hyprland crash when external monitor connected on dgpu

Already reported ? *

[X] I have searched the existing open and closed issues.

Regression?
Yes
System Info and Version
 System/Version info
Hyprland 0.44.0 built from branch main at commit f603a22af0c8890b9742d761d44f0b595740f5b1  (internal: Remove some unused lambda captures (8218)).
Date: Thu Oct 24 14:12:41 2024
Tag: v0.44.0-71-gf603a22a, commits: 5370
built against aquamarine 0.4.3

flags set:
debug

System Information:
System name: Linux
Node name: heimdall
Release: 6.11.5-arch1-1
Version: #1 SMP PREEMPT_DYNAMIC Tue, 22 Oct 2024 18:31:38 +0000

GPU information:
00:02.0 VGA compatible controller [0300]: Intel Corporation Raptor Lake-S UHD Graphics [8086:a788] (rev 04) (prog-if 00 [VGA controller])
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation AD104M [GeForce RTX 4080 Max-Q / Mobile] [10de:27e0] (rev a1) (prog-if 00 [VGA controller])
NVRM version: NVIDIA UNIX Open Kernel Module for x86_64  560.35.03  Release Build  (archlinux-builder@)

os-release: NAME="Arch Linux"
PRETTY_NAME="Arch Linux"
ID=arch
BUILD_ID=rolling
ANSI_COLOR="38;2;23;147;209"
HOME_URL="https://archlinux.org/"
DOCUMENTATION_URL="https://wiki.archlinux.org/"
SUPPORT_URL="https://bbs.archlinux.org/"
BUG_REPORT_URL="https://gitlab.archlinux.org/groups/archlinux/-/issues"
PRIVACY_POLICY_URL="https://terms.archlinux.org/docs/privacy-policy/"
LOGO=archlinux-logo

plugins:

======Config======

monitor = eDP-1, 2560x1600@240, 0x0, 1
monitor = DP-1, 2560x1440@160, 2560x0, 1
#monitor = DP-2, 2560x1440@160, 2560x0, 1

Description
been using other WM's for a bit so not sure when this has begun occuring. but instant crash if external monitor is attached. everything -git built
#0  __pthread_kill_implementation (threadid=<optimized out>, signo=signo@entry=6, no_tid=no_tid@entry=0) at pthread_kill.c:44
#1  0x00007ffff71b6463 in __pthread_kill_internal (threadid=<optimized out>, signo=6) at pthread_kill.c:78
#2  0x00007ffff715d120 in __GI_raise (sig=sig@entry=6) at ../sysdeps/posix/raise.c:26
#3  0x00007ffff71444c3 in __GI_abort () at abort.c:79
#4  0x000055555571a191 in handleUnrecoverableSignal (sig=11) at ../src/Compositor.cpp:80
#5  0x00007ffff715d1d0 in <signal handler called> () at /usr/lib/libc.so.6
#6  0x00007ffff7f24867 in Hyprutils::Memory::CSharedPointer<Aquamarine::IBuffer>::destroyImpl (this=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:282
#7  Hyprutils::Memory::CSharedPointer<Aquamarine::IBuffer>::decrement (this=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:265
#8  Hyprutils::Memory::CSharedPointer<Aquamarine::IBuffer>::~CSharedPointer (this=<optimized out>, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:177
#9  Aquamarine::COutputState::SInternalState::~SInternalState (this=<optimized out>, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/backend/drm/../../output/Output.hpp:58
#10 Aquamarine::COutputState::~COutputState (this=<optimized out>, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/backend/drm/../../output/Output.hpp:42
#11 std::default_delete<Aquamarine::COutputState>::operator() (this=0x100000018, __ptr=<optimized out>) at /usr/include/c++/14.2.1/bits/unique_ptr.h:93
#12 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::_destroy (this=0xffffffff) at /usr/include/hyprutils/memory/SharedPtr.hpp:69
#13 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::destroy (this=0xffffffff) at /usr/include/hyprutils/memory/SharedPtr.hpp:103
#14 Hyprutils::Memory::CSharedPointer<Aquamarine::SDRMPlane>::destroyImpl (this=0x7fffffffda10) at /usr/include/hyprutils/memory/SharedPtr.hpp:279
#15 Hyprutils::Memory::CSharedPointer<Aquamarine::SDRMPlane>::decrement (this=0x7fffffffda10) at /usr/include/hyprutils/memory/SharedPtr.hpp:265
#16 Hyprutils::Memory::CSharedPointer<Aquamarine::SDRMPlane>::~CSharedPointer (this=0x7fffffffda10, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:177
#17 Aquamarine::CDRMAtomicImpl::reset (this=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/src/backend/drm/impl/Atomic.cpp:352
#18 0x00007ffff7f2b817 in Hyprutils::Memory::CSharedPointer<Aquamarine::IBuffer>::decrement (this=0x555556e048f8) at /usr/include/hyprutils/memory/SharedPtr.hpp:261
#19 Hyprutils::Memory::CSharedPointer<Aquamarine::IBuffer>::~CSharedPointer (this=0x555556e048f8, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:177
#20 Aquamarine::COutputState::SInternalState::~SInternalState (this=0x555556e04890, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/output/Output.hpp:58
#21 Aquamarine::COutputState::~COutputState (this=0x555556e04890, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/output/Output.hpp:42
#22 std::default_delete<Aquamarine::COutputState>::operator() (this=0x19, __ptr=0x555556e04890) at /usr/include/c++/14.2.1/bits/unique_ptr.h:93
#23 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::_destroy (this=0x0) at /usr/include/hyprutils/memory/SharedPtr.hpp:69
#24 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::destroy (this=0x0) at /usr/include/hyprutils/memory/SharedPtr.hpp:103
#25 Hyprutils::Memory::CSharedPointer<Aquamarine::SOutputMode>::destroyImpl (this=0x7fffffffdb78) at /usr/include/hyprutils/memory/SharedPtr.hpp:279
#26 Hyprutils::Memory::CSharedPointer<Aquamarine::SOutputMode>::decrement (this=0x7fffffffdb78) at /usr/include/hyprutils/memory/SharedPtr.hpp:265
#27 Hyprutils::Memory::CSharedPointer<Aquamarine::SOutputMode>::~CSharedPointer (this=0x7fffffffdb78, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:177
#28 Aquamarine::COutputState::SInternalState::~SInternalState (this=0x7fffffffdb20, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/output/Output.hpp:58
#29 Aquamarine::COutputState::~COutputState (this=0x7fffffffdb20, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/./include/aquamarine/output/Output.hpp:42
#30 std::default_delete<Aquamarine::COutputState>::operator() (this=0x19, __ptr=0x7fffffffdb20) at /usr/include/c++/14.2.1/bits/unique_ptr.h:93
#31 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::_destroy (this=0x0) at /usr/include/hyprutils/memory/SharedPtr.hpp:69
#32 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::destroy (this=0x0) at /usr/include/hyprutils/memory/SharedPtr.hpp:103
#33 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::~impl (this=0x0, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:119
#34 Hyprutils::Memory::CSharedPointer_::impl<Aquamarine::COutputState>::~impl (this=0x0, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:120
#35 Hyprutils::Memory::CWeakPointer<Aquamarine::SOutputMode>::decrementWeak (this=0x2d4eb55ffe9ce400) at /usr/include/hyprutils/memory/WeakPtr.hpp:172
#36 0x00007ffff7f1ca1e in Hyprutils::Memory::CSharedPointer<Hyprutils::Signal::CSignalListener>::destroyImpl (this=0x555555e03a38) at /usr/include/hyprutils/memory/SharedPtr.hpp:282
#37 Hyprutils::Memory::CSharedPointer<Hyprutils::Signal::CSignalListener>::decrement (this=0x555555e03a38) at /usr/include/hyprutils/memory/SharedPtr.hpp:265
#38 Hyprutils::Memory::CSharedPointer<Hyprutils::Signal::CSignalListener>::~CSharedPointer (this=0x555555e03a38, __in_chrg=<optimized out>) at /usr/include/hyprutils/memory/SharedPtr.hpp:177
#39 Aquamarine::CDRMRendererBufferAttachment::~CDRMRendererBufferAttachment (this=0x555555e03a10, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/src/backend/drm/Renderer.hpp:30
#40 Aquamarine::CDRMRendererBufferAttachment::~CDRMRendererBufferAttachment (this=0x555555e03a10, __in_chrg=<optimized out>) at /usr/src/debug/aquamarine-git/aquamarine/src/backend/drm/Renderer.hpp:30
#41 0x0000555555a03c96 in CHyprRenderer::applyMonitorRule (this=0x0, pMonitor=..., pMonitorRule=0x0, force=128) at ../src/render/Renderer.cpp:2339
#42 0x0000555555805477 in CMonitor::onConnect (this=0x555557178860, noRule=true) at ../src/helpers/Monitor.cpp:174
#43 0x000055555573a156 in CCompositor::onNewMonitor (this=0x0, output=...) at ../src/Compositor.cpp:3062
#44 0x0000555555721ad6 in CCompositor::initServer (this=0x555555da34b0, socketName=<error: Cannot access memory at address 0x7300676e69>, socketFd=-6528) at ../src/Compositor.cpp:339
#45 0x00005555556a696c in main (argc=0, argv=0x1) at ../src/main.cpp:165

How to reproduce
just attach monitor in this case to DP-1 and it crashes on launch.
Crash reports, logs, images, videos
No response

ok interestingly its because i had these set, used to set those on kwin to kinda force all apps run on intel and yet have external monitor going on nvidia. commenting it out and runs again
#force intel for nvidia D3 power management
#env = ENABLE_DEVICE_CHOOSER_LAYER,1
#env = VULKAN_DEVICE_INDEX,0
#env = DXVK_FILTER_DEVICE_NAME,Intel
#env = VKD3D_FILTER_DEVICE_NAME,Intel
#env = MESA_VK_DEVICE_SELECT,8086:a788
#env = __GLX_VENDOR_LIBRARY_NAME,mesa
#env = __EGL_VENDOR_LIBRARY_FILENAMES,/usr/share/glvnd/egl_vendor.d/50_mesa.json
#env = VK_ICD_FILENAMES,/usr/share/vulkan/icd.d/intel_icd.x86_64.json:/usr/share/vulkan/icd.d/intel_icd.i686.json
#env = __NV_PRIME_RENDER_OFFLOAD,0
#env = __VK_LAYER_NV_optimus,non_NVIDIA_only
#env = LIBVA_DRIVER_NAME,iHD
#env = VDPAU_DRIVER,va_gl
#env = WLR_RENDER_DRM_DEVICE,/dev/dri/renderD129
#env = WLR_DRM_DEVICES/dev/dri/card0
#env = GST_VAAPI_DRM_DEVICE,/dev/dri/renderD129

which one specifically is at fault?

yeee its likely not a hl issue


Fix renderer observer, avoid unnecessary template restamp

This change is

Pull Request Test Coverage Report for Build 909

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 88.032%

Files with Coverage Reduction
New Missed Lines
%

src/vaadin-overlay.html
13
88.03%

Totals

Change from base Build 901:
-0.4%

Covered Lines:
203

Relevant Lines:
227

💛  - Coveralls


[cloud_firestore] Added source support

Description
Added source support in order to get doc(s) from server / cache / both ( default).
Related Issues
https://github.com/flutter/flutter/issues/19078
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[x] I updated CHANGELOG.md to add a description of the change.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

Can someone assist with the file formatting and hard_codec_version.
pub global run flutter_plugin_tools format doesn't  work for me.
Thanks

Can someone assist with the file formatting and hard_codec_version.
pub global run flutter_plugin_tools format doesn't work for me.
Thanks

I'll take care of it. Thank you!

A change that adds new API surface area should have unit and integration tests. I've added some.


Fix iOS deployment target in KMPNativeCoroutinesAsync

Fixes #52

Thanks for the contribution @adesentenac! Could you provides some more details about the issue you are having?
Since Xcode 13.2 the Swift concurrency features should be supported on iOS 13 and above (the deployment target has actually been downgraded from 15 to 13 in 69ae7b8eba96427b668805b69e3797f196fe0c34).

@rickclephas ah sorry, I was trying to build with Xcode 13.0 and I wasn't aware that this was changed in a newer Xcode version.
I'll close the PR and the issue then. Thanks!


Removing some clutter from ApexMocks syntax

Hey guys,
I recently started working with apex-mocks and I love it. The only thing I didn't like and where it falls a bit short with regards to Mockito is the amount of clutter in the code. Some is due to the Salesforce restrictions (no generics...) but some seemed unnecessary.
So I played arround with it a bit and found three changes that improve this. The results can be seen here: https://github.com/mischkes/fflib-apex-mocks/tree/feature/poc-for-less-clutter#a-more-complicated-case
We are already using this code in production, but it is not yet library-grade. If you are interested I would be happy to improve the code and make some pull requests.
The changes are:
Create a Utility class that encapsulates ApexMocks and Match

See https://github.com/mischkes/fflib-apex-mocks/blob/feature/poc-for-less-clutter/src/classes/fflib_Mock.cls
this is actually the same way Mockito is doing it
I would advise Projects using fflib-apex-mocks to rename this file (eg. to 'Mocks' as in the example or even 'Mk') to further reduce clutter. This is easily possible as there are no dependencies on fflib_Mock, so no other class has to be changed.
This removes the clutter from constructing fflib_ApexMocks
This removes clutter from writing fflib_Match
This removes more clutter for more complex use-cases, e.g. by the use of fflib_Mock.Answer)
This also creates a single-place-of use, which makes it easier for Newbies. "Just look at fflib_Mock"
This also hides away some of the public Methods intended for use internally, like handleMethodCall or mockNonVoidMethod
This change requires almost no change to the existing code. Currently I'm subclassing fflib_Match from fflib_Mock for convenienve, but event that could be wrapped. Which might be a good Idea to hide some internal methods like getAndClearMatchers.

Make start/stopStubbing unnecessary

there is/was probably some use-case why those methods makes sense, but I didn't see it
the main change was to always return a value in fflib_ApexMocks.mockNonVoidMethod, and move the actual parameter storing to fflib_ApexMocks.when
This remove the clutter from the startStubbing() / stopStubbing() methods
this is a minor breaking change, as mocking a method that was previously mocked to throw an exception will throw this exception during mocking. Mockito uses doReturn() statements for this case, which could be implemented
this also removes the curent possibility to check for incomplete stubbing. I guess it could still be done by setting a flag inside when() and clearing it on then()

Record call history inside methods

this change is probably mostly a matter of taste
I never really liked the way complex argument checks are done in Mockito, with the ArgumentCaptor and all. And ApexMocks is following Mockito here.
I really like the way jasmine is handling this and implemented it in the same way: simply storing the call history in the method (here: the Method mock object)
this removes the clutter to create the captor, then verifying, and then getting the object from the captor
this was also a minor change in the code

All in all these changes remove approx. 50% of the necessary mocking code, which I think is really a lot.

Great work - really appreciate the summary and your reasoning. I agree with almost everything above. I love the improved encapsulation and reduced clutter.
A few follow up points...

I would advise Projects using fflib-apex-mocks to rename this file (eg. to 'Mocks' as in the example or even 'Mk') to further reduce clutter. This is easily possible as there are no dependencies on fflib_Mock, so no other class has to be changed.
The fflib_ prefix is intentional, to avoid naming collisions when injected into another code-base as part of the Salesforce happy soup. I suspect a namespaced Unlocked Package will be the best way to give the best of both worlds.
Of course there's less chance of a collision with everything stored instead in a single utility class.

Make start/stopStubbing unnecessary

there is/was probably some use-case why those methods makes sense, but I didn't see it
I believe this prevents us from incrementing the call count, capturing arguments, throwing exceptions or invoking answers in the test setup phase. If we can still achieve all of these without explicit startStubbing and stopStubbing calls, I'm happy.

I really like the way jasmine is handling this
It's a core philosophical decision to be as close to Mockito as possible, to lower the barrier to entry and allow us to port Mockito features following their syntax. This might prove contentious but I'm ok with this in theory.

@agarciaodeian - any thoughts?

Since I did not hear from @dfruddffdc and @agarciaodeian again I guess this is not something you want to follow right now. So I will just fork the project, which makes a lot of sense anyway. Most of the code-generation stuff should be deprecated anyway and in a fork I can simply remove it 😇


feat: select_next_item and select_prev_item support "lines" option. deafults to 1

User can increase lines value to select item with pagedown/pageup.

cmp.select_prev_item({ behavior = behavior, lines = 8 })
cmp.select_next_item({ behavior = behavior, lines = 8 })

close #307

It's mostly good. I think the option name should be "count" instead of "lines".
Could you change it like that?

@hrsh7th Done. Please review it.

@hrsh7th It's done. Please review it when you have free time. Thanks.


Messages: icon for alarms

Alarm clock reports as "alarmclockreceiver" on my side but i guess it is "alarm" on some other devices

Nice - thanks!


Ability to Skip generation of source files

Looking through the configuration options I am not seeing a way to skip generation of sources - either at an execution level, or a global plugin level? Is there a way to achieve this behavior?
Basically the scenario we have is that we want to generate source files, but we also want the ability to modify these source files, say custom annotations for example(just an example). If the source files are being regenerated at every clean & build we can't achieve this. An alternative we can use is basically just generate and comment this execution out, not ideal but can be done.
If there is no way to skip the generation process for select executions, global plugin level. Is there a reason why this functionality would be opposed?

No, there's no configuration option to disable the plugin.

Looking through the configuration options I am not seeing a way to skip generation of sources - either at an execution level, or a global plugin level? Is there a way to achieve this behavior?

Primarily with up-to-date checks:
https://github.com/highsource/maven-jaxb2-plugin/wiki/Up-To-Date-Checks
The code will only be regenerated if certain source files change. But it is intended for normal use-cases like "do not recompile schemas unless *.xsd files changed".

Basically the scenario we have is that we want to generate source files, but we also want the ability to modify these source files

You should not modify the generated source code or store in a version control system. If you do, it is a wrong thing to do.

say custom annotations for example(just an example).

See jaxb2-annotate-plugin for this example.

If the source files are being regenerated at every clean & build we can't achieve this.

They should be and you should not be able to achieve this.


MIWI API converter fixes

Which issue this PR addresses:
No Jira
What this PR does / why we need it:
Fixes two issues with the API converters:

Ensures all cluster MSI / Identity fields are included in conversions
Ensures that all platform workload identities are included when converting from internal -> admin

Test plan for issue:
I plan to test in canary.
Is there any documentation that needs to be updated for this PR?
N/A
How do you know this will function as expected in production?
Will be tested in canary.

/azp run ci

/azp run ci


Incorrect transparency of ViewportTexture

Godot version
4.0.dev (9539e4e39e)
System information
Ubuntu 18.04, Vulkan
Issue description
I'm trying to display a ViewportTexture to a MeshInstance. The viewport has transparent background enabled, and the MeshInstance material's transparency is set to Alpha.
Expected result:
The texture's background is transparent.
Actual result:
The texture's background is black.
Setting the blend mode to 'Add' sets the background to transparent, but the rest of the texture also gets slightly transparent. My assumption is that the ViewportTexture's alpha channel is not set correctly - even though the background is set to transparent - , which would explain why it the background is only transparent with additive blend mode.

Steps to reproduce

Create a PlaneMesh
Create a SubViewport
Set transparent background to the viewport
Add a 3D scene as child to the viewport
Set the albedo texture of the plane's material override to the viewport's texture

Minimal reproduction project
minimal.zip

Duplicate of https://github.com/godotengine/godot/issues/40651.


fix metric reporting

ran into the same issue as #637 with partitions with 0 offsets - the patch below worked for me

/cc @d1egoaz

@bai Any update on ETA for this MR to be accepted/merged?

I came across the same issue earlier this week, sorry i wasn't keeping an eye on the repo
Fix im using is https://github.com/linkedin/Burrow/compare/linkedin:master...mwain:fix-prom-incomplete
Which is very similar to this one 👍

Hullo, is there anything I can do to help move this PR forward? :)

Sorry folks, I have been busy at work, will try to take another pass at this in the following days.

I have the same issue and would love to see this PR merged.
Has there been any progress?

Going on a full year now - how can I help get this merged?
Tagging @bai as a code owner.

Thanks all! 🎉 🚢


Use hash style for AR queries

Prefer a hash style that string style for AR queries
#bad
workflow_scope.where("id = ?", params[:id])

# good
workflow_scope.where(id: params[:id])

I think this rule should be included to the rails rubocop config so we don't need to add a separate rule

Agreed 👍


Embed a pdf in a pptx using PptxGenJS

Do we have an API that will allow us to embed on object like pdf or excel document inside a slide

Hi @Rishav12345 ,
No, I don't have any plans to incorporate something that complex into the library.  You can create hyperlinks to those documents...


Filling for DC doesn't work

I fill all states with a color including DC but DC stays gray
AK: {fill: "#B5C460"}
AL: {fill: "#B5C460"}
AR: {fill: "#B5C460"}
AZ: {fill: "#B5C460"}
CA: {fill: "#B5C460"}
CO: {fill: "#B5C460"}
CT: {fill: "#B5C460"}
DC: {fill: "#B5C460"}
DE: {fill: "#B5C460"}
FL: {fill: "#B5C460"}
GA: {fill: "#B5C460"}
HI: {fill: "#B5C460"}
IA: {fill: "#B5C460"}
ID: {fill: "#B5C460"}
IL: {fill: "#B5C460"}
IN: {fill: "#B5C460"}
KS: {fill: "#B5C460"}
KY: {fill: "#B5C460"}
LA: {fill: "#B5C460"}
MA: {fill: "#B5C460"}
MD: {fill: "#B5C460"}
ME: {fill: "#B5C460"}
MI: {fill: "#B5C460"}
MN: {fill: "#B5C460"}
MO: {fill: "#B5C460"}
MS: {fill: "#B5C460"}
MT: {fill: "#B5C460"}
NC: {fill: "#B5C460"}
ND: {fill: "#B5C460"}
NE: {fill: "#B5C460"}
NH: {fill: "#B5C460"}
NJ: {fill: "#B5C460"}
NM: {fill: "#B5C460"}
NV: {fill: "#B5C460"}
NY: {fill: "#B5C460"}
OH: {fill: "#B5C460"}
OK: {fill: "#B5C460"}
OR: {fill: "#B5C460"}
PA: {fill: "#B5C460"}
RI: {fill: "#B5C460"}
SC: {fill: "#B5C460"}
SD: {fill: "#B5C460"}
TN: {fill: "#B5C460"}
TX: {fill: "#B5C460"}
UT: {fill: "#B5C460"}
VA: {fill: "#B5C460"}
VT: {fill: "#B5C460"}
WA: {fill: "#B5C460"}
WI: {fill: "#B5C460"}
WV: {fill: "#B5C460"}
WY: {fill: "#B5C460"}

In the source I found that for filling DC two "states" are used "DC1" and "DC2". Not sure what "DC1" is doing but "DC2" works for me.
I think this should be described in README at least. Or "DC" fill color should be used for both "DC1" and "DC2".

Also, the title for DC state doesn't work and I don't see ways to set it.

@shir @gabidavila - Did you notice that DC's cursor style is different than the 50 states? In other words, it doesn't look like a clickable area when you hover over DC's circle.

DC2 works for filling in the color, but i can't seem to get the onClick handler to work. i even found the dimensions for DC and added them to the data but it still doesn't work. and when you hover over DC it says "blank US states map." this plugin made it super easy to make a clickable US map, but DC being broken complicates things.


ungreylist test for 3.6

Scope & Purpose
Un-greylist a previously greylisted test

[x] Bug-Fix for a released version (did you remember to port this to all relevant release branches?)
[x] The behavior change can be verified via automatic tests

Related Information

[x] There is a JIRA Ticket number: https://arangodb.atlassian.net/browse/BTS-27

Testing & Verification
This change is already covered by existing tests, such as resilience.
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/10396/

Tests blue


Adding Trapping Rain Water in cpp

Hey , @risingstar-bit please assign me this issue

Hey, can you assign this to me


args.foo broken in v0.19.0 ?

What reproduces the bug?
Trying to run execsnoop.bt after updating yields this error:
./execsnoop.bt:26:2-12: ERROR: Can not access field 'argv' on type '(ctx) struct _tracepoint_syscalls_sys_enter_execve *'. Try dereferencing it first, or using '->'
    join(args.argv);
    ~~~~~~~~~~

Replacing args.argv by args->argv makes it work again
I tried other scripts using args.foo and had a similar experience -- it's not difficult to fix so I'd be happy to open a PR just replacing each of these after testing, but since it's breaking existing scripts I'm wondering if it was on purpose?
bpftrace --info
System
  OS: Linux 6.1.44 #1-NixOS SMP PREEMPT_DYNAMIC Tue Aug  8 18:03:51 UTC 2023
  Arch: x86_64

Build
  version: v0.19.0
  LLVM: 16.0.6
  unsafe probe: no
  bfd: yes
  libdw (DWARF support): yes

Kernel helpers
  probe_read: yes
  probe_read_str: yes
  probe_read_user: yes
  probe_read_user_str: yes
  probe_read_kernel: yes
  probe_read_kernel_str: yes
  get_current_cgroup_id: yes
  send_signal: yes
  override_return: no
  get_boot_ns: yes
  dpath: yes
  skboutput: yes
  get_tai_ns: yes
  get_func_ip: yes

Kernel features
  Instruction limit: 1000000
  Loop support: yes
  btf: yes
  module btf: yes
  map batch: yes
  uprobe refcount (depends on Build:bcc bpf_attach_uprobe refcount): yes

Map types
  hash: yes
  percpu hash: yes
  array: yes
  percpu array: yes
  stack_trace: yes
  perf_event_array: yes
  ringbuf: yes

Probe types
  kprobe: yes
  tracepoint: yes
  perf_event: yes
  kfunc: yes
  kprobe_multi: no
  raw_tp_special: yes
  iter: yes

Oh, sorry it's a problem with my build -- I was using an old version of bpftrace through #!/usr/bin/env but I was expecting the scripts to point to the newly built bpftrace.
It works as expected when using the correct binary, with both versions (args.foo and args->foo) -- sorry for the noise!

Found this problem using package from apt.
The binary from Releases works.

@WhiverFichos your apt package version is probably <0.19.0 in which we changed the args builtin to be a structure instead of a pointer.


Collaboration sidebar

🚀 Deployed on https://67006007aa1f2829d866a175--cozy-fox-0defec.netlify.app

Merge activity

Oct 4, 5:45 PM EDT: @anle-statsig started a stack merge that includes this pull request via Graphite.


Additional tests for opam-repo-ci

In https://github.com/ocaml/opam-repository/pull/25143 we realised there is a near-collision for package names: two packages where one has dashes (-) and the other underscores (_). We (the opam-repository maintainers) would like to avoid creating such near collisions. It would be good if the opam-repo-ci could add the following tests/outputs:

When a new package file is added by a PR, and there is another package that has the same name except for non-alphanumerical characters, then there should be a warning in the CI (likely in the linting job?)

We also think it'd be nice to have additional warnings for names that are similar but not identical. Just to make typo-squatting more difficult.

When a new package file is added by a PR, and there is another package that has a name with a low Levenshtein-distance from the newly published one, then there should be a warning in the CI. (Low would probably be 2 or 3.)

For the low Levenshtein distance warning, how do you propose the value of "low" be dependent on the overall length of the package name? Without this, packages that have names 3 or 4 characters long (of which there are many) will all fail their lint checks.

The distance probably needs to only be checked after some threshold of length. I think ideally we would check the current distance distribution and then make some good guess about the appropriate threshold. As a gut feeling, I'd say the distance would be something like 1/6th. As in:

For a package up to name-length 6, trigger a warning if there is a distance of 1.
For a package up to name-length 12, trigger a warning if there is a distance of 2 or less.
etc.

This might be a bit naive though… and maybe we need something fancy with a non-linear length-distance relation.

I'll have a go implementing it in a general way, so that we can easily tune the threshold if there's annoying false positives.

The Levenshtein distance check is not yet implemented.


tunnelblick is not working

after installing tunnelblick with cask It's not starting up as it's assuming a security incident because
it's detecting that application is actually a symlink and not the actual .app folder

Which version you're trying to install?
Tunnelblick should ask you for moving itself to /Applications.
See https://github.com/caskroom/homebrew-cask/issues/9371 and https://github.com/caskroom/homebrew-cask/pull/9178 (which may be at some point definitive solution)

@radeksimko I'm trying latest version. I have export HOMEBREW_CASK_OPTS="--appdir=/Applications" however the application is not installing there as it just symlinks itself there so when I try running it it reports potential security incident and refuses to run. How do I make it install there instead of just doing symlink?

How do I make it install there instead of just doing symlink?

You don’t with homebrew-cask. We don’t support that yet. You must do it manually.

@vitorgalvao ic, thx

Closed in favour of https://github.com/caskroom/homebrew-cask/issues/12858.


Read the webhook secret from a file

Summary
Complements reading the previous webhook secret from a file by allowing
to read the current one.
Fixes: #3160
Change Type

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
make test, use the option
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 50.138% (+0.02%) from 50.119%
when pulling a8710e1c3ba0ca6f9223000d6d3a22aa02c1e6d9 on jhrozek:wh_from_file
into cea252089ecd8331bad80f0ed12529f68f949457 on stacklok:main.


issue - when   @Autowired other bean into  bean with @ActivityImpl @Component, this.workerFactory.start(); gonna start workers so that could not auto register other workers

https://github.com/temporalio/sdk-java/blob/71bc122eff4af3a1f53d98fab27d5d7a7f231d92/temporal-spring-boot-autoconfigure-alpha/src/main/java/io/temporal/spring/boot/autoconfigure/RootNamespaceAutoConfiguration.java#L152

duplicate of https://github.com/temporalio/sdk-java/issues/1597


zoom links uses document.write, removes schoology header

Currently, the zoomlinks schoology page has no header making it hard to navigate out of it. In addition, it never fully loads, but instead keeps playing the blue loading circle.

Fixed in https://github.com/Harker-Hackers/schoology-extension/commit/73162810193b0c01142c4fb24c80e8a1e901d2d5.


docs: get-url/ls-url/import-url: document --fs-config

Fixes #4824
Also adjusted metavars on dvc side https://github.com/iterative/dvc/pull/10047

Just reading this for the first time (the whole feature itself). Sorry, if some of this was discussed before:

fs-config (why is it called fs if then saying it is a remote config in the descritpion)?
config options for the target url. - I'm not sure what is the scenario / why users would need this? what is the target URL - remote URL, git repo URL? can we just describe how will it be used -  just plain english is fine :)

fs-config (why is it called fs if then saying it is a remote config in the descritpion)?

fs concept is not documented everywhere, we've been hesitating to do that for a long time, but remote concept is clearly not suitable there, so it feels like it is time to embrace it. Though properly documenting it is a separate task, but at least fs is not completely foreign to people (dask, fsspec, huggingface, dvcfs, etc). config term is reserved for dvc config, remote config for remote configs, so fs-config was the logical choice that was left.

I'm not sure what is the scenario / why users would need this? what is the target URL - remote URL, git repo URL? can we just describe how will it be used - just plain english is fine :)

This is import-url, not import, so no git repos here or even remote, just some url. The scenario is supplying additional config options (aws profile, azure account name, etc) for urls, something that people explicitly asked for in the past.


Fix Typo in CSS Flexbox Project Step 21

Describe the Issue
In Step 21, we have a typo here

The correction should be also known as. NOT also knowns as
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-css-flexbox-by-building-a-photo-gallery/step-21
Current Directions with Typo
Your images need some space between them.
The gap CSS shorthand property sets the gaps, also knowns as gutters, between rows and columns. The gap property and its row-gap and column-gap sub-properties provide this functionality for flex, grid, and multi-column layout. You apply the property to the container element.
Give your .gallery flex container a gap property with 16px as the value.
New directions with typo fixed
Your images need some space between them.
The gap CSS shorthand property sets the gaps, also known as gutters, between rows and columns. The gap property and its row-gap and column-gap sub-properties provide this functionality for flex, grid, and multi-column layout. You apply the property to the container element.
Give your .gallery flex container a gap property with 16px as the value.
How to Contribute
Here is the file you will need to update
https://raw.githubusercontent.com/freeCodeCamp/freeCodeCamp/main/curriculum/challenges/english/14-responsive-web-design-22/learn-css-flexbox-by-building-a-photo-gallery/6153a04847abee57a3a406ac.md
This will be open to first time contributors only
Please read through the contributing docs before working on this issue
Any issue marked with help wanted or first timers only label does not need assignment, and is free to work on.
The maintainers will prioritize pull requests for those who follow the directions of the contributing docs and properly fill out the pull request template.
If you need any help, please reach out on the freeCodeCamp forum or discord
Happy Contributing 💯

You must have a great eye for English before you can see that typo. That's good tho.


Tests Fixtures Update

This PR does 2 things,

it moves the user_account permissions which were originally separated from the rest of the code in the django_server into the user_account directory / keeping all the user account code together
it replaces the created database entries for each test by loading the entries in the fixtures instead. This lead to a few tests being removed as they were included within another test after simplifying

Codecov Report

Merging #181 (de5d1f6) into master (3a411e0) will increase coverage by 0.21%.
The diff coverage is 94.28%.

@@            Coverage Diff             @@
##           master     #181      +/-   ##
==========================================
+ Coverage   97.28%   97.50%   +0.21%
==========================================
  Files          19       20       +1
  Lines         516      520       +4
==========================================
+ Hits          502      507       +5
+ Misses         14       13       -1

Impacted Files
Coverage Δ

server/user_account/permissions.py
91.30% <91.30%> (ø)

server/organization/permissions.py
100.00% <100.00%> (ø)

server/user_account/tests.py
100.00% <100.00%> (+4.10%)
:arrow_up:

server/user_account/views.py
91.20% <100.00%> (-2.20%)
:arrow_down:

server/user_account/models.py
96.15% <0.00%> (-3.85%)
:arrow_down:

Continue to review full report at Codecov.

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


For input checkboxes v-model with additional @input event handlers v-model is not set to true-value and false-value bindings

Version
2.6.11
Reproduction link
https://jsfiddle.net/ventralnet/1ew78c93/2/
Steps to reproduce
Open up the console, click the checkbox. You will see two console messages logged on @input event.  this.selectedValue is logged immediately and it is blank, then in the next event loop it properly has true-value
Is this a bug?
What is expected?
this.selectedValue is set to true-value attribute
What is actually happening?
this.selectedValue is blank until next event loop

This is due to timing difference between the input and change (the one you should be listening to) events. This is something you should avoid relying on as browsers could behave differently. The event that should be used for a checkbox is change though. Using that events works fine


It is recommended to define a method to set the default value to be called when a new modification is added.

It is recommended to define a method to set the default value to set the default value of the field of the entity base class when adding/bulk new modification/batch modification, such as Id, CreateTime, Version, CreateUserId, UpdateUserId, etc. This function is very common, which is in The code in my project, the author can add it to the project.
        public async Task<long> InsertAsync(TEntity instance)
        {
            **var id = DapperActionFilterExecuter.ExecuteCreationAuditFilter<TEntity, TPrimaryKey>(instance);**
            await base.InsertAsync(instance, _unitofWork.DbTransaction);
            return id;
        }

        public override async Task<bool> UpdateAsync(TEntity instance, params Expression<Func<TEntity, object>>[] includes)
        {
            **DapperActionFilterExecuter.ExecuteModificationAuditFilter<TEntity, TPrimaryKey>(instance);**
            _unitofWork.BeginTransaction();
            var isSuccess = await base.UpdateAsync(w => w.Version == instance.Version, instance, _unitofWork.DbTransaction);
            instance.Version = Guid.NewGuid().ToString();
            await base.UpdateAsync(instance, _unitofWork.DbTransaction, i => i.Version);
            _unitofWork.Commit();
            return isSuccess;
        }

public long ExecuteCreationAuditFilter<TEntity, TPrimaryKey>(TEntity entity) where TEntity : class, IEntity<TPrimaryKey>
        {
            var id = CheckAndSetId(entity);

            var entityWithCreationTime = entity as IHasCreationTime;
            if (entityWithCreationTime != null)
            {
                if (!entityWithCreationTime.TimeCreated.HasValue)
                {
                    entityWithCreationTime.TimeCreated = DateTime.Now;
                }
            }

            var record = entity as ICreationAudited;
            if (record != null)
            {
                if (record.UserIdCreated == null)
                {
                    record.UserIdCreated = GetUserId();
                }
            }

            if (entity is IHasModificationTime)
            {
                (entity as IHasModificationTime).TimeLastMod = DateTime.Now;
            }

            if (entity is IModificationAudited)
            {
                (entity as IModificationAudited).UserIdLastMod = GetUserId();
            }
            return id;
        }

public void ExecuteModificationAuditFilter<TEntity, TPrimaryKey>(TEntity entity) where TEntity : class, IEntity<TPrimaryKey>
        {
            if (entity is IModificationAudited)
            {
                (entity as IModificationAudited).UserIdLastMod = GetUserId();
            }

            if (entity is IHasModificationTime)
            {
                (entity as IHasModificationTime).TimeLastMod = DateTime.Now;
            }

            //if (entity is IEntity<long>)
            //{
            //    (entity as IEntity<long>).Version = Guid.NewGuid().ToString();
            //}
        }

It is not for Core. We can continue discuss here: #199


Removing cookies messes up with flash

The notices/alerts I set in the flash are somehow persisted across requests when cookies are unset, whether it's through use of a before_filter or rack middleware. The result is that the same message is shown over and over, until the user reaches an action that doesn't remove cookies.

I went around the issue by adding if flash.empty to request.session_options[:skip] = true in both controller methods. That way the browser receives the instruction to clear the flash from the cookie. This has the double advantage of setting a Set-Cookie header, which tells fastly uses to not cache the page. End result: pages with a flash are not cached, yeay!
If anybody has any cleaner way of dealing with this, please let me know!

When we tried to use @blarralde's approach, we ran into a problem where the cache would repeatedly miss after a flash message was shown. We solved the whole issue using AJAX, but ESI could accomplish something similar. Our approach was inspired by this article: https://www.fastly.com/blog/caching-uncacheable-csrf-security
We extracted the flashes to an api endpoint called /flashes.json and built a controller for it.
# app/controllers/flashes_controller.rb
class FlashesController < ApplicationController
  def index
    f = flash.to_json
    flash.clear # to prevent duplicate flashes
    respond_to do |format|
      format.json do
        render json: f
      end
    end
  end
end

# in app/views/shared/_ajax_flashes.html.erb
$ ->
  $.get("/flashes.json", (messages) ->
    return if not messages
    messages.forEach (message) ->
      displayMessage(message)
  )

# in app/views/layouts/application.html.erb
<%= render 'shared/ajax_flashes' %>

We then created a request setting in Fastly to always pass on /flashes.json, and the issue was solved!


New CocoaPods release from master

Is there any chance that you could make a new CocoaPod release from master?
I am particularly keen to get the changes from https://github.com/acmacalister/jetfire/pull/36 :smile:

Done! 0.1.5 should be in Cocoapods now.

Great, thanks @acmacalister 😄


@azure/identity Node.js 20 compatibility

Package Name: @azure/identity
Package Version: 3.3.0
Operating system: macOS
[x] nodejs

version: v20.6.1

[ ] browser

name/version:

[x] typescript

version: 5.2.2

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
I am using Node.js 20 and yarn for my project and tried to install<EMAIL_ADDRESS>but<EMAIL_ADDRESS>uses<EMAIL_ADDRESS>as one of its dependencies and<EMAIL_ADDRESS>is not compatible with Node.js 20.
I noticed that<EMAIL_ADDRESS>supports Node.js 20, so is it possible to bump @azure/msal-node version to 2.1.0?
To Reproduce
Steps to reproduce the behavior:
1.
$ yarn install
yarn install v1.22.19
[1/4] 🔍  Resolving packages...
[2/4] 🚚  Fetching packages...
error<EMAIL_ADDRESS>The engine "node" is incompatible with this module. Expected version "10 || 12 || 14 || 16 || 18". Got "20.6.1"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

Expected behavior
Works with Node.js 20
Screenshots
N/A
Additional context
N/A

Duplicate of https://github.com/Azure/azure-sdk-for-js/issues/26886

@timfbfbfbfb To provide an update for you, we are already working on updating the msal-node version. The node 20 compatibility will be available with the next GA version of @azure/identity, which will be a month from now.


Replace Build.VERSION.SDK_INT with appBuildConfig.sdkInt

Task/Issue URL: https://app.asana.com/0/276630244458377/1201641519620953/f
Description
Replace the usage of Build.VERSION.SDK_INT with appBuildConfig.sdkInt.
Incidentally, also:

Remove some lint warnings related to SDK_INT
Fixed the DuckDuckGoAppLinksHandlerTest that were not testing anything because the if in all tests was never taken :facepalm:
Moved the HttpsEmbeddedDataIntegrationTest.kt to androidTestPlay test because otherwise android tests could not be run when internal flavor was configured.

Steps to test this PR

[ ] all tests pass
[ ] smoke tests

@marcosholgado this should be ready to review


[P2] Can we show an [X] red badge on sites that are not allowed?

For sites where the extension is disallowed (e.g. internal / incompatible sites), it would be useful to show the [x] red badge to indicate that the extension is unable to deliver anything.
P2 / Nice to have.

As content scripts are silently banned in the case the enterprise policy disallows it, I cannot find a good way to detect the case.


feat: Update dependencies

:tada: This PR is included in version 1.0.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Automatically apply changes

From http://code.google.com/p/ndmitchell/issues/detail?id=134
Recently I got GHC 6.10 working, and I could at last run hlint for myself.
I ran it over the Darcs repository and got lots of suggetions back:
find \( -name \*.lhs -o -name \*.hs \) -exec hlint {} +

It would be really cool if rather than just reporting a change, hlint
would actually integrate into darcs and MAKE the change, then record
it with predictable commit metadata.  For example, it would apply one
change and then run
darcs record --author hlint -m 'Eta reduction'

Which would then let me choose whether I agreed with the hint (by
picking "y" when darcs prompts me), or wanted to ignore that hint (by
picking "n" when darcs prompts me).
Or, it could just ALWAYS record the hints, and then I would choose
which hints to "darcs pull" into my main repository.
Right now I'm not sure how to build this feature as a wrapper around
hlint's existing text output, because that would require a lot of
parsing of that output.  That's why I'm suggesting it for including in
hlint itself.
Jan 14, 2009 Delete comment #1 gwern0
Perhaps the best way would be to emit XML - something like -id a = a\n+\n?
So a hypothetical tool would call hlint with --xml-output, filter out changes based
on the type (perhaps the user doesn't want to automatically make eta-reductions even
if he's willing to see them in the normal output), and then runs 'patch' on each
file/diff pair, and then runs 'darcs record --author hlint -m (file ++ type) file' etc.
(Or if not XML, then perhaps via Read/Show, if it doesn't mangle the diff information.)
Jan 14, 2009 Delete comment #2 trentbuck
I have another idea about how to implement this.
Firstly, there needs to be a way to list all the hint names that have
any matches (but not listing what the suggestions are):
$ hlint --list-hints
Eta reduce
Use a string literal

Then there needs to be a way to actually modify the files, rather than
listing the hints, and a way to only apply one kind of hint:
$ cksum x.hs
1 x.hs
$ hlint --in-place --hint 'Eta reduce' *.hs
$ cksum x.hs
2 x.hs

Then the wrapper merely needs to iterate over the list:
$ hlint --list-hints |
  while read hint
  do  hlint --in-place --hint "$hint" *.hs
      darcs record --author hlint -m "$hint."
  done

Jan 15, 2009 Delete comment Project Member #3 ndmitchell
The idea of integrating with darcs is fantatic, and a great way to do it. However, I
think the underlying issue is to get HLint to apply changes to a source file. I don't
have too much interest in doing that personally, since I worry the hints may be
wrong, I don't have perfect source code info and comments etc have been deleted from
the AST. I'd accept a patch :-)
The wrapper around that should be fairly easy to do.
Summary: Automatically apply changes
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-Low Proj-HLint
Jan 15, 2009 Delete comment #4 trentbuck

I think the underlying issue is to get HLint to apply changes

Agreed.

I don't have too much interest in doing that personally, since I
worry the hints may be wrong,

That's not a problem, because the bad hints can be reverted by the
VCS.

I don't have perfect source code info and comments etc have been
deleted from the AST.

That is indeed more of a problem, since the editing the file should
only change the bit the hint refers to, rather than rewriting the
whole file.

I'd accept a patch :-)

Ah, that will probably have to wait quite a while, as this is not
something I have experience with and I have lots of other stuff to
work on.  :-(
I guess the first step might be to get hlint to emit a standard patch
notation such as that accepted by patch(1) or ed(1); then the actual
editing of the file could be handed off to such a utility.  Guessing
how the parsing library you're using works, though, I suspect that's
going to be nearly as difficult...
Jan 15, 2009 Delete comment Project Member #5 ndmitchell
(No comment was entered for this change.)
Status: Accepted
Sep 2, 2011 Delete comment Project Member #6 ndmitchell
More people are asking for this feature, and in conjunction with the HLint API, it could prove quite useful. I think it could be done on top of the general purpose function:
reconstructModule :: String -> Module () -> String
Where parseModule (reconstructModule s m) == m, but using as much lexical information (comments/positions) from s as possible. With that function you probably just then want an --interactive switch to HLint and do darcs style step through. Once you apply one hint, you should purge all hints and try again.
Labels: -Priority-Low Priority-Medium
Sep 2, 2011 Delete comment Project Member #7 ndmitchell
Talking to Malcolm, we came up with the "trick" that makes this feasible. You take Module (), pretty print it, parse it using the exact parser to give Module SrcSpan, then you try and merge information from the original tree over to the second tree. That way you aren't too dependent on the details of HSE's SrcSpanInfo's.
Mar 24, 2012 Delete comment #8 michaelroger
For reference, JPMoresmau EclipseFP (hosted on github) does take HLint suggestions and apply them to source code, using (I assume) Eclipse/Java APIs to edit source code. I doubt there's much code reuse opportunity from there to here, but it might be helpful to look at EclipseFP as an example implementation and also as a potential use case for this feature.

I have a mostly working wip implementation of this here. It currently only works with a unreleased version of HSE and GHC 7.10.2.

Covered by #504

We can definitely do this for most hints now.


Fix more-dropdown

Thanks for contributing! 🍄

LINKED ISSUES: none

TEST URLS: https://github.com/sindresorhus/refined-github

SCREENSHOT: -

@fregante Maybe we should use [data-ga-click] ?

@fregante maybe we should use [data-ga-click]?

If it works with a simple id change in this case I don’t see why complicate it with ga attributes.

I was thinking of enterprise when they change it we may have to start supporting 2 selectors

@fregante Maybe we should use [data-ga-click] ?

We could use nav [data-content="Security"] like I did here https://github.com/sindresorhus/refined-github/pull/3795/files#r537577913, but in this case it would imply rewriting part of the feature — which is why I chose the simpler option of just updating the classes.
I think we can wait and see, if it breaks again then a partial rewrite will be warranted.

Good point, I think however that GHE uses the older part of this feature, which is unaffected.
I think, even if it breaks, it will just result in the 2 tabs not being hidden, which isn’t a big deal.

What I’m interested in at the moment is understanding why they made this change and how this affects us. Are they using the number for ordering? Should we use it too? Did they solve our old https://github.com/sindresorhus/refined-github/issues/3347 issue?

@fregante can we merge this today so it can ship. However we should open an issue to investigate the ordering.


Generic webhook notification

Is your feature request related to a problem? Please describe.
As the administrator of an application in the local travel space, I want camply to send a webhook (HTTP/POST) to a configuration-specified URL with configuration-specified headers and a JSON representation of the campsite so my application can programmatically handle the notification.
Describe the solution you'd like
When WEBHOOK_URL, WEBHOOK_HEADERS, and --notifications webhook are specified, the notification should be POST'ed to the WEBHOOK_URL with the WEBHOOK_HEADERS. The request body should be JSON with machine-readable fields (example, dates should be epoch timestamps or other common time format).
Something like:
{
  "campsite": {
    "campsite_id": "1234",
    "booking_date":<PHONE_NUMBER>,
    "booking_url": "http://blahblah",
    "...": "etc"
  }
}

and
WEBHOOK_URL="http://foobar" \
  WEBHOOK_HEADERS=$(echo '{"Content-Type": "application/json", "x-api-key": "super-secret"}' | base64) \
  camply campsites --rec-area 12345

Describe alternatives you've considered
The open Discord PR is close, as is Slack, but the fields are human friendly or formatted, not generically machine-readable.
Additional context
If this sounds reasonable to you, I am willing to contribute a PR.

Oh this is a great idea. I took a whack at this, let me know your thoughts @zdwolfe : https://github.com/juftin/camply/blob/5c747a1cacfe44ae04f8c6108d81853bcec173ce/docs/command_line_usage.md#send-a-webhook-notification

Pydantic made this super easy to dump the camply.containers.AvailableCampsite objects into JSON: https://github.com/juftin/camply/blob/5c747a1cacfe44ae04f8c6108d81853bcec173ce/camply/notifications/webhook.py#L43-L59

Thanks! I left minor feedback in https://github.com/juftin/camply/pull/296#pullrequestreview-1623326944

Love this idea!

:tada: This issue has been resolved in version 0.30.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

Here's the relevant documenation: https://juftin.com/camply/command_line_usage/#send-a-webhook-notification


This is now a clearer and better text, as far as I can see...

Although for C++ teams it might be important to inform _HAS_EXCEPTIONS value depends on _KERNEL_MODE and is best left to compile-time machinery to decide on its value when using the /kernel mode:
// <vcruntime.h> near to #100

#ifndef _HAS_EXCEPTIONS // Predefine as 0 to disable exceptions
    #ifdef _KERNEL_MODE
        #define _HAS_EXCEPTIONS 0
    #else
        #define _HAS_EXCEPTIONS 1
    #endif // _KERNEL_MODE
#endif // _HAS_EXCEPTIONS

HTH
[Enter feedback here]

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

ID: 6ab4cd21-5a26-620f-9bd4-403a02b2b915
Version Independent ID: 41e125c3-a7bc-69cb-7142-43974c5fe519
Content: /kernel (Create kernel mode binary)
Content Source: docs/build/reference/kernel-create-kernel-mode-binary.md
Product: visual-cpp
Technology: cpp-tools
GitHub Login: @corob-msft
Microsoft Alias: corob

Thank you for your patience as we work through our backlog. We have received more issues than we can manage at this time. We are closing issues based on their severity, how broadly applicable they are, and whether there is enough information provided to act on the issue. If you believe that this issue should not have been closed, please add a comment explaining why by 3/8/2024. We appreciate your contribution and encourage you to propose changes using public PRs, or submit feedback through our updated control at the top of the page, as we phase out Git Issues for this repo. For more information on our feedback systems, see https://aka.ms/ContentUserFeedback.


Seems there is a dub conflict with this package.

See for instance:
https://buildkite.com/dlang/druntime/builds/3254#175779b7-d453-438c-98f0-350a1ad3889e/115

Obviously, it is a dub bug. I got a set of similar bug reports each month.

The bug should be fixed. If rerunning the build wouldn't help, please reopen the issue.


Improve HDRI  performance

Is your feature request related to a problem? Please describe.
HDRI performance, while improved in F3D 2.1.0 could be improved further.
There may be more optimized way to read the EXR than the current implementation.

@Meakk feel free to complete

Here is the context:

https://discord.com/channels/1046005690809978911/1124206665487634513/1124351584311193600


traceback

Describe the bug
When reporting an event, we got this error from the backend
web_1     | "POST /api/ohshown-events HTTP/1.0" 500 19579
web_1     | Internal Server Error: /api/ohshown-events
web_1     | Traceback (most recent call last):
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner
web_1     |     response = get_response(request)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/base.py", line 115, in _get_response
web_1     |     response = self.process_exception_by_middleware(e, request)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/base.py", line 113, in _get_response
web_1     |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
web_1     |     return view_func(*args, **kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/views/generic/base.py", line 71, in view
web_1     |     return self.dispatch(request, *args, **kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 495, in dispatch
web_1     |     response = self.handle_exception(exc)
web_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 455, in handle_exception
web_1     |     self.raise_uncaught_exception(exc)
web_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 492, in dispatch
web_1     |     response = handler(request, *args, **kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/decorators.py", line 55, in handler
web_1     |     return func(*args, **kwargs)
web_1     |   File "/Disfactory/api/views/factories_cr.py", line 251, in get_nearby_or_create_ohshown_events
web_1     |     return _handle_create_ohshown_events(request)
web_1     |   File "/Disfactory/api/views/factories_cr.py", line 181, in _handle_create_ohshown_events
web_1     |     new_ohshown_event = OhshownEvent.objects.create(**new_ohshown_event_field)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/manager.py", line 82, in manager_method
web_1     |     return getattr(self.get_queryset(), name)(*args, **kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/query.py", line 422, in create
web_1     |     obj.save(force_insert=True, using=self.db)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 741, in save
web_1     |     force_update=force_update, update_fields=update_fields)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 779, in save_base
web_1     |     force_update, using, update_fields,
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 870, in _save_table
web_1     |     result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 908, in _do_insert
web_1     |     using=using, raw=raw)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/manager.py", line 82, in manager_method
web_1     |     return getattr(self.get_queryset(), name)(*args, **kwargs)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/query.py", line 1186, in _insert
web_1     |     return query.get_compiler(using=using).execute_sql(return_id)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1374, in execute_sql
web_1     |     for sql, params in self.as_sql():
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1318, in as_sql
web_1     |     for obj in self.query.objs
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1318, in <listcomp>
web_1     |     for obj in self.query.objs
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1317, in <listcomp>
web_1     |     [self.prepare_value(field, self.pre_save_val(field, obj)) for field in fields]
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/sql/compiler.py", line 1258, in prepare_value
web_1     |     value = field.get_db_prep_save(value, connection=self.connection)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py", line 793, in get_db_prep_save
web_1     |     return self.get_db_prep_value(value, connection=connection, prepared=False)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/contrib/postgres/fields/array.py", line 91, in get_db_prep_value
web_1     |     return [self.base_field.get_db_prep_value(i, connection, prepared=False) for i in value]
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/contrib/postgres/fields/array.py", line 91, in <listcomp>
web_1     |     return [self.base_field.get_db_prep_value(i, connection, prepared=False) for i in value]
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py", line 788, in get_db_prep_value
web_1     |     value = self.get_prep_value(value)
web_1     |   File "/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py", line 1825, in get_prep_value
web_1     |     return int(value)
web_1     | ValueError: invalid literal for int() with base 10: '森林(天然林)'
web_1     | "POST /api/ohshown-events HTTP/1.0" 500 19579

To Reproduce
Steps to reproduce the behavior:

Checkout to the latest code base 4a1dbd9281c583564227406f5bab17c64c32e26a
Deploy the code from scratch
Report as usual

Expected behavior
OHSHOWN event is reported
Desktop (please complete the following information):

OS: ubuntu-mate 20.04
Browser: firefox
Version 109.0.1 (64-bit)

Additional context
No error was found during deployment.

My educated guess is that the encoding systems between the development OS ( @Neilxx 's laptop) and the deployment env (ohshown.site, which is a ubuntu).

@tai271828  這個我在 staging 測是正常的
異常狀態是發生在你的 local 嗎？
會是你 local 的前端程式沒有更新嗎？


Change to a secret should trigger ECS service update

Secrets created by https://github.com/docker/ecs-plugin/issues/208 should trigger application update on changes.
We can use https://docs.aws.amazon.com/secretsmanager/latest/apireference/API_UpdateSecret.html to manage changes on secret value. IIUC Secrets Manager will then return the same ARN so there will be no change to the TaskDefinition_Secret, need to experiment to check if CloudFormation will detect need for service update (at least, restart) or if we need to hack into this

Proposed implementation for https://github.com/docker/ecs-plugin/issues/208, using a CloudFormation resource, won't require any specific management to support secret value replacement, as the secret is then part of the Stack.


Losing event in around million events

Hi
we ran a script to do sanity check. though we see some events being missing from json array which led to believe that something in processing records seems to be the problem. Not sure what could be but its very random.
For eg sent 2-3 million events in couple of days but lost around 10-15 events from json payload itself.
Ex: Sending [1,2,3,4,5] in one payload out of random we tend to lose events which are at the end of array for eg 5 here.
Running on msk aws
tasks.max=2
topics.regex=[0-9a-f]{8}{12}
ssl=false
hostname=
database=default
password=
port=80
value.converter.schemas.enable=false
value.converter=org.apache.kafka.connect.json.JsonConverter
exactlyOnce=true
username=clickhouse
schemas.enable=false

Any ideas?
In connector logs
We see missing count for eg we sent 29 in json array but logs mention
[Worker-04c10fb697e20c0fc] [2024-05-03 12:22:46,399] INFO [prod-clickhouse-connector-main|task-0] Trying to insert [9] records  (QueryId: [ac4516f7-772f-4e98-9343-3dfd52d0876c]) (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:161)

I believe this could be happening due to mismatched minoffset and maxoffset from replicated connect stable and zookeeper lag because the last few event are missed out of couple of million events. I am still not sure but find this the probable reason. Looking forward to your ideas :)
https://github.com/ClickHouse/clickhouse-kafka-connect/pull/380
@Paultagoras

I also had rare gaps in data after errors related to offset commit in kafka that occurred when running backups on the kafka cluster, which caused the consumer to start reading messages from the old offset again. Since the size of received messages is not deterministic, this led to the case mentioned in https://github.com/ClickHouse/clickhouse-kafka-connect/pull/291#issuecomment-1954720380. I have to use a custom build with a fix for more than half a year.

I have run the script and processing again with just one node to confirm.
For now i have removed the extra node from load balancer and trying to figure out if this could be a problem with replicated table and offset now i am pushing to one node so zookeeper lag can be eliminated.
@ne1r0n i also believe the offset seems to be causing to remove few events.

Yes this is correct. Its been 3 days and more than 3 million records. I haven't any seen any missing records yet because i removed 2 nodes from 3 node cluster so data is going on only 1 node now. We are processing millions of records every few hours.
Will update if i see problem happening on even 1 node too. Trying to figure out a solution where reading from connect state table is consistent.
Yes the infra is AWS Managed Kafka, Clickhouse Kafka Connector, Clickhouse 3 node cluster behind AWS load balancer with Keeper running for coordination.

It looks like it happened with 1 node too it took millions of event but it did happen after 3.5 millions event.
So either could be combination of keeper lag and the one @ne1r0n mentioned.
@ne1r0n did it happen to you without any broker restart or while kafka is running. I am looking to apply some solution because we are running on production so missing events is a big deal.

Sure its pretty standard but here are the files
python script to generate sample data
https://gist.github.com/abhishekgahlot2/c43d0d4509f6f4fbb536ae910d9b6f9a
sample data json
https://gist.github.com/abhishekgahlot2/01555c6586510bbca90acf9538624487
docker-compose for clickhouse
https://gist.github.com/abhishekgahlot2/568adcafbb04be4ef95425213990a757
config.xml
https://gist.github.com/abhishekgahlot2/879d6d40028b38487018f0af7e48cd57
please let me know if you need anything else.
it would be wise to log records from ProxySinkTask.java file to eliminate other potential problems. i will do that

Another reason i am expecting is having tasks.max = 2
also in KeeperStateProvider.java this could happen in future
 String key = String.format("%s-%d", topic, partition);
 String selectStr = String.format("SELECT * FROM `%s` WHERE `key`= '%s'", csc.getZkDatabase(), key);

in case of new code where virtual topic is allowed. Same keys can be stored in connect_state table with different min and max offset.
for eg i have same two table in different database the keeper state would look liike
tablename-0 | minOffset | maxOffset
I think it should be changed for multiple databases too because in virtual topic only the last part is taken for table key.

Please let me know if I am wrong here. Can the connector have race condition when using tasks.max.

I see thanks in that case shouldn't happen. I have now deployed the new main to production to see if I get overlapping issues after the new code.
I am using this PR: https://github.com/ClickHouse/clickhouse-kafka-connect/pull/389
Once I verify that its working I can add multiple nodes so that zookeeper lag can be figured out too. :)

An update, we still got the missing event issue after adding multiple nodes to load balancer. looks like zookeeper lag is still a problem.
We haven't had any case for 4-5 days with single node but as soon as we added multiple nodes. The events started to miss they are lesser in frequency after main branch is deployed but still there.
cc @Paultagoras

Sharing logs here
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,583] INFO [clickhouse-main-v2|task-0] doLogic - Topic: [2a3c_40db_a06e_e4cf52e5c5b8_metric], Partition: [0], MinOffset: [3449647], MaxOffset: [3449709], Records: [63] (com.clickhouse.kafka.connect.sink.processing.Processing:116)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,660] INFO [clickhouse-main-v2|task-0] write state record: topic 2a3c_40db_a06e_e4cf52e5c5b8_metric partition 0 with BEFORE_PROCESSING state max 3449709 min 3449647 (com.clickhouse.kafka.connect.sink.state.provider.KeeperStateProvider:110)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,661] INFO [clickhouse-main-v2|task-0] Kafka Key in clickhousewrite as kafkakey: metric (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:926)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,661] INFO [clickhouse-main-v2|task-0] ClickHouse URL: http://clickhouse.demo.com:80/2a3c_40db_a06e_e4cf52e5c5b8? (com.clickhouse.kafka.connect.sink.db.helper.ClickHouseHelperClient:93)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,667] INFO [clickhouse-main-v2|task-0] Trying to insert [63] records to table name [`metric`] (QueryId: [dac3f554-a755-4c6f-9c60-5a21a3634418]) (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:183)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,667] INFO [clickhouse-main-v2|task-0] enableDbTopicSplit: true (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:730)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,667] INFO [clickhouse-main-v2|task-0] Kallu - Records: [63], Topic: [metric], Partition: [0], QueryId: [dac3f554-a755-4c6f-9c60-5a21a3634418], enableDbTopicSplit: [true], dbTopicSplitChar: [.] (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:736)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,668] INFO [clickhouse-main-v2|task-0] ClickHouse URL: http://clickhouse.demo.com:80/2a3c_40db_a06e_e4cf52e5c5b8? (com.clickhouse.kafka.connect.sink.db.helper.ClickHouseHelperClient:93)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,706] INFO [clickhouse-main-v2|task-0] Response Summary - Written Bytes: [4475], Written Rows: [63] - (QueryId: [dac3f554-a755-4c6f-9c60-5a21a3634418]) (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:797)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,706] INFO [clickhouse-main-v2|task-0] batchSize: 63 serialization ms: 0 data ms: 1 send ms: 38 (QueryId: [dac3f554-a755-4c6f-9c60-5a21a3634418]) (com.clickhouse.kafka.connect.sink.db.ClickHouseWriter:803)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,738] INFO [clickhouse-main-v2|task-0] write state record: topic 2a3c_40db_a06e_e4cf52e5c5b8_metric partition 0 with AFTER_PROCESSING state max 3449709 min 3449647 (com.clickhouse.kafka.connect.sink.state.provider.KeeperStateProvider:110)
[Worker-04d8f7d3e0ff736a9] Kalu Database: default
[Worker-04d8f7d3e0ff736a9] Kalu Topic: 2a3c_40db_a06e_e4cf52e5c5b8.metric
[Worker-04d8f7d3e0ff736a9] Kalu Split: true
[Worker-04d8f7d3e0ff736a9] Kalu DB Topic Separator Char: .
[Worker-04d8f7d3e0ff736a9] Balu parts [2a3c_40db_a06e_e4cf52e5c5b8, metric]
[Worker-04d8f7d3e0ff736a9] Kalu Stuff: metric2a3c_40db_a06e_e4cf52e5c5b8
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,917] INFO [clickhouse-main-v2|task-0] doLogic - Topic: [2a3c_40db_a06e_e4cf52e5c5b8_metric], Partition: [0], MinOffset: [3449710], MaxOffset: [3449710], Records: [1] (com.clickhouse.kafka.connect.sink.processing.Processing:116)
[Worker-04d8f7d3e0ff736a9] [2024-05-23 04:01:22,936] INFO [clickhouse-main-v2|task-0] write state record: topic 2a3c_40db_a06e_e4cf52e5c5b8_metric partition 0 with BEFORE_PROCESSING state max 3449710 min 3449710 (com.clickhouse.kafka.connect.sink.state.provider.KeeperStateProvider:110)

Before the jump:

MaxOffset: 3449709
MinOffset: 3449647
Timestamp: 2024-05-23 04:01:22,738
After the jump:

MinOffset: 3449710
MaxOffset: 3449710
Timestamp: 2024-05-23 04:01:22,917

@abhishekgahlot2, if the issue still happens, can you ping us on ClickHouse community slack so we can discuss

Thanks @Paultagoras @mzitnik , I will let you guys know on slack. as of now I haven't had issue so far. One instance was probably due to timeout apart from that.
Thanks again in helping here :)


Enable constructing argument with tuple and buffer

@pfultz2 Looks like it needs a unit test with both shape and data to construct an argument.


Error while configure on local

Hello I am trying to setup your code but getting below error
react-native-cli: 2.0.1
react-native: 0.61.4
Building optimized bundles and generating sourcemaps... Using legacy Metro server to bundle your JavaScript code, you may encounter unexpected behavior if your project uses a custom metro.config.js file. Please upgrade your project to Expo SDK 40+. If you experience CLI issues after upgrading, try using the env var EXPO_USE_DEV_SERVER=1. Starting Metro Bundler Building iOS bundle node_modules\react-native\Libraries\Blob\URL.js: Unexpected token punc «:», expected punc «,» in file node_modules\react-native\Libraries\Blob\URL.js at 95:30 node_modules\react-native\Libraries\Blob\URL.js: Unexpected token punc «:», expected punc «,» in file node_modules\react-native\Libraries\Blob\URL.js at 95:30 Error: Unexpected token punc «:», expected punc «,» in file node_modules\react-native\Libraries\Blob\URL.js at 95:30     at C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:430:17     at Generator.next (<anonymous>)     at asyncGeneratorStep (C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:75:24)     at _next (C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:95:9)     at C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:100:7     at new Promise (<anonymous>)     at C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:92:12     at JsTransformer._minifyCode (C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:437:7)     at C:\react-native\amorist_react_native\node_modules\metro\src\JSTransformer\worker.js:367:33 iOS Bundling failed 3591ms Packager URL http://<IP_ADDRESS>:19001/node_modules\expo\AppEntry.bundle?dev=false&hot=false&minify=true&platform=ios returned unexpected code 500. Please open your project in the Expo app and see if there are any errors. Also scroll up and make sure there were no errors or warnings when opening your project.
my package.json

`{
  "main": "node_modules/expo/AppEntry.js",
  "scripts": {
    "start": "expo start",
    "android": "expo start --android",
    "ios": "expo start --ios",
    "web": "expo start --web",
    "eject": "expo eject"
  },
  "dependencies": {
    "@react-native-community/masked-view": "0.1.6",
    "@react-navigation/bottom-tabs": "^5.6.1",
    "@react-navigation/native": "^5.6.1",
    "@react-navigation/stack": "^5.6.2",
    "expo": "~37.0.3",
    "expo-constants": "~9.0.0",
    "expo-image-picker": "~8.1.0",
    "expo-permissions": "~8.1.0",
    "native-base": "^2.13.12",
    "react": "~16.9.0",
    "react-dom": "~16.9.0",
    "react-native": "https://github.com/expo/react-native/archive/sdk-37.0.1.tar.gz",
    "react-native-flash-message": "^0.1.15",
    "react-native-gesture-handler": "^1.6.0",
    "react-native-gifted-chat": "^0.16.3",
    "react-native-popup-menu": "^0.15.9",
    "react-native-reanimated": "^1.7.0",
    "react-native-safe-area-context": "0.7.3",
    "react-native-screens": "~2.2.0",
    "react-native-tab-view": "^2.14.4",
    "react-redux": "^7.2.0",
    "redux": "^4.0.5",
    "redux-thunk": "^2.3.0",
    "socket.io-client": "^2.3.0"
  },
  "devDependencies": {
     "@babel/core": "^7.12.9",
    "babel-preset-expo": "~8.1.0"
  },
  "private": true
}`

Please find the issue here #7 . You need to upgrade expo sdk version.


Corruption when rendering world.

https://map.gustapotown.xyz/?worldname=world&mapname=flat&zoom=6&x=-883&y=64&z=532
You can see here, I stopped rendering as my log was flooded with
[16:57:08] [Dynmap Render Thread/WARN]: [Dynmap] Got statelist of length 320 but expected a length of 342
Hopefully this can be fixed soon as right now this makes the mod un usable :(

Duplicate of #19


Foundation for Emails - new project setup issue

I have just installed the updated foundation-cli via npm and then tried to create a new email project using the recommended command:
foundation new --framework emails
The screen output I get is the following:
JCiMac:Emails a03pl$ foundation new --framework emails
? What's the project called? (no spaces) testinky
      _____
     /     \
    /=   . |
___| ==  | /    Thanks for using ZURB Foundation for Emails!

/   | ===  \     -------------------------------------------
|   | ====  |    Let's set up a new project.
//| =====  \   It shouldn't take more than a minute.
(   __   )
(O)  (O)
(   ,  )
__/
Downloading the project template...
Done downloading!
Installing dependencies...
events.js:85
throw er; // Unhandled 'error' event
^
Error: read ECONNRESET
at exports._errnoException (util.js:746:11)
at TCP.onread (net.js:559:26)
JCiMac:Emails a03pl$
Could you pleaser let me know how to fix this as right now I dont know if my email project is scaffolded correctly.
Thanks

Looks like npm ran into a network error while trying to install. Are you behind a firewall or proxy or anything like that?
Also, could you let us know what OS you use, and what versions of Node (node -v) and npm (npm -v) you use? Thanks :)

Hi,
My machine is wired-connected to the household router and we are not behind a software firewall or use a proxy server of any sort.
My versions of node and npm are as follows:
node -v                               v0.12.2
npm -v                                2.7.4
My machine is running OS/X El Capitan   version 10.11.3
Thanks
On 23 Mar 2016, at 22:39, Geoff Kimball<EMAIL_ADDRESS>wrote:
Looks like npm ran into a network error while trying to install. Are you behind a firewall or proxy or anything like that?
Also, could you let us know what OS you use, and what versions of Node (node -v) and npm (npm -v) you use? Thanks :)
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/zurb/foundation-emails/issues/275#issuecomment-200571218

A project folder was still created, it just didn't finish getting set up. Could you open the folder in your command line, run npm install, and tell us what happens?

What I get are a whole load of unmet dependencies:
JCiMac:testinky a03pl$ npm install
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp/node_modules/undertaker/node_modules/last-run requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp/node_modules/undertaker/node_modules/es6-weak-map,
npm WARN unmet dependency which is version 2.0.1
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp/node_modules/vinyl-fs/node_modules/glob-stream requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp/node_modules/vinyl-fs/node_modules/through2,
npm WARN unmet dependency which is version 2.0.1
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-htmlmin/node_modules/gulp-util/node_modules/dateformat/node_modules/meow requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-htmlmin/node_modules/gulp-util/node_modules/object-assign,
npm WARN unmet dependency which is version 3.0.0
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-htmlmin/node_modules/gulp-util/node_modules/multipipe/node_modules/duplexer2 requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-htmlmin/node_modules/readable-stream,
npm WARN unmet dependency which is version 2.0.6
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-imagemin/node_modules/gulp-util/node_modules/dateformat/node_modules/meow requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-imagemin/node_modules/gulp-util/node_modules/object-assign,
npm WARN unmet dependency which is version 3.0.0
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-inject-string/node_modules/gulp-util/node_modules/dateformat/node_modules/meow requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-inject-string/node_modules/gulp-util/node_modules/object-assign,
npm WARN unmet dependency which is version 3.0.0
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-inline-css/node_modules/gulp-util/node_modules/dateformat/node_modules/meow requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-inline-css/node_modules/gulp-util/node_modules/object-assign,
npm WARN unmet dependency which is version 3.0.0
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-load-plugins/node_modules/gulp-util/node_modules/dateformat/node_modules/meow requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-load-plugins/node_modules/gulp-util/node_modules/object-assign,
npm WARN unmet dependency which is version 3.0.0
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-sass/node_modules/gulp-util requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/gulp-sass/node_modules/object-assign,
npm WARN unmet dependency which is version 4.0.1
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/inky/node_modules/vinyl-fs/node_modules/glob-stream requires<EMAIL_ADDRESS>but will load
npm WARN unmet dependency /Users/a03pl/Sites/Foundation/Emails/testinky/node_modules/inky/node_modules/through2,
npm WARN unmet dependency which is version 2.0.1
JCiMac:testinky a03pl$
On 23 Mar 2016, at 22:56, Geoff Kimball<EMAIL_ADDRESS>wrote:
A project folder was still created, it just didn't finish getting set up. Could you open the folder in your command line, run npm install, and tell us what happens?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/zurb/foundation-emails/issues/275#issuecomment-200574813

Just managed to get this to work on the 4th time of trying so at least I can get testing now! I'll raise this again if the issue occurs again.
P.S.  No external software updates applied during the life of this issue.

Hmm, we'll keep an out for more reports of it - glad you got it!


Allow access to multi-tenant mongo server

While testing https://github.com/Microsoft/vscode/issues/25332
When you run your database on mlab you get a multi-tenant setup. I.e. a single mongo server has multiple databases but you are only allowed to access the db you own.
The mongoshell handles that case without issues:
 mongo mongodb://<dbuser>:<dbpassword>@<server>.mlab.com:55574/<database> works without issues.
When I try to use such a connection string the in extension I get an error since the extension tries to list the databases:
not authorized on admin to execute command { listDatabases: 1 }: MongoError: not authorized on admin to execute command { listDatabases: 1 }

Hi,
I've downloaded the extension and tried to connect to a mlab DB and got the same error.
Any hint about when is this gonna be fixed?

Hi,
I get the exact same error connecting to a locally-hosted, multi-tenant database.
Thanks in advance!

Same issue connecting to mlab db

Any progress on this?

This extension is not being actively developed anymore, all of our efforts have moved to the cosmosdb extension: https://marketplace.visualstudio.com/items?itemName=ms-azuretools.vscode-cosmosdb


Add option to disable --document-private-items for targets that have it by default

Building a documentation for a Rust project - which at this stage is just a single crate with single binary target - I've discovered that all private items are included. Running -vv helped me discover that Cargo has ran rustdoc with flag --document-private-items, even though I haven't passed it to cargo. There is no option cargo doc --no-document-private-items either.
I would like to have such option.

Thanks for the report!  Closing as a duplicate of #7963.


Performance Issues

I'm seeing drastic performance difference when using gulp-concat-css over gulp-concat

15-30s to build 600kb file on quad-core versus with gulp-concat-css
1-2s with gulp-concat

I know rework is necessary, but perhaps there's some room for performance gains?
https://github.com/Semantic-Org/Semantic-UI/issues/1464

May I know some more details about your sources? How many files are you concatenating? What's the typical dependency depth?

Concatenating about 50 files, not inlining any @import rules

Version 1.2.0 includes a performance fix (20% improvement). Please, let me know what's the improvement for you project.


Through tables for grandchildren with relationships

fixes #463

tests pass (the two failing ones are because I'm running postgres and redis on non-default ports):


Fix README typo

typo right?
PS - I'm actually enjoying reading the readme, you've thought of a lot of stuff. :+1: Of course there may be more typos I haven't gone through everything to check.

I have no idea how this happened. I could swear I just copied and pasted from my terminal.


1.2.9 Set up Recovery Phrase/Walkthrough 4

1. User Story
Info about what the recovery phrase is and why it is important.
2. Acceptance Criteria
None
3. Related Issues

[ ] Create all related issue for this issue, label them, assign them, and move them to the right projects.
[ ] tbd
[ ] tbd

4. Wire Frame
None
5. Design

https://www.figma.com/file/ERJ7bYyBWdqJw1ID9nZJ2N/Designs?node-id=7904%3A10797

button component / progress animation


RangeError: Invalid time value for some EPUBs (date/time metadata)

RangeError: Invalid time value
Deprecation warning: value provided is not in a recognized RFC2822 or ISO format. moment construction falls back to js Date(), which is not reliable across all browsers and versions. Non RFC2822/ISO date formats are discouraged and will be removed in an upcoming major release. Please refer to http://momentjs.com/guides/#/warnings/js-date/ for more info.
Arguments:
[0] _isAMomentObject: true, _isUTC: false, _useUTC: false, _l: undefined, _i: --T00:00:00Z, _f: undefined, _strict: undefined, _locale: [object Object]
Error
    at Function.createFromInputFallback (r2-testapp-js/node_modules/moment/moment.js:324:94)
    at configFromString (r2-testapp-js/node_modules/moment/moment.js:2366:11)
    at configFromInput (r2-testapp-js/node_modules/moment/moment.js:2592:9)
    at prepareConfig (r2-testapp-js/node_modules/moment/moment.js:2575:9)
    at createFromConfig (r2-testapp-js/node_modules/moment/moment.js:2542:40)
    at createLocalOrUTC (r2-testapp-js/node_modules/moment/moment.js:2629:12)
    at createLocal (r2-testapp-js/node_modules/moment/moment.js:2633:12)
    at hooks (r2-testapp-js/node_modules/moment/moment.js:16:25)
    at fillPublicationDate (r2-testapp-js/node_modules/r2-shared-js/dist/es8-es2017/src/parser/epub.js:624:52)
    at Object.EpubParsePromise (r2-testapp-js/node_modules/r2-shared-js/dist/es8-es2017/src/parser/epub.js:332:5)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)

TODO: daniel to check parsing code in r2-xxx-js, find workaround for invalid metadata format

Yep, <dc:date>--T00:00:00Z</dc:date>

We need a sanity check in the parser code, and ignore broken values. I wonder if EPUB-Check captures these types of errors?

EPUB-Check emits a warning, not an error:
WARNING(OPF-053): xxx.epub/OEBPS/content.opf(12,34): Date value '--T00:00:00Z' does not follow recommended syntax as per http://www.w3.org/TR/NOTE-datetime:[For input string: "-"] is not an integer.

Should be fixed in: https://github.com/edrlab/r2-shared-js/commit/947ddd80462d61d692722df9a545a8281b300b58
(testing with both readium-desktop and r2-testapp-js.

Ah, moment().isValid() does the job (otherwise "empty" invalid object placeholders are created!
https://github.com/edrlab/r2-shared-js/commit/130f9a6d6f5ce464650511448436ff6c3818742d


make sandbox compatible with edit server (multiple notebooks)

Description
Currently --sandbox works great for single notebooks.
However, when you start the edit server, two problems occur:

we have one isolated environment for all the notebooks
the script metadata per PEP 723 is ignored on notebook launch, instead required packages are auto-discovered.

It would be lovely to make sandbox compatibel with this mode, since we can then really utilize the power of UV with it's global cache and users can switch projects easily without worrying about dependencies and environments. Really beginner friendly that way and no terminal needed to create / activate manually managed environments.
Suggested solution
Create some sort of logic that activates and deactivates a uv isolated environment on notebook launch and shutdown from the edit server.
Alternative
No response
Additional context
No response

Oh, interesting... didn't realize this was an issue. So what is actually happening right now when I run marimo edit --sandbox is that packages for all notebooks are using the same env?
And it sounds like this would explain why I am being asked to re-install packages when reopening a notebook?
FWIW, it does seem to be working-ish:
$ marimo edit --sandbox
Running in a sandbox: uv run --isolated --no-project --with-requirements /tmp/tmpz2c8w8sx.txt marimo edit

        Create or edit notebooks in your browser 📝

        URL: http://localhost:2718?access_token=lGtxh53-cAaCqe8l1yQzGA

Resolved 5 packages in 322ms
Prepared 3 packages in 106ms
Installed 3 packages in 20ms
 + charset-normalizer==3.4.0
 + requests==2.32.3
 + urllib3==2.2.3

Resolved 6 packages in 441ms
Prepared 6 packages in 2.62s
Installed 6 packages in 405ms
 + numpy==2.1.2
 + pandas==2.2.3
 + python-dateutil==2.9.0.post0
 + pytz==2024.2
 + six==1.16.0
 + tzdata==2024.2

@gabrielgrant - it does re-install the packages, but since they are cached by uv, it should be pretty fast


Capitol letters not recognized for 144.1 Departmental radio.

Details? I need something more descriptive. Are you using a word recognized by the AI word converter? If so, the entire sentence must be in caps.

No, the department radio sender for the AI on 144.1.
Sending: [e engineering] would send [engineering] to engineering, but instead sends [E engineering] to Common.

Sorry for being so slow. That behavior is intended so it does not eat one-letter words such as "A" or "I". The only other alternative would be to force a symbol beforehand, such as "-a, -e, -c" and such.

I did not notice that the the speech change completely broke the script. I am sorry for not giving this higher priority. It is fixed now though!


Access denied on IE11

Hi,
We have more than 100 error in production environnement when IE11 (Win7 & Win 8.1) calling this function
https://github.com/logentries/le_js/blob/master/src/le.js#L263
I can reproduce in development environment
here the stack trace
Error Access is denied.

    dependencies.25681075a.js:61500:16 _apiCall
    dependencies.25681075a.js:61447:32 send
    dependencies.25681075a.js:61408:12 _rawLog
    dependencies.25681075a.js:61548:16 _log
    dependencies.25681075a.js:61562:16 error
    dependencies.25681075a.js:61630:16 error
    dependencies.25681075a.js:78770:24 d[e]
    dependencies.25681075a.js:83907:16 Anonymous function
    dependencies.25681075a.js:23431:14 k.prototype.$broadcast
    dependencies.25681075a.js:57014:8 Anonymous function

Hi @lcalvy, isn't it related to security zones? IE supports various setting which may limit access to external resources...

It's used demo an internet zone, on ssl domain.

Le 29 juil. 2015 à 13:07, Viliam Holub<EMAIL_ADDRESS>a écrit :
Hi @lcalvy, isn't it related to security zones? IE supports various setting which may limit access to external resources...
—
Reply to this email directly or view it on GitHub.

Sorry, seems that IE11 have a different implementation of CORS and block POST over http when domain is from https. It's a wrong configuration on my side.


[react] query provider component name

the provider.tsx file that gets generated for react and next js projects (which wrap the entire app) has a vague name.
we should consider renaming it to something along the lines of react-query-provider.tsx to make it more clear what the file is used for (including renaming the exported component to match)
Reason:
when looking at the complete list of providers in the app, Provider is not descriptive whereas SolanaProvider and ClusterProvider are

This change will improve code readability and maintainability. Here's how you can do it:
// In react-query-provider.tsx
import React from 'react';
import { QueryClient, QueryClientProvider } from 'react-query';
const queryClient = new QueryClient();
const ReactQueryProvider: React.FC = ({ children }) => {
return {children};
};
export default ReactQueryProvider;