Update qiskit/circuit/init.py (typo corrected)

Summary
Corrected the typo while I was going through the docs to solve some other issue.
link to section where type is - link
Details and comments
In the 3rd paragraph, "you cannot use a gate in a definition it its own definition depends on the parent." should be replaced by "you cannot use a gate in a definition if its own definition depends on the parent."

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

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

0 of 0   changed or added relevant lines in 0 files are covered.
26 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.02%) to 88.775%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/two_qubit_decompose.rs
1
91.45%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
2
88.26%

crates/qasm2/src/lex.rs
5
92.73%

qiskit/synthesis/two_qubit/xx_decompose/decomposer.py
6
90.91%

crates/qasm2/src/parse.rs
12
97.15%

Totals

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

Covered Lines:
73474

Relevant Lines:
82764

💛  - Coveralls


FLINK-1402 - Remove Serializable extends from InputFormat interface

FLINK-1402
Remove Serializable extends from InputFormat interface since the InputSplitSource interface already extend the Serializable

I think this is fine to merge.
FYI: I sometimes repeat the Serializable interface in the list of superinterfaces (even when redundant) as a form of documentation. It makes it clear upon reading the class signature that it is expected to be serializable. So anyone extending the class is easier aware of that.

Ah ok, thanks for the info Stephen, good to know it was intentional.
Do you want to keep this pattern?

I also find it quite helpful to have Serialised repeated in such cases.

I don't remember if there any best practice about this, so If we think it is useful we could keep this style and maybe document it?
But I don't think it is good practice for other interfaces.

I will close this PR due to it is on purpose


Bootstrap 3.3.5 Upgrade

We're stuck on 3.3.4 because 3.3.5 broke stuff.

Done in e92dd0151ae66d1369601a1a97175454609cd0e8


[Bug]: Browse Annotations "On" filter doesn't work for Notes

Steps to reproduce the bug
Go to browse annotations, click the "Notes" radio button, select an object type in the "On" menu, then click submit.

The results will include notes for all object types, not only the one you selected.

Screenshot

Expected behavior
No response
Additional Screenshots
No response
Environment
Sandbox, Development (native)
Sandbox Used
No response
Version
bf06c3da6
Browser Used
No response

@jlpereira on[] needs to be changed to note_object_type[]


How to resize images to 256*256

Thank you for your awesome work. I noticed that the model only accepts input sizes of 256 * 256, but the image I am currently testing is 448 * 448, so I would like to know the method of resizing the image and whether using nearest neighbor interpolation is appropriate.

def train_transforms(img_size, ori_h, ori_w):
transforms = []
if ori_h < img_size and ori_w < img_size:
transforms.append(A.PadIfNeeded(min_height=img_size, min_width=img_size, border_mode=cv2.BORDER_CONSTANT, value=(0, 0, 0)))
else:
transforms.append(A.Resize(int(img_size), int(img_size), interpolation=cv2.INTER_NEAREST))
transforms.append(ToTensorV2(p=1.0))
return A.Compose(transforms, p=1.)
We use the above function to resize the images. SAM-Med2D currently only accepts image sizes of 256x256, so it is necessary to standardize the image sizes to 256x256 before testing. After prediction, the results will be interpolated back to the original image sizes.

Thank you！I got it !


Can't get my map centralized

I'm rendering a map from Brazil and it worked fine, but i don't know how to centralize it with Svg container
https://d.pr/i/Ar3nfM
can anyone help with this?

Hi @jorgefernandes,
Check out this South America example on codesandbox:
https://codesandbox.io/s/basic-markers-3fn8g
Is that going in the direction of what you are doing?


[5.7][Runtime] Fix short-circuiting of shared cache conformance lookups.

Cherry-pick https://github.com/apple/swift/pull/59542 to release/5.7.
We were skipping shared cache queries if the ObjC metadata or descriptor were outside the shared cache, but we need to skip only if they're both outside the shared cache. We also need to check foreign types even if the descriptor is outside the shared cache.
rdar://93931813

@swift-ci please test


feature: Standardize all WhichKey shortcut names to have their first character capitalized.

Did you check the docs?

[X] I have read all the LazyVim docs

Is your feature request related to a problem? Please describe.
Currently, the capitalization of WhichKey shortcut names is inconsistent, making them look less visually appealing.

Describe the solution you'd like
It would look better if they were all standardized with the first character capitalized.
Additionally, adding some icons in front of folder name would enhance their visual appeal, like this.

Describe alternatives you've considered
I know I can map it myself. However, I think it would be better if the styles were standardized from the start. This would help somebody avoid a lot of unnecessary configuration.
Additional context
If you don't mind, I can create a pr to help address this.

which key groups are lower-case. keymaps are capitalized. It IS consistent

good point! Will change it


Fixed an error with SYNCHRONIZED block parsing.

This is an old error that seems to exist since the initial commit.
Line 106 of parseradapter.vim assumes the dictionary entry is named 'block'.

Seems like not only since initial commit, but much earlier, in original javacomplete plugin.
Thanks.


StreamChannelMixin error since 2.1.3

Since v2.1.3, i have this error
The class 'StreamChannelMixin' can't be used as a mixin because it's neither a mixin class nor a mixin.dart[class_used_as_mixin](https://dart.dev/diagnostics/class_used_as_mixin)
abstract class StreamChannelMixin<T> implements StreamChannel<T>
package:stream_channel/stream_channel.dart

A mixin that implements the instance methods of [StreamChannel] in terms of [stream] and [sink].

i don't see any updates about it in the changelog

@kevmoo 🙏🏼

my code

class MessagePortStreamChannel
    with StreamChannelMixin<String>
    implements StreamChannel<String> {
  MessagePortStreamChannel({required this.port}) {
    port.onmessage = (message) {
      _in.add(message.data! as String);
    }.toJS;

    _onPostMessageSubscription = _out.stream.listen((event) {
      port.postMessage(event as JSAny?);
    });
  }

  final MessagePort port;
  final _in = StreamController<String>(sync: true);
  final _out = StreamController<String>(sync: true);

  late final StreamSubscription<String> _onPostMessageSubscription;

  Future<void> dispose() async {
    await _onPostMessageSubscription.cancel();
    await _in.close();
    await _out.close();
  }

  @override
  StreamSink<String> get sink => _out.sink;

  @override
  Stream<String> get stream => _in.stream;
}

class WebBrowserExtensionStreamChannel
    with StreamChannelMixin<String>
    implements StreamChannel<String> {
  WebBrowserExtensionStreamChannel({required this.streamChannel}) {
    _onPostMessageSubscription = _out.stream.listen((event) async {
      _logger.info('[WBE] send command $event');
      await streamChannel.send(event as JSString).toDart;
      _logger.info('[WBE] send command Done');
    });
  }

  static final _logger = Logger('AWC-StreamChannel-WebBrowserExtention');

  Future<void> connect() async {
    final connectionCompleter = Completer<void>();

    streamChannel.onReady = () {
      _setupStreamChannel();
      connectionCompleter.complete();
    }.toJS;

    streamChannel.onClose = (String _) {
      connectionCompleter.completeError(Failure.connectivity);
    }.toJS;

    await streamChannel.connect().toDart;

    return connectionCompleter.future;
  }

  void _setupStreamChannel() {
    streamChannel.onReceive = (String message) {
      _logger.info('[WBE] command received $message');
      _in.add(message);
      _logger.info('[WBE] command received Done');
    }.toJS;

    streamChannel.onClose = (String reason) {
      unawaited(_onPostMessageSubscription.cancel());
      unawaited(_in.close());
      unawaited(_out.close());
    }.toJS;
  }

  final AWCStreamChannelJS streamChannel;
  final _in = StreamController<String>(sync: true);
  final _out = StreamController<String>(sync: true);

  late final StreamSubscription<String> _onPostMessageSubscription;

  Future<void> dispose() async {
    await streamChannel.close().toDart;
  }

  @override
  StreamSink<String> get sink => _out.sink;

  @override
  Stream<String> get stream => _in.stream;
}

I think this has to do w/ the SDK changing the model for mixins. What's your min Dart SDK?

Er, what Dart SDK are you using and what's the SDK constraint in your pubspec?

hello @kevmoo
Thank you for the reply
I would have imagined a major version change on the lib instead of a minor one with this breaking changes
[✓] Flutter (Channel stable, 3.24.5, on macOS 14.6.1 23G93 darwin-arm64, locale fr-FR)
    • Flutter version 3.24.5 on channel stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (6 weeks ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

@redDwarf03 – what's the SDK constraint in your pubspec?

name: archethic_wallet_client
description: A client dart library to interact with Archethic Wallet RPC API.

version: 2.1.8

homepage: https://github.com/archethic-foundation/archethic-wallet-client-dart

environment:
  sdk: ">=3.5.3 <4.0.0"

dependencies:
  flutter:
    sdk: flutter

  # An abstraction for two-way communication channels based on the Dart Stream class.
  stream_channel: ^2.1.2


Feature Flag for WAF Rate Limiting

In order to allow for DB stuffing we need to be able to turn off an on the WAF Rate Limiting in staging.

Not needed anymore with current approach.

Not needed anymore with current approach.


Plugins reverting to older version after updating Insiders

VSCode Version:

Version 1.8.0-insider
Commit 7499a9f3c565f4fbe5205e30c8664e5a5a11ae94
Date 2016-12-06T16:09:29.346Z
Shell 1.4.6
Renderer 53.0.2785.143
Node 6.5.0

OS Version:

Ubuntu 14.04
Steps to Reproduce:

Install IDEA Keybindings. Update the version of vscode insiders
You would see that the version of the extension vert back to 0.1.1 where as the latest version is 0.2.6

When you installed IDEA Keybindings in step one, which version was installed?
Now that you have 0.1.1, does Code warn you that there is an available update? If you update the extension, don't you get 0.2.6?

When you installed IDEA Keybindings in step one, which version was installed?

I had 0.2.5

Now that you have 0.1.1, does Code warn you that there is an available update?

No. The editor did not

If you update the extension, don't you get 0.2.6?

No.. I did not get any notification that the extension update is available, even if I check explicitly using "Check for updates" option.
I got older version continuously.
I had to manually download and install the latest version of the extension from thier github page and install.

Did you install 0.1.1 from their github page too?

No..

Moved to k--kato/vscode-intellij-idea-keybindings#51


Fix typo

@tgxworld this wants you to sign the CLA, but I think it's because you don't have your @shopify email associated with your GitHub account?

but I think it's because you don't have your @shopify email associated with your GitHub account?

Aha I had a +github in my Github email which I didn't include when signing the CLA


PWA is not working

Cannot install app on Chrome for Android and front-end code is not updating between refreshes.

Fixed in #119.


Core: Add "deadline per retry" parameter to Retry object

OS: Linux dc32b7e8763a 4.9.0-6-amd64 #1 SMP Debian 4.9.82-1+deb9u3 (2018-03-02) x86_64 x86_64 x86_64 GNU/Linux
Python version: Python 2.7.6
google-cloud: 0.34.0
It would be great if, instead of specifying a deadline and retrying a query until you hit the deadline, you could specify a deadline per retry and a number of retries. For instance, I'd like to retry a query up to 3 times with a deadline of up to 20 minutes per query.
The current workaround in the above example would be to specify a single deadline of an hour. This works fine for now, but if our queries start taking longer than 20 minutes, we'd want to know so that we can refactor the queries or investigate why it's taking so long. So it would be helpful if we could raise a RetryError if each individual retry takes longer than a certain amount of time.

You can subclass Retry and provide your own strategy, but we won't be changing our overall deadline pattern as part of the core library.


Snippets - Decorators: Verify a Google authentication token

In snippets -> decorators -> Verify a Google authentication token
I would change:
                       headers={'Host': 'www.googleapis.com',
                                'Authorization': access_token})

to:
                       headers={'Host': 'www.googleapis.com',
                               'Authorization': 'Bearer '+access_token
                       })

You can try in https://developers.google.com/oauthplayground/

Please file against https://github.com/pocoo/flask-snippets

sorry, I can't find this snippet there

We haven't actually migrated to the repo yet, but I think issues don't get as lost there.

Just open an issue on the repo for now.

Ok, thanks you, I moved it there.


docs: add changelog for 4.2.3

What type of PR is this?
docs

What this PR does / why we need it:
add changelog
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:

lgtm


Connected fonts and colors

Imported fonts from Google Fonts api, created CSS color vars and basic color classes

fancy colors


PiShrink and Virtual Box 5.2.22

I know this is a specific issue but PiShrink seems to be often used with VirtualBox.
Would it be possible to write a tutorial how to get to run PiShrink in a shared folder with the latest Virtual Box version?
There seem to be problems since about one year. I do not know enough about all this stuff to say that only Oracle can solve the problem.
I know that this is a specific problem, but PiShrink seems to be commonly used with VirtualBox.
Is it possible to get a tutorial on how to run PiShrink in a shared folder with the latest Virtual Box version?
There seem to be problems for about a year. It would be great if you could handle the mistakes of Oracle (GuestAddition...?!, loopX Input/Output fail).
https://www.virtualbox.org/ticket/17360?cversion=0&cnum_hist=1#comment:8

The problem is with VirtualBox. I can try to add a tutorial in the readme but VirtualBox is just bad and it makes me sad


Change default_returns_func to use Quandl API

I updated default_returns_func to use Quandl, created get_symbol_returns_from_quandl, and deprecated get_symbol_returns_from_yahoo.

Thanks @jaycode!
CC @richafrank for assigning a reviewer.

@jaycode sorry for the long delay, but we're finally getting back to this.

Hello @jaycode, this is a high-priority issue for us, so I'll be work on my own version of the PR, if that's alright with you. Thanks so much!


Is the documentation up to date?

Where does the documentation in this repo stand compared to the documentation that's live? For instance, I noticed the documentation for behaviors doesn't match the live site:
https://github.com/jagi/meteor-astronomy-docs/blob/master/docs/behaviors.md
http://jagi.github.io/meteor-astronomy/#behaviors
I have a couple changes I'd like to make to the documentation.

This repo is not used anymore.
Documentation is up to date. It automatically publishes when any change is made to the gh-pages branch in the meteor-astronomy repository.


Add kappnav component to the Kabanero must-gather script.

Now that the kabanero-foundation is managing a kappnav deployment, we should add kappnav component must-gather to the kabanero-foundation must-gather scripts.

@cvignola @rlcundiff Do you have any suggestions as to what we should gather here?  By default we're going to gather pod logs for anything in the kappnav namespace, and any object yamls that are created by the installation (roles, service accounts, deployment, etc).

That sounds good and I don't have anything else to suggest.  As we gain experience with the issues users encounter we may find other things to add in future releases.

This was shipped in kabanero-io/kabanero-foundation#90


Cherry pick sharded pubsub commit into 4.4

Pull Request check-list
Please make sure to review and check all of these items:

[ ] Does $ tox pass with this change (including linting)?
[ ] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
[ ] Is there an example added to the examples folder (if applicable)?
[ ] Was the change added to CHANGES file?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
Please provide a description of the change here.

Codecov Report
Patch coverage: 22.81% and project coverage change: -1.88% :warning:

Comparison is base (e85e3f7) 60.59% compared to head (240ee07) 58.71%.

Additional details and impacted files
@@            Coverage Diff             @@
##              4.4      #10      +/-   ##
==========================================
- Coverage   60.59%   58.71%   -1.88%
==========================================
  Files         116      116
  Lines       29922    29314     -608
==========================================
- Hits        18130    17213     -917
- Misses      11792    12101     +309

Files Changed
Coverage Δ

tests/test_asyncio/test_pubsub.py
18.84% <ø> (-2.15%)
:arrow_down:

redis/cluster.py
85.25% <15.71%> (-5.60%)
:arrow_down:

tests/test_pubsub.py
54.79% <20.60%> (-18.02%)
:arrow_down:

redis/client.py
67.80% <37.20%> (-1.83%)
:arrow_down:

redis/commands/core.py
78.10% <50.00%> (-0.11%)
:arrow_down:

redis/commands/parser.py
63.33% <100.00%> (ø)

... and 58 files with indirect coverage changes

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

@zach-iee That's right. It's just a cherry-pick & resolve merge conflict commit. But I don't know about push_handler_func 😲


Fix return value of ActiveThemeProvider::getThemeHandle()

I don't know why it may happen, but I've noticed this error:
Return value of
Concrete\Core\Area\Layout\Preset\Provider\ActiveThemeProvider::getThemeHandle()
must be of the type string, null returned

Let's fix it.

I don't know why it may happen

I inspected it: the problem occurs when installing a package from within the dashboard (in my tests I installed via the CLI).
When installing via the UI, the current page is a dashboard page, which has the Dashboard theme (and its controller getThemeHandle method returns NULL )


How to change the plus, move and pinned icons?

as title

In your grid options, add this code:
(This uses font awesome icons, you can use whatever you like)
Official page:
https://www.ag-grid.com/javascript-grid-icons/index.php
icons: {
    menu: '<i class="fa fa-bars"/>',
    columnVisible: '<i class="fa fa-eye"/>',
    columnHidden: '<i class="fa fa-eye-slash"/>',
    columnRemoveFromGroup: '<i class="fa fa-remove"/>',
    filter: '<i class="fa fa-filter"/>',
    sortAscending: '<i class="fa fa-sort-alpha-asc"/>',
    sortDescending: '<i class="fa fa-sort-alpha-desc"/>',
    groupExpanded: '<i class="fa fa-minus-square-o"/>',
    groupContracted: '<i class="fa fa-plus-square-o"/>',
    columnGroupOpened: '<i class="fa fa-minus-square-o"/>',
    columnGroupClosed: '<i class="fa fa-plus-square-o"/>'
},

@AmitMY
I know that, but it can't set move & pin icon

Oh, sorry, thought they existed....
After further check, there are more icons that are not changeable:

Group icon (in the group row / menu)
Tool panel shown (eye)
Too panel hidden (minus)
Menu settings (3 lines)
Menu filter (funnel)
Menu settings & Menu icon for 'tool panel' are different. one is 'check' and one is 'grid', also unchangeable

i've just stuck this in. will be in the next release. new overrides will be:
    columnMovePin // when column is to the left, before it gets pinned
    columnMoveAdd // when adding a column
    columnMoveHide // when removing a column
    columnMoveMove // when moving a column
    columnMoveLeft // when moving and scrolling left
    columnMoveRight // when moving and scrolling right
    columnMoveGroup // when about to drop into group panel

@ceolter  thx a lot!!, it will be very helpful for customization 👍


If the starting point is invalid then edges_around_point causes overflow

If the starting point is an invalid index when edges_around_point is called, then an overflow occurs. I am not sure why the point was marked as invalid, but this check at least allowed everything to run to completion.

Thanks for the contribution, merged!


pass index hints for query using server-side aggregation

here's an example in accumulo for where index hints are not used: https://github.com/ngageoint/geowave/blob/master/extensions/datastores/accumulo/src/main/java/mil/nga/giat/geowave/datastore/accumulo/query/AggregationIterator.java#L252

duplicate


[PLACEHOLDER] Handle scenarios when budget exceeded

Context

Scenario 1: running out of funds when "Renew amount monthly" is not selected.

Scenario 2: running out of funds when "Renew amount monthly" is selected.

Initial thinking: if the budget is recurring, ask user to make a one-time/ad-hoc amount authorization that...this ties the user over with budget until the monthly renewal.

Solution TBD
(on agenda for 20 Jun WM community call)

for recurring amounts, save all information about the grant but the user should be able to re-connect with the same wallet specifying a one time amount
for non-recurring amounts, notify the user and disconnect the wallet (?)

To dos
TBD

Notes:

https://interledgerfoundation.slack.com/archives/C054B631B97/p1718799633709149

Blocked by #377  and the OP depedency (@tselit to update with outlook for designs after chatting to Madalina)


Extension opens files but it shouldn't

The source folder I'm trying to add contains .bmp and .hlp files and as the extension tries to add these files it opens each and every one.  In my case there are a couple hundred .bmp files and each one opens in a new Paint window.

This is how the add files to solution folders internal functions work, strange I know. But I can have a look at if the code can be change somehow.


Use a session without a cookie jar

The replicate-python client uses a request session that has a default cookie jar. We should explicitly use a session without those!

Fixed by #102


SGLang Performance Compared with LMDeploy does not seem to have a significant improvement

SGLang Performance Compared with LMDeploy
Profiling Verify
Llama3.1-8B/Llama3-8B on 1 x A100 (bf16)

vllm means: vllm-0.5.2
sglang means: 0.2.6
lmdeploy means: 0.5.2.post1

Offline Test(same with blog post  https://lmsys.org/blog/2024-07-25-sglang-llama3/)
Starting with the small model Llama-8B, the figure below shows the maximum output throughput each engine can achieve in offline settings across six different datasets. Both LMDeploy and SGLang can achieve an excellent throughput of up to 5000 tokens per second on a dataset with short inputs, while vLLM lags behind.
Output Throughput(High is Better)

Mean Latency(Low is Better)

Median Latency(Low is Better)

Median TTFT(Low is Better)

Median ITL(Low is Better)

Online Test(same with blog post  https://lmsys.org/blog/2024-07-25-sglang-llama3/)
The online benchmark figure below shows a trend similar to the offline case. TensorRT-LLM and SGLang perform equally well and can sustain an RPS > 10, while the latency of vLLM increases significantly at a high request rate.
Output Throughput(High is Better)

Mean Latency(Low is Better)

Median Latency(Low is Better)

Median TTFT(Low is Better)

Median ITL(Low is Better)

Conclusion

In the current test case, SGLang's performance is comparable to LMDeploy's, and it is indeed much faster than vLLM, and comparable to LMDeploy's performance. Aligning SGLang's scheduling can make vLLM's performance close to LMDeploy's (further analysis and optimization are underway)
During vLLM's operation, GPU Utilization is too low, fluctuating from 0 to 60%. SGLang also has a GPU Utilization fluctuation of 20 to 80%, while the equivalent LMDeploy is basically close to 100%. From the perspective of GPU Utilization, SGLang should exceed LMDeploy to a certain extent. Do you have plans to continue to optimize and surpass LMDeploy in the future?

Reference
[1] https://lmsys.org/blog/2024-07-25-sglang-llama3/

Hi @cassiewilliam Thank you for providing this information. In fact, I am a committer for both LMDeploy and SGLang, and I have also reserved a benchmark interface for LMDeploy in bench_serving. https://github.com/sgl-project/sglang/blob/1edd4e07d6ad52f4f63e7f6beaa5987c1e1cf621/python/sglang/bench_serving.py#L247
Regarding the performance results of running Llama3 8B on a single A100 with LMDeploy, I was not surprised at all because I am very familiar with LMDeploy too. We did not include LMDeploy in the comparison in our blog post at that time because it currently does not support fp8, nor has it been specially optimized for H100 and TP. In subsequent blog releases, we will consider including it in the comparison, especially once it supports fp8. If you're interested, you can also benchmark in scenarios like A100 TP8, H100 TP8, and perhaps you'll make some discoveries.
Regarding the conclusion you mentioned, aligning vLLM with SGLang's scheduling might enhance vLLM's performance. The vLLM Team has been working overtime to reference the scheduling approach in SGLang since we published our blog, such as moving zmq-related implementations to vLLM, etc. As for how much the performance will improve after all is done, I am also very much looking forward to it. We welcome competition very much. I don't currently feel very optimistic about vLLM, the concise and efficient implementation of SGLang is still a huge advantage compared to vLLM.
GPU Util is sampled over time, showing 100% does not mean it is fully utilized, I think you should understand this basic situation. I have used both LMDeploy and TensorRT LLM, their GPU Util is very high, which is indeed well done. The low GPU Util of vLLM is due to its scheduling being implemented very inefficiently, with huge CPU scheduling overhead, which will be more evident on TP8 and H100.
SGLang has its own planning and roadmap, we will continue to advance at our own pace, maintaining a first-class level in terms of functionality and performance. SGLang will ensure both the ease of use and scalability of vLLM, as well as the performance of TensorRT LLM.

Additionally, I want to mention that the metrics we look at vary in different scenarios. For example, in offline scenarios, we pay more attention to throughput because the goal of offline is to increase throughput. In online scenarios, latency is very important, so we look at end-to-end latency. The metric for e2e latency here refers to the classic Orca paper and means TTFT + (token_num - 1) * ITL. https://github.com/sgl-project/sglang/blob/1edd4e07d6ad52f4f63e7f6beaa5987c1e1cf621/python/sglang/bench_serving.py#L178 When either TTFT or ITL cannot hold up, e2e latency will dramatically increase. I hope this background information helps you understand these metrics better. Thanks.

Additionally, I want to mention that the metrics we look at vary in different scenarios. For example, in offline scenarios, we pay more attention to throughput because the goal of offline is to increase throughput. In online scenarios, latency is very important, so we look at end-to-end latency. The metric for e2e latency here refers to the classic Orca paper and means TTFT + (token_num - 1) * ITL.
https://github.com/sgl-project/sglang/blob/1edd4e07d6ad52f4f63e7f6beaa5987c1e1cf621/python/sglang/bench_serving.py#L178
When either TTFT or ITL cannot hold up, e2e latency will dramatically increase. I hope this background information helps you understand these metrics better. Thanks.

Thank you for your replay，SGLang is a very nice job。

Thank you for your attention and support for SGLang. If you have plans to switch to the inference stack in the future, as now LLM inference frameworks all support OpenAI-compatible servers, switching has become very easy and smooth. Feel free to contact us at any time, we are more than willing to provide technical support and help. By the way, my mentee who was previously at Meituan recently moved to Shopee, maybe you are colleagues LOL. This issue is closed for now, feel free to reopen it if needed in the future. Thank you.

Hello @zhyncs may I know what you attribute the perf of sglang over vllm to? Is it primarily due to better kernels for decode like from flash infer?

Efficient scheduling and a substantial amount of engineering optimization.

Additionally, I want to mention that the metrics we look at vary in different scenarios. For example, in offline scenarios, we pay more attention to throughput because the goal of offline is to increase throughput. In online scenarios, latency is very important, so we look at end-to-end latency. The metric for e2e latency here refers to the classic Orca paper and means TTFT + (token_num - 1) * ITL.
https://github.com/sgl-project/sglang/blob/1edd4e07d6ad52f4f63e7f6beaa5987c1e1cf621/python/sglang/bench_serving.py#L178
When either TTFT or ITL cannot hold up, e2e latency will dramatically increase. I hope this background information helps you understand these metrics better. Thanks.

Shouldn't E2E latency mean TTFT + (token_num - 1) * TPOT?


fix: install button works with pyenv

Fixes #487

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

v0.48.2
GitHub release

Your semantic-release bot :package::rocket:


better error handling

for example, I tried adding a ref to a functional component and there were no errors, things just didn't work.

Ah this stinks. I'll get to this ASAP. Thanks for filing!


Vscode slowly on hackintosh

VSCode Version: 1.21.0
OS Version: hackintosh 10.13.3，
Computer Configuration：
i5-8600k
8g ddr4 *2
gtx1060
sm961 NVMe
My hard disk is normal，CLion webstorm IDEA is very fast on it，but vscode is a slowly

Steps to Reproduce:
1.open python file in vscode ， slow
2.Write code，slow

Does this issue occur when all extensions are disabled?: Yes

To use python in VS Code you must have an extension installed.
Disable all extensions by hitting CTRL+SHIFT+P and select "disable all installed extensions". Then restart VS Code.
Please report back if that improves the performance.


qsort implementation to minimal stdlib

added musl implementation of qsort
http://git.musl-libc.org/cgit/musl/tree/src/stdlib/qsort.c
related to this issue i've opened
https://github.com/zephyrproject-rtos/zephyr/issues/28896
Signed-off-by: Jonathan Bruchim<EMAIL_ADDRESS>

I have some reservations about just copying from another project, although I haven't looked at the implementation.

qsort does not actually specify which sorting algorithm should be used under the hood, which leaves a lot of possibilities open for the implementation.
if we add a sorting algorithm to the minimal libc, let's add one that has decent worst-case performance.
if we add a sorting algorithm to the minimal libc, let's add one that has all of the other random features that we could want, such as

in place sorting (no external memory required)
stability (i.e. the input order of elements is preserved in the output)

A decent reference for some of these features are here:
https://en.wikipedia.org/wiki/Sorting_algorithm#Comparison_of_algorithms
https://en.wikipedia.org/wiki/Qsort
I mistakenly opened #31107 for this same topic since I've been carrying qsort.c in my greybus module for a while (from the BSD C library). However, due to the aforementioned concerns, I never bothered to make a PR.
@JonBruchim - would you be open to considering implementing a different sorting algorithm? I would be happy to review and offer some feedback. Really, it's about testing.
I would be happy to make a suggestion about which sorting algorithm to use, although I've not personally implemented it.
https://en.wikipedia.org/wiki/Block_sort

O(N) best case
O(NlogN) worst / average case
Stable
In-place

I have some reservations about just copying from another project, although I haven't looked at the implementation.

qsort does not actually specify which sorting algorithm should be used under the hood, which leaves a lot of possibilities open for the implementation.
if we add a sorting algorithm to the minimal libc, let's add one that has decent worst-case performance.
if we add a sorting algorithm to the minimal libc, let's add one that has all of the other random features that we could want, such as

in place sorting (no external memory required)
stability (i.e. the input order of elements is preserved in the output)

A decent reference for some of these features are here:
https://en.wikipedia.org/wiki/Sorting_algorithm#Comparison_of_algorithms
https://en.wikipedia.org/wiki/Qsort
I mistakenly opened #31107 for this same topic since I've been carrying qsort.c in my greybus module for a while (from the BSD C library). However, due to the aforementioned concerns, I never bothered to make a PR.
@JonBruchim - would you be open to considering implementing a different sorting algorithm? I would be happy to review and offer some feedback. Really, it's about testing.
I would be happy to make a suggestion about which sorting algorithm to use, although I've not personally implemented it.
https://en.wikipedia.org/wiki/Block_sort

O(N) best case
O(NlogN) worst / average case
Stable
In-place


Fix documentation of matchaddpos()

The documentation of matchaddpos() is confusing:

The documentation says that the function is:
matchaddpos({group}, {pos} [, {priority} [, {id} [, {dict}]]])

A better description would be:
matchaddpos({group}, {list} [, {priority} [, {id} [, {dict}]]])

where {list} is a list of positions.

The description of {pos} is wrong. The documentation says: "The list {pos} can contain one of these items: A number ... A list with one number ... A list with two numbers ...". Unfortunately, this is contradicted by the example given in the documentation. It is wrong because the list can contain zero to eight items (not just one).

Missing word somewhere here:
https://github.com/vim/vim/blob/9faec4e3d439968e21ad74e917aebb289df8f849/runtime/doc/eval.txt#L7474-L7476
There's probably a missing word somewhere in this: " ... with the value a list like the {pos} item."

Proposed fix:
diff --git a/runtime/doc/eval.txt b/runtime/doc/eval.txt
index 03fcb655d..574edf707 100644
--- a/runtime/doc/eval.txt
+++ b/runtime/doc/eval.txt
@@ -2708,7 +2708,7 @@ match({expr}, {pat} [, {start} [, {count}]])
                                Number  position where {pat} matches in {expr}
 matchadd({group}, {pattern} [, {priority} [, {id} [, {dict}]]])
                                Number  highlight {pattern} with {group}
-matchaddpos({group}, {pos} [, {priority} [, {id} [, {dict}]]])
+matchaddpos({group}, {list} [, {priority} [, {id} [, {dict}]]])
                                Number  highlight positions with {group}
 matcharg({nr})                 List    arguments of |:match|
 matchdelete({id} [, {win}])    Number  delete match identified by {id}
@@ -7442,15 +7442,15 @@ matchadd({group}, {pattern} [, {priority} [, {id} [, {dict}]]])
                        GetGroup()->matchadd('TODO')
 <
                                                        *matchaddpos()*
-matchaddpos({group}, {pos} [, {priority} [, {id} [, {dict}]]])
-               Same as |matchadd()|, but requires a list of positions {pos}
+matchaddpos({group}, {list} [, {priority} [, {id} [, {dict}]]])
+               Same as |matchadd()|, but requires a list of positions
                instead of a pattern. This command is faster than |matchadd()|
                because it does not require to handle regular expressions and
                sets buffer line boundaries to redraw screen. It is supposed
                to be used when fast match additions and deletions are
                required, for example to highlight matching parentheses.

-               The list {pos} can contain one of these items:
+               {list} should be a list of positions, where a position is:
                - A number.  This whole line will be highlighted.  The first
                  line has number 1.
                - A list with one number, e.g., [23]. The whole line with this
@@ -7463,7 +7463,7 @@ matchaddpos({group}, {pos} [, {priority} [, {id} [, {dict}]]])
                - A list with three numbers, e.g., [23, 11, 3]. As above, but
                  the third number gives the length of the highlight in bytes.

-               The maximum number of positions is 8.
+               The maximum number of positions in {list} is 8.

                Example: >
                        :highlight MyGroup ctermbg=green guibg=green
@@ -7473,7 +7473,7 @@ matchaddpos({group}, {pos} [, {priority} [, {id} [, {dict}]]])

 <              Matches added by |matchaddpos()| are returned by
                |getmatches()| with an entry "pos1", "pos2", etc., with the
-               value a list like the {pos} item.
+               value being a position (like an item in {list}).

                Can also be used as a |method|: >
                        GetGroup()->matchaddpos([23, 11])

Vim version: 8.1.2269

The description is unclear but it's not completely wrong.  The list {pos} is a list of up to 8 items where each item is either:

a number
a list of one number
a list of two numbers
a list of three number

Thanks, I'll include the changes and tweak it a bit more.

Fixed in c8cdf0f80b3cfd88a4490d80572062c1fd1a96ca


Emit CUDA kernels concurrently/prior to LLVM graph emission

Currently the gpu_emitter emits code that is compiled at runtime by LLVM. This compiled code then generates CUDA kernels which are compiled at RT and then run. Thus, the compiled graph execution includes runtime compilation of CUDA kernels.
As far as I can tell, there isn't anything that constrains the functionality to proceed in this way. For performance optimization purposes and benchmarking, I believe it would be better to have the gpu_emitter handle the cuda kernel runtime compilation first, prior to LLVM compilation, so that when the LLVM compiled function is executed, the computations that occur are primarily based on executing the graph and not generating CUDA.
I have checked with some of the MKLDNN folks and they originally also used our current method but eventually switched to having the MKLDNN generated prior/concurrently to the LLVM compilation. This is achievable by passing a context variable from the generating code to the LLVM compiled function (m_compiled_function) which contains pointers and other information that the emitted code can use to make the appropriate CUDA kernel calls. This is how it is currently performed on the CPU transformer side, see for example CPURuntimeContext.

It's not abandoned for inference.

This raises quite a few questions. It'd be good to discuss how we plan to address this. We also want negligible compilation overhead for inference in addition to execution latency. And the current IA-side codegen definitely does not allow for this "standalone" use-case at any rate. If we end up leveraging external kernels (ex. Halide-based) for most (if not all) performance-sensitive ops, then compiling code on-the-fly adds zero value in addition to not being "standalone".

There is always some trade off, even for inference. Some might require to start faster, some require to run faster, and other might want some balanced load. All lead to different approach.

Compile time emission of cudnn and ngraph compilation of cuda kernels (prior to clang JIT) is now possible as of PR #837. Please open a new issue for further discussion of standalone binary support.


Disallow is always first

It seems like I am unable to change the order of Allow/Disallow in the robots file.. this is a problem when you only want to allow certain pages and disallow the rest as the order matters.

Hi @tsolrm
You can pass array https://github.com/nuxt-community/robots-module#array
So you group and sort as you see fit.

Example:
['@nuxtjs/robots', [
  { UserAgent: '*' },
  { Allow: ['/*?*page=', '/*.js', '/*.css'] },
  { Disallow: ['/somePage', '/*?*'] }
]]


Support for Redis SUB/PUB

Hi,
Firstly let me commend your effort in creating such a highly quality library.
I do have a request.
Is there by any chance Redis Sub/Pub is coming soon or is it available already because i can't see it in any documentation.
Thanks

NewLife.Redis/PubSub.cs at master · NewLifeX/NewLife.Redis
https://github.com/NewLifeX/NewLife.Redis/blob/master/NewLife.Redis/PubSub.cs
NewLife.Redis/PubSubTests.cs at master · NewLifeX/NewLife.Redis
https://github.com/NewLifeX/NewLife.Redis/blob/master/XUnitTest/PubSubTests.cs


AAD bypass login screen

Question
We are use Azure Active Directory to authenticate via ExternalProviders and this all sits behind an Azure Application proxy.  The proxy enforces authentication and currently redirects to the login screen of the website.  Once the user clicks on the sign in button it will re-authenticate and allow them into the site.
My question is as they are already authenticated to the proxy can I bypass clicking on the button in the login screen?
Thanks

As a workaround, you can write javascript to trigger button (external login) click when it is loaded or in a time out.

Important update
This organization is not maintained anymore besides critical security bugfixes (if feasible). This organization will be archived when .NET Core 3.1 end of support is reached (3rd Dec 2022). All new development is happening in the new Duende Software organization.
The new Duende IdentityServer comes with a commercial license but is free for dev/testing/personal projects and companies or individuals making less than 1M USD gross annnual revenue. Please get in touch with us if you have any question.


Add custom search with filter example

#192
Add section to README with example for updating index facets and running filtered search against updated index. Tested with movies example locally.

added sleep in to both code sections in src/lib.rs to make sure filterable attributes actually get set


undefined is not an object (evaluating navigation.state)

Bug
Using standalone method from docs. Having three issues:

undefined is not an object (evaluation navigation.state)

The sample calls for using AnimatedTabBarView, but the imports only list AnimatedTabBar. Which one should I use?

I am getting a type error on index within the component

Thanks!
Environment info

Library
Version

@gorhom/animated-tabbar
2.1.2

react-native
0.63.4

react-native-reanimated
1

react-native-gesture-handler
1.10.2

react-native-svg
12.1.1

Steps To Reproduce

import AnimatedTabBarView, {TabsConfig, BubbleTabBarItemConfig} from '@gorhom/animated-tabbar'

const [index, setIndex] = useState(0)

const tabs: TabsConfig<BubbleTabBarItemConfig> = {
    Home: {
      labelStyle: {
        color: "#5B37B7",
      },
      icon: {
        component: (
          <View />
        ),
        activeColor: "rgba(91,55,183,1)",
        inactiveColor: "rgba(0,0,0,1)",
      },
      background: {
        activeColor: "rgba(223,215,243,1)",
        inactiveColor: "rgba(223,215,243,0)",
      },
    },
    Profile: {
      labelStyle: {
        color: "#1194AA",
      },
      icon: {
        component: (
          <View />
        ),
        activeColor: "rgba(17,148,170,1)",
        inactiveColor: "rgba(0,0,0,1)",
      },
      background: {
        activeColor: "rgba(207,235,239,1)",
        inactiveColor: "rgba(207,235,239,0)",
      },
    },
  }

<AnimatedTabBarView
            tabs={tabs}
            itemOuterSpace={{
              horizontal: 6,
              vertical: 12,
            }}
            itemInnerSpace={12}
            iconSize={20}
            style={{ borderRadius: 25}}
            index={index}
            onIndexChange={setIndex}
        />

Fixed this. @gorhom  - we just need to fix the import to
import {
  AnimatedTabBarView,
  TabsConfig,
  BubbleTabBarItemConfig,
} from "@gorhom/animated-tabbar"


Exported custom resources aren't initialised at runtime

Tested versions

Reproduced in v4.2.1.stable.official [b09f793f5]

System information
Mac OS 14.4.1, Apple M1 Chip
Issue description
My goal is to define a custom resource (block.gd), and have this as an exported field by a Block Manager. I've defined a custom class name, and am using BlockManager as a Tool.
While I'm able to set it as expected in the editor, when run either in editor or in game, the variable shows as null in code. We are checking in the _ready function, where one would expect all exported variables to already be set.
Steps to reproduce
Run the sample project as is, and note in the console that we are outputting that the block is null, when indeed it has been set.
Minimal reproduction project (MRP)
BugRecreation.zip

Within the MRP the BlockManager.gd is added as a TextureManager autoload and such instantiated script has block with its default value which is null. The in-scene BlockManager has the block correctly assigned.

The scene tree when running the MRP:

After changing BlockManager.gd to:
@tool
extends Node

@export var block: Block

func _ready() -> void:
	prints("self=%-35s path=%-25s block=%s" % [self, get_path(), block])

the output is like:
self=TextureManager:<Node#23488103616>   path=/root/TextureManager      block=<null>
self=BlockManager:<Node#23739761861>     path=/root/Level/BlockManager  block=<Resource#-9223372013215677240>

No bug in here, closing.


How do i get the id_token to send to my backend? i am only getting the access_token

+1 this is beneficial. Google uses idToken (not access_token) for verification: https://developers.google.com/identity/sign-in/web/backend-auth
useGoogleLogin gives only access_token

For anyone who migrates to this library and wants to use it to get information about the user like their name and profile photo, you have to use the userinfo endpoint with the access token to get that info. Here's an axios request which will get all that info:
    const {data: {given_name, family_name, picture, email, email_verified}} = await axios.get("https://openidconnect.googleapis.com/v1/userinfo", {
      params: {
        access_token: access_token,
      }
    })

where access_token is a field on the value provided to the onSuccess callback.

Please check the playground https://react-oauth.vercel.app/
you will find different ways how to get user info the way you will use the library

@paymog Thank you.


VSphere machine gets wrong ip

Rancher Version:
1.0.1
Docker Version:
1.10.3
OS and where are the hosts located? (cloud, bare metal, etc):
Datancenter
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
HA Rancher
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Swarm
When we request new VSpheres we get wrong ip on em. Currently we are getting the loadbalancers ip. If we knew how you got the ip we would be able to debug and see why we get this ip instead of the machines ip.
Any help would be appreciated.

I noticed this as well. I logged into a boot2docker vsphere machine and ran the container with -e CATTLE_AGENT_IP $ip_address and that worked.

Ye that is what we are doing too but would be nice to be able to request them without workarounds.

If the agent is not given an explicit IP (CATTLE_AGENT_IP) then it is auto-registered using the address the request to the server comes in on.


⚠ Bofrost.de has degraded performance

In de2037f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 963 ms

Resolved: Bofrost.de performance has improved in 213a243 after 5 minutes.


[android] issue #116 repositioning songs in playlist - new branch

same as pull request #136 but from a new branch and with double list modification inside a critical section

Please send a fresh new Bitcoin address to<EMAIL_ADDRESS>to send you the bounty.


Drivers: Sander applies SHAKE in single-point energies

As far as I can tell, there's no way to use sander to get single-point energies from a system with bond constraints and not have a constraint algorithm applied. This is disappointing as single-point energies aren't "running" simulations, whether they're energy minimizations or dynamics runs, so the initial geometry should be respected. As far as I can tell, this is not consistent with the behavior of other engines.
This makes it tricky to test with mainline force fields (i.e. openff-2.0.0.offxml and not openff_unconstrained-2.0.0.offxml) since the input geometries are not respected.
Below is a tarball of a reproducing case, the gist of it being this (look at the bond & angle values)
$ diff run.in hbonds.in
1c1
< Run a single-point energy
---
> recomputing energies with Cartesian restraints
3,4c3,4
< imin=1,
< maxcyc=0,
---
> imin=0,
> nstlim=0,
6,7c6,7
< ntc=1,
< ntf=1,
---
> ntc=2,
> ntf=2,
$ bash compare_ntc.sh
ntc=ntf=1:

   NSTEP       ENERGY          RMS            GMAX         NAME    NUMBER
      1       1.5103E+01     6.7226E+00     1.4354E+01     C2          2

 BOND    =        0.5312  ANGLE   =       17.4385  DIHED      =        3.7980
 VDWAALS =       -0.0968  EEL     =      -14.4530  HBOND      =        0.0000
 1-4 VDW =        0.0331  1-4 EEL =        7.8521  RESTRAINT  =        0.0000
ntc=ntf=2:

   NSTEP       ENERGY          RMS            GMAX         NAME    NUMBER
      1       1.5105E+01     7.2745E+00     1.3939E+01     C2          2

 BOND    =        0.5292  ANGLE   =       17.4395  DIHED      =        3.7986
 VDWAALS =       -0.0968  EEL     =      -14.4533  HBOND      =        0.0000
 1-4 VDW =        0.0344  1-4 EEL =        7.8531  RESTRAINT  =        0.0000

ntc_repro.tar.gz

I ran into this again today working on #414. This script, taken from a failing tests, passes when OpenEye toolkits are available but fails when they aren't. Going along with the assumption that pre-applied SHAKE constraints are responsible, my guess is that Omega conformers happen to be closer to what Sage with constraints thinks it should be, and the AmberTools+RDKit conformer was a little bit off. But I’m not eager to spend the effort required to verify that. The same .inpcrd and .prmtop files are written out in each case, so I'm just ditching the constrained case in the test matrix.


Add a label to PR once the e2e passes

Proposal
In order to be able to tell which PR is ready for final review/merge, it would be nice if a label (eg e2e-passed) could be added to the PR.
If the there are additional commits/pushes to the PR, the label should be automatically removed.
Use-Case
No response
Is this a feature you are interested in implementing yourself?
No
Anything else?
No response

Isn't it enough just the CI check?

Maybe 🤔
I thinking how to do a review properly, once I trigger e2e test on a couple of PRs, I want to get back to these once they finish.

Currently we can see if e2e has passed (or being skipped) here:

Yeah I know, maybe this is more about me, knowing when a new e2e test execution finished, so I can review the result. Sometimes it is hard to keep track. I should probably solve this on my side, the label is an overkill.

@JorTurFer what would actually help is to get some kind of notification once the e2e run finishes. Now it just updates a comment which is not something that generates notifications.

Actually, the notification is sent on every comment

We could try to notify somehow to a slack channel. Could it help? If yes, we have to request the slack channel in kubernetes workspace

Actually, the notification is sent on every comment

It is not sent on comments updates, or am I mistaken? I am interested in getting a notification once e2e test run finishes.
Slack notification is probably an overkill.

sample

Actually, the notification is sent on every comment

It is not sent on comments updates, or am I mistaken? I am interested in getting a notification once e2e test run finishes.
Slack notification is probably an overkill.

I can see value in a comment post or so indeed

Modified the proposal to create a new comment, instead of just a label.

I think we should use the same for starting e2e tests instead of updating the message with test run link given that does not trigger GitHub notification, but it might be just me.


Imbalanced load on spike traffic applications

Bug description
Hi,
I've been trying to debug why one instance of our application is getting significantly more load than the other during periods of burst load.  The setup is very simple:

You can see from this graph that there was a big spike in traffic (normal):
sum(irate(envoy_cluster_upstream_rq_total{app="searchone-consumer", cluster_name=~"outbound.*search-one.svc.cluster.local"}[1m])) by (pod_name)

But the vast majority of them seemed to go to a single instance of search-one:
sum(envoy_cluster_upstream_cx_active{app="search-one", cluster_name=~"inbound.*search-one.svc.cluster.local"}) by (pod_name, cluster_name)

This results in a huge amount more load on that instance (cpu graph):

I wondered it was some locality aware routing thing but we'd see that under normal load too, also you can see here the nodes don't align with those graphs:
❯ k get pods -o wide
NAME                          READY   STATUS    RESTARTS   AGE   IP             NODE                                                  NOMINATED NODE   READINESS GATES
search-one-7c54ff8bb4-6dfnj   2/2     Running   0          14h   <IP_ADDRESS>   gke-delivery-platform-normal-20191114-de0849f6-l6vx   <none>           <none>
search-one-7c54ff8bb4-rhc88   2/2     Running   0          14h   <IP_ADDRESS>   gke-delivery-platform-normal-20191114-4e7329f7-plhb   <none>           <none>
search-one-7c54ff8bb4-s9kgd   2/2     Running   0          14h   <IP_ADDRESS>     gke-delivery-platform-normal-20191114-de0849f6-mrl2   <none>           <none>

searchone git/master
❯ k -n searchone-consumer get pods -o wide
NAME                   READY   STATUS    RESTARTS   AGE   IP            NODE                                                  NOMINATED NODE   READINESS GATES
searchone-consumer-0   2/2     Running   0          38h   <IP_ADDRESS>    gke-delivery-platform-normal-20191114-4e7329f7-nfv6   <none>           <none>
searchone-consumer-1   2/2     Running   0          17h   <IP_ADDRESS>   gke-delivery-platform-normal-20191114-6b042e4d-wpq2   <none>           <none>

🤷♂
Expected behavior
Balanced load
Steps to reproduce the bug
Unknown
Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
1.4.2
How was Istio installed?
Helm
Environment where bug was observed (cloud vendor, OS, etc)
GKE

I grabbed all of the /stats/prometheus for searchone-consumer-0 and searchone-consumer-1 whilst it was currently imbalanced to see if there is anything you can spot
istio.tar.gz
Things that stick out to me:
envoy_cluster_lb_healthy_panic{cluster_name="outbound|80||app.search-one.svc.cluster.local"} 17135 in one but envoy_cluster_lb_healthy_panic{cluster_name="outbound|80||app.search-one.svc.cluster.local"} 0 in the other

Is this caused by long-running connections?

I considered that, and restarted them (and in doing so removed outlierDetection as a test) and saw the same behaviour, so no.

@Stono Is it possible cpu load high is the cause of high active cx, not the consequence?
cx_active is not a good metric to reason about the load balancer unless it is load-based LB
It's possible that the cx_active is because high response(due to app) but equal number of incoming request(due to RR or random LB).

Hey @lambdai - i'm not sure i follow.  I can't understand how 1 instance of 3 would (consistently) be more than double the others.  They're all processing the same incoming requests (just distributed for load)


ref: use Sprintf as helper to accept type any for the key of WithValue

📑 Description
Relate to https://github.com/goravel/framework/pull/628

@coderabbitai summary

✅ Checks

[x] Added test cases for my code

This PR needs to be merged into the v1.2.x branch, and the change of goravel/framework has been merged, please run go get github.com/goravel/framework@6543aeadf1bc39901cb02df96020a1f7aabbaa03, goravel/fiber is the same.

all right, let me checkout from v1.2.x branch then and do PR again


Support KEY_VALUE schema in the python client

Is your enhancement request related to a problem? Please describe.
It's not possible to serialize/deserialize Schema.KeyValue in the python client. In particular the debezium source will return a bytes array.
A quick workaround is to do
key_size = int.from_bytes(msg[:4], 'big')
event_key = json.loads(msg[4: key_size + 4])
value_size = int.from_bytes(msg[key_size + 4: key_size + 8], 'big')
event_value = json.loads(msg[key_size + 8: value_size + key_size + 8])

where event_key and event_value are the json key and value of the message.
Describe the solution you'd like
It should be possible to define the serialization by importing a KeyValue schema from the schemas module.

The issue had no activity for 30 days, mark with Stale label.

Closed as stale. The development of the Python client is permanently moved to https://github.com/apache/pulsar-client-python.


Focal area isn't visible in CMS after editing image

Issue Summary
When going back to edit an image, the previously saved focal area is invisible.
Steps to Reproduce

Upload an image to the Wagtail CMS
Edit the image and set a focal area
Save the image
Click on the image to edit it again
The focal area that I just saved isn't visible

Any other relevant information. For example, why do you consider this a bug and what did you expect to happen instead?
I consider this a bug because if multiple people are working in a CMS it becomes very unclear whether an image has had its focal area set or not.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: (yes / no)

Yes
Technical details

Python version: 3.7
Django version: 2.2.16
Wagtail version: 2.10.2
Browser version: Firefox

Hi all,
Shameless bump of this issue since there has been no activity since I filed it. Please let me know if this is a duplicate.

Hi @kunambi,
I'm unable to replicate this on a fresh Wagtail installation with the given Wagtail / Django / Python versions - on revisiting the Edit view, the focal point is displayed with a green box as expected.

Hi @kunambi,
I'm unable to replicate this on a fresh Wagtail installation with the given Wagtail / Django / Python versions - on revisiting the Edit view, the focal point is displayed with a green box as expected.


Difference between Microsoft Defender for Cloud Apps and Office 365 Cloud App Security unclear.

Dear,
The page doesn’t specify how you can access / if there is a different portal to access “Office 365 Cloud App Security”.
Other documentations surrounding “Office 365 Cloud App Security” also doesn’t specify if there is a different portal compared to “Microsoft Defender for Cloud Apps”.
Could it be possible when references are made to “Office 365 Cloud App Security” that it is referring to the “Office 365” connected app in the “Microsoft Defender for Cloud Apps” portal? (portal.cloudappsecurity.com => Investigate => Connected apps)
Also a message in the portal is reporting that "Office 365 Cloud App Security" is not enabled, but when you look it the screen shot below the message it displays the “Microsoft Defender for Cloud Apps" portal.

The question in short: Is the ‘Office 365” connected app in “Microsoft Defender for Cloud Apps” the same as “Office 365 Cloud App Security” or is there a different portal?
Kind regards,
Ernesto

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

ID: 76a66643-56f5-dc00-5af6-3ee1d9b6f090
Version Independent ID: c319c4a4-974e-7e3a-d58a-76fd57a9efe9
Content: Differences between Defender for Cloud Apps and Office 365 Cloud App Security
Content Source: CloudAppSecurityDocs/editions-cloud-app-security-o365.md
Service: defender-for-cloud-apps
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for your question. We'll investigate and get back to you.

@ErnieBot - Thanks again for your question. After checking with the product group, I can tell you that Office 365 Cloud App Security is accessed through the same portal as Microsoft Defender for Cloud Apps. In fact, Office 365 Cloud App Security has access to all of the features of Microsoft Defender for Cloud Apps, but supports only the Office 365 app connector.
We've updated the documentation to indicate this.


fix(docs): add rename sample.env

I'm not sure this addition is any better than what is in there already. In fact, it's misleading; you mention renaming the file sample so docker-compose will read it, which is not true

I followed the readme and the docker-compose red the sample.env instead of my .env. After renaming it it worked so that's why I posted this PR but I'm not a docker expert. If you think it's misleading we shouldn't implement my pr.


Atlas texture displays whole index when read by a shader

Godot version
4.0 beta 10
System information
Windows 10 RTX 2060
Issue description
When read by a StandardMaterial3D or ShaderMaterial, AtlasTextures will display the whole atlas instead of their render regions. This happens in 3D and 2D.
Steps to reproduce

Create Sprite3D
set the main texture to icon.svg or placeholder texture
set override material the StandardMaterial and albedo_texture to an AtlasTexture

Minimal reproduction project
AtlasTextureBugMRP.zip

Duplicate of https://github.com/godotengine/godot/issues/25895.

I am aware of #25895, but the reproduction steps don't work in 3.x, which is what that issue refers to, so there's something more going on here


bazel: Force python 2 for host tools

https://github.com/bazelbuild/bazel/issues/8626
Signed-off-by: Keith Smiley<EMAIL_ADDRESS>Risk Level: Low

I'm not 100% sure this will work for us, but I want to see what CI thinks

Do we know where we are not Python 3 compliant? Can we make sure we have an issue to track fixing that? IIRC I thought we could potentially also somehow just force this on specific broken tests which would make it easier to track. cc @lizan

@mattklein123 https://github.com/envoyproxy/envoy/pull/7305 was trying to fix it in particular tests but didn't work.
@keith I'm going to merge this to see if bazel ci works. Can you create an issue for tracking?

https://github.com/envoyproxy/envoy/issues/7342


HELP

Hey so , i am actually a beginner with this library and i am getting overwhelmed by looking at soooo many functions and i need some quick help . Here's what i need :
I want to send a JSON which will be containing 2 values lets say gps values :
Lat:xxxxxx
long:xxxxx
Now i want to send it every 3 seconds , now how exactly i use your library so that under a perticular node the value gets updated every 3 seconds . Please guide me which functions to use and which examples to put my main focus upon .

See the doc at the first page (readme).

You need to play with Firebase at console.
Test with Firebase.set or Firebase.setXXX. This will create or overwrite whole data  at node or path you specified.
Test with Firebase.get or Firebase.getXXX. This will read the data from node or path you specified.
Test with example code of JSON on the bottom of this library's first github page. This will show you how to create, edit or modified the JSON object.
Test with Firebase.updateNode. This required JSON object, the data at specified path o node will be updated (just patch to the existing data or create new data if not exist).
Each test, you need to look at Firebase RTDB console on your project website. Look for how data change and its behavior.
Finally, test with stream (monitoring the changes) at the specified path or node. See the steam examples.


Enhancement doctests attempt 2

Closes #3 with extra code that should be in its own package at some point

Coverage decreased (-2.83%) to 94.37% when pulling 74d415619e07ec346c406c63253f4a363e637af9 on enhancement-doctests into 2fc5c0e85f12a4ac72db8ab21e24238e4a4a2305 on master.


ExpansionTile opened - initial state

Hi guys,
I am wondering why the ExpansionTile does not have an option to be opened at the initial state.
Is there any reason for that?
Would you accept a PR if I change the ExpansionTile to have a constructor argument to let the ExpansionTile be opened at the initial state?

Sounds like a useful PR to me.  @HansMuller and @Hixie are the experts.

We would definitely accept such a PR. It would be trivial; add a bool field (initiallyExpanded or some such, default false) to ExpansionTile, and in its state's initState, replace the fallback false with a reference to that field. The hardest part will be writing a test. :-)


Licence

This project looks promising and useful. Under what licence is this published?

It is on the MIT license (just added to the repo). Please enjoy!
On Thu, Jan 30, 2020 at 5:38 AM veghp<EMAIL_ADDRESS>wrote:

This project looks promising and useful. Under what licence is this
published?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/klavinslab/benchling-api/issues/31?email_source=notifications&email_token=ACPUKDFTK4WOEQRUBUXXE33RALJ6XA5CNFSM4KNWDN5KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IJ2YYKQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACPUKDDZ6ABLAL7EUFVEWJLRALJ6XANCNFSM4KNWDN5A
.

--
Justin Vrana


Fixing missing volatile on references in VMTransport

Fixing missing volatile on references in VMTransport to prevent a synchronization bug.
This resolves https://issues.apache.org/jira/browse/AMQ-5787

I'm closing this request because it was merged into master.


[Other] m4 mac mini 禁用sip后无限重启

Additional Information
m4 mac mini 禁用sip后无限重启， 输完密码进入桌面， 几秒后自动重启， 进入死循环， 如果停留在输密码的页面不会重启

sip禁用后重启不能算这个的bug吧

If you have issue disabling SIP, you can just reenable it in Recovery mode.
Saying that, you should inspect which part of your OS is causing the crash when SIP is disabled. (Out of scope of this repo)


Metric alerts

In what area(s)?

/area administration

/area ansible
/area autoscaling
/area configuration
/area cyclecloud
/area documentation
/area image
/area job-scheduling
/area monitoring
/area ood
/area remote-visualization
/area user-management

Describe the feature

Hi Xavier,
It would be nice if we could get alerted when disks are nearly running out of space. Today our ANF was nearly full and we've had times in the past where the OS disk was full on various VMs.
Would it be possible to set up azure monitoring to notify us by email or something when it is full? Another option might even be to automatically increase the ANF size (but this might be abused). I think this would make our deployments more reliable for customers.
cc @ltalirz

Ah I just saw #1183 .... I'll let you close this one if you think grafana is a better solution!

Yes, look at this as others had the same issue. I don't know if azure monitoring will be able to handle this as it's not the disk out of space but rather the volume created on the disk.
I won't automatically increase ANF size as this will have an impact on cost, and usually customers wants to stay in control of their ANF volumes.

closed by #1265


About Init RedisCluster hanging problem.

I found  issue 387 meant it.
If your start_nodes has dead redis server node. I am afriad of the init RedisCluster is hanging.
I try to dive into it, then i found the follow code snippt:
https://github.com/Grokzen/redis-py-cluster/blob/570d7f23faa9bde8031f5e2622e4c01aef3c4f7a/rediscluster/nodemanager.py#L226
BTW, If you specify the init parameter socket_timeout, the hanging time is socket_timeout sec
I think this project should consider about this scene.

@Milittle I am not really sure this is a problem for this package as all options and parameters is sent into the Redis class instance and this package do want to handle these special cases as little or none at all. The Redis instance that is used in the command you are refering to is called here https://github.com/Grokzen/redis-py-cluster/blob/570d7f23faa9bde8031f5e2622e4c01aef3c4f7a/rediscluster/nodemanager.py#L195 and it that allows for socket_timeout to be set during the initial cluster discovery phase of things.It is not up to this lib to ensure that you configure your client correct for your case/redis setup, but if we are missing to pass in some parameter that you set when you crease your RedisCluster instance, then we can talk about it i mean.

@Milittle I am not really sure this is a problem for this package as all options and parameters is sent into the Redis class instance and this package do want to handle these special cases as little or none at all. The Redis instance that is used in the command you are refering to is called here
https://github.com/Grokzen/redis-py-cluster/blob/570d7f23faa9bde8031f5e2622e4c01aef3c4f7a/rediscluster/nodemanager.py#L195
and it that allows for socket_timeout to be set during the initial cluster discovery phase of things.It is not up to this lib to ensure that you configure your client correct for your case/redis setup, but if we are missing to pass in some parameter that you set when you crease your RedisCluster instance, then we can talk about it i mean.

I mean, as a cluster client package, It should handle this siuation.
If the redis is cluster deploy and have 3 master node, 3 slave node. When I config the 6 node to start_nodes, the above 3 nodes are dead.  The RedisCluster have a automatic cycle these nodes to ensure the get connection?
If I understand wrong. Please let know, thank you work.

The try/except block already handles these two cases
https://github.com/Grokzen/redis-py-cluster/blob/570d7f23faa9bde8031f5e2622e4c01aef3c4f7a/rediscluster/nodemanager.py#L228
where if you timeout or get connectino error it will fail out for that node and continue to the next node. If you get any other error then that error will be raised up to the calling user. This is intended and expected behaviour imo.
If you think otherwise then you need to show me the exception you are getting that you think should probably be added to the list of cases that we should continue on with.

OK, I understand you meaning. Thanks.


General SSL_library_init():

Hi,
Is there a way to check if openssl initialization is already done?
i.e. check if SSL_library_init() or OPENSSL_init_ssl() is already invoked?
Thanks in advance.
Regards,
Amal

For which version?

Hi @richsalz ,
Openssl v1.0.2n.
Regards,
Amal

There is no official API for this.  You could do an EVP lookup with a cipher that you "know" should be there.

Thanks @richsalz
Regards,
Amal


Question: at-least-once delivery

According to the documentation NATS guarantees at-most-once delivery of a message, however, at-least-once delivery is not guarnateed.
I can iamgine only two scenarios where at-least-once delivery fails:

subscriber is not reachable anymore
subscriber cannot process the high load anymore

Both scenarios should lead to a forceful subscriber disconnect by the server.
Therefore, does the following assumption hold in general?

If at-least-once delivery cannot be fulfilled for a given subscriber, the subscriber is forcefully disconnected by the server.

Or the other way round:

As long as a subscriber is connected, he is guaranteed to receive all messages at-least-once and at-most-once (and in-order, if there is only a single publisher).

Neither will hold when publisher and subscriber are connected to different servers in a cluster and the route between the servers is partitioned.

The only statement we can make is the following.
For any given publisher, messages produced will be delivered in order with at most-once delivery semantics per subscriber.

I understand, thank you for your clarification.


Structured representation for program elements

Instead of concatenating and parsing strings everywhere, this introduces a structured data type for representing the transformations on statement labels that can happen during Boogie elaboration and VC generation. It relieves clients from needing to parse the full unsat core element labels.

Looks great, thanks !


The future of dokku-alt

this is a stub based on a request by me for commercial support on dokku-alt
see commercial support?
https://github.com/dokku-alt/dokku-alt/issues/132#issuecomment-67704415

I'm pretty new to dokku and dokku-alt. What is the difference and are there plans to merge again?
From my perspective dokku-alt worked out of the box and I had to install less plugins but I almost want to go back because things like the manager plugin do not fit to a minimalistic framework IMO.
Don't get me wrong: Great work building dokku-alt, thank you!

I am going to be using dokku-alt purely because there is an admin panel built already for it. And the fact it seems to have all the plugins I need already installed.
I am also not really a big fan of bash, personally. I would love to see someone rewrite the dokku-alt setup in a language that would lead to more easier development. Maybe Go or Python. But that is a personal opinion.


[BUG]: Payment for Warrant via Police_Delivery Channel showing police (Not provided).[UAT]

Describe the bug
Expected behavior
It should show Police address or Police station.
Reference link:

*verified in UAT env, Not working


Problems with my translated project

I tried to translate my project https://scratch.mit.edu/projects/459257487/ and found some problems:

The text said by the money sprites becomes "null".
The sum of the bills is incorrect, for example 1234 becomes 12304.

I'm guessing that 1230 + 4 being equal to 12304 is because of Javascript's propensity to interpret variables as strings.  We'll need to figure out where this is happening and ensure that it's doing proper numeric conversions first.

We made a few fixes over the last year or so which address the issues described here, and from testing it looks like this project is working correctly now!


New release needed to avoid deprecated action

What feature do you want to see added?
Action are renovated and updated main branch.
But reusable workflow was not released since long time and now cause deprecation issue when CD is running

Node16 is EOL since 5 month ago.
Upstream changes
No response

https://github.com/jenkins-infra/github-reusable-workflows/releases/tag/v1.1.3

Well for me it's not yet fixed. One release of yesterday still triggers warnings : https://github.com/jenkinsci/flyway-api-plugin/actions/runs/7947376727
https://raw.githubusercontent.com/jenkinsci/.github/master/workflow-templates/cd.yaml still point to a release that doesn't contains #29

https://github.com/jenkins-infra/github-reusable-workflows/releases/tag/v1.1.3

@lemeurherve the v1.1.3 was released on Oct 22, 2022. could you please help to release a new one?


the window is no longer responding

VSCode Version: latest version
OS Version: Windows7

sometime I open editor it showing "The Window is not Longer responding" and error message.
also I try to quick open it showing the error message.

Does this issue occur when all extensions are disabled?: No

I can't upload the profile... bcz of security policy in company...

If you launch using code --verbose in cmd.exe does it return anything interesting when the crash happens?

I don't know but this message is different from other messages.
[7868:1025/083020.600:INFO:CONSOLE(1409)] "%cDEBUG", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1409)

[7868:1025/082937.764:INFO:CONSOLE(282)] "%c[Extension Host] %cactivating extens
ion %c(at Object.activate (C:\Users\LGRnD.vscode\extensions\ms-vscode.cpptools-
0.19.0\out\src\LanguageServer\extension.js:41:13))", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (282)
[7868:1025/082937.779:INFO:CONSOLE(282)] "%c[Extension Host] %cstarting language
server %c(at realActivation (C:\Users\LGRnD.vscode\extensions\ms-vscode.cpptoo
ls-0.19.0\out\src\LanguageServer\extension.js:93:13))", source: file:///D:/#.Sec
ure Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workben
ch/workbench.main.js (282)
[7868:1025/082940.303:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/082947.078:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/082947.088:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083015.776:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083020.600:INFO:CONSOLE(1409)] "%cDEBUG", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1409)
[7868:1025/083021.364:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083021.603:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083021.968:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083021.978:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083021.979:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)
[7868:1025/083021.980:INFO:CONSOLE(1408)] "%cTRACE", source: file:///D:/#.Secure
Work Folder/9__UTILITY/VSCode-win32-ia32-1.28.1/resources/app/out/vs/workbench/
workbench.main.js (1408)

I'm having this same problem.
Here is the output of code --verbose from cmd.exe:

which subsequently opens a new VS Code session.
Visual Studio Code version:
1.29.0 5f24c93878bd4bc645a4a17c620e2487b11005f9 ia32
Windows 7 OS

This is a subsequent event:

which was showing when unlocked my computer today.

Can you follow this wiki and post the results back here to understand more about why performance is so bad for you: https://github.com/Microsoft/vscode/wiki/Performance-Issues
Thanks!


link to pre-trained weights

The link to onedrive folder with pre-trained weight is not working anymore.
Could this be updated? Thanks in advance

Thanks for the notification. I have updated the link. Please pull the latest code to have a try.


Problem with [nw] command and Question event.

The problem
When using [nw] command on dialogs before a Question event with options, if the user skips the lines with the [nw] there's a chance that the options for the question get duplicated or hidden.
Describe the bug
As explained above, the [nw] does not work correctly if the lines are skipped, making the next events in the timeline not work as intended
To Reproduce
Steps to reproduce the behavior:

Create a timeline and add text event with some lines ending in [nw] at the beggining
Add a question, (or any other event that waits for user input)
Play the scene normally and skip all the lines with the [nw] command
See how the question options get hidden or duplicated

System (please complete the following information):

Dialogic Version 1.2
Godot Version: 3.2.3

Solutions
Changing the _on_text_completed() method like this:
func _on_text_completed():
    finished = true
    if current_event.has('options'):
        for o in current_event['options']:
            add_choice_button(o)
    if current_event.has('text'):
        # [p] needs more work
        #if '[p]' in current_event['text']:
        #    yield(get_tree().create_timer(2), "timeout")
        if '[nw]' in current_event['text']:
            var current_index = dialog_index
            yield(get_tree().create_timer(2), "timeout")
            if dialog_index == current_index:
                _load_next_event()

Thanks to Rab_Loe#4334 on discord for this piece of code.
This change was already tested, but more testing never hurts

Thanks for writing it down, but the fix was actually already merged into 1.3 WIP along with more [nw] improvements. Thanks again for testing and finding this and sorry you had trouble during the game jam. (The game was nice anyways ;])

Should I close this one?


Force notifications on iOS

Hi all:
it's possible to send a notification to iOS for an accessory that not support notifications? For example, a switch cannot show notifications on iOS: It's possible to force on iPhone to show an alert when a switch is pressed?
Regards!!

Nope.
Sent from my iPhone

On Aug 6, 2018, at 3:51 AM, p3ntium<EMAIL_ADDRESS>wrote:
Hi all:
it's possible to send a notification to iOS for an accessory that not support notifications? For example, a switch cannot show notifications on iOS: It's possible to force on iPhone to show an alert when a switch is pressed?
Regards!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

oh . :(
And the StatelessProgrammableSwitch? It can receive notifications?
I have a X10 switch as doorbell, and actually it is a Homekit Switch element in HAPNode, and I want to show alerts in iOS when someone rings...
Thanks!!!

StatelessProgrammableSwitch doesn't display notifications since you configure actions for specific events (like single press, double press or hold) and those actions are executed then via HomeKit.
You could probably implement a doorbell service but this service requires a camera.
A bit ugly but you could also create a motion sensor instead, since this is a device which is able to display notifications on your iOS device.

There is a doorbell service…
Sent from my iPhone

On Aug 6, 2018, at 10:07 AM, Andi<EMAIL_ADDRESS>wrote:
StatelessProgrammableSwitch doesn't display notifications since you configure actions for specific events (like single press, double press or hold) and those actions are executed then via HomeKit.
You could probably implement a doorbell service but this service requires a camera.
A bit ugly but you could also create a motion sensor instead, since this is a device which is able to display notifications on your iOS device.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks @Supereg , the motion sensor it's my second option  :)
@KhaosT : The Doorbell service can't be configured lonely into an accessory, right? Is there another way to implement it out of a camera accessory?
Thnks!!

@KhaosT the doorbell service is not supported by the Home App as a standalone accessory. It must be implemented combined with a camera.

So I think the best choice is through a motion sensor accessory. It has an additional advantage: I have a pipe cam on the main door, if I configure the switch like a motion sensor and merge it in the same room with that cam, when someone touch the switch I will receive a snapshot attached to the notification.... This is a good behaviour...
Regards!!!


Update snagit.rb

Changes to a cask:
Editing an existing cask

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

I was under the impression I had commented this, but apparently not. When version is present, you also need to set sha256. You can find what it is by running shasum -a 256 on the downloaded file.

I have updated the condition

Thanks!


"withPermissionsCheck()" doesn't work omitted callback parameters

Overview
Thank you develop permissions-dispatcher-ktx.
I tried using 1.0.0-alpha2, it seems not to work "withPermissionsCheck()".
If add default action for these parameters, I'm very happy!
WebViewBuilder()
    .geolocationEnabled(true)
    .javaScriptEnabled(true)
    .onGeolocationPermissionsShowPrompt { origin, callback ->
        withPermissionsCheck(Manifest.permission.ACCESS_FINE_LOCATION) {
            callback?.invoke(origin, true, false)
        }
    }
    .into(findViewById<WebView>(R.id.webview_target))
    .loadUrl("https://www.google.co.jp/maps")

If omit "onShowRationale" parameter, it don't show a permissions dialog.
→ If add "onShowRationale", it show dialog.
If omit "onPermissionDenied" parameter and dialog result is "denied", it seems not to fire denied action.
→ "Never Ask" is same.

Expected
I hope behavior  like a "PermissionsDispatcher 4.3.0".
Actual
If omit these parameters, it seems not to work permission dialog.

onNeverAskAgain
onPermissionDenied
onShowRationale

Environment
Development

Android 10 Emulator
Android Studio 3.6.2

Library

permissionsdispatcher-ktx 1.0.0-alpha2

Reproducible steps
It can reproduce using ktx-sample too.
Please try omitting callback parameters.

Thank you so much! I was packed today gonna take a look tomorrow!

Sorry for the late, the issue has been addressed in https://github.com/permissions-dispatcher/PermissionsDispatcher/pull/678 and beta1 is available now! Thank you for the feedback~


use semaphore in subflow

Semaphores work fine - but if the call of Take and Release happens in a subflow, Nodered gets confused. When deploying changed flows, they are not restarted and the debug nodes can still be enabled/disabled, but the output is not updated.
To check this, simply put take and release in a subflow and restart nodered (node-red-restart) - otherwise you cannot reproduce the problem.
BR
Hubertus

Sorry not sure I follow...
This is a standard set of Nodes for Node RED, that uses Config Nodes to orchestrate and manage the queue,
what has debug nodes got to do with this?
If you mean a queue config  Node created inside of a sub flow,  is not seen by non sub flow nodes - this is standard behaviour in Node RED, a config node needs creating outside of the sub flow to see it outside and in the subflow

Yes, that's clear.
the configuration takes place/and did take place outside of a subflow. Therefore, this can actually be ruled out.
I have drastically reduced the semaphore calls - at the moment everything works. I'll let you know if I expect problems here again.
Thanks for now and best regards


Fixed #29641 -- Added UniqueConstraint support.

@charettes I've cleaned up your branch a bit. (https://github.com/django/django/pull/10271)

buildbot, test this please

I'm not sure why the mariadb builds failed - if they can be re-run that'd be great.

Feel free to grab the author attribution of the commits...

You can use Co-authored-by: signature lines if you'd like to share the attribution. GitHub also recognises this nicely - https://help.github.com/articles/creating-a-commit-with-multiple-authors/

It would be better to submit the first commit as a separate PR to ease review (and then maybe the others after that's merged). Historically, we've rarely done deprecations for changes in the database backends. It would be fine to make backwards incompatible changes as long as they're documented in the release notes.

It would be better to submit the first commit as a separate PR to ease review (and then maybe the others after that's merged). Historically, we've rarely done deprecations for changes in the database backends. It would be fine to make backwards incompatible changes as long as they're documented in the release notes.

@Ian-Foote would you like me tackle this or you'll deal with it? I just want to make sure the first commits miss the 2.2 cut as it will significantly hard to change the API after that.

@charettes I'm planning to do that today.

@timgraham I've created #10406, #10407, #10408 and #10409.

Inadvertently closed... continued in https://github.com/django/django/pull/10638.


Some analysis_server tests produce invalid analysis_options.yaml files due to passing an empty list for experiments instead of null.

While working on 352220, I came a cross a case where an invalid analysis_options.yaml was produced from some tests (namely: remove_break_test.dart).
It seems that the setUp function is passing an empty list for experiments instead of null to createAnalysisOptionsFile producing an invalid format for the analysis_options.yaml (i.e. a label without value):
https://github.com/dart-lang/sdk/blob/c3a626f729524ac8859ef8f7f06ebf4e98a43719/pkg/analysis_server/test/abstract_context.dart#L121-L138
Maybe the method above should check for both null and isNotEmpty for all the arguments if that wasn't intentional.

Great catch!

Maybe the method above should check for both null and isNotEmpty for all the arguments if that wasn't intentional.

Seems reasonable to me.
@scheglov?

It seems to me that it would be cleaner to give default const [], and check for .isNotEmpty instead of != null.

👍 I like that.

Hi @pq
I pushed a quick fix for this one as suggested by @scheglov at:

https://dart-review.googlesource.com/c/sdk/+/352866

Side question:  what does the yellow color mean for those 9 tests?

These are @SkippedTest that we were not able to make reliable.

Thank you!

Also, I've been running the tests using dart run test/test_all.dart -- is that sufficient for local testing?

Should be, yes.


[gguf] Fx type in "should parse a local file"

Follow up to https://github.com/huggingface/huggingface.js/pull/656
cc: @ngxson

Thank you! LGTM.
I was quite surprise that it isn't got catch by pnpm run lint:check. Do you have any idea why? (Or maybe because my branch is not up-to-date, so there's lint config that I didn't know about)

I was quite surprise that it isn't got catch by pnpm run lint:check

indeed,  pnpm run lint:check didn't catch it. But the tests on huggingface.js were failing. Maybe @coyotte508 knows why


Fixes #3178. v1 - Library has no strong-name.

Add strong-name signing.
Fixes

Fixes #3178

Proposed Changes/Todos

[x] Add snk file
[x] Add strong naming sign

Pull Request checklist:

[x] I've named my PR in the form of "Fixes #issue. Terse description."
[x] My code follows the style guidelines of Terminal.Gui - if you use Visual Studio, hit CTRL-K-D to automatically reformat your files before committing.
[x] My code follows the Terminal.Gui library design guidelines
[x] I ran dotnet test before commit
[ ] I have made corresponding changes to the API documentation (using /// style comments)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any poor grammar or misspellings
[ ] I conducted basic QA to assure all features are working

I suppose that NStack library should be strong-named too.

All tests passed!

BTW I'm not so expert on this library, so I don't now how to "to assure all features are working".

Created PR #91 for NStack.
I tried to build again this version of Terminal.Gui with it and strong-name warning has now disappeared.

BTW I'm not so expert on this library, so I don't know how "to assure all features are working".

Yeah, that's worded a bit unfairly.
Please, at least, do this:

[ ] Run the All Views Tester Scenario and select each view. Ensure nothing really odd happens.
 Some are more complex and tend to push the envelope and are good at sniffing out issues:

[ ] CharMap
[ ] Scrolling
[ ] Text Formatting
[ ] Editor
[ ] Dialogs
[ ] Wizards

[ ] Do the above under WSL
[ ] Do the above with -usc on both Windows and WSL

@tig, @BDisp, I just added a strong-name option without changing the code. I don't think it compromises the library's running, do you?

I don't think so and the tests pass. But you have to wait for @tig's decision.

Do I need to do anything other than merge this for it to work for publishing to Nuget?
I was expecting to have to generate a private key that we'd need to store in a github secret or something.

It is not necessary to generate a password for the .snk file for strong name purposes, so you just need to merge it. Thank you, @tig.

Ok, NStack v1.1.1 is published with an SN.
Now for Terminal.Gui...

v1.15.1 is published.
Please verify that I didn't screw something up ;-)


URL's get duplicated at the PreMiD presence creator

Reproduce

Add 2 domains in this format

Remove one

All disappeared
Fix
Just check if the value is already in the box.

This will be fixed on the next deployment. Thank you for reporting this issue. Just for your information; this page doesn't check everything (e.g. if you write https://egg, it'll think it's a valid URL since I'm using the built-in URL API which might be changed in the future), so you have to make sure to check your result even if it was created with this tool.

Fixed with https://github.com/eggsy/website/commit/2e9add4ed7224ac2cc7e675c68aae439497bb9b7.


need to increase the default dbcache size

getnameproof fails to return the proof if there is not enough coin cache to roll back to the requested block. In the current release (12.2.3) it completely boggles the application state. In the current master it simply returns too early. We presently allow up to a 50 block rollback. Upstream bitcoin ups the dbcache to 450MB. Some rough calculations on our part show that we would need 1100MB to handle 50 full blocks. 50 full blocks is highly unlikely, but 450MB was insufficient for the use case recently demonstrated. That one required 700MB: Modify the max rollback to 5000 and run getnameproof cuphead f231bbae207d4dc5ee6cf32b5f3ae87958acdc3a40a37e32cb46c95134419f39. I propose that we up it to 700. That's not a steady state RAM usage, just an occasional spike.

It depends of block size, maximum is 2MB but most of the time is pretty lower. We can set rollback to arbitrary value say ~1000 then we can throw out of memory exception, it'll be pretty noticeable what's going on.

#238


cargo-limit: init at 0.0.4

Motivation for this change
The tool is very handy to reduce noise during development.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110970 run on x86_64-darwin 1

1 package built:

cargo-limit

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 110970 run on x86_64-darwin 1

1 package built:

cargo-limit


Update README.md

We want a more standard style for the Codecov example READMD.mds. I've added and moved around a few things. @larzw it looks like you've done some work on this. Would you mind reviewing the changes?

@TomPed, it seems that this merge got "lost".
Still, I tried to follow your instructions regarding .Net Core to setup continuous integration trough Github actions, and even though all my .csproj files contain the <DebugType>full</DebugType> tag when I try to generate my reports, no report gets generated, and I get the following error message:
No results, this could be for a number of reasons. The most common reasons are:
    1) missing PDBs for the assemblies that match the filter please review the
    output file and refer to the Usage guide (Usage.rtf) about filters.
    2) the profiler may not be registered correctly, please refer to the Usage
    guide and the -register switch.

Could you provide me some assistance on what am I'm doing wrong?
My repository: https://github.com/raschmitt/date-override
Action I'm trying to run: https://github.com/raschmitt/date-override/blob/master/.github/workflows/dotnet-core.yml
Action results: https://github.com/raschmitt/date-override/runs/886072938?check_suite_focus=true

Closing in favor of https://github.com/codecov/example-csharp/pull/30