DI API renames/cleanup

cc @muratg @BrennanConroy @lodejard @davidfowl

Remove IServiceDescriptor, move ServiceDescriptor to interfaces
Remove fallback namespace for BuildServiceProvider
Rename LifeCycle Kind to ServiceLifetime

Fixes https://github.com/aspnet/DependencyInjection/issues/188
Fixes https://github.com/aspnet/DependencyInjection/issues/187
Fixes https://github.com/aspnet/DependencyInjection/issues/184
Fixes https://github.com/aspnet/DependencyInjection/issues/183

One change and :shipit:

Updated to use shared NotNull

@davidfowl Updated PR, actually deleted ServiceDescriber this time, and moved the old helpers like Transient/Singleton/Instance as static helpers on ServiceDescriptor so usage for TryAdd is basically unchanged.

:shipit:

Adding back ServiceDescriber for now, MVC relies on this heavily today for their functional tests, but talked to @rynowak and once I get the new Hosting.Create PR https://github.com/aspnet/Hosting/pull/146 in, mvc can switch its functionals to that, and then we can remove the Describer

Merged f0897c2b26413e9187f6d1bd245a3dce0ac9b0f8


Enhancement: DatadogMetricsConfigurer expansions should be configurable

In DatadogMetricsConfigurer the following means all metrics are pushed to datadog daemon:
EnumSet<DatadogReporter.Expansion> expansions = EnumSet.of(ALL);
This should be configurable via spring or otherwise

Duplicate of this one -> https://github.com/aol/micro-server/issues/258

Merged


Build webhook events properly

Solves https://github.com/rossumai/elisctl/issues/97

Solved by https://github.com/rossumai/elisctl/pull/102


How To: Add invite_for in value dynamically

I want to invite users with a different invite_for value
for example
some user: invitation token should expire in 3 days
other user: invitation token should expire in 4 days
is there any way to do so?

It they are the same user model, it isn't possible, as you can see expiration check uses invitation created timestamp (sent is used for backwards compatibility):
https://github.com/scambra/devise_invitable/blob/master/lib/devise_invitable/models.rb#L263
You can have multiple models inheriting from User, using STI, and setup invite_for setting with different values for model.
Or you can add invitation_expired_at column to your model, and some callback to your model, and override invitation_period_valid? to check that timestamp. E.g.:
before_invitation_created :setup_expiration

def setup_expiration
  self.invitation_expired_at = self.class.invite_for.days.since
end

def invitation_period_valid?
  return invitation_expired_at.future? if invitation_expired_at?
  super
end

@scambra  thanks for the reply. I thought there may exist something like this where I can use days dynamically


add our names to the template license

Since our LICENSE file was simply a template, I've added our names, as required.

Fixed that, @thpani please approve once again :)

I have changed the PR rules so they do not reset an approval on a new commit

Actually, I will close this PR, as it is superseded by #17


test(catch): add comprehensive marbles tests for catch

:+1:

Merged with b9a07cc, thanks @staltz


Show portus version in web UI

I couldn't find anything which shows current portus version running

It should be in the bottom of the page. If you don't see it, can you tell me how did you deploy Portus ?

I've deployed it using master branch. Here is what I see now.

I've deployed it using master branch.

What do you exactly mean by that ? 😅 You could've deployed this with docker-compose, vagrant, bare metal, etc. For example, I develop in bare metal and this is what I see:

I wonder why I don't see it. Ok, looks like this is my side issue. Thanks.


Change build_fat_mac.sh script to use arm64e shim

Gave a test build to user (compiled using the script locally) - fixes their issue! 😄

Please verify build_fat_mac.sh and the release action both work, before merging.

To clarify further, make sure you download the release build from the action and see that it works


Remove duplicate percentile report. Fix underflow in "network misc" repot.

Report percentiles in increment order, including user specified percentile.
p90 (duplicate percentile)
csv
Concurrency,Inferences/Second,Client Send,Network+Server Send/Recv,Server Queue,Server Compute,Client Recv,p50 latency,p90 latency,p95 latency,p99 latency
1,101,296,1102,48,8369,10,9889,10743,10796,10976

output
root@d3c067c510bb:/opt/tensorrtserver# qa/clients/perf_client -m resnet50_netdef -p 5000 -v --percentile 90 -d -f p90.csv
*** Measurement Settings ***
  Batch size: 1
  Measurement window: 5000 msec
  Latency limit: 0 msec
  Stabilizing using p90 latency

Request concurrency: 1
  Pass [1] throughput: 92 infer/sec. p90 latency: 11091 usec
  Pass [2] throughput: 99 infer/sec. p90 latency: 10884 usec
  Pass [3] throughput: 117 infer/sec. p90 latency: 10671 usec
  Pass [4] throughput: 99 infer/sec. p90 latency: 10869 usec
  Pass [5] throughput: 99 infer/sec. p90 latency: 10364 usec
  Pass [6] throughput: 101 infer/sec. p90 latency: 10743 usec
  Client:
    Request count: 508
    Throughput: 101 infer/sec
    p50 latency: 9889 usec
    p90 latency: 10743 usec
    p95 latency: 10796 usec
    p99 latency: 10976 usec
    Avg HTTP time: 9776 usec (send 296 usec + response wait 9470 usec + receive 10 usec)
  Server:
    Request count: 607
    Avg request latency: 8478 usec (overhead 61 usec + queue 48 usec + compute 8369 usec)

[ 0] SUCCESS
Inferences/Second vs. Client p90 Batch Latency
Concurrency: 1, 101 infer/sec, latency 10743 usec

p88 (sorted)
csv
Concurrency,Inferences/Second,Client Send,Network+Server Send/Recv,Server Queue,Server Compute,Client Recv,p50 latency,p88 latency,p90 latency,p95 latency,p99 latency
1,97,281,1293,56,8641,10,10695,10922,10940,11025,11248

output
root@d3c067c510bb:/opt/tensorrtserver# qa/clients/perf_client -m resnet50_netdef -p 5000 -v --percentile 88 -d -f p88.csv
*** Measurement Settings ***
  Batch size: 1
  Measurement window: 5000 msec
  Latency limit: 0 msec
  Stabilizing using p88 latency

Request concurrency: 1
  Pass [1] throughput: 97 infer/sec. p88 latency: 10898 usec
  Pass [2] throughput: 127 infer/sec. p88 latency: 7937 usec
  Pass [3] throughput: 108 infer/sec. p88 latency: 10853 usec
  Pass [4] throughput: 94 infer/sec. p88 latency: 10922 usec
  Pass [5] throughput: 97 infer/sec. p88 latency: 10922 usec
  Client:
    Request count: 485
    Throughput: 97 infer/sec
    p50 latency: 10695 usec
    p88 latency: 10922 usec
    p90 latency: 10940 usec
    p95 latency: 11025 usec
    p99 latency: 11248 usec
    Avg HTTP time: 10075 usec (send 281 usec + response wait 9784 usec + receive 10 usec)
  Server:
    Request count: 590
    Avg request latency: 8759 usec (overhead 62 usec + queue 56 usec + compute 8641 usec)

[ 0] SUCCESS
Inferences/Second vs. Client p88 Batch Latency
Concurrency: 1, 97 infer/sec, latency 10922 usec

@GuanLuo can you start CI

started

I also added the fix on underflow csv output here...


pkgdev manifest may cause manifest errors if aborted while downloading an archive

When clicking ctrl-c to abort pkgdev manifest, the (incomplete) file being downloaded will be stored with a normal file name, indistinguishable from the normal downloaded file. I can only use pkgdev manifest -f to download again. At the same time, pkgdev manifest will not display the file name of the file being downloaded when downloading the file. If someone use curl as the fetch command ( recently I got some 403 error when using wget to download from github ) , it is unable to tell which file is currently being downloaded.

See also https://github.com/pkgcore/pkgcore/issues/52 and https://bugs.gentoo.org/279096.


added support for LT100 Smart Lighting Towers

CLP Boards 0.2.0 is required to use LT100, so update it using the Arduino Board Manager. Open "Tools">"Board:">"Boards Manager...">search for "clp" and install the update.

4 leds towers direct output
LT100_4towers.zip


Pip-compatible packaging?

Really awesome work, not seen much else of this quality for visualizing hyperbolic spaces in Python!
Would be great to get this into a more pip-friendly installation (and maybe even get it on PyPI if you're open to that), had a hard time with the existing setuptools based approach but was able to repackage it to a pip-friendly format pretty easily, more than happy to help with a fork as well!

thanks, glad people are checking this out! I have been procrastinating putting this up on PyPI solely because I think the project might need a more descriptive name and I don't have one yet...
do you mind letting me know what you needed to change to get this to work with pip? I've only tested installation in pretty limited circumstances, but when I tried, pip worked as-is.


Error when running on a clean install

Describe the bug
On a clean install, the predict.py script gets errors.
To Reproduce
Steps to reproduce:

Clone the repository: $ git clone https://github.com/ekraka/TS-GAN.git
Create a new conda environment: $ conda create -n GANTS python=3.11
Install tensorflow: $ pip install tensorflow
Run the predict.py script with test xyz files.

$ python predict.py reactant.xyz product.xyz
Traceback (most recent call last):
  File "/Users/bruno/Documents/TS-GAN/test_cases/dataset-A/../../predict.py", line 20, in <module>
    from matplotlib import pyplot
ModuleNotFoundError: No module named 'matplotlib'

Install matplotlib pip install matplotlib
Run the predict.py script with test xyz files. $ python predict.py reactant.xyz product.xyz
See error.

The instruction to run the predict.py script say to add the g_mode.h5 file to the working directory where the xyz files are but this error occurs before that is needed. I've tried it both ways and the same error is given.
Expected behavior
The example should run without errors.
Error messages
$ python predict.py reactant.xyz product.xyz
2023-04-18 12:03:07.129911: I tensorflow/core/platform/cpu_feature_guard.cc:182] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
(1, 50, 50, 2) (1, 50, 50, 2)
Loaded (1, 50, 50, 2) (1, 50, 50, 2)
...
<Many TensorFlow WARNINGS>
...
Traceback (most recent call last):
  File "/Users/bruno/Documents/TS-GAN/test_cases/dataset-A/../../predict.py", line 310, in <module>
    d_model = define_discriminator(image_shape)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/bruno/Documents/TS-GAN/test_cases/dataset-A/../../predict.py", line 62, in define_discriminator
    model = Model([in_src_image, in_target_image], patch_out)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/bruno/miniconda3/envs/GANTS_new/lib/python3.11/site-packages/tensorflow/python/trackable/base.py", line 205, in _method_wrapper
    result = method(self, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/bruno/miniconda3/envs/GANTS_new/lib/python3.11/site-packages/keras/engine/functional.py", line 159, in __init__
    [
  File "/Users/bruno/miniconda3/envs/GANTS_new/lib/python3.11/site-packages/keras/engine/functional.py", line 160, in <listcomp>
    functional_utils.is_input_keras_tensor(t)
  File "/Users/bruno/miniconda3/envs/GANTS_new/lib/python3.11/site-packages/keras/engine/functional_utils.py", line 49, in is_input_keras_tensor
    return tensor.node.is_input
           ^^^^^^^^^^^
AttributeError: 'KerasTensor' object has no attribute 'node'

System information:

Operating System: macOS Ventura 13.2.1
Python version: 3.11.2
Tensorflow version: 2.12.0

Additional Information
I attach the test xyz files that I used (as txt because github doesn't recognize the xyz format).
reactant.txt
product.txt

Hello,
You are using TensorFlow version: 2.12.0. The instruction says to use TensorFlow version 2.2.0.
TensorFlow is an open-source machine-learning library developed by Google. At the time of writing TS-GAN, version 2.2.0 was the newest one. Recently Google released Tensorflow version 2.12.0.
Our code will be updated as well in the near future. However, for now, the recommendation is to create a new Conda environment and install:
pip install tensorflow==2.2.0
That will solve your error.
Best regards,
Malgorzata

The predict.py script does not use Matplotlib; therefore, this line can be commented.
I am still trying to understand why it didn't run in the first place.
In my environment, I have:
Python 3.8
numpy 1.20.0
matplotlib 3.7.1
TensorFlow 2.2.0
As I mentioned, we are working on updating the TS-GAN code with the newest versions of those libraries.

It might be because we're using Mac, we've been having many issues with code running differently


Update depandabot configurations

Change frequency from live to weekly
Allow only security updates
Closes #311

@ivantha please review!!


Doing setarch isn't enough. We NEED to pass variables down.

So this actually does what we need for 64 bits, modulo that I build 32 bits OpenSSL libraries.

Failures on i386 are expected?

No, it's not yet ready. Still working on it.


Updated to include descriptions of new features

What's changed, or what was fixed?
Added a section discussing stack packs and plugins.
Target Live Date: 2019-11-09

[X] This has been reviewed and approved by Elizabeth Sweeny.
[ ] I have run npm test locally and all tests pass.
[ ] I have added the appropriate type-something label.
[ ] I've staged the site and manually verified that my content displays correctly.

CC: @petele

@petele :) Sorry do drop this on you right before CDS...

:tada: This has been pushed live to https://developers.google.com/web/


Doge Auction[feat]

have a built-in verification for Auctions/Bids ex.

Do the bidder have enough doge to make the bid.

Both wallet should know that The bidder and auctioneer have enough money to do the transaction without the wallet owners knowing (for Privacy Reasons).

Do Unsuccessful Bid have lower to no transaction fee.

I think this is a neat idea and should be implemented cause auctions like in ebay is a way to use a currency.

You can build this on top of Dogecoin today. Please let us know how your implementation progresses on https://www.reddit.com/r/dogecoindev/


heat signaling?

I seem to be having trouble with resources not signaling complete.  Using the HA setup, the lb completes, and the infra_host builds.  Looking in cloud-init logs I see everything completed successfully:

notify_success 'OpenShift node has been prepared for running ansible.'
Feb 25 08:12:55 localhost cloud-init: + curl -i -X POST -H 'X-Auth-Token: df69949e37644d94a9a8345ae72b697b' -H 'Content-Type: application/json' -H 'Accept: application/json' http://<IP_ADDRESS>:8004/v1/be5e2313437b44f18c09889f571ae15d/stacks/my_openshift-infra_host-r6ypfk4jlz5p/ed63786b-8812-4f99-bcf5-071989ac7431/resources/wait_handle/signal --data-binary '{"status": "SUCCESS", "reason": "OpenShift node has been prepared for running ansible.", "data": "OpenShift node has been prepared for running ansible."}'

However the stack does not seem to advance, instead it stays 'create_in_progress'
deployment_tune_ansible   | 5363098c-fa5b-4807-938d-55fafaf47c24                                                | OS::Heat::SoftwareDeployment                                                                  | CREATE_IN_PROGRESS | 2016-02-25T13:06:51Z | infra_host      |
If I use heat resource-signal and manually signal that task as complete, it moves on to deploying the 3 masters, but they get stuck the same way after the work from cloud-init is done.
Any ideas on what is going on here?

deployment_tune_ansible is a SoftwareDeployment resource which is triggered by os-collect-config service (heat agent running inside VM), these sw deployments run in post cloud-init phase. If it's stuck in IN_PROGRESS state, I can think of 2 reasons:

cloud-init on infra host didn't finish yet for some reason - it's not this case though based on the successful notify_success above
os-collect-config on the infra node has issues running the deployment_tune_ansible script. Could you please check "journalctl -u os-collect-config" on the infra node - there should be a reason. Based on the fact that the heat stack is stuck in IN_PROGRESS state for a while, my guess would be that os-collect-config is failing to fetch instance metadata from heat-api-cfn server -> is heat-api-cfn up and running in your openstack deployment?

Jan - Thank you very much.  I'm missing heat-api-cfn server.  I'll get that
turned up and should be fine.
Brent Roskos
Senior Cloud Architect
Cloud Practice
Red Hat Consulting
678-744-3623
On Thu, Feb 25, 2016 at 8:45 AM, Jan Provaznik<EMAIL_ADDRESS>wrote:

deployment_tune_ansible is a SoftwareDeployment resource which is
triggered by os-collect-config service (heat agent running inside VM),
these sw deployments run in post cloud-init phase. If it's stuck in
IN_PROGRESS state, I can think of 2 reasons:

cloud-init on infra host didn't finish yet for some reason - it's not
this case though based on the successful notify_success above
os-collect-config on the infra node has issues running the
deployment_tune_ansible script. Could you please check "journalctl -u
os-collect-config" on the infra node - there should be a reason. Based on
the fact that the heat stack is stuck in IN_PROGRESS state for a while, my
guess would be that os-collect-config is failing to fetch instance metadata
from heat-api-cfn server -> is heat-api-cfn up and running in your
openstack deployment?

—
Reply to this email directly or view it on GitHub
https://github.com/redhat-openstack/openshift-on-openstack/issues/49#issuecomment-188789702
.

Cool. We should improve our documentation to clarify openstack requriements as you are not the first one hitting this issue. I opened https://github.com/redhat-openstack/openshift-on-openstack/issues/50 to make sure we don't forget.

you also need to make sure the the heat metadata service is reachable from the nova node. You can adjust the listen ip in the heat.conf on the controller


Test cluster upgrading on AWS with Dual-Stack - Test Release KubeOne 1.6

This is working as expected.


Is it possible to hide "Feed" just like "People", "CRM", "Projects", etc.?

Hello everyone. I am using Workspace. I have configured the navigation panel and hidden all the items. But the "Feed" remains by default. There is no checkbox for it. Is it possible to hide it somehow, by accessing the source code, for example?

Hello @dwpoint Sorry, but this option is not available right now. There is a suggestion for the ability to disable the module Feed (a bug 55710 in our internal bugtracker).


Missing Lists

Hey there, thanks for making this!
I am missing some lists after setting it up.  it seems to have imported exactly 50 lists - is that a hard limit perhaps? I have more than that.
Some of the lists are of the type 'shared lists' which maybe aren't supported?
Thanks for your time and effort.

You can check what lists it will display using this query in MS Graph Explorer - https://graph.microsoft.com/v1.0/me/todo/lists
I don't believe MS support shared lists in the api.

No further information provided


IndexError in CRLoss.py

I meet this problem while running train.py. I strictly followed the md file, and I hope to get a response.
Traceback (most recent call last):
File "train.py", line 228, in
main(args)
File "train.py", line 180, in main
args,
File "train.py", line 77, in train
sim_loss = cr_loss_fun(sim, labels, semi=False)
File "/ai/test/Miniconda3/envs/CFine/lib/python3.6/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
result = self.forward(*input, **kwargs)
File "/ai/test/Paper-Code/CFine-main/utils/CRLoss.py", line 82, in forward
cr_loss = self.calculate_loss(similarity, labels, self.margin, semi)
File "/ai/test/Paper-Code/CFine-main/utils/CRLoss.py", line 64, in calculate_loss
image_triplets, img_margin = self.get_triplets(similarity, label, margin, semi)
File "/ai/test/Paper-Code/CFine-main/utils/CRLoss.py", line 42, in get_triplets
loss = similarity[idx, negative] - ap_sim + margin[idx]
IndexError: index 16 is out of bounds for axis 1 with size 16

I meet the same problem, do you have solved it?
There's a strange thing.
The batch size is 32,
in train.py, line 60
img_output, text_output, img_f, text_f, sim_cs, sim_cd = network(
images, tokens, segments, input_masks
)
images.shape = (32, 3, 224,224)
But in model.py, line 110,
def forward(self, images, tokens, segments, input_masks):
in here, images.shape=(16,3,224,224)
Why this?

I have addressed this problem, the code is distributed, needing to remove all nn.DataParallel, and adding the .cuda() for model and parameters. Then, it works well.

I have addressed this problem, the code is distributed, needing to remove all nn.DataParallel, and adding the .cuda() for model and parameters. Then, it works well.

hello, have you applied for the CUHK-PEDES dataset?


Javascript ES6 class suggested code action is shown but no lightbulb appears

VSCode Version:1.22.1
OS Version: Fedora 27

Steps to Reproduce:

load this file:
https://github.com/natetarrh/frogger/blob/master/game.js
go to line 520 and see the dotted underscore, mouse over and see the suggestion to convert to a class declaration
click on it and see NO lightbulb to allow you to perform the action
copy and paste the Game function into a new file and it will now work

Yes, ran with 'code --disable-extensions'

Thanks. Duplicate of https://github.com/Microsoft/TypeScript/issues/23235


ch. 11 problem

Hi. Thanks for your code!
I would like to ask. In Ch.11, when I run this:
ax = fig.add_subplot(222, projection='3d')
ax.plot(frequencies,np.angle(fourier[:N/2+1]),zs=np.absolute(fourier[:N/2+1])**2)
ax.view_init(20, 20)
ax.set_xlabel('Frequency (Hz)')
ax.set_ylabel('Phase')
ax.set_zlabel('Power')
plt.title("3D representation of the Fourier transform")

I get the following error:

slice indices must be integers or None or have an index method

Perhaps there is come changes in Python 3?
Thanks!

I managed to figure out. The result of N/2+1 is a float. int(N/2+1) solved the problem.


Executable not readable or lacks permissions for Laravel Pint.

Tried installing the extension.  I'm using Pint in a local project so I left the executable path alone.
Seeing the following error when saving a file in VS Code:

Executable not readable or lacks permissions for Laravel Pint.

Figured it out.  Using a non-standard directory setup due to Docker.  Just needed to change the config/executable path.

Still having this error.
I'm using Laragon, PHP8.1


Migrations fail on initial start up

Checklist

[x] Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
[x] Are you sure you're not using someone else's docker image?
[x] If having problems with Lets Encrypt, have you made absolutely sure your site is accessible from outside of your network?

  I docker exec'd into the container and manually installed them with npm i pg and npm i body-parser
Interesting to note, the logs indicate that several tables were created, however when I log into the DB server, there are only two created migrations and migrations_lock
Expected behavior
I expected the container to start correctly, doing whatever it needs to do on first run.  Then logging into and doing all the things.
Screenshots
n/a
Operating System
Ubuntu 18.04
Additional context
Docker version 19.03.8
root@server:/opt/proxymanager# more config.json
{
  "database": {
    "engine": "postgres",
    "host": "XXXXXX",
    "name": "proxymanager",
    "user": "XXXXXX",
    "password": "XXXXXX",
    "port": 5432
  }
}

So I tried to use MySQL and it had no issues.  I didn't see in the docs that Postgres would not work (but, I might have missed it), so feel free to close this if you don't want to support Postgres


Store receipts in a table

Problem space
In Explorer, we don't have a way to know that some accounts were touched during cross-contract calls as we only track transactions, which has a pair of signer_id and receiver_id. However, contract execution may trigger cross-contract calls and touch "third-party" contracts.
E.g. #212, @evgenykuzyakov  thought that cross contract call was happening, no way to track. #337 was brought by @mattlockyer and points to the same issue of lack of receipts details to explore cross-contract calls.
Solution
Track receipts (only exposed in TX detail as logs with status) in another table (second step)

Currently retrieving from RPC on TX detail page
Need to build receipt table somehow, associate with accounts

Acceptance Criteria

[x] A new Receipt model is defined to cover all the needs (indices and redundant columns to avoid expensive JOINs are in place)
[x] Transaction status is saved into the transactions table resolving: https://github.com/nearprotocol/near-explorer/blob/6eae103df36df1b9918e7689aca2a3683e237eba/frontend/src/libraries/explorer-wamp/transactions.ts#L175-L179
[x] "Ghost" accounts issue #223 is resolved
[x] "Ghost" access keys should not be the case if we implement #162 first

@frol do you have any ideas what should we do for receipt table?

Just for the context (it took me a few moments to recall what kind of "table" we are talking about here) it is a database table storing the receipts.
I have not dug into it before, so let me start with the facts about the current state.
There are receipt properties:

predecessor_id (it could be either Transaction signer_id or intermediate contract's account_id)
receiver_id (a receipt destination)
receipt_id: (a unique id for the receipt; it is a hash)
receipt_type one of:

action

signer_id
signer_public_key
gas_price
output_data_receivers is a list of:

data_id (a unique id for the data; it is a hash)
receiver_id
input_data_ids (a list of hashes)
actions a list of Actions (the same as in transactions)

data

data_id (a unique id for the data; it is a hash)
data (just a blob of data)

Currently, we can only query them with the chunks RPC API (block # 2871883 and shard # 7):
$ http post https://rpc.nearprotocol.com jsonrpc=2.0 id=dontcare method=chunk 'params:=[[2871883, 7]]'

The output:
{
    "id": "dontcare",
    "jsonrpc": "2.0",
    "result": {
        "header": {
            "balance_burnt": "0",
            "chunk_hash": "Gx7CRg8goc6m3GBMyeufgSLaqkhH5HG2qEhxEHFYPW6J",
            "encoded_length": 191,
            "encoded_merkle_root": "ExpzxZNGvjMyRMphvDYY38XqD4rqJUiweDoXN6f8bQTY",
            "gas_limit":<PHONE_NUMBER>000000,
            "gas_used":<PHONE_NUMBER>00,
            "height_created": 2871883,
            "height_included": 0,
            "outcome_root": "GYEAqSdzvjQZfd9KEF6LCiza5SjPAgsqYRKciGaMUd4s",
            "outgoing_receipts_root": "5w2Cx8paWdCV4TCCwoMcc3NTpgAeBvNoGjCkGkW1pHqX",
            "prev_block_hash": "gSchiKdxvH7MtG9zqZsXLpTUpVRBRUUosqz59n1T6PB",
            "prev_state_root": "GqRtmo1upVD94xt3od9ak9goULxwAQbDrTjCou7VXo3j",
            "rent_paid": "1178834",
            "shard_id": 7,
            "signature": "ed25519:4fqeabHFfgGdV6m3ja5z42maWZ6dqnpqXeNekz2fEcCyUMGj4HvB1AA5YzbwR8AhyWeQherdLrcram78D5monp11",
            "tx_root": "11111111111111111111111111111111",
            "validator_proposals": [],
            "validator_reward": "4685722500000000"
        },
        "receipts": [
            {
                "predecessor_id": "test",
                "receipt": {
                    "Action": {
                        "actions": [
                            "CreateAccount",
                            {
                                "Transfer": {
                                    "deposit": "10000001000000000000000000"
                                }
                            },
                            {
                                "AddKey": {
                                    "access_key": {
                                        "nonce": 0,
                                        "permission": "FullAccess"
                                    },
                                    "public_key": "ed25519:36Gmm6qJ4a78qTknvqLUPXoGp5KLeAvDDTM194ngkFua"
                                }
                            }
                        ],
                        "gas_price": "5000",
                        "input_data_ids": [],
                        "output_data_receivers": [],
                        "signer_id": "test",
                        "signer_public_key": "ed25519:oNCFEmRotRHTySKqmAwifNZ8VRpYS973p8cL61y7eiE"
                    }
                },
                "receipt_id": "AhbJN8J4Nk5aFxUptkkrJ5EJWsTiEQtjBTHvGt9sPFzW",
                "receiver_id": "victorvw"
            }
        ],
        "transactions": []
    }
}

Also, there is a receipt outcome (same as we have for transactions):

block_hash (the block where this receipt was included)

gas_burnt

logs (a list of strings)

receipt_ids (a list of follow-up receipt ids)

status - it is an enum, but it is serialized as an object with the following possible keys:

Failure
SuccessValue (the final action succeeded and returned some value or an empty string; the value is the value encoded as base64)
SuccessReceiptId (the final action of the receipt returned a promise; the value is the receipt_id of the generated receipt)

proof, example:
"proof": [
              {
                  "direction": "Right",
                  "hash": "45LBFtJSxFcVSWiiamY11GZ9nq1Cmcfc5xueD49GZDZz"
              }
          ]

We get the receipt outcomes with the tx RPC API:
$ http post https://rpc.nearprotocol.com jsonrpc=2.0 id=dontcare method=tx 'params:=["EzvvJEqdxKA62oAjG32y5herDtRGYEdmZXHDDyxorJ48", "victorvw"]'

The output:
{
    "id": "dontcare",
    "jsonrpc": "2.0",
    "result": {
        "receipts_outcome": [
            {
                "block_hash": "9YGAYkA4mhafu1RWFKpQvVJMWdFeaCM6fW6Jqair2h52",
                "id": "AhbJN8J4Nk5aFxUptkkrJ5EJWsTiEQtjBTHvGt9sPFzW",
                "outcome": {
                    "gas_burnt":<PHONE_NUMBER>00,
                    "logs": [],
                    "receipt_ids": [],
                    "status": {
                        "SuccessValue": ""
                    }
                },
                "proof": []
            }
        ],
        "status": {
            "SuccessValue": ""
        },
        "transaction": {
            "actions": [
                "CreateAccount",
                {
                    "Transfer": {
                        "deposit": "10000001000000000000000000"
                    }
                },
                {
                    "AddKey": {
                        "access_key": {
                            "nonce": 0,
                            "permission": "FullAccess"
                        },
                        "public_key": "ed25519:36Gmm6qJ4a78qTknvqLUPXoGp5KLeAvDDTM194ngkFua"
                    }
                }
            ],
            "hash": "EzvvJEqdxKA62oAjG32y5herDtRGYEdmZXHDDyxorJ48",
            "nonce": 10270,
            "public_key": "ed25519:oNCFEmRotRHTySKqmAwifNZ8VRpYS973p8cL61y7eiE",
            "receiver_id": "victorvw",
            "signature": "ed25519:46WsywQYe31isWcHASrnR2pYCja4Mtjzs4n87isrTHWyDdX5uwrUaUn4SbhY9BMwWKvWwdcuyCjMsXdkXdPJv8ko",
            "signer_id": "test"
        },
        "transaction_outcome": {
            "block_hash": "gSchiKdxvH7MtG9zqZsXLpTUpVRBRUUosqz59n1T6PB",
            "id": "EzvvJEqdxKA62oAjG32y5herDtRGYEdmZXHDDyxorJ48",
            "outcome": {
                "gas_burnt":<PHONE_NUMBER>00,
                "logs": [],
                "receipt_ids": [
                    "AhbJN8J4Nk5aFxUptkkrJ5EJWsTiEQtjBTHvGt9sPFzW"
                ],
                "status": {
                    "SuccessReceiptId": "AhbJN8J4Nk5aFxUptkkrJ5EJWsTiEQtjBTHvGt9sPFzW"
                }
            },
            "proof": []
        }
    }
}

To be continued...

I would start with what users may want to know:
Q: Which transactions (including cross-contract calls) touched the given account?
A: To know that, we need to store predecessor_id and receiver_id, and a reference to the transaction (transaction_hash).
Q: What is the execution "graph" of the receipts?
A: Given that a single receipt can produce many new receipts and then gather them together, we need some way to know the parent/child relationship, and instead of having a list of child receipts, we can have a parent_receipt_id (NULL for the first receipt), as it just how we normalize the data.
Q: What are the input and the result of the receipt?
A: Input is an action (the same type as we have for transactions), and the result is the outcome status. We should take some time thinking about these receipt actions: can/should we reuse the transaction actions here?
Q: What went wrong with the execution?
A: In addition to the input and the results, we also need logs and intermediate values (arguments and return values). I would prefer not to store those in Explorer DB, so we just re-query that on the Transaction Details page on demand (e.g. when a user digs into the receipt details [NOTE: we need RPC API for that]).
It needs some better thought, but it is a starting point. I would like to invite more people to express their opinions on what they want to see on Explorer in regards to the receipts /cc @fckt @k06a @willemneal @AnaisUrlichs @ilblackdragon @bowenwang1996 @Kouprin @janedegtiareva @vgrichina

The core of this issue is resolved with #236.
Transactions that are touching a given account are now properly displayed on the Account details page as of #506.
The UI for Receipts on Transaction details page is tracked under #512.

The core of this issue is resolved with #236.
Transactions that are touching a given account are now properly displayed on the Account details page as of #506.
The UI for Receipts on Transaction details page is tracked under #512.


Question: How to programmatically add binding behaviours to view model

I'm submitting a question.
Library Version:
0.26.1
Operating System:
Windows 10
Node Version:
6.10.0
NPM Version:
3.10.8
Browser:
all
Language:
all
I have a created custom element which is going to be used multiple times, on one occasion the custom element's click event needs to be throttled by 1 second. How would I go about doing this outside of the view?
Can't seem to find any documentation. Everything I've read says I'd need to add  & throttle:1000 to the custom element's click.delegate attribute, but obviously I want this to be dynamic and set via the view-model.
Thanks!

You can always write a simple throttle function in JS. Please ask questions on Gitter or SO.

@EisenbergEffect Fair enough for closing, but was just hoping that it would be exposed in your API/documentation. Would rather do it the aurelia way, considering you've already written it.


Can't build or use in VS2019

I spent about 30 min trying to get the dependencies fixed up but had to give up.
I'd really love to give this a deeper look if you have the time to update it to build  under VS2019 (and use dotnetcore3.1 while you're at it ).

What problems are you having? I'm running VS 2019 (latest build) and after a fresh pull of master, I did:

Used nuget restore instead of dotnet restore because some of the test projects are using older frameworks AFAICT. Now build:

I'm also running .net core 3.1

I actually meant to put this issue here: https://github.com/darrenstarr/VtNetCore.UWP
My bad. Closing and opening https://github.com/darrenstarr/VtNetCore.UWP/issues/16


Bug: Addresses in logs are not checksummed

Log addresses are lowercased but they should be checksummed

Are you open to contributors?

Absolutely!

Great
Would like to take this on, however I can't view the issue on Linear

The issue is just mirrored on linear any comments here will show up there and vice versa

All yours @varun-doshi . Jump into the telegram if you want to chat about anything or need me to provide any guidance.
For context, addresses can be checksummed using createAddress(hexAddress).toString(). The logs we care about are logs returned by callHandler.js and ReceiptsManager.ts.

Understood
I'll get started

@varun-doshi any progress so far?

Apologies for the delay.
Please let me know if I've missed something


Teacher Spirit

Since September 2022 with WhatsApp ad Facebook where you go to finding a job or stage with professional proof for students.

https://github.com/parasoft/execute-job-action/assets/87576718/28ead4fd-a4f4-4572-84de-1734cf1fe7eb

Rich event for my new school year.


Text Selection not complete

Link to PDF file (or attach file here):
answer.pdf
Configuration:

Web browser and its version: Chrome
Operating system and its version: Mac
PDF.js version: online as of today
Is an extension: No

Steps to reproduce the problem:

select and copy any text that contain the little circled numbers
paste to see the result: little numbers are missing from the pasted text

What is the expected behavior? (add screenshot)
Correct text should include the numbers, but the string is missing them
What went wrong? (add screenshot)
The numbers are displayed later in the PDF, and the string around uses spaces, so visually the line looks complete, but we need to merge strings. PDFBox works well on this PDF.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

FWIW Adobe Reader does not copy and paste the text with ① as well.

Likely fixed thanks to https://github.com/mozilla/pdf.js/pull/13257.

Unfortunately this is still not fixed, however the description of the bug is quite unclear since this refers to one particular line; please try copying the outlined sentence:

Correct, I checked with the first circled 1.
Anyway, the circled numbers are drawn only at the end of the page: they aren't in the normal flow (I mean from a left-to-right-top-to-bottom pov).

Looking at this issue once more, and noticing that it's similarly broken in both Adobe Reader (i.e. the PDF reference implementation) and PDFium (as shipped in Google Chrome) the conclusion is that it's a bug in the PDF document itself; all-in-all let's close this as WONTFIX instead.


Experimental: Move path to constructor; improve path processing

This is an experimental branch that moves the path declaration from an overridden property to a super constructor argument.
What this does is gives us the ability to process the supplied path value and ensure it is formatted correctly when it is accessed, both via the path property and the (unchanged) fullPath property.
The downside of this is that we lose the const constructors and adding more processing opens up more opportunity for bugs.

Closing; will not integrate.


⚠ Samsung has degraded performance

In 24a0ea4, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 5749 ms

Resolved: Samsung performance has improved in 575effe.


PBR "basic" subtype (and its successor) has broken

It's not specific to the old pbr.basic rendertype - simply put, any PBR material that sets the new NoMetallicRoughnessMap flag true is always drawn fully reflective as if it had a Roughness value of 0, despite what the value actually is. (changing it doesn't do anything)

Thanks for testing and reporting it! I'll be sure to look into it as soon as possible.

Thanks for testing and reporting it! I'll be sure to look into it as soon as possible.

v1.4.0-preview.8 contains a fix for this. Let me know if you come across any other issues like it!

v1.4.0-preview.8 contains a fix for this. Let me know if you come across any other issues like it!


Should be a more visible link to Opptak.online from splash

What kind of an issue is this?

Feature request x

Closing this as it belongs in the glowing-fortnight repo :upside_down_face:


MIPNodeCtx::add_lazy() is not working correctly?

edit: I am currently trying to find a minimal example... maybe this gets me down to the cause of my problem and I'll close this issue. Ignore this as long as I did not remove this mention.
Hi,
it seems that MIPNodeCtx::add_lazy() is not working correctly. In my original code it seems to simply ignore the constraints added. Note that MIPSolCtx::add_lazy() works as expected.
edit: I had a minimal example here which did not work for other reasons. I have not made another minimal example, but I guess the TSP example could work. Just add something like
    if let Where::MIPNode(ctx) = w {
        ctx.add_lazy(c!(/* XXX */))?;
    }

where /* XXX */ should be replaced by a constraint that makes the model infeasible, like one variable >= 2 ...
However, the model will still have the same objective value as without it although it is expected to be infeasible.
Tested with Gurobi 9.1.2 and grb 1.0.4,
not tested if the issue also occurs using C code.

The issue is clearly related to Gurobi and not to rust-grb. I have found out that with slightly different inputs, the lazy constraints are not ignored. Sorry for the noise here.

I've released a new version that fixes a bug in the user cuts (not lazy constraints).
From Gurobi's manual,
Your callback should be prepared to cut off solutions that violate any of your lazy constraints, including those that have already been added. Node solutions will usually respect previously added lazy constraints, but not always.
I think the issue here was that Gurobi assumes that lazy constraints added at MIPNODE and MIPSOL are consistent.  For example, if at MIPNODE you add a constraint which makes the model infeasible, then you may need to add this same constraint at MIPSOL again.  I don't quite understand why (perhaps because of synchronization difficulties between threads processing MIP nodes in parallel), but I've observed Gurobi treat MIPNODE lazy constraints more as "suggestions".


Fix error message in JsonAdaptedReview

The error message in JsonAdaptedReview is actually of JsonAdaptedStall, which may be confusing to the user.


feat: replace private token in testing

Fixes #2301.
Replaces the private token in most testing. Not altering docs and some of the cli_docs testing.
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

@LHerskind do you know if the work allowing for public calls from constructor has been mapped out?

Not sure, maybe @iAmMichaelConnor and @PhilWindle started on it?

@LHerskind do you know if the work allowing for public calls from constructor has been mapped out?

Not sure, maybe @iAmMichaelConnor and @PhilWindle started on it?

There's just a github issue at the moment, sitting there for someone to pick up and discuss, then build, when they have time. https://github.com/AztecProtocol/aztec-packages/issues/2249


What is the mean of prediction's result?

bst.predict(test)
I have three class labels. 0, 1, 2
I want to get labels like 0,1,2 rather than below result.
Thank you!
the result is:
[ 2.08266497 -0.16195059 -0.53449929]
[ 2.08266497 -1.48556077  0.13058886]
[ 1.32865083 -2.025424   -0.13586384]
[-0.55171859 -1.54363155  1.26611865]
[ 1.32865083 -2.025424   -0.13586384]
[ 2.08266497 -0.70571303 -0.8340435 ]
[ 1.3542099  -2.025424   -0.53449929]
[ 2.08266497 -0.99530995 -0.8340435 ]
[ 2.08266497 -0.99530995 -0.8340435 ]
[ 3.17738867 -1.54363155  0.19758049]
[ 2.08266497 -2.025424   -0.53449929]
[ 1.57963073 -1.00376821 -0.53449929]
[ 1.3542099  -1.27166677 -0.53449929]
[ 2.62930393 -2.025424   -0.0407176 ]
[ 2.62930393 -2.025424   -0.8340435 ]
[ 2.08266497 -2.025424    0.13058886]
[ 1.52446699 -2.18625474  0.75243491]
[ 2.01449347 -1.27166677  0.07861483]
....

If you want to get the label use multi:softmax as your objective, see https://github.com/dmlc/xgboost/blob/master/demo/multiclass_classification/train.py

thanks! But, I run the program each time, the result is always changed. why? In order to get the best result?

There could be indeterministic bahvior due to multithreading and float point rounding, see #9


[SPARK-37181][PYTHON] pyspark.pandas.read_csv() should support latin-1 encoding

What changes were proposed in this pull request?
pyspark pandas will read csv with latin-1 encoding.
from pyspark import pandas as ps
ps.read_csv("<>", encoding="latin-1")

Why are the changes needed?
Earlier it throws "pyspark.sql.utils.illegalArgumentException: latin-1" whereas pandas read csv is able to read "latin-1" encoding.
In order to have the same behavior of pd.read_csv and ps.read_csv this change is needed.
Does this PR introduce any user-facing change?
Yes user will be able to provide latin-1 encoding with pyspark pandas
How was this patch tested?
unit tests

Can one of the admins verify this patch?

Jenkins test this please

Test build #145287 has started for PR 34619 at commit 1039257.

Test build #145287 has finished for PR 34619 at commit 1039257.

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

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

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49757/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49757/

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

CSV supports encoding that is only ascii compatible due to newline parsing. It does support other encodings with multiline on. I think it's fine to go ahead.

Oh, that;s a very good point @Yikun

@HyukjinKwon If you are ok with it , can we please merge it

Okay, let's go ahead as is for now

Merged to master.

Thx @HyukjinKwon @Yikun . For reviewing


Bug java-udf function open failed cause BE core dump

Proposed changes
when the java-udf open function failed, and some JNI have not set,
so in close function can't call jni.
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Typo fix: add implied "not"

@nfd9001 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


fix: linting, license dates, and import statements in react

Leaving as it is

I'll open another PR!

There we go, PTAL

Do you have any other open issues I can help with?

@evenstensberg awesome!! Feel free to grab #567 and then anything labeled with "good first issue" would make sense as well; you will probably want to also take a look at the Development Environment portion of the docs.

@evenstensberg on a second thought #129 maybe a lower hanging fruit

Thx! I'll get back to you once I have time!


Question about baseline and vertical alignment of caps/numbers

Hello,
I've been using agave for a few weeks and I'm quite enjoying it. Thank you for this beautiful work.
One thing that's been bothering me is the unusual treatment of baseline (it's higher that in other typefaces) and how it affects the vertical alignment of caps and numbers.
I'm pretty sure that it's by design, so I'm curious what led you to this decision and if there are any drawbacks of having a more traditional baseline.
Here's a comparison with Cascadia Code and PragmataPro to illustrate what I mean.
In the first 2 typefaces the baseline is lower, so caps and numbers are vertically centered across the line height.
In agave, the baseline is noticeable higher, so caps and numbers appear "off-center" vertically.

P.S. I've tried adjusting the vertical offset of characters in my terminal. While it does make the characters centered vertically, it also breaks the alignment of box-drawing characters, so I had to revert this change back to default.

Thanks for explanation!
Yeah I can see now how this solution lets you maximize the x-height while keeping the line-height as compact as possible. This "readable compactness" is the reason I love this typeface in the first place, so I think I can live with non-standard baseline :)
Thanks again, closing this.


[QUESTION] Performance Issue Caused by PreparedStatement varchar(8000) Parameter Type

Question
I have a query that yields consistent response times in the magnitude of tens of milliseconds when run as a SQL batch, e.g. from Azure Data Studio. However, as soon as I run the exact same query as a parameterized prepared statement from a Java application the query performance tanks completely (10s of seconds!) for certain parameter value sets.
The Azure Data Studio profiler plugin shows that the JDBC driver prepares the statement with a varchar(8000) parameter type:
exec sp_prepexec @p1 output, N'@P1 varchar(8000)', N'select ... where tx_type=@P1', 'CREDIT_REQUEST'

If I copy the sp_prepexec event text from the profiler view and execute it in Azure Data Studio I can reproduce the performance issue. As soon as I change the parameter data type from varchar(8000) to varchar(30), which is the correct datatype of that column, performance is again consistent and in line with the performance of the SQL batch.
My question is: How can I achieve consistent performance with a prepared statement? Either by nudging the JDBC driver to prepare the statement with the correct data type for the parameter or by any other means?
The Java application uses JPA with Hibernate as the JPA implementation.
Versions:
Hibernate: 6.4.1 (SpringBoot 3.2.1)
MSSQL JDBC driver: 12.3.0.jre17-preview (seems to be the only version that fixes this other issue)
MSSQL Server: mssql/server:2022-latest (Docker image for local tests)

Thanks for your question.
We are looking into the reported problem. At this point in our analysis we suspect that there may be something unexpected happening on the server side, but we do not have any empirical evidence yet in our investigation.
Could you please share your code exhibiting the problem?

Additionally, at this time we recommend that you specify the correct data type size to ensure consistent performance levels until we can complete our investigation.

hi @rsteppac
This is a similar issue as #1826 in that the driver does not know what the length of the column unless it makes a query to the server which will also impact performance, so it just uses the max (8k) value. To workaround this you could explicitly specify the precision of the column with something like
setObject(1, "30 precision", java.sql.Types.VARCHAR, 30);

Could you please share your code exhibiting the problem?

@barryw-mssql you mean the full SQL statement? I probably cannot (corp. policy). What I can say is that the query left joins 5 tables and inner joins a sub-query. The sub-query again left-joins 5 tables, some but not all are the same as in the outer query, and returns the value of a computed column. The query uses the "offset-fetch" windowing function to always retrieve the first window of N records.
The performance degradation is reproducible if the WHERE clause contains a single criteria on a column with very low cardinality with almost all or even all records matching one value and very few or no records match one of a handful of other values and the search value is one that yields an empty result set.
We have identified the windowing function as being part of the problem. If the windowing function is removed then execution times for WHERE criteria that yield an empty result set are generally more consistent with other combinations of WHERE criteria that do yield a result. But with one WHERE criteria in particular the inconsistent behavior is only exhibited if the query is run as a parameterized prepared statement and the parameter type does not match the column definition.

To workaround this you could explicitly specify the precision of the column with something like setObject(1, "30 precision", java.sql.Types.VARCHAR, 30);

@lilgreenbird do you happen to have a pointer to a resource describing how I can get Hibernate to call the prepared statement with setObject the way you described it?

hi @rsteppac
No we are not familiar with Hibernate you will have to reach out to their support team for anything relating to Hibernate.
We also have some more suggestions that may help with your performance issue.
You should take a look at the execution plan your queries generate and try to tune your database to execute them more efficiently. We would also suggest setting prepareMethod=sp_prepare in your connection string as that would result in a separate initial trip to the database to prepare the statement without any initial values for the database to consider in the execution plan.

@rsteppac
Please let us know if this helped to resolve your issue and/or if you have more questions? We will be closing this soon otherwise.

@lilgreenbird , you can close the ticket. Thank you.


Working on getting phoney happy with ruby 1.9.3

Phoney uses a YAML syntax that ruby-1.9.x can't parse. Attached is a commit that expands the inline hashes to the long form that both 1.8.x and 1.9.x can parse.
I also took a stab at modernizing the build plumbing so I could try to run the test suite in 1.9.x. I didn't get all the way there, alas, I don't know that the old version of rspec that seems to be required is going to be happy with ruby-1.9.x. I'll see about taking another stab at it once rspec has declared itself to be 1.9.3-friendly.

Thanks for pulling. Any chance you could push a new version to gemcutter?

OK!


Support output to console for FileExporter

In runtimes like Beanstalk, /dev/stdout or /dev/stderr may not be available, but there should be "some place" where console.log goes. We should introduce special values like console:log and console.error that will use the respective methods to print out the spans.

:tada: This issue has been resolved in version 1.22.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Repeated Liquidate BUG

We found a big bug in the recent use process. Clicking the liquidation button once will lead to the issuance of more Prize. At present, the analysis results and preliminary solutions have been obtained.
Reproduction process:

DynamoDB uses a pre-made billing model, 1-10. If the AWS performance scheduling is not timely, the request will fail.
Since DynamoDB does not have a batch update function, a for loop is set in the program to update the red envelopes. Multiple write requests will be generated in a short period of time.
Due to the above reasons, the write request will fail, and if the robot fails, it will be repeated, resulting in multiple transfer transactions.

solution:

DynamoDB enables on-demand mode.
Optimize the structure of DynamoDB and use batch delete instead of for loop to update one by one.
The robot only generates a settlement slip, and the maximum number of transactions allowed per block is packaged at a time. Batch settlement is handled by another Lambda function.

Current program:

Database billing uses on-demand mode;
When the Prize is settled and exported, turn on ttl automatic deletion, and set the retention time to 7 days.


GhidraScript to label model-specific PIC Special Function Registers

There are many different processor models of the PIC 16-bit microcontroller, each with their own specification of RAM-mapped Special Function Registers.  The Ghidra PIC processor module attempts to define some of the most common ones at the beginning of RAM, but it is far from complete (and impossible to do generically).
This PR introduces a GhidraScript that:

Searches the Microchip xc16 compiler support directory for model-specific Linker Script files (requires the free compiler to be installed)
Asks the user to choose a processor model
Creates labels for all of the defined Special Function Registers for the selected model

Should the GhidraScript be changed to make SFR labels are pinned?

Should the GhidraScript be changed to make SFR labels are pinned?

Closing due to inactivity.  The script is available to those who want it, but there's not a need to include it in a Ghidra release.


[HUDI-817] fixed building IndexFileFilter with a wrong condition in Hood…

#672  What is the purpose of the pull request
fixed bug, when building IndexFileFilter with a wrong condition in HoodieGlobalBloomIndex class
Brief change log
use a wrong config parameter in build IndexFileFilter in HoodieGlobalBloomIndex  class
Verify this pull request
No validation required
Committer checklist

thanks for the catch @Jecarm ..
@nsivabalan can you please review this. This deserves a JIRA since its an actual bug fix.. (performance should improve, correctness should be the same)

LGTM.

Nice cathc @Jecarm 👍


leader steps down when followers' disks are slow

this is related to #200. in my saturated disk testing, i was able to keep a leader around longer by setting the election timeout after writing to disk instead of before. that got around the issue of spurious leader elections being proposed by followers.
it also opened the door to a second situation, one in which there is a stable leader but sometimes no followers can respond to appendEntries in time for the leader to not step down in stepDownThreadMain. the assertion of this ticket is that a log cabin leader should rely on the discovery of a new leader to step down, and not a timeout.
i'm not sure the assertion of this ticket is correct, but i thought i'd file it to find out.

See the bulletted list in "6.2 Routing requests to the leader" in my dissertation for an explanation of why it's there. In short, an isolated leader shouldn't hold up client requests forever.

Ok, makes sense. Will close this ticket.

One thing you could do to mitigate this issue is increase the timeout for when the leader steps down. It's currently set to ELECTION_TIMEOUT (not configurable). I wouldn't go much above 2 * ELECTION_TIMEOUT or clients would be delayed a long while, but maybe that'd help the issue? It's an easy change to try out, and if it turns out to be helpful, it wouldn't be difficult to make that configurable.
I'm gonna rename this GitHub issue based on the symptom now.

Extending the step down timer is easy to try and it makes sense that should help. I'll do that now.
Not sure if this is relevant to the general case, but under common failure scenarios in my particular setup, a client that could connect to a leader that is segmented from the network would also be segmented from the network, so its progress would be inhibited regardless of the leader stepping down.

Not sure if this is relevant to the general case, but under common failure scenarios in my particular setup, a client that could connect to a leader that is segmented from the network would also be segmented from the network, so its progress would be inhibited regardless of the leader stepping down.

Ah, great point. I think if you were 100% confident in that statement, you could disable the step down thread entirely (or set a timeout of infinity). Though maybe a large timeout would be a wiser choice in case there's some unexpected wedging anywhere.

for reference, i have this set to 12x election timeout right now. seems to be helping with slow disks and failover time is within my acceptable range.

Not sure if this is relevant to the general case, but under common failure scenarios in my particular setup, a client that could connect to a leader that is segmented from the network would also be segmented from the network, so its progress would be inhibited regardless of the leader stepping down.

Ah, great point. I think if you were 100% confident in that statement, you could disable the step down thread entirely (or set a timeout of infinity). Though maybe a large timeout would be a wiser choice in case there's some unexpected wedging anywhere.

Hmm, I hadn't considered this when I wrote my earlier comment: what if machine1 can't talk to a majority of the cluster but can talk to machine2, and machine2 can talk to all of the others. And let's say machine1:server is a deposed leader and machine2:server is the current leader. In this case, machine1:client would get service if it talked to machine2:server, but without machine1:server stepping down, it could get stuck waiting on machine1:server.
This is a bit contrived, and it seems fairly unlikely with a single switch between all the machines. So you might be ok with it, especially given the moderate outage (12x election timeout). Still, I thought I'd bring it up.

I think we can set a timeout in Peer::callRPC as RPC_FAILURE_BACKOFF.
When appendEntries timeout. The LEADER could send a special message to the timeout FOLOWER to check whether the communication is ok. When the FOLOWER receive the special ack communication message, it should setElectionTimer adn update the withholdVotesUntil without lock then response. If the LEADER received the response, it could update the peer.lastAckEpoch and keep leadership.
I think it may solve the IO slow issue and does not increase the leader switch time when leader node is failed.


Change from using data to data_allocated in checkpointing

This PR:

Changes the the c contiguous test in tests/test_data.py to use data_allocated and to check for a space_order > 0 to really test the condition that is likely to fail.
Changes the checkpoint API with pyrevolve to use data_allocated instead of data.

Note: ~This PR includes #925 so merge that first.~
TODO: Remove class attributes like CC etc from CustomCompiler

Codecov Report

Merging #926 into master will increase coverage by 0.99%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master     #926      +/-   ##
==========================================
+ Coverage   88.07%   89.06%   +0.99%
==========================================
  Files         141      148       +7
  Lines       18923    20247    +1324
  Branches     2894     3204     +310
==========================================
+ Hits        16666    18034    +1368
+ Misses       1907     1839      -68
- Partials      350      374      +24

Impacted Files
Coverage Δ

devito/compiler.py
48.98% <0%> (+6.07%)
:arrow_up:

devito/types/dense.py
85.21% <100%> (-7.69%)
:arrow_down:

tests/test_data.py
97.44% <100%> (ø)
:arrow_up:

examples/checkpointing/checkpoint.py
100% <100%> (ø)
:arrow_up:

devito/types/sparse.py
71.67% <0%> (-22.72%)
:arrow_down:

devito/types/basic.py
89.87% <0%> (-5.35%)
:arrow_down:

devito/yask/transformer.py
92.06% <0%> (ø)

devito/yask/utils.py
100% <0%> (ø)

devito/yask/types.py
91.55% <0%> (ø)

... and 24 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 4921adf...1867fa4. Read the comment docs.

@navjotk I'm happy to land this, but I wanted to ask a question first: can you remind me why you want/need the *_allocated  elements, as opposed to just the domain elements ? I'd have expected one checkpoints only the domain region, not the whole array

The performance hit from dealing with non-contiguous memory is so big (introduces >1 copy), it's better to checkpoint stuff you don't need just to get a contiguous chunk of memory.

Ah! This is a very good point indeed. Would you mind adding a comment?
nitpicking: would you also mind moving def size_allocated right after def size ?

I will do those nitpicks. Thanks, merging

nitpicking: would you also mind moving def size_allocated right after def size ?

sry, hadn't noticed size_allocated is in dense.py, while size is in basic.py


When I access tapir through the domain name, I cannot create the deployKey.

When I access tapir through the domain name, I cannot create the deployKey, but I can create it directly through ip+port.
An error occurred as a cross-domain request was rejected.

The same error happens when Tapir runs on a container in AWS ECS behind load balancer.
An HTTP sessions looks like this:
POST /management/deploykey/infrahouse-cloud-init-aws HTTP/1.1
X-Forwarded-For: <IP_ADDRESS>
X-Forwarded-Proto: https
X-Forwarded-Port: 443
Host: registry.infrahouse.com
X-Amzn-Trace-Id: Root=1-65e4db15-5b1e54bb5c7bc9b33898a19a
Content-Length: 0
user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:123.0) Gecko/20100101 Firefox/123.0
accept: */*
accept-language: en-US,en;q=0.5
accept-encoding: gzip, deflate, br
referer: https://registry.infrahouse.com/management
origin: https://registry.infrahouse.com
dnt: 1
sec-fetch-dest: empty
sec-fetch-mode: cors
sec-fetch-site: same-origin

HTTP/1.1 403 CORS Rejected - Invalid origin
content-length: 0


HTTP: body parts are added in reverse order

The following code:
exec(http("post")
  .post("/url")
  .bodyPart(ByteArrayBodyPart("name1", body1))
  .bodyPart(ByteArrayBodyPart("name2", body2)))

will create HTTP request that contains body2 followed by body1. This makes constructing HTTP requests with the predefined body parts order confusing.
This should be fixed here: https://github.com/gatling/gatling/blob/master/gatling-http/src/main/scala/io/gatling/http/request/builder/HttpRequestBuilder.scala#L92

@kostya-sh Thanks for reporting!


Feature Request: Graphics Card monitoring

Hi,
Thanks for developing this wonderful conky config that just works. Wondering if there is any plan on adding the graphic card monitoring to it?
Thanks

Support for Nvidia GPUs is uber development, please stay tuned, thanks.

Available in the latest release 0.8.0, enjoy.

Supported with v0.8.0.


boost::network::http::client::request error when constructing with string

After quite some time I found this bug (?) or maybe its intended.
If I build a boost::network::http::client::request with a string like this
boost::network::http::client::request request(std::string("test"));
I get the following error
terminate called after throwing an instance of 'std::length_error'
what():  basic_string::_S_create
I had a look at the constructor of request and indeed it expects a constant reference.
request::request(std::string const& url)
Why is it implemented like this?

On 9 Oct 2015, at 12:15 AM, Giacomo Dabisias<EMAIL_ADDRESS>wrote:
After quite some time I found this bug (?) or maybe its intended.
If I build a boost::network::http::client::request with a string like this
boost::network::http::client::request request(std::string("test"));
I get the following error
terminate called after throwing an instance of 'std::length_error'
what(): basic_string::_S_create
I had a look at the constructor of request and indeed it expects a constant reference.
request::request(std::string const& url)
Why is it implemented like this? This way if I change at any point the passed url it won't work. It should make a copy of the url as is it passed or at some point during the construction.

This is implemented this way so that you can pass in a literal and it should just work.
The best way to do what you want I think is to create a request object, then use the destination modifier later on.
See http://cpp-netlib.org/0.11.2/reference/http_request.html#directives http://cpp-netlib.org/0.11.2/reference/http_request.html#directives for more details on how to use the destination directive or even the destination modifier.


feat(tool): Add linter in devcontainer config

What does this PR do?
Add linter configuration in the generated vscode devcontainer configuration file. It will allow to get linter warning error directly on the files that you are modifying, and it use the linter configuration provided in this repository.
Thanks @pgimalac for the configuration example.
This PR also add a post start configuration to setup configure git and download the project dependencies.
Motivation
Improve dev experience by providing linter information corresponding to the CI linter configuration directly in the code editor.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


Fixing the moon phase calculation

The moon phase calculation was quite poor - this one gives much better results.

Great - thanks!


Second slide doesn't appear

Looked in community forum, seems file was created in wrong folder - automatically by entering the code given by the tutorial.

https://github.community/t5/GitHub-Learning-Lab/Intro-course-completed-but-new-slide-didn-t-appear/td-p/18928

I have fixed this, but I should have created a new branch to add the new file first


[Feature Request] Auto disable on updated browsers

Chrome 32+ already disables click delay if <meta name="viewport" content="width=device-width"> has width <= device-width.
Other browsers have support for CSS touch-action, so this simple CSS would be enough:
a[href], button {
    touch-action: manipulation;
}

(please apply auto-prefixer later)
More info here: http://www.telerik.com/blogs/what-exactly-is.....-the-300ms-click-delay

In #42 which I'll be releasing soon (with a couple of other small changes), you'll be able to choose whether to initialize react-fastclick or not yourself. Would that solve this issue for you?


Always supply --stdin-filepath to Prettier

This is required for Prettier to pick up the correct settings from its config files.

Added.


Improve the way to query fast exit data

The transaction monitor checks the status of fast cdm when it finds it regardless of the estimated finalized time. It increases the API call, but it can provide a better user experience. closed #503

Codecov Report

Merging #507 (9dbf1db) into develop (443349d) will increase coverage by 9.95%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #507      +/-   ##
===========================================
+ Coverage    76.09%   86.05%   +9.95%
===========================================
  Files           81       49      -32
  Lines         3037     1936    -1101
  Branches       469      307     -162
===========================================
- Hits          2311     1666     -645
+ Misses         726      270     -456

Flag
Coverage Δ

batch-submitter
?

contracts
86.05% <ø> (ø)

core-utils
?

data-transport-layer
?

message-relayer
?

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

Impacted Files
Coverage Δ

...ckages/data-transport-layer/src/utils/contracts.ts

...ices/l1-ingestion/handlers/state-batch-appended.ts

packages/data-transport-layer/src/utils/eth-tx.ts

packages/data-transport-layer/src/utils/common.ts

packages/core-utils/src/common/index.ts

packages/core-utils/src/common/misc.ts

...ubmitter/src/batch-submitter/tx-batch-submitter.ts

packages/core-utils/src/coders/index.ts

packages/batch-submitter/hardhat.config.ts

...h-submitter/src/batch-submitter/batch-submitter.ts

... and 22 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 443349d...9dbf1db. Read the comment docs.


[Bug] Displayed S3 URI is incorrect

Description of Bug:
The Browse feature of the data catalog displays the wrong S3 URI for files.
Steps to Reproduce:

Go to https://source.coop/cworthy/oae-efficiency-atlas/data/experiments/000/01/alk-forcing.000-1999-01.pop.h.0347-01.nc
See that the S3 URI says s3://cworthy/oae-efficiency-atlas/oae-efficiency-atlas/data/experiments/000/01/alk-forcing.000-1999-01.pop.h.0347-01.nc
Realise that that is not a real bucket - this data file actually lives at s3://us-west-2.opendata.source.coop/cworthy/oae-efficiency-atlas/oae-efficiency-atlas/data/experiments/000/01/alk-forcing.000-1999-01.pop.h.0347-01.nc. The prefix is wrong.

Expected Behavior:
Display the correct S3 URI.

@TomNicholas I think its by design that since the data proxy is abstracting the real bucket involved as this s3 url is only meant to be understood by data proxy.

Thanks for the input @navdeepg2021.

the data proxy is abstracting the real bucket involved. This s3 url is only meant to be understood by data proxy.

What if I want to access in a way that requires knowing the actual bucket url though? Http requests to a url is just as common a way to access data as using the AWS CLI is. Without easily discovering this url users can't use tools like fsspec or xarray via zarr to get to the data.

Okay now I've just learned about S3 endpoints, I see that I can still get to my data using fsspec by doing
s3fs_kwargs = {'anon': True, 'endpoint_url': 'https://data.source.coop'}
fs = s3fs.S3FileSystem(**s3fs_kwargs)

ds = xr.open_dataset(fs.open(single_file_url), engine='h5netcdf')

@TomNicholas We have resolved the issue raised as per this issue. Please check the contents of the page here, https://source.coop/cworthy/oae-efficiency-atlas/data/experiments/000/01/alk-forcing.000-1999-01.pop.h.0347-01.nc.
Can you please confirm if this is the expected scenario. The cloud uri correctly reflects the uri for accessing the contents of the file using AWS s3 cli.

Thank you @navdeepg2021 ! That cloud URI looks like what I would expect now.


swapping registry coords to quay for QE / testing

/cc @cooktheryan
Ive left the references in the generate-fbc.sh because we will not be using it until either next release or we generate more FBC images

I believe this is obsolete. @Gregory-Pereira I'm closing - if we need to reopen feel free, but it seems out of date


E836 after has("pythonx") ("This Vim cannot execute :python after using :py3")

vim -Nu NONE -c 'call has("pythonx") | py print(1)' results in:
Error detected while processing command line:
E836: This Vim cannot execute :python after using :py3
E263: Sorry, this command is disabled, the Python library could not be loaded.
Press ENTER or type command to continue

has('pythonx') is used in runtime/syntax/vim.vim.
I think the check alone should not block "Python 2" then, but only when it gets used really.
While at it it might make sense to improve the error, e.g. by providing the place where "Python 2" was used.  It was quite difficult to track this down (starting from Vader's test suite).
VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Jan 26 2020 11:45:38)
Included patches: 1-148
Compiled by Arch Linux
Huge version with GTK3 GUI.

The behavior is described in :help has-pythonx.
has('pythonx') has a side effect as same as has('python') and has('python3').

I see, thanks for the pointer.
Given this side effect it should be removed from the syntax file maybe?  (AFAICS it is not really necessary, is it?).
https://github.com/vim/vim/blob/7306d6b1c9528dcfe89eb77aa7ef354ffa9605a8/runtime/syntax/vim.vim#L737
/cc @cecamp

See the remarks below ":help has-python", one can check for "python_compiled" and "python3_compiled".
Python 2 is being deprecated, so perhaps checking for python 3 is preferred.

@brammool
How about adding "pythonx_compiled" and "pythonx_dynamic" for consistency?
@blueyed
Agree. One can edit a vim script file with if_python even if if_python is disabled in his Vim.
Editing a file and executing a file are different things.

@brammool
How about adding "pythonx_compiled" and "pythonx_dynamic" for consistency?

Well, it's not that difficult to use:
if has('python_compiled') || has('python3_compiled')
And since Python 2 is going away, it's not the right moment to add more
support for this.
--
hundred-and-one symptoms of being an internet addict:
81. At social functions you introduce your husband as "my domain server."
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Status bar

whether the active note is in your queue
[x] show the priority
show notes
[x] ability to change priority of the current repetition

[ ] And please make an option to hide IW in the status bar
[ ] Able to select what to show or hide, if more options are available


Librispeech eval losses calculated inconsistently in Jax and PyTorch

See my comment here (and the whole thread for more context).

Addressed by #330.


Canonicalizer causes wrong types for flow.ex.stream.fragment op

To repro:
module  {
  func @foo(%arg0: tensor<i32>) -> tensor<1xi32> {
    %0 = flow.ex.stream.fragment(%arg0) : (tensor<i32>) -> tensor<1xi32> =
        (%arg1: tensor<i32>) -> tensor<1xi32> {
      %1 = flow.tensor.reshape %arg1 : tensor<i32> -> tensor<1xi32>
      flow.return %1 : tensor<1xi32>
    }
    return %0 : tensor<1xi32>
  }
}

Run iree-opt -canonicalize foo.mlir, and you will see
module  {
  func @foo(%arg0: tensor<i32>) -> tensor<1xi32> {
    %0 = flow.ex.stream.fragment(%arg0) : (tensor<i32>) -> %arg0 =
        (%arg1: tensor<i32>) -> tensor<1xi32> {
      %1 = flow.tensor.reshape %arg1 : tensor<i32> -> tensor<1xi32>
      flow.return %1 : tensor<1xi32>
    }
    return %0 : tensor<1xi32>
  }
}

If you pass the output to iree-opt, it will complain:
...: error: use of value '%0' expects different type than prior uses: 'tensor<1xi32>' vs 'tensor<i32>'
    return %0 : tensor<1xi32>
           ^
...:3:5: note: prior use here
    %0 = flow.ex.stream.fragment(%arg0) : (tensor<i32>) -> %arg0 =

I actually don't know if this is a blocker. I found this issue when creating repro for other bugs.

There's probably a few things hiding here, two I spot immediately are

we shouldn't be creating a stream that only does a reshape like this
the operand is tied ( -> %arg0) but the shape differs between operand and result which is invalid

Right now tied operands assume they have the same shape so disallowing tying when the shape differs would probably unblock this. It'd be really nice if they were tied, though, so one approach would be to insert the reshapes to keep the shapes the same (here, inserting a reshape to f32 on the inside of the stream and to 1xf32 outside of the stream.
A pattern to elide streams that have no dispatchable operations can come after - right now this would create some pretty silly behavior where an empty command buffer is created and submitted :P

This also happens on other cases. The reshape one is the simplest IR I found. :-D
Some other examples are:
  %97:3 = flow.ex.stream.fragment(%c6, %c1, %96, %arg4) : (index, index, tensor<6xi32>, tensor<i32>) -> (tensor<i1>, tensor<1xi32>, tensor<i8>) =
      (%arg17: index, %arg18: index, %arg19: tensor<6xi32>, %arg20: tensor<i32>) -> (tensor<i1>, tensor<1xi32>, tensor<i8>) {
    %300 = flow.dispatch @run_dispatch_44::@run_dispatch_44[%arg17, %arg18, %arg18](%arg19) : (tensor<6xi32>) -> tensor<6xi1>
    %301 = flow.dispatch @run_dispatch_45::@run_dispatch_45[%arg18, %arg18, %arg18](%300) : (tensor<6xi1>) -> tensor<i1>
    %302 = flow.tensor.reshape %arg20 : tensor<i32> -> tensor<1xi32>
    %303 = flow.dispatch @run_dispatch_3::@run_dispatch_3[%arg18, %arg18, %arg18](%301) : (tensor<i1>) -> tensor<i8>
    flow.return %301, %302, %303 : tensor<i1>, tensor<1xi32>, tensor<i8>
  }

After pasting the IR, I found that it also only apply reshape op on the argument. There are more cases, but I did not check all of them.

Awesome - thanks for the examples! I'll work on this today.


Select and integrate with a CI testing tool

Option 1: Circle CI
Option 2: Jenkins
Option 3: Travis CI
Option 4: Appveyer

Am I missing any?
We'll need to consider cross platform testing (I don't think CircleCI has a Windows env).

Went with Circle CI and CodeCov


Problem with arguments in connection

Hi,
Thanks for this implementation. When I tried to retrieve arguments with connection field graphql server raises an error:
"message": "resolve_user_subscriptions() got multiple values for argument 'parent'",
My type implementation is as follows:
        type Query {
            node(id: ID!): Node
            userSubscriptions(
                after: String
                before: String
                first: Int
                last: Int
                parent: String
            ): UserSubscriptionConnection!

        }
        interface Node {
            id: ID!
        }
        type PageInfo {
            hasNextPage: Boolean!
            hasPreviousPage: Boolean!
            startCursor: String
            endCursor: String
        }
        type UserSubscription implements Node {
            id: ID!
            name: String
            parent: UserSubscription
        }
        type UserSubscriptionConnection {
            pageInfo: PageInfo!
            edges: [UserSubscriptionEdge]!
        }
        type UserSubscriptionEdge {
            cursor: String!
            node: UserSubscription
        }

And the resolver function is:
def resolve_user_subscriptions(_, info, parent):
    # print(parent)
    request = info.context["request"]
    if request.user.is_authenticated:
        return Subscription.objects.all().filter(parent=parent)
    else:
        raise Exception("Not authenticated!")
        # return None

Kindly help.

This is because your resolver function signature is malformed.  You are missing the connection_args argument, which is present in connection resolvers.
I would suggest:
def resolve_user_subscriptions(_, info, connection_args, *, parent):

You don’t strictly need the *, but parent will always be called as a keyword argument.

Thank you very much. It is working. Please include this in the documentation also. This will be very helpful for others too.


Redwood Slabs don't act like slabs

Describe the bug
Redwood slabs take longer to mine than other wooden slabs and do not mine faster with an axe than with an empty hand.  Also, Redwood slabs cannot be used in crafting recipes that take slabs, such as a lectern.
Steps To Reproduce
Steps to reproduce the behavior:

Start up a world running Vanilla Enhanced and go to a redwood forest.
Cut down a redwood tree and a spruce tree and use them to make 6 redwood slabs and 6 spruce slabs.
Place a redwood slab and a spruce slab, they should mine at the same speed, but they do not.
Attempt to make a lectern in a crafting table.  The recipe works with spruce slabs, but does not work with redwood slabs

Expected behavior
The redwood slab should behave as other wooden slabs do.
Screenshots
Recipe Screenshots:
Lectern with Redwood Slabs
Lectern with Spruce Slabs
Environment:

VanillaEnhanced Version: 1.7.0
Minecraft Version: 1.17
Fabric Loader Version: 0.11.6
Fabric API Version: 0.36.0

Thank you for reporting. Will be fixed in the next update.


unable to upload/add new video card

Issue Summary
.../unable to upload/add new video card
Steps to Reproduce
since 4.30.0
Ghost Version
4.32.0
Node.js Version
14.18.2
How did you install Ghost?
linux
Database type
Other
Browser & OS version
No response
Relevant log / error output
none!!!

Code of Conduct

[X] I agree to be friendly and polite to people in this repository

Hey @yannickcof - I've been unable to reproduce this issue with the few details you've given. Feel free to comment with a full reproduction case on a fresh install.


add readme, update target version

@paulmon,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


An Issue about Copyright and License

Can I port the style of Blowfish to Typora and publish it on theme.typora.io?
Thanks!

Given enough time and money anything is possible :)
I would say though that that's 80% of what a theme is though... so not sure about why you would do that...
This is, however, outside of the scope of this repo so I'll be closing this ticket. Feel free to open a discussion to see if someone in the community wants/can help.


IAVL snapshot - pruning race issue tracking

Copy of https://github.com/osmosis-labs/osmosis/issues/932


fix(layer): get_and_upgrade and metrics

when introducing get_and_upgrade I forgot that an evict_and_wait would had already incremented the counter for started evictions, but an upgrade would just "silently" cancel the eviction as no drop would ever run. these metrics are likely sources for alerts with the next release, so it's important to keep them correct.

Bug introduced in #5743 in "feat: upgrade on reinit" or 9865add961440ee88419ad1de10380b5f712cc43.


Update TLD listing to support all ICANN TLDs

The official ICANN TLD listing (https://data.iana.org/TLD/tlds-alpha-by-domain.txt) has a lot of additional TLDs that are not supported by linkify.  This causes links such as http://unfiltered.news/#/2016/03/18 to not be linkified.  Assuming that all that is required is that tlds.js be updated, I will submit a pull request later.

Thanks for the fix! The PR has been merged, it'll be included in the next release.


Single-line Draftail editor

This will be done outside of Wagtail, in springload/draftail. We want to support an editor only having a single line, so we can use Draftail for plain-text input fields, and rich text that’s inline. WIP: #7249.
Tasks

[ ] Add a prop to toggle single-line mode on/off
[ ] Prevent "Enter" creating new blocks in the editor
[ ] Prevent copy-paste creating new blocks

Acceptance criteria

[ ] As an editor, I can copy-paste into the field and multiline text gets copied to a single line

Testing & review plan

[ ] Cross-browser testing @thibaudcolas
[ ] Code review @thibaudcolas

This has been implemented as of Draftail v2.0 (no stable release yet), merged in #8805. This is undocumented for the time being (needs a final release), but is just a matter of setting multiline={false} on the editor.
Demo: https://demo.draftail.org/storybook/?path=/story/docs--single-line
Next step in Wagtail will be #8249.


Remove docker tests

What type of PR is this?
https://github.com/openshift/odo/issues/3714

As per the observation in comment https://github.com/openshift/odo/issues/3714#issuecomment-670531608
/lgtm
/approve

/approve


AVC denials check for tests

First real post-test check, complementing whatever the test itself did
by checking the system for AVC denials raised during the test runtime.
Follows the design established by Restraint's AVC denial check we all
know and love, and performs the same tasks.

Ack, go ahead.


simplify cloudbuild

Google's Container Builder doesn't work so well for us at the moment:

difficult to support github tags as  docker tags
difficult to support abbrev revisions as tags
I can't setup triggers b/c I'm no org owner

I've setup a job on our internal Jenkins to do it for now and push the image to our opensource registry, i.e. docker pull registry.opensource.zalan.do/teapot/external-dns
I'm leaving the simples version of cloudbuild here for reference. You can use it to push an image to your Google project's registry without having docker installed.

/cc @ideahitme

👍


Nostr-specific Private Key from Deterministic Wallet Signature (Sign-in-With-X)

Draft implementation for NIP-XX: Nostr-specific Private Key from Deterministic Wallet Signature (Sign-in-With-X) https://github.com/nostr-protocol/nips/pull/268

I don't understand this and am not convinced it is a good thing.

Sorry, I made a mistake while housekeeping and deleted branch under PR. I have restored it. Apologies for the small mess up

@fiatjaf Can we close this?

Thanks for your time & feedback..
Closing this for now/ 🙏 🖖


[3.x] BVH - fix not refitting upward from leaf nodes

Fix leaf node possibly not updating aabb due to wrong node id.
Additionally, when requesting a new leaf, mark dirty as false in clear().
Backport of #82482.

Actually if you have time to remake #82447 I can approve that too.
That means it just needs a production team to merge.

Thanks!


enable the import of picture with alpha channel

jpg doesnt support alpha channels
when importing pictures with alpha channel error shows up
-- "Could not create the task. OSError: cannot write mode RGBA as JPEG"
fix error by converting "RGBA" to "RGB" before saving image

@nmanovic
We have same fix in the development version.

:+1: Thank you


Split io_csv feature into read_csv and write_csv

Closes #405
I completely replaced io_csv with read_csv and write_csv, didn't know if we wanted an umbrella io_csv feature that encapsulates the other two.

Closing this in favor of #408


Add RainMachine switch platform

Description:
Adds support for RainMachine sprinkler controllers (via the Switch component).
Related issue (if applicable):
Pull request in home-assistant.github.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/3135
Example entry for configuration.yaml (if applicable):
switch:
  platform: rainmachine
  ip_address: <IP_ADDRESS>
  password: my_password_123

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Thanks, @MartinHjelmare – I've made your requested changes.

I think the branch got jacked. :stuck_out_tongue_winking_eye:
If you know your git wushu, you can rebase and remove the stray commits that shouldn't be there. Otherwise there's always:
https://xkcd.com/1597/

@MartinHjelmare: That's what I had done prior to this PR; not sure what happened. I'll fix shortly.

@MartinHjelmare: The lint CI step is failing with this error:
lint runtests: commands[1] | pylint homeassistant
************* Module homeassistant.components.switch.rainmachine
E: 41,21: No value for argument 'v' in function call (no-value-for-parameter)
ERROR: InvocationError: '/home/travis/build/home-assistant/home-assistant/.tox/lint/bin/pylint homeassistant'

That isn't anywhere in my code; guessing it might be in the platform schema definition. Any advice for me?

Found this: https://github.com/PyCQA/pylint/issues/818 – key point:

Yeah, this happens because we don't have currently a proper understanding of decorators, which voluptuous seems to use, but it's on our road map. Unfortunately, the only solution that I have for now is to recommend you disabling the error locally or in pylintrc.

Going to tell pylint to ignore that issue, then re-push. Holding tight...


Calyptos: Stack trace when setting objectstorage.providerclient to walrus

[<IP_ADDRESS>] out: Ran source /root/eucarc && http_proxy="" //usr/sbin/euca-modify-property -p objectstorage.providerclient=walrus returned 1
[<IP_ADDRESS>] out: [2015-09-09T02:51:44-07:00] FATAL: Chef::Exceptions::ChildConvergeError: Chef run process exited unsuccessfully (exit code 1)

Logs:
https://eucalyptus.atlassian.net/secure/attachment/25708/calyptos-1441792304.tgz

Think this could be related to the walrus server not being registered:
[root@odc-f-09 ~]# empyrean-describe-services --region @localhost | grep 'walrus\|storage'
SERVICE  objectstorage         API_<IP_ADDRESS>  API_<IP_ADDRESS>.objectstorage   broken
SERVICE  storage               az-01             az-01-sc-1                       broken
SERVICE  walrusbackend         walrus            walrus-1                         broken

[root@odc-f-09 ~]# nc -z <IP_ADDRESS> 8779
Connection to <IP_ADDRESS> 8779 port [tcp/*] succeeded!

Retried with identical symptoms:
https://eucalyptus.atlassian.net/secure/attachment/25709/calyptos-1441800742.tgz

This isnt related to the Walrus specifically. All java components are down right now.
Is anything different about the networking in this setup? It looks like the Java components arent able to talk to the CLC because the bind addr is set on the 10.104.x.y network rather than 10.105.x.y:
[root@odc-f-09 ~]# cat /etc/eucalyptus/eucalyptus.conf | grep bind
CLOUD_OPTS=" --bind-addr=<IP_ADDRESS>"
[root@odc-f-09 ~]#

I see the issue now. I had thought this PR was already merged:
https://github.com/eucalyptus/eucalyptus-cookbook/pull/201
This does the auto detection for the bind-interface. Ill get that merged ASAP.

@dmccue Its merged now. Sorry for the oversight.

Fixed, Thanks! - https://eucalyptus.atlassian.net/secure/attachment/25611/calyptos-1441138161.tgz
[root@odc-f-09 ~]# grep bind /etc/eucalyptus/eucalyptus.conf
CLOUD_OPTS=" --bind-addr=<IP_ADDRESS>"


tf.strings.join not allowed in graph

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): 19.10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): 2.2
Python version:3.7.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the current behavior
When writing a function that is mapped on a tf.data.Dataset, using tf.strings.join() yields the following error
OperatorNotAllowedInGraphError: iterating over `tf.Tensor` is not allowed: AutoGraph did not convert this function. Try decorating it directly with @tf.function.

Even when using the @tf.function
Describe the expected behavior
Should work when calling the @tf.function decorator
Standalone code to reproduce the issue
@tf.function
def process_path(path):
    path = tf.strings.split(path, "_")[:3]
    path = tf.strings.join(path, "_")
    return path

path_ds = tf.data.Dataset.list_files(...)
path_ds = path_ds.map(process_path)

@khazit tf.strings.join expect a list as input, so it should be tf.strings.join([path, "_"]).
I will close this issue, but feel free to reopen if you encounter any other issues.

@yongtang Thanks for the quick answer but it still doesn't work properly.
This tf.strings.join([path, "_"]) gives the following
[b'data/cityscapes/train/darmstadt_' b'000035_' b'000019_']
My idea was to join every element of the tensor using the "_" separator into a single string. So I'm expecting this b'data/cityscapes/train/darmstadt_000035_000019' as an output

@khazit Ah in that case, you need to convert path into lists before the join through tf.unstack:
@tf.function(autograph=False)
def process_path(path):
    path = tf.strings.split(path, "_")[:3]
    path = tf.unstack(path, num=3)
    path = tf.strings.join(path, "_")
    return path

num=3 is needed as otherwise tf.unstack will not be able to infer the number of output and join will not work. So num=3 has to be passed in.

Great ! Thank you
I'm closing this issue

@yongtang Is there any way to not manually input the number of outputs, i.e. not have to set num=3.  I am trying to do this in a for loop where there are different sizes.


Dynamic blocks

Hi @sheadawson
Please let me know if this is the wrong way to ask, but I wonder whether you have thought about making the position/number of blocks dynamic rather than having "Sidebar", "Before Content" and "After Content".
For example, it may be useful to have any number of blocks (already doable) and have them be part of the actual page content rather than not shown in the WYSIWYG editor at all, and let them be reordered as the content editor requires.
Your thoughts.
Thanks,
Robbie

Hi Robbie. There are no restrictions around number of blocks or what areas they should be placed in. It sounds like you read the example config in the README file as being hard coded. Rather, it's just an example of one of the ways you could configure the module for a website.
This module can be used as a replacement for WYSIWYG content. But this one is designed more specifically for that https://github.com/dnadesign/silverstripe-elemental

Hey @sheadawson
Great - thanks for the quick reply.
Cheers,
Robbie


[hw,prim,rtl] Cleanup RAM connections

This cleans up some dust from the previous commits. Please look into the individual commits for more details.

Is the rv_core_ibex change actually related here? In any case, LGTM!

Yes it is. As part of adding the DFT signals, https://github.com/lowRISC/opentitan/pull/25654/commits/937b1f60f160eb5a08a0f1d9fb70ec36d1621c38 added ICacheNWays the parameter to  rv_core_ibex. As a local param though, which cannot be used in the portlist.


Provide local test facility

Start Solr Cloud and Zookeeper in-process for unit testing
Provide test data

In a first step it is also OK to:

provide a local Solr Cloud and Spark with startup scripts and a howto readme
provide a mock implementation for the Chronix Solr Cloud storage

Logic to retrieve the shard urls should be moved from ChronixSparkContext to ChronixSolrCloudStorage.