Chore: Update the go version required

Closes #

Required checklist

[x] Sample config files updated (both /etc folder and NewDemoConfig methods) (influxdb and plutonium)
[x] openapi swagger.yml updated (if modified API) - link openapi PR
[x] Signed CLA (if not already signed)

Description
1-3 sentences describing the PR (or link to well written issue)
Context
Why was this added? What value does it add? What are risks/best practices?
Affected areas (delete section if not relevant):
List of user-visible changes. As a user, what would I need to see in docs?
Examples:
CLI commands, subcommands, and flags
API changes
Configuration (sample config blocks)
Severity (delete section if not relevant)
i.e., ("recommend to upgrade immediately", "upgrade at your leisure", etc.)
Note for reviewers:
Check the semantic commit type:

Feat: a feature with user-visible changes
Fix: a bug fix that we might tell a user “upgrade to get this fix for your issue”
Chore: version bumps, internal doc (e.g. README) changes, code comment updates, code formatting fixes… must not be user facing (except dependency version changes)
Build: build script changes, CI config changes, build tool updates
Refactor: non-user-visible refactoring
Check the PR title: we should be able to put this as a one-liner in the release notes

Hey @barnettZQG can you update your commit so that Chore is actually chore? Sorry for the nitpick, that will get CI passing and helps us with our changelog generation. Thanks for the contribution!


Ability to tag reports with keywords

Reference items 4 and 20
Initial functionality is:
A list of keywords to added to the data initially without the help of an UI. #108 to improve that at a later stage.
Keywords have the following properties:

name
description
Is-a relationship to other keyword(s). Is-a is a directional relationship
When drafting a report, a user can tag a report with a keyword. Frequently used keywords to be suggested to user.

A tag should be rendered with # preceding it and on a rounded edge background. Tooltips should be displayed when clicked or (to support phone usage) or possibly also mouse-over events (todo: create a separate story for general tooltips)
Searching for keywords could be done by the search or via a custom search #109. Any report tagged with a keyword has a Is-a relationship with the search subject
Migration of old reports to be done with tools developped in #107. In normal operation, the tagging of already published reports should not be altered

Define who can add new tags (role-based).
Decide whether tags are e.g. hierarchical.
Perhaps tags also needs a description/explanation (e.g. in the form of tooltips).
Does adding a tag also add tags from the hierarchy, or does e.g. the search include it?

Only users in the administrators group should be able to add and remove key words.

Yes, tags should be hierarchical.  For example, two tags are displayed: "car" and "tire."  If "tire" is selected by the user, a query of "car" would return the record associated with "tire."

Tool tips?  Yes!  Great idea!  Can these tool tips be edited by the administrators?

When a child tag is chosen, the parent tag does not have to be included for that record.  A query will find the hierarchical parents.

I'm going to make separate issues for:

relations between tags
styling of tags in the UI
tooltips for the tags
suggested tags

After loading newReport page, focus is in Tags field, instead of Meeting Goal

The documentation is a bit unclear:
https://www.npmjs.com/package/react-tag-input#autofocus
but apparently, true is the default and not false. 🙁

I'll set autofocus to false so the tags field doesn't take focus, but note the Meeting Goal field doesn't take focus (it also didn't take focus before #130 was merged).

On IE11, on the staging server, the tag UI is not working. I suggest we replace it with a simple menu that allows to pick a tag.

@VassilIordanov Have the (default) tags been loaded in the database? No tags = no suggestions.

@gjvoosten - correct. It seems like they got overwritten at some point (possibly when copying production into staging db). I've pushed the tags and it is working now.

@VassilIordanov @gjvoosten I inserted 12 tags and tested the key word functionality.  It works great.  There was one inconsistency in the drop down list.  Sometimes it is no longer displayed after three seconds when left alone and sometimes it is no longer displayed after 15 seconds.  I was not able to consistently recreate the two scenarios to know what triggered both scenarios.  But the word tagging is great as is the announcement banner which is fantastic.

@VassilIordanov also created/saved a SQL script to re-insert the 12 test key tags should the staging database need to be refreshed again.

Alas, tags have not been that effective. Time to rethink


More than one price found in a single page product

Discussed in https://github.com/dgtlmoon/changedetection.io/discussions/2636

Originally posted by fetuffani September 16, 2024
Describe the bug
I've reached to a problem where if the price is reported many times in the single product page with different formats the plugin thinks it is two different prices
changedetectionio.processors.restock_diff.processor:get_itemprop_availability:71 - More than one price found {'2879.2', '2879.20'}, throwing exception, cant use this plugin.
My bet is that the price is both shown in the  tag and java script JSON as follows:
<meta itemprop="price" content="2879.20">
<script type="application/ld+json">{.......,"offers":{"price":2879.2,......</script>
But I haven't done any debbugging.
Version
v0.46.04
To Reproduce
Steps to reproduce the behavior:

Register this URL as price change detection: https://changedetection.io/share/MwXkGX_XPk0a
The price detector will show an error

https://produto.mercadolivre.com.br/MLB-4270080936-mini-pc-lenovo-thinkcentre-m920-i5-32gb-nvme-1tb-win11-_JM
Expected behavior
Get the single product price correctly independent of how it is presented on the webpage
Additional context
Change Detection is running from a docker image (lscr.io/linuxserver/changedetection.io:latest)

hmm they have some weird broken metadata
2922.80
2922.8

something like this


Question Suggestions!

Drop your question(s) here.

Shouldn't the first part of the less-preferred alternative in question 3c have [2] as the validation data?
Is tutorial 5 the same as tutorial 4b?

What topic(s) is this question from?
Tutorial 5: Regularization and Validation

Hey Russell, thanks for reaching out!
We're flexible with the less-preferred alternatives actually. As long as the folds aren't comparable, they aren't the best options in our books. So best to stick to something like this:
Trg        Val
--------------
[a]        [b]
[b]        [c]
[c]        [d]

As for the second question, I'm reorganising the folder structure locally (am yet to push)! Thanks for bringing it up 😃

Alright, thanks for clarifying!


New command: purview threatassessment list

Usage
m365 purview threatassessment list [options]
Description
Get a list of threat assessment
Options

Option
Description

-t, --type <type>
The type of threat assessment to retrieve. Supports mail, file, emailFile and url.

Examples
Get a list of threat assessment
m365 purview threatassessment list

Get a list of threat assessment by type
m365 purview threatassessment list --type mail

Default properties

Id
contentType
category

Additional Info
API request: https://learn.microsoft.com/en-us/graph/api/informationprotection-list-threatassessmentrequests?view=graph-rest-1.0&tabs=http
Needed permissions

ThreatAssessment.Read.All

Hi @nicodecleyre, could you add A section with what defaultProperties you would want to show when running in text output mode

Thx @martinlingstuyl, added some default properties

👏 Let's choose the three most important to make sure that they will fit in a regular terminal.

👏 Let's choose the three most important to make sure that they will fit in a regular terminal.

done

Minor remark - type is optional, now you indicate it as being required 😄

Let's ship it!

Would love to work on this one 😄

All yours!


baseUrl is not used by dockerImage

It seems that GH action is not passing baseUrl value to the docker image
https://github.com/andrcuns/allure-publish-action/blob/main/action.yml#L56-L66
Results:
baseUrl value has been ignored and not used during publishing report link.


Add Logo

Description
A small PR to add logo for dattri.
See https://github.com/TheaperDeng/dattri-public/tree/logo

Or this structure

The first one looks better to me.


RC2: Where did IReadableStringCollection go?

In RC1 I was using Microsoft.AspNet.Http to read a collection of query params.
In RC2 the namespace is Microsoft.AspNetCore.Http but the interface is missing and I cannot find any documentation mentioning it.

I just found out it is now Microsoft.AspNetCore.Http.IQueryCollection


simple space mobs cant be flashed

About the PR
all simple space mobs are immune to flashing now
fixes #20783
Why / Balance
flash shouldnt work on fishe and dragon
Technical details
no
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
no
Changelog
no cl no fun

space dragon is rev

real...

i think mobs like space bears and carps should be flashable so that heads without weapons can use their flash and run away

i think mobs like space bears and carps should be flashable so that heads without weapons can use their flash and run away

Why would a head fight a space bear or a carp. If they get carp bombed they won't have time to react fast enough and flash anyways.

Why would a head fight a space bear or a carp. If they get carp bombed they won't have time to react fast enough and flash anyways.

its usually not their choice to fight, and barratry has the space bear in maints that could find its way to a head, it would also help sec fight it

well it gives them a better chance to run away

you can already outrun a bear

I no flasha da fish


Renamed MassController to MassService

Better semantics

Current coverage is 31.14% (diff: 18.18%)

Merging #75 into master will not change coverage

@@             master        #75   diff @@
==========================================
  Files            33         33
  Lines          1374       1374
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            428        428
  Misses          946        946
  Partials          0          0

Powered by Codecov. Last update b387e43...15bd5eb


Optionally mask all local/intermediate spans

When doing network investigation, local (not remote) spans can interfere with intuition. For example, local spans make trace depth less useful, span count less useful. It also makes the trace bigger and harder to view. Adam from Atlassian suggested that the UI could disable local spans in the trace view, possibly with a checkbox or something. This would help them remove clutter when trying to figure out a problem.
cc @openzipkin/ui

I like it as optional, I still times I want to see local spans


Help Needed Compiling Windows vdexExtractor

Hi, I have to admit this isn't really an issue - it's a request.
Actually it's a begging request and I'm a little embarrassed to do it but I'm out of options.
Is anybody able to share a Windows compatible version of the latest build of vdexExtractor? This tool is invaluable for deodexing and I am very keen to update my version...but I only have access to the PC at work and can't install the necessary programmes to build it myself..
I've tried looking for ways around the restrictions on my computer but without any luck. To be honest, I'd rather not do anything too drastic in case I kinda like having a job.
So apologies for asking but...can anybody help?

Just google how to compile under cygwin and you will get your answer.

Thanks for your reply. I can't install cygwin, as I mentioned previously, so I guess I'm out of luck.


N14 dataset issue

Hello, have you solved this problem?
https://github.com/circuitnet/CircuitNet/issues/36
Why did I still encounter the same problem？

Please pull the repo, or update the csv here: https://github.com/circuitnet/CircuitNet/blob/main/routability_ir_drop_prediction/files/train_N14.csv.
This csv does not contain the names in your errors and should work fine.

But we encountered a file missing issue when executing a preprocessing script to generate a training set for the corresponding task, python generate_training_stet.cpy -- task IR_rop -- data_path [path_to-decompressible d_dataset] -- savepath [path_to-usave_output], Will the train_14.csv file you proposed work at this time? And when we tried to update train_14.csv using your method, we also encountered the same error.

I update the generate_training_set.py script, and it should be working now.

Thank you very much for your answer, but we tried your method again and still encountered an error. Later, when we searched for the corresponding missing npy file, we found that the features and labels in the original decompressed dataset files did not correspond one-to-one. When we tried to delete the inconsistent files, we found that generate_training_stet.cy would run successfully. Therefore, we believe that it may be caused by incomplete data downloaded from Baidu Cloud. We hope you can check if the data set in the cloud drive is correct?

And I have another question, on your website https://circuitnet.github.io/intro/intro.html In this table, writing DRC in CircuitNet-N14 doesn't mean anything. Is it because CircuitNet-N14 doesn't use DRC?

And I have another question, on your website https://circuitnet.github.io/intro/intro.html In this table, writing DRC in CircuitNet-N14 doesn't mean anything. Is it because CircuitNet-N14 doesn't use DRC?

Yes, the DRC for N14 has potential problems, and it can't be fixed in the short term


Doc for ligo taquito and toolkit

🏗 PR ➾
Fixes # (issue)
🪂 Pre-Merge Checklist

[ ] 🐬 I have commented my code, particularly in hard-to-understand areas
[ ] 🦀 Automated tests have been written and added to this PR
[ ] 🤿 Corresponding changes have been made to all documentation
[ ] 🏖 New and existing unit tests pass locally and in CI
[ ] 🔱 The test plan has been implemented and verified by an SDET
[ ] 🏄♂ Myself and others have built this branch and done manual testing on the change
[ ] ⛵ A summary describing the change has been written for release notes

🪁 Description
Please include a summary of the change and add the issue number this is related to above. Please also include relevant motivation and context for the change
(e.g. what was the existing behavior, why did it break, etc.)
🎢 Test Plan
Please describe the testing strategy for this change

🛸 Type of Change

[ ] 🐟 Minor change (non-breaking change of very limited scope)
[ ] 🦑 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)
[ ] 🐳 Major refactor (breaking changes and significant impact to the codebase)

I made a few comments for the PR @jchenche, there is also a sidebars.js file in the website directory which controls which things show up in the sidebar. I didn't see a change there to link to the hot reloading section which should get added in


Package manager should allow dependencies to be placed on other plugins

Currently, packages that are pluggable (such as autocomplete-plus, and AtomLinter) are dependent on their own implementations of plugin style functionality.  In order to better support packages like this I think that we should be able to support packages having dependencies.
What I would like to see is capabilities added for the following.

Subscribe to package manager to get instance of package when it is loaded
When loaded a subscribed package should expose instance of package created so sub modules can execute against loaded package, for instance if the workspace was a separate module we would get the workspace by subscribing to the workspace load rather than it being available globally on the atom object.
Ability to subscribe to loaded packages by metadata filter, something like what atom linter does to find linters that have been loaded, but subscribed like we subscribe to all current and future editors.
A simple interface for verifying version (via semver) of the subscribed package(s) are of the correct version, and when not alerting the user that the modules need updating.

This would give us the ability to dynamically, and bi-directionally link modules to each other.

have you guys noticed this npm package: https://github.com/travs/atom-package-dependencies ?

For what it's worth, in DocPad land, we use npm's peerDependencies for this.

The behavior of peerDependencies will be changed in NPM@3. They will not be installed automatically any more. I don't think it meets the requirement here.

That's correct, hence why we already add the extra checks, and will be adding such installation magic to our system, which we plan to abstract out into an independent system, if useful to others.


Movement Track on Map

How can i get the track on the map when i'm moving?

Hi @Hamza123Imran. Would you please clarify your question?
Actually, I encourage you to take questions like this over to our GeoNet iOS SDK community page where we monitor and actively answer developer questions.
In the meantime and without more clarification from your question, perhaps the following answer might help get you along?
If you would like to track the visible area of the map view as a user interacts with the map (pans, zooms, rotates, etc), you could set an key value observer on the AGSMapView's visibleArea property. This property represents the extent of the map that is currently visible within the map view.
Additionally, there are a number of delegate methods for tracking touches on an AGSMapView.
Because your inquiry is a question and not an Issue, I will be closing this issue out. Please feel free to continue to reach out to the SDK team over at the GeoNet link provided above and we can help you solve the problems you have. Thanks!


Add nvui to UI section

Checklist:

[ ] The plugin is specifically built for Neovim.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[ ] It supports treesitter syntax if it's a colorscheme.

Thanks for the PR!


Importing existing Jira tickets into Dojo

Is your feature request related to a problem? Please describe
Instead of creating webhooks for newly created jira issues, I want to export every issues in my jira to dojo.
Describe the solution you'd like
Importer for jira based on csv or xml
Describe alternatives you've considered
No alternatives that I can think of.

I'd recommend to use Jira's API for that.

GET an issue via /rest/api/latest/issue/FOOBAR-3141,
create a finding in DefectDojo,
and link back to Jira

Jira API docs: https://developer.atlassian.com/server/jira/platform/rest-apis/

@nebi-frame Is using Jira's API a sufficient solution for you?


Proof-reading executable book - Concepts map image and the curriculum overview is from Neuromatch

The concepts map and curriculum overview sections are from Neuromatch

Concept map has been updated.
Curriculum overview has been updated


🌐 Implement FoundZone

Purpose
Implement the ExecuteMsg::FoundZone message according to the current specification of the okp4-dataverse smart contract.

[ ] Consider the business rules and invariants required for data integrity, security and adherence to specification
[ ] Implement
[ ] Test

Will be achieved through #332


Change the deprecated option --link by --net and --network-alias in all playbooks

We need to change the way to link the containers.

Create the network sonata in the VM
Start the containers in the network sonata
Create the network alias to the container in order to insert the record in the local network DNS.

Fixed. Now is implemented with network name attached to the default Docker 'bridge' network - eg:
networks:
- name: bridge
aliases:
links:
After SP deploy, each Container interface can be check with:

sudo docker network ls
sudo docker inspect bridge
More info here: https://docs.docker.com/engine/userguide/networking/


Improve formatting 'Investments' rows on pool detail

Screenshot is from Patience V1 on March 17 2023.

Hi! I can help with this task!

Hi! I can help with this task!

Go for it. Send me a DM on Discord to get the env settings.


Crash when breaking a cable

This error happen when we try to break a Mekanism's cable. (I'm not alone with this glitch)
Minecraft version: 1.10.2
Forge version: <IP_ADDRESS>8
MCMultiPart version: 1.3.0
Mekanism version: <IP_ADDRESS>2
Crash-Report: http://pastebin.com/rfGq1aG0

Reported to MCMultiParto to: https://github.com/amadornes/MCMultiPart/issues/58

I ran into this as well. Tried upgrading Mekanism to 294 and Forge to 2121, but still crashes.

It crashed for me , too.

One of my players has this problem as well.


Fixed issue #3885 the versions for homebrew-core and brew are out of …

…sync
This commit will sync the repositories between brew and homebrew-core.
This should enable more stable use of brew when there are major changes
pending in the next upcoming release which are not available for the latest
tagged release in brew repository.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

We spoked about this issue over #3885 and here is the related pull request. I know that you said that you wont want to have a pull request for this, as it is not an issue, it is a feature. The reasoning for this pull request is the paper trail and in the case that you find this to be something which could be added with tests and possible changes, I am more than happy to contribute my spare time for this issue. If you feel that this is non-sense feel free to close the pull request.

As mentioned: this is not functionality we are interested in including. Read the manpage and you can find out how to not brew update if desired. Sorry.

No worries. Thanks again!
For the record, the manpage which was referenced there, includes the environment flag variable HOMEBREW_NO_AUTO_UPDATE which means that the brew update can be disabled with it.
Example:
export HOMEBREW_NO_AUTO_UPDATE=1
brew install [packagename]


Implemented GND_W_RMAX param.

Please use PX4 Discuss or Slack to align on pull requests if necessary. You can then open draft pull requests to get early feedback.
Describe problem solved by the proposed pull request
My differential rover turns around too quickly so I make use of GND_W_RMAX parameter to limit its steering rate.
Test data / coverage
Tested through gazebo. It worked.
Additional context
This solution creates a limiter on pid output of yaw control. So this parameter ranges from 0 to 1. For normal rovers, this value may be set to 1.0 . For differential ones, a value of 0.5 works fine.

Can you check the test failure? Minor code style changes. Try make check_format locally.

http://ci.px4.io:8080/blue/organizations/jenkins/PX4%2FFirmware/detail/PR-12041/1/pipeline

Can you check the test failure? Minor code style changes. Try make check_format locally.

http://ci.px4.io:8080/blue/organizations/jenkins/PX4%2FFirmware/detail/PR-12041/1/pipeline

Done.

@SalimTerryLi thanks a lot, that's great. If this is ready to be merged you can remove the "draft" and make it a real pull request.
@ItsTimmy would be good if you can review and test this.

@SalimTerryLi thanks a lot, that's great. If this is ready to be merged you can remove the "draft" and make it a real pull request.
@ItsTimmy would be good if you can review and test this.

But there is another problem. There would be some changes on QGC and document side such as the description and default values of this parameter. If not, nobody can turn it correctly. I don' t know how to make such a change of QGC.
Such as this one.

I don' t know how to make such a change on QGC.

The way this works is that the param needs to be described correctly here:
https://github.com/PX4/Firmware/blob/b984411770da07f757ebaa2f72c5600386e81892/src/modules/gnd_att_control/gnd_att_control_params.c#L127-L140
It will then appear in QGC after a release.

I don' t know how to make such a change on QGC.

The way this works is that the param needs to be described correctly here:
https://github.com/PX4/Firmware/blob/b984411770da07f757ebaa2f72c5600386e81892/src/modules/gnd_att_control/gnd_att_control_params.c#L127-L140
It will then appear in QGC after a release.

Thank you. I' ve added it.


#181 make RefSolver::$maxDepth able to configure

I am having this #184 issue also. Due to low maxDepth, JsonSchema\Exception\JsonDecodingException: The maximum stack depth has been exceeded message is keep throwing.
Make it changeable so that it can be changed based on developer needs.

@edisonthk you don't need a setter method to set the value of a public, static variable like JsonSchema/RefResolver::$maxDepth
// e.g.
JsonSchema/RefResolver::$maxDepth = 20;

The setter method would only work using a concrete instance, and the static variable value is used across all instances.


Check if file is empty

While running a task watermill should check if file is empty before executing it. One simple way to do so would be to check if file size is 0.

There is code for this but it was toggled off near the end of my GSOC.
This is part of "validators", which can be ran before a task, after a task, and later we should add the ability for users to pass per filetype validators per task, stream validators (i.e. check if each object has required keys), etc

We should close this issue once that validation is re-enabled and we have tests for this

Since I couldn't replicate this I will close it.


Fix Ui issues and some documentation errors.

Codecov Report

Merging #118 (b2b4dd2) into master (2bd7c91) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #118   +/-   ##
=========================================
  Coverage     76.14%   76.14%
  Complexity      670      670
=========================================
  Files           100      100
  Lines          1924     1924
  Branches        202      202
=========================================
  Hits           1465     1465
  Misses          402      402
  Partials         57       57

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Added additional changes for the new AccelerationStructureNV type.

BTW the CI-check-format problem is in: source/opt/inline_pass.cpp. I didn't touch that file.


Users should note down their means of transport (car, bike trailer)

Helpful for co-collectors to see how much transportation capacity is at a pickup.
Possible solutions:

per-user text field: write it down once, can't change it per pickup
per-pickup text field: different transport for each pickup possible. could be possible to remember the last value for convenience.

A free text field on the user page offers unlimited possibilities to add info... So I'm definitely in favor of that!
A free text field per pick-up on the store page adds the flexibility to request a car or whatever if needed, like it as well!

Solved it with #207


Better sequence to CI workflow

Fixes: https://github.com/runtimeverification/pyk/issues/25
This PR does:

Compresses all the PR tasks into on github workflow file.
Adds dependencies between the jobs, so that heavier jobs are only going to run if the lighter faster jobs pass.
Moves unit tests onto a public github runner.

Now we get this workflow summary:

Once this is approved, I'll adjust the relevant required checks in the github settings.

Finalized run summary.


Suppress --json option for schema subcommand.

Suppress --json option for schema subcommand.  It does not yet fully work with selectors.

lgtm, thanks


Reduce number of NumberFormatter and EtherNumberFormatter defined, if possible #1954

Closes #1954

I suppose the API is so that we are meant to use it like this:
if let string = NumberFormatter.currency.usd.string(from: NSNumber(value: change)) {

i.e. with a .usd or .short
There are these lines which omits them:
let prettyAmount = NumberFormatter.currency.string(from: NSNumber(value: dollarValueInUsd)) ?? "-"

return NumberFormatter.currency.string(from: NSNumber(value: totalAmount))

let formattedFee = NumberFormatter.currency.string(from: NSNumber(value: price.price * feeInDouble))

This introduces a bug right? Since the formatter might have been changed in a previous call, we don't actually know which format is being used.
If this is unintended, how about we make NumberFormatter.currency return an instance of another (maybe inner) class which is not NumberFormatter nor a subclass of it to make the error impossible?

damn, you right. i missed it. i will fix this.

If this is unintended, how about we make NumberFormatter.currency return an instance of another (maybe inner) class which is not NumberFormatter nor a subclass of it to make the error impossible?

i am not shure if i undarstand your last lines, but check chenges and let me know.

@hboon seems like this one is ready to check.


Add support for I2C bus configuration in order to become compliant fo…

Add basic support for configuring I2C busses. This makes use of runtime exceptions, so it shouldn't create any new requirements for users of the Foglight APIs. This is probably not an ideal long-term solution.

As far as I'm aware, the IllegalStateException is a runtime (unchecked) exception; the throws are there for documentation purposes only. Would you like the throws on the signatures removed regardless?

Throws declarations now removed from the I2CBacking; the Javadocs remain, as does the functionality.


Remove vlan support for SR-IOV

In current state it tries to read vlan id from master device while code
is executed in container namespace, but master device is available only
in host namespace.
New approach will be prepared in separate PR.

This change is

Reviewed 3 of 3 files at r1.
Review status: 0 of 2 LGTMs obtained, and 1 stale

Comments from Reviewable


LOG-4254: Upgrade eslint to remove word-wrap vulnerable dependency

/cherry-pick release-5.7

/lgtm


proxy: redo libevent handling code

The event handling code was unoptimized and temporary; it was slated for a rewrite for performance and non-critical bugs alone. However the old code may be causing critical bugs so it's being rewritten now.
Fixes:

backend disconnects are detected immediately instead of on the next time they are used.
backend reconnects happen after the retry timeout, not before
use a persistent read handler and a temporary write handler to avoid constantly calling epoll_ctl syscalls for potential performance boost.
avoids overwriting the read timeout when many requests are being queued for the same non responsive backend

Commit in progress, still hunting some other related timing bugs that seem to still happen.

RE tests: the new test suite covers this very well. During development proxyunits.t was failing on some things I had goofed up. I'm not sure what else I would add at the moment.
proxyconfig.t doesn't pass and will need to change a bit, as the proxy is detecting disconnect events fast and is emitting different log lines.

Tests should all pass now. I might add one more for proxyconfig.t to test immediate disconnect detection but I think it's optional since I've had to work around it in the tests.
Going to let this sit for a day, review it tomorrow, and probably merge it immediately tomorrow. The old code wasn't really doing anything properly and this is a lot more clear. Also nothing moved around and no new event handlers were added, just the same code split apart.

Doing an overload test with mcshredder using a basic test:

plain gets
hits primary zone
overload traffic (three mcshredder threads attempting to run 3 100k rps threads for 300k rps total)
localhost unimpeded backend nodes
10 connections per backend node (soft emulating ten total nodes)
using the memcached-debug binary, so a prod binary might get more performance.

pre-patch:

230k reads/sec average at 200-300 micros sampled latency
seems to "burst" to 270k sometimes. my pc has turbo enabled...

post-patch:

had to add an extra 100k rps thread
330-340k rps performance, more steady than pre-patch
200-300 micros sampled while overloaded

The results are repeatable by switching between binaries.
this is a lot more than I was expecting to get from the patch... but a lot closer to the performance I was expecting to get from the proxy in the first place.

did a few extra random experiments:

dropping the # of connections per pool to 1 gives similar performance between next and this branch.
so patch wins a lot as batching decreases and backend connection count increases, which is where the overhead for excess epoll syscalls comes in, as expected.

Just did first code review, added a missing stop_timeout in the cleanup routine.
going to do a set of mcshredder tests against network overload scenarios then do one more code audit and probably merge it this afternoon.

This is now merged.
@feihu-stripe - feel free to still comment on this PR if you have questions. worst case I add another patch.

this code is so much easier to follow and validate than the old original attempt.. wish I did this months ago.


Use prefix.cc to ensure that all apps use the same prefix for ontologies

We can do a GET like this: http://prefix.cc/?q=http%3A%2F%2Fxmlns.com%2Ffoaf%2F0.1%2Fname
And in the Location header, we will have the prefix: http://prefix.cc/foaf:name
Or we can parse http://prefix.cc/context but we will need to do it every time as a prefix can have been added just recently.

In general, I like the idea.
Two things though:

The site's TLS is not properly configures which indicates poor maintenance and could lead to issues.
Since items are votable, this will not guarantee deterministic results over time which could cause trouble as well.

Do you think that's a problem in practice?

The LOV website seems better maintained and it has an API, but it's more difficult to add a new ontology, while with prefix.cc it can be done within minutes. I think it's important to make it easy to add ontologies, otherwise applications won't be able to register.
For ActivityPods, we will use prefix.cc only when applications register classes from unknown ontologies. But after the app registration, the prefix will be saved (on the settings database) and prefix.cc won't be queried again. Applications will use the TypeIndex to know what is the URL of the container.
Two other alternatives:

We could create our own database of prefixes. But this is a lot of maintainance, and what rules do we apply to accept new proposals ?
Applications could declare their favorite prefixes when registering on the pod. This was the option I initially preferred, but then I thought it could create conflicts. At least with prefix.cc we can be sure that common ontologies prefixes will not change so easily.

We could create our own database of prefixes. But this is a lot of maintainance

Totally that this is probably out of scope :)

Applications could declare their favorite prefixes when registering on the pod. This was the option I initially preferred, but then I thought it could create conflicts.

I see..
Thanks for the clarifications :)


Introduce smart --max-segment-length parameter to evenly split larger scenes

Problem/Use Case
I'm using PySceneDetect to encode videos with quite large GOP size (let's say 30 seconds, this isn't a good setting for some cases but works pretty well for my home use). The issue or possible improvement comes up with scenes that are slightly longer than the GOP size: let's say a scene is 122 seconds long. The resulting segment will exhibit the downsides of the large GOP size (slow seeking for the first 120 seconds, at least if no hardware decoding is available) and yet the scene has five key frames, with the last GOP being only two seconds long. The issue is the same regardless of TIMECODE unit.
Solutions
With a setting of a maximum scene length (or another name since it's still the same scene), the scene could be split up even further, as in for the example I described, PySceneDetect could determine that five segments are needed and split the scene evenly into clips with a duration of 24.4 seconds (or whatever that would be in frames).
Proposed Implementation:
Introduce the parameter --max-segment-length to set the maximum size of a segment. Scenes surpassing the maximum segment length are split as evenly as frame numbers and the number of segments allow into the required number of segments.
Alternatives:
No idea.
Examples:
None

Are you encoding the videos via the split-video command or via some other means?  It should be possible to post-process the scene list to look for scenes that don't meet this constraint.
Is there no options that can be specified to ffmpeg to avoid slow seeking in these cases?  And what do you mean by PySceneDetect could determine five segments?

Are you encoding the videos via the split-video command or via some other means?

Via split-video.

Is there no options that can be specified to ffmpeg to avoid slow seeking in these cases?

One could make the GOP smaller, but this reduces the efficiency of the encoded video. A GOP normally has one I-frame at the beginning, which can be decoded independently, all other frames (B- and P-Frames) in the GOP rely on that I-Frame, either directly or indirectly. Because of how a video works, it makes sense to set a new I-Frame at a scene change (hence why I'm posting it here), as the bigger the change between frames, the smaller the efficiency of B- and P-Frames.
I-Frames are much larger than other frames as they need to contain the full picture (like a JPEG file), so for that reason, you'd want as few as possible. The downside comes in when you want to seek to somewhere where the last I-Frame is far away, as your player needs to go back to the I-Frame and decode all following frames until it arrives at the desired seek position.
So what I was trying to explain is that there could be such a parameter that PySceneDetect could make these splits across scenes evenly, as the underlying video codec either can't detect a scene change itself or if it can, it usually doesn't analyze the video as a whole before processing it, which means it would run into the issue I described initially: it would create four GOPs at 30 seconds, start the fifth one, process it for two seconds, detect that the scene is over and then start a new GOP.

And what do you mean by PySceneDetect could determine five segments? Sorry I am not that familiar with these encoder params.

The "detection" part would be that PyScene detects a scene of length 122 seconds and would calculate with user-supplied parameter that the scene would need to be split into five segments so that no segment exceeds exceeds the maximum segment length.
Sorry if this is explained in a convoluted way, English isn't my first language.

Based on your input, I have come to the conclusion that the functionality doesn't make too much sense in PySceneDetect itself.
The reason is that GOPs would have to be closed if the functionality was to be implemented here. It's better to write some simple logic to check if the current group is larger than the maximum GOP size and if yes, recalculate it.


fx: Orbit management - config userName + redundancy

Adding tests

PR has been updated. Reviewers, please be cautious.

PR has been updated. Reviewers, please be cautious.


Update link text to f# reference

Hi @Myoxocephalus, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@Myoxocephalus : Thanks for your contribution to the Azure documentation! The author, @christopheranderson, has been notified to review your proposed change.

#sign-off


Sample Query

Hi,
We have installed and set the permission for Grafana OCI log plugin but when we query using wild card search () it giver error that Message: GSL:line 1:0 unexpected input '' expecting one of CAST, FILE, SEARCH, SET\n*\n^\n\nOperation Name: SearchLogs\nTimestamp: 2022-12-19 14:46:58 +0000 GMT\nClient Version: Oracle-GoSDK/65.23.0\nRequest Endpoint: POST https://logging.eu-frankfurt-1.oci.oraclecloud.com/20190909/search?limit=1000\nTroubleshooting Tips: See https://docs.oracle.com/iaas/Content/API/References/apierrors.htm#apierrors_400__400_cannotparserequest for more information about resolving this error
Do you have a sample query that we can try?
Regards
Yogesh

Hi. I do not know your use case, but something like this is working (with wild card set): search "ocid1.compartment.oc1..xxx/ocid1.loggroup.oc1.eu-zurich-1.xxx/ocid1.log.oc1.eu-zurich-1.xxx" | subject = '/var/log/*' | summarize count(subject) by source,subject


Implement a Security page

We need a replacement for https://pypi.python.org/security.

Working on this!

A page has been added for this, but it's currently completely blank. Should just require filling out the template with content now.

@dstufft - should I use the same content as in https://pypi.python.org/security ?
Note: fixing this page also contributes to fixing the missing template titles exposed in #1203

Hey- So I (or someone) needs to actually come up with the correct content. The current security page is got a lot of incorrect information in it. Probably this one needs to just be done by me.

I've just been looking over the docs and had a thought...
Is there any reason to have a separate security page that is different from https://warehouse.readthedocs.io/security/ ?
Maybe instead of having a standalone page, we can just link to this?  That would mean we would only need to update the security content in one place.

Just to chime in unannounced, I find the RTD security page to be clearer than the current PyPI security page, because it offers just one course of clear action: email Donald and Richard.
Having used the security page on the current PyPI previously, and ultimately settling on emailing Donald, it was not without some reticence. I've outlined my thought process previously in a private email exchange with Donald (August 26, 2015), and looking back, there were a number of bullet points to get me to the point of making a decision vs. "just email these addresses"
As an addendum to @nlhkabu remark about linking, I'd subjectively prefer to see the security page on the PyPI site and linked to from the warehouse RTD -- trust is important, and causing a potential reporter to second guess where they've ended up isn't ideal, especially if they don't know that "warehouse" is ultimately synonymous with PyPI going forward. I don't know how much additional maintenance burden that may result in though.

As an addendum to @nlhkabu remark about linking, I'd subjectively prefer to see the security page on the PyPI site and linked to from the warehouse RTD... I don't know how much additional maintenance burden that may result in though.

I'm not too concerned about moving this content to a standalone page in Warehouse itself - the template is already there, so all we need to do is transfer the content.
My main concern with maintenance not duplicating the content, so linking to the Warehouse page from RTD seems like a good idea to me.
I agree that an end user is more likely to trust a 'page' on PyPI than the Warehouse docs.
@dstufft are you ok for me to go ahead with this?

@nlhkabu Using that content is probably roughly OK for now until I figure out a better policy that doesn't rely on me and Richard answering email. I agree it should be on PyPI itself not in our docs (which are less docs and more a collection of development notes). Should be easy to copy the data over and just delete that page from the Warehouse docs.

I'm putting my hand up for this!

This should be closed since #1985 is merged - correct?

Yup!


What is the postfix setup password?

I'm running through the postfix configuration, and in the setup wizard there is a 'setup password' field - what is this - where can I retrieve it from?

Hi, it's the master password hashed explain here : https://github.com/hardware/mailserver/wiki/Postfixadmin-initial-configuration
Make a screen if is not answer your question :)

I was reading through that wiki page at the time.
2 - Define setup password - how do I do this?
Unless this form is responsible for setting the setup password?
If I click on "lost password" I get sent to a change setup password?
Seems a bit confusing.

Ah nevermind, I must have forgotten I set the password on the first screen.


[Feature Request] Create a title page and link back to fiction page

It would be useful to generate a title page with these details:

Link to the work on Royal Road
Title
Author name
Datetime the work was scraped

I mostly just wanted a link to get back to the fiction page.

Implemented in https://github.com/Aivean/royalroad-downloader/releases/tag/v2.10.0

The title page section needs chapter class like in #22 to appear in ToC post-conversion to epub. Changing "title" class to "chapter" sort of works, but because title and description are divided by 2 distinct h1/h2 tags they are treated as separate chapters.
The chapter name is whatever in the h1/h2 tags. Maybe a good idea to instead put Title into the first h1 tag and make a link and image a paragraph content and then the same thing for description.
Something like this for the title

            Title

            [Farmer] Mage | Royal Road

          <be>
          <img class

and

              Description

by
            S.C. King

for description
so that it looks like this

Hhhhhhm, i can't really answer that question coherently. I guess for feature completeness and because it usually is that way. If it's a separate "section" its supposed to be indexed. On the other hand some publishers do leave out the title page, but index the rest.

I suppose its not truly needed or required so - sorry for the disturbance.

@ZeeWanderer, I have objection for marking title page with the chapter class — semantically it's not a chapter. If you find an alternative method that is supported by your converter, PR would be welcome.


App Crashes when you tap a profile picture

On the Scholar Detail Page, when you tap a profile picture the app crashes.

Can't reproduce the issue here. Which Scholar?

Crashes on every scholar (when you tap their picture to enlarge it).  Tested this on iPhone 6. Doesn't seem to appear any other device.

Works for me on iPhone 6

Please add the crashlog in the issue next time

Closing old issue, no crashlog attached. Feel free to reopen with a crashlog :)


s/bitciond/bitcoind

Duplicate of https://github.com/lightningnetwork/lnd/pull/851, thanks anyway :)


Adding node-polyfill-webpack-plugin breaks semi-common "msgpack" npm module

Summary: Using the npm package @msgpack/msgpack with node-polyfill-webpack-plugin fails with "The request 'process/browser' failed to resolve only because it was resolved as fully specified".
Context: I have a web app. I upgraded from Webpack 4 to 5. This broke some npm dependencies that relied on node-builtin modules. I installed node-polyfill-webpack-plugin to fix this. This broke a different npm dependency (msgpack)
Repro:
I have a minimal test repo here. To test with it, run npm install && npx webpack and a minimal web app will be built into site/.
In this repro, the commit 2f5f39b8 is a working web app using Webpack 4 and msgpack.  213e2608 is a working web app upgraded to Webpack 5 and msgpack— the breakage does not occur until I try to add node-polyfill-webpack-plugin. 51fbc8fec (HEAD) adds node-polyfill-webpack-plugin and configures it as in "Usage" in the README. This results in the error:
ERROR in<EMAIL_ADDRESS>2:38-45
Module not found: Error: Can't resolve 'process/browser' in<EMAIL_ADDRESS>Did you mean 'browser.js'?
BREAKING CHANGE: The request 'process/browser' failed to resolve only because it was resolved as fully specified
(probably because the origin is a '*.mjs' file or a '*.js' file where the package.json contains '"type": "module"').
The extension in the request is mandatory for it to be fully specified.
Add the extension to the request.
resolve 'process/browser' in<EMAIL_ADDRESS>  Parsed request is a module
  using description file<EMAIL_ADDRESS>(relative path: ./dist.es5+esm/utils)
    Field 'browser' doesn't contain a valid alias configuration
    resolve as module
<EMAIL_ADDRESS>doesn't exist or is not a directory
<EMAIL_ADDRESS>doesn't exist or is not a directory
      /Users/mcc/work/h/ts-hello/node_modules/@msgpack/msgpack/node_modules doesn't exist or is not a directory
      /Users/mcc/work/h/ts-hello/node_modules/@msgpack/node_modules doesn't exist or is not a directory
      /Users/mcc/work/h/ts-hello/node_modules/node_modules doesn't exist or is not a directory
      looking for modules in /Users/mcc/work/h/ts-hello/node_modules
        existing directory /Users/mcc/work/h/ts-hello/node_modules/process
          using description file: /Users/mcc/work/h/ts-hello/node_modules/process/package.json (relative path: .)
            using description file: /Users/mcc/work/h/ts-hello/node_modules/process/package.json (relative path: ./browser)
              Field 'browser' doesn't contain a valid alias configuration
              /Users/mcc/work/h/ts-hello/node_modules/process/browser doesn't exist
      /Users/mcc/work/h/node_modules doesn't exist or is not a directory
      /Users/mcc/work/node_modules doesn't exist or is not a directory
      /Users/mcc/node_modules doesn't exist or is not a directory
      /Users/node_modules doesn't exist or is not a directory
      /node_modules doesn't exist or is not a directory
<EMAIL_ADDRESS>59:0-86 636:30-52 637:21-33 640:21-33
<EMAIL_ADDRESS>8:0-71 10:0-63 10:0-63
 @ ./src/index.tsx 19:16-43

webpack 5.38.1 compiled with 1 error in 3975 ms

Expected behavior: I should be able to recreate Webpack 4 behavior in Webpack 5+node-polyfill-webpack-plugin.
Note: It is possible this is not a bug, NPWP's behavior is correct, and I would get my expected behavior if I properly configured excludeAliases. If so, I think this is a documentation issue, and I am filing a separate issue about that.

Identical issue, description, and possible solutions here: https://github.com/Raynos/global/issues/21
In short:

either the publisher of each polyfill publishes them as CJS/ESM combos
or the importer, in this case node-polyfill-webpack-plugin, uses an ESM-compatible entry point, like 'process/browser.js' instead of 'process/browser'

Maybe we could just upgrade this package to use ESM instead of CommonJS?

I don’t think that's the issue. The problem is that this package redirects import 'process/browser' to a package that cannot be imported that way. The ESM change needs to be in the process package itself, there's nothing this plugin can do, I think, since Webpack complains about the dependency, not the plugin

uses an ESM-compatible entry point, like 'process/browser.js' instead of 'process/browser'

Alright, let's do this then.

https://github.com/Richienb/node-polyfill-webpack-plugin/pull/15


Dataset processing error-ing on file_info = file_url.file_info()

Three datasets recently failed to upload all with the same error:
https://czi-sci.slack.com/archives/C0244PQK934/p1644017956507849
https://czi-sci.slack.com/archives/C0244PQK934/p1644017976627249
https://czi-sci.slack.com/archives/C0244PQK934/p1644017977605369
The stack trace for these are the same:
Traceback (most recent call last):
File "/backend/corpora/dataset_processing/process.py", line 554, in main
process(dataset_id, os.environ["DROPBOX_URL"], os.environ["ARTIFACT_BUCKET"])
File "/backend/corpora/dataset_processing/process_download_validate.py", line 37, in process
local_filename = download_from_dropbox_url(
File "/backend/corpora/dataset_processing/process.py", line 265, in download_from_dropbox_url
file_info = file_url.file_info()
File "/backend/corpora/common/utils/dl_sources/url.py", line 83, in file_info
resp.raise_for_status()
File "/usr/local/lib/python3.8/dist-packages/requests/models.py", line 960, in raise_for_status
raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 500 Server Error: Internal Server Error for url:

It seems like these error may be transient (need to verify with curators), but we should look into ensuring that either the logging for this error is better or that we have a failsafe if the URL returns a 500 for the requests.head call. Alternatively, I believe we can remove this code per the recommendation in this issue.

Turns out the file_info call is actually used in the upload pipeline for a few things, in particular by the progress indicator, so we might want to keep it after all. I don't know what caused the 500 here, but it could have been an excess of calls to Dropbox (since we uploaded a lot of datasets that day? I am not sure).
To mitigate, we could add retries to that call, or make it optional (although that would require disabling the progress indicator)

Is dropbox sending Retry-After response headers that are being ignored?

Closing as not reproducible anymore.


Authorizing already existing subscriptions once Fahrschein is updated

What needs to be done and why is it needed?
With the roll out of authorization support in fahrschein in this ticket. Users are able to authorize newly created subscription only. This doesn't allow authorization of historical subscriptions created in the past by the library or manually using the Nakadi API.
As a software engineer
I want to be able to authorize all previously created subscription and new ones created by application when using Fahrschein
So that the adaptation of the authorization feature increases and the coverage of authorized events increases
What needs to be changed?
1- As soon as an application starts relying on fahrchein with the latest version supporting authorization and the authorization is used by the clients.
2 - The service fetches all current subscriptions, if some are not authorized the put endpoint is used to update the subscription
3 - Consumption is resumed by the application.
Acceptance criteria
1 - As a user of the authorization feature on a specific event type this is application while creating the subscription and updated previously existing subscriptions which are authorized.
Risks and impact

Breaking the consuming application if the application itself is not added as a reader and a new deployment blocks the app creating the subscription from consuming a previously working event

This is indeed not a bad idea, it's only not so easy to implement. Code to list subscriptions by event types, getting them, updating them — this is all not implemented currently.

I see several options how to proceed with this:

Create subscriptions by hand and specify subscription id when streaming

:heavy_check_mark: Easy to implement
:heavy_check_mark: No additional call to find the subscription
:x: Not supported by Fahrschein right now
:x: If someone deletes a subscription the application needs to be repaired/redeployed

Create subscriptions automatically if absent

This is the current mode of operation that Fahrschein supports
:heavy_check_mark: Convenient for users since subscriptions are created automatically
:x: Not supported by Fahrschein right now
:x: Users need to supply admins and readers when streaming which complicates all applications
:x: Changing the subscription manually afterwards will result in client code and reality going out of sync

Create and update subscription automatically

POST, compare result, PUT if necessary to update
:heavy_check_mark: Convenient for users since subscriptions are created automatically
:heavy_check_mark: Keeps a single source of truth since the application will enforce a consistent state of the subscription
:heavy_check_mark: Applications could only allow themselves as admins, completely locking out any users
:x: Manual changes in the UI are either impossible or will be reverted automatically once an hour

@whiskeysierra I was proposing option 3 actually. but haven't thought about overriding UI changes with every run. How about POST -> Compare -> Merge -> PUT ?

How about POST -> Compare -> Merge -> PUT ?

Then you have this problem:

:x: Changing the subscription manually afterwards will result in client code and reality going out of sync

Normally I'd be against adding such logic to a library like fahrschein, since different applications might want different behaviour. The requirements sound very useful though and making subscriptions more secure is a very good goal. So we should at least expose the PUT operation and optionally have one or two convenience methods for easier usage. I don't think we should or need to update behaviour of existing methods, since applications will have to be changed anyway to specify the authorization.
This could looke like
NakadiClient.updateSubscriptionAuthorization(Subscription subscription, Authorization authorization)

To make the common usecase easier we could add
SubscriptionBuilder.subscribeAndUpdateAuthorization()
SubscriptionBuilder.subscribeAndInitializeAuthorization()

In my opinion, more complex logic like merge belongs into the application.


Configure entries layout list or grid

This is an enhancement or feature.
Summary
This allows to use grid layout on page.entries_layout on the home layout.
Context
There is support for layout grid on most places but home page.
I mimicked how it's done on posts/categories/tags to allow that on home page. See demo at www.cornerinthemiddle.com
Included a break since when using grid the post images are too close to the horizontal line bellow posts text.
There's a entries div now surrounding the posts since the first row of the grid was having a slight padding on the left. The home now behaves like posts/categories/tags pages with grid but including the paginator.
Tested on mobile renders ok and on hd desktop, for best results on desktop we can use classes: wide and paginate: 4 on _config.yml (or multiples of 4 if you want more rows)

The

 margin solution looks OK to me (as I would also do so if I were to implement this), but I'm sure @mmistakes won't be happy this way. A better approach would be to update the CSS and set margin-bottom of the element right above to a reasonable value.
In _sass/minimal-mistakes/_archive.scss, at line 35, the margin-bottom attribute of the "Recent Posts" text is currently set to zero (3rd value of margin, see W3schools).
https://github.com/mmistakes/minimal-mistakes/blob/d5d38633225cdba2635e463fd045d3da97ce6f3f/_sass/minimal-mistakes/_archive.scss#L34-L36
You can change it to 1em and remove the empty
 tag, which will result in an appearance like this:

That should look better IMO.

The

 margin solution looks OK to me (as I would also do so if I were to implement this), but I'm sure @mmistakes won't be happy this way. A better approach would be to update the CSS and set margin-bottom of the element right above to a reasonable value.
In _sass/minimal-mistakes/_archive.scss, at line 35, the margin-bottom attribute of the "Recent Posts" text is currently set to zero (3rd value of margin, see W3schools).
https://github.com/mmistakes/minimal-mistakes/blob/d5d38633225cdba2635e463fd045d3da97ce6f3f/_sass/minimal-mistakes/_archive.scss#L34-L36
You can change it to 1em and remove the empty
 tag, which will result in an appearance like this:

That should look better IMO.

As I replied, thanks for the tip. It's working nice now. I went for 0.5em instead of 1, enough space to not have images stretching up to the horizontal line.

Nice work @TarodBOFH, everything looks good to me.

Raising here the comment @mmistakes but it seems that the grid layout got broken on the last build.
Items are haivng only the first item as class grid__item while the next ones are list__item
I'll have a look on the cahnges on 4.20 that might have caused that

This is what it renders with the entries layout on home;

This is what it rendered with the layout grid prior the 4.20 release. Did something went incompatible today?

I'm not seeing anything in 4.20.0 that could have broken this.
Something else might be at play. include.type is grid on the first item in the for loop, everything else is nil. Which is why they're inheriting the default value of list.
I'm pretty sure I tested the PR and it worked, but I'm a bit foggy on it now.
I've hit this before with for loops and I think we'll need to pull it out of that and assign the variable before...

This fixes it. I'll push an update
{% assign entries_layout = page.entries_layout | default: 'list' %}

  {% for post in posts %}
    {% include archive-single.html type=entries_layout %}
  {% endfor %}

I was openning a bug, I've to leave now so probably won't be able to fix till tomorrow, I've updated my site to use a specific commit has of the remote theme, Ill have a look later. Thanks! (I'll keep the bug open).
The funny thing is that the behavior happens also on the posts and categories pages but was not happening before (those are also inheriting the layout from home).
PS> I wont submit the bug then. We're realtime chatting. Thanks!

Not able to reproduce on posts and categories, but it should be fixed for home.html layout.
Looks like if you try to read page. variables inside of a for loop unexpected things happen. Not sure why that changed but maybe there was a Jekyll version bump that affected it.
Assigning that variable outside the loop in the proper context seems to fix the issue.

It is still happening on the posts and layouts pges (home works however)
If I remove the grid from the home, posts and categories pages still broken.

I'm going to leave you a branch to experiment (has dockerfile for jeckyl if you want to fast deploy the site) that reproduces the error without customizations.

https://github.com/TarodBOFH/tarodbofh.github.io/tree/mm-bug-with-grid-layout
no entries_layout: grid defined on frontmatter == posts and categories broken and home list as expected
entries_layout: grid on frontmatter == posts and categories broken bug home is working
There are two commits on that branch that you can test. I need to go now, sry. ttyl8r
//cc @mmistakes

Nevermind, I see it on posts.html and cateogories.html layouts too.
I'll just apply the same fix to those layouts.
Only thing I can think of that's breaking this now is a Jekyll version bump. Perhaps with all the caching stuff they added there's some Liquid weirdness going on. That grid stuff has been in the posts layout for some time and the only thing that might have touched those includes were the post date stuff added in 4.20.1. And I don't see how that could have broken it.

Nevermind, I see it on posts.html and cateogories.html layouts too.
I'll just apply the same fix to those layouts.
Only thing I can think of that's breaking this now is a Jekyll version bump. Perhaps with all the caching stuff they added there's some Liquid weirdness going on. That grid stuff has been in the posts layout for some time and the only thing that might have touched those includes were the post date stuff added in 4.20.1. And I don't see how that could have broken it.

Don't think the jeyll version bump as my dockerfile has a fixed jekll version. I also don't see what might have broken it; I was looking for some post-date assiging something to the variable include.type or something like that but I am no liquid expert.
Thanks for the effort mate. You've earned a beer or dinner if you come to Madrid.

I'm stumped as to what could have broken it then. Maybe something in the Liquid further up the page is clobbering the page. variables... who knows.
For now the workaround of pulling those values out of the for loop seems to work.

Got it. It's this line on the posts date {% assign page = post | default: page %}

Got it. It's this line on the posts date {% assign page = post | default: page %}

That discovery was rather difficult but you made it! Now I know how to fix it properly.
In short: Use another variable name to avoid clobbering the page variable.

Good find @TarodBOFH.

@iBug I already pushed some fixes before discovery this bad assign above. Not sure if it's worth reversing those commits and applying some sort of fix to {% assign page = post | default: page %}. I have no idea if this has other side affects elsewhere we haven't caught yet.
I feel like there was somewhere I had to build in logic around date as things broke in page or collection documents that aren't posts. Or maybe I'm just confusing it with one of my other themes.
Point being, we'll need to test this better as I know there's some edge cases in there.

I see little value in reverting that fix (v4.20.1) if it proves effective, though the status quo is far from optimal, as overriding a built-in object is particularly prone to errors. But it's always safe to use another name (object maybe?) if you're afraid of breaking other parts.

Thanks @iBug. I'm going to take a good look and likely refactor the new post meta/data stuff that was just added. I really should have taken a better look at that PR as I'm finding some other things that could be cleaned up... inconsistent class names/include name, dropping in
 for styling purposes instead of using CSS, etc.


No documentation on how to "reset" via window.STATE_MACHINE_RESET in production outside dev tools / environment

Could you please describe how to achieve the "reset" / window.STATE_MACHINE_RESET functionality upon form completion in production?
I would like to allow a new user to fill out the registration form after another user has already completed it.

you should create an action and return your empty state.

Do you by chance have an example of this?
I can’t seem to find any on GitHub or Stackoverflow.
Thanks!
On Thu, Oct 1, 2020 at 3:41 PM Bill<EMAIL_ADDRESS>wrote:

you should create an action and return your empty state.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bluebill1049/little-state-machine/issues/56#issuecomment-702434431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXNFHUFDQLBTFBX4N45PATSIUASFANCNFSM4SA7AZFQ
.

As Bill said. Inside an action, let's call it reset return an empty object {}.
// action.js
export function resetState() {
  return initialState
}

// component.js
const { state, action } = useStateMachine(resetState)

// Call reset
action({})

Because of TypeScript you must pass something. Calling action() will not pass TS lint, but if you don't use TS you can do it.

As Bill said. Inside an action, let's call it reset return an empty object {}.
// action.js
export function resetState() {
  return initialState
}

// component.js
const { state, action } = useStateMachine(resetState)

// Call reset
action({})

Because of TypeScript you must pass something. Calling action() will not pass TS lint, but if you don't use TS you can do it.

Thanks @ilicmarko for helping ❤


Set-SecretStoreConfiguration -Authentication None often fails

If I try to set the Authentication to None the follwing command often fails.
Set-SecretStoreConfiguration -Authentication None -Password $tmpPw -Interaction None
The following error occurs:
Set-SecretStoreConfiguration :
In Zeile:1 Zeichen:1
+ Set-SecretStoreConfiguration -Authentication None -Interaction None
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (Microsoft.Power...reConfiguration:SetSecretStoreConfiguration) [Set-SecretStoreConfiguration], PSInvalidOperationException
    + FullyQualifiedErrorId : SecretStoreConfigurationUpdateFailed,Microsoft.PowerShell.SecretStore.SetSecretStoreConfiguration

But after a few attemps (sometimes 2 but often even more) it works.

@SamLue Can you provide more information on the error?  If you are running PowerShell 7+ then you can just run Get-Error, which will display details of the last error.

Hello @PaulHigin
here my complete test to reproduce this issue. Now with PowerShell 7.1.3 and Get-Error
PS C:\Users\PN118> Register-SecretVault -Name $VaultName -ModuleName Microsoft.PowerShell.SecretStore -DefaultVault -Confirm:$false
PS C:\Users\PN118> $tmpPw = Read-Host -AsSecureString
********
PS C:\Users\PN118> Reset-SecretStore -Password $tmpPw -Force
WARNING: !!This operation completely removes all SecretStore module secrets and resets configuration settings to new values!!

Performing the operation "Changes local store configuration" on target "SecretStore module local store".
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"): Y
Set-SecretStoreConfiguration:
PS C:\Users\PN118> Get-Error

Exception             :
    Type        : System.Management.Automation.PSInvalidOperationException
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            HResult : -2146233087
        CategoryInfo          : InvalidOperation: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : InvalidOperation
    TargetSite  :
        Name          : ThrowTerminatingError
        DeclaringType : System.Management.Automation.MshCommandRuntime, System.Management.Automation, Version=<IP_ADDRESS>,
Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    StackTrace  :
   at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)
    Source      : System.Management.Automation
    HResult     : -2146233079
TargetObject          : Microsoft.PowerShell.SecretStore.SetSecretStoreConfiguration
CategoryInfo          : InvalidOperation: (Microsoft.PowerShel…tStoreConfiguration:SetSecretStoreConfiguration)
[Set-SecretStoreConfiguration], PSInvalidOperationException
FullyQualifiedErrorId :
SecretStoreConfigurationUpdateFailed,Microsoft.PowerShell.SecretStore.SetSecretStoreConfiguration
InvocationInfo        :
    MyCommand        : Set-SecretStoreConfiguration
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 7
    Line             : Set-SecretStoreConfiguration -Authentication None -Password $tmpPw -Interaction None
    PositionMessage  : At line:1 char:1
                       + Set-SecretStoreConfiguration -Authentication None -Password $tmpPw -I …
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Set-SecretStoreConfiguration
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1

PS C:\Users\PN118>

But it is realy strange that sometimes the command is successful.
Have you tried these steps?
I can observe this behavior on completly different systems. Server OS with 5.1 but also on my Windows 10 with pwsh 7.1.3 and powershell 5.1

Hmm, the above error does not provide any useful error message.  Since it sometimes works, I am wondering if there is trouble writing the configuration file.  But we should get a message about that if that is the case.
Can you try assigning the error to a variable and looking to see if there are inner exceptions with more information?
$err = Get-Error
$err.Exception
$err.Exception.InnerException
$err.Exception.InnerException.InnerException
...
# At some point the InnerException property will be null.

Hello @PaulHigin ,
there is no InnerException. Only an Exception.

Performing the operation "Changes local store configuration" on target "SecretStore module local store".
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"): A
Set-SecretStoreConfiguration:
PS C:\Users\PN118> $err = Get-Error
PS C:\Users\PN118> $err

Exception             :
    Type        : System.Management.Automation.PSInvalidOperationException
    ErrorRecord :
        Exception             :
            Type    : System.Management.Automation.ParentContainsErrorRecordException
            HResult : -2146233087
        CategoryInfo          : InvalidOperation: (:) [], ParentContainsErrorRecordException
        FullyQualifiedErrorId : InvalidOperation
    TargetSite  :
        Name          : ThrowTerminatingError
        DeclaringType : System.Management.Automation.MshCommandRuntime, System.Management.Automation, Version=<IP_ADDRESS>,
Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    StackTrace  :
   at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)
    Source      : System.Management.Automation
    HResult     : -2146233079
TargetObject          : Microsoft.PowerShell.SecretStore.SetSecretStoreConfiguration
CategoryInfo          : InvalidOperation: (Microsoft.PowerShel…tStoreConfiguration:SetSecretStoreConfiguration)
[Set-SecretStoreConfiguration], PSInvalidOperationException
FullyQualifiedErrorId :
SecretStoreConfigurationUpdateFailed,Microsoft.PowerShell.SecretStore.SetSecretStoreConfiguration
InvocationInfo        :
    MyCommand        : Set-SecretStoreConfiguration
    ScriptLineNumber : 1
    OffsetInLine     : 1
    HistoryId        : 5
    Line             : Set-SecretStoreConfiguration -Authentication None -Password $tmpPw -Interaction None
    PositionMessage  : At line:1 char:1
                       + Set-SecretStoreConfiguration -Authentication None -Password $tmpPw -I …
                       + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    InvocationName   : Set-SecretStoreConfiguration
    CommandOrigin    : Internal
ScriptStackTrace      : at <ScriptBlock>, <No file>: line 1

PS C:\Users\PN118> $err.Exception

Type           : System.Management.Automation.PSInvalidOperationException
ErrorRecord    : System.Management.Automation.ParentContainsErrorRecordException
TargetSite     : Void ThrowTerminatingError(System.Management.Automation.ErrorRecord)
StackTrace     :    at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)
Message        :
Data           : {}
InnerException :
HelpLink       :
Source         : System.Management.Automation
HResult        : -2146233079

PS C:\Users\PN118> $err.Exception.InnerException
PS C:\Users\PN118> $err.Exception.InnerException
PS C:\Users\PN118> $err.Exception.InnerException.InnerException
PS C:\Users\PN118>

But I think you can reproduce this error. Have you tried?
Wouldn't that be easier to troubleshoot?
But feel free to ask more, if I can help and provide more information.

@SamLue I am assuming that your variable "$tmpPw" is an empty/Enter from a user hitting Return/Enter at the console? If so and this was no where to be found in my extensive search, that you need to enter an actual password (I used 12345) and that it is expecting a password to then clear the password!
My results that I got to work:

PS C:> Set-SecretStoreConfiguration -Authentication None -Interaction None
Confirm
Are you sure you want to perform this action?
Performing the operation "Changes local store configuration" on target "SecretStore module local store".
[Y] Yes  [A] Yes to All  [N] No  [L] No to All  [S] Suspend  [?] Help (default is "Y"): a
Creating a new Microsoft.PowerShell.SecretStore vault. A password is required by the current store configuration.
Enter password:

Enter password again for verification:

A password is no longer required for the local store configuration.
To complete the change please provide the current password.
Enter password:

PS C:> Get-SecretStoreConfiguration
       Scope Authentication PasswordTimeout Interaction
        -----  --------------     ---------------  -----------

CurrentUser                None                     900           None

I was then able to create (set) and retrieve (get) the secret without being prompted for a password.  I also ran the "Set-SecretStoreConfiguration -Authentication None -Interaction None" again, and it did not prompt me for a password this time!
Maybe the "Set-SecretStoreConfiguration" could use a "-Force" or "-AllowClobber" feature. :)

I was able to find a repro for this, and it is indeed a race condition.  Thanks for reporting this, and I'll get a fix out soon.

Ok, I have a fix for this which will be part of the v1.0.3 release.


i18n(zh-cn): Update nav.ts & ui.ts

What kind of changes does this PR include?

New or updated content

Description

Update nav.ts & ui.ts

I don't know how to update docsearch.ts.
Maybe it's because the content in modals is being translated, causing it to show incomplete?

Maybe it's because the content in modal was translated, causing it to show incomplete?
Oh nice catch, yes, probably something related to that.


Question about running a demo

What a great work!
May I ask a few questions?
If I just want to run a demo, all I need to do is downloading one of the datasets and the pretrained model corresponding to the dataset, and running the create_octree.py to extract the octree and then see the result by the Mega-NeRF-Dynamic viewer, is that correct?
Moreover, what's the meaning of --container_path argument and what should I fill in to it?
Many thanks!

Hi - I think you largely got the steps. You can either use the pretrained model with the Mega-NeRF-Dynamic viewer or by just using conventional NeRF rendering via the render_images.py script in this repo.
The --container_path should point towards the path of the merged model obtained after running merge_submodules.py or in your case it just just point towards the pretrained model.

I have the questions too, in render_images.py, how to get poses.txt, intrinsics.txt and embeddings.txt

It looks like svox was built without CUDA. https://github.com/sxyu/svox/issues/4 might be helpful for sorting that part out.
Re: render_images.py:

input should be set to a folder containing poses.txt, intrinsics.txt, and embeddings.txt files. https://github.com/cmusatyalab/mega-nerf/blob/main/scripts/render_images.py#L40 provides some intuition on how these files are used (basically each is a text file containing a camera pose / camera intrinsics / embedding index per line)
output will be where the rendered rgb / depth maps / centroid boundary estimates are written to disk
centroids_path is a somewhat niche option used to generate the right-most panel of the demo images in this repo (overlaying the cell boundaries). You can either point it to a 'params.pt' file generated by the create_cluster_masks.py script or comment all the code relating to that option if you don't care about generating those overlays


Fix a few typos

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

#sign-off


Implement infinite reconnect and cancel.

Enable a user to cancel a connection attempt.
Merge in #132 from @hijae for the settings to control the behaviour.
With this change, the following will occur:

with Infinite-ReConnect = OFF

furble will attempt a single reconnect on connection drop
a successful reconnect will reset the counter

with Infinite-ReConnect = ON

furble will attempt to reconnect indefinitely, unless Cancel is pressed
M5ez menu handling doesn't help us here, the control menu will be shown, but any item press will exit

@hijae Please test and ensure things are working as you expect (I changed quite a lot :face_with_diagonal_mouth:)
I'm doing a battery life test on an M5StickC Plus2 just to see how long it lasts continuously reconnecting ...

I'm doing a battery life test on an M5StickC Plus2 just to see how long it lasts continuously reconnecting ...

I managed to get just under 2 hours continuous reconnection.
I'll hack it and see if there's any difference with the backlight completely off.

I'm doing a battery life test on an M5StickC Plus2 just to see how long it lasts continuously reconnecting ...

I managed to get just under 2 hours continuous reconnection. I'll hack it and see if there's any difference with the backlight completely off.

I didn't manage to get to this, I think the warning is sufficient to set poor expectations.
If you really want longevity, I'd suggest the M5Core2 with the 1500mAh battery.


Binary data example is not working as expected

Describe the bug
I try to store a binary data to a table
As recomended i use
        let binaryData = new Uint8Array(2);
        binaryData[0] = 42;
    db.execute(
      `create table if not exists nodes (
        id varchar(36) primary key not null,
        label varchar,
        vectorLabel blob
      );`
    );
        const insert1 =  db.prepareStatement(
          'INSERT INTO nodes (id, label, vectorLabel) VALUES (?, ?, ?);'
        );
  insert1.bind([generateUUID(),'tea',binaryData])

I get error : [Error: Exception in HostFunction: Unknown JSI ArrayBuffer to variant value conversion, received object instead of ArrayBuffer]
I pass a ArrayBuffer as a parameter insead on Uint8Array and it solve issue but it is not align with a docs etc
Versions:
OS and version: iOS 17
op-sqlite version: 4.0.0-beta1
RN version: 0.73.6
New Architecture: Yes
Reproducible example
Create a minimal reproduction example, otherwise I won't take a look. I'm serious, no example = no solution.

ah. The examples mention the execute functions, not the prepared statements. You need to use:
insert1.bind([generateUUID(), 'tea', binaryData.buffer])

There is some convenience conversion on the other methods that is not present on the bind function

I added the same convenience conversion and also fixed a memory leak. It will be released on next version.


Update update-db-data script for v2.7 with TaskGroupV2 parameters

The missing ones are

wheel_path
pip_extras
pinned_package_versions

but we don't have a direct way to infer them, because they were not systematically tracked up to here. They likely appear in some logs of previous task collections, but for moment setting up that heuristic search is not worth it.
We can provide data-migration scripts if/when it turns out that these information are strictly needed for the lifecycle feature.


Ignore collapsible sidebar CSS when top navigation is enabled.

[x] Changes have been thoroughly tested to not break existing functionality.
[x] New functionality has been documented or existing documentation has been updated to reflect changes.
[x] Visual changes are explained in the PR description using a screenshot/recording of before and after.

Previous to this PR, the CSS for the collapsible sidebar would still be applied when a User has top navigation enabled. This resulted in the following.
Before:

The CSS applied for a collapsible sidebar should be ignored when a User has enabled top navigation. Beforehand, using ->sidebarFullyCollapsibleOnDesktop() would not result in any issues, but using ->sidebarCollapsibleOnDesktop() would result in issues. This PR improves this.
After:

That is understandable and I agree. But I think it would be good to prevent the issue out of negligence. I only came across the problem when I had dynamic settings where a user would enable top navigation while a collapsible sidebar was a fixed setting in my application.

If I were mean, I'd throw an exception when both are used together

Well good thing you aren't mean 😃

@danharrin, go for it and throw an exception, please.
Honestly, I didn't check for these cases as you'd never use both of them anyway.
Having to account for each and every combination only clutters the code and makes it harder to maintain in my opinion.


Create varsitycollege.txt

Added varsity college to the list

@rece212 First stage of verification passed, official domain is varsitycollege.co.za as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

I had added the IIE.txt file to this merge . They both fall under as our brand . Hope this is ok ?

@rece212 I made necessary corrections and merged the request. Thank you.


Packages: use the official librpm instead of libsqlite3

To count installed rpm packages, which support CentOS

Ref: https://ftp.osuosl.org/pub/rpm/api/4.11.1/group__rpmdb.html#ga0b2fac12a598a46a3f0780fab05839a0
Tested on Centos 7 (RPM version 4.11.3) / 8 (RPM version 4.14.3), should works on RHEL 7 / 8 too.
TODO: Test on Fedora

I don't have a fedora machine right now to test it, but i seems like you did a good job.

@CarterLi I installed librpm into my system without having the database. I get this error:

@CarterLi I installed librpm into my system without having the database. I get this error:

Fixed it.


Agent RPC request timeouts on a server RpcServer actor crash

Simulating an unhandled exception in the server RpcServer, the actor will crash:
E, [2017-11-13T13:13:10.806116 #17] ERROR -- : Actor crashed!
RuntimeError: oops
	/app/app/services/rpc_server.rb:88:in `handle_notification'
	/app/app/services/rpc_server.rb:45:in `process!'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/calls.rb:28:in `public_send'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/calls.rb:28:in `dispatch'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/call/async.rb:7:in `dispatch'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/cell.rb:50:in `block in dispatch'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/cell.rb:76:in `block in task'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/actor.rb:339:in `block in task'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/task.rb:44:in `block in initialize'
	/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/task/fibered.rb:14:in `block in create'

If the RpcServer actor crashes, it does not get supervised, because it is spawned in the WebsocketBackend middleware without supervise: https://github.com/kontena/kontena/blob/b9428ffc718da5427d6356705e830b127269bc6c/server/app/middlewares/websocket_backend.rb#L41-L42
The server worker with the crashed RpcServer actor will behave badly: agents connected to that worker will still remain connected, and even things like kontena node health will work, because the server -> agent RPC /etcd/health requests do not pass through the RpcServer actor. However, any RPC requests from the agent will just get stuck in the @queue because the actor is no longer processing them... that shows up as RPC timeouts on the agent:
W, [2017-11-13T13:18:50.498394 #1]  WARN -- Kontena::RpcClient: observe timeout 30.00s: Kontena::RpcClient::RequestObservable</node_service_pods/list@203381531>? (Kontena::RpcClient::TimeoutError)
/app/lib/kontena/rpc_client.rb:77:in `rescue in request'
/app/lib/kontena/rpc_client.rb:74:in `request'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/calls.rb:28:in `public_send'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/calls.rb:28:in `dispatch'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/call/sync.rb:16:in `dispatch'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/cell.rb:50:in `block in dispatch'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/cell.rb:76:in `block in task'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/actor.rb:339:in `block in task'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/task.rb:44:in `block in initialize'
/usr/lib/ruby/gems/2.4.0/gems/celluloid-0.17.3/lib/celluloid/task/fibered.rb:14:in `block in create'

One possible crash cause for the server RpcServer actor is the mongo Grid.find(...) query used to initialize the RPC handlers.

This is specific to the server worker process... restarting the kontena-agent may appear to fix the problem if the agent connects to a different server worker process with a healthy RpcServer actor.
Looks like symptoms of this kind of broken server worker process on 1.4 include a node that's stuck in the connecting state, in case it reconnects to a broken server worker process... if it remains connected to the same worker process where the RpcServer actor crashed, that symptom won't show up
NAME        VERSION     STATUS          INITIAL   LABELS
⊛ core-01   1.4.0.dev   connecting 3m   1 / 1     region=test
⊛ core-02   1.4.0.dev   online 4m       -         region=test,test=label


How to disable email in RSpec

I've just started using the :confirmable module but now my unit tests are trying to send out emails, which is failing with the following:
NameError:
       undefined local variable or method `app' for #<Devise::Mailer:0x0000000009b83090>
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/rack-test-0.7.0/lib/rack/test/methods.rb:31:in `build_rack_mock_session'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/rack-test-0.7.0/lib/rack/test/methods.rb:27:in `rack_mock_session'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/rack-test-0.7.0/lib/rack/test/methods.rb:42:in `build_rack_test_session'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/rack-test-0.7.0/lib/rack/test/methods.rb:38:in `rack_test_session'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/rack-test-0.7.0/lib/rack/test/methods.rb:46:in `current_session'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/mailers/helpers.rb:17:in `devise_mail'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/app/mailers/devise/mailer.rb:7:in `confirmation_instructions'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/models/authenticatable.rb:191:in `send_devise_notification'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise_token_auth-0.1.42/app/models/devise_token_auth/concerns/user.rb:75:in `send_confirmation_instructions'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/models/confirmable.rb:126:in `send_reconfirmation_instructions'
     # ./app/controllers/v1/users_controller.rb:48:in `update'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/test/controller_helpers.rb:33:in `block in process'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/test/controller_helpers.rb:100:in `catch'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/test/controller_helpers.rb:100:in `_catch_warden'
     # /home/vagrant/.rvm/gems/ruby-2.4.2/gems/devise-4.3.0/lib/devise/test/controller_helpers.rb:33:in `process'
     # ./spec/controllers/v1/users_controller_spec.rb:114:in `block (4 levels) in <top (required)>'

Do I need to mock something out to get around this? Is so, what?
Thanks

Please use StackOverflow for questions/help, where a wider community will be able to help you. We reserve the issues tracker for issues only. Thank you!


chore: move embark_listener to module

Rebased it once more and tested/linted locally, will merge once green!
Again, great work @andremedeiros !


Parse Unity 2019.2 asset bundles

Fixes #93
At some point between Unity 2018.1 and 2019.2, the asset bundle format changed. In particular, type tree nodes grew from 24 bytes each to 32 bytes. This PR implements that change.

Any updates on this?

I don't know the reasons, but I think this repo is on pause right now. I have been pulling some changes into a forked branch, but haven't really committed to maintaining a fork.


Default home_team and away_team to null

This is essential. When the teams of a match are not given yet, and no explanatory string has been given either: don't return an invalid id, but return null.

@lsv are there any updates on this? Would love to use this data source! <3

Nope, because its needed to show which match the winners are coming from.
If the number is above the number of teams, then its not a team, but a match id

Got it!


Add tests for DPO for VLM

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Update cast-and-convert-transact-sql.md

Fix example result set column names

@ytrikoz : Thanks for your contribution! The author, @edmacauley, has been notified to review your proposed change.

Thank you @ytrikoz for catching those mistakes and your edit.  #sign-off


Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)

I'm getting "Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)" fairly consistently (but not always). The stack looks like this:
PID 2874 received SIGSEGV for address: 0x369
0   segfault-handler.node               0x0000000104400fe8 _ZL16segfault_handleriP9__siginfoPv + 280
1   libsystem_platform.dylib            0x00007fff8497e52a _sigtramp + 26
2   libzmq.5.dylib                      0x00000001046bb3df _ZN3zmq5ctx_t13max_socket_idE + 6419
3   libzmq.5.dylib                      0x00000001046a7402 zmq_socket_monitor + 32
4   zmq.node                            0x000000010465ddab _ZN3zmq6Socket9UnmonitorEv + 39
5   zmq.node                            0x000000010465dd23 _ZN3zmq6Socket18UV_MonitorCallbackEP10uv_timer_si + 459
6   node                                0x000000010079697d uv__run_timers + 38
7   node                                0x000000010078c2be uv_run + 580
8   node                                0x0000000100667fbb _ZN4node5StartEiPPc + 583
9   node                                0x0000000100000a34 start + 52
10  ???                                 0x0000000000000004 0x0 + 4
This is with:
Node.js: 4.4.7
libzmq: 4.1.5
zmq: 2.15.3
On a Mac running 10.10+

I seemed to get this because I'm not calling .unmonitor() before .close().
Might I suggest in the .close function we call .unmonitor()?

This appears related to issue #540 . You can close as a duplicate.


Add font size for Apple Terminal

This PR extends the Apple Terminal AppleScript call that fetches the font. It now fetches the font size and appends it to the font name to make the output match other terminals.

Works!

Thank you for your contribution!
This PR is merged into hyfetch since this repo (dylanaraps/neofetch) seems no longer maintained.
HyFetch is a fork of neofetch with LGBTQ pride flags, but the repo also maintains an updated version of the original neofetch, addressing many pull requests that are not merged in the original repo.
Read the "Running Updated Original Neofetch" section for more info!


Upgrade to the latest 5.0 version of rails (5.0.4)

Should just be a minor release and not break anything.

Coverage remained the same at 97.643% when pulling 549541896b4d3d78a17e2afcbe8c8ad80a5b472f on upgradeRailsVersion into d13fdd329aadaae688587c41d5c719f0ce217394 on master.


feat - add new functionality for GitHub integration enhancements

📑 Description
feat - add new functionality for GitHub integration enhancements
✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed

☢ Does this introduce a breaking change?

[ ] Yes
[x] No

Summary by Sourcery
New Features:

Add support for updating GitHub workflows via API.

Description by Korbit AI
What change is being made?
Enhance GitHub integration by introducing constants for standardized text labels, adding the ability to make PUT requests, and updating method return types to support mixed return types.
Why are these changes being made?
These enhancements improve code maintainability and readability by using constants for frequently used labels, ensure feature completeness by supporting PUT HTTP methods, and allow for flexible method return types aligning with modern PHP practices. These modifications are aimed at better aligning with coding standards and enhancing functionality.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Summary by CodeRabbit

New Features

Added support for handling PUT HTTP requests in the Webhooks functionality.
Introduced a new method for updating workflows.

Improvements

Enhanced clarity and maintainability by replacing hardcoded strings with named constants in the GitHub class.
Updated method signatures for improved type safety across multiple methods.

Bug Fixes

Corrected the handling of sequences in the requestRerun method.

Style

Minor formatting adjustments for cleaner code presentation.

@gstraccini codacy bypass


xJea tests are failing

see https://github.com/PowerShell/xJea/pull/6

Tests are now passing https://ci.appveyor.com/project/PowerShell/xjea/build/<IP_ADDRESS>


Change the discovery url to point to production env

Change the default discovery url to point to the production environment.

@mandaryadav It seems to me that the test environment is the correct default. The majority of cases where the tool will be used will be as part of testing prior to going to production. Why change the default? Also the test environment will always have the latest validator tests, while production will lag behind.

@tylerbutler I observed that one of the 3rd party host (Edmodo) was running the validation application from the production environment so to keep it consistent, I proposed this change to make it as a default. If that is not necessary then feel free to discard my change.

@mandaryadav Yeah, I think the test environment is the right default. Most partners won't even have access to prod when using this initially, so they'd just get errors by default. I'll go ahead and close this PR.

Thanks @tylerbutler


Index out of bounds control.lua 492

 The error wad that it had an invalid index getting the cluster inventory
 In mod or slave?
 control.lia line 492. Visoble on both in- game and slave
 Index out of bounds
 clusterio.recieveInventory: Index out of bounds 101
@justarandomgeek length of http://<IP_ADDRESS>:8080/api/inventory is 6538. Is this the long rcon command  problem?

The issue (as suggested by 101 being the value it failed at) was that the Inventory Combinator only had 100 slots. I upped it to 500, which shoudl be suitable for any sensible modset.

Also, RCON must have handled this large message properly, to have made it far enough to hit that, which is nice!

Cool. Maybe the 4k char limit isn't such a big problem after all?


[ci skip] Test Kokkos Release candidate

@masterleinad Can you briefly explain the purpose of this PR, and how you plan to move forward with it/what remains to be done?

I just wanted to trigger the GitLab CI to compare performance changes between Kokkos releases making sure there weren't any for the upcoming release. I would keep it open until the Kokkos 3.6 is finalized in case we want to retest.