WebSocketServerProtocolHandshakeHandler improvements

Motivation:
Make code easier to read, make WebSocketServerProtocolHandshakeHandler.getWebSocketLocation method faster.
Modification:
WebSocket path check moved to separate method. Get header operation moved out from concat operation.
Result:
WebSocketServerProtocolHandshakeHandler.getWebSocketLocation is faster as OptimizeStringConcat could be applied. Code easier to read.
Benchmark                                          Mode  Cnt         Score         Error  Units
ConcatBenchmark.getWebSocketLocationHostMovedOut  thrpt   40  30327006.732 ± 1016440.196  ops/s
ConcatBenchmark.getWebSocketLocationOld           thrpt   40  13422614.312 ±  292622.139  ops/s

Cherry-picked into 4.1 (d1b02257248adc250cc3ac61cb5efd179675519b)


docs: Add a note on cleaning dangling images

Possibly add a note in README.md on cleaning up Docker images that Dockter builds, and in particular, cleaning up the dangling images (none), as per https://www.projectatomic.io/blog/2015/07/what-are-docker-none-none-images/
Just happened to me and only figured it out now

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


raise AttributeError(f"No {package_name} attribute {name}") AttributeError: No huggingface_hub attribute hf_api

File "run.py", line 7, in
import datasets
File "/home/appuser/miniconda3/envs/pt2/lib/python3.8/site-packages/datasets/init.py", line 37, in
from .builder import ArrowBasedBuilder, BeamBasedBuilder, BuilderConfig, DatasetBuilder, GeneratorBasedBuilder
File "/home/appuser/miniconda3/envs/pt2/lib/python3.8/site-packages/datasets/builder.py", line 44, in
from .data_files import DataFilesDict, _sanitize_patterns
File "/home/appuser/miniconda3/envs/pt2/lib/python3.8/site-packages/datasets/data_files.py", line 120, in
dataset_info: huggingface_hub.hf_api.DatasetInfo,
File "/home/appuser/miniconda3/envs/pt2/lib/python3.8/site-packages/huggingface_hub/init.py", line 290, in getattr
raise AttributeError(f"No {package_name} attribute {name}")
AttributeError: No huggingface_hub attribute hf_api

huggingface_hub version: 0.13.2
Platform: Linux-5.4.0-144-generic-x86_64-with-glibc2.10
Python version: 3.8.5
Running in iPython ?: No
Running in notebook ?: No
Running in Google Colab ?: No
Token path ?: /home/appuser/.cache/huggingface/token
Has saved token ?: False
Configured git credential helpers:
FastAI: N/A
Tensorflow: N/A
Torch: 1.7.1
Jinja2: N/A
Graphviz: N/A
Pydot: N/A
Pillow: 9.3.0
hf_transfer: N/A
ENDPOINT: https://huggingface.co
HUGGINGFACE_HUB_CACHE: /home/appuser/.cache/huggingface/hub
HUGGINGFACE_ASSETS_CACHE: /home/appuser/.cache/huggingface/assets
HF_TOKEN_PATH: /home/appuser/.cache/huggingface/token
HF_HUB_OFFLINE: False
HF_HUB_DISABLE_TELEMETRY: False
HF_HUB_DISABLE_PROGRESS_BARS: None
HF_HUB_DISABLE_SYMLINKS_WARNING: False
HF_HUB_DISABLE_IMPLICIT_TOKEN: False
HF_HUB_ENABLE_HF_TRANSFER: False

pip install datasets --upgrade


breeze.viz : plot two objects in same color

Hello,
on several occasions already I needed to plot two objects (eg. dots and a line) in the same color. I would like to avoid specifying the color explicitly but rather use the builtin palette to be robust against possible future changes.
I've checked the plot/package.scala and the plot/PaintScale.scala implementation to find a way, but I do not see anything like this implemented. Do you have any idea?
My next try would have been to explicitly pass the palette PaintScale.Category10 to plot, but it only accepts colorcode: String. A conversion function, the inverse of PaintScale.convertToColor doesn't exist. Do you know a work-around?
Alternatively, would you be interested in a PR for either one of those:

a new argument to plot, namely color: java.awt.Color to use it like this: color=PaintScale.Category10(1) to get the same color as the second plot
an inverse conversion function (might be hard to maintain)
a new magic string to pass to colorcode, indicating to take the same color as plot number x, e.g. colorcode="plot1" to use the same color as the second plot.

Cheers!

After experimenting a bit, I found out that the colorcode also accepts RGB values. So I could suggest a convertToColorCode function along those lines:
  def convertToColorCode(col: java.awt.Color): String = {
    val binCol = col.getRGB.toBinaryString
    val b = Integer.parseInt(binCol.takeRight(8), 2)
    val g = Integer.parseInt(binCol.dropRight(8).takeRight(8), 2)
    val r = Integer.parseInt(binCol.dropRight(16).takeRight(8), 2)
    "[%d,%d,%d]".format(r, g, b)
  }

Hi Andreas,
Sorry for the slow response. I will happily entertain PRs.
I say this pretty much anytime anyone asks about breeze viz: I'm generally
not super-interested in spending a lot of time on breeze.viz because I
don't actually know anything about visualization and don't currently have
plans to. But I'm happy to talk through PRs.
On Thu, Mar 22, 2018 at 5:09 AM, Andreas Maier<EMAIL_ADDRESS>wrote:

After experimenting a bit, I found out that the colorcode also accepts
RGB values. So I could suggest a convertToColorCode function along those
lines:
def convertToColorCode(col: java.awt.Color): String = {
val binCol = col.getRGB.toBinaryString
val b = Integer.parseInt(binCol.takeRight(8), 2)
val g = Integer.parseInt(binCol.dropRight(8).takeRight(8), 2)
val r = Integer.parseInt(binCol.dropRight(16).takeRight(8), 2)
"[%d,%d,%d]".format(r, g, b)
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/scalanlp/breeze/issues/701#issuecomment-375282223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAlodgP7qgxhETr8ScSRGg6aMuYLHxnks5tg5SRgaJpZM4S249_
.


Prompt not appearing and Ctrl-C closes the window

Environment
Windows build number: Windows 10 Build 19041.330
Your Distribution version: Ubuntu 18.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce

Open Ubuntu 18.04 WSL

Expected behavior

Prompt appears
Control C doesn't close the window

Actual behavior

The Prompt doesn't appear. I've had an issue for a while, where the prompt takes 5-10 seconds to load. I usually hit Ctrl-C and the prompt loads up instantly after that. But today, the prompt doesn't appear at all. I still get the blinking underscore without the prompt.
Typing doesn't do anything. Also, Ctrl-C instantly closes the window.
Additionally, can't use any of the wsl commands on cmd and powershell. The only command which works is wsl --help.
The others seem to load forever.

Found a temporary fix by doing a Full Reboot as stated here:
SO Answer
But it might happen again and the prompt still took a few seconds to load. On pressing Ctrl-C, it terminates the loading and I can see the prompt again. Is there a fix to this?

Next this happens, I would try restarting LxssManager from Services.

Yesterday this problem happen to my ubuntu distro too, I see in task manager and I noticed that the Vmmem (This is a process that makes suport the virtual machines) is not processing, so I think that this is part of the problem too.

Hello. I fix it with a tutorial in stack overflow. I don't have the link anymore but the steps are
Press win + r and paste REDEGIT
Will open the Registry Editor so go to HKEY_LOCAL_MACHINE > SYSTEM > CurrentControlSet > Services > LxssManager click on Start and set the Value data and change to 2, then you restart your computer and open ubuntu again

Will open the Registry Editor so go to HKEY_LOCAL_MACHINE > SYSTEM > CurrentControlSet > Services > LxssManager click on Start and set the Value data and change to 2, then you restart your computer and open ubuntu again. It's work for me so I hope that works for you.

This fixed didn't work for me. Still stuck at blank screen.

Next this happens, I would try restarting LxssManager from Services.

When restarting the service, it hung up on Stopping status

OS Build: 20215.1000
WSL2

Hello. I fix it with a tutorial in stack overflow. I don't have the link anymore but the steps are
Press win + r and paste REDEGIT
Will open the Registry Editor so go to HKEY_LOCAL_MACHINE > SYSTEM > CurrentControlSet > Services > LxssManager click on Start and set the Value data and change to 2, then you restart your computer and open ubuntu again. It's work for me so I hope that works for you.

I think you mean REGEDIT, not REDEGIT.

this resolved it for me without needing to change any registry settings

still happening - resolved as per here without changing settings


Separates toolbar's scss variables to an individual file so that library users can import variables without getting the rules.

I signed it. Please check. Thanks!

I signed it!

If we were to do this, it would need to be done consistently for all components with documentation and a commitment to stability. We currently consider these variables to be private implementation details such that we can't commit to keeping them stable as an API.


fix: Default root_volume_type must be gp2

PR o'clock
Description
Please explain the changes you made here and link to any relevant issues.
Checklist

[ ] README.md has been updated after any changes to variables and outputs. See https://github.com/terraform-aws-modules/terraform-aws-eks/#doc-generation

Just curious, why the default must be gp2?

Just curious, why the default must be gp2?

If I remember, it's because it doest support for launch configuration. I'll with gp3 when we'll drop LC support (soon).

Just curious, why the default must be gp2?

If I remember, it's because launch configuration doesn’t support it. I'll with gp3 when we'll drop LC support (soon).

Thanks for the explanation

On a global scale this commit causes AWS billing to increase for everyone using this module :)

@barryib Is there plan to support Launch Template instead? See https://forums.aws.amazon.com/thread.jspa?messageID=969349

I agree with @yalimpop this module will become undesirable in the future with only GP2 support.

@ivandir this issue is quite old and the latest version of the module provides support for any EBS type desired


[type: refactor] support custom origins for cors 'Acess-Control-Allow-Origin'.

…-Origin'.

Make sure that:

[ ] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[ ] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3663 (2dce50a) into master (0785508) will decrease coverage by 0.48%.
The diff coverage is 40.74%.

@@             Coverage Diff              @@
##             master    #3663      +/-   ##
============================================
- Coverage     62.58%   62.10%   -0.49%
+ Complexity     5957     5909      -48
============================================
  Files           908      908
  Lines         25027    25025       -2
  Branches       2289     2283       -6
============================================
- Hits          15664    15542     -122
- Misses         7916     8034     +118
- Partials       1447     1449       +2

Impacted Files
Coverage Δ

.../org/apache/shenyu/common/config/ShenyuConfig.java
73.91% <0.00%> (-0.24%)
:arrow_down:

...java/org/apache/shenyu/web/filter/CrossFilter.java
52.45% <45.83%> (-27.18%)
:arrow_down:

...henyu/plugin/logging/kafka/LoggingKafkaPlugin.java
22.72% <0.00%> (-72.73%)
:arrow_down:

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
75.00% <0.00%> (-25.00%)
:arrow_down:

...n/logging/kafka/body/LoggingServerHttpRequest.java
0.00% <0.00%> (-20.00%)
:arrow_down:

...gin/logging/kafka/kafka/KafkaLogCollectClient.java
34.69% <0.00%> (-18.37%)
:arrow_down:

...yu/sync/data/http/refresh/AbstractDataRefresh.java
68.96% <0.00%> (-13.80%)
:arrow_down:

.../plugin/logging/kafka/entity/ShenyuRequestLog.java
86.20% <0.00%> (-13.80%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

... and 43 more

Continue to review full report at Codecov.

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


[5.5] Load core providers before package providers

This change will ensure that package providers in the config file will be loaded before auto-discovered package providers.
Example of a problem caused by this:
In PassportServiceProvider we use the Auth facade which isn't available until the Auth service provider is registered.

I've mentioned before, this makes it impossible to override package providers.

The actual solution to this is to inject the package providers after the framework providers but before the application providers, which I do not know of a clean way to do.

In the Passport example, we should change the Passport service provider to fix this.

hmm makes sense.


Update documentation and make summary link more user-friendly

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

Comparison is base (a690f78) 94.24% compared to head (a4c0ecf) 94.24%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           typescript      #89   +/-   ##
===========================================
  Coverage       94.24%   94.24%
===========================================
  Files               5        5
  Lines             469      469
  Branches           37       37
===========================================
  Hits              442      442
  Misses             27       27

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


stop typing interactions as multiple type, but only the most specific

So, all resources a Interaction would only be typed with the most specific type, and not all of them. Since we have the WP ontology loaded, we have that info anyway.
For example, ...

This idea conflicts with https://github.com/wikipathways/GPML2RDF/issues/64


Also set inactiveBackground

VScode's behavior used to be something like that - automatically have "inactive" color be based on the "active" one. Now it's always the same, which beats the purpose.
Configure the inactiveTitlebar to match the active one.

I was just about to createa pr for this myself. This would be highly appreciated.

@tobi1220 I don't know how on Earth I managed to approve the PR, given I don't have write access to this repo. It seems abandoned, though. We should create a fork if we want this feature.

@rafaelbeckel I'm not the biggest github expert since I mostly use gitlab for work, but it seems like you don't need write access to approve, I just tried giving an additional approval and it worked^^ Also, thanks for the heads-up on how stale this repo is, I think a lot of other users don't know this as well based on the ongoing issue list. I just created my own fork but I think I'll create a new extension for this from the ground up to make sure it's not using semi-legacy code.

Oh, I actually forgot I submitted this...
I've switched over to Peacock when I realized this one's mostly abandoned.

@avivey just checked it out, Peacock looks promising, however I'm looking for an automated solution, so I guess I'll have to finish my own extension

@tobi1220 , can you please notify us here when your extension is ready. I'd gladly adopt it too. Thanks :-)

@afeblot sure thing! I gotta admit, I kinda forgot abt this project since it currently works with my fork of this extension ^^

@tobi1220, would you be willing to make your current fork available on the marketplace? Or would it be there already? I could not find it.

@afeblot I could do that, but tbh I've never published any extension on the marketplace. I don't even know what license the current extension is at, I guess MIT? The thing is, I'd like to adjust it so that it actually uses a *.code-workspace file instead of the .vscode\settings.json since this file is usually version-controlled... but I haven't found the time yet.

Would reccomend Peacock at this point


onDeviceRead listener never call.

Mobile Device Environment
Provide a list of operating systems on which this issue is relevant.

Device: [e.g. Android]
OS: [e.g. API29, Android 10]

Application Environment
Provide information about your development environment:

React Native version: [e.g. v0.63.4]
RN Bluetooth Classic version: [e.g. v1.60.0-rc.12]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Scan device
connect any device
RSSI will be undefined.
onDeviceConnected and onDeviceRead listeners never call

Going to need a bunch more information than that:

Sample code
Full descriptions of what you're doing
Etc.

With regards to your comments:

Scan device, I assume that was from RNBluetoothClassic.startDiscovery()??  When you do this, RSSI should be available in the Extras field (or maybe on it's own, I forget).
Connect to device - how? Did you pair after a scan? Or did you get this from an already bonded device?

Note - Already bonded devices don't have RSSI values, you need to discover them.  So if you want to have RSSI values for bonded devices you need to:

Get a bonded devices
Start a discovery
Merge the two lists together to get a combined bonded + rssi

Depends where you're looking for RSSI.
onDeviceConnected and onDeviceRead are definitely working, I use them thoroughly.  So you'll need way more info.

All in all, you need to provide substantially more information.

@kenjdavidson can you please share the sample code that how to connect a bonded device and as well as the device which is not bond?

It's all in the documentation of the Bluetooth example app in the other repository. You're doing too many connections, and generally reconnecting will drop the connection.
Essentially the flow is

getBondedDevices()
device.connect(). Once this promise returns you don't need to check is Connected.

And you're either good or not.
Fo unbounded you need to pair it first. Either with the OS settings or the app (which honestly I don't use and haven't tested much, it's a remnent of the original lib)
Also, what are you doing in setup Connected?

let connectToDevice = await RNBluetoothClassic.connectToDevice(DEVICE_ADDRESS);
after executing this line connectToDevice RSSI value is undefined.
And in setupConnected function, we have our business logic.

Like I said. Rssi is only available after a device is scanned or discovered. So if you want to get the Rssi you need to

discover devices first
then chekc if connected (isbonded should be available too)
then connect

Or.

connect first
then dicover
then listen for onDeviceDiscovered, filter for that device
merge rssi

This is an android thing. Not a library thing. It's just how it works.

Like I said. Rssi is only available after a device is scanned or discovered. So if you want to get the Rssi you need to

discover devices first
then chekc if connected (isbonded should be available too)
then connect

Or.

connect first
then dicover
then listen for onDeviceDiscovered, filter for that device
merge rssi

This is an android thing. Not a library thing. It's just how it works.

@kenjdavidson did you check Steps to reproduce the behavior

simple is that can we get RSSI value any moment after the device connected? @kenjdavidson

Yes. The example app has the following finctionality

discovery where rssi is available
connecting
listing bonded

All of which you need to use.
With regards to discovery the app only listens for all devices discovered and merges the list. But I have put in the ability to receive single discovery devices.
Feel free to let me know if it doesn't work. But provide details. Like exact code and maybe some debugging. I don't have a lot of time these days so I can't do it from scratch.

simple is that can we get RSSI value any moment after the device connected? @kenjdavidson

That's great. But that's not how Android works. Rssi is only available after discovery. That's in the android docs man. You need to do some research as well.
There is no rssi available if you just get paired devices or connect.
And its not wise or common enough use case for me to add the discover merge logic on connect or listing bonded. When it's easy enough for you to do yourself and no one else has asked for it.

@kenjdavidson did you check Steps to reproduce the behavior

Scan Device
I have absolutely no idea what you mean by this, since you've provided no context, but if you mean have I tested RNBluetoothClassic.startDiscovery() then yes, I have tested it and it works.

https://github.com/kenjdavidson/react-native-bluetooth-classic-apps/blob/1cba89aaa1181998dbcab7a9549a88a9bb5071a1/BluetoothClassicExample/src/device-list/DeviceListScreen.js#L159

Connect any Device
Yes, I've connected to multiple devices:

Barcode scanners
Other phones/tablets

RSSI will be undefined
This isn't actually an issue in terms of something I can replicate, but the RSSI is definitely available after you perform a discovery.

https://github.com/kenjdavidson/react-native-bluetooth-classic-apps/blob/1cba89aaa1181998dbcab7a9549a88a9bb5071a1/BluetoothClassicExample/src/device-list/DeviceListScreen.js#L174
This section of code is terrible and doesn't do what I want it to do, but if you debug that line you'll see that device.rssi or device.extras.rssi is available.

onConnectedDevice and onDeviceRead are not called
As I mentioned onConnectedDevice will never be called, I've removed it from the native side, I just haven't had a chance to remove it from the React side.  There was too much confusion around what a connected device was to make it worth while for me to maintain.

onDeviceRead is definitely being called when data comes in and you have a listener attached, this is the only method in which I received data from my scanners:
https://github.com/kenjdavidson/react-native-bluetooth-classic-apps/blob/1cba89aaa1181998dbcab7a9549a88a9bb5071a1/BluetoothClassicExample/src/connection/ConnectionScreen.js#L134
You can see in this function I either setup polling or read listener based on the parameter/state defined.
Like I said, you'll need to augment all of these things to get them working:
// This gets me a list of bonded/paired devices (currently no RSSI)
const bonded = await RNBluetoothClassic.getBondedDevices();
setBondedDevices(bonded);  // Set state

// Now start discovery
const discovered = await RNBluetoothClassic.startDiscovery();

// Now update the state with the discovered version of the device.
// this version will have RSSI and a bunch of other "extras" available
const bondedRSSI = discovered.filter(d => bonded.find(b => b.address === d.address));
setBondedDevices(discovered)

// Now you can connect to this device
// Or you could connect first, and do the discovery after, it really doesn't matter as all you're doing is managing BluetoothDevices by address.

Anyhow, unless you provide more information and specific examples I'm not going to worry about this, what you've explained you need is all doable.


Recommendations for Tuning

I am testing your work on my own time series data. However, unlike your datasets in my case increasing the encoder input surprisingly degrades the results  (not I am working with 24h output).  Do you have any recommendation to diagnose this result?
How do you recommend tuning the seq_in , label_in (token) and maybe the number of encoder and decoder level or the factor?
Additionally do you have recommendations regarding when to use the normal informer vs the stackInformer?
Last point, I am having better results when using the univariate predictions, while when trying the MS f the forecasting the results are somehow worse than the S ones, do you have any recommendations to tackle this as well?
If you recommend a tuning scheme it would save me a lot of time. Thank you for your contribution again.  Finally sorry for the many questions.

We did not do enough experiments on --features MS, this option is added based on the feedback from github.
MSE metric has some problems in evaluate long sequence prediction, low mse does not mean good prediction curve, maybe you can draw the prediction curve and groundtruth curve to decide whether to use MS or S.
seq_len and label_len can be set to some meaningful values related to periods such as 24, 48 or 720, you can also use InformerStack and customize the input length in code models/model.py line 102 and code models/encoder.py line 92 to make model get long input and short input at the same time.

I will try your recommendations and thank you for your notes.


string_length

Sir, I want to contribute to your repository please merge my code

give this repo a star and follow me for follow back😊


Fix issue 146

@jvegasbsc you code seems to trigger the pep8 check, and thus the tests now fail. Could you see what the problem is?


Add support for geodetic tiles

So you take into account both the 1x1 and 2x1 tiles scheme for zoom level 0 ?
Eventually, we should avoid '4326' as name, as it is not very well defined.

yes we kind of have to. I don't like this tile schema too much either.
Another funny thing is that our terrain tiles start at the
bottom-left
and MapProxy puts the origin at the top-left...


Only unlet b:current_syntax if it is set.

This should fix #11 and stop vim-vue from complaining when b:current_syntax has not been set.

Oh, thank you!


Add alert rules for a healthy virt-operator

What this PR does / why we need it:
To allow cluster admins to monitor their kubevirt deployment, we provide a set of default
alert rules that indicate on the health status of the components.
This PR adds such rules for virt-operator.
Added prometheus alert rules and metrics to monitor virt-operator

/retest

/retest


Add etcd scaler

Signed-off-by: ytz<EMAIL_ADDRESS>Hi, I'm a software engineer from Huawei Cloud. Thanks to KEDA for providing us with a flexible scaling project that can scale applications from zero. We find that when the pollingInterval parameter is set to a small value, the scaler will frequently pull metrics from the event source, which leads to an increase in the system load.  So by watching the an etcd key, a passively received push mode,  the scaler can activate applications with lower load usage than frequent pull mode.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] Tests have been added
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Changelog has been updated and is aligned with our changelog requirements

Fixes https://github.com/kedacore/keda/issues/3880

Relates to kedacore/keda-docs#914

Can you please fix the DCO, and Linter problems (Static Checks).
Also could you please write e2e test for this scaler? https://github.com/kedacore/keda/tree/main/tests

Sure! But I'm going to bed. I'll do it tomorrow.

Thank you for the contribution! Does Huawei Cloud want to be the maintainer of this scaler? (see scaler governance)

Thank you for the contribution! Does Huawei Cloud want to be the maintainer of this scaler? (see scaler governance)

We are very glad to be the maintainer of this scaler! The team we belong to is Huawei cloud Middleware, so I wonder if we can represent the whole Huawei Cloud.

Out of curiosity, what is the Run() method used for here?

Out of curiosity, what is the Run() method used for here?

The Run() passively accepts the push from the Etcd event source and has the same effect as setting the pollingInterval parameter to 0.

@WindNotStop there are still some static checks & validation issues, also please rebase, we are re-introducing vendor directory.
Thanks for reminding me! I have been busy with work lately. I will solve the static check & validation issues as soon as possible!

/run-e2e etcd*

@WindNotStop looking good! One last thing, could you please open an issue for this (adding the scaler) and then update the Changelog and refence that issue there (the same way it is done for other changes)?

Sure!
Opened the issue
https://github.com/kedacore/keda/issues/3880
Inserted a line into the Changelog

General: Introduce new Etcd Scaler (#3880)

Just for double-checking, did you see this? https://github.com/kedacore/keda/pull/3598#pullrequestreview-1188649008

I have left some inline comments and there is also a problem with the key. Based on the code I feel that we are using it duplicated, for the etcd key and also for the authentication key. We should use unique keys for both

/run-e2e etcd*

/run-e2e etcd*

@JorTurFer @zroubalik Can you re-review this PR please to see if we need to change things before our release on Thursday?

@JorTurFer @zroubalik Can you re-review this PR please to see if we need to change things before our release on Thursday?

I did it, I'm waiting the changes related with using "key" from metadata for etcdKey and "key" from auth for tls key, this could work but it's confusing, the discussion in in docs PR. The rest of the PR LGTM

/run-e2e etcd*

@WindNotStop e2e tests failed, could you please take a look? The link to the run is in the comment above

@WindNotStop e2e tests failed, could you please take a look? The link to the run is in the comment above:
#3598 (comment)

OMG! I think replicas >= target to be true without viewing the code if replicas == int32(target) { 			return true 		} It's now fixed
Could you review this pr https://github.com/kedacore/keda/pull/3599#issue-1351194353? This pr can solve this error.
ERROR	scalehandler	error closing scaler	***"scalerError": "json: unsupported type: func() (scalers.Scaler, *scalers.ScalerConfig, error)", "error": "context canceled"***

@WindNotStop could you please rebase, so this contains the fix for the close and then we will run e2e?

/run-e2e etcd*


Updated to fix creationTime

At least in 32 bit systems the getCreationTime() method on a comment returns invalid values, sometimes even negative values!
I've altered the following code inside the file protocolbuffers.inc.php :
I've replaced:
$i = 0;
$shift = 0;
for ($j = 0; $j < $len; $j++) {
$i |= ((ord($value[$j]) & 0x7F) << $shift);
$shift += 7;
}
By:
$i = 0.0;
for ($j = $len-1; $j >= 0; $j--) $i = $i * 128 + (ord($value[$j]) & 0x7F);
Now the command date("Y-m-d H:i:s",$comment->getCreationTime()/1000) returns the correct date!

I know it's been 3 years, but I think this is now fixed by the tests I've added, and the restructuring of the code to support integers as precisely as possible:
https://github.com/bramp/protoc-gen-php/blob/master/tests/protocolbuffers_test.php#L77
9a428b7d7e5f41674d13edcac02d8dc6ef21b569


nbText with a newline that is surrounded by code blocks can end up looking preformatted.

Hello,
If there is a nbText that has a newline inside of it, and it is surrounded by code blocks, everything before the first line break looks normal, and everything after has a pre tag. Link to example code: https://github.com/ajusa/binarylang-fun/blob/master/intro.nim#L64
Here is what it looks like:

The output markdown looks fine, but it does have the backticks surrounding it from the start and end of the code blocks it borders.

this is because the multi line string has everywhere a 4 space indentation and it is interpreted as a code block (see e.g.: https://pietroppeter.github.io/nimib/cheatsheet.html#code-and-syntax-highlighting but also https://github.github.com/gfm/#indented-code-blocks). The reason why the first block is not interpreted as code it needs an empty line before in order to do that.
the solution is to remove the indentation from multiline string. Either by using something like strutils.dedent (if you want to keep the visual indentation in the source), or removing it directly. Compare the example below:
import nimib
nbInit

nbText: """
    So what's going on here? First, we tell binarylang to go ahead and create a type called http, for parsing.
    Next, we use it to define the format of a header. A header has a string, that starts with HTTP/, and then a version.
    The version is also a string, so we prefix it with a `s`. Then, we look for a space to separate the two, and continue on
    in a similar fashion.
    An underscore (_) simply signifies that we don't care enough about that value to name it. It is good for what are called
    "magic" values, or to skip to the field you actually care about.

    Since binarylang operates on bitstreams, we turn the string into one, and then tell it to parse into an object.

    So, we have a functioning parser for the example header. What do we do if we wanted to generate a header though?
    """

nbText: """
So what's going on here? First, we tell binarylang to go ahead and create a type called http, for parsing.
Next, we use it to define the format of a header. A header has a string, that starts with HTTP/, and then a version.
The version is also a string, so we prefix it with a `s`. Then, we look for a space to separate the two, and continue on
in a similar fashion.
An underscore (_) simply signifies that we don't care enough about that value to name it. It is good for what are called
"magic" values, or to skip to the field you actually care about.

Since binarylang operates on bitstreams, we turn the string into one, and then tell it to parse into an object.

So, we have a functioning parser for the example header. What do we do if we wanted to generate a header though?
"""
nbShow

Got it, thanks! I had no idea that was a part of a markdown spec, I was under the impression that only ``` caused any sort of pre tags. I'll probably be using dedent to prevent it.

yep, it is not so used in markdown but it is in the spec.
As a further seomwhat related remark, note that 2-spaces indentation is the official convention for nim: https://nim-lang.org/docs/nep1.html#introduction-spacing-and-whitespace-conventions
Of course you are free to use whatever indentation spacing you want for your code, just thought of mentioning it.

Yep, I know it is, but I'm too used to using 4 spaces for all my other projects in other languages :smile:


question about build_gt_cloud.py

I want to generate the gt point cloud of the street and compare it with the reconstructed model. But when I execute the following code, there are some errors. If you can, please help solve the following problem. I will appreciate it very much!
docker-compose run --rm apps bash -c '\     ./pipelines/mapping/build_gt_cloud.py  \     --dataset ./data/kitti-odometry/ply \     --sequence 07 \     --n_scans 40     '

If this method doesn't work, can you tell me how to generate street gt point cloud?

Hello, according to the error msg, it says that there is no ground truth poses available. Are you sure to have the correct poses in the correct place? Where is your data?


BigQuery: timestamps not populated correctly for parameterized queries

Stacktrace

$ python sync_query_params.py timestamp
Traceback (most recent call last):
  File "sync_query_params.py", line 176, in <module>
    sync_query_timestamp_params()
  File "sync_query_params.py", line 124, in sync_query_timestamp_params
    query_results.run()
  File "/Users/swast/src/google-cloud-python/bigquery/google/cloud/bigquery/query.py", line 360, in run
    method='POST', path=path, data=self._build_resource())
  File "/Users/swast/venvs/bq-dml/lib/python3.5/site-packages/google/cloud/_http.py", line 335, in api_request
    error_info=method + ' ' + url)
google.cloud.exceptions.BadRequest: 400 Unparseable query parameter `ts_value` in type `TYPE_TIMESTAMP`, Invalid timestamp: '1.4810976E9' value: '1.4810976E9' (POST https://www.googleapis.com/bigquery/v2/projects/cloud-samples-tests/queries)

Steps to reproduce

Attempt to use TIMESTAMP type in a scalar query parameter.

Code example

https://github.com/GoogleCloudPlatform/python-docs-samples/commit/a788ea1829949a4f90d3ce42aa188ce4087f4906
The timestamp type should not be sent as a floating point value. It should take this format: https://cloud.google.com/bigquery/docs/reference/standard-sql/data-types#timestamp-type Note that this is different from RFC 3339. There is an issue open at https://code.google.com/p/google-bigquery/issues/detail?id=842 for accepting RFC 3339 in TIMESTAMP query parameters.

@tswast I have passing system tests which show the back-end sending TIMESTAMP values back as floating-point values.  Are you saying that the back-end expects those values to be sent as string timestamps, in spite of how it returns them?  For instance, here is the result from a query including a timestamp value:
{'_client': <google.cloud.bigquery.client.Client object at 0x7f399dfb33d0>,
 '_configuration': <google.cloud.bigquery.query._SyncQueryConfiguration object at 0x7f399e250090>,
 '_job': None,
 '_properties': {u'cacheHit': True,
                 u'jobComplete': True,
                 u'jobReference': {u'jobId': u'job_N75jt8A_drBKYhdW9gbvuImxAWQ',
                                   u'projectId': u'<MY-PROJECT-ID>'},
                 u'kind': u'bigquery#queryResponse',
                 u'rows': [{u'f': [{u'v': u'1.480941669E9'}]}],
                 u'schema': {u'fields': [{u'mode': u'NULLABLE',
                                          u'name': u'f0_',
                                          u'type': u'TIMESTAMP'}]},
                 u'totalBytesProcessed': u'0',
                 u'totalRows': u'1'},
 '_query_parameters': (),
 '_udf_resources': (),
 'query': 'SELECT TIMESTAMP "2016-12-05 12:41:09"'}

Your example seems to be using legacy SQL. Data types are different in standard SQL.
But yes, the format expected by query parameters is different from the format in the response.

Your example seems to be using legacy SQL. Data types are different in standard SQL.

Nope, all the queries in that system test pass the useLegacySql flag as false.

But yes, the format expected by query parameters is different from the format in the response.

That is a definite API smell, if not an outright bug:  there should be one-and-only-one canonical way to spell the wire format for any given data type (as opposed to the formats humans might type in a query).

Agreed, that's why I filed https://code.google.com/p/google-bigquery/issues/detail?id=842

Thanks! I verified this works with the sample I've written.


Authorization | Approve Recommended Credit Balance Changes (Validation/Award/Reduction) |5|

https://trello.com/c/r9x5zJQD/452-authorization-approve-recommended-credit-balance-changes-validation-award-reduction-5
As the Director, I need to be able to approve Recommended credit balance changes for a fuel supplier organization and this action needs to result in a material change to the credit balance of the organization.
Scenario:
GIVEN that I am an Organization
AND my organization type is Government
AND my role is the Director
AND I have received a Recommended credit balance change from a Government Analyst
AND the Government Analyst is recommending approval
WHEN I am viewing the recommended credit balance change in TFRS
THEN I am able to see the Approve button
AND when I click on the Approve button
THEN the credit balance change appears in the Credit Transactions list (i.e. Account Activity) of both Government and the specific fuel supplier
AND a material change occurs in the credit balance of the fuel supplier
Demo:
Show Approve button on Recommended credit balance changes
Show the change in the fuel supplier's credit balance when the recommended change is approved / completed
Show that the Approve button is not available to any other role in the TFRS system

 Authorization | Approve Recommended Credit Balance Changes (Validation/Award/Reduction) |5|


spawn op

we have default verts and color
use the array? spawn at location of self?
[r], [x, y] (all optional)
moved out of #5

it also adds the spawned entities as targets of self


Failed to execute 'postMessage' on 'DOMWindow':

Hey,
I have just added your lib to a React / Redux app that is consuming ID Server 3 for authentication.
I have everything working as expected, however my browser console is filling with errors
oidc-client.min.js?93dc:75 Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('http://localhost') does not match the recipient window's origin ('null').
I do not want silent renew on, nor do I need the lib to keep checking the state of the session. I thought disabling silent renew would prevent this?

Are you building a native/installed app (as opposed to a browser based app)? And by native/installed, that includes cordova, electron, and/or ionic.

Hey Brock,
It's a browser based app. React / Redux running in the browser. Chrome / FF and IE all show the same message in the console.

I guess what I'm puzzled by is that this is working from our normal samples... so something's not right about your setup or environment.

@brockallen Same Error on me (I use this lib with VUE).

I'm facing the same issue.
I'm using Angular 5. Both the SPA and OP are using https.

Update:
After reading the issue #225 I've got to the conclusion that this error was happening to me due to a previous error:
Refused to display 'https://mydomain.com/connect/checksession' in a frame because it set 'X-Frame-Options' to 'SAMEORIGIN'.

This was happening because my Nginx reverse proxy was adding the header 'X-Frame-Options to the response.
After disabling this on the Nginx server, both the errors disappeared. The one I mentioned in this comment and the one in the title of the issue.


Help for GitHubInterface class

Help is to implement for GitHubInterface class. It should list of all available information about class.

The code was checked with the following call:
from dynpy.utilities.creators import GitHubInterface
help(GitHubInterface)


Feature keycloak helm tests

Create a test script that checks that the Realm is present and that admin-user can retrieve the token.
Add documentation.
Add default values.

LGTM. @alvarolop Update the branch with main and I'll approve it


Let you reuse an existing Ethereum account if you really want to.

This adds a pop-up that gives you the option to connect your existing Ethereum account if you jump through a few hoops. Should make our power-users happier while scaring away casual users who might not have thought through the consequences. Text / UX open to suggestions!

Replaced by https://github.com/zorro-project/zorro/pull/123


Storage tests: Pass app name to pod object template

Pass app name parameter to pod object template, because it's used for volumes names.
Fix: https://github.com/kubernetes/perf-tests/issues/710
/sig storage
/assign @wojtek-t
cc @hantaowang @msau42

What is the app name actually used for? I removed the ability to accept appname from the pod specs.

Oh i see- its still used to populate the volume name. I reckon we can probably remove the test name form the volume and pod base name itself. There is not gonna be another vol-* or pod-* in the test namespace.

@hantaowang ok., I'll update the pr to not use testname.

@mucahitkurt - please ping me when this will be ready

@mucahitkurt - please ping me when this will be ready

@wojtek-t it's ready

/ok-to-test
/lgtm
/approve


fix nightly tests

Had failures as of 12/18, so this fixes them.

The mac failures might be a pandas bug (or an issue of a pandas dependency)?

Really strange, because the runs done less than an hour succeeded:
https://github.com/pandas-dev/pandas-stubs/actions/runs/7250869698/
Maybe something funky with Github?

he runs done less than an hour succeeded

Happy to merge anyway :)

he runs done less than an hour succeeded

Happy to merge anyway :)

I reran the two failed jobs and they succeeded:  https://github.com/pandas-dev/pandas-stubs/actions/runs/7251171162


Panic on empty MP4 AtomDataStorage::Multiple

Reproducer
No response
Summary
I'm reliably hitting a panic at:
https://github.com/Serial-ATA/lofty-rs/blob/b324bfa478298d2b08931738612b6d13437f7e51/src/mp4/ilst/atom.rs#L18
while probing one of my audio files.
Stack trace:
[ ... panic unwind ... ]

   9: core::option::expect_failed
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/option.rs:2045:5
  10: core::option::Option<T>::expect
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/option.rs:905:21
  11: lofty::mp4::ilst::atom::AtomDataStorage::first_mut
             at ./git/festival/external/lofty-rs/src/mp4/ilst/atom.rs:18:39
  12: <lofty::mp4::ilst::Ilst as lofty::traits::SplitTag>::split_tag::{{closure}}
             at ./git/festival/external/lofty-rs/src/mp4/ilst/mod.rs:475:22
  13: alloc::vec::Vec<T,A>::retain_mut::process_loop
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/alloc/src/vec/mod.rs:1647:21
  14: alloc::vec::Vec<T,A>::retain_mut
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/alloc/src/vec/mod.rs:1676:9
  15: <lofty::mp4::ilst::Ilst as lofty::traits::SplitTag>::split_tag
             at ./git/festival/external/lofty-rs/src/mp4/ilst/mod.rs:473:3
  16: lofty::mp4::ilst::<impl core::convert::From<lofty::mp4::ilst::Ilst> for lofty::tag::Tag>::from
             at ./git/festival/external/lofty-rs/src/mp4/ilst/mod.rs:631:3
  17: <T as core::convert::Into<U>>::into
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/convert/mod.rs:726:9
  18: lofty::mp4::<impl core::convert::From<lofty::mp4::Mp4File> for lofty::file::TaggedFile>::from
             at ./git/festival/external/lofty-rs/src/mp4/mod.rs:31:10
  19: <T as core::convert::Into<U>>::into
             at /rustc/84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc/library/core/src/convert/mod.rs:726:9
  20: lofty::probe::Probe<R>::read
             at ./git/festival/external/lofty-rs/src/probe.rs:518:22

I am using a fork but it is the exact same as the current 0.13.0 commit https://github.com/Serial-ATA/lofty-rs/commit/b324bfa478298d2b08931738612b6d13437f7e51 with no changes.
I'd like to provide more info but I can't actually find which file is causing this. Debug printing before the call at:
https://github.com/Serial-ATA/lofty-rs/blob/2b562c4a4b73b96d9bbc7e75268804ad12051a6e/src/mp4/ilst/mod.rs#L380-L386
only reveals Vec's that are .len() >= 1 so I'm not sure. Help on how to debug would be appreciated.
For now, would something like:
impl AtomDataStorage {
    pub(super) fn first_mut(&mut self) -> &mut AtomData {
        match self {
            AtomDataStorage::Single(val) => val,
-            AtomDataStorage::Multiple(data) => data.first_mut().expect("not empty"),
+            AtomDataStorage::Multiple(data) => {
+                if data.is_empty() {
+                    data.push(AtomData::Unknown {
+                        ..Default::default()
+                    });
+                }
+
+                data.first_mut().expect("not empty"),
+            }
        }
    }
}

be okay?
Expected behavior
No response
Assets
No response

I found the culprit files and the exact character in the exact tag causing the panic:
℗

This character was in the copyright field, alongside the company name. Removing it stops the panics.
Ideally, you would be able to reproduce this panic with the same files, although, uploading them here seems... dangerous. How should I proceed?
Other metadata programs (puddletag, id3info, music players) seem to work fine and parse the data correctly. Removing then re-adding that symbol via puddletag also seems to prevent the panics. My suspicion is that it is being converted from some weird formatting to a valid one when doing that.

Thanks for the report!
I'm not able to replicate it with by simply adding ℗ to a copyright message.

For now, would something like:
...
be okay?

AtomDataStorage::Multiple shouldn't need any extra checks for fetching. For it to reach that point means there's a deeper issue here.

Ideally, you would be able to reproduce this panic with the same files, although, uploading them here seems... dangerous. How should I proceed?

You can email them to serial@[domain on my profile] :)

I've sent an email from the one on my profile.
Sorry for all the misdirections but I'm not actually sure it's the copyright field anymore. Re-formatting any field in puddletag seems to fix it; something about the file must have invalid data, which puddletag fixes after saving tag changes.

Can you verify #202 fixes the issue? :)

https://github.com/Serial-ATA/lofty-rs/pull/202 fixes the issue. Thanks.


make a binding rule for the auth user and firestore users collection

meaning if one is deleted other will be deleted too


Refactor ConcurrentChain (WIP)

This is now ready to review (assuming tests pass)
Main area to focus is the ChainIndexer (the old ConcurrentChain) the rest of the changes are mainly renames.
@MithrilMan @noescape00 @fassadlr

some sparse considerations:

I like ChainIndexer name but what that class handles are ChainedHeaders, so maybe ChainedHeaderIndexer would be more appropriate... but at the same time this is a view only on the best chain right? wasn't this BestChainIndexer some attempts ago?

I'd like to see indexer in that class, such that we can access ChainedHeaders using syntax like
this.chainIndexer[anHeight] or this.chainIndexer[aHash]instead of .GetBlock

about GetBlock, this is a misleading name, probably just Get is enough since this class returns always ChainedHeaders (another reason to call the class ChainedHeadersIndexer, because .Get would be self explanatory

P.S.
these are non-blocking considerations, except maybe the GetBlock name that sounds really out of context here

Yeah agree @MithrilMan I will act on that (anyway tests failed)
Not sure about BestChainIndexer anymore by definition this represents the main chain (I still need to add comments)

Refactor of GetBlock  will be done in a second PR (this is already too big)

This PR simplifies the way we index the main chain (it removes the over complexity of ConcurrentChain).
The main changes to this PR are in the classes ChainIndexer (removal of ConcurrentChain and ChainBase), ConsensusManager, the rest is mainly renames.
Next will be the demoval of the dependecny on ChainState

@dangershony apart from those few comments, it looks good to me!


Fix all media types in Accept header to match RFC

Fixes the media type to align with Accept HTTP header RFC which specifies glob syntax /
See https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html
I think we should consider this for 1.3 but should sanity check with additional registries

Codecov Report

Merging #3696 into master will not change coverage.
The diff coverage is 66.66%.

@@          Coverage Diff           @@
##           master   #3696   +/-   ##
======================================
  Coverage    42.1%   42.1%
======================================
  Files         129     129
  Lines       14307   14307
======================================
  Hits         6024    6024
  Misses       7383    7383
  Partials      900     900

Flag
Coverage Δ

#linux
45.61% <66.66%> (ø)
:arrow_up:

#windows
37.05% <66.66%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

remotes/docker/pusher.go
0% <0%> (ø)
:arrow_up:

remotes/docker/resolver.go
53.97% <100%> (ø)
:arrow_up:

remotes/docker/fetcher.go
50% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


suggestion: issue/pr template

Problem:

(This is one of my fav issues to suggest 😊) Currently, there are no guidelines for what information issues and pull requests should contain. This adds ambiguity for contributors on what to include when submitting issues / prs and adds more time and mental burden for maintainers when understanding why or how a contribution could be considers.

Proposed solution:

I suggest creating an action by adding yml files to the .github directory that automatically formats incoming issues and prs with desired information that maintainers would like to know and to guide contributors to adding helpful information about their issue/pr.
The action could be as robust as creating a landing page to different types of issue templates, automatically labeling issues and closing prs or as simple as sections with headings informing contributors what information to include.

Additional Resources:

Examples:

Virtual Coffee repo — https://github.com/Virtual-Coffee/virtualcoffee.io/issues/new/choose
Open Sauced docs repo — https://github.com/open-sauced/docs.opensauced.pizza/issues/new/choose
A Call for Code repo — https://github.com/Call-for-Code-for-Racial-Justice/Incident-Accuracy-Reporting-System/blob/main/.github/pull_request_template.md
Carbon Code by IBM repo — https://github.com/IBM/carbon-ui-builder/issues/new/choose

So as far as formatting, I feel like we can check formatting in CI and just fail the build if it has issues.


Pathfinding

At some point we might need path finding for units, especially when maps become more complex (they are rather flat and easy right now).
A* is a well known algorithm. But perhaps Fringe is also a good alternative.
More resources:

https://webdocs.cs.ualberta.ca/~holte/Publications/fringe.pdf
http://aigamedev.com/open/tutorials/clearance-based-pathfinding/

@arjenvanderende maybe we should do this next? It would be a great improvement. We could even do a small step before full-fledged path finding.


Dual root partition

Have an option to use a dual root partition system like ChromeOS does to make upgrades safer. The system use two cloned read-only root partitions, and when upgrading one of them, the other is leave as a backup.

status on this? Are we still intending to implement this?


Mechanism to force image to be treated as a tensor

Not all 2D tensors are images. Data like an stft looks like an image but doesn't play nicely with the image viewer.
The tensor viewer has a number of nice features like:

support for f64 data
the ability to not lock the aspect ratio
support for colormaps

We can trick the viewer into creating a tensor by adding a trivial outer dimension, but it would be nice not to need to do this:
Example:
f, t, Zxx = scipy.signal.stft(x, fs, nperseg=1000)
Zxx_image = np.abs(Zxx).astype(np.float32)
Zxx_tensor = np.expand_dims(np.abs(Zxx), axis=0)

rr.init("stft_test", spawn=True)
rr.log_tensor("stft_image", Zxx_image)
rr.log_tensor("stft_tensor", Zxx_tensor)

Related to https://github.com/rerun-io/rerun/issues/1164

rr.log("stft_tensor", rr.Tensor(Zxx_image))

now does exactly what this describes


HIVE-25989: HBaseStorageHandler CTLT failure rollback should not drop…

… the underlying HBase table
What changes were proposed in this pull request?
Introduce a CTLT flag to check if a table was created using CTLT, the same way it's done for CTAS.
Why are the changes needed?
In the case of HBase tables, if you use a CTLT command, your new table will point to the same underlying HBase table as your original table. In case of a failure, currently the HBaseMetaHook rollback logic deletes the underlying HBase table, sweeping the rug from under the old table too, leading to data loss. However, using the CTLT flag, we can stop that from doing so.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit tests

Please update versionmap.txt as thrift is getting updated.

@nareshpr Thanks Naresh for the reminder! This file does not exist upstream, but I'll make sure to update it when doing the downstream backport.

@pvary Can you please review this patch? Thanks!


Control and unicode characters not highlighted in the editor

Does this issue occur when all extensions are disabled?: No
This issue occur when extension "C# v1.25.0" is active

VS Code Version: 1.72.2
OS Version: macOS 12.6, windows 10

Steps to Reproduce:

Create dotnet project
Insert control or unicode characters into a string.
They should be highlighted in color.
Change Color Theme
After changing the theme, the highlighting appears and immediately disappears
For example:
Console.WriteLine($"\"Issue\"\nControl characters not highlighted in editor\\\n\u00A3");
Tested for macos and windows

Can you share a screenrecording?

@hediet screenrecording

What is the expected behavior? The escape sequence is not highlighed, only the character it represents!

@hediet expected behavior


Bluetooth headphones disconnect when pulsemeeter is opened

Not sure what logs to provide, but as soon as I open pulsemeeter, my bluetooth headphones are disconnected.

Open the app in a terminal and see if there are any logs, also if you have pavucontrol try opening pulsemeeter while pavucontrol is open and see if pavucontrol will restart it's interface

Apologies for the delay, here you go:
Exception in thread Thread-1 (listen_subscribe):
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1009, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.10/threading.py", line 946, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/lib/python3.10/site-packages/pulsemeeter/MainWindow.py", line 434, in listen_subscribe
    for i in self.pulse.subscribe():
  File "/usr/lib/python3.10/site-packages/pulsemeeter/Pulse.py", line 493, in subscribe
    raise subprocess.CalledProcessError(return_code, command)
subprocess.CalledProcessError: Command '['pactl', 'subscribe']' returned non-zero exit status 1.

It seems pulseaudio crashes when you open pulsemeeter, in the latest pull request I added better logs for the app, so if you could install it (only install from source, not yet on pypi) and run it with pulsemeeter loglevel-all it would help

Also, jack is apparently installed

Delete the configuration file located in ~/.config/pulsemeeter/config.json and try again (forgot to add the key in already existing config files)

Here you go, from what I understood it is trying to connect the rnnoise monitor to the a2dp sink but failing :thinking: Also it did not disconnect immediately this time, I had to re-apply the noise reduction a couple of times, and noticed that when it's off it doesn't disconnect
pmctl init sink Virtual_Input_1
pmctl init sink Virtual_Input_2
pmctl init sink Virtual_Input_3

pmctl init source Virtual_Output_B1

pmctl init source Virtual_Output_B1
pmctl init source Virtual_Output_B2

pmctl init source Virtual_Output_B1
pmctl init source Virtual_Output_B2
pmctl init source Virtual_Output_B3

pmctl rnnoise hi1_rnnoise alsa_input.pci-0000_00_1f.3.analog-stereo 95 connect 200

pmctl rnnoise  alsa_input.pci-0000_00_1f.3.analog-stereo 95 disconnect 1
pmctl disconnect .monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1
pmctl connect alsa_input.pci-0000_00_1f.3.analog-stereo bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl rnnoise  alsa_input.pci-0000_00_1f.3.analog-stereo 95 connect 1
pmctl disconnect alsa_input.pci-0000_00_1f.3.analog-stereo bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink
pmctl connect .monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

Error connecting .monitor to bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink
pmctl disconnect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl connect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl disconnect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink

pmctl disconnect hi1_rnnoise.monitor alsa_output.pci-0000_00_1f.3.analog-stereo

pmctl disconnect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl connect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl rnnoise hi1_rnnoise alsa_input.pci-0000_00_1f.3.analog-stereo 95 disconnect 1
pmctl disconnect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1
pmctl connect alsa_input.pci-0000_00_1f.3.analog-stereo bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

pmctl rnnoise hi1_rnnoise alsa_input.pci-0000_00_1f.3.analog-stereo 95 connect 1
pmctl disconnect alsa_input.pci-0000_00_1f.3.analog-stereo bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink
pmctl connect hi1_rnnoise.monitor bluez_sink.E8_3A_4D_45_FF_3E.a2dp_sink 1

Exception in thread Thread-1 (listen_subscribe):
Traceback (most recent call last):
  File "/usr/lib/python3.10/threading.py", line 1009, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.10/threading.py", line 946, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ayush/.local/lib/python3.10/site-packages/pulsemeeter/MainWindow.py", line 449, in listen_subscribe
    for i in self.pulse.subscribe():
  File "/home/ayush/.local/lib/python3.10/site-packages/pulsemeeter/Pulse.py", line 579, in subscribe
    raise subprocess.CalledProcessError(return_code, command)
subprocess.CalledProcessError: Command '['pactl', 'subscribe']' returned non-zero exit status 1.

That might be a noise-supression-for-voice issue, I've had pulseaudio crash on me a few time because of that, instead I just use the plugin that noisetorch uses (same algorithm, but never crashed on me), I've made a pkgbuild for it if you want to try it (that's only for the plugin, having noisetorch installed is not required and won't conflict but you need still to install this plugin). I'll probably add that to the aur later

Makes sense. So the pkgbuild installs the ladspa plugin into the folder, but how would it be used? Or does it override the one provided with pulsemeeter?

It should work out of the box, I already configured pulsemeeter so noisetorch's plugin takes precedence over noise-supression-for-voice

Alright, will try the noisetorch version. Let's hope it doesn't crash pulseaudio :sweat_smile: Thanks for all the help, I love using pulsemeeter!


Replace dlfcn-win32 with simple-dlfcn-win32

I'm submitting two PR's to fix #223. Pick one. This PR replaces the dlfcn-win32 code.

I'd go with this definitely, but in this case, we dont need this reside in deps. Could you integrate it with src?
Thanks a lot!

Three test cases fail with my version of dlfcn...
1) Library should accept `null` as a first argument:
   Error: Dynamic Symbol Retrieval Error: Win32 error 127
    at DynamicLibrary.get (C:\Proj\iojs\node-ffi\lib\dynamic_library.js:112:11)
    at C:\Proj\iojs\node-ffi\lib\library.js:50:19
    at Array.forEach (native)
    at new Library (C:\Proj\iojs\node-ffi\lib\library.js:47:28)
    at Context.<anonymous> (C:\Proj\iojs\node-ffi\test\library.js:24:21)

2) Library should work with "strcpy" and a 128 length string:
   Error: Dynamic Symbol Retrieval Error: Win32 error 127
    at DynamicLibrary.get (C:\Proj\iojs\node-ffi\lib\dynamic_library.js:112:11)
    at C:\Proj\iojs\node-ffi\lib\library.js:50:19
    at Array.forEach (native)
    at new Library (C:\Proj\iojs\node-ffi\lib\library.js:47:28)
    at Context.<anonymous> (C:\Proj\iojs\node-ffi\test\library.js:64:18)

3) Library should work with "strcpy" and a 2k length string:
   Error: Dynamic Symbol Retrieval Error: Win32 error 127
    at DynamicLibrary.get (C:\Proj\iojs\node-ffi\lib\dynamic_library.js:112:11)
    at C:\Proj\iojs\node-ffi\lib\library.js:50:19
    at Array.forEach (native)
    at new Library (C:\Proj\iojs\node-ffi\lib\library.js:47:28)
    at Context.<anonymous> (C:\Proj\iojs\node-ffi\test\library.js:74:18)

These test cases depend on POSIX semantics in dlsym: searching all loaded modules to find a symbol by name.  No Windows programmer would ever think to do that.
When you search for "strcpy", which strcpy() do you expect to find?  ntdll.dll is loaded into all processes, and it exports strcpy.  msvcrt.dll is loaded into most processes, but it belongs to the OS -- node.exe doesn't use it.  In fact, node.exe static links it's C runtime, and those symbols are unavailable (even the original dlfcn-win32, which tries to emulate POSIX, can't see them).  If node.exe did use a C runtime DLL, it would be something like msvcr120.dll.  You have several DLL's loaded in your process that export "strcpy".  Which one will dlsym return?  Who knows.
It probably doesn't matter which runtime you get strcpy() from.  It matters a lot for things like malloc(), free(), fopen(), and fclose().  Memory allocated by malloc() must be passed to the free() in the same C runtime as the malloc() that allocated it.  The FILE* returned by fopen() can only be passed to stdio functions in the same C runtime.  This is why Windows DLL's generally don't use stdio FILE's in their public interface, and if they have functions that return an allocated buffer, they'll provide a function to free it (of they'll allocate the memory using a Win32 API like GlobalAlloc() or HeapAlloc(), which any caller can free).

I understand your point, and agree. I think those test are not there to test strcmp, those are there to test if binding is works or not by the given situation. If you could replace them with an invocations of an strcmp wrapper method in ffi_test.node, then I'd accept it that way. That node file is just a dynamic lib, so it gotta work that way. @TooTallNate waddaya say?

P.S. My last commit included a Unicode test case similar to PR #225.

So to sum it up:

finally we can drop another dependency, only ffi remains
we have a simplifed dlopen and co. on Windows that also fixes https://github.com/node-ffi/node-ffi/issues/183
there are some updated test cases of testing the same node-ffi behavior, but wouldn't encourage anyone to use node-ffi in a way that really never works right on Windows

@TooTallNate that's what I call pull request. Could be merged ASAP. Agree?

I celebrated too early. Unfortunattely we have some failing test cases by merging this on Linux. Please take a look at the Travis log. Do you have experience to fix them? If not, I can help you with this.

I'll fire up a Linux VM and investigate the failures reported by travis.
Not sure what to make of the failures reported by AppVeyor.  Is that a problem with the patch, or a problem with the environment?

You can ignore AppVeyor's log, thats because of iojs.exe instead of node.exe issue. Travis should work with 0.10 and 0.12. With io.js it will fail because 3.x support is blocked because of ref module is not yet implemented its NAN 2.0 support.

I'm :+1: for this, but ya we'll have to make sure all the tests are in order. It could be technically considered a breaking change for Windows users who might have been depending on the previous null behavior, but I'm contemplating whether we can consider it a bug fix instead since the behavior was incorrect from the get-go.

Cool, it's finally ready. I try to make a NAN 2.0 support PR to ref module on this week, and travis is gonna be happy again.

I'll merge this once NAN 2.0 PR gets ready (after this gets landed: https://github.com/TooTallNate/ref/pull/33), and we can release those together with all tests passed on Travis.

I have this merged in the v2 branch. v0.10 seems to be failing one test. See https://ci.appveyor.com/project/TooTallNate/node-ffi/build/80. @mcnameej Any ideas there?
[00:01:36]   1) Library should accept `null` as a first argument:
[00:01:36]      Error: Dynamic Symbol Retrieval Error: Win32 error 127
[00:01:36]       at DynamicLibrary.get (C:\projects\node-ffi\lib\dynamic_library.js:112:11)
[00:01:36]       at C:\projects\node-ffi\lib\library.js:50:19
[00:01:36]       at Array.forEach (native)
[00:01:36]       at new Library (C:\projects\node-ffi\lib\library.js:47:28)
[00:01:36]       at Context.<anonymous> (C:\projects\node-ffi\test\library.js:33:22)
[00:01:36]       at callFn (C:\projects\node-ffi\node_modules\mocha\lib\runnable.js:289:21)
[00:01:36]       at Test.Runnable.run (C:\projects\node-ffi\node_modules\mocha\lib\runnable.js:282:7)
[00:01:36]       at Runner.runTest (C:\projects\node-ffi\node_modules\mocha\lib\runner.js:419:10)
[00:01:36]       at C:\projects\node-ffi\node_modules\mocha\lib\runner.js:526:12
[00:01:36]       at next (C:\projects\node-ffi\node_modules\mocha\lib\runner.js:340:14)
[00:01:36]       at C:\projects\node-ffi\node_modules\mocha\lib\runner.js:350:7
[00:01:36]       at next (C:\projects\node-ffi\node_modules\mocha\lib\runner.js:283:14)
[00:01:36]       at Object._onImmediate (C:\projects\node-ffi\nod
[00:01:36] e_modules\mocha\lib\runner.js:318:5)

@TooTallNate: The problem was that older versions of node.exe don't export a node_module_register function.  I changed the test to use uv_fs_open instead.  PR #232 contains the fix for v2 branch.

Merged. Thanks @mcnameej!


Consider using SameSite=Lax as default for SessionMiddleware

I don't want to make it very personal but I wasted about 5 hours on a bug that was due to SameSite=Strict that is tide's default. Browsers treat SameSite as Lax if it's not specified, and I think that many developers expect the cookie behavior to be Lax intuitively. For example, when after successful payment on a different domain I'm redirected to my own site I want to send a cookie with my access_token but I can't because of Strict. I'm aware it is in the docs but I somehow managed to not notice it, because I didn't expect the bug to be caused by it.

Hi @jlkiri, thanks for reporting. Ping @yoshuawuyts this is also true for using OAuth2 with tide, you need to set the same_site_policy explicit to Lax like this:
app.with(tide::sessions::SessionMiddleware::new(
        tide::sessions::MemoryStore::new(),
        std::env::var("TIDE_SECRET")
            .expect("Please provide a TIDE_SECRET value of at least 32 bytes")
            .as_bytes(),
        ).with_same_site_policy(SameSite::Lax)
    );
    ```

Thanks!

I can't remember why I picked Strict, but a bit of research makes it seem like Lax is in fact correct. The default should change


swyxkit.netlify.app articles not found

On your website swyxkit.netlify.app it appears that all the blog articles are returning 404s.

sorry that was a bad merge of #34 .. i have pinned my deploys for now while i figure out the error

i could not figure out the error - this is my debug log where i tried all the versions https://github.com/sveltejs/kit/issues/4079 but ran out of time to figure out the source of this

@PH4NTOMiki that was amazing help, thank you so much man, i was so frustrated by this error and turned out to be not our fault lol.


Shows me as friend with my self and not my friends

here also

thanks for report !
This problem is know.
We have make a fix.


integration[minor]: Llama Cpp streaming

This PR adds streaming to the Llama CPP LLM and Chat modules.

Some small questions but I think good to go

Thank you!


Stop exposing the URL polyfill in the global scope

This moves/exposes the URL polyfill similarily to the existing ReadableStream polyfill, rather than exposing it globally, to avoid interfering with any "outside" code.
Both the URL and ReadableStream polyfills are now exposed on the pdfjsLib object, such that they are accessible to the viewer components.
Furthermore, the no-restricted-globals ESLint rule is also enabled to prevent accidental usage of the native URL/ReadableStream implementations directly in the src/ and web/ folders; see also https://eslint.org/docs/rules/no-restricted-globals
Addresses the remaining TODO in https://github.com/mozilla/pdf.js/projects/6

/botio-linux preview

/botio test

Nice work! It's good to not change the global scope anymore with the URL polyfill.


Mention the new UI in readme.md and install.md

Mentioning the new UI of bench routes to make contributors aware of it.

@muskankhedia PTAL!


AWS Lambda or Firebase Cloud Functions for mobile back-end with Google OR-tools

After figuring out it's impossible to embed my scheduling algorithm (used Google OR-tools) into my Android application, I decided to put it on a server.
I am looking for a quick solution to build a server.
Available options are AWS Lambda (Amazon) and Firebase Cloud Functions (Google).My server should receive some info about today's date, task id and task's due date, then do calculations and send the result back.
I wonder what is the best choice for using with Google OR-tools?
P.S. I understand this might be a bit irrelevant, but I'm sure it will help some other developers too!

I will only comment that I have multiple reports of crashes using python or java on Azure.
I have not investigated them. I just know the programs worked well on workstations, and crashed on the cloud.
AWS, and Google, AFAIK are good options.
I will be a bit more motivated to fix issues if they happen on Google cloud :-)


Allow {doc-title} variable in layer name

It would be useful if we could include {doc-title} or something like that in the layer name to have that part of the output file/path match whatever the PSD name is.

@JPelphrey Thanks for your vote on this. Paging @nimbupani -- might you guys be thinking about anything like this for the next version of Extract?


fix 2 minor issues in current master

This PR fixes 2 minor issues I encountered while evaluting the current 'master'-branch of atom-ternjs.
(1) added additional JavaScript grammars (ES6 and JSX)
(2) enhanced error checking in 'onWorkerMessage' ('fush' operation returns: error: 'undefined')

I'm not sure about the JSX part.
Don't know if people want the activation of the plugin within JSX files.
There are a lot of React, JSX, ... and i haven't tested the package against JSX yet.
The rest was manually added via some commits in the past.
Thx!


Document when to add to CHANGELOG

If either the author or the reviewer believe the change should be mentioned in the CHANGELOG, then it should be added.

Also, explain the "Next Version" piece. It's confusing.


ERROR: clickhouse_fdw: error while reading row: unsupported type in binary protocol

Hi! After I imported the clickhouse table data into my external table in a postgresql database, I got an error when I tried to query the table: ERROR: clickhouse_fdw: error while reading row: unsupported type in binary protocol. Later I found that the problem was in two columns, latitude and longtitude. They are defined as Decimal(12,8) type in the clickhouse database. How should I solve this problem? I cannot perform any query operations on this column.

When I want to query this field, it appears ERROR: clickhouse_fdw: error while reading row: unsupported type in binary protocol

I tried changing the protocol used, changing the binary protocol to the default protocol: CREATE SERVER clickhouse_svr2 FOREIGN DATA WRAPPER clickhouse_fdw OPTIONS(dbname '', host '', port *', driver 'http'); No error occurred. But I don't know what problems this would cause.


Updated demo site design

Changed the demo preview site's design,
Added Code Snippet

Live Preview: https://prappo.github.io/vue3-icon-picker-landing-page/

Oh I loved this PR, thanks @prappo. It has some conflicting issues, will merge asap 👍


Use gopsutil instead of gosigar for FreeBSD support

This does remove the gosigar dependency to go towards FreeBSD support.
An additional bump of the go-dockerclient dependency does make it behave on FreeBSD.
Fixes #355.

Thank you @MerlinDMC!


General & Visuals

When you plant the C4 the selfchams stop working please fix and clantag changer
(Sugestion: Add chams for ragdolls improve the grenade trajetory and improve the one window menu add a legit resolver and a legit aa (i know this is retard for a free cheat but you make a really very good work))

Try and set 'Planting' chams for the local chams problem

Should be fixed now.


Replace mkdirp with native fs implementation

As this version of mkdirp is deprecated and this is available in Node
10.x I've removed the package and declared a minimum engine version in
pacakge.json.
This was added in v10.12.0 which came out in October 2018. I'm not sure if the engines section should specify the node version as 10.12+ or if this should drop node 10 entirely. Or I can revert this and use an updated mkdirp. Let me know and I'm happy to adjust.

fixed in #43


inconsistent # of significant numbers

Hi folks,
I'm using Julia in Atom.
See the example:
unique([122 122.5 10 10.3])
displays the following result:
122 123 10 10.3
so, it is limiting the display both in console and inline evaluation to three significant digits.
Any advice on that? This leads to a lot of confusion while debugging.
Any help is appreciated.
thanks,

Yeah, this can be pretty misleading.
You can click on numbers though to get them displayed in full precision.

Should at least be mitigated by the "solution" in https://github.com/JunoLab/Juno.jl/issues/27,


How should I run khcheck with hostNetwork=true ?

I wrote a khcheck instance and run it in my cluster. And I set hostNetwork=true in the "podSpec" field. I know that I can't use the checkclient pkg because the report method use the clusterIp of kuberhealthy svc. So I write a new report method just send a POST request to the report url.  But I found the pod of my khcheck can't send report to the kuberhealthy control plane. The logs are below:
I0426 09:39:10.449461     181 main.go:392] Reporting SUCCESS
I0426 09:39:10.449545     181 main.go:425] DEBUG: Making POST request to kuberhealthy report url: http://<IP_ADDRESS>:80/externalCheckStatus
E0426 09:40:10.877513     181 main.go:436] ERROR: got a bad status code from kuberhealthy:%!(EXTRA int=400, string=400 Bad Request)
E0426 09:40:10.877552     181 main.go:396] Error reporting success to Kuberhealthy servers:bad status code from kuberhealthy status reporting url: [400] 400 Bad Request
E0426 09:40:10.877563     181 main.go:164] Error running ncr check, err: bad status code from kuberhealthy status reporting url: [400] 400 Bad Request

the code of report method is:
func sendReport (newReport status.Report)error{
	b, err := json.Marshal(newReport)
	if err != nil{
		return err
	}

	// send to the server
	exponentialBackoff := backoff.NewExponentialBackOff()
	exponentialBackoff.MaxElapsedTime = time.Second * 30
	var resp *http.Response
	err = backoff.Retry(func() error {
		var err error
		klog.Infof("DEBUG: Making POST request to kuberhealthy report url: %s", reportUrl)
		resp, err = http.Post(reportUrl, "application/json", bytes.NewBuffer(b))
		return err
	}, exponentialBackoff)
	if err != nil {
		klog.Errorf("ERROR: got an error sending POST to kuberhealthy:", err.Error())
		return fmt.Errorf("bad POST request to kuberhealthy status reporting url: %w", err)
	}

	// make sure we got a 200 and consider it an error otherwise
	if resp.StatusCode != http.StatusOK {
		klog.Errorf("ERROR: got a bad status code from kuberhealthy:", resp.StatusCode, resp.Status)
		return fmt.Errorf("bad status code from kuberhealthy status reporting url: [%d] %s ", resp.StatusCode, resp.Status)
	}
	return nil
}

what should I do?

Good timing, actually!  We are just working through a pull request that will remove source IP validation in favor of a header validation process.  Check out PR #884's description.  When this PR merges, you will need to update your checkClient package with go get -u github.com/comcast/kuberhealthy/pkg/checks/external/checkclient and rebuild your check container so that the new header is used when your check sends its report.

Thank you very much about the reply. But could you give me some quick solutions to my current problem? When I set my khcheck hostNetwork=true, the pod's ip is the same with the node's ip. I don't understand why it can't be validated.


RuntimeError: Error building extension '_splat'

nice work, I encontered this erro, could you please help me to figure it out?

Hi @hongge831,
Thank you for your interest in our work.
This issue has to do with the splatting algorithm. We used GANgealing's splatting implementation.
You may find an answer here.

I found this issue could be avoided by setting use_griddata: True in the training config file.

Yes, we also added this option in the code. Just note that it is slower :)


[LOC] "修正、 Int" needs to be replaced with "Fix, Int"

https://docs.microsoft.com/ja-jp/office/vba/language/reference/user-interface-help/math-keyword-summary

Working on a fix BUG 3256790.
@Soran-M

@hyoshioka0128
Topic was human translated and the issues is fixed.

@OfficeGlobal Thanks.
Close #884


APIStar as a cli-only API tool.

This PR is working towards removing the server framework stuff from APIStar completely, in order to focus on cross-framework tooling for APIs, as discussed initially here: https://discuss.apistar.org/t/splitting-out-the-server/551/19
The initial set of CLI tools would be:

apistar validate - Validate an OpenAPI or Swagger schema.
apistar doc - Generate API documentation, given an API schema.
apistar request - Make a client request to an API.
apistar mock - Serve a mock API.

There would also be programmatic access to any of those functions, eg. Python API client that takes a schema on instantiation, and allows the user to then interact with the API.
The type system would also be a documented part of the library, along with API for generating HTML forms out of the types, and for marshalling between JSON Schema and APIStar types.
I haven't quite figured out what I want to do about the server stuff in APIStar (or not), but I'm not happy with it in it's current state, and Starlette does a far better job of pushing forward the fundamentals of an ASGI server.
I know this won't be a particularly popular direction, but I'm keen to focus APIStar on becoming a framework-agnostic tool, for generating and serving API documentation, using as an API client etc. rather than yet-another framework.

This is very rough for projects built on APIStar. Do you plan to create a migration guide from APIStar to Starlette? Does Starlette have feature parity with APIStar?

So, Starlette takes advantage of ASGI properly, whereas for APIStar it's a bit bolted on. As a result there are a number of things that Starlette does (or soon will do) that aren't so obvious how to nicely build into APIStar, in particular:

Websockets.
Server Sent Events. (soon)
In-process background tasks. (soon)
HTTP/2 and server push. (soon)

It also does a much better job of clear, consistent design.
What it doesn't do, that APIStar does is:

Dependency Injection for handler arguments.
API documentation.

I've not been convinced that APIStar's dependency injection is necessarily that great a solution to anything much, but even if I was, I'd rather see the projects properly scoped.
Here's some different paths we could go down:

Do nothing. The existing apistar versions won't disappear so you just stay pinned to them.
Have a fork of the project, under a different name. The only concrete changes here are that someone else would be doing the maintainance other than me, and that projects would need to change the import name to update.
Forget about the DI layer, but make sure that there is suitable documentation and support for using apistar's type system and API docs together with starlette.
I start a new project that combines apistar's types and API docs, with starlette for the base framework, and just adds in the DI layer and API docs. (ie. a continuation of apistar's server, but with a clear set of boundaries between the different projects.)

@tomchristie I just noticed this after making https://github.com/encode/apistar/pull/625. Is there any chance of that landing and maybe being released as a final patch to the 0.5.x series? It's a fairly critical bug, imo. I can live with installing from my fork, but would prefer to install from PyPI (and not have to come up with a new namespace).
Re. options for current API Star users, I'd like to (shamelessly) propose my own framework, molten, as a sort of spiritual successor for those of us who don't need/want asyncio. Many of the concepts are similar (since APIStar was a huge inspiration!) so porting apps over should not be too difficult.
/end-shameless-plug

@Bogdanp Yup we can definitely do that. And great work on molten!


Not able to use a function reference as the job exit callback

Not able to use a function reference to set the exit_cb of a job. Other job
callback functions can be set using function references.
Use get_callback() instead of get_tv_string_buf_chk() to set the job exit_cb.

Coverage decreased (-0.03%) to 62.252% when pulling 333b542ae02994451c29bed292ca6650c271fb86 on yegappan:job-exit-cb into 324a78f3b649e7b14741519ecf19c4aba178772d on vim:master.

I am wondering what’s the point of enabling @coveralls messages? Given that Neovim has coverage CI build without these messages (and AFAIR less recent PR’s here used to be built without them but with coverage as well) I assume this is not required.

The Coveralls messages get sent to the mailing list, which is annoying.

Yegappan Lakshmanan wrote:

Not able to use a function reference to set the exit_cb of a job. Other job
callback functions can be set using function references.
Use get_callback() instead of get_tv_string_buf_chk() to set the job exit_cb.

Thanks.  I'll also remove the now unused struct field and unreference
the partial in free_job_options().
--
If Pacman had affected us as kids we'd be running around in dark rooms,
munching pills and listening to repetitive music.
-- Marcus Brigstocke
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

On Sa, 28 Mai 2016, Nikolai Aleksandrovich Pavlov wrote:

I am wondering what’s the point of enabling @coveralls messages? Given
that Neovim has coverage CI build without these messages (and AFAIR
less recent PR’s here used to be built without them but with coverage
as well) I assume this is not required.

I also wondered about those recent notifications. I think, I have just
disabled them. I am not sure, what caused those notifications to be
sent, I guess they were enabled by accident.
Best,
Christian
Wer zu viel verlangt, wer sich am Verwickelten erfreut, der ist
den Verirrungen ausgesetzt.
-- Goethe, Maximen und Reflektionen, Nr. 81

Changes incorporated into Vim in patch 7.4.1859.


CLOUDP-164587: Upgrade Atlas CLI to go1.20

Proposed changes
Jira ticket: CLOUDP-164587
Further comments
Update go and enable double error wrapping where possible

Should we update the go version in the GitHub workflows?

no need, they should be picking up from the go.mod file
https://github.com/mongodb/mongodb-atlas-cli/blob/7ecd228a7a049e058f19860f8bbf5da727d83504/.github/workflows/code-health.yml#L39


Cant generate Certs anymore

Restarted both Container and Host in hopes but nothing
Another instance of Certbot is already running.
Ask for help or search for solutions at https://community.letsencrypt.org. See the logfile /tmp/tmpjmp2ayq7/log or re-run Certbot with -v for more details.

    at ChildProcess.exithandler (child_process.js:308:12)
    at ChildProcess.emit (events.js:314:20)
    at maybeClose (internal/child_process.js:1022:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:287:5)

Is the contents of the Log file
2021-11-10 19:02:15,079:DEBUG:certbot._internal.main:Location of certbot entry point: /usr/bin/certbot
2021-11-10 19:02:15,079:DEBUG:certbot._internal.main:Arguments: ['--config', '/etc/letsencrypt.ini', '--cert-name', 'npm-39', '--agree-tos', '--authenticator', 'webroot', '--email', '', '--preferred-challenges', 'dns,http', '--domains', 'subdomain.mrcow.xyz']
2021-11-10 19:02:15,079:DEBUG:certbot._internal.main:Discovered plugins: PluginsRegistry(PluginEntryPoint#manual,PluginEntryPoint#null,PluginEntryPoint#standalone,PluginEntryPoint#webroot)
2021-11-10 19:02:15,090:DEBUG:certbot._internal.lock:A lock on /var/log/letsencrypt/.certbot.lock is held by another process.
2021-11-10 19:02:15,090:DEBUG:certbot._internal.log:Exiting abnormally:
Traceback (most recent call last):
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 126, in _try_lock
    fcntl.lockf(fd, fcntl.LOCK_EX | fcntl.LOCK_NB)
BlockingIOError: [Errno 11] Resource temporarily unavailable

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/certbot", line 8, in <module>
    sys.exit(main())
  File "/usr/lib/python3.8/site-packages/certbot/main.py", line 15, in main
    return internal_main.main(cli_args)
  File "/usr/lib/python3.8/site-packages/certbot/_internal/main.py", line 1558, in main
    log.post_arg_parse_setup(config)
  File "/usr/lib/python3.8/site-packages/certbot/_internal/log.py", line 100, in post_arg_parse_setup
    file_handler, file_path = setup_log_file_handler(
  File "/usr/lib/python3.8/site-packages/certbot/_internal/log.py", line 153, in setup_log_file_handler
    util.set_up_core_dir(config.logs_dir, 0o700, config.strict_permissions)
  File "/usr/lib/python3.8/site-packages/certbot/util.py", line 178, in set_up_core_dir
    lock_dir_until_exit(directory)
  File "/usr/lib/python3.8/site-packages/certbot/util.py", line 152, in lock_dir_until_exit
    _LOCKS[dir_path] = lock.lock_dir(dir_path)
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 258, in lock_dir
    return LockFile(os.path.join(dir_path, '.certbot.lock'))
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 45, in __init__
    self.acquire()
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 60, in acquire
    self._lock_mechanism.acquire()
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 112, in acquire
    self._try_lock(fd)
  File "/usr/lib/python3.8/site-packages/certbot/_internal/lock.py", line 130, in _try_lock
    raise errors.LockError('Another instance of Certbot is already running.')
certbot.errors.LockError: Another instance of Certbot is already running.
2021-11-10 19:02:15,090:ERROR:certbot._internal.log:Another instance of Certbot is already running.```

EDIT: I tried running the certbot command in the container itself using docker exec, it didnt work. Afterwards I changed the Default Redirect page to 404, still nothing via the Web GUI.
However now the certbot cmd works atleast, but the cert doesnt show up in the Web GUI

having the same problem here while renewing Certs
any workaround on this one..?

Same here

same here.
were you guys able to solve this?

Same Problem

Same here

Ok ! I found a solution.
I use docker (Docker version 23.0.1, build a5ee5b1)
I went to the terminal of my container, I installed procps :
apt install update && apt upgrade -y && apt install procps
I get certbot process with : ps -eaf | grep "certbot"
I killed all the processes and I restart the generation of the certficat from the web interface of NPM

The only process I can find is the grep process


CAN YOU PLEASE ADD SUPPORT WITH SIMHUB?

It works with simhub, I just used Dirt4 profile for it.

It works with simhub, I just used Dirt4 profile for it.

yes, this.
The default packet format in output ui should work with the dirt4 profile in simhub.


Allow users to choose if plots appear as pop up windows or in the dedicated pane

Problem Description
I just installed a fresh copy of spyder 4.0.0 (clean install of anaconda) and was stunned to discover that having plots appear as separate pop up windows was no longer supported, but that all plots are forced to be inline or in the dedicated "plots pane."
This is a major step backwards for people who routinely need multiple plots to understand complex data sets, and to iterate quickly through changes in their code. Creating multiple pop up windows and placing them strategically on a separate monitor before testing changes to your pipeline is a huge benefit. I have no desire to interact with all plots in a single pane, even if that pane is pop-out-able.
If this is already an option, then it needs to be displayed in a more prominent/user friendly place because I could not find the option after searching for it for over half an hour.
What steps reproduce the problem?
Upgrade to Spyder 4.0.0
What is the expected output?
Users have the option of using a dedicated pane for all plots OR an option of using separate pop up windows for each new plot.
Versions

Spyder version: 4.0.0
Python version: Python 3.6.9 64-bit
Qt version: Qt 5.9.6
PyQt version: PyQt5 5.9.2
Operating System name/version: Windows 10

Dependencies
cloudpickle >=0.5.0          :  1.2.2 (OK)
pygments >=2.0               :  2.5.2 (OK)
qtconsole >=4.6.0            :  4.6.0 (OK)
nbconvert >=4.0              :  5.6.1 (OK)
sphinx >=0.6.6               :  2.2.2 (OK)
pylint >=0.25                :  2.4.4 (OK)
psutil >=0.3                 :  5.6.7 (OK)
qtawesome >=0.5.7            :  0.6.0 (OK)
qtpy >=1.5.0                 :  1.9.0 (OK)
pickleshare >=0.4            :  0.7.5 (OK)
zmq >=17                     :  18.1.0 (OK)
chardet >=2.0.0              :  3.0.4 (OK)
numpydoc >=0.6.0             :  0.9.1 (OK)
spyder_kernels >=1.8.1;<2.0.0:  1.8.1 (OK)
qdarkstyle >=2.7             :  2.7 (OK)
atomicwrites >=1.2.0         :  1.3.0 (OK)
diff_match_patch >=20181111  :  20181111 (OK)
intervaltree                 :  None (OK)
watchdog                     :  None (OK)
keyring                      :  None (OK)
pexpect >=4.4.0              :  4.7.0 (OK)
pympler                      :  None (OK)
sympy >=0.7.3                :  1.4 (OK)
cython >=0.21                :  0.29.14 (OK)
IPython >=4.0                :  7.10.1 (OK)
matplotlib >=2.0.0           :  3.1.1 (OK)
pandas >=0.13.1              :  0.25.3 (OK)
numpy >=1.7                  :  1.17.4 (OK)
scipy >=0.17.0               :  1.3.2 (OK)
pyls >=0.31.2;<0.32.0        :  0.31.2 (OK)
rtree >=0.8.3                :  0.8.3 (OK)

I just installed a fresh copy of spyder 4.0.0 (clean install of anaconda) and was stunned to discover that having plots appear as separate pop up windows was no longer supported, but that all plots are forced to be inline or in the dedicated "plots pane."

This is not true. To have plots in separate windows, you just need to change your graphics backend in the menu
Tools > Preferences > IPython console > Graphics > Graphics backend

and change the Backend entry from inline to Automatic.

If this is already an option, then it needs to be displayed in a more prominent/user friendly place because I could not find the option after searching for it for over half an hour.

This has been the way to do this for a very long time (5 years or more). So we don't think it's necessary to promote it more.

Thank you for your quick response. I love spyder and appreciate the work you do on it.
Before creating this issue, I had tried switching from inline to automatic (as I do every time I install a new version of spyder), but it did not change anything. I restarted spyder and the plots continued to display in the pane.
After your comment, I closed spyder, re-launched it yet again, and the plots continued to display in the pane. I then restarted the ipython kernel, and then plots popped out in a separate window.
So somehow closing and restarting spyder did not affect the changes, but restarting the kernel did...
I don't think it's worth re opening for this, but it was a minor quirk.
Thanks again for the response.

So somehow closing and restarting spyder did not affect the changes, but restarting the kernel did...

Sorry, that's a usability issue that leads to terrible confusions like the one you just reported here. We'll try to fix it for Spyder 5.


Add lite profile scope and get email method

I Added support to the new Sign In with LinkedIn API. It only supports the new r_liteprofile scope and requires a separate call in order to fetch the user's email.
I also updated the authorization and token URLs to OAuth V2.

please update the readme then we can merge in

Sorry for the delay. I just updated the readme.


Archive this repo?

Hey Matt, curious if this repo can be archived? I was looking into an issue and was getting confused between this one and the internal one in Popmotion.

Ah just saw it's mentioned in the README 🤦♂


[New LinOp] Implementation of a SubSampling operator.

I have tested this operator and its adjoint with Numpy and Dask arrays, but not with cupy arrays as I don't have a GPU.
I have not been able to squash the 17 commits into one, as they are spread within the long history of other commits made on v2-dev. Please squash them when merging the PR.

Thank you for this addition. The code as-is is functional but lacking in a few directions:

no test suite
does not allow sub-sampling N-dimensional inputs
is stated as module-specific, while this can be relaxed
doesn't override methods such as gram()/svdvals()/etc from LinOp. (Large performance gains possible)

I will nevertheless accept the PR. Follow-up commits on my behalf expand this implementation as stated above and will also be pushed shortly.


Tauri App 2.0 adapter

RT

done. https://github.com/persiliao/axios-tauri-api-adapter/commit/db443a6d9cf8e473f62fd526fdc7b0c965ca2b61


expose UMA box topper boosters

https://mtg.fandom.com/wiki/Ultimate_Masters/Ultimate_Box_Topper
there is no entry for tcg or mkm, but these are needed for defining the booster box

fixed in https://github.com/taw/magic-search-engine/pull/261

done in #35


Update v3_main with latest main branch code

This change updates v3_main with the latest 2.x changes. This is in preparation for restructuring the tree to be more flexible and better meet the goals of the v3 product.

/azp run


request help: how to get client real-ip

Issue description
要增加IP白名单限制，发现APISIX获取远程客户端的地址为本地网络设备CNI0的地址。

This is actually a Kubernetes knowledge.
You can set externalTrafficPolicy for apisix-gateway service.

This is actually a Kubernetes knowledge. You can set externalTrafficPolicy for apisix-gateway service.

谢谢大佬，修改externalTrafficPolicy为Local，解决了。


cannot disconnect from a compute instance in VS Code once connected

I successfully connected to a compute instance in VS Code.
However, I cannot disconnect from the compute instance and switch to other machines.
I tried the command "Azure ML: Kill VS Code Server on compute instance" and it didn't work. VS Code keep trying reconnecting to the remote server and output the error "Failed to forward incoming connection to remote server: Unexpected server response: 406" repeatedly.
If I stopped the compute instance, the next time I started VS Code, it tries connecting to the remote forever.

@wangzheng0310 Closing this issue as duplicate of #73948.


Class mailer does not exist

Hello,
Thanks for making such an awesome module first off.
I'm having an issue on a clients site, when I add the DevDojo\Chatter\ChatterServiceProvider::class, in config/app I get the Class mailer does not exist.
When I comment out the DevDojo\Chatter\ChatterServiceProvider::class, in config/app the problem is gone but then ofcourse I can't use the module!.
I don't know if the Mail class is conflicting with something cause in other projects I have managed to get Chatter working ok.
All the best.

May be you missed import statement. Or you can manually check if mailer don't exist or not.
and I hope you use laravel 5.6