chore: switch to podman setup.exe

Switches the previous installation of podman (zip archive) to using the setup.exe version. Fixes #240

question: how do you define the version of podman to use ? here I see 5.2.4, before 5.0.3 so it seems for both not in sync with either the latest from podman or the one podman desktop is using
podman desktop is using the v5.2.5 https://github.com/containers/podman-desktop/blob/main/extensions/podman/packages/extension/src/podman5.json

I used the previous existing version used on other yaml files, I did not realize that it wasn't the most recent one. The older version that was being used earlier (5.0.3) probably was forgotten to be updated as time went on, unless I'm mistaken.
I've updated all instances to v5.2.5 for consistency

Regarding inputs. This does not really matters, since the parameter is only used when run on event workflow_dispatch. What we really needs it to use particular podman version based on the context of the job.
In this job we are installing the podman from podman-desktop, so no need to specify any particular podman version.
In job where we test podman-nightly installed from the nightly CI builds we might just set a new version in input parameter, but when run on nightly base on schedule, the latest podman remote version is used.
Where we really need to consider using latest podman that is available or defined by podman-desktop is this one where we want to use setup.exe podman version as defined in podman-desktop because there is no setup.exe from the podman nightly CI builds, so we need to add a code to define version like in bootc and use in when job is triggered  on schedule. Again we can use some particular latest version in input but that does not really matter (although is nice to have latest there).
On the missing two jobs, we just update input default value. that is fine since the job are only run as workflow_dispatch. thus only using user's set params.


Fix rotation in localisation view

This PR applies simple fix to rotation in NUsight localisation view.

Alternative approach #1082 1082


Limit private repos

Replacement PR for #12705 since I done f-ed up that one.
Note: this turned out a little bigger than I had anticipated... as I stated before, I'm completely new to the Gitea codebase so I probably missed a lot.
Also, I didn't take a look at the tests yet. I did do some basic manual tests and as far as I can tell, the basic functionality is working.
Anyone who's willing to help test this, help fix tests and/or review it for missed parts or improvements, please comment. (I turned on "allow edits by maintainers" for this PR by the way)
Original description
Since @kasbah does not appear to have time to continue his work on PR #6184 I started looking at continuing his work. As there have been many changes to master since, I felt it was easier to start fresh.
Fixes #3653. Feedback very much appreciated.

Apologies for the confusion and overhead by the way. :smile:

A forcepush after a rebase would have done it too ... but here we are ;)

@mvdkleijn can you commit the result of make generate-swagger ?

A forcepush after a rebase would have done it too ... but here we are ;)

Which is exactly what I thought I did but that's the part where things went wrong. Anyway, it doesn't matter. I'm happier with this PR anyway. :smile:

I have a question. The system currently keeps track of the number of repos that a user owns by storing it as a field in the DB. It causes a lot of code snippets all over the codebase. Was that done for performance reasons or otherwise? It feels like it could be a nice improvement in terms of maintenance to simply SELECT COUNT entries in the DB instead of storing the counts in the DB as a field.

Yup it's an optimisation for performance reasons.

Just checking in... my four year old gave me the gift of the flu. I'll continue on this in a few days.


SREP-1143: Prep for CI Pipeline

Adds config/config.go (for openshift/osd-operators-registry)
"Standardizes" Makefile (with helpers)
"Standardizes" Dockerfile

Remove extraneous build/bin pieces

Signed-off-by: Lisa Seelye<EMAIL_ADDRESS>

/lgtm


Switch to latest prophet

@bastienboutonnet  CI seems to be green with published prophet - can you merge this?


Get rid of internal methods calls, simplify code

This gets rid of len and str calls, and uses for loops where applicable.

Thank you so much! I will update it

@philippeitis
I updated the code as you suggested but I left out some for loops that you added to replace some of my while loops only because there were some warnings about unused variables.
For the remaining part thank you so so much, my goal is to improve everyday and that's also thanks to people like you who spend time to help other people.
Thanks again ❤


Completed ViewProfileUseCase

Completed ViewProfileUseCase Fully

Great! Thanks Tanaya

Thank you for your work - time to work test file for view profile


Update technologies.yaml

Add levelized costs (as in PR#182 which I closed as it had conflicts)

@danielhuppmann I think this PR is ok for merging. Thanks in advance


See for the list https://github.com/adobe/theblog/issues/186

Covered the major cases. They are still some really edge cases which are hard to identify / fix as part of the import. They will have to be fixed manually when found.


Save things on Flutter side

IIRC there was some api for storing "settings" type data that we could use? @futurepaul
The main one we need to save right now is client.json containing all information needed to talk to the federation.
But I think there will be more user preferences (e.g. language, do they prefer BTC or sats, etc)

here's my example of saving preferences in pln... I think this partly exists in fluttermint but it's not as fully wired up: https://github.com/BitcoinDevShop/pln/blob/master/mobile/lib/data/prefs.dart
it's a keyvalue store so you can add as many keys + values as you want just will have to write setter / getter functions (and make prefs a more complicated object rather than a single string)
it can be consumed from other riverpod providers like this: https://github.com/BitcoinDevShop/pln/blob/master/mobile/lib/grpc.dart#L21
and it's written to it here: https://github.com/BitcoinDevShop/pln/blob/master/mobile/lib/screens/welcome.dart#L21
in fluttermint we actually initialize prefs right now but don't do anything with it I guess? https://github.com/futurepaul/fluttermint/blob/master/lib/main.dart#L20


App cast generator: paths to release notes not generated properly

netsparkle-generate-appcast -e dmg -f true -o macos -u https://netsparkleupdater.github.io/NetSparkle/files/sample-app-macos -l https://netsparkleupdater.github.io/NetSparkle/files/sample-app-macos -p . -n "Sample NetSparkleUpdater macOS"
results in a changelog path of:
https://netsparkleupdater.github.io/NetSparkle/files/sample-app-macos2.0.0.md
which is clearly not right.

item.ReleaseNotesLink = (opts.ChangeLogUrl + changelogFileName).Trim(); should use Path.Combine probably

Fixed in
c3b8879


Need system reference manual

Any ETA? I really want to use a BB w/ wireless, but none of them seem to have SRMs.

The product still doesn't have an SRM anywhere to be seen. The wiki version is entirely unfinished from the looks of it as well. I do not think it is proper to ship a product and not deliver a manual in three years.

Any news?


thuang-622-download-ux-fixes

Ticket: https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/corpora-data-portal/622
QA checks:

Hide scroll bar in Chrome, FF, and Safari
Change download prompt text to: Select one of the data formats to view its download details
Update .anndata to .h5ad (AnnData v0.7)
Update Data Portal description to:

The cellxgene data portal is a repository of public, explorable single-cell datasets. If you have a public dataset that you would like hosted on this site, please let us know at<EMAIL_ADDRESS>
Thank you!
Proof:

Looking fresh!

Awesome! Thanks so much, @pdugan20 🤩 🎉 !


Update zio-schema, zio-schema-derivation, ... to 0.4.13

About this PR
📦 Updates

dev.zio:zio-schema
dev.zio:zio-schema-derivation
dev.zio:zio-schema-json
dev.zio:zio-schema-optics
dev.zio:zio-schema-protobuf

from 0.4.12 to 0.4.13
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

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

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

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

Superseded by #613.


Wrong Behavior? Clicking in Modal

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Note:I'm assuming this is a problem with my understanding of Bootstrap, not a problem with Bootstrap itself. However, if this is an issue I'm hoping it's ok to post it here.
I cannot click anywhere inside a Bootstrap 5 modal dialog box without closing it.
If I set data-bs-backdrop="static", the modal will bounce when clicked inside but will not close. Input is still not allowed.
Including a short JS to give focus to the first input field allows keyboard input, but I cannot click on the next input.
Tabbing between fields does allow input.
I'm sorry if this is a known issue - I'm relatively new to GitHub. I have searched all over the place, including stack and here (both open and closed issues), but cannot find an answer.
Cheers!
Reduced test cases

   Register

Create an Account

            <form>

                Email Address
                <input type="text" id="email_address" class="form-control" placeholder="Email">

                Email Address
                <input type="password" id="password" class="form-control" placeholder="Password">

            </form>

            <button type="button" class="btn btn-secondary" data-bs-dismiss="modal">Close</button>
            <button type="button" class="btn btn-success">Register</button>

What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome, Firefox
What version of Bootstrap are you using?
5.1; 5.1.3

@scath999 you have just missed
 on  your markup ;)

Thanks, Geo - I did figure this out, but forgot to close. My bad, and thanks so much for responding anyway to such an inane issue - your patience is really appreciated :)


unistd: add fchdir(2)

This introduces a wrapper for fchdir(2), allowing a process to change
directory based on an open file descriptor.
The underlying function is available in libc crate since 0.2.20.

Note: this has been recently added in libc crate (https://github.com/rust-lang/libc/pull/496), and it's only available since 0.2.20 (released today).

Excellent, thanks for the contribution @lucab.  Can you add an entry to the changelog for this addition?

Also, it would be good to understand what might be happening with this failure.  We currently have some known issues with aarch64 which is why it is not gating the green check, but the failure does seem suspicious:

aarch64: https://travis-ci.org/nix-rust/nix/jobs/192841376#L591
arm-unknown-linux-gnueabihf: https://travis-ci.org/nix-rust/nix/jobs/192841377#L588
mips: https://travis-ci.org/nix-rust/nix/jobs/192841378#L1335

It would appear that the system call is not working as desired in the test environment for these targets.

Oops, I forgot the changelog, added now.
Regarding the cross-test failure, I think I'm hitting a weird combination of an old kernel on travis + some chroot/mount-ns trick by docker + (likely) some bug on aufs/overlayfs. fchdir() works, but the original root-fd seems to be leading somewhere else in the FS, possibly the lowerdir of a stacked fs. I've reworked the test to avoid grabbing a root-fd and use a self-contained tmpdir instead, now waiting for travis result of this (but it looks like it is not picking it up).

@lucab Looks like macos has an issue that needs to be resolved in the tests now.

@posborne ack. It turned out the test was not canonicalizing path, so it was fooled by symlinks. test_fchdir is now passing everywhere, despite some other (I guess unrelated) failures on cross-tests. PTAL.

Well, that was fun :1st_place_medal:.
@homu r+

:pushpin: Commit 0a654e7 has been approved by posborne

:zap: Test exempted - status

@posborne great, thanks! Please ping here back once this gets into a released crate, so I can drop some private wrappers I have in a project which needs this feature.


Fix velox dylib loading paths for conda build on macos

In this PR I'm attempting to introduce a temporary workaround to fix the velox dylib issue, via a post-build hack. This is to make conda install torcharrow work out of the box.
Currently, you will encounter an error of dylibs not found when velox is dlopen-ing them. This is because the location of those dylibs were determined at compile time. For example: /usr/local/opt/gflags/lib/libgflags.2.2.dylib was installed to a default location by brew install.
We can specify the dylib dependencies in the conda recipe, but they will be installed to the conda environment locations, rather than the system ones (/user/local/...). As a result, the velox dylib (_torcharrow.*.so) won't be able to find those libraries.
On macos there is a tool called install_name_tool that can change the dylib paths, my plan is to modify those dylib paths from absolute ones to @loader_path/... so that the velox .so can find the dylibs installed by conda.

Ok after a million times of trial-and-error, I finally found a way to adjust the recipe to

Run the conda build as usual.
Use install_name_tool to change the dylib paths for _torcharrow.so post build.
Specify the right versions of runtime dependencies that velox is happy with (notably glog==0.6.0 and arrow-cpp==8.0.0).

The most tricky part is perhaps in step 2, because conda build doesn't provide a good entry point to do a final change before it packages everything into a .tar.bz2 file. Adding stuff after python setup.py in build.sh sort of worked, but there are some extra steps to specify the compatible dylibs versions after, so it will still end up creating broken build (I still don't know all the details here, just making the observations based on my experiments).
Since the conda package is just a .tar.bz2 file, the approach I ended up taking was to unpack it, install_name_tool, pack it again using the tar command, before uploading happens. One more detail here: we'll need to run tar within the target folder so that the file list doesn't have a ./ prefix, this is critical because anaconda upload specifically hard-coded this to read the metadata. It sounds and works pretty hacky, but might be fine as a temp fix before we have static link option available on velox.

Test CI run: https://github.com/facebookresearch/torcharrow/runs/6764330823


Doesn't propagate exit code correctly

Repro:
wsrun --serial --exclude-missing --fast-exit test:e2e

This should be fixed in 3.6.4 - let us know if the issue persists.


fixed background window frame for ios < 8.0

With 1.1.0 background wax fixed for ios 8.0 but broken for 7.0 and probably earlier versions (can't check). This small change fixes background window frame for ios 7.0

Thanks


[Compute] az vm/vmss update: Fix disassociation VM/VMSS from capacity reservation

Issue: #19659
This bug may be caused by the change of service behavior:  https://github.com/Azure/azure-cli/issues/19659#issuecomment-925591346
By the way, the yaml file cannot be re recorded because there is a bug in test test_capacity_reservation that the service cannot delete those test resources. This PR has passed in the local test
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Compute


Fix Docker run argument --name now works

--name works only on the same command line of others arguments. \\ break the name.

Could you please elaborate what issues you encounter, or even better open a bug ticket showing a reproducible failure example?
The --name argument never caused issues, and all docker command options are splitted this way with \\ to make them readable in the --debug and -verbose outputs. So I am wondering what you try to fix.


Task cannot be marked complete

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Have a task assigned to you
Rename the task
Get an error
Assign the task back to you
Mark the task as complete

Expected result:
Task should be completed and there is no error
Actual result:
Error appears saying I don't have permission to complete the task
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.47-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/5eaa9af7-ff32-4a14-bf50-b79c8f8aede3
https://github.com/Expensify/App/assets/43996225/74eb944a-7a18-4aa5-8787-d7053ccb3562
Expensify/Expensify Issue URL:
Issue reported by: @tgolen
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690471361981139
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01116af54c8d03f7f6
Upwork Job ID:<PHONE_NUMBER>570476544
Last Price Increase: 2023-07-31

I also found the issue to be on the back end.
Here's the response I got after trying to mark the task as complete.

On the other hand, if you follow the reproduction steps while offline, you'll be able to mark the task as complete.

@thienlnam this looks similar to some other task issues you've been working on, right? Do you have a tracker for task cleanup issues?

This should be fixed on prod now, the BE fix was deployed https://github.com/Expensify/Auth/pull/8277

I'm still working on to update permissions on the FE, but this should be good now


Conditional coloring of bars

How to get this done in Blazor? The bar's color is based on the value.
E.g.: https://codepen.io/apexcharts/pen/RwRNXzX

hello, please check https://apexcharts.github.io/Blazor-ApexCharts/bar-charts#point colors

hello, please check https://apexcharts.github.io/Blazor-ApexCharts/bar-charts#point colors

This worked for me. Thank you.
Color="e=> GetColor(e)"


Make VectorPreProcStrategy consider undefined behaviors

Vectorization pass should not introduce extra undefined behaviors.
In some particular cases, using masking strategy could result in divide-by-zero exception, as the masking strategy by default loads zero values when mask is false.
This patch addresses such issue by falling back to loop peeling, in case extra undefined behaviors could happen using mask strategy.

@hanhanW I realized deciding the vectorization strategy is actually before we do peeling (in hindsight, of course), so I created a new test file to check the decision making part instead.

@hanhanW I realized deciding the vectorization strategy is actually before we do peeling (in hindsight, of course), so I created a new test file to check the decision making part instead.

We split the tests based on CPU arch, so there are few select_*_lowering_strategy.mlir in https://github.com/iree-org/iree/tree/main/compiler/src/iree/compiler/Codegen/LLVMCPU/test. Here I'd suggest to put the test to select_x86_64_lowering_strategy.mlir because the target in the test is x86.

but I feel like we should also test the peeling part (that the computing of residual values are good). Let me know what you think.

There are few pipeline tests in pipeline_peel_and_vectorize_tests.mlir. You can add a test to the file with a preset configuration.

Hey, avoiding masking like this is a bit counter-intuitive to me TBH:

In some particular cases, using masking strategy could result in divide-by-zero exception, as the masking strategy by default loads zero values when mask is false.

In general, this would depend on the ISA, right? I've just double-check DIV for SVE and note that (extra emphasis from me):

Signed divide active elements of the first source vector by corresponding elements of the second source vector

So, masking should be totally safe, no?

Hey, avoiding masking like this is a bit counter-intuitive to me TBH:

In some particular cases, using masking strategy could result in divide-by-zero exception, as the masking strategy by default loads zero values when mask is false.

In general, this would depend on the ISA, right? I've just double-check DIV for SVE and note that (extra emphasis from me):

Signed divide active elements of the first source vector by corresponding elements of the second source vector

So, masking should be totally safe, no?

Masking it safe for SVE, yes. but generally speaking, in LLVM IR semantics:
Division by zero is undefined behavior. For vectors, if any element of the divisor is zero, the operation has undefined behavior.

Since we are lowering to LLVM IR so less undefined behavior is better..?

Masking it safe for SVE, yes.

SVE is just a specific example of an ISA with masking. Similar argument would hold for any ISA with native masking.

but generally speaking, in LLVM IR semantics:

Division by zero is undefined behavior. For vectors, if any element of the divisor is zero, the operation has undefined behavior.

Isn't the point of masking to make sure that division doesnt't happen?
I might be missing sth here, but I think that it would be good to try to compile your example to assembly to see what happens there (I am AFK, but can try SVE tomorrow). Do you have an example for an ISA without masking?

Here's the main loop for SVE:
.LBB0_1:
  sub x15, x12, x8
  cmp x15, #4
  csel  x15, x15, x13, lo
  dup v2.2d, x15
  mov w15, w14
  mov w14, wzr
  cmhi  v3.2d, v2.2d, v0.2d
  cmhi  v2.2d, v2.2d, v1.2d
  uzp1  v2.4s, v2.4s, v3.4s
  cmpne p1.s, p0/z, z2.s, #0
  ld1w  { z2.s }, p1/z, [x9, x8, lsl #2]
  ld1w  { z3.s }, p1/z, [x10, x8, lsl #2]
  .loc  1 8 12
  movprfx z4, z2
  sdiv  z4.s, p0/m, z4.s, z3.s
  mls v2.4s, v4.4s, v3.4s
  st1w  { z2.s }, p1, [x11, x8, lsl #2]
  mov w8, #4
  .loc  1 6 10
  tbnz  w15, #0, .LBB0_1
  mov w0, wzr
  .loc  1 6 10 epilogue_begin is_stmt 0
  ldp x29, x30, [sp], #16
  ret

As expected, it looks fine. This instruction computes the predicate:
  cmpne p1.s, p0/z, z2.s, #0

Here's the example that I compiled (based on your test):
module {
  func.func @test_mod_vectorizing_strategy_peeling(%3 : tensor<6xi32>, %4: tensor<6xi32>) -> tensor<6xi32> {
    %c0 = arith.constant 0 : index
    %5 = tensor.empty() : tensor<6xi32>
    %6 = linalg.generic {indexing_maps = [affine_map<(d0) -> (d0)>, affine_map<(d0) -> (d0)>, affine_map<(d0) -> (d0)>], iterator_types = ["parallel"]} ins(%3, %4 : tensor<6xi32>, tensor<6xi32>) outs(%5 : tensor<6xi32>) {
    ^bb0(%in: i32, %in_0: i32, %out: i32):
      %7 = arith.remsi %in, %in_0 : i32
      linalg.yield %7 : i32
    } -> tensor<6xi32>
    return %6 : tensor<6xi32>
  }
}

Note, upstream IREE doesn't support scalable vectorisation of linalg.generic just yet. I am a bit behind with my PRs 😅 (I generated that using my downstream version)
Anyway, as hinted in my previous messages, this should probably be disabled for ISAs that support masking. WDYT? I can add this to my TODO list and send a PR once my other bits for vectorising linalg.generic (for SVE) are ready.

Isn't the point of masking to make sure that division doesnt't happen?
I might be missing sth here, but I think that it would be good to try to compile your example to assembly to see what happens there (I am AFK, but can try SVE tomorrow). Do you have an example for an ISA without masking?

How does it happen in practice? I could have wrong understanding about masking. I think they will eventually be lowered to masked load/store, and produce native_vector_size vectors. The rest of computation ops all take the results to perform the computation. So say that if we have 6 elements and two tiles, the second tile loads [v_0, v_1, 0, 0] for LHS and [v_2, v_3, 0, 0] for RHS, and the divisor becomes zeros which could lead to UB. The main reason is that the upstream vectorizer always use zeros as padding value.
In Alan's investigation, this is the case at LLVM level:
  %28 = getelementptr i32, ptr %9, i64 %20, !dbg !81
  %29 = tail call <4 x i32> @llvm.masked.load.v4i32.p0(ptr %28, i32 4, <4 x i1> %25, <4 x i32> zeroinitializer), !dbg !81
  %30 = srem <4 x i32> %27, %29, !dbg !81

where vectorization actually loads zero as divider due to mask being zero, and the zeroinitializer.
I don't really remember how SVE codegen config work right now. IIRC, we firstly peel the loop, and apply scalable vectorization on the main loop? If this is the case, I think we're doing the same thing here. The main loop is vectorized without masking on x86 path. The distinction of the IR is that one has scalar vector and the other does not. Correct me if I'm wrong. :)

Anyway, as hinted in my previous messages, this should probably be disabled for ISAs that support masking. WDYT? I can add this to my TODO list and send a PR once my other bits for vectorising linalg.generic (for SVE) are ready.

Yes, we can disable it if we know the ISAs support the masking. Thanks for offering the help!

Hey @hanhanW 👋🏻

How does it happen in practice?

ISel :-)  (*)
Yes, looking at your LLVM example it's quite apparent that there's potential UB. But then the backend can do it's magic and leverage the ISA to get rid of that and generate the predicated sdiv instruction that I shared in my example. IIUC, that happens during ISel.

IIRC, we firstly peel the loop, and apply scalable vectorization on the main loop? If this is the case, I think we're doing the same thing here.

Correct :) (well, strictly speaking we don't have scalable vectorization for linalg.generic yet, but that's a separate matter)

The main loop is vectorized without masking on x86 path.

Sure, but then I wonder - shouldn't every iteration in the main loop "consume" exactly vector-width of elements? (where vector-width matches hardware vector-register length) So even without masking, it would be fine?
(*) I will be OOO for a while, but will double check before sending a PR for SVE.

Yes, looking at your LLVM example it's quite apparent that there's potential UB. But then the backend can do it's magic and leverage the ISA to get rid of that and generate the predicated sdiv instruction that I shared in my example. IIUC, that happens during ISel.

Yeah, I'm happy that we are the same page now. The issue is that we generated potential UB IRs, and then some backends can get rid of UB because they have such support.
I don't find a predicated sdiv instruction in x86 ISA, so we end up with disabling the masking strategy. But sure, we can enable it for SVE path if they have the support.

Sure, but then I wonder - shouldn't every iteration in the main loop "consume" exactly vector-width of elements? (where vector-width matches hardware vector-register length) So even without masking, it would be fine?

Yes, that's right on the x86 path. On the x86 path (or say without scalable vectors), there are two strategies. One is peeling and the other is masking. The operations in the main loop are all vectorized without masking. Because they have static shapes.
Things are a little different on scalable vectorization path. The "masking" strategy that we used on x86 path becomes "peeling first, then do masking on the main loop". I think you will likely hit the same issue if the ISA does not support predicated sdiv.


Depends on deprecated cookbook python

When using poise-python cookbook and setting up a venv with python 3 the installation of supervisor fails.
==> default: [2016-01-04T22:12:28+00:00] ERROR: python_pip[supervisor] (supervisor::default line 29) had an error: Mixlib::ShellOut::ShellCommandFailed: Expected process to exit with [0], but received '1'
==> default: ---- Begin output of pip install  --upgrade supervisor ----
==> default: STDOUT: Collecting supervisor
==> default:   Using cached supervisor-3.2.0.tar.gz
==> default:     Complete output from command python setup.py egg_info:
==> default:     Supervisor requires Python 2.4 or later but does not work on any version of Python 3.  You are using version 3.4.3 (default, Oct 14 2015, 20:28:29)
==> default:     [GCC 4.8.4].  Please install using a supported version.
==> default:
==> default:     ----------------------------------------
==> default: STDERR: Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-k57arkip/supervisor

It appears to be depending on the previous cookbook for python and supervisor cookbook offers no way to install from source with the master branch now supporting python 3

Working on it, there is already a new poise-supervisor cookbook in progress on github.

so.... this cookbook will be replaced with another instead of updating it?  confused.

@keen99 This cookbook will turn into a compat wrapper just like the old monit cookbook I just took over.

+1 breaking with chef 13

Also seeing this issue with Chef 13. Working on a fix right now, and I'll submit a PR when it's ready. @coderanger has this cookbook been superseded by another?

Any update on this transition? poise-supervisor hasn't been touched in years and this cookbook is broken under chef 13.

Same place it's been, when I get to it.

In the end, I used systemd to monitor processes instead of supervisor.


Fix CSV header and String array size

Fixes a bug that prevents using OfflineConnection to perform updateSQL functionality programmatically.

I support merging this pull request.

Thanks!


bug: ion-slide swipe and click

Bug Report
Ionic version:
[x] 4.2.0
Current behavior:
Hey, has a bug in ion-slide when I do a swipe and hold and click at the same time it shows error in the transform
Video:
https://streamable.com/1evdl
Screenshot:

Steps to reproduce:
Swipe to right/left and click with swipe active
Related code:

Simple code:
<ion-slides pager="false" [options]="slideOptions" #tagSlides>
        <ion-slide *ngFor="let tag of tags">
          <ion-card class="tag-image" mode="ios" (click)="goPage(tag)">

          </ion-card>

{{tag.title}}

        </ion-slide>
      </ion-slides>

slideOptions = {
    loop: false,
    initialSlide: 0,
    slidesPerView: 1,
    centeredSlides: false,
    spaceBetween: 10
  };

Other information:

Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.12.0 (/Users/kalicki/.nvm/versions/node/v10.15.3/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.2.0
   @angular-devkit/build-angular : 0.13.8
   @angular-devkit/schematics    : 7.3.8
   @angular/cli                  : 7.3.8
   @ionic/angular-toolkit        : 1.5.0

Capacitor:

   capacitor (Capacitor CLI) : 1.0.0-beta.19
   @capacitor/core           : 1.0.0-beta.19

System:

   NodeJS : v10.15.3 (/Users/kalicki/.nvm/versions/node/v10.15.3/bin/node)
   npm    : 6.4.1
   OS     : macOS Mojave

Hi there,
Thanks for the issue. I am unable to reproduce the issue you are reporting. Can you clarify the steps you are taking to reproduce this issue?
Thanks!

Hi,
We have a slideshow with cards. So when the person gives a swipe to the right or left and hold on (in the swipe) and give a touch that error is shown.
I think in the video you can see the problem well and the interaction of swipe with click.

Hi there,
Are you able to reproduce this on a clean starter application? Given the code you provided, I am unable to reproduce the issue you are experiencing.

In this other video has another problem, two clicks with two fingers.
https://streamable.com/1pje4

Hi there,
Thanks for the follow up. You need to wrap the content in your ion-slide component in

 since your swiper instance has zooming enabled. From what it looks like, I think having the two fingers on the screen at the same time is trying to trigger something with zooming, but that zoom container div isn't present and an error is being thrown as a result.
I am going to close this as it is not a bug, but I will make sure the ion-slides docs are updated to make note of this.
Thanks for using Ionic!

Actually, changed my mind. I think it's pretty easy enough for ion-slide to include the wrapper, so I'll update the slides component to do that.
In the meantime, the code I posted previously will work!


Allow deleting users and removing users from your organization

This fixes #185

This isn't quite done... I just realized I need to fix login to fail if the user is deleted

Get this green, please.


Discussion: Handle subscription (React)

Some random thoughts which went through my mind in regards to the subscription management.

Is there a way to unsubscribe from an event? I expected the EventBus to have a unsubscribe function.
What are the best practises to set up a subscription? Let's say, component B subscribes to events published by component A.

bus.subscribe("componentA.event", handler)

Now let's say component B gets re-rendered to do some changes in the data. As a result, the code above will execute again and it will subscribe 'probably' for the 2nd time to events from component A. Will the bus still keep a reference to the old handler?
If so, would it be better to use the useEffect() hook with the 'bus' dependency in order to avoid obsolete subscriptions?
useEffect(() => {
  bus.subscribe("componentA.event", handler)
}, [bus]);

And should the useEffect() also cleanup the subscription when the component B unmounts? (goes back to the first bullet point)
This here would be my preferred solution. Would be good to have a hook for that to avoid boilerplate code.
useEffect(() => {
  bus.subscribe("componentA.event", handler)

  return function cleanup() {
      bus.unsubscribe("componentA.event", handler)
    };

}, [bus]);

Ah, I saw now that the subscribe method returns a function which can be used to unsubscribe.
https://github.com/ryardley/ts-bus/blob/4fd905f0e17ea52803586620c462f342592d9666/src/EventBus.ts#L68

That is correct mind I close this?


Is it necessary to store ASID in the PTE cache?

My concern is that different ASIDs may allocate their own page tables at the same physical address. Therefore, I would like to confirm whether the PTE cache needs to match the ASID to differentiate between them.

You may need to clarify:  ASIDs allocating their "own" page tables implies separate tables.  But "at the same PA" implies that the same table is being shared by two ASIDs.
In any case, in general one does not have to include ASID in cache tags, but then one has to treat an ASID-specific sfence as a non-ASID-specific sfence.

Got it, no further question.

You may need to clarify: ASIDs allocating their "own" page tables implies separate tables. But "at the same PA" implies that the same table is being shared by two ASIDs.
In any case, in general one does not have to include ASID in cache tags, but then one has to treat an ASID-specific sfence as a non-ASID-specific sfence.


hope to use trained embedding as input

Now it seems that we can just use prompts from model, but other prompt like embedding from trained can't be used(ex: band-hands-v5, EasyNegative). How to support this?

I'm currently planning to support text embeddings, but I'm busy with real life so I don't know when that will be implemented.


pkgdev commit: generate_restrictions TypeError: parser_exit() missing 2 required positional arguments

Attempting to commit to a non-standard file: scripts/setup-and-run-repoman.sh. Git repo at commit state attached.
telans@genbox /var/db/repos/guru $ pkgdev commit -s
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/arghparse.py", line 1212, in parse_args
    delayed(args, attr)
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/arghparse.py", line 382, in __call__
    self.invokable()
  File "/usr/lib/python3.8/site-packages/pkgcheck/addons/git.py", line 378, in __call__
    namespace.restrictions = self.generate_restrictions(parser, namespace, ref)
  File "/usr/lib/python3.8/site-packages/pkgcheck/addons/git.py", line 357, in generate_restrictions
    parser.exit()
TypeError: parser_exit() missing 2 required positional arguments: 'status' and 'message'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python-exec/python3.8/pkgdev", line 4, in <module>
    scripts.run(basename(__file__))
  File "/usr/lib/python3.8/site-packages/pkgdev/scripts/__init__.py", line 38, in run
    sys.exit(tool())
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/tool.py", line 79, in __call__
    ret = self.main()
  File "/usr/lib/python3.8/site-packages/pkgdev/cli.py", line 13, in main
    return super().main()
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/tool.py", line 185, in main
    self.handle_exec_exception(e)
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/tool.py", line 170, in main
    exitstatus = func(self.options, self.out, self.err)
  File "/usr/lib/python3.8/site-packages/pkgdev/scripts/pkgdev_commit.py", line 735, in _commit
    pipe = scan(options.scan_args)
  File "/usr/lib/python3.8/site-packages/pkgcheck/api.py", line 38, in scan
    options = pkgcheck.argparser.parse_args(base_args + ['scan'] + args)
  File "/usr/lib/python3.8/site-packages/snakeoil/cli/arghparse.py", line 1214, in parse_args
    raise TypeError("failed loading/parsing '%s': %s" % (attr, str(e))) from e
TypeError: failed loading/parsing 'staged': parser_exit() missing 2 required positional arguments: 'status' and 'message'

guru.tar.gz

Actually, I suppose this should be moved to snakeoil?

Fixed in https://github.com/pkgcore/pkgcheck/commit/3d05b553214482d117b1c9bade8984e9f94a072a.


Add: illustrative home page

Please allow me to work on it with KWOC tag!

Assign me this @ItsRoy69 under iwoc24. I can work if @AaadityaG is not working on it.

@AaadityaG since its been more than a week and its been inactive so assigning to @praptisharma28

@praptisharma28 can go for it

@praptisharma28 are you still working on this issue ?

@ItsRoy69 yes I am wait for my PR

Okay

@ItsRoy69 please add the tags.

This looks good

#123 have a look @ItsRoy69


Newsletter Status

@lfarah @vsouza
hey guys is the newsletter active? on hiatus?

@dkhamsing I was in vacation,but now I'm back. I'll talk with @lfarah to restart our work on newsletter. 😄

@vsouza, @lfarah and @dkhamsing : Please let me know if you need my help on this.


Set up Doxygen

Also host the Doxygen generated HTML page on gh-pages

This is complete as of commit 7080c181450268381017407128af077bbcf0308c


HADOOP-17609. Make SM4 support optional for OpenSSL native code.

3019

 Quality gate: Passed
1 Issues

 1 Anti-patterns

Details


Cross thread allocation/deallocation bug test + potential fixes

An unsynchronised access of span->list_size in
_rpmalloc_deallocate_direct_small_or_medium can result in a span being
recycled and put back into use before the store of
span->free_list_deferred occurs in the call to
_rpmalloc_deallocate_defer_small_or_medium that initially "freed" this
span. The eventual store of span->free_list_deferred corrupts the span
resulting in crashes.
This patch adds an artificial delay between the increment of
span->list_size and the store of span->free_list_deferred to make
reproduction of the issue more likely and thus faster with the included
test case.
Two potential fixes are also included.

Fixed in #282


New Resources added for new resource types added in 1.18

Is your feature request related to a problem? Please describe.
There's new stuff generating in 1.18, so we need new resources.
Describe the solution you'd like
All the stuff that generates needs to have a resource.
Describe alternatives you've considered
Just using Registry(), but I don't think that's a good thing to do.
Description aka Additional context
Assigning this to Josh.

As far as I can tell, this is limited to just the new ore veins.
Maybe other stuff too? Not sure.


Add utility function to remove a value.

Must handle cases where the value is used as the target of annotations, and other uses outside of function bodies.
Perhaps it also suggests further values which might have become dead.

Some existing optimizations only handle this partially.  For example, they don't remove dead values when those values are the target of decorations or an OpName.

Actually, @greg-lunarg has pretty much the whole solution in DCEInst in LocalSSAElimPass https://github.com/KhronosGroup/SPIRV-Tools/pull/691
It deals with OpName, OpDecorate, and has a TODO for OpGroupDecorate.

See also DefUseManager::KillInst and DefUseManager::KillNamesAndDecorates


python3 support and make it as a vim plugin

python2/python3 is all supported with this pr,
and a vim plugin is added so that we can quick install and less configuration.

Thanks, but one issue right off the bat is that I still use this on systems that use Python 2.6, so I'm not willing to lose that support just yet. :-(

So, I didn't write a Vim plugin thinking that I didn't want to confine people to Vim (there's no reason other editors couldn't use this).  But I guess I can see the value in having it too.  I need to think more about it though.

It should running on python2.6 as well. I test success under sles11SP1 with python2.6.

It should running on python2.6 as well. I test success under sles11SP1 with python2.6.

Hrmph, you're right.  Okay.  I'll try and take a closer look at it this weekend.

There is a bug if filename and content use different encodings. I have no idea how to solve it.

I'm just not keen on this approach, and it could be useful for other editors than just Vim.  Thank you and sorry it's been languishing here so long.


Add PropExpr and ChainExpr as convenience traits to chain GtkExpressions

See #693
the closure expression api could be futher simplified by instead of having:
.closure_expression(|args| {
    let last_modified: DateTime = args[1].get().unwrap();
    gettext!("Last edited {}", last_modified.fuzzy_display())
})

it could be
.closure_expression(|last_modified| {
    gettext!("Last edited {}", last_modified.fuzzy_display())
})

but I'm not sure how to implement it with generics

but I'm not sure how to implement it with generics

You can't but see https://github.com/gtk-rs/gtk-rs-core/pull/338

also cc @jsparber as you have had to use expressions before in fractal-next

Looks all good to me otherwise

Now we only need a useful example that makes use of expressions

Now we only need a useful example that makes use of expressions

The book has some

The book has some

Not using these convenience traits :). Besides the examples work as documentation for gtk-rs and this will be hard already to discover feature for people who are not closely following the book.

The book has some

Not using these convenience traits :). Besides the examples work as documentation for gtk-rs and this will be hard already to discover feature for people who are not closely following the book.

Rest assured, that the book will make use of this after it has been merged 😉
But I agree, examples would still be nice

I do agree too with more examples

@bilelmoussaoui Let's get this in for now, and examples can happen later?
There's the color example as a PR currently which uses expressions, that could hopefully be moved to this helper API

I'll follow up an example tomorrow if that's okay

I'll follow up an example tomorrow if that's okay

Even better :)

Should I include it in this PR? Ill squash this one then append another commit for the example

I found a possible problem with chain_property while working with the example. Sometimes it is needed to specify the value type. For example
list_item
    .property_expression("item")
    .chain_property("number")

this won't work because a gtk::ListItem's item will evaluate into a GObject, thus it doesn't have a number property unless it is downcasted into a type that has a property of number.
To solve this, the this_type param for PropertyExpression will be exposed:
list_item
    .property_expression("item")
    .chain_property(TypeThatHasNumberProp::static_type(), "number")

I found a possible problem with chain_property while working with the example. Sometimes it is needed to specify the value type. For example
list_item
    .property_expression("item")
    .chain_property("number")

this won't work because a gtk::ListItem's item will evaluate into a GObject, thus it doesn't have a number property unless it is downcasted into a type that has a property of number.
To solve this, the this_type param for PropertyExpression will be exposed:
list_item
    .property_expression("item")
    .chain_property(TypeThatHasNumberProp::static_type(), "number")

You could do it like in https://github.com/gtk-rs/gtk-rs-core/pull/315 . You could list all properties of the type of the object, and from there get the type that actually defines the property.

You could do it like in gtk-rs/gtk-rs-core#315 . You could list all properties of the type of the object, and from there get the type that actually defines the property.

It should have a syntax of chain_property::<glib::Object>(prop_name), like what was stated by bilelmoussaoui?

That would work, yes

That would work, yes

I think it would be simpler to use T::static_type() where T IsAglib::Object

yes that's the same approach :-)

I meant instead of finding the property like gtk-rs/gtk-rs-core#315

That seems less user-friendly though. The user would have to know on which of the parent types the property actually is.

I read upon gtk-rs/gtk-rs-core#315, but not sure how would i translate it for this

I meant instead of finding the property like gtk-rs/gtk-rs-core#315

That seems less user-friendly though. The user would have to know on which of the parent types the property actually is.

I read upon gtk-rs/gtk-rs-core#315, but not sure how would i translate it for this
I gave it a try, and it's giving same errors as before, the find_property returns None:
pub fn chain_property(&self, property_name: &str) -> crate::PropertyExpression {
        let pspec = glib::object::ObjectClass::from_type(self.value_type())
            .unwrap()
            .find_property(property_name)
            .unwrap_or_else(|| panic!("Couldn't find a property named `{}` to lookup", property_name));

        crate::PropertyExpression::new(pspec.owner_type(), Some(self), property_name)
    }

This can't really work because you're using the value_type, which is the type of the value the expression evaluates to and not the object having such property. In other words, I think you always have to specify the type T

This can't really work because you're using the value_type, which is the type of the value the expression evaluates to and not the object having such property. In other words, I think you always have to specify the type T

But you already have the type: it's the type of &self.

Ah I think you got confused by the two chain methods; the one in the GObject trait already makes uses of the T: IsA<Object> for that; the other one inside impl Expression can't do that :-)

I think the way it is implemented now is all good, or am I missing something?

I think the way it is implemented now is all good, or am I missing something?

Yes it's all good now, i will have a quick look at the rest of the code & merge

Thanks for your contribution @SeaDve :-)

thanks for the review too!


Fix gorm remove PreparedSQL field

[ ] Do only one thing
[ ] Non breaking API changes
[ ] Tested

What did this pull request do?
gorm.io/plugin/dbresolver
/Users/smalls/go/pkg/mod/gorm.io/plugin/dbresolver@v1.5.2/dbresolver.go:147:5: unknown field PreparedSQL in struct literal of type gorm.PreparedStmtDB
User Case Description

请合并 !!!! :(

go.mod 添加一行
替换 gorm.io/plugin/dbresolver v1.5.2 => github.com/smalls0098/dbresolver v0.0.0-20240909115957-b928cd627cd8
临时解决，等待修复

@jinzhu This is breaking builds for several people. A merge & version update would be appreciated.

@a631807682 is there any chance you can add a new tag now that this has been merged? v1.5.3?

@a631807682 please add a new tag

@deefdragon @sunerpy use v1.5.3


tasmota template diffuser problem tuya

Hello, I have installed tasmota in my aromatherapy (https://es.aliexpress.com/item/32948538359.html?spm=a2g0s.90423<IP_ADDRESS>4263c0oT8boY) all very well, but I can't find a correct template for my correct aromatherapy, tried to use this:
https://templates.blakadder.com/essential_oil_400ml.html
But the light switch on HA does not work and neither does the water sensor, when I press the light switch on HA it turns me off and turns on the LED of the aromatherapy state and not the RGB light, when the water sensor runs out of water Water keeps marking OK.
Could you help me solve and make everything work correctly for me? I'm going to put some pictures of the plate that my aromatherapy has.

Thank you and sorry for my bad English.

Hi @roberdlv, glad tuya-convert was able to help you successfully install third party firmware. For support with your third party firmware, please see the respective project.


Works on given test case. Hope I didn't break anything.

Reference #143

I just saw this review request yishay, I had already replied to the commit email with a couple of quick suggestions. I will try to look at this in more detail tonight my time.
Here's what I added via email:
-You might find a place in the code where there is something already that has 'needClosure' logic, but the logic that sets it to true does not yet include the specific case you are dealing with.
-Take a look at emitClosureStart and emitClosureEnd I think those will help out.
(The second one is likely not needed if the first one is a solution, but these are just clues to help you better understand the compiler code, not any recommendation about the way to do what you are specifically working to fix - I have not looked into it yet)

Hi,
Changing to
emitClosureStart();
emitAssignedValue(assignedNode);
MemberAccessExpressionNode maenode = (MemberAccessExpressionNode)assignedNode;
emitClosureEnd(maenode.getLeftOperandNode(), definition);
Results in
org.apache.royale.utils.Language.as(c, MyButton).clickFunction = org.apache.royale.utils.Language.closure(this.parentApplication.buttonFunction, 'Function');
Which fails with
TypeError: Object.defineProperty called on non-object
Watch
Call Stack
org.apache.royale.utils.Language.closure
Language.js:281
MyBox.addElement
MyBox.js:39

The problem is that my code is in JSEmitter which descends from JSRoyaleEmitter which overrides JSGoogEmitter.emitClosureStart(). I'm not sure there's a clean way to reuse the latter function.
If you do think of something let me know. Need to sign off...
Thanks.

Sorry I did not find time last night to check that. I see what you mean about the inheritance levels. There is some code in JSEmitter which I added which likely needs to be added to an overridden method in JSRoyaleEmitter. I will try to look into that this coming weekend.
What you might consider is overriding 'emitAssignedValue' in JSRoyaleEmitter and try adding the check in there, make the changes if necessary, and call the super method otherwise.
You should also check it is not a static function you are assigning,  and avoid doing anything special in that case, I think.
I will try to more time to look more tonight.

This can be achieved in a more elegant way if the application source code is modified to make the function source explicit.


cli: add helm release management command

Closes #363.

Pending:

[x] Move push to Helm repository from GH actions to this CLI

Tested in local repo, outcome here, steps to reproduce:
$ reana-dev helm-upgrade-components
$ reana-dev release-helm -u <YOUR-GITHUB-USERNAME>


Multihit samples in oncoprint

I was wondering how difficult would be that oncoprint function show genes with more than one variant type (multihit) as half (or proportion) of each variant in the cell.
For example, take a gene which has Missense mutation (green) and Frameshift (red). Would be easy to plot the specific cell as half green and half red?
Thanks!

Hi,
That's an interesting idea. I have never thought of it. I can give it a shot and see if its doable.
Thanks.

This would be powerful to have but I imagine hard to code. The ultimate solution would also inform you of how many of each type of mutation there is and perhaps look like

Yeah, I tried to do it but it was quite hard implement it. Image command would not let me split the boxes.

Wow, @PoisonAlien, that'd be wonderful idea! (very interesting suggestion, @DarioS).


DNS resolution problems, pod hangs forever if failed to lookup

Hello, we have some problem with CoreDNS cache, why in session chrome-71-0-4d29b470-9abd-4094-99da-ac88caad0012 after first bad resolved request nothing happened? Pod and service was not killed. We know we can turn the cache off https://aerokube.com/moon/latest/#_features_not_working
Can we try to resolve it again from client?
proxy-logs.log

We a working now to implement retry on client side in case of errors...
ср, 20 февр. 2019 г., 15:48 f4hrenh9it<EMAIL_ADDRESS>
Hello, we have some problem with CoreDNS cache, why in session
chrome-71-0-4d29b470-9abd-4094-99da-ac88caad0012 after first bad
resolved request nothing happened? Pod and service was not killed. We know
we can turn the cache off
https://aerokube.com/moon/latest/#_features_not_working
Can we try to resolve it again from client?
proxy-logs.log
https://github.com/aerokube/moon/files/2884598/proxy-logs.log
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/moon/issues/130, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABWNbQ6OzgqhfliBVFyTmD2eMfYzYwMQks5vPUQSgaJpZM4bFI4K
.

This one was implemented.


Update error summary links to previous styling

TODO

[ ] Update error summary link styles

Outcomes

Error summary links should use link component if possible (visually should use the error styling)

Estimated Effort

[ ] XS: 1 to 2 hours
[ ] S: half a day
[ ] M: 1-2 day
[ ] L: 1 week
[ ] XL: Move to an epic

Out of scope

Dependency on SSR work, ready for publishing.

I received the new preview + anatomy images but I'm still missing the new alt-text description in English + french for the new anatomy images.
Here are the anatomy images that we need alt text for:

@SmartMouthWords flagging you for the missing content - thank you!

@SmartMouthWords this is the ticket for the error summary alt text

Added error summary alt text in FR and EN.

Hold off on this to release together with Wednesday 5/8 release.

@melaniebmn please update this ticket to reflect the new decisions.

Reversing the decision to use our default link styling for error summary links and going back to using the red error style instead. Here is the decision document for that decision:
https://docs.google.com/document/d/185seVPLzZr11b9lGjBieZolLp3XOk8Uedu4Pef7ucBA/edit

@adorayi closing in favor of: https://app.zenhub.com/workspaces/design-system-6100624a19f4cf000e46e458/issues/gh/cds-snc/design-gc-conception/899


Signed-off-by: Pritam Saha<EMAIL_ADDRESS>

Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@freben I raised a  fresh PR for this in #17677 .
You can continue there & close this PR.
Please let me know if other changes required .

@freben is the PR still open or closed?

Well it's open right? The other one was closed in favor of this one, and this one still does not have a green status.


Remove class and add className instead in JSX

Replace class with className

Can I work on this issue?

Issue fixed.


Server side instability

In about 5-10% of our tests, we see a server side failure of MAPDL when running within docker.
2021-12-04T07:10:08.1299565Z  Current ANSYS Traceback:
2021-12-04T07:10:08.1300044Z      Mapdl::Nodes
2021-12-04T07:10:08.1300453Z      Mapdl::VGet2
2021-12-04T07:10:08.1300858Z      prepcl
2021-12-04T07:10:08.1301274Z      Mapdl::SendCommand
2021-12-04T07:10:08.1301561Z
2021-12-04T07:10:08.1301938Z  *** ERROR ***                           CP =       0.000   TIME= 00:00:00

@FredAns, is there a way we could improve stability of the VGet2 method?

Lets keep tabs on this, but close this out for now.


Path sensitive static analysis of web applications for remote code execution vulnerability detection

http://dl.acm.org/citation.cfm?id=2486874

Data link Not working


engine: adapt engine metrics for multi-rocks

What is changed and how it works?
Issue Number: Ref #12842
What's Changed:

Share one statistics for different tablets
Iterate all tablets when flushing metrics
Put private metrics into a histogram

Related changes
tikv/rust-rocksdb#725
Check List
Tests

Unit test

Release note
None

For single-rocksdb:

For multi-rocksdb:


Add some new compression tests

Adds tests for: https://github.com/dotnet/corefx/pull/11264, https://github.com/dotnet/corefx/pull/11235
@stephentoub

LGTM


Page numbers in Index are sorted alphabetically instead of numerically

For example;

I'd expected it to be
arguments, 10, 11, 15, 22-24, 27, 30,
    47, 49, 57-58, 63, 71, 80, 87, 94,
    100, 139, 174, 192, 197, 241

Thanks for catching this!


Add a timeout option

This PR adds a new option, --cbmc-timeout which can be used for setting a timeout for the CBMC run.
The timeout applies on a per-harness basis in the Kani run.
The implementation uses tokio  to perform non-blocking reads from the CBMC process's stdout so that the timeout can be checked asynchronously.
Call-outs:
There are other names I considered for the option:

--harness-timeout: This makes it clearer that the timeout applies to each individual harness (and not to the overall Kani run). It doesn't indicate that the timeout only applies to the CBMC portion of the flow though.
--engine-timeout: This makes it a bit more general across multiple engine/backends. The notion of an engine is not really part of the Kani jargon currently though, so might not be clear to users.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

The changes seems to have slowed down Kani (regressions have been running for over an hour). Turning the PR into a draft till I investigate.

Addressed the performance issue in https://github.com/model-checking/kani/pull/3649/commits/0502cc20ac1296e10f0e7d4ad0a13c527f22f3a6.


Ghost 1.0 UI

Keepin' track of 1.0 UI things

[x] Update application UI frame
[x] New content management UI
[x] Update button styles
[x] Update modal styles
[x] Update subscribers styles
[x] Consolidate team/apps UI components
[ ] Rename team/apps UI component
[x] Update labs styles
[ ] Update general settings styles
[ ] New editor UI
[ ] New editor publish button workflow
[ ] New tags UI
[ ] New edit profile UI
[x] Night shift
[ ] Wire up night shift option in labs to load app-dark.css instead of app-css - @acburdine ?
[ ] Polish
[x] New mobile navbar
[ ] Fix mobile nav toggle

Any thoughts on moving the themes table + maybe supporting some of the new stuff we should get from the updated package.json specs?

Moving themes table, I think so yes, I need to do settings/general first to kind of figure out how that's going to work to have an idea of where it might go. But yes, def thinking about it!

Finished 🎉 ✨


fix(repl) Fix evaluation order for match expressions

Signed-off-by: Jerome Simeon<EMAIL_ADDRESS>Issue #706

Reverse the evaluation order for match cases in native/REPL eval code

Coverage remained the same at 95.542% when pulling e554e51a3559f38e909ac41b0aa02df78c2227b9 on jeromesimeon:js-fix-match-706 into bb15f634926462c728d7f4aeb394a4404494739a on accordproject:master.


Implement remark command [DO NOT MERGE]

A wise man once asked me, "how would you describe Ivan's code?"
I thought for a minute, and then replied,
"Elegant. Exquisite. Eloquent."
You've truly blown me away with this one. Well done, Ivan.


Blazor templates need updating

Not sure what might have changed with RTM, but the Blazor templates are still listed as Preview 9

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

ID: c07e57dc-526f-9479-4695-de7d8f06eb9a
Version Independent ID: 8226ed72-69b2-3c93-a931-218bd11cf9c8
Content: Get started with ASP.NET Core Blazor
Content Source: aspnetcore/blazor/get-started.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

That's correct. It's only for the Blazor WebAssembly template. Blazor WebAssembly is still in preview. After Blazor WebAssembly is released, that step will be dropped.

Thanks


Diagram mentions rebranded product

Hi - the diagram shows "OMS" but I think this was rebranded

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

ID: bdc23836-c10c-5c47-818b-42c53916b269
Version Independent ID: b5ef4736-1df9-e3d9-eafb-7782cc0d5fea
Content: Best practices for enterprises moving to Azure - Microsoft Cloud Adoption Framework for Azure
Content Source: docs/cloud-adoption/appendix/azure-scaffold.md
Service: cloud-adoption-framework
Sub-service: reference
GitHub Login: @rdendtler
Microsoft Alias: rodend

Azure Scaffold is being deprecated shortly. No additional updates planned.


feat(copy): 添加复制全部 复制行 复制列功能

👀 PR includes

✨ Feature

[x] New feature

🎨 Enhance

[ ] Code style optimization
[ ] Refactoring
[ ] Change the UI
[ ] Improve the performance
[ ] Type optimization

🐛 Bugfix

[ ] Solve the issue and close #0

🔧 Chore

[ ] Test case
[ ] Docs / demos update
[ ] CI / workflow
[ ] Release version
[ ] Other ()

📝 Description
添加复制全部 复制行 复制列功能 暂时仅透出方法
🖼 Screenshot

Before
After

❌
✅

🔗 Related issue link

🔍 Self Check before Merge

[ ] Add or update relevant Docs.
[ ] Add or update relevant Demos.
[ ] Add or update relevant TypeScript definitions.

文档需要更新不

文档需要更新不

文档等透视表的昨晚一起更新吧

文档需要更新不

文档等透视表的昨晚一起更新吧

帮你加个todo哈哈哈

:tada: This PR is included in version @antv/s2-v1.15.0-alpha.3 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version @antv/s2-v1.15.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change color on mouse move

Hello, I think this little addition can make your control more responsive

Thanks a lot! I'll get a new release out to Nuget when I can.


Unable to decode a date in nanoseconds

What happened:
If the unit of a date, is a number of nanoseconds elapsed since a reference date, xarray cannot read this dataset and throws the exception KeyError: nanoseconds.
What you expected to happen:
As this unit is valid in CF model, it must be decoded. Today the units that are recognized go from day to microseconds in the module coding/time.py.
Minimal Complete Verifiable Example:
import numpy as np
import array as xr

xr.DataArray(
    np.arange(0, 1e10, 1e9).astype("int64"),
    attrs=dict(units="nanoseconds since 1970-01-01")).to_netcdf("/tmp/test.nc")
xr.open_dataset("/tmp/test.nc")

Environment:
Output of xr.show_versions()
INSTALLED VERSIONS
commit: None
python: 3.8.3 | packaged by conda-forge | (default, Jun  1 2020, 17:43:00)
[GCC 7.5.0]
python-bits: 64
OS: Linux
OS-release: 3.10.0-957.12.2.el7.x86_64
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
libhdf5: 1.10.5
libnetcdf: 4.7.4
xarray: 0.15.1
pandas: 1.0.4
numpy: 1.17.5
scipy: 1.4.1
netCDF4: 1.5.3
pydap: None
h5netcdf: 0.8.0
h5py: 2.10.0
Nio: None
zarr: 2.4.0
cftime: 1.1.3
nc_time_axis: None
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: None
bottleneck: None
dask: 2.18.1
distributed: 2.18.0
matplotlib: 3.2.1
cartopy: None
seaborn: None
numbagg: None
setuptools: 47.3.1.post20200616
pip: 20.1.1
conda: 4.8.3
pytest: 5.4.3
IPython: 7.15.0
sphinx: None

Thanks @fbriol .
We would happily take a PR for this.

We would absolutely accept a pull request for this, but I'm not 100% sure that nanoseconds are valid per CF conventions. CF conventions references udunits.dat, which as far as I can tell includes no units smaller than 1 second:
https://www.unidata.ucar.edu/software/udunits/udunits-1/udunits.txt

We would absolutely accept a pull request for this, but I'm not 100% sure that nanoseconds are valid per CF conventions. CF conventions references udunits.dat, which as far as I can tell includes no units smaller than 1 second:
https://www.unidata.ucar.edu/software/udunits/udunits-1/udunits.txt

Nanoseconds (and seconds with any other SI prefix from yocto to yotta) are indeed valid in the CF conventions. For reference, there is this table in v1.8 of the conventions, as well as the SI prefix and SI base unit databases of UDUNITS.

i ran into this problem trying to round trip time to the nanosecond (even though i don't need it, sub micro second would be nice)
but unfrotunately, you run into the fact that cftime doesn't support nanoseconds
https://github.com/Unidata/cftime/blob/master/cftime/_cftime.pyx
Seems like they discussed a nanosecond issue a while back too
https://github.com/Unidata/cftime/issues/77
Their ultimate point was that there was little point in having precision down to the nano second given that python datetime objects only have microseconds. I guess they are right.

This issue was addressed as part of https://github.com/pydata/xarray/pull/4684.


npm package?

I saw this repo very usefull. Do you have plans to submit it in the npm directory ???
The updates in it will be easy to follow and apply

Hello, @dianjuar,
What do you mean? You can install this repository from npm by npm i material-shadows.

OH! so sorry. It was there in the Readme...
Maybe, I didn't find it because the repo doesn't have those cool stickers of npm.
Thanks for taking your time in this dumb question.


Disabling context initialization

Hello
I am trying to use sokol_app with bgfx for a webasm project. bgfx has its own context initialization and swap buffer. It is conflicting with sokol_app one. Is there a way to use sokol_app for input and other webasm goodies without 3d context initialization?
Thanks

Oops sorry somehow I missed the notification for this issue.
On some backends the window system code is quite closely connected to the 3D context/device setup, I think supporting such a feature across all backends might be a bit non-trivial (for instance on macOS everything is expected to be setup within the didFinishLaunching callback).

Thanks, flooh, yeah, I think I will keep it for the WebGL development only.


Cloud

Hi,
How can i run this model in a cloud server? Unfortunatelly realtime.py is not working since the browser can not reach a local hardware. Any advice?

cloud systems such as colab don't let you to access your webcam real-time


fix(kzip_info): don't count c++ stdlib includes as invalid absolute paths

In general, paths in the kzip should be relative (to the root of the repo), so we count absolute paths as errors. in some cases like c++ stdlib includes, these are expected and should not count as errors. There are likely other exceptions to the rule that we'll want to add in the future.

Even the C++ standard library includes should be relative; we need to update the various vnames.json to ensure this is the case. IIRC we put them in the / root (or could invent something like <system>), but they should still

be relative
not have an empty root

(Pretty sure we should do something similar for the kythe_builtins as well.)

Sounds good, thanks. A number of kzips that are built by other teams currently contain a lot of files in /usr/include/c++, so I guess we'll need to ask them to update their vnames.json configs.

Right, I think the default vnames.json we ship don't always do the right thing here.


[code-infra] Migrate to prettier async APIs

Sync APIs are deprecated in upcoming major.
Pushing these changes without bumping the major to avoid cluttering it with all the formatting changes.
Haven't deeply investigated, but it looks like there are more opportunities for parallelization in these scripts. I also noticed a few sync IO calls.

[ ] Will have to canary this on X: https://github.com/mui/mui-x/pull/11734

blocked https://github.com/mui/mui-x/pull/11734 on which is blocked on https://github.com/mui/mui-x/pull/11671


Error running 'Estimate cell-type specific expression of every gene in the spatial data (needed for NCEM)'

Hi there,
Thank you so much for your great tool!
I am trying to run this code on my data:
expected_dict = mod.module.model.compute_expected_per_cell_type(
    mod.samples["post_sample_q05"], mod.adata_manager
)

and getting the following error:
AttributeError: 'Cell2location' object has no attribute 'samples'
I will definitely provide more info if necessary but I was just wondering if there is something obvious I may have done wrong? I see that the mod object does have the following mod.sample2df_obs,  mod.sample2df_vars, and mod.sample_posterior but none of these are subscriptable.
Your assistance would be very much appreciated :)

I recommend checking previous issues about this topic https://github.com/BayraktarLab/cell2location/issues/195#issuecomment-1217202426


could not find version for r2d2_redis ^0.14.0

top level Cargo.toml has r2d2_redis = "0.14.0"
> RUST_BACKTRACE=full cargo features
The application panicked (crashed).
Message:  could not find version for r2d2_redis ^0.14.0
Location: src/parsing/mod.rs:83

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
                                ⋮ 7 frames hidden ⋮
   8: cargo_features::parsing::get_package_from_version::h9adf2ae15de3d34d
      at <unknown source file>:<unknown line>
   9: cargo_features::parsing::workspace::parse_dependency_from_item::hbc28695eb0e6fa25
      at <unknown source file>:<unknown line>
  10: <core::iter::adapters::GenericShunt<I,R> as core::iter::traits::iterator::Iterator>::next::hff79f788288508a7
      at <unknown source file>:<unknown line>
  11: <alloc::vec::Vec<T> as alloc::vec::spec_from_iter::SpecFromIter<T,I>>::from_iter::h802efb554352d864
      at <unknown source file>:<unknown line>
  12: cargo_features::parsing::package::get_packages::he0f8481e84b6b8e6
      at <unknown source file>:<unknown line>
  13: cargo_features::document::Document::new::hb8c0d65edeb81d65
      at <unknown source file>:<unknown line>
  14: cargo_features::main::h47ba66dd8e4b45f1
      at <unknown source file>:<unknown line>
  15: std::sys_common::backtrace::__rust_begin_short_backtrace::hcb0ecaaaee466828
      at <unknown source file>:<unknown line>
  16: std::rt::lang_start::{{closure}}::heaa73ce6c24cbcb8
      at <unknown source file>:<unknown line>
  17: std::rt::lang_start_internal::h103c42a9c4e95084
      at <unknown source file>:<unknown line>
  18: main<unknown>
      at <unknown source file>:<unknown line>
  19: __libc_start_call_main<unknown>
      at <unknown source file>:<unknown line>
  20<EMAIL_ADDRESS>      at <unknown source file>:<unknown line>
  21: _start<unknown>
      at <unknown source file>:<unknown line>

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.

does anything differentiate this dependency from any other? since it seems to be just a normal one
any chance deleting the target folder and the Cargo.lock helps?
does the same error occur with a nother dependency when you remove this one?

When I remove this line, the problem goes away, and the program works; relevant area
r2d2 = "0.8.10"
r2d2_redis = "0.14.0"
rand = "0.8.5"

Interestingly, it was no longer being used by any member package.

Yup; fixed. thanks


HALON-502: Ignore requests to restart mero process inside halond.

Created by: qnikst

Closing as an obsolete


fix all files using  pre-commit

Fix all files using  pre-commit

Why
It looks like some files have not been checked using pre-commit hooks at the past. By manually running pre-commit against all errors found are resolved

What

Fix all errors to skipped files
Remove whitespace from docker related files
Remove whitespace from Makefile
Add missing .nvmrc file for managing node version

Requires migrations? No
Requires dependency updates? No

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

Project coverage is 93.11%. Comparing base (223e146) to head (a539686).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1170      +/-   ##
==========================================
- Coverage   93.11%   93.11%   -0.01%
==========================================
  Files         580      580
  Lines       43649    43646       -3
  Branches     3139     3139
==========================================
- Hits        40645    40642       -3
  Misses       2393     2393
  Partials      611      611

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


[MBL-1777] Register To Receive APNs Token On App Launch

📲 What
Adds the registerForRemoteNotifications() method of UIApplication toAppDelegate.didFinishLaunchingWithOptions to request the APNs device token on app launch.
🤔 Why
Tarveen and E have called out that they haven’t been getting Push Notifications for a while.
The running theory is that their APNs token may have been replaced by the APNs Server for one of the following reasons:

restoring the phone from a backup
transferring data from one phone to another
not having reinstalled the app or logged out for an extended period of time

Currently, we're only registering the app and receiving the token at log-in when users opt-in to push notifications, so...
This PR adds that method above so that the app registers and then receives the current APNs token on app launch in the UIApplication.didRegisterForRemoteNotificationsWithDeviceToken callback.
From there, we send the token to our v1/push_tokens server endpoint as expected.
👀 See
No Visible change
✅ Acceptance criteria
It isn't easy to test this, but I've verified that this callback is now called on app launch and when opting into notifications.

For more info, check out the Apple recommended approach for Registering an app and retrieving it's device token

⏰ TODO

[ ] Check in with Tarveen and E once this goes out to make sure they are getting notifications again

https://github.com/kickstarter/ios-oss/pull/2172#discussion_r1799896847


Special cases when using FilterKeywords() with EnableFormDataLogging()

Found that if the keywordBlackList parameter contains string.Empty / "", ALL the form values are obfuscated. I understood that IndexOf() that works internally on the list, finds the empty string at the beginning of any non-null string. Is this to be expected? I think that the intention would be more obvious if one could be able to configure EnableFormDataLogging with something like .ObfuscateAllValues() while the empty string will be ignored or stripped from the keywordBlackList.
Also, if a null slips into keywordBlackList, it crashes with the same IndexOf() throwing ArgumentNullException. Null should be ignored/stripped from the list.
What are your thoughts on these?

Hi @liviriniu  ,
I would probably handle different cases in a different way.

if a null slips into keywordBlackList, it crashes with the same IndexOf() throwing ArgumentNullException. Null should be ignored/stripped from the list.

If a null slips into keywordBlackList, I think I would just through a ArgumentException in the call .FilterKeywords(...) , so that the caller detects it as early as possible and can correct it (this will then fix the issue down the line).

Found that if the keywordBlackList parameter contains string.Empty / "", ALL the form values are obfuscated

Passing a "" is indeed an edge-case , I would say the behavior does make sense and is aligned with the documentation of the method :

The Form Data values will be offuscated when the key contains one of the black-listed words.

and I would probably not change it now, so as not to break clients who might be relying on that behavior.

I think that the intention would be more obvious if one could be able to configure EnableFormDataLogging with something like .ObfuscateAllValues() while the empty string will be ignored or stripped from the keywordBlackList.

But as you propose, I do think it could make sense to have an option like ObfuscateAllValues() that will just obfuscate all values .... but I honestly have a hard-time figuring out a use case where this would make sense 🤔  (if you are enabling FormData, I'd suspect you are interested in seeing at least some of the values, right ? ).
Overall, I think we have to be a bit careful introducing possibly breaking changes in the library, as it is used in many places, and I probably don't want a change to accidentally cause sensitive information to appear in logs.
So I think I would :

happily accept some extra protection / guards against nulls in the list of keywords
not want to change the behavior when passing "" in the list of keywords so as to not introduce breaking change
maybe not accept a new method ObfuscateAllValues() as I sort of have a hard time imagining a use case for this ... and also having a hard-time finding a name that would be consistent with the other methods there (.DisablePasswordFiltering(), .FilterKeywords(), FilterKeywords())

Does this make sense ?
thanks again for your contribution 👍

It does make a lot of sense, yes. I see the bigger picture now, thanks.

Can't see through/decide between:

if I understood correctly: make .FilterKeywords(keywordBlackList) throw something like "the black list does not support null items" as to be clear why it breaks. The caller is helped to see the null early and avoid problems later in its own code. By this, logger keeps the current intolerant behavior to null although it could be made to tolerate it.
the other way, silently strip all nulls and let the filtering always happen. Null doesn't make sense as a keyword, so why not ignore it. I reckon it's the caller's responsibility to take care of own code. But yeah, caller would not know about eventual problem later on because of null. Also, existing API consumers after a change like this could wake up with code treating nulls at the point of calling FilterKeywords that is redundant. I fail to see the importance of this. Maybe let it all be as it is now :))

I agree with not changing behavior of passing "", did not see the breaking change risk. Indeed it is.

ObfuscateAllValues() is not a good idea, I agree with your arguments on this and do not see benefit in complicating things by trying to achieve consistency with existing methods. If someone would ever want to, let's say, just enumerate the form field names for the sake of some special debug and at the same time keep log volume low, a different kind of FormData property would make sense - array of strings. Well, I just explored the idea, making this happen probably necessitates a different config builder anyhow.

To choose between :

rejecting invalid inputs
tolerate slighly incorrect inputs

is choosing between :

Enforcing preconditions
or
"Be conservative in what you send, be liberal in what you accept" of the Robustness principle

Both could probably be fine, but in that specific case, if you remember from the code, there are already default keywords that are obfuscated (*password* and things like that if I recall properly). Not providing your own "keywords" means "use the default" .... but if you provide [null] ... is an array with null the same as not providing keywords ? one could decide both ways, but there is no really obvious interpretations ... so it might be better to just get rid of the ambiguity and not allow cases that could cause seemingly incorrect behavior in the first place.
So yes, I would probably add a Guard Clause, in the form of throwing a ArgumentException, indicating that the argument is not correct, the name of the argument and why it's not correct ("cannot contain null" or something of the sort, as you said).
In my opinion, ending up with an collection that contains nulls probably means something is wrong prior to there, so I'm fine rejecting, and possibly helping the caller figure out that they are indeed doing something wrong.
Does that sound sensible ? :)

(also, if we go ahead and add this behavior, it would be nice to have a test to "make it official" that is is the desired behavior)

Learning something every day :)

better to just get rid of the ambiguity and not allow cases that could cause seemingly incorrect behavior in the first place
I understand now that it's better this way, I failed to see the "use default" aspect. I do have my past experiences with PHP ambiguity in using basic functions... so yeah. This reminds me of the weak vs strong typing discussion.

I'll see what I can do


[Pilot] Add Earn screen skeleton to wallet

Description
Add skeleton of Earn screen to wallet for upcoming Earn pilot. Allows users to enter a Figure Eight ID (which will be precreated for pilot participants) and displays amount earned.
Note that the Figure Eight iFrame will be added in a future PR so this PR alone will not allow users to complete work, just to login with their ID and view the amount earned.
Tested
With celo-org-mobile-pilot firebase

No review needed on this PR just making it easier to keep track of the Earn pilot build


Update plugin dependency versions to match bigdatagenomics/adam

Re https://github.com/bigdatagenomics/cannoli/pull/19#commitcomment-21786357

After a second look, there aren't any that need updating to match bigdatagenomics/adam.
These are noted as having updates available; let us test those upstream first
[INFO] The following plugin updates are available:
[INFO]   maven-assembly-plugin .................................. 2.6 -> 3.0.0
[INFO]   maven-compiler-plugin ................................ 3.5.1 -> 3.6.1
[INFO]   maven-jar-plugin ..................................... 3.0.1 -> 3.0.2
[INFO]   maven-resources-plugin ............................... 3.0.1 -> 3.0.2
[INFO]   maven-site-plugin ...................................... 3.5.1 -> 3.6
[INFO]   maven-source-plugin .................................. 3.0.0 -> 3.0.1
[INFO]   maven-surefire-plugin ................................ 2.19.1 -> 2.20
[INFO]   org.codehaus.mojo:build-helper-maven-plugin ........... 1.12 -> 3.0.0
[INFO]   org.codehaus.mojo:clirr-maven-plugin ..................... 2.7 -> 2.8
[INFO]   org.eluder.coveralls:coveralls-maven-plugin .......... 3.0.1 -> 4.3.0
[INFO]   org.scoverage:scoverage-maven-plugin ................. 1.1.1 -> 1.3.0


Add support for Ruby 3.0

Ruby 3.0 separates positional from keyword arguments, which renders some calls in the gem not to work anymore (mostly where options is used as a Hash instead of keyword arguments). Splatting the hash makes the methods work again.

Coverage remained the same at 100.0% when pulling 0e67fbebe2012c82a64f9716c18cfe8025173630 on floriandejonckheere:ruby-3-0-0 into 08eae1d881a8324cee50d7e6bd2c6ca76779bd13 on Fullscreen:master.

Coverage remained the same at 100.0% when pulling 0e67fbebe2012c82a64f9716c18cfe8025173630 on floriandejonckheere:ruby-3-0-0 into 08eae1d881a8324cee50d7e6bd2c6ca76779bd13 on Fullscreen:master.

Thanks @floriandejonckheere, I was able to get going again by updating my gemfile to point at your branch:
gem "squid", "~> 1.4", github: "floriandejonckheere/squid", ref: "0e67fbebe2012c82a64f9716c18cfe8025173630"


Note of Backup storage redundancy

[Enter feedback here]
Note of Backup storage redundancy section
Zone-redundant storage is currently available only in specific regions. Based on the region you select; "this option will not be available for new or existing accounts."
I don't quite understand the quoted statement in bold, if the option will not be available for new or existing accounts, then the option will be available for which type of accounts? Thanks!

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

ID: c9011b1f-d923-d3a1-474e-c3431caac3d2
Version Independent ID: c123745e-1b65-0726-8168-663fef81ea73
Content: Configure Azure Cosmos DB account with periodic backup
Content Source: articles/cosmos-db/configure-periodic-backup-restore.md
Service: cosmos-db
GitHub Login: @kanshiG
Microsoft Alias: govindk

@MS-jeremy Thank you for the feedback. We are actively investigating and will get back to you soon.

@MS-jeremy This statement seems a little confusing but it tries to say that few regions would support new accounts only and not existing accounts, while few regions other way round.

Thanks @AnuragSharma-MSFT for the clarification.
So it means, some regions would only support the new accounts to enable Zone-redundant, while some regions would only support the existing and new accounts? Please correct me if I'm wrong.
And can you please kindly revise the statement so it can be more specific and clear to avoid confusing customers. Thanks!

@MS-jeremy Thanks for your reply and yes you are right. I am assigning it to the content author to further evaluate and update accordingly.

@MS-jeremy what the note means is for existing or new accounts, Zone redundancy is supported for specific regions only.
For example, when creating an account, if you select "East US", all the three storage redundancy options appear:

If you select "West US" region, the zone redundancy option doesn't appear.

I will make some tweaks to clarify.

#please-close


Document how to explicitly set GraalVM version with Buildpacks

Currently generates a java.lang.RuntimeException: file name '/cnb/buildpacks/paketo-buildpacks_java-native-image/5.3.0/.git/logs/refs/heads/fix/remove-ca-certificates' is too long ( > 100 bytes) error as discussed with @dsyer and @scottfrederick, but will hopefully be fixed on Spring Boot side.


[Snyk] Upgrade vite from 5.4.10 to 5.4.11

Snyk has created this PR to upgrade vite from 5.4.10 to 5.4.11.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.

The recommended version was released on 23 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)


Make default bagging directory not hidden?

Dart's default bagging directory is $HOME/.dart . While this is less of an issue on Windows, there can be an issue instructing users on how to actually open this directory on Mac . I've instructed users to immediately click on the output path link in the final Review and Run page after the job has completed, and then copy/paste the package elsewhere if they're not planning on uploading immediately.
Is there a reason why the default dir is a hidden directory, and at the very least, could this be changed? Ideally, a folder could be created on the desktop by default as it would be the most accessible, but it might be annoying?

Yes, the default should probably change. A common problem is that users don't notice the bags that have accumulated in the bagging directory, and then wonder why they have so little disk space after building a few very large bags.
For now, you can change the bagging directory.

From DART's top menu, choose Settings > Application Settings.
Click on Bagging Directory.
Enter a new directory name in the Value field.
Click Save.


 Doctor found issues in 1 category.

Sounds like you are passing an invalid path to NetworkImage
18:31:25.438 163 info flutter.tools flutter: Image provider: NetworkImage("", scale: 1.0)
18:31:25.438 164 info flutter.tools flutter: Image key: NetworkImage("", scale: 1.0)

looks like you are passing an empty string

@zoechi tks, but like this:
          FadeInImage.memoryNetwork(
            placeholder: kTransparentImage,
            image: widget.model.icon_url,
            fit: BoxFit.cover,
          ),

Seems widget.model.icon_url, is an empty string when this code is executed.

@zoechi I tested it, it won't be empty.

FutureBuilder<ColumnList>(
        future: refreshDate(),
        builder: (BuildContext context, AsyncSnapshot snapshot) {
          if (snapshot.hasData) {
 buildGridTileList(snapshot.data),
}

@zoechi
09:31:43.296 11 finer runtime.gc collection time 41ms • 45.2MB used of 76.6MB • isolates/1071756948
09:31:43.869 12 info flutter.tools flutter: URL: "https://test.xxx.com/zone/B1tj6zW-X.png"
09:31:43.881 13 info flutter.tools flutter: URL: "https://static.xxx.cn/FgCeTom6OnjUfEfo-ejHLB1-o_Cy"
09:31:43.882 14 info flutter.tools flutter: URL: "https://test.xxx.com/zone/SkhLN0PAz.png"
09:31:43.889 15 info flutter.tools flutter: URL: "https://test.xxx.com/zone/SkfK78FRM.jpg"
09:31:44.048 16 info flutter.tools flutter: ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
09:31:44.050 17 info flutter.tools flutter: The following ArgumentError was thrown resolving an image codec:
09:31:44.050 18 info flutter.tools flutter: Invalid argument(s): No host specified in URI file:///
09:31:44.062 19 info flutter.tools flutter:
09:31:44.062 20 info flutter.tools flutter: When the exception was thrown, this was the stack:

The exception might not come from your image urls, but from the engine trying to load a codec.

Could you please try to set a breakpoint at line

NetworkImage._loadAsync (package:flutter/src/painting/image_provider.dart:484:57)

and check the value of key.url for every time the breakpoint is hit?

@zoechi
Thank you, this error suddenly gone.
I don't know what happened.

Interesting.
Perhaps a case where flutter clean had helped. #16604
I'll close the issue then because without a way to reproduce there is no way to debug.
Feel free to create a new issue if you run into this problem again or add a comment and I'll reopen.


重装系统支持初始化密码

这个 PR 实现什么功能/修复什么问题:

支持重装系统时初始化密码
修复cloudroot和cloud-init密码冲突问题
修复同步公有云镜像时os_type不准问题
跑通windows、linux自动化测试

是否需要 backport 到之前的 release 分支:

release/2.9.0
release/2.8.0
release/2.7.0
release/2.6.0

/cc @swordqiu @yousong

/cherrypick release/2.9.0

/cherrypick release/2.8.0

/cherrypick release/2.7.0

/cherrypick release/2.6.0

Codecov Report

Merging #577 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #577      +/-   ##
=========================================
+ Coverage    7.38%    7.4%   +0.02%
=========================================
  Files         346     345       -1
  Lines       54692   54279     -413
=========================================
- Hits         4038    4020      -18
+ Misses      50275   49882     -393
+ Partials      379     377       -2

Flag
Coverage Δ

#aFlag
7.4% <0%> (+0.02%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/compute/guestdrivers/azure.go
4.54% <0%> (-2.81%)
:arrow_down:

pkg/util/billing/billingcycle.go
34.48% <0%> (-20.69%)
:arrow_down:

pkg/compute/guestdrivers/openstack.go
5.49% <0%> (-5.62%)
:arrow_down:

pkg/util/seclib2/seclib.go
91.48% <0%> (-4.26%)
:arrow_down:

pkg/compute/guestdrivers/qcloud.go
3.78% <0%> (-2.03%)
:arrow_down:

pkg/cloudcommon/db/db_dispatcher.go
0% <0%> (ø)
:arrow_up:

pkg/compute/models/guestdrivers.go
18.18% <0%> (ø)
:arrow_up:

pkg/cloudcommon/db/fetch.go
0% <0%> (ø)
:arrow_up:

pkg/compute/guestdrivers/virtualization.go
0% <0%> (ø)
:arrow_up:

... and 65 more

Continue to review full report at Codecov.

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

/lgtm
/approve


[signing] Read gpg-key from gpg-cli

Is your feature request related to a problem? Please describe.
No problem issue present
Describe the solution you'd like
Currently, env vars for GPG have to be set -- including gpg keys (both pub+priv) and the passphrase.
For builds on my local PC, I would like to use gpg to handle this stuff so I do not have to copy this data to other places in clear text.
Describe alternatives you've considered
./.
Additional context
./.

@bmarwell with #143 in place, do you still want this particular feature?

Well, yes. I have a yubikey and the idea is that I do not extract it from the key… gpg-agent? :)

I was afraid you'd say that. As I recall BC recommends not using file based keys with GPG as the format changed between gpg and gpg2. Have to figure out what alternatives may be available.

:D no worries, there is no need to rush. If it was only for me, create a tag "low priority" and maybe take a look at the maven-gpg-plugin.

I believe the maven-gpg-plugin calls the signing command using a Java process, isn't it?

Yes.
https://github.com/apache/maven-gpg-plugin/blob/master/src/main/java/org/apache/maven/plugins/gpg/GpgSigner.java#L51-L71
https://github.com/apache/maven-gpg-plugin/blob/master/src/main/java/org/apache/maven/plugins/gpg/GpgSigner.java#L89-L99
I think they want to change the command to something --with-colons, so it is machine-parsable.
The GpgSigner class is already pretty stand-alone.

Configure with mode = COMMAND. See docs for additional settings.


Bug Report: CLI command "Grant Access to the User at tenant root scope"

Describe the bug
Grant Access to the User at tenant root scope “/” to deploy Enterprise-Scale (Bash) fails in Cloud Shell.
Current snippet is:
#sign  into AZ CLI, this will redirect you to a web browser for authentication, if required
az login

#assign Owner role to Tenant root scope  ("/") as a Owner (gets object Id of the current user (az login))
az role assignment create --scope '/'  --role 'Owner' --assignee-object-id $(az ad signed-in-user show --query  objectId)

The following command works successfully:
#sign  into AZ CLI, this will redirect you to a web browser for authentication, if required
az login

#assign Owner role to Tenant root scope  ("/") as a Owner (gets object Id of the current user (az login))
az role assignment create --scope '/'  --role 'Owner' --assignee-object-id $(az ad signed-in-user show --query "objectId" --output tsv)

Note: Additions "objectId" --output tsv
Steps to reproduce

Run the documented commands in Cloud Shell

Screenshots
N/A

Thanks @lufussel for raising this. Have created PR #723 to fix this with your updated command after testing myself to confirm 👍


deps: bump for compat with cargo-component v0.10.1

It's like https://github.com/peterhuene/wasmbuilder.app/pull/16, but from a branch named deps

@peterhuene hopefully this makes it work for you!

I've deployed this PR to https://wasmbuilder.app.


include record offsets in warcio check output

What's the justification for this? While this is a relatively new interface and so it's less disruptive to change it... isn't the WARC-Record-ID that's already printed enough to debug a problem found by "warcio check"? My not-yet-ready "warcio test" is the sort of place to debug weirdness like duplicate WARC-Record-IDs.
I'm not expecting a big answer, just curious why you felt this was useful.

So that you can find the records reported on (e.g. with warcio extract). Otherwise you have to do an extra scan over the warc (e.g. with warcio index). When you have a record that fails check, you're gonna want to look at it.

Seems reasonable thanks!
(Accidentally put it on master though, now fixed -- probably should make develop the default branch)


Proposal: Provide snippets/templates for DependencyProperty

DependencyProperty standard 'propdp' template doesn't work. Add clear guidance and a VS Regex example to convert to the proper setup.
cc/ @Scottj1s

It should also be updated from the old one to use nameof instead of a hard-coded string:
public static readonly DependencyProperty ContentProperty =
            DependencyProperty.Register(nameof(Content), typeof(UIElement), typeof(Case), new PropertyMetadata(null));

@anawishnoff FYI

@angelazhangmsft Do we have docs currently that are lacking modern examples or is this a suggestion to draft up a new article about DependencyProperty?

@anawishnoff I think this is about the Visual Studio snippets that one can use to create a DependencyProperty.

It should also be updated from the old one to use nameof instead of a hard-coded string:
public static readonly DependencyProperty ContentProperty =
            DependencyProperty.Register(nameof(Content), typeof(UIElement), typeof(Case), new PropertyMetadata(null));

@bpulliam do you know where the Visual Studio snippets for XAML live? Is this something that the Windows App SDK package can update or is it in VS iteself (this would work for all XAML flavors using this snippet anyway)? It'd be nice to use nameof and make it easier to refactor property names by not having a hard-coded string for dependency properties here.


运行 docker 容器后卡住了

The cache for model files in Transformers v4.22.0 has been updated. Migrating your old cache. This is a one-time only operation. You can interrupt this and resume the migration later on by calling `transformers.utils.move_cache()`.
0it [00:00, ?it/s]
Loading pipeline components...: 100%|██████████| 7/7 [00:03<00:00,  1.99it/s]
Loading pipeline components...: 100%|██████████| 7/7 [00:00<00:00,  7.13it/s]

卡在这里不动了，重新 docker build --no-cache 然后 run 也是这样

7860 可以打开了