[Issue] Live metrics data is temporarily inaccessible

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
Run azd version and copy and paste the output here:
azd version 0.1.0-beta.4 (commit fd96b3e9b283598bc4dc736e893f1b47080fbf7d)
Output from az version
Run az version and copy and paste the output here (minimum required version is 2.38.0):
{
  "azure-cli": "2.39.0",
  "azure-cli-core": "2.39.0",
  "azure-cli-telemetry": "1.0.6",
  "extensions": {
    "azure-devops": "0.25.0"
  }
}

Describe the bug
Seeing this error on azd monitor.

To Reproduce
Run azd monitor and look at the dashboard that is opened for you.
Expected behavior
I expected to see a functioning live metrics dashboard.
Environment
Information on your environment:

WSL Ubuntu 20.04

Additional context
As a note, I was able to see the dashboard by navigating to the shared dashboard in the resource group > clicking Go to Dashboard. I assume this might be an upstream problem. However, it'd be good to think about how we can create a non-blocking path for the user in these scenarios so azd monitor doesn't feel broken.

As a note, I was able to see the dashboard by navigating to the shared dashboard in the resource group > clicking Go to Dashboard. I assume this might be an upstream problem

Does that end up linking you to a different page in the portal?  The snackbar alert at the top of the page here makes it seem like there was just an outage of some sort (however, I can't seem to access the status page, when I try to follow that aka.ms link in my browser I just get bounced to bing).
If there's some better page, we can be linking to we should go adjust but I'm not sure what else we can do here.
I wonder, is the dashboard you can access the same thing you land on via azd monitor --overview?  Today we say that azd monitor with no arguments is the same as azd monitor --logs but perhaps we should have said it was the same as azd monitor --overview.

@savannahostrowski - Can you retry today and next week to see if intermittent az issue?

The issue persists as of today but I'll try again on Monday.
@ellismg azd monitor will bring you to<EMAIL_ADDRESS>If I navigate to the shared dashboard, the URL is
<EMAIL_ADDRESS>However, the dashboard appears to be the same...just without the live metrics? Not entirely familiar with all the ins and outs of App Insights.

If I run azd monitor --overview, two different pages are opened. What's the difference? The dashboard appears the same but the URLs are different?
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>

Still persists as of today.

I think this has to do with the Python Live Metrics known issue. Filed this issue as a fix: #522


Fix pause menu triggering with enter key

Thank you kind sir, I will put your name in our appreciation for when we
present our project
On Sun, May 19, 2024 at 2:46 PM David Zhong @.***>
wrote:

You can view, comment on, or merge this pull request online at:
https://github.com/Travisbui/Capstone-Project/pull/2
Commit Summary

51607bc
https://github.com/Travisbui/Capstone-Project/pull/2/commits/51607bcc765eeb58b20895dcb98d85319c67b51b
Fix pause menu issue

File Changes
(1 file https://github.com/Travisbui/Capstone-Project/pull/2/files)

M scripts/pause_menu.gd
https://github.com/Travisbui/Capstone-Project/pull/2/files#diff-e9607ba199f381759ccca22f62d203961929edb7af17dba2fcc94a30868f3e32
(8)

Patch Links:

https://github.com/Travisbui/Capstone-Project/pull/2.patch
https://github.com/Travisbui/Capstone-Project/pull/2.diff

—
Reply to this email directly, view it on GitHub
https://github.com/Travisbui/Capstone-Project/pull/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5DGLRQ7BQJPRVVIYHTZFB3ZDEMR7AVCNFSM6AAAAABH6WBHI6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGMYDINZZGQ4TQNQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--

This message was sent from a high school student using a Gmail account at
San Francisco Unified School District. The content of the email should be
related to education in support of 21st Century learning. If you have any
questions regarding this message, please email @.***
@.***> and include "Student email" in the subject line.


Update README.md

Hey there! you have the wrong package on the README. Correct package: @codemirror/lang-javascript

Thanks!


complete some gui tests + down arrow key added function

Coverage increased (+0.2%) to 84.464% when pulling 35612eb1af2a90096c55968fd89f589cc26eafb8 on ashpa:master into 8e0b44d143125af6a095d1153eb331d0afeff7eb on CS2103JUN2017-T4:master.


chore: Set search_path for pgvecto.rs

@bo0tzz Can you release new image for v0.2.0 and v0.2.1, thanks a lot.
https://github.com/tensorchord/pgvecto.rs/issues/441

#23


fix provisionCertificate api returns NPE when 'reconnect' parameter is true

Description

This PR fixes NPE with the provisionCertificateCmd when reconnect is set to True.
Also fixes the following Marvin test failures:

test_certauthority_root.py

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

GitHub Issue/PRs

Fixes: #2753
Screenshots (if appropriate):

How Has This Been Tested?
With cloudmonkey, see screenshot
Dev environment components:

Platform: ACS-4.12
Management OS: Ubuntu 18.04 LTS
Hypervisor: 1 Host running KVM with CentOS 7

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
Testing
[ ] I have added tests to cover my changes.
[x] All relevant new and existing integration tests have passed.
[ ] A full integration testsuite with all test that can run on my environment has passed.

Great @dhlaluku
@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.  I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✖centos7 ✔debian. JID-2191

@blueorangutan package

@dhlaluku a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian.


Feature/terra3 61 external vpc

Please review PR.
By default, it will use existing VPC concept.

This PR is included in version 0.13.0 :tada:


osmomath: mutative and efficient BigDec truncations with arbitrary decimals

Closes: #6257
What is the purpose of the change
This PR:

Modifies SDKDec() conversion function to be less resource-consuming
Allows for arbitrary number of decimals truncations

Performance
The new version of SDKDec is approximately 1.84 times better than the old version in terms of time, and 1.75 times better in terms of the number of allocations:
NEW:
BenchmarkSDKRealloc-8           12195942                90.67 ns/op          224 B/op          4 allocs/op

OLD:
BenchmarkSDKRealloc-8            6852703               166.7 ns/op           328 B/op          7 allocs/op

Testing and Verifying
TODO
Documentation and Release Note
TODO

devbot help

Hi! I'm DevBot, a bot that helps with common tasks in the development process. Commands:

devbot fix: Fix basic errors in the PR. (e.g. imports, changelog conflicts, go mod conflicts)
devbot merge base: Merge the base branch into the PR branch.
devbot add changelog [misc/feat/bug/api/sec] [message]: Add a changelog entry to the PR. (e.g. devbot add changelog feat Added a new feature)

If message is blank, defaults to PR title.

devbot re-pr: Re-PR the PR, useful for external contributors where we have no edit perms.

devbot add changelog misc osmomath: mutative and efficient BigDec truncations with arbitrary decimals

devbot add changelog misc osmomath: mutative and efficient BigDec truncations with arbitrary decimals

devbot add changelog misc osmomath: mutative and efficient BigDec truncations with arbitrary decimals

devbot add changelog misc osmomath: mutative and efficient BigDec truncations with arbitrary decimals.

@pysel please see the last A/C on the issue:

Optimize this to be the first client of efficient variable truncation: https://github.com/osmosis-labs/osmosis/pull/6256#discussion_r1312117983

Could you try addressing it, please?

hey @p0mvn.
Mutative Dec conversions are in this PR: https://github.com/osmosis-labs/osmosis/pull/6393
Left a question for one of your review comments here: https://github.com/osmosis-labs/osmosis/pull/6261#discussion_r1324553486

This PR is LGTM after the remaining comments are addressed.
Note to self: try running tests before approving


CSPROPI target is not a number

I am trying to execute Microsoft TypeScript compiler (https://github.com/Microsoft/TypeScript/) under the duktape environment.
You can get my test project by the following link:
https://www.dropbox.com/s/49jkz5typ8mwnoc/test.zip?dl=0
The project is cmake-based, but you can just compile both .c files into an executable without any build system.
I have added a dummy implementations for readFile, writeFile, etc. functions and now I am trying just to get it running.
First, I had to make a modification to tsc.js code because duktape erroneously replaced string value of first argument to function createSourceFile with number 268. Adding a fake first argument to this function solved the problem (all other arguments except first are passing correctly).
Now I am getting the "CSPROPI target is not a number" error and I can't find a workaround. As far as I have understood through the code, the problem seems to be related to replacement of CSPROP with CSPROPI instruction at some moment of code generation. The latter instruction expects a number on stack?, but gets a string.
The problem occurs at line 6962 of tsc.js (line accessing ts.objectAllocator.getSymbolConstructor()). Trying to split this line into multiple expressions (keeping intermediate values in variables) resulted in "CSREGI target is not a number" error.
Can you please help to solve this problems? The tsc.js code seems to be a great stress-test to  duktape.

Here is also a test project for a problem with createSourceFile:
https://www.dropbox.com/s/966po0fj6dhtn39/test2.zip?dl=0
If you run it, you will see it printing:
createSourceFile passing: ior.ts
createSourceFile inside: 268
And if you take a look into the code (you may search by "createSourceFile"), you will see that it should not change to 268.

Thanks, I'll look into these. I tried the test with assertions enabled and got the same result (CSPROPI target is not a number).
The most likely cause is that a VM registry index has overflowed (there are some hard limits for these) without the existing compiler checks noticing it. If this is the case, it would also unfortunately mean that fixing the limit check will cause a compile error due to the function being too large to compile (e.g. it has too many constants or such).
But, we'll see what turns up. Thanks for reporting the issue!

The root cause seems to be a case where the compiler determines in pass 1 compilation that shuffle registers are not needed, but in pass 2 they turn out to be necessary after all. When this happens the shuffle registers are all zero, which causes CSPROPI to use an incorrect register for shuffling.
This also explains why Duktape would corrupt the first argument in such cases, as the first argument maps to register zero.
I can remove the issue by forcing shuffle registers to be allocated for all functions - this is not a proper fix because it impacts also functions that don't need them (which is a majority of functions). I'll need to think about a proper way to fix this, perhaps retrying pass 2 in this special case.
Anyway, I pushed a temporary fix (not merged to master) if you want to test it.

Here's the branch:
https://github.com/svaarala/duktape/tree/fix-cspropi-not-number-gh115

Managed to replicate the argument clobbering in the above test case; couldn't replicate the CSPROPI issue but a related MPUTOBJ shuffle issue is triggered instead, from the same root cause.
The hack fix in the branch makes the test case pass so I think the root cause identified above is correct.

Thank you very much! The "hack fix" seems to solve the problem. I will use it for now for my experiments with TypeScript. I hope you will be able to find a correct solution to merge into the master branch.

Good to hear! This was a pretty interesting bug because it only happens with code that is not too small nor too large (w.r.t. variable/temporary count) so it's not triggered by e.g. test262 or emscripten tests - so thanks for reporting it :)
The easiest "proper fix" is probably just to do a pass 3 in this special case.

The proper fix is now in master - the compiler now runs a third pass for the rare cases where the need for register shuffling is only discovered in pass 1.
@zapolnov Closing this now - let me know if the fix doesn't work for you.


Changed upgrade functionality

Changed upgrade functionality, so that now it reverts to old theme files when it errors out.
J=SLAP-1308
TEST=manual/automatic
Tested by running jambo upgrade on the hitchhiker theme and making sure that upgrade ran unchanged when no error occurs. Also ran with an artificial error to make sure that theme files are reverted correctly.
Created acceptance test for themeupgrader that checks to make sure that when jambo upgrade errors out, the test-theme remains unchanged.

Coverage decreased (-0.2%) to 47.018% when pulling 9a3377dc825b16651c9da8509bb0e0fddee926bb on dev/upgrade into b2bce08cc90f6d603021d1a4c458a5d2d5f465a9 on develop.


Error: Could not find or load main class SpeechRecognitionAI.jar

It is very interesting, but unfortunately, when running in Debian, it gives an error.
Error: Could not find or load main class SpeechRecognitionAI.jar
Caused by: java.lang.ClassNotFoundException: SpeechRecognitionAI.jar

Thanks everything started. And could you add instructions for integration with a Home assistant.

Then there was another question. The program has only "IP mic" at the moment, it is impossible for me to purchase the application. Is it impossible to use a local microphone with this program? (Or have I not figured it out yet?)


[BACKEND] Remote Server Updates for New Record Format

Motivation and Context
Why is this change required? What problem does it solve?:

Following significant changes to the backend introduced in #70, the remote server implementation was horrifically broken. As a result, #70 is staged in the dev branch until these remote fixes are introduced.
The negotiation, transfer, and storage of all data on both the remote server and client side are now compliant with the new backend selector module. Data backends on each side do not need to be consistent, the local copy encodes locating information (as per the design spec of #70).

If it fixes an open issue, please link to the issue here:

N/A

Description
Describe your changes in detail:
The most significant changes surround how samples are identified and grouped together (ie. identified as having a consistent schema, and therefore backend). In the previous implementation, it was assumed that a field named schema_hash was always present and could be used in order to group samples with similar content into the appropriate backend/collection containers when sent across the network. Since such assumptions are now prohibited, a new method of determining groupings for samples is intoduced here.
A push / fetch negotiation follows the following form
- Determine missing commits

For each commit:
   - Determine missing schemas defined in that commit
      For each missing schema:
        - Transfer defining information

   - Determine missing labels defined in that commit
      For each missing label:
        - Transfer key/value pairs

   - Determine missing sample hashes defined in the commit
      For each missing sample hash:
        - Determine schema_hash corresponding to the sample by mapping
           dset name to dset schema spec (at the commit level)
        - Transfer sample hash, sample bytes, dtype, shape, schema_hash

   - Transfer packed commit record data

This process is able to fully negotiate and transfer all data in a manner independent of the backend which the data is actually stored in. At the time of this initial post, fetch / push / clone operations do work for simple tests run in my development environment.
The server startup methods have been modified to allow for more convenient command line use (querying record details of the server like we can for the clients) and to enable simple plugin to our test suite. The majority of work following this initial post will focus on populating the tests and cleaning up / documenting the code.
Types of changes
What types of changes does your code introduce? Put an x in all the boxes that apply:

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Is this PR ready for review, or a work in progress?

[ ] Ready for review
[x] Work in progress

How Has This Been Tested?
Put an x in the boxes that apply:

[ ] Current tests cover modifications made
[x] New tests have been added to the test suite
[ ] Modifications were made to existing tests to support these changes
[ ] Tests may be needed, but they are not included when the PR was proposed
[ ] I don't know. Help!

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have signed (or will sign when prompted) the tensorwork CLA.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@hhsecond, please be aware of this WIP. Any feedback is greatly appreciated.

Thanks @hhsecond. I'd like to get this merged relatively soon, but I can definitely hold off for the weekend to give you time to do a thorough look.
In the meantime I'm going to start work on enabling partial fetch/clone operations in the frontend. With these changes combined with #70 all the appropriate functionality is implemented in the hangar core / backend processor.

Ok all your previous comments have been addressed. let me know if anything else pops up!

Hey @hhsecond, just a note that I need to merge this so that we can rebase the dev branch and then start making changes effectivley. as you see bugs, please make an issue for them!


Blue gets Prettier 💅

Blue gets Prettier 💅
This update implements Prettier to replace Standard. It also leverages
Husky for pre-commit fixing, improving the dev/testing working a bunch!
Thanks to @Charca for the inspiration 🤗
P.S. Will minor version bump this once… since a bajillion things have been touched.

Coverage remained the same at 100.0% when pulling a8886567ddaad91167cb4a56e6c5d0d40014128f on prettier into 564cc14b65414c1e7eecd5033f26bed86684b342 on master.


fix(linter/import): false positive for indirect export in namespace

#2863
#2862  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Dunqing and the rest of your teammates on  Graphite

Merge activity

Mar 30, 12:55 AM EDT: @Boshen started a stack merge that includes this pull request via Graphite.


Out of memory

I have 6GB memory in gtx2060. trying to use a 1024x659 image. used smallest brush with just one stroke
getting this memory error
RuntimeError: CUDA out of memory. Tried to allocate 16.00 MiB (GPU 0; 6.00 GiB total capacity; 4.02 GiB already allocated; 0 bytes free; 4.38 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

is it that memory taxing or there is an error?

Also website load speed is slow.

I have 6GB memory in gtx2060. trying to use a 1024x659 image. used smallest brush with just one stroke getting this memory error
RuntimeError: CUDA out of memory. Tried to allocate 16.00 MiB (GPU 0; 6.00 GiB total capacity; 4.02 GiB already allocated; 0 bytes free; 4.38 GiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

is it that memory taxing or there is an error?

GPU memory is not related to size of brush or number of stroke
LaMa is a big model，you need more GPU memory，or you can try cpu with --device=cpu

With CPU it works, but image is low resulation when i download. i think the dev pushed a option to not compress the image. how to enable it here?

Image is resized here，you can change maxSize

i changed it but still the image is resizing

Did you execute the yarn build?

no, how to do that?

follow README ：https://github.com/Sanster/lama-cleaner/blob/main/README.md#fronted

can't you guys add an option in the website to do keep the original image? i see a button called HD in the original project

Maybe when I have time I can add this “simple" button

That will be great. That button only limits to 2k, if possible add 3 option, 1080, 2k and original.

That will be great. That button only limits to 2k, if possible add 3 option, 1080, 2k and original.

Done. Check the new relesse


Set ErrorHandling to ContinueOnError when gathering metrics from scripts

When gathering metrics on a busy pool (or crashed/stopped) with script-collectors the fcgi error results in a 500 from the exporter, with the message as body.
An error has occurred during metrics collection:

[from Gatherer #2] dial unix /run/php-sock/php7.2-fpm.sock: connect: no such file or directory

By setting HandlerOpts we can have the error logged to stdout and metrics served as best-effort, in our case only php_fpm_up set to 0.
# HELP php_fpm_up Whether scraping PHP-FPM's metrics was successful.
# TYPE php_fpm_up gauge
php_fpm_up{socket_path="/run/php-sock/php7.2-fpm.sock"} 0

Thanks for the contribution @Meroje


Pull request merged without required tests passing

A recent pull request has been auto-merged before any tests passed. This seems to be caused by the fact that it was created from a fork so we get can't run tests (and builds) internally instead of actual testing. The check summary says 15 checks passed which might be the cause of the confusion. Shouldn't packit use some different check status when tests cannot be run? I guess this would prevent the auto-merge feature skipping the required checks.

Hi, thank you for the bug report. I can confirm Packit does not set :heavy_check_mark: when it cannot run tests:
https://github.com/packit/packit-service/blob/eb5d95cdc1c1531ac27cc8dd5999ae310b06f2c5/packit_service/worker/checker/testing_farm.py#L83
It uses the neutral status:

What you are seeing right now in the pull request is the consequence of the /packit build comment done by a repo collaborator: https://github.com/teemtee/tmt/pull/1847#issuecomment-1427939144
Hence 15 checks passed and you can inspect they are genuine test runs.
Please suggest what we can do better in Packit to make this more straightforward. At minimum we should document how to use Packit and Testing Farm together with GitHub's merge queue.
I recommend setting required set of checks to be passing for the auto-merge. @mfocko is working on adding support for this in Packit: https://github.com/packit/packit-service/pull/1909

I recommend setting required set of checks to be passing for the auto-merge.

The weird thing is that we have defined a set of required checks, but in this case they were somehow skipped. Could it be that when /packit build was called the checks were reset and because pre-commit was green the auto-merge happened? Still, I would expect that merging should not happen before all required checks are ok. Interesting...

I don't think it should merge, if you don't have the checks, it just waits, e.g. in our case we tried merge queue, Zuul ignored the branch it created for the CI checks and it timed out. It needs to have them there…
The issue I mentioned in the PR, regarding auto-merge ignoring required checks and merging anyways while having a merge queue enabled, could be related even though you don't have the merge queue enabled.
I'm not sure how I feel about auto-merge being enabled given that GitHub tends to test their changes in production environment before rolling them out. (IIRC they even made a blog about the way they can route between deployed changes just to test them on production)

Revisiting the issue
I've gone through the docs:

https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/automatically-merging-a-pull-request
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/configuring-pull-request-merges/managing-auto-merge-for-pull-requests-in-your-repository
https://docs.github.com/en/repositories/configuring-branches-and-merges-in-your-repository/managing-protected-branches/managing-a-branch-protection-rule
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/about-status-checks
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/collaborating-on-repositories-with-code-quality-features/troubleshooting-required-status-checks

Overall I cannot find any comments about the results that are accepted as successful, so we can only assume that neutral is accepted as successful which makes sense in case of code analysis tools or other CIs like that. OTOH we've already had a request to report issues with privileges as neutral as they are, technically, not failures (we didn't run anything). In the linked PR it is a problem, because it allowed bypassing the test runs on auto-merge.
If this problem affects you regularly, please let us know. One of the possible solutions to this problem would be configurable result for the permission issues, or maybe an option denoting that the check is required, so it should either succeed, or fail.

The only .skipped I rg'd  in the Packit Service is related to the release-syncing (propose-downstream).
OTOH, reading the Slack discussion, I'm not really sure if this is something that should be worked around on our side, as it is something that definitely affects other users of different CI systems too. I also hate the fact that the documentation is not explicit on this matter, cause I also got lost there (as can be seen in my previous comment).

But Packit is not using skipped, right? I don't see any neutral mentioned in the docs. So it's all a bit confusing to me.

No, we use neutral (see conclusion values in here).
The behaviour is definitely not clear, also see another related discussion.

To clarify more, so... even if the job is not completed, for example when I used /packit test in the pull request above, the neutral conclusion is used for the testing jobs? Wouldn't it be more appropriate to use one of the status values?
Can be one of: queued, in_progress, completed, waiting, requested, pending

@lachmanfrantisek mentioned pending in the chat, which would make sense to me as well. Isn't the core of the problem not to mark the job as completed, or in other word to set the conclusion before we actually get the results?

To clarify more, so... even if the job is not completed, for example when I used /packit test in the pull request above, the neutral conclusion is used for the testing jobs? Wouldn't it be more appropriate to use one of the status values?

Yes, that's correct. This originated from #760 and the idea was not to make the PR "red" (CI failure) for permission issues (and now also for some missing prerequisites issues). I can see that for your use case (and possibly also other projects), this is not desired with the auto-merge. We could either verify that pending doesn't break anyone's use case or make it configurable as already suggested in the linked issue.


fix(Storybook): fixes white flash

Prior to 6.5, we were able to set appBG in the custom Storybook Theme to set the background in the Preview iframe. This is no longer the case. There are possible fixes coming but have not seen them actually committed to a release yet. See this pr
The other known issue is a white flash when clicking from a Story to the Docs of that Story. This is related to some classes being displayed and not displayed during 'loading' of the pages. The CSS overrides added in this PR are to fix those issues until Storybook fixes them.
This PR makes a few updates to the Storybook configuration:

Adds CCS style overrides for style overrides the known appBG and the loading issue styles when clicking between Stories and Docs
Disables the following Storybook addons from the toolbar: Background, Measure, Outline and Viewport. We are not utilizing these and most of them have to do with DOM anyway.

linked wrong branch and changes


Optimize OS and Minikube

Currently we are packaging the Airy Core Platform on an existing CentOS image and in order to make the whole package smaller, we need to user either Debian or some other alternative, to make it as slim as possible, but still to run with all the features and functionalities.

We have decided to move forward with Alpine 3.10 and use k8s.io instead of Minikube, as it has the smallest footprint in terms of size.


Traceback for no auth header (python3.4)

tail -3 tabbox.py
    start(MyApp, debug=True, start_browser=False,
          port=9091, websocket_port=50000,
          username='myuser', password='mypassword')

and alling the url with a webbrowser.
python3 tabbox.py
remi.server      INFO     Started websocket server <IP_ADDRESS>:50000
remi.server      INFO     Started httpserver http://<IP_ADDRESS>:9091/
remi.server.http INFO     Authenticating
remi.server.http DEBUG    <IP_ADDRESS> "GET / HTTP/1.1" 401 -
----------------------------------------
Exception happened during processing of request from ('<IP_ADDRESS>', 45904)
Traceback (most recent call last):
  File "/usr/lib/python3.4/socketserver.py", line 617, in process_request_thread
    self.finish_request(request, client_address)
  File "/usr/lib/python3.4/socketserver.py", line 344, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File "tabbox.py", line 21, in __init__
    super(MyApp, self).__init__(*args)
  File "/usr/local/lib/python3.4/dist-packages/remi/server.py", line 347, in __init__
    super(App, self).__init__(request, client_address, server)
  File "/usr/lib/python3.4/socketserver.py", line 673, in __init__
    self.handle()
  File "/usr/lib/python3.4/http/server.py", line 398, in handle
    self.handle_one_request()
  File "/usr/lib/python3.4/http/server.py", line 386, in handle_one_request
    method()
  File "/usr/local/lib/python3.4/dist-packages/remi/server.py", line 762, in do_GET
    self.wfile.write('no auth header received')
  File "/usr/lib/python3.4/socket.py", line 394, in write
    return self._sock.send(b)
TypeError: 'str' does not support the buffer interface

Expectation a failed authentification should not  lead to a traceback in the backend.

@bernhardreiter Thank you for reporting! Now should be fixed.

Ran a quick test with 00ccfb3717f370d30a18cc371d59890a972a123f and it seems to work better now.
@dddomodossola Thanks for publishing remi as Free Software.


Release 0.19

Port all our custom changes to release 19 branch.

@johngmyers Is there something i have to do to make travis run for this PR?


DBZ-7443 [docs] correct debezium.sink.pubsub.flowcontrol.* variable names

Correct configuration variables case discrepancy between docs and pubsub sink connector
https://github.com/debezium/debezium-server/blob/df953233220e5250129135fabf1d97212d77ea40/debezium-server-pubsub/src/main/java/io/debezium/server/pubsub/PubSubChangeConsumer.java#L98C3-L98C88

Applied via https://github.com/debezium/debezium/pull/5231 Thanks!


Modals

This PR contains implementation for newly introduced modal interactions and text input component.
Useful links:

Docs
https://github.com/discord/discord-api-docs/pull/4459

Important note: this PR will be undrafted after the official release

YESSS

Undrafting it due to https://github.com/discord/discord-api-docs/pull/4459.

Should be good now.


Large Bundle Size for dash_mantine_components.js

The current dash_mantine_components.js file is quite large at 2.7MB, which can impact performance and load times.
Build Output:

><EMAIL_ADDRESS>build:js
> webpack

asset dash_mantine_components.js 2.6 MiB [compared for emit] [minimized] [big] (name: main) 1 related asset
orphan modules 1.59 MiB [orphan] 726 modules
runtime modules 1.29 KiB 7 modules
modules by path ./node_modules/ 5.18 MiB 1588 modules
modules by path ./src/ts/ 321 KiB
  modules by path ./src/ts/components/core/ 239 KiB 127 modules
  modules by path ./src/ts/components/charts/*.tsx 15.3 KiB 8 modules
  modules by path ./src/ts/components/extensions/ 14.1 KiB 8 modules
  modules by path ./src/ts/components/dates/*.tsx 24.9 KiB 7 modules
  modules by path ./src/ts/utils/*.ts 5.24 KiB
    ./src/ts/utils/charts.ts 655 bytes [built] [code generated]
    + 3 modules
  ./src/ts/index.ts 21.2 KiB [built] [code generated]
  ./src/ts/components/styles/MantineProvider.tsx 1.49 KiB [built] [code generated]
external {"commonjs":"react","commonjs2":"react","amd":"react","umd":"react","...(truncated) 42 bytes [built] [code generated]
external {"commonjs":"react-dom","commonjs2":"react-dom","amd":"react-dom","umd"...(truncated) 42 bytes [built] [code generated]

WARNING in asset size limit: The following asset(s) exceed the recommended size limit (244 KiB).
This can impact web performance.
Assets:
  dash_mantine_components.js (2.6 MiB)

WARNING in entrypoint size limit: The following entrypoint(s) combined asset size exceeds the recommended limit (244 KiB). This can impact web performance.
Entrypoints:
  main (2.6 MiB)
      dash_mantine_components.js

WARNING in webpack performance recommendations:
You can limit the size of your bundles by using import() or require.ensure to lazy load some parts of your application.
For more info visit https://webpack.js.org/guides/code-splitting/

Questions:
Orphan Modules:

Are the orphan modules necessary for the current dependencies?
If not, can they be removed?
What is the best way to identify and eliminate unnecessary orphan modules?

Lazy Loading Components:

What is the most efficient way to determine which components should be lazy-loaded?
A good starting point may be components like charts, dates, and the carousel component. Would these be good candidates for lazy loading, or are there others to consider?

Additional Strategies to Reduce Bundle Size:

Are there any other techniques or optimizations we could apply?

@alexcjohnson - I would love your input on this topic

Chunking out things like charts is probably the best solution. If you tell webpack to make a source map (in a separate .map file that you don't include in the package, for example devtool: 'source-map' like dash-ag-grid does) then you can use source-map-explorer to figure out which parts are most important to break out.
It looks as though orphan modules are not included in the bundle anyway, so we should be able to ignore those.

I have put all the (heavy) dash-leaflet plugins as separate chunks. I guess you could adopt a similar strategy :)
https://github.com/emilhe/dash-leaflet/tree/master/src/ts/fragments

Hi @emilhe
That chart is helpful - thanks!
I'm hoping that  https://github.com/snehilvj/dash-mantine-components/pull/366  will fix highlights because it reduces the file size of Highlight.py from 442.9kB to 10.8kB.
So looks like recharts is the place to start.


convert artifact level button to textfield on theorycraft page (substats)

Related to https://github.com/frzyc/genshin-optimizer/issues/2046
Update the custom inputs to NumberInputLazy

I can take a look!


deleting .cfg after update

after updating our server via cron i realized that 2 of our 3 servers running an older version of ark.
so i started a manual update via "arkmanager update @all --warn --update-mods --saveworld --backup"
it was printing some errors:

the arkIsland.cfg is as ".arkIsland.cfg.swp" on our FTP from 2 days ago and the arkEvent.cfg was deleted completly.
sry for my bad english :(
grettings
EDIT: i've protected the .cfgs now with "chattr +i"
now ark-server-tools is printing this:
Applying update from staging directory rm: remove write-protected reular file 'arkEVENT.cfg'?
then i hit "n" the server is starting with completed update

If you want to put the config file into the game server directory, please try putting the config file into the ShooterGame/Saved directory in the game server directory.
We can add *.cfg files to the list of files that are not removed when they are not present in the staging directory.

thanks, this fixed it!
is it normal that the tool does not show any progress during update process?
before V1.6 it was atleast printing "downloading mod XXX"

arkmanager will now only download mods that need to be updated, rather than always trying to download every mod.

thanks alot!


2.5.0 regression

After updating to 2.5.0 from 2.4.0 we can no longer wrap express req objects. Works fine after reverting to 2.4.0.
e.g.
app.post('/*', function(req) {
  console.log("#1");
  _(req).pipe(fs.createWriteStream("tmp");
  console.log("#2"):
})

error:
#1
TypeError: Cannot read property 'done' of undefined
    at Stream._generator (upload/node_modules/highland/lib/index.js:393:26)
    at Stream._runGenerator (upload/node_modules/highland/lib/index.js:944:10)
    at Stream.resume (upload/node_modules/highland/lib/index.js:806:22)
    at Stream._checkBackPressure (upload/node_modules/highland/lib/index.js:708:17)
    at Stream.resume (upload/node_modules/highland/lib/index.js:801:29)
    at Stream.pipe (upload/node_modules/highland/lib/index.js:890:7)
    at upload/build/index.js:204:35
    at Layer.handle [as handle_request] (upload/node_modules/express/lib/router/layer.js:82:5)
    at next (upload/node_modules/express/lib/router/route.js:110:13)
    at Route.dispatch (upload/node_modules/express/lib/router/route.js:91:3)
    at Layer.handle [as handle_request] (upload/node_modules/express/lib/router/layer.js:82:5)
    at upload/node_modules/express/lib/router/index.js:267:22
    at Function.proto.process_params (upload/node_modules/express/lib/router/index.js:321:12)
    at next (upload/node_modules/express/lib/router/index.js:261:10)
    at expressInit (upload/node_modules/express/lib/middleware/init.js:23:5)
    at Layer.handle [as handle_request] (upload/node_modules/express/lib/router/layer.js:82:5)

Looks like we're inferring that req is an Iterator because it has a next method. We should probably check is something is a readable stream before the iterator check.

yes @quarterto, the sequence in https://github.com/caolan/highland/blob/master/lib/index.js#L517 was added in this release for iterators et iterables.
currently the default Object case is the readable stream where we use pipe without checking the existence of pipe. we could check that pipe is a function early and consider we are in the stream case or have a test of the class of the object.
@svozza when you added the iterator/iterables case have you found another project where there is such a chain of type detection and do you have an idea how best we could change the current behavior to remove the regression ?

No, I've not found other libraries that detect iterators other than by the presence of a next function. I mean the most drastic fix would be for us to only support Iterables and their default Iterator but that would mean you couldn't do this any more:
var map = new Map([['a', 1], ['b', 2]]);
_(map.values())...

Something mentioned in the Mozilla docs is that when you toString() an iterator you get back a string with word Iterator in it, I guess it would be similar to how we implement the isArray function now. That would probably only work with offical Iterators like those associated with Array, Map, Set etc (I doubt that custom iterators written by developers would have the correct toString method implemented). Tbh, I really don't like this as a solution but I'm struggling to think of anything better.

what do you think about giving more priority to a test on pipe ? I think that as @quarterto mentioned it might solve the express use case.
@ronag do you confirm that the express req has a pipe function ? is it always a readable ?

Yeah, I think @quarterto 's suggestion is far preferable to mine, intuitively it makes sense that one of the first things we do is check whether an object we've been passed is a Readable stream. Failing early when passing in something Highland can't deal with is also a benefit of throwing in the default case. I can put together a PR for this right now if you want and we could do a 2.5.1 release maybe?

Yes I think we can push a 2.5.1 easily ; only some docs were commited since 2.5.0 so we can use the grunt targets for this.
There is https://github.com/rvagg/isstream with an isReadable method ; I don't know the effect that depending on stream would have on browserify though. I'd prefer something more lightweight

Yeah, I think that isStream thing is overkill and also it relies of instanceof, which is notoriously unreliable. Would it be enough to check for pipe and _read?

In the code, we use xs.pipe and xs.on so maybe we can check these 2 and consider we are in the stream case. I guess that our polymorphic constructor is already a bit of a hack so maybe even just pipe would solve our problem.
I fear that we may hit this problem again down the road with custom librabries with next or pipe or whatever ; at one point maybe we will need adaptors for custom libraries because we won't be able to make everybody happy.

Yeah, I was thinking that it's getting a bit unwieldy. Even though it's really nice to be able drop various sources into the constructor and they'll just work (mostly!) I think you're right and we may have to go down the adapter route: perhaps as part of the extensions work we've planned post-3.0.

For back compat, I think prioritizing the checks for pipe and on is the best short term fix.
What we probably really want is a separate _.fromReadable, _.fromIterator, etc and push the issue to the user, who would unequivocally have that information. That way, we don't have to worry about doing some generic coercion that may or may not work.


Fix inconsistent naming of newly introduced quasi static MultiSensor

The multi sensors of both the quasi statsic SinglePhase and MultiPhase package are inconsistently named. The names shall be changed to match the former introduced naming convention of the packages.

i_abs -> abs_i
i_arg -> arg_i
v_abs -> abs_v
v_arg -> arg_v

... an so on

The fix is backwards compatible, since these components have not yet been released.

Reset milestone such that #2483 does not show up in the (later) Release Notes.


Fix error message naming / docs. Set task.expl earlier.

Very minor cleanup and ordering change.

Cool! :)

I want to merge. Anyone else wants to review?

@dhruvg, since you're actually changing behaviour here. Can you somehow test, preferably by addinga test case, that your logical change (additional json.dumps()) fixes an existing bug?

@Tarrasch Added tests, please merge at your convenience.

"Add tests." will won't be that useful when scrolling through a commit log. Just merge the two commits I think. :)

Done. Thanks!

By the way, if you (hypothetically) would revert worker.py, would the new test cases fail?

Yes, the FailTask case would fail. The different between using json.dumps and not using it is very subtle in the test, but important if we return dicts in the on_failure calls.
F...
FAIL: test_update_result_queue_on_failure (main.TaskProcessTest)
Traceback (most recent call last):
File "test/worker_task_test.py", line 86, in test_update_result_queue_on_failure
mock_put.assert_called_once_with((task.task_id, FAILED, json.dumps("test failure expl"), [], []))
File "/Library/Python/2.7/site-packages/mock.py", line 835, in assert_called_once_with
return self.assert_called_with(*args, **kwargs)
File "/Library/Python/2.7/site-packages/mock.py", line 824, in assert_called_with
raise AssertionError(msg)
AssertionError: Expected call: put(('FailTask()', 'FAILED', '"test failure expl"', [], []))
Actual call: put(('FailTask()', 'FAILED', 'test failure expl', [], []))

Ran 4 tests in 0.054s
FAILED (failures=1)

Cool! Thanks! :)


Updating baselines to resolve VRT errors

Getting the following visual regressions so pulled these off the build from the main branch and hoping this can help resolve unrelated VRT failures on active working branches.
Ready-for-merge Checklist

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
[x] Repository compiles and tests pass.

Merging
Please squash when merging and ensure your commit message uses conventional commit formatting.
Be sure to share your updates with the<EMAIL_ADDRESS>mailing list!

Jump links Vue tests are failing in AT.  I'll open a separate PR to patch this.


Fixed #229 Added file extension append to selected path in SaveFileDialog

@kekekeks could you take a look at this?


Http api router initialization failed

Issue description
After installing the node , im getting error "Http api router initialization failed. Error message: ENOENT: no such file or directory, open '/root/certs/privkey.pem', undefined"
Expected behavior
Actual behavior
Steps to reproduce the problem

cd /root/ && curl https://raw.githubusercontent.com/OriginTrail/ot-node/v6/release/testnet/installer/installer.sh --output installer.sh
2.chmod +x installer.sh
3../installer.sh

Specifications

Node version: v6.0.0-beta.2.0
Platform:Ubuntu 22.04
Node wallet: 0xA6Ac58B669268ebC281C391Df8E0e5E9a993AAE8
Node libp2p identity:QmdbTKjSE1hHv5Z944Xbd4FZB9umMJygdDxcUDiqMcoUJN

Contact details

Email:

Error logs
[2022-08-22 12:54:04] INFO: Hub contract address is 0x256736AEb3f19AC6738E9F4D10C9B61da71CEB9F
[2022-08-22 12:54:04] DEBUG: Connected to blockchain rpc : wss://lofar.origin-trail.network.
[2022-08-22 12:54:04] INFO: Balance of 0xA6Ac58B669268ebC281C391Df8E0e5E9a993AAE8 is 73.998069639145 OTP and 99999.999999999999997 TRAC.
[2022-08-22 12:54:04] INFO: blockchain module initialized with implementation: otp
[2022-08-22 12:54:04] INFO: repository module initialized with implementation: sequelize-repository
[2022-08-22 12:54:04] INFO: All modules initialized!
[2022-08-22 12:54:04] INFO: otp blockchain identity is 0x975BbB9E41ab7A4AAc690b0f385f979476b812b2
[2022-08-22 12:54:04] INFO: Initializing http api router
[2022-08-22 12:54:04] ERROR: Http api router initialization failed. Error message: ENOENT: no such file or directory, open '/root/certs/privkey.pem', undefined
[2022-08-22 12:54:04] INFO: Initializing rpc router
[2022-08-22 12:54:04] INFO: Enabling network protocol: /store/1.0.0
[2022-08-22 12:54:04] INFO: Enabling network protocol: /get/1.0.0
[2022-08-22 12:54:05] INFO: Replay pending/started commands from the database...
[2022-08-22 12:54:05] INFO: Telemetry Injection Service initialized successfully
[2022-08-22 12:54:05] INFO: Node is up and running!
Disclaimer
Please be aware that the issue reported on a public repository allows everyone to see your node logs, node details, and contact details. If you have any sensitive information, feel free to share it by sending an email to<EMAIL_ADDRESS>

Hi @lello1231, this has been reported already. Please take a look at this discussion : https://github.com/OriginTrail/ot-node/issues/2019


glTF manipulation

Manipulate glTF files to change textures and delete meshes. May interact with #4.
Requested features:

Change materials:

Add, remove or change composite/emissive/albedo textures.
Change ambient occlusion, roughness, metalness, color and emission properties.

Remove meshes, specially decal and livery meshes.

Switched glTF development to a fork of the official Khronos Blender importer at https://github.com/leandroarndt/glTF-Python-IO


Update _config.yml

Resolves #ISSUE_NUMBER

Before the change?

After the change?

Pull request checklist

[ ] Tests for the changes have been added (for bug fixes/features)
[ ] Docs have been reviewed and added/updated if needed (for bug fixes/features)

Does this introduce a breaking change?

[ ] Yes
[ ] No

Summary by Sourcery
Build:

Change the Jekyll theme from 'dinky' to 'cayman' in the _config.yml file.

Summary by CodeRabbit

New Features

Updated the website theme to a new design (from Dinky to Cayman) for improved aesthetics and user experience.

@gstraccini create labels

@gstraccini review


Add a API to add/remove items to the "Tools" menu

Make possible add and remove items to the "Tools" menu.
The API is based on my initial proposal but i changed some things during the implementation:

I removed the has_plugin_menu_item(I don't added it since other functions don't have it's has_* counterpart).
All function were renamed to use *_tool_* instead of *_plugin_*, IMO this way it's clear that the functions are related to the "Tools" menu.
The int ID parameter of add_tool_menu_item was changed to Variant ud, used for the callback function.
The callback format now is callback(Variant ud), which made the code simpler and more flexible.
The name format for the items, were changed from "my_category/my_item"/"MyCategory/MyItem" to ANY string_Format/My Item Name where the first slice is the item category, this way the name parameter is friendlier to translation(e.g. "category/"+TTR("My Item Name") or vformat("categoty/%s", TTR("My Item Name"))).

Closes #1429.

There are now merge conflicts, could you have a look @TheHX?

Sorry for the delay, rebased to current master.

Sorry, merge conflicts again. I promise I'll have a look at it for merging then, nobody expressed any concern about the added feature.
Also note that your commits are not linked to your GH account, you might want to fix that too (or not, it's up to you).

Ping ;)

Rebased and merged it myself in e427ece. Please check it still works as intended in the current master branch :)

Alright, fixed for real with b24b52d.


Create test mode

It would be useful if there were some test mode variable that one can set to True or False.
If it's True the map is small with only a few enemies, planets and asteroids, lots of health and ammo etc.
If it's False, the map is larger normal amount of health etc.
This will be very useful especially once the map is larger.

Seems to be implemented now :)


Reducing the overhead in parallelizing simulations

The goal of this PR is to reduce the overhead in parallelizing simulations, as discussed at the end of issue #13 .

Closing the PR since this is not relevant anymore in the short term.


Typescript: Cannot assign immutable state from payload to immutable writable draft

🐛 Bug Report
Immer exposes a Immutable helper type that deeply mark a type as readonly:
type A = Immutable<{ a: number }>;

const a: A = {
  a: 4
};

a.a = 5; // error, a.a is readonly

Immer also conveniently make it so that a WritableDraft<...> will let you assign to subobjects since immer will not mutate the original state, which is correct:
const a2 = produce(a, (draft) => {
  // draft is WritableDraft<Immutable<...>>
  draft.a = 5;
});

However, writable draft don't let an immutable value be assigned to another immutable value:
type B = Immutable<{ arr: number[] }>;

const b1: B = {
  arr: [1, 2, 3]
};

const b2: B = {
  arr: [4, 5, 6]
};

produce(b1, draft => {
  draft.arr = b2.arr;
});

This is technically correct as arr will be assigned to another mutable value
This issue arises a lot when you have an immutable state in redux, use a selector and send back a state in an action payload. When action payloads are immutable, this issue will trigger.
Link to repro
A bug report without a reproduction is not a bug report. Failing to follow this templately is likely to result in an immediate close & lock of the issue.
CodeSandbox demo
To Reproduce
Steps to reproduce the behavior:
type B = Immutable<{ arr: number[] }>;

const b1: B = {
  arr: [1, 2, 3]
};

const b2: B = {
  arr: [4, 5, 6]
};

produce(b1, draft => {
  draft.arr = b2.arr;
});

Observed behavior
Typescript refuses the code
Expected behavior
Typescript should accept the code
Environment
We only accept bug reports against the latest Immer version.

Immer version: 10.1.1
[x] I filed this report against the latest version of Immer
[ ] Occurs with setUseProxies(true)
[ ] Occurs with setUseProxies(false) (ES5 only)

This might help https://immerjs.github.io/immer/typescript#cast-utilities
On Thu, 13 Jun 2024, 16:54 Guillaume Racicot, @.***>
wrote:

🐛 Bug Report
Immer exposes a Immutable helper type that deeply mark a type as readonly:
type A = Immutable<{ a: number }>;
const a: A = {
a: 4};
a.a = 5; // error, a.a is readonly
Immer also conveniently make it so that a WritableDraft<...> will let you
assign to subobjects since immer will not mutate the original state, which
is correct:
const a2 = produce(a, (draft) => {
// draft is WritableDraft<Immutable<...>>
draft.a = 5;});
However, writable draft don't let an immutable value be assigned to
another immutable value:
type B = Immutable<{ arr: number[] }>;
const b1: B = {
arr: [1, 2, 3]};
const b2: B = {
arr: [4, 5, 6]};
produce(b1, draft => {
draft.arr = b2.arr;});
This is technically correct as arr will be assigned to another mutable
value
This issue arises a lot when you have an immutable state in redux, use a
selector and send back a state in an action payload. When action payloads
are immutable, this issue will trigger.
Link to repro
A bug report without a reproduction is not a bug report. Failing to
follow this templately is likely to result in an immediate close & lock of
the issue.
CodeSandbox demo
https://codesandbox.io/p/sandbox/immer-sandbox-forked-t2n37v?layout=%257B%2522sidebarPanel%2522%253A%2522EXPLORER%2522%252C%2522rootPanelGroup%2522%253A%257B%2522direction%2522%253A%2522horizontal%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522id%2522%253A%2522ROOT_LAYOUT%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522UNKNOWN%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522clxddfhdw00062e6e3lye9ae7%2522%252C%2522sizes%2522%253A%255B100%252C0%255D%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522EDITOR%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522EDITOR%2522%252C%2522id%2522%253A%2522clxddfhdw00022e6ehboiipb8%2522%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522direction%2522%253A%2522horizontal%2522%252C%2522id%2522%253A%2522SHELLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522SHELLS%2522%252C%2522id%2522%253A%2522clxddfhdw00032e6epk5fx4i7%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%257D%252C%257B%2522type%2522%253A%2522PANEL_GROUP%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522direction%2522%253A%2522vertical%2522%252C%2522id%2522%253A%2522DEVTOOLS%2522%252C%2522panels%2522%253A%255B%257B%2522type%2522%253A%2522PANEL%2522%252C%2522contentType%2522%253A%2522DEVTOOLS%2522%252C%2522id%2522%253A%2522clxddfhdw00052e6efzk542jj%2522%257D%255D%252C%2522sizes%2522%253A%255B100%255D%257D%255D%252C%2522sizes%2522%253A%255B50%252C50%255D%257D%252C%2522tabbedPanels%2522%253A%257B%2522clxddfhdw00022e6ehboiipb8%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clxddfhdw00012e6ezo0k4uds%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522FILE%2522%252C%2522filepath%2522%253A%2522%252Fsrc%252Findex.ts%2522%252C%2522state%2522%253A%2522IDLE%2522%252C%2522initialSelections%2522%253A%255B%257B%2522startLineNumber%2522%253A6%252C%2522startColumn%2522%253A1%252C%2522endLineNumber%2522%253A6%252C%2522endColumn%2522%253A1%257D%255D%257D%255D%252C%2522id%2522%253A%2522clxddfhdw00022e6ehboiipb8%2522%252C%2522activeTabId%2522%253A%2522clxddfhdw00012e6ezo0k4uds%2522%257D%252C%2522clxddfhdw00052e6efzk542jj%2522%253A%257B%2522tabs%2522%253A%255B%257B%2522id%2522%253A%2522clxddfhdw00042e6ewmk74u5l%2522%252C%2522mode%2522%253A%2522permanent%2522%252C%2522type%2522%253A%2522UNASSIGNED_PORT%2522%252C%2522port%2522%253A0%252C%2522path%2522%253A%2522%252F%2522%257D%255D%252C%2522id%2522%253A%2522clxddfhdw00052e6efzk542jj%2522%252C%2522activeTabId%2522%253A%2522clxddfhdw00042e6ewmk74u5l%2522%257D%252C%2522clxddfhdw00032e6epk5fx4i7%2522%253A%257B%2522tabs%2522%253A%255B%255D%252C%2522id%2522%253A%2522clxddfhdw00032e6epk5fx4i7%2522%257D%257D%252C%2522showDevtools%2522%253Atrue%252C%2522showShells%2522%253Afalse%252C%2522showSidebar%2522%253Atrue%252C%2522sidebarPanelSize%2522%253A15%257D
(typescript error don't show)
To Reproduce
Steps to reproduce the behavior:
type B = Immutable<{ arr: number[] }>;
const b1: B = {
arr: [1, 2, 3]};
const b2: B = {
arr: [4, 5, 6]};
produce(b1, draft => {
draft.arr = b2.arr;});
Observed behavior
Typescript refuses the code
Expected behavior
Typescript should accept the code
Environment
We only accept bug reports against the latest Immer version.

Immer version:
I filed this report against the latest version of Immer
Occurs with setUseProxies(true)
Occurs with setUseProxies(false) (ES5 only)

—
Reply to this email directly, view it on GitHub
https://github.com/immerjs/immer/issues/1127, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAN4NBFY6V6UE5HH6M7XRTDZHGXCPAVCNFSM6AAAAABJISRW32VHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TCMZWGY2DEMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


update syntax for terraform 0.12

Quick PR to update syntax for terraform 0.12..

cc: @toVersus just in case

Thank you for the modules!
I made this PR since I'm using: baikonur-oss/terraform-aws-fargate-scheduled-task which pulls this module in. I notice that the other module has already been updated to 0.12 syntax. I was looking for a way to override this module to get rid of the warnings but it doesn't look like it would be possible for me to do without forking terraform-aws-fargate-scheduled-task so I settled on opening this PR and hoping for the best :)

I will make sure fargate-scheduled-task will be updated as soon as possible after v2.x release on this module!

Hey @roobert ,
We released v2.0.0 on this module and well as fargate-scheduled-task v2.0.2 (using iam-nofile v2.0.0). See: fargate-scheduled-task v2.0.2 on Github and Registry. Happy terraforming!

Thanks so much @prog893! Already updated and it's working perfectly. Thanks again for the great modules!


LEAN-3302

Remove unused listing/listing_element views, and the related codemirror code.

Remove unused listing/listing_element views, and the related codemirror code.

are we going to remove that node from PM document?

Is this something we will never need or we will reimplement it later without json-schema?

There are no plans to re-implement this logic any time soon. It's a bit early to get rid of the nodes from the schema, but I think we're headed in that direction.


[xhamster] Add HTTPS support

xHamster can be browsed in HTTPS even though the downloads are HTTP. This patch adds support for the HTTPS URLs.

Thanks.


URL error in examples

Since the wiki in the readme leads to the outdated repo this was forked from, I checked the examples for some information on how to use this lib. However in both the inventory and list-catchable-pokemon examples, the error Parameter "url" must be a string, not object appears quite often. The inventory example returns many errors (including the URL one), however the list-catchable-pokemon example only has the URL error.
I found that the error only occurs in the list-catchable-pokemon while attempting to get the map objects:
let mapObjects = await client.getMapObjects(cellIDs, Array(cellIDs.length).fill(0));

Did you run example from this repo or from the outdated one?

The examples are from this repo, not the outdated one.

I can't reproduce it

You tagged this as more info needed but haven't told me what information you need. So then I'll just take a guess.
I am running Node version 8.1.3 with NPM version 5.3.0 on a Windows 10 machine. My steps I took are as follows:

Install both pogobuf-vnext and node-pogo-protos-vnext
Copy the example from https://github.com/pogosandbox/pogobuf/blob/master/examples/list-catchable-pokemon.js
Replace lat and lng with my current lat and lng
I removed  hashingKey: 'hash key' and set useHashingServer to false, since I cannot find any information here on where to find/make a hashing key or what the hashing server even is
Run the example

ok, thanks for the info.
unfortunatly, there is currently no way to use the api without a (paid) hash key, so that is why it's not working.
Here is where you can find a hash key (I'm not affiliate with them, and yes it's a monopoly right now): https://hashing.pogodev.org

Damn okay, thanks for the help.


Python 2.7 system permissions error in python 3.6 environment

I'm having permission denied issues when trying to run pipenv install --dev within the shell of my python3.6 virtual environment.
I installed pipenv using the following command.
$ pip3 install --user pipenv

Collecting pipenv
  Downloading pipenv-5.3.3-py3-none-any.whl (1.2MB)
    100% |████████████████████████████████| 1.2MB 6.2MB/s
Requirement already satisfied: pew>=0.1.26 in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: pip-tools>=1.9.0 in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: semver in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: pip in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: setuptools>=36.3.0 in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: virtualenv in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: background in /home/user/.local/lib/python3.5/site-packages (from pipenv)
Requirement already satisfied: virtualenv-clone>=0.2.5 in /home/user/.local/lib/python3.5/site-packages (from pew>=0.1.26->pipenv)
Requirement already satisfied: pythonz-bd>=1.10.2 in /home/user/.local/lib/python3.5/site-packages (from pew>=0.1.26->pipenv)
Requirement already satisfied: click>=6 in /home/user/.local/lib/python3.5/site-packages (from pip-tools>=1.9.0->pipenv)
Requirement already satisfied: first in /home/user/.local/lib/python3.5/site-packages (from pip-tools>=1.9.0->pipenv)
Requirement already satisfied: six in /home/user/.local/lib/python3.5/site-packages (from pip-tools>=1.9.0->pipenv)
Requirement already satisfied: futures in /home/user/.local/lib/python3.5/site-packages (from background->pipenv)
Requirement already satisfied: resumable-urlretrieve; python_version == "3.5" in /home/user/.local/lib/python3.5/site-packages (from pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Requirement already satisfied: requests in /home/user/.local/lib/python3.5/site-packages (from resumable-urlretrieve; python_version == "3.5"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Requirement already satisfied: certifi>=2017.4.17 in /home/user/.local/lib/python3.5/site-packages (from requests->resumable-urlretrieve; python_version == "3.5"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Requirement already satisfied: idna<2.7,>=2.5 in /home/user/.local/lib/python3.5/site-packages (from requests->resumable-urlretrieve; python_version == "3.5"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Requirement already satisfied: urllib3<1.23,>=1.21.1 in /home/user/.local/lib/python3.5/site-packages (from requests->resumable-urlretrieve; python_version == "3.5"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Requirement already satisfied: chardet<3.1.0,>=3.0.2 in /home/user/.local/lib/python3.5/site-packages (from requests->resumable-urlretrieve; python_version == "3.5"->pythonz-bd>=1.10.2->pew>=0.1.26->pipenv)
Installing collected packages: pipenv
Successfully installed pipenv-5.3.3

Here are the steps I took to create the virtual environment.

First I created a virtual environment using python3.6 on my path

$ pipenv --python python3.6

Creating a virtualenv for this project...
Using python3.6 to create virtualenv...
⠋Running virtualenv with interpreter /usr/bin/python3.6
Using base prefix '/usr'
New python executable in /home/user/.local/share/virtualenvs/my-project-ySf4NbVn/bin/python3.6
Also creating executable in /home/user/.local/share/virtualenvs/my-project-ySf4NbVn/bin/python
Installing setuptools, pip, wheel...done.

Virtualenv location: /home/user/.local/share/virtualenvs/my-project-ySf4NbVn

Then I activate the created virtual environment

$ pipenv shell

Launching subshell in virtual environment. Type 'exit' or 'Ctrl+D' to return.

Then I try to install packages and development packages

$ pipenv install --dev

No package provided, installing all dependencies.
Pipfile found at /home/user/Documents/Github/my-project/Pipfile. Considering this to be the project home.
Installing dependencies from Pipfile.lock...
An error occured while installing! 3/24 - 00:00:07
Collecting babel==2.5.0
  Using cached Babel-2.5.0-py2.py3-none-any.whl
Requirement already satisfied: pytz>=0a in /home/user/.local/lib/python2.7/site-packages (from babel==2.5.0)
Installing collected packages: babel
  Found existing installation: Babel 2.4.0
    Uninstalling Babel-2.4.0:
      Successfully uninstalled Babel-2.4.0
  Rolling back uninstall of Babel

Exception:
Traceback (most recent call last):
  File "/home/user/.local/lib/python2.7/site-packages/pip/basecommand.py", line 215, in main
    status = self.run(options, args)
  File "/home/user/.local/lib/python2.7/site-packages/pip/commands/install.py", line 342, in run
    prefix=options.prefix_path,
  File "/home/user/.local/lib/python2.7/site-packages/pip/req/req_set.py", line 784, in install
    **kwargs
  File "/home/user/.local/lib/python2.7/site-packages/pip/req/req_install.py", line 851, in install
    self.move_wheel_files(self.source_dir, root=root, prefix=prefix)
  File "/home/user/.local/lib/python2.7/site-packages/pip/req/req_install.py", line 1064, in move_wheel_files
    isolated=self.isolated,
  File "/home/user/.local/lib/python2.7/site-packages/pip/wheel.py", line 345, in move_wheel_files
    clobber(source, lib_dir, True)
  File "/home/user/.local/lib/python2.7/site-packages/pip/wheel.py", line 316, in clobber
    ensure_dir(destdir)
  File "/home/user/.local/lib/python2.7/site-packages/pip/utils/__init__.py", line 83, in ensure_dir
    os.makedirs(path)
  File "/usr/lib/python2.7/os.py", line 157, in makedirs
    mkdir(name, mode)
OSError: [Errno 13] Permission denied: '/usr/local/lib/python2.7/dist-packages/babel'

[================================] 3/24 - 00:00:02

However, if I try and install the packages outside of the subshell, it installs the packages fine.
I don't understand why the pip module is trying to place things inside the system's python2.7 packages when I am running pipenv within the virtual environment's subshell.
Here is a copy of the Pipfile if you need it.
[[source]]
url = "https://pypi.python.org/simple"
verify_ssl = true

[packages]
jsonschema = ">=2.6.0"

[dev-packages]
flake8 = "*"
Sphinx = ">=1.6.1"
sphinx-readable-theme = "*"
sphinx-autodoc-typehints = "*"

sounds like you're not getting into the subshell quite right

do you set any python environment variables in your .profile or similar?

I have ~/.local/bin/ in my fish config, and it seems to come before the virtual environment's directory within the environment's subshell.
# my config.fish
set PATH ~/.local/bin $PATH
set PATH ~/Software/studio-3t/bin $PATH

$ echo $PATH
/home/user/Software/studio-3t/bin /home/user/.local/bin /home/user/.local/share/virtualenvs/my-project-ySf4NbVn/bin /home/user/Software/studio-3t/bin /home/user/.local/bin /home/user/Software/studio-3t/bin /home/user/.local/bin /usr/local/sbin /usr/local/bin /usr/sbin /usr/bin /sbin /bin /usr/games /usr/local/games /snap/bin

Problem is (i'm guessing) that pipenv prepends it's  environment to the path before starting the subshell, which is where fish prepends its paths defined within the config.fish.
Since pip also lives in ~/.local/bin pipenv is using the incorrect pip for package installation.

make sure that those parts are only running during a login session:

if status --is-login
    set PATH ~/.local/bin $PATH
    set PATH ~/Software/studio-3t/bin $PATH
end

Yeah, I should have realized that. Thanks!

@stephen-bunn did that resolve your issue?

Yes, fish now doesn't re-add those directories after jumping into the subshell. So now installations go through the correct pip and succeed.


[feature] Send email to contributors added to a project

Purpose
Addresses #3369 -- send users an email when they are added to an OSF project or component.
Changes

When a contributor with an OSF account is added to a project or component, send an email notifying them that they have been added:
If the user is added to a project and several of it's components at once, the user will only receive one email.
If they get added to another component later, they will receive an email.
Add a contributor_record field to the Node model to keep track of the last time each contributor was sent an email for throttling.

Tagged this as WIP, since I'm still trying to fix a failing test.

Before this is merged, we will probably want a way to opt out of these emails.

Related to the previous comment: this doesn't seem to use the existing infrastructure for user notifications--is there a reason for that?

I went with this implementation because I thought we always wanted to let users know when they have been added to a project. If we want to make it optional, then adding it as a "user notification" option would make sense.

I know there are some unresolved questions, but in the current state, I see no "save" button:

nevermind ... not part of this pr... see it with regular develop as well.

@GaryKriebel @lbanner @JeffSpies The desired behavior for this feature isn't well-defined. In particular:

Should you be able to opt-out of these emails?
If you are able to opt-out, should these notifications be enabled by default? Should they be enabled for existing users (will likely require a migration)?

How should we proceed?

@sloria and @samanehsan ... I suggest a proposal -- we should have these questions reviewed by community and @jlcohoon , and that might be the most efficient approach(?) It doesn't need to be extensive, rather just target the purpose/goals and use questions.

I support that plan. Would @samanehsan write up the feature as is and we'd have someone review for feedback?

Sounds good @jlcohoon @lbanner

@lbanner @GaryKriebel What is the status on this proposal?

Proposal has been reviewed ... is good to go / questions answered ... it is on @samanehsan 's priority list post 0.43.0 completion.

@sloria, the conclusion from the proposal was that users should always receive these emails. @JeffSpies and @bnosek suggested trying to send fewer emails instead of one per project/component, so I'll take a look at that post 0.43.

Now if a user is added to a project/component and several of it's child components at once, the user only get's one email. If they are added to a different child component in that project later on they will get an email. This is just like inviting unregistered contributors -- you don't want to send them multiple invites at once.

@sloria, this is ready for another pass! :sunglasses:

Confirmed that adding a user to a project and several children of that project results in only 1 email.
Looks good.
Ping @lyndsysimon for review.

Potential bug found - steps to reproduce:

Create a new project
Add a second user to the project.
Remove the second user from the project.
Add the second user back to the project.

At this point, the server returns a 400, and this error message is displayed:

I believe this is caused by the throttling, which prevents the same email from being repeatedly sent.

@lyndsysimon, I thought that was the purpose of throttling. How do you think it should behave instead?

Perhaps it shouldn't send the email when the user is added a second time, but you should be able to add the user back. As it is, the user on the admin page has no indication of why adding the user failed.
Also, as it is, if you add a user then accidentally remove them, you must wait 24 hours to add them again - at which point, it will send another email as well.

First pass complete.

Done making those changes @lyndsysimon! :sunflower:  Should I resolve the merge conflicts now as well?

The notify_contributor function saves the node after adding the time the email was sent. But this may result in confusion because even if the save flag in add_contributor is False, the node will still get saved when notify_contributor is called. @lyndsysimon, any suggestions on how to address this?

I'm going to have to defer to @sloria on this one.
You could pass the save param through to notify_contributor, but I'm unsure that's the right thing to do, since that function sends an email. The ability to send an email without keeping record of it on the Node object seems brittle to me.
Aside from that question, this looks great to me.

A rather minor consideration, but does sending an email for every call to add_contributor significantly slow down the tests suite? If so, we might want to disconnect the signal in the tests or discuss ways in which we opt in to signal subscriptions when we initialize the app.

Re: saving the node in notify_contributor, I think it's all right i this case to always save since it is necessary to save the email metadata after the email is sent (though note from my previous comment that this metadata probably belongs on the user rather than the Node).

Pass finished. :clap:

@sloria, several tests are failing because contributors are added to a project before it is saved for the first time. I could fix each one individually (either by saving the project before adding contributors or disconnecting the signal), or I could disconnect the signal for all tests. What do you think?

@samanehsan I don't feel strongly about either option, but I think disconnecting the signal might be the way to go, if only because it will allow the tests to be run without a bunch of "fake" emails being sent. We might want to do something like:
class OsfTestCase(TestCase):
    DISCONNECTED_SIGNALS = [...]

so that we can easily configure which signals are disconnected.

Done making changes @sloria :partly_sunny:

Pass finished.

Done making those changes @sloria :sunny:

One more small change. :beetle:

@sloria, this is ready for review! :bulb:

This is good to merge. I will merge once I sort out our release branches.

Nice! :+1:

Awesome! :tada:


Fixed typo in docs/topics/i18n/translation.txt.

Found this small inaccuracy while consulting docs

@beikeni Thanks :+1: Welcome aboard :boat:


Ferrodri/agora 1627 mobile UI fixes

Preview link: https://agora-next-a80ap55do-voteagora.vercel.app/
Fixes UI width error in proposals and other minor UI mobile fixes

PR-Codex overview
The focus of this PR is to improve the layout and styling of various components in the application.
Detailed summary

Added word-break: break-all to proposalVotesList.module.scss.
Updated styling classes in RetroPGFApplicationContentFundingSource.tsx.
Adjusted styling classes in ApprovalProposalVotesList.tsx.
Removed unnecessary import and adjusted styling in ApprovedTransactions.jsx.
Enhanced layout in RetroPGFApplicationContent.tsx.
Improved styling in ProposalsList.jsx.

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

@ferrodri can you add a screenshot of the before and after? Hard to know what changed here. thanks!

Fantastic, thank you! This is a great fix. The ideal modal web implementation is what we have on nounsagora.com. See below. feel free to address this either in this PR or as a separate item later
https://github.com/voteagora/agora-next/assets/5835973/278f1601-5adb-4f5f-a830-6b163b42a523

Fantastic, thank you! This is a great fix. The ideal modal web implementation is what we have on nounsagora.com. See below. feel free to address this either in this PR or as a separate item later
CleanShot.2024-02-26.at.21.17.58.mp4

Great, I just created this issue https://linear.app/agora-app/issue/AGORA-1655/improve-modal-implementation-for-proposals-like-in-nounsagoracom so I can get this merged and go for that later one, thank you!


Text input component resizes

Hi,
on start of the template font size is defined (10) and then it is used in text input component but when I publish the app and run it or play it, text input increases its height so it becomes really big. So I figured out that if I just change Font size to 8 and then change it back to 10  this will be fixed.

It is some kind of a bug

Can't reproduce, probably fixed through a new PowerApps version


feature: test circleci whether to work

Signed-off-by: Allen Sun<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

It works in circleci, closing this PR.


Why cant Github Copilot just ignore bad language?

Type: Feature Request
Do i have to behave like a 12-year-old in my own dev environment when working with GitHub copilot?
Why in god's name can't we have gated adult areas to express our adult desires?
Instead of protecting & safeguarding adult only areas, blatantly censor kinky communities. ToS Censorship?  where builders & creators lose years of content/work cuss someone somewhere decided PC isn't PC, being awake is woke & the PC MASTER RACE is less addicting than sissy Hypnos on Porn tubes... Sorry, im trans. my entriere gender identity is 50/50 a fetish & psychiatric reality.
im building an AIGF. Adults only! Yes, it's for me & my community.
Yes, MS im using guardrails. No MS, im not stealing content. No MS, im not using Copilots to generate it. No MS, im not jailbreaking.  Yes, im using both Copilots. Yes MS, i got both yearly. Yes Ms, they are the best to code & code & code.
I spend hours focused on the single task architecting bambisleep.chat instead of battling ads & browsers & guides & angry comments.
So, what i ask is this:  GitHub copilot chat window: #ignore: role.json (ignores the files role.json for adult language.)
wait? can i do this myself?
kind regards
melkanea
Extension version: 0.22.2024100702
VS Code version: Code 1.94.2 (384ff7382de624fb94dbaf6da11977bba1ecd427, 2024-10-09T16:08:44.566Z)
OS version: Windows_NT x64 10.0.22631
Modes:

const fs = require('fs');
const path = require('path');

// Directory to scan
const directoryPath = path.join(__dirname, 'your-codebase-directory');

// Function to read and process files
function processFiles(dir) {
  fs.readdir(dir, (err, files) => {
    if (err) {
      return console.log('Unable to scan directory: ' + err);
    }

    files.forEach((file) => {
      const filePath = path.join(dir, file);

      fs.stat(filePath, (err, stat) => {
        if (err) {
          return console.log('Unable to stat file: ' + err);
        }

        if (stat.isDirectory()) {
          processFiles(filePath);
        } else {
          fs.readFile(filePath, 'utf8', (err, data) => {
            if (err) {
              return console.log('Unable to read file: ' + err);
            }

            if (data.includes(`#ignore: ${file}`)) {
              console.log(`Ignoring file: ${filePath}`);
            } else {
              // Process the file as needed
              console.log(`Processing file: ${filePath}`);
            }
          });
        }
      });
    });
  });
}

// Start processing files
processFiles(directoryPath);

Thanks for creating this issue! We figured it's covering the same as another one we already have: https://github.com/microsoft/vscode-copilot-release/issues/2625. Please refer to that one for updates and discussions. Feel free to open a new issue if you think this is a different problem.


[CPP Graph] Fused Attention Doc

Type of Change: Doc
Description
Add doc for jblas-based fused attention.
Expected Behavior & Potential Risk
N/A
How has this PR been tested?
Spell check CI
Dependency Change?
No

@intellinjun @CeciliaWwq @zhenwei-intel @Zhenzhong1 Since MHA is quite vital for all coming features like TP and streaming, model owners please have a look and raise any questions you have.


Can I create a cropped image without user behavior?

I have a gallery-like component and I'm already using react-zoom-pan-pinch to get the positionX and positionY from the cropped image! I would like to create a cropped image using this library, giving only the neeeded information: scale, positionXY, etc... Is that possible?
For reference, I made this stackoverflow question: https://stackoverflow.com/questions/64286230/how-i-could-modify-a-image-by-zooming-in-and-out/64287333#64287333

Oh, I found another library that fits better with my needs. react-easy-crop. Client requests...


I have just down loaded and started using Home Assistant 0.24.1  I have it on a Raspberry PI 2 and I loaded it using the Raspberry all in one install.  Works great but when I go to Customise an item I get an error saying it can not find component Customize.  Help Please.

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
Python release (python3 --version):
Component/platform:
Description of problem:
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Please use the forum for these questions :)
https://community.home-assistant.io/


Cookie IsEssential

I had to add IsEssential = true to the CookieOptions in order to get all browsers to use the cookie.

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

ID: 47b14f57-82ac-a2e2-cbc7-22a81a60f4ac
Version Independent ID: bffca13c-223f-c61f-9cb2-9da8811eecfa
Content: Prevent Cross-Site Request Forgery (XSRF/CSRF) attacks in ASP.NET Core
Content Source: aspnetcore/security/anti-request-forgery.md
Product: aspnet-core
Technology: aspnetcore-security
GitHub Login: @steve-smith
Microsoft Alias: riande

That applies to ASP.NET Core 2.1 templates


Positive Feedback

All I really want to do is say "thank you!"
I come back to Dart about once a year after getting supremely annoyed with JavaScript. I'm very grateful that everyone in this project has accomplished these things in Atom. I was ready to download Eclipse and dive into that again, until I saw Atom plugins available.
I appreciate the care put into the tutorial page: suggested plugins, site layout; and I love that you all are bringing the features of Eclipse into Atom (the features that made Eclipse important to development since at least 10 years ago). I realize Dart makes those features a staple with the SDK, which is very good, but putting them in an editor is another problem.
I didn't want to make an "issue", but that's where I was told to direct feedback.
Thanks everyone!

Thanks for the feedback! Much appreciated.


chore: setup the project

Following changes are added by this pull_request:

Set up react project with pnpm as package manager
added prettier and eslint scripts for formatting and workflows

@Shurtu-gal , please review

@all-contributors add @Srish-ty for maintainence, code and idea.


Remove Server Explorer Azure debug attach command

Are we dropping support for debugging python on azure?

No, this is only removing the convenience command for attaching by clicking on app services in server explorer. They are removing their server explorer integration.
Debugging can still be done via the standard debug attach dialog, you just need to know the URL to use.


Updated readme.md to fix formatting issues

Hi @amitbapat, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.azure.com.

TTYL, AZPRBOT;

I am Microsoft employee, so I wonder why CLA is required?

CC @pomortaz

@hovsepm and @amitbapat thanks! LGTM.


End Translation files with a new line.

First of all, great tool. Without it I don't think translating angular application would be doable.
Files without a trailing newline are rather annoying in git diffs. Since I eventually want to run this in CI, minimizing diffs, would help us quite a bit.

Any news on this?

@firehist Are you still maintaining this?


fix: checkSession handles concurrent refresh token

Issue https://github.com/mitchelvanbever/remix-auth-supabase/issues/10 describes a use case where checkSession fails because of simultaneous calls to refresh token.

Use case :
├── invoices
│   ├── $id.js
│   └── index.js
└── invoices.js

If we use checkSession in $id and index loaders, both trigger refresh token.
Refresh token can be called once with the same session.refresh_token.

This PR add pendingHandleRefreshToken state in strategy instance.
Locally tested with a supabase JWT expiration of 10s.
@mitchelvanbever I don't know how to cover this use case in test :s if you have an idea, tell me :)
It takes me more time trying to test than coding the fix 😂

Need to test https://github.com/mitchelvanbever/remix-auth-supabase/issues/10#issuecomment-1025112231 :D

This seems to work great for me now btw.
FYI, I tested using oauth and note that this line deletes the refresh token from the auth.refresh_tokens table, which invalidates that same refresh token in the session. Should I open a separate issue/PR, or would you like to fix it in the same go?

I think I can remove this, it's not part of the lib.
Nice to see that 👍
I did that because supabase keeps a session in browser 🥲 and I didn't want that. Maybe I'm too suspicious 😅

Add an other way to remove supabase local storage session.
You may ask why :D and the reason is to make front side unaware about supabase session, outside of strategy CookieSessionStorage

I've been digging around this weekend but haven't gotten around to write a test to confirm everything should be working as expected and like you also said; It takes me more time to setup environment than to actually mess around with the code.
From what I can tell the fix is solid!
I just really want to add an accompanying test case for it, so we ensure to keep things consistent and won't break in future releases.

@jaPajaap Hello, just to warn you that there is a potential bug in the sample (onAuthStateChange listener always running, and Supabase sdk auto refresh).
I'v updated the demo ;)

@mitchelvanbever  Hey 👋 I found a way to test my case 🥳
What do you think about that ?


Add reportUserProfile support

This PR adds reportUserProfile method with predefined and custom attributes. Also, a bug in activation cofig fixed on iOS.

UPDATE: gradle config updated:

to allow a library user to define Android SDK version (it is usually placed in rootProject.ext, see example in RN community https://github.com/react-native-community/react-native-svg/blob/master/android/build.gradle)
bump target SDK version to the minimum required by Google witch is 26 for now (https://developer.android.com/distribute/best-practices/develop/target-sdk)


'currentStyle' get null on IE10

Hello,
I'm facing a problem in flexibility.js in which 'currentStyle' get null just on IE10. It does not happen on IE11 or Chrome, Firefox.
Does this library support IE10? Or it it a bug?
I have also attach the screen capture.
Thanks.

Hi @maihoanglai flexibility is AFAIK intended for IE 8 and 9. You can load it between conditional comments:
<!--[if IE]>
<script src="./flexibility.js">
<![endif]-->

if IE is enough, since IE10+ don't understand conditional comments.
For IE10, you can use PostCSS and Autoprefixer.


feat: allow --provider for all git commands (and add GIT_PROVIDER env)

added --provider flag to all git repository based commands as those will fail otherwise when autodetection doesn't work (e.g. internal git repository)

The City of Davenport needs this feature 🙏 Thanks for this great tool and you guys rock!!

as i commented in #290 - the --provider is only required when creating a new repository (using provider api calls). not when performing any of the other operations (which are git-only, and provider-agnostic).
please let me know if you think i should re-open this PR or the original issue.


Canonical form: Jumbled order of nth weekdays

This parser outputs a canonical form of the parsed opening hours string for better readability. However, there seems to be a bug for nth weekdays.
E.g. "all Sundays except the 2nd Sunday of each month": Su[1,3,4,5] is changed to Su[1,5,4,3].
This does not increase readability, actually, why change the order at all?

Without actually checking, I'm fairly sure that us an artifact of how the numbers are collected during parsing and it is just missing a Collections.reverse at some point.


Personal Downloader Example

I keep trying to use the personal self bot example and I always get an error, I'm trying to get two channels to download to separate folders.

Can you show the error you're getting as well as your config with login/sensitive info removed?

{
"_constants": null,
"credentials": {
"token": "__token",
"flickrApiKey": "__flickr_api_key"
},
"admins": [ "__USER_ME" ],
"adminChannels": [
{
"channel": "__CHANNEL_P_DDG_UBADMIN"
}
],
"overwriteTwitterPath": "/home/bots/shared/twitter_cookies.json",
"overwriteInstagramPath": "/home/bots/shared/instagram_cookies.json",
"debug": true,
"allowSkipping": true,
"scanOwnMessages": false,
"autoHistory": true,
"githubUpdateChecking": false,
"presenceEnabled": false,
"exitOnBadConnection": false,
"presenceStatus": "invisible",
"channels": [
{
"channels": [
"__CHANNEL_MEMESERVER1_MEMES",
"__CHANNEL_MEMESERVER2_MEMES",
"__CHANNEL_MEMESERVER3_SHITPOSTING",
"__CHANNEL_MEMESERVER4_MEMES"
],
"destination": "/var/www/html/files/ddg/memes",
"enabled": true,
"allowCommands": false,
"sendErrorMessages": false,
"scanEdits": true,
        "presenceEnabled": false,
        "reactWhenDownloaded": false,

        "subfolders": [
            "{{serverName}}"
        ],
    },
    {
        "channels": [
            "__CHANNEL_MEMESERVER2_NSFW_MEMES",
            "__CHANNEL_MEMESERVER4_NSFW_MEMES"
         ],
        "destination": "/var/www/html/files/ddg/nsfw",
        "enabled": true,
        "allowCommands": false,
        "sendErrorMessages": false,
        "scanEdits": true,

        "presenceEnabled": false,
        "reactWhenDownloaded": false,

        "subfolders": [
            "{{serverName}}"
        ],
    }
]

}

You have some trailing commas, those cause errors in json. The last object in a group can't have a comma. I think it's just your subfolder objects:

Thank you, of course its a comma, I got it from the example, it might be good to fix that.

Fixed it thanks, didn't realize you meant that section, I must have left that when I stripped my other settings out.


Add meta description to layout

Description
Add meta description to layout as optional field.  Update one how-to guide to test.

Deploy preview for calico ready!
Built with commit b011be476ba60fc80dc2a4291decb937084d1f91
https://deploy-preview-2867--calico.netlify.com

I've confirmed that the preview adds the meta description as expected when provided in the md (and not when it is absent from the md).
Ready for review and merge.


Initial QuestionAnsweringClient

Working on something new. Closing this since we won't use it, but will leave the branch up in case we need to revisit for any reason.


feat: Add support for using Product Name

This PR is based off the work here https://github.com/gitbutlerapp/tauri/pull/1 by @krivi. (most of this PR description was taken from there- it was very well written!)
Background:
This change in Tauri https://github.com/tauri-apps/tauri/pull/9375/files, which is noted as a refactor is in fact a breaking change as far as the app updater is concerned.
The impact is that because the binary now has a different name, it is not possible to smoothly update a Tauri application via the updater if the initial version of the app is using v1.x and the new version is using v2.x.
Approach:
I'm introducing here an option for the tauri-cli (tauri-bundler) for producing the same binary names it did in v1, allowing users to update the app via the updater. In order to enable this behavior, set the environment variable
V1_COMPATIBLE_BIN_NAME=true

Previously, the bundler did the rename just after building the binary. I poked and tried doing the same once again, but because quite a lot of the code in that bundler changed since that was merged, it felt possibly more error prone (since now the rest of the code can safely assume that the binary is the crate name)
So, instead, my approach is to do the rename the binary at the last moment (i.e. in the bundle_project function in eg. app.rs)
fixes #10548
Testing
This PR has been tested across Windows, MacOS, and Linux with the Modrinth App. You can find test builds built using this PR here: https://github.com/modrinth/code/actions/runs/10785759488

I am looking into all issues related to v1 -> v2 updater so bear with me until I get this all sorted out.

No worries- I fully tested this on Windows/MacOS/Linux and everything seems to be working as expected :)

I believe we can close this one now that we have #10939 #10996 #10977 #10969 #10967 #10962 #10952

yeah, that correct, this is not longer needed and if productName needs to be used, you can set the new mainBinaryName option to the same value as productName


feat(core): ensure KEL, ACDC, contact timestamps are used instead of record creation time

Description
In many places where we create records we are letting the createdAt timestamp be Date.now() but it should match the KERIA times.
Checklist before requesting a review
Issue ticket number and link

[x] This PR has a valid ticket number or issue: (https://cardanofoundation.atlassian.net/browse/DTIS-1417)

Testing & Validation

[x] This PR has been tested/validated in IOS, Android and browser.
[x] The code has been tested locally with test coverage match expectations.
[x] Added new Unit/Component testing (if relevant).

Security

[x] No secrets are being committed (i.e. credentials, PII)
[x] This PR does not have any significant security implications

Code Review

[x] There is no unused functionality or blocks of commented out code (otherwise, please explain below)
[x] In addition to this PR, all relevant documentation (e.g. Confluence) and architecture diagrams (e.g. Miro) were updated

In  the UI CreareIdentifier.tsx, we have the method handleCreateIdentifier that is creating the Identify record like this:
const newIdentifier: IdentifierShortDetails = {
        id: identifier,
        displayName: identifierData.displayName,
        createdAtUTC: new Date().toISOString(),
        theme: selectedTheme,
        isPending: isPending,
      };

Should we set the createdAtUTC time to the one provided by keripy instead of new Date()? @iFergal
const identifierDetail = await this.props.signifyClient.identifiers().get(identifier);
new Date(identifierDetail.state.dt)

@jimcase The state could be the datetime of a newer rotation event. But yes, this is what myself and Patrick mentioned this morning. I opened a PR just now on KERIA: https://github.com/WebOfTrust/keria/pull/335

@jimcase I also misread your comment. Yes, except Bao's current PR is changing it so that it's created via callbacks so we can consolidate the two PRs. @Sotatek-BaoHoanga and @Sotatek-TungNguyen2a please coordinate here