Created URI Signing Plugin.

This is a first round of a URI Signing Plugin that conforms with https://tools.ietf.org/html/draft-ietf-cdni-uri-signing-12 . It's still a WIP, but I believe it's effectively complete. I would be very interested in your feedback, especially as this is my first submission of any substance.
I feel sure there are critical things I've failed to consider, so feel free to nitpick on even the slightest things. :)

[approve ci]

Needs to run clang-format on it.

@zwoop  And yup, I missed the format. I'll get that run.

[approve ci]

Is this intended as a replacement for url_sig?

Yes, though because they are incompatible, people are likely to need to run both for the near future.

It's also likely to have breaking changes in the near future.

@alficles You have conflicts with master and autest is failing. I think rebasing might fix both.

Oh, you should also squash the clang-format.

All right. I've fixed the stuff I said I'd fix, including the addition of about a million curly braces. O.o I've also rebased closer to master, so hopefully I've gotten both the conflict and autest failure in one step.

[approve ci]


Pilot-agent to implement a quitquitquit endpoint like envoy

Describe the feature request
It would be great if pilot-agent implemented a quitquitquit endpoint like envoy does.  This would enable us to programatically kill pilot-agent from other containers in the same pod, for example - at the end of a CronJob, or using a tool like https://github.com/monzo/envoy-preflight.
Describe alternatives you've considered
At the moment I build a custom istio-proxy image which wraps the pilot-agent binary with a crude script that presumes if envoy is not around for 5 seconds, it's probably crashed or been quitquitquit, so sends a SIGTERM to pilot-agent.  Then from our other containers we can call curl -XPOST http://<IP_ADDRESS>:15000/quitquitquit
#!/bin/bash
echo "Starting custom autotrader pilot-agent wrapper..."

/usr/local/bin/pilot-agent-original $* &
AGENT_PID=$!
echo "Pilot agent started with pid: $AGENT_PID"

cleanup()
{
  echo "Stopping pilot-agent"
  kill -s TERM $AGENT_PID
  exit 0
}
trap cleanup SIGINT SIGTERM

n=0
until [ $n -ge 5 ]
do
  if ! pidof envoy &>/dev/null; then
    echo "envoy is not running!"
    n=$[$n+1]
  else
    n=0
    tail --pid=$(pidof -s envoy) -f /dev/null
    echo "envoy exited, waiting up to 5 seconds for a new envoy to spawn..."
  fi
  sleep 1
done
cleanup

Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[ ] Developer Infrastructure
Additional context

Just to make sure I full understand, you have jobs and when they complete the pods stick around because Envoy/pilot-agent is still running?
Seems like a reasonable and pretty simple feature.
In the mean time may I suggest pilot-agent-agent for the name of your script :slightly_smiling_face:

Haha nice name.
Yes, basically we have cronjobs that require sidecars, when the main process exits 0 the sidecar process does not.
We work around this by having the entry point of the main container TRAP the main script and then curl envoy when it ends with quitquitquit so the Pod/Job then completes.

When we do this we should also probably make sure we drain Envoy rather than force quitting. We already have code for this with TERMINATION_DRAIN_DURATION so should be pretty straightforward

/assign let me do this

/close

@hzxuzhonghu do you know what release this will be in?  Looks like it isn't in 1.2.2 unless i'm missing something.

Thanks @incfly for noticing this and we have some security concern on the PR https://github.com/istio/istio/pull/15123.
Is the 15020 port accessible to everyone? If so, It seems a malicious pod could potentially shutdown another pilot-agent with a simple HTTP request.
Is 15020 enabled on ingress gateway by default? If so, it makes the situation worse as anyone can shutdown the pilot-agent on ingress gateway with a simple HTTP request.
I'm not sure the exact impact of shutting down the ingress gateway, but I don't think it's a good idea if it could be done by anyone, which could be a potential security vulnerability.
We need to evaluate and hold this before any release, @duderino @fpesce

I tried in a previous istio release. I'm definitely able to use public
ingress gateway ip to send request at 15020 port. If same applies for this
quitquitquit It will expose ingress gateway with being restart able by
anyone...
On Tue, Jul 2, 2019, 12:57 PM Yangmin Zhu<EMAIL_ADDRESS>wrote:

Thanks @incfly https://github.com/incfly for noticing this and we have
some security concern on the PR #15123
https://github.com/istio/istio/pull/15123.
Is the 15020 port accessible to everyone? If so, It seems a malicious pod
could potentially shutdown another pilot-agent with a simple HTTP request.
Is 15020 enabled on ingress gateway by default? If so, it makes the
situation worse as anyone can shutdown the pilot-agent on ingress gateway
with a simple HTTP request.
I'm not sure the exact impact of shutting down the ingress gateway, but I
don't think it's a good idea if it could be done by anyone, which could be
a potential security vulnerability.
We need to evaluate and hold this before any release, @duderino
https://github.com/duderino @fpesce https://github.com/fpesce
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/15041?email_source=notifications&email_token=AAMVXB43SQIBJCABHJLFBL3P5OXKDA5CNFSM4HZ6UQG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZCMDHA#issuecomment-507822492,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMVXB4CLC5IDZEDVYNLC6DP5OXKDANCNFSM4HZ6UQGQ
.

The fact that 15020 is open on the public ingress gateway seems crazy, and should be closed.  I can't see any logical reason to have public ingress traffic straight to pilot-agent?
Is 15000 open too (i can't check, I don't use ingress gateways and also have a very locked down NetworkPolicy which completely mitigates this issue).  I ask this as you can currently call 15000/quitquitquit to get envoy to quit.

@Stono I believe the envoy 15000 is only accessible from the localhost which means it cannot be called outside of the pod.

Looks to me like we can access 15020.. that doesn't seem right

Thanks, so we could expose another port in pilot-agent listen only on the localhost and handle the quit request. Does this make sense?

Yeah, I talked with @howardjohn. A reasonable fix can be listening on
localhost with a separate port.
Meanwhile, we should remove 1502 from ingress gateway service as well.
On Tue, Jul 2, 2019 at 6:32 PM Zhonghu Xu<EMAIL_ADDRESS>wrote:

Thanks, so we could expose another port in pilot-agent listen only on the
localhost and handle the quit request. Does this make sense?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/15041?email_source=notifications&email_token=AAMVXB7HF74UUF5UBCUVZNDP5P6TLA5CNFSM4HZ6UQG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZC7Y2A#issuecomment-507903080,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMVXB322MCVAW5MCO6TJJLP5P6TLANCNFSM4HZ6UQGQ
.

Meanwhile, we should remove 1502 from ingress gateway service as well.

Why remove this, then how do readiness probe for gateway?

We remove the k8s service object definition's port mapping, but not on the deployment object. readiness probe should continue to work, I think this can make public-ingress-gateway-ip:15020 unaccessible from internet, but i'm not 100% sure.

I know what you mean, SG

opened #15320

I'm not sure this is the right approach - in particular considering future plans on CNI.
Even exposing 15000 - with all the /admin capabilities - to the app seems risky, and I think should be avoided. I don't know if it's a good idea to assume that the sidecar will always be in the same network namespace with the application, or that application should have access to the /admin of envoy.
If I understand this correctly, the goal is to make sure the sidecar container is shut down when the job is completed.  Can we turn this around, and add an option for the agent to connect to a port exposed by the application, and when that port stop responding - to shut down ? This is also the mechanism used for readiness/lifeness, and same port could be used for both.
So instead of app killing envoy and agent by connecting to a port, the sidecar will suicide if the app
is not ready or send a special response on the app readiness port. At the same time we can lock down /admin - for example using a UDS socket to communicate with pilot agent.

The goal I thinj is to not have to make modifications to applications in
order for them to run effectively on the mesh.
https://github.com/monzo/envoy-preflight is an example of how the community
already does this with envoy, pilot-agent breaks this
Karl
On Mon, 8 Jul 2019, 22:08 Costin Manolache<EMAIL_ADDRESS>wrote:

I'm not sure this is the right approach - in particular considering future
plans on CNI.
Even exposing 15000 - with all the /admin capabilities - to the app seems
risky, and I think should be avoided. I don't know if it's a good idea to
assume that the sidecar will always be in the same network namespace with
the application, or that application should have access to the /admin of
envoy.
If I understand this correctly, the goal is to make sure the sidecar
container is shut down when the job is completed. Can we turn this around,
and add an option for the agent to connect to a port exposed by the
application, and when that port stop responding - to shut down ? This is
also the mechanism used for readiness/lifeness, and same port could be used
for both.
So instead of app killing envoy and agent by connecting to a port, the
sidecar will suicide if the app
is not ready or send a special response on the app readiness port. At the
same time we can lock down /admin - for example using a UDS socket to
communicate with pilot agent.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/15041?email_source=notifications&email_token=AALK7CPUQPXG6HZUINTRXOTP6OUDZA5CNFSM4HZ6UQG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZOLJNY#issuecomment-509392055,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALK7CMALN4IWFFGQBPXHK3P6OUDZANCNFSM4HZ6UQGQ
.

Also we have quite a few apps that don't expose http listeners, eg cronjobs

Fixed by #15406

1.3 which will be roughly a month from now

Thanks for the work on this! We just upgraded to Istio 1.3 in our lower environments and have been making good use of this already; allowing us to retire some hacky cleanup scripts and "is container completed?" API calls to determine job status independent of the pod/proxy.
However I was wondering if someone could clarify whether localhost:15000 (admin) or localhost:15020 (status) is the right port to use for this when running from an application container itself (assuming defaults for the ports) - with the intention of causing the sidecar container to complete/exit normally.
It's anecdotal, and I didn't really dig into it, but I initially tried with :15000 and it seemed to just cause an Envoy restart internally in the container (so job didn't complete) whereas switching to 15020 caused the sidecar to exit as expected and the Job to run to completion. Is that expected? Am I missing something?

yes that is expected . 15020 is pilot agent, which manages the envoy instance - if you kill envoy at 15000 it will restart it. So you want 15020. That is what we added, envoy had quitquitquit for a while

I see, thanks for the clarification! I didn't realise that 15000 was raw envoy - that explains it.
Thanks again for the work here, makes life much easier.


In SMARTY remove any initial types that match zero atoms

Example output after 9 iterations
Iteration 9 / 10
Attempting to create new subtype: '[#17]' (c_chlorine) + 'X1' (connections-1) -> '[#17X1]' (c_chlorine connections-1)
Atom type '[#17X1]' (c_chlorine connections-1) unused in dataset; rejecting.
Rejected.
INDEX        ATOMS  MOLECULES                                                          TYPE NAME                           SMARTS REF TYPE        FRACTION OF REF TYPED MOLECULES MATCHED
    1 :        464         42 |                                                       c_hydrogen                             [#1]       HC              244 /              244 (100.000%)
    2 :        232         42 |                                                         c_carbon                             [#6]       CT              232 /              232 (100.000%)
    3 :          0          0 |                                                       c_nitrogen                             [#7]
    4 :        107         42 |                                                         c_oxygen                             [#8]       OH               68 /               68 (100.000%)
    5 :          0          0 |                                                    c_phosphorous                            [#15]
    6 :          0          0 |                                                         c_sulfur                            [#16]
    7 :          0          0 |                                                       c_chlorine                            [#17]
    8 :          0          0 |                                                         c_iodine                            [#53]
TOTAL :        803         42 |                                                                                544 /      803 match (67.746 %)

Atom type hierarchy:
	[#1]
	[#16]
	[#8]
	[#53]
	[#15]
	[#6]
	[#7]
	[#17]

We shouldn't be wasting time creating atomtypes that don't exist in the molecule set.
I will address this will all the other things I'm fixing in pull request #198

I had thought I'd fixed this long ago, but it seems like (from looking at the issue tracker) what I really fixed was to allow unused base types to get destroyed. So, these should eventually go away (and then no longer be suggested) but certainly you can save some effort by just getting rid of them initially, @bannanc .

@davidlmobley I'm updating the smarty sampler as we speak, I already added a few lines to remove unused initial types.

fixed in pull request #198


fine-grained exception handling for blog.body

Description

sqlite3.IntegrityError does not suffice for a conclusion that CHECK constraint on blog.body has failed
It should be further examined to determine the specific CHECK constraint that has failed

This python docs explain more that can help in fine grained exception handling


nixos/update-users-groups: add support for account expiry

Description of changes
Complete  #203792 by adding a NixOS test and fixing the perl script so that it also works for newly created users.
Original PR description:
Add users.users..expires. This allows giving temporary access to a user to a machine, and not have
to remember to remove it afterwards.
Ping: @zimbatm @RaitoBezarius

thanks, looks good to go to me.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1071

This breaks cross-compiled NixOS because DateTime depends on Testutf8, which doesn't cross-compile, see: #198548

I will be going on holidays soon so apologies for not being able to deal with this, @lopsided98 how much realistic do you believe it is to fix the issue? Does it fail cross compilation on all targets or only armv7l?

It should fail on all targets, since it has something to do with the use of miniperl when cross-compiling. Personally, I have observed the failure on armv5tel, armv6l and armv7l.

A commenter here reports it broken cross-compiling for aarch64 as well.

I think we can use Time::Piece instead which does cross compile and also offers the needed functions. It's also a smaller library.
I'll test this and make a PR.


Merge changes  due to PR 28270

Hard code math calls for now until they are revised as well
Target_util only has intrinsics support

@sumesh13 rocm-xla has 39 fails. could you help look into them? also rocm-v2 seems to have one related failed case.


Migrate client to use GRPC V2

Purpose
The client currently interacts with the V1 GRPC API exposed by the node. Migrating it to use the V2 GRPC API has a number of advantages, including, but perhaps not limited to better types and payload deserialization using code generated from protocol buffer schemas.
Feature proposal
Due to differences in the V2 GRPC API, changes to the client module API may be introduced. Such changes will be documented in a feature proposal.
Changes

[x] Add bootstrapping to generate code from the V2 API protocol buffer schema (in Setup.hs).
[x] Migrate GRPC call helpers to use the V2 GRPC API:
Add functional "inverse" to the ToProto class (i.e. FromProto), allowing for conversion between datatypes generated from protocol buffer schemas and their corresponding datatype in (e.g. those in Concordium.Types.*, Concordium.Common) where applicable. These are not necessarily one-to-one, so checks should be added accordingly.
(Maybe) refactor ToProto and FromProto class and instance declarations to concordium-base.
Implement V2 GRPC API calls.
Rewrite/refactor client functions and helpers to use V2 GRPC API
Add tests for V2 GRPC API calls

This is completed now and 5.1.0 is released.


Dialog buttons are not focusable

Hi,
Like issue 211
On <IP_ADDRESS> i can't focus Dialog buttons (positive, negative, neutral) with D-pad.
On VM in API 25 and 21, box and android tv.
So my app need remote controler or gamepad...
Have you solution?
My current code:
new MaterialDialog.Builder(LoginActivity.this)
                                    .theme(Theme.LIGHT)
                                    .title(...)
                                    .content(...)
                                    .neutralText(...)
                                    .neutralColor(getResources().getColor(...))
                                    .btnSelector(..., DialogAction.NEUTRAL)
                                    .negativeText(...)
                                    .negativeColor(getResources().getColor(...))
                                    .btnSelector(..., DialogAction.NEGATIVE)
                                    .onNegative(new MaterialDialog.SingleButtonCallback() {
                                        @Override
                                        public void onClick(@NonNull MaterialDialog dialog, @NonNull DialogAction which) {
                                            ...
                                        }
                                    })
                                    .show();`

Thanks for your help.

Ok,
With your app i need to D-pad Down three times to have focus on button dialogs.
I think for user experience we need methods to set focus.
With remote, we need to have focus already set for me.
Thanks for any help.

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

Ok Nice,
It's work!
Thanks.


chore(main): release 0.6.4

:robot: I have created a release beep boop
0.6.4 (2024-02-17)
Bug Fixes

bump marked from 11.2.0 to 12.0.0 (#333) (40547a8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/nzdjb/static-site-generator/releases/tag/v0.6.4 :sunflower:


Monitor - support bicycle routes

The monitor was added to knooppuntnet to perform quality assurance on long distance walking routes.  For walking routes, the assumption is made that there is a single string of ways from start node to end node.
For bicycle routes, the ways to be followed in one direction are often different from the ways to be followed in the opposite direction. This is the case when passing through roundabouts or other one-way ways.
Bicycle route GTJ-Velo was added to group fr-nwn. Several segments that are left over after analysis, simply because the analysis was done in one direction only.
Adding support for bicycle routes (and performing analysis in both directions, just like for node network routes) would be helpful.

@pyrog Thinking a bit further: comparing a GPX track recorded in one direction, against a route definition in OpenStreetMap that includes forward and backward direction will never lead to a 100% match.
The 17 extra OSM segments in GTJ-Velo are mostly (if not all) a result of this.

In release v4.3.0, the route analysis logic has been changed. Bicycle routes are supported now.
Currently route GTJ-Velo consists of a single segment.


components: i2c: fix erroneous sanity check

The predicate for the I2C_CHECK macro describes what you expect to be true.
i2c_master_read used this macro in the wrong way. Fix that.
Signed-off-by: Thomas Basler<EMAIL_ADDRESS>Signed-off-by: Ralf Ramsauer<EMAIL_ADDRESS>

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@tbnobody See #1443


BrowserHostAppCommunication demo is broken after loading form in IDE

-DefaultURL property is introduced with a value of about:blank when form is loaded into IDE form designer.  Consequently, demo does not navigate to the expected content.  Fixed by resetting DefaultURL to an empty string prior to navigation call.  I assume this is more desirable than changing TWVBrowserBase.Create to use an empty string instead of URI_ABOUTBLANK

Thanks for the PR!
I just reverted the default value of TWVBrowser.DefaultURL to an empty string and that fixes the root of this issue.
Thanks again for taking time to fix this demo.


Run Error

This is a very good job. I run the fuse_dept. py file and the following error occurs.
The follow :
[Open3D Error] (class std::shared_ptr __cdecl open3d::core::CreateDeviceHashBackend(__int64,const class open3d::core::Dtype &,const class open3d::core::SizeVector &,const class std::vector<class open3d::core::Dtype,class std::allocator > &,const class std::vector<class open3d::core::SizeVector,class std::allocator > &,const class open3d::core::Device &,const enum open3d::core::HashBackendType &)) D:\a\Open3D\Open3D\cpp\open3d\core\hashmap\DeviceHashBackend.cpp:38: Unimplemented device
Traceback (most recent call last):
File "C:\Users\23860\Desktop\深度估计与SLAM融合\参考论文\MonoNav\MonoNav-main\fuse_depth.py", line 51, in
vbg = VoxelBlockGrid(depth_scale, depth_max, trunc_voxel_multiplier, o3d.core.Device(device))
File "C:\Users\23860\Desktop\深度估计与SLAM融合\参考论文\MonoNav\MonoNav-main\utils\utils.py", line 50, in init
self.vbg = o3d.t.geometry.VoxelBlockGrid(
RuntimeError: [Open3D Error] (class std::shared_ptr __cdecl open3d::core::CreateDeviceHashBackend(__int64,const class open3d::core::Dtype &,const class open3d::core::SizeVector &,const class std::vector<class open3d::core::Dtype,class std::allocator > &,const class std::vector<class open3d::core::SizeVector,class std::allocator > &,const class open3d::core::Device &,const enum open3d::core::HashBackendType &)) D:\a\Open3D\Open3D\cpp\open3d\core\hashmap\DeviceHashBackend.cpp:38: Unimplemented device
I would appreciate it if you could reply to me！

Hi @dasda-asd , this looks like a GPU issue to me. Could you confirm your device setting in config.yml?

You can try setting device: "CPU:0" and retrying fuse_depth.py. If the code runs, it is likely the issue is with the GPU.
Can you also check whether your GPU is recognized by your computer? (E.g., try running nvidia-smi in your terminal.) This is a good way to check whether your GPU is properly recognized.

I hope this helps!

Hi @dasda-asd, was your issue resolved?


Kraken sell : Order:Insufficient funds

Note: for support questions, please join our Discord server

I'm submitting a ...
[ X] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
Try to sell, but for some reason, the gateway is in error. The code want to retry, but the order is allready passed. The error is then 'Order:Insufficient funds'.

Expected result (what you hoped would happen)
No error.

Actual result (unexpected outcome)
Loop with retry.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
I propose to add is method setOrder(...)
  if(err) {     log.error('unable to ' + tradeType.toLowerCase(), err);     if (err.message.startsWith("Order:Insufficient funds")) log.warn("Insufficient funds. Probably order canceled")     else return this.retry(this.addOrder, args);   }

Just to clarify, in this case you expect that the error will be disregarded and a retry not attempted?
Given how often error responses (especially HTTP 8xx errors) usually mean the order has posted I think this approach makes sense. However if you order is for less than half the balance it would still mean possibly posting the order twice.
Another argument here is that perhaps buy/sell orders shouldn't be retried on failure.

@cmroche a lot of people are having this problem though, see #1255 as well.

I'll add this into my list of issues to fix on Kraken. I'll implement as you suggest @askmike though it may take a week or two due to my schedule.

I might have some time later this week, if so I will take a stab at it
first :)
Regards,
Mike
On Tue, Oct 31, 2017 at 4:41 AM, Clifford Roche<EMAIL_ADDRESS>wrote:

I'll add this into my list of issues to fix on Kraken. I'll implement as
you suggest @askmike https://github.com/askmike though it may take a
week or two due to my schedule.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1259#issuecomment-340593347, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MD78xchIQKx5mPkcRwgcssCYvkhcjks5sxkJugaJpZM4QKAKz
.

--
PGP key at keybase.io/mikevanrossum
https://keybase.io/mikevanrossum/key.asc

(same fix applies for this and #1258)
@cmroche has proposed a fix which is now available in the develop branch, would you mind checking whether this solves the issue?

Also related to #1339
The poster over there was able to confirm that the fix resolved his issue.

Hi,
I keep on getting the same issue:
2018-05-07 16:20:40 (DEBUG):    [kraken.js] entering "setBalance" callback after kraken-api call, data: { error: [], result: { ZEUR: '20.0000' } }
2018-05-07 16:20:40 (ERROR):    Kraken did not return portfolio for XBT, assuming 0.
2018-05-07 16:20:40 (WARN):     Wanted to buy XBT but the amount is too small  (0.00258672 @ 7731.10000000 ) at kraken
The minimum amount in kraken is 0.002 XBT:
https://support.kraken.com/hc/en-us/articles/205893708-What-is-the-minimum-order-size-
Has the fix been published ?
Thanks
Xavier

Kraken is a lot more stable now, it had a number of fixed in the latest develop branch. Please try that out :)

Sorry to be the bearer of bad news but I still have this problem with XLM on Kraken -
2018-09-23 19:50:08 (INFO):	Trader Received advice to go long. Buying  XLM
2018-09-23 19:50:08 (WARN):	NOT creating order! Reason: Amount is too small
Having read through this thread I've updated to the latest developer branch but the problem still persists.
The minimum purchase qty is 30 and I've got enough EUR in the account to buy about 90.
I'm running 11 other tradebots on kraken without any issue - just this one that I've found so far!
Anything I can try?
Thanks! :)

Do you have more debug logs? Please post them as they include the amounts.
On Mon, 24 Sep 2018, 02:55 g1tt<EMAIL_ADDRESS>wrote:

Sorry to be the bearer of bad news but I still have this problem with XLM
on Kraken -
2018-09-23 19:50:08 (INFO): Trader Received advice to go long. Buying XLM
2018-09-23 19:50:08 (WARN): NOT creating order! Reason: Amount is too small
Having read through this thread I've updated to the latest developer
branch but the problem still persists.
The minimum purchase qty is 30 and I've got enough EUR in the account to
buy about 90.
I'm running 11 other tradebots on kraken without any issue - just this one
that I've found so far!
Anything I can try?
Thanks! :)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/askmike/gekko/issues/1259#issuecomment-423838921, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA7MDz_u-_V7_40mXXauZwc3NVyjy_jeks5ud9kLgaJpZM4QKAKz
.

Thanks for the response. I was just coming back here to reference the issue I just created. I'll post everything there in a few mins!

closing this one, it's old and outdated!


When I used Masonry in MRC ,It crashed! What should I do?

New Issue Checklist
🚫 If this template is not filled out your issue will be closed with no comment. 🚫

[ ] I have looked at the Documentation
[ ] I have filled out this issue template.

Issue Info

Info
Value

Platform
e.g. ios/osx/tvos

Platform Version
e.g. 8.0

Masonry Version
e.g. 1.0

Integration Method
e.g. carthage/cocoapods/manually

Issue Description
⚠ Replace this with the description of your issue. ⚠

This is the log of the crash:
*** Terminating app due to uncaught exception 'CALayerInvalidGeometry', reason: 'CALayer bounds contains NaN: [0 nan; 50 nan]'


Use of fields in nested attributes as keys in relations

Originally reported by @Lisio at https://github.com/yiisoft/yii2-mongodb/pull/53
Required if we have to get all files (without content) related to some documents. In this case we are recomended to use metadata field as a container for other fields. For example, Image model will have the following properties:
{
    "metadata": {
        "post_id": ObjectId("abcd01234567890123456789"),
        "width": 640,
        "height": 480
    }
}

and relation in Post model can be declared like this
public function getImages()
{
    return $this->hasMany(Image::className(), ['metadata.post_id' => '_id']);
}

I suppose if such enhancement is to be made it should be placed at ActiveRelationTrait since it may affect not only MongoDB. For example: ElasticSearch may have the same use case.
The problem with the relation by nested attribute is there is not way to declare a reverse relation to it.
How Image will relate to the Post in this case?
In order to suport this feature we need to compose full-scale nested attributes logic at the top of yii\db* layer.

need do full support for inner form - load, validate, populate

Another open questions is how such a relation is handled in a non lazy loading case.

Is it really preferable to insert a dimension inside a dimension? From an architectural perspective this is questionable

For fs.files collection the only way to store references correctly is to put them inside metadata property.

can you elaborate?


fix(devtool): use correct CSS to hide psuedo element used for hacking…

… the width of the wrapping content to be at least the width of the green class tag / decoration

Changes

Checklist

[x] Added markup tests, or they don't apply here because...
[x] Updated the changelog at CHANGES.md

This was broken for me in that the padding on the bottom was negative or something weird.  How does this look for you?

Just pushed this

So far, it seems the best way to align things is to not touch the display property on the span element... which is obnoxious, because it makes aligning things to the baseline hard

but, I did find a really weird display option that allowed the border to expand... idk 🤔 Gonna keep looking

What was wrong with what I showed you again?

I don't remember you showing me anything?

also, I think this is in the best place I can get it for now.
I am not happy with the various ways that blocks are broken up and separated from wrapping elements (because of how the code is wrapped with spans).
For example:
<style>
  code
</style>

all 3 of those lines end up being sibling elements, which makes any sort of display: inline-* not great. but, we need inline-flex for align-items: baseline.
but, with inline-flex, often, entire sections of code are moved into one line, which is worse. :upside_down_face:

Grrrr, I really thought I pushed a commit... let me take a look at this latest version.

@joshgoebel Rebased and fixed the text-alignment issue (I think)

There is also a compressed version if we want 🤷

Compressed please.

Spacing reduced and PR rebased


Update version-increment-check.yaml

Workflow can fail without stopping, which seems to be a bug as seen in the link below:
https://github.com/Energinet-DataHub/greenforce-frontend/runs/5316121408?check_suite_focus=true

@MartinSchmidt - I am assuming you thought I made the true -> false change. I did not. I am suggesting to revert that change and instead make the changes I proposed.


Implement Publishing Companies Page

Prototype: https://planettelex.github.io/Songtracker-Pro/prototypes/v_0.05/#g=1&p=publishing_companies

Add form validation.

Add an edited UI alert.


git-pwd doesn't display its result

The current rev (048e76d75cb5c0a50b10167b856c847737ef0a1d) doesn't load the XGIT context for some reason.
I am investigating.

It's actually a display problem. Results of commands aren't making it through to the printing phase of the REPL.


macOs cellpose wrapper

Hi,
I see that Linux and MacOs are not supported by the cellpose wrapper. Is there a particular blocker for this?
After installing cellpose on osx following the default procedure, I can run it from the command line like this:
conda activate cellpose ; python -m cellpose --dir /Users/jmutterer/Downloads/images/pollen --pretrained_model nuclei --diameter 0. --save_png
Thanks!

Version 0.7.0 works on MacOS 🎉

Great news thanks!


Set the default node to be the same for both http and indexer

Description
Set the default node to be the same for both http and indexer.
Steps to Reproduce
Look at the default configuration for http and indexer.
Expected Result
Both components should point to the same node.
Actual Result
Http points to https://grpc.testnet.concordium.com:20000/
Indexer points to http://localhost:20001

The difference here lies in the fact that the node for the http binary is used in the configuration for the voting dApp, meaning it needs to have grpc-web enabled (which is not the default). The node used by the indexer benefits from being local to the location of the indexer as it traverses all the transactions on chain.


[AutoPR storage/resource-manager] Adds Geo Replication Stats in Azure Storage Account's properties

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4455

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/3310


formatting error in post titles

Please sanitize formatting for special characters in post titles, such as "&"

@lorenjohnson is it possible the work you did recently might have impacted/broken this?

Unless something went wrong, this will not happen with posts after this release. A post on March 4th would be before the text formatting release last week.
The remaining work here is to migrate existing post titles at the database level to decode HTML entities from titles (the only fields that inject HTML into the page are Post.details and Comment.text). Titles and a number of other fields we were over-sanitizing are not HTML fields and shouldn't be being encoded/decoded. Previously both were happening so legacy titles in the DB were encoded and are not now being decoded. We shouldn't cautiously decode as this will cause a regression to a bug Tom fixed a month or so ago such that < and >, etc will be sanitized out of titles.
NOTE: Only Post.details/description and Comment.text need to be sanitized which always happens on the backend when retrieved.

Ah yes, ok I fixed this post just by editing the title and re-saving with an ampersand. Maybe we just run a database query looking for other ampersands to start

On Mar 15, 2022, at 2:06 PM, Loren Johnson @.***> wrote:
Unless something went wrong, this will not happen with posts after this release. A post on March 4th would be before the text formatting release last week.
The remaining work here is to migrate existing post titles at the database level to decode HTML entities from titles (the only fields that inject HTML into the page are Post.details and Comment.text). Titles and a number of other fields we were over-sanitizing are not HTML fields and shouldn't be being encoded/decoded. Previously both were happening so legacy titles in the DB were encoded and are not now being decoded. We shouldn't cautiously decode as this will cause a regression to a bug Tom fixed a month or so ago such that < and >, etc will be sanitized out of titles.
NOTE: Only Post.details/description and Comment.text need to be sanitized which always happens on the backend when retrieved.
—
Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/issues/1141#issuecomment-1068358084, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAA5HPCCO5E626FDOLJ7MTVADNURANCNFSM5QZPRPRQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were assigned.

Yes, you got it--good instincts. Sanitizing posts before the last release will give us the titles that were presenting before so I think that is the move. I removed "goodfirstticket" as this needs a little care, but is still a small thing.


[BUG] Provider produced inconsistent result after apply

What is the bug?
When applying changes to opensearch_dashboard_object resources in multiple tenants, the provider produced inconsistent results after apply. The following error is thrown for random resources, and multiple runs of terraform apply are required to successfully create all resources:
Error: Provider produced inconsistent result after apply

When applying changes to opensearch_dashboard_object, provider "provider[\"registry.terraform.io/opensearch-project/opensearch\"]" produced an unexpected new value: Root resource was present, but now absent.

This is a bug in the provider, which should be reported in the provider's own issue tracker.

How can one reproduce the bug?
Define multiple opensearch_dashboard_object in a Terraform configuration.
Run terraform apply to create the resources.
Example:
resource "opensearch_dashboard_object" "local_index_pattern_v7" {
  for_each    = var.project_list
  tenant_name = each.key
  body        = <<EOF
[
  {
    "_id": "index-pattern:ck-${lower(each.key)}-local",
    "_source": {
      "type": "index-pattern",
      "index-pattern": {
        "title": "ck-${lower(each.key)}-*",
        "timeFieldName": "@timestamp"
      }
    }
  }
]
EOF

  lifecycle {
    ignore_changes = [body]
  }
}

resource "opensearch_dashboard_object" "global_index_pattern_v7" {
  for_each    = var.project_list
  tenant_name = each.key
  body        = <<EOF
[
  {
    "_id": "index-pattern:ck-${lower(each.key)}-global",
    "_source": {
      "type": "index-pattern",
      "index-pattern": {
        "title": "ck-*",
        "timeFieldName": "@timestamp"
      }
    }
  }
]
EOF

  lifecycle {
    ignore_changes = [body]
  }
}

What is the expected behavior?
All opensearch_dashboard_object resources should be created successfully in a single run of terraform apply without producing inconsistent results.
What is your host/environment?
Operating System: macOS
Terraform Version: 1.4
OpenSearch Provider Version: 2.3.1
Do you have any screenshots?

Do you have any additional context?
No.

I have similar error when I am destroying these resources. Again, multiple terraform apply runs are required.


Support not only infura and alchemy, but regular RPC endpoint

Loking in ifura and alchemy is not always preferrable. A lot of web3 developers using own RPC endpoints (just as an example, you can get one at QuickNode and don't need any keys to use that endpoint).
Do you have any plans to support non-vendor locking RPC endpoint?

Oh, this was just given as examples, but you can just use any RPC endpoints instead. You can check in src/constants/networks.ts to see how they are used. BSC uses a different one for instance.
Ideally, you should even have 2/3 nodes for each chain that you're using, so if one goes down, you still have a backup node.


Is There an Encoder Process?

When I look at the code of Transformer-XL for TensorFlow, in model.py, in function "transformer", the parameters contain "dec_inp", which seems to be the decoder input. I can't find the parameters for encoder in the code file. Is there an encoder process? If so, where is the encoder process?

same problem, seems missing decoder in tf implementation

seems like missing encoder in tf implementation, who have solved the problem?


Add WAFERS'21 paper

Please add the WAFERS'21 paper to the publications, see instructions in readme here: https://github.com/ftsrg/theta/tree/gh-pages/publications

@as3810t maybe it's time to add it? :) (I'm just cleaning up issues and found this.)


feat(forge): account access cheatcode accounts for extcode* and balance opcodes

Motivation

vm.startStateDiffRecording should track the EXT* family of opcodes, since they affect account warmth; see #6544
Solution

Adds 3 more values to the AccountAccessKind enum and watches for those 3 opcodes as part of step.

Typically, the AccountAccessKinds indicate an EVM context switch. The Extcode* opcodes breaks this expectation. What do you think about indicating them in the StorageAccess struct? This way, all extcode* accesses are grouped under the current context.

How would you modify StorageAccess to accommodate? Could add a new extcode enum to the end and rename it something like StorageOrExtcodeAccess, but imo that just makes the struct more confusing.

You're right, that would be more confusing. I guess this is fine as-is then. With the Resume access kind, we probably could even flatten both storage and account accesses in a single array. As it's now possible to link contexts.
@JuanCoro care to chime in, as I think you may have a unique usecase for this cheat code.

Update to include balance opcode as well, since that marks the target account as warm. I think that's all of them, for real this time – any I might be missing?

@Inphi thanks for the ping! This looks good to me, I don't think it will be any trouble for what I'm using it :ok_hand:

@emo-eth FYI https://docs.github.com/en/get-started/writing-on-github/working-with-advanced-formatting/using-keywords-in-issues-and-pull-requests


UI testing for the demo app

Following up #450, when we do refactor, we may accidently break some features.
I don't know the best practice for unit testing UI. We developed a demo app (IBAnimatable Playground) to test our features. We use it for testing new features, fixing bugs and doing refactor. But it is not straight forward and we have to check them manually. If we can have some UI Testing in Xcode. We may be able to integrate it with CI and check for every PR.
We don't have to do it all in one go. We can develop it bit by bit:

[ ] User Interface: It has a list of UI components like placeholder, mask, etc, we can do it one by one.
[ ] Animations: Changing the picker view.
[ ] Activity Indicator: Changing the picker view. That would be an easy one to start with
[ ] Refresh Control: Pulling the table view to refresh.
[ ] Transitions: Clicking on the table view cell.
[ ] Presentations
[ ] Interactions

Even if the current state of XCUITest are a bit limited (can't test the layer, the animations...), we can still do basic animations with a simple navigation which ensure us to be able to launch each feature without crashing.
Let's hope that will be improved in the future version of XCUITest!


Remove all the flake8 ignores

Current setup.cfg has these lines:
# E402: module level import not at top of file
#   (standard_library.install_aliases)
# E501: line too long error
# E266: comment beginning with ## error
# F403 and F405: py2 'from module import *' error
# F999: py3 'from module import *' error
ignore = E402,E501,E266,F403,F405,F999

All of them should be removed, and the code fixed to pass the flake8 checks without requiring any kind of ignores.

Done in branch https://github.com/HDI-Project/ATM/tree/GH_105_remove_flake8_ignores
Waiting for PR #104 to be merged to fix the expected conflicts and create a PR.


#slug should not default to _id.to_s

#slug should not default to _id.to_s. This is inconsistent with the rest of the gem:

_slugs does not contain _id.to_s
clear_slug! sets _slugs to [], so #slug will return nil in this case
it will be overwritten when saving
before saving, one would expect both slug and _slugs to be nil

Ooooof I know for sure that Artsy relies on the fact that slugs default to ID, and aren't nil. This is a massive breaking change, isn't it?

@dblock actually we should make this behavior configurable. As noted above, its actually inconsistent. A "never touched" model will have slug default to ID, but once you call clear_slug! the slug value becomes nil. I can investigate further on a way to preserve the behavior for Artsy.

@dblock we should make this behavior configurable. As noted above, its inconsistent. A "never touched" new model will have slug default to ID, but once you call clear_slug! the slug value becomes nil. I can investigate further on a way to preserve the behavior for Artsy.

Sounds good. It should be backwards compatible or we need to bump major and document the breaking change.

@dblock we should make this behavior configurable. As noted above, its inconsistent. A "never touched" new model will have slug default to ID, but once you call clear_slug! the slug value becomes nil.

I actually don't think it's inconsistent, but can be misinterpreted. Don't you want to clear the slug on clear_slug!? Maybe we need a clear_slug_and_set_to_default!?

I can investigate further on a way to preserve the behavior for Artsy.

Sounds good. It should be backwards compatible or we need to bump major and document the breaking change, all while allowing the functionality to exist.


False positives when HTML optional opening tags are not present

Noticed it with the latest htmlminifier version https://github.com/kangax/html-minifier/commit/3ab98e86434857511dc33a68b32d9e8b9c334378#commitcomment-16219534
Should be reported upstream too, opened this to track the issue.

X-Ref: #249

Sounds like this is a known issue: https://github.com/fb55/htmlparser2/issues/39


Use a preprocessor

It would be cool to use a preprocessor such as Sass or LESS as it would make writing and maintaining this project easier.

Just pushed the new Sass version to master :)


fix: Capture as many channel mentions as possible

What type of pull request is this?

[x] Non-breaking code change
[ ] Breaking code change
[ ] Documentation change/addition
[ ] Tests change

Description

This adds in some extra regex matching during Message._process_dict to scan the message content for more channel mentions, and then process them as if they were normal channel mentions.
This bug is seen here: #324
While this isn't technically a bug, but rather an API oddity, we still believe that it's necessary to "fix" this for the end users' sake.
Changes

Add regex to Message._process_dict to capture more channel mentions

Checklist

[x] I've formatted my code with Black
[x] I've ensured my code works on Python 3.10.x
[x] I've tested my code

merge conflict 💀

merge conflict 💀

Fixed


pub build of a web app is no longer running transformer on dependency package?

This issue was originally filed by<EMAIL_ADDRESS>
1.2.0.dev_03_02
Win7x64
I have a web app that uses pub build.  The web app has a dependency on 1 other package (docs).  That other package itself has a dependency on 1 other transformer package:
app -> docs -> markdown_transformer
The 'docs' package applies the transformer (which converts markdown to html) on its markdown:
dependencies:
  markdown_transformer:
    ...
transformers:

markdown_transformer

Up until this version of the SDK, when I ran pub build on the web app, I got the transformed html from the transformer in:
build/packages/docs/*.html
I understand that these should now be expected in build/web/.., however they are no longer there at all.
pub serve continues to function as before.

This issue has been moved to dart-lang/pub#845.


"webkit-overflow-scrolling" Safari has add this to iOS 5 to build applications with a more...

"webkit-overflow-scrolling"
Safari has add this to iOS 5 to build applications with a more native feel.
http://developer.apple.com/library/safari/#documentation/appleapplications/reference/safaricssref/articles/standardcssproperties.html

This issue was imported from Google Moderator
Moderator votes: +4
Issue added by Anthony on 2012-04-11
To vote this issue up or down, simply include +1 or -1 in your comment.

Duplicate of #1308.


flutter: Error: google_fonts was unable to load font Oxygen-Regular because the following exception occurred:

Unable to download any fonts. It was working until Oct 18 2022. And then for some reason started getting this error.
Error
flutter: Error: google_fonts was unable to load font Poppins-Regular because the following exception occurred:
Exception: Failed to load font with url: https://fonts.gstatic.com/s/a/705290b12f58c6d70aafcaaf461dbc3d2f7f19d0f4362af1843b107d95d4960a.ttf
flutter:
See https://docs.flutter.dev/development/data-and-backend/networking#platform-notes.
flutter: If troubleshooting doesn't solve the problem, please file an issue at https://github.com/material-foundation/google-fonts-flutter/issues/new .

flutter: Error: google_fonts was unable to load font Oxygen-Regular because the following exception occurred:
Exception: Failed to load font with url: https://fonts.gstatic.com/s/a/018516876dd67d3188322375bd244200747d26c662efc13f53bc3c0d650e19c1.ttf
flutter:
See https://docs.flutter.dev/development/data-and-backend/networking#platform-notes.
flutter: If troubleshooting doesn't solve the problem, please file an issue at https://github.com/material-foundation/google-fonts-flutter/issues/new .

DebugProfile.entitlements:
I have the following configuration in my .entitlements.
<key>com.apple.security.network.server</key>
<true/>

How to reproduce?
update text theme as shown below. I also tried a few text theme. Pretty much all of them got the same error.
textTheme:  GoogleFonts.poppinsTextTheme()

or

textTheme: GoogleFonts.oxygenTextTheme()

Platform:

web
macos

Note I did not test this on android and ios.

Flutter Version:
╰$ flutter --version
Flutter 3.3.5 • channel stable • https://github.com/flutter/flutter.git
Framework • revision d9111f6402 (5 days ago) • 2022-10-19 12:27:13 -0700
Engine • revision 3ad69d7be3
Tools • Dart 2.18.2 • DevTools 2.15.0

same after upgrading to flutter 3.3.5
I/flutter (19221): Error: google_fonts was unable to load font Roboto-Regular because the following exception occurred:
I/flutter (19221): Exception: Failed to load font with url: https://fonts.gstatic.com/s/a/d1d7c5f4500eeb1a09e051781906c3642015a3f6c9b69046b905c8bf34c6ad60.ttf

I got this same error for the android platform. Is there any solution to this?

did you follow the troubleshooting steps in mentioned in the error?


In gobot model ..how should we display the intent of a particular dialog

I want to display the intent of "hi what can i help you with"

There is no such a feature in go_bot right now though I'm working on the ability to configure responses format right now. The PR is #1173 , it's WIP.


Link underlines sit too low in Firefox

As originally raised by @dashouse on GOV.UK Template, link underlines looks like they're sitting below the text (almost as if they are using text-underline-position: under; (which they aren't). This is affecting GOV.UK Frontend as well:

Dave identified that this started happening in Firefox 39, and from doing some digging it looks like Firefox 39 was the release that enabled support for WOFF2, and this seems to be related – commenting out the woff2 file from the font-face declaration 'fixes' it. We think this is something to do with the way that the text decoration is aligned with the 'baseline' (which we know is 'high' in the current cut of our font-face) in woff2 fonts.
However, it looks like updating to use the new cut of the font, with the normalised baseline, will fix this, so we might not need to do anything except prioritising the work to move to the new font:

Looks like this will be addressed with the font change in 3.0 as we expected.
Before

After

I wonder if we should list this as a fix in the changelog? / cc @aliuk2012


Fixing App.vue so it is asynchronous

Changing App.vue so Store references including rest API calls are asynchronous.

Codecov Report

Merging #984 (fe7f554) into master (1b876c1) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master     #984   +/-   ##
=======================================
  Coverage   66.85%   66.85%
=======================================
  Files         156      156
  Lines       16975    16975
  Branches     3688     3688
=======================================
  Hits        11348    11348
  Misses       5627     5627

Impacted Files
Coverage Δ

src/App.vue
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Bring dspace-oai Spring configuration / service handling in line with other modules

While trying to find a good way to use a mock OAI Solr core for integration testing, I realised how differently the dspace-oai module uses spring compared to other modules. Something that seemed like it should be relatively simple (define a mock OAI solr core/service in solr-services.xml which will be autowired in place of the usual http service) ended up getting quite tricky to pull apart and plan.
See: org.dspace.app.xoai.BasicConfiguration, the resolvers, and usage in other classes.
This issue might be closely related to https://github.com/DSpace/DSpace/issues/8112 (dependency injection issues)
cc: @mwoodiupui
Solutions?
I wanted to open this ticket to discuss possible solutions... I think a refactor to bring dspace-oai closer to how the rest of the codebase works will make things easier for developers in general.

Application context (See DSpace/DSpace#8112) - remove this idea entirely?
Load the BasicConfiguration info needed to run core OAI services from spring XML instead of a compiled class
Refactor code to use a Solr Service like discovery, statistics, authority and get rid of the 'Solr Resolver'
There are likely other things that can be moved to beans defined / referenced in XML and simplify configuration loading in the dspace-oai endpoint code
Anything else?

Much of this sounds like the way I wanted to address #8112.  I've long thought that "configuration classes" are a weird notion anyway, and here this makes it a lot harder to do simple normal things.  +1


Correction for raw affiliation King's University College at Western University

Correction needed for raw affiliation King's University College at Western University
raw_affiliation_name: King's University College at Western University
new_rors: 02grkyz14
previous_rors: 048f5b183;05cgtjz78
works_examples: W4366819300;W4386458302;W4367296517;W4379014927;W4387362260;W4384561123;W4389352341;W4385884103
contact: 32eec2ff777c2ae1890b3cb3552908a9:dd7e2706f7ea9813 @ uwo.ca

This issue was accepted and ingested by the OpenAlex team on 2024-12-21. The new affiliations should be visible within the next 7 days.


ENEError 11

<!-I am receiving the following error after 14 days of use of the corona warn app. What to do?
I even deleted the app and uploaded it again from the App Store. The error still exists.
I am using an iphone X, 13.5.1
Thanks for your support.

Hi Splashcandy, thank you for your contribution and reporting this bug. I transferred your issue to the iOS repository and now see that it's been raised 2 days ago: https://github.com/corona-warn-app/cwa-app-ios/issues/792
Therefore, I'm closing your issue as #792 is open until it's resolved.
Best regards
MC
Your Corona Warn-App Open Source Team


62 add test coverage for MethodAnalysis

Issue #62: Add test coverage for FindMethodInvocationsAnalysis and FindMethodDeclarationsAnalysis
Hi, along with the tests I have made some changes to a couple of files (due to same kept the changes in a draft PR) - please find the reason below:

MatchedMethodResult:

line#9: updated the access specifier to public as its instance is required in test package for expectedResult comparison.
line#22: overloaded the constructor to accept the matchedNode as String - this change I didn't want to make but for creating expectedResult object it is required to create an instance of this class. Since its not possible to explicitly create an ASTNode and just for testing purposes - it doesn't seem like a good idea to parse the ExampleClass and visit/search for the node (basically it would be redundant as to what is being done for the FindMethodInvocationAnalysis itself).
Also ASTNode.toString() i.e., string value for ASTNode is stored in the member matchedNode anyway.
line#40: added equals() implementation for comparison of the objects

MethodAnalysisResult:

line#13: added public access specifier for same reason as above.

DescribedPredicate:

line#33: implemetation override for equals()/hashcode()

With these two tests was able to bump the overall coverage to 80.1%
PFA the screen captures for individual class coverage.

Excellent stuff, thank you for your contribution to Astra @ade3p. Merging now.


Azure MixedReality ObjectAnchors Conversion Sample issue

Section link:

Reason: The operation has not completed yet.
Suggestion: Add
            await operation.WaitForCompletionAsync();

            if (!operation.HasCompletedSuccessfully)
            {
                throw new Exception("The asset conversion operation completed with an unsuccessful status");
            }

after line 81 AssetConversionOperation operation = new AssetConversionOperation(jobId, client);
@scottaddie , @AlexGhiondea for notification.

Close this issue, due to PR https://github.com/Azure/azure-sdk-for-net/pull/22460 merged.


Listener added through .once is not removed after event execution

Reproduction code: https://jsfiddle.net/Lfozrhdw/
The issue is in line https://github.com/tunnckoCore/dush/blob/master/src/index.js#L146
Should be fn.__sourceString = fn.toString()

should be fixed in #42


[FEATURE] add buildTemplate (instead of retrieveAsTemplate)

https://github.com/Accenture/sfmc-devtools/issues/19
create deployment templates from git instead of from MC server

closed with release of mcdev 4.0.0: https://github.com/Accenture/sfmc-devtools/releases/tag/v4.0.0


Monitors with RGB vs GBR pixel subpositioning?

I've been scouring the Internet for clarity on whether ClearType supports BGR monitors (seems like almost all 4K 43" screens currently use BGR pixel subpositioning, along with many others). Some posts seem to say yes (at least in new versions of Windows), but others say that ClearType natively only supports RGB displays.
A related question is if a user has multiple monitors where one is RGB another is BGR, will ClearType work properly for both, or can it only support one pixel rendering method at a time? What about if a monitor is in portrait mode or upside down (which I think is equivalent to BGR)?
This page would be the ideal place for Microsoft to provide clarity on this issue.

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

ID: e35c5000-b2be-b78a-dccc-bba4cd11fd4c
Version Independent ID: 629477ee-35f3-b29a-7dc1-d76d8d54abda
Content: ClearType Overview - WPF .NET Framework
Content Source: dotnet-desktop-guide/framework/wpf/advanced/cleartype-overview.md
Product: dotnet-framework
Technology: dotnet-wpf
GitHub Login: @adegeo
Microsoft Alias: adegeo

@dotnet/wpf-developers Can anyone answer this question?


[Addition]: Check for multiple perfectly positioned hotspots

Hotspot b58 Addresses
11wyGaUtwiTdmnxMWLX8tnMZxbqdL1Xh82xQynMG7ZfQGcQEha9
112LfcK5TKxt5x89U8Mw2s2LRmqi7j9ky7Dp4Qaoz152zj46rRz9
112S8BLr56ZyvpExMkqhrQdyb2pWy2qdAouYpqj7GExjVHvrvMa1
11dSdmQbq8krJjiYdyrT6B8kvqak4MBYrJLhzAVJrBXCUP7LFTF
11zHFx4bcHAYgUARsMtrWSfQDRSq4gPs9cRreMHVaFPttKftgBL
112iRQitoAMpGfVbpxtwV3maVPV7n2HzzyFwmtdhFNs7b9v3j2oc
112hRNpbtvqwgXgYKPDvGRqLWTSk7XKTscXmLZjkiSaH6ou2V9A
Discord Handle
Ms_Incognito #9864
Reason(s)
So this 7 Hotspots don't have Antenna updated or height, all of them making few 100$ a month. I think it is worth of being checked. Same as Hotspots close to this area ScreenShot: https://prnt.sc/uIJZad3dKLG4
I've marked with red ones I reported, rest seems to be a gamers as well.
Hopefully someone can check this.

Yeah, noticed the same some days ago. Definitely cheating

Yeah, noticed the same some days ago. Definitely cheating

So how does this works, who and when is checking for those?

And whole square is so suspicious https://prnt.sc/AGWAKu30VTtM


Allow some calls to not have start dates [BW-712]

Tested locally with a "recreation" workflow on dev: http://localhost:3000/#workspaces/general-dev-billing-account/cjl_test_workspace/job_history/0236a12b-0af0-46c1-a8f7-8f5d5cc1b99a/74d8ac0f-024e-4950-b93a-de447b44829c

@cjllanwarne I'd like to quickly test this for myself (I didn't fully test the code I suggested). Could you share your test workspace with<EMAIL_ADDRESS>

@breilly2 you should have access to the workspace


Proper overlap with page header.

This PR does a few things (see screenshots below):

Prevent the main-holder from pushing up against the page header.
Make the background color fill out on the page header.
Which then makes it simple for the cards or components to overlap with the page header.

This change is

 I thought this was going to take some massive change to actually work properly since the two pages were already misbehaving. Glad to see it was a small fix! Thanks for taking care of it.

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io

Yah, I did my best to make the fix as clean as possible.  What was massive was the time to figure out what's happening.  :)

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io


Fixed to enforce declare(strict_types=1); in all files

Risky :D

Indeed.

I have a codebase that already has this though, so, for that use case, this fixer would be useful to make sure i don't forget it on new files, for example, rather than for a codebase conversion.

I'm already for it ;)

have a codebase that already has this though

That doesn't mean I've written the fixer btw, it means I have a codebase that has every file with strict type hints in. :trollface:

Finally every php7 project should have it

:+1:

Please see #1921


Is there a way to programatically activate the animatioon?

I am using the Like Animation in a CardView and on double click the card view I want to be able to activate the animation. Is this possible?

try .performClick()if it exists

That worked perfectly. Thank you


Adapt GitOps tutorials to CAPI

What this PR does / why we need it

Towards https://github.com/giantswarm/giantswarm/issues/30971
Things to check/remember before submitting

If you made content changes

Run make lint dev to render and proofread content changes locally.
Bump last_review_date in the front matter header if you reviewed the entire page.

the individual PRs has been approved already, this is just integration PR, I go ahead


create / form view not working

hi mootensai, I was able to complete generation of views for all my tables without any errors. now when i go to urgencies/create ... i receive a 404. I can add urgencies in the three view but i still would like to have the option of the traditional form. Can you please give a hint how to make it work again? create.php and _form.php exist alright
haha never mind. sorry. still learning .found the missing code in the controller. many thanks!

please close the issue if it is solved. and do mind to share your solution for other refference

use gitter to ask, don't create issues..

u better learn to use Yii 2 first before use this extension..


fix function calling arguments repeat

Description
Fix issue: The demo usage of Claude Sonnet 3.5 + tool cannot be reproduced.
Major Changes:

Before appending new arguments, check to see if the new content already exists
Append only when the new arguments are not part of the existing content
Make sure that the arguments in toolCallState are consistent with those in message, and not appended repeatedly

Checklist

[x] The relevant docs, if any, have been updated or created
[x] The relevant tests, if any, have been updated or created

Screenshots

Testing instructions
Self-test passed

@moqimoqidea I think we need to solve the underlying problem, which is that the repeated parts of "arguments" get sent to the Redux store in the first place. The solution here is problematic if the stream is trying to purposefully send the same substring twice in a row. It's very possible.
Do you have any idea where this is coming from, or could you share your config.json so I can try to reproduce? It's not something i'd seen before which is why I wonder whether it's config-related

@sestinj hi, thank you for your attention, and I'll show you the details of my configuration and API request below:

my model config:

{
  "model": "claude-3-5-sonnet-latest",
  "provider": "anthropic",
  "apiKey": "MY-API-KEY",
  "title": "Claude 3.5 Sonnet"
}

my claude api curl:

#!/bin/bash

curl https://api.anthropic.com/v1/messages \
  -H "content-type: application/json" \
  -H "x-api-key: $ANTHROPIC_API_KEY" \
  -H "anthropic-version: 2023-06-01" \
  -d @claude-official-data.json

my claude-official-data.json

{
  "max_tokens": 4096,
  "model": "claude-3-5-sonnet-20241022",
  "stream": true,
  "tools": [
    {
      "name": "builtin_read_file",
      "description": "Use this tool whenever you need to view the contents of a file.",
      "input_schema": {
        "type": "object",
        "required": [
          "filepath"
        ],
        "properties": {
          "filepath": {
            "type": "string",
            "description": "The path of the file to read, relative to the root of the workspace."
          }
        }
      }
    },
    {
      "name": "builtin_create_new_file",
      "description": "Create a new file",
      "input_schema": {
        "type": "object",
        "required": [
          "filepath",
          "contents"
        ],
        "properties": {
          "filepath": {
            "type": "string",
            "description": "The path where the new file should be created"
          },
          "contents": {
            "type": "string",
            "description": "The contents to write to the new file"
          }
        }
      }
    },
    {
      "name": "builtin_run_terminal_command",
      "description": "Run a terminal command in the current directory. The shell is not stateful and will not remember any previous commands.",
      "input_schema": {
        "type": "object",
        "required": [
          "command"
        ],
        "properties": {
          "command": {
            "type": "string",
            "description": "The command to run. This will be passed directly into the shell."
          }
        }
      }
    },
    {
      "name": "builtin_view_subdirectory",
      "description": "View the contents of a subdirectory",
      "input_schema": {
        "type": "object",
        "required": [
          "directory_path"
        ],
        "properties": {
          "directory_path": {
            "type": "string",
            "description": "The path of the subdirectory to view, relative to the root of the workspace"
          }
        }
      }
    },
    {
      "name": "builtin_view_repo_map",
      "description": "View the repository map",
      "input_schema": {
        "type": "object",
        "properties": {}
      }
    },
    {
      "name": "builtin_exact_search",
      "description": "Perform an exact search over the repository using ripgrep.",
      "input_schema": {
        "type": "object",
        "required": [
          "query"
        ],
        "properties": {
          "query": {
            "type": "string",
            "description": "The search query to use. Must be a valid ripgrep regex expression, escaped where needed"
          }
        }
      }
    },
    {
      "name": "builtin_search_web",
      "description": "Performs a web search, returning top results. This tool should only be called for questions that require external knowledge. Common programming questions do not require web search.",
      "input_schema": {
        "type": "object",
        "required": [
          "query"
        ],
        "properties": {
          "repo_url": {
            "type": "string",
            "description": "The natural language search query"
          }
        }
      }
    },
    {
      "name": "builtin_view_diff",
      "description": "View the current diff of working changes",
      "input_schema": {
        "type": "object",
        "properties": {}
      }
    },
    {
      "name": "builtin_read_currently_open_file",
      "description": "Read the currently open file in the IDE. If the user seems to be referring to a file that you can't see, this is probably it.",
      "input_schema": {
        "type": "object",
        "properties": {}
      }
    }
  ],
  "messages": [
    {
      "role": "user",
      "content": [
        {
          "type": "text",
          "text": "help me make a personal site with react, tailwind, and ts"
        }
      ]
    }
  ],
  "system": "When generating new code:\n\n1. Always produce a single code block.\n2. Never separate the code into multiple code blocks.\n3. Only include the code that is being added.\n4. Replace existing code with a \"lazy\" comment like this: \"// ... existing code ...\"\n5. The \"lazy\" comment must always be a valid comment in the current context (e.g. \"<!-- ... existing code ... -->\" for HTML, \"// ... existing code ...\" for JavaScript, \"{/* ... existing code */}\" for TSX, etc.)\n6. You must always provide 1-2 lines of context above and below a \"lazy\" comment\n7. If the user submits a code block that contains a filename in the language specifier, always include the filename in any code block you generate based on that file. The filename should be on the same line as the language specifier in your code block.\n\nExample 1:\nInput:\n```test.js\nimport addition from \"addition\"\n\nclass Calculator {\n  constructor() {\n    this.result = 0;\n  }\n    \n  add(number) {\n    this.result += number;\n    return this;\n  }\n}\n```\nUser request: Add a subtract method\n\nOutput:\n```javascript test.js\n// ... existing code ...\nimport subtraction from \"subtraction\"\n\nclass Calculator {\n  // ... existing code ...\n  \n  subtract(number) {\n    this.result -= number;\n    return this;\n  }\n}\n```\n\nExample 2:\nInput:\n```javascript test.js (6-9)\nfunction helloWorld() {}\n```\n\nOutput:\n```javascript test.js\nfunction helloWorld() {\n  // New code here\n}\n```\n\nAlways follow these guidelines when generating code responses.\n\nWhen using tools, follow the following guidelines:\n- Avoid calling tools unless they are absolutely necessary. For example, if you are asked a simple programming question you do not need web search. As another example, if the user asks you to explain something about code, do not create a new file."
}

my claude-offical-result.txt

event: message_start
data: {"type":"message_start","message":{"id":"msg_01EkCqu7riyKA9DzvUPrZ9Sy","type":"message","role":"assistant","model":"claude-3-5-sonnet-20241022","content":[],"stop_reason":null,"stop_sequence":null,"usage":{"input_tokens":1497,"cache_creation_input_tokens":0,"cache_read_input_tokens":0,"output_tokens":1}}    }

event: content_block_start
data: {"type":"content_block_start","index":0,"content_block":{"type":"text","text":""}          }

event: ping
data: {"type": "ping"}

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":"I"}            }

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":"'ll help you create a modern personal website"}        }

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":" using React, TypeScript, and Tail"}}

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":"wind CSS. Let's start by setting up"}        }

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":" the project structure and essential files.\n\nFirst, let's"}}

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":" create a new Next.js project with TypeScript since"}      }

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":" it provides excellent built-in features"}          }

event: content_block_delta
data: {"type":"content_block_delta","index":0,"delta":{"type":"text_delta","text":" for React applications:"}       }

event: content_block_stop
data: {"type":"content_block_stop","index":0    }

event: content_block_start
data: {"type":"content_block_start","index":1,"content_block":{"type":"tool_use","id":"toolu_01MhMP54rrNxUVrYKF436bVb","name":"builtin_run_terminal_command","input":{}}}

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":""}          }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"{\"comm"}}

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"and\": \"np"}    }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"x create-ne"}         }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"xt-app@la"}            }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"test ."}          }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":" -"}}

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"-typescrip"}       }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"t --tai"}          }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"lwind --e"}   }

event: content_block_delta
data: {"type":"content_block_delta","index":1,"delta":{"type":"input_json_delta","partial_json":"slint\"}"}  }

event: content_block_stop
data: {"type":"content_block_stop","index":1              }

event: message_delta
data: {"type":"message_delta","delta":{"stop_reason":"tool_use","stop_sequence":null},"usage":{"output_tokens":137}        }

event: message_stop
data: {"type":"message_stop"          }

In the case where I've tested multiple times with curl, I can confirm that the content returned by Claude is not duplicated, especially the partial_json part. And after I fixed it with this pr, the situation changed and the problem was solved.
I'm not sure if this issue is only happening to me, or if someone else can reproduce it stably. If others don't reproduce the problem in the same configuration, we can ignore or close the pr.


When choosing to read the Book, focus .page-wrapper so users can scroll using keyboard

When I choose to read "The Rust Programming Language" book, the page doesn't let me scroll by arrow keys. I have to manually click on the main content (the non Table of Contents area) in order to make scroll-by-keyboard work.
To reproduce:

Go to homepage
Click Docs(Beta) -> Book
Try and navigate the page using the arrow keys on the keyboard
(The user needs to manually click on the content before scrolling by kb works)

Is this applicable anymore with the new books UI? I'm not able to reproduce this as of the latest.

Looks good :+1:


Updated bleeding indicator

Closes #890
Updated bleeding indicator in status

I came back this year to make some sprites for this project since I had a blast last year doing so.
But I see the project hasn't been updated since last november... I don't see anywhere that the project is dead though, and I've done my PRs partly in hopes it can be brought back to life :)

Oh my, all the sprite you have submitted are fantastic! .
The sprites in the repository have been used in a different project (https://aronsoos.itch.io/arena) and Project Cordon is about to be revived too (Actually, it has already started this very day. Funny timing, huh?).
It has been a very nice surprise to see you come back this year. Thanks for the faith in the project too! :D


Experiment: remove top level owners

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


Warn on output values that contain nested secrets

Resolves #10676 and resolves #10327
This PR updates the OutputsShouldNotContainSecretsRule analyzer rule to warn if an expression used inside an output value contains secrets. This analysis is performed based on the declared type of the value supplied.
Microsoft Reviewers: Open in CodeFlow

@jeskew Might not get to this until next week.

@jeskew Is there anything in the docs that will need to change?  https://learn.microsoft.com/en-us/azure/azure-resource-manager/bicep/linter-rule-secure-secrets-in-parameters
If so, please enter a new bug for that.

@jeskew Thanks for handling this! Gave it a quick look, will try to finish tomorrow.

@StephenWeatherford would you mind taking another look when you get a chance?


Cannot start service nginx: oci runtime error: caused "not a directory"

Mounting files or volumes from Linux host to container in docker-machine

I have this user rights problem which I think is an issue with the docker itself and user rights assigned to it in Ubuntu version of Linux.
I first installed docker engine, docker compose and created basic Nginx, PHP containers. I used sudo privileges when creating and running these containers with no issues using docker-compose commands. The below error shows up when tried to up the containers without sudo privilege. But with sudo privilege, it goes just fine.
Then I had to install docker-machine on the same host for a different project with profiling tools in another container of the same machine. I have used docker compose for multiple containers inside default docker machine and this error shows up again. and if I use the sudo privilege to up the machines with docker-compose, it starts the previously created machines but not the ones from docker machine.
ERROR: for nginx2  Cannot start service nginx2: oci runtime error: rootfs_linux.go:53: mounting "/mnt/sda1/var/lib/docker/aufs/mnt/6b530a05f392226181b788163191f813676f15e093f6118428c6b8f170966dad/etc/nginx/conf.d/default.conf" to rootfs "/mnt/sda1/var/lib/docker/aufs/mnt/6b530a05f392226181b788163191f813676f15e093f6118428c6b8f170966dad" caused "not a directory"
Steps to reproduce the issue:

Install docker engine and docker compose and create basic LEMP machines using compose with sudo privileges.
Install docker machine on the same host and create a new project for LEMP using compose 2 without sudo privileges.
Try to up the containers from docker machine both with and without sudo privileges using compose version 2.

Additional information important:
All other containers are created successfully without sudo privileges except nginx2 service which has a mounting file option. The composer file looks like this:
version: '2'

volumes:
    database_data:
        driver: local

services:
  nginx2:
      image: nginx:latest
      ports:
          #- 8080:80
          - 8000:80
      volumes:
          - ./nginx/default.conf:/etc/nginx/conf.d/default.conf:rw
      volumes_from:
          - phphalcon
  phphalcon:
      #container_name: phalcon
      #image: mileschou/phalcon
      #used dockerfile from phalcon repository defined above.
      build: ./php/
      expose:
          - 9000
      volumes:
          - .:/home/arch/Projects/Imgcs/docker/app2/app
  mysql2:
      image: mysql:latest
      expose:
          - 3306
      volumes:
          - database_data:/var/lib/mysql
      environment:
          MYSQL_ROOT_PASSWORD: secret
          MYSQL_DATABASE: project
          MYSQL_USER: project
          MYSQL_PASSWORD: project

and if I am commenting out the volumes option from the nginx2 service, the container is created successfully without any requirement of sudo privileges.
 Because you're running Docker in a Virtual Machine, the docker daemon does not have access to your local files. Bind-mounted files are mounted from the host that the daemon runs on (in your case, inside a Virtual Machine).
Although docker-machine / boot2docker has partial support for accessing files on the host (through VirtualBox guest additions) on Windows and Mac, this is not supported on Linux.
If you specify a bind-mount (-v <host-path>:<container-path> or through the docker-compose file volumes: -<host-path>:<container-path>), the daemon checks if the file or directory at <host-path> exists, and if not, creates a directory at that location.
In your example, you're trying to bind-mount a configuration file (./nginx/default.conf) from the host. Since that file is not present inside the VM, the daemon creates an empty directory at that location, and bind-mounts it at /etc/nginx/conf.d/default.conf inside the container.
However, at that location, a file exists, and bind-mounting a directory on top of a file fails, and produces the error.
Wondering; why are you using a virtual machine to run docker in? Ubuntu supports running docker without the need for a Virtual Machine, and will give you native performance, without the overhead of a Virtual Machine (see https://docs.docker.com/engine/installation/linux/ubuntulinux/), or VirtualBox guest additions (which has extremely poor performance)
I'm closing this issue, because this is not a bug, but I hope this helps

I am sure this is something which leads to a bug in the docker engine core as I have seen it behaving in different ways with different user privileges. But I will dig more into it later when the bug is found with more proof to bring attention here.
I guess I was not clear in my first post here. I am not using Docker in a Virtual Machine. It is installed in native machine OS of Ubuntu Trusty Distribution. If anyone kind enough to follow the steps of creating this error described in the first post of this thread, they will see that it creates this weird error which is a leading direction to a potential bug in Docker Engine.

However, at that location, a file exists, and bind-mounting a directory on top of a file fails, and produces the error.

Doesn't matter if the file exists at that location or not, the same error shows up as I have tried both with and without the file existence at that location.
I hope this gives the idea to devs here of what I am talking about on Ubuntu. If not, I can try to explain in a different way.

@dev6 your output of docker info shows that the daemon is running on a boot2docker Linux, not on Ubuntu;
Kernel Version: 4.4.17-boot2docker
Operating System: Boot2Docker 1.12.1 (TCL 7.2); HEAD : ef7d0b4 - Thu Aug 18 21:18:06 UTC 2016

What does docker-machine ls show?

The docker-machine ls shows this:
NAME      ACTIVE   DRIVER       STATE     URL                         SWARM   DOCKER    ERRORS
default   -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.12.1

I had to install docker-machine after the installation of docker-engine as I needed to have an IP address for one of the profiling tools to work with curl. And the curl was failing to get it from localhsot:PORT.

So in that case, the docker daemon is running in a VirtualBox VM (see my comment before I closed the issue)

had to install docker-machine after the installation of docker-engine as I needed to have an IP address for one of the profiling tools to work with curl. And the curl was failing to get it from localhsot:PORT.

This should work, if the container has its port published. Is the profiling tool running in a container, or on the host? Also, make sure to check there isn't a firewall rule that block it, and, it may be worth using curl -4 ... in case the machine has IPv6 enabled, and curl tries to use an IPv6 address for localhost

Yes, the profiling tool is running in a container. Tried disabling firewall and IPv6 as well but same error where curl can't find the host at localhost:Port.

That's expected. localhost in a container is the localhost of the container itself, so if you try to use localhost:port, curl is trying to connect to the localhost of the container it is running in.
You can either change this to <public-ip-of-the-docker-host>:<port> or, to prevent going "outside" of the host just to connect to another container, use a container-container network and directly connect to the other container.
For example:
# create your custom network
docker network create my-network

# run the web server, and connect it to the network
# also publish port 80 of the container at 8080 on the host
# give the container a name, so that you can connect to it
# using its "name" as hostname from another container
docker run -d --network=my-network --name web -p 8080:80 nginx:alpine

# run another container, and connect to the "web" container
# note that on the container-container network, you connect
# to the _regular_ port of the container (port 80 in this case)
docker run -it --rm --network=my-network alpine sh

/ # wget -qO- web
<!DOCTYPE html>

<title>Welcome to nginx!</title>

Welcome to nginx!

If you see this page, the nginx web server is successfully installed and
working. Further configuration is required.

For online documentation and support please refer to
nginx.org.

Commercial support is available at
nginx.com.

Thank you for using nginx.

Read about networking here; https://docs.docker.com/engine/userguide/networking/

I guess that is already taken care when you are using compose to build and start the containers. They are all attached to a new custom network. Here is my docker network ls
NETWORK ID          NAME                DRIVER              SCOPE
fa15b06ef3aa        app2_default        bridge              local
1a297b1e6ec8        bridge              bridge              local
04bc689b678f        docker_default      bridge              local
f7e82e4bfc2b        host                host                local
47347dc570c0        none                null                local

And docker network inspect docker_default gives all the containers linked together.
[
    {
        "Name": "docker_default",
        "Id": "04bc689b678fb1d036c72a69eb4aea4b86f9716881b623babf98d066842a4332",
        "Scope": "local",
        "Driver": "bridge",
        "EnableIPv6": false,
        "IPAM": {
            "Driver": "default",
            "Options": null,
            "Config": [
                {
                    "Subnet": "<IP_ADDRESS>/16",
                    "Gateway": "<IP_ADDRESS>"
                }
            ]
        },
        "Internal": false,
        "Containers": {
            "1992a97d22675c21c1a29e1b9d7de40b375817864082146544d7a57c32999b2c": {
                "Name": "docker_mysql_1",
                "EndpointID": "9eaaa16d9e58883d3aca94016367b015b4f438256d7f02b64cb34b343c61baa5",
                "MacAddress": "02:42:ac:12:00:03",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            },
            "303c25b6e74bba6cf9eb62a40b61e872093a4106d349ae5eecd39ff83888a608": {
                "Name": "docker_php_1",
                "EndpointID": "5174b700e4c7f84115b5389b98e85a55904f28efaba6bf77ff6706be0f3d05de",
                "MacAddress": "02:42:ac:12:00:04",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            },
            "6afdc60ef0771a0f7955de6507ea9d029f6849996a3f9c22e0a4af638e899520": {
                "Name": "docker_blackfire_1",
                "EndpointID": "5bb534140405754a4fa0db1d901813b886f2daec2afecddfa89cfaa5a2980ac4",
                "MacAddress": "02:42:ac:12:00:02",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            },
            "7de81c23de087d79c9b513b6f6f75a0f14b940d8e2a33524e020c7d294fecb08": {
                "Name": "docker_nginx_1",
                "EndpointID": "769be3ad76b4a0ea6f581ca734b78fe6f55a1e3d89f4759135cc7154c452ade2",
                "MacAddress": "02:42:ac:12:00:05",
                "IPv4Address": "<IP_ADDRESS>/16",
                "IPv6Address": ""
            }
        },
        "Options": {},
        "Labels": {}
    }
]

On Windows maybe "shared drives" on app's settings works.

I'm not sure, if you found already a solution for your problem, but by the way, you find the problem named in the error message: caused "not a directory"
This is because you try to mount a file "default.conf" to a file, but you can only mount directories...
So try:
./nginx:/etc/nginx/conf.d:rw

@michaelstoiber:

This is because you try to mount a file "default.conf" to a file, but you can only mount directories..

This is incorrect. You can mount files directly. The issue is what others have pointed out in this thread, that if you try to mount a file that does not exist, the container will create an empty directory at that location. If you then try to mount the file after it exists on your host, it throws an error because an empty directory exists at that location on the container. But there's no issue with mounting individual files from the host machine to the container.

@MikeMcTiernan one pitfall is that on native Windows containers, bind-mounting single files is not possible

I was on a Windows machine when this happened. I used Laradock's pre-made containers, what happened was our organization forced our PCs to change passwords so I did. I didn't know that Docker stores credentials as well, I updated it at Docker > Shared Drives > Reset credentials... > Apply
I'm no longer getting the php-fpm permissions denied error.


added change withdraw wallet and appropriate tests.

added withdraw wallet to the constructor.
added function to change withdraw wallet to contract.
added test validating withdrawal to withdraw wallet.
added test for validating change to withdraw wallet.

Closes #5

Hey @carlomigueldy @itstargetconfirmed
So I'm just following up on things now and can see this one was merged.
Just to clarify:

Does that mean that basically only contract owner / deployer initially will be able to trigger withdrawals?
If so, what are the scenarios we are solving for here, rather than letting anyone trigger a withdrawal to specified address?

I'm still trying to wrap my head around why we need a special role for that, as it seems like a drawback to me. Ie we putting more responsibility on ourselves than need be :-)
I apologize in advance if I'm misunderstanding the code / intention and digging up things that are already settled unnecessarily.


[SpotBugs] Fix REC_CATCH_EXCEPTION

What changes were proposed in this pull request?

Fix REC_CATCH_EXCEPTION violations (ignored one case).
Remove REC_CATCH_EXCEPTION from SpotBugs exclusion.

Why are the changes needed?
Sub-task of #532 - Fix REC_CATCH_EXCEPTION.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
CI.

Codecov Report

Merging #527 (1d94a28) into master (6367b36) will increase coverage by 4.04%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #527      +/-   ##
============================================
+ Coverage     60.11%   64.15%   +4.04%
+ Complexity     1769     1636     -133
============================================
  Files           205      186      -19
  Lines         11532     9089    -2443
  Branches       1033      911     -122
============================================
- Hits           6932     5831    -1101
+ Misses         4194     2942    -1252
+ Partials        406      316      -90

Impacted Files
Coverage Δ

.../org/apache/uniffle/common/config/ConfigUtils.java
95.55% <0.00%> (ø)

.../apache/uniffle/coordinator/ClientConfManager.java
92.95% <0.00%> (ø)

...inator/access/checker/AccessCandidatesChecker.java
85.50% <0.00%> (ø)

.../storage/handler/impl/HdfsShuffleWriteHandler.java
87.09% <0.00%> (ø)

...le/storage/handler/impl/LocalFileWriteHandler.java
77.77% <0.00%> (+2.26%)
:arrow_up:

...rg/apache/uniffle/storage/common/LocalStorage.java
48.36% <0.00%> (-1.97%)
:arrow_down:

...apache/hadoop/mapreduce/v2/app/RssMRAppMaster.java

...rnetes/operator/pkg/webhook/inspector/inspector.go

...rg/apache/hadoop/mapred/RssMapOutputCollector.java

... and 16 more

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

Thanks @advancedxy for the review.


Pass formContext properly to ArrayTemplateField, files array and multiselect array

Reasons for making this change
formContext should be passed to every component that renders something. Fixes #441
Checklist

[ ] I'm updating documentation

[ ] I've checked the rendering of the Markdown text I've added
[ ] If I'm adding a new section, I've updated the Table of Content

[x] I'm adding or updating code

[x] I've added and/or updated tests
[x] I've updated docs if needed

[ ] I'm adding a new feature

[ ] I've updated the playground with an example use of the feature

Released in v0.43.0.


Setup: Add self-upgrade

This PR includes some improvements to the setup.py script in order to add upgrade option. Requirements are now installed as part of the setup.
I have been using this for a long time but have been frustrated by the lack of upgrade, mainly because I never knew what changed. Because my dotfiles are all symlinked I couldn't just use the setup.py install after git pull. Instead I am relying on custom scripts.
I think it is easier to upgrade now when new code is available. If needed, migration paths can be catered, since it always uses the setup.py in master. It installs the new files without overriding cmus-osx.config.
Introduced argparse to make it easier to use positional arguments.
To develop locally, just use:
ENV=dev ./setup.py install

It may not be the best approach to it, but it works 😃
Please let me know your thoughts!

Here's the thing: I'm not particularly happy with the current setup script because it assumes that all dependencies should be installed system-wide, instead of inside a virtual environment. Your improvements definitely make it better, then it previously was, but I think the best solution would be writing and upstreaming a brew formula (which do provide virtual environments for Python applications) that installs an installation helper command line utility in the same vein as the current setup.py (which just modifies the rc and possibly also sets the status_display_program if there isn't one set already to point to the virtual environment managed by brew).

Managed virtualenv provided by brew or pipx is the right direction. Nonetheless, the setup should not care if there is a virtualenv activated or not. The user is in control wether she wants a virtualenv for cmus or otherwise use the global installation. With brew or pyenv, it is easy not to mess with the system python and is a possible option to install dependencies.
Regarding the brew formula I wasn't sure what their policies were for plugins, but after checking, it seems we should be ok.
However the repository might need some changes for it to work than I remember:

The formula can't use github releases as there hasn't been any since 2017, and if it uses master branch, there is a risk it might contain broken code (although not any different than it is now)
With a formula, it makes sense to get rid of the cmus-osx folder and store it cmus-osx.config, inside the cmus installation
How about local development?

The changes were never meant to replace a brew formula, but perhaps to serve as a preliminary solution until then.

Here's what I'm planning to do:

Write a command line bootstrapping utility that deals with modifying the rc and autosave to point to the notify.py and media-keys.py files in the correct Python environment (either a virtual environment or the global Python installation depending on the execution context)
Add an config subcommand to said bootstrapping utility that opens the config file with the default $EDITOR
Rely on appdirs to supply the config path
Migrate to poetry
poetry is a virtual environment / dependency manager that always installs packages in editable mode while developing, but can also build regular Python packages. This will make development a breeze 🌊
Properly use GitHub releases and upload the resulting Python package builds (possibly with a CI solution, but not sure about that point yet)
Publish the resulting package to pypi and create a brew formula

That's a substantial upgrade. Great plan!
I am not a fan of appdirs for this use case, since cmus is a cmd program. Doesn't make much sense to store the config away from it imo.
Not familiar with poetry, but seems like pipenv.

Fair enough, will instead place the config file at the root of the cmus configuration directory 🙂

It took way longer than expected, but v3.0.0 (technically v3.0.1 because I initially forgot to include the README.md for PyPI) is finally complete 🙂
I basically implemented every item on the feature list, expect for the brew formula, which will follow suit whenever I have some more spare time on my hands. Please take it for a spin and don't hesitate to reach out if you find any bugs 😄

Sorry for the delay.
What happened to display_mode?

No problem 🙂
I removed the setting because no other music player (that I know of at least) leaves around notifications for previous songs. Should I re-introduce it?

I prefer not leaving notifications. It is kind useless and just clutter in my opinion.

Me too. That would map to display_mode = 2, which is the current behavior.

Overall it works great, but I found some annoyances. Will open new issues.
exception_hook is great for debugging :D