SVG icons

Investigate better workflow for SVG icons. Symbols are OK, but you are not able to apply full power of styles on it. Inline SVGs are always better. Check this repo: https://github.com/iconic/SVGInjector

I want to add that today I've discovered that when you create several folders, which should compile into respective sprites, globbing the folders the way we do now does not provide a desired result. Sprites get overwritten, depending on the files in queue.
It may be that we need to parse folder first, divide into a simple Object with files as entries, and then pass each object to gulp.src().

I don't know if this is the issue with the styling, but if you get an SVG that has inline fill (or any other) attribute, you are not able to change it with CSS, e.g.:
<symbol id="icon"><path d="..." fill="#f0e"/></symbol>

But we could set up a rule inside gulp-svgmin, that would automatically remove attributes, that we would like to change in CSS, e.g.:
plugins: [{
    removeAttrs: {
        attrs: ['fill']
    },
    cleanupIDs: {
        prefix: `${prefix}-`,
        minify: true
    }
}]

FYI, the removeAttrs has to be before cleanupIDs(already in devstack), otherwise it doesn't work, don't know why :(

I have another enhancement to the svg macro that we currently have. The problem now is, that we cannot change the size of the svg, because we don't have a modifier class to the <svg> element itself, only for <use> element:
{% macro svg(name, className, sprite = "icons") %}
    <svg class="icon">
        <use{% if className %} class="{{ className }}"{% endif %} xlink:href="/gfx/svg/sprites/{{ sprite }}.svg#{{ name }}" />
    </svg>
{% endmacro %}

But I propose to have two macro elements, one with <svg> and one with <use>:
{% macro template(class) %}
<svg class="icon {{ class }}">
    {{caller()}}
</svg>
{% endmacro %}

{% macro use(name, class="", sprite = "icons") %}
<use{% if class %} class="{{ class }}"{% endif %} xlink:href="/gfx/svg/sprites/{{ sprite }}.svg#{{ name }}" />
{% endmacro %}

Then we would include the svg elements like this:
{% import "../atoms/_svg.nunj" as svg %}

{% call svg.template('icon-form icon--xl') %}
    {{svg.use('icon-email', sprite='form')}}
{% endcall %}

What do you think? I currently use it on Walmark project.

@ronaldruzicka regarding fill issue. Remove attributes will not always fix the issue. The problem here is in the SVG source (we have had a hard time with proper SVG on Conrad, and it is doable). If icon is prepared poorly, when you remove fill and strokes it will break, and you will not be able to style it with CSS at all. So this is more a question for picking right icons and asking designers to export them properly (if you want, we can have a chat to share our experience with @zoltan-kovac on svg icons :) )

Ok of course, we can talk about it :)
But anyway, regarding the svg macro, I would still propose this change, so that we can easily apply styles (modifier classes) also on the <svg> and manage e.g. the size of it.

@ronaldruzicka two macros and their composition is templates seem too complicated to me. We won’t use them separately anyway and calling just {{ svg(…) }} is more straightforward.
What about having just
{% macro svg(id, className = "icon", useClassName = "", sprite = "icons") %}
<svg class="{{ className }}">
    <use class="{{ useClassName }}" xlink:href="/gfx/svg/sprites/{{ sprite }}.svg#{{ id }}" />
</svg>
{% endmacro %}

I appreciate that your approach avoids too many arguments in one macro but I think the added complexity isn’t worth it.

After we finalize this macro tweak, we’re gonna close the issue as it’s rotting here for several months. Once someone investigates the matter on any new project, we’re gonna reevaluate it again.

I just got used to nest the macros from the Sitecore Habitat Demo project, but this solution is also good. We just need to modify both svg and use and this works.
So we can implement this updated macro.

OK, cool, let’s go for it.
We can benefit from nested macros as soon as we need something more composable (those that are going to be used both together and separately).

Do we still have some unsolved issues with the svg elements? Or can we close this issue?


Fix#27 upgrade js of ocaml

Fix #27
I took the opportunity to remove some superfluous information from the opam files.

ping @xhtmlboi


Replace getURLParam with URLSearchParams - JavaScript clean up

Is your proposal related to a problem?
Currently we have code that uses regex parse the URL location to get the query param (e.g. q=foo).
https://github.com/wagtail/wagtail/blob/f31cce6786fbe2a385367a19a16bcdd6d014a5f1/client/src/entrypoints/admin/core.js#L308-L317
Describe the solution you'd like

Replace this usage with the URLSearchParams approach used elsewhere in the code - https://developer.mozilla.org/en-US/docs/Web/API/URLSearchParams/get

Essentially, copy this code
https://github.com/wagtail/wagtail/blob/f31cce6786fbe2a385367a19a16bcdd6d014a5f1/client/src/entrypoints/admin/core.js#L281-L282
Describe alternatives you've considered

This is a small bit of clean up work that avoids complex functions when the browser now supports this with native APIs, recommend we just get this done.

Additional context

This is part of prep work for the Stimulus project, cleaning up things before we pull them out to Stimulus controllers

@Lovelyfin00  @thibaudcolas  - as discussed yesterday, this small task would be good to get out of the way.
Note: Turns out other code, within the same file, is already using the Browser API URLSearchParams so instead of making a new function that does the same, we should just adopt this approach.
Ready for you to pick up when you have worked through the other starter items Loveth.

I'll start on it today


Charting Library

[ ] Project URL:
[ ] Update contents.json instead of README
[ ] One project per pull request
[ ] Screenshot included
[ ] Avoid iOS or open-source in description as it is assumed
[ ] Use this commit title format if applicable: Add app-name by @github-username
[ ] Use approved format for your entry

Not interested in this contribution, closing. Thank you


update IOL2 application text for the OPN website

https://openpolicynetwork.org/iol/
From feedback from round 1, Give better context for what is policy (be clear in application process and screen for this), and how it is different than programs/practice.

need to clarify IOL will be conducted in English

Make it clear to fellows how their applications will be used and shared

Cable reviewed - good to go - share with advisory committee

Shared both with advisory committee. will entertain feedback until COB 25 September.

done


can not open gitea web after restart

Description
Hi folks,
I installed gitea with Docker on my SBC and I had pushed a lot code to my repository, after several days, I restarted my SBC, I can not visit to my git web anymore.
I tried to run iotop and it seems keep reading from disk for hours, has anyone knows how to solve this? the docker logs is attached, please check out, thans very much.
docker ps
CONTAINER ID   IMAGE                COMMAND                  CREATED       STATUS          PORTS                                                                                NAMES
3ce78aa0dfb3   gitea/gitea:latest   "/usr/bin/entrypoint…"   2 weeks ago   Up 22 minutes   <IP_ADDRESS>:2222->2222/tcp, :::2222->2222/tcp, 22/tcp, <IP_ADDRESS>:3000->3000/tcp, :::3000->3000/tcp   gitea

gitea.log
Gitea Version
latest
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
running on ubuntu with docker
Database
None

Can you try to enter the container and run gitea manually?

Thanks Lunny, I've entered docker container and stop/start gitea, I got same problem.
seems it's retriving data from my disk, the web is keeping in wait stauts, I could open somethings but didn't show up all of my git repository.
is there any other command should I run?

We close issues that need feedback from the author if there were no new comments for a month. :tea:


Ask for technical advice on getting a players ranking(s)

approach one: get every single qr code
pros: simplest, most accurate
cons: slow (how to cache?)
approach two: store in db
pros: simple for client
cons: really complicated to try and keep running totals of everything via cloud functions

do we need to be able to sort leaderboard by different criteria


Add consolidated test report with all workflow results

Create an artifact with all surefire reports, pit report and a single junit html report. Publish to GitHub Pages with a link from the readme

@ClaudiodelaRiva @cabaluniovi FYI:

The additional report job generates an artifact test-report-ALL with test logs, surefire results, junit test and pit reports from all jobs
When some test fails, the junit html reports link the failure to the html diff file generated by the Visual Assert assertions
Added badges at the top of the readme to view online the html reports (junit and pit) of last run (ensure that you are viewing last run by refreshing the browser)


rustup; fix debugging

@bors r+

:pushpin: Commit 354de02c24525ee2a0cf712e5d0f44f6228ad96c has been approved by RalfJung

:hourglass: Testing commit 354de02c24525ee2a0cf712e5d0f44f6228ad96c with merge 7e8b17c85f7b0ea9ff84d0c93d4fdb1070f853eb...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 7e8b17c85f7b0ea9ff84d0c93d4fdb1070f853eb to master...


[Bug] Pet gets constantly energized

After using an energy drink, the energized status remains on almost forever. Additionally, every cycle, the player is informed that the pet is no longer energized, but this is not the case.

Wrong variable name was referenced.


The accuracy of DVS dataset

Hello, I have a question about the accuracy of CIFAR10-dvs and DVS128gesture datasets. I've tried many different seeds with given configs (For CIFAR10-dvs, I used spikingjelly, not .pt files), but the accuracy is lower than your proposed results. The accuracy of CIFAR10-dvs and gesture is around 77% and under 99% respectively.
So,  I just wonder if there are other configs or settings like augmentation when you got the proposed accuracy.
Thank you.

Hi, thanks for your interest.

Thank you for the answer. I. re-check the learning rate and version of CUDA or PyTorch.

Hello,i have the same question,can you tell me what is your current learning rate and version of CUDA or PyTorch?


Supports response patching

GitHub

Fixes #66

Changes

Adds new ctx.fetch() utility to perform a bypassed request
Supports bypassing of requests that have x-msw-bypass header set
Sets the following new properties on the mocked req object:

bodyUsed
integrity
destination
keepalive

@hehehai could you please review this change? I'd be very thankful.

Thanks for the operative review, @hehehai! Much appreciated.


Fix regex handling of undefined and support user supplid error message

Work for this issue will begin begin on branch work-liquid-labs/question-and-answer/87.


Support for LDAPS certificate validation

Description
Currently OpenNebula does not support validation of the server's certificate, when connecting via LDAPS.
Since this is already a feature the ruby LDAP library supports, this could easily be implemented.
Use case
Not validating certificates is a security risk and could possibly be exploited in MITM attacks.
Interface Changes
Add an additional option to /etc/one/auth/ldap_auth.conf (e.g encryption_options) to support the definition of custom tls_options.
Additional Context
I wrote a short workaround (without option in config file):
/usr/lib/one/ruby/opennebula/ldap_auth.rb: Line 77
ops[:encryption]={ method: @options[:encryption], tls_options: OpenSSL::SSL::SSLContext::DEFAULT_PARAMS,} if @options[:encryption]

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

OpenNebula supports the validation of the server certificate. The :encryption attribute from ldap_auth.conf is used as is, therefore following configuration will validate server certificate against ca.crt
    :user: "cn=Directory Manager"
    :password: xxx
    :auth_method: :simple
    :host: localhost
    :port: 636
    :base: dc=opennebula
    :user_field: uid
    :group: cn=cloud,ou=groups,dc=opennebula
    :rfc2307bis: false
    :encryption:
        :method: :simple_tls
        :tls_options:
            :ca_file: /etc/dirsrv/slapd-opennebula/ca.crt

Sorry for the late reply. Adding the configuration to ldap_auth.conf works as expected. Thank you very much.
Could this be added to the official documentation or as an inline comment to ldap_auth.conf in a future release?


Import order being different in IDEA plugin

Detekt task is not showing this issue.

Detekt task is not showing this issue.

Are you using different Detekt configs for the Gradle task and the IntelliJ plugin? I assume not. Would you be able to provide a reproducer?

not exactly the same, final release 1.19 for IDEA, RC2 for Gradle one. I will come back when I try with both dependencies updated

@cortinico I checked and everything is using the latest version available.


Support for using alive backend in solver tests

Don't merge until all tests pass.

ok great-- this is ready to merge as far as you're concerned @manasij7479 ?

Yes, feel free to merge this.
Lots of issues to fix before we can enable it by default though.

LGTM thanks!


No privileges from Windows pushes

The output of "lxc info" or if that fails:

  driver_version: 2.0.7
  kernel_architecture: x86_64
  kernel_version: 4.10.0-22-generic
  server: lxd
  server_version: "2.12"

>lxc version
2.13

Issue description
# lxc file push build.sh lx/root/build.sh
# lxc exec lx -- ls -la
...
----------  1 root root    36 May 30 12:17 build.sh

The same with recursive uploads. I am thinking if 0o777 should be the default if no more information it given?

Just discovered --mode parameter. lxc file push build.sh --mode 744 lx/root/build.sh works for me without implicit magic.

--mode flag doesn't work with recursive pushes. Reopening.
error: can't supply uid/gid/mode in recursive mode

The problem with supporting --mode with -r is that you don't want the same mode for files and for directories.
Though maybe we can do something clever with masking there, effectively granting execute access to directories where read would be granted. @brauner

The PR above at least makes uploaded files and dirs
-rw-rw-rw-
drwxr-xr-x


Updated solution source code repository url

Issue #, if available:
No issue
Description of changes:
The solution source code repository was wrong, pointing the non-existent repository https://github.com/awslabs/awscodepipeline-integration-bitbucket.git I changed it to the new one https://github.com/aws-samples/aws-codepipeline-bitbucket-integration.git
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Issue #, if available:
No issue
Description of changes:
The solution source code repository was wrong, pointing the non-existent repository https://github.com/awslabs/awscodepipeline-integration-bitbucket.git I changed it to the new one https://github.com/aws-samples/aws-codepipeline-bitbucket-integration.git
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for your contribution @franciscomemoli


OpenSearch Policy with snapshot to s3

Hi everyone 👋 ,
this draft wants to introduce a set of patches and features that will enable the possibility to configure a snapshot of the indexes to a s3 compatible bucket before being deleted, this will prevent data loss and will allow, whenever it will become necessary, the restoration to opensearch.
This is a draft because need to be tested!

hi! I've almost tested it widely and refactored as much as possible, now potentially the example directory could be removed, I've addressed the setup of the s3 backup in the katalog/opensearch-s3-single and katalog/opensearch-s3-triple directories.

Hi @lzecca78! is this ready for review?

Hi @ralgozino , yes it should. Speaking with @nutellinoit  I dunno how relevant this PR still is because with the introduction of Loki which has native s3 storage, probably this feature won't be necessary.
Anyway, I've pretty much tested everything, so it should be ok!
Let me know if there is anything i can improve!


Warn on unsupported managed identity configuration

closes #26714

API change check for Azure.Identity
API changes are not detected in this pull request for Azure.Identity


operator [R] gitlab-operator-kubernetes (0.13.1 0.13.3)

Thanks for submitting your Operator. Please check the below list before you create your Pull Request.
New Submissions

[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want to use semantic versioning order?
[ ] Is your submission signed?
[ ] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ ] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ ] Modify more than one operator
[ ] Modify an Operator you don't own
[ ] Rename an operator - please remove and add with a different name instead
[ ] Modify any files outside the above mentioned folders
[ ] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[ ] Description of the managed Application and where to find more information
[ ] Features and capabilities of your Operator and how to use it
[ ] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

/merge possible

@mitchnielsen,@WarheadsSE,@twk3,@droopy4096 , please approve

@pursultani I don't believe this should include 0.13.1 changes.

Fixed manually. A pre-commit check is added to ensure that the PR does not contain any excess.

/merge possible

/merge possible

/merge possible

/merge possible

/merge possible


Class 'Sentry\ClientBuilder' not found when trying to install Sentry

Am currently trying to add Sentry to my Laravel project but I got hit in this issue when I did composer require sentry/sentry-laravel.
The error I got hit with is:
`@php artisan package:discover --ansi
Error
Class 'Sentry\ClientBuilder' not found
at vendor/sentry/sentry-laravel/src/Sentry/Laravel/ServiceProvider.php:122
118|                 ],
119|                 $userConfig
120|             );
121|

122|             $clientBuilder = ClientBuilder::create($options);
123|
124|             // Set the Laravel SDK identifier and version
125|             $clientBuilder->setSdkIdentifier(Version::SDK_IDENTIFIER);
126|             $clientBuilder->setSdkVersion(Version::SDK_VERSION);

• Database name seems incorrect: You're using the default database name homestead. This database does not exist.
Edit the `.env` file and use the correct database name in the `DB_DATABASE` key.
https://laravel.com/docs/master/database#configuration`

My .env file
DB_CONNECTION=mysql DB_HOST=<IP_ADDRESS> DB_PORT=3306 DB_DATABASE=homestead DB_USERNAME=homestead DB_PASSWORD=secret
Laravel version: 7.5.2
I am running everything throught Homestead and my database does exist and the project is running but I cannot figure out why do I get this error.
Thank you in advance for your help!

Hi @walegbenga this is really weird, I think the database "tip" in unrelated, it tries to be smart and fails giving a confusing message :)
Can you add "sentry/sentry-laravel": "^1.7", to your composer.json manually and running composer update to see if that resolves the issue, to me it looks like the dependency was not correctly installed.
If that doesn't fix it, can you:

share your PHP version
share your composer.json (so I can try and see if I can replicate it locally)
share the full output of composer require sentry/sentry-laravel (so we can see what composer is doing)

I am closing this issue because of inactivity, if there is still an issue please re-open / open a new issue so we can investigate 👍


I can't use ConventionInjection class

Hi, I was trying to inherent ConventionInjection or LoopValueInjection, but I couldn´t use it.
I tried build the project, but the classes are missed.
I´m don´t know If I did something wrong or is an issue, but I can use the InjectFrom method very well.

use LoopInjection
ConventionInjection has been removed

Thanks! I´m sorry, I was reading old post about ValueInjecter.


PlayerGUID bug breaks my combat log

Date: 2020-11-30 03:36:47
ID: 1
Error occured in: Global
Count: 134
Message: ..\AddOns\Skada\Skada.lua line 1136:
Usage: GetPlayerInfoByGUID("playerGUID")
Debug:
(tail call): ?
[C]: GetPlayerInfoByGUID()
Skada\Skada.lua:1136: FixPlayer()
Skada\Skada.lua:1220: get_player()
Skada\modules\Damage.lua:79:
Skada\modules\Damage.lua:78
Skada\modules\Damage.lua:235: func()
Skada\Skada.lua:3038: ?()
...ibDataBroker\CallbackHandler\CallbackHandler-1.0.lua:147:
...ibDataBroker\CallbackHandler\CallbackHandler-1.0.lua:147
[string "safecall Dispatcher[21]"]:4:
[string "safecall Dispatcher[21]"]:4
[C]: ?
[string "safecall Dispatcher[21]"]:13: ?()
...ibDataBroker\CallbackHandler\CallbackHandler-1.0.lua:92: Fire()
...s\ElvUI\Libraries\Ace3\AceEvent-3.0\AceEvent-3.0.lua:120:
...s\ElvUI\Libraries\Ace3\AceEvent-3.0\AceEvent-3.0.lua:119

Hm I got this bug and it broke my combat log, hence I did zero damage (lmao).
I also use combat log fix macro if it helps you for more information.

@foxtoopia12 did you update the addon? Because I am pretty sure that error was fixed! Please let me know.

I just updated the addon. Please give it a try and let me know if it's fixed so we can close this issue.


Support interrupt levels 4 and 5

@MabezDev you've commented in CpuInterrupt::level() the following:

// we direct these to None because we do not support interrupts at this level
// through Rust

The original implementation PR also says that 3 levels are available.
But why? According to their TRMs, the interrupt matrix in ESP32, S2 and S3 has 5 interrupt levels, and at least for level-triggered, all 5 are available. Is this some arbitrary decision, or maybe one needed by esp-wifi? It would be nice if these levels were available for use.

The Xtensa TRM recommended using interrupt levels 4 & 5 as assembly-only routines, that's why it was initially unavailable in the vectored interrupt PR. I think its possible to use them now though xtensa-lx interrupt API, but I've never tried that.
I'm still undecided if we should just ignore the Xtensa TRM and just open up all the interrupt levels to high level Rust code.

Is this an either-or situation? The 32 separate interrupt handlers map to the 7 level entry points and that's it?

I guess we shouldn't
From Xtensa Instruction Set Architecture (ISA) Reference Manual

The value of CINTLEVEL is set to at least EXCMLEVEL whenever PS.EXCM=1. Thus, all
interrupts at level EXCMLEVEL and below are masked during the time PS.EXCM=1. This
is done to allow high-level language coding with the Windowed Register Option of interrupt handlers for interrupts whose level is not greater than EXCMLEVEL.

For our targets EXCMLEVEL is 3.
My understanding is that we shouldn't have any code at those interrupt levels using the windowed register ABI (it could interfere with the register spilling etc.)

I'll be honest, I find this ISA very ... arcane. But it is, what it is 😅


RSS item descriptions do not consider actual post descriptions

Describe the bug
RSS considers content or summary as <description></> and ignores descriptions supplied in post front matters. Also, the content's or summary's HTML tags are added which make them hard to use anyway.
To Reproduce
Steps to reproduce the behavior:

Create a post with description supplied in front matter.
See item's description on the RSS feed given on https://domain.tld/index.xml

Alternatively, see https://anatole-demo.netlify.app/index.xml.
Expected behavior
Description in front matters should be considered as <description></> in RSS feed.

Hi @alexanderdavide, what you are describing is caused by this line in the rss.xml:
 {{ .Summary | html }}
As you can see the rss.xml file expects a summary. Personally, I always set up both a description and a summary. Both variables are default page variables:
https://gohugo.io/content-management/front-matter/

Hi @alexanderdavide, what you are describing is caused by this line in the rss.xml:
https://github.com/lxndrblz/anatole/blob/80312357ceee167aa9f224281363933a47e27994/layouts/_default/rss.xml#L35-L39
As you can see the rss.xml file expects a summary. Personally, I always set up both a description and a summary. Both variables are default page variables:
https://gohugo.io/content-management/front-matter/

That's fine but what I'm asking is why isn't the description front matter used as RSS <description> element? I am setting description explicitly, I want it to be the description in RSS too. The summary should merely be a fallback.

I agree. As the description is also the default in the arch type, we should exchange the summary variable for the description.


Add missing word "to" in "due to"

Signed-off-by: Timothy Gillespie<EMAIL_ADDRESS>

Thanks @TimothyGillespie


Error in last line of CLI command

Hi,
I think there's an error in the last line of the CLI command, as it's returning an error:
az deployment group create
It should be:
az group deployment create

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

ID: 9f6eb602-fe75-1400-e683-39ae6308fdf6
Version Independent ID: 3a8da44e-acc9-bb4f-af6b-ddfcabab0cd9
Content: Template with dependent resources - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/template-tutorial-create-templates-with-dependent-resources.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@ronthy Can you please share the error that you see ?
It should be "az deployment group create" but not "az group deployment create" because the command group has been deprecated and will be removed in future release and the recommended one to use to deployment group.

Check this this issue for additional reference.

Hi @KrishnaG-MSFT,
Thanks for following this up so promptly. I have since discovered the nature of the error.
It turns out that the version of Azure CLI I had installed on my Ubuntu (20.04.3 LTS) was the outdated version (v2.0.81).
Hence, it was producing the following error:
$ az deployment group create
az deployment: 'group' is not in the 'az deployment' command group. See 'az deployment --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview
I have removed that outdated version and installed the latest azure-cli package (v2.28.0) and the command az deployment group now works fine.
Thank you once again for the follow up.
I will consider this issue resolved and will close this item.
Cheers! :)


JSON to mmCIF converter in GO

Description
write a script that converts the metadata from JSON format of the defined schema into the mmCIF file that will be fed to PDB OneDep.
Context
We want to have a translator for our metadata. Now it uses all the sample/instrument metadata JSON files and the mapper of JSON Schema properties to PDBx dictionary. It expects validated JSON files. Currently, it works without defining any structs for metadata structure and is agnostic to JSON or PDBx nesting in the naming of elements. As a result, the properties in mmCIF are sorted alphabetically. This is not an issue when parsing mmCIF, according to the documentation but looks not nice. Especially, when first and last name are separated by other fields.
I will probably introduce a clear structure.
Issues

issue with PDB OneDep. It complains about the upload files without a clear error definition
I will try a brute force approach by changing properties one-by-one (within one data item) in a CIF file downloaded from EMDB.

Definition of Done

[ ] tests; especially if the third layer of JSON nesting is still supported with the current implementation
[ ] file is consumed by PDB OneDep

Not mandatory fields present in the translation need to have a "." or "?" annotation


textIndent parameter has no effect on the Text composable.

Hi!
Apparently setting the textIndent parameter in the TextStyle of a Text composable has no effect, so it is essentially broken. On Android, it works just fine, but on desktop (Fedora Linux 35 / Windows 10) it doesn't add the indent.

reproduced on Mac /1.0.1

Should be fixed in 1.1.1

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


Please add

[GS.Core](https://github.com/VincentGsell/GS.Core). `[Delphi]` `[FPC]` Core functions shared by several projects

Thanks, added


cleaned up resources.md

cleaned up and added some links, added descriptions for some of the links and the History section

I like it.  Probably can add some more readable link-text for the rest of the hyperlinks, but I agree with all these changes.


Add new status reports

The GitHub API has added TRIAGE and MAINTAIN access levels.
Support that.
Signed-off-by: Ry Jones<EMAIL_ADDRESS>

Fixes #14

@jessfraz thoughts?

ah dope thanks!


Add ability to log form auth request

From<EMAIL_ADDRESS>on November 22, 2013 17:16:34
E.g. something like this:
     given().
            auth().form("John", "Doe", log().all()).
    expect().
            statusCode(200).
            body(equalTo("OK")).
    when().
            get("/formAuth");

Note that it should log the request sent automatically sent by FormAuthFilter (1).
Another alternative would be to copy the Request- and ResponseLoggingFilter from the original request and use them in (1).  E.g.
     given().
            auth().form("John", "Doe").
            log().all().
    expect().
            statusCode(200).
            log().all().
            body(equalTo("OK")).
    when().
            get("/formAuth");

Would log the two requests and two responses being made.
Original issue: http://code.google.com/p/rest-assured/issues/detail?id=274

From<EMAIL_ADDRESS>on June 04, 2014 06:33:49
Status: Fixed


Change the weather_data_api directory to weather_data_flaskapi

Please indicate if this is a new issue or feature.

[ x] New issue request
[ ] New feature request

New Issue
Rename weather_data_api directory to weather_data_flaskapi.
Expected behavior and actual behavior.
Rename the directory to the project name.
Steps to reproduce the problem.
N/A

Could not replicate the exception issue with Postman


Fix a few spelling and typos.

I have read the CLA Document and I hereby sign the CLA

:tada: This PR is included in version 2.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


tika builds broken with debian 12

Description/Context
tika builds broken with debian 12
Just migrate tika to run in docker like everything else and then we don't have to figure out why the JRE install breaks.
https://hub.docker.com/r/apache/tika
Plan/Design

Tika is now docker-ized so this issue is moot. Yay!


[DRAFT] Automatically deploy helios-solo for use with helios-testing

TODO:

Pull the latest version of both images (onescience/alpine:latest, spotify/helios-solo:latest) necessary to test and deploy helios solo each time.
Rework error handling. Right now a lot of useful information just gets logged. I think it would be better to throw an exception with traceback details that could be wrapped by an AssertionError.
Javadoc!
Actually plumb the HeliosDeployment up up to TemporaryJob. Ideally we'd automatically deploy solo if no HeliosClient can be created from the env vars explored in TemporaryJob.Builder.
Integration and unit tests.
Propagate DNS for machines using NetworkManager (Java DBUS interface?).

Can you elaborate a bit on "Propagate DNS for machines using NetworkManager (Java DBUS interface?)"?

I think you're missing some words here

Actually plumb the HeliosDeployment up up to `.

@mattnworb Thanks, done!
@gimaker We should replicate the functionality of https://github.com/spotify/helios/blob/master/solo/helios-up#L56. @rohansingh can likely explain better, but I believe on some machines (Ubuntu desktop?) /etc/resolv.conf just points to dnsmasq on localhost, and thus we currently shell out to nmcli and pass its DNS servers through to Docker. We should be able to do the same from Java via NetworkManager's DBUS API. What's not clear to me is why we need this workaround for dnsmasq, when we don't work around a similar paradigm for unbound in production.

I like the concept of a HeliosDeployment which (if my interpretation is correct) represents a deployed cluster somewhere and HeliosSoloDeployment points to a deployment of solo.
It seems like HeliosSoloDeployment does all of it's work in the constructor; for testing and re-use (such as the addition of other methods to the interface in the future) it might be nice if this was separated out.
Do you have in mind what the use of HeliosSoloDeployment by TemporaryJobs (i.e. the API to set one up) would look like? Because I think thinking about that can help answer the previous point.
For instance, maybe something like
HeliosDeployment solo = new HeliosSoloDeployment();
solo.start();

would be a simple way to separate the instantiation of the object from the "now do some work" part of it.

Ah I missed that the way to build and start a HeliosSoloDeployment is HeliosSoloDeployment.fromEnv().build() which negates my previous comment a bit.
I think it would be good to have a unit test of HeliosSoloDeployment separate from the IT.

https://ghe.spotify.net/gist/negz/460c004fdf5816a6ce76
It occurred to me that we could just put the onus on the caller to deploy Helios solo from their tests and pass the resulting client into their TemporaryJobs instance, rather than trying to deploy it magically. Principle of least surprise and all that. Thoughts?

re the gist - is there teardown work that has to happen to clean up the helios solo container? is the @Rule on TemporaryJobs taking care of that?
I think the idea of declaratively using a self-container solo container is intriguing

Current coverage is 43.53%

Merging #769 into master will increase coverage by +0.15% as of 7e5d76f

@@            master    #769   diff @@
======================================
  Files          257     261     +4
  Stmts        10760   10957   +197
  Branches      1508    1523    +15
  Methods          0       0
======================================
+ Hit           4668    4770   +102
- Partial        417     432    +15
- Missed        5675    5755    +80

Review entire Coverage Diff as of 7e5d76f

Powered by Codecov. Updated on successful CI builds.

Soooo, shall I merge?

thanks for addressing my comments. :+1:

:+1:


Codec derivation - dropNullValues

For now it is impossible to convenietly generate encoder that does not produce null values at the output.
Scenario:
@JsonCodec case class Params(p1: String, p2: Option[Int])
def process(a: Json): Unit = ???

val params = Params("abc", None)
process(json"""{"requestId": 123, "params": $params}""")

And I want the a not to contain null values.
I know I can:

preprocess a in the process() method - apply dropNullValues, use customized printer, etc
postprocess json where I use interpolation
customize codec by .mapJson(.dropNullValues)

First and second one seems like wrong place to solve this issue - its like workaround for misbehaving codec.
Third option requires lots of bolierplate code: Switching from annotation to manual derivation (creation of companion objects, etc), and then mapping derived encoder output.
I believe most correct solution would be to add io.circe.generic.extras.Configuration.dropNullValues. Then one could use it with derived codecs as well as with @ConfiguredJsonCodec.

https://scastie.scala-lang.org/CQEocDLHTwS51Gte8lCasQ

@kamilkloch Provide a reproduct.
import io.circe._
import io.circe.syntax._
import io.circe.generic.JsonCodec
import io.circe.generic.extras.semiauto._
import io.circe.generic.extras.Configuration

@JsonCodec
case class Params(p1: String, p2: Option[Int])

val jsonString1: String = """{ "p1": "foo" }"""
val jsonString2: String = """{ "p1": "foo", "p2": null }"""

for {
  json1 <- parser.parse(jsonString1)
  json2 <- parser.parse(jsonString2)
  data1 <- json1.as[Params]
  data2 <- json2.as[Params]
} yield {
  println(data1)
  println(data2)
  println("// === problem start ===")
  println(data1.asJson == json1)
  println(data1.asJson == json2)
  println("// === problem end ===")
}

Output:
Params(foo,None)
Params(foo,None)
// === problem start ===
false
true
// === problem end ===

I can do nothing but hack Encoder with dropNullValues to make
data1.asJson == json1

return true in this code snippet.

@kamilkloch Provide a reproduct.
Yes, I do get the same result.

I believe most correct solution would be to add io.circe.generic.extras.Configuration.dropNullValues. Then one could use it with derived codecs as well as with @ConfiguredJsonCodec.

Yet another solution is to use jsoniter-scala-circe's codec instantiated with a function to filter out nulls.
The overhead of encoding of such fields with nulls will be compensated by much more efficient serialization from jsoniter-scala-core under the hood. Also, you will get an ability of serialization immediately to byte arrays, java.nio.ByteBufers, java.io.OutputStream or preallocated byte arrays without intermediate strings, that will save yet more CPU cycles and memory bandwidth.

@kamilkloch Provide a reproduct.
Yes, I do get the same result.

I believe most correct solution would be to add io.circe.generic.extras.Configuration.dropNullValues. Then one could use it with derived codecs as well as with @ConfiguredJsonCodec.

@kamilkloch Yes, I think so too. io.circe.generic.extras.Configuration.dropNullValues is the final solution.


textEditor.revealRange always scrolls to left horizontally

VSCode Version: 1.15.0-Insider
OS Version: MacOS Sierra 10.12.6

textEditor.revealRange with specified lines range and character scrolls to the 0 character:
onCursorMove(e: vscode.TextEditorSelectionChangeEvent) {
  e.textEditor.revealRange(new vscode.Range(
                        Math.max(selection.active.line - 10, 0), selection.active.character,
                        selection.active.line + 10, selection.active.character),
                        vscode.TextEditorRevealType.Default
  );
}

So when I navigate to the EOL it's okay, then I press e.g. left then cursor goes one position left and horizontal scroll goes to zero:

Also:

selection.active.character is not zero (at the end of the line),
seems like it doesn't depend on the revealType.

@kirill-kruchkov You are revealing a range of 20 lines, 10 lines above and 10 lines below the current selection. A range of 20 lines is visibly a rectangle:
In the following picture I've highlighted what a range of 20 lines looks like.

The editor will try to reveal as much as possible from the area of those 20 lines and that happens to always be at scrollLeft = 0, as if the editor would scroll to the right (for example) the lines without any characters or with a single character would not be visible.


ERROR: Error extracting LR logs: 1 validation error for DynamicSchemaModel

Hi,
I am playing around with your code and I came accross this error.
ERROR: Error extracting LR logs: 1 validation error for DynamicSchemaModel

You can reproduce with
DROP TABLE public.bets1;
DROP PUBLICATION test_pub;
select pg_drop_replication_slot('test_slot');

CREATE PUBLICATION test_pub FOR ALL TABLES;
SELECT * FROM pg_create_logical_replication_slot('test_slot', 'pgoutput');

CREATE TABLE if not exists public.bets1 (
    id bigint NOT NULL,
    deleted numeric(1,0) DEFAULT (0)::numeric NOT NULL,
    oca smallint DEFAULT 0 NOT NULL,
    modified timestamp with time zone DEFAULT now(),
    reference character varying(250),
    status character varying(20) DEFAULT 'ACTIVE'::character varying NOT NULL,
    bet_type character varying(20) NOT NULL,
    placed_time timestamp with time zone DEFAULT now() NOT NULL,
    total_stake numeric(10,2) NOT NULL,
    account_ref character varying(50) NOT NULL,
    entered_by_user_id bigint,
    settled_by_user_id bigint,
    settlement_time timestamp with time zone,
    refund numeric(10,2),
    winnings numeric(10,2),
    channel character varying(20) NOT NULL,
    currency character varying(3) NOT NULL,
    bets_id bigint,
    chan_id bigint NOT NULL,
    acco_id bigint NOT NULL,
    risk_hit numeric(1,0) DEFAULT (0)::numeric NOT NULL,
    bet_risk_group character varying(50),
    rejection_details character varying(100),
    rejection_checked numeric(1,0) DEFAULT (0)::bigint NOT NULL,
    rejected numeric(1,0) DEFAULT (0)::numeric NOT NULL,
    total_allowed_stake numeric(10,2),
    void_reason character varying(100),
    system_bet numeric(1,0) DEFAULT 0 NOT NULL,
    bet_type_id smallint,
    status_id smallint NOT NULL,
    total_price numeric(26,4) NOT NULL,
    message_time timestamp with time zone,
    external_bet_id character varying(30),
    settlement_type smallint,
    win_amt_adjust numeric,
    place_amt_adjust numeric,
    num_bets smallint NOT NULL,
    notes character varying(200),
    manual_payout numeric,
    partition_no numeric(1,0) NOT NULL,
    payout numeric(10,2),
    prod_id bigint,
    setl_type_id smallint,
    captured_stake numeric(10,2),
    win_type_id smallint,
    agent_positions character varying(100),
    agent_commissions character varying(100),
    account_path character varying(80),
    account_bonuses_id bigint,
    free_bet boolean,
    bonus_campaign_id bigint,
    country_code_iso character(3),
    bonus_stake numeric(10,2),
    operator_stake numeric(10,2),
    operator_liab numeric(10,2),
    betslip_id bigint,
    wallet_error_code character varying(30),
    wallet_error_details character varying(200),
    wallet_type_id smallint NOT NULL,
    fund_type_id smallint NOT NULL,
    betslip_settled boolean,
    num_lines smallint NOT NULL,
    void_by_user_id bigint,
    punter_colour character varying(20),
    trading_period_id smallint,
    regulator_conf_id character varying(20),
    site character varying(5) NOT NULL,
    enhancement1_type character varying(10),
    enhancement1_flag boolean,
    enhancement1_fee numeric(10,2),
    enhancement1_multiplier numeric(10,2),
    tax_on_returns_pct numeric,
    tax_on_returns_amt numeric,
    winnings_bonus numeric(10,2),
    betslip_settled_by bigint,
    tax_on_stake_pct numeric,
    tax_on_stake_amt numeric,
    bonus_deposit_stake numeric(10,2),
    bonus_winning_stake numeric(10,2),
    bonus_funds_stake numeric(10,2),
    over_ask_id bigint,
    over_ask_expiry timestamp with time zone,
    limited_account boolean,
    affiliate_id character varying(255),
    result smallint DEFAULT 0 NOT NULL,
    pct_max_stake_used numeric(7,4),
    wallet_settlement_status_id smallint,
    comment_time timestamp with time zone,
    commented_by_user_id smallint,
    click_thru_source character varying(20),
    payout_excluding_bog numeric(10,2),
    entitled_free_bet boolean,
    external_betslip_id character varying(30),
    org_cashout_bet_id bigint,
    cashout_count smallint,
    partial_cashout numeric(1,0),
    cashout_amt numeric(10,2),
    replacement_bet_id character varying(50),
    build_a_bet boolean,
    blocked boolean DEFAULT false,
    max_payout numeric(10,2),
    autocashout character varying(2000),
    referral_path character varying(25),
    external_retailer_id bigint,
    retail_bet_info character varying(2000),
    claim_by_date timestamp with time zone,
    paid_time timestamp with time zone,
    paid_by_retailer bigint,
    bonus_discount_stake numeric(10,2),
    settlement_notification_type character varying(30),
    retailer_id bigint,
    brand_id smallint NOT NULL,
    test boolean DEFAULT false NOT NULL,
    mts boolean,
    refund_tax_on_returns_amt numeric(10,4),
    extra_info json,
    estimated_reward_points numeric(10,2),
    pools_betting_coupon_id bigint,
    settlement_version smallint DEFAULT 0 NOT NULL,
    arena_bet_type_id smallint DEFAULT 0 NOT NULL,
    mts_rejection_message character varying(200),
    geocomply_details character varying(200),
    teaser_price numeric(7,3),
    price_chan_id bigint,
    limits_chan_id bigint,
    tax_federal numeric(7,3),
    tax_state numeric(7,3),
    w2g_id bigint,
    geolocation_details character varying(200),
    mts_settlement boolean DEFAULT false
);

alter table bets1 replica identity full;

COPY bets1 from STDIN CSV;
213231430000103,0,1,2021-08-14 03:21:26.57717+01,site=NORMAL;ip-address=<IP_ADDRESS>;application=web-sportsbook;,ACCEPTED,MULTIPLE,2021-08-13 10:43:56.844+01,100.00,33,,,,0.00,0.00,INTERNET,USD,,5,33,0,PRE_MATCH_MULTIPLE,,0,0,,,0,2,1,3.5340,,,0,,,0,,,3,,1,,100.00,0,,,,,f,,US ,,,,1213231430000103,,,1,1,,2,,,0,,2,,f,,,,,0.00,,,,,,,,,t,,0,90.0000,3,,,,,f,,,0,0,,,f,f,499.96,,,,,,,,,,,2,t,f,,"{""bonuses"":[{""type"":""ColumnLengthBetBonus"",""amount"":0,""awarded"":false,""updatedOn"":""2021-08-13T09:43:57.078+0000""}]}",,,0,0,,,,1,5,,,,,f
\.

update bets1 set id=id;

I tried to debug it, and I believe the error come from this function's return
    def process_update(self, message: ReplicationMessage, transaction: Transaction) -> ChangeEvent:
        decoded_msg: decoders.Update = decoders.Update(message.payload)
        relation_id: int = decoded_msg.relation_id
        if decoded_msg.old_tuple:
            before = map_tuple_to_dict(tuple_data=decoded_msg.old_tuple, relation=self.table_schemas[relation_id])
        after = map_tuple_to_dict(tuple_data=decoded_msg.new_tuple, relation=self.table_schemas[relation_id])
        logging.debug (f"before={self.table_models[relation_id](**before)}")  # HERE
        logging.debug (f"after={self.table_models[relation_id](**after)}")
        return ChangeEvent(
            op=decoded_msg.byte1,
            message_id=message.message_id,
            lsn=message.data_start,
            transaction=transaction,
            table_schema=self.table_schemas[relation_id],
            before=self.table_models[relation_id](**before) if decoded_msg.old_tuple else None,
            after=self.table_models[relation_id](**after),
        )

more specifically this part
self.table_models[relation_id](**before)

Hi @brunomgalmeida,
thank you for trying out the library and also for raising this issue
The type casting from strings to python types is still pretty basic. This function is used with dynamic models from Pydantic and I need to add handling of more of the Postgres types.
I'll look into this as soon as I have time (reproduce it, improve the error messages and fix it). I'm also considering an option to just return the before/after data as a string if it's useful.
What version of Postgres are you using?

Hi @dgea005
I have tested the above with postgres 10 and postgres 12, both yield a similar error, on update didnt test the insert.

and I need to add handling of more of the Postgres types.
I guess I am using some data types that are not being handleded then

I'm also considering an option to just return the before/after data as a string if it's useful.
I started doing this, but got sidetracked with work.... the killer of joy

Thanks for checking this

Hey @brunomgalmeida,
I tried out your example on a local container and it worked perfectly to help debug it.
It seems I specified the pydantic type for JSON wrong. I put dict but there is a real JSON https://pydantic-docs.helpmanual.io/usage/types/
I merged a fix here in case you are interested to try using it again  https://github.com/dgea005/pypgoutput/pull/14
I only had to time to reply to you until now also because of work :smile:
Thanks again for writing the issue!


sources(curl): use --next for each url in curl config

This is a clean version of https://github.com/osbuild/osbuild/pull/1812
Before this gets merged we should:
a) redo a speed test with new curl to ensure we have no regressions in the speed because of the next --nex
b) open the equivalent of https://github.com/osbuild/osbuild-composer/pull/4253 for this PR and ensure all composer tests are green (this is done in https://github.com/osbuild/osbuild-composer/pull/4281 as WIP)

Holding off pushing the big green button since I'm not sure if a speed test has been done. #4281 is passing though.

I ran the speed test now in AWS in a micro instance with:
$ sudo bash -c "time python3 -m osbuild --libdir . --store $(mktemp -d) ./test/data/manifests/fedora-boot.json"

Regular main took for the download (because micro only two parallel curls are active):
$ git branch |grep '^*'
* (HEAD detached at upstream/main)
$ git describe
v125-6-gd1dbcc20
$ sudo bash -c "time python3 -m osbuild --libdir . --store $(mktemp -d) ./test/data/manifests/fedora-boot.json"
...
manifest ./test/data/manifests/fedora-boot.json finished successfully
build:    	32e87da44d9a519e89770723a33b7ecdd4ab85b872ae6ab8aaa94bdef9a275c7
os:       	3b4b32941a016e710b292b4735cde9172cccdc4e6edef442f7808b5a50a10311
image:    	ca2c3347b6a6b5c5c4032dd3175b3462e277262fa5a03dbda5a4fc3d43846fe5

real	1m56.591s
user	0m56.399s
sys	0m5.683s

Wen hacking it to 8 curls that are forked
$ git diff
diff --git a/sources/org.osbuild.curl b/sources/org.osbuild.curl
index 8071dcba..0c19b4c0 100755
--- a/sources/org.osbuild.curl
+++ b/sources/org.osbuild.curl
@@ -258,7 +258,7 @@ class CurlSource(sources.SourceService):

     content_type = "org.osbuild.files"

-    max_workers = 2 * (os.cpu_count() or 1)
+    max_workers = 8 * (os.cpu_count() or 1)
...
real	1m5.918s
user	0m58.312s
sys	0m5.419s

And for hacked with 32 forked ones:
$ git diff
diff --git a/sources/org.osbuild.curl b/sources/org.osbuild.curl
index 8071dcba..81b70c7c 100755
--- a/sources/org.osbuild.curl
+++ b/sources/org.osbuild.curl
@@ -258,7 +258,7 @@ class CurlSource(sources.SourceService):

     content_type = "org.osbuild.files"

-    max_workers = 2 * (os.cpu_count() or 1)
+    max_workers = 32 * (os.cpu_count() or 1)

     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
....
real	1m9.256s
user	1m1.787s
sys	0m5.812s

(and even with hacked 128 forked curls I only get a real runtime of 1m11s)
And the new parallel branch.
$ git branch |grep '^*'
* use-curl-next-in-config
$ sudo bash -c "time python3 -m osbuild --libdir . --store $(mktemp -d) ./test/data/manifests/fedora-boot.json"
...
manifest ./test/data/manifests/fedora-boot.json finished successfully
build:    	32e87da44d9a519e89770723a33b7ecdd4ab85b872ae6ab8aaa94bdef9a275c7
os:       	3b4b32941a016e710b292b4735cde9172cccdc4e6edef442f7808b5a50a10311
image:    	ca2c3347b6a6b5c5c4032dd3175b3462e277262fa5a03dbda5a4fc3d43846fe5

real	0m9.065s
user	0m5.328s
sys	0m1.767s

So especially with really fast connections (like AWS) this PR really makes a difference.

This will need https://github.com/osbuild/osbuild/pull/1868 and I will do a final test from osbuild-composer (like the one in https://github.com/osbuild/osbuild-composer/pull/4281) and then this can go finally in again I think :)


Control-C handler uses obsolete Thread.Abort

Unhandled exception. System.PlatformNotSupportedException: Thread abort is not supported on this platform.    at System.Threading.Thread.Abort(Object stateInfo)    at Microsoft.Scripting.Hosting.Shell.BasicConsole.<.ctor>b__24_0(Object sender, ConsoleCancelEventArgs e)    at Microsoft.Scripting.Hosting.Shell.BasicConsole.<.ctor>b__24_1(Object sender, ConsoleCancelEventArgs e)    at System.Console.HandlePosixSignal(PosixSignalContext ctx)    at System.Runtime.InteropServices.PosixSignalRegistration.<OnPosixSignal>g__HandleSignal|10_0(Object state)    at System.Threading.Thread.StartCallback() Aborted (core dumped)
At first I assumed Thread.Abort is unsupported on Linux, but then I realized the API is now obsolete on all platforms. How hard would it be to add cancellation tokens and what would be the performance impact?

Thanks for the report. I'm not even sure how cancellation tokens would work since the threads can be running arbitrary code. There is the following issue which may be of interest: https://github.com/dotnet/runtime/issues/41291

Related issues: IronLanguages/ironpython3#64, IronLanguages/ironpython3#1024, IronLanguages/ironpython3#1312.
One possibility would be to check for a cancellation toke state at the beginning of each interpreted frame run. Ctrl-C would cancel the token and issue Thread.Interrupt. This does not guarantee that the interrupt will be always effective, like if the thread is off in the bushes inside a third-party code, but on platforms that do not support Thread.Abort the only other thing that can be then done to stop it is to bring down the whole process. This was also my experience with some interactive Unix tools: if they are really stuck in an infinite loop, Ctrl-C wouldn't help, one needed to sent a stronger signal, like SIGTERM, SIGQUIT, or even SIGKILL.
There would be some performance impact, but I don't think it will be that big, it would need to be measured. Relatively speaking, executing an interpreted frame takes orders of magnitude longer than a simple token state check.

Checking just once per frame will often result in Ctrl-C not working because many loops are contained entirely in a single frame. Performing the check somewhere that gets executed frequently like getting a CodeContext would be more reliable and hard or even impossible to get stuck using python code only. The question is whether the performance impact is going to be reasonable.

https://github.com/dotnet/runtime/pull/71661


nvm upgrade?

Would it be possible to add something like nvm upgrade? I imagine it would upgrade to the latest minor/patch version of the currently active version (or just latest patch for v0.x), and would be functionally equivalent to something like nvm install [major version] --reinstall-packages-from=$(nvm version). There could also be an optional switch to delete the old version.
I've been using this in my profile, which isn't quite as nice as I have to specify the new version to install:
# nvm-upgrade NEW_VERSION
#
# Installs the specified version of node via nvm, reinstalling all global
# packages from the currently activated version of node. Does not delete
# the previous node version.
function nvm-upgrade() {
  nvm install $1 --reinstall-packages-from=`nvm version`
  nvm use $1
  npm install -g npm
  npm upgrade -g
}

The primary use case for nvm isn't "i have one version of node and i'm using nvm to manage it".
nvm alias default node and nvm install node --reinstall-packages-from=node is usually sufficient.
I'll leave this open, but a feature like this necessitates many assumptions about the user's use cases and environment that I don't think will be worth it.

On the contrary, I have several versions of node installed, but I really only need 0.10.x, 0.12.x, 4.3.2 (thanks Lambda), and 5.x. It's not a huge problem, but would just be a nice feature to have (and to be able to instruct my less conscientious colleagues to use)

Makes sense. It's not an unreasonable request at all, I'm just not yet convinced the cost/benefit of the complexity is worth it :-)

Gotcha, that's fair. My shell scripting knowledge isn't perfect, but I'll see if I can come up with a viable PR

+1

The script in the first message is useful, thanks.

@spencerhakim
I think you shouldn't do this:
  npm install -g npm
  npm upgrade -g

Because it's hard to manage versions when there is a step that installs something unpredictable (last npm version? last version of npm dependencies?).
I think this specific behavior will not land on nvm code base.
I know that it's just a copy and paste of what you are doing on your environment, but I'm taking the opportunity to recommend you to always choose reproducible environment over last packages versions as much as possible.

I definitely wouldn't add npm upgrade -g or anything like that - nvm will only ever install the npm that comes with the given version of node.

This would be great. Literally every time I update npm I have to google for the --reinstall-packages-from= bit.

So, despite my earlier comment, the next release of nvm will have nvm install-latest-npm and nvm install node --latest-npm, since the question "what is the latest npm version that works on a given node version, and how do i get it?" is surprisingly hard to answer.
Thus, nvm upgrade will be able to be largely achieved with nvm install node --reinstall-packages-from=node --latest-npm.


What is the expected behaviour if I deploy a temporarily incorrect deployment manifest?

We have a scenario in which docker images (of iot edge modules) are pushed to container registry after applying deployment manifest (maybe a week delay). In that particular case, what happens to the deployment?
a) The IoT edge gateway, ignore the deployment forever
b) The IoT edge gateway continues running existing images. At the same time, keep retrying forever until the specified docker images are available in container registry.
I felt the docs section should explain how the edge runtime behaves when a bad deployment happens and so raised the query here. Kindly guide if this query does not fit in the docs section.
I could elobarate on the usecase if that is a need. To allow for the deployment, I skip the deployment validation with SKIP_MODULE_IMAGE_VALIDATION flag in Azure Devops.

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

ID: 877a68cb-b6ad-52a2-64ff-2965e936b3a0
Version Independent ID: 07f38c9a-bafb-bfeb-01e9-cfc420d844b5
Content: Deploy modules at scale in Azure portal - Azure IoT Edge
Content Source: articles/iot-edge/how-to-deploy-monitor.md
Service: iot-edge
GitHub Login: @kgremban
Microsoft Alias: kgremban

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

Please find the below answers.

What is the expected behaviour if I deploy a temporarily incorrect deployment manifest?

Each IoT Edge device runs at least two modules: $edgeAgent and $edgeHub, which are part of the IoT Edge runtime. IoT Edge device can run multiple additional modules for any number of processes. Use a deployment manifest to tell your device which modules to install and how to configure them to work together.
All IoT Edge devices must be configured with a deployment manifest. A newly installed IoT Edge runtime reports an error code until configured with a valid manifest.

We have a scenario in which docker images (of iot edge modules) are pushed to container registry after applying deployment manifest (maybe a week delay). In that particular case, what happens to the deployment?
a) The IoT edge gateway, ignore the deployment forever
b) The IoT edge gateway continues running existing images. At the same time, keep retrying forever until the specified docker images are available in container registry.

When you deploy a module, Each item in the deployment manifest contains specific information about a module and is used by the IoT Edge agent for controlling the module’s life cycle. Some of the more interesting properties are:
status – The state in which the IoT Edge agent places the module. Usually, this value is set to running as most people want the IoT Edge agent to immediately start all modules on the device. However, you could specify the initial state of a module to be stopped and wait for a future time to tell the IoT Edge agent to start a module. The IoT Edge agent reports the status of each module back to the cloud in the reported properties. A difference between the desired property and the reported property is an indicator of a misbehaving device. The supported statuses are:
Downloading
Running
Unhealthy
Failed
Stopped

I felt the docs section should explain how the edge agent behaves when a bad deployment happens and so raised the query here. Kindly guide if this query does not fit in the docs section.

I would suggest you, please go through the below documentation to
Understand the Azure IoT Edge runtime and its architecture
Learn how to deploy modules and establish routes in IoT Edge
I hope this helps. If you feel still information is missing, please do let us know we can help you accordingly.

Thanks for the information on status field. That helps in better understanding of iotedge. Then, I went through the docs mentioned, but none of them mentions about rollbacks of deployment.
I felt, the original question remains unanswered. The question in simple terms is: If I depoy a bad deployment manifest (e.g. with non existing docker image), will iotedge agent ignore it and fallback/rollback to last known working deployment manifest? To avoid ambiguity, I humbly request to answer in 1 word: YES or NO or DEPENDS. If DEPENDS, please help with reasoning.
Please close this issue ticket if you felt otherwise.

@kiranpradeep Sorry to hear that I couldn't address your question.
As I mentioned, if it is new deployment, IoT Edge runtime reports an error code until configured with a valid manifest.
Once an IoT Edge device receives a deployment manifest, it downloads and installs the container images from the respective container repositories, and configures them accordingly. Once a deployment is created, an operator can monitor the deployment status to see whether targeted devices are correctly configured.
If a deployment is already done and you are doing it again with a wrong deployment manifest, I don't have a definitive answer to say yes or no but it depends how you configure your deployments.
Regarding the callback:
Deployments can be rolled back if you receive errors or misconfigurations. Because a deployment defines the absolute module configuration for an IoT Edge device, an additional deployment must also be targeted to the same device at a lower priority even if the goal is to remove all modules.
Please check Understand IoT Edge automatic deployments for single devices or at scale and see if it helps to get an idea how different deployments works.
Current doc is intended to show the deployment and monitor modules.
If you feel this info be added to the doc, you can specify your scenario with more examples and we will review it and add to the doc appropriately.
Let us know if you have further questions.

Thanks for detailed explanation. Closing.

@kiranpradeep to add more information to the answer, if you apply a deployment manifest to a device and the edgeAgent is unable to pull a module image, the device will not roll back to an old version. If you specify a module, and the runtime can’t pull it, it will attempt again, based on the retry policy, which by default backs off over time until it retries every 5 minutes.

@kgremban Thanks a lot. That was exactly the piece of information I was looking for.


install commands

In the install documentation there is probably 1 line missing:
before executing "grunt", bower has to install all the dependencies listet in the brower file. therefore the list should be as follows:
git clone git@github.com:AF83/koalab.git
cd koalab
cp config/server.json.example config/server.json
$EDITOR config/server.json
npm install -g grunt-cli
npm install .
bower install
grunt
node koalab.js
firefox http://localhost:8080/
In our case not executing bower install lead to a reference error by the $ command, as zepto was not available.
One question: could you please provide the version without the mozilla persona authentication, as it works in the demo version on http://koalab.af83.com/ ?

Hi,
thanks for your message.
For bower install,  it was supposed to be ran by npm, as a postinstall step (cf https://github.com/AF83/koalab/blob/master/package.json#L38). But seems like it doesn't work. So, I've put it in README.
For the demo version, the code is in the repository. It's just a flag in the config/server.json file:
{
  "authorized"<EMAIL_ADDRESS>  "demo": true,
  "port": 8080,
  "persona": {
    "audience": "http://koalab.af83.com"
  },
  "mongodb": {
    "host": "localhost",
    "database": "koalab_prod"
  }
}


Proposal: Collect whether installations have customized access polilcies

What question will this help us answer?
Is carrying the access policy around in the database for users to edit a used feature?
Do people use Pulp in a multi use / domain framework?
How much do people customize their RBAC system?
What is a specific example of the data to be gathered?
rbac_stats:
  users: 42
  groups: 12
  domains: 3
  customized_roles: 18
  customized_access_policies: 5

How will this metric be stored in the database or gathered at runtime?
Gathered at runtime.
Will the gathering and/or storage of this cause unacceptable burden/load on Pulp?
Just a fixed number of database calls, O(1). No permanent storage.
Is this metric Personally-Identifiable-Data?
Data is anonymous, but can still be uniquely identifiable [If your installation is the only one with 23 customized policies.].
As anonymous as pure numbers.
How can we sanitize this output?
What pulpcore version will this be collected with?
>= 3.23
Alternative proposal(s)
Option 1
Option N
Summarization
Collect number of systems with certain number of .
Visualization
Pie chart of last day or time series of stacked graphs. Maybe Pie chart averaged over 7 days?

Technically, domains is distinct from RBAC, can the domain number be moved outside of the RBAC data structure.
Also:

what does the domains number report if domains is disabled?
what does the domains number report if it's enabled but only the default domain is used?

We should distiguish systems with one and switched off domains.

Technically, domains is distinct from RBAC, can the domain number be moved outside of the RBAC data structure.

Discussion: people are receptive to either "in" rbac stanza (since domains require rbac) or as it's own thing

what does the domains number report if domains is disabled?

Discussion: return 0 if disabled. Or, make it not-returned?

what does the domains number report if it's enabled but only the default domain is used?

Discussion: returns "1"

This is expected to generate 5 new graphs.


Compile OpenBLAS for Android problem

Hi, xianyi,
I am using Ubuntu to compile OpenBLAS for Android platform, and follow exactly the command:
export PATH=/tmp/my-android-toolchain/bin:$PATH
make HOSTCC=gcc CC=arm-linux-androideabi-gcc NO_LAPACK=1 TARGET=ARMV7
by changing the right toolchain path. But make complains that it cannot find "stdio.h" which I think is located in NDK Android platform folder, in my case, is android-ndk-r10e/platforms/android-19/arch-arm/usr.
Could you tell me how to modify the Makefile to let it find the right Android platform dependency? It is too complicated for me to do it myself. Thanks a lot.

I think I have find the answer [1]. Do not know how to delete this issue. Sorry.
[1] http://developer.android.com/intl/zh-cn/ndk/guides/standalone_toolchain.html

I was able to build libopenblas.so for Android using the below simple commands,
Step 1: Install standalone toolchain as given here (Choose your API level accordingly)
Step 2: Make OpenBLAS using the below make command
make TARGET=ARMV8 ONLY_CBLAS=1 AR=aarch64-linux-android-ar CC="aarch64-linux-android-clang" HOSTCC=gcc -j4
Thanks,
San


Develop Distortion Correction Function

We need a function that will apply a distortion correction as part of the L2b->L3 DRP step.

Already part of issue #4. Closing.


Space consistency on columns prefix and suffix

This depends PR on https://github.com/Laravel-Backpack/CRUD/pull/3435
It's a breaking change on v4.1.

A new inspection was created.

A new inspection was created.

The inspection completed: No new issues

The inspection completed: No new issues

A new inspection was created.

The inspection completed: No new issues

Fixed on https://github.com/Laravel-Backpack/CRUD/pull/3936.


Fix incorrect parameter name in script in Azure price doc

I updated the script to rename BILLING_ACCOUNT_NAME to BILLING_ACCOUNT_ID because I thought the Azure distinction between name (a numeric value in the case of billing accounts) and display name (a human-readable description) would be confusing to users. But I missed one instance of the old name! :sob:

Thanks for looking through this @babbageclunk ! approving.


Would you mind not calling the protobuf repository protobuf?

Please go to Stack Overflow for help and support:
http://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug or a feature request.
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):

Yes

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

Ubuntu 16.04

TensorFlow installed from (source or binary):

Source

TensorFlow version (use command below):

1.1.0

Python version:

2.7.12

Bazel version (if compiling from source):

0.5.2

CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Currently the protobuf repo in workspace.bzl is called @protobuf. Unfortunately, in github.com/grpc/grpc, the protobuf is bind to @com_github_google_protobuf//:protobuf....So if I have anything that uses the GRPC repository it cannot be built together with Bazel.
I tried rename all @protobuf// to @com_github_google_protobuf//, but it does not work, presumably because the patch file for protobuf get in the way.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

What do you mean? What problem are you seeing?

In the workspace.bzl, tensorflow binds @protubuf to the protobuf repository, while https://github.com/grpc/grpc/blob/master/WORKSPACE binds @protobuf to the build target (not the repo). Suppose I have a third repository and pulling in both tensorflow and grpc as external repositories, the build will fail due to name clash, the error message is something like external/protobuf does not have attribute testonly, which basically says Bazel is trying to use @protobuf as a target but in fact the name is pulled in as a repo by tensorflow.

Got it. I don't see a downside to renaming the protobuf. Mind sending a PR to @gunan ?

Can I work on a PR for this issue?

Sure.

Hi @lakshayg , @jhseu is upgrading grpc. #11768


Logos lexes comment start as an operator

I have a bunch of rules in my lexer, including this two:
pub enum Token {
    #[regex(r"/\*", priority = 616)]
    CommentStart,
    #[regex(r"[~!@#\^\&|`?+\-*/%<>=]+")]
    Op,
}

Then I pass /**/ to the lexer as an input.
Expected behavior: Logos should lex /* as a comment start
Actual behavior: Logos scans /**/ as an operator.
PS: This is a complete list of tokens: https://gist.github.com/suhr/d0d27a76a759821718c278a048ea1cc6

This is expected behavior because the pattern for Op can match the whole /**/, which is the longest possible match.

Yeah, I figured this out. Still super annoying though.
By the way, I'm implementing my own lexer generator based on regular expressions derivatives. It supports regular expression intersection and complement, so I can express my intent more directly.
I'd like to have some ideas about code generation from DFA. It seems that Logos relies a lot on tail calls, but the problem is Rust doesn't really guarantee tail call elimination. I believe this problem can be workarounded by using labeled break/continue for strongly connected components of DFA, but I didn't quite figure out yet how to do it.
Directed graphs with cycles are tricky.

Interesting you mention regular expression derivatives, checkout this library implementing this idea in python: https://github.com/MichaelPaddon/epsilon
I was wondering whether or not [Brzozowski derivatives[(https://en.wikipedia.org/wiki/Brzozowski_derivative) are used / considered in this library?

By the way, my lexer is already useful: https://github.com/suhr/sana. It is not as fast as Logos though.
An important thing if you consider implementing something similar: don't compute derivative classes recursively. The join operation is associative, so you can compute it from a flat list which is much more efficient.
That's how it's done in Sana: https://github.com/suhr/sana/blob/master/sana_core/src/regex.rs#L546


AZFWMigrationScript.ps1 fix and improvement

Corrected the logic of GetApplicationRuleCmd enabling it to use one or more SourceAddresses or one or more SourceIPGroups. (For rules using IPGroups, the script was crashing because of the empty SourceAddress).
Added a second ResourceGroup variable that gives the flexibility to create the Policy in a different Resource Group.

#please-close


Add the ability to resend the confirmation to a user on the admin/users page

On the user information page

if the user is not verified there will be a button that will resend the verification text message to the registered phone number
Add ability to change the verified flag

After verification

Check if they should get a gift card and create the record automatically

There is a skip button on the page where you enter the verification code, so this is how people were able to continue without being verified.

I changed the user update form so if they are not verified it will send them through the verification process when they save the account update form.
The code is/was set up to only allow them to update their phone number if they hadn't been verified.  I left that as is.
I also added an "Is Verified " checkbox to the admin participant form.  This just changes the flag without sending the verification message.  The reason for this is that the message assumes you are on the "enter verification code" page and just sends the code itself.  We could talk about making other changes to this process if you want.


Get-SPOTenant properties not documented

When running Get-SPOTenant the following properties show up and are lacking documentation.
SyncPrivacyProfileProperties
EmailAttestationRequired
EmailAttestationReAuthDays
StorageQuota (confirmation on Unit)
SyncPrivacyProfileProperties

@petey121175 if you agree with @get-itips Please close this PR. Thanks

Hello,
I am the engineer who requested the PR to be created. The documentation in 'Set-SPOTenant' is severely lagging for these properties. It just splits the name of the property in the name of documentation and doesn't provide any useful information. For example: -
-EmailAttestationRequired
Sets email attestation to required.
(what is email attestation here? Which email are we referring to)?
-SyncPrivacyProfileProperties
This parameter enables the synchronization of privacy profile properties.
(What are privacy profile properties? Where do they exist? Are we referring to the 'Show To' column in SharePoint User Profiles)? What are we synchronizing them with)? Are you referring to Delve as the destination?
-EmailAttestationReAuthDays
Sets the number of days for email attestation re-authentication. Value can be from 1 to 365 days.
(what is email attestation here? Which email are we referring to? Who receives these emails?

StorageQuota

This property does not exist in Set-SPOTenant. What's the Unit?
Some additional properties returned by Get-SPOTenant that do not exist in Set-SPOTenant are: -
AllowOverrideForBlockUserInfoVisibility
Workflows2013State

@tseward

@tseward Gentle reminder


Make Tobira work with multi node Opencast setups

Currently, Tobira assumes there is one Opencast server that can do everything: provide the harvest API, host Studio and have an ingest API. This is of course not universally true in multi node OC setups.
One easy option would be to just add new config options to Tobira to be able to configure separate servers for each use case. Alternatively, much more involved, one could talk to the OC service registry to figure out which nodes to talk to.

We should put auto-discovery at least in the backlog. If users want to add or remove ingest servers, for example, you would need to re-configure Tobira every time, while the locations are just one API call away.
Short-term, specifying them manually should be fine.


ML 2.5: Linear Regression (A)

Description
Welcome to 'ML' Team, good to see you here
This issue will helps readers in acquiring all the knowledge that one needs to know about Linear Regression.
To get assigned to this issue, add your Batch Numbers mentioned in the spreadsheet of "Machine Learning", this issue is only for Audio content.
Domain : Machine Learning
Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.

"Issue Title" and "PR Title should be the same. Include issue number along with it.

Changes should be made inside the Machine Learning/Machine_Learning/Supervised_Machine_Learning Branch.

Follow Contributing Guidelines & Code of Conduct before start Contributing.

This issue is only for 'GWOC' contributors of 'Machine Learning' domain.

To be Mentioned while taking the issue :

Full name
Batch Number
GitHub Profile Link
Which type of Contribution you want to make :

[ ] Audio

All the best. Enjoy your open source journey ahead. 😎
Domain
Machine Learning
Type of Contribution
Audio
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Name: Tisha Jhabak
Batch 6
GitHub: https://github.com/TishaJhabak1014
Audio
( no issues assigned yet)

Ishaan Yogesh Khanduja
ml-batch-6
GitHub Profile Link : https://github.com/ishaan107
Which type of Contribution you want to make : Audio
/assign

Sarmistha Sarna
Batch :01
https://github.com/ron352
type : audio
Can I work on this?


feature: Update the Hijacked Routes without Restarting Pods

What would you like to be added?
feature: Update the Hijacked Routes without Restarting Pods
Why is this needed?
https://github.com/spidernet-io/spiderpool/pull/2774#discussion_r1419963039
How to implement it (if possible)?
No response
Additional context
No response

There are no plans for it in an upcoming release.

two approachs :
(1) spider pool agent implements it
(2) sidecar ?

Maybe option 1 is better, we can refer to https://github.com/k8snetworkplumbingwg/multus-dynamic-networks-controller


lua: support logging methods on all envoy lua objects

Previously these methods were only supported on lua http filter request/response handles.
Risk Level: Low
Testing: Added new tests
Docs Changes: Updated docs
Release Notes: Added release note

confliction resolving is necessary now.
/wait


Settings TOC does not expand to full when the VS Code window is resized

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: Version: 1.65.0-insider Commit: 3112460dc48ce7e557ea9feeaae04b912164b48b
OS Version: macOS Big Sur 11.6.3

Steps to Reproduce:

Open the settings view, resize the VS Code window until the TOC disappears
Resize the window back to its original size, observe that the TOC did not re-expand fully:

https://user-images.githubusercontent.com/51720070/155616329-560a5a24-427a-4c94-ad6a-aef6081ff264.mp4

Duplicate of https://github.com/microsoft/vscode/issues/143690


The "tracy" sample profiler

https://github.com/wolfpld/tracy

Maybe also some comparison to Optick and other similar tools?

perf, anyone? #128


Defaults updates financials biopower o&m

Update:

O&M to remove values we've been double counting  (reference: https://github.com/NREL/SAM-documentation/blob/master/Defaults References and Tools/Fall 2021/O%26M Updates 2021.12.02.xlsx)
Interest rates to match ATB (Fixes https://github.com/NREL/SAM/issues/553)
PPA prices for PV front of meter cases to account for decreased interest rates (reference: https://emp.lbl.gov/pv-ppa-prices)
Reduce inverter costs in reserve accounts (reference: https://www.nrel.gov/docs/fy19osti/73822.pdf)
Apply inflation to biopower cost inputs

Run the test script to see the results with these new defaults.

A few issues:

PVWatts / Single Owner NPV is negative. NPV is very sensitive to PPA price in the 4-5 cent/kWh range:

For PV configurations, the default "Annual land lease cost" ui_land_lease of $100/yr and "Land lease escalation rate" om_land_lease_escal of 1%/yr from the Operating Costs page appear to be left over test values. I think these should both be set to zero unless we have a source for those values. This would increase the default PV / Single Owner NPV by about 10%. The ATB documentation mentions land lease is included in O&M cost and I could not find it broken out as a separate cost in the LBL report it cites.

Although this is not necessary, it would be nice for the default IRR target and IRR target year to match the results for "Specify PPA price" mode so that if a user switches from Specify PPA price to Specify IRR target without changing any other inputs, they get the same results. This would be tedious to do because it requires running a simulation for all configurations -- something that could be done by LK script.

For all FOM financial configs, retail rate inputs on Electricity Purchases page should have the following values in case user switches from PPA to retail rates.
ur_monthly_fixed_charge = 0
ur_ec_tou_mat = [ [ 1, 1, 1e+38, 0, 0.045, 0 ] ]
ur_dc_enable = 0
ur_dc_flat_mat = [ [ 0, 1, 1e+38, 0 ], [ 1, 1, 1e+38, 0 ], [ 2, 1, 1e+38, 0 ], [ 3, 1, 1e+38, 0 ], [ 4, 1, 1e+38, 0 ], [ 5, 1, 1e+38, 0 ], [ 6, 1, 1e+38, 0 ], [ 7, 1, 1e+38, 0 ], [ 8, 1, 1e+38, 0 ], [ 9, 1, 1e+38, 0 ], [ 10, 1, 1e+38, 0 ], [ 11, 1, 1e+38, 0 ] ]
ur_dc_tou_mat = [ [ 1, 1, 1e+38, 0 ] ]

Power Tower PPA price input of $0.13/kWh seems high. Calculated PPA price in SAM 2020.11.29 is $0.1032 with end of project IRR of 12.82% (which also seems high to me). Consult with CSP folks to see what default PPA prices / IRR they would like to see for CSP configs.

A few issues:

PVWatts / Single Owner NPV is negative. NPV is very sensitive to PPA price in the 4-5 cent/kWh range:

For PV configurations, the default "Annual land lease cost" ui_land_lease of $100/yr and "Land lease escalation rate" om_land_lease_escal of 1%/yr from the Operating Costs page appear to be left over test values. I think these should both be set to zero unless we have a source for those values. This would increase the default PV / Single Owner NPV by about 10%. The ATB documentation mentions land lease is included in O&M cost and I could not find it broken out as a separate cost in the LBL report it cites.
Although this is not necessary, it would be nice for the default IRR target and IRR target year to match the results for "Specify PPA price" mode so that if a user switches from Specify PPA price to Specify IRR target without changing any other inputs, they get the same results. This would be tedious to do because it requires running a simulation for all configurations -- something that could be done by LK script.
For all FOM financial configs, retail rate inputs on Electricity Purchases page should have the following values in case user switches from PPA to retail rates.ur_monthly_fixed_charge = 0
ur_ec_tou_mat = [ [ 1, 1, 1e+38, 0, 0.045, 0 ] ]
ur_dc_enable = 0
ur_dc_flat_mat = [ [ 0, 1, 1e+38, 0 ], [ 1, 1, 1e+38, 0 ], [ 2, 1, 1e+38, 0 ], [ 3, 1, 1e+38, 0 ], [ 4, 1, 1e+38, 0 ], [ 5, 1, 1e+38, 0 ], [ 6, 1, 1e+38, 0 ], [ 7, 1, 1e+38, 0 ], [ 8, 1, 1e+38, 0 ], [ 9, 1, 1e+38, 0 ], [ 10, 1, 1e+38, 0 ], [ 11, 1, 1e+38, 0 ] ]
ur_dc_tou_mat = [ [ 1, 1, 1e+38, 0 ] ]

Power Tower PPA price input of $0.13/kWh seems high. Calculated PPA price in SAM 2020.11.29 is $0.1032 with end of project IRR of 12.82% (which also seems high to me). Consult with CSP folks to see what default PPA prices / IRR they would like to see for CSP configs.

I pushed the land lease fix to this branch.
I addressed some of the utility rate issues in https://github.com/NREL/SAM/pull/790 but need to upate theur_ec_tou_mat and the ur_dc_tou_mat

@cpaulgilman The calculated PPA price for PVWatts - Single Owner with the updated utility rates and a IRR target of 11% is 4.96 cents/kWh. Should we use that?

Would be good to have something close to the Detailed PV case -- if there's PPA price for a lower IRR that results in a positive NPV that would be good. Nitpicking here, so I wouldn't hold this up for that.

My default value for utility scale systems of $13/kW-yr does not include land payments. Should I increase that based on https://github.com/NREL/SAM-documentation/blob/master/Defaults References and Tools/Fall 2021/Copy of OandM TabularForm_Graphic.xlsx? I think land prep on the installation costs should remain the same.

If I increase the fixed O&M to $15/kW-yr (to account for land lease) and the PPA price to $0.048/kWh, I get a positive NPV for both Detailed PV-Single owner and PVWatts-Single owner.


how to centered collectionview with loadmore

How to center collection view with load more pull to refresh all data.

Hey @DGP13,
Your question pertains to UICollectionView's in general and not the UICollectionViewFLowLayout` subclass in this package.
Your question would be best answered by a looking through Apple's UICollectionView documentation. Also this might be a decent resource to look at: https://cocoacasts.com/how-to-add-pull-to-refresh-to-a-table-view-or-collection-view
I am going to close this since your question is off-topic.


feat(calls): Add user to active call

What this PR does 📖

Adds the user to the active call window (chat only). The user profile pic will be displayed at bottom left of the ui

the workflow just plain failed. idk what happened there. linter looks fine though. @phillsatellite


update French translations

see @atillmanns  email Feb 7th with attached spreadsheet

done and updated translation excel data in package


Migrate to hatch

Fixes https://github.com/jupyterlab-contrib/jupyterlab_pytutor/issues/8
Fixes https://github.com/jupyterlab-contrib/jupyterlab-pytutor/issues/7
Might also fix https://github.com/jupyterlab-contrib/jupyterlab-pytutor/issues/6
To test on Binder: https://mybinder.org/v2/gh/jtpio/jupyterlab_pytutor/hatch
Changes

[x] Use the migration script to update to hatch: https://hatch-jupyter-builder.readthedocs.io/en/latest/source/how_to_guides/migrating_data_files_projects.html
[x] Align with https://github.com/jupyterlab/extension-cookiecutter-ts/
[x] Require Python 3.7+
[x] Update Node to 18 on Binder

To test on Binder: https://mybinder.org/v2/gh/jtpio/jupyterlab_pytutor/hatch

Working fine on Binder now:

Will be following up with a release with the releaser.


[VS Code] secret_password_clear_sync() failed with error code 24: Error calling StartServiceByName for org.freedesktop.secrets: Timeout was reached

Error:
secret_password_clear_sync() failed with error code 24: Error calling StartServiceByName for org.freedesktop.secrets: Timeout was reached
Steps to Reproduce:

Install the extension
sudo apt install libunwind8 liblttng-ust0 libcurl3 libssl1.0.0 libuuid1 libkrb5-3 zlib1g libicu55
sudo sh<EMAIL_ADDRESS>/usr/share/code/code /home/elandry/.vscode/extensions/ms-vsliveshare.vsliveshare-0.2.399/cascade.json
Click on "Sign In"
In the browser, login to the Outlook365 account that I got the invite
Use the resulting auth code in VS Code
Get the error message

Version Data

extensionName
VSLS

extensionVersion
0.2.399

protocolVersion
2.1

applicationName
VSCode

applicationVersion
1.22.2

platformName
Linux

platformVersion
4.4.0-109-generic

@ericslandry Can you also verify whether or not gnome-keyring is present?
#205 was reported earlier with a similar error that was missing that dependency.  Apparently our error message is not catching it.
Also, which distribution of Linux are you using?

Here's my gnome-keyring:
dpkg -l |grep gnome-k
ii  gnome-keyring                                 3.18.3-0ubuntu2                            amd64        GNOME keyring services (daemon and tools)
ii  libgnome-keyring-common                       3.12.0-1build1                             all          GNOME keyring services library - data files
ii  libgnome-keyring0:amd64                       3.12.0-1build1                             amd64        GNOME keyring services library
ii  libp11-kit-gnome-keyring:amd64                3.18.3-0ubuntu2                            amd64        GNOME keyring module for the PKCS#11 module loading library
ii  libpam-gnome-keyring:amd64                    3.18.3-0ubuntu2                            amd64        PAM module to unlock the GNOME keyring upon login

I'm using Xubuntu 16.04
cat /proc/version
Linux version 4.4.0-109-generic (buildd@lgw01-amd64-038) (gcc version 5.4.0 20160609 (Ubuntu 5.4.0-6ubuntu1~16.04.5) ) #132-Ubuntu SMP Tue Jan 9 19:52:39 UTC 2018

@ericslandry Interesting. It would be useful to see your logs to see what might be up.

Could you run the "Live Share: Export Logs" command and either add the resulting zip here or shoot them to<EMAIL_ADDRESS>Could you see if there are any errors in the "Console" tab after doing About > Toggle Developer Tools?

@ericslandry Just to sanity check - I did a fresh install of Xubuntu 16.04 and though I had to install two libs, I was able to sign in without an error.  It would be great to get your logs as I mentioned in my previous comment since it's possible this has nothing to do with the distro of Linux.

@Chuxel I've attached the file vscode-logs.zip
Here are the dev tools errors:
3/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:260 [Extension Host] Invalid message item: undefined
t.log @ /usr/share/code/resources/app/out/vs/workbench/workbench.main.js:260
/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:2217 secret_password_clear_sync() failed with error code 24: Error calling StartServiceByName for org.freedesktop.secrets: Timeout was reached
e.onDidNotificationChange @ /usr/share/code/resources/app/out/vs/workbench/workbench.main.js:2217
2/usr/share/code/resources/app/out/vs/workbench/workbench.main.js:260 [Extension Host] Invalid message item: undefined
t.log @ /usr/share/code/resources/app/out/vs/workbench/workbench.main.js:260

Can you verify "Settings > Session and Startup > Advanced > Launch GNOME services on startup" is checked?

I figured it out - I was remote desktoping into my workstation using xrdp that was running MATE desktop. Now that I'm back in the office, it works. Thanks anyways.


Fix not to push context multiple times when notifying multiple Reques…

…tLogListener
Motivation:
We push (Client|Service)RequestContext to thread local before calling RequestLogListener.onRequestLog().
When there are multiple RequestLogListeners, we push the context every time before calling onRequestLog.
This could be a problem when a user sets an expensive hook using ctx.onEnter(). So we should fix to just push once.
Modification:

Push context only once before notifying multiple listeners

Result:

Better performance

Codecov Report

Merging #2227 into master will increase coverage by <.01%.
The diff coverage is 70%.

@@             Coverage Diff              @@
##             master    #2227      +/-   ##
============================================
+ Coverage     73.52%   73.53%   +<.01%
- Complexity     9720     9722       +2
============================================
  Files           848      848
  Lines         37535    37545      +10
  Branches       4610     4613       +3
============================================
+ Hits          27599    27609      +10
+ Misses         7575     7564      -11
- Partials       2361     2372      +11

Impacted Files
Coverage Δ
Complexity Δ

...corp/armeria/common/logging/DefaultRequestLog.java
81.1% <60%> (-0.51%)
184 <0> (-3)

...eria/common/logging/RequestLogListenerInvoker.java
72.72% <73.33%> (-27.28%)
5 <4> (+3)

.../armeria/client/endpoint/dns/DnsEndpointGroup.java
80.64% <0%> (-3.23%)
13% <0%> (-2%)

...rp/armeria/common/stream/DefaultStreamMessage.java
89.36% <0%> (-1.42%)
53% <0%> (-1%)

...inecorp/armeria/server/HttpResponseSubscriber.java
84.18% <0%> (-1.29%)
73% <0%> (-3%)

.../linecorp/armeria/client/Http2ResponseDecoder.java
60.83% <0%> (-0.84%)
32% <0%> (-1%)

...com/linecorp/armeria/server/HttpServerHandler.java
81.39% <0%> (+0.66%)
82% <0%> (+1%)
:arrow_up:

...a/com/linecorp/armeria/common/util/Exceptions.java
39.47% <0%> (+0.87%)
25% <0%> (+1%)
:arrow_up:

...com/linecorp/armeria/server/saml/SamlEndpoint.java
65% <0%> (+2.5%)
11% <0%> (+1%)
:arrow_up:

...com/linecorp/armeria/client/OneEventLoopState.java
71.42% <0%> (+4.76%)
7% <0%> (+1%)
:arrow_up:

... and 3 more

Continue to review full report at Codecov.

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

Thanks for reviewing!


Add search command

This PR adds a search command which takes optional -n (limit the number of results) and --json (output as JSON) arguments.

Thank you. Merged in https://github.com/mattn/bsky/commit/c96424f7260ff210a53a73aecc772c2efaddbb07


chore(main): release 1.0.5

:robot: I have created a release beep boop
1.0.5 (2023-01-28)
update

修改配置文件 (00f587c)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/School-of-Website-Engineering/TinyWaste/releases/tag/v1.0.5 :sunflower:


Bookmarks

Does/will changes made to ITextDocumentSnapshot / ITextViewSnapshot preserve existing Bookmarks?

@LeeMSilver, yes, changes made by extensions do preserve existing bookmarks, the same way user editing code - unless some destructive edit happens, e.g. the whole line containing bookmark is deleted.


paginator at homepage

i wanted to add paginator on the hompage while having all the cards at one page i wanted to make it more user interactive by adding paginator using django paginator

You can start working on it.


Follow up on transform operation PR

Some cleanup after https://github.com/gradle/gradle/pull/23592.

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


Inject Javascript or CSS

First, this is really awesome, thanks so much for releasing this!
  For example, I use something like this on Chrome to fix a bunch of layout things about GMail.  It would be great to be able to do something similar with nativefier.  For example, just to be able to specify an arbitrary JS file when building the app that would run when any page finishes loading in the app.
Just a thought.  Otherwise, truly love this thing.

Yup, there was some work on this in #47 and is mentioned in #108 as well, I plan to expose something like this via the command line

Looking forward to your work. Bothered by the electron webview issues for days.

This has been implemented since 166aa8962e791a2dbf734deaac3fcaecaab6fbf6