Font problems in Firefox on 24 inch monitor

Issue Summary
I discovered that I am experiencing some pretty severe font problems for MathJax in Firefox (version 102.2.0esr (64-bit)) but only when using a large monitor (24 in., 1920 x 1080 resolution). I also found the equations look bad at 100% Firefox browser zoom level but fine at 110%. The same equations look correct in Firefox on a smaller laptop monitor at 100% browser zoom level (14 in., 1920 x 1080 resolution). They also look correct in Chrome and Microsoft Edge on both my large and small monitors (also 100% browser zoom level). I typically use Chrome, so I’m not sure how long this issue could have been occurring (maybe the entire time I've been working w/ MathJax). Some of my colleagues see the same issue, and some can’t reproduce the issue (even when seemingly using the same settings and screen sizes).
The problem only occurs for the main MathJax display. It looks fine in the zoom pop-up box. The issue seems to at least affect stretchy fences, summation and product symbols, and square root symbols (there could be other things, though, too). The problem is not present when I change from Math Renderer HTML-CSS to SVG. I know in the past you've warned about HTML-CSS, but this is what we've retained on our website as the default as of now because for the most part we like the look of it. Is this problem just something we'd have to live with (for some people) if we want to stick w/ HTML-CSS mode, or is there anything else we could do to help it? Also, any idea why some of us see the issue and some of us don't? Thank you in advance!
Here are some examples from https://pubs.acs.org/doi/10.1021/acs.jctc.2c00257 (it's open access, so you should be able to view it):
Eq 1:
24 in. monitor, 100% browser zoom, Firefox:

14 in. laptop screen, 100% browser zoom, Firefox:

Here’s an example showing how the equations in the zoom pop-up box appear totally fine:

Eq 2:
24 in. monitor, 100% browser zoom, Firefox:

14 in. laptop screen, 100% browser zoom, Firefox:

Eqs 22/23:
24 in. monitor, 100% browser zoom, Firefox:

14 in. laptop screen, 100% browser zoom, Firefox:

And here’s an example from https://pubs.acs.org/doi/10.1021/acs.jctc.1c01044 (inline-formula toward the end in the "Conformational Clustering" section):
24 in. monitor, 100% browser zoom, Firefox:

14 in. laptop screen, 100% browser zoom, Firefox:

Technical details:

MathJax Version: MathJax.js v2.7.9
Client OS: Windows
Browser: Firefox version 102.2.0esr (64-bit)

Supporting information:
Browser console errors (if applicable):

I don't have a larger monitor to test on, so I can't reproduce the issue.  Which version of Windows are you using, 10 or 11?
It looks like there may be an issue with Firefox's refreshing of large characters (there was an issue like this in v3 with WebKit, but I haven't seen it in v2).  If you resize the browser window, does it refresh properly?  If you zoom in or out, does that change anything?

I'm using Windows 10. If I resize the browser window on my large 24 in. monitor, the problem characters remain a problem still. If I use the full size window on my 24 in. monitor and change my browser zoom level in the top right corner from 100% to 110%, the problem goes away (the problem only seems to be present at 100% or lower). I just found that I can also replicate it on my 14 in. laptop screen if I change the zoom level in the top right from 100% to 67% or lower (looks fine at 80% and higher).

@dpvc Just to be clear, is there anything we can do to fix this ourselves? Is it an individual user or configuration issue, a MathJax issue, a browser issue, etc.? Should I report this to Firefox? Would it resolve if we switched from v2.7.9 to v3?

Sorry for not getting back to you on this.  I'm currently teaching my fall courses, so have much less free time than usual.  I haven't been able to look into this further, but it is probably a browser issue having to do with how the browser decides what should be refreshed.
We have just released an alpha version of MathJax 4.0.0 which includes support for 10 different fonts (including the STIX2 fonts), along with automatic line breaking, so it might be worth looking into that rather than v3, as I think you needed those features.


Docs: Fix the iterate callback arguments

Also, in the edited example, what if the db has just 2 keys?
The result on the SuccessCallback will be undefined I assume? So the message "Iteration has completed, last iterated pair:" does not make much sense in this case?

Oops, can't believe I missed this, thanks!
That's true, but it's just supposed to be a quick example... you could make it more accurate, but it would make it longer. For completeness, you'd want:

// Exit the iteration early:
localforage.iterate(function(value, key, iterationNumber) {
    if (iterationNumber < 3) {
        console.log([key, value]);
    } else {
        return [key, value];
    }
}, function(err, result) {
    if (!err) {
        if (result) {
            console.log('Iteration has completed, last iterated pair:');
            console.log(result);
        } else {
            console.log('Iteration has completed.');
        }
    }
});


Improve PyThaiNLP performance

PyThaiNLP wants you help us to improve the performance. You can fork this git, coding new code to improve the performance, and send your pull request to PyThaiNLP.
These are some lists for you.

[ ] Delete unused code
[ ] Port model to onnx #639
[ ] Reduce the use of external packages that are not standard Python packages.
[ ] Use Cython for any code but still keep pure python for not installed cython or cannot install cython

I propose to remove tinydb as an external dependency (as mentioned in #506). It seems to me that using a 3rd-party database library is an overkill for just querying a JSON file which is by no means large in size.
tinydb is only used twice, once in init.py for database initialization, which could be simply replaced by creating an empty JSON file, and once in core.py for querying the local database, which could be done using json in the Python standard library instead.
And it is confusing that while tinydb is used to query the local JSON database, it is not used when parsing JSON data returned from https://pythainlp.github.io/pythainlp-corpus/db.json (see core.py).
The performance improvement gained by using tinydb or other 3rd-party database libraries is, I suppose, to be negligible, but to have one more unnecessary external dependency is not a good idea, I think, for future development and maintenance of the project. Instead, JSON querying could be easily done using json in the standard library.
What is your opinion? If you agree, I would be happy to work on this and send an PR.

I propose to remove tinydb as an external dependency (as mentioned in #506). It seems to me that using a 3rd-party database library is an overkill for just querying a JSON file which is by no means large in size.
tinydb is only used twice, once in init.py for database initialization, which could be simply replaced by creating an empty JSON file, and once in core.py for querying the local database, which could be done using json in the Python standard library instead.
And it is confusing that while tinydb is used to query the local JSON database, it is not used when parsing JSON data returned from https://pythainlp.github.io/pythainlp-corpus/db.json (see core.py).
The performance improvement gained by using tinydb or other 3rd-party database libraries is, I suppose, to be negligible, but to have one more unnecessary external dependency is not a good idea, I think, for future development and maintenance of the project. Instead, JSON querying could be easily done using json in the standard library.
What is your opinion? If you agree, I would be happy to work on this and send an PR.

I am agree.

When working on #691, I found a problem relating to the version checking behavior of the corpus downloader.
The downloader checks both the name and the version here, so later if the corpus is found in the local database and a re-download is not forced, the versions should always match.
I'm not sure what the expected behavior is here. If a corpus with the same name but different version is found in local database (and force = True is not specified), should the downloader suggest the user to use force = True or just silently re-download and rewrite the latest version of the corpus?

Yes, It is not force download that you can see https://github.com/PyThaiNLP/pythainlp/runs/7957249794?check_suite_focus=true#step:5:5219 and https://github.com/PyThaiNLP/pythainlp/blob/dev/tests/test_corpus.py#L86.

@wannaphong I mean that since both the name and the version of the corpus is checked, the else block would never be reached, so the user will never be notified that a newer version of the requested corpus is available.
That is, in cases that the corpus is found in the local database, but the version do not match, the latest version of the corpus would be silently downloaded and rewritten. If this is indeed the expected behavior, the else block would be redundant.

OK. I'm agree.

@wannaphong So what is the expected behavior? If the user should be notified to use force = True when newer versions of the corpus are available, I could work on a patch to fix it.

@wannaphong So what is the expected behavior? If the user should be notified to use force = True when newer versions of the corpus are available, I could work on a patch to fix it.

I'm agree. The user should get notified when newer versions are available.

@wannaphong Should be fixed in #692, please review the PR.

I'm doing reduce import time. #719


Create AWS resources to support Defect DOJO deployment

Defect Dojo is going to run as a BOSH deployment in our tooling environment. It needs a few supplementary services in order to run. These are detailed in the Architecture documentation.
Here is a summary of what I've identified so far:

An RDS instance, probably with the Postgres engine.
A domain name that's internally / VPN routable (not public). dojo.fr.cloud.gov? To be created in Route53.
A message broker. It's not totally clear to me if this is required, but it's listed on the Architecture page. We can probably find out for sure by looking through the files in the other deployment methods, like the docker-compose files and Helm charts. Defect Dojo suggests using RabbitMQ, but we might also be able to use something AWS-hosted. If we use RabbitMQ, we would likely run it as a job in the BOSH deployment.

I propose we add these to the cg-provision repository instead of e.g. the defectdojo-boshrelease repository so we can keep all of our Terraform in one place. There is probably prior art in code related to other BOSH releases.
We'll also need to figure out how to supply credentials for the RDS instance to the BOSH deployment. I imagine we can look to resources for other BOSH deployments for templates here as well.

@jameshochadel Mind filling in the description here when you have the chance?

Staging resources have been created for use in our tooling environment

Moving this to next PI, as we don't need to worry about it until we're ready to deploy our prod Defect Dojo


React Group Project Setup with kanban board

In this milestone we achieve 👍
https://github.com/RashidMenhas/React-Group-Project/projects/1

we are two coding partners, and we assign tasks to each other.

Hi @RashidMenhas & @Ade179,
STATUS: APPROVED ✔ ✔
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations, and all the best with your capstone! 🎉

Highlights

You have set up your Kanban board correctly 👍 👍
All tasks have been assigned fairly 👏

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Updates onnxruntime to 1.16.3

Codecov Report
Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (dc6cfb1) 82.68% compared to head (9e09cb5) 82.68%.

Files
Patch %
Lines

onnx_extended/__init__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #120   +/-   ##
=======================================
  Coverage   82.68%   82.68%
=======================================
  Files          51       51
  Lines        6595     6595
=======================================
  Hits         5453     5453
  Misses       1142     1142

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


Fix version transaction

This fixes the default branch latest flag flickering we observed as part of #2571 by properly re-instating the db transaction we had in place to avoid exactly this problem.
I broke this in 5d13688873b460da1fbd984d73b4209c57b46835 during the async/await rework.
We now have a regression test in place that will prevent this from breaking again.

Placing in draft just to avoid merging this until I'm done verifying that current main that's running staging is OK and can be deployed.


Added Withdrawal Mechanism.

Title
Added Withdrawal Mechanism
Fixes  Issue #74

Proposed Changes
The original contract lacks a mechanism for the campaign owner to withdraw the funds collected, which poses a risk to both the campaign owner and potential donors. To address this issue, it's crucial to include a withdrawal function that allows the campaign owner to access the funds.
By introducing a withdrawal mechanism, the campaign owner can retrieve the funds collected after the campaign deadline has passed. This ensures that the funds can be utilized for the intended purpose of the crowdfunding campaign.
In the modified contract, we have added a withdrawFunds function that enables the campaign owner to initiate a withdrawal.

Additional Information
The withdrawFunds function includes several checks and operations:

It verifies that the campaign ID provided is valid.
It checks if the caller of the function is the campaign owner.
It ensures that the campaign's deadline has passed.
It calculates the amount available for withdrawal by subtracting the previously withdrawn amount from the total amount collected.
It checks if there are funds available for withdrawal.
It transfers the funds to the campaign owner's address using payable(campaign.owner).call{value: amountToWithdraw}('').
It emits the fundsWithdrawn event to notify external systems about the successful withdrawal.

Check List (Check all the applicable boxes)

[x] Bug(s) fixed
[x] Enhancement done
[x] New Feature(s) added
[ ] Documentation changed
[x] All new and existing tests passed.
[x] My code follows the code style of this project.

Minor Bug Fixes, Documentation Issues 10 Points


Changelog for v1.12.6

We should release final 1.12.x version before we release 1.13.0

I don't think there will be such critical issues that can not wait for 1.13.0

Sorry!

Just sharing the love of and pain I feel for our diff infrastructure

For anyone else reading we're referring to #13157

yeah... third rewrite if I counted right in few weeks :D

Well this one is a complete rewrite of ParsePatch - others were bodges because I didn't want to have to rewrite it.
We should actually rewrite at least once more though with a proper AST structure and potentially to a pure streaming structure although that won't be backportable.

Is anything happen on this branch?

Waiting for one more PR to be merged

Nope, one more issue should be fixed

@lafriks witch one is it?

Not created yet, working on it :)

Not created yet, working on it :)

nice, but when will be available?

It was not that important as I initially thought so it can also be included just in 1.13.0 final

🚀

@zeripath done

Maybe wait for an extra PR? @lafriks

What PR?

What PR?

#13527

@zeripath ping

Oh, I accidentally merged this.


Missing languages

I got to this page by working through the tutorial for creating a PowerShell function via the Azure portal.  The first step was great, easy to follow.  This second step -- adding an output binding -- has two issues.  (1) the tutorial didn't say anything about setting up storage.  I had storage settings from a previous project and was able to use those.  (2) My biggest complaint is that there are only two languages (C# and Javascript) in the step that shows how to use the storage.  How do I make use of storage in my PowerShell function?

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

ID: e952d385-d5c8-de9b-27c5-4d02add9b180
Version Independent ID: 50083880-0ec5-cbb4-363e-ba9e82f2889a
Content: Add messages to an Azure Storage queue using Functions
Content Source: articles/azure-functions/functions-integrate-storage-queue-output-binding.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@conery
Thanks for your feedback! We will investigate and update as appropriate.


Failed to create a new service mount disk

The service mount disk error is as follows:
  Normal   Scheduled               2m53s                  default-scheduler        Successfully assigned default/nginx-ceph-7455b6f7f6-kgdwq to ceph2-n
  Normal   SuccessfulAttachVolume  2m52s                  attachdetach-controller  AttachVolume.Attach succeeded for volume "pvc-27b9fd2a-a9e8-413f-9e7e-b3add6f7afcf"
  Warning  FailedMount             50s                    kubelet, ceph2-n         Unable to attach or mount volumes: unmounted volumes=[nginx-ceph-html], unattached volumes=[nginx-ceph-html default-token-ppwwg]: timed out waiting for the condition
  Warning  FailedMount             36s (x9 over 2m45s)    kubelet, ceph2-n         MountVolume.MountDevice failed for volume "pvc-27b9fd2a-a9e8-413f-9e7e-b3add6f7afcf" : kubernetes.io/csi: attacher.MountDevice failed to create newCsiDriverClient: driver name rbd.csi.ceph.com not found in the list of registered CSI drivers

@lixiaopengxiya please fill the issue template when you are creating the issue

Closing this one as there is no update on this issue. please feel free to reopen.


feat: log a valid link to starting a server with lazy-module

Super quick fix: by including the protocol (which is hardcoded to http as we don’t have access to it, and will likely not ever be a problem as HTTP rewrites would have this covered too)

This change would make it so that the output link on the console would be clickable via Cmd+Click ◡̈

Duplicate of https://github.com/elysiajs/elysia/pull/227


Trinkets metadata

Commit messages contain a little more information here. I'm not entirely thrilled by the first commit. Ideally Plethora's metadata would contain CC:T's item details, and item details would contain plethora's metadata, but doing that without getting tying yourself in knots seems quite hard.

We discussed the possibility of indexing slots like tey're stored in the data instead (head/face):

But given that it's clunkier to index this way, will probably stick with what the PR has for now


info confused by sections with the same name

Trying to read the "10.7.1 Basic Logging Tutorial" I was confused when the link to "A simple example" led not to <IP_ADDRESS> but to "6.1.1 A simple example".  Obviously info gets confused by same-named sections.
Thank you for a great package, I find it very useful.

Did updating Sphinx fix it? Otherwise, is there a sphinx bug report?


No Space left on device.

I got this error while compiling,
LLVM ERROR: IO failure on output stream: No space left on device
LLVM ERROR: IO failure on output stream: No space left on device
error: could not compile move-resource-viewer
warning: build failed, waiting for other jobs to finish...
error: could not compile tonic
thread 'rustc' panicked at 'assertion failed: pos.get() <= self.position()', compiler/rustc_metadata/src/rmeta/encoder.rs:426:9
LLVM ERROR: IO failure on output stream: No space left on device
error: could not compile move-vm-test-utils
LLVM ERROR: IO failure on output stream: No space left on device
LLVM ERROR: IO failure on output stream: No space left on device
error: could not compile move-core-types
error: could not compile move-stackless-bytecode-interpreter
The following warnings were emitted during compilation:

If I had to guess: you're running out of space for rustc to store intermediate build outputs on /tmp. The default size depends on your distro, but systemd uses 50% of RAM size by default. You have a few options here:

Unmount tmpfs, so it uses disk (and cache) instead: umount /tmp
Make tmpfs bigger: mount -o remount,size=8G /tmp (you can edit /etc/fstab to make this permanent)
Increase memory in OrbStack settings

Please share the output of df -h to confirm this.

Indeed, your tmpfs is full:
tmpfs 2.6G 2.6G 0 100% /tmp

This isn't really an OrbStack issue — any of the solutions I suggested above should fix the problem. I could consider increasing the default size % for new machines though.

Default tmpfs size has been increased for the next version, but I'd still recommend increasing the overall memory limit for your use case.


Preview infinitely expands circular ![[]] reference.

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[X] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
Circular ![[]] reference causes an infinite nesting expansion in the preview.
To Reproduce
note1.md
hi i am note  1
![[note2]]

note2.md
hi i am note 2
![[note1]]

Show the preview view while being in note1
Expected behavior
The preview detects the that there is a circular reference and breaks the circle by rendering the offending expanded note as plain link
note1 would render as if it had the following content:
hi i am note  1
[[note2]]

Thanks for the report nickolay!


FAB: list not getting active class

Short description of the problem:
My <ion-fab-list> does not get the .fab-list-active class as it's supposed to when the FAB is clicked. When I interact with another component, the class suddenly IS added to the FAB and the list is shown. Then when I close the FAB the class is not removed from the FAB until I interact with another component.
I'm using ChangeDetectionStrategy.OnPush in the page with the FAB, but setting it to Default does not fix this.
The list is working in the demos, so I'm not sure why it isn't working for me.
Another small problem I noticed while looking at the demos is that the list buttons do not animate out because the .fab-list-active class is removed, making them invisible before they can animate out.
<ion-header>
  <ion-navbar>
    <ion-title>FAB list</ion-title>
  </ion-navbar>
</ion-header>

<ion-content>

  <ion-fab bottom right>
    <button ion-fab>
      <ion-icon name="add"></ion-icon>
    </button>
    <ion-fab-list side="top">
      <button ion-fab>
        <ion-icon name="chatboxes"></ion-icon>
      </button>
      <button ion-fab>
        <ion-icon name="calendar"></ion-icon>
      </button>
    </ion-fab-list>
  </ion-fab>

</ion-content>

Which Ionic Version?
2.0.0-rc.0
Plunker that shows an example of your issue
I can't find a RC.0 compatible plunkr.
Run ionic info from terminal/cmd prompt:
Cordova CLI: 6.3.1
Gulp version:  CLI version 3.9.1
Gulp local:
Ionic Framework Version: 2.0.0-rc.0
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.0.0-beta.20
ios-deploy version: 1.9.0
ios-sim version: 5.0.8
OS: Mac OS X El Capitan
Node Version: v6.7.0
Xcode version: Xcode 8.0 Build version 8A218a

The posted markup is EXACTLY the same you are using?

@manucorporat Yep, this markup exactly and nothing else on the page! Here's a nice gif to go with it ;). I only added the text input to demo it.

As you can see the class is only added and removed after I interact with the text input. So somehow change detection fails here.

I have a possible fix, but I would like to reproduce the bug first (I can't).


[Dev] Add rerun example

Description
[Dev] Add rerun example
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Unit Tests
Manual Tests

Test Configuration:

Firmware version:
Hardware:
Toolchain:
SDK:

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

bazel build //experimental/rerun_example:rerun_example
DEBUG: /home/qcraft/.cache/bazel/_bazel_qcraft/e1d25c724e0056022d92d7bdfe62b271/external/rules_python/python/repositories.bzl:32:10: py_repositories is a no-op and is deprecated. You can remove this from your WORKSPACE file
INFO: Analyzed target //experimental/rerun_example:rerun_example (2 packages loaded, 749 targets configured).
INFO: Found 1 target...
ERROR: /home/qcraft/.cache/bazel/_bazel_qcraft/e1d25c724e0056022d92d7bdfe62b271/external/arrow/BUILD.bazel:35:11: Compiling cpp/src/arrow/memory_pool_jemalloc.cc failed: (Exit 1): gcc failed: error executing command (from target @arrow//:arrow) /usr/bin/gcc -U_FORTIFY_SOURCE -fstack-protector -Wall -Wunused-but-set-parameter -Wno-free-nonheap-object -fno-omit-frame-pointer '-std=c++0x' -MD -MF ... (remaining 207 arguments skipped)

Use --sandbox_debug to see verbose messages from the sandbox and retain the sandbox build root for debugging
external/arrow/cpp/src/arrow/memory_pool_jemalloc.cc:31:10: fatal error: jemalloc_ep/dist/include/jemalloc/jemalloc.h: No such file or directory
   31 | #include "jemalloc_ep/dist/include/jemalloc/jemalloc.h"
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
Target //experimental/rerun_example:rerun_example failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 15.386s, Critical Path: 12.44s
INFO: 45 processes: 19 internal, 26 linux-sandbox.
FAILED: Build did NOT complete successfully


Crashes with use of the 3.65 enzo

It was working on 3.60 enzo and recently updated. Just wondering if anyone figured out a fix for this or patiently wait for the update :)

Try it TheFloW version: https://github.com/TheOfficialFloW/gamecard-microsd/releases

gamecard-microsd have nothing to do with this project.
This one allows to dump gamecards and run gamecards dumps, gamecard-microsd allows to use a gamecard-to-sd adapter as a storage

Thank you, just wanted to make sure I covered everything.


Add Profile Picture

This PR is for an upload Profile Picture as per #116
Image manipulation is through a new composer package "intervention/image" and is handled in an event.
As per #151 there is also a hires version of the image that is accessible by clicking the profile picture on the public profile view.
Sorry for stuffing around on the previous pull request. (I've never really done this before :) )

Thanks for this! I've added some notes inline. Two big issues that didn't come across in the notes:

This breaks people who currently are relying on Gravatar. A good solution would provide Gravatar as a fallback.
The original plan was to have multiple profile pictures (https://github.com/tightenco/symposium/issues/2) but unfortunately that info wasn't moved to the later (https://github.com/tightenco/symposium/issues/116). I'm thinking we can just allow for a single picture for now and address adding more profile pictures later.

Thanks!

Gravatar is now a fallback for the nav icon and the public profile page. If a user doesn't have a profile picture uploaded, an alert will show explaining that Gravatar is being used and they should uploaded their own image.
Now that Storage is being used instead of File, symlinks will need to be setup from /storage/app/public to /public/storage if you are going to keep local storage
Sorry if this conflicts with my last PR. I only discovered branching after starting this feature. :)

@michaelrtm Thank you!! :)


broken link to download NEN7510

[Enter feedback here]
link to download file is broken -
https://protection.office.com/DownloadFile/ServiceAssurance/Document/compliance/Azure and Office 365 NEN7510-2011 Standard Coverage/pdf

please check.

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

ID: b0436bb6-1b01-68d5-163e-6f838d47c0cb
Version Independent ID: 33e1241e-d444-d51a-dc97-55dec81a3ce2
Content: NEN 7510 - Microsoft Compliance
Content Source: microsoft-365/compliance/offering-NEN-7510-Netherlands.md
Product: microsoft-365-enterprise
GitHub Login: @robmazz
Microsoft Alias: robmazz

@hora7ce - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/microsoft-365-docs/commit/7b8b987311a1f1095d62e15c5500524bc1745510.
Thank you for your contribution to make the docs better! Much appreciated!


con.csv causing windows error

The con filename is somehow not valid on my Windows 10 system. I cannot clone the library, exctract the downloaded zip repo, because it drops a windows error on the con.csv file.
When I would like to create a file anywhere on my disk as con.csv or con.txt I got the following error:
The specified device name is invalid.
See this:
https://superuser.com/questions/86999/why-cant-i-name-a-folder-or-file-con-in-windows
Suggestion:
use different file name

we renamed all the xxx.csv files to g-xxx.csv -- hopefully that fixes this problem.
https://github.com/git-plm/parts/pull/31


docker-machine create fails during Virtualbox version precheck

Windows 7 with Docker Toolbox (18.0.6) and Virtualbox (5.2.16 r123759) fails with the following:
$ docker-machine --debug create default
Docker Machine Version:  0.15.0, build b48dc28d
Found binary path at C:\Program Files\Docker Toolbox\docker-machine.exe
Launching plugin server for driver virtualbox
Plugin server listening at address <IP_ADDRESS>:49567
() Calling .GetVersion
Using API Version  1
() Calling .SetConfigRaw
() Calling .GetMachineName
(flag-lookup) Calling .GetMachineName
(flag-lookup) Calling .DriverName
(flag-lookup) Calling .GetCreateFlags
Found binary path at C:\Program Files\Docker Toolbox\docker-machine.exe
Launching plugin server for driver virtualbox
Plugin server listening at address <IP_ADDRESS>:49569
() Calling .GetVersion
Using API Version  1
() Calling .SetConfigRaw
() Calling .GetMachineName
(default) Calling .GetMachineName
(default) Calling .DriverName
(default) Calling .GetCreateFlags
(default) Calling .SetConfigFromFlags
Reading certificate data from C:\Users\IBM_ADMIN.docker\machine\certs\ca.pem
Decoding PEM data...
Parsing certificate...
Reading certificate data from C:\Users\IBM_ADMIN.docker\machine\certs\cert.pem
Decoding PEM data...
Parsing certificate...
Running pre-create checks...
(default) Calling .PreCreateCheck
(default) DBG | COMMAND: C:\Program Files\Oracle\VirtualBox\VBoxManage.exe --version
(default) DBG | STDOUT:
(default) DBG | {
(default) DBG | }
(default) DBG | STDERR:
(default) DBG | {
(default) DBG | }
Error with pre-create check: "We support Virtualbox starting with version 5. Your VirtualBox install is "". Please upgrade at https://www.virtualbox.org"
open C:\Users\IBM_ADMIN.docker\machine\machines\default\default\Logs\VBox.log: The system cannot find the path specified.
notifying bugsnag: [Error with pre-create check: "We support Virtualbox starting with version 5. Your VirtualBox install is "". Please upgrade at https://www.virtualbox.org"]
I've tried this in different cmd line tools and all fail the same way- cmd, powershell, git bash, cygwin
This seems to be happening because docker-machine does a version check of the Virtualbox driver by invoking VBoxManage.exe --version. As you can see in the debug STDOUT, no version information is captured, so docker-machine treats this as an error and exits.
I've manually run this version command and the output was non-empty:
C:\Program Files\Oracle\VirtualBox>VBoxManage.exe --version
5.2.16r123759
I believe this issue started to surface first when I upgraded Virtualbox to the latest version. I am not sure which version it was on before.

Reported in https://github.com/docker/machine/issues/4548

Workaround reported here: https://github.com/docker/machine/issues/4548#issuecomment-413276826


Letting a 2nd player initiate certain commands with an external controller for accessibility/difficulty reasons

This is a bit of an odd request, so I'll provide context further below.
Request
Allow a 2nd player outside of the headset to execute certain commands with an external controller or keyboard, so that the actual player inside the headset only needs to look and point.
It does not need to be particularly robust. Just reenabling the buttons and thumbsticks on the controller.
Would this be possible at all, or does the nature of how controller commands are passed to the game not really allow this to happen?
Context
I love introducing new players to Outer Wilds. But for non-gamers, low-skilled players, and especially physically handicapped players, one of the highest barriers of entry for Outer Wilds is the ship controls and the platforming sections.
My solution to this has been to play it like a co-op game, where I’m the mostly-silent “driver” who’s responsible for controlling the character and playing through the mechanical challenges, while the other player is a “passenger” who makes all the important decisions, like where to go, what to do, and how to solve all the puzzles. This works insanely well and has made it super easy for me to get people into Outer Wilds, even if they’re not an experienced gamer. (It also lets me re-experience the game vicariously through them, since their level of knowledge basically serves as my progression gate that keeps me from skipping puzzles, haha)
The opportunity to introduce people to this game in VR is honestly really exciting to me. But unfortunately, because of the nature of VR controllers, it’s difficult to control the character from outside of the headset. Mainly due to perspective reasons.
This is particularly challenging in the case of a friend of mine who is physically handicapped. They can hold the controllers, point, and look around. They also happen to have excellent VR legs, so they can deal with the lateral movement without getting sick. But the nature of their handicap means they can't engage 90% of the buttons or thumbsticks (with the exception of the grip commands). So they can't move their character, they can't control the ship, they can't jump, etc.
Having the option to initiate some of these commands from my couch with an Xbox controller while they're in VR space would go a LONG way towards letting them be able to fully experience this game in VR.

Very interesting request. NomaiVR has a big issue with this: it disables the native Unity inputs, meaning keyboards and game controllers are completely disabled. While what you request may be possible to achieve, it would be a pretty big task, not something that can be easily done in a couple hours. So unless another modder makes a NomaiVR addon for this, I doubt it will ever happen.
My suggestion is that you look for a solution that works on any SteamVR game. I'm not sure if there is such a thing. The OpenVR Input Emulator has a different goal, but maybe what you want can be achieved with it. The benefit here is that, if you manage to get it to emulate VR inputs via external keyboard/controller inputs, you'd be able to do this for any game, not just Outer Wilds.

Understood. Thanks for the detailed reply. I figured it was something like that, haha.
I will try and figure out if I can use OpenVR to hook into the game. Thank you!

Unfortunately it seems that the OpenVR Input Emulator is no longer actively being developed, and crashes when I attempt to remap controls :(


Take the shortest route when moving the hand

Steps to reproduce:
Open up the time picker, select, say 5 minutes. Then select like 55 minutes or something, the minute hand goes all the way around the clock along the clockwise route when it should be just a small hop away by going anti-clockwise.
Taking the short route would look much more subtle and pleasing from a UX standpoint, right now you can't even feel the ease-in-out interpolation, it's very in-your-face and kinda distracting and not very material-y 😄

You are absolutely right and I'm really happy that someone other than me noticed this subtle glitch. :tada:
I wasn't sure how to do this so I didn't and forgot about it. I'll look into this next weekend.

@AdityaAnand1 This is now implemented in v0.1.4 :+1:


Support for message level security

As expressed in issues #1464 and #1881 there is a need for describing signed and encrypted requests and response using OpenAPI. I would like to propose the following mechanisms to support message level security in OAS.
This proposal comes after some preliminary works summarized in the Google doc at https://docs.google.com/document/d/13THkz7867blJ464ohpz9MUrhMQp-28Cx77JvDHlyurc from OAI Google Drive (I can give access to this doc to who needs it)
The problem with the proposal in this document is that it was too complex and did not follow the Open API philosophy, so I tried here to put in place something more standard, mimicking the Security Scheme / Security Requirement mechanism people are used to.
Let's take an example that shows how you could describe required requests signature and provided responses signature using JWT tokens (as you can see it uses the key description mechanism proposed by @whitlockjc in issue #1881) :
# ...
components:

    keys:
        # Keys taken from Google's OIDC Discover Document (https://accounts.google.com/.well-known/openid-configuration)
        google-oauth-v3-1:
            description: "JSON Web Key used by Google for signing JWTs: https://www.googleapis.com/oauth2/v3/certs#/keys/0"
            type: JWK
            metadata:
                kid: 0905d6f9cd9b0f1f852e8b207e8f673abca4bf75
                e: AQAB
                kty: RSA
                alg: RS256
                n: yyeEmeK35F8P54ozfpsF79n59ZsOrcZdxQWsxrzm0qjdA5r_b-be-cQnWAw_2AoGdeWHX-Cz7uPFDMdEwzLGlpv3SELi34h8PkzjyO7xlbhsNs-ICnqUyUTA7CovKtpJ47PjiQnXcaRNCFUQbli8VlEqbVLuqFjC98igICpNYR-iiVIm0VCFtkq0p8vf1yQ493Pnx2Bm8fUx6SkeJ7wKPWQq_K4e6ZH40JWLk6c1U9W5qPKeckevdNLrdZY5lsTZ5zrRvuRBoIeZfp9bKSZGMtEja4xSCDKLrkcpb4qf6Ywx9rsZ4b8eHSLpVvUzNsj3GS7qK5flHzoccovhPVBbbQ
                use: sig

    messageSecurityOperations:
        signedJWTrequestOperation:
            securityType: signedJWTRequest
                alg:
                    - PS256
                    - PS512
                pubKeyRef:
                    - jku+kid
                mandatoryStandardClaims:
                    - iss
                    - aud
                    - iat
                    - exp
                    - jti

        signedJwtResponseOperation:
            securityType: signedJWTResponse
                pubKey:
                    $ref: '#/components/keys/google-oauth-v3-1'

    messageSecuritySchemes:
        highSensitivityScheme-PostPutPatch:
            requestSecurity:
                $ref: '#/components/messageSecurityOperations/signedJWTrequestOperation'
            responseSecurity:
                200:
                    $ref: '#/components/messageSecurityOperations/signedJwtResponseOperation'
# ...

As you can see, there are two objects added in the components top level object:

messageSecurityOperations, which is a Map of messageSecurityOperations describing the elementary signature and encryption operations that are required on API requests or performed as API responses
messageSecuritySchemes which is a Map of messageSecuritySchemes, each security scheme stating which messageSecurityOperation must be applied to the request before consuming the API and which messageSecurityOperation will be applied to the operation's responses, based on the response HTTP status code (all of these referencing the messageSecurityOperations declared in the messageSecurityOperations object

Once these objects declared in the components object, it becomes possible to reference a messageSecurityScheme directly from an Operation, just like this:
# ...
paths:
    /pets:
        post:
            description: Creates a new pet in the store.  Duplicates are allowed
            operationId: addPet
            messageSecurity: highSensitiveProfile-PostPutPatch
            requestBody:
                description: Pet to add to the store
                required: true
                content:
                    application/jwt:
                        schema:
                            $ref: '#/components/schemas/NewPet'
            responses:
                '200':
                    description: pet response
                    content:
                        application/jwt:
                            schema:
                                $ref: '#/components/schemas/Pet'
                default:
                    description: unexpected error
                    content:
                        application/json:
                            schema:
                                $ref: '#/components/schemas/Error'
# ...

where the messageSecurity: highSensitivityScheme-PostPutPatch field of the post operation object specifies which message security scheme to apply to the operation (on requests and responses).
There are a few message level security operations that could be described this way:

Request Security Schemes

JWSRequest (standard JWS compact signature)
JWSDetachedRequest (request body unchanged, signature passed in a HTTP header)
SignedJWTRequest (signed JWT)
HTTPSigRequest (HTTPSignature standard description)
EncryptedJWTRequest (encrypted JWT)
JWERequest (standard compact JWE encrypted message)

Response Security Schemes

JWSResponse
JWSDetachedResponse
SignedJWTResponse
HTTPSigResponse
EncryptedJWTResponse
JWEResponse

If you feel comfortable with the presented structure of message level security description I can create separate issues for each of these message level security operations, specifying mandatory and optional attributes that could describe these operations.
Happy to discuss!

It is correct that https://datatracker.ietf.org/doc/draft-cavage-http-signatures is the only of the described schemes supporting REST?  UK's Open Banking signature scheme does not include URI and Method although they are fundamental parts of a REST request.

HI Anders,
Yes at the moment it is the only scheme supporting URI/Method/HTTP headers signatrure available (while still in draft).
There was a proposal from Richer, Bradley and Tschofenig from the Oauth WG at IETF about that (https://tools.ietf.org/html/draft-ietf-oauth-signed-http-request-03) but it's been expired for 3 years now... Seems that IETF guys did not like the needed canonicalization of the HTP request :-)
As you know I'm thinking of proposing to the OAuth WG something simpler, based on the fact that you add Method + URI + a selection of HTTP headers in the JOSE header and then use the detached unencoded payload mechanism. Just running after time to write this down, as always :-(
Thanks,
Philippe

Hi Philippe,
You will anyway need to canonicalize the URI and the HTTP headers.  Here is my take on the latter:
https://tools.ietf.org/html/draft-rundgren-signed-http-requests-00#section-6.8
BTW, canonicalization of the JSON body is not dead, it is just resting a bit :)
It works like a charm on 5 platforms so the team behind it is currently trying to establish an "informational" RFC.  This is not a "full standard" but I think it suffices for getting it off ground.

Mmmm no actually and that's the beauty of it :-)
As the info from the HTTP message you want to integrity protect is written in the JOSE header to some extent the canonicalization process lays in how you store the information in the JOSE header.
So when you verify the sigature of the JOSE header you ensure that all this info was not modified duting transit.
Then all you've got to do is verifying that this info semantically matches the received HTTP message!
This has to be refined, a proper definition of what "semantically matches" means should be provided, but this should work and would be so easier, IMHO

This has to be refined, a proper definition of what "semantically matches" means should be provided, but this should work and would be so easier, IMHO

This is the URI and Header canonicalization I referred to.  The data is created by the sender but must be compared by the receiver using "live" HTTP data from the server.  The algorithm I showed is essentially the same as HTTP Signatures uses.  I believe this part was the reason the previous attempt you mentioned didn't got through.  Some people claim this is impossible since proxies modify data.  Since HTTP Signatures and a similar solution from Amazon actually is heavily used, this is obviously exaggerated.  Or to be more correct: Requiring that everything must work on any platform without adjustments is counter productive.  Android Apps typically does not run on all versions of Android either.
The difference between detached JSON and canonicalized JSON is another discussion.  They both have downsides.  Detached JSON is a loss in the proxy world and doesn't lend itself to serialization.

Semantic match is a different beast than byte to byte equality after transformation required by canonicalization processes. Set-Cookie addition by reverse proxies is an example of problem with canonicalization (and BTW as Set-Cookie expires is an unquoted RFC 822 date containing a comma, concatenating Set-Cookie values with a comma can be a problem: this is why you generate a Set-Cookie header by cookie) that can be easily overcome by semantic matching.
Anyway, this discussion about HTTP Signature is not totally in the scope of this issue :-)
I'll send you a PM about this.

I think I understand what you are meaning now.

You sign HTTP header data as is but in a Base64Url-safe way
After verifying the signature you try to match the signed data with the actual potentially somewhat changed data

I would be a bit cautious about trying to standardize such a scheme.  If the data is HTTP-compliant it should be possible to canonicalize the data the way HTTP Signatures, Amazon and I suggest.  If it is not HTTP-compliant, I wouldn't bother.  If an intermediary adds a Set-Cookie I don't see how that could work with any scheme unless you treat each cookie as a separate item.  Maybe that is a better approach; consider all concatenated headers as separate elements during canonicalization?

Hi all, is there any conclusion from the meeting?
I think that encryption keys management should be treated as a separate service, like user session. But instead of Authorization headers, where you can specify a global token, might be nice to select or send a keyID within the request. This assumes that the dynamic specification of the key can switch to a different security level. If no key provided then no encryption is done.
Another important thing for me is how to specify which fields are encrypted and which are "public". So every attribute that contains sensitive information has to be included on the encrypted part, and the rest will be raw to perform searches with no key. A really cool thing might be to include in the ui-docs the payload transformation process (encrypt/decrypt) by default, so you still have inputs where to put raw values and decrypt the response to show again raw values.
I'm not really familiar to JOSE, but hope to provide another point of view, thank you!


Return types not included in documentation

At one point there were examples showing what was returned and what the different enums mean, but I can't seem to find where those exist anymore. I would expect the doc page for something like az repos pr list to show me the object that gets returned so I can understand what different things mean, but no matter where I look, I get redirected back to the page that just shows me the list of options for the command.
What happened to the example responses? Can those be linked in the documentation somewhere?

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

ID: 1e48ddaa-64b8-e414-7ea9-2746827fcc5f
Version Independent ID: 952f1742-6d3b-a177-a2e0-164b563c1771
Content: az repos pr
Content Source: latest/docs-ref-autogen/repos/pr.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

route to CXP team

@jonest Thank you for reaching out, we are looking into it.

@jonest you should be able to get sample responses here https://docs.microsoft.com/en-us/rest/api/azure/devops/git/pull-requests/get-pull-requests?view=azure-devops-rest-6.0#response.
Also when you said enum, did you mean the enum of status parameter? --status {abandoned, active, all, completed}

@jonest you should be able to get sample responses here https://docs.microsoft.com/en-us/rest/api/azure/devops/git/pull-requests/get-pull-requests?view=azure-devops-rest-6.0#response.

Thanks! I completely forgot about looking at the REST response docs. Is that something that you plan to include in the CLI docs, too? Some sort of "see also" or something :)

Also when you said enum, did you mean the enum of status parameter? --status {abandoned, active, all, completed}

Yes, that's what I was referring to. The response was just returning the integer, but I didn't know which integer corresponded to which status

@jonest Thank you for the confirmation, to be honest its a lot of information for the cli documentation and we do not support links in cli help yet. but we will definitely consider this for future releases.
Let me know if you have any follow up questions, before we close this issue


OMP: Error #15

While attempting to run it im getting error:
OMP: Error #15: Initializing libiomp5md.dll, but found libiomp5md.dll already initialized. OMP: Hint This means that multiple copies of the OpenMP runtime have been linked into the program. That is dangerous, since it can degrade performance or cause incorrect results. The best thing to do is to ensure that only a single OpenMP runtime is linked into the process, e.g. by avoiding static linking of the OpenMP runtime in any library. As an unsafe, unsupported, undocumented workaround you can set the environment variable KMP_DUPLICATE_LIB_OK=TRUE to allow the program to continue to execute, but that may cause crashes or silently produce incorrect results. For more information, please see http://www.intel.com/software/products/support/.
i have no idead what libiomp5md.dll or OpenMP, failed to get any data that i actully can understand..
my run command is:
whisper-ctranslate2 "sample.mp3" --language he --output_format srt --output_dir "./audio/srv/" --verbose False --beam_size 5 --word_timestamps True --max_line_width 1 --max_line_count 1 --vad_filter True --vad_max_speech_duration_s 1 --vad_threshold 0.1 --suppress_blank False
sample.mp3 is a 365kb file - i used previously with whisper-ctranslate2, no problem with that - its also happning with anyother file.
i have tried to uninstall/install - didnt help.
i did tried to install insanely-fast-whisper, but i didnt liked it so i deleted it..
any help will do!

Hi, did you manage to fix the issue? I have the exact same issue going on my end.

Same issue

Same issue for me.    Seems totally unresolvable even after making an entirely new Conda install.

Search for libiomp5md.dll in a File Explorer and try to delete folders, where it is located.
I solved this issue taking this approach.
However, it was a week ago and i already forgot all the little details, which might have been.
But the idea is if it is already initialized somewhere, need to find its location and remove it, so there is no more a source of its ghost initialization.


Roll engine b36068c3778c..cad97fe7467c (2 commits)

https://github.com/flutter/engine/compare/b36068c3778c...cad97fe7467c
git log b36068c3778c014ac999ce03412af2690bc713bc..cad97fe7467c60de645d47acffd0e00cd83178ab --no-merges --oneline
cad97fe74 Update overlays_gr_context_ correctly (flutter/engine#8175)
c0690e6ec Roll src/third_party/skia aefecad7c0d2..69600007e278 (2 commits) (flutter/engine#8177)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


Using java 1.8 [5705]

This PR:

Updates submodules related with fastrtpsgen
Sets compatibility with java 1.8
Fixes warnings

Build status:

Linux
Mac
Windows


Create a dictionary: a TTF file

It would be nice to have a TTF file to download and use it as a dictionary. One use case could be to use the number emojis (0⃣, 1⃣, 2⃣, 3⃣, 4⃣, 5⃣, 6⃣, 7⃣, 8⃣, 9⃣) for displaying a game's score or a roulette, a level, etc. Or simply to support this great work you've done in Python texts.

1

+1

the emoji is sooooo cool, I’m sure i’ll let all my friends use it if anyone got a ttf or something can easily use on mac


Remove qiskit-ibm-provider as an example in README.md

Summary
With https://github.com/Qiskit/qiskit-ibm-runtime/pull/1285, the use of qiskit-ibm-provider is discouraged.
Details and comments
Includes some links pointing to qiskit-terra repo

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
10 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.002%) to 89.308%

Files with Coverage Reduction
New Missed Lines
%

crates/qasm2/src/lex.rs
4
91.69%

crates/qasm2/src/parse.rs
6
97.62%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
59796

Relevant Lines:
66955

💛  - Coveralls


Unable to test the warping module

I am getting issue with  cupy module and it's not part of the conda env yaml file
when running sh test_VITON.sh i am getting this

Also my gpu and cuda version is

Let me know how you got it working for youself . The readme seems to be missing this bit

pip install cupy-cuda12x in your case

@dx777  getting this issue
ERROR: Could not find a version that satisfies the requirement cupy-cuda121 (from versions: none) ERROR: No matching distribution found for cupy-cuda121

@RishiGitH  the packet is named cupy-cuda12x not cuda121

@dx777  but then i get this issue if i install cuda 12. I am having a lot of troubles with PF-AFN any idea what could be the issue
Traceback (most recent call last):   File "eval_PBAFN_viton.py", line 11, in <module>     from models.afwm import AFWM   File "/home/ubuntu/PF-AFN/PF-AFN_test/models/afwm.py", line 4, in <module>     from .correlation import correlation   File "/home/ubuntu/PF-AFN/PF-AFN_test/models/correlation/correlation.py", line 274, in <module>     @cupy.util.memoize(for_each_device=True)   File "/home/ubuntu/.local/lib/python3.8/site-packages/cupy/__init__.py", line 921, in __getattr__     raise AttributeError(f"module 'cupy' has no attribute {name!r}") AttributeError: module 'cupy' has no attribute 'util'

@dx777  Thanks got this resolved but now I am getting this issue . Is it due to image size ?
RuntimeError: Caught RuntimeError in DataLoader worker process 0. Original Traceback (most recent call last):   File "/opt/conda/envs/test/lib/python3.8/site-packages/torch/utils/data/_utils/worker.py", line 308, in _worker_loop     data = fetcher.fetch(index)   File "/opt/conda/envs/test/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 51, in fetch     data = [self.dataset[idx] for idx in possibly_batched_index]   File "/opt/conda/envs/test/lib/python3.8/site-packages/torch/utils/data/_utils/fetch.py", line 51, in <listcomp>     data = [self.dataset[idx] for idx in possibly_batched_index]   File "/home/ubuntu/PF-AFN/PF-AFN_test/data/cp_dataset.py", line 172, in __getitem__     parse_map = parse_map.scatter_(0, parse, 1.0) RuntimeError: Expected index [1, 576, 384] to be smaller than self [20, 512, 384] apart from dimension 0

@Limbor  do you have any idea ? ^^

change
@cupy.util.memoize
to
@cupy.memoize
might be helpful

I am getting the same issue. What is the solution for this cupy problem?


Sbase as a candidate for the missing coreutils.

First of all, I am sorry if it bother you that I make so many useless suggestions.
 I really recommend them. SRCs here:
https://core.suckless.org/sbase/ - repo: https://git.suckless.org/sbase/
https://core.suckless.org/ubase/ - repo: https://git.suckless.org/ubase/
Tell me what you think of them.                  :)

No worries, your suggestions have actually been pretty interesting for me to look into.
I was aware of sbase and ubase, but I'll look more into them as a replacement for the utils we are missing. ubase in particular provides quite a few utilities which are not easily ported from *BSD.
Feel free to suggest anything else as it comes up :)


User should be able to turn off/on the web server in the configurations

Is your enhancement related to a problem? Please describe
Since https://github.com/containers/podman-desktop-extension-ai-lab/pull/1544 we by default start a REST server when AI Lab is started
Describe the solution you'd like
Having an option to disable it in the configuration. And IMO have it disabled by default.
Describe alternatives you've considered
No response
Additional context
cc @feloy @benoitf @jeffmaury

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."

it should also require also some authentication else we'll start to receive some CVE

What do you mean, you would like us to generate some token that people will have to include in their authorization request ?

it should also require also some authentication else we'll start to receive some CVE

What do you mean, you would like us to generate some token that people will have to include in their authorization request >?

everytime that there is a server listening without an authentication, then people will do for example a request in a webpage calling something to the server to do things on behalf of the user (but without its agreement) so at the end, I saw security requests


Potential u-turns at junctions with inaccessible roads

When e.g. a footpath crosses a road there will be a tower-node for this junction. If u-turn costs are set to a finite value this means that the shortest path can include a u-turn at this junction, even if its not really a junction for the current vehicle (e.g. car).

Avoiding such u-turns would most likely be an improvement, but there can still be u-turns at 'real' junctions that aren't desired either.

Another (rather severe) case where this could happen are ferries that are crossing bridges (c.f. #2139).


Nested projections do not return @rid and @class, when excluding fields

OrientDB Version: 3.0.x
Java Version: 1.8.x
Hi,
Consider the following query select @this:{!a, !b} from Temp
Expected behavior

Actual behavior

Also I cannot use expand on the result anymore. Queries like
select expand(@this:{!a, !b}) from Temp
select !a, !b from Temp
select expand(@this.exclude('a', 'b')) from Temp

either throw an error or return empty result. What would be a correct way to return a proper record?
Steps to reproduce
create class Temp

insert into Temp content {"a": "1", "b": "2", "c": "3"}
insert into Temp content {"a": "1", "b": "2", "c": "3"}
insert into Temp content {"a": "1", "b": "2", "c": "3"}

select @this:{!a, !b} from Temp

Found solution: select *:{!a, !b} from Temp


JUnit Test Report job does not appear in the workflow run

Hi, we have a strange behavior that started lately - when there are failed tests,  JUnit Test Report job does not appear in the workflow run:

We do get the summary, but no not job where we can see the details and the logs

The way we expect it to work is this (it used to work like this, nothing is changed in our configuration):

the configuration is simple:
- name: Publish Test Report
          uses: mikepenz/action-junit-report@v4
          if: success() || failure() # always run even if the previous step fails
          with:
            report_paths: 'report.xml'
            fail_on_failure: true
            summary: true
            detailed_summary: true

Good day.
This is a strange one. Does it maybe get added to a different check?
Nothing indicates that there was a change in the action, as the respective API is still called here: https://github.com/mikepenz/action-junit-report/blob/v4.3.1/src/annotator.ts#L99-L100
Could be some behavior change from GitHub's side 🤔

I also tried to upgrade to v5, same result :(
What do you mean by "Does it maybe get added to a different check?" ?
Any suggestions? additional configuration?...

v5 only contains improvement to test suite handling and to the formatting of the summary.
For the different check I actually used the wrong wording. There's a long standing GitHub issue where reprots might be sent to the wrong build: https://github.com/mikepenz/action-junit-report/issues/40
So I was wondering if that possibly happens in your case

Or a different thought (which would be weird that it does not fail doing the API request) - does the CI have all permissions required?
https://github.com/mikepenz/action-junit-report?tab=readme-ov-file#pr-run-permissions

Or a different thought (which would be weird that it does not fail doing the API request) - does the CI have all permissions required?

Since it used to work (until about a week ago) I guess yes...

For the different check I actually used the wrong wording. There's a long standing GitHub issue where reprots might be sent to the wrong build: https://github.com/mikepenz/action-junit-report/issues/40

We also have runs on PR/main and some schedule runs as well, but in our case, the results are not going anywhere :(
I checked all runs for the same commit (usually it's only one), no results, only the summary as I mentioned in the description..

Running the tests on the action itself does still create the check. I am out of ideas, but the most likely would be permission issue, or something similar.

I found now that the test report indeed appears in the wrong workflow run - we have build-test and another validation workflow that both run for PR, and tests repo appears not wit the tests, just like explained in this ticket https://github.com/mikepenz/action-junit-report/issues/40
Any updates on this one?

As far as I am aware there was no new update related to the issue from the GitHub side. Unless I missed some new API or some update in the meantime.
Closing this as #40 will be the correct place to have further updates.


Rename viewInjector to providers in the developer guide

visit angular2 dev guides - displaying data and search appInjector, this need be changed to providers (angular-2.0.0.alpha.44).

long gone


chore: add tests for the sourcing package

Fixes #587
Changes proposed in this pull request:

added tests for the local file manager
added dummy test for sourcing manager

The tests for the local file manager are only concentrated around the parts where we have custom logic. We use third party libraries for reading files, and we don't want to test those.
The sourcing manager only got one dummy test to indicate that we know the actual tests are missing and that is a deliberate choice. Reading and checking for files is done using the file manager, and the actual import of a module is not tested in the context of a unit test.
For both implementations it is pretty obvious if things are not working. Then nothing works.
@MaskingTechnology/jitar

As discussed, we see mocking external dependencies as a red flag that indicates the need for more separation of concerns. Therefore we decided to redesign the implementation to separate the file system from the file management.


Export the RASTtk annotations to Anvio

We want to be able to load the RASTTk annotations to anvio to compare the annotations used in the model to what Michelle got from the pangenome.

Here's an example of that file:

I looked at using the model object directly:

But the gene ID isn't tied to any actual sequence data:

Even in the full model file, the the genes aren't tied to any sequence information:

I started doing a deep dive on the KBase code for building the models to figure out how KBase is going from the RAST annotation to the ModelSEED IDs, and I think that is best tackled in a separate issue.


💖 looking for molecule podman driver maintainers

I am unable to actively maintain this driver as much as I would like, so I am looking for to add new maintainers.
I can keep providing support for CI and integration with molecule itself but less around driver specifics.

Still looking...


Unity 2 levels fail to extract.

I'm not sure if this project plans to support Unity 2 extraction but i have 2 cases where the Unity level file fails to extract. Both from the same .unity3d file. I'll upload the files to Mediafire for you to check out.
http://www.mediafire.com/file/7h6rbfocarrbqc0/toonTropolis14.unity3d_unpacked.zip/

This project should support even Unity 1 but I've never tested it :)
There was several bugs and I fixed them in c705f5fbeedbcad612396ebe405a0de1ef70f10e. However, your archive is corrupted so I can't check sharedassets2.assets file. If last version still has errors, please reupload archive.

Oh my archive corrupted? That's the worst luck I've had in a while. I'll re-upload it if the problem still persists in the latest commit.

Latest commits fixed the issue. Great to know. I'll close this now.


Audio Stops When Banner Hides - Unable to Play 30-Second Audio Continuously

I set a 30-second audio file for playback and was informed that it should play for the full duration. However, the audio stops prematurely when the banner is hidden.
Expected Behavior:
The 30-second audio should play in its entirety, regardless of the banner's visibility state.
Actual Behavior:
The audio stops as soon as the banner is hidden, even though the duration is set to 30 seconds.
Steps to Reproduce:
Configure a 30-second audio file for playback.
Start playing the audio.
Hide the banner during playback.
Observe that the audio stops prematurely.
Additional Question:
Is it impossible to achieve continuous 30-second playback in this scenario? If not, could you provide guidance on how to decouple audio playback from UI elements like banners?

Hi there 👋
Version information, platform this is experienced, example code etc?
https://stackoverflow.com/help/how-to-ask


unexpected async result

Describe the bug
I am new to chainlit, prepare the following test, expect async to be unblocking, but the timestamps show otherwise
To Reproduce
Here is the code
import chainlit as cl
from chainlit import make_async, run_sync
import time
from datetime import datetime

def ts():
    return datetime.now().strftime("%Y-%m-%d %H:%M:%S")

def wait_for(t:float):
    ts_1 = ts()
    time.sleep(t)
    ts_2 = ts()
    return f"\tSYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

async def await_for(t:float):
    ts_1 = ts()
    time.sleep(t)
    ts_2 = ts()
    return f"\tASYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

@cl.on_message
async def main(message: cl.Message):
    # Your custom logic goes here

    msg = f"\n[{ts()}] Starting '{message.content}' ..."

    sleeper_0 = wait_for(5)
    msg += f"\n[{ts()}] sleeper_0 (S): {sleeper_0}"

    sleeper_1 = await await_for(2)
    msg += f"\n[{ts()}] sleeper_1 (A): {sleeper_1}"

    sleeper_2 = await await_for(4)
    msg += f"\n[{ts()}] sleeper_2 (A): {sleeper_2}"

    # convert to sync
    sleeper_3 = run_sync(await_for(3))
    msg += f"\n[{ts()}] sleeper_3 (S): {sleeper_3}"

    # convert to async
    af = make_async(wait_for)
    sleeper_4 = await af(7)
    msg += f"\n[{ts()}] sleeper_4 (A): {sleeper_4}"

    msg += f"\n[{ts()}] Done processing !!!"

Expected behavior
starting timestamp for sleeper_2 should be the same as that for sleeper_1
Desktop (please complete the following information):

OS: [Ubuntu]
Browser [chrome]
Version [1.3.2]

@gongwork Hi, regardless of how Chainlit should behave, I think there are 2 parts of your code that are blocking as expected in Python:

You are awaiting the async function in main - if you really wanted them to sleep at the same time, you could use asyncio.gather() instead.
In your async function, you are using time.sleep() whereas I would recommend using asyncio.sleep() and awaiting it

With some minor modifications to your code, I get the expected behaviour (pardon me for omitting the chainlit context, I didn't see it relevant):
import time
from datetime import datetime
import asyncio

def ts():
    return datetime.now().strftime("%Y-%m-%d %H:%M:%S")

def wait_for(t:float):
    ts_1 = ts()
    time.sleep(t)
    ts_2 = ts()
    return f"\tSYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

async def await_for(t:float):
    ts_1 = ts()
    await asyncio.sleep(t)
    ts_2 = ts()
    return f"\tASYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

async def main():
    msg = f"\n[{ts()}] Starting 'chainlit' ..."
    sleeper_0 = wait_for(5)
    msg += f"\n[{ts()}] sleeper_0 (S): {sleeper_0}"
    sleeper_1, sleeper_2 = await asyncio.gather(await_for(2), await_for(4))
    msg += f"\n[{ts()}] sleeper_1 (A): {sleeper_1}"
    msg += f"\n[{ts()}] sleeper_2 (A): {sleeper_2}"
    print(msg)

@hadarsharon Thank you for showing me how to use asyncio

@hadarsharon I made change per your suggestion, now it works as expected in chainlit. Good learning exercise, Thank you!
import chainlit as cl
from chainlit import make_async, run_sync
import time
from datetime import datetime
import asyncio

def ts():
    return datetime.now().strftime("%Y-%m-%d %H:%M:%S")

def wait_for(t:float):
    ts_1 = ts()
    time.sleep(t)
    ts_2 = ts()
    return f"\tSYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

async def await_for(t:float):
    ts_1 = ts()
    await asyncio.sleep(t)
    ts_2 = ts()
    return f"\tASYNC slept {t} sec \t[{ts_1}] ==> [{ts_2}]"

@cl.on_message
async def main(message: cl.Message):
    # Your custom logic goes here

    msg = f"\n[{ts()}] Starting '{message.content}' ..."

    sleeper_0 = wait_for(3)
    msg += f"\n[{ts()}] sleeper_0 (S): {sleeper_0}"

    # convert to async
    af = make_async(wait_for)

    sleeper_1, sleeper_2, sleeper_4 = await asyncio.gather(await_for(2),await_for(4), af(7))
    msg += f"\n[{ts()}] sleeper_1 (A): {sleeper_1}"
    msg += f"\n[{ts()}] sleeper_2 (A): {sleeper_2}"
    msg += f"\n[{ts()}] sleeper_4 (A): {sleeper_4}"

    # convert to sync
    sleeper_3 = run_sync(await_for(2))
    msg += f"\n[{ts()}] sleeper_3 (S): {sleeper_3}"

    msg += f"\n[{ts()}] Done processing !!!"

    # Send a response back to the user
    await cl.Message(
        content=msg,
    ).send()


Rewrite notify.py

To enhance cross-platform compatibility and achieve seamless desktop notifications, we suggest transitioning from the current desktop_notifier to the plyer library. This refactoring ensures broader support.  This upgrade aims to simplify the notification setup for a smoother and more universally compatible user experience.

Can you assign me this issue, as it's look interesting as part of JWoC'24.
By the way, i need to replace the desktop_notifier module with the plyer library right and that's all.

@ayushmorbar 1 issue at a time

yeah, i understand that. please do review my pull request as mentioned so I can get on this one quick and submit a pr

Sure

Please, Assign me this issue now!

Can you assign me this issue, as it's look interesting as part of JWoC'24.
By the way, i need to replace the desktop_notifier module with the plyer library right and that's all.
import asyncio
from plyer import notification

async def notify(title, message, timeout=10):
    """
    Show a desktop notification to the user.

    Parameters:
    title: notification title - required
    message: notification message - required
    timeout: timeout in seconds to hide the notification. Default value is 10 - optional
    """
    notification.notify(
        title=title,
        message=message,
        timeout=timeout
    )
    await asyncio.sleep(timeout)  # Wait for the specified timeout

## test
await notify('test', 'this is a test notification')

Does this work with macos?

I haven't tried it myself yet on MacOS, but Plyer is designed to be platform-independent. According to the information provided by some user on Stack Overflow, it seems to work on MacOS 12.3.1.
To ensure proper functionality, make sure you have allowed notifications from `terminal-notifier` in your System Preferences. Navigate to `System Preferences -> Notifications & Focus -> terminal-notifier`, and ensure that notifications are allowed. If you encounter any issues, consider checking your overall notification settings on your Mac, as notifications might be disabled, or you might be in a "focus" mode, which could affect notification display. Adjusting these settings should enable the desktop notifications for the provided code.

Aside from this given the functionality of the notifier function, we can pass more parameters like the app icon, name and stuff.

Great then, You can push the code once you are done


Adding initial support for running interactive commands

I looked at feature request #751 and I it seems to me that this is a recurring topic. I would give it a try. This is a straightforward implementation of running interactive command in fzf:

we specify the command using -c param
query is fed directly to the command
the command is run every time the query changes (is rerun as-you-type)
the list is fed with the new result of the command (and any current selection is gone)
The speed is acceptable, even though we run a command multiple times.
I've put some more observations in #751.

Thanks on working on this!
It would be nice if you could make args compatible with those that skim uses. As maintainer of fzf.kak I try to keep plugin generic and because skim uses pretty much the same args as fzf it was easy to support both tools with the same code.

hey!
Actually they are the same: -c and alternatively --cmd, are exactly the same that skim uses. Or I missed something?

hey!
Actually they are the same: -c and alternatively --cmd, are exactly the same that skim uses. Or I missed something?

There's also -i in skim. https://github.com/lotabout/skim#interactive-mode
Maybe it's not necessary though.

I'm taking a different approach in #1750, which I believe is more extensible and generic. It doesn't introduce a new "mode", instead, it builds upon the existing features.


[FEAT] Implement Width and Height Resizing Options for Embed Chat Widget

What would you like to see?
The chat widget is currently overlapping with a fixed header (z-index: 99) on 720p resolutions and mobile devices. To address this issue, I propose adding functionality to adjust the width and height of the chat widget. This feature would allow users to customize the widget size to better fit their specific layout needs without interference from other page elements.

Hey @timothycarambat
Please assign me this issue

I would add font size to this as well, mainly an issue on mobile devices but useful for tweaking the look on desktop as well.

@shatfield4 lets do dimensions + font
dimensions are data- attribs
font is user setting


Reduce for Objects?

I find myself trying to turn Objects into other types pretty frequently and find myself doing something like:
Object.keys(myObj).reduce((acc, key) =>  doSomething(key, myObj[key]), someAccumulator)

the other part of the time I want to return early from the iteration and that ends up being the only time I really have to reach for a for loop.
What I'm hoping for is a function like:
reduceObject :: (b -> a -> String -> b) -> Object a -> b
I know this isn't strictly the same as ramda's reduce so I can see adding a separate function to deal with this case.

#257, #364, #546, #625, #656, #920, #1067
The problem is this sort of concern:
var diff = function(x, y) {return y - x;},
    obj1 = {a: 1, b: 2, c: 3},
    obj2 = {b: 2, a: 1, c: 3};
_.reduce(obj1, diff); //=> 2
_.reduce(obj2, diff); //=> 4

obj1 and obj2 differ only in the order of key definition.
If we did something like that in Ramda, we'd violate a law that I'd like to maintain:
if R.equals(x, y), then R.equals(f(x), f(y)) for any f.

#1067 was a long discussion, and involved one proposal that would allow us to do this but was pretty controversial: apply our own sort to keys before any function where it might matter.

@CrossEye
We've been over this before 😄  But as was discussed in #1067 we're already violating the law:
R.equals({a: 1, b:2}, {b: 2, a: 1}); //=> true
R.equals(R.keys({a: 1, b:2}), R.keys({b: 2, a: 1})); //=> false

So, should we remove R.keys and the other functions you mention here? Why do these function get to stay?
The way I see it our users keeps asking us to add this function and we keep trying to convince them it's bad for them.
To me the use case seems covered by R.values and R.toPairs
R.reduceRight(([val, acc]) => ..., R.toPairs(object));

This also has advantage that R.toPairs can be used with R.reduceWhile which might be useful for the early return you mention @jethrolarson.
But this also leads me to the question: If we're already giving people something that is just as dangerous but just slightly less convenient why not give them the real thing?
I'm indifferent. I think I fine solution would be to add an example of reducing an object to the documentation for R.toPairs.

I'm in favour, provided the reducing function does not receive a key as an argument. Z.reduce supports Object#fantasy-land/reduce, and S.reduce and S.reduce_ will in the next release.

Actually, I'm in favor of closing. Realizing that this breaks structural
equality changed my mind
On Sat, Jan 14, 2017, 6:59 AM David Chambers<EMAIL_ADDRESS>wrote:

I'm in favour, provided the reducing function does not receive a key as
an argument. Z.reduce
https://github.com/sanctuary-js/sanctuary-type-classes#reduce supports
Object#fantasy-land/reduce, and S.reduce
https://sanctuary.js.org/#reduce and S.reduce_
https://sanctuary.js.org/#reduce_ will in the next release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ramda/ramda/issues/2046#issuecomment-272629353, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAB-4PS7XPn2BXnPkk3BfVHBk3MCxwNpks5rSOLYgaJpZM4LgGOY
.

@paldepind:

So, should we remove R.keys and the other functions you mention here? Why do these function get to stay?

No, they should all be fixed.
That discussion was part of the reason I backed off a bit from Ramda.  We had a larger community of people I respected and it was completely split on what I thought should have been an obvious improvement.  I've been thinking in the last few weeks, though, that it's time to try this again.  I don't see any way to avoid this if Ramda wants to stay consistent with its ideals.

If it helps, my original use case would have been solved by pipe(pairs, some(foo)). In this case order doesn't matter

Btw I think dissoc and assoc should be isomorphic as far as equality is concerned

@CrossEye Ok. Thanks for explaining. Should we then also add reduceObj with sorted keys?

How about dispatching to [the Set object] (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Set) for iteration, as suggested in https://github.com/ramda/ramda/issues/2160#issuecomment-295838881 ?
Both object and set are "Commutatively-Foldable", or some better name ;)
Actually, object is equivalent to the set of its key-value pairs.
So any method on (finite) sets will be inherited by objects, no need for any extra work.
And extending the reduce method to Sets would seem to solve the problems with best
from both worlds -- the "dirty world" of JS's set-iteration and the clean commutative case.
(I wouldn't call it reduceObj though, as reads like one artificially limited to objects.)


Node converters should return meaningful numbers for length

Description
As noted in #135 the NodeTargetConverter returns the length of the target vector for each node only in the case of categorical targets, otherwise None is returned. Can we make this more meaningful for general cases?
This may change when I've committed the new converters that aim to unify Feature and Target converters and be applicable for heterogeneous graphs.

NodeTargetConverter.len() method needs a docstring. For numeric target, this returns None - is this an intended behaviour?

I am leaving this to return None for now. This is a bit tricky - perhaps we should return the length of the values in the target attribute, but len(1) actually raises an error. Alternatively, we can return the size of the array that we need to store the values, so if the target is 1 (a float) or [1] (a list) will both return 1. I'll need to think about this.

Done Checklist (Bug)

[ ] Bug fixed
[ ] Branch and Pull Request build on CI
[ ] Branch and Pull Request pass unit tests on CI
[ ] Peer Code Review Performed
[ ] Code well commented

We are deprecating node converters.


Add nuget.org badge support

Thanks!
I see some lint error, could you fix them? npm run lint reports 7 errors now.

sure, will heck and fix the same

Awesome, merged!

Deployed https://badgen.net/#nuget 🎉


Feature/support additional http verbs

Hi David,
I couldn't have discovered your LogicMonitor API client at a better time and was dreading having to start one from scratch! I've added logic to support the other major HTTP methods and tested against our own DEV LogicMonitor environment:
./LogicMonitor.Api.Test/Tests.py -v -f
test_01_getDevice_passes (main.Tests) ... ok
test_02_updateDeviceField_passes (main.Tests) ... ok
test_03_createDeviceSdt_passes (main.Tests) ... ok
test_04_getDeviceSdt_passes (main.Tests) ... ok
test_05_updateDeviceSdt_passes (main.Tests) ... ok
test_06_deleteDeviceSdt_passes (main.Tests) ... ok
My team is particularly interested in automating the management of device SDTs so my tests were focused around CRUD activities for this.

Wow!  Nice work 👍


Create an example that demonstrates how to set 1.3 as the minimum TLS version

We have an internal requirement to show users how to do this.

This has been completed.


Validate and update the WCF getting started series - task 1385004

Summary
Initial updates for task 1385004 https://mseng.visualstudio.com/TechnicalContent/_workitems/edit/1385004

@Lxiamail Are you able to complete a review of this PR?

I'm going to review this.

Hi @Thraka, I've incorporated your review comments. Let me know if I need to make additional changes.


Refactor: Impl candidate loop for engine.

Changelog
Refactor: Impl candidate loop for engine.

Refactor: remove CandidateState, impl candidate loop in RaftCore, in
the Runtime-Engine manner.

Refactor: add two event handler to Engine:
elect() and handle_vote_resp().

Refactor: Engine does not need to update metrics_flags manually.
Instead, update these flags according to the output commands by a
Engine event handler.

fix: #293

This change is

Merge without review:) If there is any question let me know:D


Enable HUD text right-align

Currently all HUD text is left-aligned. There should be an option for all text elements to be right-aligned as well.

This is too difficult to care about.


Don't generate polyfills when script target is Latest

re: #438 :)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Can't display data cell in correct way

For example,
tealeg/xlsx display the same cell as: &{ 42740  0xc420424640 mm-dd-yy false false 0 0 2}
throught it's .String() will convert to '01-05-17'
Luxurioust/excelize: just display as 42740 (already converted to string)

@dlintw Thanks for your issue. I have fixed this issue.


Find only finds one instance of what you are looking for.

For somereason on windows (I have not tried ot on mac) when I use ctrl + F to do a find all I get is one instance of what im looking for and not others which is perticully annoying.

Can ou go to debug>show developer console>console and see if there are any errors once you do the broken find?

I am unable to replicate this issue at this time. Should this be marked for closing?


Checkbox Styling

Could you pls support other status in the checkbox styling, say, those in https://minimal.guide/Block+types/Checklists. Thx.

hey @zhanghanghitomi I just released v2.1 of Things, which supports checkbox styling. Update your theme and if all looks good, please close this issue. Thanks!

Thanks for your feedback. It works perfectly! thank you.


decrement_counters stop working in rails 4

I upgrading our app to rails 4 and found that now when user remove some record it's not decrement cached counters. Researching how to fix it. Maybe somebody has some idea.

Actually ActiveRecord::CounterCache.destroy_row should be called, but this not happened.

I just ran across this and was going in circles to trying to figure out if I was doing anything wrong or if this should be expected to work.  Hope the PR gets merged.

This should be fixed by #40
Thank you!

Unfortunately the patch in #40 didn't work with Rails 5, so it was reverted.

Whats the status on this? Anyone working on it?

@zzak wasn't this fixed by https://github.com/ActsAsParanoid/acts_as_paranoid/pull/119 ?


ciao-controller: workloads: change docker ubuntu latest workload

The latest ubuntu image does not support python3. Change the cloud
init to a simple bash script which does an infinite loop so that
the container can be verified as running for test purposes.
Signed-off-by: Kristen Carlson Accardi<EMAIL_ADDRESS>Fixes: #246

Coverage decreased (-0.02%) to 60.841% when pulling e6d07cb52cc96e03fe2fdb79f9c8eaf396c34c61 on kaccardi/issues/246 into 4d5fbe594761e4df279e514b05ab9102908d73f5 on master.


Update to Dune syntax

I am not exactly sure what the proper behaviour of the module should be. I would think the multipart module should be just moved to multipart_form_data and be the only exposed module. Thus it would avoid wrapping, and have a nice mapping from package-name to module name.

Thanks @NathanReb, good to know what your position is on this. It is a good point to add a Changelog entry. I was also thinking to maybe remove pkg/ altogether since dune-release does not need it anymore, so if you'd be willing to switch to that I can also remove it.
As an ex-Schemer and ex-Clojurist I find formatting lists this way strange, but it is your project so I also changed that 🙂

Sorry I was off for a couple weeks, I'll get back to this ASAP!

About removing pkg/, definitely in, maintaining META is a pain and I think dune-release fits perfectly for this project.
I you feel like removing it, be my guest, otherwise I can also do that later to get this merged.

I think it is better if you remove it, since I am on vacation now and access to my laptop is rather spotty :-)

Ok! Thanks for the contribution!


Fix withInsertAtBeginning and README.md

fix default padding when insert a clause at the beginning of the case
Fix titles in README.md
change FieldInTypeAliasTest to also use Run
Uncomment tests 1c and 2 in ClauseInCaseTest
Remove unused imports in all test files

Great work Mateus!


Remove scale subresource from etcd CRD

Signed-off-by: Shreyas Rao<EMAIL_ADDRESS>How to categorize this PR?

/area auto-scaling
/kind bug
What this PR does / why we need it:
This PR removes scale subresource from the etcd CRD to allow VPA to provide recommendations to the etcd pods.
The issue was observed after etcd-druid started mandatorily setting ownerRef on the etcd statefulset since g/g v1.53.0. Due to this, VPA recommender stopped providing recommendations to the etcd pods, since the statefulset had an owner that defined a scale subresource. The root problem is with VPA opting to not provide recommendations in such situations, even if updateMode is set to Off.
This is a temporary fix for gardener, in order to allow vertical scaling of etcd pods once again.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
/invite @istvanballok @andrerun @unmarshall
/cc @aaronfern @plkokanov
Release note:

Remove `/scale` subresource from etcd CRD.

/assign
/milestone v1.58

Thanks for the PR! Just one question - in the PR you mentioned that this is only a "temporary" fix. Are there any plans what a "permanent" fix would be? And should we add a todo/open an issue about it?

@plkokanov there isn't an open issue yet. I'll create on https://github.com/gardener/autoscaler for this, thanks. In any case, gardener doesn't use the etcd's /scale subresource to scale the etcd cluster anyway, so we can live without it.

There's still the Etcd CRD in the examples folder that has the scale subresource: https://github.com/gardener/gardener/blob/70d9a0b350515151da304be376d9d5f4e9a23406/example/seed-crds/10-crd-druid.gardener.cloud_etcds.yaml#L1803-L1805
It gets generated from https://github.com/gardener/gardener/blob/master/vendor/github.com/gardener/etcd-druid/api/v1alpha1/etcd_types.go which also still has the scale subresource: https://github.com/gardener/gardener/blob/70d9a0b350515151da304be376d9d5f4e9a23406/vendor/github.com/gardener/etcd-druid/api/v1alpha1/etcd_types.go#L442

Right. But gardener/example/seed-crds/10-crd-druid.gardener.cloud_etcds.yaml isn't used to deploy the etcd CRD into the seed cluster.
And the plan was to not remove scale subresource from the CRD in etcd-druid repo, because there could be community members using druid who are making use of the scale subresource for their use-case. Hence the decision to change the CRD only in gardener. If the example folder also has to be updated with this change, then we'll need to maintain a fork of etcd-druid without scale subresource, which is definitely an overhead.
WDYT?

We can merge it for now so that the problem is fixed.
Can you add a comment to the etcd CRD that's deployed by gardener explaining why the scale subresource is removed, so that someone doesn't accidentally bring it back?
People might try to copy over the generated CRD from the etcd-druid or the examples folder when some other changes are introduced and this change could get lost.

@plkokanov done, PTAL

This is a temporary fix for gardener, in order to allow vertical scaling of etcd pods once again.

🤣 Very "temporary".


Description and Internationalization for OAuth2 Scopes

The OAuth2 scopes implementation in IS doesn't support providing a human readable description that can be shown in to the end-user when providing consent.
The description should also support internationalization as in challenge questions.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[CI build scripts] Running script with 'sources.json' file

Build scripts can run with sources file.
In this case next arguments do not need to use:
--changed-repo,
--build-type,
--product-type,
--build-event,
--commit-time.
Also stages "install", "pack", "copy"
are not available with "--sources" argument
If --changed-repo and --sources are set, --sources will be ignored.

some updates in git_worker:
improving sources.json file
HEAD instead of SHA sum available in --changed-repo

This feature needed mainly for manual CI reproducing did I understood you correctly?
Or is it in use in closed source?


_testdata links pointing to themselves

I just did a go get of this repository, and then this command fails:
gopkgs -format '{{.Name}};{{.ImportPath}}'

(That command is used by VS Code to capture installed package information.)
The problem is that the recently-added _testdata symlinks in misc and math are pointing to themselves, which causes infinite loops in directory-traversing code. Were they intended to point to _testdata in the parent directory?

Thanks!


Update Firefox versions for api.HTMLMediaElement.error_event

17381

 Quality gate: NA
Details

 Quality gate: NA
Details


Remove guava - part 2

The first tranche of changes to remove Guava collections code.
Introduced a new class Collections into styx-api - mainly fairly simple things, but too verbose to inline throughout the code base. I've added it into Styx API because its methods are used throughout all Styx code.
A couple of methods/classes are complex (TransformedIterator and concat(Iterator...)) and so have been brought over as-is from Guava and properly attributed.

I have a love/hate relationship with guava as it has some awesome features but can lead to dependency hell in downstream projects (particularly of libraries which use it). So I'm curious as to your reasons for removing it? We tend to shade and relocate it (and don't expose any guava in our apis) so we can keep it using it but external users of our artifacts aren't aware.

Partly a general desire to reduce dependencies of the project, partly to avoid dependency hell in downstream projects as you say.
Also, for myself at least, I find that guava has a mix of obvious things that are nice to have and use all over the place, and really awesome/clever things that you might need once in a blue moon but really hit the spot when you do. It would be nice to be able to separate those awesome things out for use only when you need them, because it's those things (I think) that drive people to use different versions of the Guava library.
So the approach we are taking means we have our own "nice to haves" (because they are not really all that hard/laborious to do), and we pull in (by copying) only the awesome stuff that we need.

I have a love/hate relationship with guava as it has some awesome features but can lead to dependency hell in downstream projects (particularly of libraries which use it). So I'm curious as to your reasons for removing it? We tend to shade and relocate it (and don't expose any guava in our apis) so we can keep it using it but external users of our artifacts aren't aware.

@massdosage - Long time ago we attempted shading some other dependencies for styx-api module. But for some technical reasons it didn't work out. Therefore thought we could just cut off Guava as an "easy win".
But looking back my notes, I now realise back then our previous attempt was something totally different. We attempted to shade Rx.Java which we no longer use. It didn't work out because some Rx.Java types were leaking from Styx-API module. But none of this apply today, and I guess we should give shading/relocating an another shot. WDYT @chrisgresty?

So thinking further, our primary purpose is to allow other developers to use whatever version of guava they like (or none), and to allow us to change whatever version of guava we are using (if we're still using it). If we relocate guava, then we fulfil the first purpose but not the second - styx-api will still have (relocated) guava as a dependency, so third party developers can still develop against it, so we are still restricted in changing its version for our own purposes.
But if we relocate guava and use it only in the internal Styx components, but remove it entirely from styx-api (in the same manner as we are doing now), then we achieve both things. And we massively reduce the scope of the changes we are doing.

But if we relocate guava and use it only in the internal Styx components, but remove it entirely from styx-api (in the same manner as we are doing now), then we achieve both things. And we massively reduce the scope of the changes we are doing.

Yes this would be ideal.
I suspect just relocating Guava in styx-api, without removing it, would be "good enough". It won't be watertight, but would work in practice.

Given how little work remains to remove guava from styx-api (there's another PR ready to go, hot on the heels of this one, that removes almost all that's left) I think we can go for the 'ideal' option - relocating guava for only the internal components - if you're OK with that.

This PR is no longer going ahead. It has been superseded by #653, which only removes Guava from styx-api.