The output of eval_mask and Groundtruth is abnormal

Hi, thank you for your amazing work.
I tested it on the boat and multi_scene datasets you posted, and the results were good. However, some strange errors occurred when I tested other data sets. Groundtruth and eval_mask showed abnormal display. I checked many parameters and did not find the cause. The details are shown in the picture.

When I double-checked, I used the classmate's profile and code, and the results were still the same. But in his computer could get perfect results. The image size is 1241x376 and the image format is PNG. Is the question related to this? I feel very confused, Could you kindly find out what caused the problem?

Hi, is it possible that you share the scene folder with me? That would make finding the problem a lot easier :)

@darglein  Thank you for your help. I will send it to you by email.

Your dataset works fine on my system. This is the training output after 5 minutes:

If used the following parameter file:

[TrainParams]
random_seed =<PHONE_NUMBER>
do_train = true
do_eval = true
batch_size = 1
inner_batch_size = 4
inner_sample_size = 3
num_epochs = 10
save_checkpoints_its = 10
eval_only_on_checkpoint = true
name = kitti_0_500_2
debug = false
output_file_type = .png
split_method =
max_images = 4
duplicate_train_factor = 32
shuffle_initial_indices = false
shuffle_train_indices = true
split_remove_neighbors = 0
split_index_file_train =
split_index_file_test =
train_on_eval = true
train_factor = 0.0000000000
num_workers_train = 4
num_workers_eval = 2
train_crop_size = 256
train_mask_border = 16
reduced_check_point = true
write_images_at_checkpoint = true
keep_all_scenes_in_memory = false
use_image_masks = false
write_test_images = true
texture_random_init = true
texture_color_init = false
train_use_crop = true
experiment_dir = experiments/
scene_base_dir = scenes/
scene_names = kitt_data
checkpoint_directory =
loss_vgg = 1.0000000000
loss_l1 = 1.0000000000
loss_mse = 0.0000000000
min_zoom = 0.7500000000
max_zoom = 1.5000000000
crop_prefere_border = true
optimize_eval_camera = false
interpolate_eval_settings = false
noise_pose_r = 0.0000000000
noise_pose_t = 0.0000000000
noise_intr_k = 0.0000000000
noise_intr_d = 0.0000000000
noise_point = 0.0000000000
lr_decay_factor = 0.7500000000
lr_decay_patience = 10
lock_camera_params_epochs = 50
lock_structure_params_epochs = 50

[RenderParams]
render_outliers = false
check_normal = true
ghost_gradients = true
drop_out_points_by_radius = true
outlier_count = 1000000
drop_out_radius_threshold = 0.6000000238
dropout = 0.2500000000
depth_accept = 0.0099999998
test_backward_mode = 0
distortion_gradient_factor = 0.0099999998
K_gradient_factor = 0.5000000000

[PipelineParams]
train = true
verbose_eval = false
log_render = false
log_texture = true
skip_neural_render_network = false
enable_environment_map = true
env_map_w = 1024
env_map_h = 512
env_map_channels = 4
num_texture_channels = 4
cat_env_to_color = false
cat_masks_to_color = false

[OptimizerParams]
texture_optimizer = adam
fix_render_network = false
fix_texture = false
fix_environment_map = false
fix_points = true
fix_poses = false
fix_intrinsics = true
fix_vignette = false
fix_response = false
fix_wb = true
fix_exposure = false
fix_motion_blur = true
fix_rolling_shutter = true
lr_render_network = 0.0004000000
lr_texture = 0.0100000000
lr_background_color = 0.0002000000
lr_environment_map = 0.0020000000
lr_points = 0.0001000000
lr_poses = 0.0050000000
lr_intrinsics = 0.0100000000
response_smoothness = 1.0000000000
lr_vignette = 0.0000050000
lr_response = 0.0010000000
lr_wb = 0.0010000000
lr_exposure = 0.0005000000
lr_motion_blur = 0.0050000000
lr_rolling_shutter = 0.0000020000

[NeuralCameraParams]
enable_vignette = true
enable_exposure = true
enable_response = true
enable_white_balance = false
enable_motion_blur = false
enable_rolling_shutter = false
response_params = 25
response_gamma = 0.4545454681
response_leak_factor = 0.0099999998

[MultiScaleUnet2dParams]
num_input_layers = 4
num_input_channels = 4
num_output_channels = 3
feature_factor = 4
num_layers = 4
add_input_to_filters = false
channels_last = false
half_float = false
upsample_mode = bilinear
norm_layer_down = id
norm_layer_up = id
last_act =
conv_block = gated
conv_block_up = gated
pooling = average

@darglein  Thank you for your help. I also got the same result on my classmate's computer. I would like to ask what else might cause this problem in the case of the same code and data. For example, compilation problems or package saiga_OpenGL No?  I haven't started training yet. The output of GroudTruth image when eval epoch=0 is already an exception.

Can you try a full rebuild with a new conda environment? Maybe you have missed one of the changes in the last weeks.

@darglein thanks, i will try.

Sorry for troubling you. When you tranning this data, what have you done to the point cloud ? Are you rebuild the point cloud from all the lidar point from the different sequences and use the camera to color the point ?


Template Model is missing request.clientip

Description
"{{request.clientIp}}" handlebar template not resolved in text body.
Bug reports
We're a lot more likely to look at and fix bugs that are clearly described and reproduceable. Please including the following details when reporting a bug:

[X] Which version of WireMock you're using

      <dependency>
        <groupId>com.github.tomakehurst</groupId>
        <artifactId>wiremock</artifactId>
        <version>3.0.0-beta-3</version>
        <scope>test</scope>
      </dependency>

[X] How you're starting and configuring WireMock, including configuration or CLI the command line

class IpTest {

    @RegisterExtension
    static WireMockExtension SERVER = WireMockExtension.newInstance()
            .options(options().dynamicPort().extensions(new ResponseTemplateTransformer(false)))
            .build();

    @BeforeEach
    void setUp() {
        SERVER.stubFor(get("/").withHeader("accept", matching("^text/plain.*"))
                .willReturn(aResponse()
                        .withStatus(200)
                        .withHeader("content-type", "text/plain")
                        .withTransformers(ResponseTemplateTransformer.NAME)
                        .withBody("{{request.clientIp}}")));
    }
}

[X] A failing test case that demonstrates the problem
Just do a http get to baseuri.

[ ] Profiler data if the issue is performance related

Expected output
Body with resolved content ${request.clientIp}
Actual output
Empty
Analysis
The model for handlebars does not seem to have such a field.

Huh, maybe I dreamed that this did ever work and should be a feature?

Thanks @tomakehurst for merging! 👍🏻


[Bug]: Comparison fails in recordStreak

What happened?
I was testing streak continuation this morning and have found that the check at HasStreaks:51 for strict equality $diffInDays === 1.
If you see the screenshot attached you'll see that even though $diffInDays shows as 1, this evaluates to false. This is because $diffInDays is a float.

How to reproduce the bug
Try with Laravel 11 / Carbon 3.
Package Version
1.2
PHP Version
8.2
Laravel Version
11.0
Which operating systems does with happen with?
Windows
Notes
After a quick search I found this article, breaking change in Carbon 3.0 is that these will be returned as floats now.
I cloned this repo and ran the test suite on a fresh install and can confirm that your tests are also failing on this.
Thank you for this library!

I fixed this locally by adding intval at HasStreaks:51, I tried to branch and submit a PR but looks like the repo isn't open.

@PerryRylance What do you mean the repo isn't open? You should be able to open a PR in this repo

@cjmellor Tried again just now, maybe there's something wrong with my client or account.
remote: Permission to cjmellor/level-up.git denied to PerryRylance.
fatal: unable to access 'https://github.com/cjmellor/level-up/': The requested URL returned error: 403

Yeah, weird!
If you send me the code and instructions I can look at adding it.

Please open HasStreaks and go to recordStreak. Right at the top where $diffInDays is calculated, you need to wrap the expression in intval:
$diffInDays = intval($this->getStreakLastActivity($activity)
			->activity_at
			->startOfDay()
			->diffInDays(now()->startOfDay()));
			```

Cheers @PerryRylance


Highlight code within Inner line doc comments

Allows for highlighting of inner line doc comments (module  comments //!).
This is in addition to pr #7.

Thank you for the PR!


Add list of Resources to External-Resources.md

Add a lsit of external resouces of code/snippets for design/effects/animations to build interactive web-apps

Add links with a brief description provided

I have added a reference to a cool CSS library for animations of many kinds and styles in #5
No actual source code is provided; just a link to its repo where everything lives

Thanks for that,
But as mentioned in CONTRIBUTING.md
WebsiteLib aims to make high quality code easy to use for beginners, Thus try avoiding libraries
You may refer to other Github repos, or Codepen/behance etc and link specific snippets
Anyways , Thanks for trying!!👍

Please accept my pull request


Custom attributes from the Build CR could be used as parameters while defining a BuildStrategy

Parameterization
Attributes from the Build CR could be used as parameters while defining a BuildStrategy.
Examples:
$(build.output.image) implies that the Build CR's spec.output.image value be used.
$(build.parameters.skip_ssl_verify) implies that the value corresponding to key skip_ssl_verify in Build CR's spec.parameters be used.
Wider context https://github.com/shipwright-io/build/blob/master/docs/proposals/buildstrategy.md#parameterization

@sbose78 - when we presumably expand on this in an EP, the EP should also allow a means for cluster administrators to control when parameters a given user could set on their Build instances

@sbose78 - when we presumably expand on this in an EP, the EP should also allow a means for cluster administrators to control when parameters a given user could set on their Build instances


Feature/logbook

Description
Backend logic for Logbook
Changes:

Added Logbook model

Tests included/Docs Updated?

[ ] Included for each change/fix?
[X ] Passing? (Merge will not be approved unless this is checked)
[ ] Docs updated?

There seem to be a lot of changes in all models, I guess just due to reformatting of the code. Can we try to avoid this ? I guess we need to agree on a setting for the indentation level or similar.

There seem to be a lot of changes in all models, I guess just due to reformatting of the code. Can we try to avoid this ? I guess we need to agree on a setting for the indentation level or similar.

We use (should use) prettier for format code. @henrikjohansson712


[BUG] Truncated normal initialization

I'm a bit confused with usage of trunc_normal here
https://github.com/rwightman/pytorch-image-models/search?p=2&q=trunc_normal_&type=code
I'm wondering why you are not using a and b parameters. Looking into TF implementation https://www.tensorflow.org/api_docs/python/tf/random/truncated_normal

The values are drawn from a normal distribution with specified mean and standard deviation, discarding and re-drawing any samples that are more than two standard deviations from the mean.

It makes sense to change init function to
trunc_normal_(m.weight, 0, stddev, a = -2 * stddev, b = 2 * stddev)

@NProkoptsev the version here should match the official version that was eventually added to PyTorch
In terms of use, yeah, the pytorch ver is different than tf in terms of how mean/std is applied wrt to truncation bounds. If I changed it, it'd be to add a flag that does what tf does, which is:

sample the normal dist (mean 0, std 1) with specified lower/upper truncation (-2, 2 is fixed in the tf impl)
then scale the result by std, shift by mean

I believe that ends up with the same result as using the pt impl and scaling the bounds but without as much extra verbosity

this is what I mean, 'like_tf' is what tf impl does
def trunc_normal_(tensor, mean=0., std=1., a=-2., b=2., like_tf=False):
    if like_tf:
        _no_grad_trunc_normal_(tensor, 0, 1.0, a, b)
        tensor.mul_(std).add_(mean)
        return tensor
    else:
        return _no_grad_trunc_normal_(tensor, mean, std, a, b)


sipsimple: 1.5.1 -> 1.6.0

SipSimpleSDK upgraded, old version no longer available at source.

Thanks!


How do you pass an Integration JSON object for eachValue matcher?

The docs (in file rust/pact_matching/src/lib.rs) for the EachKey and EachValue matchers mention the following:

//! | EachKey | V4 | { "match": "eachKey", "rules": [{"match": "regex", "regex": "\\$(\\.\\w+)+"}], "value": "$.test.one" } | Allows defining matching rules to apply to the keys in a map |
//! | EachValue | V4 | { "match": "eachValue", "rules": [{"match": "regex", "regex": "\\$(\\.\\w+)+"}], "value": "$.test.one" } | Allows defining matching rules to apply to the values in a collection. For maps, delgates to the Values matcher. |

It is unclear as to how an FFI user can pass a matching rule definition across the boundary to matching the values of keys in this object:
{
   "unknown key 1": "string value",
   "unknown key 2": "another string"
}

e.g. this gets the correct structure to come out the other side, but no matching rules get serialised
{
  "pact:matcher:type": "eachValue",
  "value": {
    "key1: "example string",
  },
  "rules": [.../* matching rules */],
}

Related issues

https://github.com/pact-foundation/pact-js/issues/952
https://github.com/pact-foundation/pact-js/issues/1077

Fixed in 0.4.7 of the FFI. Closing.


Inspections related to includes

Inspection that reports when the same bibliography is inserted multiple times
Inspection that reports when the file extension is used in \include or \bibliography commands.

Hooray for the 100th PR.


Allow running unprivileged containers

Problem
Running privileged containers on CI/CD servers is dangerous, and an actual blocker in multi-tenant environments.
Proposal
Add isPrivileged as a Boolean property on container declarations, with default of false.

Theres a lot of things to think about surrounding this request.
Privileges are currently obviously left as a hosting concern. No hosted service is going to allow you to request privileged execution and all of a sudden you can access other tenants containers... that would be an obvious security fail.
There are many ways to achieve the above such as configuring your container runtime to dis-allow privileged execution across the board, to choosing a runtime that has enforces strict security isolation such as katacontainers, etc...
Simply adding an attribute on a container declaration in and of itself wouldn't seem to solve anything. To be of any use it would seem like you'd need to introduce some higher level concept of privilege at the level of running an op (or higher.. maybe for an entire node or net), to either allow/dis-allow any container declaration in a call graph requesting privileged execution (or other things like resources etc...)
All the above said, any higher level privilege mechanisms would need the ops to declare what they needed in order to run so maybe adding an attribute could at least be a first step.

That was a long winded way of saying this sounds fair enough but having this doesn't solve the problem alone; it's just a pre-requisite to solving the problem.

Completely agree


[Feature request]: Remove/Hide "Transcript" section at video page's description box after clicked "....more"

Describe the change you'd like
now the description box has a new section called "Transcript" and i don't think it is necessary to have it. could you remove/hide that section out?

Additional context
No response

ytd-video-description-transcript-section-renderer { display: none; }

Also, if the section is removed from the description, it should also be made so that the option in the three dots (below video player, in the section where the likes exist) works again. They made that option just show along the lines of "Transcripts have moved to the description" and then added the transcripts to the description.


docs: add debug monitoring section

Signed-off-by: Louis DeLosSantos<EMAIL_ADDRESS>

if we somehow missed that enabling debug + all other options increases complexity against that particular kernel version

I can already explain the somehow: we don't cover debug mode in any test AFAIK :sweat_smile:


Intel(R) Xeon(R) Platinum 8275CL detected as skylake

On a AWS C5 machine:
$ lscpu
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                8
On-line CPU(s) list:   0-7
Thread(s) per core:    2
Core(s) per socket:    4
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 85
Model name:            Intel(R) Xeon(R) Platinum 8275CL CPU @ 3.00GHz
Stepping:              7
CPU MHz:               2999.998
BogoMIPS:              5999.99
Hypervisor vendor:     KVM
Virtualization type:   full
L1d cache:             32K
L1i cache:             32K
L2 cache:              1024K
L3 cache:              36608K
NUMA node0 CPU(s):     0-7
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ss ht syscall nx pdpe1gb rdtscp lm constant_tsc rep_good nopl xtopology nonstop_tsc aperfmperf eagerfpu pni pclmulqdq ssse3 fma cx16 pcid sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand hypervisor lahf_lm abm 3dnowprefetch fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid mpx avx512f avx512dq rdseed adx smap clflushopt clwb avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 ida arat pku ospke

but
[wtambellini@xmt_perf bin]$ ./list_cpu_features
arch            : x86
brand           : Intel(R) Xeon(R) Platinum 8275CL CPU @ 3.00GHz
family          :   6 (0x06)
model           :  85 (0x55)
stepping        :   7 (0x07)
uarch           : INTEL_SKL
flags           : aes,avx,avx2,avx512bw,avx512cd,avx512dq,avx512f,avx512vl,bmi1,bmi2,cx16,erms,f16c,fma3,movbe,popcnt,rdrnd,sse4_1,sse4_2,ssse3

using commit 9613390.
Trying with master ....

Also detected as skylake with 0.7.0:
[wtambellini@xmt bin]$ ./list_cpu_features
arch            : x86
brand           : Intel(R) Xeon(R) Platinum 8275CL CPU @ 3.00GHz
family          :   6 (0x06)
model           :  85 (0x55)
stepping        :   7 (0x07)
uarch           : INTEL_SKL
flags           : adx,aes,avx,avx2,avx512_second_fma,avx512bw,avx512cd,avx512dq,avx512f,avx512vl,bmi1,bmi2,clflushopt,clfsh,clwb,cx16,cx8,erms,f16c,fma3,fpu,mmx,movbe,pclmulqdq,popcnt,rdrnd,rdseed,ss,sse,sse2,sse3,sse4_1,sse4_2,ssse3,tsc
cache_info      : {"level":1,"cache_type":"data","cache_size":32768,"ways":8,"line_size":64,"tlb_entries":64,"partitioning":1},{"level":1,"cache_type":"instruction","cache_size":32768,"ways":8,"line_size":64,"tlb_entries":64,"partitioning":1},{"level":2,"cache_type":"unified","cache_size":1048576,"ways":16,"line_size":64,"tlb_entries":1024,"partitioning":1},{"level":3,"cache_type":"unified","cache_size":37486592,"ways":11,"line_size":64,"tlb_entries":53248,"partitioning":1}

8275CL is supposed to be a cascadelake CPU:
https://www.cpu-world.com/Compare/354/Intel_Xeon_8275CL_vs_Intel_Xeon_W_W-11555MRE.html

todays master also id as skylake:
[wtambellini@xmt_dcos_perf build]$ ./list_cpu_features
arch            : x86
brand           : Intel(R) Xeon(R) Platinum 8275CL CPU @ 3.00GHz
family          :   6 (0x06)
model           :  85 (0x55)
stepping        :   7 (0x07)
uarch           : INTEL_SKL
flags           : adx,aes,avx,avx2,avx512_second_fma,avx512bw,avx512cd,avx512dq,avx512f,avx512vl,bmi1,bmi2,clflushopt,clfsh,clwb,cx16,cx8,erms,f16c,fma3,fpu,mmx,movbe,pclmulqdq,popcnt,rdrnd,rdseed,ss,sse,sse2,sse3,sse4_1,sse4_2,ssse3,tsc
cache_info      : {"level":1,"cache_type":"data","cache_size":32768,"ways":8,"line_size":64,"tlb_entries":64,"partitioning":1},{"level":1,"cache_type":"instruction","cache_size":32768,"ways":8,"line_size":64,"tlb_entries":64,"partitioning":1},{"level":2,"cache_type":"unified","cache_size":1048576,"ways":16,"line_size":64,"tlb_entries":1024,"partitioning":1},{"level":3,"cache_type":"unified","cache_size":37486592,"ways":11,"line_size":64,"tlb_entries":53248,"partitioning":1}

thanks for the issue. According to https://en.wikichip.org/wiki/intel/microarchitectures/cascade_lake
For cascade lake: Family 6 Model 85 Stepping 5..7
There are early CPUs which report stepping 4. Normally stepping 4 is Skylake X.

I think we can add a check on stepping

Tks @toor1245
Would you like me to take care of it/prepare a PR ?

Yes, no problem

Hi @toor1245
so I suppose we have to add such logic/case:
if stepping >= 5 and stepping <= 7: cascadelake
probably in:
https://github.com/google/cpu_features/blob/main/src/impl_x86__base_implementation.inl
?

Hi @WilliamTambellini, I think yes, but I'm worried about stepping 4 that was mentioned in wikichip There are early CPUs which report stepping 4. Normally stepping 4 is Skylake X. These are probably the first Cascade lake CPUs and perhaps they are not in public use.
@gchatelet, can we define stepping 4 as Skylake or should we parse a brand_string (see Identification section https://en.wikichip.org/wiki/intel/microarchitectures/cascade_lake)?

@toor1245 how confident are we that we can recover the information by parsing the string? AFAIU the string is not precisely defined by Intel. If it's hard to be sure I would just assume 4 is Skylake.

@gchatelet, we have similar identification by brand_string to HasSecondFMA
static bool HasSecondFMA(const X86Info* info) {
// Skylake server
  if (info->model == 0x55) {
    // detect Xeon
    if (info->brand_string[9] == 'X') {
      // detect Silver or Bronze
      if (info->brand_string[17] == 'S' || info->brand_string[17] == 'B')
        return false;
      // detect Gold 5_20 and below, except for Gold 53__
      if (info->brand_string[17] == 'G' && info->brand_string[22] == '5')
        return (
            (info->brand_string[23] == '3') ||
            (info->brand_string[24] == '2' && info->brand_string[25] == '2'));
      // detect Xeon W 210x
      if (info->brand_string[17] == 'W' && info->brand_string[21] == '0')
        return false;
      // detect Xeon D 2xxx
      if (info->brand_string[17] == 'D' && info->brand_string[19] == '2' &&
          info->brand_string[20] == '1')
        return false;
    }
    return true;
  }
...
}

but I think its hard to predict all possible cases I would prefer skylake if no objection

seen:

so all 82xx to be CascadeLake ?

@WilliamTambellini, yes, you can find the list of Xeon cascade-lake with the second generation:
https://ark.intel.com/content/www/us/en/ark/products/series/192283/2nd-generation-intel-xeon-scalable-processors.html
and all possible cascade-lake processors:
https://ark.intel.com/content/www/us/en/ark/products/codename/124664/products-formerly-cascade-lake.html
we can classify cascade-lake processors by brand:

Intel® Xeon® W-x2xxx...
Intel® Xeon® Bronze x2xxx...
Intel® Xeon® Silver x2xxx...
Intel® Xeon® Gold x2xxx...
Intel® Xeon® Platinum x2xxx...
Intel® Core™ i9-109xxx...

cpuid dumps:
16-Core Intel Xeon Gold 5218 (Cascade Lake-SP, Single 512b-FPU)
CPUID 00000001: 00050656-00200800-7FFEFBFF-BFEBFBFF -> stepping 6
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 6C6F4720-32352064-43203831-40205550 [ Gold 5218 CPU @]
CPUID 80000004: 332E3220-7A484730-00000000-00000000 [ 2.30GHz]

10-Core Intel Xeon Gold 5215 (Cascade Lake-SP, 14++ nm server, AVX512_VNNI)
CPUID 00000001: 00050657-00200800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 6C6F4720-32352064-43203531-40205550 [ Gold 5215 CPU @]
CPUID 80000004: 352E3220-7A484730-00000000-00000000 [ 2.50GHz]

12-Core Intel Xeon Silver 4214 (Cascade Lake-SP, 14++ nm server, AVX512_VNNI)
CPUID 00000001: 00050657-00200800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 6C695320-20726576-34313234-55504320 [ Silver 4214 CPU]
CPUID 80000004: 32204020-4730322E-00007A48-00000000 [ @ 2.20GHz]

24-Core Intel Xeon Gold 6212U (Cascade Lake-SP, 14++ nm server, AVX512_VNNI)
CPUID 00000001: 00050657-00400800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 6C6F4720-32362064-20553231-20555043 [ Gold 6212U CPU ]
CPUID 80000004: 2E322040-48473034-0000007A-00000000 [@ 2.40GHz]

28-Core Intel Xeon W-3275, 4200 MHz (42 x 100) (Cascade Lake-W, 14++ nm, AVX512_VNNI)
CPUID 00000001: 00050657-00400800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 332D5720-20353732-20555043-2E322040 [ W-3275 CPU @ 2.]
CPUID 80000004: 48473035-0000007A-00000000-00000000 [50GHz]

16-Core Intel Xeon 6226R (Cascade Lake-SP, AVX512_VNNI)
CPUID 00000001: 00050657-00200800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 6C6F4720-32362064-20523632-20555043 [ Gold 6226R CPU ]
CPUID 80000004: 2E322040-48473039-0000007A-00000000 [@ 2.90GHz]

28-Core Intel Xeon 8280 (Cascade Lake-SP, AVX512_VNNI)
CPUID 00000001: 00050657-00400800-7FFEFBFF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-6F655820-2952286E [Intel(R) Xeon(R)]
CPUID 80000003: 616C5020-756E6974-3238206D-43203038 [ Platinum 8280 C]
CPUID 80000004: 40205550-372E3220-7A484730-00000000 [PU @ 2.70GHz]

14-Core Intel Core i9-10940X, 4300 MHz (43 x 100) (Cascade Lake-X)
CPUID 00000001: 00050657-00400800-7FFEFBBF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-726F4320-4D542865 [Intel(R) Core(TM]
CPUID 80000003: 39692029-3930312D-45583038-55504320 [) i9-10980XE CPU]
CPUID 80000004: 33204020-4730302E-00007A48-00000000 [ @ 3.00GHz]

18-Core Intel Core i9-10980XE Extreme Edition, 3000 MHz (30 x 100) (Cascade Lake-X)
CPUID 00000001: 00050657-00200800-7FFEFBBF-BFEBFBFF -> stepping 7
...
CPUID 80000002: 65746E49-2952286C-726F4320-4D542865 [Intel(R) Core(TM]
CPUID 80000003: 39692029-3930312D-20583034-20555043 [) i9-10940X CPU ]
CPUID 80000004: 2E332040-48473033-0000007A-00000000 [@ 3.30GHz]

Affected Processors: Transient Execution Attacks & Related Security Issues by CPU

As you can see there is no info about CascadeLake with stepping 4
CPUs that didn't mention in the above link but provided by wickichip cascade-lake and intel cascade lake processors:

Xeon Gold 6246R - Intel, stepping 7, see https://www.spec.org/cpu2017/results/res2020q1/cpu2017-20200203-20784.html
Xeon Gold 6248R - Intel, stepping 7, see https://openbenchmarking.org/s/2+x+Intel+Xeon+Gold+6248R
Xeon Gold 5215M - Intel, stepping 6, see https://www.spec.org/cpu2017/results/res2019q4/cpu2017-20190930-18769.pdf
Xeon Platinum 8284 - Intel, stepping 6, see https://www.spec.org/cpu2017/results/res2019q2/cpu2017-20190317-11165.html

Hence we can set Skylake for stepping 4

Tks @toor1245
Well that issue is just about "Intel(R) Xeon(R) Platinum 8275CL CPU @ 3.00GHz" with stepping = 7.
Do you propose to set CascadeLake when 6 <= stepping <= 7 ?

@WilliamTambellini, I'm not sure about stepping 5, but this stepping was mentioned  in wikichip note that while steppings 6 & 7 are fully mitigated, earlier stepping 5 is not protected against MSBDS, MLPDS, nor MDSUM, I didn't find any other info in the documentation, so let's set CascadeLake for now when 6 <= stepping <= 7

@toor1245
ok so I suppose around that line:
https://github.com/google/cpu_features/blob/cee2648cf0cebfb5e0106a2187d4b1c33851a9ec/src/impl_x86__base_implementation.inl#L528
adding something like :
      case CPUID(0x06, 0x4E):
      case CPUID(0x06, 0x55):
      case CPUID(0x06, 0x5E):
        if (info->stepping >=6 && info->stepping <= 7)
          return INTEL_CCL; // CascadeLake
        return INTEL_SKL;

?

CPUID(0x06, 0x55): is only CascadeLake
...
case CPUID(0x06, 0x4E):
case CPUID(0x06, 0x5E):
  return INTEL_SKL;
case CPUID(0x06, 0x55):
    if (info->stepping >= 6 && info->stepping <= 7) {
        return INTEL_CCL;
    }
    return INTEL_SKL;
...


Xcode11.3.1: property cannot be marked @objc because its type cannot be represented in Objective-C

Goals
I have a code that's similar to the model example given in RealmSwift official documentation:
import RealmSwift

// Dog model
class Dog: Object {
    @objc dynamic var name = ""
    @objc dynamic var owner: Person? // Properties can be optional
}

// Person model
class Person: Object {
    @objc dynamic var name = ""
    @objc dynamic var birthdate = Date(timeIntervalSince1970: 1)
    let dogs = List<Dog>()
}

Expected Results
I expect code to compile with Xcode 11.3.1 as it does with Xcode 10.3.
Actual Results

Instead, it throws an error

property cannot be marked @objc because its type cannot be represented in Objective-C

right at the second line in Dog declaration where owner is declared.
Steps to Reproduce

This is the exact command used:
xcodebuild -workspace "${FRAMEWORK_NAME}.xcworkspace" -scheme "${FRAMEWORK_NAME}" -configuration "${CONFIGURATION}" -arch arm64 -arch armv7 only_active_arch=no CLANG_ENABLE_CODE_COVERAGE=NO GCC_INSTRUMENT_PROGRAM_FLOW_ARCS=NO BITCODE_GENERATION_MODE=bitcode BUILD_LIBRARY_FOR_DISTRIBUTION=YES defines_module=yes -sdk "iphoneos" -derivedDataPath "${OUTPUT_DIR}"

The error is thrown both in Xcode and CI environment with Xcode 11.3.1 (11C504) command-line tools. I tried changing command-line tools to 10.3 (10G8) but results are the same. Has anyone faced this issue?
Version of Realm and Tooling

Realm framework version: 4.3.2
Xcode version: 11.3.1
iOS/OSX version: macOS Catalina 10.15.2
Dependency manager + version: ? cocoapod version 1.8.4

turned out BUILD_LIBRARY_FOR_DISTRIBUTION=YES option was the cause of trouble, I have no idea why. The issue had nothing to do with the realm. Sorry for creating a non-realm issue. Closing, Thanks.

@sandeepbol Hello, i meet same issues,is there any solution if BUILD_LIBRARY_FOR_DISTRIBUTION=YES  is required. For our  framework need to be distribution to support swift evolution.

@lixiaoyu We are using xcframework now. So this issue is not relevant. We are using create_xcframework plugin in Fastlane and it works like a charm. BUILD_LIBRARY_FOR_DISTRIBUTION=YES is ensured.

oh thanks @sandeepbol,it works!


Add Bug Report Template

A bug report template is essential for documenting issues consistently and effectively. It helps developers understand the problem quickly, facilitating a faster resolution. By using this template, team members can ensure they provide all relevant details, making the debugging process more efficient and organized. This structured approach minimizes confusion and allows for better tracking of issues over time.
Bug Report
Title: [Brief summary of the bug]
Describe the Bug
A clear and concise description of the bug. What is happening that shouldn't be? What should happen instead?
To Reproduce
Steps to reproduce the behavior:
Go to '...'
Click on '....'
Scroll down to '....'
See error
Expected Behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):
OS: [e.g. Windows 10, macOS Monterey]
Browser: [e.g. Chrome, Firefox]
Version: [e.g. 118.0.5993.90]
Smartphone (please complete the following information):
Device: [e.g. iPhone 14]
OS: [e.g. iOS 17.0]
Browser: [e.g. Safari]
Version: [e.g. 17.0]
Additional Context
Add any other context about the problem here. This could include logs, error messages, or related issues.

@sid0000007 @AshuKr22 kindly assign the task under GSSOC and hacktoberfest

assign the task i can do that

@sid0000007 @AshuKr22 kindly assign the task under GSSOC and hacktoberfest

@AshuKr22
@sid0000007


Squashed letters in Nostromo interface

I'm getting the following behaviour on my 13-inch Macbook Pro (but not my 15-inch Macbook Pro at work):

It doesn't appear to happen with other interfaces.
I don't know what might have caused it, but it may have happened roughly when I upgraded to El Capitan.

I think this might be worse with some fonts. Is that Tahoma in the screen shot?

Quicksilver Preferences tell me I'm using LucidaGrande 48.0.
I don't believe I've ever changed the font intentionally.

Ah, thanks. I’m able to get characters to overlap with that font. I think I’m just going to add a preference to turn off “tightening” rather than globally disable it.
Until then, you can try another font. None of the other ones I tried overlap like that.

If you set the update type to Developer Previews and get the latest update, there should be a new preference to take care of this.

The setting works!

If you set the update type to Developer Previews and get the latest update, there should be a new preference to take care of this.

Is this a setting in the Nostromo interface prefs? Or is it a hidden defaults?

Is this a setting in the Nostromo interface prefs? Or is it a hidden defaults?

General interface prefs:

Is this a setting in the Nostromo interface prefs? Or is it a hidden defaults?

It’s in #2162 @pjrobertson, which was part of dev preview 4017.


fix: add dependency node-fetch

Continuation of #1021
Added dependency node-fetch@2 since node 14 does not support fetch.

Upgraded to node 18 so no longer needed!


(Add export/import symbols API),
the cmake minimum required version is 2.8.12.
Update cmake minimum required version accordingly.
Signed-off-by: Miguel Gaio<EMAIL_ADDRESS>

[ ]   File: CMakeLists.txt:L27-34
 Update comment
[ ]   File: skeleton-subsystem/CMakeLists.txt:L27-34
 Update comment

This pull request is now managed with CodeReviewHub. Refresh.

:+1:

:+1:

:shipit:


Idea: Extending dart_mappable Annotations for Enhanced Flexibility

I would like to propose extending the @MappableClass and @MappableField annotations in dart_mappable to include some of the missing features available in freezed and introduce new capabilities that are not currently supported.
The core idea is to add the ability to incorporate custom and predefined hooks directly within these annotations. Additionally, enabling the annotations to read and interact with predefined fields would greatly enhance flexibility.
These enhancements would bridge existing gaps and empower developers to handle more complex scenarios seamlessly within the dart_mappable library.
class ExtendedMappableField extends MappableField {
  ExtendedMappableField({
    super.key,
    MappingHook? hook,
    bool ignoreFiled = false,
  }) : super(
          hook: ChainedHook([
            _ExtendedMappableFieldHook(key, ignoreFiled),
            if (hook != null) hook,
          ]),
        );
}

class _ExtendedMappableFieldHook extends MappingHook {
  final String? key;
  final bool ignoreFiled;

  const _ExtendedMappableFieldHook(this.key, this.ignoreFiled);

  @override
  Object? beforeDecode(Object? value) {
    // some logic
  }
}

Hi @Schultek, I'd love to hear your thoughts on this. What do you think?

Is a subclassed hook like this being picked up by the generator? If not I can try to fix this.
I wouldn't want to add them itself to the package though, but its a nice option to have for users.

Unfortunately, my current implementation won't work as intended because annotations need to be const to be effective, and there must be a mechanism to register these annotations so that the generator can recognize them.
I have two ideas to address this:

Extend Annotations with Hooks: If it's possible, we could extend the existing annotations to directly implement hooks within them. This would streamline the process and make it more intuitive.

Separate Annotation and Hook Classes: Alternatively, we could keep the annotation and hook classes separate. In this approach, a registry function would associate each annotation with its corresponding hook, allowing the generator to process them together.

I would appreciate any feedback or suggestions you might have on these approaches.


Move the Account Settings and Apps links to the top of the menu bar

We are making the User Menu (the superuser dropdown) on the upper right corner only for logging out and switching to other applications. As a result, put Account Settings and Apps to the left of the User Menu along the top of the screen. For example, where Jobs and Phenotypes are in eurekaclinical-analytics-webclient.

Thanks!


If $BOSH_ALL_PROXY set, please ignore $HTTPS_PROXY

I need to set $HTTPS_PROXY for many Golang CLIs that cleverly use it to route traffic thru a SOCKS5 proxy. Can bosh move to use HTTPS_PROXY features from Golang 1.9/1.10? Or if we must keep BOSH_ALL_PROXY, can the CLI internally ignore HTTPS_PROXY if its set, please?
Currently, if both are set:
Fetching info:
  Performing request GET 'https://<IP_ADDRESS>:25555/info':
    Performing GET request:
      Retry: Get https://<IP_ADDRESS>:25555/info: http: error connecting to proxy http://socks5://localhost:9999: proxy: failed to parse port number:

I have struggled with this matter now and again, but have become at peace with the current state of things with the two variables (you know of course, excepting the bug you are encountering). When I was looking at migrating a bunch of foundation teams to jumpboxes, the CF CLI was also implementing Go 1.9 proxy support, so I started to try to crib from them. Unfortunately, BOSH's proxy and HTTPS_PROXY have slightly different semantics:

BOSH_ALL _PROXY is "a proxy for your the director"
HTTPS_PROXY "a proxy for the internet"

Some Go CLIs have hijacked HTTPS_PROXY to mean "a gateway to the API endpoint." Unfortunately, the BOSH CLI talks to both an API endpoint (the director) and the internet at large to download releases mentioned in your manifest. So if you set HTTPS_PROXY and the gateway to your director is not also a gateway to the internet, then you are out of luck.
I think we could work around your bug here while still not making life harder for those with jumpboxes that lack egress:

people who need a proxy for the internet should set HTTPS_PROXY, we should assume everything needs the proxy.
people who need a proxy for being routable to their director should set BOSH_ALL_PROXY, we can't assume their datacenter has egress, so all other CLI connections other than to the director API* should be unproxied
people who need both are too much of a headache for us to support and we will assume they meant to only set BOSH_ALL_PROXY and thoroughly ignore HTTPS_PROXY if BOSH_ALL_PROXY is set

*and any api that the director relies on, such as UAA, Credhub

people who need a proxy for the internet should set HTTPS_PROXY, we should assume everything needs the proxy.

well, in some cases, going to the Director shouldnt go thru proxy, even though all other traffic goes through it.

| well, in some cases, going to the Director shouldnt go thru proxy, even though all other traffic goes through it.
To support this usecase, it means you are missing a THIRD variable, like BOSH_NONE_PROXY or something. We can't implicitly have HTTPS_PROXY proxy everything but the BOSH endpoints.

Here's a tactical workaround in shell environments where both HTTP[S]_PROXY and BOSH_ALL_PROXY are set.
It applies only if you want all of the bosh CLI's traffic to obey the BOSH_ALL_PROXY setting, and to ignore HTTP[S]_PROXY entirely.
alias bosh='HTTP_PROXY= HTTPS_PROXY command bosh'

FYI: I ran into this issue while trying to use "bosh" and "cf" on the same machine
bosh can't use HTTPS_PROXY and cf can't use the similar bosh proxy:
ssh+socks5://jumpbox@<<IP>>:22?private-key=/tmp/bosh-jumpbox628316669/bosh_jumpbox_private.key
So I did a reverse alias... I export a variable called "cfproxy" with the needed proxy info for cf and then set:
alias cf='HTTPS_PROXY=${cfproxy} command cf'


Establish Project Story Points

Overview
Establish project story points so we can better estimate project timeline
Action Items
Product

[ ] Map user stories into user stories template
[ ] Determine priority of each story
[ ] Determine MVP functionalities

Development

[ ] Estimate effort required

Resources/Instructions
Civic Tech Index Story Points

we ended up not using story points on this project.


DNS over TLS breaks DHCP static mappings

Hi!
I am experiencing a recurring and reproducible issue with DHCP static mappings not being resolved correctly by unbound when DNS over TLS is used. It's unfortunately not that simple to reproduce as there is a long delay after reboot / fix before the behaviour exhibits itself, but it is happening to me consistently.
I am using fully updated OPNSense.
To reproduce:
-Select "Register DHCP Static Mappings" in unbound configuration.
-Have at least one DHCP static mapping set in your DHCP configuration.
-Have at least one DNS over TLS server configured and enabled in unbound config (I am using cloudflare <IP_ADDRESS>)
I am using this set-up which allows me to resolve local servers to local addresses when connected to internal network, the names also resolve externally to my external IP allowing external access to some of the same resources. The side benefit for me is that I can easily use LetsEncrypt ACME HTTP challenge to get certs for sites for internal & external audience.
After a fresh reboot of OPNSense this configuration works fine for a long period (I'm not sure exactly how long but at least 2 weeks). Some time after this period I see that the hosts stop resolving to their internal static mapping address, and start getting resolved to the external address (via DoT).
Once the problem starts:
-Restarting DHCP & DNS service has no effect.
-Rebooting OPNSense box resolves the issue, queries then resolve to internal static mapping again (for a long period but then recurs).
-Disabling then re-enabling the DoT server in unbound settings resolves the issue, queries then resolve to internal static mapping again (for a long period but then recurs).
OPNsense 22.1.2_1-amd64
FreeBSD 13.0-STABLE
OpenSSL 1.1.1m 14 Dec 2021
Intel(R) Celeron(R) CPU J3455 @ 1.50GHz (4 cores, 4 threads)

Closing and will re-open with template


[REQUEST] [b2b-search] trigger search on ENTER

I am on version 1.3.0.
Could you enhance the b2b-search so the @b2b-search event is not only triggered when pressing the search icon but also when pressing the Enter key?

Hi @OTTOxMarc, Thanks for submitting this issue! very important indeed!
This has been fixed and would be included in our next release.
Keep an eye to our comms channels for when that happens. We will update this issue as well.


logs: fix ui bug when download button not be disabled

I think we can also remount the page when search id changes, which will prevent similar issue more thoroughly.

Closed, this bug fixed in #380


Support for cross-repo code navigation

This should mostly "just work" with SCIP, but we haven't tested packages yet. We should fix this after completing:

#1
#30

High-level breakdown of the work involved here:

[x] Support manually specifying file to gem mappings. https://github.com/sourcegraph/scip-ruby/pull/149
[x] Support inferring file to gem mappings from standard Sorbet filesystem layout. https://github.com/sourcegraph/scip-ruby/pull/155
[ ] https://github.com/sourcegraph/scip-ruby/pull/158
[ ] Support inferring file to gem mappings from standard Bundler paths (for projects not using Sorbet).
[x] 🐛 https://github.com/sourcegraph/scip-ruby/issues/157
[ ] Emit external symbols in external_symbols list (not 100% sure if this is needed)
[ ] Specially add Core and Standard APIs to Sourcegraph.
[ ] Additional manual testing against Sourcegraph + refinement.

Related:

https://github.com/sourcegraph/sourcegraph/issues/44122


Fix module name in types

Type definitions were using old module name (logdna)
Changes the module name to @logdna/logger

Whoops! Thank you for the correction. We will merge this asap.

We will be doing a release of this after the holiday on Monday.


azure backup - enable VM backup with an recovery vault in different resource group

Is your feature request related to a problem? Please describe.
Enabling Backup for a VM with an recovery vault in different resource group. as it currently shows error as below
az backup protection enable-for-vm
--resource-group
--subscription
--vault-name
--vm
--policy-name DefaultPolicy
The Resource 'Microsoft.RecoveryServices/vaults/' under resource group '' was not found.
Describe the solution you'd like
An option to mention the Vault resource group specifically
Describe alternatives you've considered
It works from the portal
Additional context
Add any other context or screenshots about the feature request here.

thanks for reporting, we'll have a look

How to backup a VM in a different resource group is already documented here. Hope this helps. Please try us and let us know your response.

@ajay15283 : How to backup a VM in a different resource group is already documented here. Hope this helps. Please try us and let us know your response.

Thank you , it works with the mentioned documented procedure

Thanks for the confirmation. Closing the issue.


Dynamodb offline

Is there an option to use dynamodb offline plugin with it?

Do you mean from a Dynamo stream?
Also which plugin are you referring to, is it this one? https://www.npmjs.com/package/serverless-dynamodb-local

Yes, this one.
I want that when a function is called by the offline-sns it also will recognize my migrations with this plugin.
I try it and it did not work.
Thank you

Sorry but I'm not exactly sure the use-case.
Are you trying to listen on a Dynamo stream? e.g. http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/Streams.Lambda.html

Can you provide some sample code to demonstrate the use case?

For example:
I run two services with serverless-oflline with different ports
One of them uses serverless-offline-sns plugin with a different port.
Two of them use dynamodb-local plugin also with different ports.
When I try to call one service to another using SNS, everything works fine until I try to query something from the dynamodb that i use locally with dynamodb-local plugin.
It always said that I did not do migrations.

Hi @eladhaz05
It seems to work fine for me, here's an example: https://github.com/mj1618/serverless-offline-sns-example/tree/dev/dynamo-local

Thank you,
I checked again works fine.


chore: Confirm Flutter 3.24.0 and Dart 3.5 Support

Description
With the release of Flutter 3.24.0 and Dart 3.5 some things are breaking. Need to review the package and make any necessary fixes.
Requirements

[ ] Package works correctly on Flutter 3.24.0 and Dart 3.5
[ ] We prepare and release a new version to be released

Moving this back to "Todo" in order to create a new release with the changes.

Closing as completed #76 has now been released.


Wrong balance display of PSP22 custom token

Describe the bug
Wrong balance display of PSP22 custom token
To reproduce the behavior
Version test: v1.0.2(145)
Check with address: 5HbcGs2QXVAc6Q6eoTzLYNAJWpN17AkCFRLnWDaHCiGYXvNc
Steps:

Import PSP-22 token successfully

Contract address: 5DGHdeY4SXsZ2w8RZrgXXJwHXXe2Qg8LCnUoDTr3tnBnu2BC (Aleph Zero Testnet)

After import successfully, check the balance of the token above in the account 5HbcGs2QXVAc6Q6eoTzLYNAJWpN17AkCFRLnWDaHCiGYXvNc

Actual: Show balance = 0 PANX (correct balance = 9.852,5 PANX)

Expect: Show correct balance

The reason of this error: import wrong token type: need add token type to this import interface

Version test: v1.0.2 (150)
Validation passed!


readme: suggest cleverer npm scripts integration

This is what I did so far and I love it.

This is also the convention I have adopted as well. This way I can run lint on Travis by itself.

Concept ACK, this is also what I do

I think either way is fine, but for the readme example I prefer it the current (simpler) way.


Don't try to publish Docker image on fork

Discussion: https://github.com/lycheeverse/lychee/issues/321#issuecomment-919161530
@fauust is that how you imagined it or is there a better way?

Yep, I had prepared something very similar on a local branch:
commit 66069c1df9e6e9c70c84821ce448197e71462611
Author: Faustin Lammler<EMAIL_ADDRESS>Date:   Wed Sep 15 09:31:08 2021 +0200

    s

diff --git a/.github/workflows/docker.yml b/.github/workflows/docker.yml
index 541c542..a882169 100644
--- a/.github/workflows/docker.yml
+++ b/.github/workflows/docker.yml
@@ -8,6 +8,7 @@ on:
       - "v*" # Push events to matching v*, i.e. v1.0, v20.15.10

 name: Publish Docker Image
+if: github.repository == 'lycheeverse/lychee'
 jobs:
   build:
     runs-on: ubuntu-latest

I am not sure where the if should go in order to skip the whole job (not sure if the checkout and the start of the VM would be skipped in your case). You can test it by removing on > push > branch > master.

Great idea to move the condition higher up the chain.
The workflow syntax describes where conditions are allowed to go. I checked and it turns out there are two places:

jobs.<job_id>.if
jobs.<job_id>.steps[*].if

So I moved it up to the jobs level.
Furthermore if you use the online editor for workflow files, it will underline errors, which is very convenient.
I think it should work the way it's written now.

@mre I have just seen that there is another step that might be good to disable on forks:
https://github.com/fauust/lychee/actions/runs/1251246990

Nice one. Let me fix that, too. 😅


Unexpected TypeError: Factories must take 0 or 1 parameters.

This works:
async with aiohttp.ClientSession() as client:
    html = await fetch(client)
    print(html)

This throws an unexpected error
from typing import Final

import svcs
from aiohttp import ClientSession

REGISTRY: Final = svcs.Registry()
REGISTRY.register_factory(ClientSession, ClientSession)

..\src\my_file.py:9: in <module>
    REGISTRY.register_factory(ClientSession, ClientSession)
..\.venv\Lib\site-packages\svcs\_core.py:254: in register_factory
    rs = self._register_factory(
..\.venv\Lib\site-packages\svcs\_core.py:329: in _register_factory
    svc_type, factory, _takes_container(factory), enter, ping
..\.venv\Lib\site-packages\svcs\_core.py:433: in _takes_container
    raise TypeError(msg)
E   TypeError: Factories must take 0 or 1 parameters.

Hm yeah this is on purpose, to be unequivocal:
https://github.com/hynek/svcs/blob/c016b378d13af4c8e5251627edd5d4c2071f2ba8/src/svcs/_core.py#L460-L462
And it can be easily circumvented by wrapping it into a lambda: ClientSession()
I guess the thought process was that factories should be svcs-specific, but then it doesn't make much sense to inspect the first argument anyway, so I think we can loosen this.


Trajectory generation/following

Various kinds of trajectories could be suitable. To start with, we should implement a sinusoidal velocity profile, point-to-point (stopped-to-stopped) generator and follower.

Reference paper

We decided to start with a simple trapezoidal trajectory, initial work in #210.

I am starting to integrate the odrive into my cablecam controller. Velocity ramp support is one of the areas I need. One option would be to use the odrv0.axis0.controller.pos_setpoint command at a fixed interval and hence simulate the ramps. No problem with that. But I am trying to understand if the Trajectory approach is meant for that.
What I expect is to set the velocity ramp, e.g. 50counts/sec/sec (change the speed by 50counts/sec every second) and then I enter a goto command and the motor will accelerate slowly up to the top speed and decelerate to reach the target point.
Are we talking about the same thing here? Why is a trajectory planning needed in the python code? Why would you want a non-trapezoid speed trajectory?

@wernerdaehn Your "velocity ramp" is actually already available in the code.  Use ..controller.move_to_pos().  Please see the discourse thread here
https://discourse.odriverobotics.com/t/trapezoidal-trajectory-planner/1104

Perfect. Thanks.

This has been done for a while (for initial trajectory).  We will open a new ticket for further trajectories


Add big diagram view

Is your feature request related to a problem? Please describe.
Currently, diagram view is restricted to a little under half the screen width. This is fine for a lot of small datasets, but may get impractical for big datasets.
Describe the solution you'd like
An option to view the diagram in a full-screen modal.

Diagram can be vertically resized since 2caf6ffa680f8256fbcd24ae5a7728dedc40a27b, review in meeting whether this is sufficient


is there a Agent Middleware to handle incoming messages like in the .Net IAgentMiddleware ?

@TimoGlastra @amanji @jleach @JamesKEbert  I need Agent Middleware interface similar to IAgentMiddleware present in .Net . So I can apply custom logic on all incoming messages from middleware.

@bilal12367 Bifold uses AFJ for all communications. Better to create the issue there as we cannot address this via Bifold.


push upgrade assurance Ansible modules to the public collection

These modules are:

panos_active_in_ha
panos_readiness_checks
panos_snapshot_report
panos_state_snapshot

run sanity checks on Ansible versions:

[x] 2.9
[ ] 2.10
[ ] 2.11
[ ] 2.12
[ ] 2.13

a PR is ready

[ ] fix linting after adding new features


Incorrect flow and lack exception handling on message consuming

Use manual ack is a good and reliable choice!
But:

When fail you need to call Reject or NAck.
When ok, you must call Ack.
Call external method without serialization, can be controversial.

https://github.com/g12-4soat/tech-lanches/blob/4f74f8267f0e592ff3f0bd23ade32f7c94d7623c/src/TechLanches/TechLanches.Adapter.RabbitMq/Messaging/RabbitMqService.cs#L38-L50

Hi @luizcarlosfaria, we're delighted that you've noticed our project! 😄
Thank you for your contribution and these improvements are already added into our roadmap.
Because of project phase 3 new requirements by FIAP course, we'll be discontinuing this repository and migrating to a new one (techlanches-api) that we strongly encourage you to follow as well. But anyways, we'll be forwarding the issues you raised to that repository to fix them.
Sounds good?
Also, please let us know in case of any other improvements. We'll be glad to discuss it with you and implement suggestions as they come up.
Thanks a lot!


fix(obstacle_avoidance_planner): calculate yaw with spline interpolation

Description
The yaw of the reference path should be calculated more precisely with spline interpolation's derivatives.
This will improve narrow-driving performance in some cases.

Tests performed

planning simulator
Effects on system behavior

Better performance on narrow-road driving
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@kosuke55 Could you approve the PR.

NOTE: With this PR, the calculation time of narrow-driving in the sample map will be much higher. We have to deal with this issue before merging.


Replay Survival Mode

The idea is that you start with 50 or 100 replays and when the replay drops, they are removed from the video and so with really hard maps, you end up with only a few who are able to Fc.
This is just a thought.

#4 would need to be resolved before this can be done since there's no information in the replay file about when they miss

I’m closing this because it seems like some other people have already started doing this on youtube


custom_infowindow_class in master

Hi i'm using master branch and want to customize infobox. Is any methods to use smth like custom_infowindow_class in old versions?
Thanks!

here is my solution https://github.com/apneadiving/Google-Maps-for-Rails/wiki/Change-handler-behavior


Fix path to bundle.js

The path to the compiled Javascript file was wrong: it tries to go up a directory when it shouldn't. Amazingly the current version still works if you host directly from the dist folder I guess because there is no dir to go up to?
Anyway, this should make it work in all cases.

Hi Edwin,
Good call. You're right, the current implementation assumes NeoDash is always hosted at the top level of a domain.
Your fix is indeed better, thanks!


fix: Feature Flag only evaluates first rule in a feature

Issue #, if available:
Description of changes:

Checklist

[ ] Meet tenets criteria
[ ] Update tests
[ ] Update docs
[ ] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

The test i changed  was working because the match value of the rule was the same as the default value of the feature.
When i change expected_value_second_check to be True and run the test it fails, because it doesnt match on the second rule.
After the fix, the test works with expected_value_second_check = True or False, because it returns the correct value.

Thank you and specially @gwlester for catching it


Replace Deprecated MkDocs Extensions

Please describe the feature.
The external emoji generator dependency has been deprecated and moved into mkdocs-material.
When using squidfunk/mkdocs-material:latest and running make docs-local, I presently see these messages:

This is how it looks after the fix:


problems loading project Assembly-CSharp-Editor.csproj

<!-- To prefill this information:
1. Open Visual Studio Code
2. Bring up the command palette (press <kbd>F1)
3. press CTRL + S to  save scripts

the output windows showes:
[warn - 13:25:19]Warning while loading h:\Unity\20201f\test\JiXieBiZhuangPei\Assembly-CSharp.csproj: unable to load file state
![Uploading<PHONE_NUMBER>823.png…]()
“obj\Debug\Assembly-CSharp.csprojAssemblyReference.cache”。End of Stream encountered before parsing was completed.

@yearlog would you confirm if the project is a unity project? If so, temporarily until we fix https://github.com/dotnet/vscode-csharp/issues/5721, you could use the separate Unity extension as a workaround.

This issue is not actionable in its current state.  If you're still having problems with the latest versions of the extensions, please file a new issue with C# logs and general reproduction steps.


Install process incorrectly extracts tarball in case-retentive fashion

After manually fetching Gentoo's stage3-amd64-nomultilib-20170525.tar.bz2 and using install.exe from v0.1.0 to install it under Windows 10 (1703), I discovered that the tarball was not extracted properly:-
$ cd /usr/share/terminfo/
$ echo *
1 2 3 4 5 6 7 8 9 a b c d e f g h i j k L M n o P q r s t u v w X z

This is missing eight additional directories.  However, this particular example is interesting because it manifests as weird terminal behaviour. This is because Windows sets TERM=xterm-256color, but the corresponding terminfo file cannot be located. Normally it would be beneath the lower-case x directory but, instead, it is beneath the upper-case X directory.
I recall previously using an older version of WSL-Distribution-Switcher with a standalone Python, without issue.

Python has no way to create a case sensitive directory, the only way I can find to do this is a direct call to kernel32.dll's CreateFile with FILE_FLAG_POSIX_SEMANTICS, after changing Computer\HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Control\Session Manager\kernel\obcaseinsensitive to 0x0 and rebooting
My suggestion, revert the ntfsea and python extraction, as it seems to be unsafe

Current plan is to support all 3 ways (untar under WSL; ntfsea+python; RegisterDistribution() in wslapi.dll for current Insiders) so users can choose another method if one fails.
Regardless, I will look into creating case sensitive directories with Python. It should be possible by calling CreateFile directly with ctypes.

Hey, did someone fix this? I have the exact same problem...


bugfix: add deduplication operation for plugin load

Describe what this PR does / why we need it
Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

Is this pr fixes https://github.com/sealerio/sealer/issues/1460? If yes, please add fixes https://github.com/sealerio/sealer/issues/1460  in the pr description. When this merged, the related issue could be automatically closed. @kakaZhou719


typescript issue with vite plugin

I'm getting
node_modules/knip/dist/plugins/vite/index.d.ts:9:79 - error TS2307: Cannot find module '#p/plugins/vitest/types.ts' or its corresponding type declarations.

9     resolveEntryPaths: import("#p/types/plugins.js").ResolveEntryPaths<import("#p/plugins/vitest/types.ts").ViteConfigOrFn | import("#p/plugins/vitest/types.ts").VitestWorkspaceConfig>;
                                                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/knip/dist/plugins/vite/index.d.ts:9:133 - error TS2307: Cannot find module '#p/plugins/vitest/types.ts' or its corresponding type declarations.

9     resolveEntryPaths: import("#p/types/plugins.js").ResolveEntryPaths<import("#p/plugins/vitest/types.ts").ViteConfigOrFn | import("#p/plugins/vitest/types.ts").VitestWorkspaceConfig>;
                                                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/knip/dist/plugins/vite/index.d.ts:10:71 - error TS2307: Cannot find module '#p/plugins/vitest/types.ts' or its corresponding type declarations.

10     resolveConfig: import("#p/types/plugins.js").ResolveConfig<import("#p/plugins/vitest/types.ts").ViteConfigOrFn | import("#p/plugins/vitest/types.ts").VitestWorkspaceConfig>;
                                                                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

node_modules/knip/dist/plugins/vite/index.d.ts:10:125 - error TS2307: Cannot find module '#p/plugins/vitest/types.ts' or its corresponding type declarations.

10     resolveConfig: import("#p/types/plugins.js").ResolveConfig<import("#p/plugins/vitest/types.ts").ViteConfigOrFn | import("#p/plugins/vitest/types.ts").VitestWorkspaceConfig>;
                                                                                ~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Found 4 errors in the same file, starting at: node_modules/knip/dist/plugins/vite/index.d.ts:9

here: https://github.com/ljharb/unused-files/actions/runs/8370450998/job/22917831503#step:6:33
It started when I updated from v5.0.4 to v5.1.5. Unfortunately exclude in tsconfig isn't helping me ignore these.

The reason it broke for you is I've introduced a cleaner way to write plugins, and these subpath imports are part of that.
Using Knip in a programmatic way is not supported currently, it might break again.
Just curious, any chance you could share what made you wrap Knip?

the commit linked above your comment is to that project, https://npmjs.com/@ljharb/unused-files
knip didn’t support the concise API i needed, so i wrapped it. It’s clear it’s not supported programmatically since importing any part of it immediately invokes the command-line handling, so i had to hack around that.
It’s fine if there’s nothing you can do about it, and maybe it’s just a typescript bug with subpath imports - it’s just unfortunate because TS won’t even let me explicitly ignore the file.

I don't care about typechecking knip, is the point :-) unfortunately even when i set the max node_modules depth to 0, TS still complains about it.

Subpath imports are also a Node.js runtime feature. Going to close this as there currently isn't much Knip can do here, I believe.

@webpro is there any chance you'd be interested in exposing any parts of knip as importable libraries?

Maybe it's just bad timing, but I'm currently not interested in widening the API surface area. In general I'd love to be able to spend more time on Knip, but there are limits (I'm looking for sponsors). Mentioned similar statements before:

https://github.com/webpro/knip/issues/41#issuecomment-1829378882
https://github.com/webpro/knip/issues/540#issuecomment-2008254880
https://github.com/webpro/knip/issues/551#issuecomment-2002424076

fwiw, this issue was fixed in v5.9.2. I'm not sure what fixed it, but at least i can unpin now :-)


Documentation, website and related tools section

Ref: https://github.com/bcoe/yargs/issues/317
Suggest:

http://readthedocs.org for documentation (where a related tool section could be).
1-pager website, with a "Who's using yargs" section
Readme with development howto, list of contributors and other notes.

cc @bcoe @nexdrew

For the website, a GitHub project page would probably be the best choice here, it can then be made in either Markdown or HTML/CSS (which I would prefer), but what about the domain? GitHub would provide something like bcoe.github.io/yargs/ if I'm not mistaken, we could also buy a domain like yargs.io (currently not used).
Also, what about the logo? Is anyone here having experience in designing logos? If not, I could at least try myself as I have a good vector graphics programme which I am familiar with, I can create an .svg based on colours and style you would like to have.

I wonder if @substack has any thoughts about what might make a good logo -- his artistic style pervades optimist and yargs.

About the site, we currently have a gh-pages branch that is available at http://yargs.js.org/ (thanks to the good folks at JS.ORG). Right now the site only consists of a client-rendered version of the README (via Flatdoc), but we could add stuff to it if you want.


[sw/silicon_creator] Add functional test for sigverify

This test will run only on FPGAs since it takes a long time to complete on sim_verilator.
Signed-off-by: Alphan Ulusoy<EMAIL_ADDRESS>

CI seems to be having a bad day. I'll try re-re-running the checks later.


feat (experimental): userop cli command in plugingen

Note: This is an “experimental” PR for the sake of unblocking other eng teams for faster and easier testing/debugging iterations. Not planned per Wallet Service Accounts (client) team current or future sprints. Again, please note that this is an “experimental” branch. cc: @moldy530
https://alchemyinsights.slack.com/archives/C04RHSDSP9U/p1713169071062569
Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
The focus of this PR is on adding user operation configuration for smart accounts and improving CLI commands for building, signing, and sending user operations.
Detailed summary

Added userop.json file for user operation configuration
Updated CLI commands for building, signing, and sending user operations
Updated file paths and configurations for consistency and accuracy

The following files were skipped due to too many changes: packages/plugingen/README.md, packages/plugingen/src/commands/userop/client.ts, packages/plugingen/src/commands/userop/schema.ts, packages/plugingen/src/commands/userop/index.ts

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

Note: This is an “experimental” PR for the sake of unblocking other eng teams for faster and easier testing/debugging iterations. Not planned per Wallet Service Accounts (client) team current or future sprints. Again, please note that this is an “experimental” branch. cc: @moldy530


I would like to use heatmap in vue electron. How to do it

I use import heatmap from 'heatmap.js' and use it in the template
https://github.com/pa7/heatmap.js
Var heatmap = heatmap.h337.create,
This is not the way it is.
Because it will be given
Uncaught TypeError: Can not read property 'create' of undefined

@zsxgb
This is not a direct issue with electron-vue, so I will be closing. The documentation is a little misleading, but what you require/import is h337.
Quick Example Use
<template>

</template>


fixed the indexpage

I redone the testimonials.
I redone the projects section. Fixed overlapping  buttons.
Create parallax section and button to see progress for projects.

Looks awesome. Ready to merge, but can you resolve the above conflict first?

Ok I think is ok

👍


Continuously send requests when use getAllRecordsWithId

Target Package

@kintone/rest-api-client 1.10.0

Environment

Operating System: Windows 10
Browser: Chrome 88.0.4324.182
Node.js version: v14.15.3

Current Behavior

Step 1. Create a new kintone app, no field is OK.
Step 2. Add more than 500 records.
Step 3. On JS Customization page of this APP, add the UMD file of rest-api-client.
Step 4. On list page of this APP, open DevTools of browser and run following code.
const client = new KintoneRestAPIClient(); client.record   .getAllRecordsWithId({ app: kintone.app.getId(), condition: '$id != "" or $id = ""' })   .then((resp) => {     console.log(resp);   })   .catch((err) => {     console.log(err);   });
=> Bug occurs:
The API requests are send continuously.
Always return the first 500 records.
Memo:
If condition is '($id != "" or $id = "")', this bug does not occur.
I think adding () in rest-api-client is better.
https://github.com/kintone/js-sdk/blob/f54a4ccbaec43f90dceaf6533431f4951d37fa92/packages/rest-api-client/src/client/RecordClient.ts#L274
const conditionQuery = condition ? `${condition} and ` : "";
->
const conditionQuery = condition ? `(${condition}) and ` : "";
Expected Behavior

Do not send requests continuously.
Return the correct result.

@dong0
Thank you for reporting!
We could reproduce the problem. We'll fix it later.

@dong0
This issue was fixed in<EMAIL_ADDRESS>Thank you!

@zaki-yama
Thank you for your information.
That's great!


Compatibility Issue: Temporary Table Creation in save_as_table Conflicts with Snowflake Native Apps

I've encountered a compatibility problem when using snowflake-ml-python within a Snowflake Native App environment. The issue stems from the internal use of snowpark.DataFrame.save_as_table within the library (e.g., in OrdinalEncoder).
This method appears to create a temporary table under the hood:
https://github.com/snowflakedb/snowflake-ml-python/blob/2932445f3e0fbf42000d51ec9c9256032570ed76/snowflake/ml/modeling/preprocessing/ordinal_encoder.py#L250-L258
However, Snowflake Native Apps have a restriction that prevents the creation of temporary tables. This incompatibility renders certain parts of snowflake-ml-python unusable within Native Apps.
Suggested Resolution:
I'd like to request that the team consider a modification to address this limitation.

Thanks for reporting. We are aware of this issue. Native Apps team is considering some workaround for this.

@sfc-gh-sdas . Do you know if there is a workaround for this yet?


Redesign selector system (closes #798)

Feature summary
New selector methods
All methods are exposed both on selector and on selector promise, so you can apply it on selector with arguments, e.g.:
const withIdOrClass = Selector(val => document.getElementById(val) || document.getElemenstByClassName(val));

withIdOrClass('some').withText('Hey!');

.nth(index) - creates selector for nth element in the matching set. Replacement for index option.

.withText(text), .withText(re) - creates selector that filters matching set by given text. Replacement for text option.

.find(cssSelector), .find(filterFn) - finds all descendants of all nodes in the matching set and filters them either by CSS selector or using filterFn. filterFn is a client function predicate that receives node.

.parent(), .parent(idx), .parent(cssSelector), .parent(filterFn) - finds all parents of all nodes in the matching set.  If filter specified, filters parents either by CSS selector or using filterFn. filterFn is a client function predicate that receives node. .parent(idx) is a shorthand for .parent().nth(idx).

.child(), .child(idx), .child(cssSelector), .child(filterFn) - finds all child elements (not nodes!) of all nodes in the matching set. If filter specified, filters children either by CSS selector or using filterFn. filterFn is a client function predicate that receives node. .child(idx) is a shorthand for .child().nth(idx).

.sibling(), .sibling(idx), .sibling(cssSelector), .sibling(filterFn) - finds all sibling elements (not nodes!) of all nodes in the matching set. If filter specified, filters siblings either by CSS selector or using filterFn. filterFn is a client function predicate that receives node. .sibling(idx) is a shorthand for .sibling().nth(idx).

Methods works like filters and can be combined together, making flexible functional-style selector system.
Examples:
Selector('ul').find('label').parent('div.someClass')

Finds all ul elements on page. Then in each found ul element finds label elements. Then for each label element finds parent that matches  div.someClass selector.

Like in jQuery, if you request a property of the matching set or try evaluate a snapshot it returns values for the first element in the set:
// Returns id of the first element in the set
var id = await Selector('ul').find('label').parent('div.someClass').id;

// Returns snapshot for the first element in the set
var snapshot = await Selector('ul').find('label').parent('div.someClass')();

However, you can obtain data for any element in the set by using nth filter:
// Returns id of the third element in the set
var id = await Selector('ul').find('label').parent('div.someClass').nth(2).id;

// Returns snapshot for the fourth element in the set
var snapshot = await Selector('ul').find('label').parent('div.someClass').nth(4)();

Note that you can add text and index filters in the selector chain:
Selector('#container').parent(1).find('.someClass').withText('yo!').child('span');

Finds second parent of #container element. Then in parent finds elements that matches '.someClass' selector. Filter them by text yo!. And in each filtered element searches for a child with tag name span.

New selector properties
Like for methods, properties are exposed on both selector and selector promise.

.count - returns the number of matching elements.
.exists - returns true if at least one matching element exists.

Examples:
expect(await Selector('div').withText('42').count).eql(2);
expect(await Selector('.someClass').child(2).exists).to.be.true;

Unawaited selector calls now allowed out of test context (breaking)
Previously selector call outside of text context thrown an error:
const selector = Selector(id => document.getElementById(id));

selector('smthg'); // <-- throws

test ('Some test', async t=> {
...
});

Now it's not a case if selector is not awaited. It's useful when you need to build a page model outside the text context:
const selector = Selector(id => document.getElementById(id));
const selector2 = selector('smthg').find('smthelse'); // <-- doesn't throw anymore

test ('Some test', async t=> {
...
});

However, once you'll try to await snapshot or snapshot property values it will throw:
const selector = Selector(id => document.getElementById(id));

asynct someFunc() {
  return await selector('yo'); // Throws
}

someFunc();

test ('Some test', async t=> {
...
});

Index now is not ignored if selector returns single node (breaking)
Previously if selector returned single node index was ignored:
Selector(() => document.getElementById('hey'), { index: 2 } ); // returns element by id

however it's not a case now:
Selector(() => document.getElementById('hey') ).nth(2); // returns null

This leads to more consistent system then it used in chain:
const withId = Selector(() => document.getElementById('hey') ;

function getSecondElementWithText(selector) {
  return selector.nth(2).withText('hey');
}

getSecondElementWithText(withId);

Deprecated API (breaking)
All deprecated API will last at least for the next release. However, currently they will give deprecation warning when used.

t.select - Doesn't fit into new selector system, since it doesn't provide combination of selectors. Use selectors and property shorthands to obtain values:

var id = await t.select('.yo').id;

can be replaced with

var id = await Selector('.yo').id;

selectorOptions.index - Use .nth() instead.
selectorOptions.text - Use .withText instead.
selectorOptions.dependencies - The mechanism was vague from the start. It's not required anymore as a public API, since now we have a flexible selector mechanism, so there is no need to pass selectors around.

\cc @VasilyStrelyaev @AlexanderMoskovkin @helen-dikareva

Don't merge it, please. I've figured out that we need some changes to this mechanism

Ready for review. I've added feature summary in the OP post of PR.

lgtm. Nice changes!

FPR, there are no changes in the code since the last successful test run, so it's just an unstable test.


Form example with cancel link incorrect buttonlink

The Form example 'with cancel link' has this for markup:
<Link Variant="ButtonVariant.Outline" href="/docs/components/forms">Cancel</Button>
but should be
<ButtonLink Variant="ButtonVariant.Outline" href="/docs/components/forms">Cancel</ButtonLink>

Nice catch. I had to rename Link to ButtonLink due to bugs it was causing.

#4 is the fix for this one.


tried to create a short cut on desktop -got this bug report

Describe the bug
[A clear and concise description of the bug]
Steps to Reproduce
1.
2.
3.
Expected behavior
[A clear and concise description of your expectation]
Additional context and screenshots
[Add any other information such as screenshots, error message, or stack trace]
Sample Code
XXX

Configuration information (If you are providing a diagnostics file (see below), skip this section)
VS Version: [Can be found under "Help->About Microsoft Visual Studio"]
PTVS version: [Can be found under "Help->About Microsoft Visual Studio->Installed products->Python with Pylance"]
Python version:
Debugger Type (if applicable): [New debugger or legacy debugger?]
Diagnostics file
The diagnostics file contains information such as the Python version, environments, loaded assemblies, configuration options, etc. This file makes it easier for us to reproduce and fix bugs, so we would really appreciate it if you could provide it to us. To locate the logs, you can can follow these steps:

Open the File Explorer on your computer.
In the address bar at the top of the window, type %TEMP% and press Enter.
The %TEMP% folder should open, look for file named PythonToolsDiagnostics_*.log, where * represents a series of numbers.

This file contain the diagnostic information that can help us diagnose and resolve the issue you're experiencing.
Please note that the %TEMP% folder is a hidden system folder, so you may need to enable the display of hidden files and folders in File Explorer to see it. To do this, click on the "View" tab in File Explorer, then check the "Hidden items" box in the "Show/hide" section.
You can attach the file to this issue or email it to us at<EMAIL_ADDRESS>If you email it, please add a link to this issue.
XXX
```[Window Title]
devenv.exe

[Main Instruction]
An unexpected error occurred

[Content]
Please press Ctrl+C to copy the contents of this dialog and report this error to our issue tracker.

[^] Hide details  [Close]

[Expanded Information]

Build: 16.11.21196.2
System.Drawing.Printing.InvalidPrinterException: No printers are installed.
at System.Drawing.Printing.PrinterSettings.GetHdevmodeInternal(String printer)
at System.Drawing.Printing.PrinterSettings.GetHdevmodeInternal()
at System.Drawing.Printing.PrinterSettings.GetHdevmode(PageSettings pageSettings)
at System.Windows.Forms.PageSetupDialog.RunDialog(IntPtr hwndOwner)
at System.Windows.Forms.CommonDialog.ShowDialog(IWin32Window owner)
at System.Windows.Forms.CommonDialog.ShowDialog()
at Microsoft.VisualStudio.Editor.Implementation.PrintingClient.PageSetup()
at Microsoft.VisualStudio.Editor.Implementation.SimpleTextViewWindow.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.Undo.UndoManagerCommandFilter.Microsoft.VisualStudio.OLE.Interop.IOleCommandTarget.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.IntellisenseCommandFilter.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.BraceCompletionCommandFilter.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandHandlerServiceAdapter.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandHandlerServiceFilter.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandFilterWrapper.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.InnerExec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandChainNode.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.CommandFilterWrapper.Exec(Guid& pguidCmdGroup, UInt32 nCmdID, UInt32 nCmdexecopt, IntPtr pvaIn, IntPtr pvaOut)
at Microsoft.VisualStudio.Editor.Implementation.C

Hi there, It appears that the issue you're experiencing is related to the printer setup rather than PTVS (Python Tools for Visual Studio). The error message System.Drawing.Printing.InvalidPrinterException: No printers are installed indicates that there are no printers configured on your system.
If you're still encountering issues related to PTVS or have any other problems or questions regarding PTVS functionality, please file a new issue. Thanks!


gcc pedantically complains about that comma... :)

zstd_static.h:74:38: warning: comma at end of enumerator list [-Wpedantic]

Thanks for notification.
Unfortunately, this warning is a bit difficult to fix, since it would require to change the macro generating enums.
Probably adding a useless element at the end of the list would work. But I don't think it's cleaner than the trailing comma.

Got it. Thanks anyway. I will close this little issue.


CI Build: One time failure in MountFailsWhenNoOnDiskVersion

One time failure:
https://gvfs.visualstudio.com/ci/_build/results?buildId=9230
It appears that the test failed because the GVFS.Mount process did not respond quickly enough and gvfs mount timed out.
What was suspicious was the time between the first two lines of the mount process log (see below).
gvfs mount will wait as long as a minute for GVFS.Mount.exe, and so it's possible that something else on the machine caused GVFS.Mount.exe to take a long time to start.
At a minimum it's worth investigating what else can be logged to understand where the time went.
----- C:\Repos\GVFSFunctionalTests\enlistment\5c85ccc3f5b748be9755\.gvfs\logs\gvfs_mount_process_20190410_180702.log -----
[2019-04-10 18:07:13 +00:00] GVFSMount (Start) {"Version":"0.2.19100.3","EnlistmentRoot":"C:\\Repos\\GVFSFunctionalTests\\enlistment\\5c85ccc3f5b748be9755","Remote":"https://gvfs.visualstudio.com/ci/_git/ForTests","ObjectsEndpoint":"https://gvfs.visualstudio.com/ci/_git/ForTests","IsElevated":true,"EnlistmentRootPathParameter":"C:\\Repos\\GVFSFunctionalTests\\enlistment\\5c85ccc3f5b748be9755","StartedByService":"False"}
[2019-04-10 18:07:31 +00:00] RetryConfig_LoadedRetryConfig {"Area":"RetryConfig","Timeout":"00:00:30","MaxRetries":6,"Message":"RetryConfigLoaded"}
[2019-04-10 18:07:39 +00:00] GitStatusCacheConfig_Loaded {"Area":"GitStatusCacheConfig","BackOffTime":"00:00:02","Message":"GitStatusCacheConfigLoaded"}
[2019-04-10 18:07:39 +00:00] CachePathsLoaded {"gitObjectsRoot":"C:\\Repos\\GVFSFunctionalTests\\.gvfsCache\\b810a5d8b978423bb9fe3aa140499e0e\\gitObjects","localCacheRoot":"C:\\Repos\\GVFSFunctionalTests\\.gvfsCache","blobSizesRoot":"C:\\Repos\\GVFSFunctionalTests\\.gvfsCache\\b810a5d8b978423bb9fe3aa140499e0e\\blobSizes"}
[2019-04-10 18:07:39 +00:00] Mount_StartedNamedPipe {"NamedPipeName":"GVFS_C_\\REPOS\\GVFSFUNCTIONALTESTS\\ENLISTMENT\\5C85CCC3F5B748BE9755"}
[2019-04-10 18:07:40 +00:00] AttemptAnonymousAuth (Start)
[2019-04-10 18:07:41 +00:00] Error {"RepoUrl":"https://gvfs.visualstudio.com/ci/_git/ForTests","ErrorMessage":"Failed to parse credential string for approval"}
[2019-04-10 18:07:41 +00:00] NetworkResponse {"RequestId":1,"availableConnections":4,"CacheName":"","StatusCode":200,"ContentType":"application/json; charset=utf-8","connectionWaitTimeMS":"0.0095","responseWaitTimeMS":"987.7411"}
[2019-04-10 18:07:42 +00:00] AttemptAnonymousAuth (Stop) {"HttpStatus":"200","QuerySucceeded":true,"IsAnonymous":true,"DurationMs":2270}
[2019-04-10 18:07:42 +00:00] Information {"Message":"Git status cache enabled.  Cache state: Clean. Status scan time: 14.69s."}
[2019-04-10 18:08:10 +00:00] TryAcquireLockExternal {"LockRequest":"gvfs unmount (7272)","IsElevated":true,"Result":"Accepted"}
[2019-04-10 18:08:14 +00:00] Heartbeat {"ModifiedPathsCount":2,"PlaceholderCount":8,"GitStatusCache.StatusScanCount":1,"EnlistmentId":"4f17be1dbcc140f58432601f8b52b46d","PhysicalDiskInfo":{"DriveLetter":"C","VolumeDriveType":"Fixed","VolumeFileSystem":"NTFS","VolumeFileSystemLabel":"","VolumeSize":"136364093440","VolumeSizeRemaining":"54823940096"},"Version":"0.2.19100.3","MinutesUptime":0,"MinutesSinceLast":0}
[2019-04-10 18:08:14 +00:00] GVFSMount (Stop) {"DurationMs":72823}

Just had a second hit here:
https://gvfs.visualstudio.com/ci/_build/results?buildId=9260

Hi there,
I have this problem when trying out the VFS for GIT as a client communicating with a TFS server.
Are there any workarounds to solve this problem?
Regards,
Michael


Not working in tmux

Hi. I cannnot get this beautiful colorscheme working inside tmux. (Outside tmux it works fine)
I am using pantheon-terminal on elementary os (== ubuntu 16.04) with tmux 2.3 and vim 8.0.
tmux itself is configured to use Tc and it works (checked with the testscript mentioned here )
When I run vim however I get a more or less black and white colorscheme

I dont know if its an issue of the theme or with my .vimrc or my .tmux.conf
EDIT: Solved it by following this
Sorry for spamming in here, maybe interesting for other users though?

I might just this info to the README since this seems to be a somewhat common issue.


trademark violation

You guys are using trademarked logos of companies in your repository and put those under MIT license. You can't do that! Please clean your repository history. See examples here https://github.com/intershop/intershop-pwa/blob/develop/src/assets/mock-img/brands
Note: I'm writing this issue unrelated to any projects I'm working on.

Also I find that parts of the reference design and name look close to https://intronics.nl
Since they are using Intershop, yet not using this repository as base, from what I can see, I have a few questions as result:

Could you elaborate how the two (the intronics.nl website and this repo) fit together? Their theme reveals a lot of classes I can attribute to intronics, but not yet to Intershop. Do both use the same Open Source reference design maybe?

Are there are any copyright or trademark issues here in your opinion? Or is it safe to use this repository as a base for integration of Intershop software in that regard?

Hi Phil,
thank you for bringing up this topic. Of course we do not intend to violate any ip rights within our project. We will replace the mentioned trademarked logos immediately. They are used as as mock data only anyways.
For the relation of this repo to intronics.nl, they are not related. We use this name as part of our demo data, a fictional company named "inSPIRED" with different shops for different industries, "inTRONICS" for consumer electronics, "inSPORTS" for sports fashion and inTOOLS for a b2b shop selling tools. The "in" stands for Intershop and is part of our marketing story.
When the respective team created the demo scenario several years ago it was not aware that a company intronics existed and was even a customer of ours. We later figured out and agreed that we can use this name within our demo scenario. However as we are now an open source project, we will again get in contact and clarify how to handle the situation moving forward.

@philkunz Thank you for pointing out the trademark issues in our project.
As a result we removed the referenced content and any other questionable content we could find from the complete GitHub repository. In addition we changed our demo scenario that is supposed to give a first impression even without a connected Intershop Commerce Management server in a way that does not rely on the removed content.
The new hotfix release 0.16.1 represents now the new cleaned working state of our project.
Older releases might have problems with missing assets because of the repository cleanup.


Feature/support japanese language

Here comes Japanese
I took the opportunity to factorise the algorithm for the display hour and minutes rounding, I thought I was clever until I applied it to Dutch :)
All languages updated and tested, I would test Dutch if I were you before merging just in case!
While testing the NL version I fixed a bug (crash when midnight), there was no word for 0 (I used TWAALF)

Do tell me if I need to implement things in a different way, I am far from an advanced JS/node developper, just learning :)

Thanks! I know the code could be improved, but never took the time to do so 👍 Sorry for the late response, I had a very busy week.
To test the code I'll use this on my smart mirror for the coming week switching between some languages and I'll merge it next weekend!

No worries I also have the code ready to alternate between a list of languages every x minutes but it comes on top of this pull request I can push it if you want as well. That’s pretty cool for testing :)

Please do so!


feat: visibility to which configure pipeline corresponds to which resource request

If Platform users make 3 requests and run k get pods i see:
k get pods
NAME                                   READY   STATUS              RESTARTS   AGE
configure-pipeline-redis-59614-ls9pd   0/1     Completed           0          14s
configure-pipeline-redis-5e5d7-vg9cl   0/1     Completed           0          7s
configure-pipeline-redis-80f23-fdvt2   0/1     Completed           0          21s

It's not possible to map the pipeline to the request.
It would be nice instead to see:
k get pods
NAME                                   READY   STATUS    RESTARTS   AGE
kratix-configure-redis-cat-5678        1/1     Running   0          12s
kratix-configure-redis-jake-1234       1/1     Running   0          20s
kratix-configure-redis-sapphire-9012   1/1     Running   0          3s

Notes
We will run into name length limits (63char), so this is a best-effort improvement.  consider more shorthand than kratix-configure to increase the likelihood of the new fitting. Viewings pods with labels k get pods --show-labels is still the best way to determine exactly what a pod is related to and is a k8s convention, ensure we have the right labels set.

would also be good to know what pipeline a pod corresponds to given we are adding support for multiple in #90

To double confirm, the format will look like
kratix-<pipelinetype>-<resource request kind>-<resource request metadata.name>-<UUID truncated when we reach 63 characters>?

I think with the work from #90 it would look more like this @catmo-syntasso
<some-prefix>-<promiseName>-<resourceName>-<pipelineName>-UUID
e.g.
kc-redis-jakes-create-tf-123ab
kratix-configure-redis-jakes-create-tf-123ab
kratix-redis-jakes-create-tf-123ab

‘kratix-app-as-a-service-my-new-service-create-tf-123ab’
I tried coming up with the longest version of my preferred combo and we’re still at 54 characters so I think we’re 🆒

@richcooper95 and @kirederik to check if this has been implemented


[WIP] Possible approach to #1464

A very rough attempt at resolving #1464, but I'm assuming there's a better way this could be done: this is overriding stock behavior of react-virtualized using !important, which isn't a great idea, and is likely breaking other display behaviors along the way.
@rafaelramalho19 -- If you've got a little time between projects, I'd appreciate learning about how you'd approach this. Thanks!

I'll look into it 😄

Thanks!


Shift the '-type f' argument before the '-name' argument in find

to make it more efficient.

That’s a nice optimization; hadn’t thought of that. Thanks!


How do I get an authorization code from the library?

Core Library
MSAL.js v2 (@azure/msal-browser)
Core Library Version
2.14.2
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
2.0.0
Description
I did all the configuration as mentioned in the instruction but why after calling "loginpopup' or "loginredirect" method I am getting an access token instead of auth code. In my use case, I want to create auth code 1st and then will send this auth code to the backend server to generate other tokens(access, refresh).
One more confusing point is that even after authentication why I am getting only "access token" instead of "access token" and "refresh token" token both. I can see these values in cache though.
MSAL Configuration
No response
Relevant Code Snippets
No response
Identity Provider
No response
Source
External (Customer)

@gaut7 The msal-browser library is meant for browser-based authentication for Single-Page applications, which is why all AuthNZ is done inside the browser. For a server-side flow, you can use the msal-node library.

One more confusing point is that even after authentication why I am getting only "access token" instead of "access token" and "refresh token" token both. I can see these values in cache though.

The library takes care of token renewal, so the refresh token is not returned. This is something done in most MSAL libraries.

@pkanher617 My use case involves taking the refresh token, access token, and identity token and shipping it somewhere else so that it can be used to re-authenticate outside of the web app. Something like using device code flow doesn't work in my case, because the initial configuration must be made from the SPA. In this case, being able to retrieve the refresh token is essential so that I do not have to re-implement the authentication flow myself. It seems really interesting to me that the refresh token isn't exposed even though it's returned from the token endpoint. I don't believe "the library takes care of it, which is why we expose it" is an acceptable answer.


STT DEMO KWS -- Misleading comment in Keyword Spotting box: "Type comma separated keywords here".

Leonid,
On the STT demo page (via this link -->https://speech-to-text-demo-lr.mybluemix.net/), the user observes the following comment in the keyword spotting box, "Type comma separated keywords here".
If the user acts on this and enters his/her keywords,  he/she will never be able to test the audio samples using the sample's default keywords, unless he/she knows to leave the box empty.
Perhaps the message in the keyword spotting box should be something like this,
a) "Leave box empty for sample's default words or type comma separated keywords here". OR
b) "Type comma separated keywords here (OPTIONAL)."
See attachment for screen shots.
STT DEMO KWS DEFECT -- Misleading comment in Keyword Spotting box -- Type comma separated keywords here.xlsx
Alexandra

Leonid,
Can you also update the "Transcribe Audio" section as follows?
"You may choose to spot your keywords by entering them (separated by commas) in the text box. Use your microphone (compatible only with Google Chrome and Mozilla Firefox). Upload pre-recorded audio (WAV for uncompressed audio, FLAC or OPUS) file formats. Drag and drop recorded audio onto the page, or use the audio samples provided. The returned result includes the recognized text, word alternatives (aka confusion networks), and spotted keywords."
Thank you,
Alexandra

Leonid,
I have successfully tested your fixes.
I will close this issue now.
Thank you,
Alexandra