fix form not submitting

For some reason,
<label for="full-name">Full Name</label>
<input type="text" name="name" id="full-name" placeholder="First and Last" required="">

doesn't work but
<label>
  Full Name
  <input type="text" name="name" id="full-name" placeholder="First and Last" required="">
</label>

does.

Nope this explanation is BS never mind.


does not work

i have launched as administrator and it still asked me to run as admin

Hmm, use Launch.bat


GEODE-10089: update release scripts to handle 1.15

increased RC pipeline file size check limits as 1.15 is a little bigger than 1.14
modified RC pipeline native builds to obtain a required newer cmake than what is distributed by Ubuntu

after one more test (by way of the 1.14.4 release) this should be ready to merge


Upload-WindowsAutopilotDeviceInfo.ps1 fails on missing dll for new AzureAD module

Hi,
There was an AzureAD module version <IP_ADDRESS> released yesterday. It breaks the Upload-WindowsAutopilotDeviceInfo.ps1 script because of a missing Microsoft.IdentityModel.Clients.ActiveDirectory.dll in this version. Is there an option to update the script with a required old version? And push this to powershellgallery?
Eg.
#Requires -Modules @{ ModuleName="AzureAD"; ModuleVersion="<IP_ADDRESS>" }
And maybe later have a new script based on MSGraph module?
Kind Regards,
Jeroen

Hi,
can confirm, we have the same issue.

This is also affecting the Modern Device Drivers/Bios see this in the log

Confirming same issue on Windows 10 and Windows 11 devices

Hi,
also confirming.

Hi,
Have the same issue

A workaround for now, launch powershell then run:
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned
Install-Script Get-WindowsAutoPilotInfo
Get-WindowsAutoPilotInfo.ps1 -Online

Sign in as a Global Admin and the hash will upload, takes longer than the Upload-WindowsAutopilotDeviceInfo.ps1 script though. It will sit at "waiting for 1 of 1 to be imported" for a while then succeed (usually).

same issue here, work around checks out, just wondering when this is fixed

same here. However, it looks like the solution proposed above works fine too:
Set-ExecutionPolicy -ExecutionPolicy RemoteSigned
Install-Script Get-WindowsAutoPilotInfo
Get-WindowsAutoPilotInfo.ps1 -Online

We run Upload-WindowsAutopilotDeviceInfo.ps1 as part of a automated deployment via MDT, so in the short term we can live with Get-WindowsAutoPilotInfo.ps1 -Online and running it interactively, but would like the ability to automate this again.

I have found another workaround that has worked for me so far.
Install-Script -Name Upload-WindowsAutopilotDeviceInfo
Install-Module AzureAD -RequiredVersion <IP_ADDRESS>
Copy-Item -Path "C:\Program Files\WindowsPowerShell\Modules\AzureAD\<IP_ADDRESS>\Microsoft.IdentityModel.Clients.A*" -Destination "C:\Program Files\WindowsPowerShell\Modules\AzureAD\<IP_ADDRESS>"
After that, the Upload-WindowsAutopilotDeviceInfo.ps1 worked as it used to. Worked as of the last time I needed this, which was 5/31/2023.

@NickolajA  https://github.com/MSEndpointMgr/Intune/blob/master/Autopilot/Upload-WindowsAutopilotDeviceInfo.ps1 v1.2.0
Some issues with module install logic at line 81
Install-Module -Name MSGraphRequest
Install-Module -Name MSAL.PS

@l2g2h: yeah I noticed I was a bit too quick in committing this change. I'm testing it out currently and have updated it slightly to mimic more the behavior as of my IntuneWin32App module.
@MoondewCoUK: For your scenario, would allowing for passing a Client Secret of a custom app registration that you setup, solve your situation? I'm thinking down the lines of:
Upload-WindowsAutopilotDeviceInfo.ps1 -TenantName  -ClientID  -ClientSecret

@jerveree: My take is to make use of the MSAL.PS module instead. Would that work for you?

I've pushed 1.2.1 in this repo now. Please let me know if it works better, and I'll publish it to the gallery. Appreciate all the feedback!

@NickolajA
I have update the old script we have (1.1.0) with 1.2.1 and we use the following as part of automated MDT deployment , unfortunately I did not get the login window to authenticate , any help is appreciated:
@ECHO OFF
REM Set variables
SET SOURCE=%~dp0
SET SOURCE=%SOURCE:~0,-1%
IF NOT DEFINED LOGDIR SET LOGDIR=%SystemRoot%\TEMP
rem xcopy "Upload-WindowsAutopilotDeviceInfo.ps1" "%systemdrive%\Program Files\WindowsPowerShell\Scripts"
Install-PackageProvider -Name NuGet -RequiredVersion <IP_ADDRESS> -Force
powershell Install-Script -Name AzureAD -Force
powershell Install-Script -Name PSIntuneAuth -Force
powershell Install-Script -Name Upload-WindowsAutopilotDeviceInfo -Force
powershell -noprofile -executionpolicy bypass.\Upload-WindowsAutopilotDeviceInfo.ps1 -TenantName "xyz.com" -GroupTag "ABC"

any chance you could update the version in the gallery

It has been updated now.


Box in TreeItem label  draggable dose not work

Only 2 does not work. it can not be dragged
https://codesandbox.io/s/filesystemnavigator-material-demo-forked-wr269?file=/demo.tsx:652-660

Could you describe more what you are trying to achieve and what doesn't work? The draggable attribute is added on each of the elements you specified. What is not working?

You can take a reference from this https://codesandbox.io/s/draggable-material-ui-oj3wz?file=/src/App.tsx

Could you describe more what you are trying to achieve and what doesn't work? The draggable attribute is added on each of the elements you specified. What is not working?

there a three element that are set draggable, but the second can not be dragged , ant other two can drag.

Please try to drag the elements that setting draggable, the second can not be dragged!


⚠ Processors has degraded performance

In 5803a80, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 125 ms

Resolved: Processors performance has improved in c072572.


Make Pilet Name Configurable on Scaffold

New Feature Proposal
For more information, see the CONTRIBUTING guide.
Description
Right now the pilet's name is decided by the base name of the used directory (target). This should be made configurable, with the current setting (base name of the used directory) being the default.
Background
Maybe the directory is a bit misleading, the package should be scoped, or similar. We should support such cases without need to edit the package.json afterwards.
Discussion
What should the parameter name be? I think --pilet-name (i.e., having pilet-name / piletName) would be good.
Implementation
Right now the project name is determined here https://github.com/smapiot/piral/blob/develop/src/tooling/piral-cli/src/apps/new-pilet.ts#L132
It should be made available as options, where undefined is the default. This way, we can just assign basename(root) as default value when getting the piletName from the options.

Hello!  I would like to take this issue too. I did the #546 and it's a very similar issue.
Are you comfortable with this? Maybe you prefer that someone new fix this.
Regards

Nope you can take it @mgarciaar - much appreciated!

Hi @FlorianRappl , I was thinking... Wouldn't it be more clear if we add the parameter "--name" instead of --pilet-name?  Same goes for piral new --app-name.  As there are two separate commands for piral and pilets, doing:
piral new --name=test and/or pilet new --name=piletname
would have a better syntax.
What do you think?

What do you think?

I think this is a great idea 💡 ! It would be the same for both commands, so less confusion and easier to remember.
Yeah let's have --name for pilet new and --name for piral new :beers:


[SPARK-46955][PS] Implement Frame.to_stata

What changes were proposed in this pull request?
Implement Frame.to_stata
Why are the changes needed?
for Pandas parity
Does this PR introduce any user-facing change?
yes
In [5]: df = pd.DataFrame({'animal': ['falcon', 'parrot', 'falcon', 'parrot'], 'speed': [350, 18, 361, 15]})

In [6]: psdf = ps.from_pandas(df)

In [7]: df.to_stata('/tmp/animals_1.dta')

In [8]: psdf.to_stata('/tmp/animals_2.dta')

In [9]: pd.read_stata('/tmp/animals_1.dta')
Out[9]:
   index  animal  speed
0      0  falcon    350
1      1  parrot     18
2      2  falcon    361
3      3  parrot     15

In [10]: pd.read_stata('/tmp/animals_2.dta')
Out[10]:
   index  animal  speed
0      0  falcon    350
1      1  parrot     18
2      2  falcon    361
3      3  parrot     15

How was this patch tested?
added ut
Was this patch authored or co-authored using generative AI tooling?
no

merged to master


[ALLUXIO-2794]

The preconditions check in setWorkerAddress(WorkerNetAddress workerAddress) can be chained to save a statement:
mWorkerAddress = Preconditions.checkNotNull(workerAddress);

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14356/
Test PASSed.

Merged build finished. Test PASSed.

Hey @poloJane, do you mind signing the CLA? You can sign it here: http://cla.alluxio.org/

@poloJane can you sign the CLA?

@poloJane feel free to reopen this PR after you sign the CLA.


Appstudio update test-component-pac-jidx

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-jidx-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
7 seconds
init

✅ Succeeded
19 seconds
clone-repository

✅ Succeeded
32 seconds
build-container

✅ Succeeded
21 seconds
inspect-image

✅ Succeeded
51 seconds
deprecated-base-image-check

✅ Succeeded
46 seconds
clamav-scan

✅ Succeeded
4 minutes
clair-scan

✅ Succeeded
20 seconds
sbom-json-check

✅ Succeeded
1 minute
label-check

✅ Succeeded
7 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/test-component-pac-jidx-on-pull-request has successfully validated your commit.


FB shareViaFacebookWithPasteMessageHint not working in latest build for iOS

FB method shareViaFacebookWithPasteMessageHint is hung up on the latest plugin version (4.3.18). It never returns a value on the callbacks, and the App doesn't respond.
I'm running it on an iPhone 5S with iOS 8.3 (12F670) installed.
Last Xcode message:
"2015-04-28 11:26:10.153 App[1501:1039076] THREAD WARNING: ['SocialSharing'] took '30.614258' ms. Plugin should use a background thread."
Although this appears for other media and works so it's not related.
Great plugin by the way :D

Can you share your project with me perhaps?
Also, what is the exact thing you're sharing, and do you have the Fb app installed?
Thx

Hi Eddy,
Unfortunately I can't share the project (not personal) but i can paste the code here. I am trying to share a link. I have the FB App installed and I'm logged in. When I cancel the popup, callback returns ok, but if I send it, it gets stuck. This worked with previous version I was using (4.3.15). It also works on Android on both versions.
$(".icons .icon.facebook").on("click", function() {
window.plugins.socialsharing.shareViaFacebookWithPasteMessageHint("Share message", null, null, "Share message",
function(msg) {
// Returns true always on iOS and false always on Android
console.log("Share ok: " + msg);
},
function(error) {
console.log("Error: " + error);
}
);
});
Thanks for your time!

Can't reproduce unfortunately. You did see the paste hint popup ('share message' in your case?)
Can you try adding a URL or image to the post as well? Also, can you show a screenshot of what you mean by 'it gets stuck'?
Thanks!

Hi Eddy,
When I add a URL it works as expected. :)
But leaving that parameter null causes the App to stop working after you press the send button of the FB dialog. Never returns a callback and I can't click anywhere else. It never reaches FB either. I tested it on an iPhone 6 with 8.3 and it happened too. FB dialog cancel button works as expected with no side-effects.
Related to the hint, I believe the paste hint popup appears but only so fast and then the FB dialog is over it. I can't really tell cause it's so fast. Maybe it would work better with a timeout. Just a thought.
If you want I can close the issue, or I'll leave it open just in case you want to check those couple of things. Hope this helps.
And thanks again!!!!!

Hello, I am also having issues with shareViaFacebookWithPasteMessageHint(...) on iOS 8.3.
My share use cases require that I post:

only image (for which I use imageURL argument)
only text (using message argument)
image and text (imageURL and message arguments)

Note: I never see the hint popup.
Result for each use case:

Image is shared normally.
Apparently text is not copied to "clipboard" and I have nothing to paste, but it still allows me to send the post anyway. So when I hit "Post" it results in a status update (without content) on my FB page.
The result is the same as for (1). It only pre-fills image and does not copy any text to clipboard.

Any ideas why am I getting this behaviour, any workaround for this use cases?
Thanks

Hi Eddy,
I have the same issue. I use shareViaFacebookWithPasteMessageHint, ios 8.3.
Popup hint doesn't appear, clipboard doesn't contain shared message. :(

Any updates on this guys?

Will look at it tomorrow. In the meantime @dimoreira, how exactly do you invoke the plugin?

@EddyVerbruggen Using the same steps from ngCordova.
Using it in an Ionic app.

@dimoreira There is no example of the pastemessagehint function on that page.
@0mega (and possibly @DmytroT) if the message is prefilled then it's using the built-in iOS sharesheet, not the one from the Facebook app. Do you have the Fb app installed (and is it a recent version, where they have overridden the built-in iOS sharesheet)? Only the sheet of the Fb app will show the paste hint, which is fortunate because only then it's required.

@EddyVerbruggen Thanks for reply.
In order to test my use cases I had to uninstall and install FB app several times, so it is the most recent version I guess =). But I never see the paste hint. Didn't have time to explore it more =|

@0mega Thanks for helping! Can you show me what exactly you are posting? Not the variable names but the values: shareViaFacebookWithPasteMessageHint('message', ..
Even better: the XCode project from the platforms/ios folder, so I can debug this b*tch.

I'm sorry but I think I can't give you those because we ended up using the plugins.socialsharing.share(...) method. So now there is no fast way of providing you anything related to that issue :disappointed:.
Nevertheless if we decide to use it or if I get some spare time I'll try to reproduce the issue again.
Thanks again for your feedback, and for this plugin ;)

@EddyVerbruggen I'm facing this same issue. Debugging with XCode I found out
BOOL fbAppInstalled = [[UIApplication sharedApplication] canOpenURL:[NSURL URLWithString:@"fb://"]]; // requires whitelisting on iOS9
if (fbAppInstalled) {
  ....

Right here fbAppInstalled is false although the (up to date) app is installed in the device. When I manually added <string>fb</string> to LSApplicationQueriesSchemes in info.plist the message is displayed (like in a kind of an alert) over the share popup and is copied to the clipboard. Any idea how to get around this?
Here's the output of ionic info
global packages:
@ionic/cli-utils : 1.3.0
Cordova CLI      : 7.0.1
Ionic CLI        : 3.3.0

local packages:
@ionic/app-scripts              : 1.3.7
@ionic/cli-plugin-cordova       : 1.3.0
@ionic/cli-plugin-ionic-angular : 1.2.0
Cordova Platforms               : ios 4.4.0
Ionic Framework                 : ionic-angular 3.3.0

System:
Node       : v6.9.5
OS         : macOS Sierra
Xcode      : Xcode 8.3.2 Build version 8E2002
ios-deploy : 1.9.1
ios-sim    : not installed

While debugging iOS 11 issues I found that the paste message is not correctly placed on the clipboard. No idea which iOS version changed that, but we have to use pasteboardType public.utf8-plain-text instead of public.text. The former seems to add HTML to the clipboard which Facebook is not (or no longer) able to interpret.

This has now been fixed. The paste message is shown on iOS again. I will push an update to npm once I've tackled a few iOS11 (beta) issues.
Note that the hint is only shown on iOS < 11 when the Facebook app is installed AND the fb:// scheme has been whitelisted in the plist. That last bit will probably no longer be applicable to iOS 11.


docs(readme): Reordered and fixed typos

In some places, useOidcAuth was referred to as useUserSession
Fixed bugs in code examples
Moved the configuration example to the Configuration section
Various typo, formatting, and syntax fixes

Thanks for the catches! :)


Refactor cluster visualization

In this task/issue we need to

add more clustering methods
add num of clusters as an option in the UI
add basic clustering evaluation metrics in the UI
beautify cluster visualization
dimensionality reduction to visualize cluster
spectrogram on mouse over
@vassilinast

Branch related to this task create at https://github.com/tyiannak/amvoc/tree/vas

make duration in “spectrogram of syllable” fixed (e.g. 200 msecond)
make dimensionality reduction fixed (positions of samples in the 2d space should not change)
repair visibility of  the metrics (add a blank line bellow the 1st line..)


Prevent AdminAction redeploy if VM has an ephemeral OS disk

Redeploying a VM with an ephemeral OS disk will result in a error message,
stating that the VM has an ephemeral OS disk and cannot be redeployed.
If a VM that has an ephemeral OS disk is redeployed it will never become ready,
due to the fact that the VM's OS is re-provisioned and
the cluster machine approver will not allow the "new" machine
to join the cluster again.
Which issue this PR addresses:

Fixes : https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/14751454/
What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

The work item already exists and there is also an approved PR ready to be merged.


Chapter 4 [os error "en" model and it takes only two positions]

This is the error, I have solved this using three things.

instead of using model 'en' we can use the "en_core_web_md" model.
You will get another error like it take only 2 positions but you have given 3. for that just remove the none.
You will get another error like list of dictionary not needed, you can enclose the matcher pattern inside the two dimension array(like nested list)

Here's my code!

Should be fixed in the new versions, still thanks for the report :wave:

@DuyguA Thank you so much!


Builder problem: ssh_exchange_identification: Connection closed by remote host

Issue for builder from deis version 1.10.0:
Builder working normally(as expected) during ~20 hours, then users which try to push getting error:
$ git push deis master
ssh_exchange_identification: Connection closed by remote host
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

I don't actually testing, but i thing builder stop posting it host and port to etcd. Or maybe it posting only once for 24 hour TTL. So router cannot route requests to builder.

Restart of builder helps, but next issue is:
$ git push deis master
fatal: 'my-app.git' does not appear to be a git repository
fatal: Could not read from remote repository.Please make sure you have the correct access rights
and the repository exists.

And i solve this by:
 $ nse deis-builder
 $ cd /home/git
 $ rm -R my-app.git

Yes. Host and port no more published.
etcdctl ls /deis/builder/
/deis/builder/sshHostecdsaKey
/deis/builder/sshHosted25519Key
/deis/builder/users
/deis/builder/sshHostrsaPubKey
/deis/builder/sshHostrsaKey
/deis/builder/sshHostdsaKey

And error is
Sep 16 09:25:50 dev-node-0.novalocal sh[29179]: [error] Etcd error setting host/port: Tried and failed 2 cluster connections: Unexpected HTTP status code

And problem is here:
https://github.com/deis/deis/blob/v1.10.0/builder/etcd/etcd.go#L361

I'm facing the same issue, this happens approximated every 24 hours. Restarting builder fixes this, but it keeps recurring.
deisctl journal builder shows this
 [error] Etcd error setting host/port: Tried and failed 2 cluster connections: Unexpected HTTP status code

Sorry, I just now saw this issue.
The cause of this is etcd not answering requests. We have to refresh the etcd host/port mappings frequently or else they expire out of etcd. When they expire, SSH traffic can no longer be routed from the edge of the cluster to the builder, and then all pushes to builder fail.
I know @mboersma is working on some fixes to etcd right now. I'll consult him about this, but I am wondering if raising the TTL might help, if this is really connection related.
I'm also gradually replacing the go-etcd client with the new official etcd client, but I suspect that will not land until Deis v2.

I change break to continue on 16 September and builder still works great in my cluster with 30+ apps. I actually can't understand why this blocker issue does't fixed in 1.10.

There is one problem:
deis-builder fleet unit have not ExecStop handler, so when VM with builder have network problems for 5 - 10 min, you can get 2 builders in cluster.
In fact, with #4677 builder will never stopped automatically.


Typo in Thai language

There's a typo for the words "Account Settings" in the Thai version of the website. It's currently "ตั้งค่าบีญชี" but it should be "ตั้งค่าบัญชี". Picture for Proof

@SnowNeko9 Please take a look!

Ok! Its my bad
this word need to edit

closed in #430


refact: ing subcommands.

Update the ToolChain version to 2024-01-01, update GitHub CI to use the version and environment defined in Toolchain.toml
Change the crate to the database crate, and move the executable file to the bin directory

建议使用 conventional commits，参考：

A specification for adding human and machine readable meaning to commit messages
Angular Commit Format Reference Sheet

refactor:ing
Refactoring the ing subcommand while retaining the list and post subcommands. Add the following:

Use query instead of list and implement filtering using filtering conditions.
Use create instead of post to add private, tag, lucky, and other options.
Add delete to delete your specified ing.

中文(Chinese)
重构ing子命令，保留list和post子命令，未来可删除。新增如下：

使用query代替list，并使用过滤条件实现过滤。
使用create代替post，增加私有，标签，幸运等选项
新增delete删除自己的指定闪存

fix:ing
Fix ing create public content


Migrate to eslint

When tslint came out, it was a lot easier to use than it's eslint counterpart. Now that tslint is deprecated, we need to move to using eslint instead.

On it


tf.scatter_nd sums updates if indices are duplicated

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

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

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
tf.scatter_nd sums update values if indices are present multiple times. I use this function for moving pixels in an image. Some pixels are present twice and this gives wrong pixel values in scatter.
tf.Tensor([ 0 11  0 10  9  0  0 12], shape=(8,), dtype=int32)
tf.Tensor([ 0 11  0 10  9  0  0 24], shape=(8,), dtype=int32)
tf.Tensor([ 0 11  0 10  9  0  0 36], shape=(8,), dtype=int32)
Describe the expected behavior
I expect this behavior:
tf.Tensor([ 0 11  0 10  9  0  0 12], shape=(8,), dtype=int32)
tf.Tensor([ 0 11  0 10  9  0  0 12], shape=(8,), dtype=int32)
tf.Tensor([ 0 11  0 10  9  0  0 12], shape=(8,), dtype=int32)
Otherwise I need to find unique indices and delete multiples. And why it is adding? Maybe other options are possible then too.
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
indices1 = tf.constant([[4], [3], [1], [7]])
updates1 = tf.constant([9, 10, 11, 12])
indices2 = tf.constant([[4], [3], [1], [7], [7]])
updates2 = tf.constant([9, 10, 11, 12, 12])
indices3 = tf.constant([[4], [3], [1], [7], [7], [7]])
updates3 = tf.constant([9, 10, 11, 12, 12, 12])
shape = tf.constant([8])
scatter1 = tf.scatter_nd(indices1, updates1, shape)
scatter2 = tf.scatter_nd(indices2, updates2, shape)
scatter3 = tf.scatter_nd(indices3, updates3, shape)
print(scatter1)
print(scatter2)
print(scatter3)
Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

@RichardArendsen71 ,
The present functionality is for summation only as per the link If indices contains duplicates, then their updates are accumulated (summed).
Can you please let us know the use cases where the other operations can be applied,so that we can consider it as a feature request?Thanks!

Thanks for the answer. I missed that part. Actually I wanted to use:
image_rotate=tf.keras.preprocessing.image.apply_affine_transform(             image, theta=-angle*180/math.pi, fill_mode='constant', cval=0.0, order=1)
I works in eager execution, but with @tf.function it returns an error:

NotImplementedError: in converted code:
relative to /home/face:

projects/tensorflow2/facenettf2/crop_compare.py:27 crop_eyes_fixed1  *
    image_rotate=tf.keras.preprocessing.image.apply_affine_transform(
.virtualenvs/tensorflow2/lib/python3.6/site-packages/keras_preprocessing/image/affine_transformations.py:285 apply_affine_transform  *
    theta = np.deg2rad(theta)
.virtualenvs/tensorflow2/lib/python3.6/site-packages/tensorflow_core/python/framework/ops.py:736 __array__
    " array.".format(self.name))

NotImplementedError: Cannot convert a symbolic Tensor (truediv_4:0) to a numpy array.

If this can be fixed I am also happy. For a sort term solution I tried to write my own rotation code and needed the function tf.scatter_nd, to set the pixel values of the rotated images.
So, main reason. I want to rotate an image at an given angle. I found https://tensorflow.google.cn/versions/r2.0/api_docs/python/tf/image/rot90?hl=en , but this is only 90 degrees.
What is the best way to go? I really want to use tensorflow 2.0.

This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
Thanks!

When will this function be available for tensorflow 2.0? https://www.tensorflow.org/api_docs/python/tf/contrib/image/transform

Unfortunately TF 2.0 won't have this functionality. However we have added many other functions to tf.image module. See https://www.tensorflow.org/versions/r2.0/api_docs/python/tf/image#functions
You may use transform function from tensorflow addons package if necessary.
For this you have to install tensorflow-addons separately.
See https://github.com/tensorflow/addons#maintainers
Thanks!

I tried addons before, but that did not work with the latest TF 2.0 version. Now, I tried it again and it works, also with AutoGraph. So, I can do now what I needed. Thanks.


Refactor Footer and GetInvolved Components

Issue that this pull request solves
Closes: #416
Proposed changes
Changed the Component Structure as proposed style.
Brief description of what is fixed or changed

Changed the module.css to module.scss.
Moved GetInvolved to Pages folder.
Renamed the classes and files names as proposed.

Types of changes
Put an x in the boxes that apply

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[x] Other (changed structure as proposed):

Checklist
Put an x in the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes does not break the current system and it passes all the current test cases.

Screenshots

Footer

GetInvolved

Nav Bar for responsive site seems to get distorted. Please correct the position.

@tejan-singh My task was to refactor Footer and GetInvolved Components. I haven't touched NavBar Component. You can check in Files Changed option.
There is already a PR for fixing it (#447)

@sanjay270899 Can you please let us know if you are SWoC participant as well?


Add S3 client and update dockerfile

What problem does this PR solve?

add minio-cpp-sdk third party
add S3 client and S3 client minio
update some doc
update dockerfile

Issue link:#1809
Type of change

[x] New Feature (non-breaking change which adds functionality)
[x] Documentation Update

Duplicate with #1892


Clarify the semantics of StableHLO's sort operation.

The comparator must satisfy the same requirements as C++'s Compare concept, i.e. two elements v1 and v2 are considered to be equal by the comparator iff comparator(v1, v2) = false && comparator(v2, v1) == false. This precision is important to exclude comparators such as LE or GE, for which setting is_stable to true does not produce a stable sort.

Thanks for the suggestion! That makes sense to me.


Added Prometheus metric to count number of incoming datapoints in TCP receiver.

This metric is quite useful in following cases:

checking if storages receive expected number datapoints, e.g. if there is no significant data loss
to monitor the incoming traffic for load estimation and fluctuations

Currently, the implementation is only for TCP. Can be added for other protocols as well.

It might be better to use a counter instead of a gauge

I agree that a counter would be better. But, I have reused the implementation that was already in place for graphite. I did not add any new metric calculation.
Moving to a gauge would include re-writing the graphite metric calculation as well to avoid duplicated implementation. I think it makes sense to do it separately since it looks to fall out of the scope of this pull-request. Current PR is about exposing the already existing metric via a new interface.

You can reuse current code for Counter too
if rcv.promMetricsReceived != nil {
    rcv.promMetricsReceived.Add(float64(metricsReceived))
}

@lomik You're right. I did the change you pointed out. Please have a look.


error when trying to start

hey, so I've got everything installed from the requirements and when I try to start the bot I get this how could I fix this?

you've got to enable privileged intent

you can enable it in discord bot settings at discord.com/developers

@wathhr enable guild_members and member_presences on the application portal @ discord.com/developers

For tech support please join https://discord.gg/4NE4bk7


Renaming "chosen_cert_chain" to "server_cert_chain" in prep for Client Auth

Seems like we used that name to prepare for a multiple-cert world. We'll handle that later.


en_US.csv wrong translation

Hello
You US translation has spanish language.
Please see this in en_US.csv: "On Hold","En espera"

Hi there,
Thanks for informing us.
Rasmus has confirmed that he'll contact you in your open email thread with him.
Thank you for taking the time to reach out, have a great day!
Kind regards,
Troy
Onboarding Team Representative
Take a look at our Merchant FAQ section
https://help.viabill.com/l/en/merchant-support where you will find the answer
to most of your questions.
Receive status updates directly in your inbox! Sign up via our new Status Page
https://viabill.statuspage.io/
We would love to know your thoughts on our service!
Feel free to share your comments here
https://www.g2.com/products/viabill/reviews or on
https://www.trustpilot.com/evaluate/viabill.com
Previous-message-reference:
@.***>
Previous-message-reference: @.***>
TokeSuhr August 16, 19:06 GMT
Hello
You US translation has spanish language.
Please see this in en_US.csv: "On Hold","En espera"
—
Reply to this email directly, view it on GitHub
< https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7 >, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA

.
You are receiving this because you are subscribed to this thread.Message ID:

Hej
Ser I ikke på disse bugs?

Hello
Did you see this?

Hi there,
I'm cc'ing the Danish support team @.***) in this email for you.
Thank you for taking the time to reach out, have a great day!
Kind regards,
Troy
Onboarding Team Representative
Take a look at our Merchant FAQ section
https://help.viabill.com/l/en/merchant-support where you will find the answer
to most of your questions.
Receive status updates directly in your inbox! Sign up via our new Status Page
https://viabill.statuspage.io/
We would love to know your thoughts on our service!
Feel free to share your comments here
https://www.g2.com/products/viabill/reviews or on
https://www.trustpilot.com/evaluate/viabill.com
Previous-message-reference:
@.***>
Previous-message-reference:
@.***>
Previous-message-reference: @.***>
Previous-message-reference:
@.***>
Previous-message-reference: @.***>
TokeSuhr September 11, 20:37 GMT
Hello
Did you see this?
—
Reply to this email directly, view it on GitHub
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394

, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA

.
You are receiving this because you commented. Message ID:
TokeSuhr August 27, 10:44 GMT
Hej
Ser I ikke på disse bugs?
—
Reply to this email directly, view it on GitHub
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158

, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA

.
You are receiving this because you commented. Message ID:
Troy US MS Team Lead August 17, 12:31 GMT
Hi there,
Thanks for informing us.
Rasmus has confirmed that he'll contact you in your open email thread with him.
TokeSuhr August 16, 19:06 GMT
Hello
You US translation has spanish language.
Please see this in en_US.csv: "On Hold","En espera"
—
Reply to this email directly, view it on GitHub
< https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7 >, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA

.
You are receiving this because you are subscribed to this thread.Message ID:

Hej Toke,
Vi beklager at kontakten har været svær, men vi har tilsyneladende lidt
udfordringer med at få beskederne til vores danske afdeling via Github, når det
vedrører en dansk webshop.Kontakt os endelig direkte på @.*** i
fremtiden.
Vi får ikke mange informationer med.Hvad drejer din henvendelse sig om?
Venlig Hilsen
Rasmus
Find svar på de oftest stillede spørgsmål i vores Hjælpeartikler
http://kundeservice.viabill.com/l/da/merchant-support eller læs vores Vilkår
og Betingelser <>.
Modtag driftsmeddelelser direkte i din indbakke! Tilmeld dig via vores nye
Status Page https://viabill.statuspage.io/ ⬅
Du er naturligvis altid velkommen til at skrive til os på @.***
<mailto: @.***> for spørgsmål relateret til din webshop.
Lad din mening blive hørt 🙋
Hjælp os til at blive bedre ved at bruge 30 sekunder på at skrive et par
linjer om os påTrustpilot https://dk.trustpilot.com/evaluate/viabill.dk.
Klik her for at se vores gældende Privatlivspolitik
https://viabill.com/dk/legal/erhvervskunder/.
Previous-message-reference: @.***>
Previous-message-reference:
@.***>
Previous-message-reference: @.***>
System user September 12, 12:39 GMT
Hej ViaBill,
Tak for din mail.
Vi bestræber os på at besvare din mail inden for to hverdage.
Der kan dog i visse perioder forekomme en længere svartid.
I mellemtiden kan du finde svar på de oftest stillede spørgsmål i vores FAQ
for webshops http://kundeservice.viabill.com/l/da/merchant-support. ⬅
Tak for din tålmodighed og hav en dejlig dag!
https://viabill.dk/for-erhvervslivet/
Læs her mere om vores persondatapolitik
https://viabill.com/dk/legal/erhvervskunder/.
ViaBill September 12, 12:39 GMT
Hi there,
I'm cc'ing the Danish support team @.***
@.***>) in this email for you.
Thank you for taking the time to reach out, have a great day!
Kind regards,
Troy
Onboarding Team Representative
Take a look at our Merchant FAQ section
< https://help.viabill.com/l/en/merchant-support
https://help.viabill.com/l/en/merchant-support > where you will find the
answer
to most of your questions.
Receive status updates directly in your inbox! Sign up via our new Status Page
< https://viabill.statuspage.io/ https://viabill.statuspage.io/ >
We would love to know your thoughts on our service!
Feel free to share your comments here
< https://www.g2.com/products/viabill/reviews
https://www.g2.com/products/viabill/reviews > or on
< https://www.trustpilot.com/evaluate/viabill.com
https://www.trustpilot.com/evaluate/viabill.com >
Previous-message-reference:
Previous-message-reference:
Previous-message-reference: @.***
@.***>>
Previous-message-reference:
Previous-message-reference: @.***
@.***>>
TokeSuhr September 11, 20:37 GMT
Hello
Did you see this?
—
Reply to this email directly, view it on GitHub
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1243038394

, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA
<
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D6TS4G4XYISYTJS5M3V5Y7H7ANCNFSM56W5HTUA

.
You are receiving this because you commented. Message ID:
TokeSuhr August 27, 10:44 GMT
Hej
Ser I ikke på disse bugs?
—
Reply to this email directly, view it on GitHub
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158
<
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7#issuecomment-1229169158

, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA
<
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2DYIMID2DNWUPKKBOKLV3HWRRANCNFSM56W5HTUA

.
You are receiving this because you commented. Message ID:
Troy US MS Team Lead August 17, 12:31 GMT
Hi there,
Thanks for informing us.
Rasmus has confirmed that he'll contact you in your open email thread with him.
TokeSuhr August 16, 19:06 GMT
Hello
You US translation has spanish language.
Please see this in en_US.csv: "On Hold","En espera"
—
Reply to this email directly, view it on GitHub
< https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7
< https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7
https://github.com/ViaBill-Integrations/ViaBill-Magento-2/issues/7 > >, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA
<
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA
https://github.com/notifications/unsubscribe-auth/AONF2D43ZAWA36CL4G7LFALVZPRDDANCNFSM56W5HTUA

.
You are receiving this because you are subscribed to this thread.Message ID:

I cant see the mail.
But this is a very very easy taks to complete.
If you go to your ENGLISH translation of your magneto 2 module you will see that some of the translated strings are translated to spanish and not english. And that will affect out english store. So now we have spanish translation on our english site.
Why do it have to take so long time to fix?


rawQuery result in 500 with count(*)

Hi, thanks for seraph. I have a problem with rawQuery
db.query('MATCH(n) RETURN count(*) as total', {}, function(err, result) {});

works , but
db.rawQuery('MATCH(n) RETURN count(*) as total', {}, function(err, result) {});

doesn't work, the callback isn't called.
It's queryRaw , not rawQuery :confused:

Can't replicate. Let me know if you can get this to happen again.
> var db = s('http://localhost:10507')
undefined
> db.queryRaw('MATCH(n) RETURN count(*) as total', {}, function(err, result) {console.log(arguments) })
undefined
> { '0': null, '1': { columns: [ 'total' ], data: [ [Object] ] } }


Add rotating file logging option to celery workers

Helps to debug celery errors
🛠 Changelog

Adds rotating log file to celery worker for better debugging. Logs are rotated on hourly basis and last 12 files are kept as backup.

🎫 Tickets
This pull request resolves all or part of the following ticket(s):

TILA-####

Required Azure config map and library options have been set for all environments.

Same change made to STAGING branch, but it seems like log file is not created. Will investigate tomorrow.

Seems like the current changes are working in staging.


Make sure Cassandra gets started after restore

https://ocd.appscale.com:8080/job/Daily Build/3004/


[util] Import util tools from KUDU

MonoTime/MonoDelta
MonoTime: The MonoTime represents a particular point in time, relative to some fixed but unspecified reference point.
MonoDelta: The MonoDelta class represents an elapsed duration of time, the delta between two MonoTime instances.

CountDownLatch
This is a C++ implementation of the Java CountDownLatch

related issue is #2896


reexport dependencies used in the public api

related: https://github.com/FuelLabs/fuel-core/issues/236
All external crates with types included in the public API of this library should be re-exported.

Is there an automatic way to enforce that this change is comprehensive?

Not that I'm aware of, it's mainly a best practice.


fix half-way commit text

Pull request
Issue tracker
Fixes will automatically close the related issue
Fixes #
Feature
Fix a bug when toggle ascii mode by commit_text while the input segment of the selected candidate is shorter than the complete input (i.e. there are unsegmented raw input beyond the current segment of selected candidate), nothing gets committed at all.
Unit test

[x] Done

Manual test

[x] Done

Code Review

Unit and manual test pass
GitHub Action CI pass
At least one contributor reviews and votes
Can be merged clean without conflicts
PR will be merged by rebase upstream base

Additional Info

@groverlynn Please squash your commits.

@groverlynn Please squash your commits.

@WhiredPlanck I did in my forked repo, but it's not allowed here


Added a note regarding dummy access log data

Added: Note - the sample access logs contain 'dummy' data with random IP addresses. You will see some errors logged in your pipeline for IP addresses that are not matched in the GeoIP2 database. StreamSets correctly detects and flags these errors for analysis.
See https://github.com/metadaddy/tutorials/blob/e33796ac6ed942f2405e62a8fe2ea3e8f88cf498/tutorial-1/log_shipping_to_elasticsearch_part1.md

since graceful error handling is an important piece of the sdc, it maybe useful to point out the error handling configuration section and add some color around that.

Rather than rework that section, I extended the note a little and showed how to drill down into the errors to see exactly what the problem is. What do you think?
https://github.com/metadaddy/tutorials/blob/a243a9e03bc90f7aa984467c20abba844b25dbd2/tutorial-1/log_shipping_to_elasticsearch_part1.md#lets-ship-some-logs


Remove remote-content plugin for openAPI specs

Description
We currently have a lot of docusaurus-plugin-remote-content entries in our docusaurus config: see https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/blob/main/docusaurus.config.js#L122
This will pull in content on build time and therefore updates do happen in an uncontrolled way. Any contribution regardless of where it is made, will build the page and update the contents configured through that plugin.
Also, some openAPI specs are even pull from apps running on consortia infra. I don't known how anyone can be sure which version is running there and if it is the right one to show on the website
Proposed change
Remove the remote content plugin completely and update contents in a controlled fashion.

eclipse-tractusx/sig-infra#376

This won't be done by the System Team anymore.


review exceptions docs

Need someone to review the exceptions docs (try/catch/finally/throw). Not my strong suit.

looks good


Set -discoveryAddress only once in injector template

Hi @qfel. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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

/retest

@qfel: Cannot trigger testing until a trusted user reviews the PR and leaves an /ok-to-test message.

In response to this:

/retest

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


[Suggestion] Add line 3d and floating text 3d widgets

Such widgets exists In OpengGlasses1. Can you add it to openGlasses2 also?
https://minecraft.bymarcin.com/OpenGlasses/doku.php?id=lineworld
https://minecraft.bymarcin.com/OpenGlasses/doku.php?id=floatingtextworld

Floating Text was replaced by https://github.com/Starchasers/OCGlasses/wiki/Widget_Text3D
and line got dropped in favor of https://github.com/Starchasers/OCGlasses/wiki/Widget_Custom3D (which can be used to archive a line by writing a lua wrapper which adds the necessary vertices)

Floating Text was replaced by https://github.com/Starchasers/OCGlasses/wiki/Widget_Text3D
and line got dropped in favor of https://github.com/Starchasers/OCGlasses/wiki/Widget_Custom3D (which can be used to archive a line by writing a lua wrapper which adds the necessary vertices)

Ok, but Custom3D#setGLMODE support TRIANGLES and TRIANGLE_STRIP only, how to draw regular lines?

Ok, but Custom3D#setGLMODE support TRIANGLES and TRIANGLE_STRIP only, how to draw regular lines?

by creating a cube from point a to point b
for a 3D line segment you would draw 6 faces where each face has 2 triangles, best bet is to wrap it in a lua function, feel free to share it and i'll add it to the wiki

by creating a cube from point a to point b
for a 3D line segment you would draw 6 faces where each face has 2 triangles, best bet is to wrap it in a lua function, feel free to share it and i'll add it to the wiki

Uh, it so unoptimized. Large amount of triangles may reduce fps and too many of triangles may give extreme lags. I tested it

Uh, it so unoptimized. Large amount of triangles may reduce fps and too many of triangles may give extreme lags. I tested it

@hohserg1 sorry for necro post, but is this feature request obsolete with your addition of the Wavefront OBJ line support?
https://github.com/Starchasers/OCGlasses/issues/102


Implement TypeCheckVisitor

Resolves #31
Resolves #22 (via yeeting)
Majorly refactors Type:

Removed BuiltinType in favor of a single type struct
Removed Undefined variant in favor of the None variant in Options

@TimJentzsch I'd say for a project like this, that is the right approach :)


A device which does not exist was specified. (os error 433)

Firstly the reason I am posting this here it because it caused by redscript. My drives and OS arent the issue, being I've completely replaced my drives with brand new ones to test again.
This error happens at random, sometimes it is at launch, sometimes its while playing, it happens at random points. Normally you could just say okay and restart the game, or verify the game cache, etc. With this error specifically it locks the steamapps folder and makes all games on the drive where cyberpunk is located completely inaccessible. It entirely locks the steam apps folder, so you cannot open it via file manager either as you get the same error, but through windows.

Rebooting the pc makes the problem go away but the error will still occur again. Ive tried this with and without mods installed and get the same thing over and over again. Ive never had this problem before and after a week of trying to find a fix myself im making this post. Any help is appreciated.

@YennIsHere this is an error reported by the operating system, and simply forwarded to you by redscript, you should check your disk for errors, there are various tutorials on the internet that can help you do that, e.g. https://www.howtogeek.com/1033/how-to-use-chkdsk-on-windows/.

Thats what i figured however I have run multiple disk checks, and ive also completely swapped disk drives and did a brand new install of windows.


fix: pnpm-lock

The ci fails currently when publishing the packages.
lerna ERR! EUNCOMMIT Working tree has uncommitted changes, please commit or remove the following changes before continuing:
lerna ERR! EUNCOMMIT  M pnpm-lock.yaml
lerna ERR! EUNCOMMIT ?? .npmrc

I am unsure, but I think this may have been caused by a different PNPM version I used.

@nklomp bump on this one

We are using pnpm 8 internally, because 9 was not stable at that point. Given the impact on the other repo's and CI/CD this needs to be concerted. So unlikely we will be able to do that today

@auer-martin can you update this to pnpm 8? Or at least the same version as the ci, then it shouldn't cause issues

I did update to version 8 for compatibility, right? So this should work?

Ah wait, okay this reverts the 9.0 lockfile back to 8. Okay then we should merge this yes.


GatewayAPI GEP-91 Client Certificate Validation

Istio implementation for GatewayAPI GEP-91 Client Certificate Validation
References:

https://github.com/kubernetes-sigs/gateway-api/pull/2759
https://github.com/kubernetes-sigs/gateway-api/pull/2273
https://github.com/kubernetes-sigs/gateway-api/issues/91
https://github.com/kubernetes-sigs/gateway-api/issues/2110

As discussed with @frankbu PR https://github.com/kubernetes-sigs/gateway-api/pull/2759 needs to be merged before we can implement GEP-91 in Istio but we can start working on a POC using 2759 interfaces.

I think this needs changes to internals of Istio beyond just simple conversion. We need to support a separate reference, and it needs to allow configmaps. Fun

@SRodi @frankbu are either of you interested in an implementation? I started https://github.com/istio/istio/pull/50437 but it needs a lot of work

@howardjohn interested but unfortunately I don't have the bandwidth right now. I will get back to this if/when I have more time available.


How can i change Scroll From Horizontal to Vertical when change month?

i want to change scroll direction vertical like calendar iOS native app when change month view
Thanks

This is currently not supported, sorry!


Fail on Ubuntu 18.04

Works fine before, but fail when these day I upgrade to Ubuntu 18.04
It echos:
rm: cannot remove '/opt/bing-desktop-wallpaper-changer/.gitignore': No such file or directory

Executing bing-desktop-wallpaper-changer...
Traceback (most recent call last):
  File "/usr/bin/bingwallpaper", line 429, in <module>
    main()
  File "/usr/bin/bingwallpaper", line 424, in main
    app_notification.show()
GLib.Error: g-dbus-error-quark: GDBus.Error:org.freedesktop.DBus.Error.ServiceUnknown: The name org.freedesktop.Notifications was not provided by any .service files (2)

I installed python 2.7 and gi module
sudo apt-get install python2.7 python2.7-dev
sudo ln -s /usr/bin/python2.7 /usr/bin/python
sudo apt install python-gi

@UtkarshGpta
Maybe it's caused by a GTK module. Not sure why..

Check these links out.
https://stackoverflow.com/questions/35889348/libnotify-error-glib-error-the-name-org-freedesktop-notifications-was-not-provi
https://bbs.archlinux.org/viewtopic.php?id=75078

Check these links out.
ArchLinux
StackOverflow

@UtkarshGpta Please add bug and help wanted  tag :)

Please contribute to this bug if you're an Ubuntu 18.04 user.

Anyway, you can still use @neffo 's GNOME extension. 😉

Closed issue due to inactivity.


It would be nice to be able to use multipart::Form::file() without writing a temp file

Instead of (necessarily) taking a Path and opening the corresponding file, could it perhaps take a Read or such instead?  That way you can implement things that, say, stream generated data to into the form without having to write it to disk.
If there's some way of doing this already... my bad.
Thank you!

There is also the multipart::Part builder that can be used to, and it indeed can take a Read, and have it's mime or filename set, and then passed to form.part(name, part).
So, you could do:
let part = multipart::Part::reader(my_read)
    .mime(TEXT_JSON)
    .file_name("something.json");

let form = multipart::Form::new()
    .part("things", part);


Updated the go and grpc version for podmon

Description
Updated the go version for podmon
GitHub Issues
List the GitHub issues impacted by this PR:
| GitHub Issue # |
| https://github.com/dell/csm/issues/1435 |
Checklist:

[x] I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
[ ] I have verified that new and existing unit tests pass locally with my changes
[x] I have not allowed coverage numbers to degenerate
[ ] I have maintained at least 90% code coverage
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

How Has This Been Tested?
I have used these code changes and built image for karavi-resiliency repo and performed short-integration test.

Unit-Test code coverage:

Built image and pushed to harbor and ran short-integration-test

Can you please update the grpc and go version in other sub modules in this repository to maintain consistency.

Can you please update the grpc and go version in other sub modules in this repository to maintain consistency.

For resiliency Podman, after editing versions we have tested them and raised PR but for other modules, it would be better the respective teams test the changes and update


Use example tracts in the notebook

This PR updates the example notebooks to use the example tract set: 3830, 3831, 4028, 4029.
In the truth notebook, I also added a note about cosmoDC2 as a way to advertise it for those who are interested.
This PR fixes #8.

It was mostly to also demonstrate how tract_filter works. If people want to copy code from a cell for their analysis it seems better to have tract number spelled out.  Otherwise we could define something like
one_tract_example = tract_filter(3830)
four_tract_example = tract_filter([3830, 3831, 4028, 4029])

and use them in native_filters directly.

It was mostly to also demonstrate how tract_filter works. If people want to copy code from a cell for their analysis it seems better to have tract number spelled out.  Otherwise we could define something like
one_tract_example = tract_filter(3830)
four_tract_example = tract_filter([3830, 3831, 4028, 4029])

and use them in native_filters directly.

Thanks!

Thanks!


[FULL@MATCH!]]**Gervonta Davis vs Frank Martin LIVE STREAM@FREE ON TV at home 16 June 2024

Where To Watch Gervonta “Tank” Davis vs Frank “The Ghost” Martin Live stream FREE Boxing 15 June 2024. Two undefeated lightweights are set to collide in a high-stakes battle for supremacy. Gervonta “Tank” Davis, the three-division world champion, will defend his WBA Lightweight World Title against the unbeaten top contender, Frank “The Ghost” Martin. The fight promises fireworks, drama, and a chance for both fighters to solidify their legacies.
📺👉▶ CLICK HERE TO LIVE STREAM
📺👉▶ CLICK HERE TO LIVE STREAM
Fight Details
Date: Saturday, June 15
Time: To be announced
Venue: MGM Grand Garden Arena, Las Vegas
Fighter Profiles
Gervonta “Tank” Davis
Record: 29-0-0 (27 KOs)
Height: 5’5½" (1.66 m)
Reach: 67½" (171 cm)
Stance: Southpaw
Age: 29
Gervonta Davis is a force to be reckoned with. His explosive power, lightning-fast combinations, and defensive prowess have earned him a reputation as one of boxing’s most exciting fighters. With 27 knockouts in 29 victories, Davis has consistently delivered electrifying performances.
Frank “The Ghost” Martin
Record: 18-0-0 (12 KOs)
Height: 5’8" (1.73 m)
Reach: 68" (173 cm)
Stance: Southpaw
Age: 29
Frank Martin, the undefeated challenger, is no stranger to overcoming the odds. Confident and grateful for the opportunity, Martin aims to shock the world by dethroning Davis. His technical skills, ring IQ, and hunger for success make him a formidable opponent.
The Build-Up
Gervonta Davis last stepped into the ring in April 2023, defeating Ryan Garcia. Now, after a hiatus, he faces another undefeated foe. Frank Martin’s journey has been marked by determination and resilience. This clash represents the pinnacle of his career, and he’s ready to seize the moment.
Predictions and Expectations
The boxing world eagerly awaits this showdown. Will Davis continue his reign as the lightweight king, or will Martin pull off the upset? Both fighters possess the tools to win, and their clash promises fireworks. Expect explosive exchanges, heart-pounding action, and a fight that will leave fans on the edge of their seats.
Conclusion
Gervonta “Tank” Davis vs. Frank Martin is more than a title fight—it’s a battle for legacy. As the date approaches, anticipation grows. Mark your calendars for June 15, and witness two warriors collide in a quest for greatness. Boxing fans, get ready for an unforgettable night! 🥊🔥

📺📱 THE GERVONTA DAVIS VS. FRANK MARTIN
📺📱 CLICK HERE COVERAGE
.

➭
📺📱 THE GERVONTA DAVIS VS. FRANK MARTIN
📺📱 THE GERVONTA DAVIS VS. FRANK MARTIN
.

➭
📺📱 THE GERVONTA DAVIS VS.


SSH public key can't be used at jumpbox and failed during installation

I ran deploy task several times last days, and almost failed, one of failed issues is my id_rsa.pub can't be used to connect jumpbox VM, and I found fail messages about this:
06/19/19 09:46:24 :: openedxctjb has been identified as a member of the 'jumpbox' role
Checking /oxa/oxa-tools/scripts/overrides.sh for overrides
06/19/19 09:46:24 :: Syncing the 'https://github.com/microsoft/oxa-tools.git' repository (Branch='oxa/master.fic', Tag='')
/oxa/oxa-tools ~
Fetching origin
~
/oxa/oxa-tools ~
Already on 'oxa/master.fic'
Your branch is up-to-date with 'origin/oxa/master.fic'.
~
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
Permission denied (publickey).
lost connection
Permission denied (publickey).
06/19/19 09:46:25 :: [ERROR] Mongo installation failed for <IP_ADDRESS>
mail: cannot send message: Process exited with a non-zero status
06/19/19 09:46:26 :: [ERROR] Phase 0 Bootstrap for Mongo & Mysql failed for
mail: cannot send message: Process exited with a non-zero status
06/19/19 09:46:26 :: [ERROR] OXA stamp customization (/oxa/oxa-tools/scripts/install.sh) failed
mail: cannot send message: Process exited with a non-zero status

PS: I use [ssh-keygen -t rsa -b 4096 -f id_rsa] command to generate my SSH keys.

In order to connect to jumpbox you need to connect using private key. Let me know if this resolves your issue.

I had used private key to connect to jumbobox, thank you.
But issues shown inside logs has still exist.

Just to confirm using private key you are able to connect. If so you can close the issue.


CI is broken

Looks like the oracle-jdk-installer addon needs to be removed and we need to replace oraclejdk with openjdk.

Fixed in #321


Support alternative value resolution for BaseSettings

Feature Request
Output of python -c "import pydantic.utils; print(pydantic.utils.version_info())":
             pydantic version: 1.5.1
            pydantic compiled: True
                 install path: C:\Users\ofek\AppData\Local\Programs\Python\Python38\Lib\site-packages\pydantic
               python version: 3.8.1 (tags/v3.8.1:1b293b6, Dec 18 2019, 23:11:46) [MSC v.1916 64 bit (AMD64)]
                     platform: Windows-10-10.0.19041-SP0
     optional deps. installed: []

My use case is I'm reading a toml config file and passing what is read directly to the model constructor. However, I have no way to allow users env var or .env overrides.
Essentially, configs persisted to disk cannot benefit from BaseSettings. Or did I miss something?

This should be possible, you might just need to reimplement some of env_settings.py.

Is it a 👎 for this feature then?

I don't think it should be possible to implement what you need without changes to pydantic.
If that's possible I would rather not make any changes in the library.
If it's not possible or very annoying for some reason, please come back with requests foe specific changes and we'll consider them.


Updated CordovaPlugin to use the latest library version (0.2.1)

Hi @tompaana, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Paananen Tomi (DX/Tampere)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Reviewed 1 of 6 files at r2, 4 of 4 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from the review on Reviewable.io


style(ecc-client-ga4gh-tes): better coding style

Description
Enforce better coding styles to make code more readable

@git-anurag-hub Need to take care of 'LF' to 'CRLF' situation and vice versa, I pushed this code from linux and there were no issues but it won't be possible as of now without workarounds to push this code from windows (I tried) as husky won't allow without linting and lint fix wont work
.
please refer to these 1, 2. Maybe we could include it in elixir-eslint, this wont affect unix based operators as far as I know and will solve the issue for windows dev. :+1:

How about I merge this rn and that issue can be taken care of late in another PR ?


Feature Request - Offline diff

I have 2 pingcastle reports.
Report A = New
Report B = 1 month old
I would like to run an offline diff of the 2 reports and report the findings to a microsoft teams instance.
Does PingCastle-Notify support that?

Hello, follow this structure:
SECU-TOOL-SCAN/
    - PingCastle-Notify.ps1
    - PingCastle/
        - Reports/
            - domain.local.xml <------- your old report
        - ad_hc_domain.xml <-------- your new report

And the run the script PingCastle-Notify.ps1 :)


Update to 1.21.1

Since 1.21.1 brings some security patches for servers it would be good if the BlossomLib (and other Blossom Mods) would natively support 1.21.1.

For context: currently, Blossom Mods crash my server, even if other mods from 1.21 are compatible with 1.21.1 because of the minor changes.


Add Ctrl+A to select all current images

Now that multi-select is supported, would be useful to have Ctrl+A shortcut to select all images

Done.


Use Performance Badger Options for snapshot db

#1355  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @lazynina and the rest of your teammates on  Graphite


Migrate GitHub Actions from v1 to v2

Description
Migrate GitHub Actions from v1 to v2
Usage
name: 'Issue Reference'
on:
  pull_request:
    types: [opened]

jobs:
  issue-reference:
    runs-on: ubuntu-latest
    steps:
      - uses: kentaro-m/add-an-issue-reference-action@v2
        with:
          repo-token: "${{ secrets.GITHUB_TOKEN }}"
          branch-prefix: "issue"

Demo

TODO

[ ] Update the document

Can you elaborate on why is this important?

Hi, @Kreyren.
branch-prefix is used to get an issue number to link a specific issue from the branch name. Yesterday I wrote an incorrect branch-prefix specification in the PR description. It should contain a hyphen (e.g. issue-).

name: 'Issue Reference'
on:
  pull_request:
    types: [opened]

jobs:
  issue-reference:
    runs-on: ubuntu-latest
    steps:
      - uses: kentaro-m/add-an-issue-reference-action@v2
        with:
          repo-token: "${{ secrets.GITHUB_TOKEN }}"
          branch-prefix: "TICKET-"


Passing a URL to the Primary Property on Custom Attribute Confuses the Engine.

I'm submitting a bug report

Library Version:
1.6.0

Please tell us about your environment:

Operating System:
Windows 10

Node Version:
8.9.1

NPM Version:
5.5.1

Browser:
Chrome 62.0.3202.94

Language:
TypeScript >=1.9.0-dev || ^2.0.0

Current behavior:
When using a primary property on a custom attribute that is meant to receive an URL, the simple syntax does not pass the value to the property.
My guess is that the : on the URL confuses the parser to think I'm setting a property called http.
The behavior can be seen here: https://gist.run/?id=c1d96d1d33ac50eb1fd3ab1a3007e049
Expected/desired behavior:
The engine should recognize the value as a valid URL/URI and set the primary property with it.

What is the expected behavior?
The engine should recognize when the attribute value is set to a valid URI and set the primary property accordingly.

What is the motivation / use case for changing the behavior?
In my case, I'm trying to write an attribute that remotely load options to a select element. The attribute should receive an URL as value so the request can be made.

The attribute for debug purposes is as simples as this:
import {bindable, inject} from 'aurelia-framework';

@inject(Element)
export class RemoteSourceCustomAttribute {
  @bindable value: string;
  @bindable({ primaryProperty: true }) url: string;

  constructor(elem) {
    this.elem = elem;
  }

  attached(){
    this.elem.innerText = `someProperty: ${this.value} | color: ${this.color}`;
  }
}

It is used on the view as this:
<template>
  <require from="./remote-source"></require>

</template>

The syntax of custom attributes follow standard syntax you see in html style attribute:

Checking for url to parse correctly is unexpected. I think this is a won't fix issue.
@EisenbergEffect

@bigopon you are right, but even the style attribute accepts url by doing url(), maybe that can be a way to implement this?
I'm thinking maybe something like:
`

`
Then the view model would receive the URL string.
What do you think?

For me, I would only pass the part after https:, so it still feels invalid to me (it looks like url(https = //...). Also I'm not sure how to distinguish between what you describe and standard syntax in the parser.

@SuricateCan can you provide a use case for that value being a sting literal?
if you use a variable instead I assume everything will work fine.
So what's your "edge case"?

@Alexander-Taran you are correct. If I bind the default value to a variable, the value gets passed correctly. This behaviour only happens when using a string literal that represents a "protocol". I assume any other protocol will fall in the same issue.
Where this would be useful is, for example, a component that can receive only a URL to function properly, like a remote source for a static list, or something like this.
Here in the company I work, we have lots of components that receive a URL in the HTML so handle the calls. I don't think this is a good idea, but in some cases can be useful.
PS.: I've updated the Gist https://gist.run/?id=c1d96d1d33ac50eb1fd3ab1a3007e049

@SuricateCan that I understood.. why is it in markup though? Is it server-rendered? or just like that in code base?

Not server rendered. It fetches data to list, in my use case.

I mean why url is statically defined in markup?
why it's not dynamic?

Because the content is kind of static. It fetches a list from the server, that may or may not change. The source is always the same.

I found this issue when creating a component that fills a <select> element with <option>s from the server, where the list is built from an enumeration.

well obvious workaround is.. to have url defined in vm
Highly unlikely this will be modified in v1

@bigopon and the question is kinda answered?

This is a won't fix for now. Unless it's really desirable. There' many ways to work around it, simplest is with let as an example. Changing the attr parser in v1 isn't a high priority unless someone wants to take on that.


feat: Adding option to send world position instead of local position when spawning

Alternative to: https://github.com/vis2k/Mirror/pull/1997

It's not clear from this PR what gyrations the user has to go through to use world pos/rot.

It's not clear from this PR what gyrations the user has to go through to use world pos/rot.

I assume you are asking how they would use this??
NetworkServer.Spawn(obj, new SpawnSettings { useWorldPosition = true });

or if they need owner too,
NetworkServer.Spawn(obj, new SpawnSettings {
    useWorldPosition = true,
    ownerConnection = owner,
});

Adding SpawnSettings allows us to add other settings in the future if we want to. for example custom payloads to be used in custom spawn handlers

I'm looking for a checkbox someplace that defaults true to use world coordinates as most users expect, and that can be set false for corner cases that want locals used.

I'm looking for a checkbox someplace that defaults true to use world coordinates as most users expect, and that can be set false for corner cases that want locals used.

That "corner case" is how mirror works right now, any change from that would be a breaking change.
Mirror doesn't support object being in a hierarchy, if a user needs that they have to do something outside of mirrors code to deal with it.
It might be possible to add a check box to the inspector but that but it would make it hard to do this.

Adding SpawnSettings allows us to add other settings in the future if we want to. for example custom payloads to be used in custom spawn handlers

so what's the use case again?
I am still trying to think of a case where .localPosition wouldn't work.

so what's the use case again?
I am still trying to think of a case where .localPosition wouldn't work.

The original issue is explained in the original PR #1997  @vis2k

+1. My old-ummorpg2d-based game broke due to this when I migrated it from using UNET to Mirror.

closing for same reason as #1997
by definition, server and client get out of sync if networkidentities have parents because client will always spawn them without parents.
this can work if parent has 0,0,0 position/rotation, but a better solution would be a warning in networktransform that the user shouldn't parent it to something with != 0,0,0


Just fork one and customize it by yourself.


[Bug][r] SOMAExperimentAxisQuery drops numeric query coordinates

Describe the bug
In R, numeric soma joinids are (mostly) silently dropped when creating a SOMAExperimentAxisQuery. Numeric joinids should either be coerced to an appropriate type, or generate an error.
To Reproduce
Create a SOMAAxisQuery with integer or numeric coordinates
expt <- tiledbsoma::load_dataset("soma-exp-pbmc-small")
integer_query <- tiledbsoma::SOMAAxisQuery$new(coords = 1:2)
numeric_query <- tiledbsoma::SOMAAxisQuery$new(coords = c(1, 2))
expt_query <- tiledbsoma::SOMAExperimentAxisQuery$
    new(expt, "RNA", var_query = integer_query, obs_query = numeric_query)

The var_joinids() (integer values) are as expected
> expt_query$var_joinids() # expected -- [1, 2]
Array
<int64>
[
  1,
  2
]

but the obs_joinids() (numeric values) are not -- only the numeric value 0 maps correctly
> expt_query$obs_joinids() # unexpected -- []
Array
<int64>
[]

Versions (please complete the following information):
> tiledbsoma::show_package_versions()
tiledbsoma:    0.99.0
tiledb-r:      0.20.0
tiledb core:   2.16.0
libtiledbsoma: libtiledb=2.16.0
R:             R version 4.3.0 Patched (2023-05-01 r84362)
OS:            macOS Monterey 12.6.7

Additional context
None

Thanks for report, Martin. (And do feel free to more expressive in your Subject: or Issue: title than the minimalistic [Bug]).
We do indeed fail to cover and convert from numeric aka double.  If you are explicit all is well:
> numeric_query <- tiledbsoma::SOMAAxisQuery$new(coords = bit64::as.integer64(c(1, 2)))
> expt_query <- tiledbsoma::SOMAExperimentAxisQuery$new(expt, "RNA", var_query = integer_query, obs_query = numeric_query)
> expt_query$obs_joinids()
Array
<int64>
[
  1,
  2
]
>

Thanks for the helpful reprex!
I'm thinking we should throw a warning if the user passes a double to coords and then coerce it to an integer. We could also use rlang::is_integerish() to verify the input is integer-like but that seems like overkill.
We could also take this opportunity to update our internal function for validating coordinates to support non-numeric coordinates and verify the type of each coordinate vector is compatible with the corresponding dimension's type.
Currently it only performs some basic sanity checks and assumes the underlying tiledb array is indexed by ints, which is the default but not necessarily the case.


feat: update rbac and metrics specs

fix: update metrics import as side affect
feat: update rbac and scoped verbs

What this PR does / why we need it:

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
~[ ] Breaking change (fix or feature that would cause existing functionality to change)~

Which issue/s this PR fixes

How Has This Been Tested?

Checklist:

~[ ] My change requires a change to the documentation.~
~[ ] I have updated the documentation accordingly.~
[x] I've read the CONTRIBUTION guide
[x] I have added CI tests to cover my changes.
[x] All new and existing tests passed.

/code pr


catch IOError in ensure_file

[x] Tests added / passed
[x] Passes flake8 dask

I didn't bother with a test and can mock something if necessary. Ran into this with a read only file system when grabbing the new version.

Thanks, this is useful.


Incompatibility of pybiomart and omnipath

Hi @deeenes,  there seems to be a weird bug regarding pybiomart and omnipath. In the beginning of the pseudobulk vignette of decoupler I use pybiomart to retrieve gene symbols and then later I import collectri. The problem is that when I run the gene symbol query, it generates a hidden file called .pybiomart.sqlite which triggers a huge amount of warnings for omnipath. This used to not be the case since you can pass the argument use_cache=False but it still creates the file anyways now, I've opened an issue about it https://github.com/scverse/scanpy/issues/2861
I tried deleting the file before loading collectri but omnipath still spams warnings and falls to the static tables. The only way to make it work is to load first collectri and then biomart, which breaks the flow of the vignette. Is there anything that can be done from omnipath to solve this? Any clue what might be causing it?
You can reproduce the error by running:
import scanpy as sc
import decoupler as dc

annot = sc.queries.biomart_annotations(
    'hsapiens',
    ['ensembl_gene_id', 'external_gene_name'],
    use_cache=False
)

collectri = dc.get_collectri(organism='human', split_complexes=False)

Thanks!

Hi Pau,
I think it has nothing to do with the sqlite file (that would be indeed surprising), the most likely I can think about is pybiomart doing something with urllib3 parameters. A more minimal example to reproduce the issue:
import pybiomart
import omnipath as op

Already import time omnipath downloads some resource metadata, hence this is enough to trigger the Connection broken: IncompleteRead errors. The only condition is that pybiomart has to be imported first, so it has a chance to do something. The question is, what is this something.
Also, this seems to be a pybiomart issue, I would open an issue in their repo. I'm not sure if the project is active, it hasn't been updated for 7 years. I found that the setup of requests_cache triggers the issue:
import requests_cache
requests_cache.install_cache('foobar')
import omnipath as op

I'm not familiar with requests_cache, but the omnipath module relies entirely on requests (an API for urllib3). I found a few similar issues, for example this one. Developers at requests claim they've already fixed the issue. Indeed, the tracebacks point to requests_cache, specifically, the part requests_cache/models/raw_response.py", line 77, in from_response in the second traceback (the first one is the original urllib3 error, the third one is omnipath's error handling, the second one happens in requests_cache):
WARNING:root:Failed to download from `http://no-tls.omnipathdb.org/`.
WARNING:root:Traceback (most recent call last):
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 738, in _error_catcher
    yield
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 875, in _raw_read
    raise IncompleteRead(self._fp_bytes_read, self.length_remaining)
urllib3.exceptions.IncompleteRead: IncompleteRead(-196 bytes read, -196 more expected)

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

Traceback (most recent call last):
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests/models.py", line 816, in generate
    yield from self.raw.stream(chunk_size, decode_content=True)
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 1035, in stream
    data = self.read(amt=amt, decode_content=decode_content)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 955, in read
    data = self._raw_read(amt)
           ^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 852, in _raw_read
    with self._error_catcher():
  File "/usr/lib/python3.11/contextlib.py", line 155, in __exit__
    self.gen.throw(typ, value, traceback)
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/urllib3/response.py", line 755, in _error_catcher
    raise ProtocolError(f"Connection broken: {e!r}", e) from e
urllib3.exceptions.ProtocolError: ('Connection broken: IncompleteRead(-196 bytes read, -196 more expected)', IncompleteRead(-196 bytes read, -196 more expected))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/denes/omnipath_python/local/omnipath/_core/downloader/_downloader.py", line 144, in maybe_download
    res = self._download(req)
          ^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/local/omnipath/_core/downloader/_downloader.py", line 179, in _download
    with self._session.send(
         ^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests_cache/session.py", line 205, in send
    response = self._send_and_cache(request, actions, cached_response, **kwargs)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests_cache/session.py", line 233, in _send_and_cache
    self.cache.save_response(response, actions.cache_key, actions.expires)
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests_cache/backends/base.py", line 89, in save_response
    cached_response = CachedResponse.from_response(response, expires=expires)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests_cache/models/response.py", line 102, in from_response
    obj.raw = CachedHTTPResponse.from_response(response)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests_cache/models/raw_response.py", line 77, in from_response
    _ = response.content  # This property reads, decodes, and stores response content
        ^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests/models.py", line 899, in content
    self._content = b"".join(self.iter_content(CONTENT_CHUNK_SIZE)) or b""
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/denes/omnipath_python/venv/lib/python3.11/site-packages/requests/models.py", line 818, in generate
    raise ChunkedEncodingError(e)
requests.exceptions.ChunkedEncodingError: ('Connection broken: IncompleteRead(-196 bytes read, -196 more expected)', IncompleteRead(-196 bytes read, -196 more expected))

Note, the -196, -196 is because at this point I've already tinkered with this file, originally it was a double read error, with 392 read, -196 more expected. We have no chance to work around this issue in omnipath, because the thing happens within one method in requests_cache. So I sent them a PR. We can't either point to this PR in our packages, bc requests_cache is not our direct dependency. pybiomart could set the fixed version in their requirements, if it has any active maintainer.
Btw, pypath has a simple BioMart client:
from pypath.inputs import biomart
import pandas as pd

human_genesymbols = pd.DataFrame(biomart.biomart_query('external_gene_name', gene = True))

human_genesymbols
       ensembl_gene_id external_gene_name
0      ENSG00000210049              MT-TF
1      ENSG00000211459            MT-RNR1
2      ENSG00000210077              MT-TV
3      ENSG00000210082            MT-RNR2
4      ENSG00000209082             MT-TL1
...                ...                ...
70706  ENSG00000288629
70707  ENSG00000288678
70708  ENSG00000290825            DDX11L2
70709  ENSG00000227232             WASH7P
70710  ENSG00000290826

[70711 rows x 2 columns]

Wow @deeenes! You seems to have solved this mystery, after installing your requests_cache PR it works flawlessly. Thanks for digging into this! :wink:

@PauBadiaM I released @deeenes's fix in the latest version of requests-cache.
I see pybiomart doesn't pin any of its dependencies, so you can pin requests-cache in your own project if you'd like, i.e. requests-cache~=1.2.


Add network param for all spaces in explore endpoint

Fixes #124

@ChaituVR can you merge this?

@ChaituVR can you merge this?

@metrox-eth Done

@ChaituVR can you merge this?

I think @ChaituVR wanted to tag you

hahaha ya sorry for the wrong tag @metrox-eth
@midgerate Done Done Done! It's merged🚀


Place 'edit' and ‘remove’ functionalities within the same menu tier

Currently, you have to navigate to 'edit column' first in order to remove a column. I found it a bit counterintuitive to have ‘remove’ functionality as a subset of the 'edit' functionality; to me, they are two separate kinds of actions on par with one another.
In other words: I’m either looking to edit a column or to delete it. I wouldn't think to edit a column I'm intending to get rid of, so navigating first to 'edit' in this instance feels a bit unclear. I think they should appear on the same level.
Proposed solution: I'd like to propose adding the remove column functionality - or any kind of functionality that involves deleting information - to the first menu that pops up, not the second.
Here's a screenshot for reference:

Agree.  Remove and Rename are used quite often from my trainings I've seen in the past.  Remove could even maybe have a dedicated small X button on each column header?  Rename could be performed by double-click on the column header that makes it editable, but I feel that style of renaming a column sometimes gets in the way and accidental click/renames happen, based on my experience in other apps that do that.


x/tools/gopls: go list causes 100% CPU load

What version of Go are you using (go version)?

$ go version
go version go1.14 darwin/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users/zm/go/bin"
GOCACHE="/Users/zm/Library/Caches/go-build"
GOENV="/Users/zm/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/zm/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.14/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.14/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/_6/z1bzbqfn1h9byyv41hn2qgvc0000gn/T/go-build488495920=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

I just use VSCode to edit go files.
What did you see instead?
I see the CPU fan is loud. then I see the "go" process use 100% CPU. After I quit VSCode, the "go" process still use 100% CPU. maybe the recent update of gopls is related?

Now I have force quit the "go" process, since the CPU fan is loud.

You didn't write so, but I'm guessing you're using the go lsp? And that go list invocation is from gopls? maybe @stamblerre can tell.

@kzhui125: Are you using modules or GOPATH mode? What was the root directory you opened gopls with? Can you run go list -e -json -compiled=true -test=true -export=false -deps=true --find=false -- fmt from the command line? Does it exit normally or does it print any errors?

I opened two VSCode windows (two small repos). One use go modules, another one only has one go file, didn't run "go mod init" yet.
When I run go list -e -json -compiled=true -test=true -export=false -deps=true --find=false -- fmt on the two repos, both exit normally and quicklly.
I will see if I meet this issue again in the future. Thanks.

@kzhui125: Have you encountered this issue again, or can this be closed?

Did not encounter this problem again, thanks.

Unfortunately, I'm running into this now.

Here's the command that's running:
m                11926 100.0  0.0  5084696     24   ??   What gopls version were you using?
@heschik: I believe you had previously looked into these issues (and resolved them with the gocommand package). Do you think it's possible that go list could hang for an hour for a nonexistent package?

Hey @stamblerre I just opened another issue: https://github.com/golang/go/issues/42132


Simple Example using both dia_router and dia_body

Hi
I've been trying to get your example code working. If I provide the request body as a JSON string how do I access this data in a route? The sample code you provide outputs nothing to the browser, I was expecting to see the data in the ctx.parsed property.
Perhaps some documentation would make things easier to understand.
When I try to combine this with dia_router the code won't run, see my attempt below.
import 'dart:io';

import 'package:dia/dia.dart' as dia;
import 'package:dia_router/dia_router.dart' as dia_router;
import 'package:dia_body/dia_body.dart';

/// Custom Context with Routing mixin
class ContextWithRouting extends dia.Context with dia_router.Routing, ParsedBody {
  ContextWithRouting(HttpRequest request) : super(request);
}

main() async {

  final app = dia.App((req)=>ContextWithRouting(req));

  app.use(body());

  final router = dia_router.Router('');

  router.post('/', (ctx,next) async {
    ctx.body = '''
      query=${ctx.query}
      parsed=${ctx.parsed}
      files=${ctx.files}
    ''';
  });

  app.use(router.middleware);

  await app.listen('<IP_ADDRESS>', 8080);
  print('web server running');

}

When I run this I get the following output:
post.dart:27:18: Error: The getter 'parsed' isn't defined for the class 'Routing'.
 - 'Routing' is from 'package:dia_router/src/routing_mixin.dart' ('../../.pub-cache/hosted/pub.dartlang.org/dia_router-0.1.6/lib/src/routing_mixin.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'parsed'.
    parsed=${ctx.parsed}
                 ^^^^^^
post.dart:28:17: Error: The getter 'files' isn't defined for the class 'Routing'.
 - 'Routing' is from 'package:dia_router/src/routing_mixin.dart' ('../../.pub-cache/hosted/pub.dartlang.org/dia_router-0.1.6/lib/src/routing_mixin.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'files'.
    files=${ctx.files}

Any help would be appreciated.

Hi!
Your code will be that:
import 'dart:io';

import 'package:dia/dia.dart' as dia;
import 'package:dia_body/dia_body.dart';
import 'package:dia_router/dia_router.dart' as dia_router;

/// Custom Context with Routing mixin
class ContextWithRouting extends dia.Context
    with dia_router.Routing, ParsedBody {
  ContextWithRouting(HttpRequest request) : super(request);
}

main() async {
  final app = dia.App<ContextWithRouting>((req) => ContextWithRouting(req));

  app.use(body());

  final router = dia_router.Router<ContextWithRouting>('/perfix');

  router.post('/', (ctx, next) async {
    ctx.body = '''
    query=${ctx.query}
    parsed=${ctx.parsed}
    files=${ctx.files}
    ''';
  });

  app.use(router.middleware);

  await app.listen('<IP_ADDRESS>', 8080);
  print('web server running');
}

That fixes the errors, thanks for the super fast reply, you star! How do I get access to the JSON string in the request body. I'm setting the request header Content-Type to application/json and the request body looks like this:
{
  "name": "Mark"
}

This data is not being printed in the response. How do I access this request body in the route please?
Many thanks in advance.

Here is an example of testing your code through Postman

As you can see, the passed json is contained in the field  parsed={test: 1111}

Thanks however here is the output of the code when I run it:

Use this code https://github.com/unger1984/dia_body/issues/1#issuecomment-868980911

That's the code I'm running. It won't show me the request body.