Multiple AWS accounts

Great piece of software!

We have multiple AWS accounts and was wondering if there was a way to tell the program which one to use based on the FROM addresses or simply just which account to send it through.

Also is there a way to set throttle so it sticks to thresholds given by AWS?

Is increase capacity equal to simply upgrading the server to higher specs or do we need to make any code changes to handle higher concurrent loads?

Hi @roshanjonah,

If you want to use different AWS account based on the FROM address, your best bet is to implement this logic before you sent it to aws-smtp-relay. The relay is specifically built to not have to define AWS accounts and access permissions, but to use AWS roles instead, see also https://github.com/blueimp/aws-smtp-relay#credentials

There is no in-built throttling nor rate-limiting, but with a good use case, contributions are always welcome. :)

For horizontal scaling, you could potentially make use of AWS Network Load Balancer, with multiple aws-smtp-relay containers behind it. Otherwise yes, you can scale vertically by increasing specs.

I have a similar issue: The SES identity that I'm sending mails as is in a different AWS account. It is my understanding that for cross-account sending, the ARN of the identity needs to be passed to SES either via the SourceArn parameter in the API call or the X-SES-SOURCE-ARN email header.
Unfortunately the SMTP client I'm using does not support adding custom headers to the email so it seems to me that SourceArn support would have to be added to aws-smtp-relay. Do you have any advice or a different solution?

Yes, providing the SourceArn parameter in the API call from aws-smtp-relay to the SES API would be the right approach in my opinion.
If we add it, we probably wanna add support for SourceArn, FromArn and ReturnPathArn:
https://docs.aws.amazon.com/ses/latest/DeveloperGuide/sending-authorization-delegate-sender-tasks-email.html#sending-authorization-delegate-sender-tasks-api-sendrawemail
This capability would have to be added to the project as a configuration parameter, similar to the other options:
https://github.com/blueimp/aws-smtp-relay#options

Not sure if it helps, but we were able to do cross account sending by setting the AWS_REGION, AWS_ACCESS_KEY_ID and AWS_SECRET_ACCESS_KEY environment variables.  Obviously these are secret but it did get us what we needed

@mmacaula Setting those secrets as environment variables somewhat defeats the purpose of this project:

SMTP server to relay emails via Amazon SES or Amazon Pinpoint using IAM roles.

If you are ok to use credentials to send emails, I would recommend you to make use of the official SMTP APIs instead:

https://docs.aws.amazon.com/ses/latest/DeveloperGuide/send-email-smtp.html
https://docs.aws.amazon.com/pinpoint/latest/userguide/channels-email-send-smtp.html


core: allow ms android client protocol violation

According to [MS-RDPBCGR <IP_ADDRESS>.1.1.1] the TS_EXTENDED_INFO_PACKET structure's cbClientDir field must include the mandatory NULL terminator of the ClientDir field.
However, since version <IP_ADDRESS>, the Microsoft Remote Desktop Client for Android sets cbClientDir to 0.

Update: It wasn't just affecting the Android but also the recent iOS client from MS


process and console compatible with symfony 6

process and console compatible with S6

Thanks for your contribution @Leroy1112 🚀


Package with thousand of versions cannot be found

Hi, we have an internal registry, a package we use is setup for automated releases, and thus we have over 11K versions released!
If I try to add that package (xyz>=1.2.3 for obfuscation), I get this error:
> uv pip install "xyz>=1.2.3"
  × No solution found when resolving dependencies:
  ╰─▶ Because there are no versions of xyz and you require xyz>=1.2.3, we can conclude that your requirements are
      unsatisfiable.

I am trying to switch away from pip/pip-tools for the obvious benefits but hitting this issue, for reference I used to get this issue with pip and pip cache purge used to fix the issue for that package.
This package for us have >11K releases, specifically:
> pip index versions xyz | sed "s/,/\n/g" | wc -l
  11039

Using latest uv version: 0.3.4
I realise that I am not sharing a ton of details to get you to solve this, but I am more than happy to run debug commands, obfuscate them and share results here.

Do you have the index configured somewhere, like in pip.conf? If so, you'll need to add a uv.toml with something like:
index-url = "..."

Ohh interesting, I see... Can you run with --verbose and see if there are any warnings or similar? I know you can't share the full logs, but the more of --verbose you can share, the more we can help.

The verbose output:
> uv pip install --verbose "xyz>=1.2.3"
DEBUG uv 0.3.4 (Homebrew 2024-08-26)
DEBUG Searching for Python interpreter in system path
DEBUG Found `cpython-3.9.7-macos-aarch64-none` at `/Users/minaasham/miniconda3/bin/python3` (conda prefix)
DEBUG Using Python 3.9.7 environment at miniconda3/bin/python3
DEBUG Acquired lock for `miniconda3`
DEBUG At least one requirement is not satisfied: xyz>=1.2.3
DEBUG Using request timeout of 30s
DEBUG Solving with installed Python version: 3.9.7
DEBUG Adding direct dependency: xyz>=1.2.3
DEBUG Found stale response for: https://pypi.python.org/simple/xyz/
DEBUG Sending revalidation request for: https://pypi.python.org/simple/xyz/
DEBUG Found not-modified response for: https://pypi.python.org/simple/xyz/
DEBUG Searching for a compatible version of xyz (>=1.2.3)
DEBUG No compatible version found for: xyz
  × No solution found when resolving dependencies:
  ╰─▶ Because there are no versions of xyz and you require xyz>=1.2.3, we can conclude that your requirements are
      unsatisfiable.

Ok, so I'd suggest switch PyPI and your index -- like:
index-url = "https://pypi.python.org/simple"
extra-index-url = ["OBFUSCATED"]

We search in the extra index first, and if a package exists on that index, we stop looking, to prevent dependency confusion attacks (this behavior is configurable -- you can opt-out).
Does the package also exist on PyPI? I don't know if xyz was literal but it does: https://pypi.org/project/xyz/

xyz is just obfuscation, but yes the real package has a pypi version, so the name is shadowed, I would have thought that the index-url is looked up first liked pip does?

I was able to make it work with --index-strategy=unsafe-first-match and --index-strategy=unsafe-best-match

Yeah, either of those would work. --index-strategy=unsafe-best-match is the closest to pip.

Sorry for the confusion around index ordering. I think we would've gotten issues like this regardless of the default we chose, just because we opted for a different model (we stop as soon as we find a package on the first matching registry, by default).


Overused valibot schemata

Check which type is really validated in code.

Fixed with 6fc8d4c914e
Removal of utils package and rethinking validation with zod in www app.


Add a new section 'arch' to display CPU architecture

Description
This PR adds a new CPU architecture section, that is useful considering the current climate regarding Apple's transition from Intel (i186) to M1 (arm64) architecture. Many developers, myself included, are running two terminals (one under Rosetta2) to keep the M1 and Intel workflows separate till compatibility for M1 picks up.
This helps reduce the confusion when two such terminals are running simultaneously.
Screenshot

Two terminals running Spaceship with arch under arm64 and i386 respectively, showing how CPU architecture is important contextual information.

I'm not sure if this is something worth being visible when a user types every command. Nevertheless, this is a good candidate for an external section. I'll keep this open, if you don't mind.

I just made this because I personally find it helpful. I confuse the Rosetta terminal with the regular one quite often.
I don't mind having the PR open. Also happy to do what it takes so make it an external section if you could point me to the docs.

I'm working on spaceship v4.x. Hopefully, I'll come up with a mechanism for seamlessly including external sections in there.
You don't have to do anything at the moment, just be subscribed to this thread.

Thanks for resolving the conflict!

No problem @denysdovhan. Also thanks for making this wonderful prompt. I cannot understand how I lived before Spaceship!

Closing again because the transition to Apple Silicon is basically completed and now there's really no need to see the arch when typing every command. Also merge conflicts have come up.


Saving of app.ini should be reverted as there is no such feature in the gitea for now

Saving of app.ini should be reverted as there is no such feature in the gitea for now.

@truecode112 fixed this in https://github.com/DmitryFrolovTri/gitea/commit/c55ccfdb487216eddd636802fcc3b8c5825553d2
will close once tested

Confirmed fixed. Closing


Bitfinex handle heartbeat sequenceId

The hunt for 100% valid order books continues apparently. for #258
This is a similar update to the one I did for Gemini in #244. Sequence ID is shared across all channels on Bitfinex and if you're subscribed to l2 or l3 book you need to receive the sequenceId of heartbeat events, even if it's a heartbeat for the ticker channel.
For backwards compatibility I've just emitted an empty l2 or l3 update w/the sequenceId included from the heartbeat.
New constructor params
I've addded two new constructor params to allow new behavior that allows sequenceId validation while maintaining backwards compatibility.

@param {Boolean} [params.enableEmptyHeartbeatEvents]
*  (optional, default false). if true, emits empty events for all channels on heartbeat events which includes the sequenceId.
@param {String} [params.tradeMessageType]
*  (optional, defaults to "tu"). whether to use trade channel events of type "te" or "tu" or "all". see https://blog.bitfinex.com/api/websocket-api-update/

Explanation of changes
When the hearbeat is received on a "l2update" or "l3update" I just emit an empty Level2Update or Level3Update event w/the sequenceId. This should be backwards compatible so no harm in emitting it.
For when the hearbeat is received on a "trade" or "ticker", I've added a new boolean constructor param enableEmptyHeartbeatEvents. An empty Trade or Ticker object w/nothing but a sequenceId from a heartbeat seems non-backwards compatible, so with enableEmptyHeartbeatEvents = false (the default) the behavior will be the same as previously and these heartbeats will be ignored. When enableEmptyHeartbeatEvents = true, empty Trade and Ticker events will be emitted w/the sequenceId of each heartbeat, allowing you to validate all websocket messages.
I've also added an option to choose which type of trade update is used, "te" or "tu". The difference is "tu" comes after a short delay and includes the orderId, "te" comes immediately and doesn't have the orderId. From some testing I believe this delay for "tu" might cause an issue w/sequenceId validation so I think "te" is preferred when validating sequenceIds. The default has been  "tu" so far, so I've kept that the default and enabled optionally using "te" instead.

For reference, this only really shows up on lower activity markets since heartbeats don't get sent unless a channel doesn't have activity for 15 seconds. I was able to replicate by using the DTX/USD market which has a daily volume of $13 lol. Testing low activity markets on other exchanges may yield similar results if not accounted for.

For reference, this only really shows up on lower activity markets since heartbeats don't get sent unless a channel doesn't have activity for 15 seconds. I was able to replicate by using the DTX/USD market which has a daily volume of $13 lol. Testing low activity markets on other exchanges may yield similar results if not accounted for.

@bmancini55 I just made some changes to add a new constructor arg to keep backwards compatibility and allow sequenceId validation optionally, lmk if this seems like the right approach

@bmancini55 I just made some changes to add a new constructor arg to keep backwards compatibility and allow sequenceId validation optionally, lmk if this seems like the right approach

I forgot to create an issue, sorry about that. I've just made one here #258

I forgot to create an issue, sorry about that. I've just made one here #258

@evan-coygo I think we're still good with the approach.  I don't believe the underlying code has changed much with the move to TS.  I think the hold up last time was just on a final refactor to make the code more readable with the additional heartbeat logic.  If you'd like to fix the conflicts and update the PR I'm happy to give it a final review before merge!

@bmancini55 could I get a review?

@evan-coygo thanks for making the TS conversion and refacting things!  I pushed a few commits so we can get this across the line.
f51756dc448979743baeb7e37b7313fd5e2f5d45 - reverts the changes to the test suite. Spec files run in parallel via CI. Sibling scoped describe blocks run serially. Since the test suite wraps things in a describe block it's safe to call the test suite multiple times in a spec file and it will execute sequentially.
f51756dc448979743baeb7e37b7313fd5e2f5d45 - converts the trade event types to an Enum and just cleans up the impacted code.
0f85fabf988b0ed7c35fbbb08a7ab3e07658e1ed - handles unsubscribe events so that we don't need to override _subscribe, which breaks the idempotency of subscribe.
Let me know if you have any questions / concerns and if not, I'll get this merged and released today.


docs(assert): message callback docs

Adds docs for the change in #36.
Not sure I've done this correctly - do I need to update main/assert.md too?

thank you for going the extra mile


[QUERY] Can this package be used with cloudflare workers?

I'm currently trying to get started interacting with the Slack web api in the Cloudflare workers runtime.
Unfortunately the main client uses axios, and I don't seem to be able to get it running on cloudflare (I get a Uncaught Error: Dynamic require of "node:path" is not supported error)
Seeing as the deno runtime is very similar to the cloudflare runtime, I was wondering if it's possible to somehow install and use it there?

Hi there,
Deno's documentation suggests Deno apps should be able to run on Cloudflare Workers, so there might be a way to run this module there!
If there's a problem using this library, you could also try this project from a member of our team, which natively supports Cloudflare Workers
Otherwise we have a list of commonly used cloud providers here

@vegeris ah amazing, that looks like exactly what I've been looking for, shame I didn't stumble across it while googling earlier. Cheers!


Add option to kill active queries when the Cancel button is pressed

Thanks for the recommended process ErikEJ, I think I'm following along now. I've removed my old fork, forked from your master to get back to stable, done work in my KillQueriesOnCancel branch and are doing a Pull Request from that. I'm seeing five commits against your repository, but those are commits I did against my fork (one URL is https://github.com/ErikEJ/SqlQueryStress/commit/c05df7581249a0540644395142f12a7b83288c20) - are those an issue?
This pull request: Added option to kill active queries when Cancel button is pressed, enabled by default. I've done this by adding the Timer object "_killTimer" to LoadEngine.QueryInput, which (if the option has been enabled) checks for backgroundWorker having been given the cancel request, and if so, cancels the current Query, "KillQueriesOnCancel" flag to Form1 QueryStressSettings and  "Kill Queries on Cancel" checkbox to the Options dialog.

This looks good, I will have a look soon

Merged, looks good! Hope you get time to do the other PR re keyboard navigation as well!


⚠ Bofrost.at has degraded performance

In 6c7751d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1093 ms

Resolved: Bofrost.at performance has improved in 61ae75c.


Link to previous lesson not working

Describe the bug
A link that is supposed to take to the previous lesson is leading to a 404 error page.
To Reproduce
Steps to reproduce the behavior:

Go to '2-Working-With-Data'
Go to '06-non-relational'
Go to 'README.md'
Scroll to 'Document Data Stores with the Azure Cosmos DB'
Click on the link saying 'previous lesson'

Expected behavior
The link should take to a previous lesson.
Screenshots

fixed


test: http2 user info now works from version 7

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6426
#1480 but only for 7 as Suricata 6 is ok for the event, but not for the workaround signature

ping @victorjulien for merge if approved ?

Merged in #1484, thanks!


Getting NullReferenceException when BarcodeScanner.Update() and BarcodeScanner.Scan() are called

Hi,
I am trying to integrate this plugin in my unity project and I am using 2019.2.5f1 editor. I have followed all the steps mentioned in the docs and copy pasted the code mentioned in the sample SimpleDemo.cs with few modifications for camera permission for iPhone (the code related to camera permissions can be added as an improvement).
After running the application in iPhone, I am getting the following exception:
NullReferenceException: Object reference not set to an instance of an object.
at Scene1.g__c_statusChanged|3_1 (System.Object sender, System.EventArgs e) [0x00000] in <00000000000000000000000000000000>:0
at System.EventHandler.Invoke (System.Object sender, System.EventArgs e) [0x00000] in <00000000000000000000000000000000>:0
at BarcodeScanner.Scanner.Scanner.set_Status (BarcodeScanner.Scanner.ScannerStatus value) [0x00000] in <00000000000000000000000000000000>:0
at BarcodeScanner.Scanner.Scanner.Update () [0x00000] in <00000000000000000000000000000000>:0
at Scene1.Update () [0x00000] in <00000000000000000000000000000000>:0
and same one when I tried to call the BarcodeScanner.Scan():
NullReferenceException: Object reference not set to an instance of an object.
at Scene1.g__c_statusChanged|3_1 (System.Object sender, System.EventArgs e) [0x00000] in <00000000000000000000000000000000>:0
at System.EventHandler.Invoke (System.Object sender, System.EventArgs e) [0x00000] in <00000000000000000000000000000000>:0
at BarcodeScanner.Scanner.Scanner.set_Status (BarcodeScanner.Scanner.ScannerStatus value) [0x00000] in <00000000000000000000000000000000>:0
at BarcodeScanner.Scanner.Scanner.Scan (System.Action2[T1,T2] callback) [0x00000] in <00000000000000000000000000000000>:0    at Scene1.scanQR () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.Events.UnityAction.Invoke () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.Events.InvokableCall.Invoke () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.Events.UnityEvent.Invoke () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.UI.Button.Press () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.UI.Button.OnPointerClick (UnityEngine.EventSystems.PointerEventData eventData) [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.ExecuteEvents.Execute (UnityEngine.EventSystems.IPointerClickHandler handler, UnityEngine.EventSystems.BaseEventData eventData) [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.ExecuteEvents+EventFunction1[T1].Invoke (T1 handler, UnityEngine.EventSystems.BaseEventData eventData) [0x00000] in <00000000000000000000000000000000>:0
at UnityEngine.EventSystems.ExecuteEvents.Execute[T] (UnityEngine.GameObject target, UnityEngine.EventSystems.BaseEventData eventData, UnityEngine.EventSystems.ExecuteEvents+EventFunction1[T1] functor) [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.StandaloneInputModule.ProcessTouchPress (UnityEngine.EventSystems.PointerEventData pointerEvent, System.Boolean pressed, System.Boolean released) [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.StandaloneInputModule.ProcessTouchEvents () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.StandaloneInputModule.Process () [0x00000] in <00000000000000000000000000000000>:0    at UnityEngine.EventSystems.EventSystem.Update () [0x00000] in <00000000000000000000000000000000>:0  UnityEngine.DebugLogHandler:Internal_LogException(Exception, Object) UnityEngine.DebugLogHandler:LogException(Exception, Object) UnityEngine.Logger:LogException(Exception, Object) UnityEngine.Debug:LogException(Exception) UnityEngine.EventSystems.ExecuteEvents:Execute(GameObject, BaseEventData, EventFunction1)
UnityEngine.EventSystems.StandaloneInputModule:ProcessTouchPress(PointerEventData, Boolean, Boolean)
UnityEngine.EventSystems.StandaloneInputModule:ProcessTouchEvents()
UnityEngine.EventSystems.StandaloneInputModule:Process()
UnityEngine.EventSystems.EventSystem:Update()
I have wasted hours trying to figure out the exact root cause of these issues and think it is something related to the statusChanged as it can be seen in exceptions and which traces back to BarcodeScanner.Update() and BarcodeScanner.Scan(). Hence finally raising this issue. It would be great if some solution is provided quickly. Really stuck with this.

Project not supported anymore (Here)


Frequently land on /filament-developer-logins with 419

Hi
Thanks for this very helpful plugin!
I often find myself landing on /filament-developer-logins an expired csrf token 419, would be great if it could be redirected back to the login page instead or something.
Or perhaps remove the csrf check for only login?

I can't say I've ever encountered this issue. When does this occur?

Hmm, I have to check again, but I frequenly run artisan migrate:fresh, not sure if that could be a reason.

I will close this issue for now, if the issue persists feel free to open this issue again.


UX Design - Restricted Requests (Updates for New Ministries to On-Board)

Title of ticket:
Description
Summarize issue
Dependencies
Waiting for map out of process from @m-prodan and @liseandtea to better understand what the needs are.
Meet with Arielle to get better understanding of IAO Process
DOD

[ ] Map out
[ ] Design
[ ]
[ ]

https://miro.com/app/board/uXjVPLXHHRA=/?share_link_id=306652024529
Miro Lise made

Designs have been completed for restricted files, going to close this issue. @m-prodan can you let me know the timeline is for meeting with all future Ministries about any additional requirement they have. As the dev team wants us to have those planned earlier than later. And the sooner I know I can start working on whatever design updates need to be mad for them


Provide workaround for base dir when hosting in-process

Directory.GetCurrentDirectory() returns the worker directory of the process started by IIS rather than the application directory (for example, C:\Windows\System32\inetsrv for w3wp.exe).

@Tratcher The base dir question was just asked on the 2.2 RTM blog Q&A. Can I add a note to the end of this to surface AppDomain.CurrentDomain.BaseDirectory as a suitable way to go?

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

ID: fe69b3a9-896f-e189-0c2c-f87ff735df9c
Version Independent ID: 97c8549b-66ad-0a23-1971-784fb923042a
Content: ASP.NET Core Module configuration reference
Content Source: aspnetcore/host-and-deploy/aspnet-core-module.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: riande

@shirhatti @pakrym @jkotalik

AppDomain.CurrentDomain.BaseDirectory doesn't work when debugging project in VS because it point to the bin directory instead of project directory.

This is what I came up with:
    internal class CurrentDirectoryHelpers
    {
        internal const string AspNetCoreModuleDll = "aspnetcorev2_inprocess.dll";

        [System.Runtime.InteropServices.DllImport("kernel32.dll")]
        private static extern IntPtr GetModuleHandle(string lpModuleName);

        [System.Runtime.InteropServices.DllImport(AspNetCoreModuleDll)]
        private static extern int http_get_application_properties(ref IISConfigurationData iiConfigData);

        [System.Runtime.InteropServices.StructLayout(System.Runtime.InteropServices.LayoutKind.Sequential)]
        private struct IISConfigurationData
        {
            public IntPtr pNativeApplication;
            [System.Runtime.InteropServices.MarshalAs(System.Runtime.InteropServices.UnmanagedType.BStr)]
            public string pwzFullApplicationPath;
            [System.Runtime.InteropServices.MarshalAs(System.Runtime.InteropServices.UnmanagedType.BStr)]
            public string pwzVirtualApplicationPath;
            public bool fWindowsAuthEnabled;
            public bool fBasicAuthEnabled;
            public bool fAnonymousAuthEnable;
        }

        public static void SetCurrentDirectory()
        {
            try
            {
                // Check if physical path was provided by ANCM
                var sitePhysicalPath = Environment.GetEnvironmentVariable("ASPNETCORE_IIS_PHYSICAL_PATH");
                if (string.IsNullOrEmpty(sitePhysicalPath))
                {
                    // Skip if not running ANCM InProcess
                    if (GetModuleHandle(AspNetCoreModuleDll) == IntPtr.Zero)
                    {
                        return;
                    }

                    IISConfigurationData configurationData = default(IISConfigurationData);
                    if (http_get_application_properties(ref configurationData) != 0)
                    {
                        return;
                    }

                    sitePhysicalPath = configurationData.pwzFullApplicationPath;
                }

                Environment.CurrentDirectory = sitePhysicalPath;
            }
            catch
            {
                // ignore
            }
        }
    }

I can place the code in the topic's asset folder and only link to it (i.e., not display it in the topic). Is that ok?
... I'll stand-by on the other one for guidance :point_right: https://github.com/aspnet/Docs/issues/9774

@guardrex I'm fine having just a link but I'd like to clear it with @shirhatti first.

I'll wait for your signal to proceed.

LGTM

Will this workaround be integrated in the Aspnet Core 2.2.x or we will have to use it internally in our apps?

We are going to port the actual fix to 2.2.x

Great, thank you very much!

Is there an engineering issue yet that I can track on for the doc update?

https://github.com/aspnet/AspNetCore/issues/4599


Further progress on CuPy wrapper

Looping back on wrapping CuPy.
Summary:

Can do gradient descent-based optimization with CuPy wrapper for a simple linear model.
Tests for wrapped functions using check_grad are now annotated.

(1) Gradient-based optimization
This now works for a simple linear model that I coded up.
(2) Checking check_grad tests
test_cupy.py is a direct mirror of test_numpy.py, except with pytest annotations giving me a hint as to why they might be failing.
For some, I have an idea on how to debug (related to minute differences between CuPy and NumPy), for others, I might start reaching out for advice on what's going on.
cc'ing people who I remember are interested, and those with whom I've conferred:
@mattjj, @dougalm, @duvenaud, @jermwatt, @kmaehashi, @dwd31415, @tscohen, @oeway, @iaroslav-ai, @bingo619, @kastnerkyle, @outlace, @samuela

More progress today! In the tests for CuPy in my branch, I have 42 failed and 57 passed tests. Officially >5/9 of the way through! Of the tests that passed, they were mostly due to minor differences between the CuPy and NumPy APIs.
Along the way, I have also run the test_numpy.py suite to make sure nothing there breaks.
Finally, I have made some changes that take advantage of CuPy to automagically distinguish between the two array libraries. This is hacky, but it works. I might try to make this more generalized in the future.

More progress today! I have knocked out about 2/3 of the tests. Since >1 week ago, there's been enough passing tests coverage to write plain vanilla (Feed Forward) neural networks using nothing but dot products and loss functions.
My next most immediately useful thing would be to wrap the CuPy sparse library, which is what I'll be moving forward on.

How’s the progress on this these days?

http://github.com/ericmjl/autograd-cupy
That said... I would suggest using Jax instead!
My autograd CuPy wrapper was very educational, but I'm ultimately not in a position to maintain it long-term.


Add Firefox versions for overflow-clip-margin CSS property

17311

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: NA
Details

 Quality gate: Passed
 Everything looks good
Details

 Quality gate: NA
Details


Alluxio-Trino for Trino version 434 and higher

Page
https://docs.alluxio.io/os/user/stable/en/compute/Trino.html
Alluxio version
2.9.0 and higher.
Trino version
434 and higher.
Summary
The current documentation on Alluxio+Trino works for Trino all versions up to 433. When using higher versions, Trino throws a No FileSystem for scheme: alluxio error. This is due to some code changes for HDFS that were implemented in Trino 434.
To solve this problem, the alluxio-<version>-client.jar file needs to be copied to the path ${Trino_HOME}/plugin/hive/hdfs/ instead of the original path ${Trino_HOME}/plugin/hive-hadoop2/.

@jpohanka we tested your workaround. However, your workaround doesn't seem to be enough. My observations on that matter:

There is no alluxio-$version-client.jar on Maven Central. We added alluxio-shaded-client-2.9.3.jar to the …/plugin/hive/hdfs folder.
Beyond that we had to COPY (Dockerfile) the content of the /opt/alluxio/lib folder (from an Alluxio installation/image) into the Trino image under /opt/alluxio/lib.
Adding version 312 instead of 2.9.3 is not working! I'm still confused by Alluxio comes with semantic versioning (e.g. 2.9.4) and an incompatible seq numbering (e.g. 312).


create_csv_agent UnicodeDecodeError('utf-8' )

System Info
python
Who can help?
No response
Information

[ ] The official example notebooks/scripts
[ ] My own modified scripts

Related Components

[ ] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[ ] Callbacks/Tracing
[ ] Async

Reproduction
from langchain import OpenAI
from langchain.chains import AnalyzeDocumentChain
from langchain.chains.summarize import load_summarize_chain
from langchain_experimental.agents.agent_toolkits import create_csv_agent
from langchain.chains.question_answering import load_qa_chain
from langchain.chat_models import ChatOpenAI
from langchain.agents import AgentType
model = ChatOpenAI(model="gpt-4") # gpt-3.5-turbo, gpt-4
agent = create_csv_agent(model,"APT.csv",verbose=True)
agent.run("how many rows are there?")
Expected behavior
UnicodeDecodeError                        Traceback (most recent call last)
 in <cell line: 11>()
9 model = ChatOpenAI(model="gpt-4") # gpt-3.5-turbo, gpt-4
10
---> 11 agent = create_csv_agent(model,"APT.csv",verbose=True)
12
13 agent.run("how many rows are there?")
11 frames
/usr/local/lib/python3.10/dist-packages/pandas/_libs/parsers.pyx in pandas._libs.parsers.raise_parser_error()
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xb1 in position 0: invalid start byte

@egisgeoai answers provided by the bot is help full?


Fix concurrent withdrawal behavior

The current extension authorizes concurrent withdraw and
In this case, 'used' and 'wait time' are not properly updated by api_lnurl_callback function.
As a result, the usage limit can be exceeded if too many concurrent withdrawal are performed at the same time.
This PR is a proposition to fix this behavior.

@arcbtc please take a look at this. I'm not familiar with this extension.

Looks good to me as well. Nice spot @evildido !


How to work with update form ?

I have a data that created by validate form. I want to work to update data  with form. How can I do ?

Check initialFields in example/src/Model.elm


fix(reactivity): fix side effect computed dirty level changed unexpected

thanks to @wanghanzhen for resetTracking and test case
close #11181, #11169

/ecosystem-ci run

Excellent job! I tried it and it works fine in my project.

Thx indeed, we cannot update to current version because of this issue, nice to see that it will be fixed very soon thx to you :-)


Signer Panel (scripts and contracts)

When the user is editing a script or contract:

only one signer can be selected

Questions:
How to handle if no signers are selected?
One signer will be always selected by default.
As a rule, the first account 0x01 will be the default selected account.

If a user selects another account, the previous selection will be disabled, that way only one account can be selected at a time:

State change of the selected account (displayed on the top):

How to handle if more than one signer is selected?
Do we display an error?
Design
This design is generic and needs to above questions answered
https://www.figma.com/file/G7Z28scDlkNFvgDyR2nSnq/Playground-Library-|-Components-(External)?node-id=298%3A117977

completed


Provide own icons

There's a problem with hijacking icons like that: they're subject to change b/w Intellij versions, and at some point they might be changed so that they still will be adequate for their original purpose, but no longer match our use case. Maybe instead copy the original icon to our project?... TBD how that works with the IntelliJ's Apache license.
Originally posted by @PawelLipski in https://github.com/VirtusLab/git-machete-intellij-plugin/pull/319

In the scope of this task:

decide which icons are required
create new ones
~JB icons that are to be resused shall by baked into our codebase~ (#328)

*Provide own icons, or at least make sure the ones that we use are baked into our codebase

Comment on MACHETE_FILE icon used by Open Machete File Action:

Btw while you're here: I'd maybe use some icon with a paper & pencil (and not just paper) to indicate it's about editing, not e.g. just viewing

~ https://github.com/VirtusLab/git-machete-intellij-plugin/pull/301#discussion_r437384730

@mkondratek do you think we can close that now?

yup. I think that the current approach is good enough. We basically

use icons offered by IntelliJ where applicable (e.g. push)
use our own icons when no applicable icon can be found among IntelliJ icons (e.g. squash)
update icons with their newer versions when the newer versions are available among IntelliJ icons

closing


Specify output folder name

Is it possible to make option for specifying output folder name with the report files rather than using input files names?
Yours faithfully,
Katerina

AfterQC is designed to run in batch. So, normally AfterQC will create a QC folder, and within the QC folder there will be folders for different input fastq.
You can change the name QC to report by specifying -r report in the command line.
Then the dir tree will be like:
report/
└── R1.fq
    ├── report.html
    └── report.json

So, your requirement is not to include 'R1.fq' folder inside the report folder, and make the dir tree like:
report/
├── report.html
└── report.json

Am I right?

Yeah, so the user can specify "report" folder for each pair manually if running in -1 -2 mode, for example.

I think, that it would be perfect I user can specify some prefix for reports, e.g. --report-prefix=/path/to/dir/filename and then:
/path/to/dir/
├── filename.html
└── filename.json

@alezanalp do you agree with @serge2016 ?

I have submitted a commit to implement @serge2016 's idea. You can pull or download the latest master to have a try.
Now, you will get
QC
├── filename1.fq.html
└── filename1.fq.json
└── filename2.fq.html
└── filename2.fq.json
...

And you can change folder name from QC to report by specifying -r report .

@sfchen  Yes, I agree with @serge2016 . Thank you for the prompt reply. Will try it

There is one more "issue" or bag with this in v0.9.0:
If I run after.py --read1_file=SRR3184279_1.fastq.gz --read2_file=SRR3184279_2.fastq.gz --read1_flag=_1 --read2_flag=_2 --qc_only then I get everything ok:
report/
└── R1.fq
    ├── report.html
    └── report.json

But if I run after.py --read1_file=SRR3184279_1.fastq.gz --read2_file=SRR3184279_2.fastq.gz --read1_flag=_1 --read2_flag=_2 --qc_only --report_output_folder=$(pwd) then I get:
SRR3184279_1.fastq.gz options:
{'qc_only': True, 'version': '0.9.0', 'seq_len_req': 35, 'index1_file': None, 'trim_tail': 0, 'report_output_folder': '/home/bg/kate/AfterQC/PE_reads/', 'trim_pair_same': True, 'no_correction': False, 'debubble_dir': 'debubble', 'barcode_flag': 'barcode', 'read2_file': 'SRR3184279_2.fastq.gz', 'barcode_length': 12, 'trim_tail2': 0, 'unqualified_base_limit': 60, 'allow_mismatch_in_poly': 2, 'read2_flag': '_2', 'store_overlap': False, 'debubble': False, 'read1_flag': '_1', 'index2_flag': 'I2', 'draw': True, 'index1_flag': 'I1', 'mask_mismatch': False, 'barcode': False, 'overlap_output_folder': None, 'barcode_verify': 'CAGTA', 'index2_file': None, 'qualified_quality_phred': 15, 'trim_front': 9, 'good_output_folder': 'good', 'poly_size_limit': 35, 'n_base_limit': 5, 'qc_sample': 200000, 'trim_front2': 9, 'no_overlap': False, 'input_dir': None, 'read1_file': 'SRR3184279_1.fastq.gz', 'qc_kmer': 8, 'bad_output_folder': None}

Traceback (most recent call last):
  File "/home/bg/soft/AfterQC-0.9.0/after.py", line 221, in <module>
    main()
  File "/home/bg/soft/AfterQC-0.9.0/after.py", line 215, in main
    processOptions(options)
  File "/home/bg/soft/AfterQC-0.9.0/after.py", line 171, in processOptions
    filter.run()
  File "/home/bg/soft/AfterQC-0.9.0/preprocesser.py", line 709, in run
    stat_file = open(os.path.join(qc_dir, "report.json"), "w")
IOError: [Errno 20] Not a directory: '/home/bg/kate/AfterQC/PE_reads/SRR3184279_1.fastq.gz/report.json'

This error occurs if I set the -r dir equal to the dir, where I run AfterQC from.

@serge2016 this issue is because of v0.9.0 need to create a folder same as the R1 fastq file name, so it will conflict with the fastq file name if $(pwd) is specified as report_output_folder.
I believe with last commit, this issue is gone.

I just released v0.9.1. You can have a try with the new feature described above.

Now previous behavior is changed to more predictable:) Thank you!
But I still think about the variant when we specify -1 and -2 options: is this mode we have only one sample, so we can specify the full output name for the report.
I simply want to use your tool inside CWL environment, and it is easier to do if it is possible to specify output filenames independently from input filenames.


Fix: instantiate a new router for separated routes (fixes #15)

As mentioned in #15 there is a security issue. We need to instantiate a new router for every set of routes we pass (every time we call the function mapRoutes).

Coverage remained the same at 89.744% when pulling eb7fcde64be351a9808737258a23ab8b11063cca on fix/seperateRouters into 2cd29e71caca3675a148d24506c2bd28084f2910 on master.


feat(ngx-conveyer): reduce view updates and fix outputs

Had to remove @Directive -> outputs and switch to the explicit declaration because currently the template compiler is not happy. Event emitters also should be declared on the class when outputs is provided so it's actually easier to use the explicit @Output(...) declaration:

Alright, I'll revert my outputs updates and will add the ngx prefix.


Slicing evaluation for minimalProvideDocumentBundle

https://profiles.ihe.net/ITI/MHD/4.0.0-comment/qa.html#C__Users_john.moehrke_Git_ITI.MHD_fsh-generated_resources_Bundle-ex-minimalProvideDocumentBundle
shows errors in qa:

Bundle/ex-minimalProvideDocumentBundle: Bundle.entry[0] (l17/c6)
error
Slicing cannot be evaluated: Profile based discriminators must have a type with a profile (Bundle.entry:Documents.resource in profile http://profiles.ihe.net/ITI/MHD/StructureDefinition/IHE.MHD.Minimal.ProvideBundle)

Bundle/ex-minimalProvideDocumentBundle: Bundle.entry[1] (l36/c6)
error
Slicing cannot be evaluated: Profile based discriminators must have a type with a profile (Bundle.entry:Documents.resource in profile http://profiles.ihe.net/ITI/MHD/StructureDefinition/IHE.MHD.Minimal.ProvideBundle)

this is due to that sushi does not fill the in the profile which is needed by the ig publisher. see similar discussion on zulip
PR adjusts slicing type.code and type.profile that the errors disappear (there are still some errors in the example).

CI Build: http://build.fhir.org/ig/IHE/ITI.MHD/branches/oe_pc_slicing/index.html


NaN detected

After some iterations the following error is occurred:

...
128 samples computed
256 samples computed
384 samples computed
462 samples computed
Valid  2.78936
Epoch  0 Update  88100 Cost  0.587470054626 UD  1.63904905319
NaN detected

Reloading the model using reload=True parameter, continue the learning process.
It maybe related to this issue.

Have you tried @orhanf 's suggestions in the issue you linked? For my use case it seems like applying gradient clipping (setting clip-c to 1.0) solved the problem of NaNs.


Enable Stack Features (X-Pack) and trial by default

We're now "secure by default" with X-Pack and paid features enabled by default 🔐 (30-day trial)
If this gets merged, the x-pack branch will become the new master in a second step.
Closes #163
Closes #378

Please excuse my force-pushes. I had to find a way to set passwords for built-in users in a non-interactive manner in Travis, and was too lazy to reproduce their environment locally.
I'll get it to work eventually.

I think it's related to the fact that you changed the default elastic username and removed ELASTIC_PASSWORD from the compose file. I can try to investigate a bit later.

Yes it's by design (see new README section in "Initial setup"). We kind of abused this bootstrap password on the X-Pack branch, whereas the documentation recommends using the built-in users with less privileges instead. Just trying to apply best practices and making user aware of security. I can change it back and use the super admin everywhere.

@antoineco oh I see, you got a point with security awareness.
Argh, another battle security VS user experience :-)
My opinion is to use the super-admin everywhere in order to make the experience using this stack pretty much a "clone and up". We're not really providing a production ready stack, the main goal is still to be able to quickly give a try to the Elastic stack without any hassle.
tbh, this is kinda what I was expected when I tested this branch, just clone and up.
WDYT about default super-admin and a note about security in the README?

Sounds good! I'll do that.

@deviantony Done.
@mw-jko you review is also welcome :)

Good work ! :+1:
Not sure why the travis build fails though. I'll let you merge this one.

Looks like Travis is sometimes slower and Logstash is not ready when it should be. I'll bump the delays a little bit if I fails this round again.
Thanks for the review! :raised_hands:

@mw-jko your review is also welcome :)

Sorry, I was on vacation 🙈


further fix on memory leak

Please use latest gaugeJS build (v1.3.7). After that, the library will be leak free.
@amroessam Thanks for your excellent work!

Amazing work @chiu0602, doing it now!!

The package downloaded from npm site haven't been built
npm i<EMAIL_ADDRESS>Please help to build it, thanks a lot!

Hahaha, i forgot to build it before publishing. I'll do that now!

Done! 383f179

I have downloaded rc1 and rc2 through npm command, the files' content in dist folder are the same

Done, try now!

Thanks~ it is perfect now.


Fix output path and handle directories as output path

This PR changes the following behavior with respect to output location

If embed options are true, the output is specified by filename, it does not create a new "output" directory.
If either embed options are false, then "output" directory is created (even if output is specified by filename).
If output is not specified and embed are true, the then output file name is /path/to/input-optimized.gltf (from /path/to/input.gltf)
If output is not specified and embed are false, the then output file name is /path/to/output/input.gltf (from /path/to/input.gltf)

This PR also introduces handling directories as output. In this case,

Create directory is always false (irrespective of embed).
The output path is outputDir/input.gltf (from -i /path/to/input.gltf -o outputDir)

I have also introduced code to handle the arguments like -i input output correctly. (input -o output was handled correctly by default before).
Fixes #244

One further change we could make is to change the default created directory name from "output" to the filename of the input.
So /path/to/myasset.gltf will result in /path/to/myasset/myasset.gltf

Coverage decreased (-0.1%) to 92.727% when pulling cfa62ca2fb4b91842da1c499508814e0d287e601 on shehzan10:fix-output-path into 5e7c535b4a0cc5976e8602b12ca5b00af8159028 on AnalyticalGraphicsInc:master.

Coverage decreased (-0.1%) to 92.727% when pulling cfa62ca2fb4b91842da1c499508814e0d287e601 on shehzan10:fix-output-path into 5e7c535b4a0cc5976e8602b12ca5b00af8159028 on AnalyticalGraphicsInc:master.

Coverage decreased (-0.1%) to 92.727% when pulling cfa62ca2fb4b91842da1c499508814e0d287e601 on shehzan10:fix-output-path into 5e7c535b4a0cc5976e8602b12ca5b00af8159028 on AnalyticalGraphicsInc:master.

Coverage decreased (-0.1%) to 92.727% when pulling cfa62ca2fb4b91842da1c499508814e0d287e601 on shehzan10:fix-output-path into 5e7c535b4a0cc5976e8602b12ca5b00af8159028 on AnalyticalGraphicsInc:master.

To be similar to other CLI tools, I don't think we should ever create a directory that isn't specified by the user.  i.e. if the user sets /foo/bar/test.glb as the output then all files should go in bar.  If bar (or foo) doesn't exist, we'll create them but we shouldn't create anything under bar.
This allows the user to be able to to determine what the output will be without knowing any special rules.

To be similar to other CLI tools, I don't think we should ever create a directory that isn't specified by the user.

To incorporate this suggestion, I think only the following change needs to be made. From

If output is not specified and embed are false, the then output file name is /path/to/output/input.gltf (from /path/to/input.gltf)

We should make the output filename /path/to/input-optimized.gltf and other accompanying files would be /path/to/input-optimized.*
Does that sound good?

/path/to/input-optimized.*

Doing this style might be more complicated than I initially thought.
The problem comes down to how the properties are named within the glTF object. An example is the CesiumTexturedBoxTestEmbedded.gltf which produces the following sources when not embedded:
CesiumTexturedBoxTestEmbedded-optimized.gltf
CesiumTexturedBoxTest.bin
Image0001.png
CesiumTexturedBoxTest0FS.glsl
CesiumTexturedBoxTest0VS.glsl

These filenames are coming from the object property names. So there is no clear way to determine which filename to replace with the suggested ones.
I feel like unless we run through all cases for these names, we may not have a neat solution.
One solution could be to do something like CesiumTexturedBoxTest0FS.glsl -> CesiumTexturedBoxTest0FS-optimized.glsl, ie. add the -optimized suffix to all sources.

Is anything needed to move this along?

Is anything needed to move this along?

There needs to be consensus on how we handle the problem I described in my previous comment.

For now just deriving the filename from the id is probably fine.
With gltf 2.0 not having ids anymore, the filenames can be a combination of the filename, name (if it exists), and its index.

I agree with @lilleyse, other "sidecar" generated data is usually prefixed with the filename.  For example, if you have foo.tif and ask gdal to output foo.png, it outputs foo.png and foo.png.xml (which has the geolocation information.  So we should use model.gltf and then model.gltf.<index>.bin or something similar.
However, I'm not sure why we would wait to gltf and change things again.  Can't we just switch to a single convention now that will work in going forward?

@mramato Just to clarify, the offline discussion that @lilleyse  and I had was to maintain status quo wrt. the "sidecar" files, ie, the files would be like this:
CesiumTexturedBoxTestEmbedded-optimized.gltf (main gltf file)
CesiumTexturedBoxTest.bin
Image0001.png
CesiumTexturedBoxTest0FS.glsl
CesiumTexturedBoxTest0VS.glsl

And any changes we make would be after 2.0.
But your suggestion would make the sidecars as:
CesiumTexturedBoxTestEmbedded-optimized.gltf (main gltf file)
CesiumTexturedBoxTestEmbedded-optimized.gltf.CesiumTexturedBoxTest.bin
CesiumTexturedBoxTestEmbedded-optimized.gltf.Image0001.png
CesiumTexturedBoxTestEmbedded-optimized.gltf.CesiumTexturedBoxTest0FS.glsl
CesiumTexturedBoxTestEmbedded-optimized.gltf.CesiumTexturedBoxTest0VS.glsl

Admittedly, the input filename is quite long so all the filenames look long. But this could be a viable approach.
Comments?

How common is it for files to have shared textures?  It just occurred to me that renaming the texture output names could result in that use case being "broken".  Assuming that's not a use case we have to worry about, then the post 2.0 version looks fine with me.
How popular is the separate use case anyway (outside of textures)?  I pretty much always use combined models.

CesiumTexturedBoxTestEmbedded-optimized.gltf (main gltf file)
CesiumTexturedBoxTestEmbedded-optimized_0.bin // Index in case of multiple buffers
Image0001.png
CesiumTexturedBoxTestEmbedded-optimized_0FS.glsl
CesiumTexturedBoxTestEmbedded-optimized_0VS.glsl

This would be my preference as I rarely see the sidecar convention with gltf models. Textures ought to stay the same name.

@lilleyse's suggestion sounds good to me.

This is fixed in the 2.0 branch.


lnwire: Correct instance of double-hex encoding in log message.

Since ChannelID defines String() which hex-encodes the channel ID, fmt lines can just use the regular string formatter.
Fixes https://github.com/lightningnetwork/lnd/issues/331.

Ah, didn't see this until recently. Fixed earlier today within 31d53c6070d7b31d8c6bc8c4dd797ac95da66a1e.
Closing.


sklearn api for ranking

This PR resolves #2859. My implementation is slightly different from OP's implementation as group information has to be separately fed to fit method, instead of combining it together with features. I believe that ranking is quite a different task compare to classification or regression, so this difference in interface is probably justified. By the way, this is currently how LightGBM implements its sklearn ranker interface.
Regarding testing, I am not very sure what is the best way to test this interface. What I have done so far is to compare it with the results obtained from the train method, but this relies on the dataset used in demo/rank. If you believe there are better ways of testing, please let me know.

@hcho3 I was getting some problems with travis last night...here are the errors:
Adding APT Sources (BETA)
$ export DEBIAN_FRONTEND=noninteractive
$ curl -sSL "http://apt.llvm.org/llvm-snapshot.gpg.key" | sudo -E apt-key add -
curl: (7) Failed to connect to apt.llvm.org port 80: Connection timed out
gpg: no valid OpenPGP data found.
The command "curl -sSL "http://apt.llvm.org/llvm-snapshot.gpg.key" | sudo -E apt-key add -" failed and exited with 2 during .
Your build has been stopped.

Installing APT Packages (BETA)
$ export DEBIAN_FRONTEND=noninteractive
$ sudo -E apt-get -yq update &>> ~/apt-get-update.log
$ sudo -E apt-get -yq --no-install-suggests --no-install-recommends $TRAVIS_APT_OPTS install clang clang-tidy-5.0 cmake-data doxygen wget libcurl4-openssl-dev unzip graphviz gcc-4.8 g++-4.8
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package clang-tidy-5.0
E: Couldn't find any package by glob 'clang-tidy-5.0'
E: Couldn't find any package by regex 'clang-tidy-5.0'
apt-get.diagnostics
apt-get install failed
The command "sudo -E apt-get -yq --no-install-suggests --no-install-recommends $TRAVIS_APT_OPTS install clang clang-tidy-5.0 cmake-data doxygen wget libcurl4-openssl-dev unzip graphviz gcc-4.8 g++-4.8" failed and exited with 100 during .
Your build has been stopped.

Adding APT Sources (BETA)
$ export DEBIAN_FRONTEND=noninteractive
$ curl -sSL "http://apt.llvm.org/llvm-snapshot.gpg.key" | sudo -E apt-key add -
curl: (7) Failed to connect to apt.llvm.org port 80: Connection timed out
gpg: no valid OpenPGP data found.
The command "curl -sSL "http://apt.llvm.org/llvm-snapshot.gpg.key" | sudo -E apt-key add -" failed and exited with 2 during .
Your build has been stopped.

Is there any way I can restart the build? Thanks.

@hcho3 Thanks a lot for the help!

@Shiki-H Looking for a way to stabilize Travis.

@Shiki-H Travis CI team is currently looking at the issue: https://www.traviscistatus.com/incidents/hgmftqkw1sdl

@Shiki-H Sometimes old files stick around and mess things up. I'll go ahead and clean the workspace.

Let me know when this PR is ready for review.

@hcho3 The design issue I was uncertain about was the group argument. In LightGBM, the group argument is included for all sklean interfaces such as Classifier. I am not sure if simply over-riding the parent class in XGBRanker is the most appropriate way of doing it. Please let me know what you thinks.
By the way, regarding the ranking demo, what do think would be a good output? Or is it enough if I just make a quick walk through of the interface like it is now?

Codecov Report

Merging #3560 into master will increase coverage by 0.1%.
The diff coverage is 73.21%.

@@             Coverage Diff             @@
##             master    #3560     +/-   ##
===========================================
+ Coverage     50.27%   50.38%   +0.1%
  Complexity      188      188
===========================================
  Files           172      172
  Lines         13927    13983     +56
  Branches        457      457
===========================================
+ Hits           7002     7045     +43
- Misses         6700     6713     +13
  Partials        225      225

Impacted Files
Coverage Δ
Complexity Δ

python-package/xgboost/sklearn.py
84.49% <73.21%> (-2.64%)
0 <0> (ø)

python-package/xgboost/core.py
82.6% <0%> (+0.16%)
0% <0%> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@hcho3 yep, I think we should probably postpone custom objective for now. I will take a look at this. If I can come up with some solution, I will make a separate PR then.

@Shiki-H I notice that XGBRanker throws the following error if I tried to pass the objective "rank:map":
        elif self.objective != "rank:pairwise":             raise ValueError("please use XGBRanker for ranking task")
Is this intended? Should we just check if the object of strength contains the substring "rank:"?

@RAMitchell Oops, my bad. Yes, the check should be changed to check if the string begins with "rank:". Good catch.

@RAMitchell thanks for spotting it. @hcho3 has this problem been fixed? if not, i can work on it

It has been fixed.


Inconsistent guidance on contributor links

In the site structure templates, the contributor line is explicitly cast as a link to a github profile.  In the curated content template, there is no such guidance.  It appears that many are linking to their github profiles, but not all.

This is all done! These are old issues and can be closed.


spore-bearing organs should not be classified as animal organs

howler here spotted by @selewis:
[Term]
id: GO:0075259
name: spore-bearing organ development
namespace: biological_process
def: "The process whose specific outcome is the progression of a spore-bearing organ over time, from its formation to the mature structure. A spore-bearing organ is an anatomical structure that produces new spores." [GOC:mah, GOC:mcc, GOC:pamgo_curators]
synonym: "fruiting structure development" EXACT []
***is_a: GO:0048513 ! animal organ development
is_a: GO:0048608 ! reproductive structure development
***relationship: never_in_taxon NCBITaxon:33208 {id="GOTAX:0000457"} ! Metazoa

Although this looks like a taxon error, it's actually a naming error.
GO:0048513 ! animal organ development
is defined:
Development of a tissue or tissues that work together to perform a specific function or functions. Development pertains to the process whose specific outcome is the progression of a structure over time, from its formation to the mature structure. Organs are commonly observed as visibly distinct structures, but may also exist as loosely associated clusters of cells that work together to perform a specific function or functions
nothing there says anything about animal-specificity. The original intent was for plant, animal and even "fungal organs"
the name was changed early this year:
http://www.ebi.ac.uk/QuickGO/GTerm?id=GO:0048513#term=history
I assume this was to reflect the fact we were using the metazoan-specific uberon organ to axiomatize this:
intersection_of: results_in_development_of UBERON:0000062 ! organ

we're still in limbo here - undecided as to whether to generalize the upper levels of uberon (making them equivalent with CARO classes of the same name) which would require some changes in uberon, or to use a mix of uberon and caro
ultimately I don't think 'organ' is a particularly useful term in anatomy, and especially unuseful as a developmental grouping in GO

Hi @cmungall  and all,
Not sure who among us changed the label of GO:0048513 ’organ development’ to ‘animal organ development’ back in January; he/she may wish to comment here. @dosumis @mcourtot @ukemi @tberardini
If there are still uncertainties on the ultimate naming/generalization of that class, the safest quick fix at the moment may simply be to change the label back to ‘organ development’, as many descendants of GO:0048513 are indeed plant terms (e.g. ‘leaf senescence’). If that sounds ok, I’ll implement. We could then set this ticket to pending while we wait for a resolution on ‘organ’.

From the commit log:
r30527 | davidos | 2015-12-22 08:25:21 -0800 (Tue, 22 Dec 2015) | 1 line
fixed some of the problems arising from bringing over uberon taxon constraints + making regulates transitive. Fixed include spliting organ dev into plant and animal organ dev terms.  Fixing remaining inconsistencies will require work on Uberon and possibly CL too.
svn diff -r 30526:30527 gene_ontology_write.obo
id: GO:0048513
-name: organ development
+name: animal organ development

From the rest of that diff (above), it looks like @dosumis moved a bunch of stuff around to tidy up plant vs. animal organ development.  The term 'spore-bearing organ development' may have not been moved during this update.

It may be simpler to keep this is animal organ development for now, and remove the fungal terms from underneath.
The final strategy will need to wait until FAO is in place. We may end up getting rid of generic organs

There's definitely cleaning up that needs doing beneath the initial children of this term.  I spotted 'maintenance of floral organ identity' here:
animal organ development
--[p] maintenance of organ identity
----[i] maintenance of floral organ identity
also
animal organ development
--[p] multicellular structure septum development (slated for obsoletion, #12496, waiting for mouse annotation transfer)
----[i] bunch of plant terms
Probably need to step down the graph at each node to make sure that all children are sorted out.

Fungal structures have been described as models for multicellular development by researchers that want to pursue that angle and not because it is an established fact. There are many more articles, definitions and descriptions of fungi that refer to spore-bearing "structures" than those that use the term "spore-bearing organ."
I think the term "spore-bearing structure" should be created for fungal anatomical structures. It is technically more accurate than "spore-bearing organ" for fungi. Conidiophores would then become a spore-bearing structure.

Another +1
Thanks Diane
On Tue, Aug 16, 2016 at 12:04 AM, Diane O Inglis<EMAIL_ADDRESS>wrote:

Fungal structures have been described as models for multicellular
development by researchers that want to pursue that angle and not because
it is an established fact. There are many more articles, definitions and
descriptions of fungi that refer to spore-bearing "structures" than those
that use the term "spore-bearing organ."
I think the term "spore-bearing structure" should be created for fungal
anatomical structures. It is technically more accurate than "spore-bearing
organ" for fungi. Conidiophores would then become a spore-bearing structure.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geneontology/go-ontology/issues/12578#issuecomment-240019998,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABcuEMxwpRQ8n39j-mDHZtFrSjSNSEmGks5qgWDlgaJpZM4Jjy2Q
.

Ok, thanks @tberardini  and @cmungall  for your feedback.
I reviewed all descendants of ‘animal organ development’.
Unless anyone has objections, I will remove the links:
‘spore-bearing organ development’ is_a ‘animal organ development’
‘maintenance of floral organ identity’ is_a ‘maintenance of organ identity’
‘bract morphogenesis’ is_a ‘organ morphogenesis’
‘establishment of floral organ orientation’ is_a ‘establishment of organ orientation’
‘leaf morphogenesis’ is_a ‘organ morphogenesis’
‘floral organ morphogenesis’ is_a ‘post-embryonic organ morphogenesis’
‘floral organ senescence’ is_a ‘organ senescence’
‘post-embryonic root development’ is_a ‘post-embryonic organ development’
Links to ‘animal organ development’ also need to be removed from the following terms, but I’d need @tberardini  to suggest an appropriate parent please (either an existing one, or a new one to be created):
‘cotyledon boundary formation’
‘organ boundary specification between lateral organs and the meristem’
‘shoot organ boundary specification’
‘leaf senescence’
As for the plant-related children of ‘multicellular structure septum development’, I presume @tberardini  will take care of them as part of https://github.com/geneontology/go-ontology/issues/12496, but let me know if not.
Hopefully I haven’t missed any descendant of ‘animal organ development’ that should be housed elsewhere, but if so, they can be taken care of as they arise.
Sounds like @dianeoinglis ' suggestion to create a term for ’spore-bearing structure’ should be addressed by FAO. @cmungall , is there a tracker?
Thanks,
Paola

After reviewing the descendants of 'animal organ development', I think we need to create the following new terms with taxon constraints.  I can do some of this through templates when TG is back up. The rest needs to be done manually.
plant organ morphogenesis
plant organ formation
formation of plant organ boundary
plant organ senescence
specification of plant organ position
specification of plant organ axis polarity
establishment of plant organ orientation
maintenance of plant organ identity
I suggest renaming the apparently generic versions of 'organ %' to be 'animal organ %' to avoid future confusion when selecting terms for annotation or as new parents for new terms.
I'll need to move some Arabidopsis annotations from the animal versions to the new plant terms (when they exist) but that should further clean up things.
@paolaroncaglia: after you sort out the NTR for FAO for 'spore-bearing structure', you can assign this to me.

What do we think of a more radical suggestion and ditch all organs as a grouping and just use a pan-species pan-scale 'anatomical structure'?

You mean, for example, getting rid of 'animal organ development' (or merging it into 'anatomical structure development') and pushing all the is_a kids up a level to be direct children of 'anatomical structure development'?
Interesting.  Then put/inherit the appropriate taxon restraints on the actual terms like 'adipose tissue development' and 'floral organ development'.  Yes?

On 17 Aug 2016, at 12:40, Tanya Berardini wrote:

Interesting. Then put/inherit the appropriate taxon restraints on the
actual terms like 'adipose tissue development' and 'floral organ
development'. Yes?

Correct
Maybe best discussed on a call

I'm going to hold off on addition of any new terms until after we've discussed this possible approach as a group.

I added the topic to today's agenda (GO editors call). I won't be able to make the call, but happy to refer to minutes if there is a resolution. Thanks!

On 17 Aug 2016, at 6:55, paolaroncaglia wrote:

Sounds like @dianeoinglis '
suggestion to create a term for ’spore-bearing structure’ should
be addressed by FAO. @cmungall , is
there a tracker?

I thought Maria Costanza and I set up a GitHub repo, but I can't find
it. If we were to do this would there be interest in an informal
consortia of us (GO, Val, Diane) keeping it up to date? It's one more
moving piece but will make it easier in the long run.

After this morning's GO eds meeting, we decided that the new 'xxx plant organ xxx' terms should be created as proposed.  I'll do that and move the necessary plant terms out of '(animal) organ xxx' parentage.

I don't think Maria Costanzo set up a GitHub repo. I see that I was the last person to make changes to FAO obo to add terms for Cryptococcus...although Maria held my hand with the OBO-editor. I am certainly interested in keeping it up-to-date and it is in need of an update soon. There are a few questionable placements, some term defs that need improvement and some new terms that should be added.
All the links from the main FAO page are to SGD and are defunct. But, the Biomodels Ontology has 28 mappings to FAO out of 89 total FAO terms, Uberon has 21. Is it preferable for one of us to edit and update the obo ourselves or would it be reasonable to talk Biomodels into incorporating the complete set and updating with them? How are the terms in the GO being tied into FAO? Is it strict such that a term better be in the FAO or no-go?
What are the suggestions for how to handle the FAO and what are possible options?
Diane

@dianeoinglis - I'm checking with Stacia at SGD to see what's up with FAO editing.  Will report back.
@cmungall , @ValWood -  additional thoughts on what to do with FAO maintenance?

Response from Stacia:
We haven’t done anything with it since Maria left last November.  The documentation she left for us (last updated Feb 2014) states that "Other editors can be added by asking Suzi Lewis and Chris Mungall.”
big sigh.  i guess we need to update the contact info on it.
http://www.ontobee.org/ontology/FAO
hope this helps.

Well, there is an issue that perhaps Stacia and other SGD'ers don't realize. The phenotype curation at SGD, CGD, and formerly AspGD rely on the FAO for autoreplacement of the APO term "formation of anatomical structure" where the "anatomical structure" is called upon from the FAO. That should not be an issue for updating the FAO. The Stanford databases check the file into cvs and only the FAO terms in that file are available in the phenotype curation interface. They are unlikely to care about new terms beyond those already available in the Stanford DBs. I thought this might be worth mentioning.

Added
[Term]
id: GO:1905392
name: plant organ morphogenesis
namespace: biological_process
def: "The developmental process by which a plant organ is generated and organized." [GO_REF:0000083, GOC:tb, GOC:TermGenie]
intersection_of: GO:0009653 ! anatomical structure morphogenesis
intersection_of: results_in_morphogenesis_of PO:0009008 ! plant organ
created_by: tb
creation_date: 2016-08-24T00:21:19Z
[Term]
id: GO:1905393
name: plant organ formation
namespace: biological_process
def: "The process that gives rise to the plant organ. This process pertains to the initial formation of a structure from unspecified parts." [GO_REF:0000081, GOC:tb, GOC:TermGenie]
intersection_of: GO:0048646 ! anatomical structure formation involved in morphogenesis
intersection_of: results_in_formation_of PO:0009008 ! plant organ
created_by: tb
creation_date: 2016-08-24T00:21:27Z

For FAO, I suggest I create a github repo, seed it with the latest version of FAO (in sourceforge?). I can show @dianeoinglis how edit this - or if edits are infrequent I can check files in. I will change the OBO metadata - would you be the new contact @dianeoinglis ?
External users should largely be unaffected. If people have mappings they should still work. SGD should be unaffected.
Thanks for the info on how FAO is used in SGD. It's always useful to know how an ontology is being used (I'd like to improve the OBO site to better show this, perhaps via bioSharing). This is also useful from an AGR integration POV.

Added:
+id: GO:0090691
+name: formation of plant organ boundary
+namespace: biological_process
+def: "The regionalization process that specifies plant organ primordium boundaries resulting in a restriction of organogenesis to a limited spatial domain and keeping the organ separate from surrounding tissues." [GOC:tb]
+is_a: GO:0048859 ! formation of anatomical boundary
+intersection_of: GO:0048859 ! formation of anatomical boundary
+intersection_of: occurs_in PO:0009008 ! plant organ
Moved out plant terms under 'formation of organ boundary' to this new term.  Changed term name of 'formation of organ boundary' to 'formation of animal organ boundary' and added xp to UBERON:organ for that term.

Added:
+id: GO:0090693
+name: plant organ senescence
+namespace: biological_process
+def: "The process that occurs in a plant organ near the end of its active life that is associated with the dismantling of cell components and membranes, and an overall decline in metabolism." [GOC:tb]
+is_a: GO:0007568 ! aging
+relationship: part_of GO:0099402 ! plant organ development
Moved out plant terms under 'organ senescence' to this new term. Changed term name of 'organ senescence' to 'animal organ senescence'.  Fixed Arabidopsis annotations that were directly to the now-animal term to child terms that were plant specific.

Yes, I can be the FAO contact. Both OBOfoundry and SourceForge have the most current files. Awesome!
http://obo.cvs.sourceforge.net/obo/obo/ontology/anatomy/gross_anatomy/microbial_gross_anatomy/fungi/fungal_anatomy.obo

@dianeoinglis - you should have full admin rights here:
https://github.com/obophenotype/fungal-anatomy-ontology
Here is the updated OBO page:
http://obofoundry.org/ontology/fao.html
I took the cute space-alien mushroom image from the FAO site, but this can easily be replaced

since we're on a fungal role here I'm going ahead and axiomatizing the fungal development terms in GO, and adding an fao_imports

Great! I may have to send you questions about OBO-editor. I recall the namespace is important. Are there GO editors in addition to Val that can follow the issue tracker? If so, I will create a ticket for terms that I think should be added with defs and proposed placement. I think it would be ideal for a few additional opinions and comments on terms before I add or change. A sanity check from an experienced GO editor is what I'm hoping for.

Good questions! Let's take this here: https://github.com/obophenotype/fungal-anatomy-ontology/issues/1
Maybe some of the others can follow us over there

I'm not a GO editor ;)
But I'll pitch in on the tracker if I notice anything ....we don't use anatomy terms much for fission yeast so I haven't had much input although I am following and it all sounds good.

On 25/08/2016 02:53, Chris Mungall wrote:

Good questions! Let's take this here:
obophenotype/fungal-anatomy-ontology#1
https://github.com/obophenotype/fungal-anatomy-ontology/issues/1
Maybe some of the others can follow us over there
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geneontology/go-ontology/issues/12578#issuecomment-242260408,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHBLKEJTFKbQjwyTH4CCQ-Mj03T_cmluks5qjPWygaJpZM4Jjy2Q.

I notified SGD too...
--
Cambridge University
PomBase http://www.pombase.org/
Cambridge Systems Biology Centre http://www.sysbiol.cam.ac.uk/Investigators/val-wood

I removed the link between 'spore-bearing organ development' and 'animal organ development'.  There was another existing is_a parent, 'reproductive structure development'.

added
post-embryonic plant morphogenesis
post-embryonic plant organ morphogenesis
post-embryonic plant organ development
Then  a whole bunch of renaming the 'organ' terms to 'animal organ', moving plant terms out under the new terms, looking at parentage of plant terms and checking. More still to do but the egregious misplacements should be gone.

Will recheck for terms that still need to be addressed next week after ontology structure has been updated in AmiGO so that I can look at annotations of plant terms to child terms of 'animal organ development'.

Added:
+id: GO:0090701
+name: specification of plant organ identity
+id: GO:0090700
+name: maintenance of plant organ identity
More incremental clean up of term names to distinguish animal organs from plant organs. Added xp to UBERON:organ and PO:plant organ when possible.

Still left to sort out as they all now still fall under animal organ development but either have plant genes annotated directly to them or have plant specific terms as children terms.

[ ] specification of organ axis polarity

[ ] specification or organ position

[ ] specification of organ number

[ ] regulation of timing of organ formation

[ ] establishment of organ orientation

Added the remaining terms necessary, split out plant and animal and links to plant/animal organ morphogenesis/development terms.  Still need to clean up the annotations.  Will transfer these tomorrow when the new terms are in our curation database.

+id: GO:0090706
+name: specification of plant organ position
+id: GO:0090707
+name: establishment of plant organ orientation
+id: GO:0090708
+name: specification of plant organ axis polarity
+id: GO:0090709
+name: regulation of timing of plant organ formation

5 annotations to A. thaliana left. 3 IMP (UniProtKB), 2 IEA (InterPro).  Two of them were disputed by myself and have been corrected by UniProt just today.  The other IMP annotation I disputed today and I'm sure it will be corrected shortly.
I've opened Interpro helpdesk tickets to address the mappings leading to the two IEA annotations.
[Support #99763] AutoReply: Please update mapping: IPR031135
[Support #99764] AutoReply: Please update mapping: IPR029961
Should all be cleaned up by December 2016.  I hope.

Last IMP annotation has been cleaned up by UniProt (thanks, Emmanuel). The Interpro mappings have been updated and will be released in January.

Thanks for pointing this out Tanya.
I have corrected the mapping, but it won't be publicly visible until our next
release which will be in January.
cheers,
Lorna.


GUI Layout

User Story
As a User I want a Graphical User Interface that supports the basic media viewing concept, so that I can browse and look at various types of media.
Notes
This is a basic UI layout task, with no actual functionality.

Basic layout implemented as described in PR #2


Missing or lost funds

Describe your issue here.
Environment

Elixir Version: 1.9.0

Erlang Version: 22

BlockScout Version: d09fdbfd87d54ba1a80de57ea48c5c7e6f4f61f8

User Agent: Mozilla/5.0 (Linux; Android 8.0.0; SM-G955F) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/88.0.4324.152 Mobile Safari/537.36

Steps to reproduce
Tell us how to reproduce this issue. If possible, push up a branch to your fork with a regression test we can run to reproduce locally.
Expected Behaviour
Tell us what should happen.
Actual Behaviour
Tell us what happens instead.

Hi @sned1966! Could you give us more information here? For example, what network was this on (e.g. mainnet, alfajores)? Where were you sending the transaction from? Were you sending cUSD or CELO? Can you confirm that the funds left from the first address? Any more information here would be super helpful!

https://explorer.celo.org/tx/0xf6ec9039cb39b43a06283eff94f37c3434bb646566c518064ca30510111af833/token_transfers
this is wa hat coin base sent me to show where the money went I sent from
coin base to my Cleo wallet witch I thought was my coin base wallet but
it's not there if you could help me please
On Sat, 20 Feb 2021, 12:51 am medhak1<EMAIL_ADDRESS>wrote:

Hi @sned1966 https://github.com/sned1966! Could you give us more
information here? For example, what network was this on (e.g. mainnet,
alfajores)? Where were you sending the transaction from? Were you sending
cUSD or CELO? Can you confirm that the funds left from the first address?
Any more information here would be super helpful!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-782483987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ2AO7UP2X46XK32QILS74BSVANCNFSM4XOWUGOA
.

Sid that help or is there anything else I can do to recover this money
On Sat, 20 Feb 2021, 11:41 pm Richard Sneddon<EMAIL_ADDRESS>wrote:

https://explorer.celo.org/tx/0xf6ec9039cb39b43a06283eff94f37c3434bb646566c518064ca30510111af833/token_transfers
this is wa hat coin base sent me to show where the money went I sent from
coin base to my Cleo wallet witch I thought was my coin base wallet but
it's not there if you could help me please
On Sat, 20 Feb 2021, 12:51 am medhak1<EMAIL_ADDRESS>wrote:

Hi @sned1966 https://github.com/sned1966! Could you give us more
information here? For example, what network was this on (e.g. mainnet,
alfajores)? Where were you sending the transaction from? Were you sending
cUSD or CELO? Can you confirm that the funds left from the first address?
Any more information here would be super helpful!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-782483987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ2AO7UP2X46XK32QILS74BSVANCNFSM4XOWUGOA
.

I have sent the info over a week ago it would be nice to get some sort of
reply
On Tue, 23 Feb 2021, 3:42 pm Richard Sneddon<EMAIL_ADDRESS>wrote:

Sid that help or is there anything else I can do to recover this money
On Sat, 20 Feb 2021, 11:41 pm Richard Sneddon<EMAIL_ADDRESS>wrote:

https://explorer.celo.org/tx/0xf6ec9039cb39b43a06283eff94f37c3434bb646566c518064ca30510111af833/token_transfers
this is wa hat coin base sent me to show where the money went I sent from
coin base to my Cleo wallet witch I thought was my coin base wallet but
it's not there if you could help me please
On Sat, 20 Feb 2021, 12:51 am medhak1<EMAIL_ADDRESS>wrote:

Hi @sned1966 https://github.com/sned1966! Could you give us more
information here? For example, what network was this on (e.g. mainnet,
alfajores)? Where were you sending the transaction from? Were you sending
cUSD or CELO? Can you confirm that the funds left from the first address?
Any more information here would be super helpful!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-782483987,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ2AO7UP2X46XK32QILS74BSVANCNFSM4XOWUGOA
.

Whoops sorry about that! accidentally deleted a comment instead of replying. So it looks like the transaction successfully left your coinbase account. Can you confirm that the funds have left that wallet? Also can you confirm that your Celo wallet address is 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD (got it from the explorer link you sent)

Yes that's what coin base told me
On Fri, 26 Feb 2021, 8:21 pm medhak1<EMAIL_ADDRESS>wrote:

Whoops sorry about that! accidentally deleted a comment instead of
replying. So it looks like the transaction successfully left your coinbase
account. Can you confirm that the funds have left that wallet? Also can you
confirm that your Celo wallet address is
0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD (got it from the explorer link
you sent)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-786873666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6XALITVORVBPG4G5TTA77DPANCNFSM4XOWUGOA
.

Any update on my funds being returned to my coinbase acc yet
On Sat, 27 Feb 2021, 9:45 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Yes that's what coin base told me
On Fri, 26 Feb 2021, 8:21 pm medhak1<EMAIL_ADDRESS>wrote:

Whoops sorry about that! accidentally deleted a comment instead of
replying. So it looks like the transaction successfully left your coinbase
account. Can you confirm that the funds have left that wallet? Also can you
confirm that your Celo wallet address is
0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD (got it from the explorer
link you sent)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-786873666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6XALITVORVBPG4G5TTA77DPANCNFSM4XOWUGOA
.

Have you deleted my email again
On Wed, 3 Mar 2021, 11:22 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Any update on my funds being returned to my coinbase acc yet
On Sat, 27 Feb 2021, 9:45 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Yes that's what coin base told me
On Fri, 26 Feb 2021, 8:21 pm medhak1<EMAIL_ADDRESS>wrote:

Whoops sorry about that! accidentally deleted a comment instead of
replying. So it looks like the transaction successfully left your coinbase
account. Can you confirm that the funds have left that wallet? Also can you
confirm that your Celo wallet address is
0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD (got it from the explorer
link you sent)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-786873666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6XALITVORVBPG4G5TTA77DPANCNFSM4XOWUGOA
.

Once again I'm left waiting on a reply from you I'm starting to get really
pissed of I gave you the info you require and I feel I have been robbed
On Thu, 4 Mar 2021, 7:28 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Have you deleted my email again
On Wed, 3 Mar 2021, 11:22 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Any update on my funds being returned to my coinbase acc yet
On Sat, 27 Feb 2021, 9:45 am Richard Sneddon<EMAIL_ADDRESS>wrote:

Yes that's what coin base told me
On Fri, 26 Feb 2021, 8:21 pm medhak1<EMAIL_ADDRESS>wrote:

Whoops sorry about that! accidentally deleted a comment instead of
replying. So it looks like the transaction successfully left your coinbase
account. Can you confirm that the funds have left that wallet? Also can you
confirm that your Celo wallet address is
0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD (got it from the explorer
link you sent)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-786873666,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6XALITVORVBPG4G5TTA77DPANCNFSM4XOWUGOA
.

Hi -- apologies. When you say "Celo Wallet", do you mean Valora? and if so, can you confirm that at the bottom left of the Valora App, your address is 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD? if not, I would suggest you also start a ticket with coinbase support. The issue is that your funds aren't necessarily lost, as the address above does contain them, which you can verify here. The issue is that I'm not sure whether you have access to that address or not (do you hold the private keys for that address?). I'm still trying to understand where you got that address from. Unfortunately if you don't hold the keys to that address, there is no way to get those funds returned.
Also, apologies for the delay here -- didn't get any notifications for them. I empathize with your situation, and I'll try my best to help you out!

Coinbase told me you hold the keys when I sent them to my wallet in the box
was my email witch it said it recommends I I hit send and this is where we
are at now this wallet you said how do I get into that do I need to
download it to I though the only wallet I needed was a coinbase one
On Thu, 4 Mar 2021, 5:36 pm medhak1<EMAIL_ADDRESS>wrote:

Hi -- apologies. When you say "Celo Wallet", do you mean Valora? and if
so, can you confirm that at the bottom left of the Valora App, your address
is 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD? if not, I would suggest
you also start a ticket with coinbase support
https://help.coinbase.com/en/coinbase/getting-started/other/how-can-i-contact-coinbase-support.
The issue is that your funds aren't necessarily lost, as the address above
does contain them, which you can verify here
https://explorer.celo.org/address/0xf21490fb30a5ea8af5ac766dd53088b9ac0877fd/transactions.
The issue is that I'm not sure whether you have access to that address or
not (do you hold the private keys for that address?). I'm still trying to
understand where you got that address from. Unfortunately if you don't hold
the keys to that address, there is no way to get those funds returned.
Also, apologies for the delay here -- didn't get any notifications for
them. I empathize with your situation, and I'll try my best to help you out!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-790796601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJYQCHKKFAUM3Q2Z5STTB7AJZANCNFSM4XOWUGOA
.

I just got that wallet that you said and I never did any of that Be for was
not asked to do that when I sent them from coin base as I though I was
sending to coinbase wallet now if you could use the keys that coinbase say
you have to my funds I'm sure this will get sorted fast
On Thu, 4 Mar 2021, 6:13 pm Richard Sneddon<EMAIL_ADDRESS>wrote:

Coinbase told me you hold the keys when I sent them to my wallet in the
box was my email witch it said it recommends I I hit send and this is where
we are at now this wallet you said how do I get into that do I need to
download it to I though the only wallet I needed was a coinbase one
On Thu, 4 Mar 2021, 5:36 pm medhak1<EMAIL_ADDRESS>wrote:

Hi -- apologies. When you say "Celo Wallet", do you mean Valora? and if
so, can you confirm that at the bottom left of the Valora App, your address
is 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD? if not, I would suggest
you also start a ticket with coinbase support
https://help.coinbase.com/en/coinbase/getting-started/other/how-can-i-contact-coinbase-support.
The issue is that your funds aren't necessarily lost, as the address above
does contain them, which you can verify here
https://explorer.celo.org/address/0xf21490fb30a5ea8af5ac766dd53088b9ac0877fd/transactions.
The issue is that I'm not sure whether you have access to that address or
not (do you hold the private keys for that address?). I'm still trying to
understand where you got that address from. Unfortunately if you don't hold
the keys to that address, there is no way to get those funds returned.
Also, apologies for the delay here -- didn't get any notifications for
them. I empathize with your situation, and I'll try my best to help you out!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-790796601,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJYQCHKKFAUM3Q2Z5STTB7AJZANCNFSM4XOWUGOA
.

I'm still waiting to find out what can be done over my missing money you
say it's down to coinbase they say it's down to yourselves and all the time
I'm out of pocket
On Tue, 9 Mar 2021, 12:44 am medhak1, @.***> wrote:

If you hadn't downloaded/used Valora before, then no worries, that
shouldn't be needed now. Unfortunately we don't hold any keys; I think you
should chat with Coinbase's customer service as they will better be able to
serve this specific request, since it seems like they're the ones who
pointed you to the 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD address
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-793209530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJYDB4D5MN3GRZAK3IDTCVVQJANCNFSM4XOWUGOA
.

Are you going to tell me again you did not get notice of my email again I'm
being sent to yours from coinbase your sending me to coinbase and I'm the
one that is missing funds I need this sorted
On Fri, 12 Mar 2021, 10:56 am Richard Sneddon, @.***> wrote:

I'm still waiting to find out what can be done over my missing money you
say it's down to coinbase they say it's down to yourselves and all the time
I'm out of pocket
On Tue, 9 Mar 2021, 12:44 am medhak1, @.***> wrote:

If you hadn't downloaded/used Valora before, then no worries, that
shouldn't be needed now. Unfortunately we don't hold any keys; I think you
should chat with Coinbase's customer service as they will better be able to
serve this specific request, since it seems like they're the ones who
pointed you to the 0xf21490Fb30A5ea8af5aC766dD53088B9aC0877FD address
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-793209530,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJYDB4D5MN3GRZAK3IDTCVVQJANCNFSM4XOWUGOA
.

@sned1966 Hi I was looking over this conversation and wanted to see if I could help in any way.
You said you sent funds from coinbase to another wallet, correct?
Do you have access to this wallet? If you don't have access to this wallet then how did you acquire the address for the wallet?

When I sent them it showed my email address in the little box as I did not
really understand I just hit send I have spoken to coin base they say you
hold the keys so I should ask you for this you have told me they are the
ones to refund me I told them this and they just closed the case
On Tue, 23 Mar 2021, 6:14 pm Dominique Liau, @.***>
wrote:

@sned1966 https://github.com/sned1966 Hi I was looking over this
conversation and wanted to see if I could help in any way.
You said you sent funds from coinbase to another wallet, correct?
Do you have access to this wallet? If you don't have access to this wallet
then how did you acquire the address for the wallet?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-805122211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ3GRY6NKWH4N6BJCDDTFDLBDANCNFSM4XOWUGOA
.

Could you replicate what you did and attach screenshots of your process? Just don't actually go through with any transaction.
Also, I think the coinbase support agent may not be aware of how cLabs works because we don't hold the private keys for any user. Coinbase, on the otherhand, does control the private keys for its users, so I can see why they may have sent you a confusing message.

I just tried it won't let me take a screenshot of the transaction but you
can see from my acc that it's sitting there I can send this one it's the
one that says sent Cleo to Cleo wallet _177 and as for coinbase they just
keep telling me to ask you for the money back
On Tue, 23 Mar 2021, 10:42 pm Dominique Liau, @.***>
wrote:

Could you replicate what you did and attach screenshots of your process?
Just don't actually go through with any transaction.
Also, I think the coinbase support agent may not be aware of how cLabs
works because we don't hold the private keys for any user. Coinbase, on the
otherhand, does control the private keys for its users, so I can see why
they may have sent you a confusing message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-805315813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ44JQTG4FSLEA4BH3TTFEKPBANCNFSM4XOWUGOA
.

@sned1966
Can you send me a link to download the same wallet you used? This will give me more context.

Is this what you are looking for?
On Wed, 24 Mar 2021, 10:33 pm Dominique Liau, @.***>
wrote:

@sned1966 https://github.com/sned1966
Can you send me a link to download the same wallet you used? This will
give me more context.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-806227603,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ3EJAWG2YQBOTOVPLDTFJSCFANCNFSM4XOWUGOA
.

The link is missing

Any luck with my funds
On Thu, 25 Mar 2021, 8:13 am Richard Sneddon, @.***> wrote:

On Wed, 24 Mar 2021, 10:49 pm Dominique Liau, @.***>
wrote:

The link is missing
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-806234006,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ7UF5XUHPJIZKHMR7DTFJT6TANCNFSM4XOWUGOA
.

@sned1966 You forgot to paste the link in your last message.

https://explorer.celo.org/tx/0xf6ec9039cb39b43a06283eff94f37c3434bb646566c518064ca30510111af833/token_transfers
On Sun, 28 Mar 2021, 4:05 am Dominique Liau, @.***>
wrote:

@sned1966 https://github.com/sned1966 You forgot to paste the link in
your last message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-808836548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ7YTSMTN2T6EGR4PKDTF2MJNANCNFSM4XOWUGOA
.

Is that what you needed
On Sun, 28 Mar 2021, 9:26 pm Richard Sneddon, @.***> wrote:

https://explorer.celo.org/tx/0xf6ec9039cb39b43a06283eff94f37c3434bb646566c518064ca30510111af833/token_transfers
On Sun, 28 Mar 2021, 4:05 am Dominique Liau, @.***>
wrote:

@sned1966 https://github.com/sned1966 You forgot to paste the link in
your last message.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-808836548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ7YTSMTN2T6EGR4PKDTF2MJNANCNFSM4XOWUGOA
.

No, I need to know which wallet you tried to send your funds to.
In an earlier comment you said something about a Cleo wallet?
I need to know about your wallet so I know how your address was generated. If I have that information then I will know if it's possible to recover your funds.
So did you get your wallet from the app/play store? From a website?

I tried to send it to my coin base wallet I got that from coin base website
do you need my wallet address?
On Tue, 30 Mar 2021, 1:10 am Dominique Liau, @.***>
wrote:

No, I need to know which wallet you tried to send your funds to.
In an earlier comment you said something about a Cleo wallet?
I need to know about your wallet so I know how your address was generated.
If I have that information then I will know if it's possible to recover
your funds.
So did you get your wallet from the app/play store? From a website?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-809807252,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6CDEB7PUTJHXPLQUDTGEJGRANCNFSM4XOWUGOA
.

Is this any good
On Tue, 30 Mar 2021, 4:41 am Richard Sneddon, @.***> wrote:

I tried to send it to my coin base wallet I got that from coin base
website do you need my wallet address?
On Tue, 30 Mar 2021, 1:10 am Dominique Liau, @.***>
wrote:

No, I need to know which wallet you tried to send your funds to.
In an earlier comment you said something about a Cleo wallet?
I need to know about your wallet so I know how your address was
generated. If I have that information then I will know if it's possible to
recover your funds.
So did you get your wallet from the app/play store? From a website?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/celo-org/celo-monorepo/issues/7068#issuecomment-809807252,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASYFTJ6CDEB7PUTJHXPLQUDTGEJGRANCNFSM4XOWUGOA
.

That's great! I'll close this issue now. Have a great day!


Not Working in Monterey 12.3 on M1 Mac

The extension has stopped populating results through Alfred.

This may be related
Python 2 Workflows on macOS Monterey - Alfred Help and Support

tl;dr: Consider using kinbiko/jisho-alfred until this project is ported to Python 3.

I've been a big fan of this project for a very long time myself, and I hope (without expectations -- it's open source after all!) it'll receive some updates at some point in the future.
In the meantime, since I personally wouldn't be able to port this project from Python 2 to Python 3 in a reasonable timeframe, I've built an equivalent Alfred workflow and program in Go over on kinbiko/jisho-alfred.
I can't guarantee the same ease of use, and shiny finish that this project has, but I figured I'd share here in case it helps unblock others.


Svg memory leak

Each time the color property is updated on the Svg object a bit of memory leaks.  when color changes, reload() is called on Svg(); this in turn calls parse_tree() and render().
So far from experimentation it seems the leak is isolated in render() ; parse_tree() seems to not leak memory.
The attached example file + svg object rapidly changes the color of an Svg object and will show the memory leak.
This was observed on kivy 1.9.0
leaktest.zip

Please test that it is still valid for Kivy 1.9.2. Thank you.

Bumping the request to test on a later version (Kivy 2.2.1).
Please let us know if it is still a problem.