DataContact.kt extensions may return incorrect result if Contact has been linked/unlinked

As discovered in #140, the Contact ID may "change" due to sync or aggregation.
Problem
The ExistingDataEntity.contact function defined in DataContact.kt may return the incorrect Contact or null even if the Contact still exist.
This is due to the fact that the Contact ID may "change" due to sync or aggregation as discovered in #140.
Solution
Query the database using the Data ID (which will not change) to get the Contact ID instead of relying on the Contact ID in memory.

This is released in https://github.com/vestrel00/contacts-android/releases/tag/0.2.0


Fix travis missing package.

Patch addes ninja python module to travis qemu compilation issue.
Signed-off-by: Hariharan T.S<EMAIL_ADDRESS>

@harihare Can you figureout why CI is failing?
-Vasant

@hegdevasant  Yes, working on the  same, first step fixed the travis issue.  Can you please review the same so that I will merge it.


live preview support 1.0 and 2.0

live preview support 1.0 and 2.0

1.0 = when management token is provided, host will be previous host
2.0 = when preview token is provided, host will be new host

@harshithad0703 , we have already released v3.18.1. What should be the version here @ishaileshmishra cc : @nadeem-cs @sunil-lakshman ?

@harshithad0703 This is a new feature, make the version v3.19.0

@sunil-lakshman I think only live preview host and config keys have been updated (live preview feature already exists in the current version), so patch version update should be fine v3.18.2

updated version to v3.19.0 as there is a breaking change that might affect live preview for references


Conditionally invert axis to align annotations with elements

Fixes https://github.com/holoviz/holonote/issues/112
Invert axis if the first kdim of the element is not an annotator kdim, ensuring overlaying annotations align with underlying elements.
  Code
import holoviews as hv; hv.extension('bokeh')
import panel as pn; pn.extension()
import numpy as np

from holonote.annotate import Annotator
from holonote.app import PanelWidgets

annotator = Annotator({"x": float, "y": float}, fields=["description"])
bounds = (-1, -1, 1, 1)
data = np.array([[0, 1], [1, 0]])
img = hv.Image(data, kdims=['x', 'y'], bounds=bounds)
img_right = hv.Image(data, kdims=['z', 'y'], bounds=bounds)  # y as second kdim! so HSpan

left_plot = annotator * img
right_plot = annotator * img_right
layout = left_plot + right_plot

annotator.set_regions(x=(-0.15, 0.15), y=(-0.25, 0.25))
annotator.add_annotation(description="Test")

layout

Thanks to @hoxbro for the assist!


Fixes

Tested on Windows 10 with Cygwin.

Holy poop I forgot this repository even existed ^^

Fixes seem sensible, I'm not sure I have a good test case for the unquoted variable though.  Will merge anyway, I can't see this breaking anything!


Add module_available

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (276a5e5) to head (f2e9aeb).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #624   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           71        71
  Lines         1908      1915    +7
  Branches       233       235    +2
=========================================
+ Hits          1908      1915    +7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


b0map doesn't work with Echotimes that are Tuples

There is an instance where you have echotime * scalar:
https://github.com/MagneticResonanceImaging/MRIFieldmaps.jl/blob/e64a43f033b1c32914332fdec8f8874f019f6a9a/src/b0map.jl#L209
This could be remedied with a .*.
There is also an instance of echotime * vector':
https://github.com/MagneticResonanceImaging/MRIFieldmaps.jl/blob/e64a43f033b1c32914332fdec8f8874f019f6a9a/src/b0map.jl#L228
This would take a little more care to get right. One way could be to replace echotime*df' with outerproduct(echotime, df), and then have
outerproduct(echotime::AbstractArray, df) = echotime*df'
outerproduct(echotime::Tuple, df) = [TE * df for TE in echotime, df in df]

I am just going to use collect.  Stay tuned for a PR.

Thanks for reporting the issue and for the suggestions.
I was trying to be nice by supporting both tuples and vectors but I had never tested the tuples.
Maybe I should have just required it to be a vector 😬
(I wish Julia had a AbtractIterable type to unify all iterables...)

(I wish Julia had a AbtractIterable type to unify all iterables...)

I agree. Though whenever you need iterable * vector' you could argue you really meant to use vectors in the first place, so I think it makes sense just to use collect.

Agreed, I just meant I'd like the argument type annotation to be echotime::Iterable for full generality.
I have merged #16 which should address this.  Do you mind working with the main branch and see if you hit any other issues?  If you prefer that I tag it that's ok too.  I'm going to close this and just reopen if I overlooked any collect needed...


ergotist, decoys

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,e]

/trunk merge


chore: remove aliases in release-5.3

What is changed, added, or deleted? (Required)
chore: remove aliases in release-5.3

Which DM version(s) do your changes apply to? (Required)

[ ] master (the latest development version)
[x] v5.3 (TiDB DM 5.3 versions)
[ ] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[ ] Delete files
[ ] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

/merge


Add NetBSD support

Test with arduino-cli 0.35.3:
picohive: {/home/rxg} uname -a
NetBSD picohive 9.3 NetBSD 9.3 (GENERIC) #0: Thu Aug  4 15:30:37 UTC 2022  mkrepro@mkrepro.NetBSD.org:/usr/src/sys/arch/amd64/compile/GENERIC amd64
picohive: {/home/rxg} arduino-cli board list
Port       Protocol Type              Board Name FQBN Core
/dev/dty00 serial   Serial Port       Unknown
/dev/dty01 serial   Serial Port       Unknown
/dev/dty02 serial   Serial Port       Unknown
/dev/dty03 serial   Serial Port       Unknown
/dev/dtyU0 serial   Serial Port (USB) Unknown
/dev/dtyU1 serial   Serial Port (USB) Unknown
/dev/dtyU2 serial   Serial Port (USB) Unknown
/dev/dtyU3 serial   Serial Port (USB) Unknown
/dev/dtyU4 serial   Serial Port (USB) Unknown
/dev/dtyU5 serial   Serial Port (USB) Unknown
/dev/dtyU6 serial   Serial Port (USB) Unknown
/dev/dtyU7 serial   Serial Port (USB) Unknown
/dev/tty00 serial   Serial Port       Unknown
/dev/tty01 serial   Serial Port       Unknown
/dev/tty02 serial   Serial Port       Unknown
/dev/tty03 serial   Serial Port       Unknown
/dev/ttyU0 serial   Serial Port (USB) Unknown
/dev/ttyU1 serial   Serial Port (USB) Unknown
/dev/ttyU2 serial   Serial Port (USB) Unknown
/dev/ttyU3 serial   Serial Port (USB) Unknown
/dev/ttyU4 serial   Serial Port (USB) Unknown
/dev/ttyU5 serial   Serial Port (USB) Unknown
/dev/ttyU6 serial   Serial Port (USB) Unknown
/dev/ttyU7 serial   Serial Port (USB) Unknown
picohive: {/home/rxg} arduino-cli monitor -p /dev/ttyU0 -c baudrate=115200
Monitor port settings:
baudrate=115200
Connected to /dev/ttyU0! Press CTRL-C to exit.
Bpm:0,SPO2:90.00
Bpm:0,SPO2:90.00
Bpm:0,SPO2:90.00

Hi @picohive thanks for the patch!
Do you actually have 12 serial ports installed?
/dev/tty00 serial Serial Port Unknown
/dev/tty01 serial Serial Port Unknown
/dev/tty02 serial Serial Port Unknown
/dev/tty03 serial Serial Port Unknown
/dev/ttyU0 serial Serial Port (USB) Unknown
/dev/ttyU1 serial Serial Port (USB) Unknown
/dev/ttyU2 serial Serial Port (USB) Unknown
/dev/ttyU3 serial Serial Port (USB) Unknown
/dev/ttyU4 serial Serial Port (USB) Unknown
/dev/ttyU5 serial Serial Port (USB) Unknown
/dev/ttyU6 serial Serial Port (USB) Unknown
/dev/ttyU7 serial Serial Port (USB) Unknown

it looks like those are preallocated devices that are not connected to any actual port (more or less what Linux does with the ttySxx devices). Is there a way to filter those from the list output?

Indeed, they are just available precreated node. There is no sysfs on NetBSD yet, maybe have a better solution in the future.

Indeed, they are just available precreated node. There is no sysfs on NetBSD yet, maybe have a better solution in the future.

Indeed, they are just available precreated node. There is no sysfs on NetBSD yet, maybe have a better solution in the future.

Indeed, they are just available precreated node. There is no sysfs on NetBSD yet, maybe have a better solution in the future.


Using a PlaceholderField in a admin inline?

I can't figure out from the docs how go do this (or if this is possible). I have models like so:
class Slider(models.Model):
    name = models.CharField(max_length=128)

class Slide(models.Model):
    parent_type = models.ForeignKey(ContentType)
    parent_id = models.PositiveIntegerField()
    content_object = generic.GenericForeignKey('content_type', 'object_id')
    content = PlaceholderField("slide_content")

and an admin.py file:
class SlideInline(PlaceholderEditorInline):
    model = Slide
    extra = 1

class SliderAdmin(PlaceholderFieldAdmin):
    inlines = [SlideInline, ]

(I've left out the imports and registration), but this doesn't seem to work. Is this possible?

Hi, do you have above working sample? Wonder if you could share that? I am in need of such.


Vectorized typo

Typo fix...

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/eb08b55ba5dda4eb905f88a33a44c043e30de450/
Edited pages:

v19.2/vectorized-execution.md

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9d4b7090d1acdc96d8d06ea2087f31e944e0541a/
Edited pages:

v19.2/vectorized-execution.md


Ventur Proposal

Project Abstract
Ventur is focused on decentralizing business processes and services for professionals. Ventur’s initial goals are:

To enable decentralized marketplaces for professional engagement, encompassing business-to-business engagement and employment contracts between businesses and individuals.
To enable assignment and evaluation of technical credentials.

Project Overview on Google Slides
Project Summary on Google Docs
Grant level

[ ] Level 1:  Up to $10,000, 2 approvals
[x] Level 2:  Up to $30,000, 3 approvals
[ ] Level 3:  Unlimited, 5 approvals (for >$100k: Web3 Foundation Council approval)

Application Checklist

[x] The application template has been copied and aptly renamed (project_name.md).
[x] I have read the application guidelines.
[x] A BTC, Ethereum (USDT/USDC/DAI) or Polkadot/Kusama (aUSD) address for the payment of the milestones is provided inside the application.
[x] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted (see the announcement guidelines).
[ ] I prefer the discussion of this application to take place in a private Element/Matrix channel. My username is: @_______:matrix.org (change the homeserver if you use a different one)

Hi, thank you for taking the time to review our application.
For rmrk’s NFT implementation, we believe we will need to extend functionality beyond just the transferability flag.  Many of the Technical Certifications we want to represent have expirations.  We believe that we’ll need to add expiration functionality to appropriately represent those certifications.  Outside of our core certification/accreditation target, we have also seen significant interest in Soulbound Subscriptions with expiration functionality, which we believe our implementation could better address vs the current implementation.  Additionally, rmrk’s implementation does not currently need a functionality for discarding non transferable nfts, we believe there may be interest in having the ability to discard certain soulbound tokens (even within their expiry time should that be applicable) if their relevance is no longer beneficial to the holder.
The Escrow pallet #1094 intends to implement 2 party escrow.  While that is part of what we functionally wish to deliver, it might make sense for us to rename our feature, as we want our escrow fund to be something which can hold and distribute funds across a category of rfps (initially targeted as rfps from a single provider).  We also want to enable a configuration for escrows to accept open funding.  Enabling organizations to raise funds for a given category of work that might be of value to their community or a broader audience.

Hi @Gryczka Just checking up to see how the progress is going on M2?

Hi @keeganquigley, greatly appreciate the check in here.
Maciej from our team has had most of the technical delivery for the pallets wrapped up since late November, but we hit some delays in prioritizing the delivery submission itself and the walk-through test guides.
I've just pushed the test guides and some test coverage that should complete the M2 deliverables, and we should have the delivery milestone submitted for initial review and feedback within the next 48 hours.


add support for hashiCorp Valult

closes #63

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

for this is needed to also add support into https://github.com/CZERTAINLY/ansible-role-czertainly

support in role was resolved https://github.com/CZERTAINLY/ansible-role-czertainly/pull/51


Allow explicit specification of bands to be retrieved by Retrieve Pixel Drill tool

Allow an explicit list of bands to be retrieved to be specified as a command line argument to the Retrieve Pixel Drill tool

The Retrieve Pixel Time Series tool now supports the --band BAND command line argument in addition to --bands-all or --bands-common command line arguments.


Receive Mail - Connection timeout. Terminating client connection

Greetings,
I can see the email hitting MailSlurper, but the final step states Connection timeout. Terminating client connection.

I'm not entirely sure how to debug this, I haven't been able to find any logs.
My config:
{
"wwwAddress": "local.mailslurper",
"wwwPort": 8585,
"serviceAddress": "local.mailslurper",
"servicePort": 8085,
"smtpAddress": "local.mailslurper",
"smtpPort": 2525,
"dbEngine": "MSSQL",
"dbHost": "momentumsql.momentum.com",
"dbPort": 1433,
"dbDatabase": "mailslurper",
"dbUserName": "SqlUsername",
"dbPassword": "SqlPassword",
"maxWorkers": 1000,
"autoStartBrowser": false,
"keyFile": "",
"certFile": ""
}
Nothing gets written to the database and with every subsequent test it grows the queue.
What is exactly timing out and how can I resolve it?
http://local.mailslurper:8585/ returns client interface
http://local.mailslurper:8085/ returns 404 page not found
Any and all help is appreciated.
Thank you kindly,
Adam

I seem to have a similar problem. It stops at the same step, after logging the content type.
This project was originally in VBA and using CDO.Message. Since Visual Basic for Applications isn't exactly my strong suit I started doubting my own experience with it and so rewrote it in .NET like this:
using System.Net.Mail;

namespace MailSender
{
    class Program
    {
        static void Main(string[] args)
        {
            SmtpClient smtp = new SmtpClient("localhost", 250);
<EMAIL_ADDRESS><EMAIL_ADDRESS>                "Test Message", "This is a test message. Consider yourself chosen!");
        }
    }
}

MailSlurper config.json is:
{
	"wwwAddress": "localhost",
	"wwwPort": 8888,
	"serviceAddress": "localhost",
	"servicePort": 8085,
	"smtpAddress": "localhost",
	"smtpPort": 250,
	"dbEngine": "SQLite",
	"dbHost": "",
	"dbPort": 0,
	"dbDatabase": "./mailslurper.db",
	"dbUserName": "",
	"dbPassword": "",
	"maxWorkers": 1000,
	"autoStartBrowser": false,
	"keyFile": "",
	"certFile": ""
}

Console log is:
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Worker 11 queued to handle connection from <IP_ADDRESS>:62517
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Reading data from client connection...
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Mail from<EMAIL_ADDRESS>MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Mail MIME-Version:  1.0
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Mail Date:  2017-05-31 09:56:55
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Mail Subject:  Test Message
MailSlurper: INFO - 2017/05/31 09:56:55 libmailslurper: INFO - Mail Content-Type:  text/plain
MailSlurper: INFO - 2017/05/31 09:57:01 libmailslurper: INFO - Connection timeout. Terminating client connection

Finally, I also note that it's not reducing the queued workers until restarting MailSlurper?

The tool smtp4dev is working better for me as a (Windows-only) workaround for now.

Using nearly the same C# code, I seemed to get rid of the "terminating" problem by disposing the MailMessage object.... However, even after that, with the default basic setup, no email are being received in the DB showing in the web interface.

I have been reading that .NET SmtpClient doesn't end the session according to protocol, is this the underlying reason for this ? -https://social.msdn.microsoft.com/Forums/en-US/879f13d7-24e6-4a0f-b396-627e9da25fc1/bugs-in-systemnetmailmailmessage-net-20-sp1?forum=netfxnetcom

I just switched to using MailKit for .NET and this problem goes away, so I think we can conclude the problem is related .NET SmtpClient (even the .NET4 version which is supposed to have fixed all the RFC standard issues)

I'm having this same issue running MailSlurper on a Windows VM, and then attempting to send email to it via java. The java app actually believes that the mail was successfully received, but I get the Connection timeout error in MailSlurper on the VM and no mail shows up in MailSlurper.

@danshort thank y ou, I just realized myself, I was also running the client on a WIndows VM and connecting to my host machine where MailSlurper was running... so perhaps the problem is a more complex network issue

I'm going to close this one in favor of #53. The ability to support persistent connections (up to a point) will be in release 1.13.0.


Dynamo primary sort key

Is your feature request related to a problem? Please describe.
amplify init and amplify push api commands don't allow for Dynamo tables to be created with a sort key.
Describe the solution you'd like
There was some discussion here of a possible solution but it doesn't seem to be implemented yet https://github.com/aws-amplify/amplify-cli/issues/56#issuecomment-416736520
Also discussed here https://github.com/aws-amplify/amplify-cli/pull/277#issuecomment-428690938
Something like this seems like it would work for my case of needing a sort key on a timestamp
type DeviceData @model {
  id: ID!
  timestamp: String! @sort
  value: String!
}

Describe alternatives you've considered
Not aware of any, would be open to anything since sort keys cannot be added to already created Dynamo tables.

I would like to second this request

Thank you for the note. This is a highly request feature and your feedback really helps us prioritize it. I will make sure this is included in discussions and will report back when I have more details.

Would just like add that our organization would also love to see this feature.  As it currently stands, we're going to have to try manually modifying the cloudformation template that amplify creates in order to adjust the partition and sort keys.

@zjullion I think you can link into an existing table or manually create a table and write your own custom resolvers. It may be easier to just form your own dynamo tables and write the custom resolvers than trying to fiddle with the auto-generated CloudFormation template.

This is definitely our most needed feature at the moment. Would it be feasible to support specifying multiple sort fields? Example: sorting on createdAt or updatedAt. Clients would then specify what sort type to use in their graphql query variables.

+1 We also really need the feature.

Wanted to ping this ticket again. We are working on an RFC for custom indexes on @model types and would love to hear your feedback when it is posted to GH.

@mikeparisstuff would be great to have a way not only specify sort key but partition too.

+1 We really need the feature also

+1 It would be great to have this feature.

+1

+1

+1

What is the best temporary way to achieve this ?

How is this still not resolved?  This seems like it should be trivial.  We need to define 1 to many relationships (user videos) and require a sort key.

+1

Currently for sorting, I am getting the query back from the response in call back then creating my own sorting algorithm. I think sorting is a fundamental operation for database/querying it might not be core, but it is used so much that it seems core. I rather not use this @index implementation to interfere with schemas that I am already satisfied with. To my benefit I am only returning at most 20 results. If it was 100's or 1000's of data to sort I would be in big problem. But sorting is needed. Would appreciate if some sorting mechanism would be provided out of the box for appsync\graphql

We launched support for custom indexes today. You can find docs for the same out here - https://aws-amplify.github.io/docs/cli/graphql#key
Please let us know if you're still not able to solve your problem through this solution and we'll re-open this issue for you.

Awesome work! 👍

Thanks a lot waking up with this news, I will try it out asap.

@kaustavghosh06  I assume this will be in version 2.9.1  for android-sdk-appsync.

https://stackoverflow.com/questions/56957835/use-key-to-get-only-your-own-data-shown-with-amplify
can someone help?


feat: Add support for generic 4 byte unsigned integer

This is a rebase of #43 on top of the current master. Thanks @dolfandringa!

thanks for taking this over


Upgrade button and modal

[x] Show/hide the button based on Sofia feature flag.
[x] trigger api (https://aps-web-stage.adobe.io/webapps/access_profile/v3?include_disabled_fis=true) and get the URL to be loaded in an Iframe/modal.

Open questions:

[ ] what would be the feature flag of "free user", whom the upgrade button is to be shown ?

[ ] in the payload for the api https://aps-web-stage.adobe.io/webapps/access_profile/v3?include_disabled_fis=true .
where do we get values for: instanceId, contextual_params etc. ?


[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog
:cl:

fix: fixed some stuff i forgor on new nukeops shuttle

removing a cell recharger is cringe...

removing a cell recharger is cringe...

you still get a normal recharger + 2 borg rechargers


feat: add external resource link ui

What are the relevant tickets?
Part of https://github.com/mitodl/hq/issues/1812
Relates to https://github.com/mitodl/hq/issues/3358
Supersedes https://github.com/mitodl/ocw-hugo-themes/pull/1269
Description (What does it do?)

Adds UI that indicates an external resource link.
Adds navigation confirmation model.

Screenshots (if appropriate):

How can this be tested?

Runyarn start course ht-test-2024

Additional Context
https://github.com/mitodl/hq/issues/3167

These external resources open in the same tab. Would you like them to open in a new tab?

They should open in a new tab. I think that's the user expectation based on the icon. (@mbilalmughal let me know if you disagree)

The external resources inside the page content also have an icon and open a confirmation model.

I think this is the right thing to do. But it won't be used in the short term, so we can try it out and get feedback.

@pdpinch @HussainTaj-arbisoft Agreed, it should open in a new tab.

@pdpinch @HussainTaj-arbisoft Yes, I agree. It should open in a new tab.

This looks good to me.
@mbilalmughal have you taken a look at the icon and styling?

This looks good to me.
@mbilalmughal have you taken a look at the icon and styling?

Yes, LGTM 👍

@mbilalmughal and I discussed how the modal looks for extra-small screens. He suggested a few improvements that I have added in https://github.com/mitodl/ocw-hugo-themes/pull/1368.


[FEATURE] Cooldowns Command

Is your feature request related to a problem? Please describe.
Cant see all cooldowns at once
Describe the solution you'd like
A command which displays an overview over all cooldowns

Integrated Cooldowns in StatsCommand


fail compile depthconv

Fails to compile the depthconv function. I'm using PyTorch 1.0.

@lixiang-ucas I met the same problem as you. Have you solved it? thx

@lixiang-ucas I met the same problem as you. Have you solved it? thx

have you solved this problem~？ thanks you!


opacity

I've tried several things to get opacity to work, including editing the shaders.  Any suggestions?

nvm my bug was adding things to the scene multiple times, they were overlapping


fix: Catch the exit of the server process

Fail immediately if the server cannot be started because the
server process failed to start
Close #89

thank you @idmontie I will check the build and it should be released shortly

:tada: This PR is included in version 1.7.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BEAM-3891] Add a suborder parameter to Networks#topologicalOrder

This allows a caller to provide the order in which nodes are considered.
This permits a network to be sorted in a stable topological order,
rather than an arbitrary one based on the iteration order of the
underlying network.

Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @youngoli


Documents Class not subscriptable

Describe the bug
The "documents" class is supposed to be subscriptable but the class seems to be missing the "getitem()" method required to do so.
To Reproduce
Steps to reproduce the behavior:
import photoshop.api as ps
app = ps.Application()
app.documents[0]
Expected behavior
The following error should be returned:
"Traceback (most recent call last):
File "", line 1, in
TypeError: 'Documents' object is not subscriptable"
Screenshots

Desktop (please complete the following information):

OS: Windows10 Pro
Photoshop Version: Adobe Photoshop Version: 21.2.2 20200807.r.289 2020/08/07: 287d0d87ec  x64
Python Version: Python 3.8.3

Additional context
Please see Photoshop Scripting documentation on "Documents" class.
https://theiviaxx.github.io/photoshop-docs/Photoshop/Documents.html

Hi, @SThomasN  Thank you for your report. I checked the code again and found that I did not implement this function. I will create a new MR to update it.

Hi, @SThomasN I have been released a new version to implement this function. Thank you!
@all-contributors please add @SThomasN for Bug reports.


chore(k3s): update default Kubernetes version to 1.21

Signed-off-by: Raphaël Pinson<EMAIL_ADDRESS>

Update failure seems to be unrelated to version change.


Contributor's forms

Review and update style for all contributor's accessible forms.
Mainly:

News
Events
Library Resource (needs separate ticket, lots of things todo)
Profiles (User)
Organizations (Later MS? see also https://github.com/landportal/drupal/issues/5)
Blog post

For the update :)
it's on Ivo's sprint 2 / part 1.3 (?)

This will be done while (re)implementing the LP theme from a Bootstrap fork.
Closing this item for now as this is too vague and not really actionable.


feat/license: change license to dual MIT/Apache-2.0

CLOSES #4

@homu r+

:pushpin: Commit 66ec21b has been approved by hobofan

:zap: Test exempted - status


Release notes for v2.1.0-alpha.20180604

See https://github.com/cockroachdb/cockroach/issues/25351.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d89e75306685af5998109804156e756c05c45aa1/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/d73a7c9fc7936c23b971d8eb2589154e4d83494c/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/945636fe2d4bc6cd4f5373652b6f970222f52899/

@couchand, @vilterp, I don't see user auth called out in the release notes. Do you have specific PRs/commits that should be included?
@windchan7, @bdarnell, I don't see upgrade auto-finalization called out in the release notes. Do you have specific PRs/commits that should be included?

Review status: 0 of 4 files reviewed at latest revision, all discussions resolved, some commit checks failed.

releases/v2.1.0-alpha.20180604.md, line 35 at r3 (raw file):

### General Changes

- New clusters and existing clusters [upgraded](../v2.1/upgrade-cluster-version.html) to this version of CockroachDB will include two new empty databases, `defaultdb` and `postgres`. The `defaultdb` database is automatically used for clients that connect without a current database set (e.g., without a database component in the connection URL). The `postgres` database is provided for compatibility with PostgreSQL client frameworks that require it to exist when the database server has ben freshly installed. Both new databases behave like any other regular database and, if deemed unnecessary, can be [manually deleted](../v2.1/drop-database.html). [#24735][#24735] {% comment %}doc{% endcomment %}

has been* freshly installed

releases/v2.1.0-alpha.20180604.md, line 64 at r3 (raw file):

- [`TRUNCATE`](../v2.1/truncate.html) commands are now logged in the event log. [#25868][#25868]
- Improved [`IMPORT`](../v2.1/import.html) error messages. [#26032][#26032]
- [`IMPORT`](../v2.1/import.html) now supports `mysqldump` SQL as a data format. [#25783][#25783] {% comment %}doc{% endcomment %}

Can we move this next to the previous IMPORT improvement for mysql since they are related?

Comments from Reviewable

Auto-finalization was cockroachdb/cockroach#24987
I think UI login is being left out of the release notes until it's enabled by default.

I think UI login is being left out of the release notes until it's enabled by default.

That was the thinking.  Do we generally have release notes for experimental features?  We can start adding them to further login work.

@couchand, I don't know if we have a rule about mentioning experimental features in release notes. If we want users to try something out, in addition to having docs, mentioning in release notes is probably a good idea.

Review status: 0 of 4 files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

releases/v2.1.0-alpha.20180604.md, line 35 at r3 (raw file):
Previously, nstewart (Nate) wrote…
has been* freshly installed

Done.

releases/v2.1.0-alpha.20180604.md, line 64 at r3 (raw file):
Previously, nstewart (Nate) wrote…
Can we move this next to the previous IMPORT improvement for mysql since they are related?

Done. I usually just leave the notes in default order (by PR) but no problem to move around when it's helpful.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a1e6ba0b2df7c4289606136d787767729a50e87f/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/4ed2761bc7a0d9952d431e91a6c81b8a9fcc8541/

I was also thinking we'd leave login out of release notes because it's experimental — but then again I also thought at the time that we'd leave it out of docs, but we decided to put it in. Does seem a bit odd to have one but not the other. @piyush-singh do you have an opinion on this?
@jseldess if we decide to include release notes, I'd probably say something like this:
Require login on secure clusters if the COCKROACH_EXPERIMENTAL_REQUIRE_WEB_LOGIN environment variable is set (https://github.com/cockroachdb/cockroach/pull/25057)

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f22345c7ec5f2f68493b9fdae37e45d5b481f07f/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9ecc0250707614200f4533344effd54d1a2aa419/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9e1818f54caef80c314320a3daa1a854a5ddcefd/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b20aa313c86d2df08dfddbf743594c495db88012/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/6d1ce1a1658223b721f8b5ecb546389701b5c57b/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b7718776c22c4e8b7a13d6219e7bc940087ff08a/


The partner instant exchange SimpleSwap.io

Hi,
I am Nicolas, the manager at SimpleSwap.io, an instant cryptocurrency exchange.
We present a simple and easy-to-use platform that works without registration and limits and allows users to make swaps in just two clicks.
We successfully support USDC on our service for a long time and we have been exchanging USDC for more than 170 coins to our customers. I tried to find contact with your team but without results.
I'm sorry that I invaded here, but maybe you can tell me where I can turn. I would like to make contact so we can have an interaction.

Contact information is available at https://www.centre.io/contact


flutter pub get taking very long time to execute.

Analyzer Feedback from IntelliJ
Version information

IDEA AI-191.8<IP_ADDRESS>91312
2.7.0-edge.d45c3d15cb3cea0104a87697c085259666eec528
AI-191.8<IP_ADDRESS>91312, JRE 1.8.0_202-release-1483-b03x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1536x864

Exception
Dart analysis server, SDK version 2.7.0-edge.d45c3d15cb3cea0104a87697c085259666eec528, server version 1.27.4, FATAL error: Failed to handle request: {id: 4, method: analysis.setAnalysisRoots, params: {included: [D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield], excluded: [D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield\.idea, D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield\.dart_tool, D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield\.idea, D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield\.pub, D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield\build]}, clientRequestTime:<PHONE_NUMBER>383}
FileSystemException: Directory watcher closed unexpectedly, path = 'D:\Zakir\workspace\technologies\flutter\workspace\customdropdownformfield'
#0      _FileSystemWatcher._listenOnSocket.<anonymous closure> (dart:io-patch/file_patch.dart:309:11)
#1      _ExpandStream._handleData (dart:async/stream_pipe.dart:248:23)
#2      _ForwardingStreamSubscription._handleData (dart:async/stream_pipe.dart:164:13)
#3      _rootRunUnary (dart:async/zone.dart:1136:13)
#4      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#5      _CustomZone.runUnaryGuarded (dart:async/zone.dart:931:7)
#6      _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:336:11)
#7      _BufferingStreamSubscription._add (dart:async/stream_impl.dart:263:7)
#8      _SyncStreamControllerDispatch._sendData (dart:async/stream_controller.dart:764:19)
#9      _StreamController._add (dart:async/stream_controller.dart:640:7)
#10     _StreamController.add (dart:async/stream_controller.dart:586:5)
#11     new _RawSocket.<anonymous closure> (dart:io-patch/socket_patch.dart:1384:35)
#12     _NativeSocket.issueReadEvent.issue (dart:io-patch/socket_patch.dart:890:18)
#13     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#14     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#15     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#16     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

For additional log information, please append the contents of
file://C:\Users\ZAKIRP~1\AppData\Local\Temp\report1.txt.

Thanks for reporting this issue! This was previously reported at https://github.com/dart-lang/sdk/issues/38853; please follow along there.

Duplicate of #38853


pulsarAdmin topic list non-persistent is empty

Describe the bug
Use PulsarAdmin to create a non persistent topic, But the query topic list is empty
To Reproduce
Steps to reproduce the behavior:

pulsar-admin topics create-partitioned-topic non-persistent://public/default/my-topic -p 3
pulsar-admin topics list public/default

Expected behavior
can see the topic
Screenshots

Desktop:

OS:  mac os

version

pulsar 2.8.0

@shibd Maybe you should use list-partitioned-topics

@shibd Maybe you should use list-partitioned-topics

@shibd  Hi, Does it solved your problem?

Maybe you should use list-partitioned-topics

Thanks, It can work.


[Proposal] Replace --debug with --file for vet

Currently the vet command only accepts a --debug boolean argument to be able to vet the module against debug_values.cue file.
I find this file quite confusing, and often removes it all together with the debug_tool.cue. I think it would be useful for the vet command to take a --values (or --value if we want to keep it simple) argument, and vet against those(this) values file(s).
It would help us testing complex module where there's mutually exclusive configuration value (cloud provider for example).
What do you think @stefanprodan? If ok with you I can push a PR

I'm Ok adding -f, --values strings same as the for the timoni build command. I'm using --debug in lots of places, so please leave that as it is, if custom values are provided, those would take precedence over debug.

Copy, thanks! Will push you a MR today


Add date section to the notes

Hi, I would like to work on this issue. Should every note report a timestamp or just the date?

@Levyathanus  sure giving timestamp is much better option

Great, I've opened a PR for this!

well done


Update scalac-scapegoat-plugin version or find a flexible way to determine it.

As there are more people using scala 2.12 exclusively, it might be a good idea to update the version as there is no scalac-scapegoat-plugin 1.0.0 available for scala 2.12

+1

I tried that but as soon as i start sbt, it still wants to pull in the 1.0.0 for scala 2.12. If that would have worked, i wouldn't have opened an issue.

Really ?
I had just deleted my ~/.ivy2/cache/ folder and started a new fresh sbt session in a project that uses scapegoat. and it only downloaded the following two artifacts related to scapegoat.

com.sksamuel.scapegoat#sbt-scapegoat;1.0.9!sbt-scapegoat.jar
com.sksamuel.scapegoat#scalac-scapegoat-plugin_2.12;1.3.4!scalac-scapegoat-plugin_2.12.jar

Will give it another try and see what happens and clean the whole cache to be certain.

Ok,
If you still had the problem after that, you could create a dummy repo that replicates the problem, I will be happy to clone that repo and check what is making it fail.
Maybe one of your plugins or dependencies is the one that is trying to get the scapegoat plugin 1.0.0 ?

I haven't figured it out yet, but when creating a new project with only scapegoat plugin, the dependency resolution works as described with the scapegoatVersion in ThisBuild.
Will have to go with your comment, that it is something else in the project that triggers that depenency.

got the same issue on multi-module project.
added:

addSbtPlugin("com.sksamuel.scapegoat" %% "sbt-scapegoat" % "1.0.9")
scapegoatVersion := "1.3.8" shows red/error in IDEA

trying to reload SBT session fails with missing com.sksamuel.scapegoat:scalac-scapegoat-plugin_2.12:1.0.0:

ok, got this working...
in root aggregator module, I had to add explicitly
  .settings (inThisBuild(Seq(scapegoatVersion := "1.3.8")))

as I have
object BuildSettings {
  // ThisBuild settings take lower precedence, but can be shared across the multi projects.
  lazy val thisBuildSettings: Seq[Setting[_]] = inThisBuild(Seq(
//...
    scapegoatVersion := "1.3.8"
))
}
``` which doesn't get loaded (as it is lazy) soon enough and SBT sees hardcoded `1.0.0` version instead


separate flats and darks

[X] separate flats selector
[X] separate darks selector
[X] slurm integration
[ ] check projection size before normalization

check projection size before normalization is considered unnecessary


Write next version in a make file

Dependencies

Feature Request
Updating the make file  with the next calculated version.

Hey, I would like to work on this

hi @anoopmsivadas , ok !


GO / RHEA not 1:1

GO:0004317 and GO:0008659 both have mapping to the same RHEA id: RHEA:13097.
There may be others like this, I have not exhaustively checked.

Needs to merge; apparently one is an exact synonym of the other

oops ignore previous comment (I deleted)
GO:0004317 3-hydroxypalmitoyl-[acyl-carrier-protein] dehydratase activity
GO:0008659 (3R)-hydroxymyristoyl-[acyl-carrier-protein] dehydratase activity
Both mapped to EC <IP_ADDRESS> 3-hydroxyacyl-[acyl-carrier-protein] dehydratase.
but  has synonyms to hydroxyoctanoyl-[acyl-carrier-protein] dehydratase.
RHEA:13097 | a (3R)-hydroxyacyl-[ACP] = a (2E)-enoyl-[ACP] + H2O; this is generic
Although the EC id may be correctly mapped for mechanism, the two GO terms refer to a specific substrate.
I will request CHEBI ids for both of the these; the RHEA will be removed for the specific terms.
It will however remain for GO:0019171

4 rhea ids requested to apply to 4 reactions where the acyl-group is specified.

BTW, here is a case where if the Defs had a PMID there would be no problem getting a RHEA. I am forced to use annotation references, and for two of these, the annotations are not experimental 8-(

It turns out there were RHEA terms (hard to find).
Adding rhea to
GO:0004317 RHEA:41908
GO:0008659 RHEA:41892
GO:0008693 RHEA:41860
GO:0047451 RHEA:41844
PMID refs also added to these terms.
This addresses everything in this ticket.


Move to latest stack resolver and cleanup

I wanted to play with this and saw that it uses a old Stack snapshot. So I upgraded the resolver to the latest LTS and cleaned-up the stack.yaml file. It all compiles and works perfectly on my end, :+1:

Looks great, thanks!


fix(trading): fee table label changes

Related issues 🔗
Closes #6707
Demo 📺

Technical 👨🔧
Used muted text class in order to de-emphasise the final 3 columns.

Missing a few things

Taker fee BEFORE discount wants to go to the left to become the second column
Also I notice that we dont show the % symbole on the infra / maker / taker fee before discount
And we wanted some way to emphasise the taker fee after discount .... maybe make it bold?  Or highlight the cell? or provide some extra border on the column?  Something to emphasise that it is really the key thing to worry about


OneHotEncoder(sparse=True)

sklearn.preprocessing.OneHotEncoder has the option sparse=True, to return the output in a scipy.sparse matrix.  This can be really useful if you have categories with high cardinality.
Would it be possible to add a sparse=True option to category_encoders.one_hot.OneHotEncoder?

This is a valid feature request. Commits are welcomed.

Awesome, thanks.  I'll take a look at the code, and if I can easily implement it on my own, I'll take a stab at it.

May I ask why the project wants to re-implement encoders that are already part of sklearn? I thought it was complementing sklearn in way by only adding encoders that are not available there yet?

I see two reasons why it may be desirable:

For being able to quickly compare multiple encoders. By having them all in a single package, you may reasonably expect them to use the same interface everywhere, making it easy to compare one method with another.
Support for pandas DataFrames.

I totally agree with point 1.
For point 2 I think sklearn also supports pandas DataFrames:
>>> import pandas as pd
>>> from sklearn.preprocessing import OneHotEncoder
>>> enc = OneHotEncoder()
>>> df = pd.DataFrame([("a",), ("b", )], columns=["foo"])
>>> enc.fit(df)
OneHotEncoder(categorical_features=None, categories=None, drop=None,
              dtype=<class 'numpy.float64'>, handle_unknown='error',
              n_values=None, sparse=True)
>>> enc.transform(df)
<2x2 sparse matrix of type '<class 'numpy.float64'>'
        with 2 stored elements in Compressed Sparse Row format>

This is for sklearn version 0.21.2
Concerning the issue: What do you think about just wrapping around sklearn's OneHotEncoder. That way we would have all features available. We'd also be befitting from future updates/enhancements in sklearn.

category_encoders.one_hot.OneHotEncoder has 2 additional features I often use that are not in sklearn.preprocessing.OneHotEncoder:

drop_invariant=True to drop columns with zero variance (e.g. a categorical feature that is all one level).
handle_missing=True to encode NaNs as their own level (rather than erroring).

Honestly, I think it might make sense to open a PR to sklearn to port these 2 features from category_encoders
What do you all think?

Also, category_encoders has category_encoders.ordinal.OrdinalEncoder while sklearn has sklearn.preprocessing.OrdinalEncoder.  In this case OrdinalEncoder has 3 features missing from sklearn:

drop_invariant=True.
handle_missing=True.
handle_unknown=True to handle encoding for new categories.

What's interesting is that the sklearn OneHotEncoder has an handle_unknown option while the sklearn OneHotEncoder does not.
One thing I really like about category_encoders is that every encoder (except hashing, which doesn't need it) has an handle_missing and handle_unknown option.  It'd be really useful to have both of these options in the sklearn encoders too.

Good to see improving support for pandas DataFrames in sklearn.

What do you think about just wrapping around sklearn's OneHotEncoder. That way we would have all features available. We'd also be befitting from future updates/enhancements in sklearn.

I am leaving the decision up to @wdm0006. It would be necessary:

Wrap OneHotEncoder and OrdinalEncoder.
Get the wrappers to pass the tests. Or change the unit tests and all the remaining encoders to behave more like sklearn encoders. Possible difficulties: different/missing arguments like mapping, handle_missing or handle_unknown. And handling of "ordered" Categoricals from pandas.

The best possible outcome I can think of is adding the missing functionality into sklearn and porting category_encoders to use sklearn encoders.

Should I make a feature request on sklearn to add handle_missing and handle_unknown to their cat encoders?

Here's a request to add handle_missing to the OneHotEncoder in sklearn https://github.com/scikit-learn/scikit-learn/issues/11996

I also opened an issue for OrdinalEncoder: https://github.com/scikit-learn/scikit-learn/issues/17123


Make get_transitive public

Am new to this repo - are any other steps needed to release this change?

Am new to this repo - are any other steps needed to release this change?

No.. we don't have bazel CI yet :P I'll trust you have it working locally for now


add Pyth to Aptos

Description
Hi all, Mario from Pyth here. Would like to add Pyth oracle and how to use Pyth on Aptos for developers
Checklist

Do all Lints pass?

[x] Have you ran pnpm spellcheck?
[x] Have you ran pnpm fmt?
[x] Have you ran pnpm lint?

I think it might be good to have Oracles be its own directory, and have pyth as a subfile inside that

I agree. Adding another oracle in this single page layout would make it unwieldy. The main Oracles page could have the blurb about what oracles are, then maybe link to the sub-page on the page itself (in addition to the sidebar)?

hi all @igor-p @gregnazario @hariria let me know if there's anything I can do to expedite this!
Want to make sure that the Aptos community can leverage Pyth oracles when building their projects


Erkling abilities fired the wrong way

When dueling an Erkling, their abilities seem to trigger from the wrong party. It looks as if the player is casting the ability at the Erkling instead of vice versa. I tested this with a couple of other Magic mobs to find that they did not share this issue.
Video
Erkling Mob
Abilities
Replication

Perform /mmob spawn erkling on the latest BP and Magic
Initiate a duel with the creature and wait for the interval triggers to fire.

Fixed in latest Magic dev build.


Uncaught TypeError: Cannot read property 'toString' of null

[Enter steps to reproduce below:]

...
...

Atom Version: 1.1.0
System: Mac OS X 10.11.1
Thrown From: haskell-ghc-mod package, v1.1.1
Stack Trace
Uncaught TypeError: Cannot read property 'toString' of null
At /Users/weijiayi/.atom/packages/haskell-ghc-mod/lib/ghc-modi-process-base.coffee:163

TypeError: Cannot read property 'toString' of null
  at Socket.<anonymous> (/Users/weijiayi/.atom/packages/haskell-ghc-mod/lib/ghc-modi-process-base.coffee:163:41)
  at Socket.g (events.js:260:16)
  at emitNone (events.js:67:13)
  at Socket.emit (events.js:166:7)
  at emitReadable_ (_stream_readable.js:407:10)
  at emitReadable (_stream_readable.js:401:7)
  at onEofChunk (_stream_readable.js:386:3)
  at readableAddChunk (_stream_readable.js:129:5)
  at Socket.Readable.push (_stream_readable.js:110:10)
  at Pipe.onread (net.js:552:8)

Commands
     -0:12.7.0 editor:newline (atom-text-editor.editor.is-focused)
     -0:07.8.0 autocomplete-plus:confirm (atom-text-editor.editor.is-focused.autocomplete-active)

Config
{
  "core": {},
  "haskell-ghc-mod": {
    "onChangeCheck": true
  }
}

Installed Packages
# User
autocomplete-haskell, v0.6.1
haskell-ghc-mod, v1.1.1
ide-haskell, v1.3.0
ide-haskell-cabal, v1.0.0
language-haskell, v1.4.12

# Dev
No dev packages

Do you get this consistently, or is it a once-in-a-while issue?

Anyway, 1.1.2 shouldn't throw this error, instead providing somewhat more readable error message. If it pops up, please let me know.

I have the same issue with Atom 1.3.2.
I'm running out of a sandbox, if that has anything to do with it.

@kavigupta, you can't get this same issue, since there are literally no toString calls in that code anymore. Please open new issue (or, if Atom doesn't let you, copy error report by clicking on little clipboard icon and paste it somewhere).
I will close this issue now.


Touch events for use on touch screens

The drawing to split clusters works by default but the button down and dragging functions are not registering. Adding a touch handler in the same way as the mouse handler should make the app responsive. Additionally, handling multi-touch to move multiple pieces at once could be quick to support by iterating through the multi touch events.

I see, I didn't test it on touch screens, I will do some debugging.

I'm currently using MatterJS mouse events to handle dragging each blinks, looks like they are not supporting multitouch events on touch screens, which was discussed here: https://github.com/liabru/matter-js/issues/290
If we implement our own touch events, we will lose all the physics feature that currently applying on each blink while dragging. So this won't be a quick change. I have fixed other issues, will be included in next PR


Cycle 4 funding: uncertainties, masks, quantities, other types of arrays

A perhaps wild idea to allow myself more time for moving astropy core forward. Will need checking with the University of Toronto on what's possible.

If I can give more than 👍 , I would. Your frequent contributions are invaluable!

So happy to see this request @mhvk !    In order for us to have a realistic cost estimate for this work, we need to have that detail figured out at this stage.
Currently, we are funded by a NASA grant and we'll also need to doublecheck if we can subaward to Canada.

@kelle - I think for me paying the university would be easier (no worries about pension contributions, health benefits and things like that), but indeed I do not know whether they would charge anything beyond the fraction of my salary covered. I'll talk to my chair!

Great news!  This work is fundable by the NASA grant since it is technically "software engineering" work.  I will followup with you over email/Slack to discuss details.

Updated the PR, mostly to add explicit links to @nstarman's really nice discussion of Quantity 2.0, which means that my piece can be about implementing Quantity as a container class (rather than figuring out how to do that).

Please react to this comment to vote on this proposal (👍, 👎, or no reaction for +0)

@mhvk Do you have any update on your budget after discussing with your Chair? Is 54,000 still an accurate 1 year budget?

@mhvk Do you have any update on your budget after discussing with your Chair? Is 54,000 still an accurate 1 year budget?

I haven't gotten much further than hearing it is definitely possible, and that likely any pay would be directly to me. The latter is a bit annoying for me (pension, etc.), but it means it is all easier at least in terms of dollar figures. Anyway, bottom line: the amount is still good.

Hi Marten,
I'm writing on behalf of Astropy's Finance Committee regarding the outcome of your recent funding request PR#380. We have opened a new issue #418 to track the progress.
We are pleased to be able to let you know that, following consultation with the community, we are able to approve this request. We can currently fund the Year 1 amount of $54,000.00  (US) to carry out the project. Funding beyond this amount will be contingent upon the availability of funds. (We will be using the full budgets of all of the approved requests to craft future grant and funding proposals.)
Ana Gabela and I will be your contacts on the Finance Committee to facilitate this award. Please get in touch with us if you have any questions or concerns. Please do not reach out to NumFOCUS directly.
In addition, new to this funding cycle, is the assignment of a COTR (Contracting Officer's Technical Representative) to each funded project. This concept is borrowed from government funding agencies, although it is to be stressed that Astropy's goal is to make the COTR role as low-overhead as possible. The COTR’s primary responsibility is to make sure the work is happening at the expected pace and, if necessary, to be a liaison between the funded project and the Finance Committee or CoCo. The COTR for your project will be assigned shortly and we’ll also be sending out more details about how we see this working.
The next steps are:
Independent Contractor:
Please make sure that you are registered with Open Collective (https://opencollective.com/): this is the system you will use to submit your invoices for payment. Note that your invoices should be submitted through the NASA project on Open Collective.
We will reach out through NumFOCUS, Astropy's Fiscal Sponsor, to set up an “independent contractor agreement” (ICA; effectively a contract) and formal scope of work for the project.
Congratulations --- we are really looking forward to seeing you put these funds to good use!
Aarya on behalf of the Astropy Finance Committee


Bug/Feature with uv pip install from github

When using uv pip install from a github repository, a working SHA from any of the repository forks will be picked up as valid.
Example:
uv pip install "python-usbtmc @ git+https://github.com/python-ivi/python-usbtmc@7c15a9d2138160024c4069ecf4b2edc33f15e40e"
Problem/Feature:
The repo https://github.com/python-ivi/python-usbtmc does not contain a commit with the SHA 7c15a9d2138160024c4069ecf4b2edc33f15e40e but the fork https://github.com/FOSSBOSS/python-usbtmc does, i.e. uv pip install will install a packet from another repository than expected.
Tested on uv 0.4.10

I'll take a look at this.

I think this is a consequence of GitHub's storage model for repositories. All GitHub repositories and their forks share the same backing storage, so you can access commits from forks through any sibling repository. For example, you can see that commit here: https://github.com/python-ivi/python-usbtmc/commit/7c15a9d2138160024c4069ecf4b2edc33f15e40e.
I need to look into whether other installers also behave this way, and what we can do to avoid it (if desired).

As I wrote in the heading it might be a feature, but at the same time it might also be a security issue in this (maybe unlikely) scenario:
uv pip install  @ @
Great that you will look into this!

I agree with your assessment, though it may be somewhat unsolvable / inherent to GitHub.
Issue can be closed.


Add ARD overpass notebook + supplementary data

Supplementary data for ARD_overpass_predictor notebook in dea-notebooks/Frequently_used_code/
Proposed changes
Supplementary data for ARD_overpass_predictor notebook in dea-notebooks/Frequently_used_code/
Checklist (replace [ ] with [x] to check off)

[x] Notebook created using the DEA-notebooks template
[x] Remove any unused Python packages from Load packages
[x] Remove any unused/empty code cells
[x] Remove any guidance cells (e.g. General advice)
[x] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[x] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[x] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[x] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[x] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Hi Eric, thanks for your PR. I can't seem to find the ARD overpass notebook in question?

Sorry it’s coming I swear!
I had to edit the path in the notebook to specify this example file and was just testing that in the Sandbox. It all works now so will be submitting the notebook shortly.
Thanks
From: Matthew Alger<EMAIL_ADDRESS>Sent: Monday, 7 December 2020 10:47 AM
To: GeoscienceAustralia/dea-notebooks<EMAIL_ADDRESS>Cc: Hay Eric<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [GeoscienceAustralia/dea-notebooks] Add input dataset for ARD overpass notebook (#736)
Hi Eric, thanks for your PR. I can't seem to find the ARD overpass notebook in question?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/GeoscienceAustralia/dea-notebooks/pull/736#issuecomment-739586939, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANJD6OJ3IYKT4L7LUTKPZTLSTQJYHANCNFSM4UPT24NA.
Geoscience Australia Disclaimer: This e-mail (and files transmitted with it) is intended only for the person or entity to which it is addressed. If you are not the intended recipient, then you have received this e-mail by mistake and any use, dissemination, forwarding, printing or copying of this e-mail and its file attachments is prohibited. The security of emails transmitted cannot be guaranteed; by forwarding or replying to this email, you acknowledge and accept these risks.

Great, looking forward to it :) Just add it to this PR and we can look at the notebook + data at the same time.

Hey I have added the notebook!

Tidied up the fist MD cell with a link to DEA Sandbox, and the DEA image

This is a really useful notebook @EricHay, and it'll be a great addition to Frequently_used_code! A few comments so far that I've posted on ReviewNB, and a few I'll post here. My main request is that you add more documentation. This looks like a useful tool, but dea-notebooks is an entry point for much of the DEA environment, so we want all notebooks to be well-explained and detailed so that even beginners can understand them. Take a look at some of the other Frequently_used_code notebooks - that's the level of documentation I'd like to see! Could you please add more markdown cells that explain what you're doing and why so that people without a strong background in the topic can understand what's going on? There's also a lot of repetition going on between the satellites and locations, which could be eliminated by some loops or functions. That'd make the notebook much more easy to edit and work with, and so it'd be much more useful! Thanks :)

Great thanks Matthew. I sure can fix up the documentation and code. I did this a while back as a learning exercise and it is a pretty convoluted process!

Hey @EricHay, notebook looks awesome! To echo some of what Matthew wrote above, I think the key changes that are needed are:

Extra markdown documentation and explanation from a really basic beginner level, explaining both what each section does, but also why it is needed. A lot of users of dea-notebooks have never dealt with complex code before, so we try and walk them through everything pretty slowly. This is particularly the case for the Frequently_used_code directory which is kind of like a "library" where users can jump in and copy and paste examples of code into their own analyses.

Wherever possible, it would be great if all bits of code that require user input be moved up the very top in an "analysis parameters" section (see example here). We typically find that users somewhat blindly run through most of the notebook body without paying close attention to things that need to be changed later in the code, so having all the configurable bits in one single up-front section means they only need to focus on changing one bit rather than having to be on the look out for making changes all the way through in order to get valid results.

Happy to help out with any of this stuff! It's looking great through, and will be an excellent addition to the repo. 🚀

Thanks @robbibt, sorry i'm a bit flat out today with meetings but will get to polishing this off soon. Agree on both points 👍 and thanks for the feedback!

Take your time, no rush :)

Sorry I forgot to remove some text referring to the old Process. Should be good now!

Hey @EricHay , I just posted a few very minor comments above, it's looking great! Thanks for all your work in updating this, I think it should be much easier to use now.

Hey @EricHay , I just posted a few very minor comments above, it's looking great! Thanks for all your work in updating this, I think it should be much easier to use now.

Thanks @robbibt. Easy enough fixes, I agree on all points. This has been a bit of a backburner project, and I didn't even notice the "Site" column in the final table! That can definitely be dropped. I will do another quick tidy, and comment again once done.

Thanks @robbibt. Easy enough fixes, I agree on all points. This has been a bit of a backburner project, and I didn't even notice the "Site" column in the final table! That can definitely be dropped. I will do another quick tidy, and comment again once done.

Ok, tidied up the unnecessary commented-out code in cells, updated the final table with "Overpass" as the index, and added optional code in the Combine Dataset section to add / alter time zones, with UTC to AEST as an example (+10h). I think this is straight-forward enough for an average person with some Python knowledge to utilise now :)

Ok, tidied up the unnecessary commented-out code in cells, updated the final table with "Overpass" as the index, and added optional code in the Combine Dataset section to add / alter time zones, with UTC to AEST as an example (+10h). I think this is straight-forward enough for an average person with some Python knowledge to utilise now :)


Fix the DbKey#hashCode [ECR-2416]

Overview
DbKey#hashCode turned out to be broken: it passes array references to
Objects.hashCode, instead of passing them to Arrays.hashCode.

See: https://jira.bf.local/browse/ECR-2416
Definition of Done

[x] There are no TODOs left in the code
[x] Change is covered by automated tests
[x] The coding guidelines are followed
[x] Public API has Javadoc
[x] Method preconditions are checked and documented in the Javadoc of the method
[x] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

I think we’d require EqualsVerifier.verify for any class overriding e&h, wdyt?


Disable full push on scale from 1->0->1

Fixes https://github.com/istio/istio/issues/39652
This reverts https://github.com/istio/istio/pull/36882. At the time,
that PR was needed because EDS ServiceAccounts and CDS ServiceAccounts
were decoupled; Since https://github.com/istio/istio/pull/39133, this is
no longer true, and the fix in 36882 is not needed any longer.
This PR removes the test added in 36882 (since it tests low level
details that are not relevant anymore). It improves the existing
TestEndpointFlipFlops test -- while that test would have caught the
regression, it didn't actually set any service accounts so it was
missed. The update changes it to correctly detect the behavior (it now
fails without this PR, passes with it).

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test unit-tests-arm64

lint_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests-arm64_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test unit-tests-arm64

lint_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test lint

unit-tests_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test lint

unit-tests_istio
89b3390dd86b3ec921104aa91d46acd6ac085bde
link
true
/test unit-tests

release-notes_istio
45de4bffd635a136193f735cff1a259f562a3b68
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

cc

before on delete we clean up the SAs, so it ends up empty. Then when the
pod comes back up, new SA is added, which is a change so it triggers a full
push.
Now we avoid changing the SA when it's removed so it doesn't change when
the new one comes back up
On Thu, Sep 8, 2022, 7:30 PM Zhonghu Xu @.***> wrote:

@.**** commented on this pull request.
Could not see how you diable full push.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/40866#pullrequestreview-1101626251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXITIZ4SXCHBG7YTG4TV5KOOFANCNFSM6AAAAAAQHG56VU
.
You are receiving this because you were mentioned.Message ID:
@.***>

it is fairly subtle... but we do have a test for it at least (and now it
works... before it didn't unfortunately)
On Thu, Sep 8, 2022, 7:34 PM John Howard @.***> wrote:

before on delete we clean up the SAs, so it ends up empty. Then when the
pod comes back up, new SA is added, which is a change so it triggers a full
push.
Now we avoid changing the SA when it's removed so it doesn't change when
the new one comes back up
On Thu, Sep 8, 2022, 7:30 PM Zhonghu Xu @.***> wrote:

@.**** commented on this pull request.
Could not see how you diable full push.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/40866#pullrequestreview-1101626251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXITIZ4SXCHBG7YTG4TV5KOOFANCNFSM6AAAAAAQHG56VU
.
You are receiving this because you were mentioned.Message ID:
@.***>

In case all pods of a service is down, and at this time another event triggers a full push, we will result in having incorrect cluster.

Why? the cluster is unchanged during 1 pod, 0 pod, and then back to 1 pod -
it always has the last service account when there is 0 pods to avoid a full
push the remove the SA when it doesn't matter (if there are no endpoints SA
is never used anyways)
On Thu, Sep 8, 2022, 7:38 PM Zhonghu Xu @.***> wrote:

In case all pods of a service is down, and at this time another event
triggers a full push, we will result in having incorrect cluster.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/40866#issuecomment-1241432233, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLNCQQN6CD6FVVYOEDV5KPJFANCNFSM6AAAAAAQHG56VU
.
You are receiving this because you were mentioned.Message ID:
@.***>

But the cluster upstream tls setting can have the sas of deleted pods, we could know whether they will come back

I believe that when we scale back yo we will 'reset' the service accounts.
So typically we come back up with the same account and no update needed. If
it's a different account, however, it should clear the old one out and
trigger a full push.
Let me add a test for that case
On Thu, Sep 8, 2022, 7:43 PM Zhonghu Xu @.***> wrote:

But the cluster upstream tls setting can have the sas of deleted pods, we
could know whether they will come back
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/40866#issuecomment-1241434827, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXMOTEBFCGBS2Z3LR5TV5KP6NANCNFSM6AAAAAAQHG56VU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@hzxuzhonghu I added the new case. So this shows that we full push if the SA changes, but do a partial push otherwise, which is the expected behavior.

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.13":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
A	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
CONFLICT (modify/delete): pilot/pkg/model/endpointshards.go deleted in HEAD and modified in Disable full push on scale from 1->0->1. Version Disable full push on scale from 1->0->1 of pilot/pkg/model/endpointshards.go left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.14":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
M	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
Auto-merging pilot/pkg/model/endpointshards.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41001

In response to a cherrypick label: new issue created for failed cherrypick: #41002

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.13":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
A	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
CONFLICT (modify/delete): pilot/pkg/model/endpointshards.go deleted in HEAD and modified in Disable full push on scale from 1->0->1. Version Disable full push on scale from 1->0->1 of pilot/pkg/model/endpointshards.go left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41003

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.15":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
M	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41004

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.14":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
M	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
Auto-merging pilot/pkg/model/endpointshards.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41005

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.13":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
A	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
CONFLICT (modify/delete): pilot/pkg/model/endpointshards.go deleted in HEAD and modified in Disable full push on scale from 1->0->1. Version Disable full push on scale from 1->0->1 of pilot/pkg/model/endpointshards.go left in tree.
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41006

In response to a cherrypick label: #40866 failed to apply on top of branch "release-1.15":
Applying: Disable full push on scale from 1->0->1
Using index info to reconstruct a base tree...
M	pilot/pkg/model/endpointshards.go
M	pilot/pkg/xds/eds_test.go
Falling back to patching base and 3-way merge...
Auto-merging pilot/pkg/xds/eds_test.go
CONFLICT (content): Merge conflict in pilot/pkg/xds/eds_test.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Disable full push on scale from 1->0->1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

In response to a cherrypick label: new issue created for failed cherrypick: #41007

This should not be cherrypicked.


Unable to use constraint layout in splash screen & Also unable to set size of  in  of xml

<!-- You can insert your own image assets here -->
<item
android:layout_height="x dp"
android:layout_width="y dp"
android:gravity="center"
android:drawable="@drawable/logo_0
/>

<item
android:layout_height="x dp"
android:layout_width="y dp"
android:gravity="bottom|left"
android:drawable="@drawable/logo_1
/>

<item
 android:layout_height="x dp"
android:layout_width="y dp"
android:gravity="bottom|right"
android:drawable="@drawable/logo_2
/>

having difficulty in designing splash screen.
note logo_0,logo_1,logo_2 are imported svgs.
Thus XML
So  in  will not work

The Github issue tracker is for bugs and features requests for the Flutter SDK. Help and support questions like these are better suited for Q&A sites like StackOverflow or on one of the flutter community sites at https://flutter.dev/community .


integration-cli: try to make TestEventsTimestampFormats less flaky

- What I did
Tried to fix flakiness of TestEventsTimestampFormats
- How I did it
Added sleep before container run, so the clock is ticked. Also took nicer code from TestEventsContainerEventsSinceUnixEpoch
- How to verify it
Run TestEventsTimestampFormats billion times
- A picture of a cute animal (not mandatory but encouraged)

Fix #21749

It looks like the main correctness change is moving the sleep from before start := daemonTime(c) to after it. But I don't understand why this makes a difference. What was going wrong before?

@aaronlehmann I dunno for sure, but first event were lost. So, I decided to give some time to it. sleep before start were removed because now we filter events and don't care about events from previous runs.

But if we emit an event at t=t0 and docker events --since=t0 doesn't show this event, doesn't that indicate an acutal bug?

@aaronlehmann yeah, you're right :/ I dunno what to do though.

@LK4D4: What does the failure look like?

@aaronlehmann
14:14:45 FAIL: docker_cli_events_test.go:20: DockerSuite.TestEventsTimestampFormats
14:14:45
14:14:45 docker_cli_events_test.go:41:
14:14:45     c.Assert(events, checker.HasLen, 2, check.Commentf("unexpected events, was expecting only 2 events tag/untag (since=%s, until=%s) out=%s", since, until, out))
14:14:45 ... obtained []string = []string{"2016-04-04T14:14:44.049490810Z image untag sha256:d9551b4026f0e2950ddb557cc640871710bf88476ca938b71499305647231b82 (name=sha256:d9551b4026f0e2950ddb557cc640871710bf88476ca938b71499305647231b82)"}
14:14:45 ... n int = 2
14:14:45 ... unexpected events, was expecting only 2 events tag/untag (since=1.199519662s, until=102.791278ms) out=2016-04-04T14:14:44.049490810Z image untag sha256:d9551b4026f0e2950ddb557cc640871710bf88476ca938b71499305647231b82 (name=sha256:d9551b4026f0e2950ddb557cc640871710bf88476ca938b71499305647231b82)
14:14:45
14:14:45
14:14:46
14:14:46 ----------------------------------------------------------------------

From the since=1.199519662s, until=102.791278ms, it's apparent that this is the duration case. This runs last, so the other two cases must be succeeding.
I suspect the problem is that duration uses time.Now() instead of daemonTime to do the subtraction.
duration := func(t time.Time) string { return time.Now().Sub(t).String() }

This means the value can be smaller than it should be if daemonTime is ahead of time on the runner.

Looking at the daemon code, this makes more sense to me now. When you pass a duration for either start or until, it treats it as a relative time, and it's relative to a time.Now call in the client.
With the old code, it was easy for time to advance between time.Now().Sub(t) and the time.Now().Add(-t) that the client does. This means you wouldn't be looking back far enough to see the first event.
Your change should fix this because it sleeps after setting start.
LGTM
But note there is another problem with this test - the value of until it's passing is bogus. The daemon doesn't treat until in terms of timestamps - it starts a timer and stops returning events after that timer fires. So if you say "until=1ms", and it takes more than 1 ms to receive the events, you won't get all of them.
The test should probably do until = f(time.Now().Add(time.Second)) or something like that, instead of using end, because end is not relevant to the until parameter.

...or we should fix the daemon so until is actually evaluated in terms of timestamps if it's in the past, which I think would be the right behavior.

Hum, LGTM 🐰
I agree with @aaronlehmann, we should fix the behavior in the daemon, but I think this should be in a separate PR.

WindowsTP5 failure is unrelated, see https://github.com/docker/docker/issues/21864

Janky completed, but notification didn't arrive; https://jenkins.dockerproject.org/job/Docker-PRs/26168/console
22:16:14 Failed to remove container (docker-pr26168): Error response from daemon: No such container: docker-pr26168
22:16:14 Archiving artifacts
22:16:15 Notifying endpoint 'HTTP:https://leeroy.dockerproject.org/notification/jenkins'
22:16:15 Finished: SUCCESS

Alright, so that's all green(ish); merging

Filed #21865.


Dockerfile dokku deploy, poetry python dependencies

module 'bleach.sanitizer' has no attribute 'ALLOWED_STYLES'


add long description to create work command

Signed-off-by: ycyaoxdu<EMAIL_ADDRESS>for https://github.com/open-cluster-management-io/clusteradm/issues/162#issuecomment-1073411675

/approve
/lgtm


(PUP-7368) Add ability to declare function argument mismatch handlers

This commit adds an alternative dispatch to a function. It is named
argument_mismatch. The function that it calls must produce a String
that represents a message describing that the matched arguments are in
fact in error. An ArgumentError is then raised when the function returns
that string.
An argument_mismatch dispatcher will be excluded when the default
argument mismatch handler describes viable options. It is also excluded
from the list of dispatchers returned from the Dispatcher.
A function may have zero to many argument_mismatch declarations and
when none of them matches, the default mismatch describer still kicks
in.

CLA signed by all contributors.


feat: Add upgrade capabilities

Is your feature request related to a problem? Please describe.
Since when creating a t3-app you're kinda stuck on the version you created on with no way to upgrade, I want to create an upgrade function.
Describe the solution you'd like to see
The upgrade function should:

get the version of your app from ct3aMetadata in your package version
figure out the latest version
ask you if you want to upgrade to latest or to a specific version (?)
figure out which features you use (suggestion: should we add these into ct3aMetadata ?)
generate a diff between a ct3a of your version and the selected version

Describe alternate solutions
I'm not sure if that's the best way and also don't have an idea of how to emulate the creation of the apps to get the diff between them so I'm open to suggestions from people with more experience on the repo.
Additional information
No response

Create T3 App is an init tool, not a framework or library. Once you create the app, it's yours.
https://create.t3.gg/en/faq#how-do-i-keep-my-app-up-to-date

Actually let's leave this open :)
This approach avoids the main problem of the previous attempts at "upgrades", which is that the user could have done anything to their app, file structure, etc since initializing, can be sidestepped.

When I want to upgrade what I do is I initialize the current version of T3 in an entirely separate directory and commit locally then I initialize the latest version in a separate directory and I copy all of the files and paste them in the previous directory.
That way, I can use git diff to see all of the differences and apply them manually in my own repo.
Maybe we can describe this whole process in the docs.

In the meantime, I got this working:
https://upgrade-t3-app.vercel.app


Changing transparency on watermark?

Hi,
First off thanks for this project, it's really appreciated but how would one go about changing transpareny on watermark? I am interested in creating a "HUD" like feel.
Kind regards

Actually transparency is a mix of pixels of camera and watermark (I will change it in the future to set with parameters) but you can do your own transparency if you go to:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/blob/master/encoder/src/main/res/raw/watermark_fragment.glsl
Replace:
gl_FragColor = cameraPixel + watermarkPixel;

To:
    if (watermarkPixel.rgb == vec3(0.0, 0.0, 0.0)) {
      gl_FragColor = cameraPixel;
    } else {
      float transparency = 1.0;
      gl_FragColor = (cameraPixel * (1.0 - transparency)) + (watermarkPixel * transparency);
    }

Where transparency 1.0 is totally opaque and 0.0 is totally transparent.

I added a method to do it on fly:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/9162ae8ea7427ee0c185a53a444c471971ad7981


Migrate api/fixtures.json to OpenAPI

This PR is a followup of #14203. It migrates all the remaining bit of content in api/fixtures.json to OpenAPI.

@timabbott this PR is ready for review.

Merged as the series ending with 793c3f25e76947c9ec7a966967f5909589e66d2d, after removing changing this to remove the rest of the if statement and squashing the last two commits (as the second-to-last didn't pass lint).  Thanks @shubhamgupta2956!

I think this could be done more cleanly, and I'd like you to work on the next follow-up in that direction.  I think the /rest-error-handling examples should be moved into the components section, so that we can reuse them in individual endpoints where we want to highlight their examples more easily.


GitBug-Java v1.0.0

Is this supporting the usage of a specific act-cache? I couldn't find it. Also, we need to close the issues that are already handled, we have to take a look at that.

Not yet, we still need to take care of that actually.

@Nfsaavedra ready for review and merge :D


Dynamic dependencies in setup.py are not considered when editing dependencies

Here is a reproducible example in a docker container:
docker run --rm -it ubuntu:22.04
$ curl -LsSf https://astral.sh/uv/install.sh | sh
$ apt update
$ apt install -y curl
$ curl -LsSf https://astral.sh/uv/install.sh | sh
$ echo 'eval "$(uv generate-shell-completion bash)"' >> ~/.bashrc
$ source ~/.bashrc
$ uv init myapp
$ cd myapp
$ echo "babel==2.15.0" > pinned.txt
$ echo -e "from setuptools import setup\nsetup(install_requires=[i.strip() for i in open('pinned.txt', 'r').readlines()])" > setup.py
$ sed -i '/^dependencies/d' pyproject.toml
$ echo 'dynamic = ["dependencies"]' >> pyproject.toml
$ uv venv
$ uv pip install -e .
$ echo "babel==2.16.0" > pinned.txt

$ uv pip install -e . -v
DEBUG uv 0.4.0
DEBUG Searching for Python interpreter in system path
DEBUG Found `cpython-3.12.5-linux-x86_64-gnu` at `/myapp/.venv/bin/python3` (virtual environment)
DEBUG Using Python 3.12.5 environment at .venv/bin/python3
DEBUG Acquired lock for `.venv`
DEBUG Requirement satisfied: babel==2.15.0  <--------------------- This should have been 2.16.0
DEBUG Requirement satisfied: file:///myapp
Audited 1 package in 5ms

$ uv pip tree
myapp v0.1.0
└── babel v2.15.0

I expected that the second uv pip install -e . should have noticed that the dependencies for myapp have changed.
But it looks like it is not.
Seems related: https://github.com/astral-sh/uv/issues/2844
Workarounds I have seen:

uv pip install -e . --reinstall
uv clean; uv pip install -e .

You'll need to mark your package as requiring reinstallation if you want it to be reinstalled on every operation. This is documented here: https://docs.astral.sh/uv/concepts/cache/#dynamic-metadata.

Sighs ... not sure how I missed that in the documentation !
Thanks for the pointer, I'll read it and try it out !

@charliermarsh was thinking about this a bit more ... is there any benefit of making a heuristic here that if a package is being installed from a local path + The package has dynamic fields in pyproject.toml -- uv would automatically reinstall it in every operation to be safe ?

Local path - I assume uv already has this information
Dynamic fields - If the project.dynamic field is defined in pyproject.toml then we know if has dynamic fields ?

We have a new API whereby you can add additional files to consider when invalidating the cache. You can also include the current Git commit (i.e., invalidate whenever the SHA changes).
Looks like this:
[tool.uv]
cache-keys = [{ file = "pyproject.toml" }, { file = "requirements.txt" }, { git = true }]

See: https://docs.astral.sh/uv/concepts/cache/#dynamic-metadata.


Considerable lag when pressing j due to a key mapping.

nvim --version: v0.9.0-dev-1100+g1dd2424b12
Operating system/version:  Arch Linux (Rolling)/Sway 1.81
nvim +'checkhealth rnvimr':

rnvimr: health#rnvimr#check

OS ~
- OK Name: Linux

Ranger ~
- OK Version: ranger 1.9.3

Python ~
- OK Version: 3.10.9 (main, Dec 19 2022, 17:35:49) [GCC 12.2.0]

Pynvim ~
- OK Version: 0.4.3

Ueberzug (optional) ~
- WARNING Ueberzug is not found in Python Lib

RPC ~
- OK RPC echo: Neovim send "Give me five!" and receive "Give me five!"

Describe the bug
There is a considerable lag when pressing 'j' to go down the tree.
It is caused by this keymap:
vim.keymap.set('t', 'jk', '<C-c>')
Key 'k' was not pressed.
To Reproduce using nvim -u mini.lua
Example:
cat mini.lua
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "https://github.com/folke/lazy.nvim.git",
    "--branch=stable", -- latest stable release
    lazypath,
  })
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
  {
    'kevinhwang91/rnvimr',
  },
})

vim.keymap.set('t', 'jk', '<C-c>')

Steps to reproduce the behavior:

Open Nvim
Execute :RnvimrToggle
Press 'j'

Expected behavior
Key 'k' was never pressed but only key 'j'. The mapping should not be executed.Therefore, there shouldn't be any lag when pressing j to go down.

    aug RnvimrKeyMap
        au!
        au FileType rnvimr tnoremap <buffer><silent><nowait> j j
    aug END


Replace 'Fluent' with 'Fluentd' in logs.md

Fluentd is the correct name of https://github.com/fluent/fluentd. Fluent is the organization name.

Kindly ping

👍 Thank you!


Adding traffic signs

Is it possible to add traffic signs as Assets (correct me if I'm wrong) and place them on the side of the road? I just need some help so that I know which components in the project I should focus on.

+1

+1

+1

+1


FutureDataDriven Summit Event

Hello There,
how all is well.
I am currently launching Future Data Driven Summit event on Sep-29-2021 and would love to save the date on DataSaturday.com website. I will provide all other information in two weeks. All I am asking now is to post the event name and date on the website in order to save the date.
Call for speakers link: https://sessionize.com/FutureDataDriven
Best Regards,
Jean Joseph

Jean, this PR touches each and every file in the repository. I don't even know how this is possible, however this doesn't look good.
If you want to reserve a date for an event, please drop me an email with the information required and I will create the PR for you

Hello Gianluca,
Thanks again for your support.
please find below event details
Event name: Future Data Driven.
Date: Sep-29-2021
On Mon, May 17, 2021 at 7:32 AM Gianluca Sartori @.***>
wrote:

Jean, this PR touches each and every file in the repository. I don't even
know how this is possible, however this doesn't look good.
If you want to reserve a date for an event, please drop me an email with
the information required and I will create the PR for you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/DataSaturdays/pull/213#issuecomment-842249923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APRZMHS6VVNJ3TTDGFIJY3DTOD5GBANCNFSM446GA4OA
.

Event name: Future Data Driven Summit.
On Wed, May 19, 2021 at 4:44 PM Jean Joseph @.***> wrote:

Hello Gianluca,
Thanks again for your support.
please find below event details
Event name: Future Data Driven.
Date: Sep-29-2021
On Mon, May 17, 2021 at 7:32 AM Gianluca Sartori @.***>
wrote:

Jean, this PR touches each and every file in the repository. I don't even
know how this is possible, however this doesn't look good.
If you want to reserve a date for an event, please drop me an email with
the information required and I will create the PR for you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/DataSaturdays/pull/213#issuecomment-842249923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APRZMHS6VVNJ3TTDGFIJY3DTOD5GBANCNFSM446GA4OA
.

I also have its logo and can be found here:
https://github.com/bigdatadriven/FutureDataDriven/blob/main/FDD_logo.PNG
On Wed, May 19, 2021 at 4:45 PM Jean Joseph @.***> wrote:

Event name: Future Data Driven Summit.
On Wed, May 19, 2021 at 4:44 PM Jean Joseph @.***>
wrote:

Hello Gianluca,
Thanks again for your support.
please find below event details
Event name: Future Data Driven.
Date: Sep-29-2021
On Mon, May 17, 2021 at 7:32 AM Gianluca Sartori <
@.***> wrote:

Jean, this PR touches each and every file in the repository. I don't
even know how this is possible, however this doesn't look good.
If you want to reserve a date for an event, please drop me an email with
the information required and I will create the PR for you
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/DataSaturdays/pull/213#issuecomment-842249923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APRZMHS6VVNJ3TTDGFIJY3DTOD5GBANCNFSM446GA4OA
.


Biomes are not converted properly

The biome IDs are being converted directly. It needs might need some from-to conversion.

Closed by mistake.


Completes assignment 2

Completes assignment 2 for cs224d. Note this is the course from Spring 2016 at Stanford.
Implements named entity recognition and a recurrent neural network for sentence generation (and "understanding")
Adds starter code for assignment 3

Merges Dmitri's CS224d assignment 2


Save cursor position and restore it after recompile binary

Hi
Can you please add ability to restore cursor position after save file .scpt
And when saving on Script Editor your plugin not recompile file in Sublime when refreshing.
thanks

fixed in v2.4.3


unmarshal loaded addresses

which is needed for v0 transactions
https://docs.solana.com/developing/clients/jsonrpc-api#results-43

Thanks


Add example program: dirls.c

An example program demonstrating basic file information listing.
Example input:
dirls "C:\Users\User 1\Documents"
Might output something like:
./
../
budget.xlsx 19180
directory_1/
directory_2/
notes.txt 320
to_do_list.txt 245
Bug found upon further testing.
Input:
dirls c:
Produces the following:
./
../
<next file or directory from working directory INSTEAD of next file or directory from c:\>
<next file or directory from working directory INSTEAD of next file or directory from c:\>
<next file or directory from working directory INSTEAD of next file or directory from c:\>
etc.

Thanks, this is now merged.

@tronkko,
Thank you so much for the merge! I appreciate the opportunity to contribute to this neat project!
Best regards,
Jerad


⚠ Mysql - Hydeos has degraded performance

In 538192b, Mysql - Hydeos (https://phpmyadmin.hydeos.fr) experienced degraded performance:

HTTP code: 200
Response time: 1638 ms

Resolved: Mysql - Hydeos performance has improved in 16ca058 after 41 minutes.