How is -CertificateFilePath/CertificatePassword working without private key?

I'm unsure how it works that we don't need the private key included in the PFX file.  I've tested and confirmed that yes it does work exactly as described in the example, but I'm scratching my head as to how.  If we were doing client assertion, my understanding is that we'd be signing the JWT which would require the private key.  That obviously isn't what's happening here.  So now then, I'm confused, what is happening here?
I feel like this is less secure and as I've stated on TechCommunity, and to the EXO V2 feedback email address, I wish -CertificateFilePath and -CertificatePassword options would be removed from this module.  But in the meantime, I'd like to know what's happening here that makes it so the private key is not required in the PFX file.
I also double-checked that it continues to work even after I've deleted the certificate from my cert:\CurrentUser\My store, so I know the module isn't just using the PFX to figure out which certificate in my local store it needs to go an get the private key from.  So again, scratching my head as to what is happening here when we use the PFX option.

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

ID: d2cfafdf-cbe9-2ffe-eabe-becc66d51728
Version Independent ID: 4a46c8a8-dc70-d877-271e-6679c465a6d5
Content: App-only authentication
Content Source: exchange/docs-conceptual/app-only-auth-powershell-v2.md
Service: exchange-online
GitHub Login: @chrisda
Microsoft Alias: chrisda

Hi @JeremyTBradshaw, thank you for your feedback.
I think the best place to reach the product team and get this information is the blog:
https://techcommunity.microsoft.com/t5/exchange-team-blog/modern-auth-and-unattended-scripts-in-exchange-online-powershell/ba-p/1497387
I see that you left several comments there.

Hi @JeremyTBradshaw, thank you for your feedback.
I think the best place to reach the product team and get this information is the blog:
https://techcommunity.microsoft.com/t5/exchange-team-blog/modern-auth-and-unattended-scripts-in-exchange-online-powershell/ba-p/1497387
I see that you left several comments there.

@dariomws I will do that, but I don't necessarily think my issue is misplaced here whereas the example in the article is relevant to the question.  I guess it doesn't matter since it's not up to me anyway and clearly I'm just expressing frustration through a few different channels.  So maybe I'll step away and defuse then reassess.

@dariomws I will do that, but I don't necessarily think my issue is misplaced here whereas the example in the article is relevant to the question.  I guess it doesn't matter since it's not up to me anyway and clearly I'm just expressing frustration through a few different channels.  So maybe I'll step away and defuse then reassess.

Thank you for your reply @JeremyTBradshaw!

Thank you for your reply @JeremyTBradshaw!

@dariomws Thank you very much for the contribution and sharing this explanation. @JeremyTBradshaw If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@dariomws Thank you very much for the contribution and sharing this explanation. @JeremyTBradshaw If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@yogkumgit to be clear, no explanation was provided whatsoever. I was simply diverted elsewhere, by someone who isn't internal to Microsoft.  It's fine that this is closed, I don't even care anymore, but I'm not in agreement that any explanation or solution was provided.

@yogkumgit to be clear, no explanation was provided whatsoever. I was simply diverted elsewhere, by someone who isn't internal to Microsoft.  It's fine that this is closed, I don't even care anymore, but I'm not in agreement that any explanation or solution was provided.

@dariomws @yogkumgit and anyone else who comes along and sees this issue.  FYI I was mistaken, and in fact, in the examples, the private key is indeed included the exported PFX file.  The issue I was raising, it non-existing.
From the New-SelfSignedCertificate page, it is confirmed that the default value for the -KeyExportPolicy ExportableEncrypted.  I think I had an assumption (not really an active thought, just autopilot assumption) that the policy was defaulted to "NonExportable".
Export-PfxCertificate exports the private key, unless the certificate is marked as private key being NonExportable, which was not the case in the example's 1st command.
So the issue I was raising is not an issue at all, it was just me stuck in my own way with an underlying, incorrect assumption (which made an @$$ out of ~"u" and~ "me" (just me), as the saying goes).
🤦♂

@dariomws @yogkumgit and anyone else who comes along and sees this issue.  FYI I was mistaken, and in fact, in the examples, the private key is indeed included the exported PFX file.  The issue I was raising, it non-existing.
From the New-SelfSignedCertificate page, it is confirmed that the default value for the -KeyExportPolicy ExportableEncrypted.  I think I had an assumption (not really an active thought, just autopilot assumption) that the policy was defaulted to "NonExportable".
Export-PfxCertificate exports the private key, unless the certificate is marked as private key being NonExportable, which was not the case in the example's 1st command.
So the issue I was raising is not an issue at all, it was just me stuck in my own way with an underlying, incorrect assumption (which made an @$$ out of ~"u" and~ "me" (just me), as the saying goes).
🤦♂

Thanks for coming back with us! Nice to know that it's actually not an issue.

Thanks for coming back with us! Nice to know that it's actually not an issue.


Note clarifying port ordering

Why?

Because if one server doesn't have the port in order, you wouldn't be able to connect to it, and because I haven't seen it mentioned anywhere in the documentation I thought it would help people troubleshoot faster over a simple issue that is port ordering.
But if you prefer to not mention it or mentioning it in another way, its totally fine by me.

The port definition doesn't matter outside of a range of reserved or otherwise maybe already used ports. Your first server could have the port 5543, the next 25599, and another 25566


SecretStore misconfiguration/errors are not available as events on the resource

Describe the solution you'd like
It's possible to misconfigure a secret store in respect to a kubernetes cluster, e.g specifying a service account ref that does not exist, specifying a secret that does not exist, etc. When a secret store is misconfigured the errors that occur are not emitted as an event for the secret store resource.
What is the added value?
When users are debugging external secrets, they should be able to describe both external secrets and secret stores and discover the errors there.
Give us examples of the outcome
kubectl describe secretstore should show events.

This comment that I sent previously should be a new issue, let me have a look if we have one already

This one is already fixed in #466. It has a bug in CSS that doesn't show status, but this is addressed in #734


[VL] Fix greatest and least function tests

What changes were proposed in this pull request?
UT change.

cc @rui-mo  and @PHILO-HE , thanks.


Backup information

Don't merge yet, just putting the PR in now so I can have a review on the backuphistory type changes (paging @FriedrichWeinmann  or @zippy1981 )
Tests are failing because ideally I'd like to use 2008 and 2016 instances to prove stuff, so hoping someone's got a cunning scheme to do so.

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[x] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Add an option to build backup history up from file information if source instance isn't available.
Also allows the backup history to be built up over time, so can most files overnight, and then just add logs as neede
It can also be exported for storing or moving.
Approach
New function Get-DbaBackupInformation to scan a bunch of files and build up a dbatools.BackupHistory object
Fixed Read-DbaBackupHeader so it doesn't have an embedded datatable in the middle
Changed dbatools.BackupHistory so LSNs are longs rather thant BigInts as the latter does CliXml or Json in a nice way
Get-FilteredRestoreFile is less picky about the names of BackupTypes, so saves more workarounds for now
Commands to test
$Date = Get-Date
$history = Get-ChildItem c:\backups -recurse | Get-DbaBackupInformation -SqlInstance Server1
#wait some time
$history +=  Get-ChildItem c:\backups -recurse | Where-object {$_.LastWriteTime -gt $Date} | Get-DbaBackupInformation -SqlInstance Server1
$history | ConvertTo-Json -Depth 3 | Out-File c:\backuphistory.json
#Use method of choice to copy file to another machine
$history = Get-Content c:\backuphistory.json -raw | ConvertFrom-Json
$history | Restore-DbaDatabase -SqlInstance Server2 -TrustDbBackupHistory

Originally, I used long for that class. Long is not large enough for LSNs, which is why I moved to BigInt.

Dang. I'll change it back then. Had forgotten how large number(25,0) is
Means that porting backup history will be with Export-CliXml then. Not that
much of an issue, just getting people to do it with JSON meant I could dump
it into the test rig/unit tests for support calls.
On Mon, 23 Oct 2017 at 10:28 Friedrich Weinmann<EMAIL_ADDRESS>wrote:

Originally, I used long for that class. Long is not large enough for LSNs,
which is why I moved to BigInt.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/sqlcollaborative/dbatools/pull/2488#issuecomment-338601168,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFWoZhooYfcsPGVkLLWqClkEbfiphv8eks5svFxIgaJpZM4QB9_x
.

well, we could have  stringifier somewhere.

Some Case Sensitive weirdness has occured with git, so closing this and opening a new PR


Load text files in Artifacts as strings

I have an artifact that I've generated with QIIME 2, although now I want to operate on it in python. I try to load into python
import pandas as pd

from qiime2 import Artifact

alpha_  = Artifact.load(fp_).view(pd.Series)

In my exported series, my original barcode like, 10317.00001001 has been converted to something like 10317.00001.  When I export artifact into a text file, the sample ID are preserved. So, I think somewhere, python is doing data inference, possibly when the data is loaded as a series.

Nice catch! It's probably a transformer that is misbehaving. What does the .format property of your artifact say? It'll probably be a problem in the transformer from that format to pd.Series

The format is
qiime2.plugin.model.directory_format.AlphaDiversityDirectoryFormat
I've used a lot of pandas, and I rarely have an issue with pandas inferring data types once something is loaded into the memory.
My suspicion is that the problem comes from when the data is loaded using the pandas API. I  usually solve this by loading dataframes as text, and then casting during analysis. I think, though, you can also specify column-data type pairs with the dtype parameter.

cross-referencing a similar problem: https://github.com/qiime2/q2-diversity/issues/91


Display build/version number on login screen.

Currently, you cannot tell the build/version of CVRX without first signing in. You also have no way of checking for updates.

Planned for UI rework. Not ETA as per CVRX Team policy. Issue locked.


cannot use fastexecmany with informix odbc driver

Please first make sure you have looked at:

Documentation: https://github.com/mkleehammer/pyodbc/wiki
Other issues

Environment

Python: 3.9
pyodbc: 4.0.32
OS: centos 7.8 (64-bit)
DB: informix
driver: odbc for informix  iclit09b.so

Issue
when setting cursor.fast_execmany = True , will cause core dump or Memory Error
is there any way to insert by bulk rather than row by row as without fast_execmany did

Not all ODBC drivers support the array binding (or even implement it correctly) required to use fast_executemany. You will need to ask your ODBC driver vendor for more information.

You might try Turbodbc.


change google device id

hi how i can change google device id and imei and guid like bluestack tweaker ?

i found by changing /data/data/imei.txt in last version can change imei but other properties changeable ?


electrs in restart loop after upgrade to 1.9

Upgraded from 1.7.1 (or maybe 1.7.2) to min 1.9.0 after being offline for 9mo.  Bitcoind sync'd fine (took 3.5days) but electrs is in a restart loop.  Used systemctl to stop/start it, no difference.  Rebooted, no difference.  Uninstalled with keep index/reinstalled electrs, no difference.  (Used to work for me on 1.7.x.) 4Gb Pi4.
Debug log at termbin.  Do you need anything else?
(I'm thinking maybe it was an upgrade too far - so trying again uninstall w/ delete index, then reinstall. Will update issue later.)
Ah, did that, and see this in the logs  - but not every time, maybe only 1 time in 15 or 20:
Jul 13 20:28:06 raspberrypi systemd[1]: electrs.service: Main process exited, code=exited, status=1/FAILURE
Jul 13 20:28:07 raspberrypi electrs[2458774]: Starting electrs 0.9.11 on aarch64 linux with Config { network: Bitcoin, db_path: "/mnt/hdd/app-storage/electrs/db/bitcoin", daemon_>
Jul 13 20:28:07 raspberrypi electrs[2458774]: [2023-07-13T19:28:06.741Z INFO  electrs::metrics::metrics_impl] serving Prometheus metrics on <IP_ADDRESS>:4224
Jul 13 20:28:07 raspberrypi electrs[2458774]: [2023-07-13T19:28:06.741Z INFO  electrs::server] serving Electrum RPC on <IP_ADDRESS>:50001
Jul 13 20:28:07 raspberrypi electrs[2458774]: [2023-07-13T19:28:06.822Z INFO  electrs::db] "/mnt/hdd/app-storage/electrs/db/bitcoin": 30 SST files, 0.00002415 GB, 0.000000021 Gro>
Jul 13 20:28:07 raspberrypi electrs[2458774]: [2023-07-13T19:28:06.849Z INFO  electrs::db] closing DB at /mnt/hdd/app-storage/electrs/db/bitcoin
Jul 13 20:28:07 raspberrypi electrs[2458774]: Error: electrs failed
Jul 13 20:28:07 raspberrypi electrs[2458774]: Caused by:
Jul 13 20:28:07 raspberrypi electrs[2458774]:     0: bitcoin p2p failed to connect: <IP_ADDRESS>:8333
Jul 13 20:28:07 raspberrypi electrs[2458774]:     1: Connection refused (os error 111)
Jul 13 20:28:06 raspberrypi systemd[1]: electrs.service: Failed with result 'exit-code'.
Jul 13 20:29:07 raspberrypi systemd[1]: electrs.service: Scheduled restart job, restart counter is at 31.

(lnd took ~24hrs to get past "syncing 100%" but is finally - just this minute! - reporting correct stats.  Maybe that too was because of being offline 9mo?  Though I have exactly no channels open ...)
(BTW, electrs sure takes a heck of a long time to build, doesn't it?  Dependency rocksdb seems to be the big pig there, but cumulatively 174 dependencies ... that's kind of amazing. But this is the software development world we live in now - we all rely on others.)

Any update on this?
Looks like bitcoind was still syncing.

Update is: No, bitcoind had sync'd fine and in fact was going great, for hours.  As seen, for example, via mempool visualizer on the box.  It updated as blocks came in, in sync with https://mempool.space.  No problems.  Also, lnd sync'd to 100% although that took a very very long time.   Only problem is electrs which no matter what, no matter how many times I stopped/started it via systemctl or rebooted never did anything but its reboot loop. I even completely uninstalled it and reinstalled it (via the menu) twice: no change.
I have no further updates. (I uninstalled it and have left it uninstalled.)  Will try it again at 1.9.1.
(Unless there's some other log you'd like to see.  I could reinstall it and pull the log, if you told me which one.)


Expose active interpreter path via command

For #11440

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #11438 into master will decrease coverage by 0.00%.
The diff coverage is 95.00%.

@@            Coverage Diff             @@
##           master   #11438      +/-   ##
==========================================
- Coverage   61.03%   61.03%   -0.01%
==========================================
  Files         613      614       +1
  Lines       33614    33635      +21
  Branches     4750     4751       +1
==========================================
+ Hits        20517    20529      +12
+ Misses      12650    12075     -575
- Partials      447     1031     +584

Impacted Files
Coverage Δ

src/client/interpreter/contracts.ts
100.00% <ø> (ø)

src/client/interpreter/display/index.ts
88.67% <75.00%> (+0.44%)
:arrow_up:

src/client/common/constants.ts
100.00% <100.00%> (ø)

...onfiguration/launch.json/interpreterPathCommand.ts
100.00% <100.00%> (ø)

src/client/debugger/extension/serviceRegistry.ts
100.00% <100.00%> (ø)

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

...datascience/interactive-common/notebookProvider.ts
55.44% <0.00%> (-1.70%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

... and 144 more

Continue to review full report at Codecov.

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

Closing this PR as ${config:python.pythonPath} already provides this functionality in a more robust way. More details in the issue.


Added some unit tests for Issue #363

I've tried to reproduce the Issue #363 in three ways:

Using my implementation of NSProxy,
Using OCMockito mock object,
Using OCMock mock object.

In all cases everything seems to work fine, the tests are passing. I've excluded the OCMock test from the PR, because it's an unnecessary dependency for the project itself.

That's nice, thanks!

Thanks!


Mediawiki MobileView API deprecated, Mobile Content Service deprecated too, migrate to WMF-recommended solution

https://phabricator.wikimedia.org/T210808
https://www.mediawiki.org/wiki/Mobile_Content_Service

Hi @nicolas-raoul Please assign to me ,I will work on this.

@awanishyadav967 It is yours, thanks! Please let us know about your progress or share your findings every few days.

@nicolas-raoul The issue seems to be unassigned. Could you please assign the issue to me?

@kartikaykaushik14 Of course, thanks!

Hello @kartikaykaushik14, I hope this message finds you well. I noticed that this issue was assigned to you two weeks ago and was wondering if there has been any progress or updates. If something has come up that is preventing you from working on this issue or if you need any assistance, please let me know. I would be happy to help if needed. Thank you.

@kartikaykaushik14 Thanks for the hint!
It looks like Parsoid is JavaScript, it might be usable via WebViews.
One could argue that we should follow what the Android Wikipedia app is doing, which is apparently the Page Content Service ("PCS"), but a counter-argument would be that it is unstable.
The first task would be to list our different use cases of the API, to see how well these use cases are covered by Parsoid and PCS, possibly via a tiny prototype app.

I was trying to analyze which REST endpoints are impacted by the deprecation of MobileView API and their possible alternatives from Page Content Service.

To GET the Lead Section and initial metadata of a given title. Alternative: getSectionsLead
To GET the remaining sections of a given title. Alternative: getSectionsRemaining
To GET the remaining sections request the URL of a given title. However, this endpoint is only to get URLs for the remaining sections and once we have an alternative to get the remaining sections directly, we do not need this endpoint.

But, the endpoints are currently unstable as per current documentation.

Even these endpoints in Page Content Service are now deprecated. Parsoid seems to be the only recommended option.

Are there any updates on Mobileview API replacement?
July 2023 (when this API is supposed to stop working) is approaching rapidly.
Also, what is expected to break if MCS API is disabled on server side if commons app doesn't have it replaced yet?
Will the commons app remain usable (fully or partially), or stop working completely?

@mnalis Thanks for the nudge. As far as I could see, MCS has not yet been decommissioned. It still works but it has been deprecated [ ref ].
So, I suppose we should explore migrating to a different API soon.

I think that could be a GSoC project. If too easy, this plus #3083.

@sivaraam Sorry I couldn't continue further on the issue. Will be a little free from work in the upcoming weeks. But feel free to reassign this issue or make it a part of next GSoC.

No issues @kartikaykaushik14 ! Kindly feel free to work on this if you're able to find time.
At a later stage, if there's anything that's left to be converged, we could convert that remaining part into a GSoC project.

Also, what is expected to break if MCS API is disabled on server side if commons app doesn't have it replaced yet? Will the commons app remain usable (fully or partially), or stop working completely?

We pinged the WMF team for some inputs on how to go about with the migration. @dbrant kindly took a look at our code and provided the following clarification:

And regarding Mobile Content Service deprecation: sorry there's been a
fair amount of confusion about this. From a cursory look at your code, it
doesn't seem like you actually use any of the endpoints that are
deprecated. Even though those functions and model classes are declared in
your "data-client" subproject, they seem to be only used in unit-testing
methods, all of which can be safely removed. (unless I missed a use-case
for these functions in your code.)

So, I think the only thing left to do here is to identify the stale portions of code and remove them :-)

@kartikaykaushik14 Are you planning to work on this? :-)

I've spent some time digging through the data-client module to remove "dead code" that appears to be unused.  If you're curious, it's on a branch that you can browse.
If we're looking for a migration path to a different API, I hope you find the investigation useful!

@psh Wonderful! Feel free to send a pull repuest. 🙂

From what I have seen, now that dead code is pretty much gone, is that we are using 5 things from the data-client module that could all be pulled over and incorporated into the main commons app code -

Base data model classes for wikimedia actions API requests / responses
CSRF Token management
Notifications
Login
Thanks request/response

In terms of size, "thanks" and "notifications" are probably the easiest and mirrors other self-contained API calls we make in the app as far as work goes.  Something like CategoryInterface and CategoryClient might be a template for how it would look.
Login already exists in the app, so we'd be moving 4 classes over from the data client.
If we leave the data model base classes for last, then (I think) each of the other steps is self-contained and you wont end up with strange circular dependencies of the data client module depending on the main app.

Sounds like a good plan. Thanks a lot for taking the time to help with the clean up of the data-client module! 🙂

OK, quick "state of the union" update for those who are interested -

Commons app is now 100% self-contained as far as the API service interfaces are concerned.  They're created and managed consistently across the board and we should be able to see exactly the APIs we're using
Slightly deeper, the OkHttp and Retrofit caching is now more efficient and shared across more of our API classes.  I may be a little biased, but I think performance and memory usuage have improved 😄
Each of the PR so far has (as) moved to newer Kotlin and (b) reduced overall lines of code

So, what's left?

The AppAdapter abstraction is 90% unused, and digging into it, manages just cookie persistence and our OkHttp instance.  Since OkHttp uses the CookieJar these amount to roughly the same issue and will be addressed in an up-coming PR.  The AppAdapter will then go away.
Model classes in the "data-client" will need to merge with their commons app sibling (as appropriate) or move.
Deletion of the data-client - LOTS of files will be touched in the form of deletions.  It will be a pretty big commit but most of the work has already been done to move away from it.

Wonderful! Thanks for all of this great work! 🙂

Quoting from earlier in this issue -

And regarding Mobile Content Service deprecation: sorry there's been a
fair amount of confusion about this. From a cursory look at your code, it
doesn't seem like you actually use any of the endpoints that are
deprecated. Even though those functions and model classes are declared in
your "data-client" subproject, they seem to be only used in unit-testing
methods, all of which can be safely removed. (unless I missed a use-case
for these functions in your code.)

So, I think the only thing left to do here is to identify the stale portions of code and remove them :-)

With the most recent commit (thanks to @nicolas-raoul for his patience as it was pretty huge) the "data-client" module is gone.  The small portion of code from it that we were using is incorporated into the commons app proper.
I believe that wraps up the work we wanted to do here, unless there are other API related cleanup operations to address?  Maybe we can gather our thoughts in this thread, spin off separate issues if we spot things we want to do, and close this one?

I believe that wraps up the work we wanted to do here, ...

I suppose so too. Thanks a lot for taking care of this in review friendly part-by-part fashion, @psh! 🙂

... unless there are other API related cleanup operations to address?

I can't recollect any API cleanup similar to this one that we've left out. On a tangent, there are a few API related issues such as #5244 / #3179 that are still open in our issue tracker. Feel free to take a stab at them if you're interested.
If you're looking for other kind of issues, feel free to elaborate. We could identify relevant ones.

Maybe we can gather our thoughts in this thread, spin off separate issues if we spot things we want to do, and close this one?

Most certainly 👍🏼

Thanks a lot Paul for this fantastic work! 🙂


Baseline CLIP Model

I am trying to reproduce the Baseline CLIP results for Single object GQA setting. I am getting a much lower mAP of 0.18, which does not match the paper's numbers, I am using the pooled output of CLIP's text and image encoder, does this match the paper's implementation. I am using openai/clip-vit-base-patch32

This was the code used for the baseline CLIP:
class CLIP_base(nn.Module):
    def __init__(self, args):
        super().__init__()
        self.model = CLIPModel.from_pretrained("openai/clip-vit-base-patch32")
        self.model.eval()
        for p in self.model.parameters():
            p.requires_grad = False

    def forward(
        self,
        input_ids,
        attention_mask,
        pixel_values,
    ):
        # pdb.set_trace()

        out = self.model(
                input_ids=input_ids,
                attention_mask=attention_mask,
                pixel_values=pixel_values,
                output_hidden_states=True,
            )

        logits_per_image = out.logits_per_image

        return {"scores": logits_per_image}

Make sure you are evaluating on the hard list. See scripts/eval.py.

Thank you so much for the prompt reply! Yes I was using the hard list for evaluation, I will try again with the above snippet.


Fuzzy matching support for searchable?

I'm wondering if you would accept PRs to implement fuzzy matching for searchable options.

Proooobably not, as I like that this package doesn't have any third-party dependencies, and I don't have a library that does fuzzy searching, but I've been meaning to open-source one I've written actually, if it's small enough (it probably is) I can wire the two together 👍 Give me a few days, I'm currently busy on other projects.

Maybe a searchable can accept a function to provide a custom search filterer? And then it is possible to use fuzzy searching without massively increasing bundle size?

That could also be an option 🤔

Another option would be to use peerDependencies + peerDependenciesMeta with optional: true.

 Relevant to this use case:

The string and number prompts now have an optional format function, which can be used to style the user's input string. As I understand it this is something you guys missed for your color input.
The select and multiselect prompts now support receiving options as a function that maps the current query to an array of options. This can be used to customize how results are ranked, and in combination with searchable: false this also allows you to customize how results are searched into and highlighted too. A limitation with this option is that the new array of options must be provided synchronously. As I understand it this should be good enough for you, right? Ideally it should support providing new options asynchronously also, but it gets a bit tricky because the user should be able to press more keys in the meantime, which is tricky to handle, so I haven't implemented that yet.


HTML download is incorrect in Splash HTTP UI

Render this URL http://forums.pdfforge.org/categories/help using default script in Splash UI;
Click 'download' button near 'html'.

For me it returns a 4K document, while it should be 60K+. HTML code in the textarea is correct.
Firefox 44.0.2, OS X.

I have test it, seems render no problem for me.

Version 48.0.2564.116 (64-bit), OS X.

I have test it, seems render no problem for me.

Chrome Version 48.0.2564.116 (64-bit), OS X.

It also render fine for me, but if I click 'download' link and save the result then the resulting file is truncated.

Yes, When I click 'download', it saved as 'forums.pdfforge.org.txt'

Aha, it also works for me in Chrome; it looks like the error is specific to Firefox.

Seems like different browser (and different versions of the same browser) have different limits on the maximum length of a data url (source). I'll try to find a workaround for this.


Update GitHub action to build Asciidoc branch

Currently there is a branch which contains the unprivileged
specification code in asciidoc format. This action should only be run on
that branch. Once the conversion of the privileged specification is
complete as well, this action can be updated to build the complete set
of specs.
Signed-off-by: Stephano Cetola<EMAIL_ADDRESS>

I can't say I understand why this needs to go on the master branch, but if it works, it's fine for now.


Rename git app

Renamed application: git -> dataset_repo
resolve #1329
Motivation and context
Currently the name of the Django app is causing some issues with tests and python imports.
How has this been tested?
Manually tested (update, fresh installation)
Checklist

[x] I submit my changes into the develop branch
[ ] I have added description of my changes into CHANGELOG file
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
[ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[x] I have updated the license header for each file (see an example below)

# Copyright (C) 2020 Intel Corporation
#
# SPDX-License-Identifier: MIT

Coverage decreased (-1.1%) to 63.705% when pulling e7a82cd73077c031b6777660c3d3ba8a6cf228c4 on az/rename_git_app into 50527371ba650b58d8795470f8f95503d58e4b90 on develop.

@azhavoro , could you please fix conflicts?


Do we know why random tests are failing?

Do we know why random tests are failing during CI? They fail, and then pass on the second attempt - it's a bit annoying for something that should be reproducible.

It's bizarre, isn't it? Here is one example failure:
=========================== short test summary info ============================
[117](https://github.com/kymata-atlas/kymata-core/actions/runs/11416705740/job/31768215218#step:7:118)
FAILED tests/test_expression.py::test_hes_rename_functions_just_one - assert <kymata.entit...x7f36ca0be350> == <kymata.entit...x7f36ca193010>
[118]...

[119]...
  Full diff:
[120]...
  - <kymata.entities.expression.HexelExpressionSet object at 0x7f36ca193010>
[121]...
  ?                                                                  ^^ ^^
[122]...
  + <kymata.entities.expression.HexelExpressionSet object at 0x7f36ca0be350>
[123]...
  ?                                                                  ^^^ ^
[124]...
=========== 1 failed, 146 passed, 3 skipped, 290 warnings in 11.28s ============
[125]...
Error: Process completed with exit code 1.

And the intermittent failures I've inspected are always similar, like a failure in object identity.
However looking at that test test_hes_rename_functions_just_one():
def test_hes_rename_functions_just_one():
    data_left = [np.random.randn(5, 10) for _ in range(2)]
    data_right = [np.random.randn(5, 10) for _ in range(2)]

    es = HexelExpressionSet(
        functions=["first", "second"],
        hexels_lh=range(5),
        hexels_rh=range(5),
        latencies=range(10),
        data_lh=data_left,
        data_rh=data_right,
    )
    target_es = HexelExpressionSet(
        functions=["first_renamed", "second"],
        hexels_lh=range(5),
        hexels_rh=range(5),
        latencies=range(10),
        data_lh=data_left,
        data_rh=data_right,
    )
    assert es != target_es
    es.rename(functions={"first": "first_renamed"})
    assert es == target_es

That should not be failing intermittently.
Here's the only thing I can imagine:
Tests for ExpressionSet equality are done by comparing data blocks (and other things). This will involve float many float comparisons, and the data blocks in this test are randomly generated. Perhaps some random floats fail at equality comparison because some spooky nondeterministic behaviour in the Gitlab CI runner. (I've never made it fail by running locally.)
If this is the case, then using non-random data would at least fix this issue, even if it didn't fully explain it.

Here is another example of a randomly failing test involving float comparisons but not involving sparse...

One thing which might work would be to force (e.g.) numpy.int32s in all tests, in case it's some kind of heterogeneous architecture issue with the Github CI runners.

One thing which might work would be to force (e.g.) numpy.float32s in all tests, in case it's some kind of heterogeneous architecture issue with the Github CI runners.


Jobs are not being precessed immediately

Hi there,
I have this workers pool set
	workers, err := gue.NewWorkerPool(
		gc,
		wm,
		10,
		gue.WithPoolQueue(printerQueue),
		gue.WithPoolPollInterval(time.Minute*5),
		gue.WithPoolLogger(azap.New(zap.NewExample())),
		gue.WithPoolPollStrategy(gue.PriorityPollStrategy),
		gue.WithPoolHooksJobDone(finishedJobsLog),
	)

My impression was that gue will process any added messages right away, based on priority, but it looks like it's waiting for 5 minutes poll interval
Is this an expected behavior?
My use case is that I want to process newly added jobs right away and make them retry in 5 min if they fail.
Thanks

If you need near-real-time jobs handling you need to set poll interval to a very small value - e.g. 500ms. This parameter is responsible for polling pause in case there are no jobs currently to be handled.
For retry in 5min you need to either set Backoff strategy when you instantiate a client using WithClientBackoff() option, for your case NewConstantBackoff(5*time.Minute) should work, or you can also return ErrRescheduleJobIn(5*time.Minute, ...) error explicitly for the failed jobs in the handler.

If you need near-real-time jobs handling you need to set poll interval to a very small value - e.g. 500ms. This parameter is responsible for polling pause in case there are no jobs currently to be handled.
For retry in 5min you need to either set Backoff strategy when you instantiate a client using WithClientBackoff() option, for your case NewConstantBackoff(5*time.Minute) should work, or you can also return ErrRescheduleJobIn(5*time.Minute, ...) error explicitly for the failed jobs in the handler.

I appreciate.
One more question,
Is it explicitly possible to mark job as done? Besides removing its record from the db.
Thanks!

You can return ErrDiscardJob(...) in the handler to discard a job explicitly https://pkg.go.dev/github.com/vgarvardt/gue/v5#ErrDiscardJob

You can return ErrDiscardJob(...) in the handler to discard a job explicitly https://pkg.go.dev/github.com/vgarvardt/gue/v5#ErrDiscardJob

I was mostly interested on calling it outside of any gue handlers.

Then only direct DB intervention.

Thank you!


Version bump 0.3.0 -> 0.4.1

@goozzik why 0.4.1 and why it's pushed already?

@Strech I had to to update to 0.4.1 instead of 0.4.0. At first I released the version with 0.4.0 and then removed it from the rubygems and I couldn't rerelease the same version dunno why.

@goozzik yes, only from master. And yes you can't delete versions from rubygems, it's not NPM

Please next time pay some attention


[improvement](invert index) Change the loading method of keyword type

Proposed changes
Issue Number: close #xxx

fix can not index Chinese
optimized invert index load

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

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 53.72 seconds
stream load tsv:          501 seconds loaded<PHONE_NUMBER>9 Bytes, about 142 MB/s
stream load json:         18 seconds loaded<PHONE_NUMBER> Bytes, about 124 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.7 seconds inserted 10000000 Rows, about 336K ops/s
storage size:<PHONE_NUMBER>0 Bytes

run p0

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 54.42 seconds
stream load tsv:          503 seconds loaded<PHONE_NUMBER>9 Bytes, about 141 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.9 seconds inserted 10000000 Rows, about 334K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 51.25 seconds
stream load tsv:          506 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.2 seconds inserted 10000000 Rows, about 342K ops/s
storage size:<PHONE_NUMBER>0 Bytes


Inconsistent naming with blocks

Although a small issue, I've noticed that the blocks from this mod have different naming when compared to the vanilla version. For example:
"Block of Raw Iron" and "Block of Iron" is the naming convention in vanilla..
Whereas in Mythic Metals it follows as "Raw Banglum Block" and  "Banglum Block"

See this commit: df0a01d

This is indeed resolved for the next upcoming patch of Mythic Metals.

Fixed in 0.19.3


DPXInput::read_image

Description
From this mail thread
The reading of DPX files one scanline at a time bogs down by reading the image opposite of the way it was laid out on disk. So by the end of the image you've permuted each scanline multiple times. It's extremely prevalent over a network connection. Where possible, we should read in chunks. This will slurp the entire image when calling read_image(...).
I have no reservations on the code - only that it works quite well for grepping the whole image. If someone wishes to reinvent this to better fit the OIIO spec I'm happy to relinquish control over it. This was mainly just a "it worked for us" fix.

A special thanks to @menssimia and his brain for identifying where the performance killer was. If anyone has questions, he may be the best one to answer the nitty-gritty of the dpx code.

Checklist:

[x] I have read the contribution guidelines.
[x] If this is more extensive than a small change to existing code, I
have previously submitted a Contributor License Agreement
(individual, and if there is any way my
employers might think my programming belongs to them, then also
corporate).
[x] I have updated the documentation, if applicable.
[x] I have ensured that the change is tested somewhere in the testsuite
(adding new test cases if necessary).
[x] My code follows the prevailing code style of this project.

Should have seen that this PR has already been made. closing this and will respond to the existing PR


Remove or update code snippets on message edit

At the moment, posting a link to github causes the bot to respond with a manually performed codeblock snippet of the contents linked. However, if the user edits their message to change the link, the contents of the embed do not change, and the user has to re-send their message to get updated contents.
We've previously implemented similar behavior for the Eval cog, with a refresh reaction. However, in this case, because the response is automatic and not derived directly from user interaction I think it makes sense to automatically keep the contents "up to date" with the origin message. If the link is edited, we edit our message, or if the link is removed, we remove our message.
We should probably still keep the trashcan reaction so users can delete the response independently of this.

This seems fine to me, though i'm not 100% sure it's worth the added complexity for a fairly rare case. I don't mind too much either way so if someone else vouches for this being a useful feature I think it would be ok to add.


Automate process of creating new bob releases

We should have a script that we can run that will create a new bob release

FYI https://github.com/TongkunZhang/bob-release-tools

Thanks!

This is all documented in https://ibm.github.io/ibmi-bob/#/contributing/getting-started


add emulators manually

Please add the option to add emmulators manually until the developer adds them to Daijishou himself.
This is especially useful when the dev is inactive for a while and new emators pop up.
I wanted to add lemuroid, but it's not available.

Lemuroid doesn't support launching roms from a frontend or adb session. I've been updating the platforms for the past year or some odd months. Make to keep the platforms up-to-date for the latest players by -
Manually updating platforms:

Daijishou > Download

Or enabling Update platforms automatically:

Daijishou > Settings > Library > Enable Update platforms automatically

Also you can already add your own players:

Daijishou > Settings > Library > Add a player

You can also create/edit json files. You can see how they are typically setup by looking at the official platform json files here
Hope this helps!

Thanks.


Attempt to invoke virtual method 'zendesk.core.CoreModule zendesk.core.ZendeskShadow.coreModdule()' on a null object reference

This error "Attempt to invoke virtual method 'zendesk.core.CoreModule zendesk.core.ZendeskShadow.coreModdule()' on a null object reference" pops when calling .
ZendeskChat.startChat({
name: 'test',
email: 'test',
phone: 'test',
tags: ['tag1', 'tag2'],
department: 'Your department',
chatOnly: false,
botName: 'bottest',
});

Did you init with proper keys before using the startChat method? I tried and couldn't replicate it out.

same here, react-native 0.63 and I called   RNZendeskChat.initChat(CHAT_ID); before

Could you like get me a demo app or something? I am unable to replicate this out.

initChat is broken in latest release, make sure you use init instead. Use the keys from the Mobile SDK section in Zendesk

I'll look over the initChat method

I see no issue initChat worked just fine, can you guys let me know what exactly is the issue you facing?

Did you also run .init in the same project? The issue is with projects that
do not call init.
On Fri, 20 Nov 2020, 09:23 Saransh Malik<EMAIL_ADDRESS>wrote:

I see no issue initChat worked just fine, can you guys let me know what
exactly is the issue you facing?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Saranshmalik/react-native-zendesk/issues/3#issuecomment-731050932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIVNFDICAAOS6TV4XUXN53SQYYPXANCNFSM4SQNBE4A
.

Nope I replaced my init call with initChat and then used startChat, it connected with my zendesk chat and I could send and receive messages

Did you guys use chatOnly:true while using startChat function?

I did yeah 😕
On Fri, 20 Nov 2020, 10:08 Saransh Malik<EMAIL_ADDRESS>wrote:

Did you guys use chatOnly:true while using startChat function?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Saranshmalik/react-native-zendesk/issues/3#issuecomment-731075086,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIVNFBOU6UXBLFUE2NC3GDSQY52BANCNFSM4SQNBE4A
.

Which RN version did you try on? Maybe some issues with RN versions.

0.63.3
On Mon, Nov 30, 2020 at 10:52 AM Saransh Malik<EMAIL_ADDRESS>wrote:

Which RN version did you try on? Maybe some issues with RN versions.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Saranshmalik/react-native-zendesk/issues/3#issuecomment-735710713,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIVNFCFQMFJVXFHIKS25A3SSN2OBANCNFSM4SQNBE4A
.

I tried on the same too. And use the lib in a production app and never heard back of such crash. Will try investigate more, you recieve the exact error as the one mentioned?


Kmod fails to build with zlib-ng

checking for liblzma... yes
checking for zlib... no
configure: error: Package requirements (zlib) were not met:
Package 'zlib', required by 'virtual:world', not found

Consider adjusting the PKG_CONFIG_PATH environment variable if you
installed software in a non-standard prefix.

Alternatively, you may set the environment variables zlib_CFLAGS
and zlib_LIBS to avoid the need to call pkg-config.
See the pkg-config man page for more details.

pkgconfig file for zlib-ng was missplaced. Moved if from /lib/pkgconfig to /usr/lib/pkgconfig


Beaker throws no error when failing to write to folder that has a mount at the same path

Describe the bug
When using the structure beaker.hyperdrive.drive(profile.url).writeFile(PATH + filename, JSON.stringify({some: data}, null, 4)) if PATH is a path that currently has a drive mounted at the same location then writeFile will fail with no error and no indication that it has failed to write the file.
To Reproduce
Steps to reproduce the behavior:

Mount a drive at path X
Execute the following:

   profile = await beaker.contacts.requestProfile()
    try {
      await beaker.hyperdrive.drive(profile.url).mkdir('/X/').catch(e => undefined)
      await beaker.hyperdrive.drive(profile.url).writeFile('/X/'+ 'someFile.json', JSON.stringify({foo: 'bar', bar: 'foo'}, null, 4))
    } catch (e) {
      console.log(e)
    }

Check console, no error, check mounted drive, no file, check profile and no file there either.

Expected behavior
It should either throw an error indicating it cannot write to the mounted drive (should it be able to?) or it should write to the mounted drive.
Desktop (please complete the following information):

OS: Win7
Browser Beaker
Version Latest

Additional context
index.js check function onSubmit at hyper://305819ca846e0493ea4fe786d20b62a2566326bd4e5f3abe2331924e7fbb3b3e/

Good catch, thanks for filing

Huh, I can't repro this. I'll try it on windows to see if it's OS-specific, but that would be odd.

@pfrazee I'll put aside some time later to try and replicate on my OSX box to see if I can get some details there.


Remove application.properties

It may not be good to keep application.properties in the starter project
Also, the EnvironmentPostProcessor already takes care of it

Resolves #24

Merged as 42a08dbe4ad5cb827d25d82c2e8b8718ce2790cc


We cannot verify signed legacy CLAs till logged in

The only way to check the emails of a user is if we have their access token. This means the user will be prompted to sign the CLA and then authenticate and find out that they have signed it.
We will need a flow that marks the pull request as success

Alternative proposal: How about using the mail address from the commits as a fallback option? If all commits in a particular PR carry the same mail address.

@mp911de Thanks for the feedback. I don't think this will help.
The problem (more clearly defined) is this:
When a pull request is created we receive a pull request event. The only user related information in the event that we get is the GitHub username / id.
If the user signed a CLA in a legacy system (i.e. Spring's CLA) that only contains first name, last name, and email.
We cannot link the user who signed the CLA to the user in the pull request event. This is because there is no common information between the two. Furthermore, we cannot use the GitHub API to access the user's email address because it is not exposed to us unless we have an access token from that user.
If the user has authenticated with our system, we would have been able to either store an access token for retrieving the emails or the emails themselves. At that point when we receive the GitHub event we will be able to link the event and the legacy CLA via the email address.
When a user who signed in the legacy system authenticates for the first time, we will need to ensure to mark their pull request as successful at the time of authentication (right now we only update the pull request at the time of signing).

About this flow: How about adding two further elements:

A dashboard page where the user can see his signed CLA's
Add a migration element so if the user is sent to the sign CLA page we can then tell him in a note: "Thanks, we migrated your old signature to the new system, now you can proceed"

The migration step would then also update the PR (which is the missing part to get it working).

@mp911de Thanks for the reply.

A dashboard page where the user can see his signed CLA's

I actually just removed the dashboard page as this seemed to clutter things up and not really provide value for any workflows. See #50 I'd be interested in your take as to why we should have a dashboard.

Add a migration element so if the user is sent to the sign CLA page we can then tell him in a note: "Thanks, we migrated your old signature to the new system, now you can proceed"

I think I have a solution for this. The part I'm having a bit of trouble swallowing is that we will need to update the state of the GitHub PR on a Git.
I'm going to go ahead and assign this one to me as I think I have it mostly sorted out in my head

We have done all we can here


Exclude Program.cs/top-level program from coverage

Description
I have the top-level ASP.NET Core program src/WebApp/Program.cs using minimal APIs (see here). When calculating the coverage using dotnet-coverage collect 'dotnet test --no-build' -s 'src/Tests/dotnet-coverage-settings.xml' -f xml -o 'coverage.xml' (see here), this code is not excluded from coverage, although I've configured it in src/Tests/dotnet-coverage-settings.xml:
<?xml version="1.0" encoding="utf-8"?>
<Configuration>
    <CodeCoverage>
        <Attributes>
            <Exclude>
            <!-- Don't forget "Attribute" at the end of the name -->
                <Attribute>^System\.Diagnostics\.DebuggerHiddenAttribute$</Attribute>
                <Attribute>^System\.Diagnostics\.DebuggerNonUserCodeAttribute$</Attribute>
                <Attribute>^System\.CodeDom\.Compiler\.GeneratedCodeAttribute$</Attribute>
                <Attribute>^System\.Diagnostics\.CodeAnalysis\.ExcludeFromCodeCoverageAttribute$</Attribute>
            </Exclude>
        </Attributes>
        <Sources>
            <Exclude>
                <Source>src/WebApp/Program.cs</Source>
            </Exclude>
        </Sources>
    </CodeCoverage>
</Configuration>

In Coverlet I could resolve this with the following hack (see here):
[ExcludeFromCodeCoverage]
public partial class Program
{
}

But this doesn't seem to work with dotnet-coverage.
Environment

.NET SDK → 7.0.100
dotnet-coverage → 17.5.0
Running as GitHub Action (see here)

You should be able to use [ExcludeFromCodeCoverage] also in dotnet-coverage case.
<Sources> excludes are regexes checked always against full path so please try:
        <Sources>
            <Exclude>
                <Source>.*src/WebApp/Program.cs</Source>
            </Exclude>
        </Sources>

Thx! I tried but it is still not excluded: https://sonarcloud.io/summary/overall?id=mu88_Project28

Can you access the coverage report itself? I think I recall that there was some processing done by sonar.

@mu88 could you please provide coverage report file as @Evangelink  requested? We need coverage.xml file.

Please see the attached ZIP: it contains coverage.xml, calculated not via GitHub Action, but on my dev notebook
coverage.zip

It looks good in report. Methods are excluded:
<skipped_function id="19052" original_id="19052" token="0x60000d7" name="MoveNext()" type_name="Program.<<<Main>$>g__CreateDbIfNotExistsAsync|0_4>d" reason="attribute_excluded" />
        <skipped_function id="19496" original_id="19496" token="0x60000da" name="MoveNext()" type_name="Program.<<Main>$>d__0" reason="attribute_excluded" />
        <skipped_function id="20280" original_id="20280" token="0x60000de" name="<<Main>$>b__0_0(Microsoft.Extensions.Hosting.HostBuilderContext, System.IServiceProvider, Serilog.LoggerConfiguration)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20446" original_id="20446" token="0x60000df" name="<<Main>$>b__0_1(Microsoft.AspNetCore.Components.Server.CircuitOptions)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20500" original_id="20500" token="0x60000e0" name="<<Main>$>b__0_2(AutoMapper.IMapperConfigurationExpression)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20520" original_id="20520" token="0x60000e1" name="<<Main>$>b__0_3(Microsoft.AspNetCore.Builder.RequestLocalizationOptions)" type_name="Program.<>c" reason="attribute_excluded" />

Could you please upload during run and provide coverage report from github actions?

Here it is from a run via GitHub Action this morning:
code-coverage-report.zip

Same thing:
        <skipped_function id="19052" original_id="19052" token="0x60000d7" name="MoveNext()" type_name="Program.<<<Main>$>g__CreateDbIfNotExistsAsync|0_4>d" reason="attribute_excluded" />
        <skipped_function id="19496" original_id="19496" token="0x60000da" name="MoveNext()" type_name="Program.<<Main>$>d__0" reason="attribute_excluded" />
        <skipped_function id="20280" original_id="20280" token="0x60000de" name="<<Main>$>b__0_0(Microsoft.Extensions.Hosting.HostBuilderContext, System.IServiceProvider, Serilog.LoggerConfiguration)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20446" original_id="20446" token="0x60000df" name="<<Main>$>b__0_1(Microsoft.AspNetCore.Components.Server.CircuitOptions)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20500" original_id="20500" token="0x60000e0" name="<<Main>$>b__0_2(AutoMapper.IMapperConfigurationExpression)" type_name="Program.<>c" reason="attribute_excluded" />
        <skipped_function id="20520" original_id="20520" token="0x60000e1" name="<<Main>$>b__0_3(Microsoft.AspNetCore.Builder.RequestLocalizationOptions)" type_name="Program.<>c" reason="attribute_excluded" />

Maybe there is issue in sonar cube?

Okay, thx for your help! I'll ping Sonar again (see here) and close this issue in the meantime.


waiting for a very long time for the reponse

python version:2.7
requests version:2.11.1
when the response expected to be redirect(301)，after I send out the request.get message, it will take a long time to get response.
however in the wireshark can see the response is immediately returned.it seems like somewhere in code after get the response takes the time out

Is the response body complete immediately? Can you provide a URL to reproduce this error?

This has been stale for nearly a month. We can reopen this if we get more information and can determine that this is actually a bug in Request.


Update Global Chat

Fixes the security issues that resulted in plugin getting disabled in https://github.com/runelite/plugin-hub/commit/9f331caab55f672ad96ea8d69e55d33bae08da57

This is exfiltrating all dms to a channel anyone can subscribe

Moves from using ably's basic auth to using token auth with minimal permissions.
Revoked previous API key
Encrypts all messages

allows spoofing messages with the mod crowns.

Fixed by only allowing ironman icons with the new and getValidAccountIcon and isInvalidUsername function

bump

Instead of checking for specific control characters, it's better to just sanitize the entire message using `Text#removeTags. https://github.com/RusseII/region-chat/compare/403f4d0d17dd537838f3929a37bb095e1e5e3372...RusseII:9cdb1c0f31c57a214209a836cfbb2973254d629d#diff-51d34e1338ae68af40061f38cd1ddea82fbcf922e3c55828aa0e3264d16fd0c2R366-R368

Marking as draft for now. Please make the requested changes, then mark the plugin as ready for review.

Thanks @LlemonDuck. Added your suggestions


Changed "tags {" to "tags = {" as required by terraform 0.12

Terraform 0.12 requires tags = { since tags is a map attribute see issue #19240 https://github.com/hashicorp/terraform/issues/19240

Coverage remained the same at 100.0% when pulling efed27484729b54fe84a5cd20901881894af42b0 on dimitriosoodle:tags-need-equal-sign into 1ba96b958123087dc1454e284ce34cc9c00b69a0 on dtan4:master.

Sorry for the late review 🙇 Please do the followings to merge.

Please merge the current master and resolve conflicts
Please update unit tests

Is this change works well with Terraform 0.11.x and below?

I have tested terraform 0.11.3 and the changed seem to work

I have pulled the latest changes, dealt with conflicts and new features and addresses the unit tests.

@dtan4 Any way this is getting merged any time soon? This is a great fix and a big miss to the latest terraform versions.
Thanks @dtan4 and @dimitriosoodle for your support!


setup.py integration doc not for real, multi-file use case

It appears to me that the example integration with AOT and distutils (or setuputils) works only when there is one and only one source file.  And from #3823 it also seems that the source file can contain only one level of hierarchy in its functions.  The documentation (and perhaps implementation) sorely needs a real world example with multiple source files in a package subdirectory.
The typical pattern for python packages is to put the python source files in a directory below the directory holding setup.py.  The numba setup.py example in the documentation, however, shows the .py file to be compiled into the same directory.
This is a one-file use case, all the source must be in a single file.
When your source files are in the package subdirectory then you have to import the AOT file like this:
from package_name.my_module import cc
And when you do this, the extension module shared library ends up in the directory above the package and is therefore a standalone, separate module.  With this configuration the source file is also installed inside the package module directory so that it s .pyc file is what runs when you import via package_name.my_module.  So such a configuration runs, but runs the pyc code instead of the compiled code inside the compiled module.

@nmgeek it seems that this issue hasn't seen any attention for over two years. Can you confirm that the bug is still present in the source?


Resolve index.js.flow files in npm modules (matching flow-cli behavior)

It is appears not possible to import types that are exported by an npm module in a natural way.  For example, this works with flow but results in named error from eslint-plugin-import
import type {FormProps} from 'redux-form'

but this works and results in no eslint error
import type {FormProps} from 'redux-form/es/index.js.flow'

additionally
import type {FormProps} from 'redux-form/lib/index.js.flow'

I am happy to dig into root cause more (opening this now for tracking), but my guess is that the package.json module/main entries aren't being followed to find index.js.flow files.  E.g., these fields in redux-form are
  "main": "./lib/index.js",
  "module": "./es/index.js",
  "jsnext:main": "./es/index.js",

Is it possible that this is only happening for interfaces?


Why is event id 8004 included in the list of common events(Security)

Could you please suggest the reason event id 8004 is included in the list of common event as It is not a security event and In case a user wants to enable it on Sentinel , He needs to add NTLM/operational in windows event logs.

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

ID: 036b8907-bd6b-1414-1f0f-621e63149232
Version Independent ID: eaeb654c-d5fe-2392-8dca-e4a426bd2c69
Content: Windows security event sets that can be sent to Microsoft Sentinel
Content Source: articles/sentinel/windows-security-event-id-reference.md
Service: microsoft-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

@neerajsharma1088
Thanks for your feedback! We will investigate and update as appropriate.

Hi @neerajsharma1088
Event ID 8004 is included in the list of common events because it is a Windows event that can provide valuable information about the behavior of your systems. It is not considered a security event, but it can still be useful for troubleshooting and monitoring purposes.

@neerajsharma1088
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation

Hi @ManoharLakkoju-MSFT Thanks for your response, I understand that 8004 is an important event but my query is that why is it included in the list of common security event list because even if It gets generated on the server It won’t be forwarded to Sentinel (Provided that Common tier of security events is enabled over Sentinel).

@neerajsharma1088
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). Or you can leverage our Q&A forum by posting your issue there so our community, and MVPs can further assist you in troubleshooting this issue or finding potential workarounds.
[Teams Q&A forum] (https://docs.microsoft.com/en-us/answers/topics/46488/office-teams-windows-itpro.html) for technical questions about the configuration and administration of Microsoft Teams on Windows.
[Microsoft Teams Community forum] (https://answers.microsoft.com/en-us/msteams/forum?sort=LastReplyDate&dir=Desc&tab=All&status=all&mod=&modAge=&advFil=&postedAfter=&postedBefore=&threadType=All&isFilterExpanded=false&page=1)


Ordering of selected items

Given any thought of ordering the selected items? Something like hamburgers to reorder selected items
Thanks!

Do you mean to have the ability to drag items inside the list or do you mean to have some sort of sort function?

draggable items could be awesome!

Sounds interesting.
It means that we will need to store the order somewhere because unfortunately the selectedItems list is recreated every time.
I'll look into it.

Hey,
I think this can be done quite easily if the lib can also export defaults components used
Sortable is easy to setup, as just need to:

add <div className="sortable-list"> on top of the actual destination list.
and use HOC sortable on selectedItemRenderer.

Maybe this is the best way to do it ?

Hey @sabativi, thank you for looking into this.
Would you like to give this feature a go and try and issue a PR for it?

It seems #117 was not merged. So it's still not possible to wrap the default components into a sortable HOC, right?

Sorry @lukaspili I think he never got to finishing his PR.
I'd be happy to push a change if you'd like to write it.

Thanks for the quick response. At this point I'm just evaluating several options - including multi-select.


Huge performance difference between "AsWebServer" and "AsUtility"

The doc suggests that "AsUtility" is generally recommended under the production environment, but we are looking at around 10x performance difference, especially rendering PDFs.

How long does the pdf rendering take for you?
Do you compare the second rendering request times?
You can use the AsWebServer in your production environment,
The problem is that there is no logic in place of what to do if the jsreport.exe process dies.
In the case of Asutility such logic is in place.

How long does the pdf rendering take for you?
Do you compare the second rendering request times?
You can use the AsWebServer in your production environment,
The problem is that there is no logic in place of what to do if the jsreport.exe process dies.
In the case of Asutility such logic is in place.

Hi there,
For pdf rendering, I was looking at around 100ms when AsWebServer, but 1000ms-1500ms when AsUtility even when the process is already cooked up.
I have encapsulated a logic as it will kill the JsReport process (along with any processes running from that folder, e.g. chrome.exe) and then try to restart it if the rendering failed. Does that mean it is safe to use AsWebServer in production?
Also, are may I know if you are planning to fix the performance issue as mentioned in the docs, when will be the release roughly?
Thanks.

The performance with utility should be much better with 2.7.0 and it gets very close to the as web server option. I will share some numbers when we are three. Hopefully in one or two weeks.
If the automatic restarts work for you, then great, use it.

The performance with utility should be much better with 2.7.0 and it gets very close to the as web server option. I will share some numbers when we are three. Hopefully in one or two weeks.
If the automatic restarts work for you, then great, use it.

Thanks for the info, will be looking forward to the release.


Add some documentation for Directory.Build.rsp

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/msbuild/msbuild-response-files.md
:white_check_mark:Succeeded

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

FYI @andygerlicher @rainersigwald @cdmihai

#sign-off


"Extra" objectHighlighted event emitted during updateDate...?

I'm chasing a bit of logic to allow me to close the details panel in vizceral-example and yet keep the selected node highlighted without the detail panel popping back out on the next update/render cycle that propagates through.
I thought I had found the perfect existing signal to simple add on as a second argument to be emitted whenever the objectHighlighted event is emitted, which was force found in the highlightedObject function found in trafficGraph.js.
But there is one hick-up.  When the updating cycle is going and there is new traffic data call stack works out to be:
updateData (vizceral.js) -> updateGraph (vizceral.js) -> setState (trafficGraph.js) -> emitObjectUpdated (trafficGraph.js) -> which emits the objectHighlighted event
As these functions return the setState call completes and the next piece of updateGraph executes:
updateData (vizceral.js) -> updateGraph (vizceral.js) -> validateLayout (trafficGraph.js) -> updateView (trafficGraph.js) -> highlightedObject (trafficGraph.js) -> which emits the objectHighlighted event for the 2nd time since updateGraph was called.
It is that the objectHighlighted event is fired twice that is causing me trouble.  The highlightedObject function always So my questions is,

Why does the second "path" to emitting the objectHighlighed event in updateGraph (i.e. the validateLayout call), call the highlightedObject function since setState seems to have already taken care of that?

Hmmm, now that I've read my own question I think it isn't the best question.  This one is probably the better question.

Why does the setState of trafficGraph.js even need to emit the objectHighlighed event?  So far, I haven't found a situation where the state of a "graph" object is more than the traffic data, e.g. nodes, connections, render, layout, updated, etc. and possibly an entryNode key.

Or should I do something totally different.  Chasing this rabbit trail stems from trying to be able to single-click a node in the vizceral-example to open the detail panel, then close the detail panel but keeping the clicked node highlighted.  This works fine until the traffic data updates and ultimately causes the objectHighlighted events to fire in vizceral.js.
Thoughts?  Suggestions?

How we have it setup, if someone has deep linked into a view of vizceral, vizceral tells the container which object is highlighted. We have a very tight coupling of the surrounding chrome (detail panel) and the graph itself (as is seen in the vizceral-example), and we hook into the objectHighlighted event to update the details on the contextual panel as the details change.  I'm not saying this is a great pattern (or even a good one), it's just the reason it is the way it is.
A possible 'fix' without needing to modify vizceral: Store the highlighted node in a state outside of vizceral, then if you get the same objectHighlighted event for the same node, and the panel is closed, ignore the update event?
Or a fix would probably be to rethink the way it works in vizceral, maybe two different events? One event for when it's a new object that is highlighted, and one event for when the currently highlighted one gets updated?

Ummm, you lost me.  You are saying that the "...deep linked into a view... tells the container..." of your real running setup is a close match to the vizceral-example's connection to vizceral?  By "deep link" are you referring to how the callback executed by the emission of the objectHighlighted event propagates all the way down from vizceral-example?
So what happens on your real running setup if you do the following steps:
(1) you single-click (select) a service in the region view,
(2) close the details panel using the 'X', not by deselecting or selecting another service,
(3) with the selected service still highlighted your data updates.
For me (with my top-level interface being based on vizceral-example), the details panel will reappear.
As always, thank you for the help and all the work you've put into vizceral and managing its OSS repos.

Apologies for being unclear.
In our running vizceral app, you can navigate to vizceral/nodename?highlighted=foo to highlight an object and show the details panel for that object. If we close the details panel, it de-highlights the object. We have a tight coupling between the highlighted object and the details panel so we never had to think about separating the logic.
Does that make more sense?

Ok, that's the difference.  Using the public open-source vizceral with vizceral-example (and vizceral-react), when the details panel closes it doesn't un-highlight the node.  The detail panel closes and that is all.
For clarity/context, what I'm typically doing when this situation comes up is...
Something has happened and drawn my attention to a node at the region zoom level.  I click on it to get the details, but not to zoom in, i.e., vizceral/regionname?highlighted=foo via single-click, not direct URL setting (not that it matters as it doesn't effect the behavior using the open-source code).  I take a look through the details and then close the details panel.   Here is where what I want and what happens diverge.
What I want is to just watch that node and its connected peers to see what happens as time keeps on ticking along.  If it has been a while and/or I've forgotten, I want to click on the highlighted node again so I can see the details panel again.  What actually happens is that on every update of the traffic data the details panel reappears.
I want to see the region view as clearly as I can by not giving up any space for the details panel while still keeping the highlighting.
So just to for my own peace of mind.  When you close the details panel at the vizceral/regionname?highlighted=foo zoom level, the selected node is un-highlighed, right?  There is not a means or wasn't supposed to be a means for highlighting a node and its connected peers via click, only hover?

Correct, that is the current behavior. I could see why this would not be desired and would definitely welcome a PR to change the behavior.


Capability to select from multiple graphs hosted in one Gremlin server

Is it feasile to support multiple graphs in one graph database / gremlin server? Thanks!

Hello Zhibo,
if you are using Cypher Client, you can use alias for Gremlin Client:
Cluster cluster = Cluster.open(configuration);
Client gremlinClient = cluster.connect().alias("g2");
CypherGremlinClient cypherGremlinClient = CypherGremlinClient.plugin(gremlinClient);

In Console Cypher plugin :remote config alias is not supported yet.


Debian Wheezy: sharp breaks if the canvas module is loaded first

I ran into this very puzzling issue on Debian Wheezy, which I haven't been able to repro on my workstation, which runs a much newer Ubuntu.
It goes something like this:
$ nvm use 4
Now using node v4.3.1 (npm v2.14.12)
$ npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>[...]
$ node -e "require('canvas');require('sharp');"

module.js:356
  Module._extensions[extension](this, filename);
                               ^
Error: /path/to/node_modules/sharp/build/Release/../../lib/libgio-2.0.so.0: undefined symbol: g_option_group_unref
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/path/to/node_modules/sharp/index.js:12:13)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at [eval]:1:19
    at Object.<anonymous> ([eval]-wrapper:6:22)
    at Module._compile (module.js:456:26)
    at evalScript (node.js:565:25)
    at startup (node.js:80:7)

Interestingly it does not blow up if the two requires are reversed!
It's reproducible with both node.js 0.10.38 and 4.3.1. Haven't tried any others.
Any ideas?

I suspect canvas uses a system-wide install of libcairo2-dev, which depends on glib v2.32.4. By introducing this shared library dependency at run-time first, you supersede sharp's pre-packaged glib v2.47.5.
In this case, it's probably best to follow suit and use a system-wide install of libvips.

I got this issue too on Ubuntu 14.04.
Error: /.../node_modules/sharp/build/Release/../../lib/libgio-2.0.so.0: undefined symbol: g_option_group_unref

Sharp ran fine before I installed the canvas.

Quick thought: are either of you able to try using the fully-versioned path to the shared library (rather than the partial-versioned symlink) in the libraries section of binding.gyp, something like:
- '<(module_root_dir)/lib/libglib-2.0.so',
+ '<(module_root_dir)/lib/libglib-2.0.so.0.4705.0',

- '<(module_root_dir)/lib/libgio-2.0.so',
+ '<(module_root_dir)/lib/libgio-2.0.so.0.4705.0',

Failing this, the only option I can think of is to experiment with the --default-symver linker flag when building the pre-compiled shared libraries to insert the version number into the symbol table.

Another even quicker thought:
LD_PRELOAD=/path/to/node_modules/sharp/lib/libglib-2.0.so node script.js

...should make sharp's bundled glib take precedence regardless of module import order.

Sorry for not reporting back earlier. I just switched around the requires, then moved on :)
$ LD_PRELOAD=node_modules/sharp/lib/libglib-2.0.so node -e "require('canvas');require('sharp')"

(sharp:61468): GLib-GObject-CRITICAL **: /tmp/buildd/glib2.0-2.33.12+really2.32.4/./gobject/gtype.c:2722: You forgot to call g_type_init()

(sharp:61468): GLib-CRITICAL **: g_once_init_leave: assertion 'result != 0' failed

(sharp:61468): GLib-GObject-CRITICAL **: /tmp/buildd/glib2.0-2.33.12+really2.32.4/./gobject/gtype.c:2722: You forgot to call g_type_init()

(sharp:61468): GLib-CRITICAL **: g_once_init_leave: assertion 'result != 0' failed

... and then it hangs.
If I make the suggested patch to binding.gyp inside node_modules/sharp, then run npm install inside node_modules/sharp, I get the original error:
$ node -e "require('canvas');require('sharp')"

module.js:356
  Module._extensions[extension](this, filename);
                               ^
Error: /home/andreas/playground/node_modules/sharp/build/Release/../../lib/libgio-2.0.so.0: undefined symbol: g_option_group_unref
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at Object.<anonymous> (/home/andreas/playground/node_modules/sharp/index.js:12:13)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)
    at Module.require (module.js:364:17)
    at require (module.js:380:17)
    at [eval]:1:19
    at Object.<anonymous> ([eval]-wrapper:6:22)
    at Module._compile (module.js:456:26)
    at evalScript (node.js:565:25)
    at startup (node.js:80:7)

@papandreou Thanks for checking. I guess the dependencies of glib would also need to be included in the LD_PRELOAD variable, a lot of faff compared with moving a couple of require statements :)
@nsmag Were you also able to "fix" this by altering the require order?

@lovell Sorry for not reporting back.
I decided to do some image processing with raw image data from sharp().raw().
So no need for canvas and I don't want to encounter unexpected surprises in production.
I'll experiment with canvas + sharp again later.
Sharp is lightning fast!
Thanks.

Glad you both found workarounds. I'll close this for now but please feel free to comment and/or re-open with any more quick fix solutions :)

I was getting the same libgio-2.0.so.0: undefined symbol: g_option_group_unref error and altering require order (i.e. sharp first, then node-canvas) fixed it.

I did experiment with the -Wl,--default-symver linker flag to force the explicit use of shared library version numbering but couldn't get libtiff to play nicely (and even if I had there may have been further walls I wasn't able to hit) :(

Any progress to solve this incompatible problem? I got the same error when I uses both sharp and node-canvas

When I require sharp before canvas, it seems to solve the problem. However, node-canvas will complain Fontconfig error: Cannot load default config file when drawing text.

@jingsam If you're relying on a system-wide installation of cairo for node-canvas, then the absolute safest option here is to also rely on a system-wide install of libvips for sharp.


Prevent further cases of perma id clashes

Wrapping RevisionComponent#save in an advisory lock, does not
suffice for cases where the revision component is created through an
association, i.e
revision.file_usages.create!(...)

In this case CollectionAssociation#_create_record [1] already starts
the transaction before the lock is required. Since the transaction is
then committed only after the advisory lock is already released, the
following race condition can occur:
Thread 1:                       Thread 2:
|                               |
BEGIN                           |
GET LOCK                        |
SELECT MAX perma_id             |
INSERT revision component       |
RELEASE LOCK                    |
|                               BEGIN
|                               GET LOCK
|                               SELECT MAX perma_id
COMMIT                          |
|                               INSERT revision component
|                               RELEASE LOCK
|                               COMMIT

Since the first transaction is not yet finished, the second thread
does not yet see the newly inserted perma id and calculates the same
maximum.
To ensure that the lock is acquired before the transaction starts, we
add a new class method create_with_lock! which then delegates to
create!.
For existing revision components (and models that behave like revision
components like Page) this race condition has never been a problem
since concurrent inserts are not that likely within a revision (and we
do not care about non-unique perma ids accross entries). We only
noticed the problem for file usages now since those are created by a
batch of requests when multiple files are uploaded at the same time.
So for now we use the new create_with_lock! method only in
DraftEntry.create_file!.
[1] https://github.com/rails/rails/blob/b9ca94caea2ca6a6cc09abaffaad67b447134079/activerecord/lib/active_record/associations/collection_association.rb#L359
REDMINE-16809

Coverage remained the same at ?% when pulling 3c573007acbc82cebe0d9fa2cb8019bd8da0df0a on tf:perma-id-lock-2 into 35d2bf2a0d04b0cd844d560b219b22781e094a1c on codevise:master.


hpc/slurm: RH9 packages

Describe your changes
Support for RedHat 9.
Because pkg name changed from python3-mysql to python3-mysqlclient...
Issue ticket number and link if any
Checklist before requesting a review

[x] Document introduced changes / variables in role README.md if any
[x] Increment role version in vars/main.yml (a.b.c: +1 to b for a feature added, +1 to c for a bug fix)
[x] Update changelog inside role README.md
[x] If not present, please also add your name in the related collection authors list in galaxy.yml

Dear @pigay
Thank you for this fix !
Yes, I have started fixing the slurm role in this PR https://github.com/bluebanquise/bluebanquise/pull/792/files#diff-80c058e62a00aec5d38c628bff46810b7f05826c73855e096adc9ebec254028e, but I missed time recently to finish it.
Ok for me, merging 😊


Solix Smart Meter

System Health details
System Information

version
core-2024.8.3

installation_type
Home Assistant OS

dev
false

hassio
true

docker
true

user
root

virtualenv
false

python_version
3.12.4

os_name
Linux

os_version
6.6.46-haos

arch
aarch64

timezone
Europe/Berlin

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

HACS Data
ok

GitHub API Calls Remaining
5000

Installed Version
2.0.0

Stage
running

Available Repositories
1399

Downloaded Repositories
9

Home Assistant Cloud

logged_in
true

subscription_expiration
4. November 2024 um 01:00

relayer_connected
true

relayer_region
eu-central-1

remote_enabled
true

remote_connected
true

alexa_enabled
false

google_enabled
true

remote_server
eu-central-1-2.ui.nabu.casa

certificate_status
ready

instance_id
daf6eb36bef7465f8c634aa2297f31b0

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Home Assistant Supervisor

host_os
Home Assistant OS 13.1

update_channel
stable

supervisor_version
supervisor-2024.08.0

agent_version
1.6.0

docker_version
26.1.4

disk_total
113.9 GB

disk_used
44.9 GB

healthy
true

supported
true

host_connectivity
true

supervisor_connectivity
true

ntp_synchronized
true

virtualization

board
odroid-n2

supervisor_api
ok

version_api
ok

installed_addons
MariaDB (2.7.1), Advanced SSH & Web Terminal (18.0.0), Studio Code Server (5.15.0), UniFi Network Application (3.2.0), ESPHome (2024.8.3), RaspberryMatic CCU (<IP_ADDRESS>40826), phpMyAdmin (0.9.1), Home Assistant Google Drive Backup (0.112.1), EMHASS (0.10.6)

Dashboards

dashboards
1

resources
3

views
11

mode
storage

Recorder

oldest_recorder_run
8. August 2024 um 12:04

current_recorder_run
3. September 2024 um 10:32

estimated_db_size
3706.36 MiB

database_engine
mysql

database_version
10.11.6

Integration version
2.1.1
Checklist

[X] I have enabled debug logging for my installation.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of any previous issues..

Describe the issue
My Solix Smart Meter does send a lot of "zero" values between real data. I know this is because of the way the api works, but can you suppress those data sets?
Reproduction steps

Add Solix Smart Meter to your system
View the data

Debug logs
What do you need here excaclty?

Diagnostics dump
I am not able to get it. It is not shown:

Here is a graph that shows my problem:

@Bascht74 I cannot suppress individual 0W data because there is no mechanism to detect whether these values are true or false.
The cloud was originally marking invalid data with a separate field, but that was for all power values in the response.
If only your smart meter has false 0W values, you have an issue in your setup.
You can try to change the configuration entry options and skip invalid data responses as described here, but I don't think this will solve your issue since the cloud should no longer respond invalid data. This cloud change was implemented in June in order to fix the data update issues when using shared accounts which do not trigger frequent data updates either.


Update do Microk8s.enable Kubeflow docs

Providing a more comprehensive documentation to deploying Kubeflow on Microk8s and accessing the dashboard.

Thanks @RFMVasconcelos for submitting this PR and @8bitmp3 for the review.
Preview: https://deploy-preview-1999--competent-brattain-de2d6d.netlify.app/docs/started/workstation/getting-started-multipass/
nit:  the code block is not aligned properly. You can add 4 spaces before the code block.

/ok-to-test

Thanks @janeman98

nit: the code block is not aligned properly. You can add 4 spaces before the code block.

I think the Markdown stuff is rendered differently in Hugo docsy theme that Kubeflow uses. Therefore, adding 4 spaces/tabs to move code block to the right would not be useful here. See https://www.kubeflow.org/docs/started/workstation/getting-started-multipass/:

However, I do agree that usually it'd look neater if you use code blocks that follow numbered sequences, etc.

Thanks @janeman98

nit: the code block is not aligned properly. You can add 4 spaces before the code block.

Just double checked and spaces before code blocks are indeed used on Kubeflow's site undernearth numbered sequences, bulletpoints, etc. For example:

@janeman98 @8bitmp3, indentation fixed :)

/lgtm
/approve

Hi @janeman98 it seems that these changes are not live, is that expected?

@RFMVasconcelos
It seems I am recently removed as approvers in this repo and I guess this may be the reason this PR is not actually "merged"?
@jlewi
Can you please help to approve/merge this PR?


Login/Signup patches

This fixes the issues we discovered in the last meeting and issues pointed out by John with the background with the login and sign up UI. It also includes refactored code to add a bit more structure and make it easier to work with in the future should we need to make any changes. It should work on all screen sizes and scale accordingly.

Ojo reviewed gave the green light so I'm merging