TXT preview in repository browser not working for some txt files

I have been in communication with the Bonobo Git community at Google forums, more precisely: https://groups.google.com/forum/#!topic/bonobo-git-server/lXv9Jy8Ix-E
After having communicated forth and backup with Jakub he told me to create an issue so here it is :-)
--- SHORT DESCRIPTION, more in the forum post ---
Text files with ANSI encoding, the std. encoding for text files saved on Windows, cannot be shown via the Bonobo Git Server web interface. More specifically it throws the error: "Preview of this file is not supported."
Jakub have been able to reproduce this on the Bonobo Git demo server, with my guidance.
He told me that lkho, stanshillis and ColinNiu is the programmers to go to on this issue.
Looking forward to hear your opinion.
Thank you and have a great day.

Just a polite status enquirer. Have anybody had the time to look at this? :-) have a great day.

Confirm the problem. The bug was created by patch #273

@padremortius hmmm ok :-) - so the fix is hopefully around the corner. Looking forward to it and thank you very much for your reply.
Have a great day to you all.

Hi guys and girls :-) - is it doable :-D ? Looking forward to being able to view txt files again.

@larssb  I have made a pull request that should make it possible to view text files encoded with ansi/windows-1252 in #312.
Can you try it out?

Hi @larshg - sure I would love to. In order to test it, do you want me to just download the changed files in the repository manually or will the Bonobo Git Srv version be bumped so that it can be downloaded from the official channels? Asking because I'm not a seasoned vet when it comes to this :-)
Thank you.

@larssb as of now @jakubgarfield has already merged my pull-request to the master, so you can checkout the master and build it yourself and "publish" it to a local folder. This can then be copied as if you had downloaded from the official channels.
I don't know when @jakubgarfield has plan for a new release?

I'm not planning the release soon, waiting for more features to pack in. Might do a spontaneous one if I got a spare hour.

Hi @larshg - trying to update the server. I'm lacking knowledge on how to build this. Could you be so kind to give me a hint :-)? Also checking it out isn't that pretty much similar to downloading the zip from the main Bonobo GIT Server GitHub project page?
Thank you very much.

Hey there :-) - how do I do that build? Any help would be appreciated.

Hi @larssb
I have been quite busy lately - have you tried cloned the master and build it locally?
Else I'll try to help you, if you need it :)

Hi @larshg,
I tried to built. I assume I should use the build.cmd file. But it does not work and throws the error:
' error MSB4019: The imported project "C:\Program Files (x86)\MSBuild\Microsoft\VisualStudio\v11.0\WebApplications\Microsoft.WebApplication.targets" was not found. Confirm that the path in the  declaration is correct, and that the file exists on disk. '
It does not seem that the build.cmd file is a general way to do this or am I misunderstanding something. I have looked at the FAQ and the doc. both there and on Github and can't see a description on how to build this.
Any help is therefore appreciated.
Thank you :-D

I get the same error when running the build.cmd file. I'm not sure where it gets the v11.0 since I'm using VS2015 and that is v14.0.
Which VS are you using?
Can you just open the the solution and build it in VS? ie Bonobo.Git.server.sln...

Hi @larshg - Thank you for your response. I acknowledge I'm very noobie here. However, after having opened the solution in VS I only get an updated version of the Bonobo.Get.Server.dll - but I still have one big mess of various .cs files and so forth, that in no way resemblances a production release of Bonobo GIT Server. So basically I'm very unsure if this is correct and how to get to a state where what I have is a production build of Bonobo GIT Server. I'm sorry but I'm quite new to this.....just let me know if you need more info.
Also just to try it out I replaced the Bonobo.Git.server.dll on the Bonobo GIT server with the one build by VS. The issue still persists (the issue that this case is about in the first place). So I can't be doing things correct here :-(
Thank you very much.

Indeed most the .cs files will, when you compile the project end up in being a dll.
If you rightclick on the project, you can select publish. Then use the Release that is properly preselected and it will create a install directory that should be similiar to the official releases :)

Hmm that was waaaaaay better :-) - Thank you. I'm running the Windows Authentication setup. So both a Bonobo.Git.Server & Bonobo.Git.Server.Interface. I first started out updating the Bonobo.Git.Server but the TXT preview error is still there for ANSI files. I then thought I might have to update the Bonobo.Git.Server.Interface as well. After I did that the app pool crashes and the eventlog refers to kernelbase.dll and exception 0xe0434352 - but I don't think it has to do with kernelbase.dll. I think it is just the messenger.
This is not going as hoped :-) - any good ideas? Maybe to wait for an official release......I would just really love to have this thing fixed.
Thank you for your help and your patience @larshg @jakubgarfield

Hmm that seems weird.
Is there any error before, that can be more helpful - because the the provided is a somewhat generic error for .net managed application chrashing I think.
I'll try build a fresh and upload to a test location.
Alternatively you try edit the web.config under:
<system.web>
    <globalization enableClientBasedCulture="true" uiCulture="auto" />
    <customErrors mode="On" defaultRedirect="Home/Error" />
    <httpRuntime maxRequestLength="102400" />
    <!--<authentication mode="Windows" />-->
...

to
<system.web>
    <globalization enableClientBasedCulture="true" uiCulture="auto" />
    <customErrors mode="Off" defaultRedirect="Home/Error" />
    <httpRuntime maxRequestLength="102400" />
    <!--<authentication mode="Windows" />-->
...

And see if you get an error message before crashing!

@larssb I located the error.
In the source folder of bonobo, you can to Bonobo.Git.Server\bin and copy the either x86 or x64 folder to the bin folder on your webserver. This is where SQLite.Interop.dll is residing and it gives me the same error and crashing the app pool as you describes. It is properly not being copied in the current settings of the publish command. @jakubgarfield how can this be added? I'll try take a look but at first glance at the publishing settings, I couldn't find any list etc. should it be added to folder in the project?
Hope it helps the last way - at least it worked here again :)

I thought the SQLite.Interop.dll is added automatically to the build output when you compile the project. The nuget package should take care of it. Under which events does this not happen? (I don't have problems with normal build and release with Visual Studio)

They do get copied to the bin folder when building.
But afterwards when I publish(ie. copy all required files to bonobo_install) the project the files doesn't get copied, atleast using VS2015.

@larshg yes it works by copying the x86 & x64 folders to .\bin on the server. @jakubgarfield I have the same experience as larshg, the files are not in the 'bonobo_install' folder generated when right clicking in vs2015 and selecting publish. If this could be fixed it would help in the future :-) - thank you.
Also, most importantly, the ANSI files can now be pre-viewed in the browser gui again. Wonderful!
Thank you all and everyone :-D (I can't have missed anyone now)

If I use publish from Visual Studio (right click on the project in solution explorer) it puts the files under /bonobo_install/x86 and /bonobo_install/x64.

Closing this one, as the original issue was solved and the other issue with SQLlite.interop.dll should be solved in master.


iRep for origin of replication

Can iRep be used for predicting the ori and ter of a draft genome sequence? Are there any temporary files that could be used for that information?

iRep estimates the coverage values at the origin and terminus of replication, but does not identify where they are in the genome. Because genome coverage values can be somewhat noisy, there typically are multiple genomic regions with coverage values similar to those at the origin and terminus of replication, making it difficult to identify these regions.

On Feb 2, 2018, at 4:28 AM, sarah872<EMAIL_ADDRESS>wrote:
Can iRep be used for predicting the ori and ter of a draft genome sequence? Are there any temporary files that could be used for that information?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


ETagMiddleware

You added the ETag Middleware, but when exception occurs the page of the exceptions don't appear for me.
How can I get around this? And the exceptions page may appear again.
At the moment I did something of the type.
if (!env.IsDevelopment())
    app.UseETagger();

No longer using ETag.


Add staging config

Added staging config option. Also added logic to maintain two versions of the accountkey.pem file as the account can't be shared across environments. I only added the prepend to the staging version so existing implementations won't be affected, but a new file will be created if staging is used. Happy to make changes as requested. Fixes #13.

Please make a change also in the example config json file to include the new parameter

I couldn't find an example .json file with these settings in, but I did update the readme example to reflect the new value.


Update scala-collection-compat to 2.3.2

Updates org.scala-lang.modules:scala-collection-compat from 2.2.0 to 2.3.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang.modules", artifactId = "scala-collection-compat" } ]

labels: library-update, semver-minor

Codecov Report

Merging #134 (0f29b7f) into master (d5bdca1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #134   +/-   ##
=======================================
  Coverage   28.62%   28.62%
=======================================
  Files          41       41
  Lines        1729     1729
  Branches       33       33
=======================================
  Hits          495      495
  Misses       1234     1234

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 d5bdca1...0f29b7f. Read the comment docs.

Superseded by #141.

Superseded by #141.


WIP: DNN Initial implementation using TensorFlow

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
close #574
What are the significant changes in functionality due to this change request?
Add new capability, deep neuron network.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

tensorflow require minimum version: scipy=0.18.1 scikit-learn=0.18.2

In addition, tensorflow will only work with python 3.5/3.6 on Windows.

The other option is to use the Microsoft Cognitive Toolkit (CNTK) as the backend engine for Keras. In this case, Keras can work in both windows and linux with python 2.7. However, we need to use pip to install all these packages.

a new PR will be opened


Upgraded from 0.1.9 to 0.2.1 and URL no longer changes when switching routes

I upgraded from 0.1.9 to 0.2.1 and the URL no longer changes when switching routes even though the route does change.
I downgraded and everything worked correctly so it seems to be caused by some change in between those versions. The page history just doesn't seem to be getting updated. I can open a new tab, click around to different pages, and then click the browser "Back" button and I get taken back to the new tab page (none of the app routes were added to history).
Did something change that requires modification to how we need to implement solid-app-router?
I'm getting this error when changing paths as well:
Uncaught TypeError: start(...) is undefined
    navigateFromRoute routing.js:230
    untrack dev.js:399
    navigateFromRoute routing.js:194
    navigatorFactory routing.js:248
    handleClick components.jsx:87
    eventHandler dev.js:333
    delegateEvents dev.js:119
    <anonymous> components.jsx:240
routing.js:230:26
    navigateFromRoute routing.js:230
    untrack dev.js:399
    navigateFromRoute routing.js:194
    navigatorFactory routing.js:248
    handleClick components.jsx:87
    eventHandler dev.js:333
    (Async: EventListener.handleEvent)
    delegateEvents dev.js:119
    <anonymous> components.jsx:240
    InnerModuleEvaluation self-hosted:2388
    InnerModuleEvaluation self-hosted:2388
    InnerModuleEvaluation self-hosted:2388
    evaluation self-hosted:2349

Thanks!

Try updating Solid to 1.3.0 or later. I am pretty sure the mismatch and the reason we went to 0.2.x was because we need to make changes for Solid 1.3 or later.

That worked, thanks!


Use http://checkip.amazonaws.com to determine public IP

Done. Wanted to use this "official" AWS service instead of a 3rd party.


ENH: stats: add method parameter to differential_entropy + more accurate estimators for small samples

Reference issue
gh-13631
What does this implement/fix?
This adds a method parameter to scipy.stats.differential_entropy with three new estimation methods, all of which tend to be more accurate than the Vasicek estimator for small samples.
Question: should Vasicek remain the default since it's the best known and just as accurate for large samples, or is small sample accuracy more important?
Additional information
See here for a reference that compares the methods. It also proposes new methods that may be marginally better for small samples, but I would leave those to a separate PR.
I couldn't find other software that implements these methods, but they're all reasonably well-cited (corresponding papers have 100-200 citations), and they do seem to be consistently more accurate than Vasicek for small samples, at least for the distributions tested.
Here's a plot of the root mean squared error (between the estimate and exact entropy) over 10,000 samples from some common distributions as a function of the number of observations in a sample n. Window size is round(sqrt(n)).

Thanks for the review @tupui. I think I responded to all the comments except for some simple suggestions that we can commit along with the rest once we reach consensus.

Re: CI errors, I will fix the refguide_asv_check by specifying method='vasicek'. I should probably provide more examples anyway.
I'll fix mypy's astute observation scipy/stats/_entropy.py:243: error: "None" not callable by replacing None with _vasicek_estimator. I suppose then I can get rid of the else in the auto method selection.

@tupui The original example said:
    >>> # We use high numbered seeds in order to have good entropy in the RNG
    >>> rng = np.random.default_rng(seed=148632)

Is it really preferred to use large seeds?

Yes apparently it is better for the generator. But for the examples we should actually remove the seeding at all as we did in #13778. For the result we should then put # random so that the doctests do not fail. This is to not advertise any seed pinning.

To summarize, IMO this just needs:

fix of the type hint
removing of seed in example (we can also do it in the follow up)
remove or change the equation

removing of seed in example (we can also do it in the follow up)

gh-13778 was mostly about removing RandomState, but why does it also remove the seed? Why is it better to have results that are not repeatable? Sometimes docstests are not tested in the test suite separately (perhaps they should be, but that's not always the case), and they have helped me find bugs in the past.

removing of seed in example (we can also do it in the follow up)

gh-13778 was mostly about removing RandomState from default_rng, but why does it also remove the seed? Why is it better to have results that are not repeatable? Sometimes docstests are not tested in the test suite separately (perhaps they should be, but that's not always the case), and they have helped me find bugs in the past.

In the end we also discussed about removing the seed in the examples because users tend to copy paste this code and not touch it when it works for them. So this prevents bad practices and common seeds again. It's not a random number generator anymore if you always use the same seed ;).
I also found bugs in stats.qmc with doctests, but in the end it was because I was missing a proper test. Our test suite should be more covering than the docstrings. They are the real tests. Still we should continue to run the doctests just to verify that our examples are coherent with the actual implementation.

@tupui I think I addressed the comments, except I'll let you remove the seed in gh-13863. Could you check the doc build locally? I added another example and I want to make sure it looks alright. I'll try to check, too, but I haven't successfully built docs locally in over a year.

@tupui I think I addressed the comments, except I'll let you remove the seed in gh-13863. Could you check the doc build locally? I added another example and I want to make sure it looks alright. I'll try to check, too, but I haven't successfully built docs locally in over a year.

Sorry @mdhaber for the delay, I oversaw this! I could test your branch locally, the doctests are passing and the doc is building just fine. (You might want to merge master again to have the doc fix).
Just one thing, there is not link to references in the description of the methods.

Thanks @tupui I guess to get the numbers right in the docstring I'll just have to wait to see the values CI reports and change them to that. I have so far been unsuccessful at predicting the values because doctests for stats fail on my machine (memory error). I also tried the seed you told me, but I don't think that got me the right answer either. (I will try that again though. It's possible I did something out of order.)

Looks like the Linux Tests / Nightly Cpython (3.9) failure is unrelated and happening in other PRs (e.g. gh-13906). I'm happy with this if you are!

@tupui Looks like the Linux Tests / Nightly Cpython (3.9) failure is unrelated and happening in other PRs (e.g. gh-13906). I'm happy with this if you are! ~Update: oops, some little things noted below.~

LGTM too! The doctests where passing locally, so you did it correctly with the seed.

The doctests where passing locally,

I already fixed it I guess.

I will just wait for refguide_asv_check and merge as the rest was green before.

Thanks @mdhaber!


Support RequestScope with Helidon 4.0.0 (Nima)

Ah yes, merging this and it will only work with Java 21 as Helidon 4.0 is Java 21+ only.


Add: tailwind CSS

We have to add tailwind CSS to our crypto site repo.
Links:
Tailwind CSS docs

Hey, can i take this task?

Hey, can i take this task?

Assigned to you. Please let me know if you need any help.
Thank you.


node-exporter: Fix download URL and default to v1.5.0

Signed-off-by: Stevo Slavić<EMAIL_ADDRESS>Fixes https://github.com/prometheus-community/ansible/issues/15
Also bumps default node-exporter version to 1.5.0 for improved security (high CVE fixed) and kernel bug workaround (see changelog).

I think what we want to do is port over our automated update tools from cloudalchemy.

There is/was something like that there? It didn't seem to have been working; node-exporter 1.1.2 was released almost two years ago, there were multiple releases since then https://github.com/prometheus/node_exporter/releases

Yes, we had tools for this, but something broke and we never got around to fixing it.

Would you prefer to leave out version bump out of this PR?

Thanks @gardar for heads up! Closing in favour of https://github.com/prometheus-community/ansible/pull/17


Implement a health-check actor for c8y-device-management plugin

Implement a health check actor and integrate it with the c8y-device-management plugin so that it can respond to health check requests. Since the health check is done for the entire daemon and not for individual actors, only the health check actor must respond to health check requests and not the individual actors.
The actor must listen on tedge/health-check and tedge/health-check/c8y-device-management topics for health check requests and respond to the tedge/health/c8y-device-management topic with its health status.

The actor must listen on tedge/health-check and tedge/health-check/c8y-device-management topics for health check requests and respond to the tedge/health/c8y-device-management topic with its health status.

The name of the daemon used by this actor must be configurable (and not hardcoded as c8y-device-management.

#1815

We hold back testing/verification of health check actor for a while till its used by the mappers.

Test covered with
/tests/RobotFramework/tests/cumulocity/configuration/configuration_operation.robot


migrate ghc-project-types into the haskell-ide-engine repo

As a sub-package.

cc @DanielG

And it has a dependency on cabal-helper, it may be simplest to wait for that to appear on hackage.
I will copy it from the current submodule (https://github.com/fendor/ghc-mod/commit/7757a149a6ddb243679840ebff8949ff305c3424), and add as contributors all who show up in git blame for it, and @DanielG .

And looking closely into the hie code, this module is only used for cabalModuleGraphs. Which is only used by HaRe, and due for a complete rework in the near future.
So perhaps the best path is to remove this completely, including the cabalModuleGraphs function, and put it back when we have a solution that makes sense.
FWIW, the original motivation for the cabal-helper wrapper types is to provide a stable API to programming against multiple cabal-versions, each of which changed the underlying data structures quite a bit.
I am not sure how much of a problem that is now, as we are supporting a narrower range of cabal, and the data structure churn is not as extreme. ?


housekeeping: build process

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@74bdca8). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   81.72%
=======================================
  Files           ?        5
  Lines           ?      186
  Branches        ?        0
=======================================
  Hits            ?      152
  Misses          ?       34
  Partials        ?        0

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 74bdca8...44f5d76. Read the comment docs.


Added unit_id in daily apportioned payload

Ticket
https://github.com/US-EPA-CAMD/easey-ui/issues/6292
Changes
Added unit_id in daily apportioned payload (Compiled in dev)

@usmanahmederg, have you compiled and tested the updates in the Dev database?  I am waiting to merge until you have done that.

@usmanahmederg, have you compiled and tested the updates in the Dev database? I am waiting to merge until you have done that.

@djw4erg Yes, I have compiled and tested it on dev.


[Bug] Blood deduction is not true.

Describe the bug
Blood deduction is not true.
To Reproduce
Steps to reproduce the behavior:
Example steps (replace with your steps)

config.GUI.old-health-bar: 'true'
reload plugin
when your health > 20 (Maybe.

SkillAPI.zip


Support for languages other than english

Looks like it should be easy enough to add a command-line argument that you pass through to request.recognitionLanguages. I would open a PR, but I've never written any Swift...

Yes, it would be easy. :) It would also be nice to support customWords, which allows expanding the vocabulary used by the OCR system (to add weird technical terms or whatever).


[CP] rename generated asset manifest file back to AssetManifest.bin (from AssetManifest.smcbin)

Issue Link
https://github.com/flutter/flutter/issues/128230
Commit Hash
cfe4fed
Target
stable
PR Link
https://github.com/flutter/flutter/pull/128621
Changelog Description
Renames the asset manifest (an internal file generated by the tool and used by the framework to resolve assets) from AssetManifest.smcbin back to AssetManifest.bin. This plays more nicely with web servers.
Impacted Users
all web developers, at least those serving their site using Microsoft IIS
Impact Description
Flutter web apps served on Microsoft IIS won't be able to load and display image assets unless the server is explicitly configured to serve AssetManifest.smcbin.
Workaround
Yes, see https://github.com/flutter/flutter/issues/128230#issuecomment-1580277570 for an example.
Risk
medium
Test Coverage
yes
Validation Steps

Build a flutter web app that displays an image.
Serve the app through IIS (should be available on Windows by default).
Access the site through a web browser and make sure the image the asset is displayed properly.

CC @jonahwilliams @eliasyishak

LGTM


Support Guitar Pro files

Maybe there's an open source reader for these files?  The official app is expensive https://www.guitar-pro.com/en/index.php

Relevant open source library:
https://github.com/phiresky/tuxguitar/tree/master/TuxGuitar-android

^^ for app: https://play.google.com/store/apps/details?id=org.herac.tuxguitar.android.activity.admob

Related: #50


New parameter for Update-DbaSqlServiceAccount + few fixes/features

Type of Change

[x] Bug fix (non-breaking change, fixes ugly PS 3.0 that does not know about [securestring]::New() )
[x] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[x] 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
Improving usability and bug-fixes
Approach
Adding support for providing username/password without the credential object
Adding Read-Host to request for password when not specified
Fixing cmdlet to work under PS 3.0 by removing calls to [securestring]::New()
Commands to test

Screenshots

Learning

Thanks you Kirill! 🍰 🍣

Np!


login fails since two days ago

Version of the custom_component

0.1.3
Installation method (hacs / manual)
has install on pi-4 installed from image file onto SD card.
Installation method of hass (venv, docker, hassio,...)
Configuration

Add your configuration here.

Describe the bug
Since two days ago, I'm getting an error when HA is attempting to login to the VW connect.. It was working just fine..
Debug log
Logger: homeassistant.config_entries
Source: custom_components/volkswagen_we_connect_id/init.py:47
Integration: Volkswagen We Connect ID
First occurred: 17:35:33 (2 occurrences)
Last logged: 17:35:33
Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry
await hass.async_add_executor_job(_we_connect.login)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
self.__session.login()
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login
authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize')
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl
redirect = tryLoginResponse.headers['Location']
File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in getitem
return self._store[key.lower()][1]
KeyError: 'location'

Add your logs here.

having the same error:
Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup     result = await component.async_setup_entry(hass, self)   File "/config/custom_components/volkswagen_we_connect_id/__init__.py", line 47, in async_setup_entry     await hass.async_add_executor_job(_we_connect.login)   File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login     self.__session.login()   File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login     authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize')   File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl     redirect = tryLoginResponse.headers['Location']   File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in __getitem__     return self._store[key.lower()][1] KeyError: 'location'

Same problem here, since yeaterday. Rebooting/reloading has no effect.

I am getting the following:
Unexpected exception
Traceback (most recent call last):
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 64, in async_step_user
info = await validate_input(self.hass, user_input)
File "/config/custom_components/volkswagen_we_connect_id/config_flow.py", line 39, in validate_input
await hass.async_add_executor_job(we_connect.login)
File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
self.__session.login()
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login
authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize')
File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl
redirect = tryLoginResponse.headers['Location']
File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in getitem
return self._store[key.lower()][1]
KeyError: 'location'

Same problem as in issue #143

also issue here, seems a bit like the same
2023-02-28 07:18:27.298 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ID4 for volkswagen_we_connect_id
File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry
2023-02-28 21:26:02.944 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry ID4 for volkswagen_we_connect_id
File "/config/custom_components/volkswagen_we_connect_id/init.py", line 47, in async_setup_entry

I'm getting the same error with a really similar error log entry

same

Same problem as in issue #143

Guess not the same issue :(
#143 was "You have to login at myvolkswagen.de and accept the terms and conditions"
I did that - and it worked for a couple of days,,,
Now i get this error- which i guess is different from #143 but same as Guy-BW is starting this post with?
I can log into wv web-interface and the ID-app with my credentials, so they work just fine.
Have tried to add the integration in my test-env, but it is not getting past the username/password/submit-dialogue,,   :(
Many thanks to Mitch & contributors for the integration,, was getting used to having soc in HA :)

hopfully they will bring it back in order :)

This error originated from a custom integration.

Logger: homeassistant.config_entries
Source: custom_components/volkswagen_we_connect_id/__init__.py:47
Integration: Volkswagen We Connect ID
First occurred: February 28, 2023 at 10:47:46 (2 occurrences)
Last logged: February 28, 2023 at 11:50:08

Error setting up entry Volkswagen We Connect ID for volkswagen_we_connect_id
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 382, in async_setup
    result = await component.async_setup_entry(hass, self)
  File "/config/custom_components/volkswagen_we_connect_id/__init__.py", line 47, in async_setup_entry
    await hass.async_add_executor_job(_we_connect.login)
  File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/site-packages/weconnect/weconnect.py", line 182, in login
    self.__session.login()
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 75, in login
    authorizationUrl = self.authorizationUrl(url='https://identity.vwgroup.io/oidc/v1/authorize')
  File "/usr/local/lib/python3.10/site-packages/weconnect/auth/we_connect_session.py", line 96, in authorizationUrl
    redirect = tryLoginResponse.headers['Location']
  File "/usr/local/lib/python3.10/site-packages/requests/structures.py", line 52, in __getitem__
    return self._store[key.lower()][1]
KeyError: 'location'

Fixed in https://github.com/mitch-dc/volkswagen_we_connect_id/releases/tag/v0.1.4 but you also need to
update Homeassistant to at least version 2023.3.0.

Still missing SoC and range among other entities after the update. Still on car version 2.4. Is the latest version compatible with pre 3.0 version?

The required pull request wasn't included in home assistant 2023.3.0. It's currently just in the dev branch. You'll have to wait for the next release.

Will a manual update to request 2.28.2 fix this issue?


Automated uninstalling a plugin

Hey there,
firstly a big thank you for this wonderufl repo!
For me it works like a charm but there is one question I can't answer myself.
How can I uninstall a plugin in a CI/CD Workflow?
Right now we build a new image on push where all the composer depencies are installed an then installed and activated in shopware from the entrypoint.
If I would just remove a plugin from composer.json obviously the plugin files aren't present anymore but it still would be marked as installed in Shopware? Even if not, there would be no possibility to run the uninstall logic of said plugin?
Thank you so much for a hint 😃

Hey, great question,
Right now, the way would be:

Run bin/console plugin:uninstall <name on the current docker image
Remove the plugin from composer.json and deploy a new version

This is definitely something we want to improve. It's hard to solve this in the docker image level as we have no idea that you want to remove the plugin in future

Hey @shyim,
thanks for clarifying!
I had a very similar idea. I'm thinking in Hashicrop Nomad here but it should also work with Kubernetes, Compose, etc.
Every deployment now:

Prestart: Init container with entrypoint /setup
Start: Long running Web Container

Every deployment in the future:

Prestart (Current Tag): Init container with entrypoint /cleanup (or something like hat)
Prestart (New Tag): Init container with entrypoint /setup
Start (New Tag): Long running Web Container

The new entrypoint should run the uninstall of plugins that are not part of the new tag. I first thought about fetching the composer.json from a repository to compare it but that would at least require credentials and access to said repository. Also there should be a switch for keepUserData of UninstallContext.
Currently I'm thinking (like you) about environment variables to define which plugins should be uninstalled (an if the user data should be kept). We then can set that environment variables automatically via CI/CD or manual for non CI/CD environments.
Does that make sense? I'm trying to test it but it could take some time since I'm kind of busy right now. Open for every suggestion 😃

Quick heads up before this issue gets stale 😃
I'm testing the proposed solution by the end of September. Still open for additional input or new ideas 👍

sounds good to me :)

Hi @wv-leifel
do you have an update on this?

I am going to tackle this in Deployment Helper, so I move this issue also to there


Input Function not working

What steps will reproduce the problem?

input('what is your name')
This will not print anyhting to the conlose when I try to run it?

Hey @philhoy, thanks for reporting. This error is already fixed in our latest version (5.3.2).
Since it's still not available in Anaconda, you have two options:

Use our Windows or Mac installers, which you can find here.

Create a new environment with conda-forge packages. For that, please close Spyder, open the Anaconda Prompt and run the following commands there:
 conda create -n spyder-cf -c conda-forge spyder
 conda activate spyder-cf
 spyder

This goes in out of working. It worked for a day. Then stoped working. I tried sending another query and it started to work shortly after. Shutting down and restarting. But then the following day it will stop working?


Do not be shy about this awesome helper --- Ask for notion in AutoDarkMode Windows-Auto-Night-Mode

It took me literally 40 minutes of pestering search engines with several dozen iterations to find your theme-switcher for Visual Studio - and only because i stumbled in the results over the Default dark and light theme for Visual Studio 2022 question on StackOverflow.

Meaning: The search-engines are not yet saturated with your helper regarding the topic "auto theme switch" + "visual studio".
I am quite sure there are lots of others out there who wish for it, but simply are not persistent enough for whatever reason (mostly available time and patience) to try out tons of search-results in different search-engines on different operating-systems.
I would also propose to ask for a notion for your helper in the README of Auto Dark Mode Windows-Auto-Night-Mode , since your helper is the perfect addition to it.

I opened an issue/request in the mentioned Windows-Auto-Night-Mode by AutoDarkMode repository:
Request --- Notion in README about JasonWei512 Visual-Studio-Auto-Theme-Switcher-for-Win10-Dark-Mode and if possible collaboration - to encourage contact and possibly a collaboration.


provide an agent plugin for aliyun rocketmq

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] New feature provided
Related issues

New feature or improvement
provide an agent plugin to support aliyun rocketmq service

Make sure CI works. And how we test it? Otherwise, I would recommend to host source and jar in your personal repo, and add a pointer link in ours. Make sense?

ok

Add the link to supported list and agent optional plugin pages. Thanks

Related #3630


fix: run content script with type module

fix #13

When I have mutilple content scirpt , the output chunck  modulepreload became not inline, rather using import "xxx";
But chrome extension content scirpt not surpport ,how can i fix it ?


CommonHeight property exposed on Labels

Suggestions from SomeProgrammer
http://forums.xamarin.com/discussion/33134/suggestion-return-actual-label-size

See https://github.com/mono/CocosSharp/pull/160

New Unified Label exposes LineHeight property which allows the line height to be modified for Bitmap and SpriteFont labels.
    public float LineHeight { get; set; }

See the test LabelFntLineHeightTest https://github.com/mono/CocosSharp/blob/master/tests/tests/classes/tests/LabelTestNew/LabelFNTLineHeightTest.cs
See the Unified Label source here https://github.com/mono/CocosSharp/blob/master/src/label_nodes/CCLabel.cs


⚠ UK SLOW DNS has degraded performance

In 51cec21, UK SLOW DNS (http://ukray.serverssh.net/) experienced degraded performance:

HTTP code: 200
Response time: 24502 ms

Resolved: UK SLOW DNS performance has improved in 5e3064a.


How to set a maximum size for anchor boxes?

I am trying to modify the faster rcnn to fit with small objects. But in my images, there are only small objects and they are almost the same size as each other. I have changed these:
RPN_MIN_SIZE: form 16 to 32
anchor base size: from 16 to 2

I have 2 question:

Are these modifications right, or there are better options?
Is there a way to set a maximum size for RPN or anchor base size? (since there is no large object and some generated anchors maybe redundant.)

there is no right or wrong, especially you are the master of your project, as an outsider i don't know how to answer. if the objects are of the same size, just use a single scale should be fine. the anchors kept roughly the same size of the objects you want to detect.


[VEN-346] Add voter detail page

Jira ticket(s)
[VEN-346]
Changes
Add Voter Detail page

Nice, looks very good 👍
What do you think about adding an hover effect on the "copy" icon next to the address in the header and displaying a toast when clicking on it to indicate it's been copied to the clipboard?
Similarly to how we do with the same icon in the "Your wallet" modal:

Nice, looks very good 👍
What do you think about adding an hover effect on the "copy" icon next to the address in the header and displaying a toast when clicking on it to indicate it's been copied to the clipboard? Similarly to how we do with the same icon in the "Your wallet" modal:
Apart from this it looks good to merge for me.

yes, that was the idea, just waiting on the other PR to merge

Nice, looks very good 👍
What do you think about adding an hover effect on the "copy" icon next to the address in the header and displaying a toast when clicking on it to indicate it's been copied to the clipboard? Similarly to how we do with the same icon in the "Your wallet" modal:
Apart from this it looks good to merge for me.

yes, that was the idea, just waiting on the other PR to merge

Which other PR?
I had another look and spotted an issue with pagination. While I'm on page 1, if I click on the page 2 button the page does refresh but the pagination still displays page 1 as being the active one. If I click on page 2 again, the content stays the same but the page 2 is set as active. If I now click on page 3, it refreshed the content and displays page 1 as being active.
I also found two little design issues

user count: the icon and the text aren't vertically aligned and the text should also be grey:

header address: only the icon should be clickable, not the whole block, like in the "Your wallet" modal. The address should also be aligned with the edge of the page content:

Nice 👍


Clarify Get() sample in Azure.ApplicationModel.Configuration

In the Retrieve a Configuration Setting sample, it shows calling Set on the setting before Get.  Since the text before the sample says you’re retrieving a previously stored setting, this is a little confusing.
It might be worth considering clarifying that you can call just Get on a setting that’s already been set?  (And possibly something similar for Update and Delete?)  This could be done either by adding a comment, or by providing a longer sample preceding the Get/Update/Delete calls, to make it clear that Set is needed to add the setting to the setting store, but Get/Update/Delete can be called independently once it's there.
Additional README update:

[ ] Remove "call az extension" line, as this is no longer required.

Completed.


dashmate setup fails with error:0308010C:digital envelope routines::unsupported

Setting up dashmate v0.22.7 after upgrading from v0.22.3 fails
Expected Behavior

Successfully setup dashmate v0.22.7
Current Behavior

Setup fails:
✔ Set configuration preset
✔ Set the number of nodes
✔ Enable debug logs
✔ Set the core miner interval
✖ error:0308010C:digital envelope routines::unsupported
◼ Configure Core nodes
◼ Configure Tenderdash nodes

Steps to Reproduce (for bugs)

Upgrade dashmate v0.22.3 to v0.22.7
Remove all existing associated docker containers, images, and volumes.
Run rm -r ~/.dashmate
Run dashmate setup local

Your Environment

Version used: dashmate v0.22.7
Environment name and version: node-v18.1.0
Operating System and version: darwin-arm64

Does it work correctly for you if you use Node v16? Googling that error seems to indicate an OpenSSL incompatibility when using Node versions > v16. Examples:
https://itsmycode.com/error-digital-envelope-routines-unsupported/
https://github.com/trufflesuite/truffle/issues/4570

Node versions other than v16 is not supported at the moment. RIPEMD160 is removed from libssl on Node v18+

retest passed


Add fingerprints for russia

Here are some blockpages found in Russia to add to our fingerprint db:
https://explorer.ooni.torproject.org/measurement/20180315T230339Z_AS15378_1OT3ZGTyarLfiET0jYHJZigX2B1oQDdJKdrjkfzq5Zqr30Lvlp?input=http:%2F%2Fqha.com.ua%2F
https://explorer.ooni.torproject.org/measurement/20180315T100250Z_AS6789_KYIQLsidOroJuPSP8DNkM8WFYOwNEAKebucOYOHhr9bwel8Yl3?input=http:%2F%2Fqha.com.ua%2F
The second interestingly shows HTTP status code 451, we should probably have a heuristic that looks for that status code too.

A DNS based feature for blocked sites in crimea is the following IP:
<IP_ADDRESS>
<IP_ADDRESS>

source measurements: https://explorer.ooni.torproject.org/measurement/20180315T174422Z_AS12688_7Iy8vwd6JYREOl2E6E1PJnNeCVGVKlORQNYhUJ2tKyiWjaEFkY?input=http:%2F%2Fmaidanua.org%2F

Another blockpage fingerprint for Russia: https://explorer.ooni.torproject.org/measurement/20180315T160455Z_AS8359_t1zebVigtFmS7kxCOCe3P77uClvciZHmKIwutI13x3Wcpe5t9V?input=http:%2F%2Futronews.org

Another one:
https://explorer.ooni.io/measurement/20181229T072204Z_AS39289_xlYTdPez910QvwBFCYyD3sgv0hQq1WBgbhW3lMjIs3MVMUdGtg?input=http:%2F%2Fblackberry.com
tag:http://<IP_ADDRESS>/451/


Inversion in Context Menu

As a developer, I want to invert the current selection of the change list via a context menu that open on right click, because I don't want too many different buttons but one fitting list of actions for the context I am working in.
Acceptance criteria:

[ ] A context menu opens on a right click on the change list.
[ ] The context menu has the option "Invert selection".
[ ] The selection of the change list is inverted after clicking on "Invert selection".

The original issue was:
As a developer, I want to invert the current selection of the change list via a context menu that open on right click, because I don't want too many different buttons but one fitting list of actions for the context I am working in.
Acceptance criteria:

[ ] A context menu opens on a right click on the change list.
[ ] The context menu has the option "Invert selection".
[ ] The selection of the change list is inverted after clicking on "Invert selection".


[JS] accessing nested namespaces from require() with destructuring leads to an error

TypeScript Version:  4.2.0-dev.20201126

Search Terms: TS2305
Code
// Must be in JS mode, works in TS mode
const { HttpUriPlugin } = require("webpack").experiments.schemes;
// TS2305: Module '"webpack"' has no exported member 'schemes'.

// Syntax equivalents that work:
const schemes = require("webpack").experiments.schemes;
const HttpUriPluginAlternative1 = require("webpack").experiments.schemes.HttpUriPlugin;
const { schemes: { HttpUriPlugin: HttpUriPluginAlternative2 } } = require("webpack").experiments;

console.log(HttpUriPlugin);
console.log(schemes.HttpUriPlugin);
console.log(HttpUriPluginAlternative1);
console.log(HttpUriPluginAlternative2);

Expected behavior:
Destructuring a nested namespace in JS mode doesn't lead to an error
Actual behavior:
An error is emitted.
The error also looks a bit weird:
Module '"webpack"' has no exported member 'schemes'.
Something like Namespace 'experiments' has no exported member 'schemes'. would make more sense.
It seems like it looks up schemes on the wrong type...
Playground Link: Link
Related Issues:

Seems that the culprit may be #39770, as there are many similar issues. The palyground link works with 4.0.5 but fails with version 4.1.2.

I couldn't find any other code bases that try the combination of property accesses and destructuring. This is indeed an incorrect outcome that neither the ad-hoc alias resolution code nor the synthetic-type fallback correctly handle, but I'm not sure it's worth complicating the already-bizarre code unless we find out that lots of people want to write this code.


Question about data augmentation

Question about data augmentation: When subgraph augmentation is performed, essentially only the enge_index is modified. This means that only the subgraph is aggregated for information propagation. What puzzles me is that when computing the graph level features, the pooling function involves the features of all nodes of the graph (selected subgraph nodes and unselected nodes), does this make sense? Isn't the pooling at this point only considering subgraphs?

Hi @Struggle-Forever,
Please refer to an instantiation of the subgraph augmentation https://github.com/Shen-Lab/GraphCL/issues/24#issuecomment-839406281. We would drop the non-selected nodes via, e.g. data.x = data.x[idx_nondrop] where the pooling function is operated on.


getHTML() Method in Toast-UI/Editor Returns Unexpected Value When Editor is Empty

Describe the bug
When using the getHTML() method in toast-ui/editor, it consistently returns  instead of null when the editor is empty.
To Reproduce
Steps to reproduce the behavior:

Initialize toast-ui/editor.
Use the getHTML() method when the editor is empty.
See that "" is returned instead of null.

Expected behavior
The getHTML() method should return null when the editor is empty just like getMarkdown().
Screenshots

Additional context
This bug affects the behavior of the getHTML() method specifically when the editor is empty. It may impact applications relying on the returned value to determine whether the editor content is empty or not.

I agree. Same for getMarkdown It grabs things from the HTML of the editor itself. Here in French language you see some values:
For me empty or not !


⚠ Samsung has degraded performance

In 0dc424c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2121 ms

Resolved: Samsung performance has improved in 605aaa9.


Fix unintentional calling of property

_array_interface_base is a decorated property, but is called like a function in the public array_interface_base method.

This was merged in https://github.com/shapely/shapely/pull/1235
Thanks again @martinxsliu


keras.model.Model does not work with SparseTensor multiplied with Tensor in functional API

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Windows 10
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): unknown 2.1.0 (from conda)
Python version: 3.7.7
CUDA/cuDNN version: -
GPU model and memory: (CPU only)

Describe the current behavior
Wrapping sparse operation with Lambda layer does not help in functional model if I try to multiply SparseTensor and Tensor using tf.sparse.sparse_dense_matmul.
There is an exception:
ValueError: Sparse ops are not supported with functional models with built-in layer wrapping. Please wrap the sparse ops in a Lambda layer like:

        weights_mult = lambda x: tf.sparse.sparse_dense_matmul(x, weights)
        output = tf.keras.layers.Lambda(weights_mult)(input)

Describe the expected behavior
Sparse operation is working with usual Tensor if it is wrapped in Lambda layer.
Code to reproduce the issue
import tensorflow as tf
from tensorflow.keras import layers
from tensorflow.keras import models

def generate_sparse_matrix():
    return tf.SparseTensor(indices=[[0, 0], [1, 2]], values=[1.0, 2.0], dense_shape=[3, 4])

input_model = layers.Input(shape=(8,))
dense_layer = layers.Dense(3, activation='relu')(input_model)
sparse_dense_matmul = lambda x: tf.sparse.sparse_dense_matmul(x[0], x[1], adjoint_a=True, adjoint_b=True)
multiplied = layers.Lambda(sparse_dense_matmul)((generate_sparse_matrix(), dense_layer))
multiplied = tf.transpose(multiplied)

out_layer = layers.Dense(1, activation='sigmoid', dtype='float32')(multiplied)

model = models.Model(input_model, out_layer)

Colab:
https://colab.research.google.com/drive/1SEvbnO8lrMCZQdVvjyd6Tmiabi1b3_UE?usp=sharing
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
<ipython-input-4-2361bce90c51> in <module>
      5 multiplied = tf.transpose(multiplied)
      6
----> 7 out_layer = layers.Dense(1, activation='sigmoid', dtype='float32')(multiplied)
      8
      9 model = models.Model(input_model, out_layer)

~\Anaconda3\envs\tensor\lib\site-packages\tensorflow_core\python\keras\engine\base_layer.py in __call__(self, inputs, *args, **kwargs)
    718     # framework.
    719     if build_graph and base_layer_utils.needs_keras_history(inputs):
--> 720       base_layer_utils.create_keras_history(inputs)
    721
    722     # Clear eager losses on top level model call.

~\Anaconda3\envs\tensor\lib\site-packages\tensorflow_core\python\keras\engine\base_layer_utils.py in create_keras_history(tensors)
    185     keras_tensors: The Tensors found that came from a Keras Layer.
    186   """
--> 187   _, created_layers = _create_keras_history_helper(tensors, set(), [])
    188   return created_layers
    189

~\Anaconda3\envs\tensor\lib\site-packages\tensorflow_core\python\keras\engine\base_layer_utils.py in _create_keras_history_helper(tensors, processed_ops, created_layers)
    247               constants[i] = backend.function([], op_input)([])
    248       processed_ops, created_layers = _create_keras_history_helper(
--> 249           layer_inputs, processed_ops, created_layers)
    250       name = op.name
    251       node_def = op.node_def.SerializeToString()

~\Anaconda3\envs\tensor\lib\site-packages\tensorflow_core\python\keras\engine\base_layer_utils.py in _create_keras_history_helper(tensors, processed_ops, created_layers)
    221             'Sparse ops are not supported with functional models with built-in '
    222             'layer wrapping. Please wrap the sparse ops in a Lambda layer like'
--> 223             ': \n{lambda_example}\n'.format(lambda_example=lambda_example))
    224
    225       # Recursively set `_keras_history`.

ValueError: Sparse ops are not supported with functional models with built-in layer wrapping. Please wrap the sparse ops in a Lambda layer like:

        weights_mult = lambda x: tf.sparse.sparse_dense_matmul(x, weights)
        output = tf.keras.layers.Lambda(weights_mult)(input)

@Cheburum
Can you please refer to this issue with similar error and let us know if it helps:
link

@Saduf2019 I have read this thread, but solution "to wrap sparse ops with Lambda layer" does not work in this case.

I am able to replicate this issue, please find the gist here. Thanks!

@Cheburum
Can you please try the code on tf-nightly [2.4.0-dev20200817], i do not face any errors please find the gist here


User editor

https://github.com/CUCB/website/commit/8ee95b28872ea621e6f048c6470a49adbeb256ef is the merge of these changes, so they don't need merging into master now.


Update Storybook dependencies to include "Code" panel for stories

Depends on

https://github.com/storybookjs/storybook/pull/29253

Why?
In #1904, we created a Storybook PR to contribute a "Code" panel for single stories. We want to wait until the feature is officially released in Storybook itself and then update our onyx Storybook to include this new feature.
Acceptance criteria

[x] All Storybook dependencies are updated to the new released version which includes the new code panel feature
[x] The "Code" panel is shown correctly and shows the code snippet for the story

Definition of Done

[x] follow-up tickets were created if necessary

#2379

[x] Storybook can show the feature

Approval

Storybook
[ ] approved by designer @jannick-ux
[ ] approved by dev

Has it by default always a dark background or would it be possible to make the background pending on the chosen theme light/dark?
2 more findings:
There is a tiny shadow (when I spot it right). Please remove that
Could you make the height of the code panel filled to the bottom of the window?


feat(logger): add Check for the logger

Thank you for contributing to log!
Please add a summary of your change
Does your change fix a particular issue?
Fixes #(issue)
Check list:

[ ] Mark if documentation changes are required.
[x] Mark if tests were added or updated to cover the changes.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 58.23%. Comparing base (d8f9846) to head (bc2b7a5).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #34      +/-   ##
==========================================
+ Coverage   57.73%   58.23%   +0.49%
==========================================
  Files           4        4
  Lines         336      340       +4
==========================================
+ Hits          194      198       +4
  Misses        138      138
  Partials        4        4

Flag
Coverage Δ

unittests
58.23% <100.00%> (+0.49%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


fix(cursor): do not truncate an existing Long

This is a fix for issue #440

@jasontho-ms are you sure this is necessary? I just ran the following in the terminal and it seems okay.
const bson = require('bson');
const assert = require('assert');

assert.ok(bson.Long.fromNumber(bson.Long.MAX_VALUE).equals(bson.Long.MAX_VALUE))

MAX_VALUE has no low bits so it works, but try it with one less than MAX_VALUE so it has low bits:

bson.Long.MAX_VALUE.toString()
'9223372036854775807'
const foo = bson.Long.fromString('9223372036854775806')
foo
Long { bsontype: 'Long', low: -2, high_:<PHONE_NUMBER> }
assert.ok(bson.Long.fromNumber(foo).equals(foo))
AssertionError [ERR_ASSERTION]: false == true

@jasontho-ms Ah, good point.

Internet points don’t matter 😊 I’d appreciate it if you port it over. Thanks for taking a look!
From: Dan Aprahamian<EMAIL_ADDRESS>Sent: Wednesday, June 12, 2019 2:22 PM
To: mongodb-js/mongodb-core<EMAIL_ADDRESS>Cc: Jason Thorsness<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [mongodb-js/mongodb-core] fix(cursor): do not truncate an existing Long (#441)
@daprahamian approved this pull request.
Thank you for this @jasontho-mshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjasontho-ms&data=02|01|jasontho%40exchange.microsoft.com|9ac77dec58e2488ae35108d6ef7c08a6|72f988bf86f141af91ab2d7cd011db47|1|0|636959713346774427&sdata=r0Iulwg0t8rQiX44%2FqR9Oa1YMhrflic9KozwLD6iH5s%3D&reserved=0. As a heads up, we have merged mongodb-core and node-mongodb-native together for our next minor release. I can take care of porting this over to the new version, but you will lose credit for the commit. Do you want to take care of making this PR against the next branch of node-mongodb-native, or would you like me to take care of it. Let me know which works best.

Dan

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmongodb-js%2Fmongodb-core%2Fpull%2F441%3Femail_source%3Dnotifications%26email_token%3DAESIW5HGU36CPKCKOMSVTMTP2FSIHA5CNFSM4HXDUNE2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB3L5J5Q%23pullrequestreview-249025782&data=02|01|jasontho%40exchange.microsoft.com|9ac77dec58e2488ae35108d6ef7c08a6|72f988bf86f141af91ab2d7cd011db47|1|0|636959713346784419&sdata=k2kT4GYB2ED9nFK2gjfPduLoARuLSYmTle8CpQobGaQ%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAESIW5BDKBX743ATHK6PANDP2FSIHANCNFSM4HXDUNEQ&data=02|01|jasontho%40exchange.microsoft.com|9ac77dec58e2488ae35108d6ef7c08a6|72f988bf86f141af91ab2d7cd011db47|1|0|636959713346784419&sdata=0uoU7rImVxzsCBWTtrLCmI7RT8Axad1pigAhHmhO2EM%3D&reserved=0.

Merged. Thank you for the contribution @jasontho-ms!


Simple/Advanced mode does not persist

For the containers image, you can switch between simple and advanced mode. I always choose simple, otherwise it will prepend something like 'docker.io/' to the image name.
The problem is that whenever I click "edit/duplicate" on the container, the setting always defaults back to "Advanced mode". So everytime I redeploy I have to remember to switch it back to "Simple mode" first.
There are two possible solutions:

Persist this setting for each container instead of reverting to the default each time
Remove this annoying "docker.io" string for the image names, so that I can leave it in "Advanced mode"

@kroese
Thank you for the information.  I am going to further investigate.  I will update you as I learn more.
Thanks!

@kroese
I am able to reproduce what you are experiencing.  I am going to forward to Product for review.  I will update you as I learn more.
[1] Private Registry
[2] Container >> Add Container
[3] Simple Mode >> image:Tag
[4] Duplicate/Edit >> Advanced Mode
https://user-images.githubusercontent.com/20426210/234344435-e4081b11-4b56-4541-a353-7d563c68f915.mov
Thanks!


Unexpected error while loading subscriptions

Hi everybody,
We were having issues with azure monitor in our pre production environments while trying to setup a new datasource and we tried the latest version of grafana in docker just to make sure it wasn't our infrastructure. In Grafana we filled all the fields required to connect to Azure and whenever we try to load subscriptions we get an unexpected error message without further details. On the logs (we enabled debug level) the request is completed successfully and there are no errors.
Any idea what can be wrong? We are using exactly the same tenant id, client id, secret, etc, from an older version of Grafana (5.x).
Thank you

Got the exact same issue running Grafana 6.4.2 on Windows Server.
I get this error when trying to load subscriptions:  Azure Log Analytics: Not Found: SubscriptionNotFound. The subscription 'undefined' could not be found.

I got same error. I tryed all versions of grafana older then v6.2.5 with plugin Azure monitor v 0.3.0 - there are no problems. Newerst version of grafana including hosted at grafana.net not works properly.

I got same error. I tryed all versions of grafana older than v6.2.5 with plugin Azure monitor v 0.3.0 - there are no problems. Newerst version of grafana including hosted at grafana.net not works properly.

Well, we are running 6.2.1 and it's not working either. In 5.X works well though.

in the 6.7.1 works fine!


Wrong Titleinformation to Sonos / Last.fm

What version of Music Assistant has the issue?
2.3.4
What version of the Home Assistant Integration have you got installed?
2024.12.5 (build in)
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When I stream (doesn´t matter if local file share or Apple Music), the Titlefinormation track is scrobbeln and shown on Sonos WITH the Artist. So all played stuff is on the Last.fm side for track inforamtion "ARTIST - TRACKtitle". On my  Mac I installed the "menu bar", where you can see the Information also scrobbeld to last.fm
How to reproduce
Streaming Music from MA to sonos connected with Last.fm account
Music Providers
Apple Music, local file share
Player Providers
sonos
Full log output
music-assistant_lastfm.log
Additional information

What version of Home Assistant Core are your running
2024.12.5
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Alternative

here is a other example: correct in MA, additional artist in title in "Sonos" and Lastfm (this is how it is scrobbeld to Last.fm)

and now this:

So, for some tracks it works, for some not... all from the same provider and same tagging (streaming from apple music should be tagged correct)
I don´t expect anything, perhaps some help or tips for searching
in the log file is this shown:
AttributeError: 'NoneType' object has no attribute 'get'
2024-12-30 09:04:27.037 WARNING (MainThread) [music_assistant.webserver] Disconnected: Received non-Text message.
2024-12-30 09:32:28.821 INFO (MainThread) [music_assistant] Loaded metadata provider The Audio DB
2024-12-30 09:36:31.947 INFO (MainThread) [music_assistant.apple_music] No top tracks found for artist Sia

MA doesn’t support scrobble so I am not sure how you are doing this. Since MA doesn’t support scrobbling this doesn’t seem to be a MA issue?

that´s what I ask for: somewhere in the process (MA -> SONOS -> LastFM scrobbling) must be the time, when the IDTag getting the additional Artist in the Songtitle... So when it only would happening on playing from local files = tagged mostly correct, but there could be the wrong ID Tag.. but, from Apple music there must be the correct IDTag for scrobbling. But there is no difference between both providers.
Now I tried to deactivate the Database for metadata, but with no changes. Is there a possibility, to change the IDTags?

Sorry I am not following you at all. Your report title is that the incorrect information is sent to Sonos and last.fm
MA does not send any information to last.fm so this is confusing. Looking back at your post are you saying that in the SONOS app the track name is actually artist-track? I don’t know what that interface is which you show in the very first screenshot?

yes, in MA the ID Tag Title and Artist is shown correct, so there must be the delivery to SONOS / Last.FM with wrong or additional informations... any idea where this could be ?

No because MA streams to the player. I can only assume the Sonos app gets information from the player. I’ll need to ask the devs for more information.

ok, would be nice. after an update from sonos devices I have now this behaviour:
the entitys of the Sonos speaker now showing the Title with the artist, in MA there´s not. So I would say, that the metadata whitch are transfered from MA to Sonos are filled with to much data or a checkbox is marked wrongly... But I don´t know where to find.... I tried to disable "The Audio DB", but no change...

I strongly suggest you don’t disable things in MA if you don’t know what it does. AudioDB is used to add missing metadata to existing tracks, it has nothing to do with how information is sent to the players.

of course I made backups and just disabled for one track... but, no result, so I stopped trying.
This is what I´m searching for: where is the point, the MA is sending informations to the player and can I have some changes in the way it does or what is transmitted? thank you: happy new year


fix enum already exist declaration

Released as https://github.com/riverwaysoft/php-converter/releases/tag/<IP_ADDRESS>


Add a create notebook command to the Manage Dashboard for databases (aduxr)

Add a create notebook command to the Manage Dashboard for databases.
Currently you can create a new SQL File from the Manage Dashboard for a database but you cannot create a New SQL Notebook.

Let's work on updating the default dashboard here.


Inconsistent naming HELICS property constants

Describe the bug
When trying to use the Pythonic API, getting the time period requires using the name "TIME_PERIOD" for the property while accessing the same property with the C-API uses the name "HELICS_PROPERTY_TIME_PROPERTY". The naming should be consistent across the APIs.
To Reproduce
Steps to reproduce the behavior:
Example of working versions of the C- and Pythonic API are provided in the HELICS-Examples repository.
Environment (please complete the following information):

Operating System: macOS 14.5
Installation: pip install helics

helics --version

helics, version v3.4.0

Python HELICS version v3.4.0

HELICS Library version 3.4.0 (2023-01-19)

$ python -c "import helics as h; import json; print(json.dumps(h.helicsGetSystemInfo(), indent=4, sort_keys=True))"

{
    "buildflags": " -O3 -DNDEBUG  $<$<COMPILE_LANGUAGE:CXX>:-std=c++17>",
    "compiler": "Unix Makefiles  Darwin-21.6.0:AppleClang-<IP_ADDRESS>00029",
    "cores": [
        "zmq",
        "zmqss",
        "tcp",
        "tcpss",
        "udp",
        "ipc",
        "interprocess",
        "inproc"
    ],
    "cpu": "Apple M1 Max",
    "cpucount": 10,
    "cputype": "x86_64",
    "hostname": "WD86465",
    "memory": "32768 MB",
    "os": "Darwin  23.5.0  Darwin Kernel Version 23.5.0: Wed May  1 20:12:58 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6000",
    "version": {
        "build": "",
        "major": 3,
        "minor": 5,
        "patch": 2,
        "string": "3.5.2 (2024-04-08)"
    },
    "zmqversion": "ZMQ v4.3.5"
}

Currently, it appears that fed.property[h.HELICS_PROPERTY_TIME_PERIOD] already works as well.
Are you suggesting that we deprecate the string checking on the property accessors?

I think this is user error on my part. I thought I had used the Pythonic API HELICS property access before without having to specify the constant as being in the HELICS library (e.g. "h. ....") but I'm probably mis-remembering. I was able to replicate correct operation using the above constant.


Set up CI with Azure Pipelines

[skip ci]

 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.:x: RocioOrdonez sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Skip save step if manifest hash has not changed

Currently if the save parameter is configured, a cache will always be saved on the post-command hook.
Saving cache can be a costly operation if the file/folder is large.
It would be good if there was an option to skip saving the cache if it is detected that a cache already exists with the same cache key.
This would mimic what some other CI platforms like CircleCI do:

Hi @toote -- saw you have a PR open to fix this and it looks like it's close to landing. For our case this step is adding 2m to an otherwise <30s build step and results in CI on Buildkite taking longer than on GitHub Actions, so I'd be eager for the update!
Is there an ETA for when this will land? Thanks very much for the fix!

Is there an ETA for when this will land? Thanks very much for the fix!

Done! We'll do a release tomorrow

Thanks @pzeballos -- I'll wait for the release to integrate my pipeline!

Hi all -- I tried to integrate this but unfortunately it didn't speed things up much. The issue seems to be that it is always zipping up the directory even if the cache exists. In the case of .pnpm-store the zipping operation takes 30-40s so this nullifies the caching benefits.
Is it possible to reverse the order of operations so that this doesn't occur? Here's the example output from my build:

$ /var/lib/buildkite-agent/plugins/buildkite-i-0c1a22a08d7ad2385-2/github-com-buildkite-plugins-cache-buildkite-plugin-v1-1-0/hooks/post-command
--
  | Compressing .pnpm-store with tgz...
  | Cache of file already exists, skipping

@nealchandra I see what you mean, apparently I followed the letter of what you requested and not the spirit of it. From what I can see, though, what you actually need would require more extensive and invasive changes due to the current flow of the code but I will keep it in mind for future changes

@nealchandra FYI: #78 was opened about your scenario


Grant Type : Refresh Token not being recognised.

Please see the screenshot below. It is not recognising the grant_type as shown. Please check and confirm if this is an interim issue while using from postman or java standalone code. It is not working from both the places.
https://docs.microsoft.com/en-us/answers/questions/238954/grant-type-39refresh-token39-seems-to-be-not-recog.html

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

ID: 13b44819-ed7b-84bc-f5d2-2b937029f7df
Version Independent ID: db9f8408-c234-90d0-4817-3516e5fc3ba0
Content: Microsoft identity platform and OAuth 2.0 authorization code flow - Microsoft identity platform
Content Source: articles/active-directory/develop/v2-oauth2-auth-code-flow.md
Service: active-directory
Sub-service: develop
GitHub Login: @hpsin
Microsoft Alias: hirsin

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

Thanks James. I'd like to follow this up here. I also tried java way, and it is having the same issue.
Do you see any issue here? I'm using apache client.
	public String doPost(String oAuth2EndPoint, String clientId, String accessTokenExpiry, String refreshToken,
			String clientSecret) throws IOException, JSONException {
		// Check if Token expiry has reached using accessTokenExpiry

		try (CloseableHttpClient httpclient = HttpClients.createDefault()) {

			HttpPost httpPost = new HttpPost("https://login.microsoftonline.com/common/oauth2/v2.0/token");
			//httpPost.setHeader("Accept", "application/json");
			httpPost.setHeader("Content-type", "application/x-www-url-form-urlencoded");

			StringBuffer paramas = new StringBuffer();
			paramas.append("client_id=" + clientId);
			paramas.append("&scope=https://graph.microsoft.com/user.read");
			paramas.append("&redirect_uri=https://xcpapp/msgraph/");
			paramas.append("&grant_type=refresh_token");
			paramas.append("&client_secret=" + clientSecret);
			paramas.append("&refresh_token=" + refreshToken);

			//System.out.println("Params : "+paramas.toString());

			StringEntity stringEntity = new StringEntity(paramas.toString());
			httpPost.setEntity(stringEntity);

			System.out.println("Executing request " + httpPost.getRequestLine());

			// Create a custom response handler
			ResponseHandler<String> responseHandler = response -> {
				int status = response.getStatusLine().getStatusCode();
				if (status >= 200 && status < 300) {
					HttpEntity entity = response.getEntity();
					return entity != null ? EntityUtils.toString(entity) : null;
				} else {
					throw new ClientProtocolException("Unexpected response status: " + status);
				}
			};
			String responseBody = httpclient.execute(httpPost, responseHandler);
			System.out.println("----------------------------------------");
			System.out.println(responseBody);

			// return parseResponse(responseBody);
			return responseBody;

		}

Thanks James. I'd like to follow this up here. I also tried java way, and it is having the same issue.
Do you see any issue here? I'm using apache client.
	public String doPost(String oAuth2EndPoint, String clientId, String accessTokenExpiry, String refreshToken,
			String clientSecret) throws IOException, JSONException {
		// Check if Token expiry has reached using accessTokenExpiry

		try (CloseableHttpClient httpclient = HttpClients.createDefault()) {

			HttpPost httpPost = new HttpPost("https://login.microsoftonline.com/common/oauth2/v2.0/token");
			//httpPost.setHeader("Accept", "application/json");
			httpPost.setHeader("Content-type", "application/x-www-url-form-urlencoded");

			StringBuffer paramas = new StringBuffer();
			paramas.append("client_id=" + clientId);
			paramas.append("&scope=https://graph.microsoft.com/user.read");
			paramas.append("&redirect_uri=https://xcpapp/msgraph/");
			paramas.append("&grant_type=refresh_token");
			paramas.append("&client_secret=" + clientSecret);
			paramas.append("&refresh_token=" + refreshToken);

			//System.out.println("Params : "+paramas.toString());

			StringEntity stringEntity = new StringEntity(paramas.toString());
			httpPost.setEntity(stringEntity);

			System.out.println("Executing request " + httpPost.getRequestLine());

			// Create a custom response handler
			ResponseHandler<String> responseHandler = response -> {
				int status = response.getStatusLine().getStatusCode();
				if (status >= 200 && status < 300) {
					HttpEntity entity = response.getEntity();
					return entity != null ? EntityUtils.toString(entity) : null;
				} else {
					throw new ClientProtocolException("Unexpected response status: " + status);
				}
			};
			String responseBody = httpclient.execute(httpPost, responseHandler);
			System.out.println("----------------------------------------");
			System.out.println(responseBody);

			// return parseResponse(responseBody);
			return responseBody;

		}

@amit17051980
Thank you for the quick response!
For your Java implementation, did you follow any documentation when setting this up? This way I can attempt to set up a similar environment. I also found some documentation that might help with your issue, which I'll post below.
Java - Refresh Token examples:
How to refresh access token provided by TokenCredential / ClientSecretCredential by Azure Identity Java SDK?
ADAL4j java - use refresh token with username and password to get the access token
(Java) Microsoft Graph -- Renew Expiring Access Token (Azure AD v2.0 Endpoint)
When getting the access token, whether in Postman or Java, I'd make sure that you're copying the "refresh_token" value from the "Token Request - Auth Code" output, and pasting that into your refresh token variable/parameter when executing the "Token Request -Refresh Token".

If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

@amit17051980
Thank you for the quick response!
For your Java implementation, did you follow any documentation when setting this up? This way I can attempt to set up a similar environment. I also found some documentation that might help with your issue, which I'll post below.
Java - Refresh Token examples:
How to refresh access token provided by TokenCredential / ClientSecretCredential by Azure Identity Java SDK?
ADAL4j java - use refresh token with username and password to get the access token
(Java) Microsoft Graph -- Renew Expiring Access Token (Azure AD v2.0 Endpoint)
When getting the access token, whether in Postman or Java, I'd make sure that you're copying the "refresh_token" value from the "Token Request - Auth Code" output, and pasting that into your refresh token variable/parameter when executing the "Token Request -Refresh Token".

If you have any other questions, please let me know.
Thank you for your time and patience throughout this issue.

Thanks James.
Not sure what is the pain point while I want to use apache way, possibly I'll have a look later when I get some time to play with MSAL. But for now my standard tool OpenText xCP is fine. It is returning the access token etc using refresh token.

I'm happy for case to be closed as you have provided some additional links to refer for other community members.

Thanks James.
Not sure what is the pain point while I want to use apache way, possibly I'll have a look later when I get some time to play with MSAL. But for now my standard tool OpenText xCP is fine. It is returning the access token etc using refresh token.

I'm happy for case to be closed as you have provided some additional links to refer for other community members.

@amit17051980
Thank you for the detailed response and I'll go ahead and close this out!
If you run into any issues in the future, I'd highly recommend leveraging our MSAL or Java forums on Q&A and Stack Overflow. This way our experts and community can work with you to help resolve any issues or answer any questions you may have.
AzureAD MSAL Q&A
AzureAD MSAL Stack Overflow
Java Stack Overflow
GraphAPI forums on Q&A
Thank you again for your time and patience throughout this issue.

@amit17051980
Thank you for the detailed response and I'll go ahead and close this out!
If you run into any issues in the future, I'd highly recommend leveraging our MSAL or Java forums on Q&A and Stack Overflow. This way our experts and community can work with you to help resolve any issues or answer any questions you may have.
AzureAD MSAL Q&A
AzureAD MSAL Stack Overflow
Java Stack Overflow
GraphAPI forums on Q&A
Thank you again for your time and patience throughout this issue.


Add basic example

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (6665a67) compared to base (9b50346).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff            @@
##              main       #12   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           32        31    -1
  Lines         2860      2851    -9
=========================================
- Hits          2860      2851    -9

Impacted Files
Coverage Δ

starlette_admin/contrib/sqla/admin.py
100.00% <100.00%> (ø)

starlette_admin/contrib/sqla/helpers.py
100.00% <100.00%> (ø)

starlette_admin/contrib/sqla/middleware.py
100.00% <100.00%> (ø)

starlette_admin/contrib/sqla/view.py
100.00% <100.00%> (ø)

starlette_admin/helpers.py
100.00% <100.00%> (ø)

tests/sqla/test_basic_async.py
100.00% <100.00%> (ø)

tests/sqla/test_view_meta.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Creating ics on grid for basins of attraction is super slow for higher dimensions

Hey
I need to calculate basins of attraction for a high-dimensional system (100+), but I noticed that basins_of_attraction is super slow for these cases because of the creation of the initial conditions on a grid. Especifically
A = StateSpaceSet([generate_ic_on_grid(grid, i) for i in vec(I)])
is very slow for higher dimensions (was taking more than an hour to run...).
I think there are two reasons: evaluating the CartesianIndices becomes slow, and also maybe because generate_ic_on_grid allocates StaticVectors.
What would be a good solution for this?
Thanks a lot!

I wouldn't use basins_of_attraction for this purpose. Use directly the mapper:
lab = zeros(Int32, N)
for i in 1:N
      lab[i] = mapper( ic_generator())
end

The ic_generator can be a function or an array on the region you want to explore. The memory consumption would be restricted to the sparse matrix. For Kuramotos it works well.
A more condensend version: lab = [ mapper(ic_gen()) for i in 1:N]

Ah, nice idea! But in this case I need to use AttractorsViaFeaturizing since the dimension is so high.

So the problem is just initial condition caching? Because I assume you have a low dimensional feature space.

So the problem is just initial condition caching?

Yup. This line
A = StateSpaceSet([generate_ic_on_grid(grid, i) for i in vec(I)])
takes way too long just to create the ics when the dimension of the grid is high (100+).
I could of course just create A how I want for my specific problem, but it would be great if there was a simple and general way to create the ics that also works well in higher dims.

Because I assume you have a low dimensional feature space.

Yup, basins_fractions works great once we have the ics.

What about a special sampler with an internal queue:
sampler() =  generate_ic_on_grid(grid, pop!(I))
I am not sure it is possible, I am just throwing an idea

It is because of static arrays.
We need to allow StateSpaceSet to work with normal vectors as well. Someone needs to do a PR in StateSpaceSets.jl.

BTW I had this problem elsewhere as well, that StateSpaceSet.jl doesn't work with vectors in ComplexityMeasures.jl.

but indexing the cartesian indices is also very slow

This cannot be possible, because indexing the cartesian indices is as fast as indexing a linear range 1:N with N the total number of points. You are not measuring the time it takes to index the cartesian indices, you are measuring the time it takes to (1) transform a cartesian index to a vector and (2) collect all those vectors to another vector.

Also, your code gives me infinite stack overflow messages well before I go into the cartesian indices :D already from basins = .

 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] =
Internal error: stack overflow in type inference of unsafe_view(Array{Int32, 500}, Base.UnitRange{Int64}, Base.UnitRange{Int64}, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, I
...
, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64, Int64).
This might be caused by recursion over very long tuples or argument lists.
Internal error: stack overflow in type inference of (::Type{Base.SubArray{T, N, P, I, L} where L where I where P where N where T})(Base.IndexCartesian, Array{Int32, 500}, Tuple{Base.UnitRange{Int64}, Base.UnitRange{Int64}, Vararg{Int64, 498}}, Tuple{Bool, Bool}).

Internal error: stack overflow in type inference of print_matrix(Base.IOContext{Base.TTY}, Base.SubArray{Int32, 2, Array{Int32, 500}, Tuple{Base.UnitRange{Int64}, Base.UnitRange{Int64}, Vararg{Int64, 498}}, false}).
This might be caused by recursion over very long tuples or argument lists.
Internal error: stack overflow in type inference of isassigned(Base.SubArray{Int32, 2, Array{Int32, 500}, Tuple{Base.UnitRange{Int64}, Base.UnitRange{Int64}, Vararg{Int64, 498}}, false}, Int64, Int64).
This might be caused by recursion over very long tuples or argument lists.

Ah no sorry this is because of my printing hack :D

Okay, now I finally understand the problem. "Indexing the cartesian indices" means to obtain the indices themselves. And yes, indeed, simply doing I[1] is exceptionally slow.

Here you are creating a monster array: [i for i in vec(I)]

@KalelR you should open an issue at the main Julia repo that Cartesian Index access becomes slow and allocates in high dimensional arrays. This is too deeply internal of Julia for us to do anything.

Actually, I don't think anyone can do anything. The same limitation that comes from regular tuples goes into Cartesian Indices as well. Tuples are limited to 32 "dimensions" to be fast if I recall correctly. We would need to use Vectors to index things above some dimension.

Indeed, I haven't found a way to fix it using CartesianIndices, but Vectors do work great. Since typically we are working on a 2D projection of the basins, there are only two important dimensions, and the ics on others (which could be a lot) are fixed anyway. So a solution that's working for me now is to use CartesianIndices only for the dimensions have length > 1 (corresponding to the dimensions of the ics that are varying), and then build the grid from those indices. Then I put the grid of varying ics together with the fixed ics to construct the vector of the full ics.
It works great for my use-case, but I would like to test it further in other cases, I just don't have time at the moment. I'll put the code here anyway in case anyone is interested:
function generate_ics_on_grid(grid::NTuple{B, T}) where {B, T}
    grid_lengths = length.(grid)
    idxs_varying_dims = findall(len->len>1, grid_lengths)
    reduced_grid = grid[idxs_varying_dims]
    I_reduced = CartesianIndices(length.(reduced_grid))
    A_reduced = [generate_ic_on_grid(reduced_grid, i) for i in vec(I_reduced)]

    ics_fixed = [grid_dim[1] for grid_dim in grid]
    A = _expand_A(A_reduced, ics_fixed, idxs_varying_dims)
    return Dataset(A)
end

function _expand_A(vec_reduced, ic_fixed, idxs_varying_dims)
    vec = [deepcopy(ic_fixed) for _ in vec_reduced]
    for i in eachindex(vec)
        vec[i][idxs_varying_dims] .= vec_reduced[i]
    end
    return vec
end

Oh, and using regular vectors when the dimension gets beyond a value (in my case, 30) also helps:
function generate_ic_on_grid(grid::NTuple{B, T}, ind) where {B, T}
    mx_dimension_sparse = 30
    if B < mx_dimension_sparse
        return _generate_ic_on_grid(grid, ind)
    else
        return _generate_ic_on_grid_vec(grid, ind)
    end
end

function _generate_ic_on_grid_vec(grid::NTuple{B, T}, ind) where {B, T}
    gens = [(grid[k][ind[k]]) for k=1:B]
    return gens
end


Redesign identity and AAA

[x] Add definition of current state and problem
[x] External authentication providers
[x] ~External authorization providers~ - later
[x] ~External audit providers~ - later

Relevant to #2058.

@itaiad200 , thanks for the great review!
Closing this in favour of an upcoming new design to support LDAP directly-ish.  We only have current user requirements to do that.  We shall probably support (only) LDAP simple auth out of the box, and figure out an add-on interface for bigger configurations.
Meanwhile OAuth2 brings its own set of difficulties: most notably:

OAuth2 doesn't really provide user identities; typically you'd want to add OpenID Connect.  But even there major providers offer vastly different query capabilities (e.g. Google does not appear to provide group memberships on its OpenID Connect endpoint).
lakeFS users will typically want to create AWS API keys for S3 and lakeFS API robots.  There is no clear path for these robots, because OAuth2 is not a directory server.
OAuth2 servers typically authenticate users for short periods of time (hours) and often require exercising a "refresh token" even to get that far.  This is fine for a user logged into a browser client, but would require lakeFS to provide some other robot for refreshing tokens to keep AWS-style API keys current.

Stay tuned for LDAP.


[BUG] push_to_huggingface sometimes include tags in resulting argilla.yml file

Describe the bug
In some environments, when using FeedbackDataset.push_to_argilla, it generates an argilla.yaml file that includes tags that later prevents the from_huggingface function to work correctly as they yaml cannot be loaded back because of the tags.
allow_extra_metadata: true
fields:
- name: prompt
  required: true
  title: Prompt
  type: !!python/object/apply:argilla.client.feedback.schemas.enums.FieldTypes # <-- tag here
  - text
  use_markdown: true

Expected behavior
No matter what environment is used, the push_to_argilla should generate the yaml without tags.

Environment:

Argilla Version [e.g. 1.0.0]: 1.18.0
ElasticSearch Version [e.g. 7.10.2]: not relevant
Docker Image (optional) [e.g. argilla:v1.0.0]: not relevant

[ ] @alvarobartt create a patch
[ ] @alvarobartt @frascuchon @gabrielmbmb provide a better fix

I add a link here to a possible solution suggested by @frascuchon that it's use use_enum_values in the Pydantic config for the affected model.


Add s3 rest service audit log

What changes are proposed in this pull request?
Add s3 rest service audit log.
Why are the changes needed?
Record operations on buckets and objects.
Does this PR introduce any user facing changes?
no.

@ZhuTopher all finished.

You need to revolve the conflict first.

@yyongycy @Jackson-Wang-7 The conflict has been resolved and the availability of the audit log has been verified.

Please update your dev branch so that the failure UTs would pass

done.


Metrics update

Closes ethereum/eth2.0-devops#16

There would be two options for existing metrics:

Setup alertmanager+node_exporter with the playbook, leave the metrics untouched. No changes need to be made, same ports remain open and metrics work just as before, but with the alerting upgrade.
Deploy pushgateway + alertmanager + node_exporter. Complete change to the metrics in this case, but we can close unused ports.

I'd say we can leave the existing system as it is and just augment it with alertmanager + node_exporter (option 1). For future testnets we can go down the pushgateway route.

There would be two options for existing metrics:

Setup alertmanager+node_exporter with the playbook, leave the metrics untouched. No changes need to be made, same ports remain open and metrics work just as before, but with the alerting upgrade.
Deploy pushgateway + alertmanager + node_exporter. Complete change to the metrics in this case, but we can close unused ports.

I'd say we can leave the existing system as it is and just augment it with alertmanager + node_exporter (option 1). For future testnets we can go down the pushgateway route.

I've added code to allow for custom alerts. Since the custom alerts should be defined per testnet, you can configure it via a variable called custom_alert_rules. The variable content is copied over to remote and combined with the basic alerts to result in a alert.rules file which is used by alertmanager. I think this way we can have have the best of both worlds: Have base alerts defined for all instances/testnets + custom ones based on the importance of the testnet.
Pyrmont for example could fire alerts when participation drops below 75. We might want to set that for 80 in mainnet, for example.
Related to the PR: https://github.com/ethereum/eth2.0-devops/pull/23

I've added code to allow for custom alerts. Since the custom alerts should be defined per testnet, you can configure it via a variable called custom_alert_rules. The variable content is copied over to remote and combined with the basic alerts to result in a alert.rules file which is used by alertmanager. I think this way we can have have the best of both worlds: Have base alerts defined for all instances/testnets + custom ones based on the importance of the testnet.
Pyrmont for example could fire alerts when participation drops below 75. We might want to set that for 80 in mainnet, for example.
Related to the PR: https://github.com/ethereum/eth2.0-devops/pull/23


Add docker compose testing files

This PR adds some testing files to be able to run Loki with the paths that LokiStack uses to mimic the environment in a cluster

/lgtm


2.0.0-beta-25 TypeError: i is not a function + reload bundle.js

Since latest version of mui-datatable I have this error when I click on a button on my customBodyRender ...
index.js is main index.js of mui-datatables/dist
2index.js:1 Uncaught TypeError: i is not a function
    at onClick (index.js:1)
    at HTMLUnknownElement.callCallback (react-dom.development.js:100)
    at Object.invokeGuardedCallbackDev (react-dom.development.js:138)
    at Object.invokeGuardedCallback (react-dom.development.js:187)
    at Object.invokeGuardedCallbackAndCatchFirstError (react-dom.development.js:201)
    at executeDispatch (react-dom.development.js:461)
    at executeDispatchesInOrder (react-dom.development.js:480)
    at executeDispatchesAndRelease (react-dom.development.js:581)
    at executeDispatchesAndReleaseTopLevel (react-dom.development.js:592)
    at forEachAccumulated (react-dom.development.js:562)
    at runEventsInBatch (react-dom.development.js:723)
    at runExtractedEventsInBatch (react-dom.development.js:732)
    at handleTopLevel (react-dom.development.js:4477)
    at batchedUpdates$1 (react-dom.development.js:16660)
    at batchedUpdates (react-dom.development.js:2131)
    at dispatchEvent (react-dom.development.js:4556)
    at interactiveUpdates$1 (react-dom.development.js:16715)
    at interactiveUpdates (react-dom.development.js:2150)
    at dispatchInteractiveEvent (react-dom.development.js:4533)

And I have bundle.js reloaded on my react-app :
bundle.js | GET | 200 | http/1.1 | fetch | index.js:2429 | 1.4 MB | 615 ms
-- | -- | -- | -- | -- | -- | -- | --
bundle.js | GET | 200 | http/1.1 | fetch | index.js:2429 | 1.4 MB | 679 ms
bundle.js.map | GET | 200 | http/1.1 | fetch | index.js:885 | 1.3 MB | 311 ms
bundle.js.map | GET | 200 | http/1.1 | fetch | index.js:885 | 1.3 MB | 1.32 s

I'm not sure about what is happening ...

Try the latest version

Good for me with 2.0.0-beta-27 !
Thanks !


Tag input doesn't appear for new runtime configuration editors

Describe the issue
When editing an existing runtime configuration, a list of tags appears as one of the fields. However, when creating a new runtime configuration, the tags list doesn't appear as one of the fields.
To Reproduce
Steps to reproduce the behavior:

Open runtime configurations list in left side bar
Click "Create new runtime configuration"
There is no field for "tags"

Screenshots or log output
No errors in log.

Expected behavior
The tag list should appear as so:

This was fixed as part of #1293