feat: Add validator that leverages symfony/validation constraints.

Leverage the symfony/validation support that's being added in Silverstripe CMS 5.2

PHP linting requirements enforced by CI have gotten harsher since my last change - I'll handle that in a separate PR.


feat: color no longer tied to task execution order

Description
Related to https://github.com/vercel/turborepo/pull/9008/files

color order is no longer tied to task execution order
color is no longer tied to task hashes which change frequently

This should end up with more consistent colors between runs
Testing Instructions

Allow CI Workflow Run

[ ] approve CI run for commit: 114fe4408790a7fefd5f0f1260b860490e4ed4cb

Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer

Hey @Shaharking, 4 months ago I created a bounty for whoever solved this issue. Thanks for solving it! If you want to collect it you have to log in on Opire, set up your Stripe account and claim it.

@nabby27
I noticed that Stripe is unavailable in my country, making it challenging to receive payments.
If you can donate to one of the charities I mentioned below, that would be greatly appreciated:
Enosh
Sahar
Alternatively, a donation to a similar charity in your own country would also be greatly appreciated.
❤


#558 Replaced fixture dates in ##-AAA-## format to ISO format dates for better x-browser compat

Re: #558
Replaced fixture dates in ##-AAA-## format to ISO format dates for better cross-browser compatibility.
Motivation and Context
##-AAA-## date format does not properly work on all browsers (e.g. Firefox) and having this format used on the unit test fixtures might confuse developers into using it.
How Has This Been Tested?
yarn test - All test passed.
Types of changes

[x] Refactor (changes the way we code something without changing its functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My code follows the code style guide of this project.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

LGTM
Awesome! Thanks a lot!!


Video Hub App 2

Sike!
:trollface:
It's just a name change. Now the app

will install as Video Hub App 2
only open and save .vha2 files
have its own settings.json

This will allow users to install version 2 while still being able to use version 1 and all the hubs they've created. 🎉
Closes #168

Works well on Windows too 👌 merging


Respond to https://hypothes.is/a/ufajxqM3EeqMxwPaiyp88Q

Respond to hypothes.is annotation ufajxqM3EeqMxwPaiyp88Q on https://jrnold.github.io/r4ds-exercise-solutions/transform.html (user<EMAIL_ADDRESS>created: 2020-05-31T12:10:32.581758+00:00, updated: 2020-05-31T12:10:32.581758+00:00)

should be >=
(at least meaning 30 or more)

Link: https://hyp.is/ufajxqM3EeqMxwPaiyp88Q/jrnold.github.io/r4ds-exercise-solutions/transform.html

No changes required. Answered with reply https://hyp.is/ufajxqM3EeqMxwPaiyp88Q/jrnold.github.io/r4ds-exercise-solutions/transform.html


Replace babel-preset-react-native for metro-react-native-babel-preset…

… in README
More information in this issue.

It still works with RN < 0.56 so I'm not sure why the instructions were removed from the readme 🤷♂


Piano #176

[x] I am making a proper pull request, not a spam.
[x] I've checked the issue list before deciding what to submit.

Related Issues or Pull Requests
#176
Pls assign the issue to me under LGM-SOC'21

@nilisha-jais pls review my pr.


The quality of this dataset

This dataset contains:

duplicated bboxes
mislabeled bbox classes
one of every 5 images contains a random bbox of a random class
most of the bboxes are not properly cropped
some images that are either extremely dark or extremely bright

At first I thought it was a problem on my end, but after further investigation it doesn't seem to be the case.
Please fix

Hi which dataset did you use and you found this problems? I am currently using [TJU-DHD-traffic]. Did you find any other object detection datasets with High resolution images, that dont have the problems you list above?


Zonal mean diagnostics need to account for non uniform grid

Simple mean of xt_ocean is incorrect.

We should be doing a weighted average using dzt for the weights.   Email correspondence:

Kial just explained to me the details of correctly calculating the zonal
mean temperature using a weighted average on cell thickness.  It appears
that the field dzt is not included in the diag_table for the runs in
hh5/tmp/cosima (as it was for the APE-MOM runs).
Should we be adding back in a line like
"ocean_model","dzt","dzt", "ocean_%3yr_%3dy","all",.true.,"none",2
to diag_table so that we can calculate the true zonal mean?

Andy's response:

Grrr, sorry, yes you are right Kial. We should have included that. I have put it back into my 0.25°
runs that are still going, but this really should have been there all along. Oh well ...

Hi,
There is a chance you are saving "rho_dzt", which for a Boussinesq fluid = rho0*dzt with rho0=1035kg/m3.  I generally saved rho_dzt since I like the extra mass weighting, so there is a chance that field was inherited by the COSIMA runs...
Steve

One important point: all land values should be listed as "missing".  We do not include land as part of any horizontal average, either in the numerator or denominator.

Neither rho_dzt nor dzt are included in diag_table for these runs. (area_t is available)
The means do not include the land (I think this handled by masked arrays or NaNs internally)
I'll add in example of taking a volume weighted average to calculate zonal or horizontal means.

Most of our runs now include dzt, so we can probably address this issue soon.
Note that in my latest pull request I have shifted zonal mean functions into cc.diagnostics.zonal_mean which is in diagnostics/overturning.py.


Dell Precision 5820

May be it's only me but the code stopped working at point 5
Here where it freeze
Compatibility Checker                       #
####################################################################

CPU:

Intel(R) Xeon(R) W-2123 CPU: Up to macOS Sequoia 15

GPU:

NVIDIA GeForce GT 710: Up to macOS Big Sur 11

Connected Monitor: HP 27xi (VGA)

Sound:

NVIDIA High Definition Audio: Unsupported
Realtek Audio: Up to macOS Sequoia 15

Audio Endpoint: Speakers / Headphones

Network:

Qualcomm Atheros AR9485 Wireless Network Adapter: Unsupported

Storage Controllers:

200 Series PCH SATA controller [AHCI mode]: Up to macOS Sequoia 15
Disable Intel RST VMD in the BIOS before exporting the hardware report and try again with the new report

####################################################
As it said to disable Intel RST WMD in the BIOS but I got the same behaviour even with WMD disabled.
IDK if there is a workaround for this.

I cant. The json is missing.


Centralise frontend network error reporting

Fixes

Fixes #3468 by @krysal
Description

I've kept this PR as a draft because:

it is just a proposal for an idea for how to solve this problem, but is not necessarily the simplest (but I think it's worth it)
it still needs additional testing and probably some small refactoring, but I don't want to invest time into that until someone else has looked at this and given the general approach/idea a :+1:

Testing Instructions

Force an error to happen in a network request on the frontend. For example, append a bogus string to the end of media IDs so that they result in 404s. Confirm that those don't get sent to Plausible or Sentry anymore (try it on main as well to see that it goes to Sentry).
I don't know how to force a network error. You may be able to do it using your browser's network debugging tab and explicitly blocking a request.
Checklist

[x] My pull request has a descriptive title (not a vague title likeUpdate index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[x] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.
[N/A] I ran the DAG documentation generator (if applicable).

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

@krysal interesting in your view on this, in particular, if you have any opinions about this approach.
The simpler alternative I can think of was to either copy/paste the conditionals of error handling to the appropriate areas, or to create a similar unified fetch error processing function that just takes context as an argument, rather than creating a plugin. The plugin feels like a convenient approach to me, but it could cause additional pains with Nuxt 3, I'm not sure.
Curious for your thoughts regardless!

should create an Error class for our custom fetchingError instead of an object, in case we need to throw this error somewhere

In JS you can throw anything. We can wrap it in a class, no problem. But is the idea to be able to check it with instanceof? We can do this in a separate issue as well, now it will be easy to switch to a different format for the error if needed.

Thanks for the preliminary reviews, Krystle and Olga. I'll fix up this PR and get it ready for a final review.

Do you know where Sentry gets the "recordError" title from, and whether we can change it?

I suppose from the function name, but I don't know how to change it. Maybe in breadcrumbs? I'll have to look at the sentry JS docs, but it's a good one to change, absolutely.

Yes! I actually have staged changes for that I forgot to push, thanks for the reminder!

I remember what I got stuck on, with Playwright. I don't know how to trigger a network error as Axios identifies them. Maybe using page.route and route.abort will work, I will try it. Just wanted to mention in case you have ideas before I'm able to get back to this.

I remember what I got stuck on, with Playwright. I don't know how to trigger a network error as Axios identifies them. Maybe using page.route and route.abort will work, I will try it. Just wanted to mention in case you have ideas before I'm able to get back to this, @obulat

You can use Playwright's route.abort like this to trigger a network error:
await page.route(new RegExp(`v1/(images|audio)/`), async (route) => {
      route.abort("internetdisconnected")
    })

I tried adding e2e error tests to this PR, and found that useAnalytics does not work when we need to call it from the store and not from the setup function. The context is not passed to this composable, even if the store's fetchMedia function is called from within the setup function. The context is not available, and the onMounted also throws an error. We could move the onMounted to sync analytics to the layout files to prevent it from running for every mounted component that uses it. However, the solution for the context not being available is more complex. Maybe we should move sendEvent to a plugin?

Maybe we should move sendEvent to a plugin?

I was thinking about this as well. There are a few composables like this that might be better suited for plugins instead of composables.

This PR doesn't work due to the analytics composable, so I'm going to close it for now, but leave the branch up. I'll open another PR to move analytics composable into a plugin.


Editing external resources should not mark current edited scene as unsaved

Operating system or device - Godot version:
Antergos x86_64 - Godot master branch
Issue description:

Expected: edited external resource to be marked as unsaved, not the current scene
Happened: the current scene is marked as unsaved, even one with no root node (empty) which crashes the engine upon closing
Steps to reproduce:

New project
Open icon.png, change its name property. OR Audio > Set bus volume
The current scene tab is now [empty](*), closing it will crash the engine. (Fixed)

Related to #6797

This is still valid in the current master branch, but it's not really a bug and more a design quirk of Godot being first and foremost a scene editor. Changes to external resources or project settings are reflected on the currently edited scene, as it's the action of saving it which will also save the other changed items.
Given that this issue didn't get much traction and OP is a ghost, I'll close this one, but a follow-up could be opened on godot-proposals (if there isn't one already) to discuss how the unsaved state of non-scene resources could be better conveyed.


Turning the screen resets the app

When I turn my screen while a timer is active it resets and starts from the beginning. And the text from the start screen is overlapping with the countdown.

Android 13 on a Fairphone 3.

Thanks for reporting this, I have updated the app to be locked in portrait mode, since that is what it was designed for. It make take a couple days before Fdroid pulls the update.

Fixed in v1.0.8


Increase worker timeout to 1 minute to allow large CSV downloads

Large CSV downloads time out with network error for Anushka, e.g. from URL:
https://explorer.mediacloud.org/#/queries/search?q=[{%22label%22:%22Diseases%22,%22q%22:%22(diarrhea%20OR%20rota%20OR%20rotavirus)%20OR%20(hiv%20OR%20aids%20OR%20avahan%20OR%20naco%20OR%20antiretroviral)%20OR%20(malaria*)%20OR%20(pneumonia%20OR%20%22pneumococcal%20vaccine%22)%20OR%20(tuberculosis%20OR%20tb%20OR%20%22bcg%20vaccine%22)%20OR%20(polio%20OR%20IPV%20OR%20(OPV%20-ship%20-vessel%20-navy))%20OR%20(%22visceral%20leishmaniasis%22%20OR%20typhoid%20OR%20%20%22Lymphatic%20Filariasis%22%20OR%20Elephantiasis%20OR%20NTDs%20OR%20%22infectious%20disease%22%20OR%20%22infectious%20diseases%22%20OR%20%22Mass%20drug%20administration%22%20OR%20%22kala%20azar%22%20OR%20%22sandfly%22%20OR%20%22Acute%20Encephalitis%20Syndrome%22%20OR%20AES%20OR%20%22neglected%20tropical%20disease%22%20OR%20%22neglected%20tropical%20diseases%22)%22,%22color%22:%22%231f77b4%22,%22startDate%22:%222017-01-01%22,%22endDate%22:%222018-07-01%22,%22sources%22:[],%22collections%22:[34412118]}]

Dokku container on mcservices1 proceeds happily with API requests up until it suddenly times out, leading to a dropped connection:
...
[08:58:50][DEBUG] mediacloud.api api.py:_query:458 | query GET to https://api.mediacloud.org/api/v2/stories/list with {'q': u'((diarrhea OR rota OR rotavirus) OR (hiv OR aids OR avahan OR naco OR antiretroviral) OR (malaria*) OR (pneumonia OR "pneumococcal vaccine") OR (tuberculosis OR tb OR "bcg vaccine") OR (polio OR IPV OR (OPV -ship -vessel -navy)) OR ("visceral leishmaniasis" OR typhoid OR  "Lymphatic Filariasis" OR Elephantiasis OR NTDs OR "infectious disease" OR "infectious diseases" OR "Mass drug administration" OR "kala azar" OR "sandfly" OR "Acute Encephalitis Syndrome" OR AES OR "neglected tropical disease" OR "neglected tropical diseases")) AND (( tags_id_media:(34412118)))', 'sentences': 0, 'text': 0, 'show_feeds': 0, 'fq': 'publish_day:[2017-01-01T00:00:00Z TO 2018-07-01T00:00:00Z]', 'sort': 'processed_stories_id', 'wc': 0, 'corenlp': 0, 'ap_stories_id': 0, 'last_processed_stories_id':<PHONE_NUMBER>, 'rows': 500, 'feeds_id': None, 'raw_1st_download': 0} and None
[2018-10-23 08:58:50 +0000] [12] [CRITICAL] WORKER TIMEOUT (pid:516)
[2018-10-23 08:58:51 +0000] [592] [INFO] Booting worker with pid: 592

From the logs, it seems that worker is happily returning something up until it attempts to run some large and slow stories/list API calls, then it ends up not returning anything for a while to the client and so Gunicorn decides that it's dead.
Increasing the worker timeout is just a nasty hack (and might backfire in weird ways), but it might just work as a temporary workaround. A better architectural fix on our end would probably be to stop providing API calls which falsely claim that they're able to return Everything (a lot of rows with a lot of fields) in a timely manner :)

in the python code, this search runs and then stops
[(https://explorer.mediacloud.org/#/queries/search?q=[{"label":"Diseases","q":"(diarrhea OR rota OR rotavirus) OR (hiv OR aids OR avahan OR naco OR antiretroviral) OR (malaria*) OR (pneumonia OR "pneumococcal vaccine") OR (tuberculosis OR tb OR "bcg vaccine") OR (polio OR IPV OR (OPV -ship -vessel -navy)) OR ("visceral leishmaniasis" OR typhoid OR  "Lymphatic Filariasis" OR Elephantiasis OR NTDs OR "infectious disease" OR "infectious diseases" OR "Mass drug administration" OR "kala azar" OR "sandfly" OR "Acute Encephalitis Syndrome" OR AES OR "neglected tropical disease" OR "neglected tropical diseases")","color":"%231f77b4","startDate":"2017-01-01","endDate":"2018-07-01","sources":[],"collections":[34412118]}])]
at
Profiling: 0.369831085205s for GET to https://api.mediacloud.org/api/v2/stories/list (with {'sort': 'processed_stories_id', 'fq': 'publish_day:[2017-01-01T00:00:00Z TO 2018-07-01T00:00:00Z]', 'wc': 0, 'feeds_id': None, 'text': 0, 'last_processed_stories_id':<PHONE_NUMBER>, 'show_feeds': 0, 'key': u'8ab9905602aa074ba654665958a3d695730543af603a518c7836d829c07cc8e7', 'sentences': 0, 'corenlp': 0, 'rows': 500,


CLI deletes custom fields when updating channel

Hello,
I was trying to update a channel's image field via the CLI:
stream chat:channel:update -t 'messagingMembersOnly' -c "chatRoom_9nuDGTNQoEpF7NFzHhQ71U" --image=null
But that command had an unintended effect as it cleared most of the channel's other fields (like name, daisieRoomType, etc)
Feature request :
Is there a way to update custom fields (like daisieRoomType) using the CLI?
Additionally, is there a way to non-destructively update a field using the CLI?

This isn't applicable anymore


AzureBackupReport legacy event

[Can you please let us know when is the AzureBackupReport legacy event being deprecated? Even an eta month would be helpful for us to plan rather than nothing.]

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

ID: 451e3982-75f3-aa8c-9aab-6704c77fecb0
Version Independent ID: 8c03b201-2d67-cbb6-92ca-1362c7d3f58f
Content: Use diagnostics settings for Recovery Services vaults - Azure Backup
Content Source: articles/backup/backup-azure-diagnostic-events.md
Service: backup
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@rajkumar-munusamy Thank you for our query, we will review and get back to you at the earliest.

#reassign:adityabalaji-msft

Hi @rajkumar-munusamy - the deprecation of the AzureBackupReport event is tied to the deprecation of Azure Diagnostics mode in Diagnostics Settings. Adding @eternovsky  who might be able to provide some details on this.
At the same time, it is highly recommended to start moving to the new events at the earliest, using the instructions provided in the same article

We are aiming to move all AzureDiagnostics schemas to dedicated tables for several reasons: ingestion performance, query performance, user experience and additional optimization that we can apply to dedicated tables.
With that said, we aren't close to removing AzureDiagnostics in the near future since the migration for Azure partners isn't simple and will take time.
Once we move schemas away from AzureDiagnostics to dedicated tables, we will announce the AzureDiagnostics retirement and will give customers 3Y time to migrate. AzureDiagnostics retirement isn't planned for the foreseeing future.

#please-close


IAR - Anomalies with autoupgrade and IAR

acorn version - v0.8.0-rc4+b6a95e8a
Scenario1:

Enable IAR feature (acorn install --features image-allow-rules=true)
Deploy apps using sangeetha/myfirstacorn:v0.0.1 and when prompted for allow images , choose "yes (this tag only)".
Deploy another app in autoupgrade format -  sangeetha/myfirstacorn:v#.#.#  . This  results app being deployed with sangeetha/myfirstacorn:v0.0.1 image .

Expected Behavior:
Is this expected ? I expected app deployment to pick the latest version of sangeetha/myfirstacorn that matched the upgrade format (v#.#.#) and be promoted for allowing this image.
Scenario2:

Enable IAR feature (acorn install --features image-allow-rules=true)
Deploy another app in autoupgrade format -  sangeetha/myfirstacorn:v#.#.#  and when prompted for allow images , choose "yes (this tag only)".
This result in  IAR created with Images:  index.docker.io/sangeetha/myfirstacorn:v#.#.# and allows all images satisfying this auto- upgrade format to be deployed.

Expected Behavior:
Is this expected ? I thought only the version that gets picked should be added to the IAR. Otherwise even the new versions (that satisfy the upgrade format) would be allowed to be deployed.
Comments from @iwilltry42
#1
Auto-Upgrade will choose the latest allowed tag, that's expected.
We could discuss a feature to add an additional prompt for allowing latest found.
That'd be quite a large task though.
#2
That's also expected, since the tag that's given from the user is a pattern - so we allow that pattern.
We could additionally add the latest option to the choices. That again needs some more engineering.

Do you want to put those up for UX discussions?

Adding to my comments...
Scenario 1 - Expected
The auto-upgrade process will only propose images that are allowed by any IAR and only if none is allowed, we prompt the user to allow the pattern.
@cjellick would you want to change this behavior and ask the user in any case?
Scenario 2 - Expected
This would need to first run the auto-upgrade process (without considering IARs), get the latest image back and then prompt the user to allow that image if it's not allowed.


[PDF] final testing and fixes

Here the fixes of the bugs, which I detected after complex testing on local.

:bomb:


Font & margins styling tweaks to new UI

Notes to self for the super basic UI tweaks:

"contents" header shouldn't link anywhere
add top padding to top bar graph so that clicking "overview" doesn't scroll it up touching the header
header R padding
arrows in "back to overview"

Whole page text changes
LEFT NAV
(none of these should be bold)

[x] contents: removed
[x] overview, clusters, appendix: 1em
[x] topics in topic list: 1em

BAR GRAPH

[x] topic titles: 1em, bold
[x] descriptions: .875em, not bold

SECTIONS

[x] topic section titles: 1.75em, bold
[x] subtopic titles: 1.25em, bold
[x] "representative arguments": 1em, bold

h1: report title (2.5rem)
h2: topic titles (1.75rem, bold)
h3: subtopic titles (1.25rem, bold)
h4: bar chart titles (1em, bold)
h5: "overview", "clusters" etc (1em, not bold)
normal text: topics in bar graph (bold), topic descriptions in bar graph (not bold), "overview" "clusters" "appendix" and topic list in left nav, most report text
small text (.875rem): representative arguments, # claims
Whole page layout changes
HEADER

[ ] .right-stack:
margin-right: 16px;

LEFT NAV

[x] margin-top: 2rem
[x] custom-drawer gets:
display: flex;
flex-direction: column;
padding: 0 12px;
[x] custom-drawer > button get:
text-align: left;
margin-bottom: 0.25rem;
[x] get rid of s
[x] "contents" is not a button; margin-bottom .5rem
[x] background-color: #f2f2f2;
[ ] border-right: 1px solid #e6e6e6;

HEADER / GRAPH

[x] #description gets margin-bottom: 2rem;

SECTIONS

[x] report div: max-width: 60rem; margin: 0 1.25rem;

language picker good-enough fix (color fix, not flag sizing)
.mdc-select__dropdown-icon {
fill: #ffffff;
}
.smui-select--standard:not(.mdc-select--disabled) .mdc-line-ripple::before,
.smui-select--standard:not(.mdc-select--disabled):hover .mdc-line-ripple::before {
border-bottom-color: #ffffff;
}

closed with #131. breaking out header fix into separate bug


Running test training fails on Macbook Pro (intel)

Hi team,
I have created a tiny graph and I want to run graphstorm locally on my Macbook Pro (intel processor) laptop to do link prediction. I don't have NVidia GPU, and I install dependencies using default pip commans (pyhon 3.11):
pip install graphstorm
pip install dgl

I use PyTorch as a backend (default, didn't change it. PyTorch version 2.0.1)
I have created DistDGL graph using provided tool, and trying to run training script. I opened ssh on my laptop and allowed to do password-less login, and it can login.
The problem starts when it tries to create a server via gsgnn_lp.py. It calls rpc.reset() at dgl/distributed/dist_context.py, line 266 and fails with NameError: name '_CAPI_DGLRPCReset' is not defined.
I think this suggests that I am missing a dependency, but I am not sure.
Could someone help me track it down, please?
Thank you.

You can try to run GraphStorm in Standalone mode: https://github.com/awslabs/graphstorm/blob/74c99ec86b54c61d3184372c15e02e6efe761f2a/tests/end2end-tests/graphstorm-ec/test.sh#L30-L31

Well, if I use this line on main branch, I get ImportError: cannot import name 'use_wholegraph' from 'graphstorm.utils' (/usr/local/lib/python3.11/site-packages/graphstorm/utils.py). But I don't think it is related...
I am also doing Link prediction, not classification or regression, so I guess I need to use gsgnn_lp.py, right? If so, I don't see how can I run it in standalone mode - it tries to initialize distributed context almost immedeately...

Can you use the latest code?
WholeGraph support was recently added.

Thanks, I have updated the code and it looks like the problem is gone. Thank you!


webdav lock and unlock: ensure response must not be cached

Make sure that Responses of lock / unlock methods MUST NOT be cached, per rfc4918.
see #disableCacheable()
testcase added.

All of the WebDAV methods are "must not cache" actually (except PROPFIND, but IMO it's a bad idea to cache it), and it is 100% obvious why. If the proxy feels like it doesn't have to send the MKCOL to Tomcat, things are not going to work. I don't know if the WebDAV Servlet should have built in headers for that or the users should use a valve, filter or something instead (like the obvious: DO NOT use a caching proxy on the WebDAV location !).
Personally I am "done" with the WebDAV Servlet. The area had no tests (Litmus was sometimes run manually by Mark it seems) so according to our previous meetings there was a need to improve that. Along the way I found design problems and bugs I had to fix, and it now seems good enough (Litmus passes and the testcase scenarios are rather complex). I don't believe there are lots of users for this, so as far as I am concerned it is more than enough.
When adding tests, you might want to look at that: https://nightlies.apache.org/tomcat/tomcat-12.0.x/coverage/org.apache.catalina.servlets/index.html#dn-a
You can set test.coverage=true (and also add execute.validate=true) in your build.properties to see if new tests improve that or if things were already covered.

This change is unnecessary. As per RFC 9111, methods other than GET are only cached if the method definition allows it. RFC 4918 is explicit that these methods are not cacheable.

OTOH, +1 if you want to rewrite (they are too verbose right now) and expand the WebDAV tests.

This change is unnecessary. As per RFC 9111, methods other than GET are only cached if the method definition allows it. RFC 4918 is explicit that these methods are not cacheable.

Currently, expiresFilter with default conf will enable cache for those LOCK/UNLOCK etc. If source server do not explicitly disable it, then other component like ExpiresFilter, Nginx/Apache Httpd, F5, Proxy Server will have to be responsible for that.
If the proxy server of our user, how to resolve it - Ask their IT team to modify proxy setting?

If the ExpiresFilter is setting cache control headers when it should then that Filter should be fixed. The same goes for any other component setting cache headers incorrectly.

If the ExpiresFilter is setting cache control headers when it shouldn't then that Filter should be fixed. The same goes for any other component setting cache headers incorrectly.

OTOH, +1 if you want to rewrite (they are too verbose right now) and expand the WebDAV tests.

I started reading rfc4918 three weeks ago, and yesterday when i submitted the code i found that @rmaucher you had almost finished WebDAV rfc support.

@markt-asf @rmaucher please advise following issue need fix or not? -- If necessary, I will submit PR.
java.lang.AssertionError: 9.10.2. Refreshing Locks - Values other than 0 or infinity MUST NOT be used with the Depth header on a LOCK method. Depth header `1` is invalid. expected:<400> but was:<200>
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.failNotEquals(Assert.java:835)
	at org.junit.Assert.assertEquals(Assert.java:647)
	at org.apache.catalina.servlets.TestWebdavServletRfcSection_9_10.testRfc_9_10_2(TestWebdavServletRfcSection_9_10.java:199)

java.lang.AssertionError: 9.10.3. Depth and Locking - The Depth header may be used with the LOCK method. Depth header `-1` is invalid. 4xx status coude expected, actual status code was 200
	at org.apache.catalina.servlets.WebdavServletRfcSectionBase$WebdavClient.lockResource(WebdavServletRfcSectionBase.java:164)
	at org.apache.catalina.servlets.TestWebdavServletRfcSection_9_10.testRfc_9_10_3(TestWebdavServletRfcSection_9_10.java:280)

java.lang.AssertionError: 9.11. UNLOCK Method - 400 (Bad Request) expected due to `No lock token was provided`., Failed to unlock(lockToken=null), actual status code was 409
	at org.apache.catalina.servlets.WebdavServletRfcSectionBase$WebdavClient.unlockResource(WebdavServletRfcSectionBase.java:289)
	at org.apache.catalina.servlets.TestWebdavServletRfcSection_9_11.testRfc_9_11_1(TestWebdavServletRfcSection_9_11.java:190)

+1 for the third one (the request will not work so it's better to send the specified 400 code immediately), I just fixed it.
For the other two, about depth, I left it as is on purpose. There's no status code mentioned for a bad depth, and there's a default value if absent. IMO pretending it's either 0 if specified or inifinity otherwise is not bad.
Note: There are some items where you must not be strict like the lock timeout just after the depth header in the code.
BTW, if you submit tests, having a WebDAV compliance testsuite is not a goal for Tomcat (it was structured that way with HTTP/2 because HTTP/2 is critical for Tomcat), although we're probably not going to refuse it. The general idea is to have good coverage and cover some complex scenarios. Maybe you can submit code to Litmus and see what happens if something is not covered by it (evidently the three items here are not since Litmus is all ok), since the goal there is to be a WebDAV compliance testsuite.
WebdavServletRfcSectionBase (renamed to WebdavServletBaseTest IMO) could be a good idea to make the existing tests less verbose. Verifying it is really the case would be start, of course.

Ok, I give up on that one, since it's already done for copyResource. I gave you credit in the commit message.


Making custom Local keyboard

Dear support,
I would like to make a custom keyboard. I have two questions:
1- Can I use the KeyboardKit Pro version code for making it, or it is possible to use the KeyboardKit?
2- I tried to have an empty keyboard and put each buttons based on what I need, but I couldn't find a tutorial to do this. I appreciate if you guide me for that.
Thanks

Hi @Rosata
The demo app has a "Custom" keyboard that shows you how to create a custom input set (the keyboard's input characters) as well as a custom layout.
I will also create a completely custom demo keyboard that doesn't use an input set or keyboard layout.


glDrawElementsBaseVertex not found

I'm implementing skeleton animation and need to use the glDrawElementsBaseVertex to draw. I try to add this call in SharpGL but no success, so I'm wondering when this function will be added? Thank you.

Let me take a look and get back to you shortly!

Hi @steenmbiz can you let me know if release <IP_ADDRESS> resolves your issue? If so, please let me know and I will close this issue. If you create any interesting projects or have any great images to share please do so and I'll share them from this project too!

I'm testing the new version today and will let you know the result asap. Thanks

I ported the WPF CelShading example to WinForms using the new draw function and it works like a charm. So I can confirm that the issue is solved now.
But I also tried creating another openGLControl in the form to render the same TrefoilKnot in the second GlControl without success. The program crashed in the shader creation function. I also checked the RenderContext sample, but it use immediate mode to render and I need the retained mode to render my skeleton animation.
So would you mind providing a multi-GLControls retained mode renderContext with shader sample for us or can provide some clues for how to do it using SharpGL.  Thank you.

Hi Dave,
I can confirm that this issue is solved now. But I need your help for multi-GLControl in the same form using retained mode rendering. Please see the reply on the GitHub section for details. I’ll share my skeleton animation example in the future. Thank you.
Steven

寄件者: Dave Kerr<EMAIL_ADDRESS>寄件日期: Wednesday, October 23, 2019 12:42:20 AM
收件者: dwmkerr/sharpgl<EMAIL_ADDRESS>副本: steenmbiz<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>主旨: Re: [dwmkerr/sharpgl] glDrawElementsBaseVertex not found (#164)
Hi @steenmbizhttps://github.com/steenmbiz can you let me know if release <IP_ADDRESS> resolves your issue? If so, please let me know and I will close this issue. If you create any interesting projects or have any great images to share please do so and I'll share them from this project too!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/dwmkerr/sharpgl/issues/164?email_source=notifications&email_token=AGGCXCXMVPBVGRCVKXJ7PNDQP755ZA5CNFSM4JBVIO52YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECANIMQ#issuecomment-545313842, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGGCXCXWOKUIRE4GWYOBYSDQP755ZANCNFSM4JBVIO5Q.


Allow JSONProtoPayloadConverter to accept ignore_unknown_fields as an optional argument

What was changed
This commit enables the JSONProtoPayloadConverter to accept ignore_unknown_fields as an optional argument so that we can utilize this functionality when converting proto to json.
Why?
When deploying multiple services out-of-sync with Temporal, proto fields may change in a way that would not necessarily cause the workflow to fail, however without setting the ignore_unknown_fields option above, otherwise backwards-compatible proto changes result in conversion errors. As mentioned in the linked issue below, "People may want to ignore unknown fields and right now they have to remake the class themselves"
Checklist

Closes https://github.com/temporalio/sdk-python/issues/315

How was this tested:

Spinning up a worker / workflow that accept an initial version of proto args
Changing the proto definition in the client triggering the workflow to include an additional field
Triggering the workflow with the newer version of proto args, while worker/workflow use older version
Verifying the workflow converts the args and completes successfully

Any docs updates needed?

N/A

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jeremy-cho-skydio:x: william-almy-skydioYou have signed the CLA already but the status is still pending? Let us recheck it.


Finished Plasma theme installer + Patcher + More small changes

Also #17 - Added lightly plasma theme patch to remove panel shadows
Bumped version up to 0.2.1

Thanks for catching that spelling mistake :p. Merged. With this, I think it's ready to merge into main.


Script to create new projects

From issue #17  I created a minimal example of what I had in mind. This is a minimal example and some questions need to be answered.
Path of the new project
We need to create a folder for the project. I propose to use 2 different name : project_path and project_name. We then could have an architecture like :
+ project_path
    + project_name
        - __init__.py
        + models
            + networks
                - __init__.py
                - factory.py
                - mynetwork.py
    + logs

(+ denotes a folder, - a file)
This is an open suggestion. I choose this implementation so logs' folder is also created and is with the rest of the code.
Network as minimal example
I did a minimal example so we can discuss implementation details. Creating files and folders is done with pathlib.Path to use the / operation to join paths. The real question is : how to we fill created files with basic content ? I choose to have raw text written in the new.py I am Pull Requesting but I don't think that's the better option as this is not easily maintainable and it leads to a really long new.py file. My suggestion is to have a basic file we can copy to the project created. This requires to store quite a lot of files in the lib only used to be copied so it has some drawbacks too.
I am also working on a complete project creation with options to create any relevant directory but I didn't want to pollute the discussion with useless lines of code

@Ricocotam @MicaelCarvalho
Much needed!
However, we need a way to avoid putting executable code into a string such as this
I would suggest creating template python files in the bootstrap directory, copying them and even replacing some elements in it (maybe???).
For instance, instead of this, we would have the following file in bootstrap/template/project/models/networks/network.py:
import torch.nn as nn
class MyNetwork(nn.Module):
    def __init__(self, *args, **kwargs):
        super(MyNetwork, self).__init__()
        # Assign args
    def forward(self, x):
        # x is a dictionnary given by Dataset class
        pred = self.net(x)
        return pred  # This is a tensor (or several tensors)

Alright, so I did a sample of this on the latest commit

Thanks! @Ricocotam
I added some comments
We should probably add a test to verify that everything can work.
It implies to have some default return values (torch.tensor(0)) instead of raise NotImplementedError or pass.
What do you think? @Ricocotam @MicaelCarvalho

I'll fix your different comments soon. Anything to add ?

So we need some tests now
The only thing I didn't do is to name custom dataset after the project name, I kept mydataset.py : myexample.bootstrap.pytorch/myexample/datasets/myexample.py-> myexample.bootstrap.pytorch/myexample/datasets/mydataset.py
I can write some tests (we just have to check files have the right content and that they are created) but I know how it works already. May be, to ensure code is both clear and that agree on what/how it does it, someone else should right the tests

Solved in #35

This PR broke master and all tests... Please be more careful and make sure tests pass before merging ;)


Add Hello OpenLab job

Related-issue: https://github.com/Yikun/hello-openlab/issues/1

recheck

recheck


:bug: Verify secret reference exist

Summary
Verify secret reference exist in the ControlPlane before accessing its fields, to prevent controller-manager crash.
Related issue(s)
Fixes https://github.com/kubestellar/kubestellar/issues/1995

/approve


Unable to load python extension

Type: Performance Issue
Why do keep getting below error message ?
Extension activation failed, run the 'Developer: Toggle Developer Tools' command for more information.
Also 'Python extension loading' keeps loading...
Extension version: 2022.21.13431008
VS Code version: Code 1.74.0 (5235c6bb189b60b01b1f49062f4ffa42384f8c91, 2022-12-05T16:38:16.075Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
7.68GB (1.92GB free)

Process Argv
--crash-reporter-id 00957173-f655-42bb-ad4f-c3d1535939b1

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	    92	  7708	code main
    0	    86	  2204	   issue-reporter
    1	   152	  3572	   window (combined.py - L1T18 - Visual Studio Code)
    0	    26	  5656	   crashpad-handler
    0	    39	  6840	   utility-network-service
    0	   212	  8300	   extensionHost
    0	    76	  5728	     "C:\Users\Latitude E5470\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node "c:\Users\Latitude E5470\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=8300
    0	    80	  5984	     "C:\Users\Latitude E5470\AppData\Local\Programs\Microsoft VS Code\Code.exe" --ms-enable-electron-run-as-node "c:\Users\Latitude E5470\.vscode\extensions\mtxr.sqltools-0.26.0\dist\languageserver.js" --node-ipc --clientProcessId=8300
    0	    11	  8524	     "c:\Users\Latitude E5470\.vscode\extensions\ms-vscode.cpptools-1.13.7-win32-x64\bin\cpptools.exe"
    0	    10	 10636	       console-window-host (Windows internal process)
    0	    37	 13660	     "C:\Users\Latitude E5470\AppData\Local\Programs\Python\Python310\python.exe" completion.py
    0	    11	 11616	       console-window-host (Windows internal process)
    3	   129	 10236	   gpu-process
    0	    94	 11224	   shared-process
    0	    71	 10976	     ptyHost
    0	    74	 14268	     fileWatcher

Workspace Info
|  Window (combined.py - L1T18 - Visual Studio Code)
|    Folder (L1T18): 17 files
|      File types: txt(3) py(3) pdf(2)
|      Conf files:;

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242cf:30382550
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593cf:30376535
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
azure-dev_surveyone:30548225
pyindex848cf:30577861
nodejswelcome1cf:30587006
282f8724:30602487
gswce1:30612156
iaj6b796:30613358
dbltrim-noruby:30604474
f6dab269:30613381
fim-prod:30623723

Can you run that command it suggests you run and edit your issue to include what it says?

Do you use any of the open source forks of VSCode? If yes, then you need to enable proposed APIs for the Python extension. Find argv.json for your VSCode fork, for Code OSS on Linux it is ~/.vscode-oss, at the end of the file add "enable-proposed-api": ["ms-python.python"]. It should look like this.

Thanks for the feedback. I have deleted and re installed vs code and it
seems to have solved the issue thanks.
Regards
Jan Hendrik
On Mon, 12 Dec 2022, 18:39 Brett Cannon, @.***> wrote:

This may be a duplicate of #20247
https://github.com/microsoft/vscode-python/issues/20247 , but we need
to know what version of VS Code and the Python extension you are running to
attempt to diagnose. And as others have asked, are you running actual VS
Code or VSCodium (or some other fork we don't know about)?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode-python/issues/20354#issuecomment-1346848819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVIA46M6CXX4WOPN2ARU6WDWM5ILLANCNFSM6AAAAAASZI7E7M
.
You are receiving this because you authored the thread.Message ID:
@.***>

This may be a duplicate of #20247 , but we need to know what version of VS Code and the Python extension you are running to attempt to diagnose. And as others have asked, are you running actual VS Code or VSCodium (or some other fork we don't know about)?

I'm also showing Extension Activation Failed, but unlike that issue, it shows this in my Developer Tools:
Error: Extension 'ms-toolsai.jupyter' CANNOT use API proposal: notebookCellExecutionState.
I'm using Code-OSS currently and I have yet to try the fix shown above. I'll try that and update about how it goes.
By the way, I found issue #18109 first and tried deleting everything in the folder and reinstalling, but to no avail.


Error in init.lua:77: attempt ot compare number with boolean (ripgrep check fails)

🐛 Describe the bug
Hi! Thank you for your plugin!
I'm just getting started with Obsidian and encountered this error whenever I open (or close) a buffer containing a note in the vault folder (or switching to the note buffer from another buffer).
See screenshot, first line:

I took a quick look at the code, and as far as I understood we execute it everytime open a buffer containing a note in the vault. Anyway, the line that fails is:
  if os.execute "rg --help" > 0 then
    echo.err "Can't find 'rg' command! Did you forget to install ripgrep?"
  end

I do have ripgrep installed.
I tried running os.execute() inside nvim and I get the following:

So the issue must be in the parsing of the return code.
:h os.execute gives us:

As far as I understand, to cover all cases one should test Linux, MacOS, Windows (?) and many of the possible shell combinations (bash, sh, fish, zsh etc). Seems a bit cumbersome. Since the use of ripgrep is optional and it is clearly specified in your readme, maybe this check is not even so important.
I tried fuzzy searching in all the plugins I have and none seem to implement a similar check (would have been cool to copy paste their solution). Even telescope does not seems to enforce it as a requirement.

Sorry I didn't provide you a snippet of code issuing the error, but you probably got the point anyway ahah
Versions
nvim --version
NVIM v0.8.1
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/gcc -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -Wp,-U_FORTIFY_SOURCE -Wp,-D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -DNVIM_TS_HAS_SET_ALLOCATOR -O2 -g -Og -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/builddir/build/BUILD/neovim-0.8.1/redhat-linux-build/cmake.config -I/builddir/build/BUILD/neovim-0.8.1/src -I/usr/include -I/usr/include/luajit-2.1 -I/builddir/build/BUILD/neovim-0.8.1/redhat-linux-build/src/nvim/auto -I/builddir/build/BUILD/neovim-0.8.1/redhat-linux-build/include
Compiled by mockbuild@koji

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Run :checkhealth for more info

nvim --headless -c 'lua print(require("obsidian").VERSION)' -c q
1.6.1

Hey @Mrcuve0, thanks for all the details. I'm going to remove the check like you suggested: #81
I can't believe os.execute() doesn't have a consistent return type... 😞


display the target hierarchy when displaying the currently set target

and indicate whether targets are present/missing, or linked. For example:

@rgrover @BlackstoneEngineering @bremoran  articularly interested if you have any feedback about this

Personally, I think it's a very good thing to have.

This is really awesome and terribly handy! Now just need tab completion for 'yt target see' ... etc XP


Add simple smoke tests

I'm merging this one now - this doesn't break anything an I need it for setting up CI.


Heuristic shouldn't add overall general space-views if the existing views cover it in aggregate

https://github.com/rerun-io/rerun/pull/5164 covered the simpler case of existing space views that fully cover a recommended space-view

However, there are still transitive states during startup when the heuristic will propose a "bad" space-view with / as the root.
Consider the logging sequence:
img1/detections
img2/detections
img1
img2

Once the two images have been logged, the image-split heuristic kicks in and we end up with two views:
img1/**
img2/**

However, even if we have this blueprint initialized, we hit the following issue:
First, img1/detections on its own proposes img1/detections/** which is redundant with img/** and correctly ignored.
Next, the introduction of img2/detections causes the proposal of /** as a common ancestor of img1/detections and img2/detections
Unfortunately, the logic from https://github.com/rerun-io/rerun/pull/5164 doesn't work here since /** isn't covered by either of the existing space-views. This causes /** to get added back to the scene.
Possible solutions
One option is to do a redundancy analysis that actually considers the concrete entities that would be added.  In this case both entities are covered by existing views, even though /** would match many things that aren't.  We could do something like add a adds_entities: Vec<EntityPath> to the RecommendSpaceViews result and filter that against all the existing space-views.  This is probably the easiest thing to do if the performance cost isn't too bad.
A different approach might be to find a place in the blueprint to store some kind of state in the blueprint that explicitly tracks /** as a split-path for 2D views. Basically the heuristic could check something in the blueprint and conclude the space should continue to be split for this blueprint even though the images don't actually exist yet in this particular recording. This would have the added benefit that img3/detections would continue to do the right thing, even if img3 hadn't been logged yet.

Note, there is a particularly annoying interaction here with https://github.com/rerun-io/rerun/issues/5165
The analysis for splits leverages store subscribers, but since these leak from recording to recording, opening, then closing, then opening the same recording can produce different heuristic results.

I ran into this today, staring with no app.ron nor any saved blueprint:
https://github.com/user-attachments/assets/b991ab29-e290-4f96-ba1d-7142f90774fa


Issue #257 — support Ctrl+Enter to quick-add events.

Hello,
This is for issue #257 I added an event for when the user use Ctrl + Enter when the quick-add action is visible.

Hello,
I added all you comments into my fixes, so please check again.
Regards,
Jeremy

Thanks again, Jeremy!


fatal error C1189: #error: Your C implementation is not IEC 559 compliant, which is required for proper Vulkan interop

happened When I tried to use CMake to use Vulkan APIs. Looks like a compiler issue in w64devkit

Are you sure you're invoking the right compiler? C1189 is an MSVC error
code, and that's a Visual Studio error message.

Yes you are right, it's a MSVC issue. Thanks


Straight to order confirmation?

Is your feature request related to a problem? Please describe.
User needs to be able to use PayPal button to make an order from the cart without clicking a 'place order' button in my app.
Describe the solution you'd like
Can I use PayPal to skip checkout and go directly to an order confirmation to place the order without having to click a 'place order' button in checkout?
Describe alternatives you've considered
I have tried to use Navigation to make this happen, but I am not able to do it so far. Is there a way to do this using the SDK? Or perhaps this has already been done by someone?
Additional context
This is an existing app that uses the PayPal SDK version 0.6.2

Hi @ajsherrell,
Thank you for reporting the issue. Could you explain in details what is the current flow looks like and what exactly are you trying to do? Also, what button are you using? Is it the Paymentbutton & PaymentButtonContainer that we ship in the android-checkout-sdk or it's standalone button that you define inside your app?

Hi @mahmoud-turki ,
Update: I am able to code to have PayPal skip checkout in my app, however, I would like an option to have the PayPal UI button to say something like "Place Order" or something equivalent. Currently the button to continue back to the app says
"Review Order". Is this possible? Thanks.

I am tasked by my Project Manager to analyze if it is possible to skip the checkout screen after returning to our UI from the PayPal UI. This is to better improve the probability of the user to complete their order.
I actually do have the userAction set to UserAction.PAY_NOW. But I am still seeing the "Review Order" button label. Is this expected. How can I get the PayPal UI button label to say "Complete Purchase"?
This is my setup:
val checkoutConfig = CheckoutConfig(                         application = app,                         clientId = BuildConfig.PAYPAL_CLIENT_ID,                         environment = BuildConfig.PAYPAL_ENVIRONMENT,                         returnUrl = PAYPAL_RETURN_URL,                         currencyCode = CurrencyCode.USD,                         userAction = UserAction.PAY_NOW,                         paymentButtonIntent = PaymentButtonIntent.CAPTURE,                         settingsConfig = SettingsConfig(                             loggingEnabled = true,                             shouldFailEligibility = false                         ),                         uiConfig = UIConfig(showExitSurveyDialog = false)                     )

When you say "skip the checkout screen"  are you referring to PayPal checkout screen that you attached above? Could you please attach a video for the current experience from start to end?
| How can I get the PayPal UI button label to say "Complete Purchase"?
You need to assign UserAction.PAY_NOW as well to the order itself when you create it. Are you starting the checkout with an order object or orderId?

When you say "skip the checkout screen" are you referring to PayPal checkout screen that you attached above? --no, I mean the Checkout Screen in my app. I want to skip my Checkout Screen and go directly to my Order Completed Screen after the user clicks the "Complete Purchase"/"Review Order" button from the PayPal UI.
Are you starting the checkout with an order object or orderId? --I am starting checkout with the orderId.
Could you please attach a video for the current experience from start to end? --I am in the process of asking my manager for permission to share a video to this platform.
You need to assign UserAction.PAY_NOW as well to the order itself when you create it. --How do I assign the userAction of PAY_NOW to the order itself? I am currently using PayPal's CreateOrderAcitons class with the set(orderId: String) function:
PayPalCheckout.startCheckout(                         createOrder = CreateOrder { _it: Create OrderActions_                             it.set(data.processorTransactionId)                         }                     )

no, I mean the Checkout Screen in my app. I want to skip my Checkout Screen and go directly to my Order Completed Screen after the user clicks the "Complete Purchase"/"Review Order" button from the PayPal UI.

Just to give a little bit of context here;
PayNow flow -> The PayPal button label will be "Complete Purchase", once the user hit that button we finish the checkout after we authorize the payment. Ideally the merchant should show complete order screen and nothing else needed from the user at this point.
Continue flow ->  The PayPal button label will be "Review Order", once the user hit that button we finish the checkout after we authorize the payment, then the user have to finalize the checkout on the merchant screen. why? bc in the flow sometimes the user might want to change shipping info and the merchant wanna update the tax or cart total...etc. Then the merchant should show the complete order screen.

How do I assign the userAction of PAY_NOW to the order itself? I am currently using PayPal's CreateOrderAcitons class with the set(orderId: String) function:
I see, so it seems that the merchant backend create the order itself and return and orderId which is what you use to start the checkout experience. Could you check with your backend team what userAction they assign to the order? If they assign PayNow then you should see complete purchase but if they assign Continue then you should see review order

Hope that helps clearing any confusion :)

@mahmoud-turki  how do you set user action in order creation api, i meant whats the key and value for that.

@AnAnD4T1 There are two ways to set user action.

how do you set user action in order creation api

On server side: When you create the order you should see application_context object which include user_action
On client side: When you create the CheckoutConfig() you should see user_action

i meant whats the key and value for that.

Please check the docs :)

On server side: https://developer.paypal.com/docs/api/orders/v2/#definition-application_context
On client side: https://developer.paypal.com/limited-release/paypal-mobile-checkout/android/

Hi @mahmoud-turki ,
I have an update. I asked our backend developer to check what the user_action is set to. He discovered: "From backend, there is not value sent to PCS for paypal user action, but the acsURl received redirects to sandbox with user action as continue(Looks like default value from cardinal)."
As I mentioned earlier, I have the UserAction set to PAY_NOW in the CheckoutConfig() in my code.
Here is how I have implemented it:
`val checkoutConfig = CheckoutConfig(
application = app,
clientId = BuildConfig.PAYPAL_CLIENT_ID,
environment = BuildConfig.PAYPAL_ENVIRONMENT,
returnUrl = PAYPAL_RETURN_URL,
currencyCode = CurrencyCode.USD,
userAction = UserAction.PAY_NOW,
paymentButtonIntent = PaymentButtonIntent.CAPTURE,
settingsConfig = SettingsConfig(
loggingEnabled = true,
shouldFailEligibility = false
),
uiConfig = UIConfig(showExitSurveyDialog = false)
)
                PayPalCheckout.setConfig(checkoutConfig)
                PayPalCheckout.startCheckout(
                    createOrder = CreateOrder {
                        it.set(data.processorTransactionId)

//                            it.create(createOrderRequest())
}
)
PayPalCheckout.registerCallbacks(
onApprove = OnApprove {
checkPayPalAuthentication()
},
onError = OnError {
setState(State.IDLE)
viewState.uiEvents.post(PayPalViewState.Event.Error(it))
},
onCancel = OnCancel {
setState(State.IDLE)
viewState.uiEvents.post(PayPalViewState.Event.Canceled)
}`
As a reminder, I need to have the button label in the PayPal UI as "Complete Purchase". It is currently "Review".

Here is what we are getting from the Cardinal acsUrl:

This is the Cardinal lookup request to get acsUrl:

Please advise as to how we can get this default value from Cardinal changed to PAY_NOW.
Thanks!

@mahmoud-turki thanks

Hi @AnandASanthosh - Sorry for the delay in response. Are you still looking for support on this issue?

@AnandASanthosh - closing this issue for now. Feel free to respond here if you're still seeing the issue. Thanks!


x/website: the marketing department has taken over

What is the URL of the page with the issue?
go.dev
What is your user agent?
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/95.0.4638.69 Safari/537.36
Screenshot

What did you do?
browsed to golang.org
What did you expect to see?
the old design, or any design that has a normal desktop-friendly UI
What did you see instead?
This huge, mobile-first design, on my 13" laptop
If you have to do it for marketing purposes, I get it, but put a link at the header for an actual developers-friendly site.

I have a 13" laptop as well, but the proportions don't look like that. I suspect that you have an older generation MacBook Air with a lower screen resolution, and the style sheets falsely switch to mobile mode. This probably can be fixed by improving the CSS media queries.

That screenshot looks like 250% scaling on your side.
@drornir is there anything affecting the scaling on your browser?
For reference 100% (on a 1080p resolution screen) looks like:

I can't reproduce this either. @drornir, if you can help us reproduce it, we'd be happy to fix that. But there's only one web site now. See https://go.dev/blog/tidy-web for more. Will leave open for a few days in hope of getting a repro case.


Ventana preguntas

Agregado de Ventana de pregunta
related to: #41

Codecov Report

Merging #49 into V1.0 will decrease coverage by 5.35%.
The diff coverage is 0.00%.

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


Typo in URL of page "Compiler warnings C4800 through C5999"

The URL of this page is

https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warnings-c4800-through-c4999?view=msvc-170

which, by the page's name, should be

https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warnings-c4800-through-c5999?view=msvc-170

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

ID: ddc3041c-b233-bb3f-513f-b3df302b49a2
Version Independent ID: aad153bd-a417-01fb-f2b7-50842269dcc0
Content: Compiler warnings C4800 Through C5999
Content Source: docs/error-messages/compiler-warnings/compiler-warnings-c4800-through-c4999.md
Product: visual-cpp
Technology: cpp-diagnostics
GitHub Login: @corob-msft
Microsoft Alias: corob

@studyingegret
Thanks for the report; we're aware of the issue and currently choose to ignore it. The file was named before there were warnings in the 5000-5999 range. When they started to appear, rather than create a new article, it was easier to append them to this article and give it a new title. When there are enough warnings in that range to merit their own article, we'll split them out, and the file name will once again match the content. Until then, only the eagle-eyed such as yourself will notice the discrepancy.


Background insert doesn't update stream in foreground when opened

I have an app with a chat page. The chat page itself is grouped by rooms, e.g. another user. When the chat screen is opened and I receive a message, it works fine, also when the app is closed and I receive a message and I open the chat page it is updated properly.
But when the app is in background with a given chat page, e.g. with USER1 and a message is received and stored and I open the  chat page, the stream is not updated properly. I think it is due to  the fact that stream queries are cached and in the background it isn't synced. Because when I query the messages without stream as soon as the chat page is opened, it works fine.
Is it possible to force the stream implementation to "query" and not to use the cache somehow? Or is there another solution to trigger the stream to sync it?
Otherwise I would have to do an implementation that updates the messages in the chat page with the future query instead of the stream query and then starting the stream again.

Is it possible to force the stream implementation to "query" and not to use the cache somehow? Or is there another solution to trigger the stream to sync it?

You can use this on your database to update all streams manually:
db.notifyUpdates(
    {for (final table in db.allTables) TableUpdate.onTable(table)});

I think it is due to the fact that stream queries are cached and in the background it isn't synced

They are being cached, but that cache should invalidate after a write to the database. Are you using a separate database connection for the writes in the background?

Closing this since I can't act on this without further information. Let me know if you still run into this (and also share how the writes are happening so that I have a starting point to look into).


fix npm index to display package.json on package folder

We can change "package.json" in template easily if we decide to use another json file to present the content

Seems unittests failed.


R&D: Plum group access model + Sufia

Following a discussion with Mike Giarlo, he referred me to the group access model that Plum has built and noted that it might be a good fit for Sufia.
Please research how this is built and how it compares to our local assumptions about curator/edit groups and ownership.
If it sufficiently aligns, please consider the level of effort necessary to bring this support into Sufia. Feel free to contact Mike either via hydra-tech, hydra slack #dev channel, etc. if we get to this point.

The implementation for access control in Plum is role base, which has the following roles:
admin - with full permissions (default in hydra)
image_editor - with permissions to read, create, update works, and update files etc.
editor - with permission to read existing works, update public works, update files, and flag public and private works for review etc.
fulfiller - with permission to read existing works, update public works, and flag works for review etc.
curator - with permission to read existing works, update public works etc.
campus_patron - a user who logged in plum, who has permissions to read public and institutional visibility works.
anonymous - a public user, who has permissions to read public works only.
The workflow state gated discovery strategic is implemented basing on the default blacklight gated discovery, and the private works (own by personal) may not be exposed any roles other that admin.
I think the main different with the roles base access control in dams4 is the campus access, which is controlled by IPs but not depends on whether a user is logged in or not, and all objects (including those private ). Since workflow will be implemented as a standard component in hydra (see ticket https://github.com/ucsdlib/dams5-cc-pilot/issues/48), I'll create a PR to implement the roles base access control for the Sufia base pilot project horton, which has the following roles for now:
admin - with full permissions.
editor - with permissions to read, create, update, destroy objects and collections (public, ucsd, private).
curator - with permissions to read objects and collections (public, ucsd, private).
campus - with permissions to read public and ucsd objects and collections.
anonymous - read public objects and collections only.
Gated discovery strategic will be implemented to override the default blacklight gated discovery to expose all objects and collections (public, ucsd, private) to admin, editor, and curator roles.

I've created a PR https://github.com/ucsdlib/horton/pull/7 for the implementation, which is ready for review now. Once we add the admin role to the last user by running command rake add-admin-role, we can utilize the hydra-role-management gem to assign role to curators (http://localhost:3000/roles).


Preview image in the full size of Room component when clicking messages with image files.

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:

I think after adding the preview, you can add functions such as the previous picture and the next picture, and even enhance the interface, such as zooming in and out, rotating left and right, just like elementui.https://element.eleme.cn/#/zh- CN/component/image#da-tu-yu-lan Of course, if you are inconvenient, I can also help after the authors merge

Thanks for the PR!
However, the issue is that developers are loosing the flexibility to implement their own implementation when clicking on the image. We should probably add a prop to disable this behaviour, something like media-modal-preview: true/false

I had added  prop to disable this behavior and change some variable name for more readable. I am not sure if I need to resolve the conflicts before I commit again.

I think after adding the preview, you can add functions such as the previous picture and the next picture, and even enhance the interface, such as zooming in and out, rotating left and right, just like elementui.https://element.eleme.cn/#/zh-CN/component/image#da-tu-yu-lan Of course, if you are inconvenient, I can also help after the authors merge

This feature sounds great and I think it is not complex. I will see if I have time for doing this after author merge.

Yes please resolve the conflicts otherwise I won’t be able to merge this PR

I tested the branch locally, it works fine and the idea is good, but the UI is not very beautiful. I'll improve it a bit before we can merge this.


Does inpainting model support 768*768 or bigger images?

When I use the inpainting model, available on hugging face, I get the following error with the input size of 768 by 768. However, everything works fine with 512 by 512 images.  Does inpainting model support 768*768 or bigger images? Can I use stable diffusion v2 inpainting model with arbitrary input size?
latent_model_input = torch.cat([latent_model_input, mask, masked_image_latents], dim=1)
RuntimeError: Sizes of tensors must match except in dimension 1. Expected size 64 but got size 96 for tensor number 2 in the list.

To whom it may concern,
I found two parameters called height and width control the size of the mask and the noise tensor. When we feed an image whose size is not 512x512 to the model, the default value of height and width (512) causes an error. By setting these two parameters manually, the inpainting model supports arbitrary input sizes.
If you are using diffusers, you should predict the inpainted area using the following code:
img = Image.open(r"test.jpg").resize((768, 512))
msk = Image.open(r'mask.jpg').resize((768, 512))
.
.
.
images = pipe(prompt=[prompt] * batch, image=img, mask_image=msk, height=512, width=768
                          # ,num_inference_steps=steps, guidance_scale=guidance_scale
                          ).images

Hello,
I am new to this, I hope you won't mind me asking but where are the parameters for width and height that you found? as in what folder and file do I need to edit to add this code so that I can run 512 and 768 models at the same time in my installation?
Currently my installation is on 768 models so I can run 2.1 stability diffusion model just fine as well as any model on 768 format. However, every time I try to run something in 512 I get the "mismatch" error :(
Thank you!

Hello,
These parameters (height and width) must be a multiple of 64. In this case, I have used (768, 512), but you can use any other sizes.
I found this parameters in a file called "pipeline_stable_diffusion_inpaint.py". For me, this file is located in site-packages\diffusers\pipelines\stable_diffusion. In lines 633 and 634, height and width are defined.

useful! mark


[1.18.2] Transmutation Interface crashes when used with RS

Greetings!
As the title says, placing the transmutation interface, with an external storage from RS pointing at it, crashes the game with the following crash report:
https://pastebin.com/hwWFtFfC
Thank you very much for your time and for porting the mod to 1.18.2!

This problem has already been fixed and is going to be available in next update.

Fixed in 1.0.1 of 1.18.2
1.18.2-1.0.1


Needs updated to Reflect new User Experience

Needs updated to Reflect new User Experience. For example, where you get the Embed URL has moved under the ... in the top bar and is no long Black

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

ID: 0fc5abbd-e082-d1e1-109d-6f6cec8854a1
Version Independent ID: f8affce3-98f2-0cc6-b4de-d9af7ced0fbe
Content: Embed with report web part in SharePoint Online - Power BI
Content Source: powerbi-docs/service-embed-report-spo.md
Service: powerbi
Sub-service: powerbi-service
GitHub Login: @rkarlin
Microsoft Alias: rkarlin

Hi, @pcgeekus -- updated it now. Change will go live this afternoon. Thanks!

Thanks,
Jon
On Wed, Apr 29, 2020 at 1:33 PM Maggie Sparkman<EMAIL_ADDRESS>wrote:

Hi, @pcgeekus https://github.com/pcgeekus -- updated it now. Change
will go live this afternoon. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/powerbi-docs/issues/1668#issuecomment-621387176,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALG3YRKBLYBNU75FFGJPB7DRPBXGPANCNFSM4JWAKGXQ
.


OpenMP locks instead of busy-waiting with NUM_PARALLEL

Brief description: This pull request proposes replacing MAX_PARALLEL_NUMBER/NUM_PARALLEL feature with OpenMP locking mechanism to achieve a more refined implementation in line with OpenMP specifications and consistent with Pthreads and Win32 designs.
In the current codebase, the exec_blas function in OpenMP employs busy-waiting, utilizing the max_parallel_number feature to enable the execution of multiple instances of OpenBLAS in parallel.  With our changes:

We introduce OpenMP locks during the initialization of the queue. We have removed existing lock mechanism and have used omp_lock instead of in-house spin lock. This design is consistent with the existing Pthreads and Win32 threading backend designs.

This adjustment ensures that concurrent calls to BLAS functions do not utilize the same BLAS thread buffers.

OpenMP locks(currently using spin locks) have an expected scope for improvement in the future. Any performance gain will contribute to the new implementation without requiring manual changes.

Kindly note:

We have eliminated the NUM_PARALLEL flag and its associated code. NUM_PARALLEL was an add-on to the existing OPENMP_NUM_THREADS and requires recompilation of OPENBLAS for usage.

There is no impact to performance with default setting of NUM_PARALLEL=1. We are using the same logic but now in a more refined manner aligned with OpenMP specification with scope for future improvement.

For further details and discussions on this issue, please refer to: https://github.com/OpenMathLib/OpenBLAS/issues/4418

Hi @martin-frbg. Can you review the PR and give your insights.

Thank you, see what you mean now (and why my own experiments did not look promising). Two quick comments - I would prefer not to have an assert in the code,if it is at all possible to fall back to some safe value instead - by its very nature, OpenBLAS may be somewhere deep in the library stack of a program, and blowing up a calculation on a "bad" BLAS call when the end user may not even be aware that they use BLAS is not a good idea. And I think I would like to preserve the NUM_PARALLEL at least for now - unless you are sure that it won't be needed to address the (fairly unusual) case of multiple independent OpenMP runtimes calling into OpenBLAS (as supported by legate/legion, issue #2146)

Thank you for your comments @martin-frbg

I agree with you on assert removal. I will remove it on next commit.

There are 2 ways to handle multiple independent OpenMP runtimes calling into OpenBLAS:

Multiple buffers (Existing) : We can have multiple buffers and no lock on gemm_driver, So whenever we get multiple independent OpenBLAS calls, they will each have their own buffers (Limited by the NUM_PARALLEL) and rest of the calls will busy wait in the while loop inside exec_blas.
Single Buffer with locks (Proposed in PR and currently in use by Pthread and Win32 Backends) : The idea is to have a lock in the initial phase of OpenBLAS call inside the gemm_driver function. Now, when multiple independent OpenBLAS calls comes only one of them is granted a lock and others will sleep or busy wait (According to the inner implementation of locking mechanism).
How the test was performed :

I had a test.cpp file which called OpenBLAS inside a omp parallel region.
I had enabled nested threading in the test file.
I disabled omp_in_parallel flag in common_thread.h to not force execute single threading path if omp parallism is detected.

So, Having a single buffer with lock is fine as only one OpenBLAS call is being served at a time. It was showing consistent results during my testing on multiple OpenBLAS execution in parallel.

Thank you - I agree that having a single set of buffers and locking out any competing caller will work, I am only wondering if better performance can be achieved (in the future) by keeping NUM_PARALLEL as well and having as many buffers for parallel callers. The only drawback is that there probably needs to be more duplication of queue control structures so that a caller can not end up reusing idle threads configured for some other parallel process. (This is a problem for both pthreads and OpenMP I think, likely win32 threads as well - at this point I only suggest to avoid throwing out the NUM_PARALLEL option and its associated code,  when we may need it again somewhat later)

Hi @martin-frbg
I want your views on some points before moving forward with the modification
(Future performance gain with NUM_PARALLEL) : I want to know your thoughts on how to utilize NUM_PARALLEL to get better OpenBLAS efficiency and  your about your future plans on NUM_PARALLEL. I also didn't wanted to remove a existing feature but I failed to find any reason of future improvement using it.

Currently, Pthreads and Win32 works by allowing only a single BLAS call at a time. Is there any reason for it ?(Hardware related or Cache or any other).
Are you going to have NUM_PARALLEL type feature in Pthreads and Win32 backend as well?

(How to Proceed forward) : There are 2 ways we can have NUM_PARALLEL with OpenMP locks

OpenMP locks existing with NUM_PARALLEL : We can have different lock for each NUM_PARALLEL region.
OpenMP locks as a specific case : We can have OpenMP locks flow when NUM_PARALLEL = 1 since it behaves exactly the same as existing design and when NUM_PARALLEL != 1, We can have existing flow and disable the locks.

Plesae let me know which design is inligned with your future plans on NUM_PARALLEL ?

Sorry for the delay - some other things going on in real life - what I'm thinking about is OpenMP locks coexisting with NUM_PARALLEL (for some small number of NUM_PARALLEL, like 2 or 3) if that works out to be practical.
I think the limitation to one caller is purely historical - most of the "threading infrastructure" part of OpenBLAS is still the same as the original GotoBLAS from close to twenty years ago, written from scratch by a single developer and appropriate for the hardware of the time. (It wasn't until about six years ago that it became apparent that the pthreads build was not thread-safe at all even for a single caller.)

Yes, same number of locks (and probably just forbid NUM_PARALLEL greater than 2 or 3) - but I'm not yet sure how much duplication is needed elsewhere, it could be that it gets too expensive in terms of memory quickly.
Probably depends on the size of the hardware, some big NUMA server may be better suited for this but I'm just exploring... YIELDING is asm("nop;nop)"  on some platforms already which should make it behave better. The only reason it is still sched_yield or equivalent on others is that it seemed good enough to stay whenever someone looked into it.
No idea yet for replacing spin locks, looked into it but ran out of time, and maybe I am simply not good enough at OpenMP coding. On the other hand, I get the impression that the default OMP_WAIT_POLICY is implementation dependent and some platforms could gain from expressly setting it to "PASSIVE"

Hi @martin-frbg
As discussed,I have made some changes.
OpenBLAS is now controlling parallel execution inside OpenMP with a single lock. I have tried to explain the locks logic in comments.
I have tested it and it seems to be working fine. Please review the changes and let me know if any modification is required.

Thank you - unfortunately it looks as if I am not getting much of anything done today, so I've only had a quick look so far

Hi @martin-frbg
Any comments/suggestions on the PR?

sorry I'm currently sick, hope to get to this later in the week

Oh, Take your time. I hope you get well soon

Hi @martin-frbg
Any comments/suggestions on the PR?

Looks good (as far as I can tell today), thank you very much. I intend to merge it tomorrow.


Should we have a ThrottledFunc that works without a dispatcher?

I found that I need something like ThrottledFunc, but works in DxRenderer. There's something that's such a pain point, is that we're wasting valuable CPU times redrawing cursors when we actually don't need to . I know this long ago, just hit this again when investigating #10362 . This is an example WPR trace:

To update the cursor, the underlying buffer is frequently checked for IsCursorDoubleWidth, which consumes valuable CPU times and totally not necessary.
A simple solution would be remember the timestamp of the last update and refuse to update it until a certain period of time passes.

CC @lhecker for thoughts

@skyline75489 I wrote a til::throttled_func without a dispatcher just yesterday. hahahahaha
I can contribute it separately today if you want to. 👍


Unable to deploy on ARM server

I use the docker command line provided by this project on my ARM server, and the system prompts "Error response from daemon: error gathering device information while adding custom device "/dev/kvm": no such file or directory."
Then I used the "cpu-checker" command to query and it showed "FATAL: Module msr not found in directory /lib/modules/6.2.0-39-generic". So is it because my server does not support KVM?

May I know where to add KVM=N?

version: "3"
services:
  windows:
    image: dockurr/windows-arm
    container_name: windows
    devices:
      - /dev/kvm
    cap_add:
      - NET_ADMIN
    ports:
      - 8006:8006
      - 3389:3389/tcp
      - 3389:3389/udp
    environment:
      VERSION: "win11"
      RAM_SIZE: "8G"
      CPU_CORES: "4"
      DISK_SIZE: "160G"

    stop_grace_period: 2m
    restart: on-failure

Sorry about that…

My server is ORACLE ARM.

Can I "install" KVM to it?

@bywenshu man, were you able to use windows on oracle ampere vm with KVM disabled?! And how was the performance?, was it usable?


update example code to function component

#1579

Looks like another PR was merged first—but thank you for your work all the same! Please check #1579 for another component that needs love!


Warn if max ports less than open file limit

also attempt to guess the path to sysctl on linux if it is not already in the path.

:+1: a82d67e705aee592d025a7b111da8185f55a14a8

This seems erroneous.  What is the problem being solved?  Leveldb wants 112,000 or larger for nofiles.  That has nothing to do with ports.

@matthewvon in Erlang, ports represent any communication that leaves the vm(port driver) which includes opening files and network communication. So, the open_files limit should always at least be equal to max_ports and vice versa.  More detail on this can be found in the developers guide: http://erlang.org/doc/reference_manual/ports.html and http://erlang.org/doc/man/erl.html#max_ports

@joecaswell this should have been merged a year ago, apologies. Will get it manually tested again today to make sure nothing has changed and merge it in.

Still not convinced.  What user problem lead to this PR?

I'm sorry for the confusion, this has been made much simpler in newer version of the Erlang runtime as there is no longer a max_ports or max_open_file setting. They were combined for clarity into a single +Q setting which is the same as max_ports(which includes open file handles as well). However, since we havent moved to these newer erlangs yet, this change will warn users if they have a config which could lead to a system_limit crash because their settings do not make sense. The max_ports configuration includes both open files as well as open network ports so having max_ports set lower than the open file limit will effectively lower max_open_files to the value of max_ports.
This change will warn users on startup and once a day(by default) if max_ports is less than open_files.

@joecaswell tested on OSX and it successfully warned when I had a high ulimit with a manually set low max_port.  Going to fire up an ubuntu vm and quick test there as well.

Success, all unit and integration tests pass on riak_kv_env. Merged.


where is uv_read_start?

uv_read_start give a way to alloc buffer by user, but it seemed there is no way to do this with evt_tls.

evt-tls is an abstraction on top of OpenSSL's BIO pair(currently) which can be used with any async library including libuv.
The idea is NIO is handled by library like libuv and fed the received data for processing to evt-tls and which calls us back when processing is done.
So, It us upto the user to use his nice allocation skills with libuv. evt does not handle that.
We have sample integration with libuv but that does not to be used as it is. It is just for demo.
You can combine it any way you like.
Closing this


7/06 Publish for wictor and rubens

Docs Build status updates of commit 1448ada:
:warning: Validation status: warnings

File
Status
Preview URL
Details

rd/index.yml
:warning:Warning

Details

rd/map.yml
:warning:Warning

Details

rd/media/profiles/rubens-guimaraes.jpg
:white_check_mark:Succeeded

rd/media/profiles/wictor-wilen.jpg
:white_check_mark:Succeeded

rd/rubens-guimaraes.yml
:white_check_mark:Succeeded

rd/toc.yml
:white_check_mark:Succeeded

rd/wictor-wilen.yml
:white_check_mark:Succeeded

rd/index.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/about).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/map).

rd/map.yml

Line 0: [Warning-InvalidFileLink] Invalid file link:(~/about).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/CodeOfConduct).
Line 0: [Warning-InvalidFileLink] Invalid file link:(~/map).

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


Implicitc (space separated) Matchers

This allows for matching the other common format of matcher, the implicit equality style. Examples:
zstd 1.4.5 h9ceee32_0
python 2.7.*
_libgcc_mutex 0.1 main
backports_abc 0.5 py27h7b3c97b_0

Fully implemented now. Here's the benchmark changes:
Package name only       time:   [107.26 ns 108.28 ns 109.57 ns]
                        change: [-36.825% -36.254% -35.665%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

Package name and version
                        time:   [315.00 ns 316.84 ns 318.78 ns]
                        change: [+23.127% +24.110% +25.081%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 3 outliers among 100 measurements (3.00%)
  3 (3.00%) high mild

All possible matchers   time:   [1.0792 µs 1.0842 µs 1.0896 µs]
                        change: [+0.8051% +1.6085% +2.3674%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 1 outliers among 100 measurements (1.00%)
  1 (1.00%) high severe

Repodata depends        time:   [44.103 ms 44.351 ms 44.655 ms]
                        change: [+20.626% +22.620% +24.368%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 3 outliers among 100 measurements (3.00%)
  2 (2.00%) high mild
  1 (1.00%) high severe

In the simplest path (package_name_only) it goes through the faster implicit_parser first, so its faster. However, because it must first fail in parsing the implicit style before switching back to the full_matchspec_parser it is a fair bit slower.


Button style properties are restored to default after opening the project

ButtonStyleDisabled, ButtonStyleHot, ButtonStyleNormal, ButtonStylePressed, ButtonStyleSelected.
If you change these properties and then open the project, these properties will return to their default values.

I'm working to resolve this problem...

Fixed in 3.2.0 version


Set daemon attribute instead of using setDaemon method that was deprecated in Python 3.10

The setDaemon attribute has been deprecated in python 3.10
Due to this people using the latest python versions will get an annoying deprecation warning error.
And yes I did copy the PR title from https://github.com/gitpython-developers/GitPython/pull/1214 lol

Thanks!


fix(md-3779): column sort indicators android

Updated the sort indicator to be a top/bottom border instead of arrows.

:tada: This PR is included in version 1.1.0-alpha.57 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Disable SELinux labeling for container

This prevents issues when mounting the docker socket to the container.

/cc @BarDweller

/cc @iocanel

The pack CLI already does this:
https://github.com/buildpacks/pack/issues/1208
https://github.com/buildpacks/pack/pull/1307


added support to duckdb via ADBC

In my role as Data Engineer I've been moving out from dataframes (aka spark) as I'm building everything with DBT and BigQuery and locally with DuckDB (CLI). Using just sql to build different stages (kind of delta lake), has been a great way to facilitate the work of data scientists and for data governance.
I can see how DuckDB could work as the Ingestion and transformation layer to expose a simplified and maybe materialized "data marts" to explorer or vegalite. This is so powerful!
Amazing work with ADBC !

Thank you @matreyes! I have some questions to understand your use case better:

Why not use Explorer/DataFrames for the data manipulation?
Do you actually have a Duck DB in disk?
How are you currently importing data into Duck DB to work with it?

Answering your questions

Why not use Explorer/DataFrames for the data manipulation?
I currently work for a big corp with lot of tools and languages, the only common language between eng, scientists and analyst is sql. Also persistence (views or tables) could be useful and I can reuse DuckDB transformations in our DW (BigQuery) later on with few changes.

Do you actually have a Duck DB in disk?
I use both (memory and disk). In disk is great for having a kind of local "datamart"; in memory resembles more a typical dataframe workflow.
Also "Running on a persistent database allows spilling to disk, thus facilitating larger-than-memory workloads (i.e., out-of-core-processing)."

How are you currently importing data into Duck DB to work with it?
I usually import csv or parquet files directly to duckdb

Thank you! This looks good to me. If you are happy with it, we can merge it.

To be clear, let me know if it is ready by saying yes or no :)

Thanks @jonatanklosko ,
It's OK from my side.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Improve docstrings

There are two main issues with the docstrings as they are:

[ ] They repeat themselves a lot. There should be one docstring per input per module, and all other functions should reference the "main" docstring. See this line on tears.py on pyfolio for an example.

[ ] There aren't any examples. As good as verbose, detailed explanations of inputs are, an example would help illustrate a lot. See this line on tears.py on pyfolio for an example.

Regarding your second point, I just wanted to highlight that the input to the tears.* funtions is built by utils.get_clean_factor_and_forward_returns. So utils.get_clean_factor_and_forward_returns documentation should be taken into consideration too when dealing with tears.* funtions one

Added all data examples in utils.get_clean_factor_and_forward_returns and added a comment in each tears.* function " See full explanation in utils.get_clean_factor_and_forward_returns"


[SUGGESTION] Support for paste JSON/XML as classes

Originally filed at https://github.com/microsoft/vscode-dotnettools/issues/1038
Following discussion with @arkalyanms and @webreidi at MVP Summit, one feature I feel is missing from C# Dev Kit in VS Code is the "paste JSON/XML as classes" feature from Visual Studio.
This allows you to copy a JSON or XML object, and via the "Paste Special" menu paste it as a C# class with the relevant types created / matched (See here)
It is a hugely helpful feature when creating integrations with APIs: copy sample from docs -> paste into Visual Studio -> C# class representing the object(s) automatically created.
The Visual Studio implementation is certainly not perfect, e.g. I wish it would keep properties pascal case, or automatically apply System.Text.Json serialization attributes where relevant, but it definitely speeds things up!
In VS Code I imagine this could appear in the Edit menu, or as an option on right-click when within a .cs file.

As a workaround there is an extension in the marketplace for pasting JSON as code at https://marketplace.visualstudio.com/items?itemName=quicktype.quicktype


is it possible to read from a specific offset?

Hi,
I wanted to know if rdkafka able to read from a specific offset given to consumer.
-Niral

See here:
https://github.com/ah-/rdkafka-dotnet/issues/22


Wrong 'Authorization' header for the 'Basic' authentication scheme

Add 'Basic ' in https://github.com/cloudspout/cloudspout-button-panel/blob/743a01b174358bce41d39302ebc1dfae06f272e8/src/ButtonPanel.tsx#L52
Solution:
requestHeaders.set('Authorization', 'Basic ' + btoa(options.username + ':' + options.password));

https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Authorization

Thanks for reporting. Fixed in 6ba38a0


⚠ TorrentLeech has degraded performance

In 03cca38, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 1542 ms

Resolved: TorrentLeech performance has improved in e2f3013 after 8 minutes.


Fix resource backslash issue at source rather than during JSONify

This is an improved version of previously merged PR https://github.com/spring-projects-experimental/spring-native/pull/521 which was merged in https://github.com/spring-projects-experimental/spring-native/commit/cef5c60ad3fb8fe4bfdfc899513615234a3e49e0
The original fix just fixed the Windows backslash paths to forward-slashes during the final JSONify step in ResourcesJsonMarshaller. I went back and had a look at where the original issue stems from in BootstrapCodeGenerator and fixed it there instead (which was another use of Files.walk()).
Again, tested on Windows but would be good to have a check on a Linux system.
@aclement, I have not touched a pre-existing String.replace("\\/","/")  in ResourcesJsonMarshaller which appears to be from an earlier version of the code as I'm not sure what this is trying to do - if you have any recollection possibly this could be moved as well?
I did also consider making a change in ResourcesDescriptor.add() method but this seemed like a better approach.

Noticed that the existing check for startsWith("META-INF/native-image") was always broken on Windows as the pattern would never have a forward-slash - moved the String.replace() call above the if statement to resolve.

I have rebased this on top of my other PR https://github.com/spring-projects-experimental/spring-native/pull/546 for now given the other one is not awaiting additional review. This should not be merged until PR https://github.com/spring-projects-experimental/spring-native/pull/546 has been to avoid conflicts.

@aclement Feel free to merge this one if you think that's relevant.

Seems to behave on linux, thanks for the contribution!

@leccelecce Could you please sign Spring Contributor License Agreement if not already done?


Styling the project and solving the "N + 1" issues

In this milestone and partners and I ensured the following:

Styled the whole project to look similar to the wireframe provided in the requirements.
Solved the "N + 1" issues to enhance our code performance.

@Goruchie Thanks for reviewing the project. I have fixed the errors