tf.linalg.solve produces incorrect output when compiled using XLA

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
2.9.1
Custom Code
No
OS Platform and Distribution
Linux Ubuntu 20.04.4 LTS
Mobile device
No response
Python version
3.8.10
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
tf.linalg.solve when compiled with XLA produces different (incorrect) output when compared to a standalone TF execution.

In the sample code below, the solveEquation function solves for Ax = B and returns Ax. This function is run with and without XLA enabled. Both executions produce very different outputs!

Standalone code to reproduce the issue
import numpy as np
import tensorflow as tf

data_type = tf.float32

def solveEquation(A, B):

    ATA = tf.matmul(A, A, transpose_a=True)
    ATB = tf.matmul(A, B, transpose_a=True)

    output = tf.linalg.solve(ATA, ATB)

    return (tf.matmul(A, output))

P = 2
Q = 3
R = 5

np.random.seed(0)
mat1 = tf.constant(np.random.random((P, Q)), dtype=data_type)
mat2 = tf.constant(np.random.random((P, R)), dtype=data_type)

output_standard = solveEquation(mat1, mat2)

tf_func_xla = tf.function(func=solveEquation,
                          input_signature=[tf.TensorSpec(shape=(P,Q),dtype=data_type),
                                           tf.TensorSpec(shape=(P,R),dtype=data_type)],
                          jit_compile=True)
output_xla = tf_func_xla(mat1, mat2)

tf.print(output_standard)
tf.print(output_xla)

Relevant log output
# output with XLA disabled
[[0.43758738 0.891772866 0.963661969 0.383441627 0.791725397]
 [0.528894722 0.568044662 0.92559737 0.0710358918 0.0871287584]]

# output with XLA enabled
[[0.274795085 0.876262307 0.82626003 0.345206 0.581315041]
 [0.3589876 0.553595543 0.778286219 0.0325973332 -0.126585901]]

@Ricky-KLA,
I tried changing the P, Q, R values & tried to execute the mentioned code in the alternative approach and was able to execute without any issues with the expected output. Kindly find the gist of it here.
[[0.79968822 0.509070814 0.686773658 0.512732089 0.719699502]
 [0.788937032 0.528437436 0.846307933 0.280258715 0.788256884]
 [0.560636401 0.524819672 0.600428879 0.827999055 0.492959023]
 [0.539790332 0.374352 0.381277114 0.638788283 0.428218782]
 [0.911398709 0.371089876 0.373238 0.52072823 0.695399284]
 [0.0141778868 0.283042371 0.482044935 0.22521086 0.140789688]]

[[0.799689174 0.509071589 0.686774909 0.512732685 0.719700217]
 [0.788938284 0.528438389 0.846309364 0.280259281 0.788257897]
 [0.560636759 0.524820089 0.600429654 0.827999711 0.492959112]
 [0.53979069 0.374352425 0.38127774 0.63878876 0.428218871]
 [0.911399543 0.37109068 0.37323907 0.520728886 0.695399702]
 [0.0141781522 0.28304249 0.482045233 0.225211024 0.140789896]]

I suspect the issue is not with tf.linalg.solve and please try to provide different values and confirm if the issue still persists. Thank you!

@tilakrayal I tested with different values of P, Q and R and found that when P < Q, there is mismatch in outputs. But in hindsight, I see that this is expected since the number of equations is less than the number of variables. For the cases where P >= Q, both outputs match. So, this is not actually a bug and hence we can close this issue.

@Ricky-KLA,
Glad the issue is resolved for you, please feel free to move this to closed status. Thank you!


fix: error in non tty environments

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Jfc, do you have to change the API every version?

 Just settle on something, ffs.

v4 was released in April 2017 and v5 is v4. v6 is a change, but if you're satisfied with v4/v5, then nothing is forcing you to switch.

I know. Just settle on a damn API already, please.


#35284 Add a link to the wiki article about data import

There is no information about individual settings on our wiki. This link seems misleading in my opinion

Let's wait until the article is updated.

Verified


Pass redux-form's change prop to SimpleForm's children

Hi!
Would you consider passing the change prop through to the children of SimpleForm?
I have a case where I want to implement a relatively complex custom Field, and I think I need the change prop to make this work.
I would be willing to submit a PR if you guys are open for this change.
Cheers!

As SimpleForm is decorated by the reduxForm higher-order component, you can already pass a custom change prop on it.

I understand that I can pass a custom change prop to SimpleForm, but I want something different; I want SimpleForm to pass the change prop (that is decorated by reduxForm) through to the child Fields.
Something like this:
export const SimpleForm = ({ children, handleSubmit, invalid, record, resource, basePath, change }) => (
	<form onSubmit={handleSubmit} className="simple-form">

			{React.Children.map(children, input => input && (
				<div key={input.props.source} className={`aor-input-${input.props.source}`} style={input.props.style}>
					<FormField input={input} change={change} resource={resource} record={record} basePath={basePath} />

			))}

		<Toolbar invalid={invalid} />
	</form>
);

Note the change prop received by SimpleForm, and passed on to FormField.

You'll have to write your own Form component for that, this is not something admin-on-rest will support.

Understood. I already have my own Form component (which is basically a copy from the SimpleForm :) ). I was just curious if this is perhaps a general case that others are trying to solve as well.
Any way. Thanks for the awesome lib, having a lot of fun with it!


copy: switch to go implementation

Reimplement in go to support changing file owners.

What's current status?
@tonistiigi

The new commits LGTM

LGTM
@AkihiroSuda happy to address concerns you may have in a followup.


Diagram in read-only state after restoring a saved state

The classes' boxes can sometimes not be moved by dragging them after restoring a saved state, similarly to a read-only state.
The issue persists after modifying the code or selecting another example, but the diagram updates.
To reproduce:

Open any UmpleOnline example and quit the page
Reopen UmpleOnline to restore the saved state

The text simply gets selected instead of the box being dragged. The issue does not seem to occur on Google Chrome. On Firefox, it can be fixed by clearing browser information (cache, etc) from the browser.

The issue seems to consistently occur after Umpleonline is quit and the saved state is recovered after a longer period of time (seems to be at least 5-6 hours, perhaps more, will test further on it). The cookie is currently set to expire a full month after it was created however.
Seeing as it affects code written after recovering from the saved state, there might be a read-only setting affecting the diagram that gets loaded... However, the read only setting seems to be partially applied, as it does not affect the text editor.
Another issue I've found is that when restoring code with the "Photo Ready" option checked, a similar read-only state occurs (the boxes are normally be able to be moved around in that state), and when unchecking the Photo Ready option, the diagram can still not be edited. Restoring from a saved state after unchecking the Photo Ready option solves the issue. It occurs consistently on Chrome, Safari and Firefox. Should it be put in a separate issue?

I think the photo ready problem is part of this issue and likely should be solved together with it.
Possible causes of the 5-6 hours issue: a) There could be a cache expiry of some kind. b) Not all needed variables might be set or restored in the saved state, or not all might be being consistently set/restored (for example it might be that a variable is supposed to be true or false, but is sometimes null either in the default case).

Reloading from a saved state after a longer period of time yields behavior that isn't very consistent, and as far as I can see, Chrome, Safari and Firefox can have similar results, but not at the same frequency (I have not seen Safari restore a saved state correctly after some time so far, but I haven't been testing it for long enough to be certain it never loads the saved state correctly). Chrome almost always loads the model correctly. Part of the issue likely lies in the way the model file is saved (doesn't seem permanent enough for some browsers). Three possible states can occur when a saved state is restored:

The model loads successfully, without the unnecessary 'PhotoReady' option checked
The model loads, but with the 'PhotoReady' option
The model doesn't load and Error 500 is thrown (from the PHP)

There is possibly more than one cause. It's somewhat hard to test for the issue, especially since it doesn't occur on a local version of UmpleOnline.

This bug seems to be happening very irregularly and infrequently and the load saved state may not be being used much anyway, so downgrading priority


Gallery app

Signed-off-by: Jorma Virkkunen<EMAIL_ADDRESS>

should the nav order be 485 since that is between 470 and 500?

The previous doc is "Flashing... (470) and the next is "Importing..." (500).
Gal is closer to Fla than to Imp - and H-docs are completely missing. This is why I chose an index closer to the previous as there won't probably be many docs between "Flashing" and "Gallery". More space to add H docs.

should the nav order be 485 since that is between 470 and 500?

The previous doc is "Flashing... (470) and the next is "Importing..." (500). Gal is closer to Fla than to Imp - and H-docs are completely missing. This is why I chose an index closer to the previous as there won't probably be many docs between "Flashing" and "Gallery". More space to add H docs.

ok, makes sense.


update local installation guide for version 2.0

Local installation guide has to be updated to reflect changes done in 2.0:
Hint: Local installation guide is located in this path:
https://github.com/opendevstack/opendevstack.github.io/tree/master/docs/modules/local-install
Let's summarize changes done in verison 2.0:

new Repository ods-quickstarters (replacement of _ods-project-quickstarters)

@michaelsauter @ungerts @kiwo @clemensutschig @tjaeschke

can you all please help and provide additional information about what was changed for 2.0?


⚠ Bofrost.lu has degraded performance

In 70c1bc1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 3236 ms

Resolved: Bofrost.lu performance has improved in 82b631f after 31 minutes.


Feat contract

Add Contract extends Store, the Contract implements two functions:

run function can call remote or local service.
link function can mount functions on Contract, most of the time it is used to mount other's DApp's functions.

@Daryl-L @PainterPuppets @zhengjianhui please have a review

Please squash and merge after conflicts are resolved


fix: update unmanaged access codes type

Just had a chat with @rchodava, we do in fact support time_bound unmanaged codes now
See this test in connect which assumes that we support start and end times
https://github.com/seamapi/seam-connect/commit/873456fc78d26db400c03367edae0d582f0b7375#diff-27980da6c4e3d3c609f50b1[…]483e7b9cc1517d146764bfR85-R86

I wonder if there's a smart way to infer all of this from seam-connect

I wonder if there's a smart way to infer all of this from seam-connect

when seam connect is fully using nextlove hopefully we can infer most of the types


Change the way moving messages between folders in the database works

Previously, when moving a message, the entry in the messages table was rewritten to reference the destination folder. Then a "deleted" message entry was created referencing the source folder.
This commit changes the move code to create a new entry for the message in the destination folder and then changes the original message entry to be a "deleted" entry.
It's a first step towards being able to use a message's database ID instead of the server ID in "pending commands".
This also continues the project of moving database-related code to the storage module. There should now be enough scaffolding in place to make moving code from LocalStore and LocalFolder relatively painless.
Part of #3220

This needs a lot more work. The old code reuses LocalMessage instances after the move operation. Those instances were (somewhat) modified by the move code to point to the message in the destination folder.


test: re-organise tests for LSP6KeyManager

What does this PR introduce?
The easiest and quickest way to review this PR is to look at the folder structure of the tests for the Key Manager.
https://github.com/lukso-network/lsp-smart-contracts/tree/test/lsp6/tests/LSP6KeyManager
You will get a high level overview of how permissions and internal functions are tested.
Tests

[x] group tests into one single file LSP6KeyManager.spec.ts, that tests both version of the contract: standard contract and base contract (to be used behind a proxy).

splits tests related to permissions into an /effect folder.

add more detailed tests for executeRelayCall(...)

Bug

fix bug when reporting error message from operation type STATICCALL

I would prefer renaming the effects folder to either behavior or simply “tests”


Fix overview page typos

Description of change

Fixed some typos.
Corrected header hierarchy to not skip header levels.
Corrected capitalization to be more coherent and in line with the Wiki.
Updated note styling to be more coherent with the rest of the Wiki.

Type of change

Documentation Fix

Consolidated in #200. Closing.


Bump mybatis from 3.5.9 to 3.5.10

Fix test for type aliases

Pull Request Test Coverage Report for Build #58

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.237%

Totals

Change from base Build #55:
0.0%

Covered Lines:
193

Relevant Lines:
207

💛  - Coveralls


Re-order issuer list in bar for better readability

Signed-off-by: JoshVanL<EMAIL_ADDRESS>/assign @munnerz

/lgtm


New release wanted

When there will be a new release? I think that https://github.com/abargnesi/rantly/commit/4a0b3881ead86b0a265c992607f815c4dd904427 and https://github.com/abargnesi/rantly/commit/4a9a809bd3d683606aa53867d18f95466c9884bd are important enough to have a new release.

Thanks for the check. I put out 1.1.0 and updated the changelog.

@abargnesi thanks! :wink:


Add the ability to set a custom timeout for send method with shouldPollResponse parameter

Currently, when calling the send method with the shouldPollResponse parameter set to true, the default timeout for waiting for the transaction to be executed is 60 seconds. However, this timeout may be insufficient for busy networks where transaction confirmation may take longer.
It would be great to have the ability to set a custom timeout for the send method with the shouldPollResponse parameter, allowing developers to adjust the timeout according to their needs.
Adding this feature would improve the user experience for developers using your library and make it more adaptable to different network conditions.
Thank you for considering this feature request.

Thank you for your suggestion, we will support it  in next version. And now if  the default timeout is insufficient , you could check the transaction result by tronWeb.trx.getTransactionInfo(txID).


Skip feature handling for databases from unknown vendors

Fixes #24

Hi, thanks for the PR! Could you add a test to cover this? I guess patching one of the databases' vendor attribute and asserting that the flags aren't added should be sufficient. If that's asking too much, I can take it from here :slightly_smiling_face:

Added tests for static/callable features and unknown vendors.

failing 3.9 build is taken care of in #26

@lociii I rebased from master and squashed the commits. I also made some edits, but they're pretty minor. Some notes, though:
I changed assertTrue and assertFalse to assertIs(..., True) and assertIs(..., False) to avoid incorrectly asserting truthy/falsy values. I caught a bug in the tests: mock.patch by default uses MagicMock, which is a callable object. When the code checks the feature, it calls the mock and set the return_value as the feature value. So, your initial tests actually set the features to be True and lambda: True (not True and True). Thus, the callable test fails after I changed the assertion (it previously passed because lambda: True is truthy).
I fixed it by using mock.PropertyMock in test_static and removed the lambda in test_callable. I could've added mock.PropertyMock and kept the lambda instead, but I think that's not necessary because MagicMock (the default) is already a callable object.
Thanks a lot for the PR! I think this is the first time I actually touched unittest.mock myself and I learned a lot from it :smile:

You're totally right with the boolean assertion. Didn't think of that.
Thanks for improving the tests and publishing the new release.


[Enhancement Request] Custom Reactions

It would be really cool if you can have those fancy embedded messages have placeholders such as %user% and %target% for custom reactions. Then it would be easy to make "more professional" looking reactions for people whenever someone triggers a reaction.
It would also be nice if you could set permissions on certain custom reactions, and not just the whole module, so not all users can some of the reactions them while some can't. That way you can have permissions for literally everything! All of the permissions...

It's already in the #suggestions channel. I need to rewrite the crappy Custom reaction logic to make this happen. Probably in 1.43 or later version.


i want to use it in mysql ,what can i do it? thank you very much.

i am newer!
i want to use the mysql in the project. so,
i change the project setting  as follow:
DATABASES = {
'default': dj_database_url.config(default='mysql://root:123456@<IP_ADDRESS>:3306/saleor')
}
and : pip install the mysql-clilent.
but ,when i do 'run manage.py task' in pycharm, in the console display:
"
Failed to get real commands on module "saleor": python process died with code 1: /usr/local/src/envs/saleor/lib/python3.6/site-packages/psycopg2/init.py:144: UserWarning: The psycopg2 wheel package will be renamed from release 2.8; in order to keep installing from binary please use "pip install psycopg2-binary" instead. For details see: http://initd.org/psycopg/docs/install.html#binary-install-from-pypi.
""")"
thank you ！

Hi, we only support Postgres database as we use some of its specific features. MySQL is not supported.

get it！ thank you!~~


Add documentation using PHPDocumentor

This issue is created to enhance the package documentation using PHPDocumentor. It will ensure the codebase is well-documented and adheres to best practices.

Resolved in #6


test: fix gofmt

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

Coverage increased (+0.04%) to 41.905% when pulling cab84b9a1590bd051489c5e65967c9e568386b31 on pr/ianvernon/fix-gofmt into 3d471a5d41a7eea689eea6002e36232ac46658b5 on master.

Fixes: a1f1d0ff5f2f ("CI: Enforce sensible timeouts.") (https://github.com/cilium/cilium/pull/7720 which is marked as pending a 1.5 backport and needing earlier backports)

@joestringer I ran go fmt on the files in the v1.5 backport branch as part of the backporting process so 1.5 is good here

test-me-please

Ouch, sorry, I think I have fixed the same issue ;-/ https://github.com/cilium/cilium/pull/7738

Superseded by #7738


resolves issue 3124

Fix for https://github.com/boto/botocore/issues/3124
We've been maintaining this patch locally in our company for awhile, and just noticed. Figured it's best to get upstream.

Hi @dbrumley,
Thank you for the contribution.
I just opened up a PR for an experimental opt in header for exactly this; we are hesitant to make this change the default since customers may be relying on the existing behavior which is fully functional for S3.  In the future, this may become the default once we are confident that this is not a breaking change.
The content-length header is also not guaranteed to be present, so making a best-effort check on the length of the body will work for more use cases


Fix typo in bank-account tests

Small typo in the tests, which unfortunately suggests the opposite meaning.

Great find, thanks!

Wow that was fast, thanks!


Publish module fails to run due to missing dependency in RHEL 7

After doing a build and deploy using Andrei’s script, the Apache Publish module does not work. (So far I have only seen this on RHEL 7) This error in the GEE error log (/opt/google/gehttpd/logs/error_log):
[Thu Mar 23 20:04:02 2017] [error] [client <IP_ADDRESS>]   File "/opt/google/gehttpd/wsgi-bin/serve/publish/publish_app.wsgi", line 1, in <module>
...
[Thu Mar 23 20:04:02 2017] [error] [client <IP_ADDRESS>]     import psycopg2  # NO LINT. third party package should be before own ones.
[Thu Mar 23 20:04:02 2017] [error] [client <IP_ADDRESS>] ImportError: No module named psycopg2

This is a known issue in the latest build.  Also occurs in Ubuntu.  We are investigating.
@andreisel @asmitaw @dattapm

It is different issue, I fixed this on Ubuntu build by adding into mod_wsgi_ge.conf (third_party/mod_wsgi/conf.d/mod_wsgi-ge.conf) a path to site-packages:
/opt/google/gepython/Python-2.7.6/lib/python2.7/site-packages
I believe that problem here is that RHEL 7 may have different python version, so that the path referenced in mod_wsgi_ge.conf is incorrect.
Seems there are two options:
A) bundle python and related libs, so we can use static path in mod_wsgi_ge.conf.
B) have template with placeholders for mod_wsgi_ge.conf and generate final mod_wsgi_ge.conf when deploying Server.
For now, we can just revert bundling of python. Need to only update third_party/SConscript, and third_party/mod_wsgi/conf.d/mod_wsgi-ge.conf

@andreisel is this issue related to #36?

No it is not related to #36. In OSS version, we unbundled bunch of 3rd party libs and use python from system.
I believe we just need to update https://github.com/google/earthenterprise/blob/master/earth_enterprise/src/third_party/mod_wsgi/conf.d/mod_wsgi-ge.conf. Should be generated when installing/deploying to use system-python version in python-path.
Suggestion can be verified on RHEL 7 by updating "python-path=/opt/google/gepython/Python-2.7.6/lib/python2.7/site-packages" in /opt/google/gehttpd/conf.d/mod_wsgi-ge.conf to use RHEL python versions. After building and deploying on RHEL 7, please check what path is to site-packages dir. in /opt/google/gepython/, and put that path to /opt/google/gehttpd/conf.d/mod_wsgi-ge.conf. Then restart geserver daemon and verify push/publish.

Looks like simply changing "/opt/google/gepython/Python-2.7.6/lib/python2.7/site-packages" to "/opt/google/gepython/Python-2.7.5/lib/python2.7/site-packages" (note the python version number) does the trick.

It seems I don't have permission to assign this to myself, but I am working this issue.


get and set display name for XOpenDisplay()

can be done by just setting DISPLAY env var, but nice if you want more control then you can now set it programmatically. ran into this problem when using ssh.

If I understand correctly this allows you to pick the X server you connect to?

Thanks that argument parsing worked nicely. I fixed up the OS compat issue, although I'm not sure I like how it structured, thought about maybe moving some of it into screen.h to avoid having to use those defines in robotjs.cc.
Yes, that’s exactly the purpose of this code; directly from X(7) under the DISPLAY NAMES section:
From the user's perspective, every X server has a display name of the form:

hostname:displaynumber.screennumber

This information is used by the application to determine how it should connect to the server and which screen it should use by default (on displays with multiple monitors)...

Sorry for the delay, I've been without internet and I want to test this first. I should be able to test tonight or tomorrow.

#206 replaces this.


Add support for accessor in hvplot.interactive

Fixes #737
Fixes #646

Great! Ready to review/merge?

Yes

Thanks @Hoxbro !


Layout grid col-45 does not exists as show documentation

Hello,
Before anything, congratulations for the great framework and work.
I'm trying to use layout grid col-45, but I checked out the framework7.min.css and saw that does not exists the class col-45.
The correct implementation is this peace of code bellow?
.row .col-45 {
width: 45%;
width: -webkit-calc((100% - 15px1.2222222223) / 2.2222222223);
width: calc((100% - 15px1.2222222223) / 2.2222222223);
}
.row.no-gutter .col-45 {
width: 45%;
}
Thx,
Elysio

Make sure you are using latest version of F7 https://github.com/nolimits4web/Framework7/blob/master/dist/css/framework7.ios.css#L187


[Bug]: Unable to push report into complex S3 object key

What happened?
I tried to upload Prowler reports into S3 using -B option together with -o option specifying full path to the report file.
When it comes to file upload to S3, prowler takes only the last directory in the -o option value and uses it as a key for the S3 object.
I'd want to store reports in S3 in the same way AWS do for logs (to have a directory structure in a next way: <account_id>/<year>/<month>/<day>/csv/report_file.csv.
For now, even if I specify -o <account_id>/<year>/<month>/<day> it will store reports in S3 within the <day>/csv/report_ile.csv>, so it just drops all previous directories in the path and taking only the last one. I suppose that's because of this line: https://github.com/prowler-cloud/prowler/blob/master/prowler/lib/outputs/outputs.py#L200. This code has been intentionally done for some reason so it might not be an actual bug (the app is doing exactly what it's being told to do) but it's kinda strange to have this in place.
How to reproduce it
Steps to reproduce the behavior:

What command are you running? mkdir -p ${TARGET_AWS_ACCOUNT_ID}/$(date '+%Y')/$(date '+%m')/$(date '+%d'); prowler aws -M csv html -D ${S3_BUCKET_NAME} --log-level ERROR -R arn:aws-cn:iam::${TARGET_AWS_ACCOUNT_ID}:role/${TARGET_AWS_ACCOUNT_ROLE_NAME} -o ${TARGET_AWS_ACCOUNT_ID}/$(date '+%Y')/$(date '+%m')/$(date '+%d')
Cloud provider you are launching: aws
Environment you have like single account, multi-account, organizations, multi or single subsctiption, etc.: multi-account within the org
See error: No errors

Expected behavior
I'd want to have S3 objects to be uploaded in the same directory structure as they are written locally using -o option.
Screenshots or Logs
If applicable, add screenshots to help explain your problem.
Also, you can add logs (anonymize them first!). Here a command that may help to share a log
prowler <your arguments> --log-level DEBUG --log-file $(date +%F)_debug.log then attach here the log file.
From where are you running Prowler?
Please, complete the following information:

Resource: (e.g. EC2 instance, Fargate task, Docker container manually, EKS, Cloud9, CodeBuild, workstation, etc.): workstation, fargate task, docker container
OS: [e.g. Amazon Linux 2, Mac, Alpine, Windows, etc. ] Mac
Prowler Version [prowler --version]: Prowler 3.1.2
Python version [python --version]: Python 3.9.12
Pip version [pip --version]:
Installation method (Are you running it from pip package or cloning the github repo?): both
Others:

Additional context
Add any other context about the problem here.

Thank your for the info @daftkid !
The above PR will solve the issue.


update codeowners to be ohai

Team name has changed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

0 of 0   changed or added relevant lines in 0 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.02%) to 57.291%

Files with Coverage Reduction
New Missed Lines
%

pkg/backend/telemetryapi/harvester.go
2
87.62%

pkg/integrations/v4/logs/cfg_watcher.go
4
67.5%

Totals

Change from base Build<PHONE_NUMBER>5:
-0.02%

Covered Lines:
15350

Relevant Lines:
26793

💛  - Coveralls


THRIFT-3584 Lua: To check element of self whether nil or not,avoid the value(false) of BOOL is lost.

…e value(false) of BOOL is lost.
eg.
Before:
if self.argBool then
oprot:writeFieldBegin('argBool', TType.BOOL, 13)
oprot:writeBool(self.argBool)
oprot:writeFieldEnd()
end
After:
  if self.argBool ~= nil then
    oprot:writeFieldBegin('argBool', TType.BOOL, 13)
    oprot:writeBool(self.argBool)
    oprot:writeFieldEnd()
  end

Why:
  the value(false) for paramater self.argBool will be lost when condition is 'if self.argBool then' as before.

While it might fix a problem in sending false from Lua,
combined send + receive test does not seem to work even with this fix.
Do you want to fix reading side too ?

To read a field by field id instead of argument of 'self ' in reading side.
The problem can not be reproduced in reading side.

OK, "read" was not a precise word.
The problem I was trying to describe is about receiving boolean return value (not field) that happened to be false.
At a quick glance, it seems that if result.success then in generated code can be fixed in the same way as this.
If you're running the test mentioned in the other patch, you can uncomment the line assertEqual(client:testBool(false), ... and verify locally.


Feature Request: Convert between ACI and RGB Color Nodes

Convert between AutoCAD Color Index and its RGB value

There are already hidden functions in the Toolkit for converting from the AutoCAD color to a Dynamo color (based on RGB) and viceversa.

Oh cool, how can I find that function?


Documentation

Issue Id you have worked upon -
#390
Briefly explain your program logic -
Documentation in the code of calculators i.e. -
Factors of a number
Factorial Calculator
By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

Congratulations 🎉 @ravishrivas on your contribution.
Thanks for adding value to this repository. 😊
Let's celebrate this fest 😃 by providing more developers 👨💻👩💻 an opportunity to contribute. Help us make this repository reach far and more by sharing this to your friends and colleagues.
Your efforts can make difference.


【ZR300】Camera & IMU intrinsics and Extrinsics unreliable from API?

Required Info

Camera Model
ZR300

Firmware Version
<IP_ADDRESS>

Operating System & Version
ubuntu 14.04 LTS

Kernel Version (Linux Only)
4.4.0-75-generic

Hi,
I have a realsense zr300 camera used for vision-inertial odometry, in which case the extrinsics matrix from color camera to motion module is needed.
So how can I get the extrinsics ?
I have tried the function
dev->get_motion_extrinsics_from(rs::stream::color)
provided by librealsense API. Well the function returns error, I look into the function and find that the error comes from
throw std::runtime_error("Motion intrinsic is not valid");
But I can get the motion intrinsics using the function
dev->get_motion_intrinsics();
So how to solve the problem? Can I get the extrinsics directly from the library? If not, should I calibrate the extrinsics myself?
Btw, is the camera&IMU intrinsics results derived from the API reliable?
    colorIntrinsic = dev->get_stream_intrinsics(rs::stream::color);
    imuIntrinsic = dev->get_motion_intrinsics();

Thanks in advance.

Hi @sqn175
What about dev->get_motion_extrinsics_from(rs::stream::depth) and dev->get_motion_extrinsics_from(rs::stream::fisheye)?

Hi, @dorodnic ,
dev->get_motion_extrinsics_from(rs::stream::fisheye) works and returns a reasonable result:
    [ 0.999864817, 0.0162242502, 0.00268337899, 0.0068119932,
      -0.0161926001, 0.999803603, -0.0114245601, 0.00240068603,
      -0.00286820694, 0.0113795698, 0.999931097, -7.7811e-06,
      0.0, 0.0, 0.0, 1.0],

But I can not find a way to validate the result.

I suspect this to be a bug in the calibration process. It seems like MM-Color and MM-Depth tables are empty. As a work-around, I believe you can get the relevant extrinsics by multiplying extrinsics from Depth to FE by extrinsics from FE to MM (this multiplication might cause some small degradation in precision, that's why librealsense tries to read from dedicated tables)


[Issue] Sharing to Google + through calling shareVia method, is not working in iOS

Hello all,
I found that calling shareVia method with these 1st parameter values, does not show google plus sharing dialog on iOS: 'google+', 'com.google.ios.apps.plus', 'google'. What is the correct string parameter that I need to pass into shareVia method's 1st parameter for showing the google plus sharing dialog in iOS ?
Cheers.

You can't using this plugin, but you should be able to use the AppAvailability plugin.


Unnecessary files included with pip install

Versions: Ubuntu 20.04, python 3.9, pypandoc 1.7.5
Using pip install pypandoc creates the following files in my site-packages directory:
    site-packages/LICENSE
    site-packages/README.md
    site-packages/examples/services.py
    site-packages/filter_test.md
    site-packages/pypandoc/files/COPYRIGHT.txt
    site-packages/pypandoc/files/pandoc.exe
    site-packages/pypandoc-1.7.5.dist-info/*
    site-packages/pypandoc/*
    site-packages/tests.py

In other words, quite a few files are outside the package directories (pypandoc/* and pypandoc-*.dist-info).
This creates a few problems:

The examples/ directory looks like an installed package.
The tests.py file in site-packages broke my pytest tests (due to some sort of name collision). This took a while to debug, since I don't expect installing packages to interfere with tests.
The LICENCE and README are confusing (which package do they belong to?) and are already present in the pypandoc-1.7.5.dist-info directory.

First, it seems preferable to keep all package files in the pypandoc/* and pypandoc-*.dist-info directories (unless absolutely necessary).
Second, the file pypandoc/files/pandoc.exe is useless to me (on linux), and is very(!) large (112MB). Maybe it could be made optional?
Thanks for reading! Please let me know what you think.

Hi.
Thanks for reporting this.
I was aware of the pandoc executable that had sneaked it's way into a binary dist of pypandoc.
However, I don't know what's causing the other files to be put there.
I'll investigate, and get back to you.

Hi again.
In future versions the test, readme and examples will no longer be included in the pip installables


chore: console ui improvement added

i guess this needs L9 to run,.
but filament still supporting L8

Yeah, unfortunately that is the case :(


Auto update the "Prerequisites" section of the readme

Update the "Prerequisites" section with the language enter with the -l option, it would very convenient and more easy to use for lazy people like me

Feature will be added to mdCreator in the next update.
Thanks for report!

Issue is closed due to pull request #6.
Thanks for contributing!


Zoom event

Is your feature request related to a problem? Please describe.
I am looking for a way to use zoom event.
Describe the solution you'd like
If I zoom in or out, then I want to handle my own additional funtionality.
Describe alternatives you've considered
Currently I check the zoom level each second in a timer event. If the zoom level is changed, then I perform the additional functionality I want. However besides it can be seen as pretty ugly, it rarely appears that mapRef.GetZoom() takes relativaly long to execute.
Additional context
I want the same for zoom event like the mouse events which are currenly implemented.

Look at the extent
an event is raised on the extent change.
I think that is triggered by the zoom level as well, if the user changes
the zoom level.
Peter Marshall
07979608139
@.***
On Mon, 6 Feb 2023 at 15:17, Cor @.***> wrote:

Is your feature request related to a problem? Please describe.
I am looking for a way to use zoom event.
Describe the solution you'd like
If I zoom in or out, then I want to handle my own additional funtionality.
Describe alternatives you've considered
Currently I check the zoom level each second in a timer event. If the zoom
level is changed, then I perform the additional functionality I want.
However besides it can be seen as pretty ugly, it rarely appears that
mapRef.GetZoom() takes relativaly long to execute.
Additional context
I want the same for zoom event like the mouse events which are currenly
implemented.
—
Reply to this email directly, view it on GitHub
https://github.com/fis-sst/BlazorMaps/issues/27, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTAWK3ZS24KIOJIFY24CYLWWEI2FANCNFSM6AAAAAAUSZZIB4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This
<MapView @ref="MapView"
OnExtentChanged="OnMapExtentChanged"
@." @." Zoom="16"
Style="height: 100vh; width: 100%;">
private async void OnMapExtentChanged(Extent extent)
{
}

Peter Marshall
07979608139
@.***
On Mon, 6 Feb 2023 at 15:17, Cor @.***> wrote:

Is your feature request related to a problem? Please describe.
I am looking for a way to use zoom event.
Describe the solution you'd like
If I zoom in or out, then I want to handle my own additional funtionality.
Describe alternatives you've considered
Currently I check the zoom level each second in a timer event. If the zoom
level is changed, then I perform the additional functionality I want.
However besides it can be seen as pretty ugly, it rarely appears that
mapRef.GetZoom() takes relativaly long to execute.
Additional context
I want the same for zoom event like the mouse events which are currenly
implemented.
—
Reply to this email directly, view it on GitHub
https://github.com/fis-sst/BlazorMaps/issues/27, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTAWK3ZS24KIOJIFY24CYLWWEI2FANCNFSM6AAAAAAUSZZIB4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

OnExtentChanged

In my .razor I use
<Map @ref="mapRef" MapOptions="@mapOptions">
Map however does not support OnExtentChanged? And I do not see support of MapView, nor Extent? What do I miss?
For example if I want to do something simular for OnZoom like for mouse events. For example in case handling OnMouseDown I perform:
mapRef.OnMouseDown(OnMapMouseDown);
private async Task OnMapMouseDown(MouseEvent e)
{
}

My mistake. ignore that email.
Peter Marshall
07979608139
@.***
On Tue, 7 Feb 2023 at 10:13, Cor @.***> wrote:

OnExtentChanged
In my .razor I use
<Map @ref https://github.com/ref="mapRef" @.***">
Map however does not support OnExtentChanged? And I do not see support of
MapView, nor Extent? What do I miss?
For example if I want to do something simular for OnZoom like for mouse
events. For example in case handling OnMouseDown I perform:
mapRef.OnMouseDown(OnMapMouseDown);
private async Task OnMapMouseDown(MouseEvent e)
{
}
—
Reply to this email directly, view it on GitHub
https://github.com/fis-sst/BlazorMaps/issues/27#issuecomment-1420521265,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKTAWKYAYGLVZM3DSXONDS3WWIN4FANCNFSM6AAAAAAUSZZIB4
.
You are receiving this because you commented.Message ID:
@.***>


fixed bug where app would have overflow render crash if start date was today

Hi, thanks the for the awesome library, I ran into a bug when displaying the heatmap with one days info (todays) it would cause a render overflow and red screen in my flutter app. Heres a snippet of code that I tested this change with.
import 'package:flutter/material.dart';
import 'package:flutter_heatmap_calendar/flutter_heatmap_calendar.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    DateTime now = DateTime.now();
    DateTime today = DateTime(now.year, now.month, now.day);

    return MaterialApp(
      home: Scaffold(
          appBar: AppBar(),
          body: HeatMap(
            showColorTip: false,
            startDate: today,
            datasets: {today: 1},
            colorsets: const {1: Colors.green},
          )),
    );
  }
}

Hi, thanks the for the awesome library, I ran into a bug when displaying the heatmap with one days info (todays) it would cause a render overflow and red screen in my flutter app. Heres a snippet of code that I tested this change with.
import 'package:flutter/material.dart';
import 'package:flutter_heatmap_calendar/flutter_heatmap_calendar.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    DateTime now = DateTime.now();
    DateTime today = DateTime(now.year, now.month, now.day);

    return MaterialApp(
      home: Scaffold(
          appBar: AppBar(),
          body: HeatMap(
            showColorTip: false,
            startDate: DateTime(today.year, today.month, today.day),
            datasets: {DateTime(today.year, today.month, today.day): 1},
            colorsets: const {1: Colors.green},
          )),
    );
  }
}

Sorry for the delayed review. I'll merge your PR to the main branch and update to pub.dev ASAP.
Anyway, thanks for your contribution :)


Adds unit test for fqdn.go

Adds the unit test for fqdn.go
for #4142
Signed-off-by: Harshitha U R<EMAIL_ADDRESS>

@urharshitha Unit test failed, please check.

/skip-all


fix(nginx) Use X-Forwarded-Host instead of Host

https://github.com/longhorn/longhorn/issues/5438

@Mergifyio backport v1.4.x v1.3.x


[JBEAP-20725] - Correct java version metadata in imagestream

https://issues.redhat.com/browse/JBEAP-20725
Signed-off-by: Ken Wills<EMAIL_ADDRESS>

@luck3y thanks!


Add searchbar in the solutions component

To have a generic search functionality to filter out solutions

Work completed:
Added search input in applications. Now the cards list will be filtered based on the search term.

version : 385928e
network : devnet
search in now added and functional

https://app.testlodge.com/a/26076/projects/40893/suites/234374?expand_section=422332#case_4246719

Verified:

Devnet
d2d2622


Massive styles refactoring

This is still a work-in-progress as I attempt to abstract away some more stuff.
For the most part everything seems to be working; I might have missed something though.
Also fixes the home page which broke after #15.
I'll document this PR tomorrow (hopefully).

At this point everything is coming along nicely.  The biggest problem remaining is the .infobox class.  It was initially geared specifically towards the homepage Q&A section, but the contact and FAQ pages are also using it, and there's no (easy) way to make it adapt to all three use-cases; it's already way too generic and canceling out font-related attributes.  I plan to separate it into different components (see enclosed-box for an example) so that each section can only use what it needs to.
With that in mind, here's what I noticed that still needs fixing before this PR can be considered for merging:

[ ] Q&A boxes on the homepage are misaligned; the first one is "taller" than the other two
[ ] Poiret One font for h2 question tags on the home and Contact pages are being overridden by .infobox and defaulting to Helvetica instead
[ ] The News page is styled differently than all the other pages (it's using black Poiret One for plain text and doesn't have a background)

Only thing left is the Q&A boxes now.

Ready for review! :rocket:

While we're restyling, it might be a good idea to make link tags consistent. Some pages have href before rel while others have rel before href.

@threedot14 fixed!

Looks good. Merging now.


[ISSUE #5305] fix proxy TopicRouteService cache bug

Make sure set the target branch to develop
What is the purpose of the change
XXXXX
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #5306 (87b1a68) into develop (8d10c96) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@              Coverage Diff              @@
##             develop    #5306      +/-   ##
=============================================
- Coverage      43.20%   43.19%   -0.01%
- Complexity      7807     7809       +2
=============================================
  Files            998      998
  Lines          69451    69454       +3
  Branches        9169     9169
=============================================
+ Hits           30003    30004       +1
- Misses         35682    35683       +1
- Partials        3766     3767       +1

Impacted Files
Coverage Δ

...ocketmq/proxy/service/route/TopicRouteService.java
61.81% <0.00%> (-3.57%)
:arrow_down:

...ketmq/common/protocol/body/ConsumerConnection.java
95.83% <0.00%> (-4.17%)
:arrow_down:

...a/org/apache/rocketmq/filter/util/BloomFilter.java
60.43% <0.00%> (-2.20%)
:arrow_down:

...mq/store/ha/autoswitch/AutoSwitchHAConnection.java
70.43% <0.00%> (-1.35%)
:arrow_down:

.../apache/rocketmq/store/ha/DefaultHAConnection.java
65.86% <0.00%> (-1.21%)
:arrow_down:

.../rocketmq/broker/filter/ConsumerFilterManager.java
72.19% <0.00%> (-0.90%)
:arrow_down:

...rocketmq/broker/processor/PopMessageProcessor.java
37.63% <0.00%> (-0.54%)
:arrow_down:

...che/rocketmq/test/lmq/benchmark/BenchLmqStore.java
50.24% <0.00%> (-0.50%)
:arrow_down:

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
69.37% <0.00%> (-0.47%)
:arrow_down:

...cketmq/broker/processor/PopBufferMergeService.java
37.44% <0.00%> (ø)

... and 8 more

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


Private access modifier inside of Struct definition breaks instantiation

Input
→ View on sorbet.run
# typed: true

Foo = Struct.new(:foo) do
  private
end

Foo.new

Observed output
editor.rb:7: Non-private call to private method initialize on Foo https://srb.help/7031
     7 |Foo.new
        ^^^^^^^
    editor.rb:3: Defined in Foo here
     3 |Foo = Struct.new(:foo) do
     4 |  private
     5 |end
Errors: 1

Expected behavior
No error, because MRI doesn't complain about a private method call:
irb(main):001:0> Foo = Struct.new(:foo) do
irb(main):002:1*   private
irb(main):003:1> end
=> Foo
irb(main):004:0>
irb(main):005:0> Foo.new
=> #<struct Foo foo=nil>

Running into this as well.

Just for completeness, this should have been fixed by #4245.


Move kn-source-pkg here

As per recent discussion and previous, move code from https://github.com/maximilien/kn-source-pkg to here under pkg so it can be used by kn-source-kafka and other plugins
cc: @rhuss @dsimansk

/remove-lifecycle stale

/remove-lifecycle stale

/assign @vyasgun


Fix for issue #126

These changes should provide a fix for #126.
Now, this is my very first pull request on github, so there certainly are things that still need to be changed. But this version should give you enough information to decide what needs to be done next.
Of course, the './issue-126' folder needs to be removed before merging this branch. I've left it in just in case you wanted to see for yourself that the changes fix the issue I have described. Same with './.gitignore'.
I have added many comments to the source code, but I don't mind if you want to remove them again. As long as they are in, they should help you to understand what is going on.
Please let me know if you need more info on these changes.
Please also test the changes with another template engine. I have no idea of how to use another one other than 'handlebars'. But I am pretty sure that it won't make much of a difference.

I think I still got the issue-126 / Test 3.


Added missing import on index.d.ts

Added the Leaflet missing import to typescript definitions

I have problem with this. I wonder when it's released?

You can use the new beta 2.0


error reading arrow file

Hello,
An error was encountered when I read the arrow file with the following input:
adata = ArchR_h5ad.read_arrow('S5.arrow', use_matrix="TileMatrix")
Was it due to a lot of variables with 0 counts in the TileMatrix?  Thank you.

Reading ArchR TileMatrix to AnnData
Chromosomes:   0%|          | 0/23 [00:00<?, ?it/s]
Traceback (most recent call last):
File "", line 1, in
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_read_arrow_to_adata.py", line 66, in _read_arrow_to_adata
arrow.to_adata(use_matrix=use_matrix, write_h5ad=write_h5ad)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_Arrow.py", line 52, in to_adata
self._adata = _compose_anndata(DataDict=self._DataDict,
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_compose_anndata.py", line 27, in _compose_anndata
adata = _add_obs_var(adata, metadata, feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_add_obs_var.py", line 49, in _add_obs_var
adata.var = _add_var(feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 882, in var
self._set_dim_df(value, "var")
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 798, in _set_dim_df
value_idx = self._prep_dim_index(value.index, attr)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 810, in _prep_dim_index
raise ValueError(
ValueError: Length of passed value for var_names is 11627605, but this AnnData has shape: (1590, 6062095)

@pchiang5 thanks for your message - it's hard for me to say without more information. Can you provide some details about the # of cells / features expected based on the TileMatrix as observed in the ArchR API? Do you have success when running the same function but for the GeneScoreMatrix? I.e.:
adata = ArchR_h5ad.read_arrow('S5.arrow', use_matrix="GeneScoreMatrix")

Hi @mvinyard
Using GeneScoreMatrix also encountered a similar error (please see below). Because the arrow file was created with custom genome/annotation files of both human (chr) and mouse (mchr) sources. I wonder if the unconventional mchr was the culprit of these errors because the chromosome to read shall be > 23 and the actual tile/genescore var_names in the AnnData files were around half of those in the arrow file.
adata = ArchR_h5ad.read_arrow('S5.arrow', use_matrix="TileMatrix")

Reading ArchR TileMatrix to AnnData
Chromosomes:   0%|          | 0/23 [00:00<?, ?it/s]
Traceback (most recent call last):
File "", line 1, in
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_read_arrow_to_adata.py", line 66, in _read_arrow_to_adata
arrow.to_adata(use_matrix=use_matrix, write_h5ad=write_h5ad)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_Arrow.py", line 52, in to_adata
self._adata = _compose_anndata(DataDict=self._DataDict,
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_compose_anndata.py", line 27, in _compose_anndata
adata = _add_obs_var(adata, metadata, feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_add_obs_var.py", line 49, in _add_obs_var
adata.var = _add_var(feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 882, in var
self._set_dim_df(value, "var")
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 798, in _set_dim_df
value_idx = self._prep_dim_index(value.index, attr)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 810, in _prep_dim_index
raise ValueError(
ValueError: Length of passed value for var_names is 11627605, but this AnnData has shape: (1590, 6062095)

adata = ArchR_h5ad.read_arrow('S5.arrow', use_matrix="GeneScoreMatrix")

Reading ArchR GeneScoreMatrix to AnnData
Chromosomes:   0%|          | 0/23 [00:00<?, ?it/s]
Traceback (most recent call last):
File "", line 1, in
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_read_arrow_to_adata.py", line 66, in _read_arrow_to_adata
arrow.to_adata(use_matrix=use_matrix, write_h5ad=write_h5ad)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_main/_Arrow.py", line 52, in to_adata
self._adata = _compose_anndata(DataDict=self._DataDict,
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_compose_anndata.py", line 27, in _compose_anndata
adata = _add_obs_var(adata, metadata, feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/ArchR_h5ad/_compose_adata/_add_obs_var.py", line 49, in _add_obs_var
adata.var = _add_var(feature_df)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 882, in var
self._set_dim_df(value, "var")
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 798, in _set_dim_df
value_idx = self._prep_dim_index(value.index, attr)
File "/home/pc/miniconda3/envs/snap/lib/python3.10/site-packages/anndata/_core/anndata.py", line 810, in _prep_dim_index
raise ValueError(
ValueError: Length of passed value for var_names is 115928, but this AnnData has shape: (1590, 60058)

Thanks for this additional info. I think you are right; the unconventional concatenated human/mouse organization is most likely causing the issue. I see two options: (1) separate the objects in ArchR before conversion or (2), supplied with an example dataset that reproduces the error, I could look into extending the tool to include such a case. Not sure if this is trivial or more involved without further investigation. Let me know what you think

error reading arrow file #3
@mvinyard
Please see below for the example dataset. Thank you.
https://www.dropbox.com/s/qa54nnh6g2dgu8k/S1.arrow?dl=0
On Fri, Oct 14, 2022 at 12:16 PM Michael Vinyard @.***>
wrote:

@pchiang5 https://github.com/pchiang5 Thanks for this additional info.
I think you are right; the unconventional concatenated human/mouse
organization is most likely causing the issue. I see two options: (1)
separate the objects in ArchR before conversion or (2), supplied with an
example dataset that reproduces the error, I could look into extending the
tool to include such a case. Not sure if this is trivial or more involved
without further investigation. Let me know what you think
—
Reply to this email directly, view it on GitHub
https://github.com/mvinyard/ArchR-h5ad/issues/3#issuecomment-1278459639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG6RZ3IWFLSKVMZ2XPEBLK3WDDNBJANCNFSM6AAAAAAREC7SBY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@mvinyard
Thank you for your willingness to help. I sent you the dataset in the previous reply. Please let me know if you miss the link.


Testing for @saphe/react-table

Codecov Report

Merging #40 (7cab35f) into main (b0ea7f9) will increase coverage by 31.51%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main      #40       +/-   ##
===========================================
+ Coverage   43.37%   74.88%   +31.51%
===========================================
  Files          40       39        -1
  Lines         664      665        +1
  Branches      190      206       +16
===========================================
+ Hits          288      498      +210
+ Misses        376      147      -229
- Partials        0       20       +20

Impacted Files
Coverage Δ

packages/react-form/src/utils/helperTypes.ts
0.00% <ø> (ø)

packages/react-table/src/useVisibility.ts
100.00% <100.00%> (+100.00%)
:arrow_up:

packages/react-form/src/utils/formHelpers.ts
66.66% <0.00%> (ø)

packages/react-form/src/utils/validationHelpers.ts
15.21% <0.00%> (ø)

packages/react-table/src/useSort.ts
19.67% <0.00%> (+19.67%)
:arrow_up:

packages/react-table/src/useSearch.ts
23.43% <0.00%> (+23.43%)
:arrow_up:

packages/react-table/src/usePagination.ts
62.50% <0.00%> (+62.50%)
:arrow_up:

packages/react-table/src/useDefaultValues.ts
92.85% <0.00%> (+92.85%)
:arrow_up:

packages/react-table/src/useTable.ts
94.11% <0.00%> (+94.11%)
:arrow_up:

... and 5 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 568bcdd...7cab35f. Read the comment docs.


chore: pin eslint-plugin-import to 2.5.0

Coverage remained the same at 93.382% when pulling 668973dafa5d64a5f1160d57f94ba2439f4328a5 on greenkeeper/eslint-plugin-import-pin-2.5.0 into 2bf8dfa803647ca9ebf9e1ba090b9d5395257c34 on master.

Fixes https://github.com/bucharest-gold/fidelity/issues/51


Feature Request:  Synchronous composition

I am the tech lead for a fairly large application (10 developers) which uses durandal.   In general, we are extremely happy with the framework.   Our biggest headache however is the jerky/janky rendering experience which stems from the asynchronous nature of the compose and widget bindings.   The issue is mostly imperceptible in demo / hello world applications, but in large mature applications it becomes overwhelming.
This was also an issue with the recently-released components feature of knockout, for which I added a feature request here: https://github.com/knockout/knockout/pull/1504  (a fix has been added to their development branch, but not yet released).   Please read that issue for good background information on the problem.
This is helpful, but obviously not ideal since knockout widgets do not participate in the composition/activation lifecycle or some of the other goodness that the compose binding gives us.
I believe that in order to receive good rendering efficiency, widgets and the compose binding must render synchronously when possible (such as when the application has been bundled).  The ability to render the page top-to-bottom will not only get rid of the janky loading experience, but should greatly improve the performance hit of layout thrashing.
Note: Obviously, viewmodels which return a promise from activate must be loaded asyncronously - this issue mainly concerns UIs built up from a deep graph of viewmodel components.

We also struggle with this. Our very slight workaround for the rendering is to make our widgets etc render lists, rather than individual items so that there are fewer compose calls.
Today I have an issue with iOS devices and setting form field focus programatically which must be done on the same stack frame as a user interaction event. Unfortunately due to the async nature of compose, the element isn't rendered during that stack frame.

Here's a branch experimenting with synchronous composition. Seems to work ok on my project so far.
If someone could explain what all the setTimeouts were solving I'd be very grateful.
https://github.com/dancras/Durandal/tree/synchronous-composition

@srowan have you reviewed @dancras ' repo above?

Closing since Durandal has been superseded by Aurelia just shy of four years ago.


Example app's webview_test.dart is flaky

Most probably the cause is that we try to tap on the view before it exists. We should use something like waitUntilVisible() with a configurable timeout.
Should be fixed by #409.
Crash logs
Running webview_test.dart with flutter_driver on iPhone 14...
VMServiceFlutterDriver: Connecting to Flutter application at http://<IP_ADDRESS>:50764/rRNELTENjB8=/
VMServiceFlutterDriver: Isolate found with number:<PHONE_NUMBER>536967
VMServiceFlutterDriver: Isolate is paused at start.
VMServiceFlutterDriver: Attempting to resume isolate
Patrol: creating NativeAutomator, host: localhost, port: 8081, packageName: pl.leancode.patrol.example, bundleId: pl.leancode.patrol.Example
Patrol: Initializing PatrolBinding...
00:00 +0: navigates through the app using only native semantics
VMServiceFlutterDriver: Connected to Flutter application.
Patrol: tap() started
WebView created
WebView is loading (progress : 10%)
Patrol: tap() succeeded
Patrol: tap() started
Patrol: tap() failed
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following PatrolActionException was thrown running a test:
Patrol action failed: tap() failed with code NOT_FOUND (view with text "Accept cookies" in app
"pl.leancode.patrol.Example" doesn't exist)
When the exception was thrown, this was the stack:
#0      NativeAutomator._wrapRequest (package:patrol/src/native/native_automator.dart:137:7)
<asynchronous suspension>
#1      NativeAutomator.tap (package:patrol/src/native/native_automator.dart:370:5)
<asynchronous suspension>
#2      PatrolX.waitAndTap (file:///Users/bartek/dev/leancode/patrol/packages/patrol/example/integration_test/webview_test.dart:36:5)
<asynchronous suspension>
#3      main.<anonymous closure> (file:///Users/bartek/dev/leancode/patrol/packages/patrol/example/integration_test/webview_test.dart:20:7)
<asynchronous suspension>
#4      patrolTest.<anonymous closure> (package:patrol/src/custom_finders/common.dart:60:7)
<asynchronous suspension>
#5      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:171:15)
<asynchronous suspension>
#6      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:841:5)
<asynchronous suspension>
The test description was:
  navigates through the app using only native semantics
════════════════════════════════════════════════════════════════════════════════════════════════════
00:01 +0: navigates through the app using only native semantics [E]
  Test failed. See exception logs above.
00:01 +0 -1: (tearDownAll)
Warning: integration_test plugin was not detected.
If you're running the tests with `flutter drive`, please make sure your tests
are in the `integration_test/` directory of your package and use
`flutter test $path_to_test` to run it instead.
If you're running the tests with Android instrumentation or XCTest, this means
that you are not capturing test results properly! See the following link for
how to set up the integration_test plugin:
https://flutter.dev/docs/testing/integration-tests#testing-on-firebase-test-lab
00:01 +1 -1: Some tests failed.
Failed to stop app
flutter_driver exited with code 1
See the logs above to learn what happened. If the logs above aren't useful then it's a bug – please report it.

Should be fixed by #437


As a user I want to view the tasks that I have added easily

So that I can retrieve the wanted information with little effort.

#82

Finished this feature


Set the default debounce time for text controls to ZERO

NO TP
Quick change to set the default debounce time for text controls to ZERO

@darwinlopez: think it's worth editing your commit message to say why this change is necessary?

@rockabox/frontend do we still need this change, or can we burn it?


Pip installing can't install the required packages

I see that the this issue has occurred before and tried to fix with c29e127053694d8c055d4fca36617f5d38bc5383, but don't work. I would recommend using poetry or static dependencies.
Collecting lshashpy3
  Downloading lshashpy3-0.0.8.tar.gz (9.0 kB)
  Preparing metadata (setup.py) ... error
  error: subprocess-exited-with-error

  × python setup.py egg_info did not run successfully.
  │ exit code: 1
  ╰─> [10 lines of output]
      Traceback (most recent call last):
        File "<string>", line 2, in <module>
        File "<pip-setuptools-caller>", line 34, in <module>
        File "/private/var/folders/ds/rtdqdzb153306j602rcl9s740000gn/T/pip-install-gzk7zag1/lshashpy3_4fd763b4e79d4694a442efe6e39318d4/setup.py", line 3, in <module>
          import lshashpy3
        File "/private/var/folders/ds/rtdqdzb153306j602rcl9s740000gn/T/pip-install-gzk7zag1/lshashpy3_4fd763b4e79d4694a442efe6e39318d4/lshashpy3/__init__.py", line 5, in <module>
          from .lshash import *
        File "/private/var/folders/ds/rtdqdzb153306j602rcl9s740000gn/T/pip-install-gzk7zag1/lshashpy3_4fd763b4e79d4694a442efe6e39318d4/lshashpy3/lshash.py", line 9, in <module>
          from future import standard_library
      ModuleNotFoundError: No module named 'future'
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
error: metadata-generation-failed

× Encountered error while generating package metadata.
╰─> See above for output.

note: This is an issue with the package mentioned above, not pip.
hint: See above for details.

Did this ever get a fix?


ci: configure Dependabot version updates

Configure Dependabot for daily updates to container images in Dockerfiles.
Related-to: k3s-io/k3s#7440

Please let me know if labels and reviewers also needs to be configured.

I don't think there's much point in doing dependabot bumps for this. We don't publish a docker image from this repo; the base image is just used as a starting point for the buildroot environment, which actually builds its toolchain and then uses that to build the target environment. Given that this is already a time-consuming process that we really only need to do when intentionally updating the buildroot config, I don't want anything kicking off a bunch of superfluous PRs.

Oh, okay. I saw the issue(linked in the PR) with a label "good-first-issue" and wanted to help. Perhaps it needs a little update from someone on k3s team.


Add Unit test for Building and deploying multi-stage Recsys nbs

looks good - just the change of the dynamic path for the 2nd notebook

thanks. agreed. I tried, it did not work but I will re-try now :)


Pixi 4 flickering on Silk browser

When running a pixi example in the Silk browser on a Kindle Fire the canvas flickers. We don't experience this on any other platforms or browsers.
You can see it using this example:
https://pixijs.github.io/examples/#/basics/basic.js

Ok, so there are some switches you can try:
First is "antialias". I dont know why, but it forces browsers to behave differently as a side effect.
var app = new PIXI.Application(800, 600, {backgroundColor : 0x1099bb , antialias: true });

Second is preserveDrawingBuffer which eats the performance but affects rendering process seriously. Use with antialias.
var app = new PIXI.Application(800, 600, {backgroundColor : 0x1099bb , antialias: true, preserveDrawingBuffer: true });

Third is calling "renderer.gl.flush" just after rendering process finished.
Either call it after "Render" if you use simple renderer
renderer.render(stage);
renderer.gl.flush();

Either hack the app render method
app = new Application(...,. { autoStart: false }) ;
app.render = function() {
    this.renderer.render(this.stage);
    this.renderer.gl.flush();
}

Thanks, the perserveDrawingBuffer change fixed it. Tried it without the antialias flag and it worked that way too, so only have the one.

What about single antialias? Dont forget there's performance hit with preserveDrawingBuffer. Do not enable it on other devices/browsers ;)

Antialias by itself didn't fix the problem.
Yeah, we've limited it to just the Silk browser. It is a significant framerate hit.

Good luck! Write us more issues!

Hey there,
I've also been noticing this on and off, in Chrome too. Funnily enough, adding { transparent: true } to the renderer fixes it for me.

{ transparent: true } also fixes problem in Silk browser on Kindle Fire for me, without the performance impact of preserveDrawingBuffer - nice one : )

"transparent" has a performance impact but its small compared to preserveDrawingBuffer.
Thanks, @mafsays

I have been months trying to solve this problem, it was on Chrome on Huawei, and { transparent: true } solved it. Thanks a lot


Improve the trait input field when creating segment definitions

The trait field:

Should be better/more intelligent. It should be a combined autocomplete/dropdown/free text element.
Populating the drop down
With the global API this is going to be tricky as we don't store a canonical list of trait keys. A 99% solution would be to grab ~50 identities at random and create a list of distinct trait keys from that data. This would be a significant improvement over the current implementation.
We would need to create an API endpoint to return this list.

I'll need API before I can work on this

Hello, is there are any updates regarding the resolution of this issue?

We're not currently working on this feature I'm afraid. It's technically complex with our Edge API.


Console doesn't log errors

Use of RavenErrorHandler (in app.module.ts) causes that errors are sent to sentry, but not logged into console. Because of this line
we're not using default Angular ErrorHandler, which is why we don't see errors logs.
For example here we don't see any errors in console, but if we check network tab in devtools we can find out that they were sent to the sentry.
Maybe we should use Sentry to track errors only in production mode?

Good point!
In line https://github.com/CodeForPoznan/volontulo/blob/master/frontend/src/app/app.module.ts#L166 we're using RavenErrorHandler instead of Angular ErrorHandler. It should be selected based on the environment variable production: boolean.

Anyone doing that or can I take it?

I have plan to do that, but You're welcome to jump into it. @CorneeSean

I will not start it earlier than 1st or 2nd of May, as I will be out of the country. If you'll have time and you'll feel a hunch to do this then please do not hesitate. I can do it after I come back.


Evaluate moving to mbedTLS v2.16 branch

mbedTLS has designated its latest update v2.16.0 as of 2018-12-21 the new long-term stable production branch. We should evaluate the impact of moving to this new release:

[ ] Reconcile breaking changes in the API surface with OE SDK use
[ ] Evaluate whether this poses issues for link-time mbedTLS replacement for projects using mbedTLS updated with Everest TLS crypto.
[ ] Evaluate cost-benefit of moving to 2.16
[x] Determine if 2.7 branch is being EOL and timeframe for that.
[ ] Request OSS SDL security review of new mbedTLS version.

Per @achamayou, there's a request to move to a newer version of mbedTLS for the expanded feature set as discussed in closed PR #1414.

@CodeMonkeyLeet as far I know, the mbedTLS + Everest snapshot in use (discussed here https://github.com/Microsoft/openenclave/pull/1419) is based on 2.16, so I would not expect us to hit any issues. I'd be happy to help with testing that though, and report any problems.
We have been using a build of OE with this snapshot successfully for some time now (with a source swap though, not a link-time swap), so I do not think any OE code changes will be necessary (or at least not on Linux).

Determine if 2.7 branch is being EOL and timeframe for that.
Mbed TLS 2.7
As such, we are pleased to announce that Mbed TLS 2.7 will now become our next LTS (Long Term Support) branch, and that we will continue to maintain it for the next 3 years, until at least February 2021.
https://tls.mbed.org/tech-updates/blog/our-next-lts-branch-mbedtls-2.7

Announcing Mbed TLS 2.16 Long Term Support
We are pleased to announce that Mbed TLS 2.16.0 will be the basis of our next Long Term Support branch, and will be maintained for at least 3 years up until the end of 2021.
During that time, no new features will be added to the branch, the API and ABI will be maintained, and only bug fixes will be made to maintenance releases of 2.16.0.

After some tweaking we were able to run tests but main issue is the PASS percentage, only 77% of tests are Passing using this version.
The following tests FAILED:
14 - tests/mbed/asn1write (Child aborted)
15 - tests/mbed/base64 (Child aborted)
16 - tests/mbed/ccm (Child aborted)
17 - tests/mbed/cipher.aes (Child aborted)
18 - tests/mbed/cipher.ccm (Child aborted)
19 - tests/mbed/cipher.des (Child aborted)
20 - tests/mbed/cipher.gcm (Child aborted)
21 - tests/mbed/cipher.padding (Child aborted)
22 - tests/mbed/cmac (Child aborted)
23 - tests/mbed/ctr_drbg (Child aborted)
24 - tests/mbed/debug (Child aborted)
25 - tests/mbed/des (Child aborted)
27 - tests/mbed/ecdsa (Child aborted)
28 - tests/mbed/ecp (Child aborted)
29 - tests/mbed/entropy (Child aborted)
30 - tests/mbed/error (Child aborted)
31 - tests/mbed/gcm.aes128_en (Child aborted)
32 - tests/mbed/gcm.aes192_en (Child aborted)
33 - tests/mbed/gcm.aes256_en (Child aborted)
34 - tests/mbed/gcm.aes128_de (Child aborted)
35 - tests/mbed/gcm.aes192_de (Child aborted)
36 - tests/mbed/gcm.aes256_de (Child aborted)
37 - tests/mbed/hmac_drbg.misc (Child aborted)
38 - tests/mbed/hmac_drbg.no_reseed (Child aborted)
39 - tests/mbed/hmac_drbg.nopr (Child aborted)
40 - tests/mbed/hmac_drbg.pr (Child aborted)
42 - tests/mbed/mdx (Child aborted)
43 - tests/mbed/mpi (Child aborted)
44 - tests/mbed/pem (Child aborted)
45 - tests/mbed/pkcs1_v15 (Child aborted)
46 - tests/mbed/pkcs1_v21 (Child aborted)
47 - tests/mbed/pkcs5 (Child aborted)
48 - tests/mbed/pk (Child aborted)
49 - tests/mbed/pkparse (Child aborted)
50 - tests/mbed/shax (Child aborted)
51 - tests/mbed/ssl (Child aborted)
52 - tests/mbed/rsa (Child aborted)
53 - tests/mbed/version (Child aborted)
54 - tests/mbed/x509parse (Child aborted)

It appears that these tests are either using readv or another unsupported File I/O functions. The test should be enhanced to support these. But the upcoming IO Feature will provide support for all File I/O. So, good to hold off until the feature I/O branch is merged to master.

@anitagov thank you for the input; will be holding until IO Feature is merged.

I have updated the PR #1716 with the following changes:

Mbed TLS 2.16.1 LTS
rebase on top of latest master changes

After re-testing nothing changed regarding the list of failing tests.

Syncing the issue milestone with the pull request milestone as changed by Bruce.
/cc @yakman2020


Add Branch Protections and Checks

Check Source Branch builds before new commits are merged

Make source branch only allow pull requests, no direct changes to master

Make so that master branch can only be updated by our deployment bot

other?

The second item means that nobody can push directly to source but instead we have to make PRs right?
If so, I think these are all a good ideas.
But I don't have permissions to set that up.

@JoshOrndorff try again now?

@shawntabrizi Yep, I have access now, Thanks.
[ ] Check Source Branch builds before new commits are merged
[x] Make source branch only allow pull requests, no direct changes to master
[x] Make so that master branch can only be updated by our deployment bot

The build checks are not actually completing

The issue was that branches in forks were not triggering the build. @shawntabrizi has confirmed it's resolved now.


Add AWS proton support

The pod currently does not support AWS proton. Locally, I did manage to fix the issue by updating the dependency in deps.edn, adding the dependency to aws-libs.edn and adding :proton to the list in  resources/aws-services.edn, followed by locally compiling the image and running that. If you want I can do a PR?
 I thought this would fetch all the latest stuff, but apparently it's missing something. Could you take a look there and improve the script? After that we can run it and update the pod.

It seems the deps.edn already contains proton:
$ less deps.edn | grep proton
        com.cognitect.aws/proton                           {:mvn/version "813.2.963.0" :aws/serviceFullName "AWS Proton"}

I don't think aws-lib.edn is used anywhere.

There is a list of keys in aws-services.edn but I don't know how that was generated. It should probably be re-generated in script/update-deps.clj but from which data?
(def services (into (sorted-set) (clojure.edn/read-string (slurp (io/resource "aws-services.edn")))))

(def *clients (atom {}))

(defn get-client [config]
  (get @*clients (get config ::client-id)))

(defn -client [{:keys [api] :as config}]
  (if-not (contains? services api)
    (throw (ex-info (str "api " api " not available") {:available services}))
    (let [config (assoc config
                        :http-client @http-client)
          client (aws/client config)
          client-id (java.util.UUID/randomUUID)]
      (swap! *clients assoc client-id client)
      {::client-id client-id})))

This list was put here by @jeroenvandijk so hopefully he can enlighten us :).

Thanks a lot! I'll release a new version.

Released to the pod registry.

Thank you @jeroenvandijk  and @borkdude


[STORM-1057] Add throughput metrics to spouts/bolts and display them on web ui for 1.x-branch

Hi @HeartSaVioR, @revans2, @harshach,@d2r,@unsleepy22,
I upmerge PR apache/storm#753 to 1.x-branch.
Sorry for the delay.
Please review.
Thanks.

@HeartSaVioR Can you review this PR? Thanks.

Can I create a new PR based on the latest master version rather than upmerging this RP to master? The former would be much easier, since a lot of changes have been made from 1.x to 2.x.
Sent from my iPhone

On 28 Sep 2016, at 8:02 AM, Jungtaek Lim<EMAIL_ADDRESS>wrote:
@wangli1426
Sorry I haven't had time to review this right now. I'll take a look when I get time.
Btw, since you're modifying ported clojure files, you need to port your change to master as well. If you don't want to address both Clojure and Java, addressing master is higher priority.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Yes sure. You don't essentially need to upmerge/rebase old pull request if you want to provide new pull request based on current.

Thanks for the prompt response. I will create a new pull request for master asap.
Sent from my iPhone

On 28 Sep 2016, at 9:09 AM, Jungtaek Lim<EMAIL_ADDRESS>wrote:
Yes sure. You don't essentially need to upmerge/rebase old pull request if you want to provide new pull request based on current.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


[Question] How to install on Windows 8.1?

Is there a guide or tutorial on how to install openswoole on Windows 8.1?

WSL is required when installing on Windows. Complete openswoole installation guide: https://openswoole.com/docs/get-started/installation

But where is no WSL on Windows 8.1.
Is it possible to install using cygwin perhaps?

It is recommended to use Docker or virtual machine if WSL is not supported.


What is the difference between AlamofireObjectMapper and this ObjectMapper?

What is the difference between AlamofireObjectMapper and this ObjectMapper?

ObjectMapper is a library that handles conversion of JSON to model classes.
Alamofire is an HTTP Request library
AlamofireObjectMapper is an extension for Alamofire that uses ObjectMapper to automatically map requests that return JSON in models.


python38Packages.progressbar2: 3.53.2 -> 3.53.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.progressbar2 is: "Text progressbar library for python"
meta.homepage for python38Packages.progressbar2 is: "https://progressbar-2.readthedocs.io/en/latest/"
meta.changelog for python38Packages.progressbar2 is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.53.3 with grep in /nix/store/sd2jkis5pj969y09draf5615s7gsjfmy-python3.8-progressbar2-3.53.3

found 3.53.3 in filename of file in /nix/store/sd2jkis5pj969y09draf5615s7gsjfmy-python3.8-progressbar2-3.53.3

directory tree listing: https://gist.github.com/fc8113aeb252d552a590ae83a30b3ede

du listing: https://gist.github.com/7ad1198a9a060c28f2f714353432b727

Rebuild report (if merged into master) (click to expand)

48 total rebuild path(s)

12 package rebuild(s)

12 x86_64-linux rebuild(s)
12 i686-linux rebuild(s)
12 x86_64-darwin rebuild(s)
12 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
portmod
python38Packages.angr
python38Packages.angrop
python38Packages.baselines
python38Packages.gradient
python38Packages.progressbar2
python39Packages.angr
python39Packages.angrop
python39Packages.baselines
python39Packages.gradient
python39Packages.progressbar2
truvari

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/sd2jkis5pj969y09draf5615s7gsjfmy-python3.8-progressbar2-3.53.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.progressbar2 https://github.com/r-ryantm/nixpkgs/archive/9c7aa4b83cbef56daae1f479e1fdc75d1895d98d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/sd2jkis5pj969y09draf5615s7gsjfmy-python3.8-progressbar2-3.53.3
ls -la /nix/store/sd2jkis5pj969y09draf5615s7gsjfmy-python3.8-progressbar2-3.53.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @ashgillman @turion for testing.

Result of nixpkgs-review pr 137995 at 9c7aa4b8 run on aarch64-linux 1

1 package marked as broken and skipped:

truvari

6 packages failed to build:

python38Packages.angr
python38Packages.angrop
python38Packages.gradient
python39Packages.angr
python39Packages.angrop
python39Packages.gradient

2 packages skipped due to time constraints:

python38Packages.baselines
python39Packages.baselines

3 packages built successfully:

portmod
python38Packages.progressbar2
python39Packages.progressbar2

1 suggestion:

warning: unused-argument
Unused argument: python.
Near pkgs/development/python-modules/progressbar2/default.nix:2:3:
  |
2 | , python
  |   ^

Note that build failures may predate this PR, and could be nondeterministic or hardware dependent.
Please exercise your independent judgement. Does something look off? Please file an issue or reach out on IRC.


Add UFO SA support

See https://github.com/buttplugio/buttplug-csharp/blob/master/Buttplug.Server/Bluetooth/Devices/VorzeSA.cs. Really only needs to add command format change (device choice) and extra device name.

Was this fixed by 863a8aac21e18d131a20712adc7db1d6d69d7c06

Yup.


Implement sign in with google

[X] Added debug SHA1 key to firebase

https://developers.google.com/identity/one-tap/android/get-saved-credentials?authuser=1
https://firebase.google.com/docs/auth/android/google-signin

https://developers.google.com/drive/api/v3/about-sdk
https://developers.google.com/drive
https://stackoverflow.com/questions/27409953/uploading-video-to-google-drive-programmatically-android-api
https://developers.google.com/drive/android
https://developers.google.com/drive/api/v3/appdata?authuser=1
upload file to google drive programmatically android

https://ammar.lanui.online/integrate-google-drive-rest-api-on-android-app-bc4ddbd90820
https://stackoverflow.com/questions/56949872/can-someone-provide-an-up-to-date-android-guide-for-google-drive-rest-api-v3

As there is not enough resource and more difficulty of handling the bugs arises due to this.
I am planning to currently use Firecloud Storage as we are not expecting much users in the initial phase


Pressing back in Chrome keeps values, but removes "example" class

form has examples enabled
user submits form
user presses back button
the inputs are filled, but without "example" class

Subsequent submit results with examples sent as data.
Chrome 15.0.874.121 m
jQuery Form Example Plugin 1.5
jQuery v1.6.4

I just pushed version 1.5.1; can you please see if this addresses your issue?

Yes, it's fixed (Y)


New version release

Hi,
Can you release to maven central new version with zabbix and influxdb support? :)

@yuesong :)

I'll try to do a release sometime next week!

Thanks in advance!

@mmatloka new release is on its way to maven central.

thanks!


Add alphabetizing of bibliography entries

Add an option to sort and store bibliography entries in alphabetical order.  Have options to sort by cite key, authors name, et cetera.

This option has been added.  Currently, you can sort by key and last name of the author.


Optimization - add restrictions to parameter values

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-15593

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-16184


Documented additional parameters

These are included in a recent PR (https://github.com/tiangolo/fastapi/pull/1166) but not in the docs yet.

response_model_exclude_none
response_model_exclude_defaults

Awesome! Thank you @wshayes ! :rocket: :cake:


Exporting in-memory data to buffer.

I would like to export my better-sqlite3 in-memory database to buffer, i know there is db.backup but its creating a file, i dont want it to create a file i just need the data in buffer.

Duplicate of https://github.com/JoshuaWise/better-sqlite3/issues/573


Replace crc passwd for nested crc jobs

Nested crc image has changed its passwd back to 1..8 and now legacy jobs are failing. we need to switchover to old passwd to unblock jobs.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

/approve
/lgtm


Google Maps draws blank on iOS

Describe the bug
The rendered map stays completely blank on iOS, works in web.
To Reproduce
https://github.com/alappe/capacitor_map_example should show the example. One needs to add an API key with iOS Maps SDK enabled , mine is missing the credit card for now.
Expected behavior
I'd expect to see a map on iOS.
Screenshots

Additional context
Tested on a real device, not simulator.

iOS Maps SDK enabled, mine is missing the credit card for now

As stated in the docs you need an active billing account. So likely that is what is going wrong here.

Thanks tafeln,
since the web version shows me a message that the billing is missing but works fine I asked around in Slack and somebody there told me that the native version should behave exactly the same. I will add billing and then close the issue if this solves it.

Adding billing to my account changed the web version to fully functional, whereas the iOS version stays blank. So this was not the issue.

That's why I included the minimal code example linked in my issue description.

I am sorry. I see that now. You are using a different package. You have installed https://www.npmjs.com/package/@capacitor/google-maps instead of https://www.npmjs.com/package/@capacitor-community/google-maps . Those are 2 different packages.


Monitoring & Istio system charts do not have min/max versions set

rancher/rancher:v2.3.4-rc8
Monitoring & Istio system charts should have min/max version set in the charts to only show the latest version.
When validating, check both fresh install & upgrade scenarios

https://github.com/rancher/system-charts/pull/147
https://github.com/rancher/system-charts/pull/148

We decided to close in favor of https://github.com/rancher/rancher/issues/24956 where we will track not just that fix but our testing efforts.


parsing table frame and grid attributes

A table's grid and frame attributes are not recognized:
[%autowidth, grid=none, frame=none]
|===
| AN    | Aṅguttara Nikāya
| Cv    | Cullavagga
| Dhp   | Dhammapada
|===

I think you're saying that the "none" value is not respected. Is that correct?

Yes, "none" certainly didn't work (default "topbot" and "rows" were used instead).


Any way to get this work with Security0

Hi,
Due to some problem, our ESP32 work only with Security0(i.e without any POP pin). Can this git work with that?
I run this code in Android, it can detect the ESP32, but it failed at POP PIN.
I have little knowledge on Flutter. If it works with Security1, get it to work with Security0 is simple.
Thanks;

I  tried the app with the ESP32 security 1 config like const char *pop = ""; , and it failed, i will try to fix that. I do not tried yet the security 0 yet, i will do this in a comming future.
Your request will be added to the package's TODO list. However, i recommend using the security 1, or security 2, but security 2 is also not implemented in this package yet. For obvious security reasons, do not use security 0 if you have the option to use 1, as security level 0 simply transmits plain text without encryption.

Reference:
https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/provisioning/provisioning.html#security-schemes

I tried the app with the ESP32 security 1 config like const char *pop = ""; , and it failed, i will try to fix that. I do not tried yet the security 0 yet, i will do this in a comming future.
Your request will be added to the package's TODO list. However, i recommend using the security 1, or security 2, but security 2 is also not implemented in this package yet. For obvious security reasons, do not use security 0 if you have the option to use 1, as security level 0 simply transmits plain text without encryption.

Reference: https://docs.espressif.com/projects/esp-idf/en/latest/esp32/api-reference/provisioning/provisioning.html#security-schemes

I try your code with Security1 and it works. However, my project require Arduino and so far i can't get Security1 to work in Arduino.

@garudaonekh
This package was created and tested with an ESP32, i have not yet tested it with Arduino.
Your PR will be included in the list, but I would like to inform you that it will not be a priority, since there are others more important before, such as the one to test and perform the example with other Bluetooth packages, being this the main one, and as you commented, I would like to try to add security 2, and if time permits, add security 0, since it is not a priority, since there is no security in the connection.


Misundestrand with the Architecture

Hello,
I don't understand the architecture of the project, where are the controllers and connection string ?
Can you please can you enlighten me ?
Thanks

@anasseb this is the frontend project.  Backend is here: https://github.com/fullstackhero/dotnet-webapi-boilerplate

@matthewcorven Thank you very much sir


Add support for AsyncAPI 2.0.0

Hey Mike!
This PR adds support for AsyncAPI 2.0.0 and removes support for AsyncAPI 1.x.
We're not supporting 1.x versions anymore but I'd understand if you don't want to remove support here. Up to you :)
PS: I don't know how to test this PR. If you could do it I'd be eternally grateful 😄

@MikeRalphson any chance of getting this merged please?

Yep, just want to re-add support for AsyncApi v1 if you use a filename like asyncapi1.yaml

Thanks @fmvilas !

Thank you, Mike!

Will be releasing the new version in a few minutes, will ping on Slack.

Did you use one of the command palette commands? What is the name of your input file?

Please move this to another ticket
On Fri, 20 Mar 2020, 15:00 Mike Ralphson<EMAIL_ADDRESS>wrote:

Did you use one of the command palette commands? What is the name of your
input file?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Mermade/openapi-lint-vscode/pull/18#issuecomment-601744821,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAUJMHINWYE3KNOXWFHANWTRIOAJ7ANCNFSM4I53MLAQ
.

moved here: https://github.com/Mermade/openapi-lint-vscode/issues/19


Custom rules table with xml files uploads and windup parsing

A valid xml file containing windup rules is entirely processed to provide source, target and rules tags.
Notes:

Upload File - Multiple component has the implementation for checking the file has a valid xml content and that the xml is valid per the requirements. Meanwhile it only logs it for now. Passing the file status back to is coming in following PR.
Need to implement delete action (trash icon)

LGTM


JUnit test fixes

gradle test now runs all tests
gradle test checks if http files fail or not

This is looking good to me :)
Thank you so much for contributing.

@all-contributors please add @gsobczyk for code


Hexo setup updates

My current setup: hexo-filter-mathjax and markdown-it: an alternative without pandoc dependence with the server-side rendering of math formula.

Later I used pandoc instead for right-click menu of MathJax.


Authentication Error with New Docker Install

I am new to silver bullet & trying to run via the following docker compose file
services:
  silverbullet:
    image: zefhemel/silverbullet:latest
    restart: unless-stopped
    environment:
    - SB_USER="admin:test1234"
    - PUID=1044
    - GUID=100
    volumes:
      - /volume2/docker-silverbullet/:/space
    ports:
      - 3007:3000

Also, /volume2/docker-silverbullet is indeed owned by user 1044 with gid of 100.
When I navigate to "http://serverurl:3007"  in a browser I get the notification "You are not authenticated, redirecting to login page... ". Clicking 'Ok' then redirects me to a login page. I put in the credentials from the compose file (user: admin, password: test1234)  but I get a "Invalid username or password" message.
I checked the logs from docker but only see the following:
Updating page list cache
Updating page list cache
Unauthorized access, redirecting to auth page
Unauthorized access, redirecting to auth page
Updating page list cache
Updating page list cache
Updating page list cache
.....

I tried rolling back the docker image to the previous version 0.5.8 but ran into issues before the server even started.
Do I have a configuration issue or is this a bug?

Try removing the quotes:
- SB_USER=admin:test1234

Yep, that was it.
Thank you


markdown syntax highlighting

It would be nice to have some basic syntax highlighting for markdown/multimarkdown in nvALT as it becomes difficult to read longer notes with a few links, heading, italics etc

:+1:

👍🏽


Added ScrollBar For Better User Experience

NOTE: I USED tailwind-scrollbar
FIRST YOU HAVE TO INSTALL IT BY "npm install tailwind-scrollbar" THEN MAKE SOME CHANGE IN "tailwind.config.JS"  AFTER
THAT MAKE CHANGE IN "profile.jsx"..
changed position of "Recent Pull Request" title to make sticky..

@jalaym825 Please Review PR.

It's there already.
https://github.com/user-attachments/assets/1d177fb5-f0b6-4c63-ba13-f90eccb3820f


temp fix for symbolic shape view add [pr]

something is still wrong with symbolic shape shrink, but it should not recurse forever

ah thanks xd, I'll rebase to see the new error
is it a wrong add?

not sure what's considered a wrong add, but add should give either a merged view or None and never an error. feel free to add more tests and tag me if you see more

@Qazalin you might want to check how you got this View in the first place. It broke because vm1 has contiguous=False, even though it's contiguous, and also the mask is no-op. Creating the View with View.create when possible in the tests, which would canonicalize these.
i still think it's good to not have these function fail randomly so will keep this fix and the test

hmm - I'm 90% sure I just used ShapeTracker.from_shape() and __add__ of (some_st)+(some_st)
The scheduler doesn't ever construct Views manually. I'll trace back to how that View got created.


Using origins other than (0, 0, 0)?

I'm looking for options to show some point cloud with ipv.quickscatter(), but find that it will always use (0, 0, 0) as the origin. Is there a way to tell the output widget to show only some specific interval on the three axes? If there is not I'd need to shift the data and might end up with misleading axes values.

Ok, I've discovered xlim() and friends...

👍


Recipe Overlap in GTCE Integration

In NuclearCraft (both normal and Overhauled), there is a recipe:
250mB Oxygen + 500mB Hydrogen -> 250mB Water
registered by NC to the GTCE Chemical Reactor.
This recipe overlaps with another recipe added by GTCE:
4B Hydrogen + 1B Oxygen + 1 Carbon (circuit 3) -> 1B Methanol.
The Water recipe will always be chosen, and Methanol is currently uncraftable using this recipe when NC is included with GTCE. If a Integrated circuit of anything other than a 3 were included, or if this recipe were removed, then the problem would be addressed.

NC (at least in the overhaul edition) adds a recipe to make methanol (250 mB) by reacting carbon monoxide (250 mB) and hydrogen (500 mB) together. Does that recipe still work for you?

@DStrand1 Fixed by generalising search for recipe conflicts :)