Your X-Clip or OpenAI vanilla CLIP?

First amazing that you try to recreate this.
My question is are you going to plan to use your X-CLIP implementation or just use the basic OpenAI vanilla CLIP. From what I gathered from the official presentation I think most of the power of the DALLE-2l seems to lie in the clip embedding and obviously the move away from autoregression to diffusion and therefore also the prior diffusion into the CLIP embeddings. But OpenAI only gives out their rather weak clip models so its capabilities will be limited.

@Mut1nyJD sure! i can bring in x-clip :) https://github.com/lucidrains/x-clip (for those who do not know that line of research)

@Mut1nyJD i'm building out the prior network this morning :) should take 4-5 hours at most

loading up on :coffee: :coffee: :coffee: :coffee: :coffee:

@Mut1nyJD also, the group over at Laion have already replicated some of the CLIP models, and they will probably be on track on training a bigger better one. they are using openclip afaik. i'll probably aim to make the library compatible with all versions of CLIP, through some adapter or bridge class

@Mut1nyJD i'm building out the prior network this morning :) (i'm going for the best variant, the diffusion model that denoises the image embedding) should take 4-5 hours at most

Yeah this bit I find most confusing in terms of how they do it. What I don't quiet understood is this part DDIM and the Decoder is DDPM or are both DDIM.

spent all morning wrestling with jax jit problems 😐, i'll try to bring my mind to focus on it this afternoon!

@Mut1nyJD haha, i don't even know what DDIM is, i haven't been following the denoising literature that closely (but should probably start doing so)

also wow, so many github stars and i didn't even write any code yet lol

also wow, so many github stars and i didn't even write any code yet lol

For some reason it got in my google news feed lol. We will watch your career with great interest.

@MegaR how was it in google news? you probably mean OpenAI, who i'm not affiliated with

@lucidrains No your repo was directly linked in the google assistant news feed for me. Like when you swipe left from the homescreen on pixel phones. It probably detected I was interested in DALL-E2

@MegaR oh wow, no way... i need to give google assistant news a try, it must have gotten a lot better

@MegaR oh wow, no way... google assistant news feed recommending a day-old repository without code is not something i would have guessed

alright, wrapped up https://github.com/lucidrains/palm-jax , going to knock out the prior network (the diffusion variant) by 6-7p, then go enjoy some san francisco nightlife

Ice Cream agrees with that plan :)

Supporting https://github.com/crowsonkb/cloob-training could be nice as well as it removes the need to do a text to image mapping, cloob space is already text/image aligned.
Also there will be some bigger cloobs.

haha, so many people emailing me about this repository... just relax, is there any reason to think i can't / won't do it? ;)

@rom1504 hi Romain, you follow me everywhere, even after joining Google lol
yea, x-clip should support CLOOB-like loss already

oh, do you mean to say CLOOB like architecture removed the need for the prior neural network? Is this just speculation atm?

ok, i think i'll save this for next week. measure twice cut once kind of thing. plus i probably shouldn't make it look too easy ;)

closing since i've already answered the original question

It's probably best to stick with the paper for now, and if the open source community can rid of the prior network with some modifications to CLIP, we need to verify with experimental comparisons between the two methods

Alstroemeria trained a cloob that is almost as good as clip and released it at the link I posted, the alignements between image and text embedding is better with it.
Is it enough to remove the needed of the prior network inside dalle 2 ? That's the hope but indeed it's just an hypothesis currently
Definitely agree that comparing the 2 methods will be interesting!

Yeah agreed! Katherine is doing great work as usual :)

@Mut1nyJD haha, i don't even know what DDIM is, i haven't been following the denoising literature that closely (but should probably start doing so)

As far as I know it is a different / more efficient way of sampling in terms of required time steps. But training scheme is the same between the two.

@Mut1nyJD ahh ok, let's forget about fast for now. there are so many papers trying to make DDPMs faster, and the dust hasn't totally settled yet


V1.4

Fixed issue #92

Codecov Report

Merging #247 (c5aee98) into master (7a92aab) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #247      +/-   ##
============================================
+ Coverage     69.86%   69.88%   +0.01%
- Complexity      597      598       +1
============================================
  Files            93       93
  Lines          1978     1979       +1
  Branches        216      217       +1
============================================
+ Hits           1382     1383       +1
  Misses          516      516
  Partials         80       80

Impacted Files
Coverage Δ
Complexity Δ

.../seedu/forgetfulnus/model/flashcard/FlashCard.java
93.33% <100.00%> (+0.11%)
23.00 <0.00> (+1.00)

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 7a92aab...c5aee98. Read the comment docs.


2 issues; .NET 6 desktop runtime requirement not obvious, Failed to start up if debug.log is missing

Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'WindowsBase, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'WindowsBase, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'
at Program.Main(String[] args)

Running the new DW2Net6Win.exe causes this issue.

I need some more details on this. Are you able to contact us on Discord?
I'd like to get your OS version and to know if you have .NET 6 installed.
I believe you may need to choose the middle option from
https://dotnet.microsoft.com/en-us/download/dotnet/6.0/runtime where it
says "Run desktop apps"
On Wed, Apr 6, 2022, 6:55 AM Benc85 @.***> wrote:

Running the new DW2Net6Win.exe causes this issue.
—
Reply to this email directly, view it on GitHub
https://github.com/DW2MC/DW2ModLoader/issues/3#issuecomment-1090133172,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIC7PYEXDA5L4REJG5DTRXLVDVUTPANCNFSM5SVUBQAA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Yes, i'm about on discord. I'm running win 10, and have installed what you've suggested.
Running DW2Net6Win.exe now produces this error:
Created AppContainer isolated process 14368
DW2Net6Win v1.1.2
Unhandled exception. HarmonyLib.HarmonyException: Patching exception in method System.Void Xenko.Animations.AnimationUpdater::Update(Xenko.Engine.Entity entity, Xenko.Animations.AnimationClipResult result)
---> System.TypeInitializationException: The type initializer for 'HarmonyLib.HarmonySharedState' threw an exception.
---> System.NotSupportedException: Body-less method
at MonoMod.Utils.DynamicMethodDefinition._CopyMethodToDefinition()
at MonoMod.Utils.DynamicMethodDefinition.Reload()
at MonoMod.Utils.DynamicMethodDefinition..ctor(MethodBase method)
at MonoMod.RuntimeDetour.Platforms.DetourRuntimeILPlatform..ctor()
at MonoMod.RuntimeDetour.Platforms.DetourRuntimeNETPlatform..ctor()
at MonoMod.RuntimeDetour.Platforms.DetourRuntimeNETCorePlatform..ctor()
at MonoMod.RuntimeDetour.Platforms.DetourRuntimeNETCorePlatform.Create()
at MonoMod.RuntimeDetour.DetourHelper.get_Runtime()
at HarmonyLib.HarmonySharedState..cctor()
--- End of inner exception stack trace ---
at HarmonyLib.HarmonySharedState.GetPatchInfo(MethodBase method)
at HarmonyLib.PatchClassProcessor.ProcessPatchJob(Job job)
--- End of inner exception stack trace ---
at HarmonyLib.PatchClassProcessor.ReportException(Exception exception, MethodBase original)
at HarmonyLib.PatchClassProcessor.Patch()
at HarmonyLib.Harmony.b__10_0(Type type)
at HarmonyLib.CollectionExtensions.Do[T](IEnumerable1 sequence, Action1 action)
at HarmonyLib.Harmony.PatchAll(Assembly assembly)
at HarmonyLib.Harmony.PatchAll()
at Program.Main(String[] args)

Might this be an issue with the new <IP_ADDRESS> patch? I've just deleted all the recent DW2 Mod loader stuff and reinstalled version 1.1.0 (when i never had any issues), and its throwing the same error

Probably, I haven't worked with anything above v<IP_ADDRESS> yet. I'll get to it
this evening.
On Wed, Apr 6, 2022, 1:00 PM Benc85 @.***> wrote:

Might this be an issue with the new <IP_ADDRESS> patch? I've just deleted all
the recent DW2 Mod loader stuff and reinstalled version 1.1.0 (when i never
had any issues), and its throwing the same error
—
Reply to this email directly, view it on GitHub
https://github.com/DW2MC/DW2ModLoader/issues/3#issuecomment-1090496152,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIC7PYBJLRZPPZHJ4FLCA3LVDW7MFANCNFSM5SVUBQAA
.
You are receiving this because you commented.Message ID:
@.***>

Ive just tried the new v1.2 and im getting this error now:
System.ArgumentException, System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e
The path 'C:\Program Files\Steam\steamapps\common\Distant Worlds 2\debug.log' does not exist.
at DW2Net6Win.Isolation.Windows.StartIsolatedProcess(String appContainerName, String path, String[] commandLineArguments, IEnumerable1 capabilitySids, Boolean attachToCurrentProcess, IEnumerable1 fileAccess, String workingDirectory)
at Launcher.Run(String[] args)
=== === === === === === WARNING === === === === === ===
Failed to create AppContainer isolation environment!
You may need to run this executable as admin in order to
run the game isolated. Without AppContainer isolation,
it may be unsafe to run the game with modifications.
If you wish to continue without isolation, press the
[F5] key. The exception above has some clue as to why
the failure occurred, it is likely that directory
permissions are already more restricted than the
launcher expected for the current user.
=== === === === === === WARNING === === === === === ===
          Press any key to exit.

Same error when run as admin. Pressing F5 enters the game, seemingly with V6 running, but with the isolation disabled.
Thanks for your continued work Tyler, your AMD fix single-handedly turned this game around for me :)

Ah, I'll see if I can fix that.

@Benc85 try the latest GitHub actions artifact and let me know if it works any better for you?

I followed your advice and made a debug.log and the launcher worked flawlessly @tyler-in. I haven't had a chance to run the v1.2.1 release yet, but ill get back to you shortly if i have any issues.  Thanks for all the help mate

Everything looks good Tyler. no issues with startup at all. Nice work


Check manifest file when createing table

What type of enhancement is this?
Tech debt reduction
What does the enhancement do?
Create table may fail and will be retried by procedure. To make sure manifest files are correct we should check its content if there are any.
Implementation challenges
No response

I'd like to work on this issue if you don't mind. If so, Do you have any guidelines for addressing this issue?

@evenyag maybe you have some thoughts?

I haven't figured out a clear solution for this. What's more, I plan to refactor manifests and table engine in #1869 which might fix this issue. Let's postpone this.

Looks like we don't need to fix this in 0.4, because we removed the table manifest and replaced it with metasrv management.


Package appears as installed but it is not

bash-3.2$ brew cask list
flash-player		    iterm2-beta			mou			    sublime-text		vlc
google-chrome		    java			pgweb			    virtualbox			xquartz
intellij-idea-ce	    libreoffice			pycharm-ce		    virtualbox-extension-pack

bash-3.2$ brew cask info pycharm-ce
pycharm-ce: 2016.1.4
Jetbrains PyCharm Community Edition, PyCharm CE
https://www.jetbrains.com/pycharm
Not installed
https://github.com/caskroom/homebrew-versions/blob/master/Casks/pycharm-ce.rb
==> Contents
PyCharm CE.app (app)

bash-3.2$ brew cask remove pycharm-ce
Error: pycharm-ce is not installed

try brew cask remove --force pycharm-ce

It works! Thanks a lot!


upgrade chi to v5, prometheus to latest, fix rctx type panic

@766b any updates regarding this PR?


update code.google.com to golang.org/x/

code. google.com is going away, and the oracle has moved to golang.org/x/tools/cmd/oracle

Thanks, this would have saved me a bit of trouble during installation. Let's merge it!


Best Time to Buy and Sell Stock.

Hi, Project Maintainers. I would like to include this important algorithm in the arrays section of the code. The folder in which I'll add code is Algo-Tree/code/java. I would like you to assign this issue to me. Thank you.

Sir @tarun26091999
Please assign this problem to me. I would like to solve this in c++.
I will surely provide the code with proper documentation and comments.

already present at #462


[playground] Add workflow to Update K8s Cleanup Script

Current Behavior

Recently a new script was added to ensure that k8s cluster is cleanup nightly.
Though the script inside VM is not set to automatically update when a change is made to https://github.com/meshery/meshery/blob/master/.github/scripts/playground-cleanup.sh
Desired Behavior

When a change is made in https://github.com/meshery/meshery/blob/master/.github/scripts/playground-cleanup.sh then ensure that a new workflow is used to update the playground-cleanup.sh script places in c3-medium-x86-01-meshery VM at the root directory.
Implementation

Add a new workflow that watches for /.github/scripts/playground-cleanup.sh
SSH into c3-medium-x86-01-meshery
Update the script in /playground-cleanup.sh of root dir of c3-medium-x86-01-meshery with the new change in the master branch /.github/scripts/playground-cleanup.sh

Acceptance Tests

Reference:

Original post: https://github.com/meshery/meshery/pull/7799#issue-1743439241
https://github.com/meshery/meshery/blob/master/install/playground/readme.md

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

// @Revolyssup

I can create this workflow . May i work on this one

Yeah sure go ahead. Join today's CI call for more context

Reference diagram:


Document total resources available and resources available to each job

@moraru, @mpw6  what is the total amount of resources available to the dispatch app (or could be allocated, potentially even just temporarily)?

Cores
RAM
Disk space

I started slots for this information in the FAQ (biosimulations/apps/dispatch/src/app/components/help/faq/faq.component.html) in a new branch.

That is a loaded question.  Since biosim is running in the general pool, jobs can potentially use up to the entire amount of non-reserved resources (thousands of cores, TBs of RAM, and PBs of storage).  Obviously not the proper answer, need some thinking about how to address.
From: Jonathan Karr @.>
Sent: Tuesday, March 30, 2021 12:39 AM
To: biosimulations/Biosimulations @.>
Cc: Moraru,Ion @.>; Mention @.>
Subject: [biosimulations/Biosimulations] Document total resources available and resources available to each job (#2348)
*** Attention: This is an external email. Use caution responding, opening attachments or clicking on links. ***
@moraruhttps://urldefense.com/v3/__https:/github.com/moraru__;!!N0rdg9Wr!_mZAY44QAY6VuHJHu6izgak2j-4rC61Ak3GAgPSOKpuZh8F80AmB0HlJ1bNRXw$, @mpw6https://urldefense.com/v3/__https:/github.com/mpw6__;!!N0rdg9Wr!_mZAY44QAY6VuHJHu6izgak2j-4rC61Ak3GAgPSOKpuZh8F80AmB0HmndT26ww$ what is the total amount of resources available to the dispatch app (or could be allocated, potentially even just temporarily)?

Cores
RAM
Disk space

I started slots for this information in the FAQ (biosimulations/apps/dispatch/src/app/components/help/faq/faq.component.html) in a new branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/biosimulations/Biosimulations/issues/2348__;!!N0rdg9Wr!_mZAY44QAY6VuHJHu6izgak2j-4rC61Ak3GAgPSOKpuZh8F80AmB0Hl0CDFVXQ$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ABYODNT5TM7MIP5KYKV7N73TGFIVPANCNFSM42BGOYFA__;!!N0rdg9Wr!_mZAY44QAY6VuHJHu6izgak2j-4rC61Ak3GAgPSOKpuZh8F80AmB0HnrC-9VPg$.

This what I'm thinking of responding with and posting to the FAQ:

Currently, RunBioSimulations has access to over 50 TFLOPs from 2,168 cores, 11 TB RAM, and 8 PB storage. Of this, 48 cores and 384 GB RAM are currently dedicated to RunBioSimulations. This dedicated allocation is easily configurable and can easily be expanded as user demand grows. The additional cores and RAM are shared with other users of the HPC system at UConn Health. This sharing is managed using SLURM.


sources and sinks made optional and other minor improvements

Purpose
Make sources and sinks optional for 2M methods; and other minor improvements
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.
I have read and checked the items on the review checklist.

Thank you!

bors r+


undojoining commands that populate the results buffer for smoother undo

doesn't work due to random errors about undojoin not being allowed after undo


feat(sage-monorepo): add GH workflow to link Dockerfile

Closes #2572

Hadolint extension
The hadolint extension for VS Code requires hadolint to be installed.

hadolint: Cannot find hadolint from system $PATH. Please install hadolint.

@tschaffter is the expectation that we will name branches related to schematic following patterns: schematic/**? FAIR data team currently likes to use Jira ticket number or feature name to name a branch.. but if the expectation is to name branches with pre-fix schematic/**, we could adjust. And as dev, I kind of like to be able to trigger a GH action manually for a specific branch. Could you add that too?


Update winsorization.md

🚀 Deployed on https://656698e2e1c25e007b19fd08--cozy-fox-0defec.netlify.app


often show java.net.SocketException: Connection reset by peer

use the newest code,then often show java.net.SocketException: Connection reset by peer,how can i solve the problem?

I am facing the same issue but only for the SDK version before LOLLIPOP

Is there any other way to update TLS version for an older device by making customization in SSLSocketFactory class? And I also noticed that the class has deprecated.

thanks @bryosabian . also there is an option to use Conscrypt to solve tls and sni issue for older device


gltfio: introduce support for extras strings.

This allows clients to obtain the extras strings for the asset and
for individual nodes.  Note that extras for buffer views, accessors, etc
are not supported, although C++ clients can access the raw cgltf data
directly if they need to.

Hello!  How would one use this to get the extras from a Scene?  For example, in the attached sample glTF file, I have one scene and some extra data on it.  I'd like to get that extra data.
extra_data_on_scene.gltf.zip


UNet2D HipMRI Segmentation (47443349)

Name: Alexander Pitman
Student ID: 47443349
This pull request includes my implementation of a UNet2D, trained to segment processed medical image slices from the HipMRI Study on Prostate Cancer according to Task 1 on the COMP3710 Project Task Sheet.
The trained UNet2D model achieved an average Dice Score of 0.985, 0.981, 0.922, 0.956, 0.758, and 0.702 on the test set for the Background, Body, Bone, Bladder, Rectum, and Prostate labels respectively.
I have structured my project as follows:
README.MD: contains project documentation including a description the dataset, a description of my model, how it works, the problem it solves, the results, dependencies, and instructions for usage.
dataset.py: contains data pipeline functionality.
modules.py: contains UNet2D network architecture.
train.py: contains model training algorithm.
predict.py: contains model testing where predictions are made on the test set with dice scores reported and sample predictions visualised.
utils.py: contains various global constants and utility functions such as loss functions, dice score calculation, and seed setting.
images/: contains all the images used in the README.

This is an initial inspection, no action is required at this point

Solves problem: The problem was well solved, 5 labels reached over 0.75 Dice score and one label reached 0.7 Dice score. Training loss, prediction figures and metrics were provided. Nice and well-structured documentation.
Implementation: Nice uni tests. Data augmentation implemented. Some hpyerparameters were hard-coded. Good and clear implementation
Design: Great
Comment: Sufficient
Difficulty: Easy

Marking

Good/OK/Fair Practice (Design/Commenting, TF/Torch Usage)
Good design and implementation.
Spacing and comments.
Header blocks.
Recognition Problem
Good solution to problem.
Driver Script present.
File structure present.
Good Usage & Demo & Visualisation & Data usage.
Module present.
Commenting present.
No Data leakage found.
Difficulty : Easy. UNet2D (Easy Difficulty)-10
Commit Log
Good Meaningful commit messages.
Good Progressive commits.
Documentation
Readme :Good.
Model/technical explanation :Good.
Description and Comments :Good.
Markdown used and PDF submitted. PDF checked.
Pull Request
Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch). Submitted on time.
No Feedback required.
Request Description is good.
TOTAL-10

Marked as per the due date and changes after which aren't necessarily allowed to contribute to grade for fairness.
Subject to approval from Shakes

Segmentations not one hot encoded -2 additionally


Update DPG ResponseClassifiers

Fixes: https://github.com/Azure/autorest.csharp/issues/2006

Once https://github.com/Azure/azure-sdk-for-net/pull/27379 merges, we will need to change the CoreResponseClassifier name to StatusCodeClassifier.  Probably better to hold this until that happens.


chore(files): fix list layout

What this PR does 📖
before

after

would be even better if the bullets were missing!


Links on a main web site are different from themes.md and do not work — 404 Erorr

Solarized Light theme used as an example:
A broken link from web site (it doesn't work):
$ wget -O xt http://git.io/v3DB5 && chmod +x xt && ./xt && rm xt
...
2017-10-07 22:19:24 ERROR 404: Not Found.

A working link from themes.md (this one works as expected):
$ wget -O xt  https://git.io/v5mXi && chmod +x xt && ./xt && rm xt
HTTP request sent, awaiting response... 200 OK
Length: 7743 (7.6K) [text/plain]
Saving to: ‘STDOUT’

100%[=================================================================================>] 7,743       --.-K/s   in 0.02s

2017-10-07 22:20:14 (372 KB/s) - written to stdout [7743/7743]

It looks like some of the links have '.' where a '-' should be. For example, the One Light theme has the following shortened url:
http://git.io/vs7UQ
Which resolves to:
https://raw.githubusercontent.com/Mayccoll/Gogh/master/themes/one.light.sh
However, the correct url is
https://raw.githubusercontent.com/Mayccoll/Gogh/master/themes/one-light.sh
Notice the '-' in one-light.
As a work around, navigate to the raw page for the theme you want and use that url instead of the shortened link.


made adjustments so there server is overtime

This config just  gives the server overtime rules as its default

@JustinBenedick - rejecting this pull request as it's a request to merge the change into master and not into a new branch.

Just tried again. I think it needs an empty overtime branch to compare to.

changed base to new overtime


Built in Tile add etc follows numpy dtype conversions

If a Python rf_types.Tile built in arithmetic function results in numpy converting the underlying dtype we will honor that in the returned Tile's cell_type. Also makes already merged unit test test_udf_np_implicit_type_conversion pass.
Signed-off-by: Jason T. Brown<EMAIL_ADDRESS>

CI failure due to Py 2.7/3.6 stuff:

==================================== ERRORS ====================================
________________ ERROR collecting tests/PyRasterFramesTests.py _________________
.eggs/pytest-3.4.2-py2.7.egg/_pytest/python.py:411: in _importtestmodule
    mod = self.fspath.pyimport(ensuresyspath=importmode)
.eggs/py-1.8.0-py2.7.egg/py/_path/local.py:701: in pyimport
    __import__(modname)
.eggs/pytest-3.4.2-py2.7.egg/_pytest/assertion/rewrite.py:213: in load_module
    py.builtin.exec_(co, mod.__dict__)
tests/PyRasterFramesTests.py:5: in <module>
    from pyrasterframes import *
pyrasterframes/__init__.py:12: in <module>
    from .rf_types import *
E     File "/home/travis/build/locationtech/rasterframes/pyrasterframes/python/pyrasterframes/rf_types.py", line 143
E       def __init__(self, cell_type_name: str):
E                                        ^
E   SyntaxError: invalid syntax
!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!

Hmm not ready to merge yet because python test test_no_data_udf_handling is now failing.


Update tree-sitter-nickel to new unstable branch

We decided to change the syntax for enum tags in Nickel from `Tag to 'Tag. The newest unstable branch of the Nickel tree-sitter grammar supports that change. Additionally, it reflects the change of naming for some builtin types, namely Str became String and Num became Number.
None of these changes should affect the parse tree structure, ~so no queries should need changing.~ Well, that was a nice dream 😅

You replaced the backticks from the tests, in 696c12f, but there needs to be some changes to the query files too, with the change of Num => Number and Str to String:
--- a/languages/nickel.scm
+++ b/languages/nickel.scm
@@ -52,9 +52,9 @@
     "rec"
     "Array"
     "Dyn"
-    "Num"
+    "Number"
     "Bool"
-    "Str"
+    "String"
     "->"
     "=>"
     ; Infix operators

...also:
--- a/topiary/tests/samples/expected/nickel.ncl
+++ b/topiary/tests/samples/expected/nickel.ncl
@@ -449,7 +449,7 @@
         let length = %length% array in
         let first = %elem_at% array 0 in
         let rest = %array_slice% 1 length array in
-        let parts = partition (fun x => (cmp x first == `Lesser)) rest in
+        let parts = partition (fun x => (cmp x first == 'Lesser)) rest in
         if length <= 1 then
           array
         else
@@ -737,7 +737,7 @@
       ```nickel
         is_enum true =>
           false
-        is_enum `false =>
+        is_enum 'false =>
           true
       ```
       "%
@@ -2342,7 +2342,7 @@
       ```nickel
       from 42 =>
         "42"
-      from `Foo =>
+      from 'Foo =>
         "Foo"
       from null =>
         "null"

Thanks for the quick feedback and the suggestions! I hope the tests pass :)

Why did the Wasm tests actually succeed here?


some naming replacing in examples to make it inline with cli commands

This is discussed with the team.
This PR it the effort to unify names between different services, so we can have examples and tests consistently generated with similar name conventions. for instance "MyResourceGroup"

Can one of the admins verify this patch?

Automation for azure-sdk-for-go


socket activation from unbound fails since 2.0.15

Hi there,
I am an old fan of dnscrypt-proxy, and till version 2.0.15, I used socket activation from unbound. Namely,
in dnscrypt-proxy.toml, I have
listen_addresses = []
A dnscrypt-proxy.socket systemd activation unit with
[Socket]
ListenStream=<IP_ADDRESS>:40
ListenDatagram=<IP_ADDRESS>:40
and an unbound service with unbound.conf having a forward zone as:
forward-zone:
   name: "."
   forward-addr: <IP_ADDRESS>@40
This configuration seems to be broken with dnscrypt-proxy 2.0.15, but was working with 2.0.12. I think there is an issue with socket activation because if I stop the dnscrypt-proxy.socket unit, change dnscrypt-proxy.toml into
listen_addresses = ['<IP_ADDRESS>:40', '[::1]:40']
Then, everything works fine again. So unbound properly forward the query to <IP_ADDRESS>:40, but somehow, dnscrypt-proxy socket does not succeed to activate dnscrypt-proxy.service anymore!
Any help appreciated!
Cheers.

Hi!
This is a duplicate of https://github.com/jedisct1/dnscrypt-proxy/issues/480


Remove vm-manager project from manifest

Tracked-On: OAM-106621

Merged internally, hence merging it here.
more details, logs, and binaries: /absp/builders/celadon-mergerequest/builds/3228


truncated in XX_httplib_sprintf() is a boolean

Type of the parameter should be changed looking at the functionality of it.

Fixed: https://github.com/lammertb/libhttp/commit/3b5e4e5a75a01ad84dffae2def4d182a52668576


Pop up dark keyboard on iOS

How popup dark keyboard on iOS specific platform? Is there any api on this?

Follow-up to #18540

Duplicate of #25658

@jinfagang Please don't ask questions on Github. Stackoverflow is better platform to ask. If some thing is not working as intended and you think it is a bug, or if you have any suggestion regarding this, you are most welcome on Github. Thanks


Allowed IV lengths for AES128 and AES256 CTR mode?

What is the range of allowed IV lengths for AES CTR mode in EverCrypt_CTR.h ?
In v0.2.1, in the c source file (EverCrypt_CTR.c, line 138), I see an 'if'-statement that checks that the iv_len is at least 12 bytes.
However, in line 150 there is a calloc for 16 bytes.
Does that mean that for AES128 CTR mode the IV len range is 12 up to and including 16 bytes?
Same question for the AES256-variant.
kind regards,
Niek

Hi Niek,
Thanks for the report. First, a disclaimer: as stated on https://hacl-star.github.io/EverCryptCTR.html, EverCrypt_CTR is a WIP API. If you are authoring something serious that intends to use it, please shoot us an email to let us know so that we can try to make it better :).
From what I remember, the 16 bytes thing is just a complication of the way AES is specified in the RFC, and it's a corner case where (very fuzzy recollections here) the IV overlaps with the block number, but for some reason the RFC still allows or at least specifies this case. You should use 12 bytes in any case.
IV reduction is not available for AES-CTR since it relies on GCM.
Hope this helps,
Jonathan

@karthikbhargavan please chime in if my recollections are wrong

You should use 12 bytes in any case.

Thanks! I will do that.


failed to push some refs

I'm trying to push existing repository to fresh Dropbox repository via git-remote-dropbox.
When I try to do it I catch an error:
Pushing to dropbox:///me/reps/repository.git
debug: writing: /me/reps/repository.git/objects/a1/6cdddb802b28e9b7f52b01222117a09f1d89a7
...
debug: writing: /me/reps/repository.git/objects/2b/30cfc121e27ab54eee75b2a7dbdb6244b402f5
Traceback (most recent call last):
  File "/home/user/.local/bin/git-remote-dropbox", line 8, in <module>
    sys.exit(main())
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/cli/helper.py", line 23, in main
    helper.run()
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/helper.py", line 108, in run
    self._do_push(line)
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/helper.py", line 152, in _do_push
    self._push(src, dst)
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/helper.py", line 215, in _push
    for done, _ in enumerate(res, 1):
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 870, in next
    raise value
  File "/usr/lib/python3.9/multiprocessing/pool.py", line 125, in worker
    result = (True, func(*args, **kwds))
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/util.py", line 103, in __call__
    return method(self._obj, *args)
  File "/home/user/.local/lib/python3.9/site-packages/git_remote_dropbox/helper.py", line 287, in _put_object
    self._connection().files_upload(data, path, mode, mute=True)
  File "/home/user/.local/lib/python3.9/site-packages/dropbox/base.py", line 2861, in files_upload
    r = self.request(
  File "/home/user/.local/lib/python3.9/site-packages/dropbox/dropbox_client.py", line 346, in request
    raise ApiError(res.request_id,
dropbox.exceptions.ApiError: ApiError('dddddddddddddddddddddddddddddddd', UploadError('path', UploadWriteFailed(reason=WriteError('conflict', WriteConflictError('file_ancestor', None)), upload_session_id='AAAAAAAAAAAAAAAAAAAAAA')))
error: failed to push some refs to 'dropbox:///me/reps/repository.git'

There isn't anything in the folder
─➤  ls ~/Dropbox/me/reps/repository.git                                                                                130 ↵
ls: cannot access '/home/user/Dropbox/me/reps/repository.git': No such file or directory

I've configured remote as follows:
[remote "origin"]
	url = dropbox:///me/reps/repository.git
	fetch = +refs/heads/*:refs/remotes/origin/*

P.S.
─➤  pip list | grep dropbox
dropbox                       11.13.1
git-remote-dropbox            1.1.4

Hmm, I have not seen that kind of error before. The docs say that the file_ancestor error means that "There’s a file at an ancestor path, so we couldn’t create the required parent folders.". Is any of me, me/reps, or me/reps/repository.git a file?
Also, by the way, I recommend using selective sync and disabling sync for the directory where you are placing repos, e.g. /me/reps.

Is any of me, me/reps, or me/reps/repository.git a file?

No, ~/Dropbox/me, ~/Dropbox/me/reps are folders and ~/Dropbox/me/repository.git is absent.
What can I do to help identify the problem?

Also, by the way, I recommend using selective sync and disabling sync for the directory where you are placing repos, e.g. /me/reps.

Thanks!  Bit this recommendation isn't related to the issue?

Yeah, the recommendation is probably not related.
I'm not entirely sure what is causing this. Do you get the same error if you try pushing again? Does it seem to be deterministic (i.e. gives an error when pushing the same file), or does it fail at different points?
If you delete the directory in Dropbox (or alternatively, change the remote to point somewhere else, e.g. dropbox:///test) and push, does it still fail in the same way?
What platform are you on? (OS / Python version)
Do you have large files in this repo (say, > 20 MB)?

I've made several experiments. First conclusion: It depends on arbitrary factor.
Push to test have worked perfectly fine:
─➤  git push                                                                                1 ↵
Writing objects: 100% (5026/5026), done.
To dropbox:///test
 * [new branch]      master -> master

Pushing to me/reps/repository have worked fine. And only pushing to me/reps/repository.git have yold and error:
Writing objects:  37% (1900/5026)error: exception while writing objects (run with -v for details)

Then I've deleted repository folder from Dropbox and then push it again and everything works fine!
P.S. Repository consists of quite many files (excluding .git config):
ls -alR | wc -l
892

The biggest file has size of 261K and total size of dir is 4.8M.

Okay, so everything is working now? Hard to tell how it got into that state that caused the file_ancestor write conflict. Given that we haven't seen this reported with git-remote-dropbox before, it's possible that it was due to something else.
Yes, the repository consists of many files. git-remote-dropbox keeps all objects loose (supporting packfiles while having client-side concurrency control seemed tricky; it should be possible, but it's not currently implemented). So individual objects are zlib-compressed but there's no delta compression, so repositories take up a decent amount of space.

It seems that ok, but definitely not stable :(
Thanks for your comments!

Do you still have a copy of the folder on Dropbox that was being problematic?

I've tested git push several times in different (and sometimes same) location inside Dropbox folder.
It occasionally fails. It can be ok for location and next time not ok for the same location

git-remote-dropbox doesn't have great logging (and I'm not even sure what log output would be useful beyond the Dropbox API error). It would be helpful for debugging if I could repro this on my machine. I wonder if it has anything to do with the repository contents? Does the same issue happen if you push other repos? (E.g. clone a random github repo, like git-remote-dropbox, and then push it to Dropbox). Or is it just with your particular repo?

Closing due to inactivity.


image rotation only saves after crop frame is also moved

related trello card: https://trello.com/c/7vKgolS1

Related to #3827


Is generating-a-provider doc up do date?

What problem are you facing?
The example has the following block of code :
pc := tjconfig.NewProviderWithSchema([]byte(providerSchema), resourcePrefix, modulePath,
    tjconfig.WithIncludeList([]string{
        "github_repository$",
        "github_branch$",
    }))

but in upjet template this code looks really different
	pc := tjconfig.NewProvider([]byte(providerSchema), resourcePrefix, modulePath, nil,
		tjconfig.WithIncludeList(ExternalNameConfigured()),
		tjconfig.WithDefaultResourceOptions(
			ExternalNameConfigurations(),
		))

is this doc need to be updated? or i did smth wrong?

@IvanTurgenev1 The PR https://github.com/upbound/upjet/pull/115/ contains most up-to-date information including the TF provider documentation parsing for CRD API references!


No hint path defined for [postmark]

error.txt
The error is :
No hint path defined for [postmark].
Integration Method using: MailMessage with notifiable in Laravel

Detailed description
I am trying to send an email from existing implementation of laravel notification using postmark email template method.
toMail() which extends by Notification

call from controller using user->notify(with Notification class)

Context
Why is this change important to you? How would you use it?
How can it benefit other users?
Possible implementation
Not obligatory, but suggest an idea for implementing addition or change.
In Local environment
Include as many relevant details about the environment you experienced the bug in and how to reproduce it.

Laravel Postmark Version: v2.11.2
Laravel Version: 6.20.44
PHP Version: 7.4.0

Hey @ervshaikh, do you by chance have this package's service provider in the extra.laravel.dont-discover key inside your composer.json file? The only way I can reproduce this is by having the service provider be ignored/not run.


Update QuestionAnsweringTestEnvironment.cs with new env vars

Be sure eventually to update QuestionAnsweringTestEnvironment.cs with the new env vars. Technically, these should still work since the base TestEnvironment class falls back to prepending "AZURE_", but while we're busy fixing up QA anyway, let's get everything in sync for easier troubleshooting, should the need ever arise.
Originally posted by @heaths in https://github.com/Azure/azure-sdk-for-net/issues/28232#issuecomment-1101812246

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.12826914,Azure.Identity:0.108394265,Event Grid:0.080433756'
Label prediction was below confidence level 0.6 for Model:CategoryLabels: 'Mgmt:0.49574965,Client:0.45135722,Service:0.046528593'

Duplicated of #29958


Undefined Variable causes Helm Template Failure

Due to undefined variable check introduced in https://github.com/CrunchyData/postgres-operator-examples/pull/64 the Helm Chart is broken.
Let me copy what I commented to the PR:
helm template . --debug
install.go:178: [debug] Original chart version: ""
install.go:199: [debug] CHART PATH: /home/alex/other/postgres-operator-examples/helm/install

Error: template: pgo/templates/manager.yaml:23:23: executing "pgo/templates/manager.yaml" at <eq .Values.debug false>: error calling eq: incompatible types for comparison
helm.go:88: [debug] template: pgo/templates/manager.yaml:23:23: executing "pgo/templates/manager.yaml" at <eq .Values.debug false>: error calling eq: incompatible types for comparison

What version of Helm are you using?
I did not reproduce this while testing.

What version of Helm are you using?
I did not reproduce this while testing.

It is Helm 3.7.1:
$ helm version
version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.16.9"}
$ uname -a
Linux ubimachine 5.13.0-21-generic #21-Ubuntu SMP Tue Oct 19 08:59:28 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

I used similar. However to avoid confusion and/or other issues, I reverted it in 538c3c3.


Check filename for invalid characters before saving e.g. "a:b.obj"

Per default the Data Manager's "Save..." option via QmitkIOUtil proposes the DataNode's name as the filename. When the node name is "a:b" or simliar, this might produce files that are empty and named slightly different (in the example "a" instead of "a:b.obj"). The problem is that file systems do not accept all characters (esp. Windows)
Without a in QmitkIOUtil writers might (and some do) create empty files
that are not even named as expected (e.g. "a" instead of "a:b.obj").
The opening of a std::ofstream for such a filename does not seem to set
any fail-bits (all checked for in the affected writer class). So even writers that correctly check for the output stream state won't notice the problem.
The alternative to a check here would be a check in each possible writer class. This seems worse.
Signed-off-by: Daniel Maleike<EMAIL_ADDRESS>

Thank you!


Removes unused members from ConfigProvider

Commit Message: Removes unused members from ConfigProvider
Additional Description:
There were a few members in ConfigProvider class, and this deletes
them, which increases the test coverage as a result.
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

/retest

/retest

/retest

Think this needs your input @adisuissa


Error when streaming 'large' files

Hi there
I'm trying to use createWriteStream to transfer files from my client to a server.
When the files are bigger than 4-5MB the transfer stops at some point and hangs and then either ECONNRESET or ETIMEDOUT errors occur. If the files are smaller they are usually transferred without problems.
The server is running SSH-2.0-CompleteFTP_10.2.1
Here is a log detailing the events:
DEBUG: Local ident: 'SSH-2.0-ssh2js0.1.20'
DEBUG: Client: Trying HOST_IP on port 22 ...
DEBUG: Client: Connected
DEBUG: Parser: IN_INIT
DEBUG: Parser: IN_GREETING
DEBUG: Parser: IN_HEADER
DEBUG: Remote ident: 'SSH-2.0-CompleteFTP_10.2.1'
DEBUG: Outgoing: Writing KEXINIT
DEBUG: Parser: IN_PACKETBEFORE (expecting 8)
DEBUG: Parser: IN_PACKET
DEBUG: Parser: pktLen:548,padLen:10,remainLen:544
DEBUG: Parser: IN_PACKETDATA
DEBUG: Parser: IN_PACKETDATAAFTER, packet: KEXINIT
DEBUG: Comparing KEXINITs ...

Does sftp.fastPut() work better?

@mscdex is it a different code path?
We are streaming the file directly from a database so we would very much like to use the streaming api instead of having to buffer the file to disk first.

Regardless of our use case, it does indeed work with sftp.fastPut()

@tlbdk Yes, different code.

I suspect there is something wrong with flow control. sftp stream emits "continue" rather than "drain" on writeData() === false, and WriteStream.prototype._write never checks return value of writeData. Probably both could be fixed with change to WriteStream (on continue emit drain and wait for continue/drain in _write() if writeData returns false) but a lot of other code is fairly low-level in this package so don't know if I trust myself with a PR.


Question: 现在是否有表单提交按钮 loading 的逻辑，如果有的话怎么使用呢

Version
Vben Admin V5
Describe the bug?
现在是有表单提交按钮 loading 的逻辑，如果有的话怎么使用呢
Reproduction
现在是有表单提交按钮 loading 的逻辑，如果有的话怎么使用呢
System Info
chrome

Relevant log output
No response
Validations

[X] Read the docs
[X] Ensure the code is up to date. (Some issues have been fixed in the latest version)
[X] I have searched the existing issues and checked that my issue does not duplicate any existing issues.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

#4985


[Question] Undefined symbols, no matter what I build

I bet this might be easy to solve as pre-built libraries works, just target macOS 12 which is to high in Unreal Engine 5.
[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: my issue with pre-built SDK is that target macOS 12, for Unreal Engine I need 10.15
Firebase C++ SDK version: I tried 9.4.0 and 10.3.0
Main Firebase Components in concern: All
Other Firebase Components in use: No code issues
Platform you are using the C++ SDK on: Mac, Windows, or Linux
Platform you are targeting: All

[REQUIRED] Please describe the question here:
No matter what I build, manually building I get undefined symbols everywhere.
Nothing fancy here... building from MacOS Ventura
cmake version 3.25.1
cmake .. -DOPENSSL_ROOT_DIR=/usr/local/opt/openssl -DCMAKE_OSX_ARCHITECTURES="x86_64;arm64" -DCMAKE_OSX_DEPLOYMENT_TARGET="10.15"
cmake --build . -- -j$(sysctl -n hw.ncpu)

Some screenshots

I wonder if the root of all of this is your external download and compilation of curl:

Hi @PaTiToMaSteR,
Those libcurl errors are expected as we see them in our own internal CI builds. I believe that the curl source files process-out the definitions of functions in certain source files based on the configuration built, and some resulting object files simply don't have anything in them to link against.
By the looks of the link errors, they seem to be things that reside in libfirebase_app.a and liblibuWS.a
Could you run lipo -info <path_to_whichever_lib> to ensure that it includes both of your target architectures? If so, could you run nm on the libraries (for example nm -S <path_to_whichever_lib>) to see if you can spot the symbols in question, and if they have a definition in the library, or if they're marked as U for undefined?
Thanks!

I’m back again on MacOS making the build I’ll post the info, sorry for my late reply!

  I'm not sure what's going on here, then. I singled out one of this reported missing symbols firebase::rest::Request::set_post_fields, and I see it defined in the universal_10.15 library as __ZN8firebase4rest7Request15set_post_fieldsEPKc.
I have two more suggestions.
First is to add the CMAKE_OSX_DEPLOYMENT_TARGET to the top of the cmake file. Apparently it does odd things if it's not defined before the project() clause in cmake. If you deliberately put it at the top of the cmake file then there's no question as to when its getting plumbed into cmake. I know that defining it on the command line should work, as you've done, but I'm trying to eliminate any possible errors at this point.
Additionally set the MACOSX_DEPLOYMENT_TARGET cmake flag to 10.15 s well.


Address and File changes

Add latitude/longitude to Address
Incorporate lat/lon changes into Mapper
Add File support to Property and Mapper

Coverage remained the same at 100.0% when pulling bea94a8024e311f28378b674b813767dec2dfdb3 on files into 094b6ff408edf460b31c46823800678387c7cef0 on master.


Add Python GPU renderer implementation example

From discussion in Slack #dev channel about how to implement the OCIO GPU renderer in Python, this is a trimmed down version of an image viewer I wrote while stress testing the OCIO v2 Python bindings last year. It implements the OCIO GPU renderer with dynamic parameters for exposure and gamma adjustment. The goal here is to show just the pieces that need to be implemented for a functional and performant viewer GUI, which is quite a bit of code, but demonstrates the OCIO API in context, along with all the glue code needed to interface with the other Python libraries used here.
As @hodoulp mentioned in Slack, we should reimplement oglapphelpers in Python, which much of the rendering code in this module was adapted from. This module would be greatly reduced with that, leaving mostly Qt GUI code, but in the meantime this can serve as a working example.
Signed-off-by: Michael Dolan<EMAIL_ADDRESS>

Fantastic, this is extremely useful!
I hope we can also discuss some level of abstraction so that for a end user it can be as simple as calling apply on the GPU processor.

[Question] Why not creating an apps (like ociocheck, ociodisplay, etc.) ?
So, the Python app could be improved when oglapphelpers.py library is ready and also installed like any other apps. My concern is that an 'not used' example becomes deprecated at some point like an example in the documentation for example.

I could rebrand this example as pyociodisplay and move it to the apps folder, refining it over time.

Thanks for sharing @michdolan ! I'm working on having it work on macOS, will report when succeeded. Would be great for it to support all 3 major platforms.

@michdolan This pull request is the next one to be merged. But there is still the question about the location, and a second one (at my point of view) would be to document how to run it (or create a CMakeLists.txt with dependencies).

@hodoulp I will move this and add a README.


add data to evaporation experiment

Purpose
Add data from Lehmann 2008 Fig. 8 to standalone soil evaporation experiment. Tune parameters to ensure good fit.
Note: we should also restore experiments/Manifest.toml to use julia v1.9.3 - it was accidentally reverted to v1.9.1 in another PR
To Do

[x] read in experiment data
[x] tune parameters to ensure fit to data
[x] update parameters to more closely match experiment (zmin, t_end)
[x] clean up experiments/Project.toml, experiments/Manifest.toml

remove unused packages, use julia v1.9.3

[x] I have read and checked the items on the review checklist.

bors r+

bors r+


Verify income imputation in detail

I need to check the imputation in a bit more depth, in several ways:

When county imputation is used
When state imputation is used
When neighbor imputation is used

I also need to check:

impact on higher ed
New specific "low income" %ages

In a more concrete and repeatable way. This is somewhat high priority

Check what is happening for Islands -- appears they remain empty

Check out census_acs/etl.py around line 450 and etl_imputations.py for the code.
To do: Make sure imputed and adjusted income comes up. Take an example of 10 tracts that get filled with imputed and adjusted but dont have a not imputed value, and make sure that the imputed and adjusted value is the same as the county or neighbor average. Double-check we're not using state as the third option.


Hello, "affordance_512.ckpt" is a directory, not a ".ckpt" format file, is something wrong?

Sorry for this confusion. I used safetensor from huggingface to split the model due to the storage limitation, you can use safetensor to combine the splits. Besides, I will try to replace the splits with a whole model on huggingface. I will let you know after I replace it.


Admin UI doesn't render

Thank you for making create-rust-app available!
Using create-rust-app 8.1.1 with postgres + poem + Auth + Container + Development plugins (Storage and GraphQL disabled), I was able to get the regular Todo app working.
However, I get the following error in Firefox 109.0 (as seen in the Web Developer Tools console) on Arch Linux when accessing the Admin UI at http://localhost:3000/admin :
The script from “http://localhost:3000/admin/admin.67d0b27a.js” was loaded even though its MIME type (“text/html”) is not a valid JavaScript MIME type.
admin
The stylesheet http://localhost:3000/admin/admin.144d5956.css was not loaded because its MIME type, “text/html”, is not “text/css”. admin
Uncaught SyntaxError: expected expression, got '<'
admin.67d0b27a.js:1

The admin UI doesn't render (I get a blank page).

The admin UI is still a WIP, though from my understanding @Wulf has made a lot of progress recently, look out for the next release of create-rust-app to get your hands on these juicy updates.

Is this still an issue?


Fix issue #1319 products number text duplication.

Description
Fix products number text duplication inside [ProductBrowsingListActivity].
Related issues and discussion
#1319
Screen-shots, if any

Merged, thanks @MohamedHefny


Add pull-request number as an optional action input

This change allows to use the action with non pull_request / pull_request_target event trigger (ie. workflow_run)
fixes #141

Note: I don't see a conflict in the dist/index.js file, this one should probably be created by your CI anyway.

Hi @jprinet,
I apologize for the late response.
Thank you very much for your valuable contribution! I really appreciate it.
I am getting the below error when I do npm run build from this branch.
Could you please check?
ncc: Version 0.21.1
ncc: Compiling file index.js
Error: error:0308010C:digital envelope routines::unsupported
    at new Hash (node:internal/crypto/hash:68:19)
    at Object.createHash (node:crypto:138:10)
    at hashOf (evalmachine.<anonymous>:1:1411831)
    at module.exports (evalmachine.<anonymous>:1:1413919)
    at runCmd (evalmachine.<anonymous>:1:46102)
    at 819 (evalmachine.<anonymous>:1:43078)
    at __webpack_require__ (evalmachine.<anonymous>:1:216)
    at startup (evalmachine.<anonymous>:1:353)
    at module.exports.8 (evalmachine.<anonymous>:1:385)
    at evalmachine.<anonymous>:1:395 {
  opensslErrorStack: [ 'error:03000086:digital envelope routines::initialization error' ],
  library: 'digital envelope routines',
  reason: 'unsupported',
  code: 'ERR_OSSL_EVP_UNSUPPORTED'
}

Hi @ibakshay
No problem, I am getting a successful local run with a more recent ncc version
Would it be ok to upgrade?
➜  github-action git:(master) ✗ node --version
v16.20.1
➜  github-action git:(master) ✗ npm -version
8.19.4
➜  github-action git:(master) ✗ ncc --version
0.38.0
➜  github-action git:(master) ✗ npm run build

><EMAIL_ADDRESS>build
> tsc && ncc build

ncc: Version 0.38.0
ncc: Compiling file index.js into CJS
1151kB  dist/index.js
1151kB  [588ms] - ncc 0.38.0


add isOpen prop

@nd0ut Could you explain please what issue are you trying to address with this PR?

@moroshko Yeah, I'm sorry. I'm building a custom suggestion input control for my app based on react-autowhatever and I just wanted to control if the suggestion list opened or not. I don't want the list to be opened all the time if there is any items passed in it.

@nd0ut I see your point, but I'm not sure that isOpen deserves it's own prop in this project.
The idea of react-autowhatever is to provide a view layer without any logic. If you want suggestions to be rendered, pass them. If you don't, don't pass them. It's that simple. So, instead of doing:
<Autowhatever isOpen={false} suggestions={['item1', 'item2']} ... />

just do:
<Autowhatever suggestions={[]} ... />

Oh, I remembered why I actually did that patch with isOpen instead of passing empty items. I wanted to make leave animation of the item list but it's impossible cause the list is removing from DOM instantly.
You're right about isOpen. I think better solution is to allow the user to manually wrap an ul element with ReactCSSTransitionGroup for example. What do you think?

@nd0ut Sorry it took me so long to get back to you.
I think adding a hook to wrap the ul makes sense.

@nd0ut Would you be interested to contribute, or did you find a workaround and we can close this issue for now?

@nd0ut I added a new renderItemsContainer prop to provide a way to limit the scrolling to the items container only. I suspect it might help you with the animation implementation.


does geocoder support driving distance or just point-to-point distance

Not really an issue but just a question whether this gem supports driving distance or just point-to-point?
thx

Only point-to-point.


Add $disabledItems to check for disabled items instead of checking ag…

Adds a new variable $disabledItems to get disabled items from getDisabledItems()
Bug was caused by checking $itemValue against $defaultItems to check for disabled items.

This is my first pull request for silverstripe. Not sure if I'm doing it correctly. Hope this PR gets through.


add farthest point sampling for points sampler, and modify the uniform sampler for the meshes

Hi, Castro
I have added farthest point sampling into the points sampler, which is an algorithm applied to the cutting-edge research, see here for the relevant paper, and here for the description of the algorithm.
And I think it is probably better to sample u, v uniformly from [0, 1] and [0, u] respectively in the mesh sampler. Because to ensure the sampled points are within that face, it requires not only u + v  <= 1 but also u >= 0 and v >= 0, but for the original procedure, which is sampled from Gaussian then truncation, it did not resolve the issues with negative sampling results. Plus it is more concise now.
I am very interested to make more commitments since it intersects with my research projects during PhD, I think next step is to develop the Poisson Disk Sampling as discussed here: #239, and is it okay for me to develop a new part for the normals/curvatures calculations(directly from the points) and with reconstruction on the doc as well.
Thanks.

Hola @hansen7,
Thank you so much for the contribution! I'm leaving a few comments in the P.R. review.
Pd. I'm glad to hear that you are interested in making more commitments, don't hesitate to ask anything.

In addition to the above comments, please follow the steps described in:
https://pyntcloud.readthedocs.io/en/latest/samplers_dev.html#let-pyntcloud-know-about-your-sampler
In order to make your new sample available and easily accesible for everybody

haha no worries, is there other places for development, I am looking forward to more contributions


Optimize TreeView performance with many items

Using a tree with many items (1k+) the TreeView should only render visible parts on screen. Therefor Blazor Virtualize component can be used instead of the foreach loop to render items under a tree node.

References issue #39


Reverts deprecated ClusterClaims for odf-info

Changes

Re-adds a single ClusterClaim that stores the NamespacedName of the odf-info CM so that MCO will be able to fetch it.
Skips uninstall/delete so this cluster claim will live as long as the operator does.
RHSTOR-5117
#2440

TODO

[x] Simplify by moving ClusterClaim to be reconciled in OCSInitialization
[ ] Delete the cluster claim once the odf-info CM is removed

@raaizik Why does the claim need to be removed manually ? Can we not set it to be removed alongside StorageCluster?

The cluster claims used to be reconciled via storage cluster because they contained information about it. This is no longer the case, plus the cluster claim needs to be coupled with the operator's lifecycle where it is created once and removed manually (since there's no uninstall for OCSInit).

@raaizik Why does the claim need to be removed manually ? Can we not set it to be removed alongside StorageCluster?

@vbnrh The cluster claims used to be reconciled via storage cluster because they contained information about it. This is no longer the case, plus the cluster claim needs to be coupled with the operator's lifecycle where it is created once and removed manually (since there's no uninstall for OCSInit).

If the StorageCluster is uninstalled at a later point of time, the CM will still be present on the hub.  How are we going to indicate that cluster is not available for RDR on hub ? Are we planning or have we added the StorageCluster's status on the CM ?

@raaizik Why does the claim need to be removed manually ? Can we not set it to be removed alongside StorageCluster?

@vbnrh The cluster claims used to be reconciled via storage cluster because they contained information about it. This is no longer the case, plus the cluster claim needs to be coupled with the operator's lifecycle where it is created once and removed manually (since there's no uninstall for OCSInit).

If the StorageCluster is uninstalled at a later point of time, the CM will still be present on the hub. How are we going to indicate that cluster is not available for RDR on hub ? Are we planning or have we added the StorageCluster's status on the CM ?

@vbnrh Whenever a StorageCluster is uninstalled, its respective key in the CM gets removed. If that key was the last one to get uninstalled, then the whole CM gets removed.

/retest

/retest

/test ocs-operator-bundle-e2e-aws

@umangachapagain @raaizik
If the ODF inf config map is created when the first storagecluster is created and gets deleted when the last storage cluster is deleted (all as part of the storagecluster controller), why do we tie the claim's lifecycle to ocsinitialization, shouldn't we treat the claim and the ConfigMap as a pair? created together and deleted together?

@nb-ohad @umangachapagain QE tests for MDR and RDR is getting blocked as this PR is not in 4.17
Please take a look ASAP

@nb-ohad @umangachapagain QE tests for MDR and RDR is getting blocked as this PR is not in 4.17
Please take a look ASAP

It depends on #2712

/test ocs-operator-bundle-e2e-aws

LGTM
@iamniting @umangachapagain Do you want to take a look before merging?

/cherry-pick release-4.17


square button shape

Hi,
That is a great library, and I like square button shape more, not sure whether I can change it, if yes, how?
thanks
Eric Xin

It seems it's just a CSS change. You can add your own CSS files where you can customize buttons making them as square as you'd like.


Zpages docs

Description

Docs for Kubernetes z-pages
Issue

https://github.com/kubernetes/enhancements/issues/4827
https://github.com/kubernetes/enhancements/issues/4828

/remove-language bn de es fr hi id it ja ko pt pl ru uk zh
/remove-area release-eng
/remove-area localization

/remove language vi
/remove-sig release
/sig instrumentation

/remove-language vi

Hello @richabanker  :wave: please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review before Tuesday November 19th 2024 18:00 PST. Thank you!

Hi @richabanker , just a friendly reminder to get you PR ready for review before Tuesday November 19th, thanks!

Hi @richabanker , just a friendly reminder to get you PR ready for review before Tuesday November 19th, thanks!

/reopen

Accidentally ended up closing this PR. I have now created https://github.com/kubernetes/website/pull/48760 in its place, is this still ok?


Investigation/experiment: protobuf exposed via web API for backup and/or webconfig usage

Protocol Buffers has definitely proven itself, and a core observation is it will be more resilient to change than JSON, since we can rename fields in protobuf freely thanks to the field numbers, whereas those renames break JSON between versions. Because the JSON is different, the API methods may or may not break, or they may need additional upkeep, or the webconfig needs upkeep, and so on, to match the new field names.
This has led to us inadvertently breaking the config export (see #656), and the upkeep problem in general means that config backups are incomplete (see #309). There is also an element of discontent about the JSON libraries we use in the firmware to begin with, they are futzy and have buffer limitations, so maybe we can divorce ourselves of that by switching everything to protobuf.
This issue is meant to fiddle with ideas, which I'm going to start by exposing protobuf API endpoints and seeing what we can do with them. This might also have a positive effect on the usability of gp2040ce-binary-tools.
For now, my vague thoughts:

API endpoint /api/proto/config (GET, PUT)
reference endpoints /api/proto/foo.proto (GET) where foo is config.proto, enums.proto, etc., probably via fsdata.c
Confirm I can retrieve and edit configs using the above via gp2040ce-binary-tools?
See what people think about config backups via the above?
See if anyone better at React than me can use that to make components better/easier?
Party!?

protobuf supports json format, json can still be can be a bridge between web api and protobuf
https://googleapis.dev/python/protobuf/latest/google/protobuf/json_format.html

protobuf supports json format, json can works as adatptor format between web api and c code
https://googleapis.dev/python/protobuf/latest/google/protobuf/json_format.html

You found a language library. nanopb (the embedded C implementation we use) does not implement JSON, however; we already use ArduinoJson and it has its own complications/problems.


Improvement: new resizing method

Implemented a resize_images utility function. The new implementation uses OpenCV for image smoothing and resizing which results in big performance gains (around 4x faster on average). Updated AddCloudMaskTask to use the new function.
New function correctly computes the sigma used for anti aliasing and correctly aligns the images on resize. The function can be reused in other modules, where resizing is done.
The values in tests for the AddCloudMaskTask were also updated to allow for the new sigma computation and alignment.
Also added map_image_slices utilitiy function. It maps a given function over all (height, width) image slices in an array of shape (time, height, width, channels). Might be of use elswhere. If not we can refactor it inside resize_images.

I've realised the test_cloud_mask uses a local eopatch mask/eolearn/test/eopatch_sample.
This was probably used before we introduced a unified TestEOPatch in example_data. Can you update the unti-tests to use the unified TestEOPatch and delete the mask/eolearn/test/eopatch_sample?

I've realised the test_cloud_mask uses a local eopatch mask/eolearn/test/eopatch_sample.
This was probably used before we introduced a unified TestEOPatch in example_data. Can you update the unti-tests to use the unified TestEOPatch and delete the mask/eolearn/test/eopatch_sample?

@devisperessutti Sure. I will look into it.

I wrote the code to only work on 4d arrays. Maybe it would be better if it was generalized to work also on 3d features (timeless) or maybe even 2d if needed.

I wrote the code to only work on 4d arrays. Maybe it would be better if it was generalized to work also on 3d features (timeless) or maybe even 2d if needed.

can you please adapt the code to work also on 4d, 3d, 2d data.

I've realised the test_cloud_mask uses a local eopatch mask/eolearn/test/eopatch_sample.
This was probably used before we introduced a unified TestEOPatch in example_data. Can you update the unti-tests to use the unified TestEOPatch and delete the mask/eolearn/test/eopatch_sample?

I've added the tests for the TestEOPatch example.

I wrote the code to only work on 4d arrays. Maybe it would be better if it was generalized to work also on 3d features (timeless) or maybe even 2d if needed.

can you please adapt the code to work also on 4d, 3d, 2d data.

I've adapted the code to work on 4d, 3d and 2d data. Should I add some tests for the resizing. If so, where should I put them?


Import of .zip file fails - 80048060

Describe the bug
A clear and concise description of what the bug is.
Error when uploading CoEStarterKit.Zip into Solutions

Component (please tell us which flow or app you are experiencing issues with):
.zip file
To Reproduce
Steps to reproduce the behavior:

Select Solutions on the left navigation bar.

Select Import. A pop-up window appears. (If the window doesn't appear, be sure your browser's pop-up blocker is disabled and try again.)

In the pop-up window, select Choose File.

Choose the Center Of Excellence Core Components solution from the file explorer (CenterOfExcellenceCoreComponents_x_x_x_xx_managed.zip).

Expected behavior
The .zip file imports successfully
Screenshots
See above image.
Additional context
Add any other context about the problem here.

Hi thanks for posting. We know that the zip file is correctly formatted as others have installed with it. Can you please validate that you have covered all the pre-req's needeD?
https://docs.microsoft.com/en-us/power-platform/guidance/coe/setup#prerequisites

Extracting Components from .zip and importing each one seperately works with warnings

oh i gotcha. Yes you need to extract the starter kit into individual components. I'll make sure this is clear in the documentation. Thanks!


Conduct InternBit Presurvey

Mirabela created a survey for gathering student data about their internship search experience:
https://docs.google.com/forms/d/e/1FAIpQLSdR4z_CDx0nhLrzvTZ8yRSuPi-IO3cuZsVdRk5rxfSM8BTBbg/viewform
Before distribution, it needs some changes:

It needs an introductory paragraph explaining the purpose of the questionnaire and what the responses will be used for.

It needs one (optional) long form text question at the end:  "Is there anything else you would like to tell us about your internship experience? Please indicate below."

It needs the following optional short text question at the very end: "If you would like us to contact you with more information about this

Once ready, we can distribute this to the ICS mailing lists (through Philip) and to the ACM Manoa discord channel (anyone).

I will write up the results this weekend.


feat(welcompage): add option to disable background image

It would be nice to make the background image on the welcome page optional. I added an option for this, so it can be done in the config file. The default is enabled, so it won't break existing websites that are upgrading. I also updated the documentation and example.
I hope you find this a useful addition.

Hi @ttcremers
Thank you for the PR. It looks OK. We can merge it after you fix the typo


error: attribute 'defaultApp.x86_64-darwin' should have type 'derivation'

when running the command to add a package, here is what I get
❯ nix run . -- add github.com:dgraph-io/ratel
error: attribute 'defaultApp.x86_64-darwin' should have type 'derivation'

This one can probably be closed


Bug 1788142: bump(*)

bump openshift/api to ffde1bf
bump openshift/library-go to e135d80
pin k8s.io to v0.17.1
react to changes

use context when running operator
use client holder with resourceapply

/retest
AWS throttling on aws-upgrade, e2e-aws seems to have failed with kube scheduler losing its leader lock.

/retest

/retest

bump @mfojtik @deads2k

/lgtm


Allow linking test titles to specs.

Rather than just having this at the comment level, we can move these into the test data itself.
The links to add would need to be added to the test data in a similar way as test.cell stuff is added here:
https://github.com/w3c/vc-data-model-2.0-test-suite/blob/main/tests/10-vcdm2.js#L40-L43
Not sure that's ideal...but it does work.
I can provide readme updates, if y'all think this approach is worth doing.

p.s. also test it with a suite with no links to ensure it works with reports with and with out links, but I think the modified test data already covers that.

Yeah, I've been testing it with VCDM tests, but I think this still needs work. I'm not happy with overloading the test title parameter...it should really be more of an augmentation.

Did more local testing and this is good to go--and I plan to use it soon on the VCDMv2 test suite, so merging.


Merge latest changes to branch 'main' into development

This pull request is for merging the latest changes to the main branch into the development branch. These changes include:

Adding support for multiple solutes in the function create_solution()
Improved error handling for convert_one() and create_solution() functions
Additional testing for create_solution()
NumPy version clamping to 1.23.5
Updated release number (0.4.5 -> 0.4.7)

Stricter branch protections will be implemented in the near future which will include a requirement that branches be up-to-date when merging with main or development. Thus, it is important that development be up-to-date with main to avoid issues with future merges.


Data Labelling

Hello,
How do you label the article? For example, the data collected from the URL  http://beforeitsnews.com/awakening-start-here/2018/01/awakening-of-12-strands-of-dna-reconnecting-with-you-movie-10623.html there is no information that the article is fake.
Please explain if I miss something.

Hello @Gautamshahi.
The way of labeling articles is described in README file, see part How was the corpus created?.
There, you can find the statement:

Each article has been attributed the same label as the label associated with its domain.

So, article is labeled as fake because beforeitsnews is considered as unreliable news source.

Hello, Yes, but it adds a lot of bias if you consider only domain type for annotation. Did you publish any paper using this approach?

Yes, you are right. Actually, in research it is not recommended to use datasets labeled by source credibility, however, this annotation technique is used in majority of papers. The reason is simple - it is hard to label articles the other way. There are datasets with manual labeling, however containing only a few samples. The better way, in my opinion, is to label articles by mapping verified claims (e.g. from fact-checking sites) and determine the stance of article to claim.
If you are interested, you can check other datasets in my datasets overview for fake news detection - Fake News Datasets.


Getting exception when applying filter according to Order Date of Order grid of Admin.

Framework Version: 0.1.5

Framework Version: 0.1.5
Fixed


JavaScript destructuring

Hi everyone!
I thought recognizing destructuring in function params could be a good enhancement!
This is the actual result when it is used:
/**
 * [handleRecaptchaErrors description]
 * @param  {[type]} res        [description]
 * @param  {[type]} {          status        [description]
 * @param  {[type]} statusText [description]
 * @param  {[type]} config     }             [description]
 * @return {[type]}            [description]
 */
export function handleRecaptchaErrors(res, { status, statusText, config }) {
  // Code here
}

Getting something similar to this could be convenient:
/**
 * Handles the errors that can occur when requesting the reCatpcha API
 * @param  {Object} res                      [HTTP response of the /contactMe resource]
 * @param  {Object} recaptchaRequestResponse [Axios response received requesting the reCaptcha API]
 *         @property  {Number}  status       [Status code]
 *         @property  {String}  statusText   [Status message]
 *         @property  {Object}  config       [Original axios request config]
 */
 export function handleRecaptchaErrors(res, { status, statusText, config }) {
  // Code here
}

That being said, I'm really not comfortable with RegExp (...I should start learning) so not sure I can implement this myself

Hey there!
Same goes when you use ES6 Spread Operator as function parameter:
/**
 * [example description]
 * @param  {[type]} prop     [description]
 * @param  {[type]} ...props [description]
 * @return {[type]}          [description]
 */
function example (prop, ...props) {
  // Code here
}

I do not have a good idea how to handle Spread Operator parameters. Maybe just defining them as Array :thought_balloon:

@dirchev could you please open a new issue for adding support for the spread operator.

This should be addressed with some prio cause it's causing exceptions (See #219 and #197).

I do not have a good idea how to handle Spread Operator parameters. Maybe just defining their type to be Array

The coder may want to make it Array<{type}> so maybe that should be the placeholder.

+1 on this issue

With docblockr 0.13.7 and atom 1.31.2, the package correctly handles
function foo(x, {a, b}) { /* etc */ }
// results in
/**
 * [foo description]
 * @param  {[type]} x [description]
 * @param  {[type]} a [description]
 * @param  {[type]} b [description]
 * @return {[type]}   [description]
 */

but
function bar(x, {a, b} = {a: 1, b: "2"}) {/* etc */}
// results in
/**
 * [foo description]
 * @param  {[type]} x [description]
 * @param  {[type]} a [description]
 * @param  {[type]} b [description]
 * @param  {[type]} b [description]
 * @return {[type]}   [description]
 */

No errors or docblockr-related warnings are written to the console.


latest melodic release 0.3.6 breaks rviz on buildfarm

@dirk-thomas, yesterday's release of python_qt_binding (https://github.com/ros/rosdistro/pull/22454) breaks all rviz binary builds on the build farm, e.g. http://build.ros.org/job/Mbin_ds_dS64__rviz__debian_stretch_amd64__binary/6/display/redirect
Any idea, why cmake cannot find python_qt_binding anymore?

As of #75 the package uses the conditional attributes for the dependencies. It seems that bloom isn't including the dependencies in the control file. @wjwwood @nuclearsandwich ?

I can't assign myself to this issue but I am looking into it.

The issue is that bloom only allows conditions that it can determine from the distribution index and the python version is not currently one of them. (See https://github.com/ros-infrastructure/bloom/pull/519 for the PR which introduced the behavior and why the local environment isn't used.) An addition to the index to specify the python version is being proposed to provide a source for that information and we'll likely also need to add something to bloom which allows user-specified conditions to be used.
To resolve the immediate issue, I've used a patched version of bloom to create a package revision with the ROS_PYTHON_VERSION defined.
https://github.com/ros/rosdistro/pull/22483


feat: add values to defaultConfig

This PR adds syncMode and ipc to defaultConfig.
Also renames node to client (✔ from philipp)
My editor also did some prettier linting.

Added to #47


Missing sas-deployment-operator metrics

As per https://github.com/sassoftware/viya4-monitoring-kubernetes/issues/151, creating another issue to track missing deployment operator metrics.
Thanks.

Thank you @pilyk ... We will update this issue as we identify the root cause and potential solutions.

The SAS Deployment Operator developers removed the metrics endpoint from which surfaced metrics to Prometheus.  There are on-going discussions about if/how the SAS Deployment Operator should surface metrics.  I will update this issue when the plan has been finalized.

After further discussion, we determined metrics are actually being surfaced by the SAS Deployment Operator just not on the port our project was expecting.  We will update our service monitor definition to fix this is an upcoming release.

@pilyk Pull Request #172 was pushed earlier today and fixed this issue.  This fix will be available in our stable branch as part of our next release, due out in mid-October.  If you are especially eager for this fix, it is available now in our master branch.  However, please be aware that our master branch is our live development branch and, therefore, is updated very frequently and contains other fixes, changes and other work in progress.  The stable branch is updated less frequently, generally once a month, and contains a more cohesive, more thoroughly tested, set of changes.


IMAPClientHandler can't process very long line responses from the server

Expected behavior
Very long responses from an IMAP server in IMAPClientHandler should decode
Actual behavior
Very long responses that exceed IMAPDefaults.lineLengthLimit bytes throws an exception ByteToMessageDecoderError.PayloadTooLargeError.
The cause is IMAPClientHandler.init setting a maximum buffer size of IMAPDefaults.lineLengthLimit for decoding responses from the server and there is no way to override it without modifying the swift-nio-imap source.
Setting the limit to 8192 bytes seems to be arbitrary. IMAPDefaults.lineLengthLimit references RFC7162 section 4 for the limit, however according to the RFC, the limit is for commands sent from the client to the server, not from the server to the client:

The updated recommendation is as follows: a client should limit the
length of the command lines it generates to approximately 8192 octets
(including all quoted strings but not including literals).  If the
client is unable to group things into ranges so that the command line
is within that length, it should split the request into multiple
commands.  The client should use literals instead of long quoted
strings in order to keep the command length down.

Section <IP_ADDRESS> of RFC2683, referenced by RFC7162 section 4 also does not place limits on line length sent by servers to clients either.
While some limit may be useful, it would be helpful if there was a way to override it without modifying the source.
Steps to reproduce

Create a IMAPClientHandler()
Pass a response > 8192 bytes, like for a UID SEARCH RETURN () ALL on a very large mailbox in 10000 byte chunks to its channel

If possible, minimal yet complete reproducer code (or URL to code)
import Testing
import Foundation
import NIOIMAPCore
import NIO
import NIOIMAP

@Test func testIMAPHandler() throws {
    let clientHandler: IMAPClientHandler = IMAPClientHandler()
    let channel = EmbeddedChannel(handler: clientHandler)
    channel.pipeline.fireChannelActive()

    // this should be bigger than 8K
    var uids = MessageIdentifierSet<UnknownMessageIdentifier>()
    for i in 1...10000 {
        uids.insert(UnknownMessageIdentifier(rawValue: UInt32(i * 2)))
    }

    let command: CommandStreamPart = .tagged(.init(tag: "a3", command: .uidSearch(key: .all, charset: nil, returnOptions: [.all])))
    try channel.writeAndFlush(command).wait()

    let esearchResponse = Response.untagged(
        .mailboxData(
            .extendedSearch(.init(correlator: .init(tag: "a3"), kind: .uid, returnData: [
                .all(.set(.init(set: uids)!))
            ]))))

    // partial writes needed to trigger
    var buffer = ByteBuffer(string: esearchResponse.debugDescription)
    while buffer.readableBytes != 0 {
        let slice = min(buffer.readableBytes, 10000)
        try channel.writeInbound(buffer.readSlice(length: slice))
    }

    var maybeRead: Response? = try channel.readInbound()
    let _ = try #require(maybeRead)
}

SwiftNIO version/commit hash
2b6d7281916fe6da5229fed6a5aff3f986b37869
Swift & OS version (output of swift --version && uname -a)
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0
Darwin redacted.local 24.0.0 Darwin Kernel Version 24.0.0: Mon Aug 12 21:27:44 PDT 2024; root:xnu-11215.1.10~5/RELEASE_ARM64_T8103 arm64

Having a defensive limit is a good idea, as it's not sensible to let a remote peer force you to commit an unbounded amount of resources. However, we shouldn't prevent this being configured, as there's no one-right-value. We'd love a change that makes this configurable.


Setuptools 34.0.2 breaks new virtual envs with six < 1.10

The recent change to strip out vendored packages (like six) breaks projects that pip install a version of six earlier than 1.10
This was also reported as an issue on virtualenv with code examples showing the problem. The commit on setuptools appears to be the root cause.

This change to setuptools was made intentionally in order to avoid the ugly hacks and complicated bootstrapping scenarios imposed by such hacks.
The issue here is that the offending packages that are requiring old versions of six. They simply are no longer compatible with setuptools.
Those failing packages accepted the burden of incompatibility with any package that declared a later six dependency when they pinned their dependency to a specific version. Since those packages are demanding an old version of six, then it should be those packages that either vendor six themselves or adapt to support newer versions of six.
I did look into the possibility of expanding the supported six versions in setuptools. If I change the version to 1.6.0, some integration tests start to fail (because stevedore depends on six 1.9), but otherwise the tests pass. Perhaps setting a lower lower bound will be viable and save some headaches.

In applications, it is a best practice to pin every dependency with == in requirements.txt for reproducibility.
This usually means pip freeze --all > requirements.txt.
If one were to generate a requirements.txt with an older version of six / pyparsing / etc. they're now suddenly broken as virtualenv downloads the newest version of setuptools (demonstrated in pypa/virtualenv#1009).

@asottile: How do you suggest these inherent dependency conflicts be reconciled?

That's a good question!
In an ideal world, pip wouldn't upgrade setuptools to a version that conflicts with already installed dependencies (and would do dependency resolution) but I don't think that's really something that's doable without making pip way more intelligent and way slower.
It's possible this is just an "unfortunate" combination of two separate "unfortunate" decisions, notably:

virtualenv upgrades pip / setuptools / wheel by default (new behaviour in 15)
setuptools unvendored dependencies (new in 34)

At least personally, the first behaviour is alarming and makes it very difficult to have repeatable builds (without scattering --no-download, VIRTUALENV_NO_DOWNLOAD=1 or PIP_INDEX_URL=... virtualenv all over our ~thousands of codebases).  The second seems not a problem to me, I rather like that setuptools now acts more like a normal python package (though the pypa tools don't treat it as such: magically upgrading, excluding from freeze, etc.).
In short, big shrugs -- it might just be "unfortunate: won't fix" and that would be ok too!


No valid images after changing partition table offset (IDFGH-5408)

Environment

Development Kit:    ESP32-Wrover-Kit
Kit version : ?
Module or chip used:  ESP32-WROVERE
IDF version : .40200.210118 (4.2.0)
Build System:  idf.py / pio
Operating System:     Linux
Using an IDE?: Yes, VSCode with pio plugin (and package platformio installed for use in cli)
Power Supply:        external 5V

Problem Description
I first thought my problem is with the PIO IDE so i posted some information there:
https://community.platformio.org/t/how-to-compile-against-esp32-wrover-e/21920/12
In short:
When trying to get PSRAM running i kept getting an error that the partition table was overlapping at 0x8000, after a lot of trying around I found out that the size of the bootloader was too big and didn't fit between 0x1000 and 0x8000.
I am not sure why it grew in size but okay.
After adding CONFIG_PARTITION_TABLE_OFFSET=0x9000 to my sdk defaults and removing all the offsets from the partition table it uploaded successfully.
Sadly then it kept starting from the ota partition which had some old firmware on it and crashing in
packages/framework-espidf/components/app_update/esp_ota_ops.c", line 542
I just thought something has to be wrong there since i changed the partition table and the ota shouldn't even have any code on it right now.
After running pio run --target erase and erasing the flash and uploading it again later on, i am now stuck on an infinite loop telling me neither of my app partitions are bootable, and I have no clue why.
Expected Behavior
That the app i just uploaded to the chip is bootable
Actual Behavior
boot loop:
I (14) boot: ESP-IDF 3.40200.210118 2nd stage bootloader
I (14) boot: compile time 15:42:14
I (14) boot: chip revision: 3
I (17) boot_comm: chip revision: 3, min. bootloader chip revision: 0
I (24) boot.esp32: SPI Speed      : 40MHz
I (28) boot.esp32: SPI Mode       : DIO
I (33) boot.esp32: SPI Flash Size : 4MB
I (37) boot: Enabling RNG early entropy source...
I (43) boot: Partition Table:
I (46) boot: ## Label            Usage          Type ST Offset   Length
I (54) boot:  0 nvs              WiFi data        01 02 00009000 00004000
I (61) boot:  1 otadata          OTA data         01 00 0000d000 00002000
I (69) boot:  2 phy_init         RF data          01 01 0000f000 00001000
I (76) boot:  3 factory          factory app      00 00 00010000 001db000
I (83) boot:  4 ota_0            OTA app          00 10 001f0000 001db000
I (91) boot:  5 storage          Unknown data     01 82 003cb000 0001e000
I (98) boot: End of partition table
I (103) boot: Defaulting to factory image
E (107) esp_image: image at 0x10000 has invalid magic byte
W (113) esp_image: image at 0x10000 has invalid SPI mode 255
W (120) esp_image: image at 0x10000 has invalid SPI size 15
E (126) boot: Factory app partition is not bootable
E (132) esp_image: image at 0x1f0000 has invalid magic byte
W (138) esp_image: image at 0x1f0000 has invalid SPI mode 255
W (144) esp_image: image at 0x1f0000 has invalid SPI size 15
E (151) boot: OTA app partition slot 0 is not bootable
E (156) boot: No bootable app partitions in the partition table

Steps to reproduce
described above.
sdkconfig.txt
sdkconfig.defaults.txt
Other items if possible

[x] sdkconfig file (attach the sdkconfig file from your project folder)
[ ] elf file in the build folder (note this may contain all the code details and symbols of your project.)
[ ] coredump (This provides stacks of tasks.)

It is because platformio is using hardcoded 0x10000 flash offset for app binary. You can change offset in partition table and bootloader will try to find app at new offsets, but binary has been flashed at 0x10000.
Bootloader size usually is increased when its debug level is increased.

Okay so it happens because of the hard coding, so  i can just rearrange my partition table and this should be fixed:

# Name,   Type, SubType,  Offset,   Size,  Flags
nvs,      data, nvs,      ,  0x4000
otadata,  data, ota,      ,  0x2000
factory,  app,  factory,  0x10000,  1900k
ota_0,    app,  ota_0,    ,  1900k
storage,  data, spiffs,   ,  120k
phy_init, data, phy,      ,  0x1000

Thank you! now my issue is gone but it still crashes, now on some invalid memory region inside nvs init, but I don't think that's related to this.
I am guessing i did something illegal in my new partition table


Fix simd comparison kernels

Which issue does this PR close?

Closes #1285 .
Rationale for this change
The refactoring of the simd comparison in #1146 introduced silly mistake that was not noticed due to missing test coverage.
What changes are included in this PR?
Apart from the bugfix, this adjusts the tests so they automatically also test a larger version of the test input and so cover the chunked part.

@alamb I'm sorry for only noticing this issue so late. Would be nice if we could still include it in the 9.0 release.


undefined: ghrel.FetchLatestRelease

Summary
Error undefined: ghrel.FetchLatestRelease appears during go get installation.
Steps To Reproduce
Attempt to install gopass using go get with go version 1.13.1
in project outside $GOPATH which have go.mod
Expected behavior
Installations successes
Environment
go version 1.13.1
GO111MODULE=on
Installation happens for project that uses go.mod

OS: Manjaro
OS version: 4.19.78-1-MANJARO
gopass Version: 1.8.6
Installation method: go get (go get -u checked as well)

Additional context
After brief check I see that master branch from github.com/dominikschulz/github-releases/ghrel contains FetchLatestRelease but tag v0.0.1 doesn't. Release github-releases v0.0.2 should fix it. @dominikschulz

Console output:
> go get github.com/gopasspw/gopass
go: finding github.com/xrash/smetrics latest
go: finding golang.org/x/crypto latest
go: finding github.com/martinhoefling/goxkcdpwgen latest
go: finding github.com/muesli/crunchy latest
go: finding github.com/alecthomas/binary latest
go: finding golang.org/x/sys latest
go: finding github.com/kevinburke/ssh_config latest
go: finding github.com/jbenet/go-context latest
go: finding golang.org/x/net latest
go: finding github.com/jsimonetti/pwscheme latest
go: finding github.com/kballard/go-shellquote latest
go: finding github.com/armon/go-metrics latest
go: finding github.com/nsf/termbox-go latest
go: finding golang.org/x/time latest
go: finding github.com/mitchellh/go-ps latest
go: finding github.com/muesli/goprogressbar latest
go: finding github.com/skip2/go-qrcode latest
# github.com/gopasspw/gopass/pkg/updater
go/pkg/mod/github.com/gopasspw/gopass@v1.8.6/pkg/updater/update.go:94:10: undefined: ghrel.FetchLatestRelease

Is there is any progress on this issue done so far?

Should be fixed by #1273.