Fix warnings and some errors

Various small fixes resolving either warning/errors in the brower or during build.
9022ede fixes #353

@GregoryHamel This looks great!  Thanks especially for the fix to TimelineItemTitle.react.js

:tada: This PR is included in version 1.21.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add instructions on releasing a new version in README

Pull Request Test Coverage Report for Build 744

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.448%

Totals

Change from base Build 725:
0.0%

Covered Lines:
180

Relevant Lines:
181

💛  - Coveralls


pass compatibility for mounts?

For example, adding symlinks in the root store pointing to the mounted stores so that pass mount/foo works just like gopass mount/foo (and so that other GUIs work with mounts).

This is actually an interesting idea, but I'm not sure if this will work the same way as gopass mounts.
We'll look into it.


chore(deps): rebuilding because of lock file maintenance

chore(deps): lock file maintenance
chore: changes caused by updating lock files. See #612

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
462

Relevant Lines:
462

💛  - Coveralls


Add default team_id to WebClient constructor args

Summary
This pull request updates the Web API clients to hold the default team_id in each instance, aiming better support for Org-Wide App installation.
Category (place an x in each of the [ ])

[x] slack_sdk.web.WebClient (Web API client)
[ ] slack_sdk.webhook.WebhookClient (Incoming Webhook, response_url sender)
[ ] slack_sdk.models (UI component builders)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[ ] slack_sdk.rtm.RTMClient (RTM client)
[ ] slack_sdk.signature (Request Signature Verifier)
[ ] /docs-src (Documents, have you run ./docs.sh?)
[ ] /docs-src-v2 (Documents, have you run ./docs-v2.sh?)

Requirements (place an x in each [ ])

[ ] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[ ] I've read and agree to the Code of Conduct.
[ ] I've run python setup.py validate after making the changes.

I've verified all the unit tests and integration tests pass with the change. Let me merge this PR now.


Douai annotation underlines don't show up on Safari

Douai annotation underlines don't show up on Safari. They do show up on other browsers (we looked at Firefox and Chrome).

Nothing from @comesaignol ??? Ping???

@martindholmes: Let's just go ahead. @comesaignol must be busy with other things.

Fix committed in rev 18232. If all is well, close this.

Looks good! Closing now


Adds SkypeMiddleware to patch Skype mentions so RemoveMentionText works

Fixes #2181 Added SkypeMiddleware to patch Skype mentions so that RemoveMentionText can properly removed mention text.
ActivityExtensions.cs
Reverts commit 4daa9b9, with the addition of a Trim() added. That commit did not work for other channel mention types.
Microsoft.Bot.Builder\SkypeMiddleware.cs
Added middleware class SkypeMiddleware to remove the tags from Mention.Text for Skype mentions. This allows RemoveMentionText to properly remove user names from Activity.Text

Pull Request Test Coverage Report for Build 71670

0 of 0   changed or added relevant lines in 0 files are covered.
93 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.06%) to 77.12%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Builder/BotFrameworkAdapter.cs
93
26.67%

Totals

Change from base Build 71554:
0.06%

Covered Lines:
4975

Relevant Lines:
6451

💛  - Coveralls

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

Hey, issues are fixed. CAn you review again?

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.Luis.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.AI.QnA.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.ApplicationInsights.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Azure.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Dialogs.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.TemplateManager.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Configuration.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Connector.dll compared against version 4.3.1
:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Schema.dll compared against version 4.3.1


More Support on 1809

📝 Provide a description of the new feature
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
---All feature avalible on old versions such as 1809, 1709, and more
If you'd like to see this feature implemented, add a 👍 reaction to this post.

@Linyixuan10
1809 is used by less 1% of PowerToys users and soon it will not be supported anymore by Microsoft.
We are also planning to stop releasing updates for 1809 https://github.com/microsoft/PowerToys/issues/9417

soon it will not be supported anymore by Microsoft.
We are also planning to stop releasing updates for 1809 #9417

LTSC is 1809 and, like any other enterprise / long term channel, supported for 10 years from release which was in 2018. I guess you could say 2028 is "soon," but that'd be a little dishonest.


Global settings class for TonelReader/TonelWriter

We need a setting to define a way to define global settings that are going to be used by default when using the context menu or by all instances of TonelLoader/TonelWriter.
In these settings will be the preferred strategy classes, app suffix, user, etc.

This was closed by a4d26a1


In Sidebar File explorer, keyboard navigation focus is not visible.

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
5.3.2
Describe the bug
After using the "Reveal active file in navigation" command from the command palette, the the file or folder focused in the tree upon using the arrow keys to move around, is not made distinct.
A quick comparision of how it looks in the default theme vs how it looks in Minimal:
https://user-images.githubusercontent.com/71895835/188313416-41ba2498-769c-4731-bbc5-892b9ce2d950.mp4
Debug info
SYSTEM INFO:
Obsidian version: v0.15.9
Installer version: v0.15.6
Operating system: Windows 10 Enterprise 10.0.19044
Login status: not logged in
Insider build toggle: off
Live preview: on
Legacy editor: off
Base theme: light
Community theme: Minimal
Snippets enabled: 0
Restricted mode: on
RECOMMENDATIONS:
Custom theme: for cosmetic issues, please first try updating your theme to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.

Couldn't find a duplicate issue upon searching, but did find #353 when I narrowed down the lines of CSS that were causing the issues. Closing it for being a duplicate.


ESP32-CAM AI-Thinker green tint

Trying:
# camera construction and initialization
camera = Camera(pixel_format=PixelFormat.JPEG, frame_size=FrameSize.SXGA, jpeg_quality=85, xclk_freq=20000000, grab_mode=GrabMode.LATEST)

# to try to get rid of the green tint
camera.set_awb_gain(False)
camera.set_agc_gain(0)
camera.set_whitebal(0)

img = camera.capture()

doesn't seem to work.
I also tried taking 3 captures, which works for me on another distro.
Taking whitebal as an example in the previous distro there were 5 values.  I worked out that there is only true and false for this implementation.
A table showing the parameters values allowed would be helpful.

Have you already clicked on the link point to the circuitpython documentation in the readme?
There you will find a good description of the possible settings and their possible values. As mentioned also in the readme, you will need to append a "set_" to the respective property. For example: there is whitebal (true/false) and wb_mode (0-4).
I will also try to reproduce a green tint situation (I have it only on bad light conditions) and try some of the tricks from the other Threads and forums.

In my case, I just experience a low green tint in the initial frames, but after ~1s of video stream it disappears.
I used set the wb_mode to 3 or 4 (0 - Auto, 1 - Sunny, 2 - Cloudy, 3 - Office, 4 - Home) and I got rid of my green tint at all.
I can imagine that different targets may deliver different pictures (I have a S3 XIAO board with OV2640), and also camera sensors may have some aging, and of course everyone have a different environment and light around, so it is very difficult to fully reproduce the situation.

Documentation ... I did focus too much on the source code.  Thank you for the prompt.
The comment regarding set_ did I not do that properly in my snippet?  I also ran the get_ on each to verify that the changes had actually happened.
I was not aware that there was wb_mode as well as whitebal.
special_effect I think the numbers are:
0:,no effect 1:negative, 2:black and white, 3:reddish,  4:greenish, 5:blue, 6:retro
Appreciated your comments.

After quite a bit testing I get acceptable pictures with"
 # camera construction and initialization
camera = Camera(pixel_format=PixelFormat.JPEG, frame_size=FrameSize.SXGA, jpeg_quality=85, xclk_freq=20000000, grab_mode=GrabMode.LATEST)
#camera = Camera(pixel_format=PixelFormat.JPEG, frame_size=FrameSize.SXGA, jpeg_quality=85, grab_mode=GrabMode.WHEN-EMPTY)

camera.set_brightness(-2)

# take 10 frames, rejecting the first nine!!
for x in range(10):
    img = camera.capture()
    time.sleep_ms(50)

Happy to hear you solve the problem. Have fun!


Please provide ability to remove cache entry for a specific request

Here is my detailed request posted on StackOverflow.
http://stackoverflow.com/questions/29053156/how-to-tell-okhttpclient-to-ignore-cache-and-force-refresh-from-server

Add CacheControl.FORCE_NETWORK on your request object.

Does it evict the current cache entry first?
For example, if the call with FORCE_NETWORK fails, does any previously existing cached entry still stands?
My preference will be that it clears the current cached entry regardless if FORCE_NETWORK is set.

It'll overwrite the current entry with the new entry, if that is cacheable. If it isn't, the old, stale entry sticks around. I'll try to see if there's something better we can do here.

How do I do this with Retrofit since I don't have direct access to the request object?

This stackoverflow entry provide some clues:
http://stackoverflow.com/questions/25481403/does-retrofit-cache-the-authorization-requests
Do I set a requestInterceptor that adds the CacheControl? However, I don't see an ability to get to the OkHttp request object there.

any thoughts on how to do this with Retrofit?

@Headers("Cache-Control: no-cache") on the method should work. If you want to do it dynamically you can add a @Header("Cache-Control") String cacheControl parameter and pass null or "no-cache".

@Headers doesn't work for me since that will disable caching in all cases.
@Header method parameter looks like a good idea. I will try it out.

If I want to use FORCE_CACHE, what do I pass in the Cache-Control header?
pass "force-cache" instead of "no-cache" in here?
(@Header("Cache-Control") String cacheControl)

It would be great if OkClient provided access to the OkHttpRequest so that I can write my own Client.
In my client, I will just set request.cacheControl(CacheControl.FORCE_NETWORK)

@inder123 you can do that with an interceptor if you like.

How do I do it with an Interceptor? Interceptor does not expose any ability to set cacheControl. It only allows me to set headers. How do I access the underlying okhttp request object?

Update on this?
BTW, I saw that new OKHttp versions got released recently, and it seemed to be a little incompatible with Retrofit (OkClient needs to throw IOException for some methods).
so I presume a new retrofit is coming soon as well.
Is there a possibility of adding support for CacheControl in Retrofit interceptor?

@inder123 You can set cache control directly as a header. Cache-Control: max-age=0 for example!

@swankjesse  is max-age=0 equivalent to FORCE_CACHE or FORCE_NETWORK?
I understand how to do FORCE_NETWORK., However, I don't know how to do FORCE_CACHE.

To force a cache response you can do only-if-cached as a value. You can also add a very large max-age= value (separated by comma) if you want to allow a stale response (e.g., only-if-cached,max-age=31536000).

@JakeWharton That works for me. In our app, we have a requirement for an "offline" mode. Some countries, I think China, require user's permission for network access, even if WiFi.

No further action for us to take here.

ok


Hover contents could be single entry or multiple entries.

https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#textDocument_hover
This is a breaking change. Might need to upgrade version number as well. Just not sure which part of version should bump.

Thanks! Released as 0.13.


Editor Decoration Badges appear to be incorrect

Issue Type: Bug
My Git status is clean, but in the Editor Tab it appears to show some information as if the file has been modified. I do not know exactly if it is the git status, which should be displayed.

 And any entries on the Problems panel?

I have. I have tons of problems, as I am not bothered much about those reported by powershel Script Analyser
So the number should reflect the count of problems ? Does 9+ means more than 9 problems?
I have not figured out what the badges should say. I tend to deactivate and not care about them, but I care about VSCODE and that the features should work reliable.

So the number should reflect the count of problems ? Does 9+ means more than 9 problems?

Correct. If you don't want the editor tabs to show this information, toggle the workbench.editor.decorations.colors and workbench.editor.decorations.badges settings.

Related, see https://github.com/microsoft/vscode/issues/120345

Thanks, I close for now, until I can be more specific in my report


Fix issue in init command

Purpose

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/8272

:white_check_mark: Build ballerina-lang 28 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/74fa00aec6 by @NatashaWso2)

:white_check_mark: Build ballerina-lang 42 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/a1ea44c3cb by @NatashaWso2)

Closing this PR since this check is not needed.


Runchecker failures

Run checker failures since the beginning of March.
Currently failing (need to be fixed before beta) are:

no-asm #14980 logs indicate a time out during the tests, seems like a hang, confirmed that it works in master
enable-fuzz-afl no-shared no-module #14966 test_cmp_http fails

These have fixes in review:

no-dtls1_2 #14956, fixed by #15047
no-dtls1_2-method #14956, fixed by #15047

These worked but have been re-confirmed:

no-dso logs indicate that it runs currently which has been confirmed in master
no-pic logs indicate that it runs currently which has been confirmed in master
enable-asan no-shared -DOPENSSL_SMALL_FOOTPRINT logs indicate that it runs currently which has been confirmed in master

These have been fixed:

no-autoalginit #14960 make test fails to create fipsmodule.cnf, fixed by #14979
no-autoerrinit #14961 make test fails in test_err, fixed by  #14976
no-des #14958, fixed by #14975
no-ec2m #14959, fixed by #14977
no-sm2  #14962 logs indicate failure in test_evp which doesn't reproduce in master, possibly fixed by #14834
no-sm3  #14963 logs indicate failure in test_evp which doesn't reproduce in master, possibly fixed by #14834
no-sock #15054, fixed by #15055
no-stdio #14953 fixes this
no-stdio #15056 make test failing due to no apps,  fixed by #15057
no-tls1_2 #14964 test_sslextension fails, fixed by #14955
no-tls1_2-method #14964 test_sslextension fails, fixed by #14955
no-tls1_3 #14965 test_ssl_new fails, fixed by #14978

-d --strict-warnings no-tls1_2
-d --strict-warnings no-tls1_2-method

Should be fixed by #14955

-d --strict-warnings no-stdio

should be fixed by #14953

Adding additional run-checker jobs in openssl/tools#84

All the failing run-checker builds are good now.
The newly introduced builds can be handled elsewhere.


custom_profile: Convert to typed_endpoint.

Convert custom_profile_fields.py to use
typed_endpoint.
Use TypedDict from typing_extensions instead of typing,
to support Pydantic's type checking.

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged after fixing the above detail and cleaning up formatting in the commit message. In the future, it'd be good to spell out things like why the typing_extensions version of TypedDict is better for pydantic, since it's not obvious.


Server crash on startup

Version: QmunityLib-1.7.10-0.1.78-universal.jar
Crash: http://pastebin.com/YgeVBfHW on Server
What did you expect to happen: The server starts
What happened instead: It crashes

What version of Qmunity Lib do you get this on ? :)

Well, a new handcrafted version ;)

QmunityLib-1.7.10-0.1.78-universal ? :)

A self-compiled version, this crash is also in the latest curse version

we don't accept reports from custom compiles for obvious reasons.

Since I said the crash is also in the latest curse version.

Either way, I suggest you read/re-read the README.md so that you can report issues properly going forward.

@Quetzi I'm sorry.

Hmm, weird. I don't get this crash with only BluePower, QmunityLIb and Forge Multipart installed :/

It's a server crash.

I tested it on a server.. :3

@GirafiStudios Really?! It dosen't even work if I just install QmunityLib on a server.. :D

Weird o.O Is it a dedicated server, or on your own computer?

My own pc.

My client works fine.

That is very weird, because EntityClientPlayerMP is not even imported in that class.

Yeah, I already check it.

But it's only with QmunityLib

Its working fine in my dev env, with and without FMP

That crash error cannot just be QLib becuase it is being called  at codechicken.multipart.MultiPartRegistry. Try updating FMP?

He's using the latest version of FMP

Yep ;)

I think it was maybe fixed, thanks to @Quetzi but now there is another crash, but I'm closing this now.

other crash?

Yeah, hand-crafted BluePower version, if it's on curse I will create a issue :P


Fixed Spellings

Fixed some spelling errors throughout the readme.md files, some might be due to my local (en_GB) but most seem like little errors that are universal.

Some white-space was also removed automatically.

Thank you for the corrections.


Readme: fix a typo

Thank you for fixing this!

@ForNeVeR 3 reviewers for fixing a typo? Really? :)

Thanks guys, for the reviev :+1:


[Suggestion] Recipes start unlocked when doLimitedCrafting is true

Issue Description:
In my pack I want to use doLimitedCrafting but CT recipes are locked on creation and I want to have them unlocked by default.
What happens:
CT recipes start locked by default
What you expected to happen:
A config option or an optional parameter for scripts that changes this behaviour.
Script used (Please Pastebin or gist your script, posting an unpasted or ungist'd script will automatically close this issue):
Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue):

Affected Versions (Do not use "latest"):

Minecraft: 1.12.2
Forge: 2561
Crafttweaker: 4.0.10

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

we don't do anything special for this, like our recipes are added exactly the same as other mod recipes and "isHidden" returns false on our recipes, so I'm not sure how we can "fix" this


Review update to Puma latest stable version

We are using our own fork (https://github.com/3scale/puma) that solves a performance issue. That problem has been solved in upstream, so we should try to update.

I tried the latest release of Puma, v4.3.5. It turns out that the performance issue is still there. Also, our patch no longer works. I tried a few versions of Puma until I found the one that broke our patch, v3.9.0, more specifically, the PR #1278 in Puma.
In the latest version, applying our patch and reverting that PR seems to be enough to fix the performance issue. I still need to study that PR and understand why it has a negative impact in Apisonator. I'll post my findings here.

/me waiting for your findings, detective!

From my understanding, that PR changes how the Puma workers handle connections. In summary, instead of accepting requests and leaving them in a queue pending to be processed by the workers, now Puma only accepts a connection if there's a thread available. I guess that works in the majority of use cases, but not for ours because we only have 1 thread per worker.
In previous versions of Puma there was a "queue_requests" param to control this. Now it's present but it does no have any effect.
In practice what happens is that even with a moderate number of connections, many requests will time-out, which does not happen in previous releases with our patch nor when using Falcon.
What do you think @unleashed ? Should we update our fork to the latest version of Puma and revert that commit and/or add back the "queue_requests" param?

So there is no buffering of accepted requests, which is useful as peaks will happen and timeouts can be avoided by introducing a bit of latency. But then this breaks the clustered mode with single-thread processes, which could just be a bug unless the maintainers have decided Puma no longer supports this.
Yes, I think in the short term we will want to undo that and use our own handling. I don't know how much the reactor code has changed, but both of us were familiar with it and we can probably come up with a quick fix. But I'd say that if they haven't dropped support for clustered mode with single-thread processes this is a bug worth reporting.
Are you willing to give it a go?

I don't think the maintainers consider this a bug. In the description of the PR it's mentioned that several of the authors gathered in a conference and came up with this PR. I guess it improves things for the most typical deployment scenario, which is several workers with multiple threads for each.
In the PR someone already pointed out that the "queue_requests" param that I mentioned in a previous comment did no longer have any effect.

I tried the latest release, 5.0 and can confirm that the behaviour is the same.

The issue still exists, ok. So the next step is to port our fixes to 5.0 and try that out.

Updated to 4.3.7 in https://github.com/3scale/apisonator/pull/261


build: switch to release-please app

@enocom friendly nudge, let's make sure we figure out an approach for releasing in the new year that reduces your manual burden.

Sorry for the delay. I’ll get to this in the new year and come with questions.


Added normal mapping setting to Sprite3D

Hello, first time attempting a contribution here :)
This PR enables Sprite3D to have a configurable normal map texture.
It would come handy in a project of mine using 3D to render pixel art with day/night cycle, and I saw many people asking about this feature. Current workaround is using a quad mesh, but it can get cumbersome as it requires creating many materials and fine tune the mesh dimensions to get it right.
Bugsquad edit: This closes https://github.com/godotengine/godot-proposals/issues/3979.

Please give me a bit more time, as I am trying to figure out whether there is a way to do this more similar to the way it is done in Godot 3.x

This makes sense, although if this is the way to go, I feel we should have also ORM and emission textures.

Is it possible for this to be reviewed for 4.3?

If you mean 4.2 then we've passed feature freeze so it most likely won't be considered, 4.3 will be evaluated once 4.2 is released

No I meant 4.3 as 4.2 has passed feature freeze, hopefully the conflict can be resolved for then.

I assumed as reviews for 4.3 won't be done until then

If you mean 4.2 then we've passed feature freeze so it most likely won't be considered, 4.3 will be evaluated once 4.2 is released

No I meant 4.3 as 4.2 has passed feature freeze, hopefully the conflict can be resolved for then.

Looks like it won't make it to 4.3 neither :( really looking forward to have a Normal Map property at the Sprite3D and AnimatedSprite3D, the QuadMesh approach feels so 3.x :/

Looks like it won't make it to 4.3 neither

Nothing is certain for 4.3, we're not in freeze yet, but there's a lot of review comments that haven't been responded to and this needs a rebase, I'd say that responding to feedback will get this rolling faster

Hello and sorry for the late response. I'm sorry I can't maintain this PR because I've other priorities at the moment. If someone wants to take over this please don't hesitate!

If you mean 4.2 then we've passed feature freeze so it most likely won't be considered, 4.3 will be evaluated once 4.2 is released

No I meant 4.3 as 4.2 has passed feature freeze, hopefully the conflict can be resolved for then.

Looks like it won't make it to 4.3 neither :( really looking forward to have a Normal Map property at the Sprite3D and AnimatedSprite3D, the QuadMesh approach feels so 3.x :/

Is it possible to use the QuadMesh approach for animated sprites? Could you be so kind as to point me in the direction of any source about it?
Unfortunately 4.3 doesn't look very promising regarding this issue, and I can't find a good source for the famous "quadmesh approach" that explains how one would go about animating it.
I really need animated sprites with normal maps in a 3D environment, but I don't have enough knowledge to implement the fix myself.

Is it possible to use the QuadMesh approach for animated sprites? Could you be so kind as to point me in the direction of any source about it?

You can't use it with AnimatedSprite, but you can use it with a spritesheet that contains the animation by setting the UV1 offset/scale properties accordingly in the material (this can be automated with a script or AnimationPlayer).

Is it possible to use the QuadMesh approach for animated sprites? Could you be so kind as to point me in the direction of any source about it? Unfortunately 4.3 doesn't look very promising regarding this issue, and I can't find a good source for the famous "quadmesh approach" that explains how one would go about animating it. I really need animated sprites with normal maps in a 3D environment, but I don't have enough knowledge to implement the fix myself.

There is a short at YouTube from "FinePointCGI" with the title "Godot Quick Tip | Animating UVa Inside of Godot", that's the method I'm using right now.
I will probably create a extension to automate this process if I find time in the future but I don't know how helpful it will be as the use case is literally just us doing 2.5D projects and the nice is REALLY small lol

Up for 4.4?
it's been 2 years and a half 🧓🏻

Up for 4.4?
it's been 2 years and a half 🧓🏻

As discussed above, this still needs more work (beyond a rebase to solve merge conflicts), and the original author isn't available anymore. So this is waiting for a new contributor to pick it up to finish it. You're welcome to give it a try if you want.

I'm so sorry about that, @vxSpctr. I was toying with a small project at that time and felt the need to propose that feature. But I didn't realize the time it would require to review, rebase, and bring the code to the standard of the project. I'm in a situation right now where I'm not actively using Godot and working on a solo gamedev project (so time has become a luxury). Don't hesitate to take over this PR!


View connectivity dataset modal

PR looks fine, but deploy looks weird: https://siibra-explorer.apps.hbp.eu/fixconndatasetview

PR looks fine, but deploy looks weird: https://siibra-explorer.apps.hbp.eu/fixconndatasetview

For me it doesn't work at all:

PR looks fine, but deploy looks weird: https://siibra-explorer.apps.hbp.eu/fixconndatasetview

For me it doesn't work at all:

that's what I meant

ok found the bug. will fix


UI Issues

Rancher versions: master 2.0 8/16
Results:

[x] State without background looks like it's not aligned properly

[x] When a service is selected, the background of the scale buttons can't be seen

[x] In Stack Details, Scale need to be centered

[x] Add Container/Service - move Volumes and Upgrade policy under Show advanced options

[x] Add container/service - move Expand All to top

[x] Add container/service - Remove No Port text

[x] Container/service details page - Add No port text to Port Mapping section

[x] Container/Service details page - Port section is weird

[x] Service details page - Service log section needs updating

[x] Container/Service detail pages - Review Networking & Security/Host Config sections for the grey boxes (only some are in grey boxes and some aren't)

[x] Container/Service detail pages - Since fixing the connecting text to be centered, the stats are now too low.  I think they are connected?

[x] All the pages - the Title section is not aligned properly.  The title name is not aligning withe the buttons and the menu button is not aligned properly

[x] The status badge for Stacks is too big in both the list page and the details page

Version - master 2.0 8/22
Verified fixed


[SU-53] Allow deletion of Errored billing projects

I recommend using hermione.owner to test this, as that account has amassed a lot of errored projects over the years.
See UX considerations: https://broadworkbench.atlassian.net/browse/SU-53
List view:

New menu:

Deletion modal:

<!--
Hello, friend!
Remember to mention what sort of testing/verification you performed in the course of building this PR, i.e. checking functionality in the browser locally.
Also, so if a screen recording [1] and/or screenshots would be a helpful way to communicate context, please consider including some in your PR description.
Thanks!

[1] https://support.apple.com/guide/quicktime-player/record-your-screen-qtp97b08e666/mac
--!>

Why can you only delete failed ones? What if you wanted to delete a "healthy" billing project?
And can we just show a trash can (or some other delete icon) instead of the menu, given that there is only a single item?

It would be good to extend the billing projects test to do a deletion.

@cahrens the UX around deleting an active billing project is a much more nuanced implementation and the backend doesn't support it too well, particularly in the situation where there are workspaces in the project. I am not sure if it will ever be fully supported at this rate. This is merely being done as a courtesy to our longtime users who may have collected a bunch of these errored projects over the years, rendering their project lists unusable. We recently fixed Rawls support for this here but we don't want to force users to call the API directly.
And sure, the three dots icon was an explicit choice by Joy but I will double check with her about that versus the trash icon.

@cahrens I confirmed with UX that they would like this to remain a 3-dot icon even though there is only one option
I'll draft this PR until I add the requested test coverage


Fixing bug, updating code layout, adding native-image as well

Check List:

[X] Tests pass (./test.sh <username> shows no differences between expected and actual outputs)
[X] All formatting changes by the build are committed
[X] Your launch script is named calculate_average_<username>.sh (make sure to match casing of your GH user name) and is executable
[X] Output matches that of calculate_average_baseline.sh

Execution time: 1.360 s (almost the same as @thomaswue entry, perhaps even slightly better)
Execution time of reference implementation: very long

Lost first place, can't let that happen... giving it all.

00:03.071 now. I.e. very close to @thomaswue. It's within the range of what I can reliably measure with the current set-up, so I'd consider them equal for all practical purposes.

@gunnarmorling you can make both to run on a single thread. It would make sense for the top 3 contenders if they have similar numbers. Obviously this implicitly assume that the memory bandwidth is not the bottleneck.

it's the native image version of the JVM one?

Native.

It would make sense for the top 3 contenders if they have similar numbers.

@franz1981, yes, that's definitely something I'd like to do towards the end of the challenge, measure the top 5 as precisely as possible (will likely need the help from some of the experts with this; perhaps it's even something you might do on a fine-tuned machine in your lab?).

@royvanrijn @gunnarmorling

Native

Interesting: I believe that using EpsilonGC in JVM mode can guarantee a very similar speedup, can yo give it a spin @gunnarmorling ? @royvanrijn wdyt?

measure the top 5 as precisely as possible (will likely need the help from some of the experts with this; perhaps it's even something you might do on a fine-tuned machine in your lab?).

I'm nerd sniping in RH for this ;)

Please keep in mind that the compiler can take different decisions with different runs so you might need to compile the program several times. The runtime of the compiled executable should be much more stable, though.

@merykitty your comment is related any specific part of the comments I made? If yes, please quote it to help me understand :pray:

@franz1981 No I just gave my opinion about the evaluation of solutions in general.


feat: add hint to REST-API (link to repo and meggsimums swagger export)

Suggestion for https://github.com/geostyler/geostyler-homepage/issues/2
Changes neighbouring items to ensure alternating backgrounds.
Also contains an explicit naming for the docs
changed the order by the weights in a seemingly logical order
moved osgeo section under tutorials
sorry for the bundled PR

Thanks I'll learn Some:Day


test(install): Verify 2024 edition / resolver=3 doesn't affect resolution

What does this PR try to resolve?
I was worried there might be bugs related to this.  With this out of the way, I think we'll be ready to stabilize resolver = "3".
How should we test and review this PR?
Additional information

Thanks!
@bors r+

:pushpin: Commit 487bbc813fd116ae3d284dfe2f78063c99cadc1f has been approved by weihanglo
It is now in the queue for this repository.

:hourglass: Testing commit 487bbc813fd116ae3d284dfe2f78063c99cadc1f with merge 28274d0a51d3432ec7435ed7cc20ffd81892e9de...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing 28274d0a51d3432ec7435ed7cc20ffd81892e9de to master...


convert the example conftest.py to a pytest plugin

Instead of requiring copying an example allow using the example as a pytest plugin. Now using this is as easy as pip install pyannotate && pytest -p pyannotate_tools.pytest (assuming you would have previously run pytest and your config handled everything else already)
Fixes: #97

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


Change in behavior for S3 paths with leading slashes (/) between version 2.0.0 and 3.2.0

Summary
We're in the process of upgrading to the latest libcloud release (and Python 3) from a relatively old version 2.0.0 (and Python 2).  During testing, we discovered a change in behavior when uploading objects to AWS S3 when they had a double leading slash in the object path, after the bucket name, for example s3://some-production-bucket//some-path/some-file.txt.  Previously, this would've resulted in a folder-like object named / in the root of the path to the file, where now it does not, and some-path is the top-most folder-like object in the bucket.
While we realize this is bad practice, the leading object named / is valid by AWS S3's documentation, and for legacy reasons we cannot immediately migrate from this behavior.  We need to be able to access these double-leading-slash S3 paths as before.
Detailed Information
In trying to trace this problem, I discovered this call path:

libcloud/storage/drivers/s3.py : upload_object()
libcloud/storage/drivers/s3.py : self._put_object()
libcloud/storage/base.py: _upload_object()  invokes self.connection.request()
libcloud/common/base.py: request()
libcloud/common/base.py: self.morph_action_hook(action)

It looks like the root of this behavior issue is in morph_action_hook on line 655, which attempts to strip out any double slashes:
    def morph_action_hook(self, action):
        url = urlparse.urljoin(self.request_path.lstrip('/').rstrip('/') +
                               '/', action.lstrip('/'))
        if not url.startswith('/'):
            return '/' + url
        else:
            return url

and its this that prevents our valid use of the top-level folder-like object named /.
I verified this by modifying morph_action_hook() to just return action, which works in our case because request_path is empty.  Uploading files with this change in place returns us to the previous behavior.  We've manually patched libcloud in our production deploy, but we'd very much like for that to be a temporary fix.
I don't entirely understand the goal of the current morph_action_hook implementation (except as maybe an attempt to protect users from their own mistakes?), but whatever it was, it was clearly added since our original application of this library at version 2.0.0.
Is there something I'm missing?  Is there a provision for disabling this corrective code that I'm not seeing?
Thanks in advance for any help you can give.

Thanks for reporting this.
I assume that change was done a long time ago during the migration to requests library and likely for compatibility reasons as you mentioned.
I do think we should support your use case as well, but that change would be backward incompatible / potentially breaking which means we need to use a feature flag or similar.
To begin with, perhaps just an argument which can be passed to the driver constructor. What do you think?

I believe this should work as a workaround - #1531.
As mentioned in that PR, sadly we can't make that a default behavior since it appears to break a lot of other things (it's also backward compatible) so we need to make it a "config option".

Thanks for getting to this so quickly, we appreciate it.  I did a quick review of the PR and it looks like it'll work fine for us, it's very similar to the patch we put in place.  I'll set up a test early next week to be sure.  Thanks again.

@triplepoint Did you happen to have a chance to test it yet?
I hope to work on a new release which will include this change in the next couple of days.
One thing worth pointing out is that setting module level variable to allow duplicate slashes will modify behavior for all the requests. So this could potentially cause issues if you are using multiple drivers.
One way to avoid that would be to add allow_duplicate_slashes argument to the S3 driver and then propagate this option all the way down to the connection class. This would require more glue code, but it would mean that change would only be scoped to that instance of the driver instead of globally.
WDYT?

For now I just updated upgrade notes and noted it affects all the instantiated drivers.
If it becomes an issue, we can introduce driver constructor argument which only applies to the driver in question,

Hi @Kami, I'm sorry, but with the holiday season we haven't gotten around to testing this yet.  I think we have a product release scheuled for mid january, and I intend to have this tested as a replacement for our internal patch for that release.  I'll report back here how it goes, but looking at the change, I think it'll work fine for us.   Thanks again.


wai-aria-practices for Dropdown Menu in 6.6.3

What should happen?
The following issue relating to wai-aria-practices for the dropdown menu was discussed and changes accepted and implemented in Version 6.6.3. Link here:
Accessibility issue with Dropdown - https://github.com/foundation/foundation-sites/issues/8810#issuecomment-489297729
and an example was provided from the W3C website: https://www.w3.org/TR/wai-aria-practices/examples/menubar/menubar-1/menubar-1.htm
What happens instead?
Now that role="none" is applied to the li - We are getting accessibility errors because that role has no supported properties yet aria-haspopup and arial-label are applied.
Allowed Aria Roles, States and Properties - This table does say that it is an informative reference.  In that instance should the errors be marked as warnings instead?
If anyone has any further guidance around this, or if I have misinterpreted the implementation please feel free to contribute.
...
Possible Solution
A possible solution would be to follow the example here: https://www.w3.org/TR/wai-aria-practices/examples/menubar/menubar-1/menubar-1.html - and apply aria-haspopup and arial-label to the a element designated as the dropdown.
...
Test Case and/or Steps to Reproduce (for bugs)

Test Case:

How to reproduce:
1.
2.
3.
Context

...
Your Environment

Foundation version(s) used:
Browser(s) name and version(s):
Device, Operating System and version:
Link to your project:

Checklist

[ ] I have read and follow the CONTRIBUTING.md document.
[ ] There are no other issues similar to this one.
[ ] The issue title and template are correctly filled.

I'm still troubleshooting this in a drupal project (so it's an applied case and I don't have a patch at this point), but here is a source I'm looking at for reference: W3 Fly-out Menus
My current thoughts on what could be improved within the dropdown menu element:

adding "aria-expanded" attributes
Making sure that if the parent element is a link that doesn't actually go anywhere, screen readers can select it without reloading the page.
Making sure that both keyboard controls, and screen reader controls work to open and close the dropdown.


Added checkbox in other settings option to toggle popup confirmation …

…for clearing search result

Fixes #188
Description
Added checkbox in "other settings" to toggle popup confirmation before clearing the search result.
Other information
Bydefault it is not checked i.e. the popup confirmation won't be displayed.
Checklist:

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the master branch of the repository.
[ ] My commit messages follow best practices.
[ ] I added tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no
visible errors.

Developer Certificate of Origin
Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or
(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or
(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.
(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

@makkoncept Please have a look.

@makkoncept Updated the PR please have a look.

@techievivek It's working fine. Please enter the correct issue number in the PR description -> Fixes #159. Then, we can merge this.

@makkoncept I am extremely sorry for that silly mistake, I made the same in CC open source repo too.
I have updated it.
Thank you.


Building WebpackBIN and have question about optimization

Hi there!
I am building an experiment to use Webpack as a service to show off code, like JSBIn etc. Proof of concept is here: webpackbin.herokuapp.com.
It uses DllPlugin to move NPM packages into its own bundle, then it is cached on the client. This makes the service incredibly responsive and it is a key feature for a successful project.
The problem is that Webpack and DllReferencePlugin requires all the NPM modules to be available on the filesystem (in memory on this service) even though I have already built the DLL bundle. What I am trying to do is:

First person using a set of packages, download from NPM and create DLL of it
Store DLL in database (bundle + manifest)
When someone else uses same set of packages just download the bundle + manifest from database and use it

But as I said, this does not work because the filesystem needs all the NPM packages there as well.
The thing is that it really does not need to it seems. If I copy the application bundle created with the NPM packages in file system, to a bundle where they are not in the file system, it works just fine.
Is there any way to just force DllReferencePlugin to not resolve the paths because I know they are correct? Or am I missing something?
I also tried to use externals but it logs out the DLL function, not the actual module.

Okay, actually managed to do this by:
{
  externals: {
    'react': 'myDLL(1)'
  }
}

I just have to identify the different entry points, which is managable


Users needed to provide passport/driving licence details to third party but did not like that and stated she just wanted to give it to the council

@lizziebruce - also linked to #369 - is there anything in the journey before Verify that we can include to suggest users might need to provide details of their identity, download apps, etc?

See front-loading of explanation about companies in #588 #602


As a uni student, I want to know where the best nearby restaurants are, so that I can ensure I avoid disappointment

User Story
As a uni student, I want to know where the best nearby restaurants are, so that I can ensure I avoid disappointment
Acceptance Criteria

[x] Displayed restaurants will appear with reviews

Completed


chore(.gitignore): add .tap/ dir

See https://github.com/fastify/skeleton/pull/52

Checklist

[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

Test failure unrelated.


[!] CocoaPods could not find compatible versions for pod "RxSwift":

[!] CocoaPods could not find compatible versions for pod "RxSwift":
In snapshot (Podfile.lock):
RxSwift (= 4.5.0, ~> 4.0)
In Podfile:
RxSwift (~> 5.0.0)
None of your spec sources contain a spec satisfying the dependencies: RxSwift (~> 5.0.0), RxSwift (= 4.5.0, ~> 4.0).
You have either:

mistyped the name or version.
not added the source repo that hosts the Podspec to your Podfile.

Note: as of CocoaPods 1.0, pod repo update does not happen on pod install by default.

Hey @GTramp, we do not support RxSwift yet. We are preparing the release that would cover RxSwift 5, ReactiveSwift 6 and Alamofire 5, but it’s not ready yet.

I see, thank you !

@GTramp We actually released an alpha version 14.0.0-alpha.1 if you want to try. Gonna close this issue but let me know if I can help you with anything else :)


QR Code overflowing bounding box

See https://accounts.verida.io/did:vda:0x991546Ee69f83076c70aAdA37B04e5a10a004B4b

Please add your planning poker estimate with ZenHub @cmcWebCode40


Unable to run the 'pod install'

First of all thank you for the amazing Repo it's a life saver. I cloned the repo and the Android part seems to run like a charm.
For iOS however, every time I run cd ios && yarn upgrade flutter-module-rn && pod install I seem to get these errors.
Auto-linking React Native modules for target `ReactNativeApp`: FlutterModuleRn, RNScreens, and react-native-safe-area-context
Analyzing dependencies
Fetching podspec for `DoubleConversion` from `../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`
Fetching podspec for `Flutter` from `../node_modules/flutter-module-rn/build/ios/framework/Release/Flutter.podspec`
/Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/user_interface/error_report.rb:34:in `force_encoding': can't modify frozen String (FrozenError)
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/user_interface/error_report.rb:34:in `report'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/command.rb:66:in `report_error'
	from /Library/Ruby/Gems/2.6.0/gems/claide-1.1.0/lib/claide/command.rb:396:in `handle_exception'
	from /Library/Ruby/Gems/2.6.0/gems/claide-1.1.0/lib/claide/command.rb:337:in `rescue in run'
	from /Library/Ruby/Gems/2.6.0/gems/claide-1.1.0/lib/claide/command.rb:324:in `run'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/command.rb:52:in `run'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/bin/pod:55:in `<top (required)>'
	from /usr/local/bin/pod:23:in `load'
	from /usr/local/bin/pod:23:in `<main>'
/System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/uri/generic.rb:1490:in `find_proxy': relative URI: /Users/njohn/Desktop/Workspace/rn-package-flutter-main/ReactNativeApp/node_modules/flutter-module-rn/build/ios/framework/Release/Flutter.podspec (URI::BadURIError)
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/open-uri.rb:209:in `block in open_loop'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/open-uri.rb:226:in `block in open_loop'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/open-uri.rb:224:in `catch'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/open-uri.rb:224:in `open_loop'
	from /System/Library/Frameworks/Ruby.framework/Versions/2.6/usr/lib/ruby/2.6.0/open-uri.rb:165:in `open_uri'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/external_sources/podspec_source.rb:19:in `block in fetch'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/user_interface.rb:86:in `titled_section'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/external_sources/podspec_source.rb:11:in `fetch'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:993:in `fetch_external_source'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:972:in `block (2 levels) in fetch_external_sources'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:971:in `each'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:971:in `block in fetch_external_sources'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/user_interface.rb:64:in `section'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:970:in `fetch_external_sources'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer/analyzer.rb:117:in `analyze'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer.rb:416:in `analyze'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer.rb:241:in `block in resolve_dependencies'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/user_interface.rb:64:in `section'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer.rb:240:in `resolve_dependencies'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/installer.rb:161:in `install!'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/command/install.rb:52:in `run'
	from /Library/Ruby/Gems/2.6.0/gems/claide-1.1.0/lib/claide/command.rb:334:in `run'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/lib/cocoapods/command.rb:52:in `run'
	from /Library/Ruby/Gems/2.6.0/gems/cocoapods-1.11.2/bin/pod:55:in `<top (required)>'
	from /usr/local/bin/pod:23:in `load'
	from /usr/local/bin/pod:23:in `<main>'

Thanks again for your article and research.

Thanks for the response. this is what I got:
ruby -v
ruby 2.6.8p205 (2021-07-07 revision 67951) [universal.x86_64-darwin21]

Update: Tried running on arm64e as well. and still getting the same issues.

solved by removing all cocoapods then installing using homebrew.

gem uninstall cocoapods
gem list cocoapods
manually remove any cocoapods if left
brew install cocoapods


Unusual numerical precision value of the cell

Description

使用代码获取单元格值,得到的结果和excel表不一致

原始表格如上图
代码如下
excelHandler,_ := excelize.OpenFile("test.xlsx")
sheetMap := excelHandler.GetSheetMap()
sheet := sheetMap[1]
s,_ := excelHandler.GetCellValue(sheet, "A1")
fmt.Printf(s)

打印结果为
1.1000000000000001
附件
test.xlsx
Steps to reproduce the issue:
如上所述
Describe the results you received:
输出1.1000000000000001
Describe the results you expected:
输出1.1
Output of go version:
go version go1.13.5 windows/amd64

Excelize version or commit ID:
v2

Environment details (OS, Microsoft Excel™ version, physical, etc.):
Windows 10,Excel 2016

Excel 2016,I think it's excel soft problem,this happened when I try other xlsx library.


Additional Info on the Intro part

Why?
Closes #86
How?

[ ] Tests have been added/updated (if necessary)
[x] Documentation has been updated (if necessary)

PR for https://github.com/JasonEtco/actions-toolkit/issues/86

we have a typo in the first paragraph ;)
... Actions ...

Fixed it, thanks 🙏


🚨 Potential Deserialization of Untrusted Data (CWE-502)

👋 Hello, @ancientmooner, @microsoftopensource, @zeliu98 - a potential high severity Deserialization of Untrusted Data (CWE-502) vulnerability in your repository has been disclosed to us.
Next Steps
1⃣  Visit https://huntr.dev/bounties/1-other-microsoft/Swin-Transformer for more advisory information.
2⃣  Sign-up to validate or speak to the researcher for more assistance.
3⃣ Propose a patch or outsource it to our community - whoever fixes it gets paid.
✏ NOTE: If we don't hear from you in 14 days, we will proactively source a fix for this vulnerability (and open a PR) to ensure community safety.

Confused or need more help?

Join us on our Discord and a member of our team will be happy to help! 🤗

Speak to a member of our team: @JamieSlome

This issue was automatically generated by huntr.dev - a bug bounty board for securing open source code.

@ancientmooner @zeliu98
Take a look into CVE-2021-31200 the same bug was reported and fixed on microsoft/nni
https://msrc.microsoft.com/update-guide/vulnerability/CVE-2021-31200

@ancientmooner @zeliu98 @msftgits any updates on this 😇


Pattern AST refactoring

I have some idea that may optimize patterns' ASTs but would require some nontrivial effort

They are wrong


GPU Usage 100% at dataloading while not proceeding

Hi, I was trying to run the model via Colab provided at the README, however I encountered two issues,

I was training on the bouncingball scene from dnerf and the GPU usage spiked to 100% while dataloading and the training somehow stucks there...
I proceeded on training on the hellwarrior dataset and getting a loss is nan while training
(I am using a RTX 4090 with CUDA 11.7 and pytorch 1.13)
Wondering if there is something I am doing incorrectly, thank you!

e, most cases happens when your CUDA version not corresponds with your pytorch version.
Is your pytorch version = torch1.13+cu117?

I am using torch1.13+cu116, does that make a difference?


Compiler Error in PipelineBuilder.cpp

Bug summary
The PR #893 introduced the following new elif branch in the PipelineBuilder.cpp file:
#if LLVM_VERSION_MAJOR <= 13
    FPM.addPass(llvm::SROA{});
#elif LLVM_VERSION_MAJOR < 16
    FPM.addPass(llvm::SROAPass{});
#else
    FPM.addPass(llvm::SROAPass{llvm::SROAOptions::ModifyCFG});
#endif

which results in a compiler error for newer DPC++ versions tested with sycl-nightly/20221102 and onwards (which is a modified clang-16):
src/compiler/cbs/PipelineBuilder.cpp:91:38: error: no member named 'SROAOptions' in namespace 'llvm'
    FPM.addPass(llvm::SROAPass{llvm::SROAOptions::ModifyCFG});

To Reproduce
Build hipSYCL with a new DPC++ version.
Expected behavior
hipSYCL compiles if the LLVM_VERSION_MAJOR <16 case is removed.
Describe your setup
DPC++ has been compiled using:
python buildbot/configure.py --cuda --llvm-external-projects="clang-tools-extra,compiler-rt,openmp" --cmake-opt="-DENABLE_LIBOMPTARGET=OFF" && python buildbot/compile.py && cmake --build build -- omp && cmake --build build -- install

Additional context
Since I don't know the reason for adding this elif branch, I don't know if it is save to remove it.

Well, since LLVM 16 is actively under development, it's difficult to maintain API compatibility with all states of the LLVM 16 development, we mostly aim for LLVM upstream stable release compatibility and fix top-of-tree issues occasionally.
The API change that required the new elif branch, is due to a commit from 8.12. (https://github.com/intel/llvm/commit/4f7e5d22060e8a89237ffb93c3e7be6e92fee8fe), so DPC++ versions newer than that should work with the current hipSYCL's devel branch. (Not sure how often Intel synchs with upstream LLVM, though, so give it at least a week extra maybe?)
And of course, if you need to use your current DPC++ version, you can just remove that new case, since it was only added for (build) compatibility with ToT LLVM versions.


Homework 5

Metric
Value

alert_status
:white_check_mark: OK

bugs
5

code_smells
19

reliability_rating
C (at least 1 Major Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Solve The Beginner Problems.

Would it be okay if I contributed to this project?  If so, do you need me to open an issue?

Ok if i work on the Fibonacci sequence in python?


Add issue link to disabled TensorRT integration test

Adds a GitHub issue link for the disabled TensorRT tests since it may be disabled for a longer period of time.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

assign set of reviewers

I think precommit checks are unrelated, but we should retry. Its possible they may need a dummy commit to pass


The UvmArray may have a wrong size and bring issues when calling set()

In Lua, the default index of an array starts with 1, but it can be explicitly defined as any integer, particularly it can be a negative number.
In order to be compatible with Lua, the UvmArray in Java is implemented with two basic types: a List and a Hashmap, if the index is between 1 and List.size(), it indicates the elements in the List: (index >= 1 && index <= items.size()), while the index is out of the range of the List(), i.e. any negative numbers, it can be stored in the hashmap.
However, there is a protential issue for the UvmArray. As seen in the picture, the method UvmArray.size() returns just the item's size, but in the UvmArray.set(), the index can be any number, even if the size of the array is 0.

The index needs to be checked if it is valid or not when setting a value.
We have also considered to let the UvmArray support only positive number and starts with 0, i.e. just the same the native Java, but then we need to add the restrictions for Lua also, which could be very complex. So we'd rather keep the current design.

Considering the usage case as below:
UvmArray test = UvmArray.create();
test.add("1");
test.size();                               // should be 1
test.set(10000, "set test");      // it will work?
test.size();                              // the size is still 1?

This behavior doesn't look like a common Aarray to me. Should we add some restrictions when using UvmArray? At least we should make the the size right so that we can loop all the elements in the array.


refactor package expr

My profuse apologies up front for this massive PR.  I really thought long
and hard about breaking this into smaller chunks and made some failed
attempts, but the interdependencies between the refactoring changes
were just too much so I went for the whole shebang.
THIS COMMIT BREAKS BACKWARD COMPAT WITH THE ZNG FILE FORMAT.
Now that we're addressing changes to the ZNG spec to bring it to beta status,
I thought this PR would be a good time to introduce breaking changes, shortly
after our previous release on Friday.  I think we have a handle on the breaking changes
we  want to make and I think I Ican get them all in by the next release, so after this,
hopefully we won't have anymore backward-compat changes (though we will have a few
more additions that don't break compat as we go from beta to stable of the
ZNG format).
===
This commit refactors the expr package to be more extensible and
performant.  The zngnative package has been eliminated.  It had
good intentions but every expression resulted in the allocation
of an empty interface to hold a go native value resulting in lots
of traffic to the GC.  The implementation keeps scratch buffers
around and reuses them and aside from strings and IPs, does not
allocate any memory to evaluate expressions.  There is a "keep"
flag passed to expr.CompileExpr() that says whether you want
values that get overwritten on subsequent calls to Eval()
(e.g., ok for filtering and aggregations) or whether you want to
hold onto the values (like groupby and sort do with field keys).
We also changed the field implementation and cleaned up the AST.
There was overlap between the old FieldCall stuff and the generic
expression code, which is now eliminated.  This also makes the
grammar a bit easier to reason about as we improve and extend it.
There is now one inteface type, expr.Evaluator, to take over for
all the function closures we previously had.  This makes it
easier to cookie-cutter and add new expression functionality.
Also, the field expressions are now just an instance of
expr.Evaluator so everything is now clean and uniform.
While we were changnig the coercion logic around and getting
rid of zngnative, we laid out the ZNG type codes a bit differently
and upudated the spec.  Since we were making incompatible changes,
we changed the "byte" type to "uint8" and added "int8" so all
the integer types are now orthogonal.  Also, the integers are
all encoded using the same varint logic now, which greatly
simplifies value encoding and decoding in the coercion engine.
Most of the tests that were updated simply accommdate the change
to the zng binary format.

FYI the failing test is because the zng output format changed.  I don't know if we want to bypass this for a week or so while I finish making breaking changes.  Then, once the dust settles, hopefully by the next release, we can re-enable this check?
@philrz ?

Whatever you're doing here is already having positive knock-on effects. While writing aggregate function docs, I saw this crash with GA zq tagged v0.22.0 when run against the full 70 GB wrccdc data set:
$ zq -f table 'every 1 week count() | sort -r ts' *.log.gz
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0x13efe36]

goroutine 42 [running]:
github.com/brimsec/zq/zngnative.CoerceNativeToTime(0x0, 0x0, 0x0, 0x0, 0x0, 0x289a7d0)
	/Users/phil/work/zq/zngnative/coerce.go:202 +0x26
github.com/brimsec/zq/expr.timeTrunc(0xc000abbc00, 0x2, 0x2, 0x1724f80, 0xc0002f8028, 0x0, 0x0, 0x203000, 0x1)
	/Users/phil/work/zq/expr/functions.go:454 +0x6f
github.com/brimsec/zq/expr.compileFunctionCall.func1(0xc000a41640, 0xc0008a92c0, 0x24d9200, 0xc0008ba948, 0xc000680000, 0x289a7d0, 0x0)
	/Users/phil/work/zq/expr/expr.go:938 +0x250
github.com/brimsec/zq/expr.CompileExpr.func1(0xc000a41640, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0, 0x0)
	/Users/phil/work/zq/expr/expr.go:48 +0x63
github.com/brimsec/zq/proc/groupby.(*Aggregator).Consume(0xc000354000, 0xc000a41640, 0xc000a41640, 0x0)
	/Users/phil/work/zq/proc/groupby/groupby.go:396 +0x1c9
github.com/brimsec/zq/proc/groupby.(*Proc).run(0xc000090200)
	/Users/phil/work/zq/proc/groupby/groupby.go:269 +0xa4
created by github.com/brimsec/zq/proc/groupby.(*Proc).Pull.func1
	/Users/phil/work/zq/proc/groupby/groupby.go:240 +0x3e

Since I'd just seen your reference to eliminating the zngnative package, and zngnative appeared in this error, I tried with commit 1ddde2b for this branch and can see it's already fixed.
$ zq -f table 'every 1 week count() | sort -r ts' *.log.gz
TS                COUNT
1541635200.000000 3131
1521676800.000000 475593430
1490227200.000000 96242990
0.000000          26

👍

Hmm, I just noticed a problem with integer 0 turning into unset in expressions.  Will fix...

Hmm, I just noticed a problem with integer 0 turning into unset in expressions. Will fix and adjust the tests...

Fixed.

Regarding the failing ZNG output check, I've put up https://github.com/brimsec/zq-sample-data/pull/16 that has the test data changes necessary to make the automation pass, along with draft PR https://github.com/brimsec/zq/pull/1383 that shows it passing "green" in a scratch branch. There's comments in https://github.com/brimsec/zq-sample-data/pull/16 about the options for the order in which the changes should land. Overall, though, it's just a small loose end we can resolve right as we're merging this PR. The long pole in this effort remains the actual review of the code. 😜

Would this be a good opportunity to rename expr/temp.go?

Would this be a good opportunity to rename expr/temp.go?

I'm confused... expr/temp.go is gone as I've reworked things to remove the presumption that you can do strings.Split(field, ".") to get the path of keys as this doesn't work with arbitrary strings as key names (which zng now and must support to be a superset of JSON). There are methtods to turn ast dot expressions and expr.FieldExpr's into human-readable form with "." but the other direction doesn't work.

@mccanne I'm the one who's confused. once i got the gist of the changes on the PR i figured temp.go would be gone and saw it but on another branch. sorry for the noise.

Update: I hadn't noticed the test failures. :shipit: after they pass

The tests are all passing.  The red is from the zng format change against zq-sample-data because hte format has changed.  I proposed to check this in red and update the sample data for the zng-beta merge, which will entail further zng format changes.  Right @philrz ?

@mccanne: There's a corresponding eval branch of the zq-sample-data repo that has ZNG data based on your eval zq ready to go. It's in https://github.com/brimsec/zq-sample-data/pull/16 ready to be merged whenever we want. I just now made a new zq branch off your zq eval branch and added one modification to have it point to that updated sample data branch, and you can see in draft PR #1411 that the "zng-output-check" automation now passes "green". So if you're ready to merge on the zq side, we can merge https://github.com/brimsec/zq-sample-data/pull/16 and re-run your Actions one last time to have the satisfaction of seeing it run totally green before we merge.
That said, I do see in both your most recent Actions run and the one in #1411 that's based on your branch that there's still some failures related to the docs examples. For instance, some of the expressions math is creating some crazy high numbers:
$ zq -f table 'duration > 100 | put total_bytes=orig_bytes+resp_bytes | cut orig_bytes,resp_bytes,total_bytes' conn.log.gz
ORIG_BYTES RESP_BYTES TOTAL_BYTES
32         0          32
32         0          8224
406        1720       139337760
32         31<PHONE_NUMBER>08
32         31<PHONE_NUMBER>7296
32         31<PHONE_NUMBER>7598624
32         0<PHONE_NUMBER>701292576
32         0<PHONE_NUMBER>701292576
1          63<PHONE_NUMBER>701292576
...

@mccanne: There's a corresponding eval branch of the zq-sample-data repo that has ZNG data based on your eval zq ready to go. It's in [brimsec/zq-sample-data#16]

Cool.  We can rinse and repeat for the zng-beta branch.  I'm putting the last touches on @alfred-landrum's suggestion about "keep" and will be ready to merge soon.

That said, I do see in both your most recent Actions run and the one in #1411 that's based on your branch that there's still some failures related to the docs examples. For instance, some of the expressions math is creating some crazy high numbers:

That was a coercion bug for duration types.  Lost in translation from zngnative to eval.  I will add a system test for this since it was only the markdown tests that caught it.


Added Mevo'ot Eron to the list of schools

@Sufilevy First stage of verification passed, official domain is mevoot-eron.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@Sufilevy Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Russian localisation

Done with the latest commits. Later on, I will maintain both English and Russian localizations incrementally:

Get a challenge from type-challenges
Solve it in English first
Translate it to Russian afterwards


Reproducing CIFAR10 result from paper: Improved Techniques For Consistency Training

Hi!
Has anyone managed to reproduce the results from "Improved Techniques For Consistency Training" on the CIFAR10 dataset?
Thank you for the great repository!

Hello!
I am not sure about this. So far I haven't carried out large experiments with this.
Thank you for the kind sentiments.

@Kinyugo @noamelata
Hi，everyone. I have implemented the experiment on butterflies dataset(len=1000), which the result trained 10000 steps(max_step) is better, FID5=0.2028 with five steps sampling, and FID1 =0.2165 with one step sampling.  However, on cifar10,  I didn't get  good  result as I expected. I set : (1) N(k)=11, max_steps=400000, optimizer=RAdam(better than Adam), batchsize=32 , result: FID10(10 step sampling)=74.3594, FID5=61.3095, FID2=127.9843, FID1=285.8623; (2) N(k)=1280, max_steps=400000, optimizer=RAdam , batchsize=32, result: FID10=64.5555,FID5=56.4666, FID2=68.3255, FID1=243.7875; (3)N(k)=1280, max_steps=400000, optimizer=RAdam , batchsize=64, result: FID10=30.2732,  FID5=35.0473, FID2,=62.3042 ,FID1=238.0072. If I set the batchsize=128, The results were also poor.

I suspect there is something wrong with the structure of Unet, but i am not sure about it.

Hello,
Kindly try the experiments with the unet from the paper. This was just a random unet.  If you manage to get it working kindly share your findings.

@Kinyugo
Hello, thank you for your instant reply. I found a error in your codes as follows ：
in the timesteps_schedule function,
def timesteps_schedule(....):
num_timesteps = final_timesteps2 - initial_timesteps2
# the final_timesteps above should be "final_timesteps+1",
cf. page 26 in the Appendices of the paper ''Consistency Models" .
num_timesteps = current_training_step * num_timesteps / total_training_steps
num_timesteps = math.ceil(math.sqrt(num_timesteps + initial_timesteps**2) - 1)

return num_timesteps + 1

Thanks for the nice find. I'll correct asap.

Hello. We already compute the timesteps correctly. https://github.com/Kinyugo/consistency_models/blob/3bc6fe1350d911ecb9eca1bb9d286853b3833533/consistency_models/consistency_models.py#L38C3-L41C1

@Kinyugo However, this trivial error should not obscure the concision aesthetics of your code, this is an excellent work. As far as i know , your code is first one performed the "improved consistency models" in github.

Thank you for your kind sentiments.
Regarding the model, I agree with you that it might be sub-optimal. A recent paper "Analyzing and Improving the Training Dynamics of Diffusion Models" could be of assistance, though it's based on the EDM paper. I do plan to experiment with the architecture there but I am currently held up. You could also check issue #7 that proposes rescaling of sigmas. I think with some of those changes you might get better results.
Thanks for taking the time to share you findings with me.

@Kinyugo  Thanks for sharing your findings with me. I have not pay attention to the paper "[Analyzing and Improving the Training Dynamics of Diffusion Models]".  Regarding the issue #7  , it is worth to try it. But i want to change firstly  your unet structure with ncsn++ ,and then check the result on cifar10. To facilitate communication, we send<EMAIL_ADDRESS>each other.

Awesome. Ping me incase of any questions.
If you replicate the ncsn++ network and get good results consider contributing to the repo.

@thwgithub Hello! I was wondering if you have had the opportunity to reproduce the FID on CIFAR10 using the U-Net described in the "Consistency Models" paper. If so, may I kindly inquire about the results you obtained? Thanks in advance!

@aiihn sure.

@thwgithub Thanks! What FID results have you obtained on cifar10?

@Kinyugo Hello, I have successfully incorporated the ncsn++ network(https://github.com/openai/consistency_models_cifar10/blob/main/jcm/models/ncsnpp.py) into your consistency model. Unfortunately, I did not still achieve good results. At the same time, I looked repeatedly  your codes over, I assured it was no problem. Now, I am quite confused about it. Can you help me to check my codes?

----- 原始邮件 -----
发件人：Kinyugo @.>
收件人：Kinyugo/consistency_models @.>
抄送人：thwgithub @.>,  Comment @.>
主题：Re: [Kinyugo/consistency_models] Reproducing CIFAR10 result from paper: Improved Techniques For Consistency Training (Issue #5)
日期：2023年12月26日 18点03分
Awesome. Ping me incase of any questions.
If you replicate the ncsn++ network and get good results consider contributing to the repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.Message ID: @.***>

@thwgithub  could you provide your code for the ncsn++ as well as the training code and hyperparameters

@Kinyugo I have sent it to email. Did not you receive it?

@Kinyugo your email

@thwgithub No

@Kinyugo Thanks for your advice. please,  see more details :
https://github.com/thwgithub/ICT_NCSNpp/blob/main/ICM_ncsn%2B%2B.py

@Kinyugo I trained only on one GPU(4090)

@Kinyugo Thanks for your advice. please, see more details :  https://github.com/thwgithub/ICT_NCSNpp

@thwgithub  any breakthrough? I have been unable to checkout your repo due to time constraints on my end

@Kinyugo hey ,I find a tiny error in improved consistency training code given by your repo ,the correct mean of timesteps distribution is -1.1, but you write 1.1,just like blow:

@nobodyaaa  Thanks for catching the error. Fortunately it's a documentation issue and the correct lognormal_mean is used in the ImprovedConsistencyTraining class.

@Kinyugo yeah, I noticed that,thanks for answering. And did u repreduce icm results? I have run icm training several times but didn't get result as good as showed inthe paper ,the best of my one-step generation fid in cifar10 is about 60.

Did you run with the same configuration as the paper?

@Kinyugo  except neural network which I just use Unet copied from somewhere instead of ncsn++.I have  read the CM codes given by openai , and I used c_in and rescaled sigma to get the input of neural network just like they did.Other configuration ,like karras schedule and something else ,I just use your code .

How many iterations did you train? Batch size e.t.c

Consider using ncsn. You can get it's building blocks from diffusers.
Also there is a bug in the pseudo-huber loss that we compute the c over the channel dimension as well but it should only be over the spatial dimensions.
I'll be pushing these fixes together with other fixes for the input scaling once I'm done testing. Unfortunately I cannot run huge experiments so I'm not sure how close to the paper's results we will get.
Could you share some samples for different sampling steps?

@Kinyugo ok ,I will try ncsn++ later.And in my experiments ,I set c to 0.03 directly.
here are 1 step ,2 step ,3 step ,4 step,5step generation example ,sigma schedule is [80.,7.1273,1.8972,0.3704,0.0438]:

And I have test 50K fid of 5 step generation,nearly same to 1 step generation,now both are 50.43

@Kinyugo there are some error in my code ,and after correcting it ,I retested 5 step generation and got 18.7 fid


Feature/analytics

Add SQL knex client.
Add /v2/analytics/:library endpoint.
Includes WIP v2 docs.

Closes #62

@jimaek this should be ready to go but you'll have to add SQL_HOST and SQL_PASSWORD environment variables to the appfog instance.

Do you cache the analytics data on the API or you hit the DB every time?

I hit the db
On Mon, Jul 20, 2015, 3:30 AM Dmitriy Akulov<EMAIL_ADDRESS>wrote:

Do you cache the analytics data on the API or you hit the DB every time?
—
Reply to this email directly or view it on GitHub
https://github.com/jsdelivr/api/pull/102#issuecomment-122842620.

Ok. Well it looks good to me.
@megawac Any feedback? Is it good to go?

Btw part of the analytics data is delayed by 2 days. Any ideas what we should do?
Simply show what we have or dont allow the user to get the data for the last 2 days at all?

I'd say show what we have, and document the analytics limitations.

+1
We'd need to keep an eye on the usage and if necessary cache the results for some minutes. But otherwise this looks good :)

looks great

Is this live on the v2 demo app?
http://jsdelivr-apiv2-test.aws.af.cm/v2/analytics/jquery

ping @UnbounDev

@jimaek it should be live, yes, but the test instance on appfog is dead and I haven't been able to get a successful deploy, even manually via the cli (appfog always seems to miss installing an npm dependency and the app won't start).

The app works locally for you? Thats strange, appfog is unstable but not that unstable


Make sysctl calls on FreeBSD

The OS module get_unique_id and get_processor_name rely on linux files which don't exist on a standard FreeBSD install, make sysctl calls to get the required data.
I expect OpenBSD and NetBSD would require similar changes.

You didn't update your branch, it is still about 18 months out of date, please update your branch with git rebase -i master after updating your master branch to the latest version

You rebased incorrectly, please reset with git reset --hard 6a9caf6 and try again

There are still style issues that would be fixed by clang-format (or checking surrounding code and using the same style).
And the commits are still not linked to your GH account.

@akien-mga hi, when I look at the commit history it says I authored and committed the changes, so I'm not sure how its not linked to my account, or what I would need to change.

See: Why are my commits linked to the wrong user? for more info.

Seems there was a capital letter in my github email that makes it different to the commit info.

I think it looks right this time

Yes, looks great!

@akien-mga should be good to go now. Note that I also added the String::utf8 change which I didn't push earlier.

Cherry-picked for 4.2.2.


[Feature Request] Allow for a custom Internal Port

Is your feature request related to a problem? Please describe.
If the speedtest Tracker is bundled with other containers routing through another container's network, there can be a port conflict if more than one container tries use the same internal ports.
example: Containers routing through a VPN container, the VPN container manages all the ports
in the speedtest_tracker the network is set as:
--network=container:vpn \

I think this is at the docker level and not application, can you open the same question in the LSIO repo? Let's see what they say.
https://github.com/linuxserver/docker-speedtest-tracker

Done:  https://github.com/linuxserver/docker-speedtest-tracker/issues/20


Is it possible to hide the "copy" of an item. And only show ghost + cursor?

Question:
When i drag an element, i see both the ghost and a "copy" of the element. I know how to hide the ghost and change its appearance. But its the copy i would like not to be shown. Best solution is if i could style it myself. Second best is to just hide it and only have the cursor shown + ghost.
Is this possible?

I found forceFallback=false which solved the issue i had ( which was really that some other js within my item had a collision with sortable which made things jump and flicker and the position of cursor was off. That's why i wanted to hide the copy.
So i am happy now. But i guess my thing is related more to browser/html5 behavior than sortable.


ci(pull-request): run rbac tests with main

Thanks for submitting a PR! Please check the boxes below:

[x] I have run pre-commit to check linting
[x] I have filled in the "Changes" section below?
[x] I have filled in the "How did you test this code" section below?
[x] I have used a Conventional Commit title for this Pull Request

Changes
Include rbac tests in the main repo
How did you test this code?

N/A

A different action should be added for this that run on merge to master


Fixes all warnings except one—the remaining one is necessary

The package file actually declares the minimum deployment as 13 which is why there's still a warning. Turns out I can just delete all the availability code and only use what's necessary for iOS 13+.


'gbk' codec can't decode byte 0x96 in position 2: illegal multibyte sequence

Feature Request
Is your feature request related to a problem? Please describe:

Describe the feature you'd like:

Describe alternatives you've considered:

在windows系统上按照官方教程安装后出现类似的错误，为什么问题没解决就直接关掉Issuse了呢


Fix(DOCS): Updates broken link that redirects to faulty website

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

While looking through some documents, I found a link that redirects to a faulty site. Thank you so much.

The tools dashboard has moved to: https://tools.freecodecamp.org/dashboard/, but it's not completely maintained at the minute. Other than being used by some of our mods.
We are working towards overhauling it.
I think for now, we should remove the link and add a notice along the lines of:

We are working on creating a tools dashboard for for helping you review PRs and translations. Hangout with us in the contributors chat room to learn more.

Sounds good! Should I commit the new changes to this pull request?

Should I commit the new changes to this pull request?

Yes please.

Committed the new changes and the link to the contributor's chat room! Thank you so much!!!

Thanks 👍


[#5378] Update async and reactive of redission plugin

#5378

org.redisson.RedissonReactive

Codecov Report

Merging #6304 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #6304      +/-   ##
==========================================
+ Coverage   39.58%   39.59%   +<.01%
==========================================
  Files        3064     3065       +1
  Lines       91373    91397      +24
  Branches    11831    11836       +5
==========================================
+ Hits        36170    36188      +18
- Misses      52220    52224       +4
- Partials     2983     2985       +2

Impacted Files
Coverage Δ

...edisson/interceptor/ReactiveMethodInterceptor.java
0% <0%> (ø)

...pinpoint/plugin/redis/redisson/RedissonPlugin.java
0% <0%> (ø)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
52% <0%> (-16%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
84.1% <0%> (-4.64%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75% <0%> (-3.13%)
:arrow_down:

...point/rpc/client/DefaultPinpointClientHandler.java
73.05% <0%> (+0.97%)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.28% <0%> (+2.11%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
66.42% <0%> (+6.42%)
:arrow_up:

...pc/server/LoggingServerMessageListenerFactory.java
100% <0%> (+10%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


deploy documentation website

Using github actions
Reference

Hey @HassanBahati
I would like to work over this issue
Am I Supposed to deploy branch named g-h pages ?

Hey @HassanBahati I would like to work over this issue Am I Supposed to deploy branch named g-h pages ?

hi @maddy020 , sorry this issue was fixed by #19 ,
what was required was to recreate a github action to handle the deploy...the file that is doing is found here
If you are still interested in contributing, Please look through the issues , you may tag me in case you require any clarity.

Ok no issue


Bug: The app crashed when i click on update log on the about page.

Describe the bug
When I clicked on the main menu icon and clicked on the about page to see option/information about the app. I clicked on the update log and the app crashed.
To Reproduce

Click on the menu at the top left side
Click about
Then click on update log
See error

Expected behavior
I expect that the app shouldn't crash. There should be info to see in the log
Screenshots
https://youtu.be/V1UAiQB_Gfg
Smartphone (please complete the following information):

Device: [itel]
OS: [6.1]
Version <IP_ADDRESS>(20190526)

###Log
05-26 12:46:22.062 22516 23076 E AndroidRuntime: Process: top.geek_studio.chenlongcould.musicplayer.Common, PID: 22516
05-26 12:46:22.062 22516 23076 E AndroidRuntime: java.lang.NoSuchFieldError: No static field Companion of type Lokio/ByteString$Companion; in class Lokio/ByteString; or its superclasses (declaration of 'okio.ByteString' appears in /data/app/top.geek_studio.chenlongcould.musicplayer.Common-1/base.apk)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http2.Http2.<clinit>(SourceFile:2)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http2.Http2Writer.connectionPreface(SourceFile:5)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http2.Http2Connection.start(SourceFile:2)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http2.Http2Connection.start(SourceFile:1)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.RealConnection.startHttp2(SourceFile:7)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.RealConnection.establishProtocol(SourceFile:12)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.RealConnection.connect(SourceFile:15)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.ExchangeFinder.findConnection(SourceFile:37)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.ExchangeFinder.findHealthyConnection(SourceFile:1)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.ExchangeFinder.find(SourceFile:6)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.Transmitter.newExchange$okhttp(SourceFile:5)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.connection.ConnectInterceptor.intercept(SourceFile:5)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:14)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:1)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.cache.CacheInterceptor.intercept(SourceFile:22)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:14)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:1)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.BridgeInterceptor.intercept(SourceFile:22)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:14)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RetryAndFollowUpInterceptor.intercept(SourceFile:6)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:14)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.http.RealInterceptorChain.proceed(SourceFile:1)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.RealCall.getResponseWithInterceptorChain(SourceFile:14)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.RealCall$AsyncCall.execute(SourceFile:2)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at okhttp3.internal.NamedRunnable.run(SourceFile:3)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1113)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:588)
05-26 12:46:22.062 22516 23076 E AndroidRuntime: 	at java.lang.Thread.run(Thread.java:818)```

Emm. . What is the bug? Can you describe it in detail?🤔

I have included the details.

I have included the details.

Also due to okhttp, the intranet related operations have become abnormal. I will fix it tomorrow.

Has been fixed, you can view it on the release page

It working perfectly. :+1


HHH-11147 : fixes batch fetching bytecode-enhanced proxies

This is the Gail's https://github.com/hibernate/hibernate-orm/pull/2930 without the failing tests related with the initialisation of an non existing entity.

merged #2933 instead


Fix node rename problem

If rename node to '.' or '..', will cause problems
1、rename node to '.'
2、rename this node again
3、node's parent name changed
4、remove this node
5、node's parent has removed...
And not recommended use '/' for node name

Why this isn't merged? The problem still exists.

Looks good to me code-wise, will need to test it live.

Why this isn't merged? The problem still exists.

Congrats on posting a comment for the 1-year anniversary of the PR :D

It looks like '/' characters are already ignored somehow in the rename function (as they don't appear in the new name after validating, nor in the .xscn diff. But it doesn't hurt to "forbid" them explicitly either.
Apart from that, I get a build issue on X11:
Compiling ==> tools/editor/scene_tree_editor.cpp
tools/editor/scene_tree_editor.cpp: In member function 'void SceneTreeEditor::_renamed()':
tools/editor/scene_tree_editor.cpp:661:86: error: '_TR' was not declared in this scope
   error->set_text(_TR("Invalid node name, contain illegal character(s):\n ('.', '/')"));
                                                                                ^

Without the _TR(), it works fine. Here's a diff that proposes some rephrasing too:
diff --git a/tools/editor/scene_tree_editor.cpp b/tools/editor/scene_tree_editor.cpp
index 5312c1b..3c3eaf5 100644
--- a/tools/editor/scene_tree_editor.cpp
+++ b/tools/editor/scene_tree_editor.cpp
@@ -658,8 +658,8 @@ void SceneTreeEditor::_renamed() {
        String new_name=which->get_text(0);
        if (new_name.find(".") != -1 || new_name.find("/") != -1) {

-               error->set_text(_TR("Invalid node name, contain illegal character(s):\n ('.', '/')"));
-               error->popup_centered_minsize(Size2(400,80));
+               error->set_text("Invalid node name, the following characters are not allowed:\n    \".\", \"/\"");
+               error->popup_centered_minsize();
                new_name=n->get_name();
        }

I've actually cherry-picked your fix and added my suggestions to it in #2767.


multi configs

would be nice to add a switch on startup to pick different configs or different accounts
example in
https://github.com/gregtampa/Pokemon-Go-Bot-Working-Hack-API/blob/master/config.json

Already have this implemented.
`./pokecli.py -cf configs/filename.json