Proposal: multi-transaction

Just a weird idea I came up: a transaction that is a multi-transaction.
The idea is to have a single transaction that incorporates multiple "sequential" transactions, such that they either succeed all in one ledger or fail together. The transaction will result in a single meta.
If one of the transaction in the sequence get a tec failure, then all the other transactions are not applied.
If one of the transactions get a ter code, all the transactions are not applied in that ledger and tried in future ledgers.
XRP fee should be higher (multiply by the number of transactions?).
Application examples:

N to 1 payment: a user could send a payment in one currency using different currencies at the same time. For example he pays 100XRP using 20 USD, 0.001BTC and 500 CNY.
Advanced market making strategies: for example placing orders at different spread without risking that one or many transactions don't go trough.
Arbitrage: "close the loop" in one single transaction made up of multiple payments or offerCreate or both
More???

I think it could open up a lot of possibilities, but it might be tricky to implement.

I would expand this a bit:
There are 4 scenarios possible
1:1 payments - currently implemented
1:n payments - could be done by allowing to submit a list of issuer/amount/currency/paths instead of a single instance of these, probably needs to be applied in order and should atomically fail if any sub-payment fails for any reason (e.g. dry paths)
n:1 payments - would require serious custom logic (you are describing sending out multiple different currencies, not receiving them from multiple different accounts atomically at once). Would be probably doable with muti-signing (all inputs to the transaction must be validly signed by whatever method is possible for each sending account). Should also fail atomically (one input can't be applied - e.g. because there is no trust line - no value gets transferred at all).
m:n payments - while kinda combining the two concepts above, it maybe could be necessary to also have some logic to deal with paths. I haven't fully thought this one through if it would be just a logical next step or if there's some fundamental problem hidden somewhere (I suspect paths could be a bit problematic if they are not supplied with the transaction).

https://www.stellar.org/developers/guides/concepts/transactions.html

List of operations
Transactions contain an arbitrary list of operations inside them. Typically there is just one operation, but it’s possible to have multiple (up to 100). Operations are executed in order as one ACID transaction, meaning that either all operations are applied or none are. If any operation fails, the whole transaction fails. If operations are on accounts other than the source account, then they require signatures of the accounts in question. Here are the possible operation types:
Create Account
Payment
Path Payment
Manage Offer
Create Passive Offer
Set Options
Change Trust
Allow Trust
Account Merge
Inflation
Manage Data

@MarkusTeufelberger yeah, that is only one of the applications. But the multi-transaction could solve much more applications.
And as pointed by @yxxyun it seems stellar is already supporting it.

I would like to be able to submit a multi-transaction where an account may take the output of noripple_check to then be able to use the transactions array provided to then attempt to set the affected accounts with the ClearNoRipple flag.  This would especially help people lower their owner count/owner reserve.  I would prefer that if this were to be implemented, if a transaction were to fail perhaps due to an insufficient reserve, then to continue onto the next transaction in the array and report an array of error codes if any occurred.  Thanks for your consideration on this use case of multi-transaction's potential functionality.

JoelKatz had an "agenda" proposal at one point. Can't recall why it was
redlighted.

I would love to be able to send an "agenda" transaction that allows me to apply specific types of other ripple transactions which will either a) attempt to submit & verify the transaction, if an error occurs create an array of errors to report at the end of processing or b) if an attempt to submit a transaction is verified to have failed with an error message then the agenda transaction fails and no other transactions intended for submission are attempted.

This is an interesting proposal but not actionable as is. We very much welcome an actual proposal on the specific details. I am closing this issue for now, and we can always open a new one referencing this issue.


Removing project templates

Migrating project templates to the compiler repo.

These were migrated into the qsharp-compiler repo with PR 191


Addressing Feedback from CppCon presentation proposal

I'm looking for some good feedback, on perhaps the hardest part of any open source software project....documentation.
One review in particular is a bit worrisome for me, since the library is intended to be "intuitive" and easy to use. It was also the only non-accept rating, being "borderline." From the RaftLib org members (and the web in general) are there any improvements we can make to the documentation (either the raftlib.io front page, the wiki, or through more blogposts perhaps) to make RaftLib easier to use and understand from the start? The review in question is below and please respond directly via comments (as always, be respectful and constructive). Thanks!!

I was disappointed in the level of description, examples, etc in the Raftlib information I found.  Given that I'm at least somewhat familiar with the topic I expected that I would "get" it right away. It was harder than I thought it should be.  Also there is no mention of other approaches - HPX for example.  I think I know what he means by "Streams" but maybe I don't.  Of course this is partly due to the usage of "stream" in the standard library which has a clear meaning - but I'm hoping the authors "Streams" aren't related.  There are a number of data flow implementation methods that I'm thinking are more promising - in particular the Standard library ranges proposal in development.   I have the feeling that this is a work in progress rather than something that programmers are expected to be able to use right now.

Here is my thought.
The new description is very informative, albeit "concurrency" doesn't tell the full picture. "parallelism" might be more accurate. For audience from the academic world, linking RaftLib to a well-known parallel computing model can help them quickly understand it. Communication Sequential Processes (CSP) looks to be a close one, but I haven't verified that formally.
To help potential users get started, a step-by-step tutorial is necessary. On top of that, a cook book that provides examples for common application patterns would also be helpful.

Take a swag at incorporating more of a "parallelism" connotation. Take a look at the wiki, I'm wondering how far we are from a true step by step tutorial? I'd been reticent to write too much as we're still in alpha stage.


modified convolution document

fix #14027.
Document of _MaskedConv and MaskedConv2D could be revised too. Reasons are as follows.
https://github.com/tensorflow/tensorflow/blob/5fbda9d8da7b98f62e83a392f047adf307b48b02/tensorflow/contrib/model_pruning/python/layers/core_layers.py#L164-L171
https://github.com/tensorflow/tensorflow/blob/5fbda9d8da7b98f62e83a392f047adf307b48b02/tensorflow/contrib/model_pruning/python/layers/core_layers.py#L438-L447

Can one of the admins verify this patch?

I just changed docstring of functions. But Unit tests couldn't be passed. Some error logs are as follows. Additionally, I merged 'upstream/master' into my branch. What should I do next? Thank anyone who helps me.
-----------------------------------------error logs-------------------------------------------------
/tmpfs/tmp/bazel/execroot/org_tensorflow/bazel-out/k8-py3-opt/bin/tensorflow/examples/adding_an_op/cuda_op_test.runfiles/org_tensorflow/tensorflow/tools/ci_build/gpu_build/parallel_gpu_execute: line 40: 30163 Segmentation fault      (core dumped) $@


steps.ghd.outputs.sha does not return short sha

Hi,
(...)
      - name: Extract metadata for ${{ matrix.containers }}
        id: metadata
        uses<EMAIL_ADDRESS>        with:
          flavor: |
            latest=true
          tags: |
            type=raw,value=${{ steps.ghd.outputs.tag }}-${{ steps.ghd.outputs.distance }}-${{ steps.ghd.outputs.sha }}
(...)

return ghcr.io/vre-hub/vre-iam-rucio-sync:v1.0.0-rc.1-17-05d57f74bb467c964fc25756af6c44cd5cd7f161 not 05d57f7 as expected and described on the docs
https://github.com/vre-hub/vre/actions/runs/7225844181/job/19690127353#step:5:40

Thanks for flagging the issue!
The long SHA from steps.ghd.outputs.sha is intentional. My bad on the README.md, fixed it now. Apologies for the mix-up.
Also, I'm working on adding the short SHA output. Will get it done soon, so hang tight!

I've added the short SHA output.
Here's how you can update your workflow to get the expected results:
-             type=raw,value=${{ steps.ghd.outputs.tag }}-${{ steps.ghd.outputs.distance }}-${{ steps.ghd.outputs.sha }}
+             type=raw,value=${{ steps.ghd.outputs.tag }}-${{ steps.ghd.outputs.distance }}-${{ steps.ghd.outputs.short-sha }}

Hope this helps! Let me know if there's anything else.

Hi @proudust,
Thanks a lot for the quick answer and for solving the bug !
I confirm that everything now works as expected; https://github.com/vre-hub/vre/actions/runs/7463189852/job/20307279027#step:4:15 🥳 🥳 🙌 🙌

Thanks for confirming the fix.
Great to hear everything's working as expected now.
Looking forward to any more feedback you might have.


Testing compatibility between OSMDashboard and WeSki application

Describe briefly your feature.
Verifying the sample data between WeSki and OSMDashboard.
Describe the solution you'd like
Testing story performing manual testing to ensure data points and statistics are identical.
Describe alternatives you've considered
Perform the required physical activities and collect data in order to compare the datasets between OSMDashboard and WeSki.
Additional context
Verifying the compatibility ensures the correctness of the OSMDashboard application with respect to WeSki since WeSki behaves as an example as to how OSMDashboard and OpenTracks is expected to work.

Successfully validated that the newly developed features work seamlessly with the existing legacy code.


Log Metadata

Display log metadata such as application of origin, timestamp, etc...

Currently displays machine of origin but not application, probably both are good?


Report foreign key errors as InvalidArgument errors.

Just curious to see what happens if we return InvalidArgument instead of NotFound for foreign key violations.
(see comments inline with the changes)

From https://grpc.github.io/grpc/core/md_doc_statuscodes.html:

The operation was rejected because the system is not in a state required for the operation's execution. For example, the directory to be deleted is non-empty, an rmdir operation is applied to a non-directory, etc. Service implementors can use the following guidelines to decide between FAILED_PRECONDITION, ABORTED, and UNAVAILABLE: (a) Use UNAVAILABLE if the client can retry just the failing call. (b) Use ABORTED if the client should retry at a higher level (e.g., when a client-specified test-and-set fails, indicating the client should restart a read-modify-write sequence). (c) Use FAILED_PRECONDITION if the client should not retry until the system state has been explicitly fixed. E.g., if an "rmdir" fails because the directory is non-empty, FAILED_PRECONDITION should be returned since the client should not retry unless the files are deleted from the directory.

Update: this is on hold pending a broader discussion of existing patterns and style.


Unescape ES6 \u{xxxxxx} literals

This is for the mavericks who are wild enough to employ such an
intense level of trickery in their code

Thank you for the feature!


github-cli: bump to 1.7.0

Existing package

[Y] I am the maintainer of this package.

Update sources

I'm confused, the sources file is updated.
Check the diff

Oh jeez I'm sorry I'm just dumb.
I wasn't expecting it to be tied to a commit.


fixed save access token

Error fixed

An active access token must be used to query information about the current user.

Fixed in v1.2.0


Change polling after install

Hi,
Is there a way I can change the polling after installation?

Yes, there is a HA service for doing that, it's documented even with an automation example to use an input_number to do it. Please read the README :)


token-js: u64 should implement its own constructor

Try this code:
const bn = new BN(5);
const u = new u64(10);
const u2 = new u64(bn);

console.log("bn.toBuffer", bn.toBuffer);
console.log("u.toBuffer", u.toBuffer);
console.log("u2.toBuffer", u2.toBuffer);

Logging u.toBuffer shows a function. But logging u2.toBuffer shows undefined, even though toBuffer is defined right here! https://github.com/solana-labs/solana-program-library/blob/master/token/js/client/token.js#L54-L65
Why? It's because of BN's constructor: https://github.com/indutny/bn.js/blob/master/lib/bn.js#L21-L24
This behavior is very misleading in TypeScript, because u2 is typed as a u64, which you expect to have a toBuffer function.
In order to fix this, I think we should override BN's constructor in u64.

@joncinque gave this one a shot, could you take a look at #2975

This issue no longer applies -- @solana/spl-token uses native bigint.


mssql - connect show success, but Prepare failed:InitialBytes InitializeSecurityContext failed 8009030c

I use windows authentication, so not input user id and password.
I add sentence to detect whether connect success. It seems connect is work, however,
It show errors like: Prepare failed:InitialBytes InitializeSecurityContext failed 8009030c
Can you help on it? Thanks.
Setting:
var debug = flag.Bool("debug", true, "enable debugging")
var password = flag.String("password", "", "the database password")
var port *int = flag.Int("port", 1433, "the database port")
var server = flag.String("server", "AUSASLDATA", "the database server")
var user = flag.String("user", "", "the database user")
...
conn, err := sql.Open("mssql", connString)
if err != nil {
log.Fatal("Open connection failed:", err.Error())
} else {
fmt.Printf("Connect Success")
}
...
Errors:
password:
port:1433
server:AUSASLDATA
user:
connString:server=AUSASLDATA;user id=;password=;port=1433
Connect Success2016/01/23 22:29:18 Prepare failed:InitialBytes InitializeSecurityContext failed 8009030c

Hello @ScottHuangZL
add in your connections parameters the SQL Server Service Principle Name (SPN)
e.g. connString = "server=AUSASLDATA;ServerSPN=MSSQLSvc/AUSASLDATA:1433"
see also https://technet.microsoft.com/en-us/library/bb735885.aspx

Thank you @dimdin
That mssql server at remote site and I do not have sa access right, just able write/read to one of database in that server.
I use PHP to connect the remote mssql, it is working without any issue.
I do not have sufficient access rights to complete execute "setspn -A MSSQLsvc/AUSASLDATA:1433 "
I am not sure why we need set SPN when use golang??
Any hints can help me? Thanks again.

What connection string do you use to connect from php?
Is the golang machine on the same windows domain with the sql server?

As for php, I use Yii2 framework, the connection string as below:
'connectionString' => 'sqlsrv:Server=AUSASLDATA;Database=EOL',
The drivers provided by Microsoft for php, thanks.
That server at USA, I am in China, anyway, should be different domain, however we are in same company and can access that DB. Thanks.
I am switch to ODBC to connect mssql , it is work for my go program now.
And waiting anybody help for go-mssql, thanks.

@ScottHuangZL If you are using windows auth, what versions of NTLM does the server support? I think we only support NTLMv1 at the moment.

@kardianos - Can you link to the duplicate issues when you close them and mark them like this? It is really helpful when people stumble to these issues through google or outdated links.

See https://github.com/denisenkom/go-mssqldb/issues/146

hi @ScottHuangZL @kardianos I get same issue when use gorm with go-mssqldb, could you give me some advice?


improve performent of hash join in some case

improve performent of hash join  when build table has to many duplicated row, this wii cause hash table collisions and slow down the probe performent.

Hi, @yangzhg
Could you attach some benchmark for this PR?

Hi, @yangzhg
Could you attach some benchmark for this PR?

done

This is a wonderful job! 💯


space after constructor name breaks syntax JS highlight

VSCode Version: Version 1.7.1 (1.7.1)
OS Version: OS X El Capitan 10.11.6

Steps to Reproduce:

write somewhere in file:

const a = new Promise(resolve => {
    const file = this.props.doc.file;
    resolve({
        abc: Math.random()
    });

    class Foo extends React.Component {
        render() {
        }
    }

});

syntax highlight below is correct for now:

2. Now add space between Promise and (
3. syntax highlight is now broken.

Thanks for opening this. This is a grammar issue so moving to that repo.

This issue was moved to Microsoft/TypeScript-TmLanguage#347


Add support for logging to a file

This could be useful for troubleshooting messages delivered by another tool such as Nagios Core that (afaik) doesn't capture output from executed notification commands.

Note: Have example use the value for log_file from nagios.cfg.
For example:
# LOG FILE
# This is the main log file where service and host events are logged
# for historical purposes.  This should be the first option specified
# in the config file!!!

log_file=/var/log/nagios3/nagios.log

In our case, /var/log/nagios3/send2teams.log should be the fully-qualified log file path used in the example(s). Specifying /tmp/send2teams.log may not work for Nagios' purposes unless SELinux or AppArmor policy changes are applied to permit that access.

Snippet used for debugging/troubleshooting work on #225:
	f, err := os.OpenFile("/var/log/nagios3/send2teams.log",
		os.O_APPEND|os.O_CREATE|os.O_WRONLY, 0644)
	if err != nil {
		log.Println(err)
	}

	defer f.Close()
	log.SetOutput(f)

Needs work:

permissions adjustment
handle error from f.Close()

The fixed path/filename would be replaced with a user-specified value and sanitized where possible.


Restore test runs, add jacoco, and fix resulting broken tests

This resolves shortcoming in junit 5 migration.  I had initially wrote that code months ago and coming back to it only ran on command line.  Outside of missing import changes in a few spots at that time, it appeared my work was done.  However, the jump to junit 5 resulted in command line tests no longer running which then highlighted this was not complete on a small number of tests.  To ensure this type of thing never happens again, I've added jacoco to the mix.  Each pom has jacoco coverage number set.  The top one which is just pom is set to 100% for clarity reasons.  The other three are set to their appropriate current code coverage (git module 81%, svn module has no tests, and main plugin at 77%).  The svn module didn't actually need the setting as jacoco is skipped when no tests but presume we want to provision tests there at some point.  We may want to go further here and add coveralls to show the coverage overall for the repo itself.
note: jacoco reports will get added, these are not shown from main site but that site needs some retooling to fluido and few other things so it looks modern rather than legacy apache LAF. [update: noticed this behavior locally to not show jacoco from the site directly was due to attach of descriptor only rather than full site run that is going to happen after merge with github action.  I have updated to fluido locally though and it looks improved and also noting that main site on readme points to the old gh pages unless there is some other magic].

thanks! let's merge. there is an automated github action that generates the snapshot site at each commit: https://mycila.carbou.me/license-maven-plugin/reports/4.2-SNAPSHOT/


ja: Make docs/concepts/services-networking/dns-pod-service.md follow v1.19 of the original text

This is a Feature Request

What would you like to be added
Update docs/concepts/services-networking/dns-pod-service.md to follow v1.19 of the original (English) text.
Why is this needed
content/ja/docs/concepts/services-networking/dns-pod-service.md is outdated.
Comments
/language ja
/good-first-issue
File to update
https://github.com/kubernetes/website/tree/dev-1.19-ja.1/content/ja/docs/concepts/services-networking/dns-pod-service.md
Original
https://github.com/kubernetes/website/tree/release-1.19/content/en/docs/concepts/services-networking/dns-pod-service.md
diff between translated and v1.19
https://gist.github.com/9ea7b74f8d1485abec01ae9024520a7d
Note
Currently, we use dev-1.19-ja.1 branch. Please open a PR targeting the branch.
ref
https://kubernetes.io/ja/docs/contribute/localization/
If you have a question, feel free to ask us at slack.k8s.io #kubernetes-docs-ja channel.

/triage accepted

/assign

/close


Histogram Equalisation

Implement histogram equalisation as part of an enhancement module: algorithm

https://github.com/jturner314/ndarray-stats/pull/29 waiting on CDF to be merged


Implement asset manager

Asset Manager are needed to push data to executor backends that cannot initiate internet connections to Covalent.

Hey team! Please add your planning poker estimate with Zenhub @AlejandroEsquivel @cjao @kessler-frost

@FyzHsn can you please add more details to this issue's description? Thanks!


Xnero: Watch uzynskidigital(?!.com)

Xnero requests the watch of the watch_keyword uzynskidigital(?!\.com). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

uzynskidigital(?!\.com) has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Send message option to save for next outlines saves now also title an…

…d message

Codecov Report

Merging #250 (fcb4040) into master (43fbcba) will decrease coverage by 0.01%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #250      +/-   ##
==========================================
- Coverage   87.52%   87.51%   -0.01%
==========================================
  Files          79       79
  Lines        5492     5496       +4
==========================================
+ Hits         4807     4810       +3
- Misses        685      686       +1

Flag
Coverage Δ

unittests
87.51% <50.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

base/views/outline_create.py
92.59% <ø> (ø)

base/views/results_get_deff.py
85.07% <0.00%> (-2.62%)
:arrow_down:

base/models/profile.py
97.36% <100.00%> (+0.14%)
:arrow_up:

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


⚠ net.chenshaowen.com has degraded performance

In 35bc12f, net.chenshaowen.com (https://net.chenshaowen.com) experienced degraded performance:

HTTP code: 400
Response time: 2837 ms

Resolved: net.chenshaowen.com performance has improved in 2be5ffb after 12 minutes.


API: json: unknown field "allowedActions"

started getting this error from SLO API
grafana:index:SLO (requests-slo-panel):
error: 1 error occurred:
* updating urn:pulumi:grafana-slo-transfer-claim-service::grafana-slo::grafana:index/sLO:SLO::requests-slo-panel: 1 error occurred:
* Unable to read SLO - API: json: unknown field "allowedActions"
{"alerting":{"fastBurn":{"annotations":[{"key":"name","value":"SLO Burn Rate Very High"},{"key":"description","value":"transfer-claim-service Requests error budget is burning too fast, this is affecting Ticketing."}]},"slowBurn":{"annotations":[{"key":"name","value":"SLO Burn Rate High"},{"key":"description","value":"transfer-claim-service Requests error budget is burning fast, this is affecting Ticketing."}]}},"description":"transfer-claim-service Requests SLO Panel","destinationDatasource":{"uid":"grafanacloud-prom"},"labels":[{"key":"service_name","value":"transfer-claim-service"},{"key":"Location","value":"prodblue-us-west-2"},{"key":"Feature","value":"Ticketing"}],"name":"transfer-claim-service Requests","objectives":[{"value":0.9,"window":"28d"}],"query":{"ratio":{"groupByLabels":["route"],"successMetric":{"prometheusMetric":"requests_total{location='prodblue-us-west-2',route!='/health',service_name='transfer-claim-service', status='200'}"},"totalMetric":{"prometheusMetric":"requests_total{location='prodblue-us-west-2',route!='/health',service_name='transfer-claim-service'}"}},"type":"ratio"},"readOnly":{"allowedActions":["read","write"],"drillDownDashboardRef":{"UID":"ddtbst2dpqm80e"},"provenance":"api","status":{"type":"created"}},"uuid":"1er2zuzey1at2l35ecjgk"}
it happens in all versions of @pulumi-grafana

It seems to be fixed in Terraform provider v3.5.0: https://github.com/grafana/terraform-provider-grafana/pull/1697


Neogit seems to call TermOpen via setup

Description
One of the recent Neogit updates seems to be causing Neovim to call TermOpen when the setup function is called. I like to have my terminals in Neovim open in insert mode, so I have the following in my config: vim.api.nvim_create_autocmd("TermOpen", { command = "startinsert" }).
This results in Neovim's normal buffers opening in insert mode, rather than normal mode, when it should only be applied to the terminal buffers.
I installed Neogit for the first time yesterday and didn't have this problem, but after updating my plugins today I noticed it, which is why I think a recent update is the culprit.
Neovim version
NVIM v0.9.1
Build type: Release
LuaJIT 2.1.0-beta3
Operating system and version
Arch Linux
Steps to reproduce

Update Neogit to the current release with the TermOpen autocmd
Open Neovim

Expected behavior
Neovim opens in normal mode
Actual behavior
Neovim opens in insert mode
Minimal config
vim.cmd([[set runtimepath=$VIMRUNTIME]])
vim.cmd([[set packpath=/tmp/nvim/site]])
local package_root = "/tmp/nvim/site/pack"
local install_path = package_root .. "/packer/start/packer.nvim"
local function load_plugins()
  require("packer").startup({
    {
      "wbthomason/packer.nvim",
      {
        "TimUntersberger/neogit",
        requires = {
          { "nvim-lua/plenary.nvim" },
          { "sindrets/diffview.nvim" },
        },
        config = function()
          print("loaded neogit")
          require("neogit").setup()
        end,
      },
    },
    config = {
      package_root = package_root,
      compile_path = install_path .. "/plugin/packer_compiled.lua",
      display = { non_interactive = true },
    },
  })
end
if vim.fn.isdirectory(install_path) == 0 then
  print("Installing neogit and dependencies.")
  vim.fn.system({ "git", "clone", "--depth=1", "https://github.com/wbthomason/packer.nvim", install_path })
end
load_plugins()
require("packer").sync()
vim.api.nvim_create_autocmd("TermOpen", { command = "startinsert" })

From git bisect seems the commit causing this behavior is 66bdfe592b038fbeb129921a05e2d45b1ebd097a

Seems to be this line: https://github.com/NeogitOrg/neogit/blob/015c27c5b74eba9b151c683579c838e742bf36a2/lua/neogit/process.lua#L104

This line is the root where this problem is coming from ultimately:
https://github.com/NeogitOrg/neogit/blob/ea570537d6ee1893a6ec66d2c46cc646d602e493/lua/neogit/lib/git/cli.lua#L451
This is because the process lib spawns a terminal in the background.

Hey @KodyVB, mind giving my patch a shot?
https://github.com/treatybreaker/neogit
That defers the nvim term open until absolutely needed so it doesn't interfere with your insert enter autocmd.

From what I've seen, those changes do seem to fix it and I haven't noticed any unwanted behaviors from it.

I have actually found one issue that I'm currently battling with it. The console key (by default #) doesn't scroll to the end of output, it scrolls to the entire end of the buffer with the modifications I have made.
Currently working on resolving it.

Fixed.


Eventsource streaming job in databricks

Commission new streaming cluster in DBR instance
Streaming job code is created in python
Job code is deployed into our databricks instance and verified running
Microbatch code structure is prepared so that we in the other stories can insert our eventhandling code

Databricks:
pros:
When using protobuf we would have schema tightly coupled with delta table
Master data is stored in delta lake after event playback, for immediate comsumption
cons:
No native protobuf integration
Azure function as intermediate between service bus and eventhub to get messages into databricks
More infrastructure needed to complete pipeline
No Databricks event sourcing reference implementation
Databricks instance would require to always be on and running for streaming jobs
C#
pros:
Solution can be shared with other teams
POC for entire chain already exists
Increased development speed
Easier integration test, due to easy access to output
Cosmos db can be substituted with other data storage options, eg. JSON files in blob storage, Mongo DB, SQL database and other
Nuget/Protobuf usage (C# classes distributed by Nuget and not manually generated by proto contract itself)
cons:
Cosmos db connector 3 used in Databricks jobs is new technology
Thoughts:
We would only need to read master data from databricks, never update.
Updates/substitution would only be triggered by integration events.


duplicate notifications

Every notification generated by my icinga2 setup is generating duplicate messages in hipchat. Have you seen this before?

Let me know what would be useful in debugging this isseue.
Also, thank you so much for putting this project out there, it's a very convenient thing to have =)

Sorry I have not encountered this before. Are you sure that you haven't created duplicate contacts or something like that?

Sorry for the false alarm. I think I've found what's happening. The configuration provided in your documentation suggests just sending to the user_groups = [ "icingaadmins" ] so for each user in that group it sends a message which isn't a problem unless you have multiple users in the icingaadmins group =).

Ah! Good to hear you solved it. :)


RFC for lower precision execution through the dynamo frontend

https://github.com/pytorch/TensorRT/discussions/2455


Security/Authentication handling is weird

I have the case of NTLM authentication (yay, right? :P). Even when using basic auth (which I hoped would work as a fallback), I get a 401 with a WWW-Authenticate header. So I started digging through the code. First reading the code from the provided BasicAuthSecurity, then reading how request.js likes to have it: https://github.com/mikeal/request#http-authentication
Turns out request.js already has (seemingly) way sophisticated authentication capabilities but they go unused and instead special headers are injected.
EDIT: I just saw addOptions, I'll try porting the BasicAuthSecurity to using that instead of addHeaders (which is not documented btw) and see where I'll get.

@FrankyBoy did you find a solution for this that can be shared with the community?

nope, sorry, just fell back to request-ntlm (because ntlm is what I needed) and extended that a little bit -> request-ntlm-continued

@FrankyBoy thanks


Add Debug supertype to iri parameter in get function of Loader trait

impl Loader for ResourceLoader {
    fn get<T: Borrow<str>>(
        &self,
        iri: Iri<T>,
    ) -> std::result::Result<(Vec<u8>, String), LoaderError> {
    iri.borrow_term() // This is not possible because T does not implment Debug
    iri.clone() // This is also not possible, because T does not implement Clone
...
}

Just to clarify: your suggestion is to change the declaration of Loader::get from
fn get<T: Borrow<str>>(
        &self,
        iri: Iri<T>
    ) -> Result<(Vec<u8>, String), LoaderError>;

to
fn get<T: Borrow<str> + Debug>(  // <-- added "+ Debug" here
        &self,
        iri: Iri<T>
    ) -> Result<(Vec<u8>, String), LoaderError>;

right?
I'm guessing that you are encountering issues when generating a log or error message, because something like format!("{:?}", iri) does not work?
Given that Iri<T> can easily be converted to strings, I think you can actually do without the Debug trait: format!("<{}>", iri.as_str() would give an even better  message (in terms of user-friendliness, at least).

Addung either Debug or Clone is fine.
I needed just two owned values of iri.

Sorry, I think I initially read the body of your issue too quickly.
The immediate solution to your problem is to use Iri::as_ref, which gives you a "copy" of the IRI, borrowing data from the original.
I understand that it was tempting to use Term::borrow_term instead, and indeed, it is surprising that Iri<T> does not implement Term in this case (even I was surprised :stuck_out_tongue: ). I just opened #170 to remove this surprising exception.


CodeMirror concat issue

It looks like the concat task is searching for files that no longer exists src/codemirror-4.inline-attachment.js, src/codemirror-3.inline-attachment.js.
https://github.com/Rovak/InlineAttachment/blob/master/Gruntfile.js#L29

fixed in #76


Contact with no Image

Hi, First the first, Awesome work thanks for this library
I got an issue when testing on simulator and tried to get a contact image, returning in a crash .. I located the problem here (SwiftAddressBookPerson.swift) :
public var image : UIImage? {
	get {
		return UIImage(data: ABPersonCopyImageData(internalRecord).takeRetainedValue())
	}
}

If a user does not have picture the crash comes in, I updated to this, not sure if it is the best way.
public var image : UIImage? {
    get {
        if ABPersonHasImageData(internalRecord) {
            return UIImage(data: ABPersonCopyImageData(internalRecord).takeRetainedValue())
        } else {
            return nil
        }
    }
}

Actually, I should be able to initialize UIImage with nil, for my taste... There are failable initializers in Swift, so why not? But you´re right, the documentation tells that it must be nonnull. Your way looks perfect to me, you could do it in one line by using the ? operator like  return ABPersonHasImageData(internalRecord) ? UIImage(data: ABPersonCopyImageData(internalRecord).takeRetainedValue()) : nil, if that line would not get longer than the others. I´d love to have a pull request from you (please don´t forget to sign the contributors agreement first)

Same problem here. Hope this get fixed soon. Until there, manual fix for me. Thanks for the library and @ed-mejia thanks for the fix. :)

I thought you would maybe send a pull request but if not, I will fix it myself this weekend.

I'm Very sorry I will do it this weekend, I been short of time :(

Me, too :) No need to be sorry!


SE Home Meter stopped working

Describe the bug
SolarEdge uploaded a firmware update to my energy meter device yesterday (1.0.64 => 1.0.90). I cannot get any values through Modbus anymore since today morning. Any ideas on this?
Device ID: MTR-240-3PC1-D-A-MW

https://www.solaredge.com/de/produkte/zaehler-und-sensoren/solaredge-home-energiezaehler

Expected behavior
A steady flow of sensor values. ;)
Screenshots

Diagnostic File
config_entry-solaredge_modbus_multi-b9021318c3c4e0163dd38bf0171164c9(5).json
Debug logs
No response
Home Assistant Version
2024.10.0
solaredge-modbus-multi Version
3.0.2
Installation Type
Container
Read the Instructions

[X] My configuration follows https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Configuration
[X] I have read the Known Issues https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Known-Issues
[X] This issue is not about a template error (do not report template errors, read https://github.com/WillCodeForCats/solaredge-modbus-multi/wiki/Template-Design-Notes)

Additional Context
No response

The meter is being identified as a meter, but all of the values the inverter returns for the meter are 0x8000 or 0x0.
You can try turning everything off and then back on to cold boot the system, but this isn't an issue to fix in the integration. The firmware in the meter or inverter needs to be fixed if a cold boot doesn't help.

The meter started working again 12 hours later without rebooting. Looks like a firmware issue. Thanks for your support anyway!


Any write to misa may cause UXL or SXL to change as a side effect

What of:
The SXL field is only implemented if S-mode is implemented, and is
otherwise hard-wired to zero.  An implementation must enforce that
SXL$\leq$MXL. Thus, a write to {\tt misa} changing MXL may cause
SXL to  change as a side effect.
The wording "any write to {\tt misa}" seems to imply that even following Write Preserve {WPRL} precautions writes to misa can do magic things to SXL. Perhaps, though, the Write Preserve Precautions should be made more evident or perhaps explicit, not just for Reserved space in CSR but any write.

misa doesn't have any WPRL fields, but yeah, the intent is that any write to misa may have side effects on SXL/UXL, even if misa is preserved.

This would establish a strong precedent of co-dependence between CSR.
I understand that misa is a fundamental register, and significant in that it contains MXL.
All the extension fields are also potentially invasive to core processing, however,
none of these extension transitions need to invalidate MXL/SXL/UXL.
If the MXL or SXL/UXL functionality is implemented retaining SXL/UXL when MXL is not changed in a misa write should not be an insurmountable challenge.


Command to return number of linting errors or warnings found

Feature request
Describe the solution you'd like
I am working in a codebase with thousands of lint errors. I would like to be able to make a cool to return the number of warnings or errors. This way, I can incrementally move my codebase to a cleaner, "more linted, but not perfectly linted" version by not allowing more lint errors in subsequent pull requests. The goal is to add a hook in our build process that does not allow new PRs to pass the build if they add more linting errors to the codebase.
Describe alternatives you've considered
I've searched online for this solution. I haven't found one yet. If you have a solution or workaround, i would be very interested in hearing it.
Additional context
Trying to move to a cleaner code base incrementally and this would help me get there.

Related: #2267

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


LiveView Bindings for SwiftUI -> Focus Events: phx-window-focus into ScenePhase

LiveView doc: https://hexdocs.pm/phoenix_live_view/bindings.html#focus-and-blur-events
SwiftUI doc: https://developer.apple.com/documentation/swiftui/scenephase

Done with PR #166


adjust pdf thumbnail

archivematica pdf thumbnail should generate image thumbnail of pdf

do not do. preference is to do on the fly thumbnail generation, we don't want to store thumbnails


Problematic hardcoded relative path to modulescmd executable

While running the regression tests locally on my laptop, one test fails because payu assumes that the modulescmd executable is located in $MODULESHOME/bin, which is not the case for all the linux distros (e.g., Debian and Ubuntu). Obviously not a problem when running on Gadi, but could be annoying for someone else trying to use payu in a different machine.
PS: The CI does not see the problem because no environment modules are installed in the Ubuntu image being used.

Thanks @micaeljtoliveira
There are a couple of other issues related to porting to other machines so I've made a specific label for that as it is highly desirable.
Having said which, running payu on a laptop is nothing something I ever imagined, so good to have my mind expanded.


fix(client): integrate queryRaw improvements

Closes https://github.com/prisma/prisma/issues/12802
Closes https://github.com/prisma/prisma/issues/12800
Fixes https://github.com/prisma/prisma/issues/13085

Tests will be failing for now, waiting for the engines version 1c2129f3eaa9bf16c6cd7c14b69b7eb9cc88bbed to be published.


#trivial Clean up cross domain communicator

Summary

How did you test your changes?

fyi @fan-zhang-sv i am closing this.
let's revisit with our new strategy of not allowing batching.

Review Error for fan-zhang-sv @ 2024-05-01 20:17:55 UTC
User failed mfa authentication, public email is not set on your github profile. see go/mfa-help


cant appear /rankshop

pls fixx huhu

The command is working. Also, you didn't make a right bug-report.

can you pls make atleast 2mins or 1min tutorial? plssss


Fix error with library version

Description:
This component is currently broken. Somehow an error was introduced, but using a previous version of this library fixes this component.
Related issue (if applicable): fixes #
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#<home-assistant.io PR number goes here>
Example entry for configuration.yaml (if applicable):

Checklist:

[ ] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Make sure you open PRs from new branches. This PR also includes an unrelated PR.
Can you add a comment that this version should not be upgraded without a fix for the issue. Include a link to the issue too (stacktrace)


How to train network for custom dataset?

I've used your pre-trained weights for face recognition and it seems to work well. Thank you.
Now, I am interested to use a similar network to verify if two images match or not (no faces). Do you have any idea about how could I train my network?
In your code for face recognition, an image is used as input for network and a vector is generated as output. Both images (faces) are fed to network such that -both- generated vectors are then compared with cosine or euclidean similarity.
But in my case I just have two images that match or not.

You should study Siamese Network. This face recognition model is trained by its procedures.

This is not an issue. This is closed.


Use and acknowledge lock status

Consistently use and acknowledge the 'lock' status of an element.
a) new commands:
element lock
element unlock

b) Visual aknowledgement by

removing all corresponding modification actions from the context menus

vs

displaying an appropriate icon in the element-tree

removing all modification elements from the scene-item-manipulation

vs

All now in 0.8b6


Google Meet Joiner

Google meet joiner
main.py -
**python script built to join google meet classes automatically **
Issue no.(must) - #278
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in proper folder (Name of folder should be in lowercase with no space in between)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[x] I have added README.md and requirements.txt with my script

Thank you

Hello,
what was the error here?

Hey,
try installing flake8 and then run it on your py file, it will show you what is wrong.
py -m pip install flake8
py -m flake8 main.py

Alternatively, you can go to https://github.com/python-geeks/Automation-scripts/pull/302/checks?check_run_id=1274832343

Hey,
try installing flake8 and then run it on your py file, it will show you what is wrong.
py -m pip install flake8
py -m flake8 main.py

Alternatively, you can go to https://github.com/python-geeks/Automation-scripts/pull/302/checks?check_run_id=1274832343

I mean i checked the page u linked. But it just say failed after x number of seconds. It is not showing what the issue is.

Hey,
try installing flake8 and then run it on your py file, it will show you what is wrong.
py -m pip install flake8
py -m flake8 main.py

Alternatively, you can go to https://github.com/python-geeks/Automation-scripts/pull/302/checks?check_run_id=1274832343

Also i installed flake8 and ran it on my file.
No error showed up, all that came up was this indentation contains tabs

Did you see this?

for example, on line 9 it must be a blank line in order to have 2 blank lines between whatever you wrote and your function
With this code, I have no linting error:

import time
import webbrowser
import schedule
import pyautogui

url1 = "https://meet.google.com/"
url2 = "https://www.google.com/"

def class1():
    webbrowser.open(url1)
    time.sleep(3)
    print("Joining Class")
    time.sleep(2)
    pyautogui.hotkey('command', 'd')
    print("turned off camera")
    time.sleep(2)
    pyautogui.hotkey('command', 'e')
    print("turned off mic")
    time.sleep(3)

    for i in range(4):
        pyautogui.press('tab')

    time.sleep(3)
    pyautogui.press('enter')

def class2():
    webbrowser.open(url2)
    time.sleep(3)
    print("Joining Class")
    time.sleep(2)
    pyautogui.hotkey('command', 'd')
    print("turned off camera")
    time.sleep(2)
    pyautogui.hotkey('command', 'e')
    print("turned off mic")
    time.sleep(3)

    for i in range(4):
        pyautogui.press('tab')

    time.sleep(3)
    pyautogui.press('enter')

if __name__ == "__main__":
    schedule.every().day.at("16:37").do(class1)
    schedule.every().day.at("14:00").do(class2)

    while True:
        schedule.run_pending()  # check if we need to run anything
        time.sleep(10)  # wait 10 seconds before checking each time again

See: https://www.flake8rules.com/rules/E101.html and https://www.flake8rules.com/rules/W191.html
Out of curiosity, which IDE do you use?

Flake8 is a tool that will check your code syntax and provide instructions on how to clean it. So yeah even you don't pass Flake8, your code can run well
It allows us to have a certain consistency on all these scripts, that's why we have a check 😉

Did you see this?

for example, on line 9 it must be a blank line in order to have 2 blank lines between whatever you wrote and your function
With this code, I have no linting error:
import time
import webbrowser
import schedule
import pyautogui

url1 = "https://meet.google.com/"
url2 = "https://www.google.com/"

def class1():
    webbrowser.open(url1)
    time.sleep(3)
    print("Joining Class")
    time.sleep(2)
    pyautogui.hotkey('command', 'd')
    print("turned off camera")
    time.sleep(2)
    pyautogui.hotkey('command', 'e')
    print("turned off mic")
    time.sleep(3)

    for i in range(4):
        pyautogui.press('tab')

    time.sleep(3)
    pyautogui.press('enter')

def class2():
    webbrowser.open(url2)
    time.sleep(3)
    print("Joining Class")
    time.sleep(2)
    pyautogui.hotkey('command', 'd')
    print("turned off camera")
    time.sleep(2)
    pyautogui.hotkey('command', 'e')
    print("turned off mic")
    time.sleep(3)

    for i in range(4):
        pyautogui.press('tab')

    time.sleep(3)
    pyautogui.press('enter')

if __name__ == "__main__":
    schedule.every().day.at("16:37").do(class1)
    schedule.every().day.at("14:00").do(class2)

    while True:
        schedule.run_pending()  # check if we need to run anything
        time.sleep(10)  # wait 10 seconds before checking each time again

See: https://www.flake8rules.com/rules/E101.html and https://www.flake8rules.com/rules/W191.html
Out of curiosity, which IDE do you use?

I am using sublime text at the moment.
So if i fix the spaces and indentation, where it says its a mix of spaces and tabs. So if make another PR will it clear?

Yes but you don't need to open another PR, you just have to commit your changes on the same branch you did the PR (  googlemeet-joiner)
It will automatically update the PR

Yes but you don't need to open another PR, you just have to commit your changes on the same branch you did the PR (  googlemeet-joiner)
It will automatically update the PR

Yes but you don't need to open another PR, you just have to commit your changes on the same branch you did the PR (  googlemeet-joiner)
It will automatically update the PR

Alright got it. Thanks for your help.

Okay I found why you got these errors.
I use vscode, and the indent is automatically done with spaces rather than tab.

It seems that Sublime text default is using tab

Okay I found why you got these errors.
I use vscode, and the indent is automatically done with spaces rather than tab.

It seems that Sublime text default is to use tab

Oh, now i get it. Thanks a lot for your help. I appreciate it.

My tab width size is 4 in sublime as well. I tried writing the program in vsc but got the same error. I don't know where the lint error comes in. @Sacrezar

My tab width size is 4 in sublime as well. I tried writing the program in vsc but got the same error. I don't know where the lint error comes in. @Sacrezar

I just checked, you corrected these lint error. Good work 😉
Your errors are different now:

for your second function you wrote class1 instead of class2
and you have to add somes spaces when you use inline comment.
Something like that should work:
if __name__ == "__main__":
    schedule.every().day.at("16:00").do(class1)
    schedule.every().day.at("06:00").do(class2)

    while True:
        schedule.run_pending()  # check if we need to run anything
        time.sleep(10)  # wait 10 second before checking again

Hey @Sacrezar , I ran the flake8 on the file again and there are not lint errors anymore.
I will make sure to add argvs and remove class1 and class2.

[ ]  Correct Lint errors

[ ]  Delete one of the function since they are the same and use the url as an arg

def attend_my_class(url):
    webbrowser.open(url)
    ...
    ...

if __name__ == "__main__":
    schedule.every().day.at("16:00").do(attend_my_class(url1))
    schedule.every().day.at("06:00").do(attend_my_class(url2))
    ...

If you want, you can even allow the user to put the link when launching the script. See sys.argv here and there. That's not an obligation 😃

Hi again. I added the changes to my script. Where do i check the boxes for the changes that were assigned to me?

Nice work! 👍

Thank you.
Thanks for all your help too.

@all-contributors add @m4dummies for code


VK API issues

Hello,
We currently have 3 PRs for the same issue.
#227 @alexus1024
#228 @imaleksandr
#229 @diseks
Can we use this issue to discuss the problem, and decide which PR we should go with moving forward. Or propose a new PR with the union of all 3 approaches.

I propose we go with #228

I vote for 228
I don`t think this version setting in options from #229 is necessary: older version is not supported by VK anymore - it is the source of entire issue.
And my quick fix #227 is just too quick )

Thanks guys. Cool that my code suits you.

Great, I'll merge and deploy this tonight (barring any personal issues)


kill process

I have a luaproc process that hangs on a receive() like this:
local s, err = socket.connect(ipAddress, 8989)
s:receive(2)

Is there a correct way to stop/kill this process from outside?
Cheers
/Joakim

No, there is no "kill" equivalent in luaproc. You cannot finish Lua processes from the outside.


Map a couple of random methods

I realise this is incredibly disjoint - sorry. This is just me going through methods that I use and trying to clean them up a little.

Updating, any further comments should go in an issue or PR


Error in README file

In the readme file it says
final dio = Dio();
dio.interceptors.add(DioLoggerInterceptor)

should be
final dio = Dio();
dio.interceptors.add(dioLoggerInterceptor)

Thx for the fix! 🙏


Update opensearch docker-compose: set max field number to 2000.

Reject: the index level config can't be set in config file.


2.0.0

Add analysis options
Improved type checking with MiddlewareBuilder
Bump version to 2.0.0

Codecov Report

Merging #14 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #14   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           5      5
  Lines          47     47
=====================================
  Hits           47     47

Impacted Files
Coverage Δ

lib/src/built_reducer.dart
100% <ø> (ø)
:arrow_up:

lib/src/store.dart
100% <100%> (ø)
:arrow_up:

lib/src/middleware.dart
100% <100%> (ø)
: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 2c18790...695d0a1. Read the comment docs.


Bug fix: copyright not updating on home page

@vsvipul, now copyright is also being updated on home page. Have a look.

@LalitNM working as expected 💯


add the crs version file to the repository

The CRS is a Google Spreadsheet that users can copy to create their own simulations. The spreadsheet includes a formula that fetches a file from an external source, parses its content, and, if the copy is outdated, shows a message to the user telling them to update to the new version.
Currently, the file with the version is hosted in https://raw.githubusercontent.com/Charles-CovidActNow/Covid-Response-Simulator/master/Covid Response Simulator
We will soon update the CRS to use a new version (see https://trello.com/c/o9TKhGdY/458-update-crs-link-version-125-https-docsgooglecom-spreadsheets-u-4-d-1vs-ojzbue96sky-8cnjwlpwnxkjwsdkw5xbejyr5-uk-copyuspsharing).
This PR adds the file to our assets, so we can provide Gil with the URL of the version file. Once this PR is deployed, we will update the CRS URL and the version number.

Makes sense 😅  - updated. I might update the link URL as well once the spreadsheet is updated

 Update CRS link: version 1.25 https://docs.google.com/spreadsheets/u/4/d/1vs-OjzBuE96sky-8CNJWLPWnXkJwsDKw5XBejYR5-uk/copy?usp=sharing


Add Subheader option

I think it would be a good idea to make it possible to add a header for the FrontLayer while it is not visible
According to the Material Design guidelines it should also be part of a backdrop
It should look like this (at the bottom):

It was originally planned. But I don't know what is the best way to implement this. Do you have any idea?
Also @WieFel what do you think about it?

Should it be an attribute (say Widget subHeader) in BackdropScaffold ? or Should we make another widget (say BackdropSubHeaderFrontLayer) which can be dropped in as frontLayer (As we did this with BackdropNavigationBackLayer).
@felixwortmann @WieFel Let me know your views.

I think just a widget (SubHeader) would be good. I did not read the code where this has to be added so I can not say which would be easier to implement.

I think subHeader attribute is better given that it is of type Widget so that it is customisable enough but also default TextStyle and Padding

Also with stickyFrontLayer we need to make sure that the subHeader is visible when backLayer is active

It was originally planned. But I don't know what is the best way to implement this. Do you have any idea?
Also @WieFel what do you think about it?

I did not read all the code so I also do not know the best way. This might help though: https://medium.com/@CORDEA/implement-backdrop-with-flutter-73b4c61b1357

I think it is already possible to implement it with the package as it is right now. Every front layer would have to consist of a Column that contains the subheader and the front layer content.
Another way would be to add a BackdropSubHeaderFrontLayer, as @daadu already mentioned. I think it could look like this:
BackdropSubHeaderFrontLayer(
    header: ...,
    body: ...,
)

Where header and body would be of type widget.

I just wrote a litttle POC for myself. It only shows the SubHeader when the FrontPanel is closed. You can take a look here

I think it is already possible to implement it with the package as it is right now. Front layers would have to consist of a Column that contains the subheader and the front layer content.
Another way would be to add a BackdropSubHeaderFrontLayer, as @daadu already mentioned. I think it could look like this:
BackdropSubHeaderFrontLayer(
    header: ...,
    body: ...,
)

Where header and body would be of type widget.

Currently there is a filter which makes it

@WieFel I would suggest going with subHeader attribute to BackdropScaffold instead of BackdropSubHeaderFrontLayer widget - primarily because of handling stickyFrontLayer when subHeader is presents. MDG mandates that the subHeader should be visible. Also because sub-header is MDG is defined as main component (although optional) of the Backdrop Anatomy.
I would suggest:

add final Widget subHeader; as attribute to BackdropScaffold
modify _buildFrontPanel method to accommodate it.
wrap widget.subHeader with DefaultTextStyle as mentioned in MDG for subHeader. So for user simply setting Text("Sub Header") should do the work and off course he can then modify it if it wants. This how it works in say AppBar where setting title with simple Text widget does the magic.
handle stickyFrontLayer make sure the height of the subHeader widget is atleast visible - for eg. If BackLayer has full screen content then the frontLayer should drop completely (which would happen in current version) but with subHeader visible.

Moreover, we could provide a helper widget (Like ListTile in ListView) say BackdropSubHeader which can be simply dropped in as subHeader if user wants standard implementation with additional options like  (apart from title)

bool showDivider
Color dividerColor
double dividerHeight
bool automaticallyImplyLeading (used to show Icon to show upward arrow when backLayer is visible as leading)
bool automaticallyImplyTrailing (used to show Icon to show upward arrow when backLayer is visible as trailing)
Widget leading
Widget trailing
BackdropSubHeader would be recommended way to add subHeader to Backdrop.

Let me know your thoughts @WieFel @felixwortmann

I will be working on this weekend.

Sounds good

@daadu Yes, sounds good! :)

@WieFel @felixwortmann I have written this feature. Please check and give me your feedback.
BackdropSubHeader is undocumented. I need your help for this!


Action doesn't work in forks

Got a somewhat unintuitive error with the action:

github.GithubException.GithubException: 403 {"message": "Resource not accessible by integration", "documentation_url": "https://docs.github.com/rest"}

What does it mean? See https://github.com/Mudlet/Mudlet/pull/4885/checks?check_run_id=1998372846#step:11:1297

Is it because the PR is from a fork? It looks like it's failing when creating the review. One option is to disable the check on forks, using something like: https://github.community/t/dont-run-actions-on-pull-request-from-fork/17560/4
There may be a way to give frequent collaborators access to create reviews -- I'm not sure how to do that myself

Hm, but that's not how Github works: anyone should be able to make a PR from their fork without needing any kind of access. Requiring access is non-scalable and wouldn't make these kinds of actions really work - does Github really not permit this?

It's not the PR itself, its the token used to create the review which only has read permissions when used from a forked repo: https://docs.github.com/en/actions/reference/authentication-in-a-workflow#permissions-for-the-github_token
So it can do everything but write to the repo -- review comments count as a "write", which is why it's failing when trying to create the review.
I think the issue is basically in order to post the review comments, the user would need (some level of) write permissions, but if the PR is from a fork, they could've modified the action to do other things.
It looks like there's a similar limitation on creating Checks Annotations, so even if I switched it to them, there would still be the same problem.
I could make it an option to do everything but post the review comments, that way it'd still be able to run on forks and you'd have the option of failing if there were clang-tidy warnings, you'd just have to go into the check to see them.

What if a custom token is generated and passed to the action which has the review write capability, would that work?

Solved it in the end with the pull_request_target hook :+1:


license file

Hi,
thanks for your work providing such a useful theme.
In the footer of each page, there is a broken link to "license".   the README.md mentions a page/license.md file - but that renders as ROOT/page/license/
Is there just a mismatch in the documentation?
Stephen

Home plate only shows the list of articles blog plate,
I want the home plate to show all the plates.
How should I do?
thank you.

Sorry, I'm not sure.  But its probably best to ask this as a separate issue.
I have a file content/home/index.md  to make the home page as mentioned on
https://discuss.gohugo.io/t/homepage-content-from-content-home-md/54/12

& fyi here is my page http://sje30.github.io


[Proposal] Rename DictionaryLiteral

This proposal renames the confusing and misnamed DictionaryLiteral type. This type is neither a dictionary nor a literal.
There is no strong motivation to deprecate. The type does not produce active harm. Instead, it adds measurable (if small) utility and will be part of the ABI. A sensible renaming creates a worthy compromise.
This proposal was discussed in the Swift Forums on the 100% bikeshed topic: DictionaryLiteral thread.

cc @lattner

Added draft implementation assuming we do not deprecate.
https://github.com/apple/swift/pull/16577

Merging and kicking off this review, thanks!!


Switch teams from hydra to phoenix

What this PR does / why we need it
Use new team.
Checklist

[X] Update changelog in CHANGELOG.md.

Trigger e2e tests

/test upgrade
/run cluster-test-suites

/test upgrade

/test upgrade

/run cluster-test-suites


Propagate errors from subprocesses to main process

In mltsp.Flask.flask_app, errors that occur in calls run in a subprocess do not propagate up to the main app

I think this is the same as #107 (although my description there was a bit vague)...would be great though!


[MINOR] Improve the logic when building bloom filter KeyRangeLookupTree

Change Logs
Refer to the implementation of Math.max() and Math.min().
This way you can unify the style of the if branch statement above.
Make source code more readable.
Impact
NONE
Risk level (write none, low medium or high below)
NONE
Documentation Update
NONE
Contributor's checklist

[x] Read through contributor's guide
[x] Change Logs and Impact were stated clearly
[x] Adequate tests were added if applicable
[x] CI passed

Azure CI is green.


Added tests for plugins

Fixes and help for PR #49 that @mspeedy kindly contributed.

Add Celery to requirements-dev.txt
Sort import statements
Fix counting stock plugin modules in test_autodiscover
Fixes to Python 2 and 3 compatibility
Fixes to inherited class initialization semantics (add calls to base classes in __init__ methods). For some reason QuantifiedCode is still not happy with the Python 2 compatible initialization syntax and suggests just calling super().__init__, which we can't do while supporting Python 2 (unless done with future super which would add a dependency).

@mspeedy @codingjoe this PR should fix testing problems with #49 and be a smooth merger. I did not alter any of @mspeedys work.

Looks good, thank you for the help!

@mspeedy @aleksihakli cool, but who's implementing review changes now?
Anyways, this is pretty cool, but I don't like that it changes existing tests and transforms everything to UnitTest where we used py.test. I personally perfer pytest, but I appreciate everyone how takes the time to write a test. Pytest allows executing both UnitTests and pytest. Therefore it's fine if the new tests are written UnitTest style, but please don't change the old once.

I thought to help since @mspeedy said he doesn't have time to do all the updates.
I'll roll back the test migrations from object inherited tests to TestCases and rebase this on top of the current master.

On closer inspection the commit chain is broken and inclues and then deletes 600kloc of libraries. I'll open another pull request with those additions and removals squashed and tests done in the original style.


docs: transocder route config

Signed-off-by: Lizan Zhou<EMAIL_ADDRESS>#3456
Description:

What does this PR do? What was the behavior before the PR?
What is the behavior with the PR? If fixing a bug, please describe what
the original issue is and how the change resolves it. How does this
feature get enabled? By default, config change, etc...

Risk Level: Low (docs only)
Testing: N/A
Docs Changes:
Release Notes:

Done, thanks!


Create ACED-IDP Microsoft/Azure account.

@kellrott When you get a chance, let's create an ACED-IDP Azure account.


Handle white-list processing

While reading this book ... http://shop.oreilly.com/product/0636920033011.do
I was reading about "White Lists" ... and was then looking in the ESP32 docs for related APIs and found

http://esp-idf.readthedocs.io/en/latest/api-reference/bluetooth/esp_gap_ble.html#_CPPv228esp_ble_gap_update_whitelistb13esp_bd_addr_t
http://esp-idf.readthedocs.io/en/latest/api-reference/bluetooth/esp_gap_ble.html#_CPPv230esp_ble_gap_get_whitelist_sizeP8uint16_t

Since we have APIs at the ESP-IDF that are exposed ... let us now investigate how we might expose and use these through the C++ classes.

Another step in ble security

New code/functions added:

BLEAdvertising::setScanFilter()
BLEDevice::whiteListAdd()
BLEDevice::whiteListRemove()

No breaking changes and no changes in existing apps introduced by these functions.


Add exit application method to system channel

Add System.exitApplication and System.requestAppExit method.
System.exitApplication requests two types (required, cancelable).
https://github.com/flutter-tizen/embedder/issues/47

System.initializationComplete method should be added.

Rebased patch


Chat UI

Description
I'd like to design the chat ui in figma which can be used in dummygram
Screenshots
No response
Additional information
No response

i wish to work on this issue

Okay, you may place the Ui in assets folder under figma design or whatever you think is good :) and create a pr.
Also, for now which chat approach do you intend to follow, would something like linkedin's chat ui work ?

it would be better if we use a goofy ui ( like that of insta  ) since it's a social media app (?)

it would be better if we use a goofy ui ( like that of insta ) since it's a social media app (?)

okay makes sense.

I'd like to work upon this issue if this hasn't been done yet.I have a good knowledge of UI as well

@bhoomikhurana i am working on this issue

@bhoomikhurana i am working on this issue

I think no one is :/ ,closing temporarily , will reopen if needed


Removed unnecessary use of Long.toString from BlobStoreRepository.

Signed-off-by: Mani<EMAIL_ADDRESS>Description
Removed unnecessary use of Long.toString from BlobStoreRepository.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #5833 (8a528a2) into main (fb78f11) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #5833      +/-   ##
============================================
+ Coverage     70.81%   70.90%   +0.08%
- Complexity    58686    58760      +74
============================================
  Files          4768     4768
  Lines        280575   280575
  Branches      40514    40514
============================================
+ Hits         198685   198935     +250
+ Misses        65636    65349     -287
- Partials      16254    16291      +37

Impacted Files
Coverage Δ

...ch/repositories/blobstore/BlobStoreRepository.java
64.47% <100.00%> (+0.56%)
:arrow_up:

.../java/org/opensearch/node/NodeClosedException.java
50.00% <0.00%> (-50.00%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
42.50% <0.00%> (-48.75%)
:arrow_down:

.../java/org/opensearch/search/dfs/AggregatedDfs.java
54.83% <0.00%> (-41.94%)
:arrow_down:

...ava/org/opensearch/search/dfs/DfsSearchResult.java
47.87% <0.00%> (-39.37%)
:arrow_down:

...h/action/ingest/SimulateDocumentVerboseResult.java
60.71% <0.00%> (-39.29%)
:arrow_down:

...pensearch/action/ingest/DeletePipelineRequest.java
31.25% <0.00%> (-37.50%)
:arrow_down:

...ensearch/client/indices/DetailAnalyzeResponse.java
20.54% <0.00%> (-24.66%)
:arrow_down:

...opensearch/persistent/AllocatedPersistentTask.java
44.23% <0.00%> (-21.16%)
:arrow_down:

...tions/bucket/terms/ParsedSignificantLongTerms.java
73.68% <0.00%> (-21.06%)
:arrow_down:

... and 487 more

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


weave-net pods restarting on kubernetes

This has been happening for a long time and we have no idea what is going on. weave-net pods are restarting with the following error: Warning SandboxChanged Pod sandbox has changed, containers will be killed and recreated.
Below is how the weave-net restarts look in kubectl:
weave-net-44z47                                                       2/2       Running             3          4d
weave-net-4gpkz                                                       1/2       CrashLoopBackOff    940        4d
weave-net-5rjbs                                                       2/2       Running             0          4d
weave-net-5wbsb                                                       2/2       Running             0          4d
weave-net-84rbx                                                       0/2       ContainerCreating   0          53s
weave-net-bvxgz                                                       2/2       Running             1          4d
weave-net-cgdkc                                                       2/2       Running             10         4d
weave-net-dbkfm                                                       2/2       Running             1          4d
weave-net-h8rkv                                                       2/2       Running             0          4d
weave-net-lwnfv                                                       2/2       Running             0          4d
weave-net-lxxmp                                                       2/2       Running             0          4d
weave-net-mbsmp                                                       0/2       CrashLoopBackOff    983        4d

What you expected to happen?
No restarts on weave-net pods. We are seeing over 500 restarts on some weave-net pods.
What happened?
Warning SandboxChanged Pod sandbox has changed, containers will be killed and recreated.
How to reproduce it?
No idea.
Anything else we need to know?
We use kops version 1.8.1 to configure our prod and staging clusters on AWS.
Versions:
weave-net: 2.2.1
docker version: 17.09
$ weave version
$ docker version
$ uname -a
4.9.0-6-amd64
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.3", GitCommit:"d2835416544f298c919e2ead3be3d0864b52323b", GitTreeState:"clean", BuildDate:"2018-02-07T12:22:21Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.10", GitCommit:"044cd262c40234014f01b40ed7b9d09adbafe9b1", GitTreeState:"clean", BuildDate:"2018-03-19T17:44:09Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Logs:
$ docker logs weave

or, if using Kubernetes:
$ kubectl logs -n kube-system <weave-net-pod> weave

Network:

$ ip route
$ ip -4 -o addr
$ sudo iptables-save

Can you get the kubelet log too, for one of the hosts where it is restarting?

The file is huge but here is a sample of the weave errors. Lots of socket errors and "CNI failed to retrieve connection namespace path."
kubelet.log
Would you like me to upload the whole journactl output somewhere?

The errors you show are caused by the Weave Net container being restarted, in other words that is a symptom not the cause.  We need to know more about why kubelet has restarted the Weave Net container.
Generally it's a waste of time to try to cut the file down to just what is needed, because neither of us knows which bits are relevant.

Doing some searching, it seems this can be caused by the docker daemon restarting.  Please could you have a look at its logs too?
If there's nothing in the docker logs, check that the docker-healthcheck script installed by kops is not killing it.  I've seen that before.

I'm seeing this same issue on kubernetes 1.9.3, weave 2.2.0, kops 1.9.0-beta-2.  When it happens again I'll try and capture logging information.  It has happened twice in 2 weeks.

To be clear, "sandbox has changed" is Kubelet reacting to what it sees as a change, and "sandbox" is how it refers to the underlying container. For example, Docker restarting, causing all containers to restart, would be one way to cause this.  If you have an issue on your node(s) that causes Docker to restart, then everything will be impacted, including Weave Net.

So we knew about the issue that dockerd was restarting but not sure why that was happening. We made slight changes last night. One of the changes was clearly define memory and cpu limits for most pods in our k8s cluster. The other change was to provision more IOPS to the nodes and masters. We have since seen significantly less restarts, almost none.

Good to hear - do you think it was the healthcheck job that was killing it?

@bboreham Yes, exactly! The docker-healthcheck script was restarting docker since this line if timeout 10 docker ps > /dev/null must have been taking longer than the allowed timeout because of low IOPS on the nodes...

Hope this helps other people out there. I will open an issue with kops. Maybe there should be another healthcheck service running benchmarking iops on the server or the docker-healthcheck script can be refactored as it turns into a single point of failure when the servers are under high load and with low IOPS?

@sstarcher : I discussed with my team again to check if there was anything we changed that was not related to IOPS. The only other thing we noticed is that it could be resource limits based. We noticed when we did not have strict resource limits on some services they would hog cpu and memory from the nodes. In our case it was a logging service that was not buffering the logs correctly and therefore requesting lots of cpu and memory to push them out.
Is it possible you had some service running on your cluster that was using lots of cpu or memory at intermittent times? That was the only other problem we noticed.

It looks like by default kops is setting weave up as
      limits:
        memory: 200Mi
      requests:
        cpu: 50m
        memory: 200Mi```

So it should be unlikely to get evicted.

@sstarcher : it's not that weave-net is getting evicted. it's that other pods that do not have memory/cpu limits and requests could be crashing docker service on the nodes. weave-net restarting is the symptom of that problem.

The docker service itself should also have a higher adjust score so it should be pretty unlikely.  Again all guessing until I can reproduce it and debug the issue fully.

Great, @Cryptophobia I'm going to close this issue since you found the source of the restarts.
@sstarcher when you get some details please open a new issue so we can keep the conversation threads clear.


App TMurgent/TMEdit being added

This app is being worked on.  It has been integrated into the Tim branch.

Fixes as part of PR #18


Empty Array after using getSubscriptions or getProducts (IOS)

I'm currently having a problem when using this library related to the getProducts() and getSubscriptions() functions.
They are not loading not products nor subscriptions. All in-apps has ready to submit status. It's more than 24h since in-apps configured and got ready to submit status.
In-App purchases capability enabled
Version of react-native-iap
I'm using version 6.0.6
upd. latest version (7.5.0) showing same behaviour
Version of react-native
I'm using version 0.66.1
Platforms you faced the error (IOS or Android or both?)
Faced this problem on IOS. Android works well.
Expected behavior
initConnection(): Showing true as a result.
getProducts(): Getting the list of all active products registered on App Store Connect
getSubscriptions(): Getting the list of all active subscriptions registered on App Store Connect
getProducts([id]) : Getting the requested product
getSubscriptions([id]): Getting the requested subscription
Actual behavior
initConnection(): Showing true as a result.
getProducts(): Getting an empty array
getSubscriptions(): Getting an empty array
getProducts([id]): Getting an empty array
getSubscriptions([id]): Getting an empty array
Tested environment (Emulator? Real Device?)
I've tested both: Emulator and real device (including TestFlight app version)
Steps to reproduce the behavior
for 6.0.6:

clearTransactionIOS()
initConnection()
flushFailedPurchasesCachedAsPendingAndroid() // used for android
getSubscriptions([id]) or getProducts([id]) // with or without id array

for 7.5.0:

initConnection()
clearTransactionIOS() // also tested withoud this call
getSubscriptions([id]) or getProducts([id])

I am having the same problem

the problem was due to not submitted tax form

@jblancoh Worked for me. (15.1) I was on 15.0.2 previously
What will happened to my users using iOS 15? Would they be able to make a purchase?
My app was already live.

Same as @Desintegrator, it stopped working only on iOS without any code changes. That was the reason, after accepting the paid apps form it worked normally again.

I am also not getting products , @Desintegrator where I can submit tax form can you please help me
I am facing a issue of not getting products for IOS after Testflight upload , but working when I debug (run) locally on physical device. I want to validate receipt
I tried below methods

const itemData= await getIosModule().getItems(constants.productSkus)
alert("itemData ==>" + JSON.stringify(itemData))

const product= await getProducts({ skus: constants.productSkus });   // by hook useIAP()
alert("product=> "+JSON.stringify(product))

const product= await IAP.getProducts({ skus: constants.productSkus });
alert("product=> "+JSON.stringify(product))

Please help me to resolve this :)
I hope you will reply soon

I am also not getting products , @Desintegrator where I can submit tax form can you please help me
I am facing a issue of not getting products for IOS after Testflight upload , but working when I debug (run) locally on physical device. I want to validate receipt
I tried below methods

const itemData= await getIosModule().getItems(constants.productSkus)
alert("itemData ==>" + JSON.stringify(itemData))
const product= await getProducts({ skus: constants.productSkus });   // by hook useIAP()
alert("product=> "+JSON.stringify(product))
const product= await IAP.getProducts({ skus: constants.productSkus });
alert("product=> "+JSON.stringify(product))

Please help me to resolve this :) I hope you will reply soon

I have the same issue


allow config to update based on specified path

Fixes #1223 .
allows a config file path to be specified either in initialisation of the Config object or when running update. This path is then used for future updates.
How's this @jenshnielsen? I haven't tested it yet.

Judging from the tests, this does not quite work yet.

@WilliamHPNielsen the bug is a typo, I think you can get the main points from my comment or looking at the code and I was hoping for some feedback on the concept before I put time into testing

@nataliejpg and I was hoping for actually working code before I put time into reviewing... oh, no, now we're stuck in a deadlock 😄
I think the idea of passing a kwarg path to force load from that path is A-OK.

@WilliamHPNielsen should work now

I guess I volunteered to clean this up. I am going to push one or two commits now, fixing the return and extending the tests.
Another issue is improving the documentation...

thanks @WilliamHPNielsen

Note to @QCoDeS/core: Configuring_QCoDeS.ipynb was last updated two years ago. We might want to have another look at that one.


Add extraPropA and extraPropB to details page

should show extraPropA and extraPropB if they are enabled in the table on the details page.
The table also changes widths when PV monitoring is turned on/off. Would be nice to keep the widths the same so the table doesn't change appearance to user when toggling checkbox

completed with https://github.com/ChannelFinder/pvinfo/pull/26


Custom code actions don't work with kind other than explicit

Testing #194031
ESlint introduced a custom source.fixAll.eslint to have control over the fixes for eslint. Setting this to always has no effect since the code action provider is never called with a context.only value of source.fixAll or source.fixAll.eslint. In the always case context.only always seems to be undefined

User error. Got it working.


Update Cargo dependencies as per Dependabot recommendations

Changes

Cargo.lock dependencies

Comments

Dependabot found a few issues to address.

#435  👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

@chrisaddy this is just some Dependabot stuff.


Processes the messageId from the push notification send and passes it…

… along with the push open call.

lgtm

lgtm

LGTM


Update New-AdminAuditLogSearch.md

Typo fix

Docs Build status updates of commit 47cee3e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-AdminAuditLogSearch.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@JohanFreelancer9