Log based replication background worker and log parser

Code originally written by Kfir Payne<EMAIL_ADDRESS>Explain the changes

Add log based replication background worker, responsible for scanning all the replication rules, and based on candidates provided by the log parser, decide which objects should be deleted from pre-existing replicas.
Add log parser, responsible for getting the next log to check, parse the log, and returning a list of keys (candidates) for the logs replication scanner to check.
Add a function to replication store that returns all the replication rules containing log based replication info.

This PR adds support for deletion of the replicas (replicated via currently existing replication scanner). AWS logs are not chronological in nature and also do not guarantee that each event will be captured in the logs (during testing it was observed that the logs kept missing crucial events such as of object deletion as well) which AWS calls "best effort log delivery". Due to these constraints we have the following caveats in the current implementation:

If the replication scanner encounters a delete type record, it will delete the object from the replica. However, if there was no log corresponding to the delete - the object will not be deleted from the replica.
Deliveries of the logs are excruciatingly slow. It may take more than an hour for a DELETE change to be replicated in a replica.
We most likely cannot implement a full fledged replication tool due to the fact that

Events might be missed (eg. missing a PUT - that won't be good).
Logs are not chronological. Logs do have timestamp in them which are accurate but that isn't of much help because the only guarantee that AWS gives is that timestamp in the name of the log file will always be greater than the timestamp of the events it stores (obviously, though).

[ ] Doc added/updated
[ ] Tests added

Lesson learnt. DeepScan does not understand optional chaining properly yet 🙂 .


Add Ayedo to Adopters

Signed-off-by: Hrittik Roy<EMAIL_ADDRESS>Add Ayedo to Adopters.md

Thanks @cwayne18! 🚀


Allow getting type from define() node

fixes #286
DefinitionResolver::getTypeFromNode() was missing a case for define() function call nodes.
This caused the stubs file to contain lots of constants without an associated type.
Added the missing case and added checks to other parts concerning define() nodes so that incomplete defines (only one argument) are not resolved as constants.

Would it be possible to add a test that fails before the changes to prevent a regression?

Sure. Should I maybe add a folder for regression tests?

No, all tests are to prevent regressions.


libkbfs: introduce SyncAll for syncing multiple files

This PR introduces KBFSOps.SyncAll, which syncs all dirty files together to the server in a single MD update.  Note that this PR does not enable SyncAll use by kbfsfuse yet, it is only used in tests right now.
This breaks up the original folderBranchOps.SyncLocked into two parts, so it can be used by SyncAll. SyncAll uses the chains and prepper code also used by CR to summarize a set of changes into a single MD update and sync them all together.
This ended up exposing some issues in conflict resolution, especially around how it deals with resolutionOps and syncOps that are part of a previous resolution.  I'll try to give some details about each one in PR comments.
Issue: KBFS-2069

Ok I'm done making annotations. Ready for review!

Thanks for the comments @akalin-keybase.  I addressed some of them and responded to the others.  Ready for another look!

(Test flake should be fixed in #916, is unrelated to this PR.)


Adopts standard input syntax for GitHub Actions

Current
in action.yml

production_branch
staging_branch
dry_run

refs

lowerCamelCase<EMAIL_ADDRESS>kebab-case: https://github.com/actions/cache/blob/main/action.yml
snake_case<EMAIL_ADDRESS>

Result
in action.yml

token
repository
production-branch
staging-branch
label
draft
dry-run


x/pkgsite: add support for github's markdown footnotes

What is the URL of the page with the issue?
<EMAIL_ADDRESS>What is your user agent?
Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Screenshot
Github:

pkg.go.dev:

What did you do?
Added footnotes to README.md file per Github markdown syntax.
What did you see happen?
Works fine in Github. Footnotes not respected in pkg.go.dev.
What did you expect to see?
Footnotes should look the same in both places if possible.

This is interesting. footnotes don't seem to be mentioned in the github flavored markdown spec here: https://github.github.com/gfm/. Could you link us to official documentation of footnotes in github?

https://github.blog/changelog/2021-09-30-footnotes-now-supported-in-markdown-fields/
https://docs.github.com/en/enterprise-cloud@latest/get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax#footnotes


ChatCompletion request fails

Hi, I have the error:
OpenAiHttpException: Invalid URL (POST /v1/chat/completions)

Please help me

With this code works fine, but with createChatCompletion throws error. HELP ME

How ?

thank's very much.
Y try create but.

I also get the null error, but it happens when I try to add more than one chat message to the list, maybe there is a clue in that?

https://github.com/forestwanglin/openai-java


adjust readpool size by the number of cpu core

Signed-off-by: zhangjinpeng1987<EMAIL_ADDRESS>
What have you changed?
Adjust readpool size by the number of cpu core, make it works better under sysbench and other benchmark cases when using default configuration value.
What is the type of the changes?
Pick one of the following and delete the others:

Improvement (a change which is an improvement to an existing feature)

How is the PR tested?

Manual test (add detailed scripts or steps below)

[ ] benchmark

Does this PR affect documentation (docs) or should it be mentioned in the release notes?

If there is a document change, please file a PR in (docs) and add the PR number here.
If this PR should be mentioned in the release note, please update the release notes.

Does this PR affect tidb-ansible?
No

/bench

/bench

/run-integration-cop-push-down-test tidb-test=pr/899


Good page but!

In X64 calling convention i want find more explanation of r10-r15 register, thank you for you already work. sorry for my bad english i am french...

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

ID: dc07ad07-7d1c-e42d-506d-de67e4a02494
Version Independent ID: ed140094-1331-1502-6618-158307cd73f3
Content: x64 Architecture | Microsoft Docs
Content Source: windows-driver-docs-pr/debugger/x64-architecture.md
Service: unspecified
GitHub Login: @DOMARS
Microsoft Alias: domars

@maitreyotta, thanks for this input. I added links in a new see also to these resources in a new see also section:
X86-64 Wikipedia
AMD 64 Developer Resources
Refer to these resources for more information about the registers.


Include relations from emission coefficient to the processes which cause emissions

Description of the issue
We need to include some relations from emission coefficient to the processes which cause emissions.
Ideas of solution
primary energy production has_process_attribute emission factor
combustion has_process_attribute emission factor
industrial process has_process_attribute emission factor (when industrial process is implemented #534 )
energy transformation has_process_attribute emission factor (maybe we need to include subclasses of energy transformation or wait until #372 is solved and include relations like that:
coal power unit participates in energy transformation has_process_attribute emission coefficient)
Workflow checklist

[x ] I discussed the issue with someone else than me before working on a solution
[x ] I already read the latest version of the workflow for this repository
[ x] The goal of this ontology is clear to me

I am aware that

[x ] every entry in the ontology should have a definition
[x] classes should arise from concepts rather than from words

Not every energy transformation has emissions, e.g. transformation of electricity to heat. So has process attribute cannot be related to the general energy transformation, but maybe to subclasses.
For a coal power unit, there are a couple of (sub)processes:

Combustion of coal in a heater. This is at the same time an energy transformation of chemical energy in heat.
Energy transformation of heat to mechanical energy in a steam turbine.
Energy transformation of mechanical energy to electrical energy in an electromotive

The partial energy transformations can be abstracted to an overall energy transformation from chemical energy to electrical energy.  The emissions are solely associated to the combustion process. Concluding from that I would argue that we need the following statements in the OEO:

coal power unit participates in energy transformation
coal power unit participates in combustion
combustion has process attribute emission factor

Similar for industrial processes and energy production: Only a subset of these have emissions and hence an emission factor.

I think, a better solution would be a subprocess relation as emission is a process and we want to related this emission process to other processes. The emission factor is only a process attribute, so I think we should deal with the relations between the processes itself first.
Maybe someone of @OpenEnergyPlatform/oeo-general-expert-formal-ontology could give some input here?

I think it would help to use has part relations here for structuring.
For the example above, we could use a "overall chemical to electrical energy process" as a starting point. This process has the relations

has participant some coal plant
has part some "Combustion of coal in a heater"-combustion
has part some "heat to mechanical energy"-transformation
has part some "mechanical to electrical energy"-transformation

The combustion could have the relations

has part some "chemical energy to heat"-transformation
has part some emission

And the emission has the relation has process attribute some emission factor
This means that we have several different processes which are either proper parts of other processes or temporally coextensive to another process.

I would try to keep it as simple as possible.
We don't have the subprocess of a coal power unit in the ontology yet and I think they are not neccesary to describe an energy system model. We consider the coal power plant in the model more like a black box, which has some inputs, some outputs, some ratios between them, some costs etc.. One output of the process are the emissions and the emission factor describes how much emission per unit activity of that process is produced. So the emission factor is related to the (power generating/combustion) process, not to the emission itself.

Only a subset of these have emissions and hence an emission factor.

Can we solve that problem by changing the relation name to can have process attribute. Or explaining that the emission factor could also be zero?

One output of the process are the emissions and the emission factor describes how much emission per unit activity of that process is produced. So the emission factor is related to the (power generating/combustion) process, not to the emission itself.

According to our definition, emission is not an output but a process. The output you are talking about would probably be something like an emission value that has a mass unit as unit. We currently do not have this but can define such a value class if needed.

Only a subset of these have emissions and hence an emission factor.

Can we solve that problem by changing the relation name to can have process attribute. Or explaining that the emission factor could also be zero?

I am not sure whether it is useful to have statements in an ontology that are not clearly true or false. What do you think, @sfluegel05?

The output you are talking about would probably be something like an emission value that has a mass unit as unit. We currently do not have this but can define such a value class if needed.

Thank you for pointing that out. I think a class emission value is definitely helpful for describing model results or datasets.

When I think about it, I am not sure if we even need a concept of emission. emission is currently defined as a process releasing byproducts from human activity (e.g. production, distribution or consumption) into the environment.
So it always depends on another process and is just special because of who participates in the process (humans) and where some of the outputs go.
That means we could treat emissions just like any other output: We have a process (e.g. the power generation in our coal plant). This process has several inputs and outputs (e.g. coal, electrical energy, ...) and one of these outputs happens to be a carbon dioxide (or another emission gas). The carbon dioxide is then related to a quantity value.
In that case we wouldn't need an emission-process or emission value. If we still want to explicitly state that something is an emission we could make that a quality of the carbon dioxide (similar to origin, maybe called destination). Also, an emission factor as a process attribute could be used if we don't want to say what kind of emission we have but just how much. So an emission factor would be directly related to a quantity value.

Only a subset of these have emissions and hence an emission factor.

Can we solve that problem by changing the relation name to can have process attribute. Or explaining that the emission factor could also be zero?

I am not sure whether it is useful to have statements in an ontology that are not clearly true or false. What do you think, @sfluegel05?

I think it would be best to leave out this relation if there are processes which don't have it. That way, anybody who wants to use an emission factor for his process can do this but no one has to.

Regarding:

We don't have the subprocess of a coal power unit in the ontology yet and I think they are not neccesary to describe an energy system model.

I don't think that we need coal power unit process in detail, but we could define a more abstract process: Process XYZ is a transformation that contains at least two sub process: An energy transformation of chemical energy from fuel into electrical energy and an emission process.
This process XYZ could then be applied to all fuel power unit indicating that they all have emissions.

That means we could treat emissions just like any other output: We have a process (e.g. the power generation in our coal plant). This process has several inputs and outputs (e.g. coal, electrical energy, ...) and one of these outputs happens to be a carbon dioxide (or another emission gas). The carbon dioxide is then related to a quantity value.
In that case we wouldn't need an emission-process or emission value. If we still want to explicitly state that something is an emission we could make that a quality of the carbon dioxide (similar to origin, maybe called destination). Also, an emission factor as a process attribute could be used if we don't want to say what kind of emission we have but just how much. So an emission factor would be directly related to a quantity value.

Please be aware that we already had a lot of discussion on emissions, e.g. in #56, #234 and several meetings. It was a difficult compromise as a lot of aspects had to be reflected, as this is also closely related to the whole portion of matter topic. So please let's find a solution that builds upon what we already have in the ontology without restructuring to much.

Process XYZ is a transformation that contains at least two sub process: An energy transformation of chemical energy from fuel into electrical energy and an emission process.
This process XYZ could then be applied to the fuel power unit class via participates in.

I like the proposal.
And I think we don't need emission value necessarily if we have all the emissions individually described in the OEO (thanks @sfluegel05 for reminding me). The emission factor should be always specific about which greenhouse gas or pollutant is released.

I think it would be best to leave out this relation if there are processes which don't have it. That way, anybody who wants to use an emission factor for his process can do this but no one has to.

Seems like I had a wrong understanding of relations. I thougt if there is no relation implemented, I cannot use the "attributes"/in this case the emission factor to describe a specific process.

I think the process can be applied to fuel power plants and fuel power units. Are they supposed to be the parent class for the individual fuel power plant/units? Because I see them all on the same level in the OEO.
A label for the XYZ process could be fuel-powered electricity generation (process)?

To sum up what I understand we agreed so far:

New process:  Fuel-powered electricity generation is a transformation that contains at least two sub process: An energy transformation of chemical energy from fuel into electrical energy and an emission process.
has part some energy transformation
has part some emission
has participant some fueled power plant
has participant some fueled power unit
has process attribute some emission factor
Add relation: fueled power plant participates in some fuel-powered electricity generation
Add relation: fueled power unit in some fuel-powered electricity generation

Additionally we should also add the relation between emission and emission factor via:

emission has process attribute some emission factor
emission factor process attribute of some emission

I think that would solve this issue, wouldn't it?

I agree with the first 3 bullet points, but I have another view on the relation between emission and emission factor. I think only the fuel-powered electricity generation has the process attribute emission factor. One would multiply the activity of the fuel-powered electricity generation process with the emission factor and gain one or several of the portion of matter emissions.
So I think the relations between the emission process and emission factor are not necessary or could be even confusing.

Sorry, I don't understand this. In #243 #581 we just defined the emission factor as: An emission factor is a process attribute that quantifies the emissions or removals of a gas per unit activity. This clearly gives the link to the emissions.

I don't agree to the has process attribute between emission and emission factor.
The def of has process attribute is: A relation between a process attribute and the process it existentially depends on.
How much emissions are produced depends on the activity of the fuel-powered electricity generation process. If there is no activity (electricity generation) in a certain time period, there are no emissions. So I would say that the emission factor existentially depends on the fuel-powered electricity generation process and not on the emission process.

Any opinions by @OpenEnergyPlatform/oeo-domain-expert-energy-modelling  or @OpenEnergyPlatform/oeo-general-expert-formal-ontology ?

From my side I agree with what you have proposed @l-emele

Okay, seems like my understanding of how I would built an energy system model cannot be fully translated to how we built the ontology.
Shall I implement it according to the summary above?

Go for it!

Looks fine to me, too.

Since it's already implemented, I'm closing this issue.


KERNELBASE.dll Warning and dump file

Starting Gw2 with Clarity throws the following warning in "gw2clarity.log" and creates a dump file for each logging. Addon works just fine.
[11:12:38.5338499|WRN] Intercepted exception in module 'C:\WINDOWS\System32\KERNELBASE.dll', address 0x7ffc0423cd29, code 0xe06d7363.
How can I solve the issue or stop creating dump files?

desactivate in settings graphics "dx9"

Without further activity I'll be closing this now. Feel free to reopen with more details if the issue persists.


Better document cluster agent & Update chart

What this PR does / why we need it:

Better document the Install for the Datadog Cluster Agent
Add a codeowner
Move default log level to info

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

I'm ok with commenting it out in the docs to be more clear but from my testings and given that:
https://github.com/helm/charts/pull/8786/files#diff-19938b6253581d6defa2db7d73496176R61
The leader election will be ON if the Cluster Agent is enabled. Unless set to false.
If the Cluster Agent is not enabled however, the default value will be the one from datadog-agent/pkg/config/config.go which is false. So the value in values.yaml does not do anything.

/approve
/ok-to-test

/lgtm


Add option to set image as wallpaper

Description (required)
Fixes #1525
Lets the user set an image as phone's wallpaper.
Tests performed (required)
Tested on Redmi 7.1.2, with ProdDebug.
Screenshots showing what changed (optional)

Do we set thumbnail or original image to set @maskaravivek ?

It sets whatever imageUrl is there in the Media object. AFAIK it's the original image.


CIS workload ping-pong reconcile loop

Due to concatination in https://github.com/statnett/image-scanner-operator/blob/bdaec856a8414c9e761e328f2fcf43d97f6508bd/internal/controller/stas/workload_controller.go#L235 it is now possible to get a ping-pong reconcile loop of a CIS when two workloads have the same container name and image, and share A LOT of characters in their names. Specifically, the workload within a CIS is updated back and forth.

Any suggestion for how to fix this @tenstad? One of our goals have been to make the CIS name human-readable.


Add tests

Tests lots of tests.
This PR adds test for elements which were not tested before and extends already implemented test files.
This is a big PR. I had to make a lot of changes as some parts of the code was malfunctioning and I did not notice it since there were no test cases. Maybe I can break it down to smaller PRs if it is not acceptable in the current form.

PR too big


'NoneType' object has no attribute 'delete_cookie'

Hello Mr. Grinberg:
I'm learning your book "Flask Web Development" , now I've deployed my VPS, and now I can visit the index page by IP address .
But when I tried to click "Login" button to the  /auth/login page , it failed and occurred "Internal Server Error" , I checked the error information .
    Python3.4 + Nginx +Gunicorn

The main issues it showed are :
1: Exception on /auth/login [GET]
2:  [2016-10-17 21:14:13,399] ERROR in app: Exception on /auth/login [GET]
Traceback (most recent call last):
  File "/home/test/deploy/Flasky_on_Github/venv/lib/python3.4/site-packages/flas                                             k/app.py", line 1988, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/test/deploy/Flasky_on_Github/venv/lib/python3.4/site-packages/flas                                             k/app.py", line 1643, in full_dispatch_request
    response = self.process_response(response)
  File "/home/test/deploy/Flasky_on_Github/venv/lib/python3.4/site-packages/flas                                             k/app.py", line 1862, in process_response
    response = handler(response)
  File "/home/test/deploy/Flasky_on_Github/venv/lib/python3.4/site-packages/flas                                             k_login.py", line 467, in _update_remember_cookie
    self._clear_cookie(response)
  File "/home/test/deploy/Flasky_on_Github/venv/lib/python3.4/site-packages/flas                                             k_login.py", line 503, in _clear_cookie
    response.delete_cookie(cookie_name, domain=domain, path=path)
AttributeError: 'NoneType' object has no attribute 'delete_cookie'

I was confused currently ...because in the [GET] method , it should only render the login.html to show the LoginForm...
Could you please give us any suggestion ? Thanks a lot!

What was the return value of the function that handles the /auth/login URL?

Hello Mr. Grinberg , below are my code for /auth/login  URL
And I tried to modify the return of /auth/login       to      return '
 Test
'
But it didn't work , when I tried to visit /auth/login  page , it still render the base frame of website and displayed the "Internal Server Error"
from flask import render_template,redirect,request,url_for,flash
from flask.ext.login import login_user,current_user
from . import auth
from ..models import User
from .forms import LoginForm,RegistrationForm,ChangePasswordForm,PasswordResetRequestForm,PasswordResetForm,ChangeEmailForm
from flask.ext.login import logout_user,login_required
from app import db
from ..email import send_email

@auth.route('/login',methods=['GET','POST'])
def login():
	form=LoginForm()
	if form.validate_on_submit():
		user=User.query.filter_by(email=form.email.data).first()
		if user is not None and user.verify_password(form.password.data):
			login_user(user,form.remember_me.data)
			return redirect(request.args.get('next')or url_for('main.index'))
		flash('Invalid username or password.')
	return render_template('auth/login.html',form=form)

Below are my login.html code

{% extends "base.html" %}
{% import "bootstrap/wtf.html" as wtf %}
{% block title %}Flasky - Login{% endblock %}
{% block page_content %}

Login

New user?

Click here to register

	{{ wtf.quick_form(form) }}

Forget Password?

{% endblock %}

Hello Mr. Grinberg :
I found the problem finally...............
  The return response of below function .... I added "one more indentation" in the return line.....

  Now I cancelled one , now it works ........... Still thanks for your reply !~

  Have a good day

@main.after_app_request
def after_request(response):
	for query in get_debug_queries():
		if query.duration >= current_app.config['FLAKSY_SLOW_DB_QUERY_TIME']:
			current_app.logger.warning('Slow query: %s\nParameters: %s\nDuration: %fs\nContext: %s\n' %
				(query.statement, query.parameters, query.duration,
				query.context))
	return response


feat(db) add generate_admin_api schema option

Still needs tests.

Can you elaborate about the context? It seems like a decent patch otherwise.

Can you elaborate about the context?

Sorry for not explaining; this patch came out of a phone call with @hishamhm.
We don't want to expose the new CA entity (see https://github.com/Kong/kong/compare/next...james-callahan:feat/mesh-ca-bootstrap) via the admin api. This PR introduces a flag to opt-out.


components/Editable: unsubscribe from redux store

Editable component was missing unsubscribing from redux store on component unmounting. This one should fix #404.

Thank you!

@arekkas sir could you please release new version and deploy on npm? :) Thank you!

Sorry, absolutely!

Thank you! 🥇


Unable to use custom tslint formatters after update to angular (cli) 6

Bug Report or Feature Request (mark with an x)
- [x] bug report -> regression
- [ ] feature request

Area
- [x] devkit
- [ ] schematics

Versions

"angular/cli": "6.0.8"
"angular-devkit/build-angular": "0.6.8"
"angular": 6.0.6
Repro steps

installed npm package
"tslint-teamcity-reporter": "2.0.0"
run
ng lint project_name --format=tslint-teamcity-reporter
The log given by the failure

Schema validation failed with the following errors:
Data path ".format" should be equal to one of the allowed values.
Desired functionality

ng lint should use custom formatter, as states e.g. in #5253

This is still not working for me. Trying to use tslint-teamcity-reporter. Using Angular CLI 6.1.2 same reporter version as in original issue (2.0.0)
Gives me
Data path ".format" should be equal to one of the allowed values.


Custom sorting functions

It would be nice to be able to supply custom sorting functions via column definitions.

:+1:


Windows Release

Is there any plan to release this for Windows too?

yes, please. Any thoughts on a Windows release?

See #128


Empty changelogs are commited

leads to

Fixed by https://github.com/space-wizards/space-station-14/commit/2ef0089152478a33616e69250165604cdd13d7f5


WordProcessor (terrible name)

added an intermediate step in the splitting-tokenization pipeline (WordProcessor).  This required adding a tiny bit of extra code to word_tokenizer and word_character_tokenizer, but should be backwards compatible.  Please check though.  When this PR is ready, I can update the docs (didn't want to do that until the structure was finalized).

I'm going to make the changes I recommended here, so that you can work on your instance type PR, then the actual model.

Ok, this is done.  @bsharpataz, can you look over the changes that I made?

I like the changes, and I agree with you about splitting apart the stemming from the filtering (btw, stop word list was shown to me by tushar -- it exists in a code base somewhere I just don't know how to load it -- it's a Peter Clark list, apparently).
The thing that worries me is that if an old experiment doesn't specify "processor" in their experiment file, but they did specify perhaps the NLTK WordSplitter, wouldn't this change break their expected result? since the default would be {} for processor and then "simple" for the splitter?

You're right that this is a non-backwards-compatible API change, but I think that's ok.  There isn't anything that uses the NLTK word splitter.  Going forward, we'll probably need to think about API changes and backwards compatibility, but I don't think we're to that point yet.

ok, great, thanks
On Mon, Feb 20, 2017 at 11:39 AM, Matt Gardner<EMAIL_ADDRESS>wrote:

You're right that this is a non-backwards-compatible API change, but I
think that's ok. There isn't anything that uses the NLTK word splitter.
Going forward, we'll probably need to think about API changes and backwards
compatibility, but I don't think we're to that point yet.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/allenai/deep_qa/pull/212#issuecomment-281165444, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AFIniYnRCZSreTGjQG41hL18WmWonw5Yks5reevdgaJpZM4ME6xN
.


Rails upgrade 5 2

Upgrade to rails 5.2, remove usage of simplexml (cut ties with HDS)
Pull requests into cqm-parsers require the following. Submitter and reviewer should :white_check_mark: when done. For items that are not-applicable, note it's not-applicable ("N/A") and :white_check_mark:.
Submitter:

[x] This pull request describes why these changes were made.
[x] Internal ticket for this PR: https://jira.mitre.org/browse/CYPRESS-461
[x] Internal ticket links to this PR
[x] Code diff has been done and been reviewed
[ ] Tests are included and test edge cases
[x] Tests have been run locally and pass

Reviewer 1:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Reviewer 2:
Name:

[ ] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[ ] The tests appropriately test the new code, including edge cases
[ ] You have tried to break the code

Codecov Report

Merging #30 into master will decrease coverage by 2.65%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master     #30      +/-   ##
=========================================
- Coverage   76.76%   74.1%   -2.66%
=========================================
  Files         110     110
  Lines        5307    5307
=========================================
- Hits         4074    3933     -141
- Misses       1233    1374     +141

Impacted Files
Coverage Δ

...ers/dc_definition_from_template_or_type_extract.rb
60.78% <0%> (-23.53%)
:arrow_down:

...rser/2.0/document_helpers/doc_population_helper.rb
78% <0%> (-19%)
:arrow_down:

...ic_occurrences_and_source_data_criteria_extract.rb
73.33% <0%> (-16.67%)
:arrow_down:

lib/hqmf-parser/2.0/precondition.rb
85.36% <0%> (-14.64%)
:arrow_down:

lib/hqmf-parser/2.0/data_criteria.rb
81.69% <0%> (-13.15%)
:arrow_down:

lib/hqmf-parser/2.0/types.rb
83.82% <0%> (-11.77%)
:arrow_down:

lib/hqmf-model/types.rb
72.93% <0%> (-6.77%)
:arrow_down:

.../hqmf-parser/2.0/document_helpers/doc_utilities.rb
94.11% <0%> (-5.89%)
:arrow_down:

lib/hqmf-parser/2.0/population_criteria.rb
93.44% <0%> (-4.92%)
:arrow_down:

lib/hqmf-model/data_criteria.rb
65.85% <0%> (-1.96%)
:arrow_down:

... and 1 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 c2a8667...94d45be. Read the comment docs.


AttributeError: 'NoneType' object has no attribute 'toggleVerify'

When clicking the verify image box, if I cancel rather than selecting save the program closes with the error:
Traceback (most recent call last):
  File "labelImg.py", line 1173, in verifyImg
    self.labelFile.toggleVerify()
AttributeError: 'NoneType' object has no attribute 'toggleVerify'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "labelImg.py", line 1178, in verifyImg
    self.labelFile.toggleVerify()
AttributeError: 'NoneType' object has no attribute 'toggleVerify'
Aborted (core dumped)

Looks like there needs to be a check to make sure that labelFile is not None. I would be happy to make a pull request to fix this issue.

Ubuntu 16.04 LTS
PyQT5

This issue should be resolved now. Thanks for bringing it to our attention.


Fix copy-paste error in Bash task's help text

can you separate out the package lock as a separate PR?  How does it affect developers who are using npm 5?  It's LTS (what our docs use and what most of our build agents are using)

It's a more lenient version spec that lets the leftmost digit slide (so bug fixes will get taken).  So it shouldn't be a breaking change.
I don't see it in NPM v6's changelog, but I only noticed it after upgrading.  I can revert and do a separate PR.


Add authentication support

Technical description in https://github.com/open-eid/hwcrypto.js/wiki/Authentication

Is there any guess when will it be implemented?
Until then I use a closed source crappy alternative from
https://portal.signwise.org/BE/en/p/about/web-browser-module

I see that in the APIv1 wiki the parameter filter of the getCertificate() method has changed from

TBD - filter for the certificate selection

to

AUTH filter certificates with non-repudiation key usage

Can this be used to ask for auth cert/PIN1? Please add more information about this parameter, ie example how to use it, will it require new browser plugin or will be upgrading just the hwcrypto.js be enough etc.

It is not released, but requires updated plugins (chrome-token-signing 1.0.6 and browser-token-signing 3.13.0) also upgrading hwcrypto.js.
Yes this can used to ask for auth cert.

I'm very interested in this functionality. Could you possible provide some code examples or some guidance on how to ask for PIN1 and get basic information from the certificate? Thank you in advance!

For verified information, you shall have to verify and parse the certificate in your backend. For a secure user verification, the actual functionality to do it in a secure way is not yet fully implemented.

I am working on an implementation here:
https://test2.zigis.id.lv/
Go to "Iestatījumi" and "Autorizēties" (I am from Latvia)
If the authentication is successful, You will be able to access and save settings.
I am making digital signature system built on PHP and javascript

Thanks @martinpaljak , is there any example of how to verify the signature from the demo code? I mean, in openssl. I am trying to sign something, then send the certificate to the other end alongside a signature, but I get always a false indication.

Interesting, thanks @expiorer . I happen to live in Latvia, I am an e-Resident and I am developing a social network for e-Residents (that's why I want to have e-Card authentication). Do you live in Riga? Maybe we can meet!


Don't delete MCO resource for 4.8

As part of ae6029b we removed the MCO from override list but missed
this one..

Ack, but also wondering if we have a way of blocking upgrade attempts?

@cfergeau don't know yet but  will figure this out.


Most efficient way of DataStore refresh

Hi,
Currently running the PaaS version of the Azure API for FHIR.  I am currently looking at the most optimal way of refreshing the data store or bringing the service back to a default state with no data.
A few options I have been looking at:

Create a script which stores all the id's then created individual delete requests to the server.
Conditional deletes (not-supported)?
Destroy and recreate the PaaS service (Would have to go through all apps and reassign permissions)

The only feasible option seems to be number 1 so far unless there is another method I have not uncovered.
Thanks for your help!
Kind Regards,
Glen

I use this little tool I made:
https://github.com/hansenms/ClearFhirService
I am not sure you need to first store all the ids. You can just do a search for /*, it will return all resources then run through and delete. add the ?hardDelete=true query parameter to get rid of all revision history.

Thanks Michael, that is really handy.
Kind Regards,
Glen


Fixed flaky unit tests

Description
Changes proposed in this pull request:

Fixed flaky unit tests

Related issue(s)

/test pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

/test pull-eventing-manager-unit-test

/cla-recheck


Format members amount with commas and update youtube link in the footer

What:
Format members amount with commas and update youtube link in the footer
Why:
Closes:

https://app.asana.com/0/1159164196409129/1198974685116941/f
https://app.asana.com/0/1159164196409129/1198220574842733/f

How:

Create a function to format numbers with commas in a new folder called formatters
Replace the old youtube link channel with the new one

Media
https://www.loom.com/share/dd6360c87f514cd196a703eea88ea5fa

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


Ships not interactable on a server, but are okay on singleplayer worlds.

This issue occurs when only Valkyrien Skies and addons are installed and no other mods

[X] I have tested this issue and it occurs when no other mods are installed

Minecraft Version
1.16
Eureka version
alpha6
Mod Loader
Forge
Issue description
Playing forge 1.16.5 with eureka installed. On a server, whenever anyone tries assembling a ship, the ship only exists as blocks you can't interact with, only being able to collide with them. This does not happen in a singleplayer world with the same modpack. Is there any way to fix this? I know 1.16 isnt supported but I have no other choice.
Issue reproduction
Go into a server and try assembling a ship.
Logs
debug.log
latest.log

Looks like your issue might be caused by the other mods you have installed. Please try to replicate the issue with just VS and addons, and then create a new issue.


Module 1.1 Assignment

Given what you have learned about Git, why would you want to use Git in your classroom?

Of course.


Structured logs for HTTP 404 endpoints

Hello,
I realized if a client requests a non-existing endpoint (say, /foo) no record of this appears in any structured form (i.e. logfmt/json). If you leave the Apache access.log (CLF) logs on, an entry appears there, and that is all:
<IP_ADDRESS> - - [19/Oct/2016:02:22:16 +0000] "GET /foo HTTP/1.1" 404 19 "" "curl/7.49.1"

I think there should be some structured log entry for those

who want to unify all the logs coming out of registry in structured form (like below)
and therefore disabling CLF entries

Ideally I'd love to see this log from a nonexisting endpoint:
time="2016-10-19T02:24:06.460645555Z" level=info msg="response completed" environment=development go.version=go1.6.3 http.request.host="localhost:5000" http.request.id=060b3c1c-02df-4f3c-91a1-74d4bfef70a8 http.request.method=GET http.request.remoteaddr="<IP_ADDRESS>:43418" http.request.uri="/foo" http.request.useragent="curl/7.49.1" http.response.contenttype="application/json; charset=utf-8" http.response.duration=1.477627ms http.response.status=404 http.response.written=19 instance.id=082c15c4-ddf9-4b60-8689-95d4fe8852f5 service=registry version=v2.5.0-rc.1-180-g8234784
But when I look at app.go I see this:
        status, ok := ctx.Value("http.response.status").(int)
        if ok && status >= 200 && status <= 399 {
            ctxu.GetResponseLogger(ctx).Infof("response completed")
        }

this was added in https://github.com/docker/distribution/commit/54f7e5d54b0e8f547dd2bae804b36694cd0f1601 but I am not sure if this code path gets invoked on routes that are not handled at all, because later in that file there is a log message response completed with error that gets printed when there is an error from an endpoint...
Any opinions?

The registry will never log non-existing endpoints because the multiplexer we use (gorilla/mux) will not have anything registered to call.   (The CLF line comes from gorilla).
It is possible to register a default handler for a non-existing endpoint, I'm not sure if we can get a context with all of the information you wish though.   I think logging some like 'non-existent endpoint' or similar would be more useful than the standard 'response completed' though.

@rscothern I agree with "non-existent endpoint message". Are you open to having a context constructed from the request and have that logged, just like we add CLF to mux?

I am open.

@ahmetalpbalkan All that needs to be done is to move the instrumentation up the handler stack. Should be very straightforward. With that change, we can probably removed the combined logs.

@ahmetb Is this still an issue? Do we need to make change here?

I suppose we can close. It's only an issue for 404s.


Executing with PowerShell

Similarly to how you are given a choice to choose between PowerShell or bash when you start CloudShell, you can start local CloudShell with either PowerShell or bash. The code is already there but the README can be slightly clearer. This is just a small fix on the README.

I can't seem to find a way to re-run the workflow.

No problem :)


fix: Correctly Detect Pictures Directory and Clone Path for Background

Pull Request
Title
Correctly Detect Pictures Directory and Clone Path for Background
Type of Change

[ ] New feature
[x] Bug fix
[ ] Documentation Update
[x] Refactoring
[ ] Hotfix
[ ] Security patch
[ ] UI/UX improvement

Description

The configure_background function couldn't correctly detect existing Pictures directory and installed the nord-background repo to the top level of the Pictures directory
https://github.com/ChrisTitusTech/linutil/issues/303
Testing
Tested both having no Pictures directory and an existing Pictures directory, then ran the DWM-Titus script from the TUI. Both times the images from the nord-background repository wound up in ~/Pictures/background
Impact
N/A
Issue related to PR

Resolves #303

Additional Information

This will still fail if you already have a ~/Pictures/nord-background folder that's not empty but that seemed a bit extra to go clobbering that if it existed.
Checklist

[ ] My code adheres to the coding and style guidelines of the project.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no errors/warnings/merge conflicts.

Sorry for the inconvenience. We had a massive restructure of the codebase to improve future development. Because of this can you update your PR to the new structure. Thank you for your assistance and contribution.

Rebased on top of the new change


We've come a long way :)

r? @ryankurte
(rust_highfive has picked a reviewer for you, use r? to override)

bors r+


Add relationships between a project and its admins and participants

Rather than relying on the short_name, which can change.

Done


Update docs for tab switch keybinding

docs: keybinding reference for tab switching
dendron-site PR: https://github.com/dendronhq/dendron-site/pull/217
Noticed that the "unique to your system" ctrl/cmd key wasn't applicable for tab switching. Made an extra note in the dendron-site PR too, as I (and perhaps many others too) have specific key bindings to match the defaults across other apps:
    {
        "key": "ctrl+shift+tab",
        "command": "workbench.action.previousEditor"
    },
    {
        "key": "ctrl+tab",
        "command": "workbench.action.nextEditor"
    },

NOTE: Leaving the checklist in below in case I missed something 🤷
Pull Request Checklist
You can go to dendron pull requests to see full details for items in this checklist.
General
Quality Assurance

[ ] add a test for the new feature
[ ] make sure all the existing tests pass
[ ] do a spot check by running your feature with our test workspace
[ ] after you submit your pull request, check the output of our integration test and make sure all tests pass

NOTE: if you running mac/linux, check the windows output and vice versa if you are developing on windows

Special Cases

[ ] if your tests changes an existing snaphot, make sure that snapshots are updated
[ ] if you are adding a new language feature (graphically visible in vscode/preview/publishing), make sure that it is included in test-workspace. We use this to manually inspect new changes and for auto regression testiing

Docs

[X] Make sure that the PR title follows our commit style
[X] Please summarize the feature or impact in 1-2 lines in the PR description
[X] If your change reflects documentation changes, also submit a PR to dendron-site and mention the doc PR link in your current PR

Example PR Description
# feat: capitalize all foos

This changes capitalizes all occurences of `foo` to `Foo`

Docs PR: <URL_TO_DOCS_PR>

Special Cases
First Time PR

[ ] sign the CLA which will be prompted by our github bot after you submit the PR
[ ] add your discord alias in the review so that we can give you the horticulturalist badge in our community

Analytics

[ ] if you are adding analytics related changes, make sure the Telemetry docs are updated

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

@all-contributors please add @codeluggage for doc


RealFuels install error

GameData/SolverEngines/SolverEngines.version
Installing Mod : ModuleAnimateEmissive v1.5
Owning Mod     : SolverEngines
CKAN Version   : v1.10.0-0-gb447380 (beta)
Your GameData has been returned to its original state.
Getting this error when installing RealFuels.

Closing as duplicate of #676


Debug Adapter Features don't seem to work (Wolfram Engine 12.3, MacOS 11)

Hi, it's me again. Hope I do not bother you. Now that I got the language server running in VSCode on MacOS with Wolfram Engine 12.3 everything seems to be working except the features which are referenced in the "Debug Adapter Features" section of the readme.
Example:
Let's say I open a new .wl file and type Integrate[x^2,x] and I want to evaluate this expression in the debug console, VSCode tells me that there are no available code actions. Also no light bulbs do appear (which are activated in VSCode). The readme shows that there should be a code action named Evaluate in Debug Console.
Any hints on what needs to be done to fix this?

Sorry for the late reply, but you can see the instruction here: https://github.com/kenkangxgwe/vscode-lsp-wl#debugger-support


Fix invalid chars

The ¥ character probably used a character code that was not expected. See the image for details.
Therefore, it was changed to the character that corresponds to 00A5 as Unicode.

Thanks! Fixed in 1.1b1.


Plot confusion matrix and metrics table

It will display as following:

Codecov Report

Merging #970 (b59051b) into main (3664780) will decrease coverage by 0.11%.
The diff coverage is 10.71%.

@@            Coverage Diff             @@
##             main     #970      +/-   ##
==========================================
- Coverage   68.72%   68.61%   -0.12%
==========================================
  Files         201      201
  Lines        9149     9170      +21
  Branches     1530     1536       +6
==========================================
+ Hits         6288     6292       +4
- Misses       2413     2431      +18
+ Partials      448      447       -1

Impacted Files
Coverage Δ

towhee/utils/sklearn_utils.py
0.00% <0.00%> (ø)

towhee/functional/mixins/metric.py
29.11% <11.11%> (-4.22%)
:arrow_down:

towhee/hub/repo_manager.py
80.18% <0.00%> (-0.91%)
:arrow_down:

towhee/operator/__init__.py
100.00% <0.00%> (ø)

towhee/functional/data_collection.py
91.80% <0.00%> (ø)

towhee/operator/stateful_operator.py
68.42% <0.00%> (+0.85%)
:arrow_up:

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 3664780...b59051b. Read the comment docs.

/lgtm


Provide explicit exit-code for helm upgrade --atomic failures

Scenario:
We use helm extensively in my role, for application and kubernetes configuration deployments.
We find that for some of our app deployments, we have failures that cause the release to enter a PENDING error state. This could be an underlying issue, unrelated to helm's functionality.
But to address any intermittent failures, we have a script that performs some retries and if a failure occurs on the helm upgrade --install we then perform a helm rollback manually. This is because I want to clearly distinguish between errors on the first upgrade, and errors with the subsequent rollback.
I'm aware helm has a flag for this scenario called --atomic which handles these cases, however as a user I cannot distinguish if the error scenario is:

An error with the initial release.
or
An error with the initial release -> followed by an error with the uninstall attempt.

As these both exit with a 1 exit-code. I would need to scrape the stdout/stderr stream to distinguish this.

Proposal:

Allow helm built-in functionality to provide specific exit-codes for failure scenarios? Or is this going against design?

I noted this code, which generally handles the exit-code and allows for some override based on plug-ins. So i would essentially do something similar.
https://github.com/helm/helm/blob/33ab3519849a90549f734fbbbc0aecb7f37f7570/cmd/helm/helm.go#L83-L91
I think I would essentially need to propogate an exit code along with errors from this layer:
https://github.com/helm/helm/blob/main/pkg/action/install.go#L485C19-L485C30

thanks for the suggestion. I think the proposal is valid here. As an operator, it would be valuable to be able to distinguish different failure scenarios.
Though there might be alternatives to determine the failure cause e.g. helm install --upgrade -o json or helm get ...


Update README.md

GitHub Actions Integration updated according to the latest working case.

@mknapik can you merge ?

Sure, thanks for your contribution!


WSL 2 backend updates for Edge release

WSL 2 backend updates for Docker Desktop Edge <IP_ADDRESS> release.
Changes include:

Modified the prereqs and installation instructions
Updated screenshots to reflect the latest UI
Added new sections on developing with Docker and WSL 2, best practices
Moved Feedback to the end of the page
Updated headings to h2 to match the rest of the docs

@simonferquel @nebuk89 Could you PTAL?

You can preview the updated page here - https://angry-visvesvaraya-c62c8e.netlify.com/docker-for-windows/wsl-tech-preview/.


Updating Coder

Just, a thought, how would you update coder, in-app?
I am only thinking about the core node.js app, would it be possible for a notification, on login, with a single-click update process?

FYI - the latest image has an updated version of Node.  It's sort of a pain to install because it needs to be done manually and copied to /opt/node/. Unfortunately, the apt-get version is really old.

what is a node?  who or what is Coder? I just do not understand the words being used, I am an old hobbyist with Arduino and Raspberry but two PC's with faults!
means starting all over as most projects disappeared with two house moves on top of faulty PC's.
I am only trying to get sensors and key pads to work.  but  for some reason  # Keypad.h downloaded from
Arduino website is not recognised,up to this point.   latest IDE downloaded last night may now work?
passwords for Raspberry may now be hard to remember?


Added new site in hosting.yml

Just added hosting https://www.ukraine.com.ua/ (tfa via SMS, Google Authentificator, hardware tokens)

Still need logo and docs link.

"Still need logo and docs link."
Done.

@Nanotraktor please see FAQ_New_TFA_Site

@Nanotraktor Please remove the unnecessary twitter:, email: No, and phone: No fields.

jamcat22, many thanks for the tip. Done.
Done!

Great!


Replace references to this being a beta

We currently mention in some places in the app that this is a beta.
In preparation for the stable release we should drop all references to it being a beta release.
For example the BETA  button on windows and the fact that the about window opens automatically.

Merged into master via #74


Provide default select lists for bool and bool? expressions

Similar to #438 should provide automatic (well, static) select lists for bool and bool? expressions.  This amounts to using DefaultDisplayTemplates.TriStateValues() (and something similar for non-Nullable Booleans) when the user does not pass a select list  to @Html.DropDownList() and @Html.ListBox().
Current implementation gives up when no select list is passed or found in ViewData.

Triage: this helper is fairly trivial but simple for customers to do themselves.  suggest we not fix this one for RC.

@dougbu Could you please add a quick code sample to this bug that shows what a user would have to do to solve this problem?

I wouldn't recommend using a <select/> for any regular bool.  just use a checkbox.  <select/> is possible but would look strange.
for bool?, assuming you want localization and have the resources in your resx file:
var selectListForNullableBool = new List<SelectListItem>
{
    new SelectListItem
    {
        Text = Resources.Common_NotSet,
        Value = string.Empty,
    },
    new SelectListItem
    {
        Text = Resources.Common_True,
        Value = "true",
    },
    new SelectListItem
    {
        Text = Resources.Common_False,
        Value = "false",
    },
};

the first SelectListItem isn't :100:% required when using Html.DropDownList[For]() because you can call that method as follows:
@Html.DropDownListFor(m => m.BoolQuest, selectList, optionLabel: Resources.Common_NotSet)


Fix Ec2RemoteShellProxy.execute() args order (#51)

Change Summary
Description
Fixes #51
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

will be replaced by another PR


small/big face buttons (A and B overlays), hide/show option, osk_toggle

Also added a retropad fast overlay, the full featured one takes almost 4 seconds or more to load...

Btw, is it hard to get this set as the default in android? :)
The retropad overlays has almost all things needed now

shouldn't be hard to change defaults. I think it's just an entry in a config somewhere. we can ask radius where to find it.

I mean, is this better than the default now?

I haven't used it to see if it controls better, but it looks nicer, I think, and they seem better-designed (e.g., the extra L1 by the buttons)

They should be used with Overrides Files ON. I'll try to make for every console.

Did you change the original flat overlay? that one had some offset between the buttons and the actual touch zones that made it great.
Does your overlay have that too?

Are you speaking about this?

I kinda didn't like, and the d-pad touch zones are too small

The touch zone surpass something like 3 to 6 pixels of the picture, for d-pad is more, 9~11

you are not answering my question, I asked if you modified the original one
not for an explanation
On Wed, Sep 2, 2015 at 12:13 PM, sergio-br2<EMAIL_ADDRESS>wrote:

The touch zone surpass something like 3 to 6 pixels of the picture, for
d-pad is more, 9~11
[image: retropad-b - 960x540 - green marks]
https://cloud.githubusercontent.com/assets/5431285/9638284/4e694814-517c-11e5-88bc-29b785a1adf4.png
—
Reply to this email directly or view it on GitHub
https://github.com/libretro/common-overlays/pull/9#issuecomment-137175341
.

For context, the guy who made the original one made a little program that registered touch locations and then he drew the overlay on the screen and had people touch the overlay buttons, then he based the hitboxes on that data. Interestingly, the hitboxes were not lined up with the overlay images, presumably because the places where people put their thumbs register differently from where they're expecting.

yeah I remember he studied the thing and it does work better than just
"bigger" overlays so I'd like to keep that one as it was
On Wed, Sep 2, 2015 at 2:47 PM, hizzlekizzle<EMAIL_ADDRESS>wrote:

For context, the guy who made the original one made a little program that
registered touch locations and then he drew the overlay on the screen and
had people touch the overlay buttons, then he based the hitboxes on that
data. Interestingly, the hitboxes were not lined up with the overlay
images, presumably because the places where people put their thumbs
register differently from where they're expecting.
—
Reply to this email directly or view it on GitHub
https://github.com/libretro/common-overlays/pull/9#issuecomment-137224713
.

I barely based in the old one, I got only the idea of the shoulder buttons.


Bilateral sweep setpoints

Hey, this is my first issue submission!
I'm interested in doing bilateral sweeps (let's say, back/forth IV curves), but the setpoints dependencies are not recognized correctly when saved in the way given by the example below.

from qcodes import Measurement

meas = Measurement()
meas.register_custom_parameter('s1')

meas.register_custom_parameter('y0_forth', setpoints=('s1',))
meas.register_custom_parameter('y0_back', setpoints=('s1',))

sweep = np.linspace(0,1,11)

with meas.run() as datasaver:
    dataid = datasaver.run_id

    for _s1 in sweep:
        datasaver.add_result(('s1', _s1),
                             ('y0_forth', _s1))

    for _s1 in sweep[::-1]:
        datasaver.add_result(('s1', _s1),
                             ('y0_back', _s1))

This is not so much a bug, but a situation that in general cannot be resolved. look at the data that qcodes acquired:
`ds.get_parameter_data()
{'y0_back': {'y0_back': array([1. , 0.9, 0.8, 0.7, 0.6, 0.5, 0.4, 0.3, 0.2, 0.1, 0. ]),
's1': array([1. , 0.9, 0.8, 0.7, 0.6, 0.5, 0.4, 0.3, 0.2, 0.1, 0. ])},
'y0_forth': {'y0_forth': array([0. , 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9, 1. ]),
's1': array([0. , 0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8, 0.9, 1. ])}}`
you have two arrays, both with name 's1', but they are not equal. That's a problem in general for plottr and it plays it safe here to prevent us from ending up with corrupted data so it renames one -- that's an easy fix because there's no logical dependency conflict here. you need some human insight to see that in this case you can actually sort them such that they can be mapped onto each other.
(Note: you'll see that also the export methods qcodes provide for pandas or xarray give warnings about this)
i'm open to suggestions on how detection of 'fixable' cases like this can be improved, but my first suggestion would be to improve the acquisition code.

my first suggestion would be to improve the acquisition code.

i agree with this, that would be my first instinct as well. would it make sense for this measurement to NOT create separate dependent parameters for up and down sweeps, and store them together? or, another idea, just make two separate datasets - you won't be able to plot them in plottr on the same window (right @wpfff ?) but e.g. in qcodes using plot_dataset that would be possible if axes are re-used.

I agree with both of you but everything I tried has its pros and cons.
For instance:

storing everything in only one parameter (as suggested by @astafan8) would not work well for 2d maps
registering two parameter setpoints for s1 would not allow to plot the traces together in the same plot
saving the whole array with the back sweep flipped would miss the real-time plotting

Moreover option 2), when doing 2d maps, also gives me some issues in recognizing the setpoints while the measurement is running (see picture and code below). Of course both option 2) and 3) can work, but I'm not completely happy with both of them.

import numpy as np
from qcodes import Measurement
rng = np.random.default_rng()

meas = Measurement()
meas.register_custom_parameter('s1_forth')
meas.register_custom_parameter('s1_back')
meas.register_custom_parameter('s2')

meas.register_custom_parameter('y0_forth', setpoints=('s1_forth','s2'))
meas.register_custom_parameter('y0_back', setpoints=('s1_back','s2'))
meas.register_custom_parameter('y1_forth', setpoints=('s1_forth','s2'))
meas.register_custom_parameter('y1_back', setpoints=('s1_back','s2'))

sweep_1 = np.linspace(0,1,11)
sweep_2 = np.linspace(0,10,11)

with meas.run() as datasaver:
    dataid = datasaver.run_id

    for _s2 in sweep_2:
        for _s1 in sweep_1:
            sleep(0.2)
            datasaver.add_result(('s1_forth', _s1),
                                 ('s2', _s2),
                                 ('y0_forth', _s1+rng.random()),
                                 ('y1_forth', _s1+rng.random()),)

        for _s1 in sweep_1[::-1]:
            sleep(0.2)
            datasaver.add_result(('s1_back', _s1),
                                 ('s2', _s2),
                                 ('y0_back', _s1-rng.random()),
                                 ('y1_back', _s1-rng.random()))

I'm realizing that the issue you're facing also has a bit to do with how data is represented/loaded from the qcodes database. with plottr's own dataformat (which we're using in my lab and a few others), this problem does not actually occur, turns out.
for testing this you can run plottr-monitr . from your working directory to start the data browser.
the following snippet generates and saves data in hdf5 files with specific conventions (make sure to update plottr to the most recent version from github, though):
from time import sleep
import numpy as np
from plottr.data.datadict import str2dd, DataDict
from plottr.data.datadict_storage import DDH5Writer

sweep = np.linspace(0,11,11)

with DDH5Writer(
    str2dd("y0_forth(s1); y0_back(s1)"),
    name='Test measurement'
) as writer:

    for s in sweep:
        writer.add_data(
            s1=s,
            y0_forth=s**2 + np.random.normal(scale=1),
        )
        sleep(2)

    for s in sweep[::-1]:
        writer.add_data(
            s1=s,
            y0_back=s**2 + np.random.normal(scale=1),
        )
        sleep(2)

Notes:

To run this you need to update plottr -- i just had to fix a bug that was relevant and showed up due to a recent change in numpy, i believe.
the reason this behaves differently from the qcodes dataset is that the dataformat here explicitly saves everything as a single table: during the 'forth' sweep None is recorded by default for the 'back' data and vice versa. our dataformat deliberately demands that each addition of data must include all fields (even if data is None or NaN). conversely, when we process data in plottr we simply ignore all None/NaN type data during plotting.  In this specific case we have two traces that both contain a bunch of NaNs that simply don't show up when we plot.

Looking at your most recent comment, though, i'm not sure that what i wrote will work for 2D maps for example -- probably will work when you create 2d scatter plots, but not with placing data on a grid (but i might be wrong :))
The issue here in general is really that you effectively operate with two different and what i'd call incompatible sets of data. The similarity between the different sets is such that i wouldn't know immediately how to find a generic routine that allows bringing them onto equal footing. a solution to this would probably have to enter in the data loading/selection step somewhere. happy to give feedback on any solution proposals ;)

I understand, thanks for your time :) For now I decided to stick with saving the whole array at once after flipping it.

ok, sounds good. i'm closing this for now then.


Ruby 3 - URI.escape

in Ruby 3, URI escape is no longer supported, a simple fix is CGI escape

Not sure if i will find other issues ... but came across this as I upgraded to Ruby 3 (Rails 6.1.1), from Ruby 2.7 (Rails 5.2)
URI.escape is no longer supported. In my search I found that CGI.escape is a substitute; source: https://ruby-doc.org/stdlib-2.4.1/libdoc/uri/rdoc/URI/Escape.html
As this gem has not been updated in ~6 years ... I forked it and made the one line change. This is my first time doing anything like this and unsure if I am doing this the right way, but it is working.
gem 'madmimi', :git => 'https://github.com/charlie-prezzano/madmimi-gem/'

Not sure if i will find other issues ... but came across this as I upgraded to Ruby 3 (Rails 6.1.1), from Ruby 2.7 (Rails 5.2)
URI.escape is no longer supported. In my search I found that CGI.escape is a substitute; source: https://ruby-doc.org/stdlib-2.4.1/libdoc/uri/rdoc/URI/Escape.html
As this gem has not been updated in ~6 years ... I forked it and made the one line change. This is my first time doing anything like this and unsure if I am doing this the right way, but it is working.
gem 'madmimi', :git => 'https://github.com/charlie-prezzano/madmimi-gem/'

Thanks, Charlie. I will check it out today.

Thanks, Charlie. I will check it out today.

"Today" == 10.days.from_now 😄
A couple of things I discovered:
CGI.escape !== URI.escape
2.7.0 :003 > URI.escape('foo bar')
(irb):3: warning: URI.escape is obsolete
 => "foo%20bar"
2.7.0 :004 > CGI.escape('foo bar')
 => "foo+bar"

This means that the specs on this branch fail.
Before even being able to run the specs I needed to update the safe_yaml and json gems.
Do you want to fix these? Or I can see if I can find some time in the next couple of weeks to handle them.


Initial commit of xcorr

Description
This is an initial commit of a version of xcorr into SOLikeT.
I have made use of the xcorr code here:
https://github.com/simonsobs/xcorr
to calculate galaxy-galaxy (gg) and galaxy-CMB lensing (kappag) cross-correlations.
The tests check the the log-likelihood is finite, and produce a comparison plot between the data stored in the simonsobs/xcorr repository and that produced by this code, as shown below:

ToDo: broad

[ ] The current treatment of binning for comparison to data is very ad-hoc. Would be good to unify this across SOLikeT
[ ] Unify calculation of kappa kernel with lensing likelihood(?)

ToDo: moderate

[ ] Check source of low power at mid-ells in kappag
[ ] Make work with CLASS
[ ] Include more of the xcorr bias models
[ ] Check calculation of magnification kernels
[ ] Include redshift distribution nuisance parameters
[ ] Run some test chains

ToDo: minor

[ ] Investigate increasing resolution of self.zarray above 149 (which I think is set in CAMB)
[ ] Expose a number of settings which are currently hard-coded in the initialize.
[ ] Remove dummy_pk

New comparison to CCL (which is used in solike.cross_correlation) shows previous problem was due to inconsistencies in specification of cosmology.


Version numbers are out of sync

package.json is 3.6.1
animate.css is 3.6.0
latest release tag is 3.5.2

It'll get fixed soon as I finally got some time to work on it. Thanks.

This should be fixed now!


Remove Test.sol and TestOwner.sol

It doesn't appear that either of these contracts are being used anywhere? Can they be removed?

Test.sol
TestOwner.sol

@Sabnock01
Hi there! Thanks for reaching out and opening this issue. Will add this to our backlog so one of the team members can take a closer look! 🤝

@Sabnock01
Hi there! Thanks for reaching out and opening this issue. Will add this to our backlog so one of the team members can take a closer look! 🤝

Thanks! Am fine with opening the PR myself in the event you guys want to proceed.

@Sabnock01 We often use the contracts from this repo in https://github.com/base-org/contract-deployments - the smart contract files you pointed out are sometimes used in that repo for testing.


Recordset app login and logout

This depends on ermrestjs login/logout
https://github.com/informatics-isi-edu/ermrestjs/issues/28

Hopefully this work will yield a reusable page or nav-bar component we can reuse consistently in record, viewer, and other future apps...?  Also, I think this will overlap with the site-customization issues we've been trying to sort out in other existing apps.  We need a way for a deployed server to have Chaise and static site (i.e. Jekyll) configured with integrated branding.

@karlcz: right, in fact, I was thinking we should factor out all of the index.html.in templates into a single template (see #266) and then we include common parts like login/logout so that they are always consistent across apps.

The recordset app should also have the same header (and footer?) as the rest of the chaise apps.

Angularize login/logout and put them in common folder.

Where should I got to see this in a working deployment? And where can I see the code? Also, is this task meant to cover Karl's comment asking for a reusable navbar component?

It is on the record-recordset branch. https://github.com/informatics-isi-edu/chaise/tree/record-recordset
I see this branch has been merged to the master. And yes it includes the reusable navbar component.
Redirect to login at page loading
https://github.com/informatics-isi-edu/chaise/blob/record-recordset/recordset/recordset.js#L481
navbar and login/logout
https://github.com/informatics-isi-edu/chaise/blob/record-recordset/recordset/recordset.js#L155
Recordset html
https://github.com/informatics-isi-edu/chaise/blob/record-recordset/recordset/recordset.html#L3
Navbar templage
https://github.com/informatics-isi-edu/chaise/blob/record-recordset/common/templates/navbar.html
Navbar directive
https://github.com/informatics-isi-edu/chaise/blob/record-recordset/common/views.js


Update Proof and ProofOp

https://github.com/informalsystems/tendermint-rs/issues/198
adding to #196
Note: changes from https://github.com/informalsystems/tendermint-rs/pull/206/commits/79bc33a16461e16e4d4f9f6fc973e2b6732523a3  can be ignored when reviewing (they are from master),
relevant changes here: https://github.com/informalsystems/tendermint-rs/pull/206/files#diff-fb2318e2d9a483eac0f27bed09958988

[ ] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGES.md

Looks good I think for now. Will try to run IBC with this. I am wondering though why is proof.rs in .../abci/.. directory and module and not in merkle?

Hmm, that is a good point. I left it this way because it was already there. I'll go on and merge now but I think you are right, should be in merkle.


Store algorithm package output metadata in Unity Data Catalog using a common metadata format

The application Package create data products which have required and custom metadata. The metadata for output products is needed for rich metadata search in the U-DS Data Catalog. We need to define a format for the data catalog operations while simultaneously making it "simple" for an application Algorithm Developer to populate that metadata.
Required metadata:
Start Time / Stop Time OR Sol/SCLK?
Creation Time
Optional Metadata
Bounding coordinates?
cycle, Pass
Custom (Project) Metadata (examples)
Cloud Cover

Duplicate of #66


[Feature]: Arrow functionality for the search

Current behaviour: No action for the arrow keys in the search filter
Expected behaviour: Selection of the filtered search
https://github.com/code100x/daily-code/assets/108384868/6e501af9-6039-4f64-998e-f1a65ceb4bd8

Implemented this thing fully. Can I update it with a PR?


Debug: Call stack view should use more ellipsis

Testing #33305
IMO the behaviour, under tight space constraints, should be:

First, the line numbers go away
Then, the file name starts compressing, with ellipsis
Finally, the same happens to the actual stack frame label

@joaomoreno thanks a lot for help. I love this change

@isidorn I do not see ellipsis for stack frame label

@sandy081 good catch. Will tackle that in october

@joaomoreno dear wizard, my span always wraps in 2 lines no matter what I do. I tried the following:
box-sizing: border-box;
white-space: nowrap;
overflow: hidden;

Might you have some advice?

Not sure I follow. 😕

Due to the
min-width: fit-content my span overflows to the second line even though it shuold only remain in one line. However I have to use this rule due to the flex layout and the priorty on which elements get hidden.
Closing as will not fix


License file

Firstly, thanks very much for posting this awesome library. It's just the ticket for our large Angular 1.5 app that needs a sprinkling of onPush style change detection for some optimizations.
What license are you releasing this under? Based on your other libraries, I'm assuming MIT. If so, would you mind adding a license file? :)

Hi @peterhodges,
Thank you for the compliment :). Really happy that it could help you. If you have any issues or queries, don't hesitate to get in touch.
I just added the MIT license file.
Thanks !


Upgrade Django to 3.2.0

Changes:

Upgrade Django to 3.2.0
Upgrade pytz to 2021.3
Upgrade asgiref to 3.4.1
Upgrade typed-ast to 1.5.1
Update migrations

Approved!


Constant POST requests to https://.table.core.windows.net:443/Tables() failing with 409

Hi,
We are using the Serilog.Sinks.AzureTableStorage v4.0.0 for logging from a web app running on .net framework 4.6.1, hosted in Azure as an App Service. Logging is currently the only thing in the system that writes to Azure Table Storage.
During high traffic periods where a lot of log entries are produced we are seeing (logged as Dependency Call Failures in Azure App Insights) a very high rate of POST requests to https://<account>.table.core.windows.net:443/Tables() that fail with 409 from the Azure Table Service API.
The log entries that the application produces accumulate normally under the LogEventEntity table.
What could be the source of these requests? Does the AzureTableStorage sink implement some kind of health check mechanism that could be doing this kind of requests? (That seem invalid)
Thanks,
Adrian

Hi @adrianbontea
409 is the status code for conflict (as per here). Azure Table Storage gets back a conflict when the (Partition Key, Row Key) pair is identical to another row in the table.
Can you please give us some more details about how you use this sink (in particular, how and if you generate your own partition / row keys)?
Catalin

We are seeing the same thing here @cdavid this is not happening on an insert, rather what appears to be the account table list query.

You can close the issue, this seems to be expected behavior for the client library 8+ CloudTable.CreateIfNotExists throws a 409 error and moves on if the table already exists.

I feel like we should address this by checking manually if the table already exists as outlined here: https://medium.com/medialesson/application-insights-azure-table-storage-createifnotexists-errors-3e2584a38b24

Fixed


Restore support for matching a scenario by multiline step argument line numbers

Description
The Cucumber 4.0 release introduced some regressions in matching scenarios by line numbers other than the exact scenario line. The feature line, tag line, background lines, and step lines all stopped matching their related scenario(s). More context at https://github.com/cucumber/cucumber-ruby/issues/1469
This PR builds on top of https://github.com/cucumber/cucumber-ruby-core/pull/238 to also match multiline arguments like tables and docstrings. Note that I didn't update the CHANGELOG because I think this falls within the changelog line added in the previous PR.
Type of change

Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] Tests have been added for any changes to behaviour of the code
[x] New and existing tests are passing locally and on CI
[x] bundle exec rubocop reports no offenses
[x] RDoc comments have been updated
[x] CHANGELOG.md has been updated

@luke-hill @mattwynne I have some free time coming up if this needs any work. WDYT?

I have probably done about 2-3 hours on cucumber over the last month, owing to real life stuff.
I can maybe get around to contributing here or offering some insight later in the month. If you're ok to continue solo I can check it in a while.

@mattwynne Okay, just pushed a commit incorporating your suggestions. Ready to merge?
Then I can move onto a new PR plugging the last few holes in the larger scenario-matching regression.

Hi @botandrose,
Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾
In return for this generous offer we hope you will:

✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
ℹ Don't feel obliged to help, just do what you can if you have the time and the energy.
🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

Superb, thanks @botandrose!

You can use branches in this repo now for PRs, if you prefer.


Multivalue A record has changed to a single record

My cluster has 5 worker nodes and everything has been fine until yesterday early evening. Before then, all DNS entries has the 5 IP addresses of the worker nodes and all of the nodes were receiving traffic. However, without any deployment or any work in Rancher, all entries in Route53 now has a single IP address of a worker node and this is the only worker node that is now receiving traffic. Please can you help

Useful
Info

Versions
Rancher v2.3.3 UI: v2.3.25

Route
workload

I changed the record in Route 53 to contain all the 5 IP addresses of the worker nodes, the next minute, it was updated back to 1 IP address!

Once I restart the ingress controllers, it seems the last one that gets restarted is the IP address that is updated in Route53. In other words, it's almost like a race condition and the last one that gets started takes control in informing DNS what information to send to Route53.

Another observation is that there is only a single IP address that is associated with the LB in a project space. So, for example, `apiVersion: extensions/v1beta1
kind: Ingress
metadata:
annotations:
certmanager.k8s.io/cluster-issuer: letsencrypt-prod
field.cattle.io/publicEndpoints: '[{"addresses":["<IP_ADDRESS>"],"port":443,"protocol":"HTTPS","serviceName":"p-j4lfz-pipeline:auth-master-service","ingressName":"p-j4lfz-pipeline:auth-master-ingress","hostname":"auth.master.apps.wsp-lattice.io","path":"/","allNodes":true}]'
kubectl.kubernetes.io/last-applied-configuration: |
{"apiVersion":"extensions/v1beta1","kind":"Ingress","metadata":{"annotations":{"certmanager.k8s.io/cluster-issuer":"letsencrypt-prod","kubernetes.io/ingress.class":"nginx","kubernetes.io/tls-acme":"true","nginx.ingress.kubernetes.io/secure-backends":"true","rancher.io/globalDNS.hostname":"auth.master.apps.wsp-lattice.io"},"name":"auth-master-ingress","namespace":"p-j4lfz-pipeline"},"spec":{"rules":[{"host":"auth.master.apps.wsp-lattice.io","http":{"paths":[{"backend":{"serviceName":"auth-master-service","servicePort":80},"path":"/"}]}}],"tls":[{"hosts":["auth.master.apps.wsp-lattice.io"],"secretName":"auth-master-cert"}]}}
kubernetes.io/ingress.class: nginx
kubernetes.io/tls-acme: "true"
nginx.ingress.kubernetes.io/secure-backends: "true"
rancher.io/globalDNS.hostname: auth.master.apps.wsp-lattice.io
creationTimestamp: "2020-02-04T14:22:36Z"
generation: 1
name: auth-master-ingress
namespace: p-j4lfz-pipeline
resourceVersion: "12329978"
selfLink: /apis/extensions/v1beta1/namespaces/p-j4lfz-pipeline/ingresses/auth-master-ingress
uid: b970c97c-c788-4ba6-907c-d4f72c5d99e2
spec:
rules:

host: auth.master.apps.wsp-lattice.io
http:
paths:

backend:
serviceName: auth-master-service
servicePort: 80
path: /
tls:

hosts:

auth.master.apps.wsp-lattice.io
secretName: auth-master-cert
status:
loadBalancer:
ingress:
ip: <IP_ADDRESS>
`
I expect the IP addresses of all the nodes to be listed here.

The log on reprovisioning a node is the following:
W0221 17:17:27.481950 6 flags.go:221] SSL certificate chain completion is disabled (--enable-ssl-chain-completion=false)
nginx version: openresty/<IP_ADDRESS>
W0221 17:17:27.485215 6 client_config.go:541] Neither --kubeconfig nor --master was specified. Using the inClusterConfig. This might not work.
I0221 17:17:27.485422 6 main.go:183] Creating API client for https://<IP_ADDRESS>:443
I0221 17:17:27.491689 6 main.go:227] Running in Kubernetes cluster version v1.16 (v1.16.3) - git (clean) commit b3cbbae08ec52a7fc73d334838e18d17e8512749 - platform linux/amd64
I0221 17:17:27.495494 6 main.go:91] Validated ingress-nginx/default-http-backend as the default backend.
I0221 17:17:27.663487 6 main.go:102] Created fake certificate with PemFileName: /etc/ingress-controller/ssl/default-fake-certificate.pem
I0221 17:17:27.690878 6 nginx.go:274] Starting NGINX Ingress controller
I0221 17:17:28.033142 6 event.go:258] Event(v1.ObjectReference{Kind:"ConfigMap", Namespace:"ingress-nginx", Name:"nginx-configuration", UID:"e92b384e-cd3c-452f-b183-74a18cde28ff", APIVersion:"v1", ResourceVersion:"592", FieldPath:""}): type: 'Normal' reason: 'CREATE' ConfigMap ingress-nginx/nginx-configuration
I0221 17:17:28.033361 6 event.go:258] Event(v1.ObjectReference{Kind:"ConfigMap", Namespace:"ingress-nginx", Name:"udp-services", UID:"d48b28df-00dc-4e0a-bc5a-acd55da0a272", APIVersion:"v1", ResourceVersion:"597", FieldPath:""}): type: 'Normal' reason: 'CREATE' ConfigMap ingress-nginx/udp-services
I0221 17:17:28.044994 6 event.go:258] Event(v1.ObjectReference{Kind:"ConfigMap", Namespace:"ingress-nginx", Name:"tcp-services", UID:"f4e6d726-5a26-499f-8d64-274a9ff312f1", APIVersion:"v1", ResourceVersion:"596", FieldPath:""}): type: 'Normal' reason: 'CREATE' ConfigMap ingress-nginx/tcp-services
I0221 17:17:29.165791 6 backend_ssl.go:66] Adding Secret "p-j4lfz-pipeline/auth-master-cert" to the local store
I0221 17:17:29.166671 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-auth-service-cert" to the local store
I0221 17:17:29.167972 6 backend_ssl.go:66] Adding Secret "p-fvvtq-pipeline/wits-stag-cert" to the local store
I0221 17:17:29.168097 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-j4lfz-pipeline", Name:"auth-master-ingress", UID:"b970c97c-c788-4ba6-907c-d4f72c5d99e2", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345219", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-j4lfz-pipeline/auth-master-ingress
I0221 17:17:29.168135 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-auth-service-ingress", UID:"4f6e5aa4-c900-4968-a45a-cda789a4d423", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345224", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-auth-service-ingress
I0221 17:17:29.168183 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-fvvtq-pipeline", Name:"wits-stag-ingress", UID:"0134fa0c-ddfb-4f8d-bb56-5e848c2d6bdc", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345147", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-fvvtq-pipeline/wits-stag-ingress
I0221 17:17:29.168366 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-master-ingress", UID:"3f7ee5e4-a824-42a7-99d2-209eeccee31e", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345181", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-master-ingress
I0221 17:17:29.170037 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-master-cert" to the local store
I0221 17:17:29.170357 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-px2w9-pipeline", Name:"geoservice-test-ingress", UID:"88bbaa20-e151-4348-a139-0c6ac31e2e92", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345207", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-px2w9-pipeline/geoservice-test-ingress
I0221 17:17:29.170860 6 backend_ssl.go:66] Adding Secret "p-px2w9-pipeline/geoservice-test-cert" to the local store
I0221 17:17:29.171086 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-fix-sidebar-indent-ingress", UID:"ad1d78e2-a5b2-4f39-8d7b-a9060b0a91f9", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345239", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-fix-sidebar-indent-ingress
I0221 17:17:29.171699 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-fix-sidebar-indent-cert" to the local store
I0221 17:17:29.172432 6 backend_ssl.go:66] Adding Secret "p-7p5bc-pipeline/fpt-test-cert" to the local store
I0221 17:17:29.174169 6 backend_ssl.go:66] Adding Secret "p-82cnf-pipeline/imannotate-test-cert" to the local store
I0221 17:17:29.174781 6 backend_ssl.go:66] Adding Secret "p-fvvtq-pipeline/wits-prod-cert" to the local store
I0221 17:17:29.174824 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-7p5bc-pipeline", Name:"fpt-test-ingress", UID:"15fbc195-e2a5-4251-b87f-7cc0e750d402", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345124", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-7p5bc-pipeline/fpt-test-ingress
I0221 17:17:29.174847 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-82cnf-pipeline", Name:"imannotate-test-ingress", UID:"a1e51a30-9a6d-4830-87a9-150587b92f09", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345174", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-82cnf-pipeline/imannotate-test-ingress
I0221 17:17:29.175651 6 backend_ssl.go:66] Adding Secret "p-fvvtq-pipeline/wits-test-cert" to the local store
I0221 17:17:29.176314 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-prod-cert" to the local store
I0221 17:17:29.176343 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-fvvtq-pipeline", Name:"wits-prod-ingress", UID:"7da4f365-9ff3-42f3-be12-a8a4453f9846", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345093", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-fvvtq-pipeline/wits-prod-ingress
I0221 17:17:29.176362 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-fvvtq-pipeline", Name:"wits-test-ingress", UID:"18c930a3-e8f4-4f6c-9334-bfc632f1d85b", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345115", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-fvvtq-pipeline/wits-test-ingress
I0221 17:17:29.178064 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-stag-cert" to the local store
I0221 17:17:29.178910 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-demo-cert" to the local store
I0221 17:17:29.184690 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-prod-ingress", UID:"7055edcb-ba2c-4da5-b89f-cfad04624ba4", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345157", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-prod-ingress
I0221 17:17:29.184941 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-stag-ingress", UID:"87b20ef5-e6ec-4e59-b96a-73395bedf1cc", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345165", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-stag-ingress
I0221 17:17:29.185831 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-geolocation-cert" to the local store
I0221 17:17:29.189906 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-modularization-cert" to the local store
I0221 17:17:29.195167 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-demo-ingress", UID:"9e0cd88e-17cf-4de9-85f7-80ef3cad1148", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345104", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-demo-ingress
I0221 17:17:29.195332 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-geolocation-ingress", UID:"72953841-3d15-4c37-9292-497a7d7ddd32", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345230", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-geolocation-ingress
I0221 17:17:29.196653 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-modularization-ingress", UID:"2a8ca5d5-470a-4afe-9a6f-82209f5aa90b", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345213", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-modularization-ingress
I0221 17:17:29.199538 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-geoservice-ingress", UID:"41e3c900-f59f-402d-b123-de2aa4e150c0", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345236", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-geoservice-ingress
I0221 17:17:29.199712 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-geoservice-cert" to the local store
I0221 17:17:29.200569 6 backend_ssl.go:66] Adding Secret "p-jvztj-pipeline/corgis-review-style-updates-cert" to the local store
I0221 17:17:29.203720 6 event.go:258] Event(v1.ObjectReference{Kind:"Ingress", Namespace:"p-jvztj-pipeline", Name:"corgis-review-style-updates-ingress", UID:"5a2b6158-5ff8-440e-bf8c-df4bb382c271", APIVersion:"networking.k8s.io/v1beta1", ResourceVersion:"12345141", FieldPath:""}): type: 'Normal' reason: 'CREATE' Ingress p-jvztj-pipeline/corgis-review-style-updates-ingress
I0221 17:17:29.224297 6 nginx.go:318] Starting NGINX process
I0221 17:17:29.224516 6 leaderelection.go:235] attempting to acquire leader lease ingress-nginx/ingress-controller-leader-nginx...
I0221 17:17:29.226090 6 controller.go:133] Configuration changes detected, backend reload required.
I0221 17:17:29.227312 6 status.go:86] new leader elected: nginx-ingress-controller-qbtzq
I0221 17:17:29.380766 6 controller.go:149] Backend successfully reloaded.
I0221 17:17:29.380973 6 controller.go:158] Initial sync, sleeping for 1 second.
I0221 17:17:5

Sidenote from external-dns point of view:
If the ingress.status has only one IP/hostname, external-dns can only assign all mentioned spec.rules[].host to the given one.
To fix the issue the ingress controller configuration or the controller needs to be fixed.

I can confirm that the problem is with the latest image (rancher/nginx-ingress-controller:nginx-0.25.1-rancher1)
I have reverted to the previous image (rancher/nginx-ingress-controller:0.21.0-rancher3) and everything seems to be working fine.

The actual problem that caused this is described in #26677. I filed it separately because that let me put the right title on the issue as a bug report. (This Issue instead describes a symptom of the bug.)


Devices that were checked working

This issue is for devices which were checked working with pyrgbdev.
Since I do not have every device in planet Earth, I need some help from people who owns those devices.
If you have checked them working, please make a small issue comment that specific device had been confirmed working.
Also, if you would like to attach a picture of how it works, you can!
Please do not close this issue, this is a place for users reporting that their device works
An example of reporting devices that were working is like below:

Working devices
Corsair

K95 RGB Platinum
GLAVE PRO RGB
MM800RGB
VOID PRO RGB

Razer

Deathadder Elite RGB

Pictures

By
Gooday2die


Catching InvalidOperationException when refreshing web perf counters

Fix issue #1686
Changes
Catching InvalidOperationException when refreshing web perf counters, more details in the comment.
Checklist

[x] I ran Unit Tests locally.
[x] CHANGELOG.md updated with one line description of the fix, and a link to the original issue if available.

For significant contributions please make sure you have completed the following items:

[ ] Design discussion issue #
[ ] Changes in public surface reviewed

The PR will trigger build, unit tests, and functional tests automatically. Please follow these instructions to build and test locally.
Notes for authors:

FxCop and other analyzers will fail the build. To see these errors yourself, compile localy using the Release configuration.

Notes for reviewers:

We support comment build triggers

/AzurePipelines run will queue all builds
/AzurePipelines run <pipeline-name> will queue a specific build

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


Allow overriding image src property

[X] I tried updating to the latest version.

[ ] I can't, there is an issue.

[ ] I Am on Windows

[ ] I Am on Bash on Windows

[X] I Am on Linux

[ ] I Am on OS X

Description
The best way of achieving image lazy loading is by replacing the src property with thumbnail and moving the actual full image path to another like data-src. Then javascript is used to replace src with data-src when the image is to be loaded.
This isn't possible afaik with jekyll assets as you can't override the src property or ask for it to be set at a different property.
Steps
Output

Expected

I feel like this is complex and out of scope. And this is a classic case of why plugins exist, because to me this really feels like I wish to keep convention over configuration. We also provide numerous other ways to do this as well.

I was scratching my head around how to express this elegantly, and I have to agree with you.

We also provide numerous other ways to do this as well.

Could you elaborate? How would I accomplish the lazy load? Thanks!

The documentation provides ways for you to manually build out HTML and get the same data we have access to.  On top of that, it also provides an example of a plugin that would give you access to the full Ruby scope.
Thanks!

A workaround would be:

But it goes against the Liquid philosophy I guess.


set-certificatetemplateacl contraint error

Hello there,
thanks for this great module, but I ran into an issue which I was not able to solve. I cannot set template ACLs, it always produces this error no matter what user or group I try. I am Enterprise Admin and have local admin on the CA:
PS C:\> import-module pspki
PS C:\> get-certificatetemplateacl -Template WebServer

Path                                    Owner                                   Access
----                                    -----                                   ------
CN=WebServer,CN=Certificate Template... AD\Enterprise Admins                    {PKI.Security.AccessControlEntry2, P...

PS C:\> get-certificatetemplateacl -Template WebServer | Add-CertificateTemplateAcl -User ad\ca-templateadmins -AccessType Allow -AccessMask Read,Write | Set-CertificateTemplateAcl
Set-CertificateTemplateAcl : Exception calling "CommitChanges" with "0" argument(s): "A constraint violation occurred.
"
At line:1 char:158
+ ... k Read,Write | Set-CertificateTemplateAcl
+                    ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,Set-CertificateTemplateAcl

I am using the latest version of your module on a Windows 2008 R2 enterprise CA with a domain/forest also on 2008 R2 level. I have looked into the source files, added debug output and tried to manually do the steps in the module, but I could not solve this isse. Can you please help me?

Sorry, I can't repro the issue. Just ran latest public build and it works as expected for me. I can suspect that although you are logged on with Enterprise Admins permissions, the PowerShell console is not running in elevated mode.

I'm having the same issue as well. I've confirmed that I can perform the same operating through the MMC Certificates Template without a problem. I'm run an elevated Powershell prompt, but keep getting this error.

That's strange, because I was unable to repro the issue. I tried various command parameters, but everything succeeded for me and never saw this error.

What's strange now is if I set the user to be the owner of the template and run the command, I don't get an error and it just outputs the full path to the template but does not actually change the ACL on the template.

I made some research and found that this error may come from insufficient priveleges of the caller. Your attempt to explicitly add user to owner implicitly confirms my findings. This is certainly an issue with priveleges required by AD.

closing as inactive.