changed mailchimp signup form style to "horizontal"

mailchimp signup merge committed

Thanks, the horizontal style looks better.

Yes :)
On Wed, 29 Jul 2020 at 19:24, Jonah Sullivan<EMAIL_ADDRESS>wrote:

Thanks, the horizontal style looks better.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/foss4g-oceania/foss4g-sotm-oceania-2020/pull/17#issuecomment-665549069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMZ4UARNWY3A67EE6GJMTR57TGRANCNFSM4PLMPPSQ
.

quick comment - it's not clear at all what people are signing up for here. This process should somewhere explain:

what communications a person should expect (related to this conference only, or for this and future conferences)
what we do with data collected (ie your e-mail address will be retained by OSGeo Oceania for FOSS4G SotM conference communication purposes)

thnx, and the website is coming together nicely. Hope the experience of building it is going well ;)

+1 Agree
On Wed, 29 Jul 2020 at 21:02, Adam Steer<EMAIL_ADDRESS>wrote:

quick comment - it's not clear at all what people are signing up for here.
This process should somewhere explain:

what communications a person should expect (related to this
conference only, or for this and future conferences)
what we do with data collected (ie your e-mail address will be
retained by OSGeo Oceania for FOSS4G SotM conference communication purposes)

thnx, and the website is coming together nicely. Hope the experience of
building it is going well ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/foss4g-oceania/foss4g-sotm-oceania-2020/pull/17#issuecomment-665597630,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMZ4QGZHNOTIS5ZJ3UOHTR576SRANCNFSM4PLMPPSQ
.

Hi Jonah,
Just following up on Adam's input. What is the signing up for? Is it for
registering interest in participating in the conference?
cheers,
Edwin
On Thu, 30 Jul 2020 at 10:28, Edwin Liava'a<EMAIL_ADDRESS>wrote:

+1 Agree
On Wed, 29 Jul 2020 at 21:02, Adam Steer<EMAIL_ADDRESS>wrote:

quick comment - it's not clear at all what people are signing up for
here. This process should somewhere explain:

what communications a person should expect (related to this
conference only, or for this and future conferences)
what we do with data collected (ie your e-mail address will be
retained by OSGeo Oceania for FOSS4G SotM conference communication purposes)

thnx, and the website is coming together nicely. Hope the experience of
building it is going well ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/foss4g-oceania/foss4g-sotm-oceania-2020/pull/17#issuecomment-665597630,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMZ4QGZHNOTIS5ZJ3UOHTR576SRANCNFSM4PLMPPSQ
.

Have changed it to "Sign up for event updates...Get updates in time and
more."
cheers,
Edwin
On Sat, 1 Aug 2020 at 21:23, Edwin Liava'a<EMAIL_ADDRESS>wrote:

Hi Jonah,
Just following up on Adam's input. What is the signing up for? Is it for
registering interest in participating in the conference?
cheers,
Edwin
On Thu, 30 Jul 2020 at 10:28, Edwin Liava'a<EMAIL_ADDRESS>wrote:

+1 Agree
On Wed, 29 Jul 2020 at 21:02, Adam Steer<EMAIL_ADDRESS>wrote:

quick comment - it's not clear at all what people are signing up for
here. This process should somewhere explain:

what communications a person should expect (related to this
conference only, or for this and future conferences)
what we do with data collected (ie your e-mail address will be
retained by OSGeo Oceania for FOSS4G SotM conference communication purposes)

thnx, and the website is coming together nicely. Hope the experience of
building it is going well ;)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/foss4g-oceania/foss4g-sotm-oceania-2020/pull/17#issuecomment-665597630,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANMZ4QGZHNOTIS5ZJ3UOHTR576SRANCNFSM4PLMPPSQ
.


ydb.Open(ctx, dsn)

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[x] Refactoring (no functional changes, no api changes)
[x] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #203
What is the new behavior?

Other information

Codecov Report

Merging #204 (561d05d) into master (98938c9) will decrease coverage by 17.76%.
The diff coverage is 41.17%.

@@             Coverage Diff             @@
##           master     #204       +/-   ##
===========================================
- Coverage   57.79%   40.03%   -17.77%
===========================================
  Files         132      137        +5
  Lines       15583    15532       -51
===========================================
- Hits         9006     6218     -2788
- Misses       5985     8852     +2867
+ Partials      592      462      -130

Flag
Coverage Δ

1.14.x
26.79% <27.43%> (-0.40%)
:arrow_down:

1.15.x
26.79% <27.43%> (-0.40%)
:arrow_down:

1.16.x
26.79% <27.43%> (-0.40%)
:arrow_down:

1.17.x
40.03% <41.17%> (-17.77%)
:arrow_down:

1.18.x
26.79% <27.43%> (-0.40%)
:arrow_down:

connection
15.67% <14.47%> (+0.04%)
:arrow_up:

discovery
14.78% <12.21%> (+0.04%)
:arrow_up:

e2e
23.57% <32.12%> (-24.36%)
:arrow_down:

integration
23.57% <32.12%> (-24.36%)
:arrow_down:

macOS-latest
26.79% <27.43%> (-0.40%)
:arrow_down:

ratelimiter
16.92% <30.76%> (+0.15%)
:arrow_up:

scripting
17.28% <14.02%> (-0.09%)
:arrow_down:

table
?

ubuntu-latest
40.03% <41.17%> (-17.77%)
:arrow_down:

unit
26.79% <27.43%> (-0.40%)
:arrow_down:

windows-latest
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

errors.go
14.28% <ø> (-7.15%)
:arrow_down:

internal/backoff/type.go
0.00% <0.00%> (ø)

internal/lazy/scheme.go
22.00% <0.00%> (-56.00%)
:arrow_down:

internal/operation/status.go
0.00% <0.00%> (ø)

internal/wait/wait.go
0.00% <0.00%> (ø)

internal/xerrors/transport.go
36.92% <0.00%> (-29.24%)
:arrow_down:

options.go
63.23% <ø> (-11.77%)
:arrow_down:

retry/backoff.go
0.00% <0.00%> (ø)

retry/context.go
0.00% <ø> (ø)

table/options.go
0.00% <ø> (-50.00%)
:arrow_down:

... and 76 more

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 98938c9...561d05d. Read the comment docs.


[ARM32/ObjWriter] Implementation thumb mode

@dotnet/arm32-corert-contrib please review

@jkotas please review again


Intermittent crashes with OpenJDK 9

I have an issue that is possibly not addressable within Honest Profiler itself and that might be more suitable for the HotSpot development mailing list, but I thought this might be a good place to start and see if anyone can provide insights.
I've been getting intermittent SIGSEGV crashes running (under FreeBSD) the BSD port of OpenJDK 9, roughly equivalent to early access build b117, and a version of Honest Profiler with the commit in my pull request #108 (which fixes a different sort of crash). The new crashes do not occur with OpenJDK 8.
An example trace is as follows:
#17 0x000000173a8c63c1 in VMError::report_and_die (thread=0x0, sig=<error reading variable: Value out of range.>, pc=0x7d0 <error: Cannot access memory at address 0x7d0>, siginfo=0x1738b9593a <_write+10>, context=0x0) at /home/olivier/openjdk9/hotspot/src/share/vm/utilities/vmError.cpp:1071
#18 0x000000173a68889b in JVM_handle_bsd_signal (sig=11, info=0x7fffd605d0b0, ucVoid=0x7fffd605cd40, abort_if_unrecognized=<optimized out>) at /home/olivier/openjdk9/hotspot/src/os_cpu/bsd_x86/vm/os_bsd_x86.cpp:785
#19 0x000000173a6845a9 in signalHandler (sig=11, info=0x7fffd605d0b0, uc=0x7fffd605cd40) at /home/olivier/openjdk9/hotspot/src/os/bsd/vm/os_bsd.cpp:2972
#20 0x0000001738838977 in handle_signal (actp=<optimized out>, sig=11, info=0x7fffd605d0b0, ucp=0x7fffd605cd40) at /usr/src/lib/libthr/thread/thr_sig.c:243
#21 0x000000173883806c in thr_sighandler (sig=11, info=0x7fffd605d0b0, _ucp=0x7fffd605cd40) at /usr/src/lib/libthr/thread/thr_sig.c:188
#22 <signal handler called>
#23 frame::safe_for_sender (this=<optimized out>, thread=0x17b8cc1800) at /home/olivier/openjdk9/hotspot/src/cpu/x86/vm/frame_x86.cpp:135
#24 0x000000173a8674bc in JavaThread::pd_get_top_frame (this=0x17b8cc1800, fr_addr=0x7fffd605d510, ucontext=<optimized out>, isInJava=<optimized out>) at /home/olivier/openjdk9/hotspot/src/os_cpu/bsd_x86/vm/thread_bsd_x86.cpp:77
#25 0x000000173a171412 in AsyncGetCallTrace (trace=0x7fffd605d5b8, depth=128, ucontext=0x7fffd605e240) at /home/olivier/openjdk9/hotspot/src/share/vm/prims/forte.cpp:598
#26 0x000000173c22ca6e in Profiler::handle (this=0x173bc31330, signum=<optimized out>, info=<optimized out>, context=0x7fffd605e240) at /home/olivier/honest-profiler-3/src/main/cpp/profiler.cpp:79
#27 0x0000001738838977 in handle_signal (actp=<optimized out>, sig=27, info=0x7fffd605e5b0, ucp=0x7fffd605e240) at /usr/src/lib/libthr/thread/thr_sig.c:243
#28 0x000000173883806c in thr_sighandler (sig=27, info=0x7fffd605e5b0, _ucp=0x7fffd605e240) at /usr/src/lib/libthr/thread/thr_sig.c:188
#29 <signal handler called>
#30 0x00000017408a9860 in ?? ()
#31 0x0000001911e62cf8 in ?? ()
#32 0x00007fffd605e858 in ?? ()
#33 0x0000001766d85d8b in ?? ()
#34 0x00007fffd605e8b0 in ?? ()
#35 0x000000179fa3d308 in ?? ()
#36 0x0000000000000000 in ?? ()

The crash occurs in frame 23, which maps to the following line in openjdk9/hotspot/src/cpu/x86/vm/frame_x86.cpp:
sender_pc = (address) this->fp()[return_addr_offset];

There seems to be a check just above that fp is "safe"; perhaps that test is deficient.
This code seems to have changed between OpenJDK 8 and OpenJDK 9. I don't know enough about the way the JVM deals with frames to understand what the code is doing and how it ends with a return value for fp() that is invalid (if I remember correctly the value for return_addr_offset was 1 according to GDB, and was thus unlikely to be the problem).
Does anyone have insights as to what the problem might be, and how to fix it?

As you state, this looks like a JVM bug rather than an Honest-Profiler bug. It would be valuable to report it on the relevant OpenJDK mailing list and link here maybe?
I'm vaguely familiar with the code in this area, but there are far more qualified people in the OpenJDK team.

@nitsanw Yes I think I'll report it to the HotSpot folks. It would be useful to know if the problem is specific to my OpenJDK build or platform (especially since I don't have the problem reduced to a simple test case, and don't have a very convenient way of trying to reproduce it on an officially-supported OS).
Has anyone been using Honest Profiler with OpenJDK 9? How is it working for you?

I have not, I think its worth trying to reproduce the issue without honest profiler running to see if its just a pure JDK9 problem.

reproduce the issue without honest profiler running

The crashes only occur with Honest Profiler. (And the stack trace in my original post shows that the problem is in a method called by AsyncGetCallTrace, which is itself of course called by Honest Profiler.)

My apologies - I was reading this thread too fast ;) +1 to @nitsanw 's suggestion of reporting it on the openjdk mailing list.

@cinquin if you can make Solaris Studio work on that platform you should see same issue, also with other AGCT profilers:

https://code.google.com/archive/p/lightweight-java-profiler/
https://github.com/apangin/async-profiler

JMC might have same issue.

@nitsanw Yes I imagine I would see the same issue. But to get this fixed by the OpenJDK people it would probably help if the problem could be reproduced on an officially-supported OS; I'm not set up to do that right now.
Incidentally, it might be a good idea to add a set of "stress tests" to the tests that are already part of the project. The other types of crashes that I've fixed so far only occur when the timing is just right (or rather, just wrong) and I don't remember them being triggered by the test suite.


mitogen crashed with "error: unpack requires a string argument of length 12"

Hi,
trying the lastest mitogen is crashing with "error: unpack requires a string argument of length 12" while running an apt task - the traceback is:
TASK [jmetex : install jmetex] *******************************************************************************************************************************************************************************************************************************************************************************************************************************************
task path: /Users/tom/repo/infrastructure/ansible/roles/jmetex/tasks/main.yml:6
[pid 9967] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 D mitogen: register(Context(9, None), mitogen.core.Stream('unix_client.9967'))
[pid 9967] 17:29:19 D mitogen: unix.connect(): local ID is 9, remote is 0
[pid 9967] 17:29:19 D mitogen: register(Context(0, None), mitogen.core.Stream('unix_listener.9953'))
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1000, '\x80\x02}q\x01(U\x08usernameq\x02X\t\x00\x00\x00teichhornq\x03U\x0fcheck_host_key'..298))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1001, '\x80\x02}q\x01(U\x08usernameq\x02X\x04\x00\x00\x00rootq\x03U\x03viaq\x04cmitogen.core\n'..224))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').call_async(<function run_module at 0x10a470140>, *('ansible.modules.packaging.os.apt',), **{'args': {'_ansible_version': '<IP_ADDRESS>', '_ansible_selinux_special_fs': ['fuse', 'nfs', 'vboxsf', 'ramfs', '9p'], u'name': u'jmetex', '_ansible_module_name': u'apt', '_ansible_syslog_facility': u'LOG_USER', '_ansible_verbosity': 3, '_ansible_socket': None, '_ansible_diff': False, '_ansible_debug': False, '_ansible_shell_executable': u'/bin/sh', '_ansible_check_mode': False, '_ansible_no_log': False}, 'env': {}})
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').send_async(Message(None, 9, 9, 101, 1002, '\x80\x02(U\x17ansible_mitogen.helpersq\x01NU\nrun_moduleq\x02U ans'..486))
[pid 9968] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 E mitogen: <mitogen.unix.Listener object at 0x10a47e3d0> crashed
Traceback (most recent call last):
  File "/Users/tom/repo/mitogen/mitogen/core.py", line 1185, in _call
    func(self)
  File "/Users/tom/repo/mitogen/mitogen/unix.py", line 81, in on_receive
    pid, = struct.unpack('>L', sock.recv(4))
error: [Errno 35] Resource temporarily unavailable
[pid 9953] 17:29:19 D mitogen: <mitogen.unix.Listener object at 0x10a47e3d0>.on_disconnect()
[pid 9968] 17:29:19 D ansible_mitogen.connection: Call run_module('ansible.modules.packaging.os.apt',) took 7 ms
The full traceback is:
Traceback (most recent call last):
  File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 130, in run
    res = self._execute()
  File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 528, in _execute
    result = self._handler.run(task_vars=variables)
  File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 128, in run
    return super(ActionModuleMixin, self).run(tmp, task_vars)
  File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/plugins/action/normal.py", line 45, in run
    results = merge_hash(results, self._execute_module(tmp=tmp, task_vars=task_vars, wrap_async=wrap_async))
  File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 308, in _execute_module
    env=cast(env),
  File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 137, in call
    return self._connection.call(func, *args, **kwargs)
  File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 238, in call
    return self.call_async(func, *args, **kwargs).get().unpickle()
  File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 224, in call_async
    self._connect()
  File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 192, in _connect
    self.router, self.parent = mitogen.unix.connect(path)
  File "/Users/tom/repo/mitogen/mitogen/unix.py", line 100, in connect
    mitogen.context_id, remote_id, pid = struct.unpack('>LLL', sock.recv(12))
error: unpack requires a string argument of length 12

fatal: [DE-IX-001-02-02-05-4]: FAILED! => {
    "msg": "Unexpected failure during module execution.",
    "stdout": ""
}

Ansible and python versions:
ansible <IP_ADDRESS>
  config file = /Users/tom/.ansible.cfg
  configured module search path = [u'/Users/tom/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.10 (default, Jul 15 2017, 17:16:57) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)]

The relevant playbook part is this:
- name: install jmetex
  apt:
    name: jmetex

If you need further outputs please ask!
You're work is awesome!
Thanks,
Tom

Hi Tom! Thanks for the report,  I'm mobile just now but will check when i
get home. Can you let me know:

how many cpus?
what ansible version and host os

On 11 Mar 2018 22:21, "Tom Eichhorn"<EMAIL_ADDRESS>wrote:

Hi,
trying the lastest mitogen is crashing with "error: unpack requires a
string argument of length 12" while running an apt task - the traceback is:
TASK [jmetex : install jmetex] *******************************************************************************************************************************************************************************************************************************************************************************************************************************************
task path: /Users/tom/repo/infrastructure/ansible/roles/jmetex/tasks/main.yml:6
[pid 9967] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 D mitogen: register(Context(9, None), mitogen.core.Stream('unix_client.9967'))
[pid 9967] 17:29:19 D mitogen: unix.connect(): local ID is 9, remote is 0
[pid 9967] 17:29:19 D mitogen: register(Context(0, None), mitogen.core.Stream('unix_listener.9953'))
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1000, '\x80\x02}q\x01(U\x08usernameq\x02X\t\x00\x00\x00teichhornq\x03U\x0fcheck_host_key'..298))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1001, '\x80\x02}q\x01(U\x08usernameq\x02X\x04\x00\x00\x00rootq\x03U\x03viaq\x04cmitogen.core\n'..224))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').call_async(<function run_module at 0x10a470140>, *('ansible.modules.packaging.os.apt',), **{'args': {'_ansible_version': '<IP_ADDRESS>', '_ansible_selinux_special_fs': ['fuse', 'nfs', 'vboxsf', 'ramfs', '9p'], u'name': u'jmetex', '_ansible_module_name': u'apt', '_ansible_syslog_facility': u'LOG_USER', '_ansible_verbosity': 3, '_ansible_socket': None, '_ansible_diff': False, '_ansible_debug': False, '_ansible_shell_executable': u'/bin/sh', '_ansible_check_mode': False, '_ansible_no_log': False}, 'env': {}})
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').send_async(Message(None, 9, 9, 101, 1002, '\x80\x02(U\x17ansible_mitogen.helpersq\x01NU\nrun_moduleq\x02U ans'..486))
[pid 9968] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 E mitogen: <mitogen.unix.Listener object at 0x10a47e3d0> crashed
Traceback (most recent call last):
File "/Users/tom/repo/mitogen/mitogen/core.py", line 1185, in _call
func(self)
File "/Users/tom/repo/mitogen/mitogen/unix.py", line 81, in on_receive
pid, = struct.unpack('>L', sock.recv(4))
error: [Errno 35] Resource temporarily unavailable
[pid 9953] 17:29:19 D mitogen: <mitogen.unix.Listener object at 0x10a47e3d0>.on_disconnect()
[pid 9968] 17:29:19 D ansible_mitogen.connection: Call run_module('ansible.modules.packaging.os.apt',) took 7 ms
The full traceback is:
Traceback (most recent call last):
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 130, in run
res = self._execute()
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 528, in _execute
result = self._handler.run(task_vars=variables)
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 128, in run
return super(ActionModuleMixin, self).run(tmp, task_vars)
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/plugins/action/normal.py", line 45, in run
results = merge_hash(results, self._execute_module(tmp=tmp, task_vars=task_vars, wrap_async=wrap_async))
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 308, in _execute_module
env=cast(env),
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 137, in call
return self._connection.call(func, *args, **kwargs)
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 238, in call
return self.call_async(func, *args, **kwargs).get().unpickle()
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 224, in call_async
self._connect()
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 192, in _connect
self.router, self.parent = mitogen.unix.connect(path)
File "/Users/tom/repo/mitogen/mitogen/unix.py", line 100, in connect
mitogen.context_id, remote_id, pid = struct.unpack('>LLL', sock.recv(12))
error: unpack requires a string argument of length 12
fatal: [DE-IX-001-02-02-05-4]: FAILED! => {
"msg": "Unexpected failure during module execution.",
"stdout": ""
}
Ansible and python versions:
ansible <IP_ADDRESS>
config file = /Users/tom/.ansible.cfg
configured module search path = [u'/Users/tom/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible
executable location = /usr/local/bin/ansible
python version = 2.7.10 (default, Jul 15 2017, 17:16:57) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)]
The relevant playbook part is this:

name: install jmetex
apt:
name: jmetex

If you need further outputs please ask!
You're work is awesome!
Thanks,
Tom
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dw/mitogen/issues/131, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAJC0oAHtNyEDFJBIP9NI0Qd3ddTlhrks5tdVKZgaJpZM4Slwef
.

Still not home,  but i think i see the problem. What version of OS X?
Somehow a blocking mode socket is returning EAGAIN. Makes no sense :)
On 11 Mar 2018 22:51, "David Wilson"<EMAIL_ADDRESS>wrote:

Hi Tom! Thanks for the report,  I'm mobile just now but will check when i
get home. Can you let me know:

how many cpus?
what ansible version and host os

On 11 Mar 2018 22:21, "Tom Eichhorn"<EMAIL_ADDRESS>wrote:

Hi,
trying the lastest mitogen is crashing with "error: unpack requires a
string argument of length 12" while running an apt task - the traceback is:
TASK [jmetex : install jmetex] *******************************************************************************************************************************************************************************************************************************************************************************************************************************************
task path: /Users/tom/repo/infrastructure/ansible/roles/jmetex/tasks/main.yml:6
[pid 9967] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 D mitogen: register(Context(9, None), mitogen.core.Stream('unix_client.9967'))
[pid 9967] 17:29:19 D mitogen: unix.connect(): local ID is 9, remote is 0
[pid 9967] 17:29:19 D mitogen: register(Context(0, None), mitogen.core.Stream('unix_listener.9953'))
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1000, '\x80\x02}q\x01(U\x08usernameq\x02X\t\x00\x00\x00teichhornq\x03U\x0fcheck_host_key'..298))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(0, None).send_async(Message(None, 9, 9, 500, 1001, '\x80\x02}q\x01(U\x08usernameq\x02X\x04\x00\x00\x00rootq\x03U\x03viaq\x04cmitogen.core\n'..224))
[pid 9953] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: put() done. woken? True
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').call_async(<function run_module at 0x10a470140>, *('ansible.modules.packaging.os.apt',), **{'args': {'_ansible_version': '<IP_ADDRESS>', '_ansible_selinux_special_fs': ['fuse', 'nfs', 'vboxsf', 'ramfs', '9p'], u'name': u'jmetex', '_ansible_module_name': u'apt', '_ansible_syslog_facility': u'LOG_USER', '_ansible_verbosity': 3, '_ansible_socket': None, '_ansible_diff': False, '_ansible_debug': False, '_ansible_shell_executable': u'/bin/sh', '_ansible_check_mode': False, '_ansible_no_log': False}, 'env': {}})
[pid 9967] 17:29:19 D mitogen: Context(5, u'ssh.<IP_ADDRESS>.sudo.root').send_async(Message(None, 9, 9, 101, 1002, '\x80\x02(U\x17ansible_mitogen.helpersq\x01NU\nrun_moduleq\x02U ans'..486))
[pid 9968] 17:29:19 D mitogen: unix.connect(path='/var/folders/jd/tvrdj3v55_z5ql3mgb4jky1c0000gn/T/mitogen_unix_Dc8KhF')
[pid 9953] 17:29:19 E mitogen: <mitogen.unix.Listener object at 0x10a47e3d0> crashed
Traceback (most recent call last):
File "/Users/tom/repo/mitogen/mitogen/core.py", line 1185, in _call
func(self)
File "/Users/tom/repo/mitogen/mitogen/unix.py", line 81, in on_receive
pid, = struct.unpack('>L', sock.recv(4))
error: [Errno 35] Resource temporarily unavailable
[pid 9953] 17:29:19 D mitogen: <mitogen.unix.Listener object at 0x10a47e3d0>.on_disconnect()
[pid 9968] 17:29:19 D ansible_mitogen.connection: Call run_module('ansible.modules.packaging.os.apt',) took 7 ms
The full traceback is:
Traceback (most recent call last):
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 130, in run
res = self._execute()
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/executor/task_executor.py", line 528, in _execute
result = self._handler.run(task_vars=variables)
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 128, in run
return super(ActionModuleMixin, self).run(tmp, task_vars)
File "/usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible/plugins/action/normal.py", line 45, in run
results = merge_hash(results, self._execute_module(tmp=tmp, task_vars=task_vars, wrap_async=wrap_async))
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 308, in _execute_module
env=cast(env),
File "/Users/tom/repo/mitogen/ansible_mitogen/mixins.py", line 137, in call
return self._connection.call(func, *args, **kwargs)
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 238, in call
return self.call_async(func, *args, **kwargs).get().unpickle()
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 224, in call_async
self._connect()
File "/Users/tom/repo/mitogen/ansible_mitogen/connection.py", line 192, in _connect
self.router, self.parent = mitogen.unix.connect(path)
File "/Users/tom/repo/mitogen/mitogen/unix.py", line 100, in connect
mitogen.context_id, remote_id, pid = struct.unpack('>LLL', sock.recv(12))
error: unpack requires a string argument of length 12
fatal: [DE-IX-001-02-02-05-4]: FAILED! => {
"msg": "Unexpected failure during module execution.",
"stdout": ""
}
Ansible and python versions:
ansible <IP_ADDRESS>
config file = /Users/tom/.ansible.cfg
configured module search path = [u'/Users/tom/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
ansible python module location = /usr/local/Cellar/ansible/<IP_ADDRESS>/libexec/lib/python2.7/site-packages/ansible
executable location = /usr/local/bin/ansible
python version = 2.7.10 (default, Jul 15 2017, 17:16:57) [GCC 4.2.1 Compatible Apple LLVM 9.0.0 (clang-900.0.31)]
The relevant playbook part is this:

name: install jmetex
apt:
name: jmetex

If you need further outputs please ask!
You're work is awesome!
Thanks,
Tom
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dw/mitogen/issues/131, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAJC0oAHtNyEDFJBIP9NI0Qd3ddTlhrks5tdVKZgaJpZM4Slwef
.

This is MacOS 10.13.3 (17D102), with ansible <IP_ADDRESS>, and 4 cores.

This might be another manifestation of issue #123. Looking at this today.
Does this crash for you reliably, and does it always crash in the same spot?  I'm running on a 2 core / 4 thread 10.12  MacBook and have never experienced this. But there is a probably related bug (#123) where a 16 vCPU box exposes some synchronization issue.  I may attack that bug first to see if it has the same root cause as this one, as I am able to reproduce that bug.

FWIW these ordering and synchronization issues are the hardest ones to find and diagnose, so please allow me a little time in figuring this one out :)

Hi, yes this always crashed at the same spot. No pressure, this is not like a burning issue, I can run ansible fine without mitogen ,)

I get this crash at different tasks when re-running. OS X 10.11.6 host, MBP 2 cores / 4 threads, tried with 1, 2, 3, and 4 hosts.

Head swirling in other issues today, sorry for the delay. Can you confirm, are you using the Apple-supplied Python build or e.g. one from MacPorts?

Mine is Homebrew:
ansible <IP_ADDRESS>
  config file = /Users/andy/gr/myproject/ansible.cfg
  configured module search path = [u'/Users/andy/gr/myproject/ansible/library', u'/Users/andy/gr/myproject/ansible/library/vendor']
  ansible python module location = /Users/andy/gr/myproject/.tox/deploy/lib/python2.7/site-packages/ansible
  executable location = .tox/deploy/bin/ansible
  python version = 2.7.14 (default, Mar  1 2018, 19:23:40) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-8<IP_ADDRESS>)]

@rouge8 thanks! Can you confirm, does the issue present on e.g. run_hostname_100_times.yml in the examples/playbook/ directory against localhost and/or your real target machine. And if not, does examples/playbook/issue_131.yml tickle it?

@rouge8 Can you possibly also include the output of ansible-config dump --only-changed from your project directory :) Thanks! Will try and have this one nailed tomorrow

Config:
ANSIBLE_FORCE_COLOR(/Users/andy/gr/myproject/ansible.cfg) = True
ANSIBLE_PIPELINING(/Users/andy/gr/myproject/ansible.cfg) = True
ANSIBLE_SSH_ARGS(/Users/andy/gr/myproject/ansible.cfg) = -F ansible/ssh_config -o ControlMaster=auto -o ControlPersist=60s
DEFAULT_CALLBACK_WHITELIST(/Users/andy/gr/myproject/ansible.cfg) = ['timer']
DEFAULT_FORKS(/Users/andy/gr/myproject/ansible.cfg) = 50
DEFAULT_GATHERING(/Users/andy/gr/myproject/ansible.cfg) = smart
DEFAULT_GATHER_SUBSET(/Users/andy/gr/myproject/ansible.cfg) = all,!ohai,!facter
DEFAULT_HOST_LIST(/Users/andy/gr/myproject/ansible.cfg) = [u'/Users/andy/gr/myproject/ansible/inventory_from_ssh_config.py']
DEFAULT_MODULE_PATH(/Users/andy/gr/myproject/ansible.cfg) = [u'/Users/andy/gr/myproject/ansible/library', u'/Users/andy/gr/myproject/ansible/library/vendor']
DEFAULT_ROLES_PATH(/Users/andy/gr/myproject/ansible.cfg) = [u'/Users/andy/gr/myproject/ansible/roles', u'/Users/andy/gr/myproject/ansible/shared_roles']
DEFAULT_STRATEGY(/Users/andy/gr/myproject/ansible.cfg) = mitogen
DEFAULT_STRATEGY_PLUGIN_PATH(/Users/andy/gr/myproject/ansible.cfg) = [u'/Users/andy/forks/mitogen/ansible_mitogen/plugins/strategy']

run_hostname_100_times.yml
All runs are on 4 hosts.

Worked the first time I ran it
Crashed the second, immediately after
Waited a couple minutes (longer than the ControlPersist=60s), reran. No crash
Reran several times immediately. No crash. 😕

issue_131.yml
All runs on the same 4 hosts.
Ran immediately after run_hostname_100_times.yml:
PLAY [all] ************************************************************************************************************

TASK [command] ********************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: error: unpack requires a string argument of length 12
fatal: [env1-stage-bastion]: FAILED! => {"msg": "Unexpected failure during module execution.", "stdout": ""}
changed: [env2-stage-bastion]
changed: [env2-prod-bastion]
changed: [env2-demo-bastion]

TASK [command] ********************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: error: [Errno 61] Connection refused
fatal: [env2-stage-bastion]: FAILED! => {"msg": "Unexpected failure during module execution.", "stdout": ""}
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: error: [Errno 61] Connection refused
fatal: [env2-prod-bastion]: FAILED! => {"msg": "Unexpected failure during module execution.", "stdout": ""}
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: error: [Errno 61] Connection refused
fatal: [env2-demo-bastion]: FAILED! => {"msg": "Unexpected failure during module execution.", "stdout": ""}
	to retry, use: --limit @/Users/andy/forks/mitogen/examples/playbook/issue_131.retry

PLAY RECAP ************************************************************************************************************
env2-demo-bastion        : ok=1    changed=1    unreachable=0    failed=1
env2-prod-bastion        : ok=1    changed=1    unreachable=0    failed=1
env2-stage-bastion       : ok=1    changed=1    unreachable=0    failed=1
env1-stage-bastion        : ok=0    changed=0    unreachable=0    failed=1

Ran a second time, same results. Ran a third time, all hosts succeeded for the first command, env2-demo-bastion failed the second, env2-prod-bastion and env2-stage-bastion succeeded, and env1-stage-bastion appears to have hung.

You got me excited when I saw ANSIBLE_FORKS=50, because I never test with that, only to have hopes dashed on seeing my playbook of a single step (run_hostname..) also crashing :P~
So it looks a hell of a lot like a threading issue in the top-level process itself, not in any worker. It simply shouldn't be possible for ECONNREFUSED to happen during a run, so this has already been very useful.
Thanks again so much for your help!

Finally managed to reproduce this locally. WIP

Sorry for the delay, this was actually a really simple bug and I should have spotted it much earlier.
As for why you saw it and I did not, either that is due to a large forks setting, or because your machines are sufficiently busy to starve the new worker process of the chance to write its identity to the socket before the master attempted to read it.
Can you please confirm the issue is now gone on latest master.

Hi, I can confirm this error is gone

Same, no longer crashing for me. 🎉
My actual playbook is slower than previous non-crashing runs though. I'll try and debug and/or simplify that and open a separate issue when I figure it out. 😕

That does not sound helpful! Please keep spamming me these tickets, and always be sure to mention the modules involved. Note if you were timing using the ansible.cfg in the examples directory, it now has forks=50 by default. I have zero clue how much perf impact that will have (but I guess we're going to find out over the coming days :))

It might also be something going with the network -- that same playbook is also slower today without Mitogen than it was yesterday 🙃  I'll open issue(s) if I figure out more!

I've got some vague thoughts around a built-in profiling function that would capture this kind of stuff "for free". Something like that popular callback plug-in, but on steroids, since we have an obscene level of visibility into the remote process (e.g. per task getrusage()) and have smart code at every network endpoint (e.g. latency or bandwidth monitoring via adding message timestamps, etc)


[gRPC][Generator tool] Add new param to pass dependent proto file directories

Description:
Current implementation doesn't have a way specify give different dependent proto file directory path. dependent proto file need to add inside same proto directory.
We need to add new param to pass dependent proto file directories like below,
ballerina grpc --proto_path=src/ --input sample.proto --output service/

I am closing this issue since we do have another issue to capture the same. please check #1612 to track the status


add mooBeetSolidBeets vault

wantToken = "0xf2b2103e1351c4d47b27efc8100bb264364fa92c"
wantPoolId = "0xf2b2103e1351c4d47b27efc8100bb264364fa92c00020000000000000000025a"
inputSwapPoolId = "0xcde5a11a4acb4ee4c805352cec57e236bdbc3837000200000000000000000019"
nativeSwapPoolId = "0xcde5a11a4acb4ee4c805352cec57e236bdbc3837000200000000000000000019"
chefPoolId = 59
strategist = "0xCbd4082a6057a01217bAd206708ACDf1dC4bba41"
input =  "0xF24Bcf4d1e507740041C9cFd2DddB29585aDCe1e"
output = "0xF24Bcf4d1e507740041C9cFd2DddB29585aDCe1e"
native = "0x21be370D5312f44cB42ce377BC9b8a0cEF1A4C83"

submitted new PR with fixes to contract addresses and asset names which was leading to failed build


fix: subject length to long

Issue #, if available:
It looks like there's an issue with the TEAM application that arises when the SUBJECT of an Amazon Simple Notification Service (SNS) message is too long. The specific problem is that the TEAM application doesn't validate or limit the maximum length of a string variable, causing message sending to fail. The error is captured in CloudWatch, and a screenshot of the error has been provided.

Description of changes:
To address this issue, the proposed solution is to shorten the SUBJECT line by removing the {requester} variable, which can potentially be very long and contain a significant number of characters.  {requester} variable is the email address which is raising an request. The original long subject line includes the {requester} variable, and the modified shorter subject line omits it. Here's an example to illustrate the change:
example:
Long Subject Line (Original):
subject = f"{requester} requests access to AWS account {account} - TEAM"
Short Subject Line (Modified):
subject = f"Requests access to AWS account for {account} - TEAM"
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@abita-reply  PR merged. please close the associated issue once validated

Thanks, it's working!


CdsTag is clickable by default [React]

Describe the bug
By default, with no additional properties CdsTag behaves like a button, even without the clickable property.
In addition, adding a closable property, does not render an X like expected.
How to reproduce
Stackblitz: https://stackblitz.com/edit/react-ts-9sxe8z?file=index.tsx
Expected behavior
Tags should render without clickable behavior by default.
"Closable" tabs should render with an X
Versions
App

Clarity: cds/core and cds/react 5.0.0-rc.1

@jeeyun : since you are in this area of the code already, could you take a look?

@jeeyun : since you are in this area of the code already, could you take a look?

I just realized that I believe this is the expected behavior and you have to add readonly if you don't want it clickable.
However, I'm still unable to get the X to appear on closable.

I just realized that I believe this is the expected behavior and you have to add readonly if you don't want it clickable.
However, I'm still unable to get the X to appear on closable.


MINOR Don't run tests for Draft PRs

This patch updates our CI workflow to skip the JUnit tests if the PR is marked as a Draft. Also, change to using the more modern "checks" API instead of "statuses" for the feedback from CI Complete.

Example of the tests and build scans being skipped (from my fork):

When I click "Ready For Review", the CI kicks off and the tests run
https://github.com/user-attachments/assets/cb32c568-87de-453d-b153-cf6c5679d830

Example with "WIP" in title: https://github.com/mumrah/kafka/pull/25


test: replica test cases fix

What type of PR is this?

master nodes don't hold the necessary azuredisk topology label that is required to satisfy azuredisk volumes' required node affinity.
Hence, this label needs to be added to fully utilize the 3 nodes for replica test cases.
There were cases observed where the tests were querying for wrong AzVolumeAttachments for the wrong volume, resulting in inconsistent test results.
Hence, the change involves getting PVC name from the pod spec and querying for PV name directly from the PVC object.

/kind test
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.925%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
182

Relevant Lines:
424

💛  - Coveralls

FYI- I had @landreasyan disable the test so that we can make progress before the fix is checked in. You'll need to undo #284 as part of this PR.

FYI- I had @landreasyan disable the test so that we can make progress before the fix is checked in. You'll need to undo #284 as part of this PR.

Sounds good

I will merge this PR as the pod affinity PR has some dependencies on the changes. I will revert the merge if necessary.


Bumping to latest dependencies v0.21

Changes
Upgrading to v0.21 using: ./hack/update-deps.sh --upgrade --release v0.21

:broom: Update dependencies to latest Knative release

/kind cleanup
/assign @julz

hm, dont think that worked, looks like it's still looking at OWNERS and Im not in there 😢
/assign @mattmoor

I believe this PR makes the switch to CODEOWNERS.
Meanwhile:
/lgtm
/approve

at least I tried :-)


Clarify/simplify military-feature-data install steps a bit

Watching @dfoll try to set up this repo made me realize that the readme is somewhat complicated and unclear.
We just need to tell folks that ant is optional and tell them "go grab the data folder from : https://github.com/Esri/military-features-data " (for the Converter Tools Anyway, other tools may need all of the steps)
But maybe we could have a table with list of the tools, purpose, what data folder needed, etc.)
Also Add a link to the template.
Related to #17

Closing. The toolbox has been moved to https://github.com/Esri/solutions-geoprocessing-toolbox/tree/dev/military_features.


Features that does not work with Basilisk

They are hardcoded - using 'subsquid', using ss58 2, using KSM
Collection

[ ] History
[ ] Holders
[ ] Flippers

Profile

[ ] History
[ ] Holdings
[ ] Gains

Detail
Error: Network error: Response not successful: Received status code 400
Blocked by

#660

:wave:

@vikiival can you create child issues for this one?

I will check the Gallery Passion filter.

https://github.com/galacticcouncil/amsterdot-bounties-2022/issues/3

[ ] Create a class
Creates a class, (i.e. collection) of tokens that can all be described by certain set of common properties.

[ ] Mint
Mints a token that is unique within a class.

[ ]  Transfer
Transfers ownership of a token to another wallet.

[ ] Burn
Destroys a previously minted token.

[ ] Destroy a class
Destroys a previously created class. In our implementation, such a class cannot contain any minted tokens inside.

[ ] Set a price
Also commonly called a listing. Owner of a token can list their token on the marketplace with a specified price for which it can be bought instantly by other users.

[ ] Buy
A user can buy a listed token for previously set price by its owner.

[ ] Make an offer
Even if a user does not set price for their token, other users can make offers for which they would buy the token. This reserves a full offered amount on offerer's account. Only one offer can be made by one account for a token.

[ ] Withdraw an offer
Retract a previous offer and unreserve funds.

[ ] Accept an offer
Owner of a token can decide to accept an offer which then acts as a regular buy action.

[ ] Add a royalty
Owner of an NFT can embed a royalty to a token. That means from that point on, each trade of this NFT will split the full amount to the royaltee (usually the owner) and the seller of the NFT in the defined ratio.

I think it's close to save this one to prevent having 2 chief Basilisk issues, both of the features which are still not done here are linked here

https://github.com/kodadot/nft-gallery/issues/660


Getting invalid token in incognito mode and firefox

Hi! I am working on a greenfield nextjs project and recently deployed to vercel. Everything was working fine until yesterday, when I changed browsers (for fun) and noticed I am always hitting handleInvalidToken in incognito mode and firefox related browsers.
I am unsure how to debug it. I am happy to help getting this fixed!

I'm having the same issue - out of the blue.  I even have a branch that was deployed a few weeks ago that is still working - but the latest npm is always falling into handleInvalidToken (and there's no context to log at that point).
The only thing I've noticed is that my (working) branch writes two session cookies on a successful login: CookieName + CookieName.sig.  The (failing) current code is only writing the first cookie and not the .sig

My unresolved issue I created yesterday might be related: #146
It's working in local, but the authentication flow is falling in handleInvalidToken in the firebase deployment environment.

@awinogrodzki Any thoughts on why the signed cookie is not being written?
Looking at the code (https://github.com/awinogrodzki/next-firebase-auth-edge/blob/926f23656e1fd935d064cb360e424597d80431af/src/auth/cookies/index.ts#L1) it seems like this is failing to be set.  But I'm not sure why a cookie IS being set without the .sig

Hey @kucho,
Thanks for reporting the issue. Could you share a link to your Vercel environment, as well as output of npx next info?
Also, are you on Windows or Mac?
Could you also test this issue on https://next-firebase-auth-edge-starter.vercel.app/ and let me know if it's reproducible?

I am testing https://next-firebase-auth-edge-starter.vercel.app on number of browsers (Chrome, Firefox, Opera, Safari, Edge) in and out incognito mode and cannot reproduce the issue on my side, so it might be related with a configuration.
BTW. could you also share your middleware.ts (without credentials) :-)

@awinogrodzki I can not reproduce the error on the demo page.
I have tried:

Upgrade nextjs
Upgrade next-firebase-auth-edge
Disabled preview secure auth cookies
Disabled vercel preview protection

Still getting

I am using mac. The npx next info output is:

Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 23.3.0: Wed Dec 20 21:30:44 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T6000
Binaries:
  Node: 21.4.0
  npm: 10.2.5
  Yarn: 1.22.21
  pnpm: 8.15.1
Relevant Packages:
  next: 14.1.1-canary.51 // Latest available version is detected (14.1.1-canary.51).
  eslint-config-next: 14.0.4
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.3.3
Next.js Config:
  output: N/A

The middleware.ts:
import { authMiddleware, redirectToHome, redirectToLogin } from "next-firebase-auth-edge";
import { NextRequest, NextResponse } from "next/server";
import { env } from "~/env";

const AUTH_PATHS = ["/login"];

export async function middleware(request: NextRequest) {
	return authMiddleware(request, {
		loginPath: "/api/login",
		logoutPath: "/api/logout",
		apiKey: env.FIREBASE_API_KEY,
		cookieName: "AuthToken",
		cookieSignatureKeys: ["secret1", "secret2"],
		cookieSerializeOptions: {
			path: "/",
			httpOnly: true,
			secure: env.SECURE_AUTH_COOKIE,
			sameSite: "lax" as const,
			maxAge: 7 * 60 * 60 * 24, // 7 days
		},
		serviceAccount: {
			projectId: env.FIREBASE_PROJECT_ID,
			clientEmail: env.FIREBASE_CLIENT_EMAIL,
			privateKey: env.FIREBASE_PRIVATE_KEY,
		},
		handleValidToken: async ({ token, decodedToken }, headers) => {
			if (AUTH_PATHS.includes(request.nextUrl.pathname)) {
				return redirectToHome(request);
			}

			return NextResponse.next({
				request: {
					headers,
				},
			});
		},
		handleInvalidToken: async () => {
			console.warn("Invalid firebase token");

			return redirectToLogin(request, {
				path: "/login",
				publicPaths: AUTH_PATHS,
			});
		},
		handleError: async (error) => {
			console.error("Unhandled authentication error", { error });
			return redirectToLogin(request, {
				path: "/login",
				publicPaths: AUTH_PATHS,
			});
		},
	});
}

export const config = {
	matcher: ["/api/login", "/api/logout", "/login", "/logout"],
};

My middleware:
`import { NextRequest, NextResponse } from "next/server";
import { authMiddleware, redirectToHome, redirectToLogin } from "next-firebase-auth-edge/";
import { authConfig } from<EMAIL_ADDRESS>const ANON_PATHS = ["/account/signin", "/account/create", "/account/reset"];
const PUBLIC_PATHS = [...ANON_PATHS, "/monitoring"];
export async function middleware(request: NextRequest) {
return authMiddleware(request, {
loginPath: "/api/account/signin",
logoutPath: "/api/account/signout",
apiKey: authConfig.apiKey,
cookieName: authConfig.cookieName,
cookieSerializeOptions: authConfig.cookieSerializeOptions,
cookieSignatureKeys: authConfig.cookieSignatureKeys,
serviceAccount: authConfig.serviceAccount,
    handleValidToken: async ({ token, decodedToken }, headers) => {
        if (ANON_PATHS.includes(request.nextUrl.pathname)) {
            return redirectToHome(request);
        }

        return NextResponse.next({
            request: {
                headers,
            },
        });
    },

    handleInvalidToken: async () => {
        console.log("Server Config", authConfig);
        console.log("Invalid token");

        return redirectToLogin(request, {
            path: "/account/signin",
            publicPaths: PUBLIC_PATHS,
        });
    },

    handleError: async (error) => {
        return redirectToLogin(request);
    },
});

}
export const config = {
matcher: [
"/", // Match all paths
"/((?!_next|favicon.ico|api|.\.).)", // Matches all paths except /_next, /favicon.ico, /api, and files with extension
"/api/account/signin", // This API endpoint is not protected by authMiddleware
"/api/account/signout", // This API endpoint is not protected by authMiddleware
],
};
`

@kucho,
I am trying to login to https://portal-nextjs-git-eng-401-ridepanda.vercel.app/ via email link. I receive e-mail as expected, but application fails to process my login, when I click on the link.
Could you copy the contents of your login page? I am interested mainly in signInWithEmailLink call, which seems not to be occurring.
In starter example, signInWithEmailLink is handler here: https://github.com/awinogrodzki/next-firebase-auth-edge/blob/main/examples/next13-typescript-starter/app/login/LoginPage.tsx#L68-L93
If you open dev console in starter example and follow the email link, eventually you'll see a call to https://identitytoolkit.googleapis.com/v1/accounts:signInWithEmailLink?key=XXX, which does not occur in portal-nextjs-git-eng-401-ridepanda.vercel.app

I still don't have a proper explanation to what happened with my early deployment that succeeded on chromium, but failed on firefox. I made a bunch of changes, added and removed a lot of bits, and this time around I missed the signInWithEmailLink call.
After handling it like in your example, it just worked. I think I assumed (incorrectly) the login params were being handled by the middleware and the redirection was all happening at this level, not client side.
Thank you for your help and  patient 🙏! Feel free to close this issue.

I'm glad to hear that!
If anything else comes up, don't hesitate to create another issue.
Cheers 🎉


Option to parse existing Deployment object to generate the one for telepresence

Given an existing Deployment it should be possible to extract environment variables and the like for the Telepresence Deployment.

Jean-Paul expressed a preference for doing this on level of config files, rather than talking to Kubernetes cluster, since it gives more visibility over how things are done.

An idea in the opposite direction might be to have telepresence actually edit the existing Deployment for the user.  A command line:
$ telepresence --deployment foo --container bar --new-shell

could patch foo so that its bar container uses the telepresence image and then start a shell in the proxied environment for that container.  When the shell exits, telepresence could even roll the deployment back.
That seems kinda snazzy to me though it may be sufficiently automagical that there are some unwanted obscure corner-cases or side-effects.

Current plan:
$ telepresence --swap-deployment existingservice --run-shell

This swaps out Telepresence proxy for existing pod, and reverts on shutdown.

Remaining work for swap-deployment branch:

Get all tests passing (hopefully just dealing with fiddly bits in Runner.check_call).
Test against OpenShift, make sure it works.
Make sure no k8s resources are leaked by telepresence or the test suite.
Update documentation and asciinema video (that could be done in separate branch, though.)


Rewrite throttle logic to prevent request flooding

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: 1303
What is the new behavior?
I've made some adjustments based on my suggestion in #1303 in order to add a single record and retrieve all info in a single function. This also allows external storage providers made by the community to add and retrieve in a single roundtrip (i.e. my redis storage provider).
The initial behavior was that getRecord() returned an list of sorted TTL timestamps, then if it didn't reach the limit, it will call addRecord(). This change was made based on Redis where I fiddled around on how to prevent this issue. I found out that express-rate-limit is incrementing a single number and returning the information in a single roundtrip, which is significantly faster than how NestJS throttler works by called getRecord(), then addRecord.
The rate-limit-redis did have a very fast version with EVALSHA. I tried to integrate the EVALSHA script they used into my own storage provider, which immediately worked. Then I adjust the changes to my version, but that did not work all the way. There were still coming +10 requests through with my version. This version was much faster than before where ±400 requests came through, but still wasn't not perfect. This was because SMEMBERS <key> that I personally used - which returns a list of TTL timestamps from <key> - seems to be much slower than it is to increment a single number and retrieve this number. Therefore, I made the new logic similar to how express-rate-limit works by requiring two things from the addRecord() (which I have renamed in this PR):

totalHits: amount of requests done for key
timeToExpire: amount of seconds until the totalHits expire for key

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Breaking ThrottlerStorage changes:

removed getRecord
addRecord(key: string, ttl: number): Promise<number[]>; changes to increment(key: string, ttl: number): Promise<ThrottlerStorageRecord>;

Other information

@jmcdo29 I still feel like we should put { totalHits: number; timeToExpire: number } in a variable, what do you think?

@jmcdo29 I have adjusted some small issues and have refactored some of the code. The things to notice is:

timeToExpire will be reset in the ThrottlerStorageService once it is 0 or less (this was already done in my redis storage provider, but I forgot to add it here)
two new interfaces for the sake of extensibility: ThrottlerStorageOptions and ThrottlerStorageRecord (the latter will be very useful for my own package as well, the former is more for internal use of this package, but is still nice to have because we obviously don't want to repeat ourselves as developers :))
renamed addRecord to increment: I felt the addRecord was not the main purpose of the function as we're incrementing the totalHits as of now.
Adjusted the regex in test/controller.e2e-spec.ts for x-ratelimit-reset so that we only require whole numbers. The previous version was only checking if it contained 1 or more numbers, which isn't what we should check, because that means any arbitrary string can come through, as long as it has a single digit in it.

Everything looks good to me here.
@kamilmysliwiec this would be a part of a "breaking" change in that standard use of the library should not change at all, but the underlying API that developers could extend and work with for custom storages would change. I wanted to bring this to your attention before publishing a major release for it in case you noticed anything you wanted addressed.

Hey do you have an ETA on merging this PR?

@jmcdo29 Hey do you have an ETA on merging this PR?

@jmcdo29 Hey do you have an ETA on merging this PR?

I'll likely get this merged and published later this week

Thanks that will be great, we noticed some issues on requests spike which related to this issue

Hi @jmcdo29 , any news about merging PR?

I'll get the docs change started up tomorrow and get the ball rolling on all of that

Welp, that rebase mucked things up :sweat_smile: I'll get that resolved and merged today.

@kkoomen heads up, I had to force push to your branch to remove most of those renovate/dependabot commits. All should be clean now though. Just a heads up for next time you work on this library unless you just delete and re-fork.

@jmcdo29 Thanks, I'll start merging things on my side.


Uploading a data file sometimes fails

When uploading a second data file to an existing project, it sometimes fails
@kyao
(from meeting-- please add details/files so we can try to replicate)

this was on version 2..8.0

I have not been able to replicate this yet

@kyao  this is the same issue you reported in #407?


Upgrade to parsel-js 1.2.0+

It looks like the 1.2.0 release is missing the dist folder.
I have ignored it using dependabot, so we need to dependabot unignore  parsel-js once the release is available.
See https://github.com/puppeteer/puppeteer/pull/13345#issuecomment-2509668548

@dependabot unignore parsel-js


Excuse an amateur -- what do I upload where?

I am not a developer, but I want to understand how Parsedown would work on my Linux shared hosting.
Given that I know how to FTP files and edit them:

what do I have to upload to my server?
where do I put it?
what commands do I then put into an index.php file to make use of Parsedown?

Again, please excuse the fact that I don't understand what to do, or if there is a "Dummies" guide to this kind of stuff, please point me to it!

@philipca depends on what you want to do with Parsedown. I doubt it would be of any use if you do not know how to write php.

But how would somebody who does know how to 'write PHP' install it on my site. I like to understand how things work before I pay such a person to make use of it for me.
I know where and how to create and edit css files, I know where to install javascript files and how to call them.  Is using Parsedown very much different from those??

Parsedown is a library that can convert Markdown to HTML, it comes as a PHP class without any frontend or user interface. It can be used as part of other software to provide some functionality, it does not make sense to use it alone.

So, if a PHP file (ie a webpage) is going to call Parsedown, where does it find Parsedown?
I thought something was uploaded to the server, or is it dynamically loaded like javascript?

Correct me if I wrong. I still learn PHP.
I think you should reference the parsedown.php using "require"

@philipca This is not really an "issue".
Anyways, an absolute "bare bones", just-make-it-work example would look like this:

Download the Parsedown.php class file.
Upload it to your server.
Place index.php file in the same directory as Parsedown.php and include this code in index.php:

<?php

require 'Parsedown.php';

$Parsedown = new Parsedown();

$theText = file_get_contents('EXAMPLE.md');

echo $Parsedown->text($theText);

Write some markdown and save the content as EXAMPLE.md in the same directory as the other files.
View index.php in the browser.

However, as others have said, some knowledge of PHP and classes will go a long way - but this will get you a "Hello World" example.

Thank you, Jonathan Bell. A clear and polite instruction, which I could follow easily.
Parsedown now works for me, and so I understand better how it can be invoked and used.
Thank you for making this clear to me.

Actually, now I understand Parsedown (and Markdown) more fully, it looks like a better fit for my needs would be the 'editor' supplied on this GitHub page that I am typing in now!!
Could somebody please explain to me what it is, and where I could obtain it??


Search in report panel is not honoring config settings

I configured a custom locator on report search config panel, but the settings are not honored in the search on report panel.
In the image below, I have added San Diego locator but when I search 'Redlands' it's providing suggestions, but it's not the same experience on the search button on map. Is this a bug or by design?

Related issue: https://devtopia.esri.com/WebGIS/arcgis-portal-app-templates/issues/10613

@skrishn
We are able to see the same results for the input redland

Could you check this once, it could be due to some dev env issues.

@skrishn
Please check this and let us know if any changes required from our end


Deno support for sqrl core library? (murmurhash3 dependency has invalid spec)

Hey, thanks for posting this on HN, it looks really cool! I was wondering was if you'd accept some small tweaks to get the core sqrl library working with Deno? I've been exploring it for deploying JavaScript libraries and I think SQRL would be a great fit for this; it makes it super easy to deploy things like custom servers based on JavaScript APIs like this.
Deno has support for importing some subset of npm packages with the npm: qualifier on an import. So, we might expect the following typescript program to work:
import { Instance, Execution, AT } from<EMAIL_ADDRESS>import { run } from<EMAIL_ADDRESS>
export async function register(instance: Instance) {
  instance.register(
    // @todo: Add type for name once it's required
    async function sayHello(state: Execution, name) {
      return "Hello, " + name + "!";
    },
    {
      // @todo: Add argument documentation
      args: [AT.state, AT.any],
    }
  );
}

run({ register });

But, deno run -A sqrl-server.ts fails. This is expected I suppose, but what we get is:
error: npm package<EMAIL_ADDRESS>
Caused by:
    0: error parsing version requirement for dependency: murmurhash3.js@https://github.com/qix/murmurHash3.js/releases/download/3.0.0-qix-0/murmurhash3.js-3.0.0-qix-0.tgz
    1: Invalid npm version requirement 'https://github.com/qix/murmurHash3.js/releases/download/3.0.0-qix-0/murmurhash3.js-3.0.0-qix-0.tgz'.
    2: Unexpected character.
         https://github.com/qix/murmurHash3.js/releases/download/3.0.

I think this is just because Deno doesn't support the direct https:// method for node dependencies. The actual code itself works fine, actually.
In fact, the following program does work fine, and all it does is import all the dependencies of the core sqrl library (except the murmurhash dep, but that's fine). None of these have any issues, and I don't see why the core library would either:
import * as _bigint_buffer from "npm:bigint-buffer"
import * as _bluebird from "npm:bluebird"
import * as _double_ended_queue from "npm:double-ended-queue"
import * as _eventemitter3 from "npm:eventemitter3"
import * as _fast_stable_stringify from "npm:fast-stable-stringify"
import * as _jsonschema from "npm:jsonschema"
import * as _moment from "npm:moment"
import * as _moment_timezone from "npm:moment-timezone"
import * as _node_nice from "npm:node-nice"
import * as _sqrl_common from "npm:sqrl-common"

if (import.meta.main) {
  console.log("hello world");
}

So the Murmurhash dependency seems to be the sticking point. Maybe this could be bugfixed on the Deno side but it's easy enough to workaround too, I think. There are like, 3 options here that I see:

Release the murmurhash3.js package to npm, then start depending on that with a normal version constraint
Migrate to a different pure-JavaScript murmurhash3 package
Just inline murmurhash3.js into this package, because it's a small single file and an implementation detail anyways?

Honestly, I would vote for number 3, since it's less work to test, no need to release another new package, and just eliminates a dependency.
Note that the Redis package also depends on murmurhash3.js. Frankly I think it would be fine to just inline a copy of this file there, too; yes it's copy/paste, but I don't think it's a huge sin. Frankly you could use other hashes too here I think, so it's all a bit of a moot point — just something needs to be done.

Please note that sqrl-cli and its ilk don't necessarily need to work immediately. Deno provides its own HTTP server APIs, so really there should maybe be a sqrl-deno-http package or something. I'd actually be happy to write a simple server connecting SQRL to Deno's HTTP system. But the core library needs to work first.

There are actually many more changes needed than this for the pure TypeScript core to depend on Deno, e.g. its enforcement of isolated modules, lack of require, etc. A big one is that a local import must use a file extension (because import ... from "foo" is ambiguous as to whether it means foo.ts or foo.js.) So it's not just this. But I think it's possible to support both with some cleanups, e.g. moving all the CommonJS require imports to use ESM, etc.
So, assuming Deno support for the core lib sounds like something OK to add, I think it's probably doable if you'd accept some patches to shuffle stuff around.


create inclusive naming check

So, there ARE failures bu they've been filtered by reviewdog because essentially we're not introducing new failures -- only eliminating existing ones.  I still need to manually test an upgrade from a previous deployment.  Diligent reviews are appreciated

i've tested an upgrade from  cs:kubernetes-master -> kubernetes-control-plane --channel=latest/edge -> locally built charm with these changes:
juju upgrade-charm kubernetes-master --switch kubernetes-control-plane --channel=latest/edge
# switches charm metadata name from `*-master` to `*-control-plane`
juju-wait
juju upgrade-charm kubernetes-master --path  ./kubernetes-control-plane.charm
# tests the charm upgrade
juju-wait
juju run -u kubernetes-master/0 -- charms.reactive get_flags

all flags still look good.  no control-plane flags exist


Jesse: Watch meshok.irlink.ru

Jesse requests the watch of the watch_keyword meshok\.irlink\.ru. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

meshok\.irlink\.ru has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


how to save all the uploaded file name in database

Hi to All,
              Your script is verynice......I want one help..please help me....how to save all the uploaded file name in database........

pls help me.........

Hi ricmetal,
I have a problem with your solution. The error reads:
PHP Notice:  Undefined variable: file_name in /pathto/php/index.php on line 305.
Any ideas?

Nevermind, I got your code to work. I needed to change: $stmt->bind_param('s', $file_name);
to: $stmt->bind_param('s', $name);
Thank you for your code ricmetal!

no problem.
cheers mate.

Hello ricmetal, I don't know if you are still monitoring this Thanks to your above help I have gotten the title into the sql, but now I am having trouble deleting it when the corresponding file(s) are deleted from their directory. Do you have any solutions for this?


Prettier

Checklist:

[x] Test your work and double check you didn't break anything
[x] Update any relevant READMEs and docs

Description:
Adding prettier to format code equivalently for all developers. Followed the way origin.js uses prettier, open to make it better or easier to use.

Looks like there is an issue with the glob patterns, I don't think the ** does what I think it does, it isn't penetrating directories

Should this be closed in lieu of https://github.com/OriginProtocol/demo-dapp/pull/189 ?

Yes, I'm going to close this in favor of #189 for now. Single quotes ftw. 💪


Unable to load vscode-remote://codespaces%<REDACTED: user-file-path>: Request through port forwarding failed.

Error: Unable to load vscode-remote://codespaces%<REDACTED: user-file-path>: Request through port forwarding failed.
at Q.M (https:/<REDACTED: user-file-path>:2000:23849)
at async Promise.all (index 0)

Go to Errors Site

That string doesn't show up in our codebase, I think that's just the codespaces embedded.


Adding util class

NetflixOSS » eureka » eureka-pull-requests #593 SUCCESS
This pull request looks good


Enable unit tests in workflow

Unit Tests are now disabled because we're dependent on a storage account.
Can enable using Azurite as a storage emulator.

This issue was closed with commit c1a8cf9f07a24f851386a5ee561b1edfc5032e22


impl from_mdb_value for bool is unsound

The source of unsoundness
https://github.com/vhbit/lmdb-rs/blob/3a4bd66eb92716e5115568bc101246255dececf7/src/traits.rs#L135-L142
The function from_mdb_value is implemented through macro on several primitive types including integers, float numbers, and bool. However, it is unsound to transmute data with arbitrary bit patterns to bool.
To reproduce the bug
use lmdb_rs_m::core::MdbValue;
use lmdb_rs_m::FromMdbValue;

fn main() {
    let a: i32 = 3;
    let mdbval = MdbValue::new_from_sized(&a);
    let res = bool::from_mdb_value(&mdbval);
    println!("{:?}", res);
}

run with Miri would show that UB happen.
    |
157 | mdb_for_primitive!(bool);
    | ^^^^^^^^^^^^^^^^^^^^^^^^ constructing invalid value: encountered 0x03, but expected a boolean
    |

UB should not happen with safe function in Rust.

Misaligned pointer dereference could also happens here:
fn main() {
    let a: i32 = 3;
    let mdbval = MdbValue::new_from_sized(&a);
    let res = i64::from_mdb_value(&mdbval);
    println!("{:?}", res);
}

run the code and panic
thread 'main' panicked at 'misaligned pointer dereference: address must be a multiple of 0x8 but is 0x7ffeedce1fbc', /${HOME}/.cargo/registry/src/index.crates.io-6f17d22bba15001f/lmdb-rs-m-0.7.7/src/traits.rs:154:1

At line 138, *i32 behind *c_void is indirectly transmuted to *mut i64 leading to misaligned pointer and dereference at line 139.

[UPDATED] The implementation here could also cause to another undefined behavior.
To reproduce the bug, please see the following code:
#[repr(align(2))]
#[derive(Copy, Clone, Debug)]
struct Padding {
    a: u8,
    b: u16,
    c: u8,
}

fn main() {
    let la = Padding { a: 10, b: 11, c: 12 };
    let mdbval = MdbValue::new_from_sized(&la);
    let res = i32::from_mdb_value(&mdbval);
    println!("{:?}", res);
}

to run with miri,
error: Undefined Behavior: using uninitialized data, but this operation requires initialized memory
   --> /home/rafael/.cargo/registry/src/index.crates.io-6f17d22bba15001f/lmdb-rs-m-0.7.7/src/traits.rs:152:1
    |
152 | mdb_for_primitive!(i32);
    | ^^^^^^^^^^^^^^^^^^^^^^^ using uninitialized data, but this operation requires initialized memory
    |
    = help: this indicates a bug in the program: it performed an invalid operation, and caused Undefined Behavior


Single choice list view regression API 16 v0.7

On API 16 there seems to be a problem with the icon colour (selected and not-selected) for single-choice list view dialogs. Correct colours are used on API 22.
Setting md_widget_color changes the selected colour correctly, but the selected colour is not derived from colorAccent.
In the image below, version <IP_ADDRESS> is on the left and looks correct. Version <IP_ADDRESS> is on the right and has the incorrect (inverse?) colours for the circular checkbox-like thing.

Are you changing the dialog theme with the Builder's theme method? Black is the default color for the light theme, white is the default for the dark theme.
Widget color only works for API 21 and above.

The only issue I would expect here is that you manually changed the dialog background color, but there could also somehow be a bug.

I should note that Material teal is the default colorAccent value in the Material theme or AppCompat, that's why it used to be there.

The dialog theme is set to dark in styles.xml. On 0.7 it picks up the dark theme correctly, so setting the theme explicitly has no effect (same results with it set to black theme vs not setting any theme in xml).
colorAccent is set to #3ADBBC using AppCompat dark theme.
Perhaps using a TintRadioButton from AppCompat instead of a standard RadioButton would apply the correct tinting for API < 21?

Well, that's a good idea, but those views only pull the tint colors directly from your Activity theme. However, I might try to sub-class them later to pull out the custom values dynamically.

Commenting as a note to myself.
The tintDrawable method in TintManager (line 135) needs to be overridden and changed to use a custom color rather than extracting colors from theme attributes:
https://android.googlesource.com/platform/frameworks/support/+/c497221ee0e2e873f49653ff1a046df2e5276759/v7/appcompat/src/android/support/v7/internal/widget/TintManager.java

I actually believe PR https://github.com/afollestad/material-dialogs/pull/416 will solve this

https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>

Perfect, works as expected in <IP_ADDRESS>. Thanks for the quick response!


feat: Async render window/tab

Codecov Report

Merging #158 into master will decrease coverage by 1.75%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
- Coverage   27.94%   26.19%   -1.76%
==========================================
  Files          15       16       +1
  Lines         204      210       +6
  Branches       35       35
==========================================
- Hits           57       55       -2
- Misses        116      124       +8
  Partials       31       31

Impacted Files
Coverage Δ

src/js/components/Window/index.js
45.45% <ø> (-8.4%)
:arrow_down:

src/js/components/Tab/DraggableTab.js
10.52% <ø> (ø)
:arrow_up:

src/js/components/Tab/Tab.js
16.66% <0%> (-1.12%)
:arrow_down:

src/js/components/Window/Tabs.js
0% <0%> (ø)

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 32d17cc...276da5d. Read the comment docs.


Add Note for deploying resources to a different Subnet

See comments section:
https://azure.microsoft.com/en-us/documentation/articles/app-service-app-service-environment-intro/
Stefan Schackow:

"Hmm - is the Linux VM in the same subnet as the ASE? I think what is happening is that <IP_ADDRESS> is in the subnet address range containing the ASE. If that is the case, the behavior is expected because we block connectivity (except for specific intra-ASE routing) to any other IP address in the subnet containing the ASE. If you can redeploy the Linux VM to a different subnet, then connectivity will work.

Hit the same problem myself. Deploying to a different Subnet fixes connectivity. Decided to send PR for make benefit glorious nation of cloud.

Hi @snobu, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@stefsch can you sign-off on the note this customer is adding?

#sign-off

Thanks @stefsch  and thanks to @snobu for helping improve the quality of the Azure docs!


Union type support

Hi again, this library has no problem dealing with multi-option Literals, but when I try to use Union, it crashes with
ERROR:root:Unknown LMFormatEnforcer Problem:
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/tokenenforcer.py", line 78, in _compute_allowed_tokens
    self._collect_allowed_tokens(state.parser, self.tokenizer_tree.root, allowed_tokens, shortcut_key)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/tokenenforcer.py", line 119, in _collect_allowed_tokens
    self._collect_allowed_tokens(next_parser, next_tree_node, allowed_tokens, None)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/tokenenforcer.py", line 117, in _collect_allowed_tokens
    next_parser = parser.add_character(character )
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 40, in add_character
    clone.object_stack[-1].add_character(new_character)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 394, in add_character
    super().add_character(new_character)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 316, in add_character
    self.root.finish_parser(new_character)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 63, in finish_parser
    self.object_stack[-1].add_character(character_to_pass_to_parent)
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 245, in add_character
    self.current_key_parser = get_parser(
  File "/opt/homebrew/lib/python3.10/site-packages/lmformatenforcer/jsonschemaparser.py", line 168, in get_parser
    raise Exception("Unsupported type " + str(value_schema.type))
Exception: Unsupported type None

Example schema:
class test(BaseModel):
    test_value: float | int

Hi! Thanks for the report! Indeed unions are not supported yet. I'm working on adding a limitations / known issues section to make it easy to know what isn't supported yet, and also allow people to vote on what matters most to them.

Added support for this in v0.7.0!


Requirement already satisfied 'kamene'

I had installed all the requirements successfully. But while running 'kickthemout.py' by './kickthemout.py' - I'm getting error :
Details: No module named 'kamene'
To resolve that - when I entered 'pip install kamene' to install "kamene" - I'm getting this -
"Requirement already satisfied"

try python3 -m pip install kamene

Hi there @arshit09. Please follow the installation guide, located in the readme file of this repo. If you have any further problems, please open a new issue. Thanks!


e19: update

efl: 1.15.0 -> 1.16.1
elementary: 1.15.0 -> 1.16.1
emotion: 1.15.0 -> 1.16.0
enlightenment: 0.19.8 -> 0.20.3
evas: 1.15.0 -> 1.16.0
terminology: 0.8.0 -> 0.9.1
pythonefl_1_16: 1.15.0 -> 1.16.0

I've made sure it all compiles and runs as well.

@ftrvxmtrx Thank you very much for upgrading EFL and Enlightenment! Any chance that you've tried the wayland version of E20? I'm just curious :)

@ftrvxmtrx nice work, but I think that naming e19 is not appropriate any more, we should rename it to e20 or even better, name it enlightenment .. I am talking about folder/file names and nixos module name in configuration

@ts468 nope, haven't tried wayland. I mostly use e* stuff for efl and elementary, to write UI applications.
@matejc agreed, I'm gonna add another commit to this PR later today, hopefully.

Yeah, besides, e19 or even e feels too short.

Note: there's a nixos module that handles configuration reames – that would be useful for services.xserver.desktopManager.e19, for example, so that configs aren't broken.

@vcunat is that nixos/modules/rename.nix?

Yes. I hope the file is self-explanatory. I can't see any documentation about this...

@vcunat @matejc the "enlightenment" package currently points to e16. Do you think e19 should be the replacement for it?

No, it should not be a replacement, e16 vs e17+ is like gnome2 vs gnome3, but unlike gnome2, e16 is still maintained, we should upgrade it, not replace it.
That said... I have done some research on that how other distributions handle this, some do not have e16 at all, and some have enlightenment16. I would like that we rename enlightenment to enlightenment16 and then e19 to enlightenment.
But probably all of this weights on, if we find someone to maintain e16, last and first contribution to e16 was in 2009 by @civodul ... so we might be better of to just remove e16

I have done some thinking, if no one protests I will make a pull request that removes the enlightenment package (enlightenment16) from the nixpkgs, because it is not maintained, has no nixos module and because it solves the issue of naming e17+
What do you guys think?

Merging as some really good work has been put in this. Renaming of e19 -> enlightment can happen independently.


Rewrite DefEq by separating the lr map and the rl map

It is obvious. Fixes #275. Don't merge yet.

/cc @wsx-ucb

Ohhh!! I was trying to make a program compile, but it turns out that the program was wrong

bors r+


can this read Chinese paper?

Hi, you may see my implement. It can't read Chinese paper due to the PyPDF2 package. I have fixed this problem, please refer to my fork

@MrPeterJin Really interesting stuff! Might implement some of your changes. Let me know if you'd like to become a collaborator!

@MrPeterJin Really interesting stuff! Might implement some of your changes. Let me know if you'd like to become a collaborator!

Thank you for your comment! I'm glad that you found my work interesting and I would be happy to collaborate with you.

@MrPeterJin sent you an email! feel free to submit a pr with your new features!

你好，你可以看看我的工具。由于包装原因，它无法阅读中文文件PyPDF2。我已经解决了这个问题，请参考我的fork

您好，请问您的researchgpt私有化部署时需要配置代理吗？

你好，你可以看看我的工具。由于包装原因，它无法阅读中文文件PyPDF2。我已经解决了这个问题，请参考我的fork

您好，请问您的researchgpt私有化部署时需要配置代理吗？

您好，是需要的，因为它需要从 OpenAI API 接收信息


PCF8574 problem podczas kompilacji

Sorry answered wrong issue (deleted)

My PCF8574 library has no pinMode() function as it needs not to be set.
This is different from some other IO expanders e.g. MCP23017 which do have a pinMode().
So just leave out the line and it should work.

Any progress to report?

The adafruit pcf8574 library apparently has another interface that does support the pinMode command.
It probably does write a 0 or 1 to the device to simulate a pull up.
My library has no pinMode command but just uses the write command directly.
Note that the device has no DDR ==DataDirectionRegister.
If the adafruit lib works for you I assume your problem is solved and you may close the issue.

![1704460327517](https://github.com/RobTillaart/PCF8574/assets/126965848/255dd5b8-1999-4025-b784-5a9f6104a564)

Have a look at the examples to see how to use the library.
Thanks for trying.


Use "standard-version" in our release pipeline

Ask
While releasing our binary, we'd like to use a tool like standard-version to generate a CHANGELOG, new version number, and release notes.
Note that this tool is for javascript packages and we need to do some research to see how we can use it for our Go project.

See https://github.com/awslabs/goformation/blob/master/.releaserc from Goformation on how we can onboard our Go project.


Wrong date conversion -> the data 2024-06-14 is being converted to 2080-03-01 whereas it should b 2080-02-32

this issue is solved in the latest update. Please do update this package to solve this issue.


Future Amount incorrect in transaction data in QT (testnet)

Reproduce

Receive a future transaction
Navigate to Transactions
Double click transaction
Check "Future Amount"

This will be fixed with PR #73

fixed in develop


chore: Renaming inconsistent template folder casing - 2

#minor

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.486%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19900

Relevant Lines:
22308

💛  - Coveralls


navbar's icon-only link should have title in mobile mode

Is your feature request related to a problem? Please describe.
The current implementation(screenshot below) can be improved in terms of accessibility.

Describe the solution you'd like
like this.

My current implementation is something like this:
{% for page in site.pages %}
    {% unless page.title == null or page.hide or name_page contains page.title %}

 |

            {% if page.icon %} {% endif%}
            {{ page.title }}

    {% endunless %}
    {% assign name_page = page.title | append: name_page %}
{% endfor %}

This is a breaking change, but I can send a PR if you like this proposal.
Describe alternatives you've considered
Align icon-only links in a single row?
Additional context
Add any other context or screenshots about the feature request here.

Hi thanks!
I was thinking about it, but did not take the time to implement it.
I could take a look at your implementation and see where we can go from there.
Everything one online could maybe be done via some CSS rules using the fact that they are icons (so we could do it without breaking stuff).

Closing this one as it will be done in #394


Remove the xcode-install binstub

To ease the transition period I have left the old binstub in place for now, but it is undocumented. It should be completely removed in the next release.

:+1: Thanks for considering this !
Anyways, already upgraded my scripts to use xcversion.
By the way the current xcode-install script is broken for me in 1.0.1:
xcode-install --version
/Users/nghimbovschi/.rbenv/versions/2.2.3/bin/xcode-install:23:in `load': /Users/nghimbovschi/.rbenv/versions/2.2.3/lib/ruby/gems/2.2.0/gems/xcode-install-1.0.1/bin/xcode-install:2: syntax error, unexpected tIDENTIFIER, expecting keyword_do or '{' or '(' (SyntaxError)
	from /Users/nghimbovschi/.rbenv/versions/2.2.3/bin/xcode-install:23:in `<main>'

@neonichu this is actually not working:
$ xcode-install install
/Users/fkrause/.rbenv/versions/2.2.1/bin/xcode-install:23:in `load': /Users/fkrause/.rbenv/versions/2.2.1/lib/ruby/gems/2.2.0/gems/xcode-install-1.0.1/bin/xcode-install:2: syntax error, unexpected tIDENTIFIER, expecting keyword_do or '{' or '(' (SyntaxError)

It seems like the shell script is interpreted as a Ruby script for some reason.

Yeah, all things in bin need to be ruby -- they're not execed but rather loaded by rubygems

Thanks @segiddins. Is there a good way to do an alias in Ruby there?

@KrauseFx I've already made a PR ;)

@neonichu Ready to be closed. Thanks @segiddins :+1:


Two minor issues with todo-api example

Issue 1. The todo-api example also mentions a .env-example file which does not exist
Issue 2. When compiling the todo-api example:
error: current package believes it's in a workspace when it's not:
current:   ~/src/rust/sqlx/examples/postgres/todo-api/Cargo.toml
workspace: ~/src/rust/sqlx/Cargo.toml
this may be fixable by adding examples/postgres/todo-api to the workspace.members array of the manifest located at: ~src/rust/sqlx/Cargo.toml
Alternatively, to keep it out of the workspace, add the package to the workspace.exclude array, or add an empty [workspace] table to the package's manifest.

We have the other examples in the workspace so I think this was just an accidental omission.
The fix is definitely to add "examples/postgres/todo-api", here: https://github.com/launchbadge/sqlx/blob/e1d22a1840c07e0bbdd97791fc4993413e151384/Cargo.toml#L10-L12

However, the .env-example file does exist: https://github.com/launchbadge/sqlx/blob/master/examples/postgres/todo-api/.env-example
The leading dot causes it to be sorted above the other files.


fix: update range specifier for gts version with ^

This fixes the build by pinning the version of gts.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   90.14%   90.14%
=======================================
  Files           2        2
  Lines          71       71
  Branches        4        4
=======================================
  Hits           64       64
  Misses          7        7

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 2b7720b...0610407. Read the comment docs.

Updated :)  I can merge this since I don't have admin.

You should be able to merge this now.
On Mon, May 14, 2018 at 10:17 AM Justin Beckwith<EMAIL_ADDRESS>wrote:

Updated :) I can merge this since I don't have admin.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/google/google-p12-pem/pull/38#issuecomment-388894560,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE0qTHW9FNZxj_EENgYrDm7gHl2jbPBks5tybwdgaJpZM4T-CW3
.

--
Ali

I still can't 🤔


Drag and Drop between Columns with Mat-tableI

Bug, feature request, or proposal:
feature request
What is the expected behavior?
I would like to move the order of columns by drag and drop in mat-table
What is the current behavior?
I can only move between rows
Which versions of Angular, Material, OS, TypeScript, browsers are affected?
Angular 7.0.0
Angular Material 7.0.1
Is there anything else we should know?
Any demo solution in StackBlitz will be appreciated.

Any update on this?

Seen this
And found a working stackblitz:
column drag and drop

I am not sure what causes the issue but this is not working on latest angular and material

it's not working on these versions

I wrote a quick stackblitz for this. Angular Material 9 made the drag and drop columns much easier:
https://stackblitz.com/edit/mat-table-column-reorder

Thank you, @daveccrit 👍

I wrote a quick stackblitz using Angular Material 9, if that's an option. It made the drag and drop columns so much easier:
https://stackblitz.com/edit/mat-table-column-reorder

Could it be added sort function ?

Yes,  It can be added with sort function but it’s a different material api.
On Tue, Jul 21, 2020 at 2:13 AM DuoWeng<EMAIL_ADDRESS>wrote:

I wrote a quick stackblitz using Angular Material 9, if that's an option.
It made the drag and drop columns so much easier:
https://stackblitz.com/edit/mat-table-column-reorder
Could it be added sort function ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/angular/components/issues/13776#issuecomment-661251354,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHA33WYKAUIXGW6RHZW3RSDR4SCNXANCNFSM4F62H6KA
.

--
Best regards,
Romel Lauron
SOFTWARE ENGINEER|E: r rlauron<EMAIL_ADDRESS>|C:
(+63)<PHONE_NUMBER>

I wrote a quick stackblitz using Angular Material 9, if that's an option. It made the drag and drop columns so much easier: https://stackblitz.com/edit/mat-table-column-reorder

you really save me right now, thanks


Remove only custom button

Hi,
I want to show the custom button if layer geometry is "polygon" and remove if layer geometry is "linestring"
Is there any function to remove the customButton.
I tried following:
const customButton = {
        name: 'custom',
        title: 'Custom',
        className: 'control-icon leaflet-pm-icon-edit',
        onClick: () => {
        },
        afterClick: (e, obj) => {
        },
        doToggle: true,
        toggleStatus: false,
        disableOtherButtons: true,
        tool: 'custom',
        block: 'edit',
        actions: ['finishMode'],
    };

map.pm.Toolbar.createCustomControl(customButton) //Adding the button

map.pm.Toolbar.getButtons()['custom'].remove(); // It is not removing the button

You can remove it like that:
map.pm.addControls({
  custom: false
})

And showing it again with:
map.pm.addControls({
  custom: true
})

You can remove it like that:
map.pm.addControls({
  custom: false
})

And showing it again with:
map.pm.addControls({
  custom: true
})

map.pm.addControls({
custom: false
})

Not Working

map.pm.addControls({
custom: false
})

Not Working

Look into this example: https://jsfiddle.net/falkedesign/pk2jyg71/
It should work.
Change between custom: false and custom: true

Look into this example: https://jsfiddle.net/falkedesign/pk2jyg71/
It should work.
Change between custom: false and custom: true

Look into this example: https://jsfiddle.net/falkedesign/pk2jyg71/
It should work.
Change between custom: false and custom: true

Still not working. May be issue with npm package

Look into this example: https://jsfiddle.net/falkedesign/pk2jyg71/
It should work.
Change between custom: false and custom: true

Still not working. May be issue with npm package

In the demo it is working how you expecting?
Do you use the latest version of Geoman 2.9.0 and Leaflet 1.7.1?

In the demo it is working how you expecting?
Do you use the latest version of Geoman 2.9.0 and Leaflet 1.7.1?

Yes, i am using the latest ones

Yes, i am using the latest ones

You also use the same name as in your shared code? name: custom

You also use the same name as in your shared code? name: custom

You also use the same name as in your shared code? name: custom

Yes.
I tried locally. Issue with npm package

You also use the same name as in your shared code? name: custom

Yes.
I tried locally. Issue with npm package

Hmmm Typescript, vanilla, or which framework you are using?

Hmmm Typescript, vanilla, or which framework you are using?

using reactjs

using reactjs

Can you please create a demo like this: https://codesandbox.io/s/elated-villani-q0gei?file=/src/map/MapWithGeoman.tsx

Can you please create a demo like this: https://codesandbox.io/s/elated-villani-q0gei?file=/src/map/MapWithGeoman.tsx

I have a similar issue with #796


9.2 Late Requests

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Before: https://main--prisma-cloud-docs--hlxsites.hlx.page/
After: https://--prisma-cloud-docs--hlxsites.hlx.page/
Worker: https://prisma-cloud-docs-production.adobeaem.workers.dev/?branch=

added 9.2 updates that came in late


Fix Infinite Recursion

I think I've found and fixed two more places where infinite recursion can occur in java symbol solver.  At the very least, I tracked down infinite recursions in my runs to these lines and these changes fixed them for me.  It makes since that you shouldn't recurse onto yourself.  The better question might be why self is an ancestor by getAncestors, but I couldn't find a clear way to detect/fix that since the classes weren't the same, which is why I went down this route.

This seems a good thing. The fact it does not break any test should reassure us that the recursion was not intended here.
Thanks!


Allow mounting of ServiceAccountToken / specifying of ServiceAccount in Repo Server

Currently the repo server component has automountServiceAccountToken set to false. When attempting to use the lookup function with helm, this causes us some issues because it has no idea how to communicate with the cluster. It would be nice to be able to override this if this was desired and also to specify the serviceAccount that I would want to use here.

Thanks for your continued contributions! I am going to merge this and will go out in the next release.


What is the purpose of function model_completion_bfm?

Hey @jimmysue, this function tries to complete a full head using the some parts from FaceWarehouse model. Since 3DDFA fitting result cover only the main face region (no back head, no ears, no neck), augmenting using this mesh is difficult and error prone. Thus, we need to complete the face mesh before using it for augmentation. I hope this answers your question well?

Hey @jimmysue, this function tries to complete a full head using the some parts from FaceWarehouse model. Since 3DDFA fitting result cover only the main face region (no back head, no ears, no neck), augmenting using this mesh is difficult and error prone. Thus, we need to complete the face mesh before using it for augmentation. I hope this answers your question well?

@jacksoncsy But in 3DDFA_V1's training code, the shape of w_exp_sim.npy in train.configs folder is (159645, 10). 159645 is corresponding to  53215 points which contains ears and neck. Why did you say "3DDFA fitting result cover only the main face region (no back head, no ears, no neck)" ? I am really confused about this.


Ability to manually enter crop location / size

Some input fields that let you manually adjust the location and size of the crop box, for precise pixel control.

I'm thinking for a first iteration it can look something like this, with the row with inputs only opening when the crop editor is active.