[SPARK-16822][DOC] Support latex in scaladoc.

What changes were proposed in this pull request?
Support using latex in scaladoc by adding MathJax javascript to the js template.
How was this patch tested?
Generated scaladoc.  Preview:

LogisticGradient: before and after

MinMaxScaler: before and after

Can one of the admins verify this patch?

@srowen could you take look?

Jenkins test this please

Jenkins add to whitelist

Nice, does it work? :)

Test build #63077 has started for PR 14438 at commit 2a480ca.

Test build #63077 has finished for PR 14438 at commit 2a480ca.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63077/
Test PASSed.

Merged build finished. Test PASSed.

Nice one, those equations are pretty. Merged to master


Add NeedFinding to Interview resources

http://internetfreedom.secondmuse.com/needfinding/ has good resources for preparing an interview around research goals

YOUR USING THE ICON:exclamation: :exclamation: :exclamation:

Also, this looks really interesting... but cyber-!


Incorrect pages transition animation when setPage is called and the new page isn't next or previous from current page

Environment
Platform: iOS
react-native: 0.69.6
react-native-pager-view: 6.0.2
Description
When I change the page calling the setPage method and the new page value is not next or previous from current page it shows an incorrect transition animation.
For example I have 3 pages. I'm in the page 1. And I will change with setPage to page 3 the animation that I'm expecting it's the transition from page 1 (current) to 3 but it shows from page 2 to 3.
Reproducible Demo
https://user-images.githubusercontent.com/19580424/200635670-bf965360-0dda-4bfb-8c21-f9ea2f1ae7f7.mp4

@krozniata How can I test the fix? I have already upgraded to 6.1.1 but still having same issue...
I'm also using expo 46.0.16 to test the application here (don't know if its relevant)

Are you testing it locally or in expo go ? Could you check, if correct version of pager has been used ?

I'm testing in expo go (SDK 46, Client version <IP_ADDRESS>) from iOS (16.1) Simulator.
I've upgraded the react-native-pager-view package with yarn to 6.1.1

@CarlosSLoureiro Pager View isn't yet updated in Expo SDK. In order to test the fix you need to test it locally

@krozniata Oh thanks for the info. I've used expo eject to test it without expo go and the fix works! Thank you!


Fix the CI

And also fixing the CI

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

Project coverage is 69.68%. Comparing base (163381e) to head (1e3bd94).
Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #529      +/-   ##
==========================================
+ Coverage   69.63%   69.68%   +0.05%
==========================================
  Files          80       80
  Lines        9277     9277
==========================================
+ Hits         6460     6465       +5
+ Misses       2817     2812       -5

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


V0.10.0

This release focuses on updates to the histogram's appearance including:  #16, #17, #18, #25

@brittsikora This is ready for feature testing. v0.10.0 example is here. Simplest way to test is to compare to v0.9.0 example here.


swb: annotate BaseDictionary entry items

BaseDictionary Key or Value can be of GC pointer type, potentially needs
write barrier on each entry item.
Use macro and template type traits to replace underlying entry Key/Value
type with WriteBarrierPtr when Key or Value is pointer type and used with
recycler allocators.
BaseDictionary methods still use original Key/Value types. Only the
underlying EntryType may contain swapped type. Most methods are not
affected. However, when WriteBarrierPtr is used, only immutable
TryGetReference should be used. (We cannot get a mutable reference and
possibly change the pointer underneath WriteBarrierPtr.)
Allow WriteBarrierPtr to take immutable address to support TryGetReference
(used a lot).
Update: Apply template traits to field types to simplify write barrier field
annotations. Now most annotations are Field or FieldNoBarrier (for pointers
known not needing write barrier).

@dotnet-bot test Linux tests please

@dotnet-bot test Ubuntu daily_ubuntu_linux_release please

@leirocks @digitalinfinity @curtisman Please help CR, thanks!

@dotnet-bot test Linux tests please

:shipit:

Thanks @leirocks !


Reintroduce, replace or drop comparator for Hash

Extracted from #2443 so we don't forget about it and have a dedicated place to discuss.
#2443 removed the comparator functionality from Hash. Let's decide whether

we want to change the compiler to make it possible (and fast?) again.
replace it by a functionally equivalent construct (separate Hash type,  different implementation, different, less problematic interface, ...)
or conclude that we don't actually need it ever again.

Thanks for adding this.
I personally never used Hash's comparator, I don't know if somebody else did. There's also the thing that you can easily achieve the same behaviour by using a struct wrapper as keys, providing custom == and hash methods (this is what we originally did in HTTP::Headers):
struct Wrapper
  getter value

  def initialize(@value : String)
  end

  def ==(other : Wrapper)
    value.compare(other.value, case_insensitive: true) == 0
  end

  def hash
    value.downcase.hash
  end
end

a = {} of Wrapper => Int32
a[Wrapper.new("FOO")] = 1
a[Wrapper.new("foo")] = 2

puts a[Wrapper.new("fOo")]

More code is involved, but I don't know how frequently you need this feature. Alternatively, a separate Hash type could be added.

4 years without any supporting comments might qualify as number 3: "conclude that we don't actually need it ever again."


MD17

Added the MD17  datasets.
These are essentially the MNIST of Neural Network force fields just as QM9 is the MNIST of molecular property prediction.

Codecov Report

Merging #2843 (74e7704) into master (4e47b6f) will decrease coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2843      +/-   ##
==========================================
- Coverage   84.54%   84.28%   -0.26%
==========================================
  Files         206      208       +2
  Lines        9401     9425      +24
==========================================
- Hits         7948     7944       -4
- Misses       1453     1481      +28

Impacted Files
Coverage Δ

torch_geometric/data/temporal.py
28.57% <0.00%> (-2.68%)
:arrow_down:

torch_geometric/data/dataloader.py
88.23% <0.00%> (-1.57%)
:arrow_down:

torch_geometric/utils/homophily.py
85.71% <0.00%> (-1.25%)
:arrow_down:

torch_geometric/nn/models/schnet.py
18.90% <0.00%> (-0.41%)
:arrow_down:

torch_geometric/data/data.py
91.45% <0.00%> (-0.37%)
:arrow_down:

torch_geometric/nn/models/dimenet.py
18.92% <0.00%> (-0.29%)
:arrow_down:

torch_geometric/transforms/gdc.py
78.10% <0.00%> (-0.22%)
:arrow_down:

torch_geometric/data/graph_saint.py
97.97% <0.00%> (-0.05%)
:arrow_down:

torch_geometric/nn/models/gnn_explainer.py
94.33% <0.00%> (-0.04%)
:arrow_down:

torch_geometric/nn/conv/rgcn_conv.py
98.51% <0.00%> (-0.03%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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

Super cool :) Thanks a lot!


H2 Client connection brakes when ALPN resolved to H2

Description:
public function main() returns error? {
    http:Client restCountriesEp = check new ("https://www.dataaccess.com/webservicesserver");
    http:Response countries = check restCountriesEp->get("/NumberConversion.wso");
    test:assertEquals(countries.statusCode, 200, msg = "Found unexpected output");
}

Steps to reproduce:
error: HTTP/2 stream 3 reset by the remote peer

Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Debugged the code and the following are the findings.

ALPN negotiation is getting successfully negotiated to h2 and then we create the h2 pipeline and move on.
Once we send the request client receives an RST frame from the server with the error code 1 which means an HTTP2 protocol error. That doesn't give us much information on the error.
HTTP2 Protocol Error - The endpoint detected an unspecific protocol error. This error is for use when a more specific error code is not available.

trace logs
[2023-03-16 12:50:57,182] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] OUTBOUND SETTINGS: ack=false settings={MAX_HEADER_LIST_SIZE=8192}
[2023-03-16 12:50:57,265] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] INBOUND SETTINGS: ack=false settings={=2, MAX_CONCURRENT_STREAMS=100, INITIAL_WINDOW_SIZE=1048576}
[2023-03-16 12:50:57,267] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] OUTBOUND SETTINGS: ack=true
[2023-03-16 12:50:57,269] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] INBOUND WINDOW_UPDATE: streamId=0 windowSizeIncrement=983041
[2023-03-16 12:50:57,292] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] OUTBOUND HEADERS: streamId=3 headers=DefaultHttp2Headers[:path: /webservicesserver/NumberConversion.wso, :method: GET, :scheme: http, :authority: www.dataaccess.com, user-agent: ballerina] streamDependency=0 weight=16 exclusive=false padding=0 endStream=true
[2023-03-16 12:50:57,485] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] INBOUND SETTINGS: ack=true
[2023-03-16 12:50:57,521] TRACE {http.tracelog.upstream} - [id: 0x8ced4fa1, L:/<IP_ADDRESS>:59482 - R:www.dataaccess.com/<IP_ADDRESS>:443] INBOUND RST_STREAM: streamId=3 errorCode=1

I tried the same code with some other HTTP2-supported sites. And those work fine, which means that this is not happening for all the HTTP2 backends.

code
public function main() returns error? {
    http:Client restCountriesEp = check new ("https://www.wappalyzer.com/lookup");
    http:Response countries = check restCountriesEp->get("/wordpress.org/");
    io:println(countries.getTextPayload());
}

Tested this with a nghttp client. These were the outgoing headers,
:method: GET
:path: /webservicesserver/NumberConversion.wso
:scheme: https
:authority: www.dataaccess.com
accept: */*
accept-encoding: gzip, deflate
user-agent: nghttp2/1.52.0

However, ballerina client outgoing headers are follows.
:path: /webservicesserver/NumberConversion.wso
:scheme: http
:authority: www.dataaccess.com
:method: GET
user-agent: ballerina

The difference here is that even though the url is https, ballerina client sends the scheme as http. When the scheme is updated to https in ballerina client, request is successful.


RBAC Access

Documentation out of date?
The only IAM roles available are:

I also do not see the following in built-in roles via the documentation in here https://portal.azure.com/#view/Microsoft_AAD_IAM/AllRolesBlade which is where it should be?:

Built-in role definitionsAzure Cosmos DB exposes two built-in role definitions:

ID
Name
Included actions

00000000-0000-0000-0000-000000000001
Cosmos DB Built-in Data Reader
Microsoft.DocumentDB/databaseAccounts/readMetadataMicrosoft.DocumentDB/databaseAccounts/sqlDatabases/containers/items/readMicrosoft.DocumentDB/databaseAccounts/sqlDatabases/containers/executeQueryMicrosoft.DocumentDB/databaseAccounts/sqlDatabases/containers/readChangeFeed

00000000-0000-0000-0000-000000000002
Cosmos DB Built-in Data Contributor
Microsoft.DocumentDB/databaseAccounts/readMetadataMicrosoft.DocumentDB/databaseAccounts/sqlDatabases/containers/Microsoft.DocumentDB/databaseAccounts/sqlDatabases/containers/items/

I tried adding cosmos db operator and backup in IAM but it still was not enough access to handle containers.

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

ID: aba612b3-f404-11b2-45d9-6e10f1fbf11d
Version Independent ID: 86ccb46e-5ebd-50a4-a911-2ac83c8a41f4
Content: Configure role-based access control for your Azure Cosmos DB account with Azure AD
Content Source: articles/cosmos-db/how-to-setup-rbac.md
Service: cosmos-db
GitHub Login: @seesharprun
Microsoft Alias: sidandrews

@WayneAASPLLC Thanks for your findings,We will look into it and update as appropriate.

I ended up creating a custom REST API and created an internal VNET link to the Cosmos DB but this documentation should still be updated, these options do not exist.

The roles do not exist. They are not returned in the Azure Portal and I don't see them in the results from Get-AzRoleDefinition:

Please either get them published or remove the built-in role definition section from the doc at https://learn.microsoft.com/en-us/azure/cosmos-db/how-to-setup-rbac#built-in-role-definitions.

I was wrong. The roles do exist but they are in a different place than most users are expecting. Commonly when we refer to "role definitions" or "role assignments" in the context of Azure, we mean the Microsoft.Authorization/roleDefinitions and Microsoft.Authorization/roleAssignments resource types, respectively. However in the context of Cosmos DB they are microsoft.documentdb/databaseaccounts/sqlroledefinitions and microsoft.documentdb/databaseaccounts/sqlroleassignments, respectively.
And the built-in roles do show up if you use the right PS cmdlets:

My suggestion on this issue would be to make it more explicit in the docs so the user doesn't confuse Cosmos DB role definitions and role assignments with the standard Azure RBAC resource types.
Docs:

https://learn.microsoft.com/en-us/azure/templates/microsoft.documentdb/2021-10-15/databaseaccounts/sqlroledefinitions?pivots=deployment-language-bicep
https://learn.microsoft.com/en-us/azure/templates/microsoft.documentdb/2021-10-15/databaseaccounts/sqlroleassignments?source=recommendations&pivots=deployment-language-bicep

Thank you @majastrz, we will get the docs updated. We had a product PM confirm the same findings.


Add PATCH method for Fuel

What's in this PR?
This PR mitigates the issue where Fuel don't expose PATCH method for client to use. This PR will remedy that.

Something wrong with tests?

It seems like httpbin.org doesn't support POST method for httpbin.org/patch endpoint.

🎉 🎉 🎉

Wow, let's merge this.


nit on the need for transpose rules

I love the tutorial, thanks for putting together this great resource!
You write:

We're lucky in this case, and we don't need to add a "transpose rule", since JAX can actually work that out by itself (our JVP is linear in the tangents).

Every well behaved JVP is linear in the tangents, by definition -- the tangents are the "vector"  part of "Jacobian-vector product."
What's special about this primitive (and which means that we don't need the transpose rule) is that it is non-linear. That means it that it can't appear in the tangent calculation (because again, output tangents are always a linear function of input tangents, per the chain rule), and only things that appear in the tangent calculations needs to be transposed to calculate cotangents (VJPs).
So the key question is actually whether a primitive is a linear function of one or more of its arguments. If so, then yes, you need a transpose rule to support reverse mode autodiff.

Good point - I'll try to fix the terminology shortly!

Good point - I'll try to fix the terminology shortly!


[arm2pulumi] Site should specify that it's in beta

To avoid customer confusion, https://www.pulumi.com/arm2pulumi/ should have a "Beta" badge on it to let clients know that it is not finished and some errors are expected. Maybe also link to known issues.

My suggestion to address this is that we link to known issues as part of the help text we present when users get a conversion error.  We've improved these conversion errors to be a bit more actionable recently - but adding the GitHub issue list would help guide folks to the right place to both understand where there might be known gaps, and where thy can report any new issues they are seeing.


ddel: AttributeError: type object 'HttpError' has no attribute 'resp'

I've created too many jobs "by accident" (or rather I hoped it would re-use compute engines). When I tried to delete all of them using: ddel --provider google-v2 --project my-project-name --jobs '*'
I am getting the following exception at some point:
Traceback (most recent call last):
  File "/path/to/venv/bin/ddel", line 11, in <module>
    sys.exit(main())
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/commands/ddel.py", line 137, in main
    create_time_min=create_time)
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/commands/ddel.py", line 184, in ddel_tasks
    user_ids, job_ids, task_ids, labels, create_time_min, create_time_max)
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/providers/google_v2.py",line 1069, in delete_jobs
    tasks)
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/providers/google_base.py", line 445, in cancel
    batch_fn, cancel_fn, ops[first_op:first_op + max_batch])
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/providers/google_base.py", line 409, in _cancel_batch
    batch.execute()
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/providers/google_v2.py",line 400, in execute
    self._response_handler(request_id, response, exception)
  File "/path/to/venv/local/lib/python2.7/site-packages/dsub/providers/google_base.py", line 383, in handle_cancel_response
    msg = 'error %s: %s' % (exception.resp.status, exception.resp.reason)
AttributeError: type object 'HttpError' has no attribute 'resp'

It could be that there are so many tasks to delete. HttpError should have the resp set in the constructor, not sure why it hasn't in that case. Maybe it's a different object (although the full classname was googleapiclient.errors.HttpError).
I got around it by putting a try/catch google_base, something like (which is obviously a workaround, not a proper solution, but enough to get everything deleted - which took a while):
      try:
        msg = 'error %s: %s' % (exception.resp.status, exception.resp.reason)
        if exception.resp.status == FAILED_PRECONDITION_CODE:
          detail = json.loads(exception.content)
          status = detail.get('error', {}).get('status')
          if status == FAILED_PRECONDITION_STATUS:
            msg = 'Not running'
      except AttributeError:
        msg = 'error %s' % exception

Thanks for reporting this @de-code!
We can certainly work around this, although I'd like to be able reproduce it or otherwise reason about why you are seeing this. We have not seen this particular error in our end-to-end tests that call ddel and looking at the HttpError constructor, the "resp" member has been there for the last 8 years.
Are you able to reproduce this error consistently? Can you dump out anything further on this object using repr or inspect?

I have since been able to cancel all tasks and I am not sure I feel brave enough to get into the same situation again (I am a bit worried about the cost - unless I can limit the compute engines to a very small number).
I can give you more information of what lead to it:
I created a tasks.tsv with around 5700 entries and fired that off. What I haven't reported is another issue, that it failed and existed (with the --wait) but left the tasks running / pending. I can raise it but don't have more information than that.
All of the tasks are very short lived (probably not the intended use-case). Therefore the overhead of creating a compute engine is much greater. If however it re-used the VM and just run docker exec or run many times the overhead would be much lower. The UI cloud console showed around 4/5 pages of compute engines running and being started. First I deleted them there, which probably wasn't a good move. Then I went to the Pipelines page which only showed 32 pending tasks which I cancelled there as well. But it was still creating new compute engines. That is when I tried to cancel all of the tasks via ddel (eventually successfully). It's possible that it was caused by deleting the VMs or because the tasks are short lived.

Also working with a dsub user that just showed me the same error.  He's using the latest version of sub and has a tasks file with thousands of entries.   The only difference is that they aren't deleting using a '*', but they are using a job id.  The error is very similar:
xxxxxx@bxxxxx:~$ ddel --provider google-v2 --project phs-207015 --jobs 'bg-noise--xxxxx--181106-032254-56'
Delete running jobs:
user:
set(['xxxxxxx'])
job-id:
['bg-noise--xxxxxxxx--181106-032254-56']
Found 2544 tasks to delete.
Traceback (most recent call last):
File "/usr/local/bin/ddel", line 11, in
sys.exit(main())
File "/usr/local/lib/python2.7/dist-packages/dsub/commands/ddel.py", line 137, in main
create_time_min=create_time)
File "/usr/local/lib/python2.7/dist-packages/dsub/commands/ddel.py", line 184, in ddel_tasks
user_ids, job_ids, task_ids, labels, create_time_min, create_time_max)
File "/usr/local/lib/python2.7/dist-packages/dsub/providers/google_v2.py", line 1093, in delete_jobs
tasks)
File "/usr/local/lib/python2.7/dist-packages/dsub/providers/google_base.py", line 445, in cancel
batch_fn, cancel_fn, ops[first_op:first_op + max_batch])
File "/usr/local/lib/python2.7/dist-packages/dsub/providers/google_base.py", line 409, in _cancel_batch
batch.execute()
File "/usr/local/lib/python2.7/dist-packages/dsub/providers/google_v2.py", line 412, in execute
self._response_handler(request_id, response, exception)
File "/usr/local/lib/python2.7/dist-packages/dsub/providers/google_base.py", line 383, in handle_cancel_response
msg = 'error %s: %s' % (exception.resp.status, exception.resp.reason)
AttributeError: type object 'HttpError' has no attribute 'resp'

Thanks for reporting this Joe.
Would you be able to send me the output for "pip list" from their machine? I'm interested to see if there is a package versioning issue we should understand better.
In the meantime, you can drop in the code change suggested above to handle the original exception more cleanly so that the user can delete their tasks.

Package                  Version
------------------------ ------------------
asn1crypto               0.24.0
avro                     1.8.2
awscli                   1.15.62
boto                     2.49.0
botocore                 1.10.61
CacheControl             0.11.7
cachetools               2.1.0
certifi                  2018.1.18
chardet                  3.0.4
colorama                 0.3.9
crcmod                   1.7
cryptography             2.1.4
cwltool                  1.0.20180302231433
docutils                 0.14
dsub                     0.2.2
enum34                   1.1.6
future                   0.16.0
futures                  3.2.0
gax-google-logging-v2    0.8.3
gax-google-pubsub-v1     0.8.3
gcloud                   0.18.3
google-api-python-client 1.7.4
google-auth              1.5.0
google-auth-httplib2     0.0.3
google-compute-engine    2.8.3
google-gax               0.12.5
googleapis-common-protos 1.5.3
grpc-google-logging-v2   0.8.1
grpc-google-pubsub-v1    0.8.1
grpcio                   1.15.0
html5lib                 0.999999999
httplib2                 0.11.3
idna                     2.6
ipaddress                1.0.17
isodate                  0.6.0
jmespath                 0.9.3
keyring                  10.6.0
keyrings.alt             3.0
lockfile                 0.12.2
mistune                  0.8.3
oauth2client             4.1.2
parameterized            0.6.1
pip                      10.0.1
ply                      3.8
protobuf                 3.6.1
pyasn1                   0.4.3
pyasn1-modules           0.2.2
pycrypto                 2.6.1
pygobject                3.26.1
pyOpenSSL                17.5.0
pyparsing                2.2.0
python-dateutil          2.7.3
pytz                     2018.5
pyxdg                    0.25
PyYAML                   3.12
rdflib                   4.2.1
rdflib-jsonld            0.4.0
requests                 2.18.4
retrying                 1.3.3
rsa                      3.4.2
ruamel.yaml              0.15.34
s3transfer               0.1.13
schema-salad             2.6.20171201034858
SecretStorage            2.3.1
setuptools               39.0.1
shellescape              3.4.1
six                      1.11.0
SPARQLWrapper            1.7.6
tabulate                 0.8.2
typing                   3.6.2
uritemplate              3.0.0
urllib3                  1.22
webencodings             0.5
wheel                    0.30.0
You are using pip version 10.0.1, however version 18.1 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.

Thanks Joe.
One of our engineers was able to reproduce the problem.
It looks like it has always been there for the google_v2 provider and is a 1-line (1-character) change:
Line 415 change:
exception = sys.exc_info()[0]

to:
exception = sys.exc_info()[1]

We'll have the fix included in the next release.

Its nice when its simple.   Thanks!

From: Matt Bookman<EMAIL_ADDRESS>Sent: Thursday, November 08, 2018 9:48 PM
To: DataBiosphere/dsub
Cc: Slagel, Joseph L; Comment
Subject: Re: [DataBiosphere/dsub] ddel: AttributeError: type object 'HttpError' has no attribute 'resp' (#133)
Thanks Joe.
One of our engineers was able to reproduce the problem.
It looks like it has always been there for the google_v2 provider and is a 1-line (1-character) change:
Line 415 change:
exception = sys.exc_info()[0]
to:
exception = sys.exc_info()[1]
We'll have the fix included in the next release.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_DataBiosphere_dsub_issues_133-23issuecomment-2D437257540&d=DwMFaQ&c=KoC5GYBOIefzxGAm2j6cjFf-Gz7ANghQIP9aFG9DuBs&r=wItZQv9607iYGyuIVq_Pk3Qr-Yy3oS1MW_vcxPgKYrE&m=EsPN6UMIxAWfaPzX_VTO2huuEtnJFCIkW6G9p1eY3VA&s=BGosgJCDfTHtzPt7iEk9EaQNB_5nT409XiMn4eUR-_Q&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AKBQyN4KR-2D-5FL7UKfCJ2lBqqi3m0ZqVd9ks5utRdIgaJpZM4Xz-2Ddh&d=DwMFaQ&c=KoC5GYBOIefzxGAm2j6cjFf-Gz7ANghQIP9aFG9DuBs&r=wItZQv9607iYGyuIVq_Pk3Qr-Yy3oS1MW_vcxPgKYrE&m=EsPN6UMIxAWfaPzX_VTO2huuEtnJFCIkW6G9p1eY3VA&s=E3pG21k-ohCfn_Fo6WltPi77GfQbLzUwJ0UmKDn5tYw&e=.

This message is intended for the sole use of the addressee, and may contain information that is privileged, confidential and exempt from disclosure under applicable law. If you are not the addressee you are hereby notified that you may not use, copy, disclose, or distribute to anyone the message or any information contained in the message. If you have received this message in error, please immediately advise the sender by reply email and delete this message.

This was fixed in release 0.2.3 (#136)


Docs: search on developer machines

Is your feature request related to a problem? Please describe.
Without Algolia keys a contributor cannot use the search feature on their local machine, limiting their ability to contribute in this area.
Describe the solution you'd like
The documentation should include instructions on how to get keys or set up a 'mock' search: https://github.com/freeCodeCamp/freeCodeCamp/issues/37076#issuecomment-539172200.

For me, I think very few people would work on the search itself. So, we would be okay to getting asked for developer keys on a need basis.
And as for everyone else, will it be possible to render a message like "search is not available during dev" in the dropdown?
/cc @scissorsneedfoodtoo

I wouldn't mind having a public API key and app ID available in case someone wants to work on the search components or tests.
We have a dev app available on Algolia that I can set up with the News index. I believe there's a limit on 50,000 operations a month for dev apps, which should be enough for anyone who wants to work on the search. If we hit the limit Algolia should just return an error for the rest of the month.


Added handling for comments between keywords/tests in rule 1004

Fixes #513

Codecov Report

Merging #599 (5d08a34) into master (04d8864) will decrease coverage by 0.98%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #599      +/-   ##
==========================================
- Coverage   97.34%   96.36%   -0.99%
==========================================
  Files          23       23
  Lines        2639     2640       +1
==========================================
- Hits         2569     2544      -25
- Misses         70       96      +26

Impacted Files
Coverage Δ

robocop/checkers/spacing.py
98.76% <100.00%> (-0.31%)
:arrow_down:

robocop/checkers/comments.py
95.45% <0.00%> (-4.55%)
:arrow_down:

robocop/utils/misc.py
94.54% <0.00%> (-4.10%)
:arrow_down:

robocop/checkers/documentation.py
96.42% <0.00%> (-3.58%)
:arrow_down:

robocop/checkers/__init__.py
96.38% <0.00%> (-2.41%)
:arrow_down:

robocop/checkers/errors.py
94.18% <0.00%> (-2.33%)
:arrow_down:

robocop/checkers/misc.py
96.96% <0.00%> (-2.17%)
:arrow_down:

robocop/checkers/lengths.py
98.12% <0.00%> (-0.63%)
:arrow_down:

Continue to review full report at Codecov.

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


Prepend BUILDKITE_BINPATH to PATH iff not already in the PATH

It seems like the buildkite agent is prepending BUILDKITE_BINPATH to PATH:
https://github.com/buildkite/agent/blob/master/bootstrap/bootstrap.go#L402-L406
This is causing problems when we want to change, for example, the default python to be /opt/conda/bin/python instead of /usr/bin/python, when the buildkite agent is installed in /usr/bin (e.g. buildkite is installed as apt package).
Even if we set PATH=/opt/conda/bin:/usr/bin to make sure that /opt/conda/bin/python is tried first, the agent changes to PATH=/usr/bin:/opt/conda/bin:/usr/bin, so /usr/bin/python is used.
It would be nice if the agent only prepend the PATH if it is not already there, or just append the PATH at the end.

Sorry you've run into problems with this @w4-sjcho, we do that so that calls to buildkite-agent will always refer to the same binary as buildkite-agent start was invoked with, otherwise folks run into subtle bugs where they have different versions on their system.
One fix for you might be moving the buildkite-agent binary to its own directory? E.g /usr/local/buildkite-agent/bin/buildkite-agent or similar?

Yes that works, and is the workaround I'm using right now.

This should be fixed with https://github.com/buildkite/agent/pull/1465 and will be included in the next release!


KAFKA-17720: Remove zookeeper_migration_test.py and migration-related functions in kafka.py

Remove zookeeper_migration_test.py and functions in kafka.py related to migrations
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mingyen066 fyi, there was a build issue on trunk. I have fixed the issue and updated your branch with the fix.


error when scanning a server that is offline

as stated, an error encountered when scanning a server that is offline or not connected to the internet.

hi @jamhulk
Thank you for raising the issue, please can you provide some further information?
version of ansible
version on python
which control it stops at?
In short there are multiple things that will fail is the system doesn't meet all the pre-requisites and it is not connected to the internet. The baseline itself does extensive testing so it is setup to work with each different setup (especially on brownfield servers or those not connected to the internet).
Ansible provides each control with a title to help explain what it is trying to achieve at each step and if you have set a control to run that has prereqs it will try to add them on your behalf. In your case you will have to ensure you have the control you want and the prereqs are met in each case.
I hope that makes sense
many t hanks
uk-bolly

hi @jamhulk
I have looked at this a little further already, as this is mentioned only when running a scan ( i assume it should be for the audit component only?). If this is the case we should move the discussion to the audit repository.
CIS benchmarks require several tests that would be needed in order to pass or fail, with each control having the ability to be enabled or disabled the following should be turned off.
Without having knowledge on your configurations, I am assuming the following.

You have added the goss binary and the audit content manually to the system or used the copy options to do so within remediation?
You don't have a satellite server or any update server to test for any updates

Your main culprit for rhel9 in the audit will be
rhel9cis_rule_1_9
If other OS remediation benchmarks are also being used they will do other checks that may need to be disabled.
This asks the system to ensure there are no outstanding updates available by running dnf to test, if this is not connected this will fail i'm afraid.
Please feel free to drop by discord server to discuss further with us or any other members of the community on how they are approaching any issue that you maybe experiencing.
If i have mis-assumed, if you can please let us have further information about your setup, variables being used from default and maybe an output of the actual error.
many thanks
uk-bolly

hi @jamhulk
I hope this is now working for you, i have noticed this is also raised on another repo, which has further feedback regarding prerequisites that are required. will need to be installed if the system is disconnected.
If you are still seeing problems, please let us know that all the required packages are added to the system and some log extraction of the failures.
many thanks
uk-bolly

hi @jamhulk
I will close this request as stale, please feel free to open a new issue if you are able provide further information.
Many thanks
uk-bolly


Added 2 methods for choosing the correct country if they have the same phone extension.

This update provides two ways to ensure that the correct country is selected if there are multiple countries with the same phone extension.

The first way is pretty straightforward. I added a static variable in NKVPhonePickerTextField called samePhoneExtensionCountryPriorities that can be used to set the priority of countries if they have the same area code. The variable is properly commented with an example and is added to the example view controller. Please see the comment for usage instructions.

The second added feature is the automatic detection of the correct country based on a local area code. This feature requires no extra effort from the user of the library so no need for the user to change anything. The feature currently works only for US and Canada. Local area codes for both US and Canada are stored in Sources/Bundles/Countries.bundle/Data/localAreaCodes.json file and you can modify that file to add this features for any other countries that share the same phone extension.

Wow! Great addition 👏


Improvements for unit testing

Some of these tests do still fail for me, however at least they don't seem to be crashing.
Mostly I wanted to restore CsoundObj.m to use a little ifdef's as possible. I noticed you were not using setup/teardown methods in the tests, and it made more sense to me to do the kind of thing you were doing within those, so I exposed the necessary code through CsoundObj and AKManager.
I hope I didn't break anything as I'm not too sure of the purpose of some of your new code, but I think this should be an improvement.
As a side effect, this also simplifies a little bit of the boiler plate for individual tests.

Hum, lots of odd failures happening in Travis that didn't happen on my system...

@megastep Try triggering a rebuild on Travis; there have been times where I've seen it fail on Travis but passed on my machine, and a rebuild showed nothing was wrong.

From looking at the Travis log I see these are not the trivial Travis hanging errors, but instead the files not really being created correctly for the MD5 hash to run on them.  None of the tests are broken on my own system or on Travis running the current branch so this is fixable

Actually, I was wondering if the renderToFile:duration: method is supposed to be blocking? I think it might have something to do with this. Your performSelector call would wait for completion, whereas my NSThread code doesn't as is.
This could explain how this is getting tested before the file gets a chance to get created properly.

These last changes seem to help for me, I no longer get crashes in tests on either iOS or OS X, although some of those tests do fail for me (but that seems to be another issue).
I also fixed the way the OS X test project was setup, apparently there's no need to duplicate the linking configuration for test target.
I really wonder if we should change the record:.. APIs to either say that they are blocking, and maybe give an option for them to spawn their own threads? This really depends on what the intended usage for these is, you probably don't want to run Csound on the main thread if you have a UI.

Woohoo, Travis passed!

Fantastic!


🔧 MAINTAIN:  Add .nox to gitignore

So I can use nox locally 😄
FYI I also use it with --force-venv-backend mamba  (https://mamba.readthedocs.io, https://nox.thea.codes/en/stable/usage.html#forcing-the-sessions-backend)

Updated to make tests pass

Thanks @danwos, although I would note that I usually use "Squash and Merge", to keep the commit history nice and tidy 😅


Focus state on ActionBar looks kinda buggy

Description of the bug
This doesn't actually break any accessibility rules, but it looks kinda weird. It should at least show all the sides of the border and probably also encompass the arrow-down icon to the left. Whether it should outline the entire bar (including the actions) is a question that should probably be discussed with UX.
We should also look into unifying the look of all our focus states so that they look and act similarly to that of the Design system which has recently changed theirs into a black and yellow outline.

Steps To Reproduce

go to Overvew page
use tab to focus on an ActionBar

Additional Information
No response

Sjekket, syns det ser bra ut nå som også nedover pilen er med i fokuset.   Er dette knapp og fokus fra design systemet @allinox og @Albertlarsen  ?

@Randiabs Det er basert på fokus fra design systemet ja


Resolving issue when using setInputParameters

When I call
op.setInputParameter("y",yi, "column");
where yi is of type double[], I get the following error message:

The type cern.colt.matrix.tdouble.DoubleMatrix2D cannot be resolved. It is indirectly referenced from required .class files

I've added  the colt library 1.2.0 to my maven pom but this doesn't help.
https://search.maven.org/#artifactdetails|colt|colt|1.2.0|jar
Any suggestion?

I should read your installation manual better...


add small contributing guide

Someone in twitch chat asked how to contribute, here's how to do it.

Thank you for the PR!


Issues with getting a base64

With any combination of base64 encoding i run into issues. this may be due to linux having slightly different encoding tools than mac? Once i figure this out. Ill make a PR to include a note in the docs

The last character here always looks odd when using the included command on linux, witch produces a encoding sveral thousand letters long

Alternativly, this syntax
echo <file.json>| base64 -w 0

produces a much shorter encoding still with the strange character

hexdump says last two chars are this

not sure whats helpful here and whats not, thanks for reading, will update you

Sorry to add more to your plate, i may not be able to help with that but i suspect that the linux base64 encoding is just maybe a tad different than the BSD one/mac? What would an appropriate encoding look like i can spend some time testing to see if i can get it working

Ran a diff after piping back and forth through base64 with the same string in readme, looks like its not capturing the newline

bleh

Also having this issue with linux. I just had to use an online converter to test and that worked
I think supporting raw keys directly would be a good idea. I've seen other actions do this. I'd try to change that myself but I've never touched go

Yeah I think adding an option for a 'non-encoded' secret might be the only fix.
Working on it.


Sentry + Swarrot Processor

Hi to everyone. I use very nice bundle https://github.com/swarrot/SwarrotBundle for Symfony.
Because current version of this bundle does not supports newest version of sentry-symfony, I use own sentry processor.
Main problem: all errors sent when consumer is died. Realtime error tracking doesn't works.
Current configuration:
sentry:
    dsn: '%env(SENTRY_DSN)%'
    options:
        release: '%env(RELEASE_NAME)%'
        environment: '%env(SENTRY_ENV)%'
        enable_compression: false

Installation was via that documentation https://docs.sentry.io/platforms/php/symfony/#installation
<?php

declare(strict_types=1);

namespace App\Infrastructure\Swarrot\Processor;

use Sentry\State\HubInterface;
use Sentry\State\Scope;
use Swarrot\Broker\Message;
use Swarrot\Processor\ProcessorInterface;

class SentryProcessor implements ProcessorInterface
{
    /**
     * @var ProcessorInterface
     */
    private $processor;

    /**
     * @var HubInterface
     */
    private $hub;

    /**
     * SentryProcessor constructor.
     *
     * @param ProcessorInterface $processor
     * @param HubInterface       $hub
     */
    public function __construct(ProcessorInterface $processor, HubInterface $hub)
    {
        $this->processor = $processor;
        $this->hub = $hub;
    }

    /**
     * @param Message $message
     * @param array   $options
     *
     * @return bool|null
     *
     * @throws \Throwable
     */
    public function process(Message $message, array $options): ?bool
    {
        try {
            return $this->processor->process($message, $options);
        } catch (\Throwable $e) {
            $this->handleException($e, $message, $options);
        }
    }

    /**
     * @param \Throwable $e
     * @param Message    $message
     * @param array      $options
     *
     * @throws \Throwable
     */
    private function handleException(\Throwable $e, Message $message, array $options): void
    {
        $properties = $message->getProperties();

        $messageBody = $message->getBody();
        $queue = $options['queue'];
        $routingKey = $properties['routing_key'];

        $this->hub->withScope(function (Scope $scope) use ($messageBody, $queue, $routingKey, $e): void {
            $scope->setExtra('message', $messageBody);
            $scope->setExtra('queue', $queue);
            $scope->setExtra('routing_key', $routingKey);

            $this->hub->captureException($e);
        });

        throw $e;
    }
}

Also consumer use a lot of memory when I use SentryProcessor.

This issue is related/duplicated of https://github.com/getsentry/sentry-php/issues/811. This is not an issue with the bundle but with the base SDK.
That issue seems to be fixed now, and will be released as 2.2. You can try to use sentry/sentry:dev-master to see if that solves your issue.


feat: salud to record peer latency

Checklist

[x] I have read the coding guide.
[ ] My change requires a documentation update, and I have done it.
[ ] I have added tests to cover my changes.
[x] I have filled out the description and linked the related issues.

Description
Since salud acquires the status msg from every peer (reachable or not), we can use this mechanism to record peer latency instead of kademlia's ping.
Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

Is the 'Check' from time to time？ Because my VPN or VPS will change my server IP occasionally. Sometimes a bad server can cause my network to disconnect, and then a well-performed server can re-establish my network.
Besides， I find that when every time I start my bee full node， it can connect a lot of peers. however, the number of my peers will decrease over time and finally decays to just a few dozen nodes. I am wondering whether it is caused by the potential 'Checking' issues.

@0xCandy the check does not disconnected from peers.
a few dozen nodes signals that your node is having serious network issues. on average, the expected peer count is around 150.

@0xCandy the check does not disconnect from peers. a few dozen peers signals that your node is having serious network issues. on average, the expected peer count is around 150.

I mean, if my network is bad and peers disconnected, then periodic re-check may help recover my peers.


Does XSQL have plan to support Delta lake or Spark Structed Streaming?

Does XSQL have plan to support Delta lake or Spark Structed Streaming?

We have noticed the development of Delta lake. If we really want to support it which needs a plan.
XSQL has supported Spark Structed Streaming for Kafka.


Fix version badge

The absence of style=flat parameter in the svg link made the version badge invisible in the README (perhaps it leads to a CAPTCHA problem). This pull request fixes that.

Coverage remained the same at 75.915% when pulling 84bec619b6c3117a3d4906844a6907a780b96f22 on schedutron:patch-1 into 4e39dcfdddbc085269a8d3c63b5869e47c35615f on cmu-db:master.


Remove ANTIALIAS from list of filters (deprecated in PIL) (Fix #119)

Remove ANTIALIAS from list of filters (deprecated in PIL) (Fix #119)

Thanks!


OpenAL-<IP_ADDRESS> does not compile

Citing from http://hydra.cryp.to/build/659710/nixlog/1/raw:
Preprocessing library OpenAL-<IP_ADDRESS>...

src/Sound/OpenAL.hs:48:8:
    Could not find module ‘Graphics.Rendering.OpenGL.GL.ObjectName’
    Perhaps you meant
      Graphics.Rendering.OpenGL.GL.Selection (from OpenGL-<IP_ADDRESS>)
      Graphics.Rendering.OpenGL.GL.Bitmaps (from OpenGL-<IP_ADDRESS>)
      Graphics.Rendering.OpenGL.GL.Feedback (from OpenGL-<IP_ADDRESS>)
    Use -v to see a list of the files searched for.

src/Sound/OpenAL.hs:49:8:
    Could not find module ‘Graphics.Rendering.OpenGL.GL.StateVar’
    Perhaps you meant
      Graphics.Rendering.OpenGL.GL.Shaders (from OpenGL-<IP_ADDRESS>)
      Graphics.Rendering.OpenGL.GL.Bitmaps (from OpenGL-<IP_ADDRESS>)
      Graphics.Rendering.OpenGL.GL.Hints (from OpenGL-<IP_ADDRESS>)
    Use -v to see a list of the files searched for.

Ooops, the OpenAL package was missing an upper bound on the OpenGL dependency. I added it, please do a cabal update, retry and tell me if that worked.

Well, the upper bound is not going to help us (the NixOS distribution) because we've already updated to OpenGL-<IP_ADDRESS>.
Anyway, it's good that the upper bound is now there for those installing with cabal-install.

OK... I just uploaded new versions of the OpenAL and ALUT packages to Hackage. It would be nice to know if that fixes things for you.

Yes, the new version fixes the build error: http://hydra.cryp.to/eval/294660#tabs-now-succeed. Thank you for the quick update!


Problems using new OpenSeadragon.MouseTracker

Hello, can I prevent the event bubbling and default events of the bound element when using new OpenSeadragon.MouseTracker? For example, the bound element is an overlay, and I can avoid the event when I click on this overlay Bubbling delivers this event to the view

@zmomozq
This pen has examples of most of the ways you can fine tune event propagation


Add Jenkins GitHub Issue filer plugin for master build failures

Jenkins plugin: https://plugins.jenkins.io/github-issues
Have this plugin file an issue on build failures for the master branch so we can triage all build failures through GitHub.

Superseded by #3482.


Controls V-72063, V-72069, V-72071, V-72073 - aide.conf controls

These controls need to be implemented once the aide resource is developed.

Implemented in commit f29d7d89cd34f964341de28a4a91f3dfb3b654ec


Rank data nodes by topological order in the data node selector

A tradeoff has been proposed.
The idea is to display in the data node selector the data nodes in a topological order for a scenario. With this proposal, the scenario is the context used to set the data node rank.
A more formal proposal should come soon.

Originally posted by @jrobinAV in #1363

Branch in progress: https://github.com/Avaiga/taipy/tree/feature/%232071-rank-datanodes-in-selector
Corresponding PR: https://github.com/Avaiga/taipy/pull/2072
@FredLL-Avaiga Does it make sense for you to take over this issue and the related branch?

The issue why not but the branch not really.

I understand we want to rank the data node for a given scenario, which is not necessarily the owner of the data node. The given scenario could be:

the data node's owner, as you said.
the scenario passed as a property of the data node selector
the "current" scenario group, when the property display_cycles is False.

So the code could look like:
given_scenario
datanode_rank = datanode._ranks(given_scenario.config_id], None)

What you propose is to have an API on the data node's side like the following so you don't need to have a given scenario. Right?:
datanode.get_rank_for_owner()

I believe your proposal only covers the first case. Am I missing something?

I was not aware there's another case ...
datanode selector builds a tree based on datanode.owner_id
cf. https://github.com/Avaiga/taipy/blob/55bdd46584d3141d75aa242bbcfb516e5d4201f3/taipy/gui_core/_context.py#L698


Unable to open interface after enabling SSL.

Reporting Issues: Unable to get into server after turning on SSL. How to disable or correct?

[x] Version c0ed30551f183437753773e317cacf7c00750f6c

[x] Branch master2

[x] Commit hash

[x] Operating system - Debian 8

[x] Python version - 2.7.9

[x] What you did? Enabled SSL with a provided SSL cert. Entered it as a full path.

[x] What happened? I cannot get back into the interface. When going to main port, non-ssl, it reports SSL is enabled and to use SSL. When I try SSL I get a connection refused.

[x] What you expected? Find a way to either remove the setting or get it working.

[x] How can we reproduce your issue? Not sure

[x] What are your (relevant) settings?  I specified the path to the CRT and KEY file. Folder was reset to 777 just in case of permission issues or what not.

[ ] Include a link to your FULL (not just a few lines!) log file that has the error. Please use Gist or Pastebin.

Close your issue when it's solved! If you found the solution yourself please comment so that others benefit from it.
Link to log: https://1drv.ms/u/s!AtItXRNac1nKhe47NcmKoJmg9dEYgQ

Please use pastebin and the full log. Why is thw interface bond to port 80 try selecting another port. Jill htpc manager manually and start it with --port 8085

Okay. So I removed the SSL cert the app makes and restarted on port 7075. It worked. Killed the process and started again, this time using the normal startup which is set for port 80. It no longer responds.
Port 80: https://pastebin.com/1LfUNv1j
Port 7075: https://pastebin.com/RF7hWCsj


SSL Connection error

Traceback (most recent call last):
File "C:\Users\mboyle\Desktop\tmp.py", line 5, in
user = sandbox_client.get_current_user()
File "C:\Python27\lib\site-packages\coinbase-2.0.6-py2.7.egg\coinbase\wallet\client.py", line 213, in get_current_user
response = self._get('v2', 'user', data=params)
File "C:\Python27\lib\site-packages\coinbase-2.0.6-py2.7.egg\coinbase\wallet\client.py", line 129, in _get
return self._request('get', _args, *_kwargs)
File "C:\Python27\lib\site-packages\coinbase-2.0.6-py2.7.egg\coinbase\wallet\client.py", line 115, in _request
response = getattr(self.session, method)(uri, *_kwargs)
File "C:\Python27\lib\site-packages\requests-2.10.0-py2.7.egg\requests\sessions.py", line 487, in get
return self.request('GET', url, *_kwargs)
File "C:\Python27\lib\site-packages\requests-2.10.0-py2.7.egg\requests\sessions.py", line 475, in request
resp = self.send(prep, *_send_kwargs)
File "C:\Python27\lib\site-packages\requests-2.10.0-py2.7.egg\requests\sessions.py", line 585, in send
r = adapter.send(request, *_kwargs)
File "C:\Python27\lib\site-packages\requests-2.10.0-py2.7.egg\requests\adapters.py", line 477, in send
raise SSLError(e, request=request)
SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)

I have the same error when trying to use the client.
Using python 2.7.11 on Mac 10.11.5
I saw a similar issue here. Perhaps the problem is with homebrew??

Are you able to establish a verified SSL connection to a different host using the Python requests library? Say, https://www.coinbase.com/ or https://www.google.com/ ?

@aianus Yes, using the python requests library I was able to get an HTTP 200 response to my request. Python terminal pasted below:
import requests
r = requests.get('https://api.github.com/user', auth=('anthonywittemann', 'my_password'))
r.status_code
200

I'm going to close since this is an old issue, but if you are still able to reproduce please open a new one and we'll look into it! Thanks.


Documentation for SMART App Launch

Task to track the documentation of SMART App Launch functionality
"Well known" urls:

https://api.medplum.com/.well-known/jwks.json
https://api.medplum.com/.well-known/openid-configuration
https://api.medplum.com/fhir/R4/.well-known/smart-configuration
https://api.medplum.com/fhir/R4/.well-known/smart-styles.json

SMART App Launch:

The launch UI that implements the actual flow is a work in progress.
For example to test, manually constructed the URL as follows: https://inferno.healthit.gov/suites/custom/smart/launch?iss=https%3A%2F%2Fapi.medplum.com%2Ffhir%2FR4&launch=

The base URL is the destination app you're calling
iss is the FHIR server base URL
launch is the id of a SmartAppLaunch resource

SMART Backend Services:
Private key JWT -

This has passed the Inferno test, but needs to be documented
Here are the docs we used:

https://www.rfc-editor.org/rfc/rfc7523
https://www.hl7.org/fhir/smart-app-launch/example-backend-services.html#step-2-discovery
https://docs.oracle.com/en/cloud/get-started/subscriptions-cloud/csimg/obtaining-access-token-using-self-signed-client-assertion.html
https://darutk.medium.com/oauth-2-0-client-authentication-4b5f929305d4

The main 3 steps:

Generate and host the JWKS
Set the "jwksUri" parameter on your ClientApplication resource
Then, request the token with  client_assertion_type and client_assertion

Client Secret
Client Secret Documentation
Scopes for Limiting Access
This has passed Inferno test.  Needs documentation.
Token Introspection
This is under development,  similar to the /auth/me endpoint

This is done https://www.medplum.com/docs/integration/smart-app-launch


Warn when symlinks are present in a continous build

We don't support symlinks with continuous build, meaning the build will not re-trigger if an input referenced via a symlinked directory changes. This is mentioned in the documentation briefly (see https://docs.gradle.org/current/userguide/command_line_interface.html#sec:changes_to_symbolic_links), but it's unlikely that all users of continuous build would be aware of this, or that they would necessarily understand if their build contains symlinks.
We should print a warning at the end of the build in a continuous build session if we detected a symlinked input during the build, like so:
BUILD SUCCESSFUL in 417ms
2 actionable tasks: 1 executed, 1 up-to-date

Waiting for changes to input files of tasks... (ctrl-d to exit)
Warning: inputs are referenced via symlinks; some changes might not be detected.
<-------------> 0% WAITING
> IDLE

cc: @gradle/bt-execution

We already traverse the VFS at the end of the build to see if there were any symlinked files stored in it during the build (and then we discard them to avoid retaining stale data). We could use the result of this process to decide if we need to print the warning.


Grid not work as aspected if nested inside -Component

Easy Grid #57

See commit https://github.com/GeekyAnts/native-base-docs/commit/9c4f4f10b2b08fd077afc7d8a32e86c052ac16c2

Check latest docs


created base for phase 2 games storybook

Next steps are to implement non-click on the correct answer and work with integrating the timer header on phase 2

Altered GamesInProgress.jsx to begin combining the QuestionCard, Answers, and Footer. It looks like this:

Approached changes to the answers when entering phase2 based on conditional rendering of a boolean value prop "phase2". When phase2 is the boolean true, then it will change stylistically to this:

I'm having difficulty on having two separate stories that look like this.
Phase 1 Answers:

Phase 2 Trick Answers:

The last answer for both of these screenshots depend on two boolean arguments being correct and phase2.
<HostAnswerDropdown answer={correctAnswer[0].choice} explanation={correctAnswer[0].explanation} correct={true} phase2={true}/>
GamesInProgress.stories.jsx uses GamesInProgress.jsx which will be the page wrapper for all the components that will make up the GamesInProgress page, i.e. Header, QuestionCard, Answers, Footer.  I don't know how to access the argtypes of HostAnswerDropDown because it's being wrapped by a parent component named GameDetails in AnswersInProgressDetails.jsx which has no args.


fix: fix __vite__mapDeps code injection

Description

Closes https://github.com/vitejs/vite/issues/15702

~I copied and adjusted mapFileCommentRegex constant from https://github.com/thlorenz/convert-source-map/blob/1afbeee2f2a42a3747c31dfcfc355387afdf42e2/index.js#L14 so that it will only matches the last line of the source code.~ (EDIT: now prepending __vite__mapDeps at the top)
I manually verified that sourcemap is working via pnpm -C playground/js-sourcemap build + preview:
Show screenshot
스크린캐스트 2024-02-21 13-28-46.webm

Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines, especially the Pull Request Guidelines.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Update the corresponding documentation if needed.
[x] Ideally, include relevant tests that fail without this PR but pass with it.

/ecosystem-ci run


Dropdown: search & selection allows disabled options to be selected

Bug Report
Steps

Create a searchable dropdown, see: codesandbox
Within the options, mark one as disabled
Search in the field such that only the disabled item is displayed (in my example above, type the letters Be to filter to Belgium)
Blur the field with a mouse click

Expected Result
Nothing is selected
Actual Result
The disabled item is selected in the field
Version
0.88.1 (also occurs on 0.80.2 where originally discovered)
Testcase
https://codesandbox.io/s/semantic-ui-example-yqlkk
Additional Information
This is not necessarily a bug if the developer disables selectOnBlur, like so:
 <Dropdown
    placeholder="Select Country"
    fluid
    search
    selection
    selectOnBlur={false}
    options={countryOptions}
  />

The results of disabling select on blur can be found on this codesandbox

Confirmed that this is reproducible when searching. Triaging to needs engineering for now. Thank you for the note on the workaround.

I will work on this


SQLite Target not available from Nuget

I'm using MetroLog in my Windows 8.1 app via Nuget and it seems that the SqLite target is not available (due to a refactoring perhaps?). Will a package be provided in the future?

Any news on this?

I am wondering the same. I am using MetroLog v. 0.8.9. The SQLiteTarget is not part of the namespace MetroLog.Targets. Any updates on why that is?

Same here.
So I added SQLiteTarget source files to my project to be able to use it

Any update on SQLiteTarget  to be included in Nuget Package.. Waiting for It

Bump.

Working on it...issue has been getting CI builds working with VSIX :(
Fortunately SQLitePCL.raw fixes this so it should be soon.

Is adding SQLiteTarget source files still the best/only solution?

Actually, a better workaround is to download source, build MetroLog.Targets.SQLite (in release mode), put MetroLog.NetCore.SQLite.dll in a lib directory, and reference it there.  I'm including the dll below if you want to skip the first couple of steps.
MetroLog.NetCore.SQLite.dll.zip


django-paypal version

I'm not sure if this is a real issue. I'd email you about it, but it might at least be a documentation issue.
I can't but realize that the pypi version of django-paypal (https://github.com/johnboxall/django-paypal) hasn't had a commit in three years. Looking at the github graph, https://github.com/dcramer/django-paypal seems to be the most updated one (meaning, the one most forked-off and which seems to be accepting the most pull requests).
Do you have any preference on which version of django-paypal to use?

I have no particular preference over which is used - the pypi supplied one worked well enough for my use cases.
If you think using dcramer 's version is more relevant, please feel free to do so.
Are there any incompatibilities that would require a patch to django-shop-paypal? If so we should probably sit down and discuss the best way forward (I don't think recommending a dead branch is a good idea).

I will be using this for a shop site skeleton I'm working on. I'll use the newest django-paypal and see if any trouble arises. I'll keep you posted


Admin

Could I be added as an admin to the repository So I can set tags and actions?

I'm at work now, but sure, I'll look into it tonight!
--
François Charron

Le 4 févr. 2021 à 16:06, Wylie Fowler<EMAIL_ADDRESS>a écrit :

Could I be added as an admin to the repository So I can set tags and actions?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sent you an invite to a new Github organization. I'm working on transferring the repo to that organization, so I can give you admin access. Simple personal repos can only add "Collaborators", but as you've already figured out, it's pretty limited.

Transfer is done, you're an admin. Just give me some time to update the atom repo with the updated URL before starting to push commits.

Sounds good, when I add something how do I push It to atom?

https://flight-manual.atom.io/hacking-atom/sections/publishing/ has the info about publishing. In short, once atom is installed, you can use apm publish <version_type> to publish. This will take care of pushing to Github, bump the version number according to the <version_type> (which can be "major", "minor", or "patch"), create a tag, and publishing to the atom repo.

I've had trouble pushing to the repo just now (needed to mess around with my git config to make atom use ssh instead of https, which was causing problems on my machine), so I suggest you try and follow the steps in the link I provided, and let me know if you run into problems.

It worked, Thanks so much for letting me help with this.

Glad it worked, and I should thank you for helping out 🙂
--
François Charron

Le 4 févr. 2021 à 22:34, Wylie Fowler<EMAIL_ADDRESS>a écrit :

It worked, Thanks so much for letting me help with this.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Failed to reproduce the results on Mutag dataset

Hello,
I retrained the downstream model following the documentation (e.g. python mutag_gnn.py) and also the Refine model using the recommended: python refine_train.py --dataset mutag --hid 100 --epoch 100 --ratio 0.4 --lr 1e-3 --batch_size 64
However, at the end of the training I get an AUC of 0.82 which is quite far from the reported 0.955. Am I missing something?
2022-07-01 18:01:23,919 - refine_train.py[line:163] - INFO: ACC:[[0.86 0.81 0.78 0.78 0.77 0.79 0.77 0.8 0.86 1. ]] ACC-AUC: 0.822 Mean P: 0.099 2022-07-01 18:01:44,651 - refine_train.py[line:158] - INFO: Epoch: 98, LR: 0.00001, Ratio: 0.40, Train Loss: -163.027, ValLoss: -1.704

Hi,
Could you try

run evaluate.py after the training
train a GNN model using a different seed and then run the cmd again


Review site issue

The queries don't seem to be working on the review site. It just says "Searching" and 'Show this query' says "Nothing to see here..."

Seems to be working now.
Closing this issue.


added favicon image in the title

Favicon image increased the overall personability of the website in the top section

for the smooth integration added favicon to the whole website

do acknowledge and merge it


How to show empty map when there is no data available?

First, I'm not very familiar with maps in react-native.
It looks silly, but I need the map to be empty at certain times, without Markers. It worked well before with normal maps.
I don't know what I'm missing here. Since data is required, using undefined, null, empty array doesn't work. The following error always occurs:
TypeError: Cannot read property 'latitude' of null.
Couldn't find a solution searching about it.
Please, what I might be missing about that?

The problem was in the data I was receiving from third party API. Sorry for opening this useless issue.


Please invite me to the GitHub Community Organization

Name
Joriz Ivann Villanueva
Discord Username (if applicable)
jrzvnn
Additional Context
I encountered jaidevstudio in a welcoming Open Source community that prioritizes collaboration over code. The inclusive nature and emphasis on meaningful connections impressed me. I'm eager to join and contribute to a space where diverse perspectives create impactful projects.

It's great having you contribute to this project
If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord Server and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.

Invitation sent for the GitHub Organisation. Welcome to the community 🎉#
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see, you can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/jaidevstudio/people
Like this

Tips for practising:
Remember contributing to open source is not just about code, its about collaboration, communication and adding value
I hope that helps

Please check your email or organisation page 🙏


Inconsistent Light State

I'm submitting this issue as partly a reminder to myself to fix this and partly as a request for others to review my code (I wrote the bridge support which enables the LED Lights).
Specifically I can reproduce this when feeding wemo this:
wemo light <lightname> off 255
The bulbs do not like to receive a dim setting when being asked to turn off as this would break the slow dim effect built into the bulb.
This produces a status like this:
Light: <lampname> 	{'dim': '255', 'state': '255'}
I believe this is what is being returned from the bulb and not an issue with my code (unsure). Obviously this could cause problems and so we need to ensure that these issues are addressed:

Ensure only good data is fed to the bulbs
Ensure rate of data sent to bulbs is controlled (have caused bulbs to act funny by sending too many requests quickly)
Error check response from bulbs

FYI: (To quickly switch off a light without the dim effect you can tell the bulb wemo light <lightname> on 0 meaning to turn on and set dim to zero)

I get the same status just using the off command.
melba:~ jluke$ /usr/local/bin/wemo light Light1 status
{'dim': '255', 'state': '1'}
melba:~ jluke$ wemo light 'Light1' off
melba:~ jluke$ /usr/local/bin/wemo light Light1 status
{'dim': '255', 'state': '255'}
This also has the side effect of greying out the app button
with the "Not Detected" message. See attached screen shots.
Macbook Pro
Mac OS 10.10.3
iOS 8.3

Thanks for your work with ouimeaux, much appreciated.
John Luke

Closing due to #123 (without testing)


Support HTTP2

We already have SSL support, just need to hook up http2.

:+1:

So just require in http2?
require('http2').createServer(options, app).listen(8080); ?

There appear to be some issues regarding http2 and express. Also does http2 allow for TLS connections over http1?


internal: improve pubsub javadoc

This is addressing some friction I had, as someone who hadn't used executors before. I think it's good for this code to be as copy-paste-able as possible.
Also, being new to executors, let me know if there's a better executor to be the default recommendation.

@pongad Whoops, didn't try compiling (doh). Fixed the issue


Accuracy fails when: 1) fp16 is enabled, and 2) multi-dimensional, multi-class

🐛 Bug
When 1) fp16 is used and 2) preds is multi-dimensional, multi-class, accuracy (perhaps also other classification metrics, though I haven't tested) fails to compute.
To Reproduce
# multidimensional, multiclass with logits
# preds shape: (N, C, D) where N=1, C=3, D=2

preds = torch.tensor([[[0.2, 0.7, 0.1], [0.1, 0.5, 0.4]]], dtype=torch.float)
preds = preds.transpose(1, 2)

target = torch.tensor([[1, -100]], dtype=torch.long)

When pred is of fp32 (like what's in the above code block), accuracy computes successfully:
torchmetrics.functional.accuracy(preds, target, mdmc_average='samplewise', average='micro', ignore_index=-100)

However, if we change preds to fp16 like below:
preds = torch.tensor([[[0.2, 0.7, 0.1], [0.1, 0.5, 0.4]]], dtype=torch.half)
preds = preds.transpose(1, 2)

then accuracy fails to compute. The error log:
File ~/OneDrive/App/python/python-env/py39-base/lib/python3.9/site-packages/torchmetrics/functional/classification/accuracy.py:471, in _accuracy_update(preds, target, reduce, mdmc_reduce, threshold, num_classes, top_k, multiclass, ignore_index, mode)
    469     raise ValueError("You can not use the `top_k` parameter to calculate accuracy for multi-label inputs.")
    470 preds, target = _input_squeeze(preds, target)
--> 471 tp, fp, tn, fn = _stat_scores_update(
    472     preds,
    473     target,
    474     reduce=reduce,
    475     mdmc_reduce=mdmc_reduce,
    476     threshold=threshold,
    477     num_classes=num_classes,
...
--> 848     preds = preds[target != ignore_index]
    849     target = target[target != ignore_index]
    851 return preds, target

IndexError: The shape of the mask [2, 2] at index 1 does not match the shape of the indexed tensor [2, 3, 2] at index 1

Environment

TorchMetrics version: 0.9.3 & 0.10.0dev0

Hi @XiaomoWu,
Thanks for raising this issue. I am happy to report that this issue already seems to have been solved in by a large refactor of the classification package, described in this issue #1001 and changed merge in this PR #1195. Below is an example using the new multiclass_accuracy implementation. Note that some arguments may have changed names.
import torch
from torchmetrics.functional.classification import multiclass_accuracy
preds = torch.tensor([[[0.2, 0.7, 0.1], [0.1, 0.5, 0.4]]], dtype=torch.float)
preds = preds.transpose(1, 2)
target = torch.tensor([[1, -100]], dtype=torch.long)

# float32
multiclass_accuracy(preds, target, num_classes=3, average='micro', multidim_average='samplewise', ignore_index=-100)
# returns tensor([1.])

# float16
multiclass_accuracy(preds.half(), target, num_classes=3, average='micro', multidim_average='samplewise', ignore_index=-100)
# returns tensor([1.])

The changes are available if you install directly from master
pip install https://github.com/Lightning-AI/metrics/archive/master.zip
else we plan to release v0.10 of TorchMetrics at the end of this month where the changes will be included.
Closing issue.

Hi @SkafteNicki,
I confirmed that accuracy is now successfully computed in v0.10dev0. Thank you!


Opacity problem with EditMode

In EditMode the opacity should be 1 in order not to clicking trough the EditMode and accidentally click the image zoom

The problem was that the Image Zoom wasn't with the property z-index: 0, by default was set to 1 for some reason


Implement metric MetricTabs component

Implement Tabs that show the metric overview and a metric chart in the tab panel. The component should receive a region and a list of metrics. The tabs and content should be responsive.

#479


Update Known-issues.md

SharePoint tab issue that is being worked on by SP PG: https://icm.ad.msft.net/imp/v3/incidents/details/104422007/home

@LolaJacobsen Hi Lola! Changes look correct could you please take a look and merge? Thanks!

All changes to this article must be approved by @marclauinger. I've added him as a reviewer.


Generate Documentation

New employees can become productive quicker by having (docstring) documentation to refer to.

[x] Use a tool that generates HTML documentation
[x] Generate and automatically update when new commits are pushed to master (CD)
[x] Host it somewhere

cleansio.readthedocs.io


Error when loading gold file

Hello,
I am trying to generate predictions from the sample of the development set provided here. However, the script fails upon loading the gold data.
Input: python natural_questions/make_test_data.py --gold_path sample/v1.0_sample_nq-dev-sample.jsonl.gz  --output_path whatever
Error:
I0201 17:10:07.177878<PHONE_NUMBER>68704 eval_utils.py:261] parsing sample/v1.0_sample_nq-dev-sample.jsonl.gz .....
multiprocessing.pool.RemoteTraceback:
"""
Traceback (most recent call last):
  File "XXX/miniconda3/lib/python3.7/multiprocessing/pool.py", line 121, in worker
    result = (True, func(*args, **kwds))
  File "XXX/miniconda3/lib/python3.7/multiprocessing/pool.py", line 44, in mapstar
    return list(map(*args))
  File "XXX/natural-questions/eval_utils.py", line 264, in read_annotation_from_one_split
    for line in input_file:
  File "XXX/miniconda3/lib/python3.7/gzip.py", line 374, in readline
    return self._buffer.readline(size)
  File "XXX/miniconda3/lib/python3.7/_compression.py", line 68, in readinto
    data = self.read(len(byte_view))
  File "XXX/miniconda3/lib/python3.7/gzip.py", line 463, in read
    if not self._read_gzip_header():
  File "XXX/miniconda3/lib/python3.7/gzip.py", line 406, in _read_gzip_header
    magic = self._fp.read(2)
  File "XXX/miniconda3/lib/python3.7/gzip.py", line 91, in read
    self.file.read(size-self._length+read)
  File "XXX/miniconda3/lib/python3.7/codecs.py", line 322, in decode
    (result, consumed) = self._buffer_decode(data, self.errors, final)
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8b in position 1: invalid start byte
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "natural-questions/make_test_data.py", line 120, in <module>
    app.run(main)
  File "XXX/miniconda3/lib/python3.7/site-packages/absl/app.py", line 300, in run
    _run_main(main, args)
  File "XXX/miniconda3/lib/python3.7/site-packages/absl/app.py", line 251, in _run_main
    sys.exit(main(argv))
  File "natural-questions/make_test_data.py", line 53, in main
    n_threads=FLAGS.num_threads)
  File "XXX/natural-questions/eval_utils.py", line 303, in read_annotation
    dict_list = pool.map(read_annotation_from_one_split, input_paths)
  File "XXX/miniconda3/lib/python3.7/multiprocessing/pool.py", line 290, in map
    return self._map_async(func, iterable, mapstar, chunksize).get()
  File "XXX/miniconda3/lib/python3.7/multiprocessing/pool.py", line 683, in get
    raise self._value

Specifically it fails at line 264 of eval_utils.py when trying to iterate over the gold file.
Can you help me solve this issue?
Thanks a lot!

I also have the same problem.

Indeed this fixes the problem, thanks.


feat: file upload & ffmpeg example

[ ] Attachment type (TODO figure out the exact type shape)

ComposerPrimitive

[x] ComposerState.attachments
[x] ComposerPrimitive.AddAttachment
[ ] ComposerPrimitive.Attachments
[x] Styled Components styles (WIP)
[ ] shadcn-ui styles
[ ] EditComposer state
[ ] Drag and drop

MessagePrimitive

[x] AssistantMessage.Attachments
[ ] MessagePrimitive.Attachments
[ ] Styled Components styles
[ ] shadcn-ui styles

AttachmentPrimitive

[ ] AttachmentState
[ ] AttachmentPrimitive.Root
[ ] AttachmentPrimitive.Remove
[ ] AttachmentPrimitive.Preview (TODO spec this)
[ ] Styled Components styles (composer)
[ ] shadcn-ui styles (composer)
[ ] Styled Components styles (message)
[ ] shadcn-ui styles (message)
[ ] Attachment type (image vs file) ?

AppendMessage types

[x] AppendMessage.attachments

AI SDK

[x] Vercel useChat
[x] Vercel useAssistant
[x] Vercel RSC
[ ] useAssistant handling is confusing and diverges from OAI

Local Runtime

[ ] LocalRuntime attachment handlers

ComposerPrimitive.Input style change

[x] Styled Components
[ ] shadcn-ui Thread
[ ] shadcn-ui Thread Full

Upload

[ ] File upload / upload progress tracking
[ ] ExternalStore onAddAttachment, onRemoveAttachment

super excited about this feature! 🤗


Create Craptopia Template

Create Craptopia dedicated server template

“Craptopia” lol

“Craptopia” lol

Oops. Not exactly what I meant. How embarrassing and hilarious.

as i know, you must set the port in ServerConfig.ini


Update OWA GEP to better reflect current state

How to categorize this PR?

/area documentation
/kind cleanup
What this PR does / why we need it:
This PR updates the OWA GEP to better reflect the current state of the project.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

NONE

/invite @vpnachev

/cc @vpnachev

Typically, we don't maintain the GEPs after implementation but rather add dedicated documents for the respective features. I'm fine with merging this, though.

Ok, thanks! I just wanted to clarify/remove some things from the GEP that were never implemented so that no one reading it gets confused.

/approve


Install guide

Please could you write few lines how to install it?

Just updated README.md with installation guide.
Please let me know if this don't work for you.


Feature Request: Extensible internationalization support

From what I've gathered so far, i18n (internationalization) support is currently limited and baked into the code.
It would be nice to have an API for scripts to access the i18n "backend" to query for translations, formatting, pluralization, and also be able to insert new information into it without the need for a new compiler version to be shipped to include them.
That would enable, for instance, the creation of a library like TeX's babel that centralizes this info and users could turn to for the most common cases and still be able to address their own specific needs.

Please see #216

Please see #216

Yes, fluent would definitely be great to be the backend of i18n!
(And maybe I should take a more thorough look on this repo before posting more suggestions 😅 . That's the second time today...)
Should I close this issue in favor of it?

Yes, please do close duplicate issues. I just scatters the discussion for everybody to have more than one with the same scope.


Use more extensive test suite

Only testing the import of the python packages is not optimal. We should be using as much of the upstream test suite as possible in our CI (and with a timeout of 6h, we have ample time for testing). After a quick search upstream, these are things that are probably relevant:

https://github.com/ray-project/ray/blob/ray-1.0.1/.bazelrc#L107-L111
https://github.com/ray-project/ray/blob/ray-1.0.1/cpp/BUILD.bazel#L60-L85
https://github.com/ray-project/ray/tree/ray-1.0.1/python/ray/tests
Some more tests in other places, e.g. https://github.com/ray-project/ray/tree/ray-1.0.1/release

CC @vnlitvinov

I agree with testing Ray core functionality by running some of the unit tests.
Here is our current Travis pipeline. The following three suites (Streaming, Worker, Tests) should be sufficient and run in ~20 minutes total. We could also opt to run ASAN tests (adding another 30 minutes).
https://github.com/ray-project/ray/blob/master/.buildkite/pipeline.yml#L102-L121
We can also find a (small) subset of sub library tests (for Ray Tune, Serve, RLLib) to run. In my opinion a very narrow set of basic functionality should suffice, as it should generally just serve as a smoke test to decide if the build succeeded and the sub libraries can actually be used at all. For this we will probably add another tag to the bazel rcs in upstream ray to be able to filter these.

I think we should be good with running some smoke tests, as we take already tested Ray sources for the start.
So we just want to make sure that we didn't screw something up in patching and packaging, thus a bit of everything that pokes most binaries but runs fast is a good thing to get.
This would also benefit the core developers, as it would allow running smoke tests during development...

I don't know how long the full test suite for all things would run, but I prefer to be more comprehensive than just smoke tests, because various corner cases are often only caught by a handful of tests. I'd be fine if running the test suite adds an hour or a bit more to our CI.

@krfricke
Hope you don't mind, I took the liberty of assigning you to this issue. 🙃


XML and std::vector

Luckily XML has real char types in contrast to JSON ( #310). However std::vector is serialised to
        <value1 size="dynamic">
                <value0>A</value0>
                <value1>C</value1>
                <value2>A</value2>
                <value3>C</value3>
                <value4>A</value4>
                <value5>C</value5>
                <value6>A</value6>
                <value7>C</value7>
                <value8>A</value8>
                <value9>C</value9>
                <value10>A</value10>
                <value11>A</value11>
                <value12>T</value12>
                <value13>T</value13>
                <value14>A</value14>
                <value15>T</value15>
        </value1>

while std::string is serialised to:
<value2>ACACACACACAATTAT</value2>

Obviously the second is much more space efficient and also easier to read. Could you change for the standard containers+char to serialise like string in XML?
Thanks!
PS: I don't know if this change could be made without breaking compatibility 😰

This is something you can implement on your own very easily by overloading the default serialization for vector and specializing it for char. I imagine this will be slower to serialize or deserialize due to the string construction, but the basic outline is:
template <class A> inline
void CEREAL_SAVE_FUNCTION_NAME( cereal::XMLOutputArchive & ar, std::vector<char, A> const & vector );

template <class A> inline
void CEREAL_LOAD_FUNCTION_NAME( cereal::XMLInputArchive & ar, std::vector<char, A> & vector );

Converting to string appropriately in those functions. You'll likely also want to specialize prologue and epilogue not to create a node in the XML for the vector<char> serialization function, otherwise you'll end up with output that looks like:
<value1>
  <value0>ACDADFDASDFDASF</value0>
</value1>

Instead of just the inner term.

I imagine this will be slower to serialize or deserialize due to the string construction

I suspect that this would even out in the wash actually, since the construction/output of all the tags seems like it would be a bunch more work.
That said, MY  implementation would use 'string_view' instead if possible (or a cereal::string_view if not), since std::vector::data() is compatible with a string_view.
Pulling it out would likely be able to use the string in place to load it to the vector.

Thanks for your comments. Yeah, I thought about doing something along the lines, but copying is not ideal.

That said, MY implementation would use 'string_view' instead if possible

string_view serialises to actual data? But its a non-owning data structure!?

You'd have to write a serializer function most likely (and perhaps teach rapid XML about string view), but there's no reason a non-owning string-view couldn't be copied from into the xml structure.

You'd have to write a serializer function most likely (and perhaps teach rapid XML about string view), but there's no reason a non-owning string-view couldn't be copied from into the xml structure.

Sure it could be done, I was only questioning whether something like that should be done 😄
Anyway, we are getting off-topic.


Scipy

I'm trying to run this on Ras Pi OS lite on a PI Zero. I keep getting the error: no module named 'scipy'. Please help 😅

use command sudo apt-get install python3-scipy and do that for numpy and all of those things in the insatll.py and also i recommend doing this as well
sudo pip3 install rpi_ws281x adafruit-circuitpython-neopixel
this command fixed alot for me

Thank you!!! I'll run all that and update.

let me know how it works!

Worked perfectly! I went through the individual installs in the old directions also. I'll add a picture of the final project once it's finished!


Reformat the project

I remember that Arthur talked about this time ago and we decide not to do it but maybe now is the time. #3579 has a lot of changes that provides 0 information about what the PR really does. The problem is that the old code isn't properly formatted.
I vote to activate the formatting plugin of detekt or add ktlint to the project directly so we ensure that the format in all the code is consistent. This will create a big PR changing a lot of lines but we know that those changes doesn't affect the behaviour so they are safe.

+1

Yup, I'd say let's apply the formatting plugin so we have a direct way to test it on the repo.


UP-4839: Update underscore version in uPortal

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines

Description of change

Updates version of underscore used by uPortal.

Coverage increased (+0.01%) to 22.896% when pulling ca82715c88803688132c7ca64a6fffa6d6e10238 on ChristianMurphy:UP-4839 into 503c0243556a7b165839e03ba0a3a52c99b0f9fa on Jasig:master.

Thank you @ChristianMurphy!

LGTM :+1:
Tested locally.  I have I fix for the page jumping when buttons are clicked, which I will apply after the merge.


No Error Message when creating a duplicate of a taxonomie element

Tried to add a Folder "Aufgaben zu SQL" but accidentally added a Topic with this name. After that I was not able to save a Folder with the Name "Aufgaben zu SQL" but also got no error message (just kept loading)

Moved to document with feature requests


Support Unions usage

Is this feature request related to a new or existing Amplify category?
api
Is this related to another service?
AppSync
Describe the feature you'd like to request
Appsync supports working with Interfaces and Unions (GraphQL Union Types Reference ) but amplify doesn't like them. Would like to see @model directive auto detect Union types and generate mutations for each of the types in the Union.
Describe the solution you'd like
GIVEN the schema
type EpicTask {
  id: ID!
  name: String
  isEpicTask: Boolean
}

type StoryTask {
  id: ID!
  name: String
  isStoryTask: Boolean
}

union Task = EpicTask | StoryTask

type Todo @model {
  id: ID!
  task: Task!
  description: String
}

WHEN amplify transforms it
THEN generate AppSync equivalent schema
input CreateTodoWithEpicInput {
	id: ID
	task: EpicTaskInput!
	description: String
}

input EpicTaskInput {
	id: ID
	name: String
	isEpicTask: Boolean
}

input CreateTodoWithStoryInput {
	id: ID
	task: StoryTaskInput!
	description: String
}

input StoryTaskInput {
	id: ID
	name: String
	isStoryTask: Boolean
}

type Mutation {
	createTodoWithEpic(input: CreateTodoWithEpicInput!, condition: ModelTodoConditionInput): Todo
	createTodoWithStory(input: CreateTodoWithStoryInput!, condition: ModelTodoConditionInput): Todo
}

type Query {
	getTodo(id: ID!): Todo
	listTodos(filter: ModelTodoFilterInput, limit: Int, nextToken: String): ModelTodoConnection
}

Sample Mutations
mutation CreateEpicTodo {
  createTodo(input: {task: {name: "First Epic", isEpicTask: true}, description: "First Epic"}) {
    description
    id
  }
}

mutation CreateStoryTodo {
  createTodoWithStory(input: {task: {name: "First Story", isStoryTask: true}, description: "First Story"}) {
    description
    id
  }
}

Sample Queries
query getTodoList {
  listTodos {
    items {
      description
      id
      task {
        __typename
        ... on EpicTask {
          isEpicTask
        }
        ... on StoryTask {
          isStoryTask
        }
      }
    }
  }
}

sample output:
{
  "data": {
    "listTodos": {
      "items": [
        {
          "description": "First Story",
          "id": "3cdd3dcf-71d7-4710-8978-4e4bffb77686",
          "task": {
            "__typename": "StoryTask",
            "isStoryTask": true
          }
        },
        {
          "description": "First Epic",
          "id": "f96b9bec-f4f1-4b28-a840-6c3224100cd6",
          "task": {
            "__typename": "EpicTask",
            "isEpicTask": true
          }
        }
      ]
    }
  }
}

Describe alternatives you've considered

Manually creating AppSync types by hand - this obviously beats the purpose of using Amplify code generation
Using separate fields for each Type in the union - this won't scale and not intuitive, bloats the model

type Todo @model {
  id: ID!
  epicTask: EpicTask
  storyTask: StoryTask
  description: String
}

Additional context
This requires parity with AppSync and we hit by this during our schema design and require this in shaping our model.
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Were you able to apply the first approach :
Manually creating AppSync types by hand - this obviously beats the purpose of using Amplify code generation
I am trying to implement a union and is at the crossroads to redefining the schema vs implementing the mutations and Appsync schema by hand. Can you provide your experience?
Thanks!


Feature to add calendar to see past outages / incidents and upcoming maintenance

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
API, UI Feature
🔖 Feature description
Feature to add calendar to the statusPage see past outages and upcoming maintenance
✔ Solution
Feature to add calendar to the statusPage see past outages / incidents and upcoming maintenance.
❓ Alternatives
No response
📝 Additional Context
No response

I think this is a duplicate of https://github.com/louislam/uptime-kuma/issues/1984
If you agree, could you please close this Issue, as duplicates only create immortal zombies and are really hard to issue-manage?
If not, what makes this issue unique enough to require an additional issue? (Could this be integrated into the issue linked above?) ^^

yes it can be integrated to the issue #1984 as this feature request requests for adding the incident and upcoming maintenance on the statusPage for each service.

@sachintandon-nexla
Could you please close this issue then?
Only you or Louis can do so..

yes it can be integrated to the issue https://github.com/louislam/uptime-kuma/issues/1984

Closing as a duplicate


iPad orientation change bug

The background doesn't stretch entirely when I change from portrait to landscape:

However it worked when I removed the window.pageYOffset condition from:
if (this.isBody && window.pageYOffset === 0) {
  window.scrollTo(0, 1);
  this.resize();
}

So, is this condition really necessary? why scrolling to (0, 1) instead of (0, 0)?
Env: iPad 4 iOS 7.0.4, Safari

Same issue here. Did this change resolve your issue without any unintended consequences?


make log level lower to consist with reconcile message

This message is not necessary. It makes log message not readable because lots of them printed quickly.
Refer to https://github.com/kubernetes-sigs/federation-v2/pull/617#discussion_r261435218 for information

I think you've got the right idea here - this message doesn't have to appear repeatedly.  How often will it be logged (regardless of level)?

2 messages is logged repeatly every 2 seconds in my environment.
I0306 05:58:00.515905       1 controller.go:163] Skipping start of sync & status controller for cluster-scoped resource "Fede│I0306 05:57:12.933206       1 controller.go:134] Shutting down "ingress" DNSEndpoint controller
ratedClusterRole".  It is not required for a namespaced federation control plane.                                            │I0306 05:57:12.933605       1 controller.go:134] Shutting down "service" DNSEndpoint controller
I0306 05:58:00.812859       1 controller.go:163] Skipping start of sync & status controller for cluster-scoped resource "Fede│rpc error: code = Unknown desc = Error: No such container: 5a62176bf72013f3d513bf8ffca53c36d0602f0e9436f
ratedNamespace".  It is not required for a namespaced federation control plane.                                              │8a6568bd0142e717ec1panpxpx@lsftest01:~$ kubectl get crd
I0306 05:58:02.510431       1 controller.go:163] Skipping start of sync & status controller for cluster-scoped resource "Fede│NAME                                                        CREATED AT
ratedClusterRole".  It is not required for a namespaced federation control plane.                                            │clusterpropagatedversions.core.federation.k8s.io            2019-03-06T05:56:03Z
I0306 05:58:02.714683       1 controller.go:163] Skipping start of sync & status controller for cluster-scoped resource "Fede│dnsendpoints.multiclusterdns.federation.k8s.io              2019-03-06T05:56:03Z
ratedNamespace".  It is not required for a namespaced federation control plane.
...

@xunpan I think that suggests logging the message only once per type rather than decreasing the log level.

Resolved. The message only print once for each type.

Nicely done, thank you!
/approve
/lgtm


SSL connection to Azure Cache for Redis

Hi,
I have problem with connection to Azure Cache Redis and I'm not sure if it is possible or I did something wrong.
So far in my sevices I used connection string like: rediss://:[password]@[host]:6380. Default not ssl port is disabled by server configuration.
Tried Authentication "User & Password", "No Auth" with option Use SSL turned on and off and with parameter "sslprotocols=tls1.2" and "ssl=true".
DataGrip shows: "Failed to connect to any host resolved for DNS name."
Redis version: 6.0
Connection string works just fine with Jedis and Lettuce.
Thanks in advance.

nvm, figured it out.
Server needs public endpoints turned on and new cluster connection is required.


Update on the footer

Design on the footer is updated.
Also added links to the other pages.
Needs to be add to rest of the pages.

Could we speed up this PR for merge please?

You have not requested a review from QA. We haven't gotten a notification about this and that's why our SM has not assigned it yet.

Oh right. Understandable if you have not get pinged :D Are you on it now then Eli ?

Great! Thank you! I only looked at this PR because I was curious since it was about the footer, and then I saw your comment :D I'm on it now!


Menu and Weapon Quick Change

The main menu was moving to a scene we aren't using and certain weapons are too far away, causing the player to be unable to shoot at enemies near the player.

I'm getting ready to merge and updating the change log - please @me if any other changes are pushed.


Show the icon in VS Code extenions list

Currently their is a blank logo in the extension. Link the already presented logo to extenion.
Preview

Hey, I have added the logo can you check out if it appears or not?

Hey, I have added the logo can you check out if it appears or not?

You have to send a pull request with your changes done in the project, then only I can review and merge them.

@iamAbhishekkumar assign me this issue.

@iamAbhishekkumar assign me this issue.

@Pritam0077 Are you still working on this issue or not?

No

@AkhileshMattam go ahead I you want to work on this.


Doc insertion speed is much slower with SQLite 3.8.5

See https://github.com/couchbase/couchbase-lite-ios/issues/487

SQLite3 versions on Android

Version 5.0.0 (lollipop) API 21 - 3.8.6 2014-08-15 11:46:33 9491ba7d738528f168657adb43a198238abde19e
Version 4.4.4 (Kitkat) API 19 - 3.7.11 2012-03-20 11:35:50 00bb9c9ce4f465e6ac321ced2a9d0062dc364669

Note: version is checked by android emulator (Genymotion) by following command
adb shell
sqlite3 --version

The patch to improve the document insertion speed (https://github.com/couchbase/couchbase-lite-ios/commit/f20387ebfb98fefeb60041056c15dad50a0f57f4) was already applied by @tleyden (https://github.com/couchbase/couchbase-lite-java-core/commit/219a74bd71a77adccc022724353a97b21d3c5b6f#diff-ddbec9e62a53b23c377faa1d1051c0af) to fix other ticket (https://github.com/couchbase/couchbase-lite-android/issues/444).

Verification
Compared document insertion performance between version without fix (one commit earlier) (https://github.com/couchbase/couchbase-lite-java-core/commit/d4f96a4ce2655afea652ff0cb4ea0690b2a270f7) and current master (https://github.com/couchbase/couchbase-lite-java-core/commit/8de482e430b246eb286760bfbc3f6ef71a207769)
Test Environment: 4.4.4 (KitKat) and 5.0.0 (Lollipop) on Genymotion Emulator on OSX.  KitKat includes SQLite3 v3.7.11, Lollipop includes SQLite3 v3.8.6.
Test Case: testCreateDocsPerformance() method of Test1_CreateDocs.java  2000 document size and 2000 documents. Measured 3 times each test case.
https://github.com/couchbase/couchbase-lite-java-core/commit/d4f96a4ce2655afea652ff0cb4ea0690b2a270f7 (Before applying fix)

KitKat - 3697ms, 3871ms, 3993ms
Lollipop - 1867ms, 1779ms, 1790ms

https://github.com/couchbase/couchbase-lite-java-core/commit/8de482e430b246eb286760bfbc3f6ef71a207769  (After applying fix)

KitKat - 3458ms, 3519ms, 3520ms
Lollipop - 1340ms, 1493ms, 1461ms

Result: I observed around 30% performance improvement for Lollipop, and around 10% improvement for KitKat