Fully qualify all global function calls.

Modern php has several optimizations it can apply to global functions at compile-time, but only when they are fully-qualified.
For example it can translate several functions into specific opcodes which are faster than their function equivalent.
It can also do compile-time evaluation of constant operands for several functions.
Additionally, fully-qualifying global functions avoids a local namespace lookup and as such makes calls faster in general. The improvement will be pretty minor but every little helps, especially in situations where these functions may be called thousands of times in a single analysis.

Did you manually do this? I wonder if we could use Rector to automate this and if we should add a rule to make sure functions are always imported.
I know they are in PHPStan, it'd be worth checking how that project enforces this

Yeah i did do it manually
not sure about the rector aspect with regards to the automation, as that would require either pre-commit hooks or for github actions to add commits onto opened prs to apply changes.
It would be easy to add a phpcs sniff to enforce they're always fully-qualified though, phpstan itself uses https://github.com/slevomat/coding-standard and the sniff is SlevomatCodingStandard.Namespaces.ReferenceUsedNamesOnly.ReferenceViaFallbackGlobalName
Adopting the full slevomant/coding-standard ruleset would likely be quite a large piece of work as the slevomat standard is fairly opinionated, but i could require-dev it and add the single sniff if you'd like.

Drupal core also uses parts of slevomat/coding-standard (https://git.drupalcode.org/project/coder/-/blob/8.3.x/composer.json?ref_type=heads#L21)
It's used by Drupal core https://git.drupalcode.org/project/drupal/-/blob/11.x/core/phpcs.xml.dist?ref_type=heads#L255
I think it'd be great to add it as a dev dependency and include that one rule to start

Turns out slevomat/coding-standards was already installed to the repository :sweat_smile:, but ooking into it the rule does more than just enforce the use of use function:
https://github.com/slevomat/coding-standard/blob/master/doc/namespaces.md#slevomatcodingstandardnamespacesreferenceusednamesonly-
it also prevents the use of inline namespaces and a few others, which increases the size of the changeset. I've resolved all the violations it came up with in a separate commit to make it easier for you to review.


Session is not Maintaing in app

After cloning the Ethereum-react-native-boilerplate project I am facing an issue. In App everything is working fine but every time I open my app it requires me to authenticate
it’s not maintaining the session in app.
On the other hand session is maintained in metamask and moralis panel and isAuthenticated is also coming false after reopen.

Same issue. I'm trying to find a solution

@fatiherdogan01 did you find something?


Allow agent to become leader again after another agent became leader.

What does this PR do?
If for whatever reason the leader loses connection to the API server and another agent becomes the leader. Allow the previous leader to try to become the leader again.
Motivation
Previously, if an agent stopped leading then it would never try to become the leader again.

My attempt at a unit test seems to have upset the race detector.


Send other remote keys

Created a new "remotekey" command which sends as a keypress any of the remote keys already defined in Keycodes along with some more I added (for DPAD navigation).  Adding any additional keys (set and code) under protocol - Keycodes would then work with this command.  Maybe this would be helpful for other people?
examples:
pyvizio --ip=ip --auth=auth remotekey VOL_DOWN
pyvizio --ip=ip --auth=auth remotekey DP_UP
pyvizio --ip=ip --auth=auth remotekey DP_DOWN
pyvizio --ip=ip --auth=auth remotekey DP_LEFT
pyvizio --ip=ip --auth=auth remotekey DP_RIGHT
pyvizio --ip=ip --auth=auth remotekey DP_ENTER
pyvizio --ip=ip --auth=auth remotekey BACK

Hey, @aschain thank you for your work!
These changes were included as a part of https://github.com/vkorn/pyvizio/pull/18


Fix message size discrepancy in chat history

@rsmahoney211 said:

model messages stream at modal size but user messages are input at fab size. fixes when you select the chat from history

#229 related


"react-router has no exported member 'Location'"

I know this is technically probably because of a recent change to the @types for react-router but your types file must reference it and it is no longer exported.
Error dump:
ERROR in [at-loader] node_modules/@types/react-router-redux/index.d.ts:9:10
    TS2305: Module '"/Users/~/Documents/~/node_modules/@types/react-router/index"' has no exported member 'Location'.

I realise this should be an issue for the actual typedef file not this project, my bad


Search target selected panel view keyboard handling

Note

Handle UIModalPresentationStyle.formSheet
Handle toggle software keyboard in simulator

Codecov Report

Merging #46 into v5.2 will increase coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##             v5.2      #46      +/-   ##
==========================================
+ Coverage   69.72%   69.73%   +<.01%
==========================================
  Files         181      181
  Lines        8027     8026       -1
==========================================
  Hits         5597     5597
+ Misses       2430     2429       -1

Impacted Files
Coverage Δ

...ringUI/ShareTargetSearchResultViewController.swift
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Public (early) alpha of free-programming-books search

https://4dylanregan.github.io/myapp/
you can leave comments and suggestions here.
There's a parser that converts the markdown to json,
and  a react app that loads the json and searches it.
by @4dylanregan @nrfq @brogan20 @dpekata @LeoOuyang24 @pkelly10439594

Thans for that first.
A suggestion : maybe should open each link in a new frame(tab or window in browser). not in current frame.
:)

Nice!!!

it would be great have yml format to address frontend using github pages

My 2 cents to UI...
It would be great considere responsive design.

Eg. above screenshot shows irregular gaps on breaking layout line when content is irregular

This link: https://4dylanregan.github.io/myapp/ show a blank page, there is no UI at all.
this is normal? or I use the link as API  or there something I miss?

This link: 4dylanregan.github.io/myapp show a blank page, there is no UI at all. this is normal? or I use the link as API or there something I miss?

Project was transfered to the organization recently. Try with
https://ebookfoundation.github.io/free-programming-books-search/

deployed


Add opinionated Anki export

(I've revised this issue based on new thoughts)
Summary
I wrote Lute based off of LWT, but dropped the SRS feature of LWT: the code was brutal, and for the initial MVP (minimal viable product) release of Lute I didn't feel that it was a necessary feature.  I still don't :-) for a few reasons:

A brute-force approach of "just test everything" isn't the best.  In some cases like verb inflections, I don't need to test every permutation -- perhaps I should only see the parent, and a few child examples.  Also, there are many words that I've only seen once so far in my reading, and may never see again.  I think I should be able to select the terms I want/need to test.
I question whether Anki testing falls within the primary use-case of Lute, which is just to get you reading, and to hopefully encourage you to keep reading.  I read a lot with Lute, and would vastly prefer to focus on reading, rather than testing.
Testing by seeing sentences I've read, and then regurgitating those (or similar), isn't very fun for me!

Even Steve Kaufmann of LingQ doesn't really recommend using their testing feature, probably for the same reasons as I have above. :-)  (He does recommend their "sentence mode" for building sentences, I believe.)
Exporting terms to a CSV, and images to somewhere else, may be trickier than needed, so I'll with using AnkiConnect as the first iteration of this.
This will be an opinionated export: it will assume certain note types, deck name, field names, etc.
Design/UX notes
The following are some rough ideas only.  I'd need to try implementations to really get a handle on the UX.
Config

Some ppl don't use Anki, so they shouldn't see it if they don't want it.  Default should be to see it.
add setting to set the Ankiweb address and port; default should be whatever needed

Exports from term listing
The term listing has a checkbox.  Users could select the terms they want to export, and then click an "export to anki" button.
Ankiconnect supports exporting images, see https://github.com/FooSoft/anki-connect/issues/158.
Lute doesn't store sample sentences for terms, but it does has a reference lookup that could get the latest sentence for any term.  The sentences table is loaded on opening a page for reading, so even if the page isn't marked read something should be in the table.  Need to update the export to include a non-read page's sentences.
Fields to export:

term id (can change if people delete terms)
language id
term
audio (left blank, as lute doesn't have audio, but people might add it later)
language
image
translation
parent term
tags
sample sentence as "manual cloze" (with term replaced by ____)
sample sentence with cloze removed

failed exports

model doesn't exist
deck name doesn't exist
ankiconnect isn't on
misc errors?

Anki note/card templates
I'll put some kind of pre-designed note type in a public place so people can access it ... that's probably the easiest thing to do.   Creating a single-card shared deck on AnkiWeb would be easiest.  AnkiConnect apparently does let you create models using the API, I'm not sure how tough that would be.  https://foosoft.net/projects/anki-connect/index.html#model-actions
It would be nice to have Anki cards be able click back to Lute, if Lute's running, so that people can see the term and its sample sentences again.

Some of the above is no longer valid (e.g. any mention of the .env file, which is replaced with settings).  But the ideas are still good.

Would love to see this being implemented. I agree with your initial thoughts on Anki not being a suitable place for regurgitation, however there's an adjacent use-case that we can target.
Exporting terms which are a bit mature in the learning cycle (i.e. let's say learning levels 2/3/4) along with sentences would probably be better (That way we ensure that the term would've been seen at least twice or more number of times by the user)

I want to chime in on this, as I have been developing something based off AnkiConnect in the last few months. I crave the possibility of adding something like this to Lute, because, well — I really do enjoy Anki 😄

Exporting terms to a CSV, and images to somewhere else, may be trickier than needed, so I'll with using AnkiConnect as the first iteration of this.

In fact, I also tried this approach, but it proved a bit too cumbersome for the possible gains. It's of course doable. But AnkiConnect seems indeed to be the way to go.

This will be an opinionated export: it will assume certain note types, deck name, field names, etc.

I don't think it is necessary to assume any of these. The current implementation of my service will query Anki for available note types, deck names and field names, and leaves to the user the choice of matching as needed.

(source and target are the field names of my custom note type).

Design/UX notes
The following are some rough ideas only. I'd need to try implementations to really get a handle on the UX.
Config
* Some ppl don't use Anki, so they shouldn't see it if they don't want it.  Default should be to see it.

I feel this can be handled as the MeCab config: leave it on the settings, and users can find it if they want to? A toggle should then enable AnkiConnect interactions when appropriate in the interface?

Exports from term listing
The term listing has a checkbox. Users could select the terms they want to export, and then click an "export to anki" button.
Ankiconnect supports exporting images, see FooSoft/anki-connect#158.
Lute doesn't store sample sentences for terms, but it does has a reference lookup that could get the latest sentence for any term. The sentences table is loaded on opening a page for reading, so even if the page isn't marked read something should be in the table. Need to update the export to include a non-read page's sentences.
Fields to export:
* term id (can change if people delete terms)

* language id

* term

* audio (left blank, as lute doesn't have audio, but people might add it later)

* language

* image

* translation

* parent term

* tags

* sample sentence as "manual cloze" (with term replaced by ____)

* sample sentence with cloze removed

I think these are all doable.

failed exports
* model doesn't exist

* deck name doesn't exist

* ankiconnect isn't on

* misc errors?

If implementation follows the pattern of first querying the decks, models and note types of the running Anki instance, then these are not a problem.

Anki note/card templates

I'll put some kind of pre-designed note type in a public place so people can access it ... that's probably the easiest thing to do. Creating a single-card shared deck on AnkiWeb would be easiest. AnkiConnect apparently does let you create models using the API, I'm not sure how tough that would be. https://foosoft.net/projects/anki-connect/index.html#model-actions

I tested something like this as well. In fact, I think it suffices to load a deck for Anki to include its note types, models, etc. But I think this problem is circumvented by matching to whatever the user already has on their own decks.

It would be nice to have Anki cards be able click back to Lute, if Lute's running, so that people can see the term and its sample sentences again.

Yes! I think a tremendous use case is to sync from Anki to Lute and match learning percentages to Lute's system of 1-5 familiarity scale.
As we discussed on Discord, I know you still have some thoughts on your mind — I'll be awaiting those. In particular, my very naive implementation — the one I already disclosed on Discord — places deck matching at the global settings level, and I understand (and it makes sense) you would prefer it to be language specific. So there's that.
In the meantime, I'll work locally on my end and see if I can arrive at a good compromise.

Hi @paotsaq - thanks for the notes.  This is a tough problem.

current implementation of my service will query Anki for available note types, deck names and field names, and leaves to the user the choice of matching as needed.

Your idea is very good, but I feel it belongs in a different issue, "flexible Anki export".  I feel that the flexibility adds a fair amount of complexity, and would make the initial delivery of an Anki export feature more difficult.  For an initial implementation, to get exporting done, I think that that the "opinionated export" is a good choice, as it reduces scope significantly.
A more flexible implementation would need:

a dynamic form to be built, with dynamic data posting
question of how to handle things that need some more processing before being sent.  e.g. if I sent a term, I (likely) would also want to send some kind of Cloze data, but Lute doesn't have that stored anwhere
a way to handle mismapping :-)
a way to handle incomplete mapping.  e.g. if we Lute to be able to update Term data, then we'd likely want to also export the Term ID.  if the user's deck didn't have that, they'd need to change the deck.  But if exporting the Term ID is optional, then that might cause problems down the road, in some way.  I can't say.

Adding flexibility in the export also adds complexity to the usage.  For example, if I'm doing German, and want to export my terms after a day of reading, will I need to re-map everything every time?  I'd quickly get tired of that, and I'd want to keep track of my pre-set mappings -- but that entails:

a way to store the flexible form design (need some kind of flexible table, but perhaps just a json mapping table would be sufficient)
reloading flexible form from flexible table data
a way for Lute to adjust that mapping data?  Hm, no, but there would need to be a way to show any "extra fields" that Lute might make available for use in the term exports
to handle mismapped/outdated maps (e.g. changed Anki deck name, field name, etc)

We can chat on Discord to work this out some more.  :wave:

For me, I also want some things that I'm not sure how to handle generally at all.  :-)  e.g. for German, there are some tricky things:

terms have genders. I tag terms as "masc", "fem", "neut", for their gender.
some German plurals are very strange.  I tag them as "plural" and set the singular as the parent.

Since I use tags in this way, I'd somehow want a specialized export that consults tags and maybe sends special cards -- but that will be way too specialized to create a general-usage form form, so I'll like create some kind of "export job" and figure out how to make that work behind the scenes.  Tricky stuff.
ps - making changes to Lute to standardize this (e.g., add a "gender" field, and a "singular form" field) are non-starters.  Not all languages have genders, most words don't have genders/singular forms anyway, most things aren't nouns.


Fallback to use new discovery uri pattern when the old one fails.

Fallback to use new discovery uri pattern when the old one fails.

Please squash commits with each round of code review.
Also please ensure that your commit log message conforms to these guidelines.


Support D410/415/435 with unique PID in USB2 Mode

DSO-8789: Add support for the new SKUs to Librealsense core.:

Adjust default selection to accomodate low-res USB2-imposed FPS and resolution values.
Provide kernel patches and WinOS registry script to enable metadata attributes for the new SKUs.
Modify unit-test to support the new PIDs
Resolve segmentation fault when an unhanded exception was thrown from dtor.

DSO-8733 Bug-fix. Resolve improper default assignment for legacy USB2 device

Closing for rework


Storybook Provider and Component examples broken

Search terms you've used

I didn't use any search terms but looked through the open issues, and didn't find anything addressing this directly.
Bug description

Visiting any of the links similar to https://solid-ui-react.docs.inrupt.com/?path=/story/components-link--with-children under components or Providers is broken and doesn't render an example.
To Reproduce

Navigate to the link above in any browser

Expected result

Visible examples of how to use the components
Actual result

The examples are broken due to webpack modules or nonexistent objects.

Hi @zg009 . We are now deprecating the React SDK, so unfortunately we will not be addressing the open issues.


CircleCI/codecov.sh is consistently failing.

https://circleci.com/gh/istio/istio/1479?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
The codecov.sh is consistently failing in CircleCI. I've been unable to glean any useful information at the moment.

It looks like the failure is due to proto files going out of sync with the generated pb files.
@guptasu is working on changes to the build system to run regular go tools as part of Prow jobs. This should avoid this type of breakages in the future.

Can we close this?


Add missing details for the installed Firmware retrieval

As described in the response body of the OAS, only Firmware components with the FirmwareComponentClass== "firmware-1-0:FIRMWARE_COMPONENT_CLASS_TYPE_PACKAGE" should be returned.
For these components only the attributes "FirmwareComponentName", "FirmwareComponentVersion" and "FirmwareComponentStatus" are to be returned.
While this is described in the OAS, the details are missing in the ProfileInstances and Callback as well as in its representation on the diagram.

Second thoughts:
Not necessary, as the descriptions and examples are detailed enough.
No need to add fieldsFilter for FirmwareComponentClass and relevant attributes; the entire firmare-collection can be retrieved first with a single request.


faster loading async Google Analytics script

as recommended by HTML5Boilerplate.com
https://github.com/h5bp/html5-boilerplate/blob/master/dist/index.html
...and Google (Alternative async tracking snippet)
https://developers.google.com/analytics/devguides/collection/analyticsjs/#alternative_async_tracking_snippet

Yep, lgtm. Surprised beacon still isn't the default.

Thank you @coliff!


[code-completion] Add filter rules for description in addition filter-name

Explanation: This is an extension to our "filter-rules" API for code-completion, which allows the client of sourcekitd to create custom filters to hide/show various kinds of completions.  The new rule is for filtering based on the description text of the completion.
Scope: Only affects clients that adopt the new API.
Radar: rdar://problem/28920034
Risk: Low; only affects clients that adopt the new API.
Testing: Regression tests added.

@swift-ci please test


Adds maliput_integration_tests source to foxy.

Please add the following dependency to the rosdep database.
Package name:
maliput_integration_tests
Package Upstream Source:
https://github.com/maliput/maliput_integration_tests
Purpose of using this:
Only source is being added for triggering CI for each pull request event.
No release is expected.

CC: @tfoote


about null value

class:
public class Person
 {
        public int Id { get; set; }
        public string Fullname { get; set; }
        public DateTime? Birthday { get; set; }
  }

Test:
var col = db.GetCollection<Person>("Person");
col.Insert(new Person { Fullname = "John" });
col.Insert(new Person { Fullname = "Doe" });
col.Insert(new Person { Fullname = "Joana" });
col.Insert(new Person { Fullname = "Marcus" });

delete result:
var delResult = col.Delete(x => x.Birthday < DateTime.UtcNow);

OR
find result:
var findResult = col.Find(x => x.Birthday < DateTime.UtcNow);

I think delResult = 0  or findResult.Count() = 0, but result of test is delResult = 4  or findResult.Count() = 4.
Is it a bug?

Hi, that happend because null are a type of value (like int or string). And, when you compare 2 diferent types (like DateTime with null) LiteDB comprare data type order only. And null are less than Date. This will happend if you add another datatype too, like
db.Engine.Insert("col", new BsonDocument { ["_id"] = 5, ["Fullname"] = "Demo", ["Birthday"] = 5 });
So, when I compare 5 with DateTime.Now I just compare order of each data type:
https://github.com/mbdavid/LiteDB/blob/master/LiteDB/Document/BsonType.cs
I will take a look on how MongoDB resolve this and return here.

You right @codeyu, it's wrong result in LiteDB. Thats mongodb results:
> db.col1.find()
{ "_id" : 1, "a" : 1 }
{ "_id" : 2, "a" : 2 }
{ "_id" : 3, "a" : null }
{ "_id" : 4 }
{ "_id" : 5, "a" : "str" }

> db.col1.find({a:1})
{ "_id" : 1, "a" : 1 }

> db.col1.find({a:{$lte:1}})
{ "_id" : 1, "a" : 1 }

> db.col1.find({a:{$lte:2}})
{ "_id" : 1, "a" : 1 }
{ "_id" : 2, "a" : 2 }

> db.col1.find({a:null})
{ "_id" : 3, "a" : null }
{ "_id" : 4 }

Mongo does not return different types when using > <
It's not too much clear to me after read this:
https://docs.mongodb.com/manual/reference/bson-type-comparison-order/#bson-types-comparison-order

Thank you for your job:)


Update quickstart and add feature examples

This PR simplifies quickstart instructions and adds feature examples

@tatiana I ran black, but the CI is still complaining.

Not sure how to go about fixing this


Installation instructions

Please provide the installation instructions.
Thanks!

Shouldn't be too bad.
In my case, my accessories are installed in /usr/lib/node_modules. CD into that directory
$ git clone https://github.com/hplato/homebridge-httpmulti.git
This should create a homebridge-httpmulti directory.
Then create your config.json file (mine is in /root/.homebridge/)
{
"bridge": {
"name": "Homebridge",
"username": "CC:11:3D:C3:DE:35",
"port": 51826,
"pin": "123-45-678"
},
"description": "Somthing",
"accessories": [
	{
	"accessory": "HttpMulti",
	"name": "A Fan",
	"on_url": "http://ip/fan&select_state=on",
	"off_url": "http://ip/fan&select_state=off",
	"deviceType": "fan"
	},
	{
	"accessory": "HttpMulti",
	"name": "relay",
	"on_url": "http://...",
	"off_url": "http://...",
	"deviceType": "switch"
	},
	{
	"accessory": "HttpMulti",
	"name": "a light",
	"on_url": "http://...",
	"off_url": "http://...",
	"brightness_url": "http://...",
	"deviceType": "light"
	},
	{
	"accessory": "HttpMulti",
	"name": "Door Lock",
	"lock_url": "http://...",
	"unlock_url": "http://...",
	"deviceType": "lock"
	},
	{
	"accessory": "HttpMulti",
	"name": "Garage Door",
	"open_url": "http://...",
	"close_url": "http://...",
	"deviceType": "garagedoor"
	},
	{
	"accessory": "HttpMulti",
	"name": "Room Blinds",
	"on_url": "http://...",
	"off_url": "http://...",
	"deviceType": "blinds"
	},
	{
	"accessory": "HttpMulti",
	"name": "A Thermostat",
	"setpoint_url": "http://...",
	"deviceType": "thermostat"
	}
]

}

Very great pluggin, thank you !
But I've got 3 question for you.
First, what is the name of the variable used for the brightness ? I tried %b as in the Http plugin, but it doesn't work.
Second question, I created blinds accessory without success. The accessories are created, and available in Insteon+ but when I asked Siri about blinds, he told me that I havn't blinds.
Third question, is it normal that "switch" accessories are considered as light in Siri. If I told to turn on all my light, it turn on my switch too... I should remember that it wasn't the case with Http.
Thanks for your help

Here’s an example of a brightness URL that works for me:
"brightness_url": "http://<IP_ADDRESS>:80/SET;none?select_item=$back_deck_light&select_state=%VALUE%”,

What version of IOS are you using. I seem to recall that v9 is needed for blinds. I use this successfully with IOS 9.3
 "accessory": "HttpMulti",
 "name": “Main Blinds",
 "on_url": "http://<IP_ADDRESS>:80/SET;none?select_item=$b1&select_state=on",
 "off_url": "http://<IP_ADDRESS>:80/SET;none?select_item=$b2&select_state=off",
 "deviceType": “blinds"

I think so. I’ve never tried turning on all the lights...

On May 20, 2016, at 11:41 AM, Zorgonaute84<EMAIL_ADDRESS>wrote:
Very great pluggin, thank you !
But I've got 3 question for you.
First, what is the name of the variable used for the brightness ? I tried %b as in the Http plugin, but it doesn't work.
Second question, I created blinds accessory without success. The accessories are created, and available in Insteon+ but when I asked Siri about blinds, he told me that I havn't blinds.
Third question, is it normal that "switch" accessories are considered as light in Siri. If I told to turn on all my light, it turn on my switch too... I should remember that it wasn't the case with Http.
Thanks for your help
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub https://github.com/hplato/homebridge-httpmulti/issues/3#issuecomment-220671488

Thanks for the brightness URL, I'll try.
About blinds, in Insteon the accessory look very strange because it havn't any button.
For exemple a switch have an ON and OFF button in Insteon, a light have ON, OFF and a slide, a garage door have 2 icons to open and close the door. But about the blinds, there is nothing...
My iOS version is 9.3.2

is it possible to use/add a status_url ( or similar ) on the  "garagedoor" ? then I can set up a read switch connected to an Arduino board which will show if its open / closed / obstruction ?

There is probably a way, but it's not clear to me how that connects in.

Any Updates on blinds for IOS10?
Thanks :)

I haven't looked at this yet. Hopefully over the next few weeks.

I am always getting this error, with your new httpmulti or with legacy httpmulti:
Garage Door Obstruction Read:
Garage Door - current door state Read
And the message comes up as soon as I open the Home app on iphone, with all status showing as updating.
When I use Insteon accessory (using Insteon.js) I can open my garage but I dont like how I have to say "turn garage on" rather than turn garage open ...
Any help will be appreciated.

And BTW, when I type in that http directly in my browser, my garage opens (same http for open and close):
http://user:password@<IP_ADDRESS>:25105/3?0262347Y4D0F12ff=I=3

I'm going to have to redo the accessory since it looks like a few things have changed. No big deal, but will take some time. Have blinds working with IOS10

And now lightbulbs... Probably in a few days this will be updated.

OK, the v2 of this accessory seems to mostly work. I can control all the objects with either the IOS10 home app or Siri. More work needs to be done for status, but for now this should be a decent update for anyone using it.

Are the config.json examples posted here still valid in v2? I cant see to be able to get this plugin to work?

The error seems to be the response of your device.
As the StackTrace says, it is expecting a key named StatusCode.
Have a look at index.js, line 501 and find the accessory type it is about. Also check your statusurl you listed in config.json!
Greetings phpfs

Well, its not the neatest, but I copied up my config.json to the wiki that has a working set of parameters for each httpmulti type.

On Dec 15, 2016, at 2:55 PM, phpfs<EMAIL_ADDRESS>wrote:
The error seems to be the response of your device.
As the StackTrace says, it is expecting a key named StatusCode.
Have a look at index.js, line 501 and find the accessory type it is about. Also check your statusurl you listed in config.json!
Greetings phpfs
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/hplato/homebridge-httpmulti/issues/3#issuecomment-267455723, or mute the thread https://github.com/notifications/unsubscribe-auth/AExbSLCrosSwvtPeCNJhiO47QuzI8gIrks5rIbdOgaJpZM4ID6_x.

@hplato  Great.! This will surely help ... Where can i check out the accepted inputs form the server, so i can modify my openhab outputs?

Probably the simplest way is to look at the perl code I did for Misterhouse.
https://github.com/hplato/misterhouse/blob/homebridge/code/common/homebridge.pl https://github.com/hplato/misterhouse/blob/homebridge/code/common/homebridge.pl
The subrotine hb_status shows how I parse object states and what value I return to the httpmulti accessory

On Dec 18, 2016, at 3:13 PM, TRIROG<EMAIL_ADDRESS>wrote:
@hplato https://github.com/hplato Great.! This will surely help ... Where can i check out the accepted inputs form the server, so i can modify my openhab outputs?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/hplato/homebridge-httpmulti/issues/3#issuecomment-267847063, or mute the thread https://github.com/notifications/unsubscribe-auth/AExbSEsmFIePlW4YSW85u3N7bVj3yiVKks5rJaIQgaJpZM4ID6_x.

So after messing around a bit i figured it out:
My config was wrong i.e. based on the config example from april 13th. :\ i was using
    "open_url": "http://...",
    "close_url": "http://...",

as opposed to:
"up_url": "http://...",
"down_url": "http://...",


@action decorator silently ignores empty string parameter.

It should either give me an error or it should honor the empty name (and thus omit the route suffix). Honoring the empty name can be useful for custom actions/verbs on resources/collections. E.g. bulk-update for PATCH operations against a collection:
using Cadl.Http;
using Cadl.Rest;

using Azure.Core;

@resource("rooms")
model Room {
  @key
  @visibility("read")
  roomId: string;
}

@parentResource(Room)
@resource("rooms")
model Participant {
  @key
  @visibility("read")
  participantId: string;
};

@action("") // <-- (low priority) bug - empty action name should either work (be honored) or yield an error
@doc("Update participants in bulk.")
@patch op bulkUpdateParticipants is Azure.Core.ResourceCollectionAction<Participant, { @header contentType: "application/merge-patch+json", participants: Record<Participant>}, {}>;

(see playground)

nit: the second @resource in the sample should be
@resource("participants")
model Participant {

est: 3

Let's make it an error. We can always turn error into works without breaking, but not vice versa.

If error, we need to add some other way to support the scenario described in the issue.


RTU Hardware flow control with RTS pin (half-duplex)

I would like to toggle a RTS pin while sending out data from tx to control half duplex chips. I was able to do this with my own unfinished modbus library, but I'd really like to port my code to this one for it's additional tcp and async functionality.
I tried to look around the source, but am quite unsure how deep the serial transaction is handled. Should I rather look in the mio_serial crate, or is there some callback functionality built-in?
Thanks in advance! Awesome project!

So, after browsing many many unsuccessful forums threads on how to get a working UART with GPIO controlled RTS pin, I finally got it!
I knew the correct way to do this is via a kernel driver, but it took a long time to get it working so I first did it in userspace code, which I tried here too. I am running a Raspberry Pi 4 board with the PL011 connected to uart0 (GPIO 14 and 15). The amba-pl011 driver already supports an RS485 mode since this commit.
So what you need to do is:

Have a kernel with the above mentioned commit, since the driver is builtin (You most likely will with a newer Raspberry Pi OS)
Enable the RS485 flags on /dev/ttyAMA0 (I have PL011 connected there with /boot/config.txt)
Set the dedicated GPIO17 on its alternative function 3, this makes it a RTS pin (I used raspi-gpio cli)
Enable Hardware flow control in tokio_serial or any other library you might be using (might actually not be necesseary, this sets the tty crtscts property, and is used with hardware based circuits I guess. In this case I tested a working example without it.)

Here is an example:
use rs485::Rs485;
use rs485::SerialRs485;

// point number 2 above
async fn enable_rs485(tty_path: &str) -> Result<(), Box<dyn std::error::Error>> {
    let fd = tokio::fs::File::open(tty_path).await?;
    let mut rs485_settings = SerialRs485::new();
    rs485_settings.set_enabled(true);
    fd.set_rs485_conf(&rs485_settings)?;
    Ok(())
}

#[tokio::main(flavor = "current_thread")]
async fn main() -> Result<(), Box<dyn std::error::Error>> {
    use tokio_serial::SerialStream;
    use tokio::time::{sleep, Duration};
    use tokio_modbus::prelude::*;

    let tty_path = "/dev/ttyAMA0";
    let slave = Slave(0x1);

    enable_rs485(tty_path).await?;

    let builder = tokio_serial::new(tty_path, 9600).flow_control(tokio_serial::FlowControl::Hardware); // point number 3 above
    let port = SerialStream::open(&builder).unwrap();

    let mut ctx = rtu::connect_slave(port, slave).await?;
    println!("Reading a sensor value");

    loop {
        let fut = ctx.read_holding_registers(30, 2);
        let rsp = tokio::time::timeout(Duration::from_millis(4), fut).await;
        match rsp {
            Ok(data) => println!("Sensor value is: {:?}", data.unwrap()),
            Err(_) => println!("Sensor timed out")
        }

        sleep(Duration::from_millis(50)).await;
    }
}


The LMA App in Stream Audio Mode is transcribing even when I am on mute!

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
The LMA application in stream only mode listens to the microphone and picks up surrounding speech/audio and then transcribes the same even when I am on mute within the Microsoft Teams application. This will lead to bad transcriptions and unwanted data especially when there are background chatter. (office environments)
Describe the solution you'd like
A clear and concise description of what you want to happen.
When on mute, I dont want the application to listen in and start transcribing. I would also ideally want a way to control when the application can listen in.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
NA
Additional context
Add any other context or screenshots about the feature request here.
NA

Fixed in v0.1.4


d3-sankey is missing in package.json and index.js

I have read the warning and i still believe this is the correct repo for this issue
I tried the following (reduced test case):
import * as d3 from 'd3';

d3.sankeyLinkHorizontal()

and d3.sankeyLinkHorizontal does not exist
I have noticed that d3-sankey is neither in package.json nor listed as export * from ... in index.js
If this is actually an issue, i'm happy to make the PR

Like other modules (d3-hexbin, d3-geo-projection etc), d3-sankey is a module but is not part of the main distribution.


Support removing spacing between paragraphs

In Jetpack Compose, LineHeightStyle can be specified as an option to ParagraphStyle to remove additional spaces between paragraphs. According to the documentation:

trim feature is available only when PlatformParagraphStyle.includeFontPadding is false.

https://developer.android.com/reference/kotlin/androidx/compose/ui/text/style/LineHeightStyle
In Compose Desktop, setting LineHeightStyle has no effect, and there is no includeFontPadding in the PlatformParagraphStyle class.
This is one of the blockers of implementing a soft-wrap-enabled code editor with line numbers.

Thanks for Issue. We will discuss it in our team.

Hi, does it work on AnnotatedString?
withStyle(                 ParagraphStyle(                     textIndent = TextIndent(                         firstLine = 0.sp,                         restLine = 16.sp                     ),                     lineHeight = 16.sp,                     platformStyle = PlatformParagraphStyle( includeFontPadding = false ),                     lineHeightStyle = LineHeightStyle(                         LineHeightStyle.Alignment.Center,                         LineHeightStyle.Trim.Both                     )                 )             ) {                 content()             }
I tried with this but there is still margin after every paragraph.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


InsertRoute uses a recursive algorithm

The InsertRoute function traverses the tree until it finds the node it needs to act on. Currently, it maintains a list of visited nodes as it traverses so that it's able to cut off any nodes that, as a result of inserting, create a completed subtrie (e.g. if a node has two children, one representing a "0" and the other representing a "1", then the subtrie node itself represents the complete subtrie below it, and this situation has a more efficient representation. As a result of pruning child nodes for this purpose, it is possible we need to percolate that change upwards in the trie, and we do that using the list of visited nodes.
This PR maintains the ability to prune redundant children from nodes and percolate that change upwards in the trie, but it replaces the list with a recursive algorithm. Will this be more performant? We'll see!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
4 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 83.627%

Files with Coverage Reduction
New Missed Lines
%

pkg/routesum/rstrie/rstrie.go
4
97.65%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
332

Relevant Lines:
397

💛  - Coveralls

Commit 01d43cf was cherry-picked and merged into main. The remainder of commits are in the performance-feature-branch and don't belong in main.


feat: display same source and destination token warning on widget

Summary
If the source and destination tokens are selected as the same, we should show a warning to the user.
Fixes # (issue)
How did you test this change?
Select the same token for both the source and destination, and also enter an amount
Checklist:

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

In the current implementation, the warning appears after the debounce period for the input. If the user changes the input, the warning will be cleared and then reappear after the debounce delay. I suggest leaving the existing logic for the quote unchanged and simply adding a check and warning for this specific state in the relevant component, so the warning displays immediately without delay.

Please review the UI design again. The spacing around the newly added component does not align with the design specifications


Change the text when loading game assets

Currently the loading screen to get to the menu is identical to the loading screen when getting into a game. Would be nice to at least change the text to something like "Loading game assets..." to differentiate the two.

Added in 0d8284d.


DeepBGC failed with TypeError: rsplit() takes no keyword arguments

Part of the error log:
ERROR   19/11 15:51:37   rsplit() takes no keyword arguments
Traceback (most recent call last):
File "/mnt/volume/miniconda/3/lib/python2.7/site-packages/deepbgc/main.py", line 113, in main
run(argv)
File "/mnt/volume/miniconda/3/lib/python2.7/site-packages/deepbgc/main.py", line 102, in run
args.func.run(**args_dict)
File "/mnt/volume/miniconda/3/lib/python2.7/site-packages/deepbgc/command/pipeline.py", line 173, in run
step.run(record)
File "/mnt/volume/miniconda/3/lib/python2.7/site-packages/deepbgc/pipeline/annotator.py", line 36, in run
pfam_annotator.annotate()
File "/mnt/volume/miniconda/3/lib/python2.7/site-packages/deepbgc/pipeline/pfam.py", line 120, in annotate
short_pfam_id = pfam_id.rsplit('.', maxsplit=1)[0]
TypeError: rsplit() takes no keyword arguments
ERROR   19/11 15:51:37   ================================================================================
ERROR   19/11 15:51:37   DeepBGC failed with TypeError: rsplit() takes no keyword arguments
ERROR   19/11 15:51:37   ================================================================================

Hi @thinkgenome, good catch!  I fixed it in #26.
You can install the updated version using pip install deepbgc==0.1.15, or if you are using Conda, please wait until version 0.1.15 appears here (can take up to a day or two): https://anaconda.org/bioconda/deepbgc/files and then install using conda install deepbgc==0.1.15.
This bug is only happening in Python 2.7. In general, I recommend using Python 3.6+ for DeepBGC since Python 2.7 will not be supported in the future. You can create a new deepbgc conda environment with Python 3.7 using conda create -n deepbgc python=3.7 deepbgc

Thanks @prihoda .


update readme about events

the description  for events in readme is  outdated after the rework .
Event.Data  no longer exists in the current version

You're right and that's something I'm aware of. My thinking was that the README should reflect the latest release of termui where events have not been reworked. But if someone wanted to use the latest commits to master, they would be free to do so but at their own risk.
I'm open to feedback on this though, I wasn't sure what the best strategy of making breaking changes was but this seemed fine to me.

Best strategy is to make Readme content relevant to revision of code associated with it. Eg. when you switch branches content of Readme updates according to source code in this revision

Hi, I figured updating the readme to the latest commit was the best decision so I went ahead and did that and I also added a wiki with some general info. Let me know if anyone has any more suggestions.


The font class on  should be a parameter

First thing I wanted to customize was the font. Ran straight into https://github.com/gohugoio/hugo/issues/3505

That should do it. You can add the parameter body_classes to config or to a page.

great, thanks


Add support for a global task search feature based on query filters

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.
[X] I am only requesting a single feature. Multiple changes should be split into individual requests, with links between them.
[X] I believe my requested feature will be generally applicable to other users, not just me: it is not uniquely tied to my personal workflow.

🔖 Feature description
Context:
I just thought about how often I need to look at a particular task, but don't want to bother writing a query fast to search for it.
Like oh, I needed to do something but can't remember if I did and when. I'd like to be able to search for something and adjust the search using some keywords I know I'd use for that tasks, and maybe some filters such as done in 2023 or something.
Or when you want to know if, like your gf just repeated to you kindly, you already planned your dentist appointment for 2024.
I'm using a temp file with premade basic queries for that now, but it would be great to be able to execute a command Search tasks that would run in a dialog box.
✔ Solution
Add a command that allow us to search for a task using task description matching, for example, or any of the query filters.
The dialog box would have two parts : A query visualization at the top, a code block in the bottom.
It would allow us to type in some filters, just like in a query. And then click on a task to access its file and location, or edit it using the edit task dialog.
❓ Alternatives
Maybe add an extension to the obsidian search functionality. There already exist a tag: search for looking at tags. Maybe one could search for tasks this way ? But this wouldn't support query filters.
📝 Additional Context
No response

Thanks for the suggestion.
As a workaround, I do something similar with a frequently-edited note in the root of my vault called Temp Search.
My main concern about the Todoist example shown is the assumption that Tasks is going to be able to distinguish between:

commas separating search filters
commas in the description string

Also, can you please expand on what you mean by 'task description approximation' - is the search for 'approximation', or are you imagining some other kind of search....
Basically, the more info you can provide about what you are currently imagining, the better....

Thanks for the suggestion.
As a workaround, I do something similar with a frequently-edited note in the root of my vault called Temp Search.
My main concern about the Todoist example shown is the assumption that Tasks is going to be able to distinguish between:
1. commas separating search filters

2. commas in the description string

Also, can you please expand on what you mean by 'task description approximation' - is the search for 'approximation', or are you imagining some other kind of search....
Basically, the more info you can provide about what you are currently imagining, the better....

You're right, I don't know what symbols could we use to separate different filters. Or maybe type them one after the others in a search dialog box ? But then we would need to be able to render a pseudo query on that same dialog box just like the create/edit tasks.
For the task description approximation it is two-sided :

Match substring in string : Search for cal would match calendar, calisthenics, etc.
Take small typos into account : Search for baseboll would match a description that contains baseball

These are all good ideas...

Take small typos into account : Search for baseboll would match a description that contains baseball

Indeed - so this is quite a large task.

These are all good ideas...

Take small typos into account : Search for baseboll would match a description that contains baseball

Indeed - so this is quite a large task.

Yes, the typo matching implementation would be quite a task.
Maybe we could :

First implement a basic description includes TEXT that would manage substrings matching
Add a regex matching for N similar characters (I think I heard of it a while back, it would match strings even if N characters don't match, ideal for small typing typos.
Add a complete similar description matching like Obsidian search.

Maybe we could :

First implement a basic description includes TEXT that would manage substrings matching
Add a regex matching for N similar characters (I think I heard of it a while back, it would match strings even if N characters don't match, ideal for small typing typos.
Add a complete similar description matching like Obsidian search.

I understand what you're saying....
Based on past experience, I would suggest starting with just 1, and then waiting and seeing what feedback we get from users.
Perhaps 1 may be good enough in practice.
Or perhaps Obsidian may have implemented better things by the time 1 is released...
Keeping features focussed and minimal has several bebefits:

it can motivate someone to try them out
it prevents huge PRs being stuck forever in feedback cycles, sometimes dealing with repeated conflicts...

(I know you didn't mean it like this, but I slightly feel the need to point out that this is free software after all, developed by volunteers in finite spare time... so keeping requests realistic helps too...)

Maybe we could :

First implement a basic description includes TEXT that would manage substrings matching
Add a regex matching for N similar characters (I think I heard of it a while back, it would match strings even if N characters don't match, ideal for small typing typos.
Add a complete similar description matching like Obsidian search.

I understand what you're saying....
Based on past experience, I would suggest starting with just 1, and then waiting and seeing what feedback we get from users.
Perhaps 1 may be good enough in practice.
Or perhaps Obsidian may have implemented better things by the time 1 is released...
Keeping features focussed and minimal has several bebefits:
* it can motivate someone to try them out

* it prevents huge PRs being stuck forever in feedback cycles, sometimes dealing with repeated conflicts...

(I know you didn't mean it like this, but I slightly feel the need to point out that this is free software after all, developed by volunteers in finite spare time... so keeping requests realistic helps too...)

You're totally right abouth everything, I'll edit the FR to match with our discussion.
I'm learning some Javascript and Typescript in my spare time on some basic projects in order to be able to help one day with things, slow but steady ✌

Awesome - the more the merrier!
I'm happy to pair with you on an early contribution to help you get started!

Hi @mathisgauthey, please could you have a look at #2579?
I'm not sure but I think it is a more succinct and perhaps focused version of what we were heading towards in this ticket.
If you agree, then I would propose to close this one, and keep that other one open.
What do you think?

Hi @mathisgauthey, please could you have a look at #2579?
I'm not sure but I think it is a more succinct and perhaps focused version of what we were heading towards in this ticket.
If you agree, then I would propose to close this one, and keep that other one open.
What do you think?

Hey there, looks like the first step we discussed (below). I do agree then, it's more focused and simple.
1. First implement a basic `description includes TEXT` that would manage substrings matching
2. Add a regex matching for N similar characters (I think I heard of it a while back, it would match strings even if N characters don't match, ideal for small typing typos.
3. Add a complete similar description matching like Obsidian search.

Thanks very much for closing it in favour of #2579, and for adding the summary.


#14690: Reorganize gtests under tests/tt_metal/tt_metal

Ticket
#14690
Checklist

[ ] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

I am not going to block this PR, because I applaud that some reorganization work is being done here.
My only comment is: Since this is being reorganized ... why not put everything under tt-metal/tt_metal/. The tests are very clearly associated with the tt_metal library, why do they need to be outside of that directory in a global tests/ directory?
This is a minor gripe that @afuller-TT and I have.
If they are in the same tree it simplifies things like assigning code owners to tests.

is this PR ready for merge?
I checked and don't see any passing post-commit runs.
With these changes, I would be extra careful and run all the workflows to ensure all tests pass.

is this PR ready for merge? I checked and don't see any passing post-commit runs.
With these changes, I would be extra careful and run all the workflows to ensure all tests pass.

I'm just fixing up some minor issues regarding failing pipeline runs. Once I fix those, this will be ready to merge.
This PR should only affect the all post-commit tests, TG unit tests, TGG unit tests and T3000 unit tests pipelines.

Thanks boss, please post passing links once done

All post-commit pipeline: https://github.com/tenstorrent/tt-metal/actions/runs/11991509497
T3000 pipeline: https://github.com/tenstorrent/tt-metal/actions/runs/11991511278
TG pipeline: https://github.com/tenstorrent/tt-metal/actions/runs/11991513649


tilemap regression: Image setPixel API doesn't work with legacy tilemap

works  on arcade.makecode.com but not on beta
https://makecode.com/_P99evo0webDd

repro
run and press button A
strawberry should be appear at (0,0)

I am able to reproduce this.
    tm.setPixel(0, 0, 3)

Above API

The underlying representation for the tilemap has changed from just being the provided image to support the new features, so changing pixels in the image directly won't be reflected in the tilemap; https://makecode.com/_irpFrCFevXK2 is the way of doing that that has been supported rather than relying on the passed image being the source of truth

A breaking change like this in middle of school year would be very bad...  tilemaps are used everywhere and arcade has quite a few users (despite still being in beta).

We should fix for backward compat.

This is still an issue.

I need to use this code instead of setPixel:
                    const tm = game.currentScene().tileMap;
                    tm.setTileAt(x, y, pixel);

But getPixel does work.

Closing this as we have a workaround for the tile issue (Use settile and gettile)


The min default text size isn't set correctly

Expected Behavior:
When you retrofit existing TextViews and change them to AutoFitTextViews in xml, they should resize correctly as more text is added to field.
Observed Behavior:
TextViews changed to AutoFitTextViews Don't resize correctly to the Default min size (8sp)
Reproducible Test:
In the project sample, remove the following line in main.xml
autofit:minTextSize="8sp"
see screenshot below
Reason:
AutoFitHelper.java:66 int minTextSize = (int) helper.getMinTextSize();
AutoFitHelper.java:75 minTextSize = ta.getDimensionPixelSize(R.styleable.AutofitTextView_minTextSize, minTextSize);
both of the returned sizes above are in Pixels...
AutoFitHelper.java:80 helper.setMinTextSize(minTextSize)
This method signature applies SP as no TypedUnit is specified. So, taking my density into account (xxhdpi) the density factor is 3. The min was/should be 24px but then the above method changed it to 72px
Proposed Solution:
change AutoFitHelper.java:80 to
helper.setRawMinTextSize(minTextSize);
helper.setPrecision(precision);

change AutoFitHelper.java:80 to
helper.setRawMinTextSize(minTextSize);
helper.setPrecision(precision);
ALSO not working


FileStore: create dst dir before move

Codecov Report

Merging #13 into master will increase coverage by 1.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master     #13      +/-   ##
=========================================
+ Coverage   27.48%   28.5%   +1.01%
=========================================
  Files           7       7
  Lines         211     214       +3
  Branches        8       8
=========================================
+ Hits           58      61       +3
  Misses        153     153

Impacted Files
Coverage Δ

core/src/main/scala/blobstore/fs/FileStore.scala
96.77% <100%> (+0.34%)
:arrow_up:

Continue to review full report at Codecov.

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

nice! I remember coming across this a few weeks ago.


Steps don't align with what is presented within Azure Enterprise Application integration

This page is really out of date with the settings that are presented to the integrator. You don't make mention of the Reply URL which is a mandatory field. The only way to find the answers to this is to Google and find other people who have configured the integration correctly (most likely by engaging Professional Services or Support).

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

ID: e69b84a3-f907-118d-8a02-76b1c4ba267c
Version Independent ID: c1468889-6502-372c-a718-3b59fe399375
Content: Tutorial: Azure Active Directory integration with AirWatch
Content Source: articles/active-directory/saas-apps/airwatch-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

https://communities.vmware.com/t5/Workspace-ONE-Discussions/Azure-Active-Directory-SAML-Federation-integration-with-AirWatch/td-p/2268300/page/2

@michaelcadogan Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.

#please-close


Performance improvement gap between the val and test set on ScanNet

Hi, thanks for your great work on 3D instance segmentation. I have a question about the performance on the ScanNet dataset.
I noticed that your results on the test set improved by 3.2% on the AP50 metric compared with the val set results. However, when I tried my own model on the ScanNet benchmark, I found that my val set and test set results are very similar. Do you know why ISBNet performs much better on the test set? I would love to know how to improve my own models.
Looking forward to your reply, thanks in advance!

I guess one reason is that the official test set comprises only 100 scenes, leading to more unstable results.


Documentation

Hey guys,
I'm currently trying your amazing package. During documentation reading I came to the point where you extend the model like this:
class User extends \Eloquent {

}

Are you sure this works? In my opinion there is no class with that name. What you mean is Illuminate\Database\Eloquent\Model, right?

class User extends \Eloquent is the laravel 4 way, some stuff of the documentation is updated, some stuff isn't
This is updated: https://github.com/jarektkaczyk/eloquence/wiki/Builder-searchable-and-more
This is not: https://github.com/jarektkaczyk/eloquence/wiki/Validable

Ah okay, perfect. Thanks!
I'm trying to use the Mappable trait but getting the same result as without the mapping :(

@bart About the Eloquent - I personally never use this alias, but it's perfectly fine to extend \Eloquent instead of the Model, (in L, not sure about Lumen) since it's alias.
About the mappings - let's talk about it in the forum

Thanks a lot @jarektkaczyk !


AQ (Antarctica) is not supported

Found on the log.

There is no standard currency code on Antarctica.


Discord API update: migrate to arikawa

Migrate Boe Tea to https://github.com/diamondburned/arikawa from discordgo to support new API features.

The need to migrate ASAP is gone, since discordgo updated their API, however it's still desired.


Dev/2 messages restructure

Restructure the repository to improve organisation and clarity.
Uses Visual studio projects/solution on Windows, and cmake to generate makefile targets under linux.
Provisionally adds cpplint configuration file ( and cmake target on linux) for code linting.

Build failure due to appveyor timeout - the instance didn't seem to come back up after the post-cuda install reboot.


vendor: point vishvananda/netlink back to upstream

Given PR https://github.com/vishvananda/netlink/pull/498 has been
merged officially, move the repo back to upstream. Final follow-up
to 515654b58beb ("vendor: fix stack corruption from retrieving
veth peer index"). Re-performed same update as in 515654b58beb.
Signed-off-by: Daniel Borkmann<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please

from master branch, please apply:
diff --git a/go.mod b/go.mod
index 4d2dc94e8..267428368 100644
--- a/go.mod
+++ b/go.mod
@@ -497,7 +497,7 @@ replace (
        github.com/ugorji/go => github.com/ugorji/go v1.1.4
        github.com/urfave/cli => github.com/urfave/cli v1.20.0
        github.com/urfave/negroni => github.com/urfave/negroni v1.0.0
-       github.com/vishvananda/netlink => github.com/cilium/netlink v1.0.1-0.20191111102915-47ae870a10ed
+       github.com/vishvananda/netlink => github.com/vishvananda/netlink v1.0.1-0.20191113183427-d71301a47b60
        github.com/vishvananda/netns => github.com/vishvananda/netns v0.0.0-20190625233234-7109fa855b0f
        github.com/vmware/govmomi => github.com/vmware/govmomi v0.20.1
        github.com/xiang90/probing => github.com/xiang90/probing v0.0.0-20190116061207-43a291ad63a2

and then run:
./contrib/go-mod/update-vendor.sh

This was exactly what I did. Are you asking as I need to rebase, or why ./contrib/go-mod/update-vendor.sh is pulling in all these direct dependencies? I don't know yet.. maybe golang version related?

test-me-please

Coverage decreased (-0.01%) to 46.038% when pulling c9ecd9d77291dfe73479a59d8058361a4f6b2030 on pr/netlink-update-vendor into 3b6e7373cdd904f33c9c82b22cd56a8baa51e62c on master.


Fix #909 - C2D README use remote

Fixes #909

@trentmc Shouldn't we keep a local c2d readme as well? All readmes should have a local version due to dependency issues, and I think we should discuss a solution to mitigate the differences between barge versions and remotely deployed versions. E.g. we could have a perfectly working Provider version on rinkeby, but we could have updates on barge that we are completely new and different. This renders the readme tests useless, since they will run on a "frozen" version set in the remote branch (Provider, Aqua), but will not trigger when updating the remote networks with new versions. This is opposed to the current flow where we always test the newest changes from barge and then adapt the remote networks, after making sure everything works.
Not sure if I'm explaining this right, but we can discuss more when available.

@trentmc Shouldn't we keep a local c2d readme as well?

No, not in this case. It means docker-inside-docker, which gives a Pandora's box of errors. (E.g. it doesn't work on my relatively vanilla ubuntu 22 setup.)

All readmes should have a local version due to dependency issues

I see that this will change over time: we will be having more remote-only readmes. That's where the practical usage lies. We'll keep simple local readmes and some remote counterparts, but let's not force 1:1.


error while running java sample

Log trace while running prepare.sh. Is there any maven project sample to test it out. Also please provide the path to download the jar directly, It will be really helpful for created a from scratch.
pradeep@pradeep-VirtualBox:~/iroha/example/java$ ./prepare.sh
-- -DCMAKE_BUILD_TYPE=Debug
-- -DTESTING=ON
-- -DBENCHMARKING=OFF
-- -DFUZZING=OFF
-- -DCOVERAGE=OFF
-- -DPACKAGE_ZIP=OFF
-- -DPACKAGE_TGZ=OFF
-- -DPACKAGE_RPM=OFF
-- -DPACKAGE_DEB=OFF
-- -DENABLE_LIBS_PACKAGING=ON
-- -DSWIG_PYTHON=OFF
-- -DSWIG_JAVA=ON
-- -DSUPPORT_PYTHON2=OFF
-- -DSWIG_CSHARP=OFF
-- -DSWIG_NODE=OFF
-- Could NOT find gtest (missing:  gtest_INCLUDE_DIR gtest_LIBRARY gtest_MAIN_LIBRARY gmock_INCLUDE_DIR gmock_LIBRARY gmock_MAIN_LIBRARY)
-- Could NOT find spdlog (missing:  spdlog_INCLUDE_DIR)
-- Could NOT find protobuf (missing:  protobuf_INCLUDE_DIR protobuf_LIBRARY protoc_EXECUTABLE)
-- Could NOT find grpc (missing:  grpc_LIBRARY grpc_INCLUDE_DIR gpr_LIBRARY grpc_CPP_PLUGIN)
-- Could NOT find rapidjson (missing:  rapidjson_INCLUDE_DIR)
-- Could NOT find pq (missing:  pq_INCLUDE_DIR postgres_INCLUDE_DIR pq_LIBRARY pg_config_EXECUTABLE)
-- Could NOT find pqxx (missing:  pqxx_INCLUDE_DIR pqxx_LIBRARY)
-- Could NOT find gflags (missing:  gflags_INCLUDE_DIR gflags_LIBRARY)
-- Could NOT find rxcpp (missing:  rxcpp_INCLUDE_DIR)
-- Could NOT find tbb (missing:  tbb_INCLUDE_DIR tbb_LIBRARY)
CMake Warning at /usr/share/cmake-3.5/Modules/FindBoost.cmake:725 (message):
Imported targets not available for Boost version
Call Stack (most recent call first):
/usr/share/cmake-3.5/Modules/FindBoost.cmake:763 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.5/Modules/FindBoost.cmake:1332 (_Boost_MISSING_DEPENDENCIES)
cmake/dependencies.cmake:70 (find_package)
CMakeLists.txt:113 (include)
CMake Warning at /usr/share/cmake-3.5/Modules/FindBoost.cmake:725 (message):
Imported targets not available for Boost version
Call Stack (most recent call first):
/usr/share/cmake-3.5/Modules/FindBoost.cmake:763 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.5/Modules/FindBoost.cmake:1332 (_Boost_MISSING_DEPENDENCIES)
cmake/dependencies.cmake:70 (find_package)
CMakeLists.txt:113 (include)
CMake Error at /usr/share/cmake-3.5/Modules/FindBoost.cmake:1677 (message):
Unable to find the requested Boost libraries.
Unable to find the Boost header files.  Please set BOOST_ROOT to the root
directory containing Boost or BOOST_INCLUDEDIR to the directory containing
Boost's headers.
Call Stack (most recent call first):
cmake/dependencies.cmake:70 (find_package)
CMakeLists.txt:113 (include)
-- Could NOT find ed25519 (missing:  ed25519_INCLUDE_DIR ed25519_LIBRARY)
-- Could NOT find SWIG (missing:  SWIG_EXECUTABLE)
-- Installed package SWIG not found. It will be pulled to /home/pradeep/iroha/external/src/swig_swig/share/swig/3.0.12 at compile time.
CMake Error at shared_model/bindings/CMakeLists.txt:49 (swig_add_library):
Unknown CMake command "swig_add_library".
Call Stack (most recent call first):
shared_model/bindings/CMakeLists.txt:91 (myswig_add_library)
-- Configuring incomplete, errors occurred!
See also "/home/pradeep/iroha/example/java/build/CMakeFiles/CMakeOutput.log".
See also "/home/pradeep/iroha/example/java/build/CMakeFiles/CMakeError.log".
make: *** No rule to make target 'irohajava'.  Stop.

@pradeeppadmarajaiah please install latest CMAKE (3.8 at least, 3.11 idealy). Also, AFAIK you need to have boost 1.65 installed.

Seems resolved


NoMethodError: undefined method `key?' for #JSON::Ext::Generator::State:0x007f77fc9a6818

I'm receiving this error NoMethodError: undefined method 'key?' for #<JSON::Ext::Generator::State:0x007f77fc9a6818> when some users view certain Customer records in my database that are clearly there.  The URL someone will visit is www.domain.com/api/v1/customers/147791?key=af0shsdg0sg as an example.  key is passed and checked against a field called account.api_key for authentication.  Can't seem to figure out why an error appears on some records and not others, any idea?


Add an ability to check class naming

I was thinking about the ability to use this tool to check and enforce proper class naming. For example, every class using the Symfony's AsMessageHandler attribute should have a suffix Handler. Such assertion could be called ShouldHaveNameMatchingPattern. I was looking into the source code to see how it could've been implemented, and I was thinking about something similar as it is currently in shouldApplyAttribute() where you define selectors in the next TargetStep.
Let me know what you think about this feature.

Hi. This would be a declaration assertion instead of a relation assertion. But right now (AFAIR) declaration assertions do not accept parameters.
I think some magic should be done (maybe with generics) in order to accept them.
I've been busy and traveling lately, but I think I can have a look between today and tomorrow.

Hi. Sorry for the delay, it's been a challenging month in both personal and professional context.
I have finally been able to make some quick-and-dirty changes to make this assertion possible.
Please, let me know if you see any weird behaviour.
Released with https://github.com/carlosas/phpat/releases/tag/0.10.12 🚀

Thanks for your work @carlosas !


Do not run the notify_ebpf_complexity_changes job on deploy pipelines nor on main

What does this PR do?

Do not run the notify_ebpf_complexity_changes job on deploy pipelines nor on main
Declared the ebpf team jobowner

Motivation

This job consistently fail on deploy pipelines because the env variable that is used (CI_COMMIT_BRANCH) is not available when running a pipeline on a tag (see https://docs.gitlab.com/ee/ci/variables/predefined_variables.html)
I don't think it makes sense at all to run this job on tags

Additional Notes

I dropped the - when: on_success from the rule because the other except rules don't have it
Having it forces you to use this rule as the very last one in your config (see the windows config for instance), so you can't mix several of them that might have this `- when: on_success
Moving the except rule should prevent this job from running on main
We tested this config with the simulator on GitLab and it seems to work as expected

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Include // camel-k header to allow camel-k tooling assistance

Lets include in the top comment as below
# camel-k: language=yaml

❯ camel init bar.yaml
❯ cat bar.yaml
# camel-k: language=yaml

# Write your routes here, for example:
- from:
    uri: "timer:yaml"
    parameters:
      period: "1000"
    steps:
      - setBody:
          constant: "Hello Camel from yaml"
      - log: "${body}"

As this allows tooling assistance for camel-k textual tool

Instead of doing this then karavan can name the files xxxx.camel.yaml - then tooling will have code assistance
https://issues.apache.org/jira/browse/CAMEL-18622
This requires Camel 3.18.3 to have the loader supported in jbang


bug: $fromMillis various issues

Steps to reproduce
Pattern: $fromMillis($millis(),'[Y01][M01][D01]')
Evaluating this pattern will result in two separate issues

arg[0] $millis() panics with 'Not a number'
[Y01] also errors with 'Unsupported datetime picture string: Y01'

fix available in v0.3.1 and above.


Fixed compatibility of unit testing with Xcode 8

codesign libraries to make Xcode 8 beta >= 4 happy
got rid of VerifySimulators since private API SimVerifier is no more
detect frameworks being loaded through overriding NSBundle behavior
private DTiPhoneSimulatorSystemRoot API disappeared. Then we replaced it with traversing the filesystem and parsing .plist files.

@ExtremeMan

Let's wait unit tests to go through. I think I got all them to work but couldn't a last one because I probably don't have a proper local environment for testing.

#704

Very nice. Have you tried running tests both under Xcode 7 and Xcode 8?

Surprisingly, I haven't tried the unit tests on Xcode 8. I'm doing it.

Looks like build failed. Try running ./scripts/travis-ci.sh locally.

Making progress: 4 tests failing instead of 6.

Very nice. Should we just drop SenTestingKit support?

ha yeah, everything boils down to it :)
Could you give me some guidance for that?
I can start with removing the unit tests and then in second pull request we can try to burn everything else SenTestingKit.

Sounds good to me. Kill all SenTestingKit tests to unblock this PR. In follow up we can assert in Xcode 6 + remove everything related to that old guy.

🎉


Unable to set API_URI

What I'm trying to achieve
I am unable to set API_URI via env variable it always defaults to localhost
Steps to reproduce the problem

set API_URI="http://saleor-api.default:8000/graphql/"
deploy container
check env var inside of  container echo $API_URI > http://saleor-api.default:8000/graphql/
open js console in the browser all api requests points to http://localhost:8000/graphql/

What I expected to happen
API_URI env variable to be respected  and points to http://saleor-api.default:8000/graphql/
All containers are build from Dockerfile provided in  2.10.4.zip
I'm also experiencing the same issue with Dash container 2.10.0.zip
I am using k8s internal dns

Did you fix API_URI in Dockerfile.dev?
I fixed in Dockerfile.dev and run docker compose build again.
And API_URI changed.

I do not try  Dokerfile.dev assumed its not for production use

Dockerfile.dev does not do build phase, I suspect there is something wrong with npm run build ?

Oh okay. Never mind. You used Dockerfile file.
Did you fix API_URI in the Dockerfile and build the image?
I faced a problem similar to yours. What I did is I built the docker image again after changing API_URI.

Looks like it was caching issue works now


[BEAM-6918] Changed github link to not require login and fixed broken example link

git clone git@github.com:apache/beam.git requires the user to be logged in, while https://github.com/apache/beam does not.
Spaces in the example link need to be escaped.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @qinyeli

Thanks, David. I can't merge the PR, though, as I am not a Beam commiter.
R: @pabloem Would you be able to review this?

Done! Thanks!


fixes: Most of the grammar issues in the code. #11429

Fixes #11429
Just a smile modification of grammar
changing an to a wherever it was wrong earlier.
Please check the following:

[x] Do the tests still pass?[^1]
[x] Does the code comply with the style guide?

[x] Run make lint from the Wagtail root.

[ ] For Python changes: Have you added tests to cover the new/fixed behaviour?
[ ] For front-end changes: Did you test on all of Wagtail’s supported environments?[^2]

[ ] Please list the exact browser and operating system versions you tested:
[ ] Please list which assistive technologies [^3] you tested:

[ ] For new features: Has the documentation been updated accordingly?

Please describe additional details for testing this change.
[^1]: Development Testing
[^2]: Browser and device support
[^3]: Accessibility Target

I have rebased and will add you to the contributors list as 'Vaishnav Dasari' - let me know if you want a different name listed Thanks @SpeedGotFried  - really appreciate the help here.

Just a reminder for the future, it's recommended you create a fresh branch for contributions.
Have a read of this section and maybe this page for more details.
https://docs.wagtail.org/en/stable/contributing/first_contribution_guide.html#create-a-fresh-branch-for-your-contributions


Update detect-enable-and-disable-smbv1-v2-v3.md

Article incorrectly stated that both SMBv1 and SMBv2 status could be determined by reading LanmanWorkstation service entry.  It will show one of the dependencies for LanmanWorkstation service being mrxsmb20 however that does not tell you if that is enabled or not.

@robinharwood, @Xelu86
Can you review the proposed changes?
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Not clear how to add sub-segment seed points

Seed points are added by Shift + Click but this is not made clear by the UI, nor is it clear how seed points can be removed.
Reported by Vince Smith.

To be handled by #185


Fix formatting of Kubernetes Meets High Performance blog post

@urzds Thanks for this fix! Things look good on the test server, so let’s approve it.
/lgtm
/approve


Use single connection for both schema history table management and applying migrations

What

Fix for GH Issue 23327 (Flyway migrations with ALTER TABLE ... RENAME COLUMN may fail with YBDB 2.20+)

Currently, the plugin uses a separate connection to manage schema history table and migrations are applied in another thread. This was causing catalog version mismatch issues when one thread altered catalog version while another was querying a pg_catalog.pg_class table inside a transaction.
The fix is to use single connection for both schema history table management and applying migrations

Also, made tableEntries map a static variable in YugabyteDBExecutionTemplate

This map keeps track of which entries are added to the lock table across different migrations from within the same app. Earlier, being an instance variable it was getting reinitialised for each migration and not serving its purpose.

Testing

Ran modified tests from flyway-tests repository. PR available here.

This PR is for review only. It'll be raised with upstream repository's main.

lgtm.
One minor clarification seeked.

Just in case, different threads in an application doing migrations try to update the map at the same time.


Installation Failure on Ubuntu 22.04 Due to Docker Compose Command Recognition

Issue: Installation Failure on Ubuntu 22.04 Due to Docker Compose Command Recognition Error
Description
Upon testing on Ubuntu 22.04 with the latest versions of Docker and Docker Compose, the installation and activation of the script ./docker-setup-ubuntu.sh fail. This is attributed to Docker Compose without the hyphen not being recognized as a valid command. After rectifying this factor, the script functions properly.
Proposed Solution
To resolve this issue, we need to ensure Docker Compose commands are recognized correctly without requiring a hyphen.
Action Plan
Open a PR with the fix
Additional Information:
OS: Ubuntu 22.04
Docker Version: Latest
Docker Compose Version: Latest

Another error with --quiet command

Hi @ethanlacerenza
We will review your pull request and merge. Thanks for the reporting the issue with a fix in a pull request.

I approve :+1:

@0xbharath  @Rancho-rachit @Prateek-Thakare
Hi, I've noticed that the scanning data doesn't appear on the dashboard, even after following the tutorial and guide. I've tried both on Mac and Ubuntu.
Is this a known issue?
I've precisely followed the steps outlined in the readme and documentation.

Hey @ethanlacerenza
This should not happen, are you entering the correct db host and port? Also, was your setup successfully completed?

Hey @Prateek-Thakare,
The installation is progressing smoothly without any errors. I ran this command before setting up the dashboard and completed the scanning: mantis onboard -o org_name -t example.in
I followed this video tutorial to set up the database, following the same steps, and encountered no errors: Tutorial Link. However, the result doesn't seem to appear on the dashboard.

Hi @ethanlacerenza
Can you try ssh-ing into MongoDB docker and see if mantis db and collections are created? Conversely, can you run Mantis again and check if you are getting any errors?

I see only assets_collections

I found the issue, and it was related to a permissions matter, which wasn't related to Mantis since the databases were being created correctly. Thank you very much.
If you need me, I'm here.

Thanks, closing this. Also thanks for the PR, we have merged the same.


[Bug] SABnzbd - Paths and Categories - nested incomplete

Which Guide
SABnzbd - Paths and Categories
Describe the Error in the Guide
It shows the incomplete folder being nested inside of the completed folder.
/data/usenet/incomplete-downloads vs /data/usenet
Expected behavior
The incomplete folder should be adjacent to prevent issues if a script/app is processing the completed folder recursively.
Screenshots
For the context of that page it may just be easier to blur out incomplete folder since the context is showing the whole completed folder + category concept.

Thnx will try to blur it this evening

one is not nested while one is.
so if something was to process your example "/data.usenet" recursively, it would see the "/data/.usenet/incomplete" stuff as its nested within it. its recommended to have incomplete and complete adjacent of each other.

/downloads/incomplete
/downloads/complete
/data/usenet/incomplete
/data/usenet/category-name
So the issue would only be if you don't use categories ?

Yes
As I noted on discord, it's not a huge deal because the content of that page is showing the whole let's do categories to segment stuff. If they follow the whole page then it's fine, it's just of course people doing half way or doing dumb things where there ends up being collisions.

i will create a new screenshot, and change the instructions, to cover is correctly


Day200

Resolves #411.

Pull Request Test Coverage Report for Build 761

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

Totals

Change from base Build 750:
0.0%

Covered Lines:
5

Relevant Lines:
5

💛  - Coveralls


Thank you

This is: minimal, useful, and ruthlessly simple.   Thanks!  🚀

Thank you 🙌 I needed that, had a brutal week.


Add port in development to fix password reset link sent to email

related to #1266, doesn't fix it yet
It fixes email link sent in development environment to have correct port
http://localhost/... --> http://localhost:3000/...

Codecov Report

Merging #1269 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1269   +/-   ##
=======================================
  Coverage   93.54%   93.54%
=======================================
  Files          60       60
  Lines        1332     1332
=======================================
  Hits         1246     1246
  Misses         86       86

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 3abb737...7a58cce. Read the comment docs.


Add CI via Github Actions

Based on updates made in InitialMassFunctions.jl.

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:

Actions are running and completing except for Julia 1.7, which succeeds in a few minutes on my local machine but which runs for 10+ minutes on the github runner (1.10 runs in <3 minutes). Not sure why 1.7 is so slow here. It is useful to test our compatibility lower bound (we support versions >= 1.7) but not very useful if the runs never finish.


key.released event doesn't always seem to fire when a key is released

This makes it look like certain keys are stuck and are always pressed.
This is especially bad if I want to respond to a key-combination like Alt+F1.
I can likely create a small repro case if this is not a known issue and is doubted, just let me know.
I also realized just now that maybe I can also create some kind of an abstraction to hide this issue, because genuine stuck/pressed keys repeatedly fire new key.pressed events, so if for about a second I don't get any of those, I can safely assume that its due to this bug and the key is in fact not pressed.
This can work, though of course it would be preferable that this is fixed somewhere upstream because this would introduce a noticeable lag on occasion when responding to a user releasing a key.

Somewhat off-topic, but I am also baffled by the lack of available and reliable global keyboard hooks for node.js. For a while I was using iohook, until I realised that it doesn't at all work when two apps require it at once. It makes Windows OS freeze. That's a pretty big deal-breaker.
Now, gkm, this package, seems to work much better, but like I say above, it's not fully reliable. And it also seems to not be actively maintained? And yeah, maybe I can smooth out these rough edges and live with this, but I am perplexed as to what people actually use if/when they want to respond in Node to a global keyboard shortcut. Do people just never want to do that? Do they use something I am not aware of?

Attempting to work around the issue with this:
let pressMap = {}
let onKey = fn => {
  gkm.events.on('key.*', async function([key]) {
    if (this.event == "key.typed") return
    if (this.event == "key.pressed") {
      clearTimeout(pressMap[key])
      pressMap[key] = setTimeout(_ => {
        pressMap[key] = null
        // this.event = "key.released"
        // fn.call(this, [key])
        fn.call({event: "key.released"}, [key])
      }, 1000)
    }
    if (this.event == "key.released") {
      clearTimeout(pressMap[key])
    }
    fn.call(this, [key])
  })
}

And noticed a weird behavior: When I keep pressing alt, the alt event repeatedly fires; but when I then also start long pressing control, the repeated alt firing stops and repeated ctrl firing takes its place, even though alt is still pressed. This can be a problem if the user hesitates for a long time while pressing stuff.
As in, the above code works around the initial issue, but fails in the edge-case when the user is slower than 1s in a long key combination.
Though I might have an idea on how to overcome this issue too by treating repeated firings as a key group being pressed. Though that might also run a different risk of consecutive long-held combinations appearing as if being held together.
However, I also noticed a much more worrying problem: If multiple events coincide, some of them seem to be fully dropped! Even key.pressed events! In my logs I see stuff like:
pressed    alt
released   alt
released   control

Note how according to gkm, control was never pressed, it's just released out of nowhere.


Support text alignment for typography

Allow textAlign css property to be used when defining text styles.
Fixes #16

Closing this as required more thought. See discussion in #16


External Props extend broken

Environment
[6:23:12 PM] Working directory: C:/code/test-app
[6:23:12 PM] Nuxt project info:

Operating System: Windows_NT
Node Version:     v20.8.1
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-js9bi8-ygsjlv?file=app.vue
Describe the bug
Extending props from an external file seems to be broken in the latest version (3.8.2). After starting the app you can see the error:
[plugin:vite:vue] [@vue/compiler-sfc] Failed to resolve extends base type.
If this previously worked in 3.2, you can instruct the compiler to ignore this extend by adding /* @vue-ignore */ before it, for example:

interface Props extends /* @vue-ignore */ Base {}

I created a new vue app (no nuxt) to see if it was a vite or vue issue but it seemed to work ok without nuxt.
Additional context
No response
Logs
at ScriptCompileContext.error (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:15670:11)
    at resolveInterfaceMembers (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:17936:13)
    at innerResolveTypeElements (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:17762:14)
    at resolveTypeElements (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:17750:35)
    at innerResolveTypeElements (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:17795:16)
    at resolveTypeElements (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:17750:35)
    at resolveRuntimePropsFromType (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:19186:20)
    at genRuntimePropsFromTypes (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:19162:17)
    at genRuntimeProps (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:19152:18)
    at Object.compileScript (C:\code\test-app\node_modules\@vue\compiler-sfc\dist\compiler-sfc.cjs.js:20260:21

This is a very bad error message but Vue is trying to tell you that you need to install typescript to make this work. (It is required for inferring runtime prop types from pure TS).
Working reproduction: https://stackblitz.com/edit/github-js9bi8-pz6t3q?file=package.json
Ideally we could somehow intercept the error and show it (cc: @antfu).

Closing as https://github.com/vuejs/core/pull/8883 is merged and released


Added rules and build scripts

Replaces https://github.com/Azure/azure-open-api-validator/pull/1

@dsgouda,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@dsgouda,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

https://gist.github.com/sergey-shandar/f175b4343dfbe923263275d6181f0dfb

@olydis this is the very first PR I wish to merge.

Nope, fixed.


FederatedTypesPlugin: Type defention in "lib-app"

Hello.
Is it possible to expose library in FederatedTypesPlugin like it possible in ModuleFederationPlugin (https://github.com/module-federation/module-federation-examples/blob/master/complete-react-case/lib-app/webpack.config.js#L17)?
I got an error when I tried to do this:  [FederatedTypesPlugin] Error: rootDir must be a directory

it should be, there is some ongoing effort/requests to remove module federation from type plugin so that you can use both independently instead of type plugin configuring module federation for you.
Happy to take a PR if you have a desire to move it forwards

@special-k I'm developing a federated types library: https://github.com/ilteoood/native-federation-typescript
Would you like to try it and give me a feedback?

Hi @special-k,
the native-federation-typescript plugin is now part of module-federation!
Did you had the opportunity to try it?
https://www.npmjs.com/package/@module-federation/native-federation-typescript

Hi @ilteoood
I have tried native-federation-typescript plugin, Able to create @mf-tests.zip but it throws error Unable to compile federated types, Error: File not found: dist/@mf-types. I have used default config and also custom config but facing the same issue. Could you please help

@sudheerkumar-golla please provide a repro

Sorry @ilteoood, I can't share the company repo. If you want to see certain config. I can share.

A repro is not your company repository, is a basic repository which can be used to reproduce the issue. You can create it by yourself

I am having OP's issue; I don't know enough about out repo or this package to make a minimal example that reproduces, but perhaps a log would still be helpful. This started happening when I tried to upgrade from legacy yarn 1.x to the newer 3.x line.
We are using "@module-federation/typescript": "2.4.2", but this seems to happen with "@module-federation/typescript": "3.1.1", too.
    [FederatedTypesPlugin] Preparing to download types from remotes
    [FederatedTypesPlugin] No Remote components configured
    [FederatedTypesPlugin] Preparing to Generate types
<e> [FederatedTypesPlugin] Error: rootDir must be a directory
<e>     at TypescriptCompiler.getFilenameWithExtension (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:231:13)
<e>     at TypescriptCompiler.getNormalizedPathWithExt (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:113:34)
<e>     at map (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:48:34)
<e>     at Array.map (<anonymous>)
<e>     at TypescriptCompiler.normalizeFiles (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:104:18)
<e>     at TypescriptCompiler.generateDeclarationFiles (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:45:46)
<e>     at FederatedTypesPlugin.compileTypes (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/plugins/FederatedTypesPlugin.ts:105:23)
<e>     at /home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/plugins/FederatedTypesPlugin.ts:85:31
<e>     at Hook.eval (eval at create (/home/dhorkin/repos/*****/.yarn/cache/tapable-npm-2.2.1-8cf5ff3039-3b7a1b4d86.zip/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:7:1)
<e>     at Hook.CALL_DELEGATE [as _call] (/home/dhorkin/repos/*****/.yarn/cache/tapable-npm-2.2.1-8cf5ff3039-3b7a1b4d86.zip/node_modules/tapable/lib/Hook.js:14:14)
<e> [webpack-dev-middleware] Error: rootDir must be a directory
<e>     at TypescriptCompiler.getFilenameWithExtension (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:231:13)
<e>     at TypescriptCompiler.getNormalizedPathWithExt (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:113:34)
<e>     at map (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:48:34)
<e>     at Array.map (<anonymous>)
<e>     at TypescriptCompiler.normalizeFiles (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:104:18)
<e>     at TypescriptCompiler.generateDeclarationFiles (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/lib/TypescriptCompiler.ts:45:46)
<e>     at FederatedTypesPlugin.compileTypes (/home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/plugins/FederatedTypesPlugin.ts:105:23)
<e>     at /home/dhorkin/repos/*****/.yarn/__virtual__/@module-federation-typescript-virtual-deb392ac40/0/cache/@module-federation-typescript-npm-2.4.2-a02ec556e9-0a270d2e74.zip/packages/typescript/src/plugins/FederatedTypesPlugin.ts:85:31
<e>     at Hook.eval (eval at create (/home/dhorkin/repos/*****/.yarn/cache/tapable-npm-2.2.1-8cf5ff3039-3b7a1b4d86.zip/node_modules/tapable/lib/HookCodeFactory.js:19:10), <anonymous>:7:1)
<e>     at Hook.CALL_DELEGATE [as _call] (/home/dhorkin/repos/*****/.yarn/cache/tapable-npm-2.2.1-8cf5ff3039-3b7a1b4d86.zip/node_modules/tapable/lib/Hook.js:14:14)
<

@FFdhorkin  did you resolve this problem I just got around to setting up typescript and I'm running into the same issue.

Just use the built in typescript capabilities in federation v2?


Add a render function for keyboard shortcuts panel

Summary
Add a prop renderKeyboardShortcutsPanel that allows a custom panel for keyboard shortcuts to be rendered.
Screenshots
Default

renderKeyboardShortcutsPanel

@ljharb how long should the coverage check take to run? 🤔

This seems to be the issue @noratarano and i were working on; i believe recent permissions changes caused this to be broken. I’m not sure how to fix it :-/
A short term workaround could be to disable coveralls as a required check; a longer term workaround could be to replace coveralls with another service; ideally we can just figure out how to fix coveralls.

@ljharb ahh that'd do it, any chance it's fixed now? coveralls finished running at least but now I'm seeing "Coverage decreased (-68.7%) to 11.246%" on this PR which seems odd

@caseklim yep, should be all set now! try a rebase, and if it's still claiming coverage has dropped then more tests need to be added.

@ljharb but a coverage drop of -68.7% to 11.246%? I added a test for my new render function, and it's complaining about files I didn't even touch. e.g.,

It might be related to the issues coveralls was having - ie, if master was updated while coveralls was down, it might not know about the new coverage.
@noratarano can merge through the failing coverage check, which i think is fine here.

@caseklim @ljharb Merging!