notebook

Dear InternVideo team,
Is there a notebook to show how to load the model and extract video feature? Thx a lot.
Best Wishes,
Zongze

Hello! You can find it in initial files on HF https://huggingface.co/OpenGVLab/ViCLIP/blob/main/demo.ipynb

Dear InternVideo team,
Is there a notebook to show how to load the model and extract video feature? Thx a lot.
Best Wishes, Zongze

Hello! You can find it in initial files on HF https://huggingface.co/OpenGVLab/ViCLIP/blob/main/demo.ipynb

Thx for your reply. But the link is broken, could you update the link? Thx

Thx for your reply. But the link is broken, could you update the link? Thx

GitHub - https://github.com/OpenGVLab/InternVideo/blob/main/Data/InternVid/demo.ipynb
Or HF - https://huggingface.co/OpenGVLab/ViCLIP/blob/main/demo.ipynb

Thx a lot, it works now. Could you tell me how many frames it can process? I do not find this number in the paper. If I provide 60 frames to it, can I get a continuous 60 frame video embedding from it? Thx a lot

Thx a lot, it works now. Could you tell me how many frames it can process? I do not find this number in the paper. If I provide 60 frames to it, can I get a continuous 60 frame video embedding from it? Thx a lot

As I know, it's 8 frames. In demo notebook of ViClip you can find this function, where video processor select first 8 frames of input video:
def frames2tensor(vid_list, fnum=8, target_size=(224, 224), device=torch.device('cuda')):
    assert(len(vid_list) >= fnum)
    step = len(vid_list) // fnum
    vid_list = vid_list[::step][:fnum]
    vid_list = [cv2.resize(x[:,:,::-1], target_size) for x in vid_list]
    vid_tube = [np.expand_dims(normalize(x), axis=(0, 1)) for x in vid_list]
    vid_tube = np.concatenate(vid_tube, axis=1)
    vid_tube = np.transpose(vid_tube, (0, 1, 4, 2, 3))
    vid_tube = torch.from_numpy(vid_tube).to(device, non_blocking=True).float()
    return vid_tube

get it. thx a lot.

Authors also mention that 4 or 8 video frames work best:

The point is that, is it necessary to re-trained model for different video frame numbers? I think yes, if so, did authors train 3 different model for 4/8/16 ?


Run Dialyzer with known problems

I have a code base with some known dialyzer issues, I would like to run dialyzer in my CI Build with some way of saying "only fail" if the number of errors is > N, or if an error not on this list shows up. Is there a way to do this?

@zkessin in 18.0 -dialyzer attribute was added that allows you to selectively ignore warnings. There is no mechanism in rebar3 itself to ignore warnings because it would be inferior to using the attribute. The attribute is not effected by line changes nor different OTP releases.

@zkessin did this answer your question? Can we close this?

Yes close it


Add panel guide and examples

Congrats on the launch  of jupyter-bbox-widget.
I would like to contribute a guide for using it with Panel to enable users to create notebook tools and data-driven apps.
Ideas
The tool.py example could be further improved by

Persisting the bbox state across images
Enabling the user to more easily navigate the list of images than incrementally.
Show how to make jupyter-bbox-widget size responsively in a Panel data app.

I don't know much about Panel so I can't give this a thorough review. But setting the image property on bboxes feels unnecessary as I mentioned in #27.
Overall I think this example would be better off in its own repo or gist. Because I'm uncomfortable about adopting Panel code and guides that I will have to keep updated and relevant (and answer questions on?..). It would also be interesting to see something more fleshed out - to let the features of Panel shine through =). I can add a link to the readme if you publish this elsewhere.

Thx. For your feedback. I will just share it at awesome-panel.org when I get the time.
Just to understand my context. I'm used to other projects wanting to present how they can be used in multiple environments. For example like lonboard https://developmentseed.org/lonboard/latest/ecosystem/panel/ or https://github.com/whitphx/transformers.js.py#panel. That is why I made the contribution.


daemon: remove sysInitPath, lxc leftover

it's always initialized and/or set to an empty string because it was lxc only in the past https://github.com/docker/docker/commit/3b5fac462d21ca164b3778647420016315289034#diff-1a1f3e7ad9b1d7584e2d3e7d0c4c3db9L789
docker info now either return an empty string or the result from utils.DockerInitPath directly (which could be an empty string as well)
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

LGTM

LGTM


Update identity to 2.0.0-beta.4 for  ai-anomaly-detector

Update identity to 2.0.0-beta.4 for @azure/ai-anomaly-detector
Related epic - #14581
Instructions:

Replace tenant id. tenantId now can't be an invalid tenantId, so use the recorder's replaceableVariables to set one like 12345678-1234-1234-1234-123456789012.
Re-record the NodeJS tests, since MSAL does some more recordings.
If you find any issues while upgrading, please report them in this issue.
Reference PR - #15704 and #15654

@joheredi Can we target this milestone to get this done? We are almost at the end of updating all packages to use the latest Identity preview package in their tests!


fixing bystatusandtype

could this view emit the status and type, and not the workflow name in addition ?

in https://cmsweb.cern.ch/couchdb/_utils/document.html?reqmgr_workload_cache/_design/ReqMgr

This view is going to be useful for injecting ReReco in Unified @amaltaro should be easy to fix.

it prints request name, request status and request type:
https://github.com/dmwm/WMCore/blob/master/src/couchapps/ReqMgr/views/bystatusandtype/map.js
what's the problem printing workflow name? Can't you simply drop it in your client?

Hi, I'd like to search by status and type i.e key=assignment-approved,ReReco and get directly the workflow names. the view cannot be used as a search as it stands.

Hi Jean-Roch, yes it makes more sense to have that way. Fixes #6352
However, I remember this was requested specifically this way. I think a lot of Ops script need to be changed to get the value correctly.  As well as McM script.
https://github.com/cms-PdmV/stats/blob/master/tools/statsMonitoring.py#L183
I will try to announce that if everyone agree we will merge the patch, if not we have create the new view.


operator project-quay (3.10.0-nightly.20230815)

Add 3.10.0-nightly.20230815 to community operators

/merge possible

/merge possible


[decompilation] Replace char primitives with integer values

This boils down to seeing '\xffff' vs 65535 in the code. The latter being preferred.
It's sometimes difficult to tell at first glance if examples like the following are the same integer value.
this.readUnsignedShort() - '쀀'
this.readUnsignedShort() - '耀'
This might require an upstream fix in fernflower. Not really sure, just thought I'd open this for discussion.

This should probably be scoped to places where chars are used in arithmetic statements.

Mentioned in discord:
fesh0r/fernflower@e44ba99

Further notes from @Adam- regarding the work required.

The decision is to either update fernflower completely or simply apply that patch. the version of fernflower I use now is https://github.com/runelite/fernflower and it is versioned and deployed up to repo.runelite.net which the updater pulls in using maven. So you could PR that patch against that repo if you wanted. But to change fernflower you need to manually run the full decompile and mapping procedure locally since the CI does not cover it and I dont know of what impacts it will have
[...]
I wouldnt mind seeing fernflower updated completely if it still functions
[...]
Also I find that the decompiled result of the compiled fernflower decompile is generally drastically different from the decompiled result of the output of the deobfuscator. which I think is due to javac more than anything.  so when I update things I will re-map everything and then re-decompile it instead of redecompiling runecsape-client
so you would have to basically get 154, deobfuscate it, map the existing client to it, and then decompile that with the newer ff version and commit that

Fixed in #176

Nice!


add pod reflect controller

What type of PR is this?

**feat:**add pod_reflector_controller
What does this PR do?

reflect member cluster daemonset manage pod to host controller to keep  consistent
Which issue(s) does this PR fix?

Fixes #
Special notes for your reviewer:

experience
Screenshots (if applicable):

Additional Comments:

/lgtm


[now deploy] Check if builds are ready right away

Check if builds are ready right away so we don't show Building....
Fixes #2698

Codecov Report

Merging #2699 into canary will decrease coverage by 1.03%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           canary   #2699      +/-   ##
=========================================
- Coverage    9.27%   8.24%   -1.04%
=========================================
  Files         260     260
  Lines       11041   10530     -511
  Branches     1546    1344     -202
=========================================
- Hits         1024     868     -156
+ Misses       9955    9604     -351
+ Partials       62      58       -4

Impacted Files
Coverage Δ

src/commands/deploy/latest.js
0% <0%> (ø)
:arrow_up:

src/util/dev/router.ts
75.71% <0%> (-9.14%)
:arrow_down:

src/util/dev/yarn-installer.ts
38.09% <0%> (-4.44%)
:arrow_down:

src/util/dev/builder-cache.ts
39.9% <0%> (-3.66%)
:arrow_down:

src/util/dev/builder.ts
12.23% <0%> (-1.9%)
:arrow_down:

src/util/dev/server.ts
12.33% <0%> (-1.76%)
:arrow_down:

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 b1b1529...518dbc8. Read the comment docs.


Feature: Export PanzoomEventDetail type

What problem does this feature solve?
I want to use CustomEvent<PanzoomEventDetail> for the panzoom events in my application. But PanzoomEventDetail is not exported from @panzoom/panzoom.
Describe the solution you'd like
Export PanzoomEventDetail in panzoom.ts. I can make a PR if you want.
Describe alternatives you've considered
I don't think there are clean alternatives. My project currently imports the type from @panzoom/panzoom/dist/src/types.

Good call! I may as well export all types.


Navbar can be improve

Current navbar has simple design and we can change it by changing it position and the button can be added instead of link.
Please assign me this issue to solve the problem and to fix it as a GSSoC '23 Contributor.

Hiii, @Rupantar-Borphukan
This issue is similar to #86
Please msg here Thanks :)


'confidence' for single character is different between 'MeanTextConf()` and 'Confidence(level)'

Current Behavior
It's unconvinent to compile exe so I here used c++ bind from python. (I examed its code and it's directly using TessBaseAPI in C++)
api = tesserocr.PyTessBaseAPI(r"", 'chi_sim', 10)
api.SetVariable("save_blob_choices", "T")
api.SetVariable("lstm_choice_mode", "2")

api.SetImageFile(r"...")
api.Recognize()

choices = []
ri = api.GetIterator()
level = tesserocr.RIL.SYMBOL
for r in tesserocr.iterate_level(ri, level):
    choices.append([r.GetUTF8Text(level), r.Confidence(level)])
    for _ in r.GetChoiceIterator():
        choices.append([_.GetUTF8Text(), _.Confidence()])

print([(_.GetUTF8Text(), _.Confidence()) for _ in api.GetIterator().GetChoiceIterator()])
print(api.GetUTF8Text().strip(), api.MeanTextConf(), choices, api.MapWordConfidences() if api.GetUTF8Text() else 0)

api.End()

output:
[('员', 91.3373794555664), ('灵', 0.0)]
员 93 [['员', 99.01185607910156], ['员', 91.3373794555664], ['灵', 0.0]] [('员', 93)]

[('会', 92.27279663085938), ('针', 0.0), ('，', 0.0), ('|', 0.0), ('。', 0.0)]
会 96 [['会', 99.53471374511719], ['会', 92.27279663085938], ['针', 0.0], ['，', 0.0], ['|', 0.0], ['。', 0.0]] [('会', 96)]

[('患', 64.94859313964844), ('理', 0.0), ('写', 0.0), ('上', 0.0), ('雪', 0.0), ('由', 0.0)]
患 0 [['患', 80.64909362792969], ['患', 64.94859313964844], ['理', 0.0], ['写', 0.0], ['上', 0.0], ['雪', 0.0], ['由', 0.0]] [('患', 0)]

for most of the single characters, the MeanTextConf() returned seems to be the mean of (like '会' (99 + 92)/2 ~= 96), but for the first case '员' (99+91)/2 ~= 95, rather than 93 as the shown above.
moreover, the single character "患" has both MapWordConfidences and MeanTextConf 0 (zero), and obviously that's wrong.
pictures of above examples: (all are extracted from font files with same point size)

Expected Behavior
in the above example, since it's a single character, the confidence should be '99.01185607910156' '99.53471374511719' (those returned by r.Confidence(level)), because that's the correct character.
Suggested Fix
as above
tesseract -v
bunded, but:
tesseract 5.3.1
leptonica-1.83.1 (Jun 13 2023, 19:19:21) [MSC v.1935 LIB Release x64]
libgif 5.2.1 : libjpeg 6b (libjpeg-turbo 2.1.4) : libpng 1.6.39 : libtiff 4.5.0 : zlib 1.2.13 : libwebp 1.3.0 : libopenjp2 2.5.0
Operating System
Windows 10
Other Operating System
No response
uname -a
No response
Compiler
No response
CPU
No response
Virtualization / Containers
No response
Other Information
No response

We do not support 3rd party projects (tesseract wrappers).
Please reproduce the problem with C++ code (full working code + input image for testing and maybe desired output, AFAIK we do not active Chinese developer).
Also, provide information if some previous version (e.g. 4.x) was working as you expected.
Thanks.


Low-priority node quota

The sample code (https://github.com/Azure-Samples/batch-dotnet-ffmpeg-tutorial.git) failed to run the job as the code uses low-priority nodes and the default quota for my Batch account was 0 low-priority nodes.
Setting the instance variable LowPriorityNodeCount=0 and the DedicatedNodeCount=5 fixed the issue and allowed the job to complete.
Alternatively, I suppose you could request an increase in the low-priority node quota.

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

ID: 0d35e3d8-a644-dfb6-a16a-93679276c079
Version Independent ID: 8c6bcc7c-4428-be5b-2221-0c0302cdd281
Content: Run a parallel workload - Azure Batch .NET - Azure Batch
Content Source: articles/batch/tutorial-parallel-dotnet.md
Service: batch
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@jj-moore Thank you for providing this level of detail and taking the time to file this issue. I am having a service area engineer investigate this further and make the necessary changes so that users can move past this issue.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@mscurrell @pkshultz - Please review feedback and provide update info.

@mscurrell @pkshultz  Did you get a chance to look into this issue? Please share your insights.
We have another customer here on this issue :https://github.com/MicrosoftDocs/azure-docs/issues/52617

#reassign:@JnHs

@jj-moore This appears to be due to a recent change. Details and a path to request a quota increase are available in the note here. Thanks for the feedback!
@MicrosoftDocs/azure-cxp-triage #please-close


getting ambiguousactionexception on IDS

AmbiguousActionException: Multiple actions matched. The following actions matched route data and had all constraints satisfied:
IdsAdmin.STS.Identity.Controllers.HomeController.Index (IdsAdmin.STS.Identity)
IdsAdmin.Admin.Controllers.HomeController.Index (IdsAdmin.Admin)
for some reason this came up today. it was working fine before. not sure if it's because I updated the nuget packages.

Do you have the references between these projects - AdminUI and STS?

yes, been on it for half a day with beyond compare. turns out that when adding new project references to both project, it somehow cause that to happen. if STS add new project referece somecomm.proj then also adding that to AdminUI, the error will happen. I think in the new project structure, this won't happen since you have refactor all the share stuff into their own projects and STS won't have any of the Admin UI controllers.
btw the latest master branch throws an error when initializing the database.

Which error - can you share the details please?

Request matched multiple actions resulting in ambiguity. Matching actions: "IdsAdmin.STS.Identity.Controllers.HomeController.Index (IdsAdmin.STS.Identity)
IdsAdmin.Admin.Controllers.HomeController.Index (IdsAdmin.Admin)"
Request matched multiple actions resulting in ambiguity. Matching actions: {AmbiguousActions}
Microsoft.AspNetCore.Mvc.Internal.AmbiguousActionException: Multiple actions matched. The following actions matched route data and had all constraints satisfied:
IdsAdmin.STS.Identity.Controllers.HomeController.Index (IdsAdmin.STS.Identity)
IdsAdmin.Admin.Controllers.HomeController.Index (IdsAdmin.Admin)
at Microsoft.AspNetCore.Mvc.Internal.ActionSelector.SelectBestCandidate(RouteContext context, IReadOnlyList`1 candidates)
at Microsoft.AspNetCore.Mvc.Internal.MvcRouteHandler.RouteAsync(RouteContext context)
at Microsoft.AspNetCore.Routing.Route.OnRouteMatched(RouteContext context)
at Microsoft.AspNetCore.Routing.RouteBase.RouteAsync(RouteContext context)
at Microsoft.AspNetCore.Routing.RouteCollection.RouteAsync(RouteContext context)
at Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke(HttpContext httpContext)
at Microsoft.AspNetCore.Localization.RequestLocalizationMiddleware.Invoke(HttpContext context)
at IdentityServer4.Hosting.IdentityServerMiddleware.Invoke(HttpContext context, IEndpointRouter router, IUserSession session, IEventService events) in C:\local\identity\server4\IdentityServer4\src\IdentityServer4\src\Hosting\IdentityServerMiddleware.cs:line 72
at IdentityServer4.Hosting.MutualTlsTokenEndpointMiddleware.Invoke(HttpContext context, IAuthenticationSchemeProvider schemes) in C:\local\identity\server4\IdentityServer4\src\IdentityServer4\src\Hosting\MtlsTokenEndpointMiddleware.cs:line 60
at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
at IdentityServer4.Hosting.BaseUrlMiddleware.Invoke(HttpContext context) in C:\local\identity\server4\IdentityServer4\src\IdentityServer4\src\Hosting\BaseUrlMiddleware.cs:line 36
at Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)
at NWebsec.AspNetCore.Middleware.Middleware.MiddlewareBase.Invoke(HttpContext context)
at NWebsec.AspNetCore.Middleware.Middleware.MiddlewareBase.Invoke(HttpContext context)
at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)

think with the new project structure, this error won't happen since IDS won't have reference to admin UI project.

I think we solve this issue on gitter. You should not reference both web projects together.
If you have any another issue, please let me know.
Thanks.

no I wasn't referencing both projects. IDs references Idm and if they both references another project, the issue will show

If you want can you please paste .csproj of Admin and STS? I will check it. It sounds weird to me. :)

it's easier to create a dummy project say  common.proj and add it to both projects, the issue will show up but this issue won't occur in the new solution structure since IDs does not reference Idm directly anymore

OK, thanks for feedback, I will test it. 👍

thanks Skoruba. Great work! 👍👍👍


F# 121699 : Syracuse installer 2.999 9.6 doesn't update in automated mode

-double timout from 30000 to 60000
-update deprecated orders (getDB, command....)
-suppress unused imports

LGTM


Always displays the "NC" warning message

The "NC" warning message displays even after I installed netcat and verified that typing "nc" in the terminal works.

I've got the same problem in Ubuntu 16.10. I've tried both the nc from the netcat-openbsd package and from the netcat-traditional package.

It checks for nc's availability in a weird way - looking for hostname in result of nc --help. There are two things wrong with that: the command for help is nc -help (at least for netcat-openbsd) and hostname is nowhere to be found in there. Changed index.js:32-33 to:
const result = spawnSync('nc', ['-help']);
if (result.stderr && result.stderr.toString('utf8').indexOf('netcat') !== -1) {

and seems to be working.


Publish refactor pages helper

Context
We use site prism, which requires adding explicitly defining a page object every time we test a new page. We would like to start automating this process.
This PR aims to address this by using metaprogramming to define these methods on the fly using the file path.
Note: There are a few more ways we can refactor this such as combining the two method generators into one and moving the DfE sign in page object into the parent folder, however as this PR now touches 108 files I thought it would be good to do as a follow up PR.
Changes proposed in this pull request

Use the file paths spec/support/page_objects/publish and spec/support/page_objects/support to grab the file names, process them into a more respectable form, and use this as the method name when defining the method on the fly.

Rework the existing specs to make use of this new automatic method generator.

Guidance to review

Are we happy with the naming conventions used

Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased main
[x] Cleaned commit history
[x] Tested by running locally

 Publish - Refactor pages helper

Nice work on this so far. The logic seems to be the same so maybe extract it out so it's not duplicated then we can define any number of namespaces by just updating the location of each one via https://github.com/DFE-Digital/teacher-training-api/blob/8312feaf03921c48f55554cb6d8825c51825c63b/spec/support/feature_helpers/publish_pages.rb#L3

Nice work on this so far. The logic seems to be the same so maybe extract it out so it's not duplicated then we can define any number of namespaces by just updating the location of each one via
https://github.com/DFE-Digital/teacher-training-api/blob/8312feaf03921c48f55554cb6d8825c51825c63b/spec/support/feature_helpers/publish_pages.rb#L3

As discussed offline, this will be done in a follow up PR


feat: GitHub OAuth POC

Hey,
This PR closes #364
I have added a Auth component that makes a POST request to backend with code. Backend can ask GitHub for access_token and send frontend a JWT auth token that is being logged with consle.log for now.
I have added a .env and .env.example files.

This is merged in development with the hidden "Login with Github" button to make it easier for @charnysh-dev to start working on the whole login/logout/profile flow.


Report error with lib file path, not only line and columns

For a single file it works through.

Interpreter may need to track current file path, and a new error struct wrapping the old SchemeError with an extra file path may need to be introduced


Fix data workspace vscode telemetry

Finally got around to fixing and verifying sending the correct extension name with the data workspace vscode extension's telemetry. Similar to the fix done in https://github.com/microsoft/azuredatastudio/pull/19645.
I also noticed the fullName in IPackageInfo in the sql database projects extension was never getting used, so removing that to clean up.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 6 (83.33%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.002%) to 42.107%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/data-workspace/src/common/utils.ts
5
6
83.33%

Totals

Change from base Build<PHONE_NUMBER>:
0.002%

Covered Lines:
28152

Relevant Lines:
62380

💛  - Coveralls


Chudnovsky algorithm Python

@Kumar-laxmi , The Chudnovsky algorithm is a fast method for calculating the digits of π, based on Ramanujan's π formulae.
Please assign me this issue, I would implement it in python using binary splitting which can calculate 10000s of digits of pi in less than 0.0005 secs

Since this is an easy issue you need to implement this in all four languages.
Assigned!
@Mayureshd-18 : C, C++, Java & Python


Error with loadWithIsolate method

Illegal argument in isolate message: object is unsendable - Library:'dart:async' Class: _Future@4048458 (see restrictions listed at `SendPort.send()` documentation for more information)

same issue


Fixed undefined behavior in BASISU_OFFSETOF

The original implementation was dereferencing a nullptr which is an undefined behavior according to the C++ spec. It was triggering some automatic errors during our sanitizer tests (asan with nullptr check).
The fix replaces the offset with standard implementation of offsetof that uses compiler built in instructions when available (avoiding the undefined behavior).

Got it thanks!


Run formatter

Now that we've reenabled the jit, I thought it might make sense to re-run the formatter. I disabled formatting around the list of tests in b9test.cpp, since google's style doesn't binpack the arguments ever, and the huge list of strings was hideous.

Not too many changes, looks good to me.


Execution time of Activity Functions

Hi,
Is there a way to find out the execution time of durable function (activity functions) from Azure Portal after deployment? Would like to log these details.
Thank you!

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

ID: b934ad1d-aabb-707d-d296-02cfbb22ec5c
Version Independent ID: 5d8f8cfb-be79-fac8-62ef-3a396a3be3e8
Content: Durable Functions Overview - Azure
Content Source: articles/azure-functions/durable/durable-functions-overview.md
Service: azure-functions
GitHub Login: @cgillum
Microsoft Alias: cgillum

Closing as duplicate of: #86382


Check legal name in PDF Invoices of Incognito contributions

Current behavior: "Display name" is used
Expected behavior: "Legal name" is used

The same issue applied to the address, we want to use the main profile info. I'm preparing a comprehensive fix to take that into account at the API level.


Add flags validation

Signed-off-by: Zhou Hao<EMAIL_ADDRESS>

LGTM
bot, retest this please

Changes LGTM, but papr seems to be having deadline issues here as in #876

FWIW, I don't think there's an issue with this PR, but we probably need an adjustment in paper (Red Hat CI) and/or we need to tweak deadline settings in lint.

LGTM
@rh-atomic-bot r+

bot, retest this please

@rh-atomic-bot retry

@rh-atomic-bot r+

@rh-atomic-bot retry

@rh-atomic-bot retry

@rh-atomic-bot retry

@rh-atomic-bot retry

@rh-atomic-bot retry


chore(conf): client_max_body_size to 10m by default

Summary
Important Changed the default value of nginx_http_client_max_body_size in Kong configuration from 0 (unbounded) to 10m (10 megabytes).
Checklist

[x] The Pull Request has tests
[x] A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
[x] There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference
Fix KAG-3361

@kikito / @hbagdi what is final decision: merge or not merge this on 3.7? It may break some installations that deal with large client bodies (larger than 10mb).

Changing this to draft while we discuss on the internal ticket.


Not showing the information about a video/link to it when using Youtube XML

When adding a channels XML file to my feed and get a notice that a new video have been uploaded I just get (empty) in the *elfeed-entry*.
Reproduction of this problem should just be to add a channel, like https://www.youtube.com/feeds/videos.xml?channel_id=UC7YOGHUfC1Tb6E4pudI9STA, update and select an entry.

Was a problem with an extension called elfeed-goodies, not elfeed itself

Just to add a general note about this: YouTube used to include entry
summary / content in their Atom feeds with the video description, but at
some point they stopped using these (required!) Atom elements and switched
to the defunct Yahoo! Media RSS XML namespace. The result is no feed
summary unless the client supports Media RSS extensions… in an Atom feed.
Why did they do this? Probably just some people at Google didn't know any
better and didn't bother to check if their changes were actually valid and
correct.
Fortunately this doesn't really matter since the actual entry is the video
itself, not the description, and YouTube's mistake saves Elfeed time
processing an unneeded, and usually unwanted, video description.


Powershell note for kubectl patch command

This command does not work in Powershell. A specific note would help the developers to take care of the same.

/check-cla

/check-cla

I have signed the corporate cla. I have tried forcing the bot to re-check the CLA but still its not working. Can anyone please help on this?
/check-cla

/check-cla
Forcing cla check

/assign @jimangel

@manishbansal8843 👋 Thanks for your patience. This is a useful clarification!
Instead of a note, I'd prefer to see it formatted with a shortcode for multiple tabs. Please feel free to ask in #sig-docs if you have any questions about formatting.

@zacharysarah I have made the changes. However, i am not able to add a note in PowerShell tab to explain the reason for removing quotes around command substitution. If this is acceptable, then kindly approve it. Anyway, i have posted a query on slack. Will update the PR if there is any way to do that.

@manishbansal8843 Thanks for the changes! ✨

However, i am not able to add a note in PowerShell tab to explain the reason for removing quotes around command substitution.

The idea of multiple tabs is hopefully to make the necessary difference self-explanatory. In this case, I think it works great.
/lgtm
/approve


Stop WebServer

Hello, thank you for your work!
How can i stop webserver ?

Hi!  Thank you for the suggestion. I'll implement IDisposable on this in the next few days!
Sent with thumbs, please pardon brevity, spelling, and grammar

On Apr 8, 2017, at 10:03 PM, SkyWar-design<EMAIL_ADDRESS>wrote:
Hello, thank you for your work!
How can i stop webserver ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi @SkyWar-design , I just committed v1.1.4 and pushed an update to NuGet.  This one has IDisposable implemented so you can either use using (var s = ... or you can manually call s.Dispose().  Please let me know if any issues!

I don't think the Dispose() method is working as intended, after disposing the Server I can still reach it as well as all its routes.

Hi @AChmieletzki thank you for this, I will test it again and report back.  Cheers

Thanks, was able to reproduce.  Working on this now.

Hi @AChmieletzki I just pushed a new version to Git (https://github.com/jchristn/WatsonWebserver/commit/60adc700d226055a6bc442becd833a82aeb66ea7) and also updated NuGet to v1.2.5 with the fix.  The TestDefault project now has a console menu to allow you to call/test .Dispose().  It is working now.  Thank you for catching this.  Closing now, please re-open if the problem persists!


Feat: mux-uploader and mux-uploader-drop

Description
Getting this up so I can get OSS eyes on the code. Based off this TDD.
To-Dos (Blocking Merge)

[x] Initial README

To-Dos (Fast Follows)

[ ] Tests

Blocked by upchunk not playing well with the testing environment

[ ] Accessibility (screen reader)

@luwes @cjpillsbury Addressed all blocking comments and most fast-follows.

@cjpillsbury @luwes Updated with last blocking item: README
Christian will look into pipeline failures 👼


bug: default templates are rendering incorrectly

Couple issues with {0} and {1}s appearing in the live site, in addition, the built-in templates aren't working quite right.

quasar vite template had a missing config setting (see https://github.com/quasarframework/quasar/issues/13951)


nvim-lspconfig shortcircuits if on_attach contains vim.api.nvim_exec2 statements

Description
If the user defined on_attach functions contains a vim.api.nvim_exec2 statement (see minimum repro example below), then nvim-lspconfig will mysteriously and without error stop execution on this line and not proceed:
https://github.com/neovim/nvim-lspconfig/blob/dd11ba7b3c8f82d51b6d4dd7d68fce2d78bf78a0/lua/lspconfig/configs.lua#L333
This is bad because the next couple of lines register the user passed commands:
https://github.com/neovim/nvim-lspconfig/blob/dd11ba7b3c8f82d51b6d4dd7d68fce2d78bf78a0/lua/lspconfig/configs.lua#L335-L340
So this means that the commands field is completely ignored, and this is the reason that my rust-tools had none of its commands on start up (took way too long to root cause 😳)
Neovim version
NVIM v0.9.0-dev-1319+g10baf8971
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Nvim-lspconfig version
dd11ba7b3c8f82d51b6d4dd7d68fce2d78bf78a0
Operating system and version
Linux main 6.4.3-zen1-2-zen #1 ZEN SMP PREEMPT_DYNAMIC Sat, 15 Jul 2023 20:18:15 +0000 x86_64 GNU/Linux
Affected language servers
all
Steps to reproduce

nvim -u min.lua sample.rs (need a sample rust file to trigger rust_analyzer)
Try :SampleCommand

Actual behavior
E492: Not an editor command: SampleCommand
Expected behavior
hello world
i.e. SampleCommand was defined
Minimal config
local opt = vim.opt
local api = vim.api
local fn = vim.fn

--- Lazy bootstrap
local lazypath = fn.stdpath('data') .. '/lazy/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
  vim.notify('Bootstraping lazy.nvim...')
  fn.system({
    'git',
    'clone',
    '--filter=blob:none',
    'https://github.com/folke/lazy.nvim.git',
    '--branch=stable', -- latest stable release
    lazypath,
  })
end
opt.rtp:prepend(lazypath)

-- Problematic function
local function bad_on_attach()
  -- This statement causes SampleCommand to not be registered.
  -- Comment it out and it will regsiter
  api.nvim_exec2[[
    augroup lsp_document_highlight
    autocmd! * <buffer>
    autocmd CursorHold <buffer> lua vim.lsp.buf.document_highlight()
    autocmd CursorMoved <buffer> lua vim.lsp.buf.clear_references()
    augroup END
  ]]
end

require'lazy'.setup {
  {
    'neovim/nvim-lspconfig',
    config = function()
      local lspconfig = require'lspconfig'
      lspconfig.rust_analyzer.setup {
        on_attach = bad_on_attach,
        commands = {
          SampleCommand = {
            function()
              print('hello world')
            end,
          },
        },
      }
    end,
  },
}

LSP log
see minimal lua

nvim_exec2 takes 2 arguments not one thats why its failing

Ah you're right, just tried it manually via :lua vim.api.nvim_exec2('echo hello'):
E5108: Error executing lua [string ":lua"]:1: Expected 2 arguments
stack traceback:
        [C]: in function 'nvim_exec2'
        [string ":lua"]:1: in main chunk

But for this issue, I never got an error message. Execution just stopped silently.

I never got an error message. Execution just stopped silently.

If it's during startup, that is a Nvim core issue. Using :messages manually, should show the error.


Problem during openVINO inference

Dear Anomalib Team,
I am using Anomalib to detect anomalies in my dataset, which contains maps of displacements. I have followed the getting_started file from the notebooks section and encountered an issue during the inference phase using OpenVINO.
Here is a brief overview of my process:
Dataset and DataModule:
I used the Folder class to create the datamodule with the following code:
datamodule = Folder(
    name="abejas",
    root=Path.cwd() / "mapas/datos_noviembre",
    normal_dir="normales",
    abnormal_dir="anomalas_rural",
    task=TaskType.CLASSIFICATION,
)
datamodule.prepare_data()  # Create train/val/test/predict dataloaders

datamodule.setup()  # Split the data into train/val/test/prediction sets.

Model Creation:
I created a basic Padim model:
model_padim_final = Padim()

Training the Model:
I used the following code to train the model:
engine = Engine(task=TaskType.CLASSIFICATION)
engine.fit(model=model_padim_final, datamodule=datamodule)

The training process completed successfully.
Testing the Model:
The testing phase also yielded good results:
test_padim = engine.test(
    model=model_padim_final,
    datamodule=datamodule,
    ckpt_path=engine.trainer.checkpoint_callback.best_model_path,
)

Results:
Copiar código
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃        Test metric        ┃       DataLoader 0       ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
│        image_AUROC        │    0.9957906007766724    │
│       image_F1Score       │      0.9931600689888     │
└───────────────────────────┴──────────────────────────┘
Exporting the Model to OpenVINO:
I used the following code to export the model to OpenVINO:
engine.export(model=model_padim_final, export_type=ExportType.OPENVINO)

output_path = Path(engine.trainer.default_root_dir)

openvino_model_path = output_path / "weights" / "openvino" / "model.bin"

metadata = output_path / "weights" / "openvino" / "metadata.json"

inferencer = OpenVINOInferencer(
    path=openvino_model_path,  # Path to the OpenVINO IR model.
    metadata=metadata,  # Path to the metadata file.
    device="CPU",  # Run on an Intel CPU.
)

Inference Issue:
When I use the inferencer to predict a new image from the test data, the prediction always indicates a 100% probability of the image being abnormal. I used the following code for inference:
i, data = next(enumerate(datamodule.test_dataloader()))
images_test = data['image']
image_numpy = images_test[0].numpy()

predictions = inferencer.predict(image=image_numpy)

print(predictions.pred_score, predictions.pred_label)`

Results of Prediction:
1.0 LabelName.ABNORMAL
Dataset Verification:
I have verified the number of images in each class to ensure the Folder class accesses the folders correctly:
Number of normal images: 2327
Number of abnormal images: 727
Image Shape Verification:
I have checked the shape of the images at various stages:
Before using the Folder class: (768, 1366, 4)
In the Folder class: torch.Size([3, 768, 1366])
In the test datamodule: torch.Size([3, 768, 1366])
Transposed for OpenVINO inference: (1366, 768, 3)
Installation:
I installed Anomalib from source using the following commands in a new conda environment:
!git clone https://github.com/openvinotoolkit/anomalib.git

%cd anomalib

%pip install .

%anomalib install -v

Despite achieving good results during testing with various models (Dfm, PathCore, Padim), the inference using OpenVINO consistently fails, indicating the problem is likely not with the model itself. I have tested with many images and used various methods for loading the images (from the Folder datamodule and directly from my original folder), but the issue persists.
I dont know if the problem is related with the dataset or even if the model is overfiiting.
I have attached a file with an example from my dataset in case the problem is related to the images.

I would appreciate any guidance or suggestions you could provide to resolve the inference issue with OpenVINO.
Thank you for your assistance.
Best regards,
Alex

I got this problem too. I checked the read_image() function and I found that they divide the image by 255.
After I divided the image(in np.array format) by 255, this problem was solved,

I got this problem too. I checked the read_image() function and I found that they divide the image by 255. After I divided the image(in np.array format) by 255, this problem was solved,

@him192021 Hi, could you please elaborate on exactly what steps you take to correctly infer an image?

@haimat
something like:
image = Image.open(path).convert("RGB") image=np.array(image) / 255.0
In my case, the result was fine when using anomalib.data.utils.read_image.
However, the result was always 100% defective when I use cv2/PIL to read the image. Then I printed the heat map and found that the image becomes strange.  It seems that the OpenVINOInferencer.predict expect the input between 0 and 1.

@him192021 Thanks for your response!

Thanks for the replies @him192021 and @haimat.
In my case it still doesn't work despite rescaling the images between [0,1] so that the inference with OPENVINO works correctly.

In this image you can see how I am reading the images (normal and anomalous) with the function read_image of anomalib.data.utils.

When doing inference of the normal image, it still comes out abnormal 100% in spite of having images scaled between [0,1].
It is probably a problem with my data as they are quite complex to analyse by neural networks.


Are expired domains being removed from the list ?

Hello @lindell
Awesome package, thank you for maintaining it.
I just have a question, do you have a script that loops through that list and remove expired/unavailable  domains ?
Cheers.

Hi :)
This repo does not contain the source list. It's merely synced against https://github.com/wesbos/burner-email-providers


OpenVPN: Instances [new] - proto tcp is ambiguous

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
When creating a TCP-OpenVPN server the following message will be logged and the OpenVPN instance will fail to start.
Options error: --proto tcp is ambiguous in this context. Please specify --proto tcp-server or --proto tcp-client
To Reproduce
Steps to reproduce the behavior:

Go to 'VPN', 'OpenVPN', 'Instances [new]'
Click on '+'/Add
Select 'Role':  'Server'
Select 'Protocol':  'TCP'
Click on 'Save'
See error in 'VPN', 'OpenVPN', 'Log File'

Expected behavior
I expect the OpenVPN server to start.
Describe alternatives you considered
I've search the source code find any way to set the proto tcp-server, but was unable to find any way.
Additional context
I believe the following line(s) should be adjusted to handle this case:
https://github.com/opnsense/core/blob/21bd623835e9dfa437a61f77a4f1531e17581e25/src/opnsense/mvc/app/models/OPNsense/OpenVPN/OpenVPN.php#L499C58-L499C58
The old instances handled it in the following section:
https://github.com/opnsense/core/blob/21bd623835e9dfa437a61f77a4f1531e17581e25/src/etc/inc/plugins.inc.d/openvpn.inc#L495C12-L495C12
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 23.7.5-amd64
FreeBSD 13.2-RELEASE-p3
OpenSSL 1.1.1w 11 Sep 2023

@rbrouwer what did you configure for the server (IPv4/IPv6]) option (and possibly type)? I'm not 100% sure if this is a configuration issue about a missing validation and/or a scenario we have missed.
Looking at this, I would expect it is clear for openvpn we intend to run a server when any "server" clause is provided (which offered the tunnel), for tap it might still result in the same issue as no tunnel network would be provided there.

In my case I did try to migrate my existing OpenVPN server instances to the Instances [new] section. I ran into this case when I was attempting to migrate the TAP TCP server instance, so I provided no Server IPv4/IPv6.

@rbrouwer thanks, just wanted to check, I'll put this on my list to fix. As I see it we need an extra validation for tun and the -client / -server option for tap.

@rbrouwer https://github.com/opnsense/core/commit/163a0796f72c73f5c174c183d12138c733bf19fc + https://github.com/opnsense/core/commit/997b4d6e001492ae2157057e483cfc7c4fb0f0df should fix the issue. Out of curiosity, what made you decide to use the new instances module already? I noticed the popularity, but always a bit curious about why that is.

To be honest I was setting up OpenWrt 23.05.0-rc4 on one of my travel routers and therefore needed to be in that section and figured I could attempt get my 4 OpenVPN instances into there. I prefer to try it out earlier and rather find out issues before the server section becomes deprecated/gets removed.
That gives me time to find issues like this one I mentioned or like that one the config appears to be missing "tls-server" like the old version is doing on this line:
https://github.com/opnsense/core/blob/21bd623835e9dfa437a61f77a4f1531e17581e25/src/etc/inc/plugins.inc.d/openvpn.inc#L583
Select 'Role': 'Server'
Select 'Protocol': 'TCP'
Select 'Type': 'Tap'
Select 'Authentication': 'Local Database'
Select a 'Certificate Authority'.
I believe that gets OpenVPN to rage-quit with:
Options error: Parameter ca_file can only be specified in TLS-mode, i.e. where --tls-server or --tls-client is also specified.
But I cannot find that issue yet and you cannot reproduce that issue yet, because OpenVPN currently will rage-quit for proto being  "ambiguous in this context".


Add some logic to find readmes for images like cert-manager-webhook

I added a bit of logic to look for README.md files in directories based on an image's annotations.
For example cert-manager-webhook looks like this:
cat workdir/cache/cert-manager-webhook.latest.json  |jq -r '.predicate .annotations ."org.opencontainers.image.source"'
https://github.com/chainguard-images/images/tree/main/images/cert-manager

So this code explodes that URL, uses the trailing cert-manager element and looks for a readme in the correct workdir/sources/images/images/cert-manager/README.md location for the image.
This isn't a substitute for pulling this data from the API or chainctl img ls longer term, but it should solve the problem of some of the missing readme files for today.

That's great, thank you Jamon!


Update DevMode consensus to handle non-DevMode blocks.

DevMode consensus is updated to, before looking at block header data:

if the new fork head is not a DevMode block, raises an exception
if the current fork head is not a DevMode block and is the immediate predecessor to the new fork, it immediately accepts the new fork head as that indicates that consensus mode is changing
if the current fork head is not a DevMode block and also is not the immediate predecessor to the new fork head, it raises an exception

Easiest way I found to test was manually (fingers crossed these instructions work) in multiple xterms:
Start up a validator:
sawtooth admin keygen --force
sawtooth config proposal create -k /home/ubuntu/sawtooth/keys/validator.wif sawtooth.consensus.algorithm=poet -o config.batch
poet genesis -k /home/ubuntu/sawtooth/keys/validator.wif -o poet.batch
sawtooth admin genesis config.batch poet.batch
validator -v --public-uri tcp://<IP_ADDRESS>:8800

Start up a config TP:
tp_config -v tcp://localhost:40000

Start up REST API:
rest_api

Propose a config setting for DevMode consensus:
/sawtooth config proposal create -k /home/ubuntu/sawtooth/keys/validator.wif sawtooth.consensus.algorithm=devmode

Ensure that after you create the config proposal that in the validator log you see that the blockchain is udpated (you should see a message about the new fork head being chosen).  Depending upon the wait timer chosen by the PoET consensus module, this may take a minute or so (it is random after all).
Propose a config setting for PoET consensus:
/sawtooth config proposal create -k /home/ubuntu/sawtooth/keys/validator.wif sawtooth.consensus.algorithm=poet

Monitor the log for the validator - you should see a message when you propose the config setting sawtooth.consensus.algorithm=poet that DevMode has chosen the new fork head and it should also log a message that it detected a switch to DevMode consensus.
Repeat the propose Poet/propose DevMode several times.  You should see DevMode detecting the switch.
Then propose a config setting that does not switch consensus modes:
/sawtooth config proposal create -k /home/ubuntu/sawtooth/keys/validator.wif sawtooth.poet.target_wait_time=5

You should see DevMode fork resolver choose the new fork, but no message about detecting a change as it should be building upon a DevMode block.

Rebased on master

Tested locally

Rebase before merge


Question: move texts or  images with mouse inside trumbowyg doesn't work

Hi guys,
I working with a project that include trymbowyg. From a couple of month the move of text selected or image with mouse cursor doesn't work anymore.
Is not a trubowyg issue, but I would like to find some info to understand what is generate this strange behavior to fix it.
Thank you for any kind of information.

I can move text and image on the Trumbowyg demo: https://alex-d.github.io/Trumbowyg/
I'm on Chrome / Windows 10

Excatly... it work ... but I don't know why inside this project doesn't work now ... I would like to understand what can condition this functionality and fix what is wrong in the application where I work now.
Over this project work a lot of people and I'm not a genius on front end debugging...browser console is clean and I don't know where point my eyes.

Hi @Alex-D ,
I have solved...someone have decided to bind event on the document:
$(document).bind('drop dragover', function (e) {
                e.preventDefault();
});

I'm sorry for my inappropriate question, but it's been a couple of days behind this problem.


IOREDIS Pool. is it missing???

Is there is a way in IOREDIS to specify the pool settings, so that automatically it takes care whether we are working in a cluster, or in a sentinel.
I found another library ioredis-pool, that doesn't look impressive. Any suggestion will help.

We are using generic-pool

A generic pool like https://github.com/coopernurse/node-pool should be enough.


ninja: build stopped: subcommand failed - 'remove_if' is not a member of 'std'

Hello!
I'm trying to compile dxvk from source, and I encountered the following.
OS: Fedora 40, kernel-6.11
[299/303] Compiling C++ object src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj
FAILED: src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj
i686-w64-mingw32-g++ -Isrc/d3d9/d3d9.dll.p -Isrc/d3d9 -I../../src/d3d9 -I../../include -I../../include/vulkan/include -I../../include/spirv/include -fdiagnostics-color=always -D_GLIBCXX_ASSERTIONS=1 -D_FILE_OFFSET_BITS=64 -Wall -Winvalid-pch -Wextra -std=c++17 -O3 -msse -msse2 -msse3 -mfpmath=sse -Wimplicit-fallthrough -Wno-missing-field-initializers -Wno-unused-parameter -Wno-cast-function-type -Wno-unused-const-variable -Wno-missing-braces -DNOMINMAX -D_WIN32_WINNT=0xa00 -DDXVK_WSI_WIN32 -MD -MQ src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj -MF src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj.d -o src/d3d9/d3d9.dll.p/d3d9_mem.cpp.obj -c ../../src/d3d9/d3d9_mem.cpp
../../src/d3d9/d3d9_mem.cpp: In member function 'void dxvk::D3D9MemoryAllocator::FreeChunk(dxvk::D3D9MemoryChunk*)':
../../src/d3d9/d3d9_mem.cpp:52:25: error: 'remove_if' is not a member of 'std'; did you mean 'remove_cv'?
   52 |     m_chunks.erase(std::remove_if(m_chunks.begin(), m_chunks.end(), [&](auto& item) {
      |                         ^~~~~~~~~
      |                         remove_cv
[300/303] Compiling C++ object src/d3d8/d3d8.dll.p/d3d8_shader.cpp.obj
ninja: build stopped: subcommand failed.

Adding the following to src/d3d9/d3d9_mem.cpp seems to have fixed this compile issue:
#include <algorithm>

Which build environment is this on?

Hi there. Sorry I just updated the post to include the following

This is most likely a gcc-14 change with it removing implicit header includes. They do this about every other major release with a header or two.
https://gcc.gnu.org/gcc-14/porting_to.html

Feel free to make a PR with the extra include

Thank you. Learned a lot today. :bow:


Unclear license about the derivative fonts

STHeiti and STKaiti are both proprietary. So I think some data doesn't fit the MIT license unless you have been granted the right to include them.

Hi iology,
Thanks a lot for pointing that out! I'm not familiar with licenses etc., what would you suggest I do?
Currently I'm not including the proprietary fonts themselves in the repo but only data generated from them. Could you recommend a license that fit this situation? Or do I have to remove those data entirely?
Thanks!

Obviously those two fonts are largely based on the original works, though they look very different. I don't think there is any open source license for distributing derivates of proprietary products, even if you clarify it in the document.
So better remove them from the repo. This will also prevent other open source projects from misusing the data. :)

Thanks. I've now replaced the proprietary fonts with open source alternatives (e6d1866cddbb0bbeed750ab1db641b3d24de0ea8).

Nice. Those two sources provide very permissive licenses for derivative works.


Support docker 18.09 in the test script.

We also tested containerd 1.2 with Docker 18.09, so this change is needed in 1.2 as well.
See test failure caused by this: https://k8s-testgrid.appspot.com/sig-node-containerd#containerd-e2e-shielded-gci-1.2
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

/lgtm


Adds JavaDocs to @JobWorker

Added docs to @JobWorker as this is one of the primary interfaces to create workers and they lack important info such as unit of measurement for timeouts.
Most of the wording has been inherited from io.camunda.zeebe.client.api.worker.JobWorkerBuilderStep1.
Please double check the UoM for requestTimeout (the only one in seconds as opposed to milliseconds). This has been inferred by looking at JobWorkerManager.java#L77.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Corrado Alesso seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @namero999 !


Responsive mjml-column not working in some gmail versions

I'm using mjml for creating 2-3 column, responsive layouts. Sadly it does not work consistently in web gmail. Sometimes it looks like it's ignoring @media query and shows "mobile", one by one columns layout.
Reproduction Steps:
It's kind of hard to reproduce.
First time I notice the problem was using G Suite's Gmail variation. After opening new email, then pressing show images, email looked like "mobile" version (ignoring @media queries). After going back to main inbox and opening email again it looked "fixed", showing perfectly fine, three column layout. It was about month ago. As for now, it looks fine in G Suite, but at my friend's Gmail above problem happened again, but now email does not fix itself when going back to Gmail's inbox.
She's using quite same setup as me, we're both using Macbooks (I've got Air and she's got Pro), we are both using newest versions of Google Chrome and Mac OS High Sierra. At my computer in Inbox/Gmail all those emails looks perfectly fine...
Screenshots:
Broken gmail version:

Ok gmail version:

MJML version:
4.1.0
Email clients the bug is seen on:
Gmail and G Suite in Google Chrome v65+, Mac OS High Sierra.
Possible solutions?
Do you have any idea what might be causing it and how to fix it? As for what's causing I'm guessing Google is serving us both different versions of Gmail and the one she's using does not like @media query and it's stripping code lines containing it and thus layout looks like it should in mobile devices?
As for temp solution we are changing layout by removing columns and using only 100% width blocks, but it's not really a desirable solution, as for we started to use this framework because of it's multicolumn capabilities in the first place.
Any ideas would be much appreciated :)

As of today, gmail released a new update for Old gmail interface that entirely remove classes on every element. For now we're not aware if it's a bug or a permanent thing on old gmail.

It is indeed a general Gmail issue and looks like it's rendering emails like it used to be doing a few years ago. It will probably be fixed soon on their end, you can read more on this thread: https://litmus.com/community/discussions/7560-old-gmail-completely-broken-as-of-this-morning.

@iRyusa, @ngarnier thanks guy, that's good to hear. So maybe we won't have to redesign our newsletter :)
Did you ever encounter such a behavior:

First time I notice the problem was using G Suite's Gmail variation. After opening new email, then pressing show images, email looked like "mobile" version (ignoring @media queries). After going back to main inbox and opening email again it looked "fixed", showing perfectly fine, three column layout.

and maybe found a way to fix it?

It looks like it has bee fixed yesterday on gmail part.
The display image bug is always here, but it's fixed in the new gmail UI anyway and there's no way to prevent it.

I've already figured out that issue. I can fix such things from my end. If anyone interested. you can mail me at
<EMAIL_ADDRESS>


ADMIN PANEL ISSUE

The admin panel doesn't work anymore, even the official admin panel from the documentation doesn't work

Thank you for bringing the issue with the admin panel to our attention. We appreciate your proactive report.
Rest assured, our team is actively investigating the problem, and apologies for any inconvenience caused,
We are dedicated to fixing the admin panel and restoring its functionality as soon as possible.

We appreciate your patience and assistance with the issue regarding the admin panel. Our team has investigated and successfully resolved the problem. The admin panel is now fully functional as expected.
Your contribution in identifying this problem has been invaluable to us. Please do not hesitate to report any further issues or suggestions you might have, as we continuously strive to improve our system.
Thank you once again for your input. Your diligence helps us maintain the quality and effectiveness of our software.


Sidekick Release Candidate

This PR has been auto-generated from https://github.com/adobe/helix-sidekick/.

Changes in this release:

https://github.com/adobe/helix-sidekick/pull/165
https://github.com/adobe/helix-sidekick/pull/167
https://github.com/adobe/helix-sidekick/pull/168


Implement capnproto replication

Our profiles from production show that a lot of CPU and memory in receivers is used for unmarshaling protobuf messages. Although it is not possible to change the remote-write format, we have the freedom to change the protocol used for replicating timeseries data.
This commit introduces a new feature in receivers where replication can be done using Cap'n Proto instead of gRPC + Protobuf. The advantage of the former protocol is that deserialization is far cheaper and fields can be accessed directly from the received message (byte slice) without allocating intermediate objects. There is an additional cost for serialization because we have to convert from Protobuf to the Cap'n proto format, but in our setup this still results in a net reduction in resource usage.
We have a split router-receiver setup and this is our resource usage in staging after enabling the new replication method:

Router usage did go up as well, but we still got an overall net reduction:

We can also experiment with other formats by having a more generic flag: receive.replication-format=...

[ ] I added CHANGELOG entry for this change.
[ ] Change is not relevant to the end user.

Changes

Verification

Really interested in this too! Our profiles show similar for unmarshaling

Doesn't this include a whole new RPC framework? I'm very interested in this. Glad to finally see #7071 tackled. How do you think this compared to dRPC? And is it worth eschewing gRPC altogether?

Yes this comes with its own RPC framework which can be a double edge sword. It's one more thing to learn and troubleshoot if it goes wrong. I haven't looked into dRPC yet in practice, but maybe there is a way to abstract the RPC framework and not expose internals to end users, this way we can swap it out in future releases in a transparent manner. Right now we expose the listen port as a parameter since we need separate connections for Cap'n Proto.

DRPC allows serving both gRPC and DRPC on the same socket to allow for incrementally upgrading/migrating a fleet. We might consider doing something similar whether we go with capnproto or DRPC

Mhm, we probably need to experiment more with the transparent approach before deciding what to do. For example, we could maybe deprecate --grpc* command line parameters and have a generic listener for RPCs. Perhaps let's just continue as is and mark this as experimental everywhere?
My only suggestion would be:
          {"address": "node-1:10901", "capnproto_address": "node-1:19391"},

Instead of having two addresses here, let's have "rpc_protocol": "grpc/capnproto" or something like that. I think it would be more future-proof with regard to transparent switching between RPC protocols on the receiving end.

Perhaps we can use something like https://github.com/soheilhy/cmux to multiplex multiple protocols on the same port. This way we don't need to pollute arguments with flags for each protocol. Let me see if this is easily doable, and also add some tests for the new RPC.

This PR grew a lot, so I've moved the unrelated changes here https://github.com/thanos-io/thanos/pull/7833.

Adding e2e test: https://github.com/fpetkovski/thanos/pull/7


linetrs check

add the responsive desktop version
add contact me form

The project is indeed your hard work but there is a slight issue of
Linters / Webhint (pull_request)  it is suggested that clear that issue either by running required code or reinstalling linters file.


Upload more than one model at the same time

Hello!!!
I would like to ask you a question. In fast-stable-diffusion in Google Collab, is it possible to upload more than one model at the same time and choose between them?
And at the same time, I came across, in one of the models, this thing: "If you don't have it already, download the original VAE from Stability A.I and activate it alongside the SimpMaker3K1 model. There usually are multiple folders for models, VAE goes in the vae folder, and my model goes in the main stable diffusion folders."
Does it work in fast-stable-diffusion and if so, how to unable it.
Thanks for any answer.

for the models, you can simply rename the existing model in the the models folder in your gdrive, and for the vae, you need to put them manually in the right folder, trying to automate that process may cause other issues, so some things have to be done manually.


Exact command for obtaining 3D Pose?

Hello everyone,
I'm trying to run the following command for getting 3D output
python src/openpose_3dpose_sandbox_vmd.py --camera_frame --residual --batch_norm --dropout 0.5 --max_norm --evaluateActionWise --epochs 200 --load 4874200 --openpose output/
But I'am getting the following error
INFO:openpose_utils:start reading data: openpose_output_output//json
Traceback (most recent call last):
 File "src/openpose_3dpose_sandbox_vmd.py", line 481, in <module>
   tf.app.run()
 File "C:\Program Files\Anaconda3\envs\pose3Drnn\lib\site-packages\tensorflow\python\platform\app.py", line 125, in run
   _sys.exit(main(argv))
 File "src/openpose_3dpose_sandbox_vmd.py", line 78, in main
   start_frame_index, smoothed = openpose_utils.read_openpose_json("{0}/json".format(openpose_output_dir), idx, FLAGS.verbose == 3)
 File "C:\3d-pose-baseline-vmd\src\openpose_utils.py", line 28, in read_openpose_json
   json_files = os.listdir(openpose_output_dir)
FileNotFoundError: [WinError 3] The system cannot find the path specified: 'openpose_output_output//json'

Can anyone suggest me the proper command for getting the 3D output?
Thanks in advance

I face the same error as you faced.
Have already solve it?
If solved, could you please share how to?
Thanks


task/88/update-cypress

Codecov Report

Merging #89 (9008fa8) into main (469382b) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   72.40%   72.34%   -0.07%
==========================================
  Files          56       56
  Lines        1627     1627
  Branches      235      235
==========================================
- Hits         1178     1177       -1
- Misses        437      438       +1
  Partials       12       12

Impacted Files
Coverage Δ

client/src/components/SideBar.vue
67.28% <0.00%> (-0.94%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Make sure kie-tools is on Java 17, Maven 3.9.6, and Quarkus 3

There's a complication here, since we still have three big packages (Serverless Workflow Diagram Editor, BPMN, DMN, and SceSim Editors, and DashBuilder) on kie-tools that need GWT, and as you know, GWT together with Java upgrades usually require some work. We're on GWT 2.10, which should already support Java 17, but we'll know more once we try it.

@tiagobento I already did this analysis some time ago (https://github.com/apache/incubator-kie-issues/issues/40#issuecomment-1424170663), I'm afraid that setting any Java 9+ version to compile our GWT code will result in a failure. Consider that we're currently still compiling it with 1.8 ( see here ).
I guess the only realistic solution is to keep 1.8 as target as source in maven with JDK 17, but that will require some adjustments (on tests side in particular)


try to use stdlib 0.16 with Gleam 16.1 - still fails

rebar3 shell fails, but docker seems to be ok.

Hello! Could you share the error? I tested the main branch locally and it worked OK for me on 16.1. Thanks

Now example-echo-server git:(main) ✗ rebar3 compile is OK. Also running rebar3 shell.
Compiling echo/application
Done!
===> Analyzing applications...
===> Compiling echo
Erlang/OTP 24 [erts-12.0.2] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:1] [jit] [dtrace]

Eshell V12.0.2  (abort with ^G)
1> ===> Booted echo
POST 200 /echo

Updated test


Public inputs are appended to the proof

Noticed this inconsistency, as the Noir smart contract expects proofs to be separate from public inputs:
function verify(bytes calldata _proof, bytes32[] calldata _publicInputs) external view returns (bool)

While bb.js expects them to be appended to the proof. This requires devs to slice them out before making calls to the smart contract:
await api.acirVerifyProof(acirComposer, proof, false); // we can just pass the proof here because it already has the public inputs

const publicInputs = proof.slice(0, 32);
const slicedProof = proof.slice(32);
await contract.verify(slicedProof, [publicInputs]); // but we have to slice them before calling the contract

Would be a nice devex improvement to make bb.js output (and consume) proof and public inputs separately, to be consistent with the verifier contract

cc @rahul-kothari not sure if this is low-hanging, but certainly a neat DevEx fruit for Noir devs making use of bb.js.

Can we do it so that the verify function is able to differentiate between the publicInputs and proof by receiving only one argument?

yea this seems reasonsable! If it isn't too much work and doesn't break anything in bb.js, let's do it!

Update: With NoirJS, publicInputs and proof are now split at the Noir level.


Insert pull request description here

Insert pull request extended description here

Code review passed: code sufficiently shitty. Thank you for your contribution.


Building add-ons link is 404.

The link https://docs.microsoft.com/en-us/office/dev/add-ins/overview/office-add-ins-fundamentals is a 404.

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

ID: e62dfa25-e420-38e4-ff47-05cca62d14ec
Version Independent ID: 5ea03ce0-38b9-e005-a430-e98442e2ec83
Content: Deploy add-ins in the admin center - Microsoft 365 admin
Content Source: microsoft-365/admin/manage/manage-deployment-of-add-ins.md
Service: o365-administration
GitHub Login: @Sirkkuw
Microsoft Alias: sirkkuw

@russrimm Thank you for your feedback. We have made changes to the broken link via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/3651
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @russrimm Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Why does brew cleanup require latest version to be installed?

I don't like to delete older versions of formulas immediately after installing new versions because experience has taught me that, very occasionally, there will be a problem with the new version and I will want to revert to the earlier version.
I like to wait at least a few days, ideally a couple of weeks before deleting the older version.
In practice, the time that I both want to run brew cleanup and remember to do so, is immediately before upgrading to the next version.
However, brew cleanup does not allow me to do this:
> brew cleanup mercurial
Warning: Skipping mercurial: most recent version 3.8.3 not installed

The cleanup section of man brew doesn't mention a --force option, but I tried it just in case:
> brew cleanup --force mercurial
Warning: Skipping mercurial: most recent version 3.8.3 not installed

What is the reasoning behind disallowing the cleanup of older versions of formulas when the most recent is not installed?
Is there a simple way of removing older, inactive, versions of formulas when the most recent version is not yet installed?
(In case it isn't obvious, I'm posting this as an issue because it seems to me to be incorrect behaviour that should be fixed, but I've worded it as a question because I'm guessing there is actually a good reason the way it works as it does.)

Please see I had the same questions and opened #401 a few days ago. :) Waiting for review.

What is the reasoning behind disallowing the cleanup of older versions of formulas when the most recent is not installed?

Technically, Homebrew doesn't consider a formula to be installed unless the latest version it knows of is installed. And because only installed formulae (in the sense I explained here) are checked for older versions that could be cleaned up, this isn't happening.
I guess for most users of Homebrew this isn't an issue, as they tend to update as soon as a new version is published. Thus, for them the brew cleanup logic behaves as expected. But I can sympathize with your use case and I think it is perfectly reasonable for cleanup to remove all but the most recent installed version of a formula, even if this version happens to be older than the latest available version.
That's not exactly an answer to your question, but to be honest I don't know why it was implemented as it is (this happened before my time with Homebrew) and I agree that this behavior is not ideal. What definitely needs to be factored into the decision is what version of a formula is currently linked into the prefix, as that one cannot be cleaned up. But I guess in your case this will always be the latest one you upgraded to (even if that's not the one Homebrew considers to be the latest).

Is there a simple way of removing older, inactive, versions of formulas when the most recent version is not yet installed?

None that I'm aware of. You'd basically had to do this manually by inspecting the “rack” of a formula, i.e. the Cellar/<formula> directory in your Homebrew prefix, and then removing the ones that are no longer needed and making sure to retain the one that is currently linked and/or pinned. Certainly not ideal.
Closing this as there's already PR #401 open to address this and I'm heading over there to provide feedback. If you have something to add to the discussion in that PR, feel free to chime in. Thanks!

Thanks for the information. I did wonder if maybe that was what was happening under the hood.
Just for pedantry's sake, I feel compelled to respond to:

I guess for most users of Homebrew this isn't an issue, as they tend to update as soon as a new version is published. Thus, for them the brew cleanup logic behaves as expected.

I also update as soon as a new version is published. It's just that I want to run brew cleanup immediately before updating, rather than after, in order to ensure I can easily revert to a known-working version if I encounter problems later on.
It's super moot, though: your suggested behaviour in #401 works great for me.

The discussion in issue #401 seemed to indicate that the maintainers agreed that this was something that should be implemented, but was then closed because the person working on it ran out of steam.
Is it possible to reopen this case, or is this issue effectively dead, now (unless someone else discovers the problem independently and decides to implement it)?
N.B. I realise the above sounds passive-aggressive: it's not my intention. I'm just genuinely unclear on your process for things that you agree are issues but don't intend to work on right now. The "new issue" template suggests that only things that are actively being worked on can remain open, but if that's the case I'm not sure how you track lower-priority bugs.

@sedm0784 please don't comment on 17 month old issues asking process questions.

is this issue effectively dead, now (unless someone else discovers the problem independently and decides to implement it)?

This is the case pretty much. We welcome pull requests on this and similar issues.

but if that's the case I'm not sure how you track lower-priority bugs.

We don't. Lower priority bugs end up sitting in the issue tracker not getting fixed so we close them. If someone cares enough to fix them: great.


Mongo Introspection improvements for scalar types

Added proper support for Mongo BSON data types.
Added useful errors for type conflicts.
Removed Skip for Mongo Unit tests.
Fixes the second part of Nico's bug reported in #3529
You might want to cherry-pick this commit directly into beta as well.

Thanks a lot! Just merged this into beta and then alpha.


Use futures-await in the README examples

Big thanks for this crate, also amazing work on the examples in the readme, i was able to get up and running with it very quickly.
But i believe the readme could be made better by using futures-await. since the goal is to demonstrate how to use the crate. The callbacks are very hard and are very error-prone (even the example isn't entirely correct see #130 ) to follow for someone who isn't very familiar with the futures crate
Proposal
#[macro_use] extern crate log;
extern crate lapin_futures as lapin;
extern crate futures_await as futures;
extern crate tokio;

use futures::prelude::*;
use futures::future::Future;
use futures::Stream;
use tokio::net::TcpStream;
use tokio::runtime::Runtime;
use lapin::client::ConnectionOptions;
use lapin::channel::{BasicPublishOptions,BasicProperties,QueueDeclareOptions};
use lapin::types::FieldTable;

fn main() {
  Runtime::new().unwrap().block_on(
    init().map_err(|err| error!("{:?}", err))
  ).expect("runtime exited with error");
}

#[async]
pub fn init() -> Result<(), std::io::Error> {
	let addr = "<IP_ADDRESS>:5672".parse().unwrap();

	let stream = await!(TcpStream::connect(&addr))?;
	// connect() returns a future of an AMQP Client
    // that resolves once the handshake is done
	let (client, heartbeat) = (await!(lapin::client::Client::connect(
		stream,
		ConnectionOptions::default()
	)))?;

	tokio::spawn(heartbeat.map_err(|_| ()));
	// create_channel returns a future that is resolved
    // once the channel is successfully created
	let channel = await!(client.create_channel())?;
	let id = channel.id;
	info!("created channel with id: {}", id);

	let queue =
		await!(channel.queue_declare("hello", QueueDeclareOptions::default(), FieldTable::new()))?;

	info!("channel {} declared queue {}", id, "hello");

	await!(channel.basic_publish(
		"",
		"hello",
		b"hello from tokio",
		BasicPublishOptions::default(),
		BasicProperties::default()
	))
}

I think this is much easier to understand and showcases the crate's api better.

Even though futures-await makes the code a lot easier to read, I'm not sure adding a nightly-only dependency to the readme will make it easier for people to get started with this library especially since it does support Rust stable. Especially since async/await is to be integrated into the language proper by the end of the year futures-await is seeing a lot less activity lately. We're probably better off waiting for official support from the language.

even the example isn't entirely correct

Regarding this aspect:

this is not an error/a warning to the eyes of the compiler, and won't be until rust-lang/rust#51560 is resolved
readme examples are not tested in CI so it is currently impossible to automatically detect errors anyway (maybe something like rust-skeptic could be used ?)

I agree @kureuil, there's little reason moving to futures_await when futures are moving into std and standardized async/await support will be built into the language. Maybe in September (or October; whenever they decide to release the official version) we should revisit this?

Yup, we'll wait for official async/await + futures 0.3 before we do that

Closing, we won’t use futures-await, just stable async-await once available


[changed] Don't render div on dom when isOpen is false

I noticed that when I wanted to animate the fadein of a modal it only worked for the first time. This was because the css animation of ReactModal__Overlay--after-open only works when a div is rendered for the first time. That's why I changed the logic to render nothing if there is no modal.
I'm aware that this can break a lot of things (like using a ref for ReactModal) for other people, so maybe we need to bump a major version if we want to have this.
Changes proposed:

When not open, render null instead of div

Upgrade Path (for changed or removed APIs):
When you want to use a ref on ReactModal and use it after it's rendered, you will have to place the call to the ref in a setTimeout so it's accessible after it's rendered.
Acceptance Checklist:

[ ] All commits have been squashed to one.
[ ] The commit message follows the guidelines in CONTRIBUTING.md.
[ ] Documentation (README.md) and examples have been updated as needed.
[x] If this is a code change, a spec testing the functionality has been added.
[ ] If the commit message has [changed] or [removed], there is an upgrade path above.

This allows users to add an animation for opening the modal.

@claydiffrient any thoughts? looks good.

@CompuIves Sorry for the delay on this.  Could you change the commit message from [fixed] to [changed] ?  I think that will make it easier to indicate that it's more of a breaking change vs. a simple bug fix. I believe this is a valuable change.  I'd like to get it in for v2.

No problem! I changed it.

Hi @CompuIves, we have released v2.0.0. This will need a rebase.

Okay! Will do it today.

There were a lot of changes, I decided to just recreate the changes based on new master.
I wasn't able to reproduce the rendering of div instead of null in the tests anymore, maybe I'm doing something wrong or something has changed.

Coverage decreased (-13.5%) to 71.429% when pulling dc14e2d9b7b94b3828635e9704fa5a0f26972ff4 on CompuIves:master into 8f3898a6b613a6edb8c0d2b1a255f03fc9367d71 on reactjs:master.

Awesome, @CompuIves!
Perhaps after close the modal you can check if modal.portal == null.

Ah, that works, thanks! Added test.

yeah, that is correct.

Coverage remained the same at 84.956% when pulling 9f3fd854f229e83b8381e1747af43ccb88ecc888 on CompuIves:master into 8f3898a6b613a6edb8c0d2b1a255f03fc9367d71 on reactjs:master.

You can squash the commits and append [chore] at the begging of the commit message.

Done!

Coverage remained the same at 84.956% when pulling 1639de8e47c1dc58ae5ebd4813afc8b103097b50 on CompuIves:master into 8f3898a6b613a6edb8c0d2b1a255f03fc9367d71 on reactjs:master.


Support for Web Sockets in addition to HTTP

Enable support for Web Sockets in addition to HTTP end point.
Requested by Syed Jafri - Cafe

Try to compile this plugin - https://github.com/acoutts/chintai-zeromq-watcher-plugin
[100%] Built target unit_test
/usr/local/lib/libzmq.so: undefined reference to std::__throw_out_of_range_fmt(char const*<EMAIL_ADDRESS>/usr/local/lib/libzmq.so: undefined reference to<EMAIL_ADDRESS>collect2: error: ld returned 1 exit status
make[2]: *** [programs/nodeos/CMakeFiles/nodeos.dir/build.make:210: programs/nodeos/nodeos] Error 1
make[1]: *** [CMakeFiles/Makefile2:6422: programs/nodeos/CMakeFiles/nodeos.dir/all] Error 2
make: *** [Makefile:163: all] Error 2
    >>>>>>>>>>>>>>>>>>>> MAKE building EOSIO has exited with the above error.

@pipilink - @acoutts might be able to help. or please create a ticket on https://github.com/acoutts/chintai-zeromq-watcher-plugin

@pipilink it is complaining that it can't find libzmq on your system. Please ensure you follow the dependency installation instructions to setup zmq for your system.
http://zeromq.org/intro:get-the-software


Outdated: RtMidi

RtMidi is outdated, the current version is 4.0.0.
http://www.music.mcgill.ca/~gary/rtmidi/index.html
https://github.com/thestk/rtmidi

Thanks for posting this issue. Latest version is 4.0.0, you can update vcpkg and rebuild.


Delete tmp directory from coverage report after running

Clear and concise description of the problem
The /coverage/tmp directory is preserved after the code coverage report is finished. That's fine for local development, but it's not on CI when we need to publish the results to an analysis tool like CodeClimate.
For example, this is the size of the temp directory for a project I'm working on:
435M	coverage/tmp
4.9M	coverage/xxx-api
1.7M	coverage/coverage-final.json
988K	coverage/clover.xml
 84K	coverage/index.html
 20K	coverage/prettify.js
8.0K	coverage/sorter.js
8.0K	coverage/base.css
4.0K	coverage/sort-arrow-sprite.png
4.0K	coverage/prettify.css
4.0K	coverage/favicon.png
4.0K	coverage/block-navigation.js

Suggested solution
Add an option to the coverage config to remove the tmp directory.
Alternative
For now, I can remove the files after running the coverage script, like:
  "test:coverage": "vitest run --coverage && rm -rf ./coverage/tmp"

Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

@sheremet-va I'd like to work on this feature if you think it's worth adding a new config :)

I think it's a bug. We have code for deleting coverage folder.

Do you mean to delete the entire coverage folder on rerun?
I saw this line:
https://github.com/vitest-dev/vitest/blob/0cae8ac0aec7fd25cf13aa6d4154d8412687870c/packages/vitest/src/node/core.ts#L153
But what I'm looking for is to delete only the tmp directory.

I think I've looked into this before. Even if you tell v8 to stop coverage collection with require('v8').stopCoverage(), it will still output the final v8 coverage reports into directory pointed by process.env.NODE_V8_COVERAGE when vitest's main process exits. Simply running fs.rm for the tmp directory is not enough.

@AriPerkkio, I didn't notice that behavior, but it's still worth deleting the tmp dir. In my case, it created a new 1MB file, and it's way better than the current behavior.

Sure, I'd like it to work that way. If possible, the tmp directory should always be removed after coverage report is generated. Users should not be interested in pure v8 reports.
It might be worth looking into whether we even need to set process.env.NODE_V8_COVERAGE in the main process. After #1278 was implemented I think we only need to set the flag in workers.

Would it be possible to remove the whole coverage directory if it only includes tmp, for instance when using a console text reporter?

@kaioduarte #2144 will reduce the size of leftover coverage/tmp size. I think that is the best we can do at the moment.
@everett1992 I don't think it's possible. It seems that vitest has to set NODE_V8_COVERAGE in its main process in order to get coverage reported. I did some testing and it looks like V8 coverage reporting cannot be enabled in worker_threads (used by Tinypool) directly. This is the only comment I found about this issue https://github.com/nodejs/node/issues/28283.
We could remove the coverage report here outside vitest process, but adding coverage logic here is not ideal.
https://github.com/vitest-dev/vitest/blob/6f856c4036248430bcca5f0e40398c4a4e981ea9/packages/vitest/src/node/cli-wrapper.ts#L125-L126
The "leftover" v8 reports are written when this process exits:
https://github.com/vitest-dev/vitest/blob/6f856c4036248430bcca5f0e40398c4a4e981ea9/packages/vitest/src/node/cli.ts#L97-L98

Manual testing to see whether NODE_V8_COVERAGE works in worker_threads
> node run.js
#1 existsSync(NODE_V8_COVERAGE) false

# Run in child_process
script::NODE_V8_COVERAGE /workspaces/example-project/temp-coverage
#2 existsSync(NODE_V8_COVERAGE) true

Removing /workspaces/example-project/temp-coverage
#3 existsSync(NODE_V8_COVERAGE) false

# Run in worker_thread
script::NODE_V8_COVERAGE /workspaces/example-project/temp-coverage
#4 existsSync(NODE_V8_COVERAGE) false

// script.js
console.log('script::NODE_V8_COVERAGE', process.env.NODE_V8_COVERAGE);

// run.js
const { execFileSync } = require("child_process");
const { Worker } = require("worker_threads");
const { existsSync, rmSync } = require("fs");

const NODE_V8_COVERAGE = "/workspaces/example-project/temp-coverage";

if (existsSync(NODE_V8_COVERAGE)) {
    rmSync(NODE_V8_COVERAGE, { recursive: true, force: true })
}

checkDirectory('#1')

// This will enable V8 coverage report
execFileSync("/usr/local/bin/node", ["script.js"], {
    stdio: 'inherit',
    env: { NODE_V8_COVERAGE }
});

checkDirectory('#2')

if (existsSync(NODE_V8_COVERAGE)) {
    console.log('Removing', NODE_V8_COVERAGE)
    rmSync(NODE_V8_COVERAGE, { recursive: true, force: true })
}

checkDirectory('#3')

// This will NOT enable V8 coverage report
const worker = new Worker('./script.js', { env: { NODE_V8_COVERAGE } })

new Promise((resolve, reject) => {
    worker.on('exit', resolve);
    worker.on('error', reject);
}).then(() => {
    checkDirectory('#4')
});

// Helpers
function checkDirectory(label) {
    console.log(label, 'existsSync(NODE_V8_COVERAGE)', existsSync(NODE_V8_COVERAGE))
}

Thanks for investigation, @AriPerkkio!


Move plist install dir to /Library/LaunchDaemons

@jsingle Fixes #402. Also cleans up existing plist install location.

Can we change it from org to com?  while we're doing this

Fo sho yo

We'll need to make sure we test this properly with our next RC

Gonna add some changelog stuff too, actually.

@jsingle Review

Looks good to me :rocket:


[Feature] Descent 3

hello I'm thinking that it could be possible to create scenarios for descent 3 (I don't know how to program), but I think it could be done in 2D with a layer of translucent color that identifies what level the floor pieces are on (1-2-3 ) and you could generate events like you can with mansions of madness. Thanks for your work

It could be a possibility, as you pointed, like a mod for the second edition. But I seriously doubt that someone is really considering starting the work.


Update tutorial.md

additional comments:

line 119: parameter is called Logistic Regression
line 121 and 125: (as tabular) is shown as selection
line 249: Nearest Neighbor Class. tool in Statistics has different parameters to select (wrong ones for this tutorial). NNC in ML section is the right one

@anuprulez @khanteymoori ...

Thanks @erxleben. Looks ok to me

Thanks to @erxleben
we have two NN, the right one is in Machine Learning part.
Logistic Regression <----- Logistic Regression model, in the new version
and maybe I remove the extract parameters part,  because the new Pipeline builder tool creates both of the model and parameters.
I will apply the changes

I have applied additional comments in this PR: https://github.com/galaxyproject/training-material/pull/1929


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/VSC-Service-Account/docs-archive-test-source.git

Docs Build status updates of commit 28a5b15:
:warning: Validation status: warnings

File
Status
Preview URL
Details

archiveTargetFolder/content/someContent.md
:warning:Warning

Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

.openpublishing.redirection.json
:white_check_mark:Succeeded

archiveTargetFolder/TOC.yml
:white_check_mark:Succeeded

archiveTargetFolder/breadcrumb/DocsE2EArchiveTarget/toc.yml
:white_check_mark:Succeeded

archiveTargetFolder/content/sql-database-engine.png
:white_check_mark:Succeeded

archiveTargetFolder/docfx.json
:white_check_mark:Succeeded

archiveTargetFolder/content/someContent.md

Line 2, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


HowTo migrate mailboxes from bare metal postfix/dovecot server?

Subject

[ x ] I would like some feedback concerning a use case

Description
Hi, I am evaluating using this container project for "refreshing" my "friends and family" mailserver.  I'm currently running on CentOS 6, using postfix, dovecot (IMAP only), clam, amavis, spamassassin, TLS with an public SSL cert (not letsencrypt), but it's all tied together with mysql.  I'm not running sieve, opendmarc, or opendkim, but am interested to have those features, as well as running on a modern platform like docker.
The server's been functioning around 9-10 years, we've got about 10GB of mail in /var/mail/virtual, and I'm very interested to get off this bare-metal and into a more modern solution.
I'd like to get an idea of how I might migrate my users mailboxes, aliases and such into this container platform to make the transition as seamless as possible.
P.S.   On a side note, I would also be interested to know how people might be deploying this in a public way...e.g. I was thinking about using a digital ocean droplet, perhaps CentOS 8 with tight hardening (selinux, auditd, etc.), but am looking for opinions from experience ppl that might use this container system.  Thanks!

I've migrated two setups from bare metal CentOS 7. Moving the mailboxes was just a matter of copying files and fixing ownership/permissions. You will probably have to setup the user accounts and aliases manually. That can be a chore, depending on how many you have, but hopefully not that bad.
As for hosting there are many options, not sure if I can recommend one in particular over the others. CentOS 8 probably means podman rather than docker, which is cool and should work, but not as mainstream. That is always risky.
Good luck!

@erik-wramner thanks for the reply.  I'm looking toward CentOS 8, despite the issues with docker (which can be worked around) because I'm a fan of tight selinux and auditd policies on internet facing systems.  I don't have enough experience with that stuff on debian/ubuntu to have a solid comfort level.
Can you speak to the ownership/permissions issue?  I assume you mean UID/GID deviations between the old server and volume mounted to the container.  All mailboxes on the current system reside under /var/mail/virtual/<domain> and are owned/grouped to a username specified in /etc/dovecot/conf.d/10-master.conf.  Does this container do similar, and I just have to chown/chmod the old files to match the container's config?
I am also curious about how to configure where the volume(s) reside.  I had planned to setup a block volume on the host just for the mail container project directory and volumes.  My thought with using a dedicated block volume on the host was to keep everything related to this container (code and data) kept in one place that is easily mounted on another node, snapped, grown in size later, and etc.  In the docker-compose files you have named volumes, but not indication of specifying a hostpath.  Is there a recommended configuration here?  Should I locally modify the compose file to hard code the hostpaths for the volumes?
Again, thanks for your answers.

You're right about the ownership, all mail files are owned by the same user (docker) and reside under /var/mail.
As for the volumes many users are mapping host paths. There are pros and cons. The main advantage with volumes is that it "just works", with host paths there can sometimes be permission issues. On the other hand you know what you get with a host path, there have been bugs with docker volumes after all. You can certainly change the compose file to map host paths if you like.

Hi again,
So what is the best practice for mapping the host paths to the volumes?  I've tried a few ways, and they always end up under /var/lib/docker.  The cloud node I'm deploying will have a block storage volume mounted.  I "could" put that in /var/lib/docker, but I'd rather have it mounted as something like /data/mailserver and also have the compose file and env files on it.
Is there a good example available?

Well, the main template in the readme actually includes an example (partial extract):

    volumes:
      - maildata:/var/mail
      - mailstate:/var/mail-state
      - maillogs:/var/log/mail
      - ./config/:/tmp/docker-mailserver/
volumes:
  maildata:
    driver: local
  mailstate:
    driver: local
  maillogs:
    driver: local

Here you have three docker volumes that will end up in /var/lib/docker and one local folder (config) that is mapped directly. The folder has no mapping in the second volumes group and is a path. Please refer to the docker documentation for detailed syntax.

I have taken the route of removing the named volumes section from the docker-compose file and hard coding the volume paths into the container volumes.  This seems to work just fine.
My hardware server had my domain setup as a virtual domain, so I had to adjust the source and destination paths appropriately and set ownership permissions to what the container is using as you mentioned.
All seems well, closing.


Use https git URL for cloning srdf repo

Description
If git is not setup with ssh the cloning step will fail.
vcs import < moveit2_tutorials/moveit2_tutorials.repos

I was following this moveit2 tutorial:
https://moveit.picknik.ai/main/doc/tutorials/getting_started/getting_started.html#create-a-colcon-workspace-and-download-tutorials
Checklist

[x] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

Would be nicer to spec https URLs everywhere.
Users who'd prefer to checkout over SSH could just add:
[url<EMAIL_ADDRESS>	insteadOf = https://github.com/

to their $HOME/.gitconfig.


Update README

Update to match current branding style and grammar edits

Leaving this for jimmy as this is his baby now

@theotherjimmy

Thanks @yennster


Running failed with GLib-GIO-ERROR, Settings schema 'io.github.hugopl.Tijolo' is not installed

Running failed after build successful.
 ╰─ $ bin/tijolo
Tijolo running 🚀

(process:94082): GLib-GIO-ERROR **: 14:54:00.429: Settings schema 'io.github.hugopl.Tijolo' is not installed

Thanks.

There's a ArchLinux package on AUR, did you tried it?
https://aur.archlinux.org/packages/tijolo
If not using the package, you need to tell glib to compile the gschema, bla bla... package manager does that automatically, but I added a Makefile target for this too.
Try
make post-install

It's on README, but now I think I should just put the commands there instead of trying to explain about Gtk stuff that doesn't matter for who is just installing the application.

Run make post-install works, thank you!


Error happened when finished the test and answered questions

An error happened to complain "AttributeError: 'str' object has no attribute 'value'" in the module "CLIHandler.test_Finish_Message_For_Logs"
So the 'value' in following code might need to be removed.
        return self.loggerHandler.finish_Test(consts.RESULTS_OF_TEST_MESSAGE + self.testName + " is - " + str(testStatus.value), True, testStatus)
    else:
        return self.loggerHandler.finish_Test(consts.RESULTS_OF_TEST_MESSAGE + self.testName + " is - " + str(testStatus.value), True, testStatus,finalResults[1])

Thanks

It seems that install new version enum library enum34 can fix the issue. i.e. run the pip command
pip install enum34
Can say the issue be closed.


Player break block face context.

I don't know how possible it is but could <context.face> or <context.side> be added to get the side that a player broke the block on. Possibly returning 1-6 or north, south, east, west, bottom, top.

You should be able to do this by comparing the block location and the player location
<EMAIL_ADDRESS>Returns the compass direction between two locations.
If no second location is specified, returns the direction of the location.
Example returns include "north", "southwest", ...

That does NOT tell you what side was broken. What you are suggesting tells you what side they MIGHT have broken with accuracy as low as 50%.

<player.eye_location.precise_impact_normal>, a tag I just added, will tell you the answer :)


Persist state of the world between node restarts

Each full node will rebuild the state of the world by processing the blockchain since the genesis block. This is a time-consuming operation so periodically saving the world to disk (RocksDB?) would be helpful. The node can restore the world after a restart and bring it up to update it based on the latest information received from peers.

We keep state of the world in internal (deserialized) block structure. We can leverage this and the mechanism for persisting blocks to persist state (before serializing we can extract trees and decorate the input to db)

https://www.pivotaltracker.com/story/show/152471640