  This can be confusing, because many editing functions such as adding Series or Related Items are actually done directly in the Django admin.
In principle, Django CMS provides overlay functionality that makes this change unnecessary (see http://docs.django-cms.org/en/release-3.4.x/how_to/toolbar.html).  However, in the experience of Boston Lindy Hop we found this to be buggy, and so we changed all of the sideframe/overlay links to be direct links to admin pages.
Getting the toolbar on the admin pages should just involve adding the appropriate template tag to an overridden version of the admin base template.  However, there is the potential issue for repeated loading of toolbars if an admin page is loaded via an overlay or sideframe.  Since overlays and sideframes are just iframes, getting the Django CMS toolbar on admin pages should be achievable by just overriding the admin base template and adding a piece of Javascript that loads the toolbar only if the page is not in an iframe.
Also, this change will require the overridden base admin to be loaded before templates from django.contrib.admin, and so we will need an app, similar to the djangocms_admin_style app, which is listed in INSTALLED_APPS before django.contrib.admin.

Actually, looking at this a bit more, it's simpler than I anticipated.  Django already handles the popup/iframe cases, so all that we need to do is override the template admin/inc/branding.html that is provided by djangocms-admin-style to include all of the CMS toolbar links, not just "Change password" and "Logout" links.

OK, actually, we will also still need to override admin/base_site.html to ensure that django-sekizai blocks are rendered in the appropriate place in admin.  However, I've confirmed that getting the CMS toolbar to show up is straightforward by overriding templates/admin/base_site.html and templates/admin/inc/branding.html.
I'm not sure why this isn't default behavior for the djangocms-admin-style package (I think it just may be that configurable toolbars are relatively new in Django CMS), so I've posted an issue to that repo to see if they have plans to just go ahead and implement this improvement within that package.  If they end up doing so, then I may not make any changes to this repo.  If they aren't going to change that repo anytime soon, then I'll figure out how best to implement this change in a way that doesn't complicate project setup too much.

Now that INSTALLED_APPS is reordered by default, this issue has been solved by including the modified templates in danceschool.core.


Prevent UI from making requests too often

Describe the bug
It's possible (probably, in fact) that the UI is making more requests to the API than needed, and only taking notice of the latest request to return. This would cause unnecessary strain on the API, and if the cache isn't quite working properly, also the DB, which is more of an issue. This may or may not be the reason for the EC2 instance locking up and falling over periodically when multiple users are loading/actively using the UI simultaneously.
To Reproduce
Steps to reproduce the behavior:

Open the Quartz UI
Open browser DevTools > Network tab
Watch the requests being made by the UI as you click on charts/change view etc.
See many requests, and possibly then no requests returning successfully if the API falls over.

Expected behavior
Should only be requesting new data periodically, perhaps at the 5 minute auto-update interval that the app runs internally, or at least not more than once every couple of minutes.
Additional context
N/A

this now done?


How do I use this in the browser?

the demo tries to get browser.js from a nonexistent dist folder

Sorry @benzmuircroft I didn't see your message till today, I'll add an example of how to build for the browser.


Add support for default decimal places for users commonly performing math related to currency such as those in accounting roles

Problem Statement
Accounting users that utilize traditional desktop calculators often want to default to having 2 digits after a decimal because they are constantly entering currency values.
Evidence or User Insights
Having to enter a period to denote cents adds a great deal of additional data entry throughout the course of a day and is one of the primary barriers to switching from a printer with a paper spool to a computerized calculator.
Proposal
Add a setting to configure the starting position when inputting data. The default would be as it currently works, but setting it to -2 would place it in the hundredths, -3 would be thousandths and 3 would be thousands, etc.
Alternate solution: Add an accounting mode option just as there is a scientific mode option. This option would be designed for users working with currency values regularly.
Goals
Increase user adoption of the calculator by meeting needs of accountants and office workers that deal with currencies regularly.
Low-Fidelity Concept
If the user enters "1" it would display as "0.01"
If the user enters "1" followed by "2" it would display as "0.12"
If the user entered "1 followed by "2" followed by "3" it would display as "1.23"

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

We thought this idea was interesting, and we have all used input similar to this (particularly on banking websites), but we are not aware of any electronic financial calculators that handle input in this way. Ultimately, we felt that this behavior, even as an option, might cause confusion for the broad userbase of Calculator. We would be happy to revisit this if you could help provide any additional evidence of the user need this feature would address.

Thanks Dave, there is a huge userbase for having an "adding machine" mode in calculator. Just as scientific calculators are different from standard calculators, so are adding machines, and they deserve their own special mode in Calculator.
Adding machines are still a hugely popular purchase in accounting departments worldwide and is one of the few holdovers from the 90s that I continually see on people's desks.  Every physical adding machine offers some functions that are unique from a standard or scientific calculator, such as the use of postfix notation, but when I surveyed accountants and administration employees on "why di you still use a physical calculator" the must have feature was consistently the function to lock decimals using a physical switch.
This feature is universal on all adding machines across all brands, from the Royal EZVue 550HD to the Victor 1460-4, to the Casio HR-100TM and Sharp EL-1801V.
Other software calculators do offer this mode and many such as deskcalc default to this mode out of the box, with the number of decimals available for selection in the lower left corner and defaulted to 2, as this is the most standard for currency related financial calculations.
A few other examples are:

Mark's Adding Machine
Superb Calc
Schoettler CalcTape

The venerable Hammacher Schlemmer  even tries to bridge the gap between the need for electronic record keeping and the need for simple adding machine input by featuring a "transfer to pc" physical adding machine that dumps the adding machine output as HID output via wireless usb receiver!
I do not believe this would cause confusion if the default is 'standard' because users are not confused by 'scientific' mode, and the decimals can appear initially on the screen in adding machine mode just as they do on a physical adding machine calculator or in an application such as deskcalc.

Thank you for following up with these details! Reopening for reconsideration.

I would definitely use this feature as I am trying to get rid of my desk calculator.

@datapharmer After reviewing the pitch again, I think everyone felt that this would be a useful setting to have, but also that it might be confusing to have this setting exist globally across all modes.

Alternate solution: Add an accounting mode option just as there is a scientific mode option. This option would be designed for users working with currency values regularly.

The idea of adding a "finance calculator" has come up in the past, which sounds similar to the accounting mode you suggested as an alternative solution here. What are your thoughts on closing out this pitch but folding the idea into a new pitch to add a finance calculator mode (with this as a setting in that mode)? If so, would you like to take a first pass at submitting that pitch? I imagine there are others in the community who would be interesting in more finance/accounting focused features that could help develop the idea further.

(with this as a setting in that mode)

Do you add the concept of per-mode settings in general?

Do you add the concept of per-mode settings in general?

I think this is an opportunity to explore how best to expose per-mode settings, but we kind of already have settings in some modes. For example, you can toggle scientific notation in Scientific Mode:

And in Programmer Mode to specify word length:

They are exposed inconsistently, so I am not suggesting we follow those examples as a pattern moving forward, but rather noting that we already have per-mode settings today. I think there might be an opportunity to leverage the operator panels from #428 to expose input and display options:


Emmet: Support css.color.short setting

"emmet.preferences": {
"css.color.short": true
}
Why not working?

@sergeche Does the new modular emmet support this feature?

@ramya-rao-a yes, it should be passed as shortHex option: https://github.com/emmetio/stylesheet-formatters/blob/master/format/css.js#L7

Looks like it is enabled by default. Since VS Code doesnt change that particular property, we will need to debug why this doesnt work.

I'm having the same problem, using the latest VS Code version: 1.35.0.
After adding the setting "css.color.short": true as mentioned above, Emmet in VS Code still expands #1 to #111111 instead of #111.
Hope this could get some love, because small color values are just way more sexy!

@agragregra @Friksel I updated the Emmet dependency, and now css.color.short is enabled by default in the latest Insiders.
I still have to add the setting to VS Code if one wants to disable it for whatever reason, so I'll leave this issue open.

@agragregra @Friksel I updated the Emmet dependency, and now css.color.short is enabled by default in the latest Insiders.
I still have to add the setting to VS Code if one wants to disable it for whatever reason, so I'll leave this issue open.


feature(instant-buy): Support BiAddress system (Xverse)

What this PR does / why we need it:
Xverse uses two address: taproot & nested-segwit: one is for payments, and the other is for inscriptions.
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] No console errors
[ ] Unit tests*
[ ] Added e2e tests*

Thanks for creating the PR, Jordan. The changes looks specific to XVerse and from my understanding doesn't fit the standard arguments that ensures everything is wallet agnostic.
I think the approach to have a destinationAddress as the arguments option would be a better idea as we use it in other functions.


Feature Request: Tumblr: Parse HTML directly for Notifications instead of RSS

On tumblr I got the official statement from Dev team now that you cannot change the RSS rendering. To add webmention publish links, I will have to type them in manually into the Rich Text HTML textarea every time. If Bridgy could check the actual HTML representation for changes / updates I could easily add webmention publish URL.

:+1:
specifically, when superfeedr tells us there's a new post, we'd fetch its full HTML and extract links from there, instead of from the content that superfeedr gives us.

Decided to just add this as a NOTE or Reminder, since its driven by the same Use Case:
"	... I had thought implementation wise, since you already have code behind those webmention url grabbing the post as HTML, you could use that and modify it to a 'webmention detect' URL, so when bridgy gets something from Tumblr you then first throw it against that and find the actual webmention URL in the HTML"
snarfed replied:
"right now tumblr webmentions and publish are separate features, they don't trigger each other like that. i could definitely add that though [...] definitely related! this one sounded like triggering publish specifically. 590 sounds like fetching HTML to get all URLs, including outside of content."


src: add new metrics APIs

src: add new metrics APIs

The following metrics are now always recorded and available via the new
uv_metrics_info() API.

* loop_count: Number of event loop iterations.
* events: Total number of events processed by the event handler.
* events_waiting: Total number of events waiting in the event queue when
  the event provider request was made.

Benchmarking has shown no noticeable impact recording these metrics.

This is a patch we are currently using internally at NodeSource. We have been testing more reliable ways to measure the health of the event loop and have found that tracking the number of events processed, and more importantly, the number of events waiting in the event queue to be processed before the event provider is called is a better measure of event loop health.
Allowing a callback to be called upon completion of the event loop iteration, we can measure how long the individual iteration took. Using this we can calculate the statistical average amount of time each event waited. Not only events that were received by the event handler, but also how long each event waited in the event queue.
As an example, here is a graph that plots number of requests (req/sec), total number of events (poll_events), number of event loop iterations that had events waiting to be handled (loop_w_e), and the total number of events waiting (events_waiting):

This curvature is similar to almost every HTTP server workload. Using this data we can determine the optimal time to scale the server without knowing the actual processing time the event loop has taken.
In addition to this data we use an exponential moving average using the Poisson window function combined with a time constant adjustment so even though the calculations are done once every loop iteration, the smoothing curve acts as if it was time-series data. The equation for this is:
                       /-ΔT \
                       |----|
                       \ τ  /
s   =  s      + (1 - e       ) * (x  - s     )
 t      (t-1)                      t    (t-1)

ΔT - the difference in time since the last calculation in seconds
τ  - the time constant for the calculation in seconds
x  - new value to add to the exponential rolling avg

Using the moving average data can be reliably stored in a time series database and used for analysis of the health of an application over more than one process.
Hopefully, these advantages outweigh the burden of adding additional metrics APIs.

@cjihrig Thanks for the review. Fixed all comments.

I'm converting this to a draft and rethinking the implementation to make things simpler. Will ping when the update is complete.

@cjihrig @santigimeno I've redone the implementation and simplified the API along with adding more tests. PTAL.

@santigimeno Everything's taken care of.

Looks like several errors are from the build failing to find stdatomic.h.

@cjihrig Any feedback on whether I should leave it a uint64_t* or switch to a void*?

@cjihrig if that's the case then I'll leave it as is.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/2260/

CI failures unrelated: https://github.com/libuv/libuv/issues/3714

uint64_t* reserved[13];

So the structure has three uint64_t and then 13 pointers?  That feels really weird on 32-bit platforms.  Did you mean to use this?
uint64_t reserved[13];

Or is it too late and the 32-bit ABI set in stone now?  (though the struct length will be a PITA to keep when "unreserving" in future)

It's released so it's set in stone now, yes.

Then I suggest at least a comment in the struct to warn about unreserving, as that won't be easy to do while keeping both 32-bit and 64-bit struct lengths.

 I can't believe I made it a pointer. Sorry.

You were also discussing it during review but noone seems to have noticed 🤷🏽

though the struct length will be a PITA to keep when "unreserving" in future

Actually, using union is possible to simplify that at least.  I even now saw this somewhere in libuv.  (though it won't warn you about getting over)

This is ugly, but here's an idea on how to handle this to make it easier to use the reserved space in the future:
struct uv_metrics_s {
  uint64_t loop_count;
  uint64_t events;
  uint64_t events_waiting;
  /* private */
  uint64_t reserved[6];

#if UINTPTR_MAX == 0xffffffff
  uint64_t* dead[1];
#elif UINTPTR_MAX == 0xffffffffffffffff
  uint64_t* dead[7];
#else
#error "Platform not supported"
#endif
};

What ideas did you have in mind for using a union?

I meant e.g. https://github.com/libuv/libuv/commit/b6bf6d2d1f4c4ec263ee9064060e983ac3ee555f


Proposal: Support Projects in "Kickstarter" Fashion

How about hacking a kickstarter-like tool, allowing us to support each-others projects with RHOC?
RHOC will be exchangeable for utility on the coming RChain Platform. The Coop can support RChain Projects by distributing RHOC to members, who in turn send those RHOC to the projects they support. Many of these projects will rely on RHOC to fuel the bandwidth, compute, and storage needs of the apps they are developing.
Jake Gillberg demonstrated a web3/metamask Chrome Extension that enables us to use RHOC to vote on GitHub issues. Could the same hack be done to a project management platform, enabling RHOC to be transferred?

At the moment it seems like most holders of RHOCs are hodlers (myself included). This doesn't bode well for liquidity in the market. The Co-op is also not distributing RHOCs. A few thousand have gone out as bounties but – if you 'do the math' – after a few months it'll still be a few thousand. Being hodled.
"...distributing RHOC to members..." with the intention that they put them into play is a great idea.

I would be ready to trade a bit more.
Get the RHOC's rolling. When we pay out more RHOC to activists, there will
be more people interested in liquidating those RHOC's.
IMHO the first community driven pay-outs of 100,000 RHOC will be just the
start. Let's spend it quickly. Show the results to the board and ask for
more.
Jake's system looks to me a great tool. It's excellent for cleaning up the
issues in GitHub. And I think that GitHub is good enough for our project
management.
--
Cheers,
HJ
On Thu, Aug 17, 2017 at 7:13 PM, kitblake<EMAIL_ADDRESS>wrote:

At the moment it seems like most holders of RHOCs are hodlers (myself
included). This doesn't bode well for liquidity in the market. The Co-op is
also not distributing RHOCs. A few thousand have gone out as bounties but –
if you 'do the math' – after a few months it'll still be a few thousand.
Being hodled.
"...distributing RHOC to members..." with the intention that they put them
into play is a great idea.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rchain/Members/issues/83#issuecomment-323136477, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB0x96Qv3rDQYEore_lt3Cfq4hjwCfLcks5sZHShgaJpZM4O4fEp
.

Just added Jeroen van Dijk in the spreadsheet. Will be looking for more


Galaxy on Fire 2 - Black Screen and Unresponsive Game with RTX Remix

Issue:
When attempting to play Galaxy on Fire 2 on Windows 11 with RTX Remix, the Laucher works but after that the game starts with a black screen and becomes unresponsive.The game needs to be manually closed. I have tested this issue with both versions of Remix.
System Specifications:
GPU: NVIDIA GeForce RTX 3090
Driver Version: 531.61
OS: Windows 11
Game version: Steam

Steps to reproduce:
Install RTX Remix by dragging and dropping the files into the game folder
Launch Galaxy on Fire 2
Observe black screen and unresponsiveness

Expected behavior:
The game should start normally, displaying the main menu or an intro sequence, and RTX Remix should be compatible with the game.
Logs:
NvRemixBridge.log
GOF2_d3d9.log
d3d9.log

i have a bit of progress:
i have updated to the newest version of rtx-remix (remix-0.2.0) but the injecktion still faild. but it was a shader issue. i used this conf file to set the shader model to 1 or 0 ( dxvk.txt ) then the games started and i could see the rtx logos and menues.

but sadly the screen was black. i could hear the game music and with shortcouts i could start the first mission and i could here the sound from that mission.
GOF2_d3d9 (1).log
here is the log file . i notice that the USD Plugins failed to load (warn:  [GameCapturer] LSS USD Plugins failed to load.) when i try to make a usd snapshot nothing happens not even a log file gets created. also i have some formart issues i am not sure but i think its a rare image formart that is not detected or something but could also be wrong (warn:  ConvertFormat: Unknown format encountered: D3D9Format::AL16
warn:  ConvertFormat: Unknown format encountered: D3D9Format::R16) and it also says that no camea is found and raytracing has faild (RTX-Compatibility-Info] Trying to raytrace but not detecting a valid camera.) i have tried to enable freecam in a settings menue but this did not help the screen was always black. lastly something strange. i have disabled antialiasing in the game settings but i still get this error (warn:  D3D9DeviceEx::SetRenderState: Unhandled render state D3DRS_MULTISAMPLEANTIALIAS) . i have seen lots of other people have simular issues on the discord manly the no camera found issue but also the usd plugins failing to load. i think the main proplem is that these games use shaders. but maby its easier to work out the shader issues with gof2 becasue the shaders are hlsl shaders that are in plain text in the game files and they even have debug info and comments. so we could get the camera possision from one of the shaders somehow, for a bit of testing i have modified the bloom shader to invert collers insted. and it worked.

thats it thanks for the cool tools :)

Hello! Will you please test this on the latest release and let us know if the issue persists? Thank you!


not able to export to sepal workspace

Exporting a mosaic (tested with optical mosaic) to sepal workspace gives an error below
{
"id": "4a1ac8f9-214d-47e5-8527-f3dece54f64b",
"recipeId": null,
"name": "Retrieve Bugoma_NDVI_dif-19-23 to SEPAL workspace",
"status": "FAILED",
"statusDescription": "{"defaultMessage":"Failed to execute task: ","messageKey":"tasks.status.failed","messageArgs":{"error":"ServerException: Failed to set band names."}}"
}
Thank you!

Hi Teopista is it an old recipe or an old one ?

New one!

then @cdanielw do you have access to the log directly from here ?

I believe you still actually got the image downloaded, but it failed to set some metadata on the image. So it's a GDAL related bug I'll have to check.


[1-Click-Confetti] Add confetti with sound command

Description
Add raycast confetti command, so you can set hotkey to run confetti with sound. It's so inspirational.
I want to make it more often with hotkey.
Also small refactoring to make the changes look appropriate.

Can we just update the Changelog please? Thanks!

@peduarte Changelog updated

@peduarte What now needs to be done to ensure that the changes get into the main branch?

@velios thanks for your contribution!


Cutscene freezes game

I am at week 2, and after the opening cutscene, my game freezes.

prob cuz you have low storage or smth idk


Use xss_scanner.py crashed

Problem Description
Use xss_scanner.py crashed
Steps to reproduce the behavior:

Use the file xss_scanner.py in example-comple
Command mitmproxy -s xss_scanner.py
Use the proxy and open some websites
crashed!

Please lodge a bug report at: https://github.com/mitmproxy/mitmproxy
mitmproxy has crashed!
Traceback (most recent call last):
  File "/Users/yuechen/.pyenv/versions/3.7.3/envs/mitm-3.7.3/lib/python3.7/site-packages/mitmproxy/proxy/server.py", line 121, in handle
    root_layer()
  File "/Users/yuechen/.pyenv/versions/3.7.3/envs/mitm-3.7.3/lib/python3.7/site-packages/mitmproxy/proxy/modes/http_proxy.py", line 9, in __call__
    layer()
TypeError: 'NoneType' object is not callable

Please lodge a bug report at: https://github.com/mitmproxy/mitmproxy

System Information
Paste the output of "mitmproxy --version" here.
Mitmproxy: 5.0.1
Python:    3.7.3
OpenSSL:   OpenSSL 1.1.0j  20 Nov 2018
Platform:  Darwin-18.7.0-x86_64-i386-64bit

refs #4031


[Windows Arm64] Update tool to target Arm64 on Arm64 hosts

Background
See: https://flutter.dev/go/windows-arm64
Once https://github.com/flutter/flutter/issues/129805 lands, Arm64 hosts will be able to build x64 executables. These executables run using x64 emulation.
Solution

Add Windows Arm64 to the HostPlatform and TargetPlatform enums
Update the Flutter Windows plugin templates to show how to use the target architecture to bundle pre-built libraries
If on Arm64 host and the "beta" or "stable" release channels, produce a warning that an x64 executable will be built.
If on Arm64 host and the "master" release channel,

Use Abi.current() to determine the target architecture
Download Windows Arm64 artifacts

Update Windows Arm64 devicelab tests: build, run, benchmark

One this work lands, we should clean up the packages repo tooling: https://github.com/flutter/packages/pull/4826/files#diff-b525444e626a654538177afff3418a5da77038d43a58e3b52118523f87e9b7bfR316-R339

@pbo-linaro is the contributor working on adding Windows Arm64 support.
@dcharkes It looks like the native assets support added by https://github.com/flutter/flutter/pull/134203 works already on Windows Arm64 machines thanks to x64 emulation. FYI we're still working on running tests on Windows Arm64 machines, your new tests won't run on Arm64 machines yet.
@pbo-linaro, Daco is working on adding "native assets", which is a new way to support native code in pub packages. From my understanding, this will be an alternative to Flutter's existing support for native plugins. The package author will be able to create a build.dart file that uses the user's installed compilers to compile the native assets. In https://github.com/flutter/flutter/pull/134203, Daco allowed Windows build to use x64 MSVC to compile C++ code. To enable targeting Arm64, we will need to update the logic added by In https://github.com/flutter/flutter/pull/134203 to use the correct MSVC toolchain depending on the host/target.
(Daco feel free to correct any details I may have gotten wrong!)

Hi, I'm currently working on windows-arm64 support and managed to get native (x64, arm64) and cross compilation (x64 -> arm64 and arm64 -> x64) of flutter apps to work on my local branch.
One issue I'm investigating now is support for native_assets, where it fails because cl compiler can't be identified for arm64 architecture. After digging into this, I reached this native_toolchain_c/lib/src/native_toolchain/msvc.dart, and found that support need to be done here.
I don't think flutter project will accept a windows-arm64 enablement without native assets, so it would be nice if we could fix this together @dcharkes. Once native repository has correct support, I should be able to do the rest on flutter side.
@loic-sharma: Do you see native assets as something mandatory for first enablement?
@dcharkes: Do you think anything more beyond toolchain detection has to be done to support new windows-arm64 platform?
Thanks,
Pierrick

Hi, I'm currently working on windows-arm64 support and managed to get native (x64, arm64) and cross compilation (x64 -> arm64 and arm64 -> x64) of flutter apps to work on my local branch.

🎉

One issue I'm investigating now is support for native_assets, where it fails because cl compiler can't be identified for arm64 architecture. After digging into this, I reached this native_toolchain_c/lib/src/native_toolchain/msvc.dart, and found that support need to be done here.

Yep, that sounds correct!

I don't think flutter project will accept a windows-arm64 enablement without native assets, so it would be nice if we could fix this together @dcharkes. Once native repository has correct support, I should be able to do the rest on flutter side.

Yes, please feel free to create PRs to the dart-lang/native repo to do toolchain detection. ❤
I don't have access to an arm64 win machine myself. Neither do the GitHub actions have support for Windows arm64 as far as I know.
You should be able to have your local copy of the dart-lang/native packages and use a path dependency in the pubspec.yaml of flutter_tools to test your changes directly from flutter tools.

@dcharkes: Do you think anything more beyond toolchain detection has to be done to support new windows-arm64 platform?

I'm not aware of anything. The Abi.windowsArm64 is already in dart:ffi, and @rmacnak-google has been fixing corner cases recently. (There are some remaining corner cases: https://github.com/dart-lang/sdk/issues/53829. If you run into a crash when doing an FFI call please give us a ping. But these are relatively uncommon cases: e.g. structs by value with specific struct sizes. This shouldn't stop you with implementing support for native assets for Windows arm64, as most tests just pass integer values.)

Thanks for your reactive answer!
The advice about using a path in pubspec.yaml is what I was missing to try this locally, that should help.
I'll propose a PR to add support in native then.


Project Logo / Project Icon

We need a Logo / Icon for the project.
If possible as .SVG (Scalable Vector Graphics).

I don't think we need an icon for a project that's about to be abandoned.


Example for not-common frameworks

Hello,
I am working with a custom web framework and I would like to ask if there is an example to bind it (e.g. with the raw embedded webserver in python.

There isn't! If you're writing a custom framework, I suspect you'll be at home reading the source. Start with the Resource class and work out from there.


lxd operation show not very useful

Required information
ubuntu bionic
  driver: lxc
  driver_version: 3.0.3
  kernel: Linux
  kernel_architecture: x86_64
  kernel_version: 4.18.0-25-generic
  server: lxd
  server_pid: 4137
  server_version: 3.0.3
  storage: zfs
  storage_version: 0.8.1-3~18.04.york0
  server_clustered: false

Issue description
lxc operation show is not very useful
LXD status codes created_at updated_at identical even after several hours. May be better if the updated_at is recent.
Steps to reproduce

make a copy some huge containers (containing AOSP code)
wait for it to run
After few hours still the updated_at is so old.
Would be nice if status or  updated_at is refreshed regularly (say every few minutes)

Information to attach

~ $ lxc operation  show dffggfd18-278a-4f19-b3e4-11ece89bd208
id: d9d6fd18-278a-4f19-b3e4-11ece89bd208
class: task
description: Creating container
created_at: 2020-03-08T10:25:06.712379023+01:00
updated_at: 2020-03-08T10:25:06.712379023+01:00
status: Running
status_code: 103
resources:
  containers:
  - /1.0/containers/xenial-quack-aosp
  - /1.0/containers/xenial-los-rename
metadata: {}
may_cancel: false
err: ""

~ $ date
Sun Mar  8 13:22:19 CET 2020

That would just needlessly require running a go routine and burning cpu without actually saying anything about whether something is happening.
Operations where progress information is available will show that under metadata.


Rouote::resource - parents ?

Is there any way to add parent to resource route ?
I try second argument like:
Route::resource('users', UserController::class)
->breadcrumbs([
'index' => 'Users',
'create' => 'New User',
'show' => fn(User $user) => $user->name,
'edit' => 'Edit',
], 'dashhboard');
but it is not working ...

You'll need to use the callback syntax to do this. Try this:
Route::resource('users', UserController::class)
  ->breadcrumbs(
    fn($breadcrumbs) => $breadcrumbs
      ->index('Users', 'dashboard')
      ->create('New User')
      ->show(fn(User $user) => $user->name)
      ->edit('Edit')
  );

This will set the dashboard route as the parent of users.index but leave all the other implicit parents set (eg. users.index is the parent of users.show). If you want all the routes to be children of dashboard, just set the second parameter to 'dashboard' in each method call.


nukie planet update

About the PR
the airlocks now use nukeops access
chem goggles in chemistry
an RCD in powergen with two cartidges
two incendiary shotgun shells at the firing range
fills the backpack water tank with 100u clf3 and 900u napalm
two teargas grenades where the EMP and flashbangs are
the drop pod now has is now made out of plastitanium and has LOS blocking doors
power monitoring console in gravgen
Why / Balance

the fire weapons are good because it requires nukies to give up spacing to use them and incendiary ammo is otherwise unobtainable (perhaps a firestarter bundle with incendiary ammo could be added, but thats off topic)
the tear gas grenades are non lethal and easily counterable with internals
engis RCD the nuke into a corner so nukies RCD it out
nukies RCD the nuke into the corner so engis RCD it out because they have the exact location at a map
is this powercreep?
no, its catching up. every engi has an RCD and every officer has teargas
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:JoeHammad

tweak: The nukie planet has been slightly updated

i dont really want this stuff mapped, firestarter bundle would be fine to add to uplink

i dont really want this stuff mapped, firestarter bundle would be fine to add to uplink

why not test it out like this first? if it turns out to be awful in practice it can be removed
also, should i add the RCD and water tank updates to the CL?

joe we need to talk about your power creep

joe we need to talk about your power creep

the solution isnt to elevate the other side, thats literally power creep

well it doesn't look like the roundstart RCD for engis is going away, and it looks like its going to get better, too #15708

yea thats gonna nerf them

im putting the fireaxe on infiltrator anyway

im putting the fireaxe on infiltrator anyway

if thats a joke, based
if it isnt, based

also im closing your infiltrator pr ill put a turret in through

Imo kill the incendiary shells, they are stupidly overpowered, and then this'll be fine-ish, it's powercreep yet the items will probably be underused because its just neat gimmick stuff.

over..

I guess you do have the fire axe which can space floors and break into windows about as fast but trades the construction feature for high damage
maybe we could have some kindof item that forces you to decide between the axe or the RCD for the team? but id rather not plan around hypotheticals like that

Joe you literally got banned for making powercreep prs despite over a dozen warnings and then you start going back to big powercreep prs, just make new content instead of making everything needlessly stronger.

I dont think this is powercreep. the station gets more tools (notably the RCD) and this aims to bring nukies up the same amount. the important part of powercreep is things being left underpowered
remember what syndrome said? if everybody's special, nobody's special

im not a game designer by the way

Game design is about choice, giving people literally every tool they might possibly need for any situation, also completely bypassing TC in the process, is not choice.

fair point

nukies get enough c4 already

anyway, times up. final verdict, emisse, what do i need to remove?

well i removed the RCD
thinking about it, having this EVERY NUKIE GAME would just be bad

ive removed the backpack water tank fill but im keeping the two incendiary shells

changes made

i dont think sometimes :trollface:

anyway i go to bed

by the way i'm going to go on a holiday tomorrow so i'm gonna give this PR 30 minutes and after that ill remove anything that'll prevent this from getting merged (and then go to sleep because it's 12:30 right now)

i still got some time actually

remove incendiary stuff it should be its own uplink bundle

i removed the stuff from the tank, are the two shells okay?

no

zamn

removed

alright goodbye for a week


[Chore] ⚙Ignore URI's that are not informative

Some class and properties URI's are not informative when comparing two mappings.
For example:
Properties such ashttp://www.w3.org/2000/01/rdf-schema#label, https://schema.org/name, http://www.w3.org/2004/02/skos/core#prefLabel should be present in all mappings and thus are not interesting for mapping comparison.
Class such as https://schema.org/Thing is a valid type for every resources in the semantic web thus are not interesting for mapping comparison.
A solution to implement this is to create a lists
IGNORED_PROPERTIES = ['http://www.w3.org/2000/01/rdf-schema#label', ...]
IGNORED_CLASSES = ['https://schema.org/Thing']

and use them during comparison:
if property not in IGNORED_PROPERTIES:
...

Solved in commit 17c36d998be7e6a6283b5d09e83a8d4f76d775a1


Vault cross account role permission denied with 0.2.26

@bonifaido  We are seeing the following error in configurer pod only when using 0.2.26 and not anything below.
msg="error configuring vault: error configuring aws auth roles for vault: error putting vault-crossaccount-role-platformlive aws role into vault: Error making API request.\n\nURL: PUT https://vault.secrets:8200/v1/auth/aws/role/vault-crossaccount-role-platformlive\nCode: 400. Errors:\n\n* unable to resolve ARN "arn:aws:iam::************:role/VaultCrossAccountRole-PlatformInfrastructure" to internal ID: AccessDenied: Access denied\n\tstatus code: 403, request id: dbd4ecf0-c65e-11e8-ad97-d3c20a6b5ad9"
#140 was submitted to support cross account access.

It looks to me that the credentials that Vault was given are not allowed to access the role API in your AWS account.

@bonifaido Found the issue related to our EKS IAM role. Thanks !

Great! Thanks :-)


summary_tool_code.js needs cleaning up

From hacking it to work standalone, there are still commented code sections that aren't needed and missing .fail() for various XHRs.

Fixed by #4.


Numba can't properly match ListType of Arrays in function signature

Let's start right away with a reproducer, because i'm not sure of how to properly describe this kind of behavior. Let's say I want a function that takes a list of arrays as an input (because these arrays might have varying lengths):
@nb.njit(nb.float32[:](nb.types.ListType(nb.float32[:])))
def f(list_of_arrays: List[np.ndarray]):
    return list_of_arrays[0]

r = np.random.random(10).astype(np.float32)
f(nb.typed.List([r, r, r]))

If we actually display the function's signature before running, we get
(ListType[array(float32, 1d, A)],) -> array(float32, 1d, A)

So far, so good. However, this code raises a TypeError: No matching definition for argument type(s) ListType[array(float32, 1d, C)]. Yet,  you can obviously see that the argument types listed in the error are a subtype of the function's argument signature (the array(float32, 1d, A) type should match both array(float32, 1d, C) and array(float32, 1d, F) array types).
To try and have this working again, the "trick" is to force the layout of the array inside of the ListType to contiguous (C):
@nb.njit(nb.float32[:](nb.types.ListType(nb.float32[::1]))) # <--- array is now of type array(float32, 1d, C)
def f(list_of_arrays: List[np.ndarray]):
    return list_of_arrays[0]

r = np.random.random(10).astype(np.float32)
f(nb.typed.List([r, r, r]))

This works fine, but this is an unexpected (and very confusing) behavior.
I'm using numba 0.50.1 on Ubuntu 18.04, in a regular python 3.6 venv.

Thanks for the report, I can reproduce, your reasoning follows what I would expect from my memory of there being no cast on construction for non-scalar types. CC @esc, think this ought to be looked at as a use case? Thanks.

We could include a cast, but I don't think casting from C to A is supported?

Typing layer is ok with C->A as this is generalising, whereas A->C is not permitted.
In [13]: from numba import types

In [14]: c2d = types.Array(types.float64, 2, 'C')

In [15]: a2d = types.Array(types.float64, 2, 'A')

In [16]: from numba.core import typing

In [17]: ctx = typing.context.Context()

In [18]: c2d.can_convert_to(ctx, a2d)
Out[18]: <Conversion.safe: 3>

In [19]: a2d.can_convert_to(ctx, c2d)

In [20]:


a full AirPlay and AirPlay 2 support?

It is possible to have a full AirPlay and AirPlay 2 support?

Possible? Yes.
With this code? No.
This code only works for extremely old devices, and perhaps not even after they got software updates. Also, this code only pushed a few frames per second and I never fixed audio synchronization, so even during it's time it was not super useful.
I gave up on this because of how awful the documentation and reverse engineering their protocols was, and I doubt it's gotten any easier since then.

You can look here: https://github.com/mikebrady/shairport-sync/issues/535 ;)

Seems like there are plenty of other people who are more willing to work on that problem than I am, I hopefully won't ever have to deal with proprietary Apple protocols again.
Seems like the discussion there has long eclipsed the scope and interest of this project, so I'll just archive this repo.
Feel free to fork and continue, but if I were to ever attempt something like this again I'd prefer to start fresh and leave this code written by my high school self behind.


chore(storybook): fix deploys

Closes #28
The latest deploy to storybook failed because it couldn't find the tailwind css file.
https://github.com/PradaIng/structuraid-desktop/actions/runs/4071179705/jobs/7012740451
Compiling the tailwind styles before doing the storybook build should fix this issue

no @ImMPrada. Right now the npm run storybook command is concurrently running watch:css and the storybook compilation
I think that on the actual build we do need to make sure that tailwind compiles the css first before doing the compilation. So those 2 commands are running one after the other, not concurrently.


Resource editing disabled error message [staging <IP_ADDRESS>]

I'm still having this error pop up and it's happening more frequently especially if switching from one map to another.
I'm just switching between maps nothing odd or weird.  In this case I was trying to load sci_lowerhospital.
I've been working w/ Hab_Airport and Tech_Hub though. So I have seen it with those maps too.
I'm on staging <IP_ADDRESS>  since it's release last week.

Thanks for separating into a new issue - this is indeed different to the previously fixed #91.
I'll take a look!

Thank you.  I'm not sure if it's actually doing what it says about editing disabled.
I've been purposely exiting and restarting the script editor each time I see the error.
Just to be certain it's not borking anything.

Yeah it should all work fine without it, you just won't be able to edit any resources connected to entities like models/materials.

Turned out to be a 32-bit memory issue. I'm now favouring 64-bit (and using some new parsers), so this should be fixed!


Modrinth/CurseForge release page

Having a public release for the mod outside of this GitHub page would increase visibility for the project and allow users to automatically download it using tools like https://github.com/Senth/minecraft-mod-manager, among other things

The mod is currently quite unpolished and I don't feel it's ready for a release on CurseForge, but I'll release the next update on Modrinth.


chore(main): release 0.6.0

:robot: I have created a release beep boop
0.6.0 (2024-08-29)
Features

basic logging support (#26) (6ebbbbc)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.6.0
:sunflower:


Add additional info to stepup error pages with new error theming

This issue is imported from pivotal - Originaly created at Aug 12, 2019 by bstrooband
There needs to be additional information on the error page. This will be addressed later on due to changes on the error pages in 5.11.
The information is the error response from GW.
Also if user does not have a token at the required level. Tell them to register a token. Provide link to page (https://support.surfconext.nl/stepup-noauthncontext) that can be customized with advice e.g. the right link to the self-service portal and add translation.
See: https://www.pivotaltracker.com/n/projects/1425900/stories/166729843 to validate all additional info requirements are met.

@thijskh ,The status code and status message feedback information labels might need an abbreviation and could I place the link directly under the text?

Do you have any suggestions for the abbreviations? (bstrooband - Aug 22, 2019)

I have no suggestion to abbreviate the labels without making it too cryptic.
You can add the link directly under the text for now yes. If this turns out not to be OK we can easily change it later.
Note that there are a Status Code (in this case Reponder), a Substatus code (in this case Example) and a Status Message.
    <samlp:Status>
        <samlp:StatusCode Value="urn:oasis:names:tc:SAML:2.0:status:Responder">
            <samlp:StatusCode Value="urn:oasis:names:tc:SAML:2.0:status:Example" />
        </samlp:StatusCode>
        <samlp:StatusMessage>Something went wrong</samlp:StatusMessage>
    </samlp:Status>
``` (Thijs Kinkhorst - Aug 23, 2019)

Update with representative invalid status:
I've used an already available/used format:
StatusCode/Subcode (bstrooband - Aug 23, 2019)

It's true that the supplied url does not exists at the moment?
(bstrooband - Aug 23, 2019)

@bstrooband Yes, it will be created. "registratieproces" as one word please. (Thijs Kinkhorst - Aug 23, 2019)

FYI: (bstrooband - Aug 23, 2019)


The competition between quic-bbr and cubic

As we all know, when tcp-bbr flows coexist with a large number of tcp-cubic flows, there will be a catastrophic speed problem. So,I'd like to ask, have you tried the competition between quic-bbr and cubic? Is quic-bbr better than tcp-bbr in term of competition with cubic?   In other words, what are the differences between quic-bbr and tcp-bbr, and whether they will be upgraded to bbr-v2?

I'd expect same unfaireness when quic-bbr and cubic coexist. And I have seen that in very simplified test of 1 quic-bbr stream and 1 cubic stream in a simulation.
I don't know when we will start to work on bbr v2 in mvfst. It's in our plan but right now we have other priorities.
If you send a Pull Request of BBRv2, we are more than happy to review it.


New method to build debian package using of this repo

The way to build the debian package of this node is to run:
bash build.sh /deb/output/dir
The ROS dependencies are handled by underlay.repos file. This allows the developer to updated dependencies but this is something that we don't want in CI so a flag which indicates CI build would be helpful.

This is something I missed; why wouldn't we want to use the same dependencies in CI?

We could us the same dependencies in CI. The scenario that I am thinking is when the developer uses a development branch of a dependency which is not yet in master. Would this be a problem in CI when for example creating a release?
While writing my response I realized that maybe this is something that we want to have :). That way we can test in CI work which is still in different branches. =)


win-19 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
win-19 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

win-19 is out of sync with Salt master, rebooting to restart network service.


Restful EPP

Software name
Restful API for registry-registrar integration
Software webpage
https://internetee.github.io/repp-apidoc
Software description
Restful alternative to EPP
Editing organization
Estonian Internet Foundation
Software license
No response
Category name
API
New category

[X] Yes, it's a new category
[ ] No, it's an existing category

Motivation
Got a suggestion to propose .ee Rest-EPP implementation for the list. Not sure if this qualifies as its API descritpion not a piece of software on its own, but I'll leave it for you to consider.
Request checklist

[x] I have checked for other similar issues and pull requests.

[x] I have replaced "Software name" by the actual name of the software in the title

[X] I have filled all mandatory fields

Thanks for your suggestion, Timo! My preference would be to limit the scope of this list to tools / software.
Your API description is great, but if we were to include it, we would actually have to include other DNS related specifications, standards and RFCs as well. I'm afraid the list will then become too long and confusing.
But since this is a communication effort, I am open to other opinions.

No worries! As I mentioned in the motivation bit, I do agree with your opinion, but as the suggestion came from the community I did my part and created the ticket. The fact that you like the API description made it worth the effort :) You may close the ticket if there is nothing else to discuss here.


Appstudio update test-component-pac-irpw

Pipelines as Code configuration proposal

Pipelines as Code CI/test-component-pac-irpw-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
13 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
17 seconds
sanity-inspect-image

✅ Succeeded
17 seconds
deprecated-base-image-check

✅ Succeeded
45 seconds
clamav-scan

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
16 seconds
sbom-json-check

✅ Succeeded
11 seconds
sanity-optional-label-check

✅ Succeeded
11 seconds
sanity-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/test-component-pac-irpw-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
18 seconds
clone-repository

✅ Succeeded
32 seconds
build-container

✅ Succeeded
8 seconds
clair-scan

✅ Succeeded
12 seconds
sanity-inspect-image

✅ Succeeded
14 seconds
deprecated-base-image-check

✅ Succeeded
30 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
6 seconds
sanity-optional-label-check

✅ Succeeded
6 seconds
sanity-label-check

✅ Succeeded
6 seconds
show-summary


Udpate map viewer to the latest version.

Update map viewer to 1.3.1
.
It includes the following changes:

General Improvement: hover to highlight features
General Improvement: Display connectivity info without obstructing current map view
ScaffoldVuer: Improve primitives controls and annotation mode

The new version can be tested on this site.

Three things.

Show connectivity info in the sidebar, which is false by default. We may need to add :connectivityInfoSidebar="true" to use it. But if it will not be used at the moment. Ignore this one.

The sidebar tag close sign style seems to be affected. Doesn't look good to me. But depends on you.

A small style issue in its sub-component popup. It does not affect any functionality. But we can increase the height .gallery-popper > height: 4rem; in style to fix.

allery-popper > height: 4rem;

Thanks.

Three things.

Show connectivity info in the sidebar, which is false by default. We may need to add :connectivityInfoSidebar="true" to use it. But if it will not be used at the moment. Ignore this one.
The sidebar tag close sign style seems to be affected. Doesn't look good to me. But depends on you.

* A small style issue in its sub-component popup. It does not affect any functionality. But we can increase the height `.gallery-popper` > `height: 4rem;` in style to fix.

Thanks @ddjnw1yu, I'll update the connectivity close button issue in sidebar.


fixing broken URL in translation

Can you remove the span? We should try to avoid hard-code styling in the translation files. Also, we're about to merge in the rails4 branch, so you might want to make this change there.


[simple-romp] Webcam device number selection

Problem
https://github.com/Arthur151/ROMP/blob/ff5ea02b2fb524ba55631fc398f722af04e053e4/simple_romp/romp/main.py#L197
The webcam source might not be necessarily number '0'.
Especially if virtual webcam drivers are used.
Recommended solution
Add command line and configuration option to change the device number.

Vivian, thanks for your constructive comments. This function has been added in the last commit.

@Arthur151 Please publish this feature as an release at PyPi. 🙂


Images with alt text shows up twice

See https://google.github.io/comprehensive-rust/pt-BR/, which currently looks like this:

The underlying HTML looks weird, with the images being duplicated:

Build workflow

GitHub contributors

GitHub stars

I tested this kind of Markdown with the current main branch and it's fixed there. The problem above seems to be the nested tags: the image inside the link causes us to extract things twice.

This was fixed with the 0.2.0 release. Please upgrade and run mdbook-i18n-normalize on your PO files.


[RFC] [BC Break] [3.x] Prototype - merge/detach removal

This PR simply attempts to get rid of EntityManager#merge() and EntityManager#detach() operations.
While merge and detach can be useful in contexts where entities are being serialized:

serializing entities is something we suggest NOT doing
the semantics really don't fit PHP
the merge operations are quite aggressive and cause a lot of issues
cascade merges not configured correctly = hell
EntityManager#clear() is a much better API for long-running and multi-threaded processes
the initial use-case around EntityManager#merge() was to merge data without further queries (when avoidable): the Second Level Cache now solves that problem in a more elegant way.

As discussed with @beberlei, this path is viable, but a few things must be provided instead:

merge and detach operations are actually operations that are part of a serializer: we need to move the interface there
this obviously breaks the authentication system of symfony, and @nicolas-grekas is probably needed for feedback here
generally, we won't support serializing entities in future, which means that alternate approaches need to appear

I somehow agree we should get rid of merge, but disagree of detach.
We still manage detached entities internally and there's no way to avoid it.
As I've asked on IRC, we should wait a few weeks (beginning 2016) before actually focus our work on 3.0.

@guilhermeblanco the idea is to move detach to something similar to clear instead.
As per discussion with @beberlei yesterday, detach can lead to horribly broken object graphs, and should lead to cascaded detach in most cases with bi-directional associations.
We need something safer and more specific.

I find myself thinking of it as:

"Detach-All", which is basically clear(). Useful, easy, a no-brainer.
"Detach-One", which requires finding all the collaborating entities/collections and replacing references to the target with a lazy-loading proxy. This requires accurate metadata and badly-written entities can throw a monkey-wrench into the whole process if they maintain a reference somewhere that Doctrine doesn't know about.
"Detach-Subgraph", where you cascade and affect all referenced entities. Same difficulties as above for the most part. Better, in that you don't need to handle "replacing" references in still-managed entities. Worse, in that effects are less-predictable.

We need something safer and more specific.

If the main driver is (de)-serialization, what about leaving the managed-object graph totally unchanged, and instead doing cloning/state-overwriting? For example:
$plainEntity = $entityManager->cloneUnmanaged($managedEntity);
$str = $mySerializer->save($plainEntity);

And later:
$plainEntity = $mySerializer->load($str);
$entityManager->overwriteDataFrom($managedEntity, $plainEntity);

"Detach-Subgraph", where you cascade and affect all referenced entities.

Same thoughts here - we would need some sort of detach mechanism that safely applies detach, without breaking portions of the graph, but entirely cutting it out when there is any reference.

My feeling is that avoiding accidental data-bugs is a higher priority than saving memory/CPU-cycles in the PHP runtime. Big organizations can spin up additional PHP instances, and people with high-performance data-dumping needs can probably find other ways to get the data out.
If doctrine returns a copy of the subgraph for serialization, that's easier to control and has cleaner failure-scenarios. Forgot to configure some child entities as important? Then the data isn't present. Configured too many? Then you get a huge subgraph.

If doctrine returns a copy of the subgraph for serialization, that's easier to control and should have cleaner failure-scenarios.

Agree 100% - that's why we'd need a "serializer", rather than a detach()

Or just declare that (de)-serialization is a problem to be solved by some other library: All Doctrine has to do is:

Create subgraphs of Plain Old PHP Objects
Given subgraphs of POPOs, attempt to overwrite the managed objects' with the new state

Hi,
I'm currently using a lot of clear and merge operations in a long running operation. I had seen that merge operation is not as robust as persist operation (I encounter 2 bugs for which I made a PR).
So I try the import from external API some entities, in order to keep my data synchronized. I totally agree that doctrine doesn't fit for that type of operation and a raw SQL requests will be awfully much faster. But I have not enough time to manage this kind of operation, and as my script took less than 10 minutes make me happy.
In order to do this I need to manage my memory usage and the only way to do this I :

flush all entities
clear doctrine in order to garbage collect
merge the entities that I need to keep for the next steps

@moroine

merge the entities that I need to keep for the next steps

Since this operation would cause queries anyway: wouldn't it be simply safer to just re-fetch the objects?

@Ocramius Not really.
I made a bundle dedicated to import data from any source such as API, Excel ... The aim of my bundle is to add new source as much simple as possible ! In order to do this, I create a factory which is composed by one builder by entity that I can import. I tried to be flexible as much as possible, so I needed to handle of the complexity in my builders.
The developer need to provider a parser which provide sequentially (by a function next) a pattern which is a representation of my entity (better than a raw array). The problem is that a pattern could be a graph (a big one) so when I traverse my graph I need to flush / clear to garbage collect useless memory. So when I acsend back my graph I need to recover my cleared entities which data are partially imported (so different from database, included relations) so merge operation fit really fine for that case.

when I traverse my graph I need to flush / clear to garbage collect useless memory

@Ocramius I wonder if the identity-map could optionally use weak-references... I'm not sure if the extension exists for PHP7, but I really wish that they were built-in.

I'm not sure if the extension exists for PHP7, but I really wish that they were built-in.

That would open a completely different can of worms about how to keep transaction integrity.
Code like following would just break in a very unexpected way:
$em->transactional(function ($em) use ($userIds) {
    array_walk(
        $userIds,
        function ($userId) {
            $user = $em->find(User::class, $userId);

            $user->ban();
        }
    )
});

I suggest looking at #5550 instead (for this sort of operation)

I don't think serialization or partial detachment of subgraphs is really within the scope of what the ORM should do. I think within reason we can look at what needs to be possible with the ORM to enable those sorts of use cases, but IMO they're special cases that should be implemented in userland code.

@zeroedin-bill I definitely agree in terms of serialization and "detach-subgraph". Do you think "copy-subgraph" would be a reasonable replacement?
On the other side--with data flowing in--what about a "merge" like:
/*
 * Try to load/overwrite state represented by subgraph, throwing
 * exceptions if things don't line up exactly. Fails if any entities
 * in the subgraph do not have a @MergeMethod
 */
$entityManager->overwriteState($plainEntity);

And the ORM could require that any affected entities must designate a merging-method (e.g. @MergeMethod so that it is handled in user-space:
/*
 * @Entity
 */
class Foo{
    /** @Id @Column(type="integer") @GeneratedValue */
    protected $id = null;
    /** @Column(type="string") */
    protected $a = "hello";
    /** @Column(type="string") */
    protected $b = "world";
     /** @OneToMany(targetEntity="Bar", inversedBy="foo") */
    protected $bars;

    public function __construct(){
        $this->bars = new ArrayCollection();
    }

    /*
     * @MergeMethod
     */
    public function myMergeMethod(Foo $other){
        // Can we assume framework already checked that IDs match?
        $this->a = $other->a;
        $this->b = $other->b;
        // Not sure if there are any gotchas in terms of lazy-proxies here
        $this->bars = new ArrayCollection($other->bars->toArray());
    }
}

// Class Bar omitted

Alternate configuration mechanisms could be used for a static merge-method like Foo::Merge(Foo $managedRecipient,Foo $unmanagedSource)

If an entity object is kept immutable and changes for a data row has to be handled in a new object, can this change also be handled without a EntityManager#merge() method?

If an entity object is kept immutable and changes for a data row has to be handled in a new object, can this change also be handled without a EntityManager#merge() method?

Immutability is not handled by the ORM: only one instance of an entity per identifier is allowed in a single UnitOfWork, and its spl_object_hash() is tracked. This is the wrong tool for that job.

@guilhermeblanco I went over this again and rebased it on develop. It's mostly removals and I still have 3 TODOs, but I'd still like to have your opinion on the change.

@Ocramius I've done some digging in Symfony and I could not find any merge/detach in the security anymore. The user provider that loads the entities either does a find($identifier) or calls a refreshUser() on the repository.
https://github.com/symfony/symfony/blob/master/src/Symfony/Bridge/Doctrine/Security/User/EntityUserProvider.php#L70

That's awesome news!

@lcobucci another round when you can.
@guilhermeblanco this is ready, IMO. This does intentionally not cover EntityManager#clear() === new UnitOfWork() yet: that will happen in the future.


[stdlib] Revise and expand floating-point documentation

What's in this pull request?
Expanded documentation for FloatingPoint, BinaryFloatingPoint, and the conforming types.
Resolved bug number: n/a

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please smoke test

@stephentyrone Hows this look?

I reviewed these changes internally already; I'll go through this pull request as well as a sanity check, but based on what I've already seen this doesn't need to wait on my approval.  As far as I'm concerned, this should be merged now.

Ok. SGTM!

Thanks @stephentyrone and @gottesmm! I'll watch for comments on this PR and catch them in the next batch of revisions.

@natecook1000 The only thing that jumped out at me is that I believe the exponent is required for hexadecimal floating-point literals, but you describe it as being optional.  If you can spare a couple minutes to fix that up, it would be great.

@stephentyrone It's optional in string -> floating-point conversions, where it's assumed as 0 if missing.
(swift) Double("0x10.8")
// r0 : Double? = Optional(16.5)

I don't think I discussed hexadecimal numeric literals—do you see any specific places where I'm doing that? I should probably spell "hexadecimal" correctly, though. 😳

Hmmm.  That should probably be a stdlib bug, then; init(String) should use the same format as we do for literals unless there's a good reason not to, and I don't know of other languages that allow hex float without an exponent.  I'll look into this in the morning.


Changelog has not been updated in 9 months

Can we please start updating the changelog again? It's difficult to know what bug fixes have been made and/or features added.

@johnjiang You're absolutely right, you can see what has changed by scrolling through the release-tags but we'll also update the changelog.md soon :yum:

Done with https://github.com/Mozharovsky/CVCalendar/commit/8a10b090a247008a48cc878166fc6274bb5272fc


Improve the information page

Add more detailed descriptions and illustrations on the information page.

and link info page from home page

closing this issue because it was resolved in #117


Add ability to control where text overlay goes on product pages

Currently, the text overlay on the images' behavior is hard coded.
We need to be able to control if it goes left or right, as not all product pages follow the same logic (it's not always zig-zag).
I think we already have a column we could use for this in Excel, the Style column.
Example: See https://www.caesars.com/caesars-palace/shows/absinthe

Checking with Spencer first. I don't think we should complicate the authoring experience. They should just enter the content and not worry about whether the text will be left or right and it is up to the consuming app to decide that look and feel.

Checked with spencer. We want the authors to decide whether the text overlay would be on the left or right from the excel.


[SPARK-42712][PYTHON][DOC] Improve docstring of mapInPandas and mapInArrow

What changes were proposed in this pull request?
Improve docstring of mapInPandas and mapInArrow
Why are the changes needed?
For readability. We call out they are not scalar - the input and output of the function might be of different sizes.
Does this PR introduce any user-facing change?
No. Doc change only.
How was this patch tested?
Existing tests.

CC @HyukjinKwon @hvanhovell

Merged to master and branch-3.4.


Add support for Swift 2.3

This updates the code to work under both 2.2 and 2.3 while developers handle their transition period up to Swift 3. The project file is also updated to indicate the last migration, and the version of swift. The version instructs Xcode 8 which internal toolchain is to be used.

@ketzusaka any reason for the baseline change? Was Xcode giving a compiler error on this?

Indeed, that was the onl compile error.

@ketzusaka ok I'll check it out in a bit, the API diffs don't show any change to that so I'm curious as to what is erring because the change you've made isn't necessarily right.

Sounds good! This was what the Xcode migratory suggested; they may of gotten it wrong though :)

I believe this is correct – it seem like Baseline is no longer included in Xcode 8.

@nickkjordan this is tricky, because Baseline is supported down to iOS 6 where as LastBaseline is only available in iOS 8 yet we support iOS 7…


Example?

It would be useful to put a link to an example page in the README.

We'll rather build proper documentation, see #32 (and of course link it from the README).


QoL Setting: Set your Z Target to Hold by Default via the Generator

This is a very, very minor thing compared to everything else but both current randomizers allow the user to set their default Z target so they don't have to set it in-game. Most players choose to set it to Hold but providing it as an option to toggle (whether you want default to stay on Switch or the default to switch to Hold) is good as players often forget to set it in-game as well.
Edit: Basically a setting that allows you to set the default to either Switch or Hold, so just a little drop-down choice between Switch and Hold on the generator.

Celeste did this last week or so. Done. No issues.


Remove empty RuntimeUtility class

Create branch: issue-32-remove-empty-runtimeutility-class from main.


Fixed label for CSI matching nodeSelector in values.yaml

Currently, the documentation states that the command to label a CSI node as enabled is kubectl label node <nodename> cubefs-csi-node=enabled.
However, in the values.yml file, the nodeSelector is "component.cubefs.io/csi": "enabled"
This PR updates the README.md file to reflect the correct command.

LGTM, thanks for your contribution, the README is forgotten to update.


Issues with Mongoid

Hi,
I am getting problems using Mongoid and client_side_validations
i am using Mongoid 2.0.2
when enabling validations: true on my form
i am getting these errors
undefined method `entries' for BSON::ObjectId('4e085cbc816f461eee000005'):BSON::ObjectId
Extracted source (around line #16):
13:
14:
15:     		<%= f.label :title %>
16:     		<%= f.select :title, ["Mr", "Miss", "Mrs", "Dr", "Sir"] %>
17:

Hey I've switched to MySQL for my project was having too many issues with mongoid. I prefer to use the rails form helper rather than formats tic or simple form. Anyways with the default setup this gem works beautifully


Support 'changes' for updates when replica identity = full

#72  👈
main

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

Join @acco and the rest of your teammates on  Graphite


Include Gradle build system

To quote the Gradle documentation, it is "an open-source build automation tool for the Age of Continuous Delivery (CD)."  Gradle offers significant improvements over Maven in terms of simplicity, performance, and overall developer experience.
See https://hub.docker.com/r/niaquinto/gradle/ for an image focused specifically on Gradle.

I've opened https://github.com/cloudbees/jnlp-slave-with-java-build-tools-dockerfile/issues/13 but I think it's better suited here.


[libsndfile] Update to 1.2.2

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

This port has install Findxxx.cmake for its dependencies, which should be avoided. We need patch it to follow the usage provided by vcpkg.
See https://github.com/libsndfile/libsndfile/blob/2c6ffea6ac8bb1b89e1f778bdad8bf8b0342d32f/CMakeLists.txt#L853-L856

https://github.com/libsndfile/libsndfile/blob/2c6ffea6ac8bb1b89e1f778bdad8bf8b0342d32f/CMakeLists.txt#L821

vcpkg conflicts with upstream regarding dependency mp3lame.

No, upstream is broken, it lacks keyword TARGET at the failing line:
https://github.com/libsndfile/libsndfile/blob/2c6ffea6ac8bb1b89e1f778bdad8bf8b0342d32f/cmake/Findmp3lame.cmake#L61

@FrankXie05, ready.

@evpobr We need remove all the Findxxx.cmake in portfile.cmake before vcpkg_cmake_configure

@FrankXie05, ready.

libsndfile[*]:x64-linux doesn't build. Please test all features and CMake usage (with a symbol from the lib!) in x64-linux which is a static triplet.
I didn't look at the (implementation) details but I think the brief comments about avoiding the find modules were misleading. Available CMake configs usually aren't drop-in replacements. They have different names, provide targets instead of  variables, and using the targets during the build implies using find_dependency(...) in the exported config. It at least unlikely that non of the removed Find modules needs more action in the patch.

@dg0yt , what do you mean?
https://github.com/microsoft/vcpkg/pull/34388/checks?check_run_id=17729086236

It builds in CI, with default features. This is necessary, but is it sufficient?

Feature regtest failed before in 1.2.0, so this isn't a regression. Let's look at libsndfile[experimental] which is everything minus regtest. experimental maps to using/supporting speex.
libsndfile development is somewhat coupled to vcpkg. Both projects share contributors, and the libsndfile build system has cmake code which reacts to vcpkg variables.
The portfile passes CMAKE_FIND_PACKAGE_PREFER_CONFIG=ON, and that's why the find modules were not used when a config for a package name was available.

This explains why it is not as bad as I initially thought. It already uses the config and targets, and it has find_dependency. There are still some rough edges.
This is from the 1.2.0 config log:
-- Could NOT find ALSA (missing: ALSA_LIBRARY ALSA_INCLUDE_DIR)
...
-- Found Speex: /vcpkg/installed/x64-linux/debug/lib/libspeex.a (found version "1.2.1")

Not finding ALSA is a (installation order) problem and a port bug, given port alsa, but that's another story.
Speex (for experimental) is found.

The new version should do the same:
-- Could NOT find ALSA (missing: ALSA_LIBRARY ALSA_INCLUDE_DIR)
...
CMake Warning at /vcpkg/scripts/buildsystems/vcpkg.cmake:859 (_find_package):
  By not providing "FindSpeex.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "Speex", but
  CMake did not find one.

  Could not find a package configuration file provided by "Speex" with any of
  the following names:

    SpeexConfig.cmake
    speex-config.cmake

  Add the installation prefix of "Speex" to CMAKE_PREFIX_PATH or set
  "Speex_DIR" to a directory containing one of the above files.  If "Speex"
  provides a separate development package or SDK, be sure it has been
  installed.
Call Stack (most recent call first):
  cmake/SndFileChecks.cmake:60 (find_package)
  CMakeLists.txt:95 (include)

Not finding ALSA is not a regression, so we don't care now.
It no longer finds Speex. This is a regression from this PR. It isn't reported a build error because it is not marked as required.
The fastest fix for the regression is to not remove this Find module. It isn't one which would get installed.
(The best solution is to move it to config mode, similar to the other packages. But this task is more involved.)

@dg0yt , did i miss something ot it is ok?

You missed the point of my comment regarding the patch.
Remove the first chunk of the patch. That chunk removes an effect on FIND_MODULES_INSTALL_LIST which already has no effect due the last chunk of the patch.
And the variables you want are in the code with or without the patch.


Add Plugin Dependencies to documentation

The PHP Interpreter configration features is only available for IDEs with PHP support (PhpStorm and IntelliJ Ultimate).
As this feature is optional, the intellij platform does not validate it's sub-dependencies. It requires the remote execution and the docker plugin enabled. This should be added to the documentation.
See #25.

Added required plugins for automatic remote interpreter configuration to the feature documentation as they are only required for this feature.
https://github.com/php-perfect/ddev-intellij-plugin/wiki/Features#prerequisites
Other features like the terminal integration does require the terminal module of the IDE but won't trigger any error or warning if it is disabled.


Console shows incorrect approval statuses for smart contract proposal

The error occurs when the name and id of the organization differ in configtx.yaml, e.g.
...
Organizations:
    - &Org1
        Name: Org1
        ID: Org1MSP
...

The error does not occur with test-network because the id and name of the peer organization are the same.
The status value depends on this line of code:
https://github.com/hyperledger-labs/fabric-operations-console/blob/f6230be047e44264e44808043db61b1588c3b89b/packages/apollo/src/components/ChaincodeModal/ChaincodeModal.js#L232 The keys in the commited object are the identifiers of organizations, but member.id is actually the name of the organization.
Members are passed from the ChannelDetails component and are based on the part of the channel configuration where the names of organizations are used.
https://github.com/hyperledger-labs/fabric-operations-console/blob/f6230be047e44264e44808043db61b1588c3b89b/packages/apollo/src/components/ChannelDetails/ChannelDetails.js#L229

No, I haven't tried to fix it.


Add golangci-lint

Description:
golangci-lint is a fast Go linters runner. It runs linters in parallel, uses caching, supports yaml config, has integrations with all major IDE, and has dozens of linters included.
References:
https://golangci-lint.run/

@dnnrly I want to work on this.

I would like you to consider that this project is aim at beginners to Go and that some users (me) use this on Termux on their smart phones. This means that there could be some restrictions about downloading and running a linter.
Having said that, if you can find a way of getting a linter working, PRs are welcome!

@dnnrly yeah it works locally also by running golangci-lint run command


Advanced Tables

Hi!
At first I'd like to thank you for the product! Thank you a lot for the great job!
I am asking for the advanced tables feature. I would like to be able to add a list of hosts into the table, where each row corresponds to the particular host with its items text data located in different columns.
(Just like the NewRelic server list table) Correct me if I am wrong and it is already possible to make this kind of a table.
Kind regards!

Hi, thanks! Yes, it's useful feature. There's #240 issue already opened. I should implement special table mode (format zabbix response as grafana table object). I'll try to do that soon.


Bugfix for data_parallel_model - add Copy ops

Bug was exposed by this test:
caffe2/python/data_parallel_model_test.py::GPUDataParallelModelTest::test_equiv

Fixes this error:
========= Invalid __global__ read of size 4
=========     at 0x000000a0 in caffe2::math::_Kernel_float_Add(int, float const *, float const *, float*)
E0522 10:33:44.497241 18450 context_gpu.h:126] Encountered CUDA error: unspecified launch failure
F0522 10:33:44.497547 18450 operator.h:200] Computation on device returned error in operator
input: "gpu_0/fc_b_grad" input: "gpu_1/fc_b_grad" output: "gpu_0/fc_b_grad" name: "dpm" type: "Sum" device_option { device_type: 1 cuda_gpu_id: 0 }

I have never seen this error, and we run this code a lot.

Do you ever run it without NCCL and without GLOO?

Yes. We disable NCCL by default. No-one uses NCCL currently (unless using Gloo). I wonder if your GPU setup incorrect somehow?
From: Luke Yeager<EMAIL_ADDRESS>Reply-To: caffe2/caffe2<EMAIL_ADDRESS>Date: Tuesday, May 23, 2017 at 10:41 AM
To: caffe2/caffe2<EMAIL_ADDRESS>Cc: Aapo Kyrola<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [caffe2/caffe2] Bugfix for data_parallel_model - add Copy ops (#633)
Do you ever run it without NCCL and without GLOO?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/caffe2/caffe2/pull/633#issuecomment-303477494, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABgM17NP6jNrwwmAxFmRDMEi7AYIP-uaks5r8xpGgaJpZM4NjCmV.

Looks like you might be right. I'll close this assuming it's a problem with my machine for now (I may reopen if I discover something). Thanks for the review!

@akyrola have you ever tested with non-heterogeneous GPUs? The error was because my GPUs don't have P2P access.

No we haven’t! Good catch. Is it possible to add a check whether there is uniform access or not and use different reduction then? I am concerned of using Copy always, since it will slow things down.
From: Luke Yeager<EMAIL_ADDRESS>Reply-To: caffe2/caffe2<EMAIL_ADDRESS>Date: Tuesday, May 23, 2017 at 2:25 PM
To: caffe2/caffe2<EMAIL_ADDRESS>Cc: Aapo Kyrola<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [caffe2/caffe2] Bugfix for data_parallel_model - add Copy ops (#633)
@akyrolahttps://github.com/akyrola have you ever tested with non-heterogeneous GPUs? The error was because my GPUs don't have P2P access.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/caffe2/caffe2/pull/633#issuecomment-303536902, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABgM14YIWTuKl8XXbTHNk9WcC0r5OfYHks5r807MgaJpZM4NjCmV.

I am concerned of using Copy always, since it will slow things down.

Agreed. I'll look into it now.


Is there a JSDoc intermediate format?

I'm wondering if there is an existing intermediate format which could be emitted by other documentation projects such as 'typedoc' that could be read into JSDoc to leverage the JSDoc ecosystem of templates and plugins?
eg:
file : { node: { line: 123, comments: {..}, context{..} }, ...  }

check out https://github.com/jsdoc2md
and there are other projects that parse jsdoc to json too (eg. https://github.com/documentationjs/documentation/blob/master/docs/NODE_API.md among others) -- i personally used jsdoc2md on my site for a while.

Sorry, there's no intermediate format like that. The closest thing is the JSON output you get when you run jsdoc -X, and JSDoc isn't capable of reading that JSON, only generating it.


feat: add function list to auto-complete to Clickhouse datasource

SUMMARY
add function list to auto-complete to Clickhouse datasource
Fixes #15477
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS
use https://gist.github.com/Slach/ad926dfa30ceb4fe1d168c00c3da98e0#file-bootstrap-dev-sh
bootstrap-dev.sh script
open https://localhost:8088
add clickhouse+native://default@clickhouse:9000/default as new datasource
after add, open SQLlab and try to use auto-complete

Hi @Slach, Thanks for the following updates. LGTM excepts CI didn't pass. Could you have got time to fix it?

@zhaoyongjie try to fix CI/CD https://github.com/apache/superset/pull/16234/commits/33cbd2585cc500c038b3545eeaf677fa04f14186

@zhaoyongjie try to fix CI/CD 33cbd25
how to trigger CD/CD workflow?

I will trigger the CI. thanks!

@zhaoyongjie sorry, i forgot apply black formatter
fixed on https://github.com/apache/superset/pull/16234/commits/368c2a62d9f977b55870be23df6d87ae1d806d2b

@zhaoyongjie ;) i also apply isort to clickhouse.py
https://github.com/apache/superset/pull/16234/commits/2e003cea822188fa026b40cc31b5b3be54cfa98f
could you trigger ci/cd again?

@zhaoyongjie ;) i also apply isort to clickhouse.py
2e003ce
could you trigger ci/cd again?

Sure.

@zhaoyongjie could you suggest about cypress-matrix tests?
https://github.com/apache/superset/pull/16234/checks?check_run_id=3398189875
look like it not related to my changes

@zhaoyongjie could you suggest about cypress-matrix tests?
https://github.com/apache/superset/pull/16234/checks?check_run_id=3398189875
look like it not related to my changes

Cypress might be stuck in unknown reason. I will restart this job.

@zhaoyongjie looks like all checks passed, could we merge pull request?


vdirsyncer: use Python3 by default

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
/cc @matthiasbeyer @jgeerds

Even if Python2 is still supported, the maintainer says that some
features won't work and the support will soon be discarded.

Did you check any other applications depending on it also use python3?

@domenkozar I think that's the role of nox-review, but I also did it manually. khard's derivation depends on vdirsyncer and is only compatible with Python 2: nevertheless, I think there is no reason for khard to depend on vdirsyncer and I will fix that now in another PR.


Fix lint error in register.go

fixes #102
fix lint error by adding the comment on exported const group name
Signed-off-by: Andrea Thniah<EMAIL_ADDRESS>

Codecov Report

Merging #119 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   55.77%   55.77%
=======================================
  Files          41       41
  Lines        1533     1533
=======================================
  Hits          855      855
  Misses        621      621
  Partials       57       57

Continue to review full report at Codecov.

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

@andreathniah Can you please fix the DCO signoff error !

@andreathniah I think you have to rebase all the commits and send it into one commit with DCO sign-off.

e all the commits and send it into one commit with DCO

I rebased and add the sign-off message for each of the commits. I'm not sure why it still doesn't seem to work?

Thanks @andreathniah . Can you please do "git commit -s --ammend " ?

Thanks @andreathniah . Can you please do "git commit -s --amend " ?

That worked! Thank you for the help, apologies for the countless mistakes made!

Great @andreathniah . Good job.
Thanks for your PR !

Thanks @a4abhishek .
@andreathniah Just one last change as @a4abhishek suggested, Please make it something like "GroupName is the constant used for ...."


A bit pissed about https://github.com/swc-project/swc/issues/8058

Describe the feature
https://github.com/swc-project/swc/issues/8058
Wait, I spend 20mn doing all of that and you close the issue automatically when I did provide the link given here?
And then, I cannot even update the main post because it's locked? How bad a joke is that?

cc @balazsorban44 It seems like #8058 has a link to play.swc.rs, which is listed at
https://github.com/swc-project/swc/blob/89676946e6db6e23ac48878c5f54a702a1a325fe/.github/workflows/nissuer.yml#L23
Is this a misconfiguration?

Yeah, it's missing https://github.com/balazsorban44/nissuer#validate-reproduction-urls
reproduction-link-section. the default ### Link to reproduction(.*)### To reproduce
won't match your template

This should not be an issue anymore @DNA-PC! It was a flick in the initial setup 🙏

Cloisng as fixed by https://github.com/swc-project/swc/pull/8060
Thank you!


[SPARK-50524][SQL] Lower RowBasedKeyValueBatch.spill warning message to debug level

What changes were proposed in this pull request?
This PR aims to lower RowBasedKeyValueBatch.spill warning message to debug level.
   public final long spill(long size, MemoryConsumer trigger) throws IOException {
-    logger.warn("Calling spill() on RowBasedKeyValueBatch. Will not spill but return 0.");
+    logger.debug("Calling spill() on RowBasedKeyValueBatch. Will not spill but return 0.");
     return 0;
   }

Why are the changes needed?
Although Apache Spark has been showing a warning message since 2.1.0, there is nothing for a user to be able to do further. This is more like a dev-side debug message. So, we had better lower the level to DEBUG from WARN.

#14349

Does this PR introduce any user-facing change?
No behavior change. This is a log message.
How was this patch tested?
Manual review.
Was this patch authored or co-authored using generative AI tooling?
No.

Could you review this PR, @viirya ?

Looks good to me.
On Mon, Dec 9, 2024 at 2:04 PM Dongjoon Hyun @.***>
wrote:

Could you review this PR, @viirya https://github.com/viirya ?
—
Reply to this email directly, view it on GitHub
https://github.com/apache/spark/pull/49116#issuecomment-2529621569, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAQZ576KCNM2HUMBT5RVLT2EYHWFAVCNFSM6AAAAABTJVOV4CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRZGYZDCNJWHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you!

Merged to master.

Thank you, @HyukjinKwon !


Add Re-Lax

Pull Request Description
Re-Lax is a tvOS framework for generating Parallax images utilizing low level frameworks and APIs such as Accelerate, lzsfe, and unsafe memory access. This is licensed under MIT.
Acceptance Criteria
To be accepted into the Swift source compatibility test suite, a project must:

[x] be an Xcode or swift package manager project
[x] support building on either Linux or macOS
[x] target Linux, macOS, or iOS/tvOS/watchOS device
[x] be contained in a publicly accessible git repository
[x] maintain a project branch that builds against Swift 3.0 compatibility mode
and passes any unit tests
[x] have maintainers who will commit to resolve issues in a timely manner
[x] be compatible with the latest GM/Beta versions of Xcode and swiftpm
[x] add value not already included in the suite
[x] be licensed with one of the following permissive licenses:

BSD
MIT
Apache License, version 2.0
Eclipse Public License
Mozilla Public License (MPL) 1.1
MPL 2.0
CDDL

[x] pass ./check Re-Lax

@swift-ci test


Improve language on addon structure page

Consistently use 'you' over 'we'
Some grammatical improvements

Thanks!


⚠ Taitopia Cloud Rendering Service Status has degraded performance

In 58c25d1, Taitopia Cloud Rendering Service Status (https://status-api.taitopia.design/health/taco) experienced degraded performance:

HTTP code: 200
Response time: 5 ms

Resolved: Taitopia Cloud Rendering Service Status performance has improved in 1aa4f9f after 6 hours, 5 minutes.