Grafana v5.4.3 (6539180) and Zabbix 4.0.4 acknowledge not working

Hi,
I installed newest Zabbix Version and Grafana v5.4.3 with the grafana-zabbix plugin. I test with some PROBLEMS in Zabbix. Erros are displayed correct in grafana interface.
But I can't acknowledge them in the grafana interface, nothing happend. Look at my screenshot. What is the problem?

Hi,
okay I used your document to run from master:
https://github.com/alexanderzobnin/grafana-zabbix/blob/master/docs/sources/installation/run_from_master.md
Is there no other way to install the plugin with working acknowledge? I won't installed some extra on my monitoring server. I tested your document on my test system and acknowledge is working fine.
Is there no other way to use the master plugin?
Regards

I'm working on the new release, hope, it will be available soon.

Hello,
Thanks for fast reply. Okay understand but I have short question. Is it possible to upgrade from the master branch to the next release in the future if you will release it? Because now I installed it via yarn setup. Is that a problem? Or will I have problems with the next release?

I'm working on the new release, hope, it will be available soon.

Hello,
Thanks for fast reply. Okay understand but I have short question. Is it possible to upgrade from the master branch to the next release in the future if you will release it? Because now I installed it via yarn setup. Is that a problem? Or will I have problems with the next release?


[Bug] Error when typing accent grave (like "à")

Hello,
I'm using MarkEdit version 1.20.3 (68) on a MacBook Air M2 with Sequoia 15.1.1.
I write in Catalan and Spanish, and I can't get the grave accents to work correctly. When I write a vowel with a grave accent, MarkEdit writes the grave accent and then the accented vowel: "`à". I have detected this problem in the latest version. I don't remember if the previous version also had it. I know that in previous versions, the issue did not exist.
Acute accents work correctly ("á").
Thanks for your excellent Markdown editor!
Francesc Rosés

Hi,
Yes, exactly!
The accents in Spanish keyborards are what we call dead keys. You hit the accent key, nothing happens because is a dead key, and then, when you type the vowel, the accent appears over the vowel.
It works fine with acute accents (like “á”), but not with grave accents (like “àèò”).
Note: Spanish do not have grave accents, but Catalan, as French, for instance, does.
Regards,
Francesc

On 23 Nov 2024, at 01:27, Gnohz Gniy @.***> wrote:
I just learned a bit about typing grave accents in Spanish, are you talking about typing ` and a consecutively?
—
Reply to this email directly, view it on GitHub https://github.com/MarkEdit-app/MarkEdit/issues/767#issuecomment-2495143508, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6VTSLZP4Z6TOINR7GBWDT2B7DXBAVCNFSM6AAAAABSJ7MZVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGE2DGNJQHA.
You are receiving this because you authored the thread.

Thanks a lot!
Francesc

On 23 Nov 2024, at 01:47, Gnohz Gniy @.***> wrote:
The fix is in #768 https://github.com/MarkEdit-app/MarkEdit/pull/768; I'll tag a release in the next few days.
As a temporary workaround, you can disable editor.autoCharacterPairs https://github.com/MarkEdit-app/MarkEdit/wiki/Customization#editorautocharacterpairs.
—
Reply to this email directly, view it on GitHub https://github.com/MarkEdit-app/MarkEdit/issues/767#issuecomment-2495158771, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6VTSIBZ6WFXFLQEEBSEDT2B7GCTAVCNFSM6AAAAABSJ7MZVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGE2TQNZXGE.
You are receiving this because you authored the thread.

Hi, Gnohz,
It works!
Regards,
Francesc

On 23 Nov 2024, at 01:47, Gnohz Gniy @.***> wrote:
The fix is in #768 https://github.com/MarkEdit-app/MarkEdit/pull/768; I'll tag a release in the next few days.
As a temporary workaround, you can disable editor.autoCharacterPairs https://github.com/MarkEdit-app/MarkEdit/wiki/Customization#editorautocharacterpairs.
—
Reply to this email directly, view it on GitHub https://github.com/MarkEdit-app/MarkEdit/issues/767#issuecomment-2495158771, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6VTSIBZ6WFXFLQEEBSEDT2B7GCTAVCNFSM6AAAAABSJ7MZVKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIOJVGE2TQNZXGE.
You are receiving this because you authored the thread.


TypedArray subarray method is slow in V8

Which makes using buffers pretty slow.
Follow up https://bugs.chromium.org/p/v8/issues/detail?id=7161 and https://github.com/nodejs/node/issues/17431 for possible solutions or workarounds.

You might find this interesting, people have proposed removing the species subclassing behavior altogether, because it also has security issues.  And sounds like it's related to the perf issue here.
https://github.com/tc39/proposal-rm-builtin-subclassing


[WIP][POC] Introduce json report generation

That is an ugly hack to make leapp preupgrade produce
a json report alongwith default text one.
Json report will be present in the same directory as the
text report.

Can one of the admins verify this patch?

Looks good to me personally.

Before we merge, shouldn't the json report be mentioned in the output too? Or will we make a separate PR for that?

@fernflower Additionally, when we are doing the change, I would like to see change in leapp-repository as well. One of reasons to have the paths defined was to have just one source of true. See the line. When you do change, I expect there is a way to get the correct path so it will not be hardcoded here. Or the actor will be modified so the path will not be needed.

but still I am unhappy that path is again hardcoded (and not sure how to provide the info to actors). even the dir doesn't look to me like something that's needed as it is the same one used for logs. I think that better solution would be to provide paths report_txt and report_json as with that the info doesn't have to be hardcoded anywhere else.

@pirat89 I get you, I created a bug/enhancement https://github.com/oamg/leapp/issues/531


Fix SD card root image

Apparently media.discordapp.com and cdn.discordapp.com have stopped working unless viewed from within discord itself. Clicking "open original" and removing everything after the ? also results in messages like "This content is no longer available."
For now, just copy the images into the repo. The only instances of this were for English and Spanish, so I have handled both of them, though a French version of the image also exists, but is not used on the guide.
The better solution is to copy the HTML that is used to generate the image into the repo itself, so it can be translated with the guide. However, I don't have the web dev skills or knowledge to fix that. If anyone can step up to do it, please do!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

Can you add the Spanish hyperlink through Crowdin? Nothing wrong with adding the image in the PR, but Crowdin is going to reset the link upon next translation merge until someone goes there and fixes it

meh actually, I think I'll just do it

Thanks!


:white_check_mark: test(core): Add 100% test coverage

Description

Bump min test coverage to 100 100 100 100

Testing
Lots
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #366

PR #367   👈

This comment was auto-generated by Graphite.


Fixing the 'Save Changes' button inconsistency with owner dropdown menu

Description:
In the current version, if a user were to edit a bot, and click on the dropdown to change the owner but clicks on the same owner, the 'Save Changes' button will become enabled. This is obviously wrong because no changes have been made, but the bot settings is implying that a change has been made. There are 2 things to keep note when fixing this issue:

If the 'Save Changes' button is currently disabled, and a user clicks on the owner dropdown and selects the same owner, it should remain disabled - since no change has been made.

If the 'Save Changes' button is currently enabled because other edits have been made for this bot, and a user clicks on the owner dropdown and selects the same owner, the 'Save Changes' button should remain enabled.

Fixes #24568
Added changes to make sure the save changes button remains disabled if the user clicks on the current bot owner from the dropdown menu. However, if there have already been changes in the bot settings, this change is making sure to not disable the Save Changes button.
Demo of my fixed changes:
https://user-images.githubusercontent.com/16692028/230746382-0cde8b02-730b-4924-95cf-71219e5e0a5b.mov

Thanks @tpark1218 ! Did you test changing the owner and then changing it back (without saving changes)? Can you post a video confirming that the button becomes disabled again?

Closing as the original issue was fixed via https://github.com/zulip/zulip/pull/24910.


cannot update an action with just parameters

Steps to reproduce the issue:
1.  touch foo.js
2.  wsk action create foo foo.js -p a A
3.  wsk action update foo -p b B
4.  wsk action get foo parameters

Provide the expected results and outputs:
ok: got action foo, projecting parameters
[
    {
        "key": "b",
        "value": "B"
    }
]

Provide the actual results and outputs:
ok: got action foo, projecting parameters
[]

The test coverage was missing a case of parameter update without an exec defined. Also we are missing tests for parameter update from CLI, and tests missing for projecting properties (these are not supported by go CLI).
@mdeuser @dubeejw FYI.

Closed by #818.


Lovelace sensor card not showing graph

Hey guys,
Updated to most recent HA because id love to use the new sensor graph card! Great job on that one!
Updated and implemented it in my lovelace config; it nicely shows the value, however no graph...
These are the config lines i got:
- type: vertical-stack
cards:
- type: sensor
entity: sensor.gewicht
name: Gewicht
hours_to_show: 168
This is my weight so i got it set to  168 to show a graph for last week. What am i missing?

/move to home-assistant/home-assistant-polymer


Add SVCB and HTTPS record type support

/cc https://github.com/octodns/octodns/pull/1176 which added support for them in octoDNS core
/cc @pieterlexis-tomtom fyi

This will fail until https://github.com/octodns/octodns/pull/1176 & https://github.com/octodns/octodns/pull/1179 have shipped in a new release.

Werid, still not getting 1.8.0. Maybe it's pulling from a pypi mirror or something. Will pick this back up tomorrow.


Update free-programming-books-langs.md

Add PROGRAM = PROOF Book

I think you meant to put this in the Mathematics section, not the Mathematica section


Data UI

Front-end UI modified
@CodiumAI-Agent /review

PR Review

 ⏱ Estimated effort to review [1-5]
2, because the PR includes both backend and frontend changes, but the modifications are straightforward and mostly involve adding new features and configurations.

 🧪 Relevant tests
No

 🔍 Possible issues
CORS Configuration Duplication: The CORS configuration is defined both in routes/routes.js and server.js, which might lead to confusion or misconfiguration in the future.

 🔒 Security concerns
No

 Code feedback:
relevant fileroutes/routes.js
suggestion

Consider removing the manual CORS configuration here since you've already configured CORS globally in server.js using the cors package. This will help avoid potential conflicts or confusion regarding CORS settings. [important]

relevant linerouter.use((req, res, next) => {
relevant fileserver.js
suggestion

Since you're using the cors package for handling CORS, you might want to configure it more precisely for security reasons, such as specifying the exact origins that should be allowed instead of using '*'. This will help prevent unwanted cross-origin requests. [important]

relevant lineorigin: '*',
relevant fileclient/frontEnd/src/Components/GraphicsCard.jsx
suggestion

It's a good practice to handle loading and error states when fetching data. Consider adding state variables for loading and error, and display appropriate feedback to the user based on these states. [medium]

relevant lineconst response = await axios.get('http://localhost:5000/GPU');
relevant fileclient/frontEnd/src/Components/Processor.jsx
suggestion

Similar to the GraphicsCard component, consider handling loading and error states for a better user experience. This can be done by introducing state variables for loading and error, and updating them appropriately during the data fetching process. [medium]

relevant lineconst response = await axios.get('http://localhost:5000/CPU');

 ✨ Review tool usage guide:
Overview:
The review tool scans the PR code changes, and generates a PR review which includes several types of feedbacks, such as possible PR issues, security threats and relevant test in the PR. More feedbacks can be added by configuring the tool.
The tool can be triggered automatically every time a new PR is opened, or can be invoked manually by commenting on any PR.

When commenting, to edit configurations related to the review tool (pr_reviewer section), use the following template:

/review --pr_reviewer.some_config1=... --pr_reviewer.some_config2=...

With a configuration file, use the following template:

[pr_reviewer]
some_config1=...
some_config2=...

See the review usage page for a comprehensive guide on using this tool.


Hi!

Just wanted to say thank you for your theme! Tested it in complex languages like typescriptreact. What a consistent result! Types remain types, constants are constants everywhere and so on. Even a so popular among treesitters dracula theme gave me different colors for same syntax types in different places in the code.
The only suggestion, please, promote your theme! A lot of people enjoy onehalf colorscheme, and you work in this field is robust. With lua configs it became a pain for some developers to switch to similar treesitter light themes. It would be great if you created a PR in nvim-treesitter repo  to add your theme into the list of suggested. Plus you can create a thread in the r/neovim reddit channel.

Thank you for the kind words :heart: Most of the work is done by Lush. I just copied the colorscheme from onehalf.
Sure, I can make a post on Reddit. I updated the readme to include a preview.

I posted it on reddit.
https://www.reddit.com/r/neovim/comments/sh8y3w/onehalflush_a_light_colorscheme_based_on_one_half/


Editing the date of a post

Is your feature request related to a problem?
Hello, no, Thank you for your hard work on this amazing piece of software.
Describe the solution you'd like
Being able to also edit the date of a post, when you click on edit memo.
Additional context
I would like to get my diary from another platform to this one, so in order to import them I would need the option to change the date of the posts.

try to double click the time on the left-top of one memo.

Thank you so much!

try to double click the time on the left-top of one memo.

Doesn't work for me. After double-clicking the note opens in the details view that is non-editable

Doesn't work for me too.

@TheNomad11 @u23a
Hey, try Alt+click the date.

thanks, but doesn't work either. maybe it is an configuration issue then? Maybe my reverse proxy is too basic? In the docs there is only an example for nginy, but I am using Apache, might this be the reason?
<IfModule mod_ssl.c>
<VirtualHost *:443>
    ServerName memos.mydomain.com
    AllowEncodedSlashes On
    ProxyPreserveHost On
    ProxyPass "/"  "http://localhost:5230/"

SSLCertificateFile /etc/letsencrypt/live/memos.mydomain.com/fullchain.pem
SSLCertificateKeyFile /etc/letsencrypt/live/memos.mydomain.com/privkey.pem
Include /etc/letsencrypt/options-ssl-apache.conf
</VirtualHost>
</IfModule>

@TheNomad11 @u23a
Hey, try Alt+click the date.

Hi, is there a way to achieve this on mobile?

@itsfeng
Unfortunately no.
We are thinking about how to make this operation more user-friendly.


[TIR] Expand debug symbol output for CodeGenLLVM

Prior to this commit, the CodeGenLLVM would include DWARF symbols specifying the function signature of each TIR function.  This commit expands the information exposed in the debug symbols.

Name functions based on the name of the corresponding TIR function. This is taken either from the attr::kGlobalSymbol if present, or the PrimFunc's GlobalVar otherwise.

Name function parameter based on their TIR variable name.

Annotate the name, the type signature, and parameter names for the private function produced by the tir::attr::compute_scope attribute.

Name local variables based on the name and type of the corresponding TIR variable.

This came about while debugging the implementation of https://github.com/apache/tvm/pull/16542, but is otherwise unrelated.

Merged from main to PR branch, to resolve CI breakage that was fixed in https://github.com/apache/tvm/pull/16546.


California - Newport Children’s Medical Group - Interim Research

have not sent data yet -- @rachelhanster moving them to closed b/c cannot include them in the interim accessment due to delays


Support chain upgrades in voyager

Voyager currently expects that chain ids never change, we either need to document this as expected behaviour or handle upgrades (specifically cosmos chains).

shelving this for now as chains do not upgrade their chain id in practice. if we hit a case of this in the future, we can investigate possible solutions then.


Remove embedded documents complexity

Remove some of the additional complexity when working with embedded documents. Treat embedded documents like normal documents in the UnitOfWork as much as possible.
Fix issue with ClassMetadata::associationMapping not inheriting associations correctly.
Fix issue with re-persisting removed embedded document. #719
Fixes #1058, #999 and #1059

I wanted to test this with our project but got an error:
Fatal error: Class 'Doctrine\ODM\MongoDB\MongoDBInvalidArgumentException' not found in lib\Doctrine\ODM\MongoDB\UnitOfWork.php on line 942
This MongoDBInvalidArgumentException doesn't exist at all in the project. Maybe you have forgotten to git-add it?

@AronSzigetvari I apologize. I have a little more work to do here still. I realized it after I asked people to test and didn't update the comments here.

@AronSzigetvari I pushed up a change in c62b532a886e6a76d5668a5af19a8590fa635ba7 that fixes this. This PR originally had some other changes in it that I removed and I missed a few references.

I tried this today - it works fine except for a tiny issue.
I'm using DoctrineExtensions to add translatable behavior to documents. In this case, it's an embedded document which makes things a bit complicated.
In order to switch locales, the locale field of the embedded document is changed, after which the document is refreshed. This no longer works after the refactoring, since doRefresh() will not work with embedded documents. However, refreshing the parent document will reset the (unmanaged) locale field in the embedded document, which causes an error while loading.
Now, this could be a case of "you're using it wrong", I'd have to take another look at that. Just thought I'd mention this since I promised to run it against our application.

@alcaeus Do you think you can make a unit test that demonstrates the problem? I can see if I can make it work like it did before in that case.

Will do, give me a couple of days to investigate. As I said, this could be a case of "you're using it wrong".

@jwage I created the above issue to track the underlying issue with DoctrineExtensions, or at least what I think the underlying issue is. From what I can see, the only way to currently handle those behaviors in embedded documents is to refresh the embedded document which is no longer supported with this branch. Given the nature of embedded documents, I agree that they should not be refreshed directly.

I am a Doctrine ODM user and I have run up against a problem that sounds related to this PR. My PR https://github.com/doctrine/mongodb-odm/pull/1094 demonstrates a data consistency problem which I thought would be fixed by this branch but I just rolled a quick test and I see we still have this same problem even on your PR. How do you suggest I proceed with the resolution?

@blockjon we should try to fix your case in this branch.

@blockjon I take that back. This looks like it be solved by an entirely new feature. Possibly in #1066

@jwage any chance for rebase? Now that I have upgraded my app to BETA13 I will be able to run our test suite against this branch

Ok, so the branch is rebased and all my functional tests are still passing using this branch :)
@alcaeus @blockjon @AronSzigetvari (and everyone who step by here) may as you gentelmen to test this branch?

@malarzm Im checking this against my repo right now...

My project's unit test all seem to pass with this branch.
On a slightly different topic, you are all at this point aware of my interest in pushing forward with the atomicSet feature that @malarzm has been helping to build.
As I have been thinking about Doctrine ODM, I have come to the opinion that all the whole project should use atomicSet for EmbedMany everywhere. If it were up to me, I would remove the option for other "strategies" pretty much entirely. In my view, all new coders of this codebase (or any ODM/ORM for that matter) would reasonably expect a data persistence layer to use atomic queries such that no data loss & corruption would even be possible. My app isn't even "highly concurrent" per se. At any one moment, I have at most 2 users trying to update a record and I have run into integrity problems.
With the way the system supports different strategies, it means that some strategies are more stable than others. For example, the atomicSet strategy currently does not support editing of documents (https://github.com/doctrine/mongodb-odm/pull/1153).
I realize that there are advantages to having smaller & more compact queries but I think data integrity and consistency supersedes all else.

@blockjon thanks for the test :) As for #1153 I will come back to that my tomorrow and (hopefully) it'll be the last issue, I'll try to find more embedded documents tests, double them for being atomic and see if they'll be still working. Hopefully atomic strategires will be all fixed and hardened once we reach 1.0.0, which I hope is around the corner, we can even discuss making it default one by then - soon we shall see :)

Thanks @malarzm

@blockjon I agree completely with atomicity being entirely important, however...

My app isn't even "highly concurrent" per se. At any one moment, I have at most 2 users trying to update a record and I have run into integrity problems.

Here you are talking about concurrency, which I believe is not completely up to the database from a user using an application perspective. Such concurrency must be up to the application, at least partially.
From what I've learned, in an application you can have either pessimistic concurrency, where you lock out everyone from editing a document or documents, until the user who "checked out" the documents checks them back in.
Or, you can have optimistic concurrency, where you have versioning of the data. If someone tries to update the data with an older version, they get an "Oops, user X was faster than you and made modifications to the data already. Please refresh your page and make your changes again."
Either way is suboptimal from a user perspective and I'd say, for running a web based application, the latter with versioning is the better solution.
To be honest though, I am not sure if or how this all fits in with the strategy(ies) used in MongoDB-ODM for atomicity. I think the two are actually different topics, aren't they? If not, I'd like to understand how they are connected.
Scott

@malarzm, I can test it on Monday the earliest. I did tested the previous version but bumped into very mysterious problems. The previous version  broke some tests, and caused some type check errors in Doctrine that didn't happen when I inserted a gettype()  call (but no if,  just a gettype!)

@AronSzigetvari weird indeed... Obviously I'd be very interested in a failing test case if you'd run into such issues again :)

Looks good to me. The issues I previously reported are gone. :+1:

@AronSzigetvari got any feedback for me by any chance? :)

@malarzm, I have made a testing for the original problem and it seems to be OK, but I still have a failing test in the application test suite that I want to simplify to a minimal test or find out if we just used it wrong and had worked until now just because of some quirk.

@AronSzigetvari sure, do you think you'll be able to pull it together today or tomorrow?

@malarzm I am on it now so I very much hope so.

@AronSzigetvari thank you very much! :+1:

@malarzm I have found the cause of our failing test, but I am not sure if it is a bug or not (a feature, or a fix of a previous bug):
Scenario: merging 2 carts. Cart is document, Item is embeddedDocument.
When we merged cart A and cart B, we added Items of cart B to cart A. Since it is the same Item object, after we call $dm->remove($cartB), we still have both items in $cartA->getItems(), but after calling $dm->flush(), the item that belongs to the removed $cartB is removed from $cartA, too.
I could make the test pass 2 different ways:

If I don't use the same Item object, but create a new one with the same data: $cartA($item->copy())
It also seems to work if we call $cartB->removeItem($item) when transferring items, so when we flush, the items doesn't belong to the removed parent document any more. (I am not sure if it is really OK, or may cause problems later)

It makes sense since remove is cascaded, but it used to work differently before, so it is a BC in this sense, but I believe it is not a bug. What do you think?

@AronSzigetvari I would love to be able to just reuse embedded documents by passing them around to different documents but unfortunately we're not aware of any good solution for that :( I'd suggest going down 1), I'm personally always (deep) cloning objects that are about to change owner to ensure nothing will clash or have any side effects.

Merged manually in https://github.com/doctrine/mongodb-odm/commit/b1089b723df8ae2230694a3899941e05d560c41f. @AronSzigetvari @blockjon @alcaeus @danez thank you very much for all your help!

I have made a test that creates an owner document and an embedded document inside, then replaces the embedded, creates a new one with colliding hash value and checks the document state in the UOW. This fails with BETA10 and passes after this fix.
Probably too late to commit it in, since you have already merged the fix.

@AronSzigetvari I've merged @danez test in https://github.com/doctrine/mongodb-odm/commit/4817908f95ff98bb59242d16ca6e4d53a9082504 but if you think it's not enough feel free to submit a PR with passing test only :)

@malarzm I have added a test, it is a bit similar to @danez 's, but it is a slightly different case (no detached documents were used)


Add "array" Method

These changes add an array type coercion bloblang method similar to the bytes method: If the caller is already of type array the input is returned. If it is any other type the input is wrapped into a one-element array. Such a method helps to avoid constructs like this.with("name").values().flatten() to ensure an array returned.
The docs for the connect repo can be extended by:
### `array`

Marshal a value into an array. If the value is already an array it is unchanged.

#### Examples

```coffee
root.my_array = this.name.array()

# In:  {"name":"foobar bazson"}
# Out: {"my_array":["foobar bazson"]}
```

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


feat: add highlight for nvim-treesitter-context

Before:

After:

I simply used the same highlights as SignColumn.

Maybe we should choose a lighter highlight color instead so it does not fade into the background as much?

I kind of like colors.selection:

Hey again, thanks for the PR!
I personally prefer the more subtle active background and I think this should be the default since the plugin seems to take from NormalFloat but you can always override this in your own config if you want.
Also do you mind adding this plugin to the list of supported ones for documentation?

Hey, ok then let's leave it set to active. I added the plugin to the list of supported plugins in the readme.

Sounds good! And also, sorry I should have specified but there's both the readme and the vim help docs: https://github.com/rmehri01/onenord.nvim/blob/main/doc/onenord.nvim.txt.

No worries, thanks for reminding me :)

Looks great, thanks again!

Thanks for your work on this great color scheme! :D


Support loop_one mode

Roon's loop mode has three states: 'loop' |  'loop_one' |  'disabled'
source: https://roonlabs.github.io/node-roon-api/Zone.html
Currently this library only exposes loop and disabled
It would be great to have support for this in the API (and eventually HA).

I'm happy to write a PR for this.
What is this projects stance on breaking changes?
Should I break the function signature by changing the bool to a str type?
https://github.com/pavoni/pyroon/blob/master/roonapi/roonapi.py#L302
Or to maintain backwards compat should we support a boolean and string value?

With only a couple of users - I don't think it's a big deal to change the API.
We should just make it clear in the release.

As far as I can see the HA wrapper doesn't currently use Roon's repeat mode - and HA does support the 3 mode model.
So will be a good addition if you decide to do this.

Yea i'll write a patch in the new year!

@doctorfree I'm tempted to make a breaking change to do this.
Will it break your current code?
Anything I can do to make it easier?

@pavoni it would break the current RoonCommandLine support for repeat which I treat as a toggle. The changes I would have to make to support this change to the API are fairly simple and confined to a few lines of code. The only problem would be those users who upgrade the Python Roon API while running a version of RoonCommandLine prior to my changes to support this.
I'd say go ahead and add support for loop_one and I will update RoonCommandLine to support it as well.

Will it break your current code?
It would yes, but I can update my code easily enough when I update the pyroon dep. Thanks for asking.

Sorry I didn't get round yet to writing that PR, been busy with other things.

I'd say go ahead and add support for loop_one

+1

@doctorfree if it helps I can flag you when I've merged the PR - but before I release the library.

@doctorfree It was easy to make this backwards compatible - so shouldn't be an issue for you.


Reduce NamedPipe Chatter (take 2)

2nd attempt, first was https://github.com/dotnet/razor/pull/10756
This uses checksums for RazorProjectInformation to determine if it should be serialized and sent to razor. The benefit is to reduce work done when unrelated changes are made (such as adding/changing a csharp file)

Question, how frequently the checksum recalculated? is it done on every change or if else what is the main trigger?

Every time we get a workspace update we have to fully calculate the checksum. The reduction here would be not in computation on the workspace listener, but in work done for updating project information and serialization/deserialization.

Moving to draft. This has some good ideas in it but want to re-evaluate this in light of recent conversations about architecture


codeceptJS + nightmare + nightmare-download-manager not working

What are you trying to achieve?
i am trying use this plugin nightmare-download-manager
What do you done?
I have seen that Codecept instance Nightmare and I have tried in all the ways that have occurred to me to add the plugin that needs to be added before Nightmare is instanced and as I have not seen how I ended up making a fork of codecept, there is some other way to add actions to Nightmore before it is installed or has to be added as a new functionality for this type of Nightmore plugins. Below I leave information of what I have researched and done.
The problem comes from this issue here is implemented this plugin.
https://github.com/segmentio/nightmare/issues/151
Info:
https://electronjs.org/docs/api/download-item
And this line is the one that I had to add to my fork so that the plugin works because it needs to be added before Nightmare instants
https://github.com/tastafur/CodeceptJS
thanks

Let me look for it. Send a pull request from your fork and I will look if this is ok to get it into the main codebase.


Add support for output file extension

Tools like cutadapt use the input file name extension to determine what it should do.


Pop Os

Is possible to run the new Pop Os using crouton?

No, at least not at this time:
Here are the releases that crouton recognizes and can install:
chronos@localhost ~ $ crouton -r list
Recognized debian releases:
    potato! woody! sarge! etch! lenny! squeeze! wheezy! jessie stretch buster
    bullseye sid
Recognized kali releases:
    moto! kali! sana! kali-rolling
Recognized ubuntu releases:
    warty! hoary! breezy! dapper! edgy! feisty! gutsy! hardy! intrepid! jaunty!
    karmic! lucid! maverick! natty! oneiric! precise! quantal! raring! saucy!
    trusty! utopic! vivid! wily! xenial yakkety! zesty! artful! bionic* cosmic!
    disco! eoan* focal*
Releases marked with ! are upstream end-of-life, and should be avoided.
Releases marked with * are unsupported, but may work with some effort.

Hope this helps,
-DennisLfromGA


Improve time complexity for 'in' operator

"in" operator is better handling in set. Complexity for set is O(1) and complexity for list, tuple is O(n)

@charettes sorry for bothering with that PR. It was evening time and I was not attentive.


LED strip with WS2811 controller

Hi, I'm taking on a new endeavor, which is to get control over a newly purchased Alitove Neon strip .
I used your Holiday Lights sample to get started, and was hoping that, bare minimum, the ManualControl special effect would get me started to have the strip LEDs all set to the same color.
After realizing that the strip supposedly had no white LED (I was hoping to purchase an RGBW one, looks like this is only RGB), I corrected my sketch to initialize the LED strip using this:
  strand = new Pixel_Strand(4,140);  // GPIO 4, 140 IC (controller) with 3 LED each = 420 LEDs on strip

Using Eve, selecting a red color gives me a full red strip.
Switching to blue, the strip remains red.
Switching back to red, the strip remains red.
Switching to neon green, the strip goes pink.
Switching to pink, the strip goes red.
If I simplify the code to control only the first pixel, I have realized that I need to "convert" the Hue of HSV to 180 - (px->H.getNewVal<float>()-180);
And then, if I try to illuminate only the 2nd pixel (IC), then I get totally different colors.
So it is all over the place.
The WS2811 specs at https://cdn-shop.adafruit.com/datasheets/WS2811.pdf seems to say this is a 24-bit color.
Any tip to help me troubleshoot the issue? Where can I see raw bits sent to the strip?

Let's start with a direct test of the Pixel code using the following script (change the PIN number accordingly):
#include "HomeSpan.h"

#define PIXEL_PIN 33    // set this to whatever pin you are using - note pin cannot be "input only"

Pixel testPixel(PIXEL_PIN);

void setup() {

  Serial.begin(115200);
  delay(1000);

  Serial.printf("Pixel Test on pin %d\n",PIXEL_PIN);
}

//////////////////////////////////////

void loop(){
  Serial.print("Red\n");
  testPixel.set(Pixel::RGB(100,0,0));
  delay(2000);

  Serial.print("Green\n");
  testPixel.set(Pixel::RGB(0,100,0));
  delay(2000);

  Serial.print("Blue\n");
  testPixel.set(Pixel::RGB(0,0,100));
  delay(2000);
}

This should cycle the color of the very first LED set on your strip from red to green to blue every two seconds.
Based on the above link, it looks like your strip needs 12 volts - make sure to power it directly from a 12V supply and connect the ground from the strip to the ground pin on the ESP32.   Also make sure the pin you choose to control the strip is connected to the digital INPUT on the strip (the strip allows for chaining, so one side will be a digital input and the other a digital output).
If this works for changing the colors of the first LED set on the strip, you can then modify the test to set all the LEDs on the set to the same color at the same time as a second test.

Instead of blinking red, green, blue, it flashes red, blue green.
Connected to GPIO 4 of a Keeyes dev board.
12 volts supply to the red wire, ground to white wire & ground pin of the dev board, and green wire to gpio pin 4.

If it's blinking correctly but the colors do not match this may be a result of how the driver chip is connected to specific LEDs  within the strip.  I suppose different manufacturers can use the WS2811 differently.  I can add a hook that allows one to specify the order of the bytes.
If you modify to control multiple pixels at the same time does it still operate correctly (apart from the green and blue being swapped)?

If you modify to control multiple pixels at the same time does it still operate correctly (apart from the green and blue being swapped)?

Yes it does. I tried with 140 pixels and the whole strip illuminates the same colors. Blue and green inversed.
I also tried with HSV, in case something was wonky with it as opposed to RGB, so I used with 0 and 359 (for red), 120 for green (got blue) and 240 for blue (got green).

The fact that it works with 140 pixels is very good news - suggests that everything is working fine, with the exception of the color mapping.   I pulled the specs for the WS2811 and compared with the WS2812 and indeed the order of the colors when transmitting data to the chip is different!  I'll come up with a generic way of specifying the color order in the library so it can easily be configured to reflect the details of each chip and specific led strip.
Note the HSV functions is just a mathematical mapping to RGB, so if the RGB data is sent in the wrong order, the HSV won't work either.  I'll probably work on a patch this weekend --- will let you know when I post the changes to the dev branch.

Thanks, and no rush. Illuminating a led strip is definitely less critical than having my alarm system send me push notifications! ;-)

I added a new method to Pixel called setColorMap(uint8_t *map).  It allows you to set the order in which the color bytes are transmitted.   Six color maps are pre-defined in the ColorMap namespace as follows (though I think only the first two will  ever be needed):

ColorMap::GRB (this is the default)
ColorMap::RGB (this is a common alternative)
ColorMap::RBG
ColorMap::BRG
ColorMap::BGR
ColorMap::GBR

It looks like only the first two are used by most manufacturers, though there is a lot of confusion as to which chips and devices use which order (for example, see this discussion).
To use the new method, call it right after you instantiate a new Pixel.  For example, in the test sketch above, to switch the order of byte transmission to RGB (which I think is what you need for your specific device), you would add:
Pixel testPixel(PIXEL_PIN);
testPixel.setColorMap(ColorMap::RGB);   // add this new line

The new code is available in the "dev" branch (which now builds off of 1.9.0).
Let me know if this works.  If so I'll update the documentation accordingly.  I also have a nice test script that helps users determine what order they need, as well as whether or not they have an RGB or RGBW device.

Just tested with the Sketch above and it works perfectly. May I ask, where would this new line go in the Pixels example?

In the Pixel example you would enter it below the following line:
 pixel=new Pixel(pin);               // creates Pixel LED on specified pin <- Existing Line 79
 pixel.setColorMap(ColorMap::RGB);   // add this new line

I'm still contemplating the exact format of how this should work.  For now I've added setColorMap but I'm going to try out adding this as second parameter to the constructor itself, effectively replacing the isRGBW flag with a something like Pixel::Type::RGB, Pixel::Type::RGBW, Pixel::Type::GRB, etc.

I will give it a try soon, but keep also in mind “all-white” led strips while re-working on your solution. I may end up having to purchase that instead, because Holidays colors all year long in the house will not pass the voting committee…

I had to use ColorMap::GBR, but something is strange, setting to Red is now Pink.
  testPixel.setColorMap(ColorMap::BRG);   // r=g, b=r, g=b (Setting to Red gives me Green, ...)
  testPixel.setColorMap(ColorMap::RGB);   // r=b, b=g, g=r
  testPixel.setColorMap(ColorMap::GBR);   // r is pink!, g=g, b=b

.. unless I have an RGBW after all???

If using RGB you see blue followed by green followed by red, use ColorMap::BGR

ok.. definitely not RGBW, as when I set it, this is where the strip becomes multi-colored! :)

If using RGB you see blue followed by green followed by red, use ColorMap::BGR

No luck.
Red is green-ish
Green is red (a real red!)
Blue is blue.

I'll give so more thought - these changes worked fine for @honear, but there may one something specific to your model.  Have you tried using a different driver (unrelated to HomeSpan) - perhaps the Adafruit NeoPixel library as a test?

UPDATE:
The final version of the code is now checked-in on the "dev" branch.  To simplify use, I eliminated the recently-added setColorMap() method (which was mostly for initial testing purposes) and instead changed the constructor from Pixel(int pin, boolean isRGBW); to Pixel(int pin, pixelType_t pixelType); where pixelType should be set to one of the following:

PixelType::RGB
PixelType::RBG
PixelType::BRG
PixelType::BGR
PixelType::GBR
PixelType::GRB
PixelType::RGBW
PixelType::RBGW
PixelType::BRGW
PixelType::BGRW
PixelType::GBRW
PixelType::GRBW

Example: Pixel testPixel(12, PixelType::RGBW);
This allows you to simultaneously set the order of the RGB colors as well as specify whether your device also has a White LED.   In the new constructor, setting pixelType is optional.  If left blank, the default is PixelType::GRB, which matches the behavior in all previous versions of HomeSpan.  The original constructor, Pixel(int pin, boolean isRGBW) still works as expected, but I've flagged it as deprecated so you will receive a warning when compiling if used.
Finally, I added a standalone PixelTester sketch to help determine which setting to use for any given device.  You'll find it under Other Examples.

Thanks, I ended up having to use PixelType::GBR for things to work.
Using your new sample that ends up increasing the brightness of R-G then B, I noticed that in the low brightness of Red, the strip was pink. Then as brightness increased, it went to bright red.
So I took this, and re-uploaded the Holidays Light demo. With the demo and Eve app, I could now better control the colors, and using a bright red, within Eve with full brightness, the strip is red. Then decreasing the brightness from the app, the color is still red with lower brightness, and not pink. That code uses HSV, not RGB.
Nevertheless, I can now enjoy all but the Twinkle effect which has no lights at all and will try to figure out why and push a PR to update that different project if I can find the cause.
I owe you one for fixing this! Thanks!
and I'll let you close that issue if you are all done with it.

Note that the Twinkle effect uses the white LED and requires an RGBW strip.

Good point. I edited the effect to use the selected color, and now it all works fine.
I was also noticing flickering of incorrect colors, and I ended up using setTiming(0.5, 2.0, 1.2, 1.3, 50); to match the values found on page 3 of https://cdn-shop.adafruit.com/datasheets/WS2811.pdf. If I am not mistaken, this is especially meant for that, right? And no more flickering since then btw.
Is the documentation somewhere about the return value of the update() methods like at https://github.com/HomeSpan/HolidayLights/blob/main/HolidayLights.ino#L165 ?
Does this impact the speed of update? Speed of repeat?

Yes, the setTiming() method is meant for exactly that -- really glad it solved the flickering issue.  I was doing some research and apparently some other Pixel controllers experienced the same problem. Certain manufacturers changed the timing cycle for their WS2811 chips a while back and flickering is the result of using timing parameters that are too close for the chip to distinguish a 1 from a 0.  One thing I noticed on the WS2811 doc is that it explicitly shows the order of bit transmission as RGB, but you needed to use GBR.  Perhaps Altilove connected the chip differently to the LEDs on their strip?  Regardless, as long as it works I guess all is good.
And yes, the return(20) in update() is the delay (in milliseconds) between updates to the effect.  The delay itself is implemented in the "alarm" code in lines 134-139.  Note that the update() method for the effects is not the same as the HomeSpan update().  I probably should have picked a different name to avoid confusion.

oh! I understand what you mean now. In PixelStand.h, I ended up:

renaming alarmTime to effectDelay
renaming the SpecialEffect function virtual uint32_t update(){return(60000);} to virtual uint32_t run(){return(60000);} everywhere where corrections were required, and now it all makes sense.

Since everything seems like it is working with the new dev code I'll close out this issue.  Please feel free to re-open if needed.

Hi, is it possible to reopen the issue? Something strange is happening. Still using the Holiday Lights project to get things to work, I realized that if I use the Manual effect, it initially works fine with the color selected. But then, if I toggle the color (using Eve app) back & forth between 7 preset colors, then all of a sudden the strip start flashing at ~1 second intervals.
Or, if I change the brightness multiple times, then the flashing also begins.
The only way I can get the flashing to stop is to keep changing the brightness and out of the blue the flashing stops and the color is solid and accurate.
If I set the effect on the Knight Rider, I can change the brightness & color as much as I want with no issues.
Setting to Random, the flashing will occur.
Setting to Twinkle (modified to use RGB & HSV), no issues.
Recetrack: no issues.
I got myself a full white strip that I will later try, but I would like to resolve this strange issue before.

Unfortunately I won't be able to work on this for a while - and it may be very difficult to debug.  It may be that the speed of change is too rapid, in which case adding a short delay after the effect updates might help.  Other thing to check is wiring - make sure you connect the ground wire on both sides of the strip to the ESP32 ground.
You can also try  to reduce the nPixels so that only the first half of the strip is used.  I'm not exactly sure what that would tell us, but it is a variable to test.

Thanks, I tried something based on your suggestions.: I set the # of pixel to half of what the strip has: 70 (instead of 140).
Then I kinda noticed something and when on in isolating steps to reproduce:

Using the Home app (I haven't confirmed yet with Eve), if I set the brightness to 73% and quickly change colors, no issues.
If I increase brightness to 74% and go about changing colors, then it starts to flicker.

I will see if there is a specific count of color changes that the flicker starts to occur...

update: no consistency in how many color changes I have to do before flickers happens... :(

I also wired the other end of the ground, as suggested. No changes, flickering always starts at 74% brightness...
hum... I used an old router 12v power plug, says 1.0 amp output.
While the strip has 140 "pixels", this is actually 420 LEDs. I may be pulling to much amps.. need to look into this.

With a multimeter in the loop, it got up to 1.7amps, but without flickering. If I remove the multimeter, it flickers. But the fact that I see 1.7 amps from an old Netgear power plug with max output saying 1.0 amp is a bad sign. I will find another power source and provide results later. ok to keep this issue closed for now.

That was it. I found an old Netgear power plug, 12v 5amps output, and no flickering. All set!

Excellent!  Thanks for the update.


Aggregating stream data

I am looking to consume data from a stream, and then aggregate it before merging it into another table. How can this be done using incremental_streams
Here is some pseudo code:

{{
config(
materialized='incremental_stream',
unique_key=['key']
)
}}
WITH
new_data AS (
SELECT
key,
value
{{ incr_stream.get_stream_metadata_columns() }}
from {{incr_stream.stream_source('src', 'table')}}
),
aggregated_data AS (
Select key
Max(value)
from new_data
Group by key
Select * from aggregated_data

The problem is that the get_stream_metadata_columns()  are not included in the aggregation.
Is there any way around this?

Hi @sternalon,
Apologies for the delay in getting back to you.
What you're attempting to achieve—incrementally updating aggregated data—can indeed be complex. I would suggest considering the use of ❄ dynamic tables, which allow Snowflake to handle the updates to your aggregation automatically. Dynamic tables are well-suited for this type of incremental processing.
However, if you prefer to stick with streams, I would recommend the following approach, which avoids directly aggregating the data but allows incremental_stream to handle the changes (inserts, updates, or deletes) based on the new rows and existing data:
{{
config(
  materialized='incremental_stream',
  unique_key=['key']
)
}}

WITH new_data AS (
  SELECT
    key,
    value,
    {{ incr_stream.get_stream_metadata_columns() }}
  FROM {{ incr_stream.stream_source('src', 'table') }}
),

aggregated_data AS (
  SELECT
    key,
    MAX(value) OVER (PARTITION BY key) AS value,
    {{ incr_stream.get_stream_metadata_columns() }}
  FROM new_data
)

SELECT * FROM aggregated_data;

This version does not directly aggregate data in the way you may initially expect (with a GROUP BY clause). Instead, it uses a window function to ensure you maintain access to the stream's metadata columns in the output. This allows for more flexibility in handling incremental updates without losing the ability to track the metadata alongside the new data.
I hope this helps, but feel free to reach out if you'd like further clarification or assistance!


vertical autoscaling of graphs doesn't handle peaks well

For some reason I can trigger horrendous packet loss on my local network simply by uploading a file to Google Drive. When I do this, I get latency spikes as high as 500-1000ms, whereas previously everything was nice and stable under 25ms.  This means that when the spikes appear, the auto-scaling kicks in and the previous data under 25ms gets squashed so thinly at the bottom of the graph that it becomes invisible.  Firstly, even when it is squashed like this it should still be visible.  But secondly, I would suggest a logarithmic graph or horizon graph so that wide variations can still be easily viewed on a single graph.

Thanks for the feedback, and apologies for the late response, just letting you know your request had been noted, but it requires some significant additions to https://github.com/20c/graphsrv/issues but we're looking into it.

No problem and thanks a lot for the update!  Please do consider horizon graphs as an option, because they are pretty awesome in terms of space efficiency and visual clarity! :-)


Add first version of the migration guide to Krane 1.0

What are you trying to accomplish with this PR?
Add a migration guide to help people migrate off of kubernetes-deploy and into krane.
This can still change before 1.0, but it might be easier to maintain this guide if we merge it and update as needed (e.g. we still need to send a PR to update flag names: https://github.com/Shopify/kubernetes-deploy/pull/602#discussion_r339247531).

Is there a difference between version 1.0 and version 1.0.0 (1 decimal vs 2)  it seems to be used interchangeably throughout this doc but not really consistent

There shouldn't be a difference, but in any case we should make it consistent. Good shout!

Just a nit. It's a great migration guide, thanks @dirceu


add custom message explaining CMake find_package error messages

Many user questions are based on the not easy to read and interpret CMake error message in the case that find_package(... REQUIRED) can't find a certain CMake config file, e.g.:

http://answers.ros.org/question/227296/catkin_make-error-when-using-mavros/
http://answers.ros.org/question/189686/catkin_make-fails-weirdly-after-trying-to-create-a-new-package/
http://answers.ros.org/question/227151/adding-rplidar_ros-as-package-dependency/
http://answers.ros.org/question/221359/ros_controlconfigcmake-not-found/
http://answers.ros.org/question/222041/rosaria-make-error-jade/
http://answers.ros.org/question/67079/how-to-depend-on-pr2_controllers_msgs/

This patch prints a custom message before that which should provide a more helpful description.

The same logic could be applied to the generated package specific CMake config files (https://github.com/ros/catkin/blob/5975eca2c172f1c889833352306413d468a4d0cc/cmake/templates/pkgConfig.cmake.in#L165). But I have never seen it being a problem / being asked on answers.ros.org.

What about suppressing the CMake message entirely and printing this:
set(component foo_pkg)
message(SEND_ERROR "Catkin could not find a package named '${component}'.\n"
  "If the package name is correct, this problem can be resolved in one of the following ways:\n"
  "1. Install the '${component}' package to your system if it is distributed as a binary package,\n"
  "2. Add the '${component}' package to your workspace or one of your underlay workspaces,\n"
  "3. Add the installation prefix for '${component}' to CMAKE_PREFIX_PATH,\n"
  "4. Explicitly set the `${component}_DIR` variable to the path to the directory containing the '${component}' package's CMake configuration files.")

Why should it convert a fatal error in a non-fatal error? I think there is a very good reason why this case is fatal.

Why should it convert a fatal error in a non-fatal error? I think there is a very good reason why this case is fatal.

You're right, it should be FATAL_ERROR.

I would rather not replace the output of CMake completely. I think keeping the actual error message but providing additional information before is better.
Also I don't think the message should contain to much information. Simply because it is pretty difficult to ship changes to the message. If more information is valuable it could reference a URL.

I would rather not replace the output of CMake completely. I think keeping the actual error message but providing additional information before is better.

It's up to you guys, but you're talking about novice users who appear to be confused by pretty basic CMake errors. Adding more stuff for them to read probably won't save them. In the context of a Catkin package, the CMake error is practically misinformation.

Also I don't think the message should contain to much information. Simply because it is pretty difficult to ship changes to the message. If more information is valuable it could reference a URL.

Yeah, referencing a URL makes sense, I only suggested that since it would replace the specific (bad) advice that CMake gives when a package isn't found.

Can anyone recommend a URL which would contain helpful information in this case? Or does it require a new wiki page or ticket to document.

I don't know of a good wiki page that currently fits the bill.

If we don't have a good page to reference I won't have the bandwidth to set one up. Therefore I would propose tho merge the PR as is since the added message provides much better context then without.


[SPARK-33482][SPARK-34756][SQL][3.0] Fix FileScan equality check

What changes were proposed in this pull request?
This bug was introduced by SPARK-30428 at Apache Spark 3.0.0.
This PR fixes FileScan.equals().
Why are the changes needed?

Without this fix FileScan.equals doesn't take fileIndex and readSchema into account.
Partition filters and data filters added to FileScan (in #27112 and #27157) caused that canonicalized form of some BatchScanExec nodes don't match and this prevents some reuse possibilities.

Does this PR introduce any user-facing change?
Yes, before this fix incorrect reuse of FileScan and so BatchScanExec could have happed causing correctness issues.
How was this patch tested?
Added new UTs.

cc @dongjoon-hyun

Test build #136449 has started for PR 31952 at commit 03b9b0b.

Test build #136449 has finished for PR 31952 at commit 03b9b0b.

This patch fails executing the dev/run-tests script.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/136449/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41033/

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41033/

thanks, merging to 3.0!

Thank you, @peter-toth and @cloud-fan .


Notebooks +Code/+Markdown is distracting

Testing #114969
It keeps showing and displaying when you move the cursor vertically

Can we improve this somehow? Some thoughts:

Only show after a timeout has happened (reset on cursor move?)
Show all the time at lower opacity, only setting to opacity: 1 when the cursor is over it?

I think this is a bug when hovering over the output, the buttons should always be visible when hovering over the entire cell.

I think this is a bug when hovering over the output, the buttons should always be visible when hovering over the entire cell.

@misolori it also shows when the panel is being hovered.

@misolori it also shows when the panel is being hovered.

it also shows when the panel is being hovered.

oh right, this the other state where you have an active cell selected and your cursors moves away from the editor.

it also shows when the panel is being hovered.

oh right, this the other state where you have an active cell selected and your cursors moves away from the editor.

I'd expect that to be the same as the mouse hovering within the notebook but outside the cell though.

I'd expect that to be the same as the mouse hovering within the notebook but outside the cell though.

@Tyriar not sure what you mean? When you hover a cell those buttons always appear.

@Tyriar not sure what you mean? When you hover a cell those buttons always appear.

@misolori hovering the green part is where it shows now:

I think we both agree that it not showing when hovering the webview is a bug, but I think it should look like this:

If hovering outside the cell within the notebook hides it, hovering outside the notebook should as well because that's even more "unfocused".

@misolori hovering the green part is where it shows now:

I think we both agree that it not showing when hovering the webview is a bug, but I think it should look like this:

If hovering outside the cell within the notebook hides it, hovering outside the notebook should as well because that's even more "unfocused".

If hovering outside the cell within the notebook hides it, hovering outside the notebook should as well because that's even more "unfocused".

Hovering outside the cell doesn't hide it you are just hovering over a different cell and that's where it is being shown it's just not visible. The logic goes:

If mouse is inside of the editor/notebook, show actions (run, cell toolbar, +Code/+Markdown) for the hovered cell
If mouse is outside of the editor/notebook, show actions for the focused cell

If hovering outside the cell within the notebook hides it, hovering outside the notebook should as well because that's even more "unfocused".

Hovering outside the cell doesn't hide it you are just hovering over a different cell and that's where it is being shown it's just not visible. The logic goes:

If mouse is inside of the editor/notebook, show actions (run, cell toolbar, +Code/+Markdown) for the hovered cell
If mouse is outside of the editor/notebook, show actions for the focused cell

Why do we hide the focused cell buttons at all then? It's the flickering the moving your mouse around that bothers me.

Why do we hide the focused cell buttons at all then? It's the flickering the moving your mouse around that bothers me.

We only want to show one set of actions, we got feedback that it was too noisy when you have more than one set of actions and can be confusing on which cell is the focused one as the focus border wasn't enough of an indicator.

We only want to show one set of actions, we got feedback that it was too noisy when you have more than one set of actions and can be confusing on which cell is the focused one as the focus border wasn't enough of an indicator.

We increased the delay of showing the insert toolbar and also added setting to hide it.


udt should be editable

The udt item should be editable

When clicking the udt item in the main panel, it should go to the edit form, instead of an add form.

Now support edit udt info

Now support edit udt info


Allow all bots to be used for chameleon projector

About the PR

allow to use chameleon projector on any bot
this pr is opposite of #32004
Why / Balance

funny
Technical details

new tag for bots and add for projector whitelist
Media

no
Requirements

[ ] I have read and am following the Pull Request and Changelog Guidelines.
[ ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

:cl:

tweak: All bots are available for disguise with the Chameleon Ppotlight

Hold still, please.

This needs to be merged as soon as possible

I dunno if this is very well within the spirit of the projector. The projectors main drawback is that if you're seen moving the jig is up.

The person talking would probably not have the pAI "beep" sound so IMO this would be even funnier when someone is going to try replicating a medibot to get a kill objective

The person talking would probably not have the pAI "beep" sound so IMO this would be even funnier when someone is going to try replicating a medibot to get a kill objective

BotOps

I dunno if this is very well within the spirit of the projector.

yeah, the only reason i opened this pr, becasue medibot was whitelisted for projector while other bots weren't
there has to be some certainty about this, so i opened #32004, but with count of dislikes i opened this as opposite
should just wait for maintainers discussion about this

if this gets merged people will probably use this and expect this more which is likely good

lets try it out


CPU goes 100%

Issue Type: Performance Issue
Hey VSCoders,
recently I am experiencing problems with high CPU in one workspace. Especially when I try to go to a definition of local node modules package. Trying to understand what could be the problem, since other projects run just fine.
Can it be connected with file watcher, file or search exclusion within workspace settings?
VS Code version: Code - Insiders 1.34.0-insider (d234937aedcbc265d2272ba5861125293239f9c7, 2019-04-05T05:14:06.827Z)
OS version: Darwin x64 18.5.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8559U CPU @ 2.70GHz (8 x 2700)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
2, 2, 3

Memory (System)
16.00GB (2.95GB free)

Process Argv

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
   18	    98	 68438	code-insiders main
    8	    98	 68439	   gpu-process
   13	   295	 68440	   window (Page.js — infogram-frontpage)
    0	     0	 68712	     /bin/bash -l
    0	    66	 68955	       node /usr/local/Cellar/yarn/1.12.3/libexec/bin/yarn.js dev
    0	     0	 68956	         /bin/sh /var/folders/ml/7fh8hsc12799l7tx3sbvfpkh0000gn/T/yarn--1554576937035-0.4920864411067647/node server.js
    0	   213	 68957	           /Users/alexey.vorobyov/.nvm/versions/node/v8.15.0/bin/node server.js
    0	    98	 68724	     extensionHost
  101	   279	 68917	       electron_node tsserver.js
    0	    66	 68940	         electron_node typingsInstaller.js typesMap.js
    0	    66	 68921	       electron_node eslintServer.js
    0	    66	 68929	       electron_node server.js
    0	    49	 68831	     watcherService
    0	    49	 68920	     searchService
    5	    82	 68946	   shared-process
   52	    66	 68978	   window (Issue Reporter)
    0	     0	 68979	   /bin/ps -ax -o pid=,ppid=,pcpu=,pmem=,command=

Workspace Info
|  Window (Page.js — infogram-frontpage)
|    Folder (infogram-frontpage): 105 files
|      File types: js(42) svg(13) png(11) map(10) json(10) yml(3) DS_Store(2)
|                  env(1) md(1) eslintrc(1)
|      Conf files: package.json(1) launch.json(1) settings.json(1);

Extensions (44)

Extension
Author (truncated)
Version

vscode-css-formatter
aes
1.0.1

Bookmarks
ale
10.4.2

project-manager
ale
10.5.1

copy-relative-path
ale
0.0.2

unique-lines
bib
1.0.0

vscode-toggle-quotes
Bri
0.3.2

npm-intellisense
chr
1.3.0

path-intellisense
chr
1.4.2

bracket-pair-colorizer
Coe
1.0.61

vscode-svgviewer
css
2.0.0

vscode-eslint
dba
1.8.2

vscode-quick-select
dba
0.2.7

javascript-ejs-support
Dig
0.4.4

es7-react-js-snippets
dsz
2.2.1

gitlens
eam
9.5.1

auto-close-tag
for
0.5.6

macros
ged
1.2.1

rest-client
hum
0.21.2

search-node-modules
jas
1.3.0

vscode-styled-components
jpo
0.0.26

expand-region
let
0.1.2

dotenv
mik
1.0.1

quicksnippet
mou
0.0.1

vscode-json5
mrm
1.0.0

vscode-scss
mrm
0.6.2

vscode-stylefmt
mrm
2.5.0

debugger-for-chrome
msj
4.11.3

color-highlight
nau
2.3.0

copy-file-name
nem
1.0.0

copy-json-path
nid
0.0.5

fix-json
oli
0.1.2

vscode-graphql
Pri
0.1.7

sass-indented
rob
1.5.1

vscode-paste-and-indent
Rub
0.0.8

vscode-sass-format
sas
1.1.6

code-settings-sync
Sha
3.2.8

js-jsx-snippets
sky
0.3.0

vscode-fileutils
sle
2.13.5

guides
spy
0.9.3

autoimport
ste
1.5.3

code-spell-checker
str
1.6.10

language-stylus
sys
1.11.0

vscode-todo-highlight
way
1.0.4

change-case
wma
1.0.0

(1 theme extensions excluded)

I experienced this issue when i add >= 8 projects in a single workspace.
So i fixed this problem by using one project the workspace.

Please follow these instructions to collect performance information. Share the output of code-insiders --status

Hey, @mjbvz.
Output with enabled extensions

Output with disabled extensions

Okey, seems problem similar to https://github.com/Microsoft/vscode/issues/49885.
When I try to go to file definition, CPU just spikes 100% and stabilises.
Also noticed that after removal of node_modules folder, seems CPU goes down.
I have already excluded it from files.watcherExclude, files.exclude and search.exclude

Actually CPU goes 100% after opening any file, so after some service/watcher starting on file open.
Before opening any file in workspace everything is ok, even if node server is running in dev environment.

Sounds like https://github.com/Microsoft/vscode/issues/71886
This may be caused by an old version of  @types/styled-components @peacefulseeker can you please follow these instructions to see if they fix the issue for you: https://github.com/Microsoft/vscode/issues/71886#issuecomment-481441653

Thank you @mjbvz.
Will take a look after a while. :)

@mjbvz, for now seems this https://github.com/Microsoft/vscode/issues/71886#issuecomment-481584323 approach helped me + upgrading the typescript
globally to version 3.4.3

Thanks. I'm closing this as a duplicate of  #71886  since it sounds like those steps fixed things


WIP - Add jitsi

A work in progress.
things to be solved:

[ ] public IP of videobridge service
[ ] certs of prosody
[ ] docker image of jitsi services

@pierreozoux let us know when this is ready to review :)

What's the status of this?

I stopped working for that client and jitsi didn't show much interest
Closing this for now, but hopefully it gives inspiration for anybody that needs it :)


Getting error “The credentials supplied to the package were not recognized” while requesting Azure API Management URL

Team, when I request APIM url , I get the below exception.
The credentials supplied to the package were not recognized
I am attaching the certificate while requesting the APIM.

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

ID: e4e98f59-eb03-4692-db13-ac0c61a6dd3c
Version Independent ID: 55059f88-2aa2-b916-03e7-0589cc166555
Content: Azure API Management overview and key concepts
Content Source: articles/api-management/api-management-key-concepts.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

Here is my C# code which makes a request to the Azure API Management
       HttpClientHandler requestHandler = new HttpClientHandler();

       X509Certificate2 certificate = // Get certificate from windows store
       try
       {
             using (var client = new HttpClient(requestHandler))
             {
                client.BaseAddress = new Uri(baseAddress);
                client.DefaultRequestHeaders.Accept.Clear();
                requestHandler.ClientCertificates.Add(certificate);
                client.DefaultRequestHeaders.Add("Ocp-Apim-Subscription-Key", subscriptionKey);
                client.DefaultRequestHeaders.Add("Ocp-Apim-Trace", "true");

                var responseTask = client.GetAsync(string.Format("/api/resource"));
                responseTask.Wait();
                var result = responseTask.Result;  // exception raises from here

              }
        }
        catch (HttpRequestException exx)
        {
            if (exx.InnerException != null)
            {
                return exx.Message + exx.InnerException.Message;
            }
            return exx.Message;
        }
        catch (Exception ex)
        {
            if (ex.InnerException != null)
            {
                return ex.Message + ex.InnerException.Message;
            }
            return ex.Message;
        }

Here is my APIM Policy
<inbound>
<base />
<set-backend-service base-url="http://echoapi.cloudapp.net/api" />
<choose>
    <when condition="@(context.Request.Certificate == null)">
        <return-response>
            <set-status code="403" reason="Invalid client certificate" />
        </return-response>
    </when>
</choose>

If I remove the below null check, I won't get the exception.
      <when condition="@(context.Request.Certificate == null)">

This means reading certificate from context throws an exception

@codemaker Thanks for the question! We will review and get back to you!

@codemaker A bit of research suggests that the error that you are seeing arises from your client not having access to the private key on the certificate when fetching it from the store. Could you confirm?
While the OP in this SO thread is not related to APIM, they should hold true for checking the private key.

Yes, You are right.
I ran my client code in administrator mode, now it's working!

@kudlatiger Glad we could help!
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.
Also, please do note that this repository is primarily for addressing document feedback. Its best to post generic queries on MSDN or StackOverflow to engage the wider community.


imhex: set XDG_DATA_DIRS properly

This allows ImHex to find pre-defined patterns for example.

Things done

Built on platform(s)

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

[ ] Tested, as applicable:

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

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

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

Whoops, turns out this is an (almost-exact) duplicate of #354906.


How many services can be registered on the esp32 boarder router srp server? (TZ-504)

hi, can i ask is there a max number of services that can be registered on the esp32 boarder router as a srp server? And how could i increase it? And could i set up a central SRP server on a linux server for several different thread networks all connected in the same local Ethernet  network? If this can be achieved how should i do that?

For your question about the max number of services which can be registered on the BR, the only limitation is the size of the static array which is used for record the MDNS service. You can set this via idf.py menuconfig
(Top) → Component config → mDNS  →  (10)Max number of services

And for your question And could i set up a central SRP server on a linux server for several different thread networks all connected in the same local Ethernet network? If this can be achieved how should i do that
Do you want that the host can access the service which is pubilshed on any thread devices?
If yes, I think currently the implemention of the SRP can reach your demand. Each thread network should have one BR to forward the packets between wifi network and thread network. And the host can access any service published at any thread netwrok. No other things are needed to be done.

ok thanks a lot


Lack of server timeouts

Http servers created in https://github.com/movio/bramble/blob/24b50dca339947b8e1424d9f70af15d3299a7284/main.go#L66-L69
don't have anything set in their "timeout" field which may lead to various problems. Zero value means no timeout
Links:

https://blog.cloudflare.com/exposing-go-on-the-internet/
https://adam-p.ca/blog/2022/01/golang-http-server-timeouts/
https://github.com/ory/graceful/blob/ecb4894ce882f348f659da531a47aaf99796d489/http_defaults.go#L51-L60 -> some library I personally like using, sets http.Server defaults to the same as in cloudflare blogpost

If you want I can prepare a PR which sets those defaults to
	// DefaultReadTimeout sets the maximum time a client has to fully stream a request (5s)
	DefaultReadTimeout = 5 * time.Second
	// DefaultWriteTimeout sets the maximum amount of time a handler has to fully process a request (10s)
	DefaultWriteTimeout = 10 * time.Second
	// DefaultIdleTimeout sets the maximum amount of time a Keep-Alive connection can remain idle before
	// being recycled (120s)
	DefaultIdleTimeout = 120 * time.Second

but I'd like to first hear what do you think about it. Maybe some different values, maybe somehow expose it via configuration json?

@aerfio good catch, we haven't hit any issues with the timeouts not being specified yet but that's probably luck more than anything.
An initial PR setting to recommended defaults would be appreciated, and if we do find a need to tweak them we can add the ability to configure them.


Support for Facebook SDK 7.1.0

The current latest Facebook SDK for Unity has some compatibility-breaking changes.
More info:
https://developers.facebook.com/docs/unity/change-log#v7_1_0
When can we expect to have Getsocial Unity SDK updated to work for this new FB SDK?

Hi there,
Thanks for the note. Our engineers are aware of this and currently plan to work on it, as soon as we have a release with the latest Facebook SDK we shall inform all our customers.

I see release v4.0.0 just came out. Does it resolve this issue yet? Thanks.

Hi @p0w1nd,
Ok, to update the project to the newest Facebook SDK with GetSocial please follow these steps:

Open the project with GetSocial in Unity 5 and upgrade the project. There should be no problems with Unity updating the API methods.
Download the latest Facebook SDK for Unity
Remove old Facebook folder from the project, remove all files that came with Facebook SDK from Plugins folder in Unity
Import the new .unitypackage into the project. Now we need to fix all the API calls to Facebook
First add using Facebook.Unity; everywhere as the namespace has changed.
Change FB.Login method to following FB.LogInWithReadPermissions(new [] { "public_profile", "user_friends" }, result => ...
FB.Logout(); to FB.LogOut();
IdentityInfo.CreateWithSocialNetworkInfo(IdentityInfo.ProviderFacebook, FB.AccessToken, FB.UserId), to IdentityInfo.CreateWithSocialNetworkInfo(IdentityInfo.ProviderFacebook, AccessToken.CurrentAccessToken.TokenString, AccessToken.CurrentAccessToken.UserId),
In FacebookInvitePlugin.cs change OnAppRequestFinished to

	private void OnAppRequestFinished(IResult result, Action<string, List<string>> completeCallback, Action cancelCallback, Action<Exception> errorCallback)
    {
        if(result != null)
        {
			var responseObject = Json.Deserialize(result.RawResult) as Dictionary<string, object>;
        ...

on Android you will have to update the manifest to <uses-sdk android:targetSdkVersion="23" android:minSdkVersion="15" />

This pretty much should do it. There is no explicit dependency between GetSocial and Facebook, so you can just update the SDK and resolve API changes issues.
Hope this helps. Please let me know if I have missed something.
P.S. Carefully remove all the previous Facebook SDK stuff so you don't get conflicts. Also if yo

Hi @TarasOsiris
Thanks for the detailed instructions.
One question: why don't you put those changes to the SDK?

Hi @p0w1nd,
These detailed instructions are only if you are updating FB SDK from 6.x to 7.x and transitioning existing app from Unity 4 to Unity 5 (aka worst case scenario). If you are importing GetSocial unitypackage into the existing project (Unity 5.x + FB SDK 7.x), you will have no compilation issues after automatic Unity API updater is run, as our SDK has no explicit dependencies on Facebook SDK API. The only thing you will have to correct is FacebookInvitePlugin.cs by changing FBResult to IResult.
We do not update our test app for now because most of our clients use Unity 4 and it will be complicated to maintain two versions of the test app in parallel, because of the Unity 4 -> 5 API changes and changes in FB SDK API. We are planning to do it in the nearest future though.

I see. Thanks a lot for the clarification. We're going to integrate GetSocial this week to our upcoming game. I got the proposal integration design doc from Milly ;)


Uncaught ReferenceError: timeago is not defined

Thanks for you work.
I config my blog comment  according to project readme docs. But it is not work . The error is 'Uncaught ReferenceError: timeago is not defined'. Can you help me？Thanks advance！

Have you add timeago.min.js into your header?
<!-- stylesheet -->
<link rel="stylesheet" href="path_to_comment_css/comment.css">

<!-- javascripts -->
<script src="https://cdnjs.cloudflare.com/ajax/libs/jquery/2.0.3/jquery.js">/script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/markdown.js/0.5.0/markdown.min.js">/script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/timeago.js/3.0.2/timeago.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/spin.js/2.3.2/spin.min.js"></script>
<script src="path_to_comment_js/comment.js"></script>

@pylixm if it helps, please close the issue.

  I did not import the js file correctly.
In addition, the css has some confused . I can solve it. I hope new theme to come quickly . Thank  you a lot .


How can relay-subscriptions be used with ReactNative and ReactNavigation?

I am using relay-subscriptions with reactnative and exnavigation. This lib is not tied to react native or any navigation lib. It should work the same way as it would on web.