Missing Capability Check

Hi! I followed these instructions and they were great, but sadly they did not work because of missing capabilities, which resulted in several hours of debugging. Users following these instructions should know that they need to set

Internet (Client and Server)
Private Networks (Client and Server)

Also another issue is that when I set these in Unity under Player settings, they didn't not transfer to the appxmanifest in VS, which was very confusing. They have to be set directly in the built project's Package.appxmanifest :)

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

ID: b0213cb7-2364-a9f3-6bbf-6b3d9501531d
Version Independent ID: faba539d-c785-bc56-d5f3-9ec2997b19e9
Content: PC Holographic Remoting tutorials - 2. Create a Holographic Remoting PC application - Mixed Reality
Content Source: mixed-reality-docs/mr-learning-pc-holographic-remoting-02.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @jessemcculloch
Microsoft Alias: jemccull

@martinwilter This seems like an issue with the tutorials, can you confirm?

Hi @hferrone,
I'm not familiar with this tutorial, but my guess is:

Internet (Client and Server) capabilities are not needed, probably the desktop app connects to HL app over wifi?
Private Networks (Client and Server) capabilities are not related to the the tutorial, but depends on local environment:

Private Networks (Client & Server): Provides inbound and outbound access to Intranet networks that have an authenticated domain controller, or that the user has designated as either home or work networks. Inbound access to critical ports is always blocked.

Personally, I've never seen those capabilities not being transferred to VS, I remember eye tracking not falling into that, but that was changed in latest MRTK releases, I could be wrong, but as far as I know, all capabilities settings are now transferred to VS.

@martinwilter
The tutorial is explicitly for connecting to HL over wifi to stream frames so internet capabilities are 100% needed. This is what it says on the tutorial: "In this section, you will configure your project to stream your Mixed Reality experience to your HoloLens 2 device from your PC in real-time over a Wi-Fi connection"
I was doing this on my home network, which I guess falls under "private networks".
If this helps, I'm using Unity 2019.4.9f1 and MRTK 2.4.0. It may be possible that the capabilities get transferred the first time the solution is generated but not the succeeding times it is built. However, I tried to build several times to test whether the settings get transferred and they did not. Setting them explicitly in the VS solution was the only step I needed to take to make everything work.

Thanks for the additional information, @ckskylight. I've notified the tutorial creators so they can take a look at this.

Hi @hferrone,
We have checked this issue and we need to add the below details on "Build details" sections of the tutorial.
"In the Project Settings window, expand the Publishing Settings, scroll down to the Capabilities section and check the below-shown capability checkbox in addition to the existing capabilities:
•	Internet Clint server
•	Private Network Client Server
"

@varunsiddaraju Can you be clearer on where this content needs to go and clarify this part of the sentence: "check the below-shown capability checkbox in addition to the existing capabilities:"

Hi @hferrone, I'll be stepping in to help with this and some of the other issues.
I see someone on Varun's team had made the update to the old docs fork when they looked into this a while back, they made these two commits:
https://github.com/onginnovations/mixed-reality-old/commit/6ecb21f798d43075fdf86cb43c1b63f3667340d0
https://github.com/onginnovations/mixed-reality-old/commit/3be9f2f472bd3e710734066e22e2de65144cd7e3
In the md https://github.com/onginnovations/mixed-reality-old/blob/pc-holographic-remoting/mixed-reality-docs/mr-learning-pc-holographic-remoting-02.md:

This is the same concept as described here https://docs.microsoft.com/en-us/windows/mixed-reality/develop/unity/tutorials/mr-learning-sharing-02#enabling-additional-capabilities.
Hope this helps. Let me know if you need anything else.

@hferrone, were those lines added to the docs repo? (The md I linked in my previous post was from our fork of the old docs repo). Let me know if want me to make a quick PR.


[Feature Request] Implement Another NoSQL Database

Tell us about your feature request
As currently the database only supports 5 databases, I would like to request the implementation of another NoSQL database. I have experience with NoSQL databases like Firestore/Datastore on Google Cloud, and they are quite useful for serverless applications.
Justification
NoSQL databases offer a flexible schema, horizontal scaling, and better performance for certain use cases compared to traditional relational databases. By adding support for another NoSQL database, the project can cater to a wider range of use cases and provide more options for developers.
Disclaimer

[X] I agree

For now, we are not accepting new databases. The aim is to minimize the number of db-s and avoid bloat. Implementing any database in the blueprint is straightforward, simply as adjusting the connection string, driver, and specific env-s


Document maxHeight

Feature added #6. We should document it

@jeffchan

maxHeight: Truncate the content to fit in the specified height (in px).

Is it enough as a documentation for this option?


Double Messages

Hi there,
Thanks very much creating this tool.
I noticed ( I think with the latest version ) that I'm getting double messages within my console screen, I've attached an example

Can you see the text is repeated?
Gary

I have also been having this bug happen to me, and I've come to find it's not just a visual glitch. Each command is actually being run twice, as evidenced by having cinst cygwin anywhere in your script, it will open the interactive installer and refuse to progress because its target destination already has files in it.

Seeing this and using 2.9.26 of boxstarter, because boxstarter.azure requires it

Same on 2.10.3. I tired putting just a Write-Host in my script and the output is shown twice.
Any ideas on how to debug this?

I can see that this is called 3 times, but not why:
Invoke-ChocolateyBoxstarter -bootstrapPackage 'C:\Users\vagrant\boxstarter42.txt'

Does anyone know what the latest stable version is? I have tried backing out release by release to 2.9.14 but I am still running into problems with the duplicate commands causing the process to hang.

Just getting started with Boxstarter myself, and encountering this issue as well, every single line of output is duplicated. Very strange.

I nthe "boxstarter.log" file, only some lines are duplicated. Here's a snippet:
Chocolatey installed 1/1 packages. 0 packages failed.
 See the log for details (C:\ProgramData\chocolatey\logs\chocolatey.log).
Sending message 'PostRunMessage' out if there are subscribers...
[2018-05-12T23:50:52.0780568-04:00:::PID 5324] Boxstarter: restoring current directory location to C:\Windows\system32
[2018-05-12T23:50:52.0780568-04:00:::PID 5324] Boxstarter: found C:\Users\james\AppData\Local\Boxstarter\Boxstarter.5324.restart we are restarting
[2018-05-12T23:50:52.0940571-04:00:::PID 5324] Boxstarter: found C:\Users\james\AppData\Local\Boxstarter\Boxstarter.5324.restart we are restarting
[2018-05-12T23:50:52.1090538-04:00:::PID 5324] Boxstarter: Exit Code: 0
[2018-05-12T23:50:52.1411566-04:00:::PID 5324] ++ Boxstarter finished Calling Chocolatey to install tmp93D.tmp. This may take several minutes to complete... 00:04:18.9957038
[2018-05-12T23:50:52.1411566-04:00:::PID 5324] ++ Boxstarter finished Calling Chocolatey to install tmp93D.tmp. This may take several minutes to complete... 00:04:18.9957038
[2018-05-12T23:50:52.1575554-04:00:::PID 5324] Boxstarter: Removing C:\ProgramData\chocolatey\lib\tmp93D.tmp in progress
[2018-05-12T23:50:52.1730616-04:00:::PID 5324] Boxstarter: writing restart file
[2018-05-12T23:50:52.1730616-04:00:::PID 5324] Boxstarter: writing restart file
[2018-05-12T23:50:52.4630540-04:00:::PID 5324] Boxstarter: Restore Automatic Updates from Windows Update
[2018-05-12T23:50:52.4945508-04:00:::PID 5324] Boxstarter: UAC Enabled. Disabling...
[2018-05-12T23:50:52.5100548-04:00:::PID 5324] Boxstarter: Disabling UAC
[2018-05-12T23:50:52.5413599-04:00:::PID 5324] + Boxstarter finished Installation session. 00:04:22.2055366

I would love to help fix this but I have no idea how to debug it further. Any suggestions appreciates :-)

Same issue here.

Is there any update or work around for this?

Not at the moment. Happy to accept PR's as it's 'Up For Grabs'.

I don't believe it is actually running the installation scripts twice.
I believe the issue is with Write-Host in which it is outputting the value twice to the powershell cli.
Interestingly if you execute boxstarter from within Powershell ISE you don't get the duplicate outputs from Write-Host.
To prove this I modified the Write-HostOverride function within the chocolatey.ps1 script.
I commented out all the code within this function and added two lines
Microsoft.PowerShell.Utility\Write-Host @PSBoundParameters
Log-BoxStarterMessage $object

This means that the overridden Write-Host functions writes to the console output and also to the log file.
There results was that it was NOT entered twice in the log file.
But in powershell CLI it was outputed twice.
I then changed the line for Write-Host to Write-Output $object
Microsoft.PowerShell.Utility\Write-Output $object
The result was that the output was only shown once in Powershell CLI.
I have not gotten to the bottom of it but don't believe the packages are getting executed twice.
I believe it is some type of bug with the Write-Host function because called within an Invoke-Command as the duplicate messages only show for the installations of the child packages and not the parent package.
I hope that helps with trying to find a solution.
But it seems to only be a UI issue and not a functional issue.

@adrianhalid Thanks for that detailed troubleshooting. It is indeed only a UI / display issue and not a functional one.
If you get to the bottom of it then PR's are welcome.

I think its a bug in PowerShell as per this Open issue.
https://github.com/PowerShell/PowerShell/issues/7814
I can't absolutely prove this but looking at the behaviour I think its related.
If you read the issues comments you see it also affect the Invoke-Command.
I was trying to figure out if/which Invoke-Command might be causing the issue but couldn't figure it out.
I wanted to try use the -InformationAction Ignore option to see if I could stop the message being passed up the nested Invoke_Command's.
Maybe someone who knows the code base better than me can figure out where the -InformationAction Ignore options could work. I am thinking it might actually have to be applied to the Chocolatey code base. But couldn't figure it out where.
Here is the example they provide which demonstrates the issue.
You would expect the text "Test" to be displayed once.
But with the nested Jobs/Commands it is echoed out to the display 6 times.
Start-Job {
Start-Job {
Start-Job {
Start-Job {
Start-Job {
    Write-Information Test -InformationAction Continue
} | Receive-Job -Wait -AutoRemoveJob -InformationAction Continue
} | Receive-Job -Wait -AutoRemoveJob -InformationAction Continue
} | Receive-Job -Wait -AutoRemoveJob -InformationAction Continue
} | Receive-Job -Wait -AutoRemoveJob -InformationAction Continue
} | Receive-Job -Wait -AutoRemoveJob -InformationAction Continue

Also, it is not jobs specific. It is how PowerShell remoting work in general:
Invoke-Command . {
Invoke-Command . {
Invoke-Command . {
Invoke-Command . {
Invoke-Command . {
    Write-Information Test -InformationAction Continue
} -EnableNetworkAccess -InformationAction Continue
} -EnableNetworkAccess -InformationAction Continue
} -EnableNetworkAccess -InformationAction Continue
} -EnableNetworkAccess -InformationAction Continue
} -EnableNetworkAccess -InformationAction Continue


Created users can not navigate to my account security section. Something went wrong

Describe the issue:
Created users can not navigate to my account security section. Something went wrong

How to reproduce:

Login to management console
Create user
Login to my account
Navigate to security page

Expected behavior:
User should be able to navigate to the security page successfully.
Product version: https://wso2.org/jenkins/view/products/job/products/job/product-is/3735/

This issue is not reproducible after the identity-apps version 1.2.621


[otbn] Add support for CSRs/WSRs to ISS

(EDIT: The commit message changed dramatically. This is the new one)
This gets rid of the SingleRegister dictionary that was defining MOD
and ACC and defines a new class (WSRFile) to hold the values. These
are stored internally as unsigned integers, but that's hidden: you
have to access them via functions like read_signed() or
write_unsigned().
We also move the flags inside the CSRs. I'm not completely convinced
this is the right way to do it: alternatively, you could have the
flags sitting inside the state and pass them (as well as the WSRs) to
the CSR functions. If we decide to do that, it should be reasonably
easy to change back. The flags code also gets moved into its own
file (to break a circular include dependency).
There's a certain amount of noise in insns.py from updating how we get
to the flags and how MOD/ACC work. Incidentally, I think the previous
code was probably wrong for MOD as it treated it as a signed number.
While touching this, I also got rid of some bogus assertions (which
only pass because 1 == True in Python: aargh!) and added explicit int
casts to go between bool and int objects.
Finally, this gets the CSRRS/CSRRW instructions working (and, of
course, ports the wide equivalents to the new framework).

Erk. Of course, I didn't actually wire up the instructions. I'll do that in a sec and force-push.

Right, I've just pushed up an amended version that actually does the instructions. I wouldn't be surprised if there are still problems with it: @GregAC, I'm going to grab your ALU/CSR RTL code and check the new smoke test passes with the results.

Eugh. There are also problems with e.g. BN.AND. I'll fix them up in a separate PR.

Force-push rebases and sets RND to always evaluate to 1 for now.

Thanks for your comments. Sounds like a plan, let's see how it works out.


added new helper function: getStatusCode

resolves #94

Codecov Report

Merging #95 (1f21f24) into develop (3c6c04f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##           develop       #95   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           21        22    +1
  Lines          242       255   +13
  Branches        11        15    +4
=========================================
+ Hits           242       255   +13

Impacted Files
Coverage Δ

src/helpers/getStatusCode.ts
100.00% <100.00%> (ø)

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 3c6c04f...1f21f24. Read the comment docs.


When using group layer Manager is showing error that Map does not have any editable layer but the layers are editable

When using group layer, Manager is showing error that Map does not have any editable layer but the layers are editable
Test map: https://sg-em-devext.mapsdevext.arcgis.com/home/item.html?id=92a2b0c8b3f9467aa868925428e3ec87
Test service : https://sg-em-devext.mapsdevext.arcgis.com/home/item.html?id=be53bed2c5e3459cba7c30bb9b7c93ba&view=service

@sarahmcdonald7

@kellyhutchins

@sarahmcdonald7  can you create an issue for me in the config panel repo and I'll fix this one.

@kellyhutchins yep! I will get something logged

@azizaparveen Can you share the map/layer that you mentioned in the issue?

@sarahmcdonald7 the map is publicly shared now, but it is in one of our org, will this work?  https://sg-em-devext.mapsdevext.arcgis.com/home/item.html?id=92a2b0c8b3f9467aa868925428e3ec87

@azizaparveen Thanks, I actually think I am good. I thought the issue was with all group layers but was confused because all the maps I tested with yesterday used group layers. But I found that the error only appears when maps use group layers that were created in Map viewer. All the maps I was using yesterday had "group layers" because the feature layers were in a feature service.

Closing this since this needs to be fixed at the template and tracking it in https://devtopia.esri.com/WebGIS/arcgis-template-configuration/issues/4058


Automatically show the hidden dock

Describe the bug
If the dock is hidden, make it show with tags then hide after users trigger or dismiss.

v1.2.0


Renamed references to puppetlabs to voxpupuli rabbitmq #844

Pull Request Checklist
Renamed references from puppetlabs/rabbitmq to puppet/rabbitmq
The module is basically the same, just changed ownership.
Going to make some tests around it before removing the WIP:
Description

Related Issue

Fixes # .
Motivation and Context

How Has This Been Tested?

General

[ ] Update README.md with any necessary configuration snippets

[ ] New parameters are documented

[ ] New parameters have tests

[ ] Tests pass - bundle exec rake validate lint spec

Didn't find other references to puppetabs-rabbit or problems in working with puppet-rabbitmq.
Clear to merge for me.

in v2.41.0


Some text disappeared in this PDF.

Attach (recommended) or Link to PDF file here:
1.pdf.lin (1).pdf
Configuration:

Web browser and its version: FireFox 84.0
Operating system and its version: macOS 10.15.7
PDF.js version: 2.7.262
Is a browser extension: No

Steps to reproduce the problem:

Open the PDF with pdf.js.
Found the text of this PDF disappeared.

What is the expected behavior? (add screenshot)

What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Your PDF document is corrupt, since one of the fonts is missing necessary data, as indicated by the errors in your screen-shot.
While we perhaps should handle the actual error better, please note that Adobe Reader (i.e. the PDF reference implementation) renders that document that same way as the PDF.js library.

Your PDF document is corrupt, since one of the fonts is missing necessary data, as indicated by the errors in your screen-shot.
While we perhaps should handle the actual error better, please note that Adobe Reader (i.e. the PDF reference implementation) renders that document that same way as the PDF.js library.


docs(v2): add docusaurus-prince-pdf to resources

Motivation
I found several PDF generation tools like mr-pdf can generate PDF for Docusaurus sites. But the output doesn't in good quality with some glitches:

Doesn't work well with system Dark Mode. You will get a dark background in generated PDF. But it's not an issue in Ci/CD environments
No Table of Contents support
Based on Puppeteer make the text cannot be copied or searched correctly
Link anchors (links start with #) not well handled

So I created a separate tool to generate PDF using Prince.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
No tests required
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)
No related PRs

thanks 👍  that would be interesting to display an example output of your project, maybe you could add a generated pdf link in your doc, using the init template of Docusaurus, or at least a few screenshots?

@slorber A screenshot is not included in readme. I also create a GitHub Action to automatically generate PDFs for docusaurus.io. You can grab the artifacts in each run.

thanks, that looks great :)


Handshapes: add detail view

Create a detail view with edit option for editing all handshape features

Looks good! One request: to leave the finger Selection and Finger Configuration "1" unnumbered, and consistently add "2" as a suffix for the second pair; in the screen shots it's a bit mixed.

https://signbank.science.ru.nl//dictionary/handshape/4/
https://signbank.science.ru.nl//dictionary/handshape/29/
https://signbank.science.ru.nl//dictionary/handshape/80/

@susanodd , could you move the field "Finger configuration 2" down, to be below "Finger selection 2"?

Fixed. (Pushed to github master branch.)


Youtube-dl no longer downloads from videa.hu

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.07.28
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'-F', u'https://videa.hu/videok/zene/operahaz-image-film-werkfilm-CyLClwXXHbfnDxfW']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.07.28
[debug] Python version 2.7.17 (CPython) - Linux-4.15.0-112-generic-x86_64-with-Ubuntu-18.04-bionic
[debug] exe versions: ffmpeg 3.4.8, ffprobe 3.4.8
[debug] Proxy map: {}
[Videa] CyLClwXXHbfnDxfW: Downloading XML
ERROR: Unable to download XML: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 627, in _request_webpage
    return self._downloader.urlopen(url_or_request)
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 2238, in urlopen
    return self._opener.open(req, timeout=self._socket_timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 467, in error
    result = self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 654, in http_error_302
    return self.parent.open(new, timeout=req.timeout)
  File "/usr/lib/python2.7/urllib2.py", line 435, in open
    response = meth(req, response)
  File "/usr/lib/python2.7/urllib2.py", line 548, in http_response
    'http', request, response, code, msg, hdrs)
  File "/usr/lib/python2.7/urllib2.py", line 473, in error
    return self._call_chain(*args)
  File "/usr/lib/python2.7/urllib2.py", line 407, in _call_chain
    result = func(*args)
  File "/usr/lib/python2.7/urllib2.py", line 556, in http_error_default
    raise HTTPError(req.get_full_url(), code, msg, hdrs, fp)

Description

The program worked previously.

Duplicate of #25973.


[VCoDeS] - Windows: File System Server Connector

actionAfterProcess:"NONE" works fine for windows but MOVE and DELETE doesn't work properly. May be because of the path, not as in the LINUX environment.

Created https://github.com/ballerinalang/connector-file/issues/8 to track this


General Question on updates and modules

Hi,
I have the greek translation ready to submit, but I am not sure. What is the process for providing this to you to incorporate into your master branch? (Not a github expert by any means)
Additionally. I have written a Flickr module, how can that be added to the list of modules available?
[MMM-Flickr] (https://github.com/kapsolas/MMM-Flickr)
Thanks!

Please sign up to the forum at https://forum.magicmirror.builders/ and post your questions there for more help. In the mean time, here's a run-down of what you need to do:
First put your code up on GitHub, then submit a PR request for the translation file against the develop branch. Do it in steps (and ask help along the way):

Fork the main MagicMirror² project into your own account
Push your changed/added files to the 'develop' branch
Submit a PR request against the 'develop' branch

If all that sounded foreign, well it kinda is. Github has a learning curve. But on this page that you are reading, at the top on the right, you should see a button that says 'Fork' with a number behind it. Click on it. That will fork the project into your own account.
Once forked into your account, change the branch to 'develop'. Look at the top of this page, right below your username/MagicMirror, and you should see a menu, 'Code, Issues, Pull requests, etc., etc'. Make sure the 'Code' one is selected. A little below that, there's a button that read 'Branch: master'. Click on it and select the 'develop' one.
Now you can use Github's editor and manually post and commit your changed files into the branch, or you can use a facility such as 'git' on your own computer to clone the whole thing to your computer, add/change the files there, then push it all back up to Github. Either way, this is a more involved process and I recommend you research it some more using Github's guides at https://guides.github.com/activities/hello-world/ (main page is at http://guides.github.com/).
Once your changed code is on Github, you can create a pull-request (PR) against the same 'develop' branch and submit it.
For the Flickr module you create, you need to create your own Github repository and post it on the forum so others can find it/talk about it/etc., etc.
Hope this helps.

I did not have any success logging into the forum. I kept getting an error that said invalid session no matter what I did, thus I decided to post here.
The link to the module is already on my github. (MMM-Flickr)[https://github.com/kapsolas/MMM-Flickr]

Hi @kapsolas. That login issue should be solved now. @paviro made some changes to the forum earlier this week. You might need to to remote your cookies.

If you have any additional questions, feel free to ask. I'll close this issue for now.


For consistency - python sdk marked with

Python sdk twice mentioned, but only once marked that it doesn't support policies (Reliability not yet implemented)
Fixed inconsistency.

@SychevIgor : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@JimacoMS3 Can you take a look at this? First, does it use the right version of the Python SDK, and it the added comment accurate? (I don't remember seeing that anywhere else.) Thanks. -- @robinsh

@Robinsh -- looks good to me! @SychevIgor thanks for your contribution!


NormalMap with compressed texture

Hello, it's a shame that we can't use a normal map file with the new compressed texture API, it makes the API unusable for a lot of projects and so the framework. Thanks to think about it.

This is now supported via compressed textures + multi atlases.
I genuinely think "unusable for a lot of projects" is completely overselling it, though! I bet hardly anyone uses this.


Move CanIsolate CompletableFuture instance to 2.12+ only

There might be also a flaky CompletableFuture test, I'll fix it as well

LGTM 👍

I think CompletableFutureLocalSuite is failing because sometimes f doesn't get isolated reference before it is scheduled.
I will try to figure it out tomorrow but I am not familiar with CompletableFuture and why would it happen so if you guys have any ideas then go ahead :D

I vaguely remember that pool management in CompletableFuture is worse than it is in our Scala one, and it often falls back to its default global pool. Maybe we can't really support it, then.

@oleg-py @alexandru

I vaguely remember that pool management in CompletableFuture is worse than it is in our Scala one, and it often falls back to its default global pool. Maybe we can't really support it, then.

Yup, I have been playing around and if we use thenCompose (equivalent of flatMap) it pretty much always uses ForkJoinPool.commonPool-worker-0 . Maybe it is still possible to support it but I don't think there is a lot of interest for it and it doesn't seem worth the effort right now. What do you think about removing the support in this PR? We can always revisit it later.

Yeah, let's do that.

I vaguely remember that pool management in CompletableFuture is worse than it is in our Scala one, and it often falls back to its default global pool. Maybe we can't really support it, then.

Wait, what do you mean?
Is it our fault? Are we using global somewhere where we shouldn't?

Is it our fault?
No, Java 8 CompletableFuture uses it's own "global" (ForkJoinPool.commonPool()), and it's hard-coded in methods since Java doesn't have implicits - unless you pass your own on almost every method call, it will reschedule on that commonPool

Also note that thenComposeAsync doesn't have a variant which takes a thread pool (at least to my very limited knowledge)

I pushed a change to remove it so we can easily merge it if we decide to go this way

Yeah, I don't think we can or even should support anything that bypasses TracingScheduler.

👍 yes, screw it.


Obsolete function resolution_matrix

What and Why
The function resolution_matrix is obsolete. It also appears in the documentation and the tests.
I propose to remove it.
Quote from @abelsiqueira
"I am not sure because who knows what will be required to achieve the speed up required for the project.
That being said, with proper commit naming, it should be easy to find it back if we remove it, so it is fine to remove it, in my opinion"
Possible Drawbacks
No response
Related Issues
No response

Go for it!


Add children per promise

Hi, I'm new in Ember, and I have a little problem.
I'm trying to create a tree, that on opening the node I will send request to the server, to get it's children.
I don't want to load whole tree in the beginning.
How can I do it?>
When I tried to do it, but what when I click on some node, I get response from server (children), but then whole the tree becomes only the tree of children
Sorry, if it is not the right place.

I would run the demo. There's an example of lazy-loading:
Demo
Both dynamic (AJAX loaded) and static examples are in the dummy demo.
Clone this repo: git clone
Install packages: npm install then bower install
Run ember serve
Visit the sample app at http://localhost:4200.

The thing is that I'm trying to get children of specific node, when I expand this node.
And to get these children I call "store.queryRecord".
Now I have 2 problems:

I want call "store.queryRecord" on the event of expand of the node. What event should I use?
I still don't realize how do I use promise(thats' what returns from "store.queryRecord", not the real data)

Or maybe you can tell me how I can override toggle_node method?

Never mind, I found what was my problem.
Thanks

What was your problem?

Wrong usage of promise... like I said I'm new in Ember

@AlexLvovsky How you solved the problem. what was logic which you used?

@PankajWorks, actually I don't remember. And I don't have an access to this project any more.
Sorry


Add missing spaces in README.md

In the Special Thanks part, there is a space between the session and the mention, but only the last API Server session does not contain a space between the mentions. Added missing whitespace for document readability and unifying style.
Screenshot

Before
After

👀


Error with setdisplaymessage function and getdisplaymessage function

Request function in CSMS
async def request_set_display_messages(self):
request = call.SetDisplayMessagePayload(
message={
"id": 380,
"priority": "InFront",
"message": {
"format": "HTML",
"content": "string",
"language": "C"
}
}
)
response = await self.call(request)
Response Function in CS
@on('SetDisplayMessages')
async def response_set_display_messages(self, message):
    return call_result.SetDisplayMessagePayload(
        status="Accepted"
    )

Error
INFO:ocpp:CP_1: receive message [2,"3bf4cfff-b791-48ff-be94-f6c6c4fff8f7","SetDisplayMessage",{"message":{"id":380,"priority":"InFront","message":{"format":"HTML","content":"string","language":"C"}}}]
Traceback (most recent call last):
File "/usr/local/lib/python3.9/dist-packages/ocpp/charge_point.py", line 162, in _handle_call
handlers = self.route_map[msg.action]
KeyError: 'SetDisplayMessage'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/ruban/Desktop/OCPP-implementation-tracker/cs_clean.py", line 5226, in
asyncio.run(main())
File "/usr/lib/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
return future.result()
File "/home/ruban/Desktop/OCPP-implementation-tracker/cs_clean.py", line 5213, in main
await asyncio.gather(cp.start(), cp.request_sign_certificate(), cp.request_security_event_notification(),
File "/usr/local/lib/python3.9/dist-packages/ocpp/charge_point.py", line 126, in start
await self.route_message(message)
File "/usr/local/lib/python3.9/dist-packages/ocpp/charge_point.py", line 144, in route_message
await self._handle_call(msg)
File "/usr/local/lib/python3.9/dist-packages/ocpp/charge_point.py", line 164, in _handle_call
raise NotImplementedError(f"No handler for '{msg.action}' "
ocpp.exceptions.NotImplementedError: NotImplementedError: No handler for 'SetDisplayMessage' registered., {}
Process finished with exit code 1

Hi @Ruban-A , it seems the problem you are experiencing is due to a misspell of the Action to route. You should use the word "SetDisplayMessage" and not "SetDisplayMessages". You can use the official and tested enums we have in our repo, to avoid this problems: https://github.com/mobilityhouse/ocpp/blob/a6f57307d2a8a1c8f85f802742b5855ccf48756d/ocpp/v201/enums.py#L57
So, in your case, would be:
from ocpp.v201.enums import Action
@on(Action.SetDisplayMessage)
async def response_set_display_messages(self, message):
    return call_result.SetDisplayMessagePayload(
        status="Accepted"
    )

Thanks i figured it out . I added s  @on('SetDisplayMessages') , i cleared the error by removing s @on('SetDisplayMessage'). Now its working Fine

Thanks i figured it out . I added s @on('SetDisplayMessages') , i cleared the error by removing s @on('SetDisplayMessage'). Now its working Fine

Great! But as I said, is easier and faster if you use the enums. I realise that we dont have an example with it in our docs and we should. I will take care of opening a PR to solve that.


Issue with the order of as: and default: options on param declarations

Am fairly new to Grape, so apologies if this has come up elsewhere already but doing a quick scan I couldn't see anything covering this issue I was having with using both as and default param options.
If I use the following to set an endDate param with a default value and convert it to end_date then the default value behaviour works as expected.
params do
  optional :endDate, type: Date, default: (Date.current + 50.days).end_of_month, as: :end_date
end

Would result in the default param value being set when no param was sent:
(byebug) params
{"end_date"=>Wed, 30 Sep 2020}

However if I use following for the same request, and specify the as option first, the default value is never set on the params, even when using declared (although not sure this would make a difference anyway).
params do
  optional :endDate, as: :end_date, type: Date, default: (Date.current + 50.days).end_of_month
end

Would result in:
(byebug) params
{}
(byebug) declared(params)
{"end_date"=>nil}

I'm assuming this might be a bug with how the param options are being parsed by Grape but wanted to also make sure I wasn't doing something dumb.

As a side note, if there is a better way to convert all incoming camelcase params to snake case without having to use the as option when declaring a param I'd love to know 👍

This looks like a possible bug indeed, try turning it into a spec?
Regarding camelcase, I would try swapping the parameter builder to a Hashie extension that underscores all keys. I believe Rash does that, but it's built on top of Mash and I wouldn't use that because that's crazy. My recommendation would be to write a Hashie extension that only does that. I'd be interested in seeing the results, including both a contribution to hashie and documentation in Grape/blog post for how to accomplish this.
Btw, you probably want a lambda to evaluate on every request, default: -> { (Date.current + 50.days).end_of_month }.

Hey @dblock, I can certainly try putting a spec together for this when I get some time to, same as the Hashie extension.
Regarding the default: -> { (Date.current + 50.days).end_of_month }, I had mine set as this originally, but this produced "default": { } in the Swagger JSON so I thought I had set it up wrong and removed the lambda.
The Swagger JSON looked like:
{
  "in": "query",
  "name": "endDate",
  "description": "End date of query",
  "type": "string",
  "format": "date",
  "default": { },
  "required": false
}

If this is another bug I can put this in another ticket if that helps keep things separated?

That's a grape-swagger issue, so you should probably open a bug there. That said, it's not supposed to interpret a dynamic default value into some kind of documentation. Not sure what one would expect here.
For your API, I suspect you don't want Date.current to be some value every time your server starts/grape loads, but actually change on every request, hence you need a ->.


[Windows] Update Kotlin installer hash check

Description
Kotlin developers changed method of publishing hashes for installer archive.
Related issue:

https://github.com/actions/runner-images/issues/9466

Check list

[x] Related issue / work item is attached
[ ] Tests are written (if applicable)
[ ] Documentation is updated (if applicable)
[ ] Changes are tested and related VM images are successfully generated

/azp run windows2019,windows2022

Hey @robertreemsrivm!
Yep, it should resolve Kotlin hash related build issue on Windows. I hope so at least. 😄


Proposed implementation of an Annotation Plottable

Reference to issue  #3356
Outstanding issues:

Currently missing the MarginX/MarginY implementation
Needs firming up on the default formatting
Needs firming up the default (minimum) offset from respective axes to ensure uniform and predictable spacing
Additional tests?
Example code/contrib. to the Cookbook

Thanks so much @dlampa, this looks great! I'm going to merge this now but feel free to open a new PR if you have ideas for refinements.
I moved some stuff around to simplify the code in this plottable (and future plottables as well). I'm excited to have this in the next release! 🚀


Asset URL generation causing DB writes on the front-end

I was not sure where exactly to report this as I noticed it coming via Retour & SEOMatic, but the common element is the Vite plugin, so here we are!
The bug
On the front-end of a site I have I am seeing DB writes like this:
INSERT INTO `resourcepaths` (`hash`, `path`) VALUES ('7facf4f9', '@nystudio107/retour/web/assets/dist') ON DUPLICATE KEY UPDATE `path`=VALUES(`path`)

INSERT INTO `resourcepaths` (`hash`, `path`) VALUES ('8dc133d6', '@nystudio107/seomatic/web/assets/dist') ON DUPLICATE KEY UPDATE `path`=VALUES(`path`)

Going up the stack, VitePluginService::init() is the common element, specifically, here:
https://github.com/nystudio107/craft-plugin-vite/blob/4ff8dfabd51663fcb081f092eeeba1ed2ed88a50/src/services/VitePluginService.php#L67
This will eventually trigger the DB writes.
Background
The "why" for this is that I am trying to reduce the number of DB writes on the front-end of our sites, as due to the fix for this https://github.com/craftcms/cms/issues/12557 long-ish running db backups can block the front-end if there are writes during the backup.
To reproduce

On a site using SEOMatic or Retour (probably others that use the Vite plugin too)
Enable Debug toolbar
Load a front-end URL
Notice DB write queries on every request

Expected behaviour
Some mechanism to prevent the Vite plugin asking Craft for a published asset URL when no Vite plugin functionality is used on the front-end of a site. This could be a setting, config variable or potentially moving the relevant code out of VitePluginService::init().
Versions

Plugin version: 4.0.10
Craft version: 4.8.9

hmmmm. Yeah seems to me that for frontend requests, we could just skip this entirely, since it'll never be used

Addressed in the above commits.
I just cut a release version too, lmk how you go. Just do:
composer update nystudio107/craft-plugin-vite

...and that should do it

In the interests of an update: This seems to have done the trick. Thank you.


There are no typings for the typescript

TS7016: Could not find a declaration file for module 'react-stomp-hooks'

Thanks for bringing this to my attention. I will create the typings and package them with the library in a few days.

I have just released a typescript rewrite of the library. Let me know if you need anything else.


Which version is the last?

I ask because:

if I do get github.com/xtaci/smux/ (without initialized modules) go will download the version 1.5.10
if I do get github.com/xtaci/smux/ with initialized modules go will download version 2.0.1+incompatible

I'm using modules I realized that I am still using version 2.0.1. So I want to update the version. Which one should I use? (1.5.10 or 2.1.0?)

1.5.10 , all features in 2.x.x branch have been merged into 1.x.x

Thank you for the information. It is safe to change version from 2.0.1 to 1.5.10 or should I expect problems and major changes in the code?

it's safe!

Great :)

just remember to set the Version in smux.Config，if you are using version:2


[Cherry-pick-204][API]fix releaseWorkflowAndSchedule api and relation binding api

pick #8464

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.0.4-prepare@3158b2c). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             2.0.4-prepare    #8466   +/-   ##
================================================
  Coverage                 ?   21.03%
  Complexity               ?     1586
================================================
  Files                    ?      536
  Lines                    ?    22491
  Branches                 ?     2664
================================================
  Hits                     ?     4732
  Misses                   ?    17274
  Partials                 ?      485

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 3158b2c...c18ec48. Read the comment docs.


Add package: ncdu

The C-based ncdu version distributed via entware has stalled for months. This PR brings the latest fixes and optimizations of the LTS version to toltec. Tested on rM1 and rM2.

Do any of you have an idea, why the package only falls back to 1.17-1 after manually installing this build? Maybe we can fix something here, or the most recent version might be available in a bit anyway.

It appears that in-between my submitting this PR and your review, Entware actually updated ncdu (along with many other packages). I just confirmed this by running the usual opkg update and opkg upgrade.

Would it be likely that the 1.X series will get updated in the future given that it seems to be pretty separate to the new 2.X versions?

It's hard to say, but 1.X series should be usable for a very long time. There are also interesting alternatives inspired by ncdu, such as gdu.

How do they differ and why not go with 2.X?

If couldn't get the Zig-based 2.X series to compile and run on the rM.

Do you have any idea why it might not work for me on the rM 2?

I can't say for sure. I may have customized my device in such a way that it happens to work. I will try on a fresh install next time.

Not sure how to go with this package.

Given the Entware update, there's no point in duplicating efforts. I'll go ahead and close this PR. Thank you for the thorough review!


FW: Remove the old "SANDSTONE_GA" macros

They're unused.

If you wonder, why this PR failed, the commit is not signed-off.
What is more please check out meson_options.txt  We also mention ga and ga-dev
option('flavor', type : 'combo', choices : ['default', 'ga', 'ga-dev'], value : 'default',
    description : 'Select one of the novel build flavors (ga, ga-dev) or the default')


Crash with TFCompat and Mods.

TFC Tech did some change which removed wood dust. since some recipes use it, it crash.

any estimates on how long this will take to fix and is there a workaround other than removing tfc or mekanism?

downgrading tfc to earlier version seemed to work

any estimates on how long this will take to fix and is there a workaround other than removing tfc or mekanism?

already fixed. i just didn't have time to push a fix to curseforge.

will this fix be pushed to curseforge soon?


Refactor JS Code

Refactor JS code for Windows following #2508

Thanks @chiaramooney

Feel free to merge when it's OK on your side @chiaramooney


Certain torrents return "-1" for seeders and leechers.

Always returns -1 for seeders and leechers
magnet:?xt=urn:btih:0a36a61cb46fc1444c95eb62185b777d65362ca9&dn=elementaryos-freya-i386.20150411.iso&tr=https%3A%2F%2Fashrise.com%3A443%2Fphoenix%2Fannounce&tr=http%3A%2F%2Flocal.localtest.me%2Fphoenix%2Fannounce.php

Binary:  6¦´oÁDL•ëb[w}e6,©
Works fine
magnet:?xt=urn:btih:fc85dc999730a42de3924444aadbcfa183b5f388&dn=elementaryos-freya-amd64.20150411.iso&tr=https%3A%2F%2Fashrise.com%3A443%2Fphoenix%2Fannounce&tr=http%3A%2F%2Flocal.localtest.me%2Fphoenix%2Fannounce.php

Binary: ü…Ü™—0¤-ã’DDªÛÏ¡ƒµóˆd

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/25922350-certain-torrents-return-1-for-seeders-and-leechers?utm_campaign=plugin&utm_content=tracker%2F10163092&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F10163092&utm_medium=issues&utm_source=github).

The first one, the one that fails, has what appears to be a line break in it. That seems like it might be an issue.

Found. I will update my fork to send a pull request.


bump go-containerregistry dep

Codecov Report

Merging #404 (2e9a819) into main (8c7b9cb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #404   +/-   ##
=======================================
  Coverage   42.10%   42.10%
=======================================
  Files          35       35
  Lines        1805     1805
=======================================
  Hits          760      760
  Misses        906      906
  Partials      139      139

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 8c7b9cb...2e9a819. Read the comment docs.


Rename the repo

In the README file, the software is named RiverCatch. Even if we were to pretend this name isn't terrible, the repo should really be renamed to reflect the name of the software. The file/variable names could also better reflect your choice in software name.

The README file shows the software has been completely renamed since this issue was raised. RiverAnalyserPRO now needs to be reflected in the repo name.

For renaming the repo, follow the instructions in the link:
https://docs.github.com/en/repositories/creating-and-managing-repositories/renaming-a-repository


Standard Product Check errors

Describe the bug
If a standard product flag is provided as false in a UMM-C record, pyQuARC still produces an error stating that the flag is missing.
To Reproduce
Steps to reproduce the behavior:

Run pyQuARC on sample UMM-C record (C1577484501-LARC_ASDC)
See error even though metadata indicates a value of "false" as the Standard Product flag

Expected behavior
No error should show when a value is provided in the Standard Product field.

Merged and released - closing


Remove Major predicate in favor of empty predicates (fixes #150)

Building will fail until a published version of steveklabnik/semver-parser#29 is available.
Bumped version to 0.9.0 because:

WildcardVersion lost the Major variant.
Ordering of VersionReq is changed for the empty set of predicates which are now wildcards.

Oh. The author needs to push new version of semver-parser

Yes, I've been doing some work and haven't cut a new release yet.

I am not going to be accepting this PR, sorry. There are some breaking changes, and I want to move away from semver-parser entirely anyway. Thank you / sorry


session id changes with every request

Hey, i have been building my api system on top of your boilerplate, which saved me a huge amount of time, but im having trouble after authentication, the AUTH works fine and i get the token in response, but in my API i have set a session variable and its active only for that request , in the consequent request the session is flushed out and i get a different token in response. i have enabled Sessions in API middleware, and and set the session domain.
the session seems to retain when i make the request through postman, but not when i use axios or guzzle. and i can see the session files created in storage folder with different token
array:2 [
  "_token" => "JpRUe19KiVojlXvnahHSgcsJmbI8Cj4zb9FP1kEE"
  "use_company" => 1
]

[
"_token" => "OzfWDpUykwt0OnASz627e7o3C7LtlYgONTs6qqiH"
]

Hi @sunder6218 - REST is supposed to be stateless - it does not make any sense to have a "session" with the consumers of your API, because there is no reliable way to track it, and the whole concept of REST APIs is that they are not meant to have sessions with the client.

Closed due to no response.

Hi @sunder6218 - REST is supposed to be stateless - it does not make any sense to have a "session" with the consumers of your API, because there is no reliable way to track it, and the whole concept of REST APIs is that they are not meant to have sessions with the client.

Thank you , i have changed the approach accordingly to better suit the API.


[Rating] Generate random name if none provided

The Rating component shipped by Material-UI in the "Lab" section has a bug when declared in a file and reuse.
I assigned a prop called "name" which was unique and a function "getRating" which would take value and name as its parameters as,

getRating = (value, name) {
console.log("Value ===>", value, "\n Name ===>", name)
}

I had to use this Rating component five times and give it unique "name" prop and a function to get rating of each. But it behaved by only updating the first rating component only.
I have implemented the same logic for React-Rating Library and it works perfectly.
Kindly, the rating component is a beautiful library of Material-UI but do fix this bug.
Best Regards,

[ * ] The issue is present in the latest release.
[ * ] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

Expected Behavior 🤔

Steps to Reproduce 🕹

Steps:

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v4.?.?

React

Browser

TypeScript

etc.

How can we reproduce the issue?

Hey, can I help with this issue?

@Vitao18 Sure :), I think that a strategy similar to
https://github.com/mui-org/material-ui/blob/ba1f645a2dbed510ca40a53de1707dda78adae53/packages/material-ui-lab/src/useAutocomplete/useAutocomplete.js#L80-L87
could do it


【bug】android.content.res.Resources$NotFoundException: Unable to find resource ID

Version: 10.0.0
what: When use @OnClick int kotlin ，it don't work，but@BindView work。
But if you use both (@OnClick and @BindView)，it work。EveryThing is normal in Java。
Sample:
first:
    @OnClick(R2.id.btn)
    fun confirmBill(view: View) {
       // don't work，and the error is android.content.res.Resources$NotFoundException: Unable to find resource ID
    }

second:
    @BindView(R2.id.btn)
    lateinit var btn: Button

    @OnClick(R2.id.btn)
    fun confirmBill(view: View) {
       //  work
    }

the  ViewBinding file is created. the code same is normal ,but is only use @OnClick ,it will be an error.
eg: kotlin version: 1.4.0

it's happen in library module，ViewBinding use R2.id not R.id，and the are diff


Latest Artix 7 ROI breaks the 018-clbram fuzzer

make[2]: Entering directory '/big/github/SymbiFlow/prjxray/fuzzers/018-clbram'
bash generate.sh specimen_001
+ source /big/github/SymbiFlow/prjxray/utils/genheader.sh
++ '[' -z artix7 ']'
++ set -ex
++ test 1 = 1
++ test '!' -e specimen_001
++ mkdir specimen_001
++ cd specimen_001
+ python3 ../top.py
Traceback (most recent call last):
  File "../top.py", line 108, in <module>
    loc = next(slices)
  File "../top.py", line 58, in gen_slicems
    assert ((12, 28) == SLICEX)
AssertionError
Makefile:15: recipe for target 'specimen_001/OK' failed
make[2]: *** [specimen_001/OK] Error 1
make[2]: Leaving directory '/big/github/SymbiFlow/prjxray/fuzzers/018-clbram'
Makefile:19: recipe for target 'run' failed
make[1]: *** [run] Error 2
make[1]: Leaving directory '/big/github/SymbiFlow/prjxray/fuzzers/018-clbram'
Makefile:19: recipe for target '018-clbram/run.ok' failed
make: *** [018-clbram/run.ok] Error 2

Maybe;
diff --git a/fuzzers/018-clbram/top.py b/fuzzers/018-clbram/top.py
index b2cac6c..2024b0f 100644
--- a/fuzzers/018-clbram/top.py
+++ b/fuzzers/018-clbram/top.py
@@ -55,8 +55,8 @@ def gen_slicems():
         x = 6, 8, 10, 12, 14
     '''
     # TODO: generate this from DB
-    assert ((12, 28) == SLICEX)
-    for slicex in (12, 14):
+    #assert ((12, 28) == SLICEX), repr(SLICEX)
+    for slicex in (8, 12, 14):
         for slicey in range(*SLICEY):
             # caller may reject position if needs more room
             #yield ("SLICE_X%dY%d" % (slicex, slicey), (slicex, slicey))

The comment above says;
'''
x
   Always even (ie 100, 102, 104, etc)
   In our ROI
   x = 6, 8, 10, 12, 14
'''

also note : ./019-ndi1mux/top.py:41: assert ((12, 28) == SLICEX)

019 fix: https://github.com/SymbiFlow/prjxray/pull/163

The core issues should be fixed. I'm going to also replace anywhere that has slice_xy(), although I think because most of these use regular expressions, they shouldn't be affected by the addition of the BRAM and DSP strings


chore: ini solution test

Current dependencies on/for this PR:

main

PR #165   👈

This comment was auto-generated by Graphite.


Add project info to "list reports" payload

The project ID and name were included in the response payloads for individual
reports, but not in the payloads for getting lists of reports.

Current dependencies on/for this PR:

main

PR #1606   👈

This stack of pull requests is managed by Graphite.


Fix lqr badarg

Fix for the Travis CI bug identified in PR #242 regarding LQR test.  The problem was that one of the test cases used in tests/matlab_test.py was incorrect.  It used a set of weighting matrices that did not give a positive definite cost function for the LQR problem.  This should have generated a error, but that case is not tested in the code (see issue #252, created to track this).
The fix here is to change the state space weighting matrix (multiplying it by 10) so that the overall weighting function is now positive definite.  In addition, a skipped unit test has been put in with the original test case.
The reason this was generating spurious errors is because the slycot LQR function (sb0md) was creating a closed loop system whose eigenvalues had very small realparts (on order 1e-16) and apparently these were sometimes positive and sometimes negative.
Also fixed a small typo in a docstring while I was at it.

Coverage remained the same at 78.914% when pulling cc4508c82c8b12df74601c13ac0c6cfaf78b4b40 on murrayrm:fix_lqr_badarg into 4b0101cf00a413013f1805d8a331b26e3c96e211 on python-control:master.


Update icon padding

Fixes #52
@ozgunozerk can you check bundles on mac and windows before PR?

macOS icon is still a bit big, I believe the linked issue is not resolved yet.
I will try on windows too

This is the dock (without hover):

This is the magnification effect on the dock when hovered:

On windows, there are 2 problems:

Orange and green circled icons resolutions are very low. It was better before this PR imo.


gmail login error "This account already exists on your device."

Installed gmail v2<IP_ADDRESS>0951155.release.62129274
A "New in Gmail" screen presented. Press "GOT IT".
Press "Add an email address".
Press "Google"
Press "Sign In" at the microG Services Core screen.
"Just a sec..."
Enter email and press next.
Get the following error: "This account already exists on your device."

I have got solution for google after 3 weeks brain storming and spent all day and night time to resolve the issue
any one need solution can contact on wire @abampalxender

I fixed it by installing Android 10 and system-MinMicroG-Standard-2.7-20200727053128.zip.


"Error: 'error' is read-only" in Firefox web extension content script

Environment
## System:
 - OS: macOS 13.5
 - CPU: (8) arm64 Apple M1 Pro
 - Memory: 1.14 GB / 16.00 GB
 - Shell: 5.9 - /bin/zsh
## Binaries:
 - Node: 18.17.0 - ~/.asdf/installs/nodejs/18.17.0/bin/node
 - Yarn: 1.22.19 - ~/Library/pnpm/yarn
 - npm: 9.6.7 - ~/.asdf/plugins/nodejs/shims/npm
 - pnpm: 8.6.12 - ~/Library/pnpm/pnpm
## npmPackages:
 - styled-components: ^6.0.7 => 6.0.7

Reproduction
Steps to reproduce

Use styled-components within a Firefox web extension content script.
Ensure that process.env.NODE_ENV in the bundle is not set to production.
Load the web extension content script in Firefox.

Expected Behavior
styled-components should work properly within a Firefox web extension content script.
Actual Behavior
When process.env.NODE_ENV is not set to production, styled-components attempts to modify console.error, leading to the Error: "error" is read-only error.
The issue can be traced to this line of source code: https://github.com/styled-components/styled-components/blob/0a27d797c018075b9f2c90e573c9a37fb7a80196/packages/styled-components/src/utils/checkDynamicCreation.ts#L21

Related: https://github.com/styled-components/styled-components/issues/3695


Return value from hydrate not used in EloquentController

Is it supposed that the return value isn't used in EloquentController->hydrate()?
See: https://github.com/cloudcreativity/laravel-json-api/blob/master/src/Http/Controllers/EloquentController.php#L233

Good spot! The hydrator interface specifies that the object should be returned, so the Eloquent controller should be using the return value.
Thanks for the PR - I'll be able to merge and tag it later this evening (UK time)


[SecurityFix] Updating alibaba fastjson to take care of CVE-2022-25845

What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)
[ ] I have added documentation for my new feature or new function

@gdrossi46 Can you sign off this PR by git commit -s?

@gdrossi46 LTGM overall. As alvin said, please sign off the commit

@alvin-celerdata @banmoy Done, I have signed off the commit now.
When will this be merged and a new version released?

@gdrossi46 Version 1.2.10 was just released. While there's no plan yet for the next version release, would it be possible to create a separate package based on the latest code for use?

Yes @banmoy, that's what we have done. We have generated our own package and are using it to unblock ourselves.


[Not For Review]Triage flow aggregator e2e flakes

Codecov Report

Merging #2299 (603d70d) into main (6c350e0) will decrease coverage by 20.54%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #2299       +/-   ##
===========================================
- Coverage   61.77%   41.23%   -20.55%
===========================================
  Files         276      141      -135
  Lines       21342    17408     -3934
===========================================
- Hits        13184     7178     -6006
- Misses       6773     9593     +2820
+ Partials     1385      637      -748

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
41.23% <ø> (ø)

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


Fix flaky tests that defines pk for objects

Fix flaky tests.

drop defining pk in tests

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2852 Flaky tests due to fixtures hardcoded auto-increment primary keys


Can't make it work

Awesome plugin ;-) but unfortunately, it doesn't work on my end.

VSCode version
Version: 1.41.1
Commit: 26076a4de974ead31f97692a0d32f90d735645c0
Date: 2019-12-18T14:57:51.166Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Darwin x64 19.2.0

+1

+1

getting a better error message, but same basic problem:
Command 'Python init Generator: Generate __init__.py' resulted in an error (command 'extension.generateInit' not found)

it it as if the extension activate method is not called

same error on my side :)

i have the same error on my side.
Command 'Python init Generator: Generate init.py' resulted in an error (command 'extension.generateInit' not found)

I have made a fix for this   #3 , hope the maintainer @SeeLog  can have a look soon :) Will be nice to use this extension!

Now, merged #3 and publish this extension. But it still seemed to be occurred this problem.

When debugging, this extension works fine.
But compile and install this extension or install from Market, same issue is occurred.
Does anyone have any idea to solve this problem?

sometimes its nice to find someone else with a similar problem, even if it is not your own: https://github.com/microsoft/vscode/issues/23300
when I download, npm i, and then vsce ls in your repo, I see your node_modules ... but I'm not sure they are in your package. in general, everything looks right now (albeit with old typescript -- I'll submit a pr just to freshen style a bit)

Finally, I might fixed this problem!
The problem was glob package is not included in dependencies.
I have fixed it, also have published!
https://marketplace.visualstudio.com/items?itemName=SeeLog.python-init-generator
I test this extension on MacOS and It works.

I'll close this issue when someone else can confirm it works.

although I submitted the PR so maybe it should not be me, I uninstalled and restarted vscode, then installed 0.0.9 from the extensions tab, and found it is now working

Thank you for your cooperation!
Now, close this issue.


Change Dictionary#get to return null on missing keys instead of throwing an exception

That'd be more aligned with what other collection types do, including Map and Set.

A vote from me on having it not throw an exception, this would simplify a good amount of code in some of my projects 👍


Pipelines: Add page listing modules

Each module says which pipelines use it, but we don't have the inverse.
It'd be nice to have the inverse: a tab on the pipeline pages listing all modules that are used. Can be summary boxes, similar to the main modules listing page.

we are running out of space for tabs on <20" screens imo. how about adding it to the sidebar similar to how it is on the modules pages


Error when entity must be specified as a string

Under some circumstances when exposing an association using an entity, the entity must be specified as a string.  In that case, if grape-swagger-entity is also used, grape-swagger fails with the following error:
undefined method ancestors' for "Permission::WithoutRoleEntity":String /home/ubuntu/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/grape-swagger-0.24.0/lib/grape-swagger/endpoint.rb:275:in block in expose_params_from_model'
Currently fixed locally with a monkey patch to the expose_params_from_model method:
def expose_params_from_model(model)
  model = model.constantize if model.is_a? String
  model_name =model_name(model)
  .
  .
  .

@dmazan … please can you provide some more informations, a PR would be best including you fix …

seems realted to: https://github.com/ruby-grape/grape-swagger/issues/427

Should be fixed with: https://github.com/ruby-grape/grape-swagger/pull/527

@dmazan … thanks for the fix


s-listen only updates once

I have a Commerce site with a cart count in the header.
_partials/store/basketCount:
{% set order = craft.orders.id(orderId).one %}

{% if not order %}
    {% exit 500 %}
{% endif %}

    Cart count = {{ order.totalQty }}

_partials/store/basket:

    {# functioning commerce cart form updating as expected with sprig #}

Layout:
{{ sprig('_partials/store/basketCount', { orderId: cart.id }, {'s-listen': '#basket'}) }}

When the cart (#basket) is first updated, the listening component updates. On subsequent updates, no update occurs.

Craft v4.2.2
Sprig v2.1.0

You should be listening on the entire basket component, rather than a specific div inside it, which will be overwritten on each cart update. So putting the ID on the component (instead of inside the component) should solve this for you.
{{ sprig('_partials/store/basket', { }, { 'id': 'basket' }) }}

Makes sense - thanks.

@bencroker should this listen to multiple components? For example:
More than one of these:
{{ sprig('_partials/store/basket', { }, { 'class': 'add-to-basket' }) }}
Listened to by this:
{{ sprig('_partials/store/basketCount', { orderId: cart.id }, {'s-listen': '.add-to-basket'}) }}


how to define customer view(can change any position in the main screen )

hi cruffenach:
how can i setting customer view in this plugin

Hello Cruffenach,
How can I present Toast  from bottom of the screen.

Hi @kmuddapu, toasts from the bottom are not supported in this library, this library mainly deals with animations involving the status and navigation bars at the top of the screen.


Return wonky expression coersion-ishness to the way it was before

@cjllanwarne @mcovarr
This is the change we discussed the other night

:+1:

:+1: with maybe a new ticket to validate other known options too?

Going directly to the new repo


script to show test coverage for files that changed in the last month

Admittedly, I like writing tests and I think writing or fixing a few tests is a good way to get to know a codebase. So I've been poking around the failing tests in Teiserver in order to learn my way around the codebase.
I realized, though, that another way to think about the problem of improving test coverage (without boiling the ocean) is to focus on the test coverage of recently changed files.
So, as an experiment, I wrote up this script to filter "test coverage per-file" with "only files changed in the last month". I think I will find it useful. Maybe it will be useful to others.
Example output:
...(end of actual test run output)...
[os_mon] cpu supervisor port (cpu_sup): Erlang has closed
[os_mon] memory supervisor port (memsup): Erlang has closed
COV    FILE                                                                                LINES RELEVANT   MISSED
  2.4% lib/teiserver_web/live/battles/match/show.ex                                                              428      125      122
 48.6% lib/teiserver/autohost/tachyon_handler.ex                                                                 192       37       19
 69.6% lib/teiserver/tachyon/transport.ex                                                                        238       69       21
 86.6% lib/teiserver/player/tachyon_handler.ex                                                                   249       67        9
100.0% lib/teiserver/tachyon/handler.ex                                                                           76        0        0

I think this script could be used in place of just these lines in the CI. Maybe in another PR if you can't be bothered, but I'm a bit worried that these kind of scripts are barely ran, so having it run automatically would be nice.
Bonus point if the github action could report/comment on the PR, but I'm really not too fussed about that, may be too noisy.

I just tried it out (setting set -euxo pipefail so the script fails rather than swallowing errors). However, it seemed to also fail on warnings when I set that, so perhaps we'd have to expand the GitHub actions to do all the same thing this script does.

No, mix test doesn't fail on warning. The error you got was likely insufficient coverage:
FAILED: Expected minimum coverage of 80%, got 18.5%.

that isn't shown because of the grep. I don't know how to configure that, but that should be possible. Also, we could not care about this return code, we do run the test suite in another github action anyway.

No, mix test doesn't fail on warning. The error you got was likely insufficient coverage:
FAILED: Expected minimum coverage of 80%, got 18.5%.

that isn't shown because of the grep. I don't know how to configure that, but that should be possible. Also, we could not care about this return code, we do run the test suite in another github action anyway.

Yeah, that was my conclusion as well. (maybe I should just figure out how to set the test coverage requirements instead of the following hack:  I've re-arranged the sequence so that it runs the tests twice, once failing on error, and the second time using it to generate test coverage... )
However, I have not yet figured out why the GitHub action only goes back a few seconds -- I thought I had changed the action to do a deep clone rather than a shallow clone...
Run git diff b35d2d5fec7bfec473fe988937bb67672c7c124b '@{last month}' --name-only | tee ./cover/files_changed_in_last_month.txt
warning: log for 'HEAD' only goes back to Wed, 11 Dec 2024 03:56:03 +0000

I'm a little unsure if this simple report will even be used. It seemed possibly useful as a ad-hoc script but how often will this convince a developer to go back and actually improve test coverage?

I didn't find a way to set custom minimum coverage, although I admit I haven't searched for very long.
what you could do is set +e before running the coverage test so you effectively ignore the non 0 exit code?
I quite like the output of your script really, I think that would be a nice addition to the CI. But if that proves too much of a hassle, having the script readily available + a mention in the readme would already be quite nice really.

a way to set custom minimum coverage

I did find it, it's in coveralls.json. I adjusted it down to match the current test coverage level. (33.4% as of this writing. Obviously, adjust it whenever you need to.)
I added a blurb about it in the documentation.

I think I got it working. Of course, the tests failed, but had they succeeded, we might have seen a separate coverage report get generated.


Increased max diff limit from 100000 to 150000

See #198

Thanks @k1o0 🙏 I've deployed this change, I'm going to leave it going for a little bit and see if there are any significant performance issues before merging (probably a day or two).


Remove assignee_but_unconfirmed

Coverage increased (+0.06%) to 25.49% when pulling 8059fb89fb43fb85c0ebbd7f2632f01cd5090a85 on calixteman:rm_assigned into 71acf1d6df315770676d1205f51f9c3e7a44ae9e on mozilla:master.


Analytics - Filter out points on Map to display only points in selected Service Area

Steps to reproduce:

Upload service area file in Service Area drawer.
Upload PCPs file in Providers drawer.
Go to Analytics drawer.
Change dropdown from "All service areas" to "Amador/94501"

Expected behavior:
Map zooms to selected service area and displays only points in that service area.
Actual behavior:
Map does not react and all points are displayed.

oops here are the files I used:
service areas: https://drive.google.com/open?id=1U-rOMy3BE9ruPs8kW8pEwUrO5QOxnVEH
PCPs: https://drive.google.com/open?id=1x31zlVpEi6QPC1xQl5XT1SZBzIaUiBJ4


Add homebrew formula

Unfortunately, homebrew's user experience hides the complexity of dealing with different node versions. This is a problem in the context of a modern desktop where developers are likely to deal with multiple versions of node simultaneously. Thus, although homebrew can offer initial ease of installation, the setup can easily break later on without any obvious clues as to the culprit. Thus, homebrew does not seem to offer any real advantages in the long run.


Invalid number of channels

File "C:\Users\chibi\Desktop\Nero\main.py", line 25, in
main()
File "C:\Users\chibi\Desktop\Nero\main.py", line 6, in main
waifu.initialize(user_input_service='whisper',
File "C:\Users\chibi\Desktop\Nero\waifu.py", line 46, in initialize
self.update_tts(service=tts_service, output_device=output_device, voice=tts_voice, model=tts_model)
File "C:\Users\chibi\Desktop\Nero\waifu.py", line 99, in update_tts
sd.check_output_settings(output_device)
File "C:\Python\lib\site-packages\sounddevice.py", line 697, in check_output_settings
_check(_lib.Pa_IsFormatSupported(_ffi.NULL, parameters, samplerate))
File "C:\Python\lib\site-packages\sounddevice.py", line 2747, in _check
raise PortAudioError(errormsg, err)
sounddevice.PortAudioError: Invalid number of channels [PaErrorCode -9998]
I'm stumped, I don't even know where to start

Same problem

Thank you for opening the issue and contributing to this project!
I've implemented error handling. Now, before this error message will be a short explanation and device list.
To fix this:

Check the list of devices that print's before an error message
Find the first device that starts with CABLE Input. For me, it's CABLE Input (VB-Audio Virtual C, MME (0 in, 2 out)
Copy ID of this device (it's printed before device name)
Paste it as value of output_device in main.py inside initialize method

If your CABLE Input device has ID 3, update it in initialize method:
waifu.initialize(user_input_service='whisper',
                     stt_duration = None,
                     mic_index = None,

                    chatbot_service='openai',
                    chatbot_model = None,
                    chatbot_temperature = None,
                    personality_file = None,

                    tts_service='elevenlabs',
                    output_device=3,
                    tts_voice='Rebecca - wide emotional range',
                    tts_model = None
                    )

I'm planning to add GUI in the future, so then I'll make a better way to solve it! Thanks for bringing this to my attention!

I am happy to say that it worked, my value was 7, now this is supposed to work with VTubeStudio right? is there any special action I need to take to do that? since currently it's just audio.

Right and does it just record until you say something? I feel like that would fill up storage real quick if it even records silence.
I'm just worried about wasting my tokens.

I'm glad to hear that it fixed your issue!
To make Waifu work, you'll need to set a few settings in VTube Studio. You need to select CABLE Output as microphone, select preview option, and setup Mouth Open setting. You can find more detailed explanation in README file, under 6th list item in "Installation" part, there's Settings: you can click on to reveal screenshots with explanation.
Additionally, you can go through most of parameters and check the "Auto-breath" box to add idle movent for 2D avatar.
As for audio recording, that's a really good question! I'm afraid it really does record silence. Unfortunately, I'll be busy for the next few days on my job. If there's free time to spare I'll look into this issue. But it's more likely that I'll be able to fix it only on Monday

I followed the instructions yet it is still unable to lipsync, also take your time with the audio recording issue most people have a keybind which records your audio, but i am more interested in simply chatting and it automatically picks up on it.

Also another potential issue I found is that while you're using V-CABLE for the output, you cannot hear anything else from your computer other than the AI, no music, no videos. I'm just bringing it into your attention if you weren't aware and I was wondering if there was a possible fix for it.


[Feature] add star mgr s3 region config

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/staros/StarMgrServer.java
1
1
100.00%
[]

run starrocks_admit_test


Hi, is this still maintained this crate?

Hi, is this still maintained this crate?

I don't think so

Thanks mate
On Thu, Jun 3, 2021 at 8:24 PM Félix Lescaudey de Maneville <
@.***> wrote:

I don't think so
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PoiScript/docx-rs/issues/35#issuecomment-853763548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AIKO7IIR64BWVOYWUILXFLTTQ5J6XANCNFSM4WIFWEZA
.

Do you know of any actively maintained alternatives to this crate?

This crate is actively maintained but idk if it has the same features

This crate is actively maintained but idk if it has the same features

Thank you! I had a brief look at this project earlier and I had the same question. Can't tell yet if it has similar features.


When I try to upload and replace a file, there is an error: 409.

When I try to upload and replace a file, there is an error: 409.
The error is only one figure: 409.
Any file will be wrong.
I have just updated the latest version.
The previous version without this error.

Oh! I presume you downloaded via Caddy. I forgot to release the newest release there. It will probably be fixed in ~20 mins.

Ok, already can be used normally!


clinical complaint pipeline refinement

Related issue

Issue Id
Context

List of changes

change a
change b

Database migration/schema changes (if any)

List your migrations here...

Self review checklist

[ ] Confirm that the changes made by you meet the resolution criteria and nothing else.
[ ] Ran all the tests locally to check that you have not introduced any new regression.
[ ] If there are any failing tests, you have already fixed them. If not
[ ] Followed coding and styling guidelines.
[ ] Made sure the changes are is not overly complex, unreadable and unnecessarily smart. Simplicity wins!
[ ] Made sure the comments are not too verbose and not to cryptic.
[ ] Made sure the code metrics are not deteriorated.
[ ] Made sure you have updated (if necessary)-

[ ] Documentation
[ ] Translation
[ ] Tests/test data
[ ] Postman collection
[ ] Version

Additional info (if any)

Add anything you feel worth mentioning...

screensshots (if any)

Original
Updated

** Original screenshot **
** Updated screenshot **

Merged with dev_consolidation


feat: 🎸 Update polkadot-types and docs

Description
Updates types after latest core changes. Fixes all the existing extrinsics and txtags to match the newly created ones.
Breaking Changes
Even though it has some breaking changes, I'm not marking them (since this is not yet live)
JIRA Link
NA
Checklist

[] Updated the Readme.md (if required) ?

:tada: This PR is included in version 24.0.0-confidential-assets.7 :tada:
The release is available on:

npm package (@confidential-assets dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Add Dataset] 💵Indian startup & funding news dataset💰

Description
@nikhil25803 I would like to add this
Screenshots
No response

Go ahead @ighoshsubho :))

Hello 🫱🏼🫲🏼 !! I would like to contribute; please assign it to me !!


[SMALLFIX] Add OBS constants and property keys.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18611/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.cli.GetConfKeyTest.getConfKeyWithAllPropertyNames
Test FAILed.

Merged build finished. Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18612/
Test PASSed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18619/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/18621/
Test PASSed.


Serve Block Producer List

We need to serve a list of block producers to app containing relevant info from their bp.json in order to generate their profile pages.
We want to have job on nodejs that continually aggregates this data on the MongoDB.
Use https://github.com/eoscostarica/eosjs-camel-api for querying the EOS API.

Ref https://developers.eos.io/eosio-nodeos/reference . see get_producers

Ref bp.json standard https://github.com/eosrio/bp-info-standard

@murillojorge @rubenabix the idea is to aggregate all data to display, logo, location and all info we can gather from a BP on their profile page

@rubenabix usage example of EOS Camel API pkg https://github.com/gaboesquivel/eos-actions/blob/master/index.js

Another option re-enabling a unix cron we use to have to generate that list on one of EOS Costa Rica nodes monitor.eosio.cr/mainnet-producers.json .. It's not currently working.  Thoughts @xavier506 ?

Actually I think we don't want to be running a node on Zeit just to able to execute cleos to get the system producerslist

I'd look this
const got = require('got')
const producersList = await got('https://monitor.eosio.cr/mainnet-producers.json')
const getBpJson = (url) => got(url)
const producersData = procudersList.map(bp => getBpJson(bp.bpJsonUrl))
// and finally save the data to MongoDB using the Block Producer mongoose Model

I just talked to @xavier506, we are going re-enable the mainent producers json cron job / service and we'll query that.
No need to run an EOS node on this project. @rubenabix

Para el mainnet-producers.json estoy corriendo el siguiente cron job en el server
docker exec eosio /opt/eosio/bin/cleos -u https://api.eosio.cr --wallet-url http://<IP_ADDRESS>:8888 system listproducers -l 1000 -j > /var/www/monitor.eosio.cr/public_html/mainnet-producers.json

Cool, please let us know when we can start using it.

Here is an initial list of BPs in JSON format
https://github.com/eoscostarica/rate.eoscostarica.io/blob/master/mainnet-producers.json

hey @rubenabix we should be using the On-Chain Data instead.
https://validate.eosnation.io/data/
here you get all BPJSON data from Blockchain.
https://validate.eosnation.io/bps.json .

I think we can just configure demux to listen to eosio.system:regblockproducer and producerjson_contract:set
https://github.com/EOSIO/eos/blob/master/contracts/eosio.system/eosio.system.cpp
https://github.com/greymass/producerjson/blob/master/producerjson.cpp

solved by https://github.com/eoscostarica/eos-rate/commit/4838b60dc652714d1c6b911ea5a7a31e6428cea9


Invalid links to code on GitHub

There are two links in the text: "Once the blob-triggered function is created, use the run.csx and function.proj from GitHub in the Function. " which targets are missing (displaying 404 error page).

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

ID: 959c889e-ce7b-2480-b24d-ad11915a9dc8
Version Independent ID: 69cd330c-9702-3a09-c3df-03e9e1975ec5
Content: Trigger a Batch job using Azure Functions
Content Source: articles/batch/tutorial-batch-functions.md
Service: batch
GitHub Login: @laurenhughes
Microsoft Alias: peshultz

Thanks for the feedback! We are currently investigating and will update you shortly.

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

@laurenhughes Did you get a chance to take a look at this one? The links are throwing 404 error. If you can provide me with correct urls, I can update the document. Thanks. :)

Both links work now, thanks for raising the issue!
#please-close


Added Option of Select hidden element : Link Selector

Have created this PR specific to add SelectHiddenElement Option for LINK Selector, Kindly review and let me know, i will proceed the same for other selectors
Part of - #811

@jayandran-Sampath I have approved the changes. There are some conflicts though, can you try to resolve them?


Updated autoNumeric-min.js

@BobKnothe Updated the minified version.
What will happen with tag v1.9.40 https://github.com/BobKnothe/autoNumeric/tree/v1.9.40 could you please update  with the new changes or create a new tag?

Mic - i update the file and version number - thanks again for the help.


[Accessibility]A11y_VSCode_VSCAdditionalTasks_Screenreader: NVDA is not reading the "Back","Next" and "Cancel" buttons as buttons.

Environment Details:
VSCode Version : 1.24.0
OS Version: Win10
OS Version : HighSierra10.13.5
Additional Details:
ScreenReader: NVDA
NVDA Version: NVDA2018.2
Repro Steps:
1)Launch VS Code while NVDA is ON.
2)Launch Visual Studio code desktop application.
3)Click on next button-->Next-->Next-->Next Button
3)With the help of Tab just observe the NVDA Readings.
Actual:
NVDA is not reading the "Next" and "Cancel" as buttons.
Expected:
NVDA should read the appropriate role for all the controls on the screen.
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
User will not come to know that there are buttons present on the screen.
Attachment for Reference:
[Accessibility]A11y_VSCode_VSCAdditionalTasks_Screenreader.pptx
Does this issue occur when all extensions are disabled?: Yes

Repro Steps:
1)Launch VS Code while NVDA is ON.
2)Launch Visual Studio code desktop application.
3)Click on next button-->Next-->Next-->Next Button
3)With the help of Tab just observe the NVDA Readings.

What does this even mean? Can you share a screenshot of where those Next buttons are?

@AccessibilityTestingTeam-TCS This needs to go into https://github.com/jrsoftware/issrc

This issue not reproducing in new version VScode 1.24.1.


To many false positives

To many false positives.

With the maxCharactersBetweenNumbers in the validator you can specify how many character are allowed between numbers. This will decrease the false positives.


Bug? All hold invoices are private

Background
When adding a hold invoice, it automatically adds private channel routing hints, regardless if the flag --private is active, or not.
Your environment
lnd v0.6.1
Steps to reproduce
lncli addholdinvoice [command options] hash [amt]
An invoice with routing hints of all private channels is produced.
lncli addholdinvoice --private [command options] hash [amt]
Again, an invoice with routing hints of all private channels is produced.
Expected behaviour
When the --private flag is not set, there should not be any routing hints included in the invoice. I'm not sure if this is on purpose, or if this is undesired behavior.

lncli creates invoices with routing hints by default. If you'd like to omit them, you need to explicitly set --private=false.

Oh, I did not notice that it's private by default.
As you mention --private=false works.
Sorry for the FUD then :)
Closing the issue...


Registry search results improvements

Searching the registry for https://opentelemetry.io/registry/?s=python+exporter returns results for all items containing python and all items containing exporter. I would expect it to return only python exporters.

This can now be accomplished by the following query: https://opentelemetry.io/registry/?s=&component=exporter&language=python


When Upgrading k83 version for an  EKS cluster fails, k8s version in the edit cluster view reflects the newer version.

What kind of request is this:
Bug
Steps to reproduce (least amount of steps as possible):

Deploy an EKS cluster selecting k8s version 1.13
Once the Cluster is Active go to Edit cluster.
Select k8s 1.15 from the Kubernetes Version dropdown and click Save.
The result is shown below
After the failure the Rancher UI stays in k8s 1.15 as if the upgrade was a success. But that isn't the case.
Edit Cluster shows k8s 1.15 and can't change it back to 1.13 unless we use the API.

Result:
Cluster is stuck in Updating in Rancher.
InvalidParameterException: Unsupported Kubernetes minor version update from 1.13 to 1.15 status code: 400, request id: 3e33935c-9352-xxxx-xxxx-xxxxxxxxxxxx

Other details that may be helpful:

EKS Console shows upgrade to 1.15 as unsupported from 1.13
Upgrading from 1.13 to 1.14 works.
Upgrade path 1.13 -> 1.14 -> 1.15 works.

Environment information

Rancher version: v2.4.3-rc6
Installation option: HA k8s

This is also reproducible in v2.3.6 with EKS k8s 1.13

Trying to edit the cluster it shows 1.15 immediately:

I recommend not allowing them to update to version that are more than one minor version ahead of their current. We know EKS will not allow it.
In my opinion, we should also hold off on adding this type of validation logic in the backend as @mrajashree works on kubernetes native validation. We likely will not have the same kind of flexibility w/ API level validation going forward. We also want people to be able to remedy these types of scenarios using the raw API.

EKS UI is broken in latest 2.4-headcommit id: 8e20687a4 and master-head commit id: b85ea695f
Uncaught (in promise) Error: Could not find module Semver imported from shared/components/cluster-driver/driver-amazoneks/component
UI is just stuck here:

Rancher version: 2.4-head (05/19/2020) commit id: 8e20687a4
Rancher version: master-head (05/19/2020) commit id: b85ea695f
The UI is now preventing to upgrade from 2 versions behind the current k8s version in EKS.

Upgrading K8s is possible from 1.13 -> 1.14 and then from 1.14 -> 1.15 as expected and supported by EKS.


Added support for proxy_ssl_server_name to automate-es-gateway

:nut_and_bolt: Description: What code changed, and why?
When connecting to external Opensearch clusters they may be configured in a way that requires passing along the hostname via the http request. One example of this is Opensearch hosted in Kubernetes which is using a shared ingress that depends on the hostname to be passed for internal routing to the correct service. This code updates the config api to support a new boolean value AND updates the nginx config for automate-es-gateway to check for this config value to determine whether or not to set proxy_ssl_server_name on;
:chains: Related Resources
:+1: Definition of Done
You should be able to set or patch in this config:
[esgateway.v1.sys.ngx.http]
proxy_ssl_server_name = true

and the nginx.conf for automate-es-gateway should include the line
proxy_ssl_server_name on;
:athletic_shoe: How to Build and Test the Change
I need help testing this as attempting to run the dev environment even on the main branch with no changes is failing for me.
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[x] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.
:camera: Screenshots, if applicable

fyi @chef/automate-team

Thanks @mskdenigma  for the PR, we will get it Reviewed and Tested by next week.


Update Licensing and copyright page

[x] Update page to make licensing/permissions tagging as clear as possible with a map of how editorial should write stickies and how to convert this to xml
[ ] Becky approved page
[ ] Melissa approved page
[ ] Fred approved page
[ ] James approved page
[ ] Naushin approved page
[ ] Editorial approved page
[ ] Share page with Exeter

Definition of done

[ ] Team reviewed and approved page?
[ ] Add changelog to page
[ ] Assign Fred to update the Schematron if relevant

@bcollins14 @JGilbert-eLife @FAtherden-eLife @griffithsc @Melissa37
This page has now been updated! I've added to the Figure adaptations and reproductions sections - let me know if anything could be clearer. @griffithsc I'd especially value your feedback on this as someone with fresh eyes! :D

@naushinthomson Hi Naushin - this looks great. I would perhaps like to know at what point in the 'journey of an article' the XML tagging would take place (if needed)?

@naushinthomson One other question - How you would go about checking the license of the previously published content from which the figure has been redrawn/reproduced? Apologies if this has already been addressed!


Solana - create agreement

Is this a Solana version of Pay agreement#9?
I can start drafting something.

The original issue for create agreement was not created in github. But it already exists in the code.
Now, for Solana, we separated into different issues:

#45
#41
#43

Since merge was accepted, Im closing this issue.


Texture? poster.jpg

I've hacked up a downloader/uploader/Sketchup editor importer.
And I feel I need a quick local viewer.
Please please consider adding a texture to this?
Пожалуйста :)

This project was a toy. I no longer have time or interest to play with this any further. Feel free to fork and submit PR if you want to add anything. I doubt I will be adding any features to this. I don't have an app either.

Fair enough.
Спасибо за первоначальный импульс.
Я продолжил в сторону Sketchup, если видели
PAF

15 апр. 2015 г., в 3:01, Dmitry Yakimenko<EMAIL_ADDRESS>написал(а):
This project was a toy. I no longer have time or interest to play with this any further. Feel free to fork and submit PR if you want to add anything. I doubt I will be adding any features to this. I don't have an app either.
—
Reply to this email directly or view it on GitHub.


Feature: Allow scrolling past the end of the list.

I dread authoring my reply on the very bottom of the screen. Please add some element after that, so I can scroll the textarea to the center of the screen while writing.
Actually this maybe 80% screen height of extra scrolling is also nice to have when reading, too.

this is a good suggestion. done for the event page. let me know if there was another page you think this is needed for.


Create alertmanager service scrape by default (#385)

I'm trying to get vmagent to scrape alertmanager by default. Since I'm quite new to Golang and Kubernetes operators, please let me know if my change makes any sense :)
Fixes https://github.com/VictoriaMetrics/operator/issues/385

Thanks for contribution!
It defiantly makes sense)

Thanks! Any ETA on a release that will contain this change? No rush, just trying to plan if I should work around it by adding the service scrape manually until then :)

Monday or Tuesday at this week, there are few issues, that i want to investigate and fix before release.