bluez: patch gamepad pairing regression

Description of changes
bluez releases 5.69 and 5.70 introduced the regression that makes pairing Playstation 3/4/5 controllers impossible: the gamepad disconnects after a few seconds. This patch fixes that.
See for more info: https://github.com/bluez/bluez/issues/614
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Duplicate https://github.com/NixOS/nixpkgs/pull/271864/files


Plugin not found (in Package Control)

I tried to install it via Package Control, but it doesn't show up.
When I clone it to my packages, on ST3 load console shows this:
File "D:\Applications\Sublime Text 3 x64\Data\Packages\SublimeLinter-jshint\linter.py", line 14, in <module>
    from SublimeLinter.lint import Linter, util
ImportError: No module named 'SublimeLinter.lint'

I'm also running into this issue, and none of the above suggestions have aided in a solution. I'm on OS X 10.10.1 w/ Sublime 3066. I've reverted my Sublime install (https://www.sublimetext.com/docs/3/revert.html) and even tried a fresh install; neither let's me install SublimeLinter. When I search in Package Control for 'SublimeLin' I am not given an option to install SublimeLinter. Any suggestions?

Same problem here as @jamesmehorter

Oddly enough, I tried again this morning and now SublimeLinter is showing up! Definitely feels like some sort of caching???? Super weird!

same prob here, not showing up.

@thewinterwind https://github.com/wbond/package_control/issues/926 this was a package control issue, and has been resolved.

In order to get it to show up, I had to go into my package control settings and update the following line:
"repositories": ["https://github.com/SublimeLinter/SublimeLinter-jshint"],

Package Control is currently down.


Clean npu dtype branch

Description

sym_int4_rtn and sym_int8_rtn are only used in NPU
only support intel_npu_acceleration_library > 1.10

https://github.com/intel-analytics/ipex-llm-workflow/actions/runs/9803772728


ARTEMIS-3221 Migrating to Jakarta EE 8 artifacts

I havent actually properly looked at the changes, but skimming them it seems like you are going to be missing an update to the LICENCE file (and perhaps NOTICE file) that goes in the binary convenience distribution, given the aim here is to change a bunch of deps in ways which make them all go from using ALv2 to something else.

@gemmellr thank you for pointing me to LICENSE file, I have just updated it but I'm not sure how I should update the NOTICE file, do you have any suggestions?

thank you for pointing me to LICENSE file, I have just updated it but I'm not sure how I should update the NOTICE file, do you have any suggestions?

The NOTICE file should only convey 'required notices', and distributing the license text itself often already satisfies most 'must include a copy of  type requirements, so changing NOTICE is often unnecessary, especially for using things that dont themselves have a NOTICE file. In this case, im not sure anything will need added, but its worth checking nothing needs removed, and just generally checking the file seems correct and up to date.

The commit message here says.. ARTEMIS-3215 Upgrade hawtio version to 2.13.2"
I guess you had some git-fu issue when you squashed the wrong comment?

I will do a couple tests before merging it

if you do mvn dependency:tree from integration-tests you still see Geronimo being pulled down. Is that expected?

I still see a Geronimo usage from openwire server's protocol, and openwire clients...
if I look a mvn dependency:tree I still see this a couple times... but it seems to be the last one now?
[INFO] +- org.apache.activemq:artemis-junit:jar:2.18.0-SNAPSHOT:test
[INFO] +- org.apache.activemq:activemq-client:jar:5.16.0:test

I pushed a commit to fix it.


[NTR] add device temperature setting

Preferred term label
device temperature setting
Synonyms
Textual definition
A setting datum that specifies the operating temperature of a device.
Suggested parent term
IAO:setting datum
Examples
Attribution
ORCID:0000-0002-1595-3213

This issue was created retrospectivly to have a term tracker item to link to within the ontology, for future discussions about and the tracking of possible changes of this term.


add nvim side for zellij, waiting for zellij side for full support

zellij #967

If you are waiting for zellij to implement something, then better mark this PR as draft.


Stop Button Do Not Work When on Pause

As the audio is playing, press pause, then stop, and play the audio again, it doesn't start from the beginning but continues where the pause was.

What browser? Working fine in chrome.

Safari, and Firefox is working fine too.

I don't think this is an issue then

No, I meant it's not stopping right in Safari, and for Firefox it's working like Chrome is.

oh ok


Optional support for rustls

What did you implement:
Optional support for using rustls via hyper-rustls instead of native-tls via hyper-tls.
How did you verify your change:
It builds.
What (if anything) would need to be called out in the CHANGELOG for the next release:
I added a stub CHANGELOG message.

@rocallahan thanks for keeping this short and sweet.


how to get file name in error message in which validation fails

HI
I am using the ajv schema validator for my project..i need to validate a group of 100+ json files against one schema rule.....i traversed the json files one by one and error is consoled...but in error message i cant get the name of json file in which error occurred..is there any method to get that data also..
Thank you

This is not part of ajv scope, you have to manage it in your application code.


ceilometer-polling role was renamed (back) to ceilometer-central

For both Cloud6 and Cloud7
See https://github.com/crowbar/crowbar-openstack/pull/373 and https://github.com/crowbar/crowbar-openstack/pull/376

+1 Thanks!!!

+1

+1

Note: this is needed to make Cloud 6 gating green

Still +1

+1


Font rendering in WSL

This is screenshot from Windows console running WSL bash:

While this is from Windows terminal:

Why can this be an issue?
I noticed it with font shipped with WT and now trying with Consolas, but it's the same result

Can you copy one of those weird characters for me?

@DHowett how do you mean I copy the characters?

No, I mean can you copy one to the clipboard and send it to me as a github comment?

If you cannot select text because of mc or tmux, you can hold down Shift and drag to select, then Shift+right click to copy.
Otherwise, you can leave off Shift

ok, character before mc keys

Bingo! This is a /dupe of #3487. Thanks!

Ok thanks :)


Karabiner 10.17.0 crashing in -[PreferencesManager clearNotSave]

Twice so far today. I was not using my Mac at either time.
Karabiner_2016-03-06-150337_mary.txt
Karabiner_2016-03-06-102708_mary.txt
I've never had a crash with any previous version of Karabiner and my configuration hasn't changed.

Thank you for feedback!
I've fixed this issue at v10.18.0.
Please upgrade Karabiner.


cast_storage method from datasets.features

Describe the bug
A bug occurs when mapping a function to a dataset object. I ran the same code with the same data yesterday and it worked just fine. It works when i run locally on an old version of datasets.
Steps to reproduce the bug
Steps are:

load whatever datset
write a preprocessing function such as "tokenize_and_align_labels" written in https://huggingface.co/docs/transformers/tasks/token_classification
map the function on dataset and get "ValueError: Class label -100 less than -1" from cast_storage method from datasets.features

Sample code to reproduce the bug
def tokenize_and_align_labels(examples):
tokenized_inputs = tokenizer(examples["tokens"], truncation=True, is_split_into_words=True, max_length=38,padding="max_length")
labels = []
for i, label in enumerate(examples[f"labels"]):
    word_ids = tokenized_inputs.word_ids(batch_index=i)  # Map tokens to their respective word.
    previous_word_idx = None
    label_ids = []
    for word_idx in word_ids:  # Set the special tokens to -100.
        if word_idx is None:
            label_ids.append(-100)
        elif word_idx != previous_word_idx:  # Only label the first token of a given word.
            label_ids.append(label[word_idx])
        else:
            label_ids.append(-100)
        previous_word_idx = word_idx
    labels.append(label_ids)

tokenized_inputs["labels"] = labels
return tokenized_inputs

tokenizer = AutoTokenizer.from_pretrained("distilbert-base-uncased")
dt = dataset.map(tokenize_and_align_labels, batched=True)
Expected results
New dataset objects should load and do on older versions.
Actual results
"ValueError: Class label -100 less than -1" from cast_storage method from datasets.features
Environment info
everything works fine on older installations of datasets/transformers
Issue arises when installing datasets on google collab under python3.7
I can't manage to find the exact output you're requirering but version printed is datasets-2.3.2

Hi! We've recently added a check to the ClassLabel type to ensure the values are in the valid label range -1, 0, ..., num_classes-1 (-1 is used for missing values). The error in your case happens only if the labels column is of type Sequence(ClassLabel(...)) before the map call and can be avoided by calling dataset = dataset.cast_column("labels", Sequence(Value("int"))) beforehand. The token-classification examples in Transformers introduce a new labels column, so their type is also Sequence(Value("int")), which doesn't lead to an error as this type unbounded.


Create Income and Create Expense mutations

Keypoints:

Add createIncome mutation
Add createExpense mutation
Resolve merge conflict

Related Issue(s)
closes df-69
closes df-70
Checklist:

[ ] New/Modified Tests are Passing
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings or errors.

Additional Notes:

@MaryBruff

Code looks good - though I find it interesting that you ran into a merge conflict in your app.js file - seems like git should have handled that fine 😆
Generally, Its interesting to see only 1 of 4 tasks completed on a PR that is under review-request.  I think you might want to add some comments as to the status of things like testing, documentation, and code comments.  Since its included in your PR template, its worth a mention.  This also could be an opportunity to revisit what is expected in a PR template!

@

Code looks good - though I find it interesting that you ran into a merge conflict in your app.js file - seems like git should have handled that fine 😆
Generally, Its interesting to see only 1 of 4 tasks completed on a PR that is under review-request. I think you might want to add some comments as to the status of things like testing, documentation, and code comments. Since its included in your PR template, its worth a mention. This also could be an opportunity to revisit what is expected in a PR template!

That's good feedback! Thank you. I will bring this up at standup.

@MaryBruff @chisPmama This has been updated. Please review latest push.


Update machine-groups.md

Updated tip to clear the confusion

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7436

@JohanFreelancer9 Please copy review

@mjcaparas Please merge this PR. Thanks


Update MD file on new branch 2017-11-19_05-26-50

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update release-notes-containers.md

@eric-urban @AleksandrGontcharovMS Update the April release notes as we only fixed vulnerabilities and didn't do any model updates.

@eric-urban

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Terms output in a strange order when paging in the Terms admin interface

Terms output in a strange order when paging in the Terms admin interface.
I believe this is because sort is applied after paging is applied.
https://github.com/OrchardCMS/Orchard/commit/356e8230ab2c4704ab300fb6915606137873bf22

Replacing line 45 in TermAdminController.cs with the following code fixes the issue:
            var terms = TermPart.Sort(_taxonomyService.GetTermsQuery(taxonomyId).ForPart<TermPart>().List());

            if (pager.PageSize > 0)
            {
                terms = terms.Skip(pager.GetStartIndex()).Take(pager.PageSize);
            }

This applies the paging after the sort.

Your change is just adding another issue, as it would load all the terms in memory before sorting and paging it.
However we agree there is an issue here, first the paging is not based on an order, so loading different pages doesn't mean we'll get the correct page, and also different hierarchies can be mixed in the same page.
The solution is not simple, but there is one. We need to change the FullPath property to have its components padded. Instead of have "1/2/3" we'd have "00000001/00000002/00000003" this way it is lexicographically orderable in database.
1- Change the code to store it this way.
2- Add an index to this field as we'll sort it in database, and also because we filter by prefix to load children of a specific term. Add the sort order in the paged query of the terms.
3- Make TermsComparer obsolete so that we can force users to use the database sorting.
4- Create an upgrade step in the Update module to update all existing terms.

Wouldn't it be better to add a new int property to taxonomy termpart called "Level".
Its value will be the number of '/' in FullPath. In migrations we should set it for existing terms.
Then the order will be obtained ordering by Level and Weight.
This is how we have solved this issue recently. Furthermore it helped us to add methods that get child elements of a node limiting the number of levels returned in an efficient way.
Don't you think it is more elegant and efficient than using padding?

Problem of that solution is ignores the weight of the terms. Other more complex solution would be to have a FullWeigthPath Property that stores weights of parents: instead of its id: "2/5/2".
The extra complexity would be caused because after update the weight of a term we will need to visit all their children updating the FullWeigthPath to update the related value.

So maybe we can keep the current Path, but create the lexicographically sortable structure using weights. Or like in menu items, use a double for weight that would just represent the order in the tree.

Problem I see is to update the double or the lexicographically sortable structure for all terms affected in an import scenario for each imported term is expensive. Unless we can execute the reorder operation for  the taxonomies affected only after a complete step of a recipe execution finished.

What size of dataset will see all these performance issues come into play
1000+ terms? 10,000+ terms?
Maybe just put warnings in place on relevant recipes/imports to say that
large sets of terms may take a long time to process.
There can't be that many people using datasets of this size. I think anyone
using really big term sets will be faced with a whole host of problems - I
mean how would the interface deal with selecting from 1000+ terms when
editing a content item?
Also with that number of terms surely the primary (and possibly only) sort
bring used would be alphabetical, which I assume doesn't have these issues,
as that is a sort naturally available already via DB. Sorting by weight
wouldn't have any practical use with sets this size.
So perhaps we make the weight feature something that is enabled/disabled
separately - so can be disabled for these large dataset scenarios.
On 30 Jul 2016 10:21 pm, "Sergio Navarro"<EMAIL_ADDRESS>wrote:

Problem I see is to update the double or the lexicographically sortable
structure for all terms affected in an import scenario for each imported
term is expensive. Unless we can execute the reorder operation for the
taxonomies affected only after a complete step of a recipe execution
finished.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/OrchardCMS/Orchard/issues/7055#issuecomment-236357586,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAm8xTR_UBgJ6ZoNIGuoSpBi1YK-xqE7ks5qayW6gaJpZM4JTtv1
.

Well for example we currently for this page https://www.sayandsail.es we use a taxonomy containing all the ports in Spain, a total of 467 ports categorized by country and province. As soon as we add new countries it is expected to grow up considerably.
Currently importing it consumes considerable time.  In fact, in our daily develop cycle we create the site from scratch using a setup recipe but we avoid to create the complete taxonomy when it is not necessary for saving time.
You are right we order this taxonomy terms setting the weight to values that follow an alphabetically order. But for ordering a hierarchy of terms alphabetically we still need to take into account the hierarchy, that means use the text of the parent of a term. So, we will need a lexicographically orderable structure based on the text of the parent terms: grandfather text/father text/my text which should be updated when the text of parent term changes. In our case this happened to correct text encoding mistakes in the original import we made to production site.
We have other multitenant configuration with 30 sites where each site can have around 360 terms of families of products localized in Spanish and English for a taxonomy. Every night those sites sync products and families from an ERP. Currently this operations takes more than it will be desirable. I know it can be improved on our side replacing current brute force sync per a syncing based on changed or removed families, but this another discussion about how to deal with legacy systems. The point is that in general orchard 1 is not very fast importing data so please don't make it slower.

But, I agree it would good to know the real cost of performing this operation for different numbers of terms to check if my concerns are well founded.

I think I didn't read with enough attention what @sebastienros said: "Or like in menu items, use a  double  for weight that would just represent the order in the tree."
This double won't need to be recalculated after update each term. It will be just the one you provide in the import. So, when I create the xml for the import I will take care of providing the right value.
So, I agree with that proposal. 👍

We are evaluating to provide a PR (me and @jardg) .
After looking how menuitems work I found they don't use double for weight https://github.com/OrchardCMS/Orchard/blob/dev/src/Orchard.Web/Core/Navigation/Controllers/AdminController.cs#L140
They use a string with position of ancestors and their own separated by dots. When the order of a parent is updated they update their children. They could be improved using a lexycographyc sortable position, but this is sth for another issue.
Coming back to taxonomies:

Working with a double for the weight will solve the issue of adding leading 0 to the position, but will force us to load and update all the terms of a taxonomy when one term is updated.
Working with a string lexicographically sortable, will reduce the number of load and updates needed to the minimum possible only doing that for terms affected.
Instead of using a handler for updating positions, as for menuitems, leaving the responsibility of providing the position to the code that adds or modifies a term allows a good performance for import scenarios, cause we don't need to set positions cause we rely on the position provided in the recipe. So, you need to provide in your recipe all the childrens affected by a change in position or to use a command that recalculate the position of those terms (or all of them) in a taxonomy.
TaxonomyTerm driver should export positions without leading 0s and accept them without leading 0s

@sebastienros @pilotbean If do you agree with that we could work on a PR


Use cdnjs for angular.js

Google cdn does not work in China. So, I can't view the demo from China.

Can you update the PR to fix merge conflicts please.

@ojhaujjwal this is outdated, in gh-pages we use a local copy of angular now.

@jacobscarter Hey, if you want you can use jsDelivr which pulls files directly from npm instantly

@jsdelivrbot npm angular-wizard

@jimaek thanks for the info, we have jsdeliver listed as CDN in the README, for the gh-pages we will just leave it pulling locally, that way there wont be any domain access issues for anyone


Not working in 1.0

I justs discovered and installed the css file as per instructions, but nothing is happening either in live editor or reader mode. Can anyone confirm that this works in 1.0 ?

Also not working for me

me too v1.0.3

Did you guys installed the contextual typography plugin?
It is a requirement for the snippet to work.

Yes, contextual typography is installed. I've tested on both MacOS and Windows and no luck. Is it working for you in the latest Obsidian?

Yes, contextual typography is installed. I've tested on both MacOS and Windows and no luck. Is it working for you in the latest Obsidian?

It's working for me.
Obsidian 1.0.3. Windows 10.
I tested it on my own vault and on the sandbox vault.

Hmm ok on further testing it is working in Reader view, which I thought I'd tested but apparently not.
Is there a way to make it work in Live Preview? I never really use reader view

@sizhky @timmb @berot3
Hey guys, there is a workround for OBS 1.0~ :

Open the css file with VSCode

Replace ".is-readable-line-width" to ""
( or you can just delete the words ".is-readable-line-width" & all its occurrences. )

snippet is working again ~~ !!

for me it works in one vault and doesn't work in other vaults, that's strange, isn't it!!

On 1.1.9 works fine with reading mode, wish for live mode

@sizhky @timmb @berot3 Hey guys, there is a workround for OBS 1.0~ :

Open the css file with VSCode
Replace ".is-readable-line-width" to ""
( or you can just delete the words ".is-readable-line-width" & all its occurrences. )
snippet is working again ~~ !!

Great. It's working again.


Scenes push super slow on iOS

It takes about a minute and a half to push a scene from a listView on iOS 7.

@jeffreymendez1993 You need to provide more details.

Is it on simulator?
Is debug mode enabled?
Have you tried iOS 8+?
Have you tried on a device?
Have you tried on the release mode?
Which RN version?
Which machine configuration? (ram, ...)

yes
yes
no
no
5 no
0.42.0
ram

@jeffreymendez1993 I've been burned by Slow Animations setting of the iOS simulator so many times, cmnd+t is only a key away from cmnd+r for reload.  Triple check you haven't accidentally enabled that setting?  From iOS simulator: Debug -> Slow Animations

@thorsonmscott  Thanks that was the problem. Closing the issue now.


Extending schema with autoValue/Custom functions

I want to extend schema with it's own custom and autoValue functions and have the option for an original function to be called with the option being the default.  I suppose you could make it a cleaning parameter and prevent previous schema functions from running by operating on this.originalValue. If it's a custom function that it returns it's value or one of the previous ones or can clear previous returns.

If I'm understanding this correctly, it should be possible, though not necessarily pretty.
Something like:
const originalCustom = baseSchema.schema("someKey").custom;
baseSchema.extend({
  someKey: {
    custom() {
       // implement new logic or call `originalCustom`
    }
  }
});

Yes its not pretty. Maybe this could be an enhancement?


Zabbix problem tag

Good morning,
yesterday after the Grafana upgrade to version 9.4.7 we are experiencing an anomaly in the selection of tags for the "Zabbix problem" widget.
Before yesterday when I put 2 tags, only the problems that respected both tags appeared, so the rule was (Tag1) AND (Tag2)
With the new version, he shows me all problems that contain either Tag1 or Tag2, no longer just problems that contain both.
Zabbix version 6.0
Grafana version 9.4.7

Hi! Yes, this behavior was changed. But I think since there're different use cases, it makes sense to add And/Or switch for tag evaluation.

I labeled it as a bug because it breaks existing behavior. What I think is needs to be done is:

Revert default behavior to AND operator
Add operator selector AND/OR


Highlight search results case-insensitively

Close #88

Thanks @zarvox

This change is now live.
See: https://apps.sandstorm.io/search?term=board


composer error: The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)     Unzip with 7z command failed, falling back to ZipArchive class

Under Windows 10, I try to install with composer require bacon/bacon-qr-code but get this error:
C:\inst>composer require bacon/bacon-qr-code
Using version ^2.0 for bacon/bacon-qr-code
./composer.json has been created
Running composer update bacon/bacon-qr-code
Loading composer repositories with package information
Updating dependencies
Lock file operations: 2 installs, 0 updates, 0 removals

Locking bacon/bacon-qr-code (2.0.8)

Locking dasprid/enum (1.0.3)
Writing lock file
Installing dependencies from lock file (including require-dev)
Package operations: 2 installs, 0 updates, 0 removals

Installing dasprid/enum (1.0.3): Extracting archive

Installing bacon/bacon-qr-code (2.0.8): Extracting archive
0/2 [>---------------------------]   0%    Failed to extract dasprid/enum: (7) D:\Lua\5.1\7z.EXE x -bb0 -y C:\inst\vendor\composer\tmp-00ee02c8f4af9f85331dd569cf5cc4fb -oC:\inst\vendor\composer\ef3967dc
The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)
Unzip with 7z command failed, falling back to ZipArchive class
Failed to extract bacon/bacon-qr-code: (7) D:\Lua\5.1\7z.EXE x -bb0 -y C:\inst\vendor\composer\tmp-657cd265e68422b530a8a1796bfb8cdf -oC:\inst\vendor\composer\a6e8c1f9
The archive may contain identical file names with different capitalization (which fails on case insensitive filesystems)
Unzip with 7z command failed, falling back to ZipArchive class
1 package suggestions were added by new dependencies, use composer suggest to see details.
Generating autoload files
No security vulnerability advisories found

You probably need to check this problem with Bacon. Sorry.


fix: google storage bucket storage and bucket prices

Fix #451

 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.

Fixed 2 different bugs. One is related to the issue. Based on this page each region can be transformed to the lowercase to query the API except the multi-regional Europe that its region In the terraform config is EU but europe in the APU.
Another bug was that I think google added a free tier for its operations or I completely missed it when I initially implemented it so it was returning 0 prices.


[Notification] Start notification with command-line interface

What is the purpose of the change
Make notification scripts using cli.
Brief change log
(for example:)

Make notification scripts using cli.

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.

Codecov Report

Merging #210 (8bc259c) into master (09efea0) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #210      +/-   ##
==========================================
- Coverage   65.54%   65.52%   -0.02%
==========================================
  Files         203      203
  Lines       15222    15225       +3
  Branches     2291     2292       +1
==========================================
- Hits         9977     9976       -1
- Misses       4721     4722       +1
- Partials      524      527       +3

Impacted Files
Coverage Δ

...otification_service/cli/commands/server_command.py
92.95% <100.00%> (+0.31%)
:arrow_up:

...ow_plugins/blob_manager_plugins/s3_blob_manager.py
73.61% <0.00%> (-2.78%)
:arrow_down:

ai_flow/store/sqlalchemy_store.py
75.14% <0.00%> (-0.19%)
:arrow_down:

Continue to review full report at Codecov.

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


request to merge.

Hello, hikalkan I'm free today, the question answering system upgrade to the latest version of ABP version, request to merge.

Thank you very much. Once I check changes, I will merge it.


bug: app flickering

If you name an app exactly like this, it will flicker:

GIF was captured at 16 FPS and flickering is much faster & more apparent in real life.

Fixed in #645


refmt incorrects formats curried arguments

external use :
  t => (Request.t => Response.t => Next.t => unit [@bs]) => unit = ""
[@@bs.send];
external static :
  path::string => (Request.t => Response.t => Next.t => unit [@bs]) = ""
[@@bs.module "express"];

is converted to
external use :
  t -> (Request.t -> Response.t -> Next.t -> ((unit)[@bs ])) -> unit = ""
[@@bs.send ]
external static :
  path:string -> Request.t -> Response.t -> Next.t -> ((unit)[@bs ]) = ""
[@@bs.module "express"]

Reproducible case here https://github.com/vramana/Rexpress/tree/e06f9fb14f4e06b8d9288da9da0827555b2c7858
cc @bobzhang

To be clear, this is when converting from re to ml right?
Are you passing the flags -print re or -print ml ?

@jordwalke Yes.
https://github.com/vramana/Rexpress/blob/e06f9fb14f4e06b8d9288da9da0827555b2c7858/package.json#L17

Curious, why are you printing to ml? Btw: That looks like a bug in OCaml's printer (which we use for refmt -print ml). We could always fork the printer and fix the bug too.

@jordwalke I was just mimicing build steps from this repo https://github.com/bsansouci/ReSocket.io/

I wonder if that library does that because it was made when Reason didn't have bucklescript support?

@jordwalke I had to do this because Bucklescript doesn't support bsc -c -bs-files *.ml for .re files. At least at the time that I wrote this (a couple days ago lol)

okay, cc @bobzhang

I would like to support -bs-files for reason too, but I don't have reason parser.
Btw, you don't need print to ml files to get buckle support (CC @chenglou )

@bobzhang Can bs-files support raw binary ASTs, as you said the ocaml compiler can?

or if bs-files supported ml-synonym and pp then it would work too

It is more complicated if we accept raw ASTs, we also need to mark it is applied(or not) the transformations by Buckle( e.g handling those external attributes).
The main purpose of -bs-files is not to replace a build system, but just to make life easier for small to medium sized project(you don't need a build system for most of your projects)
It is nice to have Reason parser as a stand alone self contained library, we can also provide native windows support in that case

Fixed in master. A bug from us. #1523


lowercase all String fields?

I've noticed that queries are case sensitive.
The analyzer we use in the ruby version
"analyzer": {
                "searchable": {
                    "type": "custom",
                    "tokenizer": "whitespace",
                    "filter": ["lowercase", "custom_ascii_folding"]
                },

Full file here: https://github.com/acaprojects/ruby-engine/blob/master/db/es_template.json
This is probably not the best implementation or even a valid one on the latest version of ElasticSearch - but might give us some ideas on improving the tokenisation process.

Fixed by #6

Fixed by #6


In filesystem lib add size_tree to get the size of whole directory

added fielstystem.tree_size and tree_size_hr in 2b8c98f


Improve Magnetometer SNR by Sampling Twice

Looking at the I2C transactions with a scope, it is apparent that there is sufficient time to perform additional reads of the magnetometer during each fusion cycle. Currently one one read is performed, and because the magnetometer portion of the FXOS8700 does not have a FIFO, that read only obtains a single data point.  It should be feasible to perform a magnetometer read before and after the block read of the accelerometer, and possibly also after the gyro read.  Thus there might be 2 or 3 data points per fusion cycle, which would improve the signal to noise ratio if they are averaged.

No - this approach does not noticeably improve the performance of the heading readings. Several long-term stability recordings showed that the standard deviation of readings collected every 1.0 second for about 10 hours was about 0.8 degrees, and this did not change when performing 5 magnetometer readings per fusion cycle and averaging them.
Here's a histogram comparing the 1- and 5-sample approaches:


TensorRT with GroundingDINO

Hi, I am a little stuck on how to use TensorRT to speed up GroundingDINO inference. GroundingDINO takes in both an image and text prompt and I am a bit lost on how to convert the text prompt to tensor. Can someone please give me some example code or suggestions on how to make it work? Thank you!

Hi，I have been stuck here recently. Do you have any demo so far？ Think you！

I am try to transfer grounding-dino to onnx, but it seems like difficult, because of multi-model in Grounding dino Module and NestedTensor

i try to use torch.onnx.export to transfer, but it always problem:
torch.onnx.symbolic_registry.UnsupportedOperatorError: Exporting the operator ::__ior_ to ONNX opset version 13 is not supported
anyone know which part of model make this problem.

any updates here?

I had to disable checkpointing in order to prevent another error, but now stuck at the same place. Seems like the aten::_ior op is not yet supported by torch onnx opset.
https://pytorch.org/docs/stable/onnx_supported_aten_ops.html

I got the same error. Has anyone succeeded in converting to onnx or torchscript model?

here is the onnx I have convert:

it's a difficult way, modify many codes(contains torch framework),just now it's only support fixed shape:

img: 800x800
token: 1x5 ( it's really two words without the head,tail and dot)

and I have speed up performance for the model at this fixed shape

performance killer, I have fix for the fixed shape situation:

nonezero
arange

performance killer, I have fix for the fixed shape situation:

nonezero
arange

hi,can you show me the tokens you used to trace the model or show me the code?

performance killer, I have fix for the fixed shape situation:

nonezero
arange

amazing!!Could you provide the onnx export code. Want to learn from it very much~~~

performance killer, I have fix for the fixed shape situation:

nonezero
arange

hi,can you show me the tokens you used to trace the model or show me the code?

I‘m newer about the algorithm， can you share your code ? and how you change it?

performance killer, I have fix for the fixed shape situation:

nonezero
arange

HIi @oylz ! I met this error:
input: "/transformer/enc_out_class_embed/Unsqueeze_3_output_0"
input: "/transformer/enc_out_class_embed/Unsqueeze_4_output_0"
input: "/transformer/enc_out_class_embed/Unsqueeze_5_output_0"
input: "/transformer/enc_out_class_embed/Constant_7_output_0"
output: "/transformer/enc_out_class_embed/Slice_output_0"
name: "/transformer/enc_out_class_embed/Slice"
op_type: "Slice"
[07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:774: --- End node ---
[07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:776: ERROR: builtin_op_importers.cpp:4493 In function importSlice:
[8] Assertion failed: (axes.allValuesKnown()) && "This version of TensorRT does not support dynamic axes."=====would you give me some advices?

here is the onnx I have convert:

it's a difficult way, modify many codes(contains torch framework),just now it's only support fixed shape:

img: 800x800
token: 1x5 ( it's really two words without the head,tail and dot)

and I have speed up performance for the model at this fixed shape

Batch Inference?

performance killer, I have fix for the fixed shape situation:

nonezero
arange

can you please share your code to export grounding-dino to onnx?

here is the onnx I have convert:

it's a difficult way, modify many codes(contains torch framework),just now it's only support fixed shape:

img: 800x800
token: 1x5 ( it's really two words without the head,tail and dot)

and I have speed up performance for the model at this fixed shape

Can you provide an infer.py for test your onnx?

@oylz
Hello, I don't know if you can show your code example of converting onnx, I want to learn your code. Thank you so much

@oylz
Hello, could you please show your onnx converting code? Thanks a lot.

Hello, I found onnx convertor for GrondingDino model here
https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino
as aт intermediate step to ge in convert to openvino model: https://github.com/wenyi5608/GroundingDINO/blob/main/demo/export_openvino.py
I get gronded.onnx and think how to convert it into tensorRT now

Hello, I found onnx convertor for GrondingDino model here https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino as an intermediate step in converting to openvino model: https://github.com/wenyi5608/GroundingDINO/blob/main/demo/export_openvino.py
I get grounded.onnx and think how to convert it into tensorRT now

Hello, I tried using this as well, but I was not able to successfully export it. Could you please tell me about your environment setup? I'm not sure if the failure was due to a version issue. Thank you!

my local env are:
pip freeze --local
addict==2.4.0
certifi==2023.7.22
charset-normalizer==3.2.0
cmake==3.27.5
coloredlogs==15.0.1
contourpy==1.1.1
cycler==0.11.0
defusedxml==0.7.1
filelock==3.12.4
flatbuffers==23.5.26
fonttools==4.42.1
fsspec==2023.9.2
-e git+https://github.com/wenyi5608/GroundingDINO.git@a3256bbca7fd365d1240d1830eaf8c13987e666d#egg=groundingdino
huggingface-hub==0.17.3
humanfriendly==10.0
idna==3.4
importlib-metadata==6.8.0
Jinja2==3.1.2
jstyleson==0.0.2
kiwisolver==1.4.5
lit==17.0.1
MarkupSafe==2.1.3
matplotlib==3.8.0
mpmath==1.3.0
networkx==3.1
numpy==1.26.0
nvidia-cublas-cu11==<IP_ADDRESS>
nvidia-cuda-cupti-cu11==11.7.101
nvidia-cuda-nvrtc-cu11==11.7.99
nvidia-cuda-runtime-cu11==11.7.99
nvidia-cudnn-cu11==<IP_ADDRESS>
nvidia-cufft-cu11==<IP_ADDRESS>
nvidia-curand-cu11==<IP_ADDRESS>
nvidia-cusolver-cu11==<IP_ADDRESS>
nvidia-cusparse-cu11==<IP_ADDRESS>
nvidia-nccl-cu11==2.14.3
nvidia-nvtx-cu11==11.7.91
onnx==1.14.1
onnxruntime==1.16.0
opencv-python==<IP_ADDRESS>
openvino==2023.1.0.dev20230811
openvino-dev==2023.1.0.dev20230811
openvino-telemetry==2023.1.1
packaging==23.1
Pillow==10.0.1
platformdirs==3.10.0
protobuf==4.24.3
pycocotools==2.0.7
pyparsing==3.1.1
python-dateutil==2.8.2
PyYAML==6.0.1
regex==2023.8.8
requests==2.31.0
safetensors==0.3.3
scipy==1.10.1
six==1.16.0
supervision==0.6.0
sympy==1.12
texttable==1.6.7
timm==0.9.7
tokenizers==0.13.3
tomli==2.0.1
torch==2.0.1
torchaudio==2.0.2
torchvision==0.15.2
tqdm==4.66.1
transformers==4.33.2
triton==2.0.0
typing_extensions==4.8.0
urllib3==2.0.5
yapf==0.40.2
zipp==3.17.0
I make all  like  described in   https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino
and get
-rw-rw-r-- 1 cnn cnn 694130127 сен 27 17:58 grounded.onnx

Hello, I found onnx convertor for GrondingDino model here https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino as an intermediate step in converting to openvino model: https://github.com/wenyi5608/GroundingDINO/blob/main/demo/export_openvino.py
I get grounded.onnx and think how to convert it into tensorRT now

hello, I want to use onnx by using gpu runtime
have you ever work gpu with your onnx model ?
i encountered this error when i try
/usr/local/lib/python3.10/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py:69: UserWarning: Specified provider 'CUDAExecutionProvider' is not in available provider names.Available providers: 'AzureExecutionProvider, CPUExecutionProvider

my local env are:
pip freeze --local
addict==2.4.0
certifi==2023.7.22
charset-normalizer==3.2.0
cmake==3.27.5
coloredlogs==15.0.1
contourpy==1.1.1
cycler==0.11.0
defusedxml==0.7.1
filelock==3.12.4
flatbuffers==23.5.26
fonttools==4.42.1
fsspec==2023.9.2
-e git+https://github.com/wenyi5608/GroundingDINO.git@a3256bbca7fd365d1240d1830eaf8c13987e666d#egg=groundingdino
huggingface-hub==0.17.3
humanfriendly==10.0
idna==3.4
importlib-metadata==6.8.0
Jinja2==3.1.2
jstyleson==0.0.2
kiwisolver==1.4.5
lit==17.0.1
MarkupSafe==2.1.3
matplotlib==3.8.0
mpmath==1.3.0
networkx==3.1
numpy==1.26.0
nvidia-cublas-cu11==<IP_ADDRESS>
nvidia-cuda-cupti-cu11==11.7.101
nvidia-cuda-nvrtc-cu11==11.7.99
nvidia-cuda-runtime-cu11==11.7.99
nvidia-cudnn-cu11==<IP_ADDRESS>
nvidia-cufft-cu11==<IP_ADDRESS>
nvidia-curand-cu11==<IP_ADDRESS>
nvidia-cusolver-cu11==<IP_ADDRESS>
nvidia-cusparse-cu11==<IP_ADDRESS>
nvidia-nccl-cu11==2.14.3
nvidia-nvtx-cu11==11.7.91
onnx==1.14.1
onnxruntime==1.16.0
opencv-python==<IP_ADDRESS>
openvino==2023.1.0.dev20230811
openvino-dev==2023.1.0.dev20230811
openvino-telemetry==2023.1.1
packaging==23.1
Pillow==10.0.1
platformdirs==3.10.0
protobuf==4.24.3
pycocotools==2.0.7
pyparsing==3.1.1
python-dateutil==2.8.2
PyYAML==6.0.1
regex==2023.8.8
requests==2.31.0
safetensors==0.3.3
scipy==1.10.1
six==1.16.0
supervision==0.6.0
sympy==1.12
texttable==1.6.7
timm==0.9.7
tokenizers==0.13.3
tomli==2.0.1
torch==2.0.1
torchaudio==2.0.2
torchvision==0.15.2
tqdm==4.66.1
transformers==4.33.2
triton==2.0.0
typing_extensions==4.8.0
urllib3==2.0.5
yapf==0.40.2
zipp==3.17.0
I make all like described in https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino
and get -rw-rw-r-- 1 cnn cnn 694130127 сен 27 17:58 grounded.onnx

Thanks!!!!!

here is the onnx I have convert:

it's a difficult way, modify many codes(contains torch framework),just now it's only support fixed shape:

img: 800x800
token: 1x5 ( it's really two words without the head,tail and dot)

and I have speed up performance for the model at this fixed shape

hi  i want to download the onnx model ,but the google cloud need the right to download  ,i request some times,but not yet got the permission,so how to download the onnx model directly or can you agree my request on goole cloud my username is<EMAIL_ADDRESS>

Hello, I found onnx convertor for GrondingDino model here https://blog.openvino.ai/blog-posts/enable-openvino-tm-optimization-for-groundingdino as an intermediate step in converting to openvino model: https://github.com/wenyi5608/GroundingDINO/blob/main/demo/export_openvino.py
I get grounded.onnx and think how to convert it into tensorRT now

hello, I want to use onnx with using gpu have you ever work gpu with your onnx model ? i encountered this error when i try
/usr/local/lib/python3.10/site-packages/onnxruntime/capi/onnxruntime_inference_collection.py:69: UserWarning: Specified provider 'CUDAExecutionProvider' is not in available provider names.Available providers: 'AzureExecutionProvider, CPUExecutionProvider

Hi @minuenergy , dose the onnx model you got support the inputs with dynamic scales? I got a bug when modifying the size of the input image.

performance killer, I have fix for the fixed shape situation:

nonezero
arange

HIi @oylz ! I met this error: input: "/transformer/enc_out_class_embed/Unsqueeze_3_output_0" input: "/transformer/enc_out_class_embed/Unsqueeze_4_output_0" input: "/transformer/enc_out_class_embed/Unsqueeze_5_output_0" input: "/transformer/enc_out_class_embed/Constant_7_output_0" output: "/transformer/enc_out_class_embed/Slice_output_0" name: "/transformer/enc_out_class_embed/Slice" op_type: "Slice" [07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:774: --- End node --- [07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:776: ERROR: builtin_op_importers.cpp:4493 In function importSlice: [8] Assertion failed: (axes.allValuesKnown()) && "This version of TensorRT does not support dynamic axes."=====would you give me some advices?

I met the same problem, is there a solution now?

performance killer, I have fix for the fixed shape situation:

nonezero
arange

HIi @oylz ! I met this error: input: "/transformer/enc_out_class_embed/Unsqueeze_3_output_0" input: "/transformer/enc_out_class_embed/Unsqueeze_4_output_0" input: "/transformer/enc_out_class_embed/Unsqueeze_5_output_0" input: "/transformer/enc_out_class_embed/Constant_7_output_0" output: "/transformer/enc_out_class_embed/Slice_output_0" name: "/transformer/enc_out_class_embed/Slice" op_type: "Slice" [07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:774: --- End node --- [07/10/2023-12:45:07] [E] [TRT] ModelImporter.cpp:776: ERROR: builtin_op_importers.cpp:4493 In function importSlice: [8] Assertion failed: (axes.allValuesKnown()) && "This version of TensorRT does not support dynamic axes."=====would you give me some advices?

I met the same problem, is there a solution now?

My error is :
[12/15/2023-10:39:37] [E] [TRT] ModelImporter.cpp:726: While parsing node number 3432 [Slice -> "onnx::Slice_3626"]:
[12/15/2023-10:39:37] [E] [TRT] ModelImporter.cpp:727: --- Begin node ---
[12/15/2023-10:39:37] [E] [TRT] ModelImporter.cpp:728: input: "onnx::Slice_3616"
input: "onnx::Slice_22788"
input: "onnx::Slice_3622"
input: "onnx::Slice_22789"
input: "onnx::Slice_3625"
output: "onnx::Slice_3626"
name: "Slice_3432"
op_type: "Slice"

[12/15/2023-10:39:37] [E] [TRT] ModelImporter.cpp:729: --- End node ---
[12/15/2023-10:39:37] [E] [TRT] ModelImporter.cpp:732: ERROR: builtin_op_importers.cpp:4531 In function importSlice:
[8] Assertion failed: (axes.allValuesKnown()) && "This version of TensorRT does not support dynamic axes."

Hi, I am a little stuck on how to use TensorRT to speed up GroundingDINO inference. GroundingDINO takes in both an image and text prompt and I am a bit lost on how to convert the text prompt to tensor. Can someone please give me some example code or suggestions on how to make it work? Thank you!

Brother, have you transferred it out? GroundingDINO to Tensorrt

Hi, I am a little stuck on how to use TensorRT to speed up GroundingDINO inference. GroundingDINO takes in both an image and text prompt and I am a bit lost on how to convert the text prompt to tensor. Can someone please give me some example code or suggestions on how to make it work? Thank you!
I have successfully transferred the engine file, but the output accuracy and pytorch cannot be aligned. May I ask if you have resolved it?

Hi, I am a little stuck on how to use TensorRT to speed up GroundingDINO inference. GroundingDINO takes in both an image and text prompt and I am a bit lost on how to convert the text prompt to tensor. Can someone please give me some example code or suggestions on how to make it work? Thank you!

Brother, have you transferred it out? GroundingDINO to Tensorrt

哥们儿请问你现在有tensorrt的G-dino了吗？

Hey there! I have an onnxruntime implementation below with a tensorrt execution provider backend at the link below thanks to the foundation built by @wuqingzhou828, and some optimizations, simplifications. It's stripped of the need for the the huggingface transformers library too to try to push for lightweight dependencies.
In all honesty a FP16 engine is only roughly 2x faster (working on rigourous benchmarks) on an Ampere GPU because of the unsupported ops, and adding an automatic mixed precision context to the pytorch model is a lot faster at around 3x right now.
I'm working on building out BERT as a separate inital text encoder stage now so we can mess around with different calibrations and platforms. The second stage can then stay in pytorch with amp or compile etc and we'll get the best of both worlds!
https://github.com/rhysdg/vision-at-a-clip
https://github.com/rhysdg/vision-at-a-clip?tab=readme-ov-file#grounding-dino

Has anybody the onnx checkpoint for the grounding dino and inference code of it??
TIA

Good afternoon. Could some kind person please share the ONNX model or the code to perform the conversion?


[BotBuilder-Core] ActivityHandler class does not match C# counterpart

Versions
What package version of the SDK are you using.
<EMAIL_ADDRESS>What nodejs version are you using
12.16.1
What browser version are you using
-
What os are you using
Windows 10 Pro
Describe the bug
Give a clear and concise description of what the bug is.
The JavaScript implementation of ActivityHandler contains different method's implementation compared to C#. The main difference we found is in the onConversationUpdateActivity method.
To Reproduce
Steps to reproduce the behavior:

Clone BotBuilder-DotNet
Navigate to /libraries/Microsoft.Bot.Builder
Open ActivityHandler.cs
Scroll down to line 172
Clone BotBuilder-JS
Navigate to /libraries/botbuilder-core/src/
Open activityHandler.ts
Scroll down to line 664
See the differences

Expected behavior
Give a clear and concise description of what you expected to happen.
I expect ActivityHandler in JavaScript to use the same implementation as in C#.
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.
Using Direct Line Channel, the Virtual Assistant and the Skill receives the ConversationUpdate Activity by duplicated sending the welcome message by duplicated as well.
Reviewing the TypeScript bots present in BotBuilder-Samples, we noticed that the logic that should be present in the ActivityHandler > dispatchConversationUpdateActivity, is present in the membersAdded event of the bots. See example.
See related information:

Microsoft Bot displays unnecessary duplicated messages in WebChat?
Identical Threads in Microsoft Bot are Duplicating Welcoming Message

This differing behavior was shipped in the initial implementation, to avoid breaking changes for customers we've left the ActivityHandler behavior as is. However, the expected C# logic was implemented in the ActivityHandlerBase class and could be reimplemented in a subclass of ActivityHandler. Alternatively, @Batta32 you could directly subclass ActivityHandlerBase which would make the VA Template/Solution more aligned. (No event listeners, all desired methods must be implemented in a subclass)
https://github.com/microsoft/botbuilder-js/blob/150dfe563c9d3132bec55c4604503bceb462ff78/libraries/botbuilder-core/src/activityHandler.ts#L84
https://github.com/microsoft/botbuilder-js/blob/150dfe563c9d3132bec55c4604503bceb462ff78/libraries/botbuilder-core/src/activityHandlerBase.ts#L116-L134

Thanks @stevengum for the detailed information! That's a great idea, however, we are following a parity with the C# bots extending the DefaultActivityHandler to TeamsActivityHandler (VA) and ActivityHandler (Skill).
We think that the best approach is to add the same logic that the TypeScript samples bots contain in the onMembersAdded method as they are extending to the ActivityHandler and not to the ActivityHandlerBase class.
Applying this we can have a parity between C# VA/Skill - TypeScript VA/Skill and also, parity with the BotBuilder-Samples bots.

Thanks for filing this issue and providing additional information!
Given that the TypeScript samples should mirror the behavior of the JS samples, the team will need to have a discussion whether or not to change the JS & TS samples' behavior to match C#. In the meantime, I'm closing this issue as a wontfix because these changes in the SDK would be breaking.


Replace XmonDevice from placement

XmonDevice is currently used by the placement code, which is in core.  A suitable way to specify the geometry for devices outside of cirq.google should be made so that we don't have core cirq depending on cirq.google

The number of issues on how the line place should really work grow, with more and more conflicting each other.
In #751 we agreed with @Strilanc to move line code inside the cirq.google, so it can freely depend on Xmon* classes. This move would make placement code work only within cirq.google and limit its behavior. After discussion with @dabacon we came to conclusion that we might want to rethink this decision.
In #95 I proposed a set of interface methods that grid or graph device would need to support in order for the placement code to be implementable. This would keep line placement generic. One current problem with that approach right now is that we have only Device and XmonDevice that implements it. We don't have any GridDevice or GraphDevice, where connectivity concepts that I'm about to introduce would have a meaning. To make placement generic we should probably have it.

I'm going to close this. We now have two placement/routing codes in Cirq, one in Google and one in contrib.  I think we should think about whether a general location is suitable.  Of course there are also a lot of algorithms to explore in this space, so we would likely want to leverage previous algorithm work.


QuasiQuote parse error on input

Using the package: https://hackage.haskell.org/package/neat-interpolation-<IP_ADDRESS>/docs/NeatInterpolation.html
With code:
f :: T.Text
f =
  [text|
    function(){
      function(){
      }
    }
  |]

it shows the error:
/tmp/haskell-lsp59186/Debian.hs-00000-4856614137838069181.hs:107:15: error:
    parse error on input `('
    [text|
      function(){
> function(){
        }
      }

Seems like you need to enable QuasiQuotes extension.
Try hlint -X QuasiQuotes .  or use {-# LANGUAGE QuasiQuotes #-} pragma.

@jorpic this issue is not related to the code, the code compiles and the language extension is enabled in the .cabal file.
I am using the Haskell IDE Engine so I am not calling HLint myself.

An issue for the Haskell IDE Engine team then - it seems they aren't passing QuasiQuotes from the .cabal file properly.

@ndmitchell Thank you, I passed that to them.

This was not fixed in the IDE engine before archival.
I'm seeing the same error with QuasiQuotes in the extensions in the .cabal file.


SOCKS5 proxy worked but the regex verification only accept "https"

VSCode Version: 1.33.0
OS Version: Win10

Steps to Reproduce:

Go to Settings User Settings Application Proxy proxy
Input a SOCKS5 proxy like socks5://<IP_ADDRESS>:8080
The SOCKS5 proxy actually worked.
The problem is the regex verification only accept https

Related Issues: #58080 #27838 #9971
These Issues are locked and I can't make a  comment. This is really inconvenience.
@joaomoreno

The ProxyResolver and vscode-proxy-agent will need to add support for SOCKS5. https://www.npmjs.com/package/socks5-client might help.


The Clarity Icon under Custom Triggers does not receive keyboard focus

[Clarity Version]
3.1.1
[Test URL]
https://clarity.design/documentation/signposts#examples
[WCAG 2.1 Guideline]
2.1.1 Keyboard
(Level A)
https://www.w3.org/TR/WCAG21/#keyboard
[OS]
MacOS 10.15.3, Windows 10
[User Agents/Assistive Technologies]
Safari 13.0.5, Firefox 75.0, Switch
[Component]
Signposts
[Description]
The Clarity Icon under Custom Triggers does not receive keyboard focus
[User Impact]
Switch users cannot navigate to the Clarity Icon via keyboard and must simulate a mouse click.
[Steps to Reproduce]
See the following instructions for switch usage then follow the steps below.
Windows (see Scan through keys)
https://support.microsoft.com/en-us/help/10762/windows-use-on-screen-keyboard
Mac
https://support.apple.com/en-us/HT202865

Using a switch, try navigating to the Clarity Icon with the Tab key.
For MacOS only:
Navigate to and activate Pointer --> Mouse Click.
Select the vertical area of the click, and then the vertical coordinate of the icon.
Select the horizontal area of the click, and then the horizontal coordinate of the icon.

[Recommendation]
The CLR button does not have a tabindex attribute, and is not a natively interactive element. Adding a tabindex attribute with a value of "0" and updating the role attribute to be "button" will both ensure this element receives focus, but also that its role is properly announced to users.	<clr-icon _ngcontent-clarity-c224="" shape="avatar" clrsignposttrigger="" class="signpost-trigger is-solid has-badge-info" role="button" aria-label="Signpost Toggle" aria-expanded="false" aria-controls="clr-id-1" tabindex="0">...</clr-icon>

also needs to open and close the popup.

Hi there, this is an automated message.
In an effort to dedupe and track accessibility reports consistently, this issue is being closed and tracked internally with our Accessibility team. It will still be triaged and addressed as part of ongoing work.


add CLIP-ViT-L-scope

Thanks for the super quick feedback, I have tried to make the names more consistent as suggested, I have also changed the repo to point to the github repo for using the files (clipscope) rather than the one I used to train the models (vitsae).  The clipscope repo is much more likely to be used by others in future than vitsae.
I seem to be able to access the clipscope pypi package (https://pypi.org/project/clipscope/) and pip install clipscope from the usage section works for me, does it fail for you?
In terms of tracking, these models are all derived from the same training process and a common use case for them should be to compare how they perform against one another. In a sense collectively they comprise a single tool. https://huggingface.co/google/gemma-scope-2b-pt-res from which we took inspiration takes a similar approach and places large numbers of related models together as a single "model".
Personally I am most interested in the extent to which people are using the tool in any respect than I am in which parts of it are being used most. I would say that probably the most common usage will to be to follow the usage code and just download that one model and play around with it a bit.

apologies for the delay, accepted the suggested commit and changes made to the readme https://huggingface.co/lewington/CLIP-ViT-L-scope/blob/main/README.md
I think we should be ok to merge now

aah, I should have picked that up, much obliged


Consider adding another maintainer and cargo owner

With bzm3r no longer involved in the Bevy space and @StarArawn being quite busy, we could use another Maintainer.
I think I'm the only other user with permissions on this repo, and things are set up in such a way that I can press the merge button only with a GH approval from a user with write access.
This is making it very difficult to get work merged, or to be motivated to help get work into a mergable state.
Additionally, we haven't had an official Cargo release targeting Bevy 0.13 or Bevy 0.14, which is making it difficult to maintain crates like bevy_ecs_ldtk which depend on bevy_ecs_tilemap.
I am proposing that @ChristopherBiscardi be added as a maintainer and cargo-owner of the crate, and if possible, to formalize some sort of process for getting PRs reviewed and merged.

I'm okay with this. Ultimately though the goal for this repo is for it to be merged directly into bevy. I've done a little bit of work to facilitate this but its not super clear how to start the process.

I'm comfortable with taking on maintenance and between @rparrett and myself we can probably figure out what the path to upstreaming looks like. My guess is it turns into a working group of some kind, if upstream is amenable.

I've been added and have merged code, so going to close this.

Thanks @StarArawn !


SoapHeadersNotPermitted does not exist in some suds releases

Hello,
This exception does not exist in suds release 0.7 which is being use in Debian distributions. According to the code there's no replacement so I turned it optional.
Regards, Adam.

I also created Debian's package available here:
https://cloud.le-vert.net/public.php?service=files&t=7f106d986b07256ba7efcf88e26788f6
They need suds-passworddigest package from here:
https://cloud.le-vert.net/public.php?service=files&t=665bdfb0abe467cf6b6a28d50370dc65

Thanks a lot for your pr. But before I merge this pr, I need test it with suds-0.7.  😄


Use lodash collection manipulations such as map/each

Example json file loaded to the db
{
 "shelves":
[
  {
    shelveId: 0,
    books: [{...},{...},{...}]
  },
  {
    shelveId: 1,
    books: [{...},{...}]
  }
]
}

then i want to get each shelve Id with number of books (count).
Doing this in chaining lodash functions i could not find a way... any hints?
The end result i wish to get is this:
{
  "0":  3,
  "1": 2
}

Thanks!

To get your desired result no lodash function is required. You can use  array reduce
var obj = {
   "shelves": [
    {
      shelveId: 0,
      books: [{name: 'a'},{name:'b'},{name:'c'}]
    },
    {
      shelveId: 1,
      books: [{name: 'i'},{name:'j'}]
    },
    {
      shelveId: 2,
      books: [{name: 'x'},{name:'y'},{name:'z'},{name:''}]
    }
  ]
}

var counts = obj.shelves.reduce(function(acc, item){
  return Object.assign(acc, {[item.shelveId]: item.books.length});
}, {});

console.log(counts); // output 1
console.log(JSON.stringify(counts)); // stringify object

output
// output 1 will be an object
{
  0: 3,
  1: 2,
  2: 4
}

// stringify result
{"0":3,"1":2,"2":4}


Missing jobs?

Could you please explain where we can find two referenced jobs and COBOL apps:
Job	Program	Function
CBTRNLDJ	CBTRNLDC	Load Transactions
CBTRNRPJ	CBTRNRPC	Reports

Pepito,
The 2 jobs named in the list are not required in this release of card demo.
We are updating the list of JCLs in the documentation to reflect the current inventory

Revision to README file done and merged into main


allow binding to LAN address instead of <IP_ADDRESS> for zeroconf

It seems like librespot binds to all interfaces (<IP_ADDRESS>) by default for zeroconf discovery.
Pardon my ignorance but does it need to bind to all interfaces for zeroconf discovery? Would it be possible to bind it to the assigned LAN address (such as <IP_ADDRESS>) and still have the Spotify Discovery feature (which is what zeroconf is used for?) work as expected? If yes, can we get an option to change the address to which librespot binds to for zeroconf?
I've raised a similar issue in the spotifyd repository as well.

Duplicate of #486.


Raw Code View UTF8 Problem

Description
My code is filled with turkish comments and when i tried to view the code via clicking the 'raw' button, characters are garbled. No problem on default viewing though. Thank you...

Thanks your feedback!
Can you make up an example on GitHub so I can see how GitHub treats this situation?

Hello, you can see the github repo file at this address:
https://github.com/Kambaa/gogs-raw-code-test

My server has the same exact file and it prints raw data like this:

Got it, thanks.

Hmm? I made a test repo on https://try.gogs.io/unknwon/issue-2888/raw/master/test-code-with-turkish-comments.php, and if I visit, I can see utf-8 characters correctly. Latest chrome...

@Unknwon For me this file is displayed with incorrect characters. I tested with Latest Firefox, and Chromium (on Ubuntu).

Maybe latest Chrome is a bit smarter :sweat_smile:

I'll try to fix this, but not today :)

Latest chrome

The fix is now available on https://try.gogs.io
My example file: https://try.gogs.io/moltam/utf8-test/raw/master/README.md
I think we can close this issue now.

Nice! looks good.

ping @Kambaa


If have a log file open and the file gets removed plugin throws error.

Steps to Reproduce:

open file with view-tail-large-files
delete file externally, view error.

What should happen (IMO):

open file with view-tail-large-files
delete file externally, text view goes blank except first line is "File does exist"
recreate file, text view updates to show file.

why this effects me, when I run my program its first step is to delete/achrive the old log file, it then after a little bit creates a new one and starts to write to it. I would like it to not throw and error nor create an alert, but if i want to view the log tab, and the file is no existant it will tell me so.

I just published version 1.0.7 to help with this problem.  Now when a file disappears the tab also disappears, which is what Atom normally does.  Then for 10 minutes it checks once a second to see if a file with the same path appears.  If it does then that new file is opened in a new tab.
Let me know if this works for you.  I couldn't give you exactly what you asked for without major changes.


Uncaught TypeError: patch.getChanges is not a function

Was working on (existing, gfm) document for some time
Error occurred on hitting enter (not for the first time)

Now every time causes this error. Pasting (or vim putting) a \n character is fine, though; it's the actual keypress that is triggering a (attempted) call to patch.getChanges. (That seemed to be true for a while; no longer the case.)
It's now also happening with other characters; mostly space.
Seemingly related (I imagine - without looking at the source - it needs the result of getChanges) the cursor is now positioned after e.g. a closing paren, rather than inside the pair, on typing an open.
Atom Version: 1.7.0-beta0
System: Mac OS X 10.11.3
Thrown From: Atom Core
Stack Trace
Uncaught TypeError: patch.getChanges is not a function
At /opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:1333

TypeError: patch.getChanges is not a function
    at TextBuffer.module.exports.TextBuffer.emitDidChangeTextEvent (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:1333:60)
    at TextBuffer.module.exports.TextBuffer.transact (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/text-buffer.js:843:12)
    at TextEditor.module.exports.TextEditor.transact (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/text-editor.js:1274:26)
    at atom-text-editor.newCommandListeners.(anonymous function) (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/register-default-commands.js:648:22)
    at CommandRegistry.module.exports.CommandRegistry.handleCommandEvent (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/command-registry.js:260:29)
    at /opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/command-registry.js:3:61
    at KeymapManager.module.exports.KeymapManager.dispatchCommandEvent (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:570:16)
    at KeymapManager.module.exports.KeymapManager.handleKeyboardEvent (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/node_modules/atom-keymap/lib/keymap-manager.js:378:22)
    at WindowEventHandler.module.exports.WindowEventHandler.handleDocumentKeyEvent (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/window-event-handler.js:98:36)
    at HTMLDocument.<anonymous> (/opt/homebrew-cask/Caskroom/atom-beta/1.6.0-beta3/Atom Beta.app/Contents/Resources/app.asar/src/window-event-handler.js:3:61)

Commands
     -5:38.1.0 spell-check:correct-misspelling (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  3x -5:34.2.0 core:move-up (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
     -5:08.9.0 autocomplete-plus:confirm (atom-text-editor.editor.vim-mode.insert-mode.is-focused.autocomplete-active)
     -5:08.6.0 editor:newline (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  2x -5:08.2.0 core:backspace (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
     -5:07.8.0 editor:newline (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  3x -4:48.2.0 core:backspace (atom-text-editor.editor.vim-mode.insert-mode.is-focused.autocomplete-active)
  2x -3:38.3.0 editor:newline (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  2x -3:25.8.0 core:backspace (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  3x -3:19.1.0 core:move-right (atom-text-editor.editor.vim-mode.insert-mode.is-focused.autocomplete-active)
  2x -3:17.2.0 core:move-down (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
     -2:09.3.0 pdf-view:zoom-out (div.pdf-view)
     -1:26.6.0 core:move-up (atom-text-editor.editor.vim-mode.insert-mode.is-focused)
  4x -0:52.9.0 core:backspace (atom-text-editor.editor.vim-mode.insert-mode.is-focused.autocomplete-active)
     -0:03.5.0 core:move-right (atom-text-editor.editor.vim-mode.insert-mode.is-focused.autocomplete-active)
  2x -0:02.3.0 editor:newline (atom-text-editor.editor.vim-mode.insert-mode.is-focused)

Config
{
  "core": {
    "closeEmptyWindows": false,
    "destroyEmptyPanes": false,
    "disabledPackages": [
      "markdown-scroll-sync"
    ],
    "excludeVcsIgnoredPath": true,
    "projectHome": "~",
    "themes": [
      "atom-dark-ui",
      "solarized-dark-syntax"
    ]
  },
  "editor": {
    "autoIndent": true,
    "fontFamily": "Anonymous Pro for Powerline",
    "fontSize": 11,
    "invisibles": {},
    "showIndentGuide": true,
    "showInvisibles": true,
    "showLineNumbers": true,
    "softTabs": true,
    "softWrap": true,
    "softWrapAtPreferredLineLength": true,
    "tabLength": 4,
    "tabType": "auto"
  }
}

Installed Packages
# User
markdown-preview-plus, v2.2.2
pdf-view, v0.50.0
tabs-to-spaces, v1.0.2
term3, v0.21.4
vim-mode, v0.65.0

# Dev
No dev packages

Same case here (new markdown document). Every newline suddenly caused that error. Closing the document and  opening it again made it go away, if at least temporarily.

@joushou Are you also using vim-mode?

No vim-mode. If we ignore my syntax theme, go-plus and its dependencies, I'm working from a completely stock atom.

I've got the same thing happening, no vim-mode either, editing a markdown file.

Same error, no vim-mode. I was editing a shell script.

It shows up after a while when typing markdown files rather consistently, and doesn't only show up on return. It is incredibly annoying, as the popups fill your screen.
After a short bit of debugging, the issue appears to be here:
(text-buffer.js:840)
      compactedChanges = this.history.groupChangesSinceCheckpoint(checkpointBefore, endMarkerSnapshot, true);
      this.history.applyGroupingInterval(groupingInterval);
      this.emitMarkerChangeEvents(endMarkerSnapshot);
      this.emitDidChangeTextEvent(compactedChanges);

this.history.groupChangesSinceCheckpoint returns false, which this.emitDidChangeTextEvent tries to call "getChanges" on, which obviously fails.
The reason this.history.groupChangesSinceCheckpoint returns zero, is that this.undoStack is an array of 0 length, skipping the checkpointIndex search, and returning false at line history.js:131.
I have quickly hacked out the call to patch.getChanges in emitMarkerChangeEvents in the debugger when a "false" patch is provided, but I suspect that the real issue lies in the fact that my undo buffer is empty at this point, which I suspect was not the intention. After all, I made a change (a newline), and should therefore have something to undo. If no other progress is made, I might try to dive a little deeper, but I have no effin' clue how these parts of Atom work.
For now, I shall retreat to my static type systems.

Is there a workaround for this? I am now getting it with every keystroke, and the undo queue is indeed empty (Ctrl+Z does nothing) as joushou says.
Disabling the Markdown preview doesn't seem to help much.

The workaround is to reopen the affected file, which works for a while.
No better patch is currently available. We need to figure out why the undo buffer dies.

After leaving my first comment I ended up closing and reopening Atom, and the problem has indeed not come back.

It keeps coming back for me, however.
Has anyone tried a recent master? I don't have a suitable toolchain.

Running on 1.7.beta started from CLI.
Editing markdown file (now 180 lines, will be probably 200 when I am finished).
Installed packages:

go-plus + deps
git-blame
git-diff-popup
git plus
linter-swagger
minimap
minimap-autohide
minimap-hide
minimap-(git|split)-diff

Also no vim-mode...

This will be fixed soon in 1.7.0-beta3. Keep in mind if you're having trouble on beta in the future, you can always fall back to the stable channel until your issue is resolved. Thanks for your help everyone.


.azdo

Hi, shouldn't it be .azdo/pipelines ?

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

ID: 4f953ea6-ae2f-b258-3eac-e86a12398d5a
Version Independent ID: 4f953ea6-ae2f-b258-3eac-e86a12398d5a
Content: Configure a pipeline and push updates using GitHub Actions or Azure DevOps
Content Source: articles/azure-developer-cli/configure-devops-pipeline.md
Service: azure-dev-cli
GitHub Login: @alexwolfmsft
Microsoft Alias: alexwolf

Thank you for raising this issue - this will be fixed soon in an upcoming PR.

#please-close


Floating Widget improvements

Hi, thank you for the the great app.
Could you please make a few enchancements to the floating widget functionality?

Add floating widget to a watch face (similar to Juggluco floating glucose).
Add floating widget to Always On screen for Galaxy phones.
Make an option to fix a position of the floating widget.
Thanks a lot for your work.

Hi, thank you for the the great app. Could you please make a few enchancements to the floating widget functionality?

Add floating widget to a watch face (similar to Juggluco floating glucose).

Wear OS only support the permission using adb. So the most user can not use it.
I also see no real advantage. So maybe if I have some boredom.

Add floating widget to Always On screen for Galaxy phones.

This is no floating widget. For Samsung there is already an app bringing widgets to AOD.
But I also thinking about this feature, but it has no prio...

Make an option to fix a position of the floating widget.

Do you really need it? I always have the floating widget at the same position.
What do you expect? Always the same start position or complete fix position?

Thanks a lot for your work.

Thank you for your feedback.
Michael

This is why I like and prefer Floating Widget to complications:

Customizable size, I need at least font size 14 for my eyes.

As of fixing a widget position - this is for a pure convenience to prevent it from accident move with a "fat finger".
Thank you.


[QUESTION]Longhorn csi plugin reports errors

Question
Longhorn csi plugin cyclically reports errors
Environment:

Longhorn version: v1.1.1
Kubernetes version: v1.20.7+k3s1
Node config

OS type and version ubuntu 20.04
CPU per node: master01=16C master02=16C  master03=16C
Memory per node: master01= 32G ,  master02= 33G, master03= 64G
Disk type :master01 = ssd , master02 = sata, master03= sata
Network bandwidth and latency between the nodes:  master01 ->master02  2ms     master01 ->master03  0.545ms  master02 ->master01  2ms  master02 ->master03 1ms  master03 ->master01  0.553ms   master03 ->master02  2ms

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal):  kvm

Additional context

root@k3s-master01:~# kubectl logs -n longhorn-system longhorn-csi-plugin-hgstc longhorn-csi-plugin
2021/05/24 08:42:27 proto: duplicate proto type registered: VersionResponse
time="2021-05-24T08:42:27Z" level=info msg="CSI Driver: driver.longhorn.io csiVersion: 0.3.0, manager URL http://longhorn-backend:9500/v1"
time="2021-05-24T08:42:37Z" level=fatal msg="Error starting CSI manager: Failed to initialize Longhorn API client: Get "http://longhorn-backend:9500/v1\": context deadline exceeded (Client.Timeout exceeded while awaiting headers)"
Add any other context about the problem here.

I just saw a similar issue https://github.com/longhorn/longhorn/issues/1478 I think I have enough memory to run longhorn , can anyboby help me to resolv it?

Hi! Can you check if longhorn-manager is running on each node? Error message exceed dateline mean there are some issue to reach it.  Are others Longhorn process properly running? Can you also provide support-bundle?

Hi , cclhsu , I am pretty sure  that other longhorn pod are all running and the status are healthy

I just redeployed longhorn , seems that longhorn-ui pod has same problem too

Hi! Can you check if longhorn-manager is running on each node? Error message exceed dateline mean there are some issue to reach it. Are others Longhorn process properly running? Can you also provide support-bundle?

Hi , cclhsu , I am pretty sure that other longhorn pod are all running and the status are healthy

Please help us by creating an issue in https://github.com/longhorn/longhorn/issues and provide us with a support bundle either attach it to the ticket or send it to<EMAIL_ADDRESS> We will take a look! Thanks!

Please help us by creating an issue in https://github.com/longhorn/longhorn/issues and provide us with a support bundle either attach it to the ticket or send it to<EMAIL_ADDRESS>We will take a look! Thanks!

Sorry to trouble you, I just re-deployed longhorn again, but this csi problem no longer appeared, and I didn't do anything during this period. The problem cannot be reproduced, I will raise the issue again when there is a problem next time . Thank you very much


improvement task group scan entity

scan task group queue
Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

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

run buildall

run buildall

run p0

run buildall

run buildall

run buildall

run buildall

run buildall

@wangbo do you have time to merge this pr？

run buildall


No removal/uninstalling procedure for Interception

From: https://github.com/snoothy/ucr/wiki/Core_Interception
To use Interception, it needs to be installed from cmd with administrator privileges.
But there's no documentation about uninstalling/removing it.

simply use install-interception.exe with /uninstall rather than /install. This information can be found by calling install-interception without any arguments.

simply use install-interception.exe with /uninstall rather than /install. This information can be found by calling install-interception without any arguments.

I'd rather know it first by reading the documentation than executing it firsthand.


Compare numeric suffix of pre-release strings separately

Hey Paul!
I've put together a Nightly release channel for Atom and ran into an interesting issue while trying to generate delta packages with Squirrel.Windows.  On the day that the 1.30.0-nightly10 release was being produced on VSTS I saw the build fail with the following error:
2018-07-19T04:51:24.8313461Z Error: Failed with exit code:<PHONE_NUMBER>
2018-07-19T04:51:24.8313925Z Output:
2018-07-19T04:51:24.8314245Z System.IO.FileNotFoundException: The base package release does not exist
2018-07-19T04:51:24.8314666Z File name: 'D:\a\1\s\out\atom-1.30.0-nightly1-full.nupkg'
2018-07-19T04:51:24.8315221Z    at Squirrel.DeltaPackageBuilder.CreateDeltaPackage(ReleasePackage basePackage, ReleasePackage newPackage, String outputFile)
2018-07-19T04:51:24.8315990Z    at Squirrel.Update.Program.Releasify(String package, String targetDir, String packagesDir, String bootstrapperExe, String backgroundGif, String signingOpts, String baseUrl, String setupIcon, Boolean generateMsi, String frameworkVersion, Boolean generateDeltas)
2018-07-19T04:51:24.8316677Z    at Squirrel.Update.Program.executeCommandLine(String[] args)
2018-07-19T04:51:24.8317066Z    at Squirrel.Update.Program.main(String[] args)
2018-07-19T04:51:24.8317420Z    at Squirrel.Update.Program.Main(String[] args)
2018-07-19T04:51:24.8317661Z
2018-07-19T04:51:24.8317999Z   at ChildProcess.proc.on.code (D:\a\1\s\script\node_modules\electron-winstaller\lib\spawn-promise.js:62:16)
2018-07-19T04:51:24.8318923Z   at emitTwo (events.js:126:13)
2018-07-19T04:51:24.8319275Z   at ChildProcess.emit (events.js:214:7)
2018-07-19T04:51:24.8319668Z   at maybeClose (internal/child_process.js:925:16)
2018-07-19T04:51:24.8320356Z   at Process.ChildProcess._handle.onexit (internal/child_process.js:209:5)

After tracking it down a bit further, I found that when generating the delta nupkg, Squirrel is looking for the previous version of 1.30.0-nightly10 in the RELEASES file and it's choosing 1.30.0-nightly1 instead of 1.30.0-nightly9.  This happens due to a bug in the SemanticVersion class of Squirrel's fork of NuGet which compares the prerelease section of the version string using an ordinal comparison.  This comparison treats nightly10 as a string directly following nightly1 instead of nightly9.
An easy fix for this issue is to leverage the native method StrCmpLogicalW so that the numeric parts of the prerelease string are compared the same way as how Windows Explorer sorts them.
Another option would be to try and parse the prerelease string format to detect the numeric portions and compare them separately but I think that change would be more likely to introduce regressions.
I'll be sending another PR in a moment to add a test to Squirrel.Windows that verifies the new sorting behavior.
Related PR from atom/atom: https://github.com/atom/atom/pull/17723

Hey Paul, finally got a chance to get this updated.  Mind taking a look to let me know what you think?  I've verified that the test I added to Squirrel.Windows still passes after these changes.
Thanks a lot!


Document how to deploy influx enterprise on google cloud infrastructure

we should have enterprise documentation for deploying to Google's Cloud.

Closing in favor of https://github.com/influxdata/docs.influxdata.com/issues/2063.


AssetManager fix

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed: Argument 1 passed to Yiisoft\Aliases\Aliases::isAlias() must be of the type string, null given, called in src/Aliases.php on line 132.
Fixed PHP_EOL in testcase

Merged. Thanks!


In this project the client will connect but then is immediately disconnected

RemoteControl.zip

Hi @lxman thanks for sending this.  Appears to be a problem in the client form.  I'm investigating this now.  When I connect from a console client, it works fine (see attached).  Will post findings and source when finished.

Client is somehow being immediately disposed.

Ok so it appears that when the Send methods are embedded within forms event handlers, Dispose gets called on the object (!!!).  I don't know why this is happening or what to do about it.  Any suggestions?

Hmm, well, that's odd - I do know a bit about winforms, so I'll think about it. Here's something else to add to the puzzle. I have a project done with a similar design, only using WPF. That project does not have this issue. I realize that at this point it only confirms your diagnosis, but I thought that might be good to know.
When I get a bit of time around my day job I'll take a look.

Wait, I got it.  Give me a moment.

Simple fix.
Set either MessageReceived or StreamReceived event on your client.
I'll modify .Start() to throw an exception if neither is set.
Thanks for catching this!  I was about to pull my own hair out...

FYI there was a code path inside the main DataReceiver loop where a break was present without any logging.  DataReceiver was actually terminating because the main loop was broken and Dispose is called at the end of DataReceiver.

Hmm, okay, I've set a handler on  MessageReceived and that particular error does not appear any more. Thanks!

I updated the package to v3.1.4 on NuGet: https://www.nuget.org/packages/WatsonTcp/3.1.4
Here's the commit: https://github.com/jchristn/WatsonTcp/commit/fcfc5487832f34cd422a3f4deb5f457951cd8ada
It adds better logging for cases in code where the DataReceiver loop is broken due to the lack of a message or stream handler, and throws an InvalidOperationException if Start or StartAsync is attempted without these being set.
Thanks for finding this!  It proves the fact that if you have a code path, somehow it will be found!

By the way @lxman would you mind if I posted (public) the modified source from the .ZIP file above?  It may benefit others using the project.  If not, I understand!

That's fine with me - that was just a barebones to demonstrate the issue. Nothing really proprietary there.

Okay, it looks like I'm on to step 2, and I've got the same thing. I've started to fill out the project and  the client is getting suddenly disconnected again. Would you like me to open up a new ticket or just continue this one in order to send you the source?

Let's reopen this one.

No, sorry, wait. I see what it is now.  It was causing the server to crash. Just saw that as I was uploading to you.
RemoteControl.zip
I'll try to be a little more diligent. In short, I created a list of Control, tried to access the first element, but had forgotten to actually add a control to the list. I guess trying to reach in and get element 0 when the list is actually empty caused the server to go belly up.
My bad.

Hey @lxman don't sweat it, happy to help.  Reclosing for now.  Don't hesitate to file new issues if/when they arise.  Cheers and stay safe.


Update links to build and coverage status in Readme.md

updated coverall to support new repo name, closing / opening to retrigger tests.


Added blockchain APIs

missing ? https://neoscan.io/doc/Neoscan.Api.html

Closing this in favor of first collecting infos in #38


Corrected documentation of Log database function.

I tried to follow the docs and was confused by the description for the Log function. I'm not an English native, but I have the impression that the docs are incorrect. The current phrasing makes you think that Log(2, 4) = 0.5.
The code example, however, shows that Log(2, 4)  = 2, I think this would be phrased this way: Log of 4 to the base of 2, i.e. log of the second number to the base of the first.
An alternative and more concise phrasing would be this:

Accepts two numeric fields or expressions and returns their logarithm. expression1 is the base and expression2 is the number.

Additionaly the function signature could be represented like this:
.. class:: Log(expression_base, expression_number, **extra)

Nice catch 🏆

@StefanBrand Thanks :+1:


How to customize the display pattern of measures in  matrix visual

I just have some customized demand in using matrix visual, could anyone give some suggestion？
As seen in the picture, in this dashboard, I use two matrices ,but I have set them to display different messuare values of the same row label(namelly inventory sku code),based on user's filter selection.
What I need  is to ensure when I scroll up and down in one matrix, the other one may scroll in the same direction.
Since synchronous scroll in several matrices seems unavailable yet, so I try to put all measures into  one matrix, but the result is not so satisfactory,
Specifically, when the  measures in  the left matrix are dargged into the right one,  they will be expanded to compute and list according to each date column label in the right matrix.---this looks a little weird, because the results of these two measures keep constant  in  a whole month, not variable to each date of a certrain month. In another words,  I need set these two measures  to  display in  each single column

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

ID: 3825d5bd-51e3-468a-0f7d-a936083f0dc5
Version Independent ID: 59eea08a-c392-951e-2ba6-860cfcc58281
Content: Create a matrix visual in Power BI - Power BI
Content Source: powerbi-docs/visuals/desktop-matrix-visual.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @mihart
Microsoft Alias: mihart

Thanks for your comment, @JTTonywujie. This issue has been assigned to the doc author for review.

Thanks for posting – note that we can only answer feedback related to the documentation, and can’t provide direct support or respond to feature requests.
Your best venue for getting an answer or feedback on your question is from the community (https://community.powerbi.com) or from support at https://support.powerbi.com. You can also provide feedback using the Give Product Feedback button at the bottom of the article page.


Make Run() use a terminal only if it has one

Make Run() run the command with a terminal if we're being run with a terminal.  This lets us avoid a weird workaround in the tests.

 (8) [Buildah] Add "dockah build-using-dockerfile" aka "buildah bud"

@rh-atomic-bot r+ ca3ce93

@rh-atomic-bot retry


Move Flexion senders and receivers to production

Moving Flexion senders (e.g. simulated-hospital, simulated-lab, etor-service-sender) and receivers (e.g. simulated-hospital, etor-service-receiver-orders, etor-service-receiver-results) to production.
1.) Update setting for all senders and receivers
2.) Test connectivity

We are in testing phase.

Due to RS can only handle PKCS#1 key files (BEGIN RSA PRIVATE KEY) come from the PEM encrypted messaging project, it causes issue when it try to read from the vault.  I happened to create the key pair for PKCS#8 and registered in the vault.  Therefore, it can't handle it.  I corrected the private key in the vault and now it is good to go.

Done.


Bug: changing type of playing

All Submissions:

[x] Have you followed the guidelines stated in CONTRIBUTING.md file?
[x] Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

changing type of playing from str to bool
closes #1983

AppHarness tests are failing until #1987 can merge. Will take this PR after that.


Implement divmod

Hello @philippeitis! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file polynomial/core.py:

Line 387:1: W293 blank line contains whitespace


Moved populate_instance_names

Cleanup to move method into config class.
Added tests.
yapf cleaned up some unrelated test files

:+1: aside my small remark.

Ah, thanks, got that updated!

:100:


Add HTML show

Codecov Report

Merging #55 (cc983c4) into master (2520d74) will decrease coverage by 2.40%.
Report is 4 commits behind head on master.
 Consider uploading reports for the commit f7120ef to get more accurate results

@@            Coverage Diff             @@
##           master      #55      +/-   ##
==========================================
- Coverage   94.42%   92.02%   -2.40%
==========================================
  Files           6        6
  Lines         269      276       +7
==========================================
  Hits          254      254
- Misses         15       22       +7

Files Changed
Coverage Δ

src/abstractgeotable.jl
95.69% <58.82%> (-3.32%)
:arrow_down:


Add documentation on Math features.

Coverage remained the same at 100.0% when pulling 1ba5f983519d80fd5636188b1660fdb9da973fb6 on docs into 38654cfd5fc455aff53151fb15247cb92bf1b70b on master.

Coverage remained the same at 100.0% when pulling 1ba5f983519d80fd5636188b1660fdb9da973fb6 on docs into 38654cfd5fc455aff53151fb15247cb92bf1b70b on master.

Coverage remained the same at 100.0% when pulling 978f10d8d325cd1dc4a4a2a5ceb769b1c010fb01 on docs into 38654cfd5fc455aff53151fb15247cb92bf1b70b on master.


Clarify usage with Immutable.js

In the readme, it says

Provide your own reducer function that will receive actions of type LOCATION_CHANGE and return the payload merged into state.

This isn't enough information, and the user needs to look through the source code to understand where to actually put the data.
Can we change this to something like

Provide your own reducer function that will receive actions of type LOCATION_CHANGE and return the payload merged into the locationBeforeTransitions property of the routing state. e.g. state.set("routing", {locationBeforeTransitions: action.payload}).

I'd be happy to accept a PR for this change. We can discuss specific changes that way.


build(config): downgrade find-up

tablegen wasn't working for me because of some cjs/esm conflicts in the dependency chain<EMAIL_ADDRESS>and below should support both cjs and esm, which also will help building for both if we want that later

would be curious if it works for you with #660

would be curious if it works for you with https://github.com/latticexyz/mud/pull/660

yeah just tested, it does. This PR still can be useful if we want to support cjs, up to you

seems reasonable to downgrade and then upgrade again if it works with tsup? looks like there's not many changes since 5.0.0 that we'd need take advantage of

closed as it is unnecessary, for now at least


isLinkedWithAuthType returns true after failed account linking attempt

After attempting to link to a Facebook/Twitter account that already exists, the correct error is returned:
"Another user is already linked to this facebook/twitter id"
using the "isLinkedWithUser" method returns the correct result, however when the application is closed and re-opened the "isLinkedWithUser" method returns true for the failed attempt
one is then able to retrieve information (friends list/email) etc for the account that failed to link
logging out and back in resets the behaviour back to normal.. I have spend hours attempting to debug this issue.. if it is something within my implementation then I need to address this further.. however if this is a bug with the SDK then it is a security flaw so I thought it was worth reporting, even if it turns out to be my implementation

Hey @bybyron, thanks for the report.
Will take a look in-depth, but I guess it is indeed an issue.

Valid bug, just reproduced with a test app. Fix is incoming...
Workaround: Try saving the user again - it will be persisted to disk at that point.

thanks for the response, glad to have helped


surrealdb-migrations: 0.9.8 -> 0.9.9

Description of changes
for anyone else having a look at this. I need to add the cargo.lock here since the cargoHash is giving me a weird download error
 > error: failed to sync
       >
       > Caused by:
       >   failed to download packages
       >
       > Caused by:
       >   failed to download replaced source registry `crates-io`
       >
       > Caused by:
       >   failed to verify the checksum of `surrealdb v1.0.0-beta.9+20230402`

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 236336 run on x86_64-linux 1

1 package built:

surrealdb-migrations


Relation_extraction.jinja not working

As mentioned in the code I executed the code and downloaded everything from requirements.txt other codes are working but
Code for Relation Extraction
result= nlp_prompter.fit('relation_extraction.jinja',
                                   domain = 'clinical',
                                   # examples = [prompt_examples[0]],
                                   text_input  = sent)
print(result['text'])

gives the error "TemplateNotFound". What should I do?

Hi, did you install from github directly?

Hi, @monk1337 . Yes, this worked.
Is there any way for getting the same relations every time we execute the code because right now on every execution the relations extracted are different?