Subscription issues with vernemq

Hello,
First thanks for the great library!
I have some rather strange issues with vernemq 1.8.0. From time to time it looks like tortoise (v0.9.4) is loosing its subscriptions. I can't reproduce this issue locally but I suspect that this happens after vernemq is restarted. The Connection is still present (can see the process in the Tortoise.Supervisor) but somehow the client is not receiving any messages.
I also noticed that the handler's subscription/2 callback is not called if the client reconnects to vernemq. I created a small repository to illustrate the issue: https://github.com/kbredemeier/tortoise_sub_error
The repository comes with a docker-compose.yml that starts a vernemq and a mosquitto container. The application starts a client for each broker and logs the subscription events. If I stop the containers and start them again I can see the subscription logs for mosquitto but not for vernemq. The subscriptions are still active though.
I tested the same thing with a much older version of vernemq (1.3.0) and got a slightly different behaviour. On the first reconnect I get the expected log messages but on any subsequent disconnect the subscription/2 callback is not invoked anymore.
Maybe the two issues are related to each other. I would love to jump in to fix this issue but don't have an idea where to start. Help with this would be much appreciated :)

I know I keep saying this; but I hope I can make some time to focus on the MQTT 5 branch of this project, such that I can get it out the gates—that branch will make significant changes to the structure of the code, so merging changes from master into the mqtt5 branch is kinda troublesome.
I hope you understand. I don't know if it is a Tortoise issue, or a VerneMQ issue. Hopefully I will make a deep dive into tortoise and get it updated, and communicate the architecture such that more people than me can work on it :)


PubSub Routing docs

Documentation for PubSub Routing dapr/dapr#2582

[x] Read the contribution guide
[ ] Commands include options for Linux, MacOS, and Windows within codetabs
[x] New file and folder names are globally unique
[x] Page references use shortcodes instead of markdown or URL links
[ ] Images use HTML style and have alternative text
[x] Places where multiple code/command options are given have codetabs

Issue reference
Closes #1746

@pkedy - can you add a .NET example to this, since this is obviously missing.


Alpha » Components Pane

Parent: #409
Legend
⬜ - Hasn't started
📝 - In Progress
✅ - Complete
TODOS

Issue
PR
Feature
Progress

????
????
ability to DND any native or custom element to the stage
⬜

????
????
pre-built elements such as H1, H2, H3, div -- separated by "custom" and "native"
⬜

????
????
dropping elements to the stage should create a new preview window of it
✅

????
????
Should be able to create a new component from the panel (+ button in upper corner)
⬜

????
????
Should be able to delete a component in the right panel (needs to warn of assoc. files)
⬜

????
????
Hooks to allow extensions to register custom components (dropping behavior may be different depending on component type -- react for instance may need to also import a new file)
⬜

????
????
API back-end for returning available components
✅

Quick update on this:


Improve typing of core

Description
This PR adds some additional types to core. Mainly focused on command and app modules.
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )

Reviewer Checklist:

[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Code is clear and sufficiently documented
[ ] Correct, in line with design

Processing this pull request

Failed to merge changes into iso4 due to merge conflict.Please open a pull request for these branches separately.

Merged into branches master in 3a12628834f77693c6ad82daab6be2420a637501

Not closing this pull request due to previously commented issues for some of the destination branches. Please open a separate pull request for those branches. You can safely close this pull request and delete the source branch.


Updates DTLSv1_listen to support DTLS 1.3 client hellos.

Checklist

[x] documentation is added or updated

The CI is relevant.

This PR is waiting for the creator to make requested changes but it has not been updated for 30 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

Looks like this needs a rebase

This PR is waiting for the creator to make requested changes but it has not been updated for 30 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR is waiting for the creator to make requested changes but it has not been updated for 61 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

@andrewkdinh Thanks for your comments. This is still a draft and is missing a rebase etc. I will reply to your comments when I have time to work on this part of the DTLS 1.3 feature.

This PR is waiting for the creator to make requested changes but it has not been updated for 30 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR is waiting for the creator to make requested changes but it has not been updated for 61 days.  If you have made changes or commented to the reviewer please make sure you re-request a review (see icon in the 'reviewers' section).

This PR has been closed.  It was waiting for the creator to make requested changes but it has not been updated for 90 days.


faction-manager.js: Problem with rep cost estimates

Sometimes when i let the scripts run by themselves, i go back to them having quit to an error message that goes like this (copied from this line):
(CONTEXT: BN: 4.3, SF:1.1, 4.2)
ERROR: We were only able to purchase ${purchased} of our ${purchaseableAugs.length} augmentations. Expected cost was ${getCostString(totalAugCost, totalRepCost)}. Player money was ${formatMoney(playerData.money)} right before purchase, is now ${formatMoney(await getNsDataThroughFile(ns, 'ns.getPlayer().money'))}
So I looked into it and it seems like there is a problem when estimating the money needed to buy the rep for an aug because the expected cost is always smaller than the player money and when the script uses the estimated money to buy the rep, it ends up with less rep than needed to get the wanted aug.

If this ever happens again, it would be helpful to:

Copy+Paste or take a screenshot of the actual error message you received so that I can see what the values of the variables ${purchased}, ${purchaseableAugs.length}, playerData.money etc... were when you encountered the error (I put them in the log message because they are essential to figuring out which of a myriad of possible things went wrong)
Export and attach your save file so that I can load it up, reproduce, and step through the code

As is, I can't reproduce the conditions that let to the error you saw, or even begin to speculate on what scenario you encountered. The only thing I can think is that perhaps you were playing the unstable DEV version of the game (e.g. https://bitburner-official.github.io/bitburner-src/) which sometimes contains temporary backwards-incompatible changes to the game logic, and which don't always make their way to the main (steam-release) branch.

Oh... that's because the function to get bitnode multipliers in a script is only available once you have SF-5.1... otherwise it calculates as if faction rep multiplier is 1... which it isn't in bitnode 4.

Oh... that's because the function to get bitnode multipliers in a script is only available once you have SF-5.1... otherwise it calculates as if faction rep multiplier is 1... which it isn't in bitnode 4.

That is a phenomenal catch! Next time I'm working on this I'll hard-code this bitnode mult into faction manager as a fall-back for when SF5.1 isn't available

Hiya;
I've been having this same issues sporadically and I've come to realize it's because managePurchaseableAugs generally makes our list of purchasables without NFG; then adds in NFG after. The issue itself comes from it adding NFG after all augs cheaper than first NF, but before all augs more expensive than first NF. So, as a result, if we happen to be able to afford 4 augs more expensive than NF, and then be able to afford a few levels of NF, adding the NF aug purchases with change the actual purchase price of the more expensive augs (making them even more expensive) due to the aug purchase count multiplier.
If we simply push the NF purchases (instead of splicing them in) then the total costs should remain as expected and we shouldn't have any issues in the purchase logic (unless there's some other logical reason for not doing this).

I wouldn't think it's sorting them from cheapest to most expensive as you're suspecting - it should be the exact opposite - its always optimal to buy augs in order of most expensive, to cheapest.
The algorithm gets more complex when you consider that some augs depend on one another, so the code performs a greedy sort to determine what purchase order minimizes the cost.
I'm more inclined to think the first theory above is accurate - if the game alters the formula for the cost of reputation (which it does in some bitnodes), but you don't have SF5.1 to get the multipliers, then we're now no longer buying sufficient rep, then that would explain why all the subsequent purchase attempts failed.
If the rep is only off by a small amount (e.g. <1) due to a rounding error, then that could explain why some report that the initial run fails to buy anything, but then re-running works (because in that time their rep with that faction might have creeped ever so slightly higher)

Current formula for getting the required donation amount (in dollars) for reputation with a faction is:
let getReqDonationForRep = (rep, faction) => Math.ceil(1e6 * (Math.max(0, rep - (faction.name ? faction : factionData[faction]).reputation)) / (playerData.mults.faction_rep));

It already takes into account the faction rep multiplier you have with a given faction (e.g. from a prior install), which acts as a divisor to the cost of buying additional reputation. My guess is that value 1e6 is what is changing in BN 4 and needs to be adjusted based on some currently unknown bitnode multiplier.

Hmm, I don't think this has always been here:

Looks like the "FactionWorkRepGain" mult (which affects how fast you gain reputation from working) also affects how much reputation you get from donating.
This was added Oct 7, 2022: https://github.com/bitburner-official/bitburner-src/commit/ea39e5e1f46ddc962a7463d09b646d80ca83a059
which is probably around the time I stopped agressively playing / maintaining these scripts.


Add the ability to reset modal on close

Add the ability to reset the modal to step 1 when it is closed instead of saving its position
Add attribute of "reset on close" to the form to enable reseting

@dacoolman thanks for the PR!
I would prefer it to be "true" rather than "1", what do you think?
Also the comparison should be using === instead of ==, and spaces around the comparison, i.e.
a === b // yes
a===b // no

Sure thing, should be updated now

@dacoolman 2 last nit:

use single quotes instead of double quotes for 'true'
x === 'true' ? true : false is the same as x === 'true', so we can remove the ternary check
Thanks for your PR! :D

Oh right, okay should be updated

Looks great! Thanks :)


[Bug]: Wrong cd directory in readme

What happened?
cd sudo-flix should be cd smov in:
git clone https://github.com/sussy-code/smov.git
cd sudo-flix
git pull
pnpm install
pnpm run dev

What browsers are you seeing the problem on?
No response
Steps to reproduce?
No response
Other relevant information
No response

ight ill do it when i make a change next


support AppleClang with openmp

support #1492
Tested with cmake 3.12.0-rc3 and Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>) and libomp 5.0.1.
Howto

install libomp by homebrew brew install libomp
download and install cmake >= 3.12.0
mkdir build ; cd build
cmake ..
make -j

Oh, I didn't know AppleClang has supported OpenMP. Good news!

master fails too :(
Travis problems... again!

There is something wrong with OSX on Travis.
I don't know why curl: (3) <url> malformed happns.
==> Installing gcc
==> Downloading /bottles/gcc-8.1.0.sierra.bottle.1.tar.gz
curl: (3) <url> malformed
Error: Failed to download resource "gcc"
Download failed: /bottles/gcc-8.1.0.sierra.bottle.1.tar.gz
Warning: Bottle installation failed: building from source.
==> Downloading https://ftp.gnu.org/gnu/gcc/gcc-8.1.0/gcc-8.1.0.tar.xz
==> ../configure --build=x86_64-apple-darwin16.7.0 --prefix=/usr/local/Cellar/gc
==> make

Travis seems to be OK now. Rerunning builds.

@levyfan , could you also help on the updating of Documents?

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

I think we could update the Travis for macOS to using AppleClang as default. I' ll create related PR

@chivee

I think we could update the Travis for macOS to using AppleClang as default. I' ll create related PR

Don't it too early, because CMake 3.12 is Release Candidate now? I mean, can we install it via brew at Travis?

@StrikerRUS we could use brew --HEAD for cmake now and switch to stable release later.

Why is VSTS: LightGBM-Build-CI still running? 😿

refer to https://github.com/Microsoft/vsts-agent/issues/1699#issuecomment-404731721


Stream functionality breaks on Web after 1.9.7

Issue
The chat completion stream on Flutter Web platforms returns all chat at the end. Not streaming.
Using version dart_openai: ^2.0.1
See video:
flutter_chatgpt_notok.webm
Note: Works ok on other platforms (Tested on Windows)
Expected
After downgrading & pinned the package version: dart_openai: 1.9.7. The stream completion on web works normally again.
See video:
flutter_chatgpt_ok.webm
Code snippet
Stream<OpenAIStreamChatCompletionModel> chatStream =
    OpenAI.instance.chat.createStream(
  model: "gpt-3.5-turbo",
  messages: _aiMessages,
);

chatStream.listen((chatStreamEvent) {
  debugPrint(chatStreamEvent.toString());
  // existing id: just update to the same text bubble
  if (chatResponseId == chatStreamEvent.id) {
    chatResponseContent +=
        chatStreamEvent.choices.first.delta.content ?? '';

    _addMessageStream(chatResponseContent);

    if (chatStreamEvent.choices.first.finishReason == "stop") {
      isAiTyping = false;
      _aiMessages.add(OpenAIChatCompletionChoiceMessageModel(
        content: chatResponseContent,
        role: OpenAIChatMessageRole.assistant,
      ));
      chatResponseId = '';
      chatResponseContent = '';
    }
  } else {
    // new id: create new text bubble
    chatResponseId = chatStreamEvent.id;
    chatResponseContent = chatStreamEvent.choices.first.delta.content ?? '';
    onMessageReceived(id: chatResponseId, message: chatResponseContent);
    isAiTyping = true;
  }
});

Full code: https://github.com/iqfareez/flutter_chatgpt/blob/main/lib/screens/chat_page.dart

Thank you for figuring this out @iqfareez, downgrade to 1.9.7 fixed streaming on web for me

Did this get fixed? I am using 3.0.0 and Streams don't work on the web.

I don't think it was fixed. I've stayed on 1.9.7 for anything web


FIX: gradle install error caused by javadoc generate.

execute ./gradlew install in Java8
falled in JavaDoc task. like:
spring-webflow/src/main/java/org/springframework/webflow/engine/Transition.java:32: 错误: 意外的结束标记:

state, called the target state. A transition may become eligible for execution on the occurrence of an

Fix all javadoc error. and execute "./gradlew install" correctly.

These have been addressed since.


chore: refactor build grpc

add grpc-tools to dev dep
use submodule to import layotto
mv pb to proto/runtime/v1 prepare for proto/extension/v1

What this PR does:
Which issue(s) this PR fixes:

Special notes for your reviewer:
Does this PR introduce a user-facing change?:

本地已跑过。
@seeflood 后续帮忙看看 github action 的 layotto 如何起来。

1.0.0


cargo-make: fix shell completion

Description
cargo-make includes bash completions in the repository. I'm loading it for both bash and zsh.
https://github.com/sagiegurari/cargo-make/issues/565
Note

the completion targets makers, cargo-make make style looks not supported.
I've referred to https://github.com/NixOS/nixpkgs/pull/332696 for this change.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349104

x86_64-linux

:white_check_mark: 2 packages built:

cargo-make
open-scq30

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 349104

x86_64-darwin

:white_check_mark: 2 packages built:

cargo-make
open-scq30

Thank you!


Feature request: allow user to copy the path from the top panel

the top panel I mean is this:

for example, if I right click on build, I will get ZoeDepth/zoedepth/models/base_models/midas.py/class MiDasCore/def build in the clipboard, if I right click on MidasCore, I will get ZoeDepth/zoedepth/models/base_models/midas.py/class MiDasCore and so on.
By the way, if user can copy the link of code (e.g. ZoeDepth/zoedepth/models/base_models/midas.py#L333), just like on the web, it will be very great !

/duplicate of #58678

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


The cluster's authentication mode must be set to one of [API, API_AND_CONFIG_MAP] to perform this operation.

I encountered an issue where the authentication_mode parameter is missing in version 18.29.1 of the terraform-aws-modules/eks/aws module, which is required for the aws_eks_access_entry module. Consequently, this results in the following error:
Error: creating EKS Access Entry (cluster:arn:aws:iam::339712812380:role/Karpenter-cluster-20240207214924404600000005): operation error EKS: CreateAccessEntry, https response error StatusCode: 400, RequestID: 4fa53b84-263c-40f9-97b1-ea8871cc8652, InvalidRequestException: The cluster's authentication mode must be set to one of [API, API_AND_CONFIG_MAP] to perform this operation.
I removed the local .terraform directory (only applicable if state is stored remotely): rm -rf .terraform/
I re-initialized the project root to pull down modules: terraform init
i re-attempted the terraform plan or terraform apply commands to check if the issue persists.
Terraform v1.6.6
on linux_amd64

provider registry.terraform.io/gavinbunney/kubectl v1.14.0
provider registry.terraform.io/hashicorp/aws v5.35.0
provider registry.terraform.io/hashicorp/cloudinit v2.3.3
provider registry.terraform.io/hashicorp/helm v2.12.1
provider registry.terraform.io/hashicorp/kubernetes v2.25.2
provider registry.terraform.io/hashicorp/local v2.4.1
provider registry.terraform.io/hashicorp/null v3.2.2
provider registry.terraform.io/hashicorp/random v3.6.0
provider registry.terraform.io/hashicorp/time v0.10.0
provider registry.terraform.io/hashicorp/tls v4.0.5
provider registry.terraform.io/terraform-aws-modules/http v2.4.1

module "eks_blueprints" {
  source = "github.com/aws-ia/terraform-aws-eks-blueprints?ref=v4.32.1"

  cluster_name    = "${var.project}-cluster"
  cluster_version = "1.29"
  enable_irsa     = true

  map_roles = [
    {
      rolearn  = module.karpenter.iam_role_arn
      username = "system:node:{{EC2PrivateDNSName}}"
      groups = [
        "system:bootstrappers",
        "system:nodes",
      ]
    },
  ]

  vpc_id = module.vpc.vpc_id

  private_subnet_ids = module.vpc.private_subnets

  managed_node_groups = {
    role = {
      capacity_type   = "SPOT"
      node_group_name = "general"
      instance_types  = ["t3a.medium"]
      desired_size    = "1"
      max_size        = "3"
      min_size        = "1"
    }
  }
}

module "karpenter" {
  source  = "terraform-aws-modules/eks/aws//modules/karpenter"
  version = "20.2.0"

  cluster_name           = module.eks_blueprints.eks_cluster_id
  irsa_oidc_provider_arn = module.eks_blueprints.eks_oidc_provider_arn
  enable_irsa            = true
}

Expected behavior
I expected the authentication_mode parameter to be available with a default value in the terraform-aws-modules/eks/aws module, allowing the aws_eks_access_entry module to function properly without encountering the authentication mode error.
Actual behavior
The authentication_mode parameter is missing from version 18.29.1 of the terraform-aws-modules/eks/aws module, causing the aws_eks_access_entry module to fail with the authentication mode error described above.

Additional context
I believe adding the authentication_mode parameter with a default value in the terraform-aws-eks-blueprint variables would resolve this issue. And or upgrade terraform-aws-modules/eks/aws 18.29.1 version.
variable "authentication_mode" {
  description = "The authentication mode for the cluster. Valid values are `CONFIG_MAP`, `API` or `API_AND_CONFIG_MAP`"
  type        = string
  default     = "API_AND_CONFIG_MAP"
}

I'm not sure how to answer this one - in short, authentication_mode is not supported in v18, or even v19, of the EKS module - it was only recently added in v20 of the EKS module.
this is not going to be supported in the outdated v4.x of EKS Blueprints module since that version is no longer active


⚠ Dovecot has degraded performance

In fad9be6, Dovecot (https://en.wikipedia.org) experienced degraded performance:

HTTP code: 503
Response time: 3264 ms

Resolved: Dovecot performance has improved in 7d6a77c after 5 minutes.


🐛 | GenericMultiTimerClass broken

Version
3.5.3
Branch
future3/main
OS
Ubuntu 22.04
Pi model
Laptop
Hardware
No response
What happened?

https://github.com/MiczFlor/RPi-Jukebox-RFID/blob/future3/main/src/jukebox/jukebox/multitimer.py#L249
self._function = self._callee(*self.class_args, iterations=self._iterations, **self.class_kwargs)
Instead of modifying the function call, so that later self._function can be called, here the function is already executed. In my case the return value is None, so later the MultiTimer tries to call none, and the program dies.

GenericMultiTimerClass tries to pass on the paramter iterations and MultiTimer tries to pass on parameter iteration so this clashes.

Should those two classes be interchangeable or not? I don't get the value of GenericMultiTimerClass because you can build just the same thing by simply using the MultiTimer so my cleanest suggestion is: Delete the GenericMultiTimerClass.
If you want to keep it, here's a fix that worked for me, on first shallow testing:
FixGenericMultiTimerClass.patch.txt
Logs
No response
Configuration
No response
More info
No response

Also see typos:
typo.patch.txt

Thanks @hoelger .. In which situation did you call this function. I am currently working on a PR to implement the timers. You did not use this branch, correct? #2332

Ah sorry, yes, I did not thoroughly look through other issues nor the state on development. So no, I just used this functionality on top of future3/main.

I wanted to come back to this again. Thank you for the detailed review initially!
Thank you for the detailed review! You've identified two significant issues:

You're absolutely right about the function execution bug. The current code executes self._callee immediately instead of storing it for later execution. I will fix this here #2332
The naming conflict between iterations/iteration parameters is indeed problematic and could cause unexpected behavior.

I disagree with your assessment about GenericMultiTimerClass as we will need this in same specific use cases (e.g. IdleShutdown). I will add some documentation to explain the classes better


Camera models

delete space "simple_pinhole": CameraModel.SIMPLE_PINHOLE,

Actually I'm not familiar with PR, so I did it in a strange way! Thank you for the comment.

@wooni-github no problem.


Fix for 664, open for comment.

This is a fix for #664 where the window will sometimes be null on doing a reload.
I am not quite sure it fixes the issue wholesale but it does prevent the javascript error from occurring.  More testing is required and this PR is here for evaluation and comment.  I do not consider it complete at this time.

note I missreferenced one of the commits.  it was meant to reference #664


Thymeleaf 3: Is loading the template four times really necessary?

I'm checking the Snapshot implementation currently, and I found out that the template which is used to decorate (surround) our main template is loaded four times:
one time completely at DecoratorProcessor 92
one time with selector head at HtmlDocumentDecorator 72
one time with selector body at HtmlDocumentDecorator 101
one time with selector html at HtmlDocumentDecorator 119
Do you think that's necessary? Couldn't you clone the one loaded the first time? We have remote access for the template, so we have a little performance issue with that.

Thanks for raising this.  Yes, the loading of a template 4 times seems totally unnecessary.  At the time I added those, it was the only way I knew how to obtain a model from a template, not knowing that it went through all of these Thymeleaf methods to reach out to template resolvers and everything else.
Now, I know how to pull out a sub-model from a model, so I'll be adding that to those areas you've identified.
Just curious: since this goes through the template manager, does turning the Thymeleaf cache on stop the excessing loading?

Each of those four loadings is done with a different cache key since the selector is included in the cache key. Of course, when the page is loaded the second time with caching turned on, we already have those four entries in the cache.
Thanks for adressing this! Tell me when it's included in the latest snapshot :-)

Didn't think I'd be working on this for a while, but I came across something else that needed changes to how I fetch models, and so this got updated in the process 😁
I've just uploaded another 2.0.0-SNAPSHOT, so that should reduce the crazy amount of template fetching that you found.

Thanks! Now I got https://github.com/ultraq/thymeleaf-layout-dialect/issues/110 ;-)


Clarify that test folder is a sibling to the src folder

The previous rendering of the tree made it look like the test folder was a sibling to the unit-testing-using-dotnet-test folder.

Hi @breyed, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). 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, DNFBOT;

Thanks @breyed! LGTM! I'll :shipit:


MacOs - no window

Describe the bug
I created a test project.
The application runs successfully in Windows and glitches in macOS.
A screenshot is attached. It shows that the shell is running but there is no window.
I am attaching the logs. Something about  UnhandledPromiseRejectionWarning: Unhandled promise rejection.
This bug is only in macOS.
To Reproduce
Steps to reproduce the behavior:

Copy folder https://github.com/RouR/learn-project/tree/master/electron
npm install
npm run electron:start

Expected behavior

A window should appear.
The behavior should be the same as in Windows

Desktop:
OS: MacOs BigSur 11.1
@capacitor-community/electron: "3.0.1",
@capacitor/core: "3.1.1",
web-app created with "@ionic/angular": "6.0.0-dev.202106282030.9f3b1d9"
Terminal logs
user@mac electron % ls
app			capacitor.config.json	package-lock.json	tsconfig.json
assets			dist			package.json		user-preload-script.js
build			node_modules		src
user@mac electron % npm install
npm WARN<EMAIL_ADDRESS>No repository field.

audited 449 packages in 11.386s

43 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

user@mac electron % npm run electron:start

><EMAIL_ADDRESS>electron:start /Users/user/Desktop/electron
> npm run build && electron ./

><EMAIL_ADDRESS>build /Users/user/Desktop/electron
> tsc

(node:7538) UnhandledPromiseRejectionWarning: Error: Cannot find module<EMAIL_ADDRESS>Require stack:
-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>- /Users/user/Desktop/electron/build/src/index.js
- /Users/user/Desktop/electron/node_modules/electron/dist/Electron.app/Contents/Resources/default_app.asar/main.js
-
    at Module._resolveFilename (internal/modules/cjs/loader.js:887:15)
    at Function.n._resolveFilename (electron/js2c/browser_init.js:257:1128)
    at Module._load (internal/modules/cjs/loader.js:732:27)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Module.require (internal/modules/cjs/loader.js:959:19)
    at require (internal/modules/cjs/helpers.js:88:18)
    at Object.setupCapacitorElectronPlugins (/Users/user/Desktop/electron/node_modules/@capacitor-community/electron/dist/core/util.js:93:21)
    at ElectronCapacitorApp.init (/Users/user/Desktop/electron/build/src/index.js:102:20)
    at /Users/user/Desktop/electron/build/src/index.js:127:26
(Use `Electron --trace-warnings ...` to show where the warning was created)
(node:7538) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 2)
(node:7538) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

I guess the reason is  https://github.com/capacitor-community/electron/blob/4ce5fbc49cda07bccbd6f3941c3a489d574219d2/src/electron/util.ts#L90
File<EMAIL_ADDRESS>does not exist

Workaround: npx cap update electron
https://github.com/capacitor-community/electron/issues/110
but, it must work out-of-box

By your example repo I don't see the web app containing the electron folder.
Normally the flow of creating a project would be to:

Create web app project via ionic cli, create react app, vue cli, etc.
Add the Capacitor core and cli as shown in the capacitor docs
Build the project via npm run build or similar
Follow the getting started guide in the electron platform docs

If I've missed something in your demo repo then please reopen an issue but I can not reproduce this on the same version of macOS when I go through the process above.


adds language-family-membership file

We have to merge this pr first: https://github.com/paketo-buildpacks/tini/pull/13


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1044

Questions ?
Please refer to the Backport tool documentation


Read FNC1 character GS1-128

It would be very interesting to enable the reading of GS1-128. It would be enough to read the characters FNC1
IMAGE: https://ibb.co/ZdJpprN
Required to read variable length GS1-128 identifiers
Thanks

I rectify. Reading does capture FNC1 characters. That you will not see in the string. But you will be able to detect it when you go one by one through the characters of the captured text.


Include .git when copying into the Docker build image

Type of change

Bugfix

Description
We now use a maven plugin to extract information about the git commit we are building against, so we need the .git dir included when we run maven.
Additional Context
The docker build action failed during ./mvnw -B clean verify -Pdist,withAdditionalFilters -Dquick -pl :kroxylicious-bom,:kroxylicious-app -am" with:
Error: ERROR] Failed to execute goal io.github.git-commit-id:git-commit-id-maven-plugin:6.0.0:revision (get-the-git-infos) on project kroxylicious-app: .git directory is not found! Please specify a valid [dotGitDirectory] in your pom.xml -> [Help 1]

Checklist
Please go through this checklist and make sure all applicable tasks have been done

[ ] Write tests
[ ] Make sure all tests pass
[ ] Review performance test results. Ensure that any degradations to performance numbers are understood and justified.
[ ] Make sure all Sonar-Lint warnings are addressed or are justifiably ignored.
[ ] Update documentation
[ ] Reference relevant issue(s) and close them after merging
[ ] For user facing changes, update CHANGELOG.md (remember to include changes affecting the API of the test artefacts too).

This fixed the problem locally but we don't have a way to trigger the docker workflow for non-main to test

Yeah it adds a little time to copy it in, 22M currently but it'll grow.
I think we have bigger problems like doing the maven build from scratch every time. Going to be a pain for local dev.

I've got a run to fire , had to push the branch to origin so we can do a manual workflow run targeting the branch. I didn't see PRs in the list of things you can run against.


Rebase upstream

Thanks to @0x7FF.
Fixes support for Apple Sillicon (running MacOS).

has this been tested on both macos & linux?

I don't have linux; I can check now on macOS.

 Feel free to ignore it, but definitely do not squash it together with the rest.

i cant ignore it when merging; please revert it.

Sorry.


Pattern final fixes

@r2dedios Documentation is done for now, and linting works. I still need to fix the failing unit tests. We can merge this PR regardless to get the documentation into the main branch. I can fix the tests using a different PR. What do you say?


FindSFML: Missing dependency and wrong library path

Two fixes for FindSFML.cmake:

Environment variables (e.g. PATH on Windows) are looked up early by CMake, and libraries found there are prioritized over those in SFML's own folder. To avoid paths to wrong libraries, this commit modifies the CMake find_library() call, such that environment variables are no longer considered as suitable paths.
See also: http://en.sfml-dev.org/forums/index.php?topic=17774

The dependency libvorbisenc is not listed in FindSFML. Thus, libraries using SFML get an incomplete SFML_DEPENDENCIES variable in CMake, leading to linker errors.

Could NO_SYSTEM_ENVIRONMENT_PATH have a negative effect on OS X ( @mantognini ) or Linux?

I guess it should be fine. At least Jenkins didn't complain.

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.

Isn't it possible to just add
find_library(${output} NAMES ${ARGN} PATHS ${FIND_SFML_PATHS} PATH_SUFFIXES lib NO_SYSTEM_ENVIRONMENT_PATH)

before the original line without removing it? According to the documentation calling find_library multiple times in succession is perfectly normal and is just a convoluted way of specifying a precedence order for which there is no mechanism by just calling the function a single time. This would also guarantee not breaking anything that currently works as well, because the original line remains as a fallback if the first call doesn't find anything.

That's what I suggested at the end of this post. The question is whether it's necessary? Do we know systems that take library paths from environment variables?
In my opinion, we should avoid unneeded complexity in the configuration files... The CMake scripts are already a massive beast, at one point no one will understand them anymore :wink:

Do we know systems that take library paths from environment variables?

That's the question. Even if we don't know, it might still break somewhere. :wink:

The question is whether it's necessary? Do we know systems that take library paths from environment variables?

Since NO_SYSTEM_ENVIRONMENT_PATH only affects reading from PATH and LIB, we shouldn't generalize the question to all environment variables. The fact that we don't need to read from environment variables in the typical case doesn't mean we go breaking it for those hackers that do make use of this.
PATH isn't/shouldn't really be used to specify library search directories so I don't mind breaking this. However, we mustn't forget that even if we suppressed reading from PATH, during execution, the dynamic linker on Windows will try to resolve any imports using a certain directory search order. It begins in the current directory and at some point resorts to searching PATH. If we forbade CMake from searching PATH, we could link to a library that might not be in PATH although the PATH library would be the one that is loaded when the application is run. I wouldn't want to be the one who has to find out why applications start to mysteriously crash in this case. It works out most of the time, but I'm sure this was part of the rationale behind why CMake chooses to search PATH before anything else.
As for LIB, this is pretty much the standard environment variable where some Visual Studio users might append whatever library they intend on using. The idea is that you download some library with an installer, it installs its .dll files somewhere, adds that directory to PATH and modifies LIB to add the directory where the import libraries are located. This way, the installer can basically take care of everything for the developer automatically (isn't this the reason why people still use VS? :wink:). Excluding or de-prioritising LIB would break this for those people. They might have installed their own freetype or jpeg somewhere using some sort of installer, and intend to use that instead because it might have some new features. We're going to break this by de-prioritising LIB. MinGW went the other route and (AFAIK) hardcoded their search paths into the compiler/linker so they wouldn't have to make use of environment variables. This means that if you wanted a "fully automated" installer for some MinGW compatible library, it would have no choice than to dump them into the directories where the system headers/libraries are located. More messy, but no environment variables.
I remember the times when I still used my own version of GLEW (since it was newer than SFML's one) and it became really really annoying to always have to manually edit the CMake entries just because SFML insisted its own version is the one I want to use when building. If we can save less persistent users the time of having to do this, then I don't see why we shouldn't try to do so.
TL;DR: You are eventually going to break building for someone by assuming reading from PATH and/or LIB is never necessary or even intentional. The original problem was that the search order was wrong for us. We solve that problem by searching our own paths first, then the environment paths if that failed. This might actually already be enough to "break" behaviour for some users on Windows, since they would be using a different jpeg in SFML and in their own code, so I don't know what to think of this. If you ask me, the "perfect" solution would be to search for .lib files first, no matter where they come from and then .dll files after that. The system directories should be preferred over our own if they are compatible (this can be/is tested in the Find scripts themselves if we specify a version).

In my opinion, we should avoid unneeded complexity in the configuration files... The CMake scripts are already a massive beast, at one point no one will understand them anymore

This is more of a shortcoming of the CMake find_library command. If there was a way to specify the search order using a single command, we would probably do that, but there isn't. So, as stated in the documentation, we need to call it multiple times. I don't think applying something that is explicitly mentioned in the documentation constitutes making the scripts more complicated. If it ever gets improved one day, we could just combine them back into a single line again.

We solve that problem by searching our own paths first, then the environment paths if that failed.

Are LIB and PATH considered at all, even if we have a second find_library call? Since SFML ships the .lib files on Windows, they will always be found first -- unless the user manually removes them. So is the intended usage manual modification of SFML's installed lib directory?

This might actually already be enough to "break" behaviour for some users on Windows, since they would be using a different jpeg in SFML and in their own code, so I don't know what to think of this. If you ask me, the "perfect" solution would be to search for .lib files first, no matter where they come from and then .dll files after that. The system directories should be preferred over our own if they are compatible (this can be/is tested in the Find scripts themselves if we specify a version).

Can we really assume that the user uses the system's libjpeg version? The user might as well use SFML's (not so unlikely if his whole game is written around SFML), or yet another one in his own local folder... So chances that we break something are there either way.

Well, I guess since this only really affects the building of SFML, I guess we can live with that. If we ship every single dependency ourselves anyway and only rely on OpenGL and the system libraries, I don't think anything should go wrong even if we excluded more stuff from the search paths.

If we ship every single dependency ourselves anyway and only rely on OpenGL and the system libraries

That's not 100% true: when building SFML for package managers one should probably use the dependencies from that package manager. That's how homebrew work at least. (I guess it's always possible to use cmake variable to specify where deps should be taken from but that's not really convenient.)

That's the question. The way I see it NO_SYSTEM_ENVIRONMENT_PATH essentially means "don't let the user use their own installed libraries on Windows". I also find it misleading because other environment variables are still read and even used on other operating systems. This is just a Windows thing, and because Windows doesn't have a package manager it becomes more tedious for developers who know what they are doing and have to go and manually alter CMake entries at the end of the day.
As usual, it's all Windows' fault :grin:.

So what's the verdict?

I guess if it doesn't change anything for users who intentionally override SFML's supplied extlibs with their own, it shouldn't make a difference. People who count on PATH and LIB to override any of SFML's extlibs are doing something wrong to begin with anyway :grin:.

This PR has been added to my merge list, meaning it will be merged soon, unless someone raises any concerns.


CompareTo Generation fails if fields include underscores

Description
If a field in a proto message contains underscores in the name, the new CompareTo generation fails with two messages:

Exception while processing file: /Users/user/Projects/Hashgraph/Pbj/pbj-integration-tests/src/main/proto/comparable.proto
Cannot invoke "com.hedera.pbj.compiler.impl.Field.optionalValueType()" because "f" is null.

Note that no further information is given, so it is a particularly difficult error to track down.
Steps to reproduce

Modify pbj-integration-tests/src/main/proto/comparable.proto

change bytes bytes to bytes bytes_field on line 26
change , bytes", to , bytes_field" on line 18

run ./gradlew clean assemble -Dsonar.gradle.skipCompile=true --warning-mode=summary --no-daemon --no-build-cache --no-configuration-cache

The compilation will fail with the referenced error

Here is a patch that resolves this issue (and includes an abbreviated stack trace in compiler output for compiler exceptions).
fix-175-CompareTo-Generation-fails-if-fields-include.patch

@jsync-swirlds is your branch ready for review? Do you need someone to take it over?

I sent a draft PR.  I have bandwidth to complete the PR if that's easiest.


offline-dbg: make collect-sos-ovn check all db locations

The current sos report checks a number of potential locations
for ovnnb_db and ovnsb_db. collect-sos-ovn should check each
of these locations for db files.
Signed-off-by: Salvatore Daniele<EMAIL_ADDRESS>

I was trying to add the ovn_db from an sos report run on my kind set up and noticed the databases were stored in /etc/openvswitch.
Should we check all of these locations when collecting from sos?
https://github.com/sosreport/sos/blob/main/sos/report/plugins/ovn_central.py#L137

LGTM


feat: add a leaderboard of top contributors

Resolves #5

NOTE: to be merged after #28

Please, go through these steps before you submit a PR.

[x] My Pod Leader knows I'm working on this Pull Request
[x] I've explained what the Pull Request is adding.
[x] I've explained why this is important.

Resolved a merge conflict - great work!

@KhudaDad414 just a bit confused I was not able to find the leaderboard in the web UI. Just a suggestion using Github API instead of Github Actions could have been an easier approach and you could have fetched more params in total.

@Aniket762 why we are not using GitHub API? well, there are two reasons.
Security
Since we have no backend and the website is statically generated, we have to put the Github Token on the page itself, which anybody can inspect the page and pick it up which is a risk.
GitHub API call limit
Github only allocates 5000 points per hour which if we want to scrap the whole repo each time someone opens the page would consume those points like butter.


[docs] Warning card icon misaligned

Duplicates

[X] I have searched the existing issues

Related page
https://mui.com/x/api/data-grid/data-grid/
Kind of issue
Other
Issue description
The warning card in the API docs has a misaligned icon

Context 🔦
No response

Transfered :ok_hand:


Citaten-plugin does not trigger on citaten-only page

This PR just includes package upgrades to a version of the citaten-plugin that has the restorations to make it trigger in the citaten-only page again. With the new citaten-plugin, a new editor version was also necessary.

Watch out: this was al tested locally with npm link and the last commit contains a version number that does not exist yet, but will be after the release of the citaten-plugin, blocked by this PR.


Posts should be ordered by publish date descending

Right now the posts are insered into the MemoryMarkdownPostIndexer according the name.
The website should show all posts by date descending.
        public override IContentIndexer Index(string contentPath, string slugPrefix = null)
        {
            slugPrefix = slugPrefix ?? "/";

            IDictionary<string, Metadata> metadataTemp = new Dictionary<string, Metadata>();

            foreach (var subDir in Directory.GetDirectories(contentPath))
            {
                if (IndexContent(contentPath, subDir, MetadataType.Post, slugPrefix, out Metadata metadata))
                {
                    metadataTemp.Add(metadata.Slug, metadata);
                }
            }

            // order all posts descending by date
            foreach(var entry in metadataTemp.OrderByDescending(x=>x.Value.PublicationDate))
            {
                Metadata.Add(entry);
            }

            return this;
        }

This is the fix. I'm mobile and cannot create a PR.
It's more a reminder for me.

It looks like this is now implemented?


Type Error 'Valid Route Paths' when using useNavigate on nested routes

Describe the bug
Getting type error Error 'Valid Route Paths' when attempting to use useNavigate to navigate to a child route. May be related to #555
Your Example Website or App
https://stackblitz.com/edit/tanstack-router-fx77y7?file=src%2Froutes%2Fdashboard%2Finvoices%2Finvoice.tsx
Specifically line 92
Steps to Reproduce the Bug or Issue

Create parent & child routes
Use navigate

Expected behavior
Not to see type error
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: 0.0.1-beta.100

Additional context
No response

I faced this issue too and actually didn't find any solution

The trailing slash on fullPath was not matching up with the latest fixes for resolving index routes. This is now fixed by using the route.to property instead of fullPath or id. Examples have been updated.

id: The ID of the route, including layout route identifiers and wildcards
fullPath: The full path of the route, including trailing slashes so as to differentiate index routes from their parents
to: The navigable full path of the route without any trailing slashes.

@tannerlinsley I'm getting an error when I use the route.to property.
navigate({
  params: {
    organizationId: data.data.id,
  },
  replace: true,
  to: organizationsRootRoute.to,
})

Argument of type '{ params: { organizationId: string; }; replace: true; to: "/$organizationId"; }' is not assignable to parameter of type 'MakeLinkOptions<"/onboarding/", "/$organizationId">'.
  Property 'search' is missing in type '{ params: { organizationId: string; }; replace: true; to: "/$organizationId"; }' but required in type '{ search: SearchReducer<Partial<{}>, unknown>; }'.

Can you send me a CodeSandBox or stackblitz link please?

Please double check that your routes are referencing the correct parentRoute.

Ok, I'll check. Thank You!


fix: ensure to only remove npcs that are currently spawned

Motivation
When stopping a service and npcs are already removed before the actual disable of the npc plugin the call to remove all npcs might result in a NPE.
Modification
Ensured to only remove spawned npcs.
Result
No NPE anymore.

Shouldn't remove be guarded against throwing an exception in case the npc is already removed instead?


Definition of labels and reuse of CHEBI terms in the CDNO mineral nutrient class

This issue has been created to discuss the labels and reuse of CHEBI terms in the CDNO mineral nutrient class.
Having checked some CHEBI labels that were incorrectly re-labeled in CDNO (see issue #64), e.g. re-labeled as 'calcium', we identified some terms where the representation of mineral nutrients could be improved in the dietary nutrition domain:
1. We propose the reuse of other CHEBI terms. For example the ionic form (atom with more or less electrons) of an element is often measured when quantifying e.g. the 'concentration of calcium' in milk, typically using a method (eg ICP-MS) that estimates the concentration of calcium (2+) ions. However, the quantity being calculated is typically expressed as atomic Ca, even though it may be present in multiple bound, ionised or other molecular forms (eg salts). In some situations, there may be an assay that allows the estimation of concentration (via stoichiometry) of a specific salt.
Thus, within the 'dietary nutritional component' and 'dietary nutritional component concentration' classes, we would include eg. 'calcium(2+)' [CHEBI:29108] in addition to 'calcium atom' [CHEBI:22984].
2. To ensure there is vocabulary more tailored to dietary nutrition, we propose the adoption of the terms ‘dietary calcium’, ‘dietary magnesium’ etc, and under these have options for hierarchical sets of terms including: ‘calcium atom’, ‘calcium ion’, ‘calcium salt’.
We welcome feedback to know whether this provides a way forward to help nutritionists navigate to appropriate ChEBI terms and distinguish in what form an element may be present, quantified/estimated or reported depending on assay.
3. The hierarchical classification would look something like this, using 'calcium' as an example:

@ddooley, @CropStoreDb, @kaiiam, @laurenechan, @LuciaSegovia, @maweber-bia, @mateolan, what do you think?

Looks good to me - your classes like "dietary calcium" reflect the amount of information available from existing datasets. The subclasses can be referenced if more specific calcium form testing data is available.

I fully agree with the proposals, well done!

Brilliant!! I really like it :) Just one suggestion, for iron, you may want to include haem and non-haem into the hierachy.

Great thank you all :) @LuciaSegovia thanks, I will take that into account, I might need your help to check the hierarchical classification at some point, we will be in touch.

Sure! No problem :)

Hello, we have completed a framework proposal for the 'mineral nutrients', please follow the link and feel free to make any comments in column "N", your feedback is much appreciate it :)
@maweber-bia, @LuciaSegovia, @ddooley

Looks good @LilyAndres (from a quick glance I didn't study it all in depth). I'd make sure to add a column of the CHEBI iri's or CURIE's so this can get turned into a robot template.

Nice job! just wondering why you did not include "dietary" in the label of silicon dioxide
What will you do with the scientific publications referenced in the spreadsheet? Actually, it is good to know such references

We can (and I think should) add the publication links as cross-references to the terms.

My apologies if we have already discussed this, but I am slightly concerned with categories like 'dietary lead' or 'dietary arsenic' being subclasses of 'mineral nutrient' given that lead and arsenic are not nutrients. Similar concerns arise for any elements that may be consumed in the diet but do not have actual 'nutrient' capacities for humans. Do we have options for ways to include these dietary components with a different parent class?

Good points @laurenechan. We should be really clear about what is meant by mineral and  nutrient and based on is definition make sure we don't subsume anything under it that should be.
Like Lauren suggests we should have a different parent, something high-level and perhaps from CHEBI for chemicals that we import from CHEBE and present but don't want to call 'nutrients'

As previously discussed (see issues #65,  #30)  a "dietary component" can have either a "nutritional" or a "non-nutritional" role
This should be clear in the hierarchy of components as well :
"dietary"  indicates that the component is provided by food (in a diet)  but we should have dietary nutritional components and dietary non-nutritional components (those components which have a biological role - either positive effect or negative effect- but which are not nutrients)

Thanks for your comment @laurenechan. We have discussed this matter and we think maybe replacing the label of the 'dietary nutritional component' class with 'dietary chemical component' could work? As @maweber-bia mentioned, we need to think about how to define chemical components that are not necessary nutrients but are still in the diet. We presented a Venn diagram of these categories in one of the IFOW monthly meetings in Sept 2021.

In this case, the minerals that are ingested in the diet but that are not nutrients could be classified in a separate subclass, for example ‘mineral non-nutrient’.  I’m not sure whether we yet have the ideal label but this is the idea. Here is an example of the hierarchical classifications:

What do you think?

This issue has been solved in the 3.1 release. I will close it now.


How can I deal with errors thrown during resolving?

I'm guessing this has come up before :-) But what sort of approaches are there for dealing with errors that are thrown during resolving?
For example, if I have this (which won't compile at the moment):
public extension Container {
    var service: Factory<MyProtocol> { self { try MyService() }.singleton }

And I want to somehow capture the error from the service init so I can display a message to say something went wrong, is there a way to neatly handle it?
one thought is to make the service optional and use a try?. But in my case the registration is for a singleton and the error is  possible but highly unlikely so I'd kinda like to keep the registration non-optional and somehow redirect the error.
any thoughts?

You do have a knack, don't you? In all the years I've done this I've not had a single request for some way to handle a throwing initializer.
Regardless, if it throws then you're not going to have much of choice. That factory closure has to return something. You have two options. The first, as mentioned, is making it optional and nil.
The second depends on the type of object being returned. You could, perhaps, do...
public extension Container {
    var service: Factory<MyProtocol> {
        self {
            do (
                return try MyService()
            } catch {
               // log error
               return StubMyProtocol()
         }
        .singleton
     }
}

If the library could fail, but is some sort of logging or analytics library, then return a mock for the rest of the world to interact with. If it's an API, then just have the mock error out when accessed.

Yeah. It's a tricky one. I have a service which is loading a bunch of data from JSON files on startup, so the problem I was addressing was what happens if something in that data is off. I've since worked around it, but it's an interesting question.


the bottom to up button collides with items in mobile screen

Describe the bug
the bottom to up button collides with items in mobile screen
Screenshots
If applicable, add screenshots to help explain your problem.

assign me this issue.

how will you proceed?

i will try to adjust its position.


Stored Backups multiple select delete

I did a search and did not a find the feature request I am proposing.  So please do not flame if the request is somewhere on the site I did not see it.
In the Stored Backups screen give us the ability either by checkbox or CTRL/SHIFT Selecting more than backup at a time.
A simple and short request.

Closed for not following template.


Why we use squeeze_embedding

I'm currently somehow confused about why we use squeeze_embedding module to pack the padded sequences and then unpack it ?

mark，same confusion

It works like layers/dynamic_rnn.py, we use torch.nn.utils.rnn.pack_padded_sequence and torch.nn.utils.rnn.pad_packed_sequence to compress variable length sequences according to the longest sequence in a batch.
You can debug this module to see its working detail.
For more information of how this two function works, please search pytorch variable length sequence.

Now I get it.


Mazumder - ngsQC data selection for all data teams + genome assembly

All ngs and assembly data selection, Org tracking, and selection criteria protocol development will happen in tandem with this task:

[x] Select data for Marburg, Influenza (H1N1 + H3N2). More XBB.1.5 and RSV (time permitting). Select Fastq + associated Fasta (genome assembly - preferable a Ref Genome) + (recent data). 1-3 different data points per virus strain. Check FDA BioProject first and include, then check NCBI as a whole. Ping me to review after this step
[x] Use the selection criteria protocol described by Raja and build on it in the Documentation folder (provide link in ticket)
[ ] Update this sheet to organize everyone's work and who you delegate selected data to: https://docs.google.com/spreadsheets/d/1Nf9Eey2ruwKRkoSMoNhGCAYmFAw2Wb73TzA3sYA3va0/edit?usp=sharing
[x] Delegate data tasks as a sub-ticket in the following tickets (I can show you how to do this if you are unsure): Tasks #212 #213 #215 depend on this task being completed and require data delegation

[ ] test

complete


Security check for PHP libraries

Apigee security asked how we are checking for known vulnerabilities in the libraries we depend on, added hooks to run sensiolabs/security-checker on installs and updates

Looks like there is more to do to get this working with TravisCI: https://github.com/travis-ci/travis-ci/issues/6339#issuecomment-355158125

I though about adding sensiolabs/security-checker to this library but then I realized that this is a library not a project, therefore the consumer should check whether their dependencies (including this one and its dependencies) are up to date.
sensiolabs/security-checker could be maximum a dev dependency instead of a mandatory dependency of this library but based on the above reasoning I am giving a -1 for this pull request.

PS.: A security checker for your composer.lock  we do not ship composer.lock with our library, we rather do highest-lowest testing to ensure that our library is compatible with the minimum and the latest version of its dependencies.

Yes, the security check should be a dev dependency, made that change. @mxr576 do you have another solution for checking for known vulnerabilities?  We need to be alerted if an issue arises.

@cnovak I can not image what security issues we could possible check in our dependencies if we are not depending on exact versions of the libraries we use. (Because basically this is what composer.lock descripes in case of PHP project.) Our library just says please install the latest version or at least x.x.0 from the listed packages, in case of "symfony/serializer": "^3.2 || ^4.0" it means symfony/serializer >= 3.2.0 or >= 4.0.0.
As I mentioned the consumer app should check whether its dependencies (and dependencies of dependencies) defined in its composer.lock contains any know vulnerabilities or not.

Or do you want to keep tabs on minimum versions that our PHP library requires and ensures that those does not contains any known security risks - in other words, monitor new releases day by day of this library's dependencies and increase the minimum version in this library's composer.json? That would be a huge overhead...

If we don't have PRs often then yes, we would not know about vulnerabilities until the CI ran again.  I am asking our security team for guidance.

https://blog.martinhujer.cz/17-tips-for-using-composer-efficiently/ Tip #5 and #6: About why we should not ship composer.lock in this library - consequently sensiolabs/security-checker does not work for us - also we are not depending any secure or insecure dependency directly. The consuming application decides which versions are being installed from its dependencies.


Dont use RequestLoggingFilter all the time

Not having @Logged meant that every request was going through the requestLoggingFilter when it was meant for only methods with @Logged on them.

Not having @Logged meant that every request was going through the requestLoggingFilter when it was meant for only methods with @Logged on them.

alluxio-bot, merge this please

alluxio-bot, merge this please


Create and Maintain a Common Nomenclature

#Create and Maintain a Common Nomenclature
##Establishing a common terminology for the Clear My Record Project
Clear My Record is a unique combination of legal procedure, substantive law, design and coding. The team will include many people from different academic and vocational disciplines. Having a common lexicon for terms that are core to design and operation of an expungement application is an important resource. The inital lexicon will be developed in the early weeks of the project and will be maintained as part of the project and, ultimately, the operating application.

Here is a test comment.


Add mod find

Users cannot find friends who have taken or are taking a specific module.
Adding an additional mod find will allow them to find friends who have taken or are taking a particular module.
Let's
implement the mod find command
implement modcontainskeywordpredicate
include tests for mod find command
include tests for modcontainskeywordpredicate

Codecov Report

Merging #101 (28deb95) into master (bc50cee) will decrease coverage by 0.07%.
The diff coverage is 70.83%.

@@             Coverage Diff              @@
##             master     #101      +/-   ##
============================================
- Coverage     71.67%   71.60%   -0.08%
- Complexity      524      539      +15
============================================
  Files            84       86       +2
  Lines          1691     1715      +24
  Branches        210      214       +4
============================================
+ Hits           1212     1228      +16
- Misses          397      403       +6
- Partials         82       84       +2

Impacted Files
Coverage Δ

...a/seedu/address/logic/parser/ModCommandParser.java
86.58% <0.00%> (-8.16%)
:arrow_down:

...ss/model/person/NameContainsKeywordsPredicate.java
80.00% <ø> (ø)

...a/seedu/address/logic/commands/ModFindCommand.java
88.88% <88.88%> (ø)

...ess/model/person/ModContainsKeywordsPredicate.java
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Configure access to Cloudflare

Provide access to the following members of Blink Labs:

[x]<EMAIL_ADDRESS>(@agaffney)
[x]<EMAIL_ADDRESS>(@verbotenj)
[x]<EMAIL_ADDRESS>(@wolf31o2)

Permissions should be:

ZeroTrust, all domains (isn't scoped)
Administrator, dmtr.host domain

I think we still need to do some tuning on the access, but everyone has access to the account.


Embed plot into PyQt gui

I have not been able to find any reference in the documentation for embedding bqplot into PyQt gui. Being able to exploit both the power of bqplot and PyQt would be amazing.
Does the current release of bqplot allow for that? In the case of matplotlib this is easy done and in the official website one can even find example code for that.

I only know @astrofrog has done it in combination with Voila and the Qt WebEngine, see this example: https://github.com/astrofrog/voila-qt-app. I don't know if this fits in your use case though.

Thank you. Unfortunately this does not apply to my case since a different python subprocess is created. I would like that the Python process which I use to create the PyQt window would also be used to render the plot. That is, basically I would like that the library could use PyQt5 as a plotting backend (as matplotlib does), and not being limited to use a Jupyter Notebook.


Add link to SmartNews in README

SmartNews uses airflow to manage our data pipeline. I mentioned airflow in the slide below.
http://www.slideshare.net/smartnews/20160127-building-a-sustainable-data-platform-on-aws

Sweet!


I found a bug about the readUInt32BE.

Buffer.prototype.readUInt32BE = function readUInt32BE (offset, noAssert) {
  offset = offset >>> 0
  if (!noAssert) checkOffset(offset, 4, this.length)

  return (this[offset] * 0x1000000) +
    ((this[offset + 1] << 16) |
    (this[offset + 2] << 8) |
    this[offset + 3])
}

In some Android browser (Chrome 71), it returns a negative number.
I think it should be add a check.
  return (this[offset] * 0x1000000) +
    ((this[offset + 1] << 16) |
    (this[offset + 2] << 8) |
    this[offset + 3]) >>> 0

readUInt32LE also need add this >>> 0

Can you check that it's not possible set negative number (or number more than 255) in Uint8Array in described environment?

The buffer data is [172, 68, 0 , 0 ]
It uses for mp4a sampleRate.  it returns -1404829696.

Do you have a minimal example that demonstrates this issue in action? It's not possible to debug this issue without additional information. If this is still an issue, please leave a comment or open a new issue.


Social Media Profile URLs

Title
Social Media Profile URLs
Description
Added new URLs for Codeforces, Linktree, Peerlist, Quora, Snapchat, Telegram, Twitter in social-links.json.

Issue Ticket Number
Fixes #2

Type of change

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

Checklist:

[x] I have followed the contributing guidelines of this project as mentioned in CONTRIBUTING
[x] I have checked to ensure there aren't other open Pull Requests for the same update/change?
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes needed to the documentation

Amazing work! ⭐
Merging it to main branch.
Thanks for Contributing to SocialRepo
Keep contributing 🙌


add filtering capabilities to the results table

Adding filtering capabilities to the table view::

In addition, for more clarity moving the config.py file under the folder "utils"

@rachelt44 look on the slide I shared above --- ^^^ it is showing you numeric filtering ^^^

@rachelt44 look on the slide I shared above --- ^^^ it is showing you numeric filtering ^^^
(( The code automatically detects for all the clomns in the table if they are numeric or categorial and creates relevant filters for the data basedon that., ))

When I ran the code locally, all features were detected as categorical, so it didn't work for me. Also, in the slide you showed, it shows real numbers for integer features, which doesn't make sense. We don't want a 21.97 filter for number of processors. Is there a way to fix it?

Code is updated @rachelt44 please check :-)

@rachelt44 look on the slide I shared above --- ^^^ it is showing you numeric filtering ^^^
(( The code automatically detects for all the clomns in the table if they are numeric or categorial and creates relevant filters for the data basedon that., ))

When I ran the code locally, all features were detected as categorical, so it didn't work for me. Also, in the slide you showed, it shows real numbers for integer features, which doesn't make sense. We don't want a 21.97 filter for number of processors. Is there a way to fix it?

Code is updated @rachelt44 please check :-)

@eranra works now!


Checking location when running the dev server

Opening this for @bebraw
For the Splash page I'd like to tweak the styling Navigation bar. The cleanest way to do this through React/React-Router is usually to check props.location like so:
let { section, location, config } = props,
      { pathname } = location,
      { home = '/' } = config;

...
    <Navigation home={ home } pages={ pages } splash={ pathname === '/' } />
...

However this always evaluates to false on the dev-server. After some logging it seems that the pathname variable is equal to '/' when using npm run build but when developing always evaluates to 'antwar_devindex' no matter which page you're on. See the screenshot below:

Not sure if this is an issue with antwar or our particular setup so I figured I'd create the issue here to start...
Update: Just pushed a new branch for the Splash page if you want to take a look.

Maybe it would be cleanest to go through the DOM location API? That wouldn't tie us to react-router. Maybe we could avoid bundling it to the final build altogether.

Works for me, I'll use that for now on the splash-page branch.


Add redirect to DTR release notes

Deploy preview ready!
Built with commit 6de36b9dce66fd0c808a9b77a9c4e90a755999d3
https://deploy-preview-567--docker-docs.netlify.com


Change Dockerfile to use non-root user

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (78ea6df) 99.90% compared to head (82e10b0) 99.90%.
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   99.90%   99.90%
=======================================
  Files          38       38
  Lines        3317     3317
=======================================
  Hits         3314     3314
  Misses          2        2
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[stable/nextcloud] Add insecure option for curl

Signed-off-by: Aurélien Bertron<EMAIL_ADDRESS>
What this PR does / why we need it:
Add cronjob.curlInsecure value. This is used to set the "insecure" option (-k) to curl in the cronjob. This prevents errors when using self-signed certificates in development configuration.
Which issue this PR fixes

fixes #16232

Special notes for your reviewer:
Feel free to suggest a new name for this value as I am not sure it is clear enough for a user.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/lgtm

Thx for the PR


Popularity Boost

Love this add-on team! I'm trying to figure out if i can add a popularity boost to make these results even better. Relevance is good, but sometimes we e.g. brands are better (based on e.g browsing history) etc.
Any tips on how I might achieve this?

Hi @aaronweatherall and thanks for your nice words!
Have you tried mixing the score with some other fields via a function score? I haven't tried it, but both sort and bf should work with autocomplete and do what you're after.

You can integrate popularity boost in autocomplete searchHandler.
That can be edismax handler with bf or bq where you can add popularity boost.

dismax_ac_query

dismax
100%
XXX


Does it support Chinese

Does it support Chinese

There are no chinese translations.
I have no clue how dirs paths will work with these special chars.


Feature/outbreak service

Adds call to retrieveOutbreakEvents to get data from the server vs db.json
Added news OutbreakProvider
Updates protobuf file (covidshield.proto ) to latest schema

@jeberhardt
Tested and this is now working 🎉
Let's move this from draft + merge to unblock other work on the feature.

PR Demo
https://www.youtube.com/watch?v=RaZQXXJTOgo


Curve LP token price

Using the endpoint /lps, one can fetch all the LP prices crosschain. Below you can find a subset of the response:
"curve-poly-atricrypto3": 1938.4656787747067

Could you please explain where the value 1938.46 comes from? I cannot derive this calculation from any value displayed on the curve website pool (https://polygon.curve.fi/atricrypto3). Thank you!

It come from counting how many USDT, ETH and WBTC are on the LP contract, multiply by the price of each of those tokens and divide by the total supply of LP. That price is how much 1 LP token is worth.
https://github.com/beefyfinance/beefy-api/blob/master/src/api/stats/common/curve/getCurvePricesCommon.js

Thank you very much, that answers my question.


Fix shakapacker version in Gemfile.lock to fix CI

https://github.com/shakacode/shakapacker/commit/756318738dda09d6432d8b06418ad40abf98cd11 breaks CI 💔
Like https://github.com/shakacode/shakapacker/runs/5090267102.
It may be better to remove Gemfile.lock maintenance easier.
In this PR, this commit just update gem version.

Nice one. Don't think removing Gemfile.lock is necessarily a good idea. Some discussions around this
https://github.com/rails/rails/pull/18951
https://github.com/heartcombo/devise/pull/3147
This is definitely something to keep track off during release process though! @justin808 What are your thoughts? Maybe a check or bundle install run somewhere before https://github.com/shakacode/shakapacker/blob/master/rakelib/release.rake#L29 ?

for a followup, I think we should either:

remove the Gemfile.lock from the root directory (that is what React on Rails does)
update the release script so that this file gets updated.


Updating version to 0.0.9 for release

Current coverage is 51.41%

Merging #64 into master will not affect coverage as of 9963d0f

@@            master     #64   diff @@
======================================
  Files           41      41
  Stmts         1873    1873
  Branches       284     284
  Methods          0       0
======================================
  Hit            963     963
  Partial         55      55
  Missed         855     855

Review entire Coverage Diff as of 9963d0f

Powered by Codecov. Updated on successful CI builds.

LGTM


{:ok, {:ok, user}} returned from Addict.Repository.create

At least for me, I'm getting this behavior; it causes the last stage in registration, sending user email, to explode with:
(UndefinedFunctionError) undefined function: :ok.username/1 (module :ok is not available)
        :ok.username({:ok, %Flashbang.User{__meta__: %Ecto.Schema.Metadata{source: {nil, "users"}, state: :loaded}, email: "az", hash: "$pbkdf2-sha512$60000$tv82nurn18WwchXaLHdxhg$qDPWQVlr/lg3WQznHCms84CL/Oek8VQ93whhznVkyt6Qm.rlQPB4YA2JjLdKk.Vo8LSheUibSrkKSJ1KjSwGtw", id: 3, inserted_at: #Ecto.DateTime<2015-08-14T22:27:01Z>, recovery_hash: nil, updated_at: #Ecto.DateTime<2015-08-14T22:27:01Z>, username: "z"}})
        lib/addict/email_gateway.ex:7: Addict.EmailGateway.send_welcome_email/2
        lib/addict/interactors/addict_manager_interactor.ex:162: Addict.ManagerInteractor.send_welcome_email/2
        lib/addict/controller.ex:95: Addict.Controller.register/2
        lib/addict/controller.ex:89: Addict.Controller.phoenix_controller_pipeline/2
        (flashbang) lib/phoenix/router.ex:265: Flashbang.Router.dispatch/2
        (flashbang) web/router.ex:1: Flashbang.Router.do_call/2
        (flashbang) lib/flashbang/endpoint.ex:1: Flashbang.Endpoint.phoenix_pipeline/1

After digging a bit I believe a recent commit that touched Addict.Repository.create caused this.

@mrluc what ecto and phoenix versions are you running?

Closed via update on #55


[ur] Localize 'Cloud Computing' Term in Urdu

Describe your changes
Related issue number or link (ex: resolves #issue-number)
resolves #1874
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

LGTM Zubair!


MINOR: [Dev][JS] Fix lint issues introduced

Fixes some trailing whitespace, and removes what appears to be an accidentally committed test file.

CC @domoritz not sure if js/test.ts was meant to be committed

Ups. No it wasn't. Thank you!


Don't upgrade when the version is already installed

When using an upgrade_strategy with a specific version, it's possible to skip the upgrade if the requested version is already installed.
I'm not sure where such behavior should go, but I am able to test the rvm version with this command:
rvm version | grep -q '^rvm 1.16.20'

So perhaps something like this in upgrade_rvm?
upgrade_check = rvm_wrap_cmd(
  %{rvm version | grep -q '^rvm #{opts[:upgrade_strategy]}'}, user_dir
)

@cgriego Can you please submit this PR here?


Question: Does the latest version no longer support kibana v5.4.3 now?

It could be installed, but run with exceptions.

There are some API changes between 5.5.x and 5.4.x versions of Kibana. So the latest logtrail version is not backward compatible with older releases of Kibana.

Is there a way to see what versions work with what versions of Kibana? I have 5.4.1 and was about to install the latest but thought I'd double check and found this, now unsure what the latest version of logtrail works with 5.4.1

Kibana requires exact match between plugin and kibana version. The plugin installer name will contain the Kibana version supported and the plugin version. e.g logtrail-5.6.4-0.1.23.zip supports Kibana 5.6.4. Whenever there is new release of Kibana, a new release of logtrail latest version will be released.
Currently I am not releasing logtrail for olders Kibana version everytime when there is a new logtrail release.


docs: Improve the readme

I wanted to make the readme more friendly, so that you can glance over the important information but be able to copy and paste code to get started which is why I moved the comments outside of the examples and included more information for users that use pnpm because I ran into some issues with peer dependencies.
I prefer to have a section on the reasoning behind the package to understand why I would use it but it can be improved or removed if you want.
Thank you for making this! 😄

Sorry for the disorg here, but on second thought, I'm thinking I'll probably want to remove the tweet screenshot, because it adds a lot of jargon to the front of the repo that might confused people.
Everything else is fantastic.


fix: re-implement status update flags

What this PR does / why we need it:
This patch re-implements the --status-update flag for KIC 2.0
but also deprecates the --update-status-on-shutdown flag as the
behavior of tearing down on cleanup is somewhat in conflict with
the idempontent and eventually consistent design of KIC 2.0.
Which issue this PR fixes
fixes #1304
PR Readiness Checklist:

[x] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

What are the other methods mentioned in the issue comment? I'd naively expect that we can't be eventually consistent when shutting down, because there's no "eventually" after shutdown: the controller is gone and will not be able to issue further status updates.

What are the other methods mentioned in the issue comment? I'd naively expect that we can't be eventually consistent when shutting down, because there's no "eventually" after shutdown: the controller is gone and will not be able to issue further status updates.

The "other methods" refers to the assumption that the controller will be restarted, and that the re-entrant nature of our logic leads to eventual consistency. Trying to support teardown mechanisms with a Kubernetes controller seems like a juxtaposition and without some strong influence I'm not seeing why we would start adding this kind of functionality right now. Let me know your thoughts.

Ah. I was thinking of the case where the controller+proxy are deleted entirely, possibly never to return, which is the only case where the current shutdown handler does anything.
This is maybe not hugely important--if you're keeping Ingresses around and care about their status it stands to reason that you're probably going to start a controller again in the future--but we can leave inaccurate information around without it (status info will remain indefinitely otherwise). This could potentially impact some setup where another application watches Ingress status to update DNS and either populates a fallback address if there is none or there are downstream applications that behave differently for an empty NOERROR response versus an HTTP timeout or refusal. Probably not that common, but not unreasonable.
Research doesn't turn up much of interest to say whether or not we definitely need this, unfortunately. It's another carryover from the NGINX controller, and the only info I can find there indicates that shutdown updates have been around since time immemorial. The flag was added to instead disable the behavior, since there are some cases where you don't want it--prior to that the controller always did shutdown updates.

Research doesn't turn up much of interest to say whether or not we definitely need this, unfortunately. It's another carryover from the NGINX controller, and the only info I can find there indicates that shutdown updates have been around since time immemorial. The flag was added to instead disable the behavior, since there are some cases where you don't want it--prior to that the controller always did shutdown updates.

Yes, I've seen https://github.com/kubernetes/ingress-nginx/issues/881 and while this does provide some justification I still feel like this functionality is dubious: at best it's "best effort" functionality, as the pod may be killed without any grace period and the status cleanup can't run. And then amidst what is effectively "best effort" the following things need to be true for there to be value:

the status updates need to be able to complete  before the configured container shutdown grace period
the statuses of the ingress records is being used by an external source that specifically needs to know when the IP/Host becomes unavailable

However for that second point isn't that potentially a bug as well? If your controller gets stopped and then it removes the IP/Host status from the resource but then the controller comes back online and puts it back doesn't that have the potential to break such integrations, especially in the case that the proxy is still up during this time and the IP is actually valid? Why would we ever make the control plane update the status of an object which we can't validate is actually a valid representation of the data plane? Wouldn't we only want to remove the status if we were certain that the backend/dataplane was actually not serving it?
We have an opportunity here to cut ties with this bit of functionality which is arcane, if not just questionable because we're about to release a new major version and we can let the users know the change occurred, I think we should take that to reduce maintenance as I'm not seeing strong justification otherwise.
Let me know what you think. Ultimately if we really want to make this work I believe we only need to add a purge mechanism to the Proxy cache implementation.


Headless CMS

In my first talk, I will talk of the benefits of:

Headless CMS
Compare Wordpress RestAPI (CMS) Vs. Strapi.io (CMF)
Alternatives to Strapi

I will demonstrate:

Live coding and creation of a app/blog using GatsbyJS + Strapi with offline support

Sounds great! Where would you like to give this talk?

@bromso thank you for the presentation. Can you upload it somewhere? :)

Thank you! :)
Sure!
My presentation
https://drive.google.com/open?id=1NmP_XX2PLqnlDYXnyK4DsbpQeItndtJ8tOYeOxmBmJ8
Sources I've used for creating my slide presentation:
Jamstack.org
HeadlessCMS.org
Storyblok Article of Headless CMS - Article "might have a LITTLE biased approached because Storblok is a Headless CMS ;)
Resources:
NodeJS
Strapi.io
GatsbyJS
YouTube Tutorial of Strapi & Gatsby
Stack Used in the Tutorial
NodeJS
NPM
Strapi.io
GatsbyJS
SailsJS
MongoDB
Strapi also supports these databases, out-of-the-box
PostgreSQL
MySQL
MariaDB
SQLite
Open-source alternatives to Strapi
Open-Source
Netlify CMS
Ghost
Closed-source
Contentful
Flamelink (based on the Firebase eco-system)
(Mutated) Headless CMS as a service
Shifter
For even more information, please go to HeadlessCMS.org. You can search and filter your way forward in order for you to find a Headless CMS based out of your needs.
Tech that would have improved the app that I was trying to show "on stage"
StorybookJS - Manage all of your UI components better
LernaJS - Manage multiple NPM packages


How does infer works?

I am an iOS developer, and I am trying to use infer in our project.
But I really want to know how the infer works and "static analysis principle" infer uses.
I have read the documents in http://fbinfer.com/. But it is not detailed enough.
Can you give me some wikis or webs to learn about that?

This paper on Infer's bi-abduction analysis is a good resource. The bi-abduction is only one of many Infer analyses, but it is the basis for many of the checks.

More information can be found in the "Foundations" section of the online documentation. Some more papers are listed here. Hope you can find the answers you're looking for there.


#158 layout on why us section optimised for desktop

Current coverage is 47.95%

Merging #177 into master will increase coverage by +0.05% as of 38d8061

@@            master    #177   diff @@
======================================
  Files           20      19     -1
  Stmts          311     294    -17
  Branches         0       0
  Methods          0       0
======================================
- Hit            149     141     -8
  Partial          0       0
+ Missed         162     153     -9

Review entire Coverage Diff as of 38d8061

Powered by Codecov. Updated on successful CI builds.


Storybook env setup

implement #17

storybook environment setup
storybook.scss aims to customize welcome of story in the future.

Thanks 💯


Various built-in functions are not exact.

Various built-in functions are not exact and rely on doubles. If possible these should be swapped to BigDecimal implementations.
I'm not sure where to get those, though. It's just something that I just noticed.

big-math seems to implement these functions already, but depending on it doesn't seem to be a very good option.
Everybody can fix this, though, by overriding the functions in the Expression after it is being created. I'm not sure how to approach this. The big-math library is MIT licensed, theoretically the source could be embedded, but depending on it would be nicer. But I'm not sure if a dependency is something you want.

FYI: Just started working on an add-on library that replaces and extends the current implementations.

I have just released the first version of the EvalEx big-math extension.
See the announcement here: https://github.com/ezylang/EvalEx/discussions/335


Add setCellTextRaw for faster mass updates

For faster mass-updates, we need some function to set cell texts without slow features like validation and history

looks good to me, would you please update the types as well (index.d.ts) ?


HELTEC Wireless Stick No Display

Hello OM,
No display status on heltec wireless stick.

we are currently testing as it should work and don't know why it does not... please share the exact model of your board
HELTEC WIRELESS STICK (which version? v2 or v3 ? or v3.1)?

we are currently testing as it should work and don't know why it does not... please share the exact model of your board
HELTEC WIRELESS STICK (which version? v2 or v3 ? or v3.1)?

V3

if seen the schematics of V2 and V3
when using oled pins for V3 , wifi does not work and also oled does not light

No display status on heltec lora32 v2
downgrade to [V<IP_ADDRESS>] and now ok


bug: No compile-time errors in dev, consecutive errors in build

Provide environment information
System:
OS: Windows 10 10.0.19045
CPU: (16) x64 AMD Ryzen 7 2700 Eight-Core Processor
Memory: 14.65 GB / 31.95 GB
Binaries:
Node: 16.19.0 - C:\Program Files\nodejs\node.EXE
Yarn: 1.22.18 - ~\AppData\Roaming\npm\yarn.CMD
npm: 8.19.3 - C:\Program Files\nodejs\npm.CMD
Describe the bug
I have added two very obvious, compile-time errors to index.tsx and _app.tsx, namely:
const x: string = 4 and const a: string = 4.
Issue 1:
When running the dev script, I would expect the code not to compile, but to tell me about this issue,
so I could fix it, but the dev script runs successfully and I can use the app.
Issue 2:
When running the build script, the error in '_app.tsx' is actually shown, but not the one in 'index.tsx'.
The error in 'index.tsx' is only shown, once I fix the errors in '_app.tsx' -> Meaning the compiler stops after the first error.
This leads to extremely cumbersome and annoying issues, when moving a component library from one file to the other and having to deal with all the errors consecutively.
Link to reproduction
https://github.com/Tobjoern/ts-turbo-demo
To reproduce
Download the repo,
'pnpm install',
'pnpm run dev',
'pnpm run build'.
Additional information
No response

I don't see how this has to do with t3-turbo. This is how it works in any Next.js app.

a typeerror in dev shouldn't kill your server - that would be quite bad DX.
build is cancelled after first error cause what if you'd have 10 more errors following just cause you had that 1st error? should your entire terminal be literred or just show the root cause?

here's a sandbox with a minimal next.js app: https://stackblitz.com/edit/nextjs-ce2hjm?file=pages/index.tsx
And here's a small demo, as you can see build stops at the first error it sees:
https://cln.sh/fLz4BZ4b

I see how this error has nothing to do with t3-turbo,
but I disagree with your assessment that this would be quite bad DX.
Let's say I move my 'FormGroup' component from 'shared' to 'inputs' and this very component is used in 100 other components.
With the current setup, I would have to run 'npm run build' 100 times, and jump to each component separately, to get it to work again.
I find the way nuxt.js handles this to be a lot better.

I suggestiv you open an issue in Next.js then


Blog Post: How to mine BALN

Write a guide on how to:

Mine BALN on Homepage
Mine BALN on DEX
Claim BALN
Stake BALN

Draft: https://docs.google.com/document/d/1yqD_GhG5nqiEcYUhKOIEMLPojxmD11J3-pEd8cHcbGw/edit?usp=sharing

Final post: https://balanceddao.medium.com/how-to-earn-balance-tokens-6aafb7d6dd5


Tell which websites are updated

On ipfs-cohost sync

@alanshaw in reality, all domains are updated. Even if we don't have a new snapshot to take, we copy the old snapshot to a new directory with the current date.
I'm not sure if it's worth printing anything on this case.


Try to avoid table and object creation

For performance reason you want to avoid table and object creation as much as possible. In your on_input function you create a new table every frame (which will generate garbage which will cause a gc hitch):
Highlight_Button({x=action.x,y=action.y})

Wouldn't it be better to either pass the action table or the x and y values directly.
Highlight_Button(action.x, action.y)

or
Highlight_Button(action)

The same goes for Click_Button where you create a vector3 which you really don't need since you only care about x and y for node picking

I'm not quite sure why I did it that way, although I do remember my thinking for Click_Button's vector3. I thought I needed the z value to use gui.pick_node, and then I didn't modify it once I found out.
Modified in Version 0.2.


Fix 18n fallback when "mount_at_root" is false

This fixes #2596.

@tdreyno time to merge 😃?

@markets enabled auto-merge, needs a rebase


Add dynamic pilot name parsing

Hi,
Some task validation software such as the one used by the PWCA appears to be slightly changing the IGC headers format. This results in a wrong parsing of the pilot's name when reading IGC files validated by such software.
As an example, PWCA uses HFPLTPILOT field rather than HFPLTPILOTINCHARGE, leading to an incorrect slice of the name in the low level reader. The change proposed allows a dynamic slicing of this field in order to handle correctly various IGC header formats.
GpsDump also seems to be using HFPLTPILOT rather than HFPLTPILOTINCHARGE.

can you add a few tests for this new behavior to avoid regressions?

Done.

sweet, thanks!


isAdmin() needs to check if the user is currently logged in

Hi,
I am checking if a user is an admin with the following IF - Yii::$app->user->identity->getIsAdmin() or Yii::$app->user->identity->isAdmin.
Both of them throws PHP errors if no user is logged in. Can the method check if the user is currently logged in and then return TRUE if they are admin? if not logged in, then return FALSE?
Thank you.

Fails while User identity is none object, Needs to be checked before?!

I want to show "edit" buttons to an administrator only. So, I am trying to use the above commands to check if the logged in user is an admin or not. IF admin, then show edit buttons. But, when no one is logged in, it says "Trying to get property of non-object". Wouldn't it be better if the getIsAdmin() method checks if there is a user logged in first and then check if that user is an admin?

I solved it by using IF(!Yii::$app->user->isGuest && Yii::$app->user->identity->isAdmin). both conditions together like I found in a piece of code. Although I don't see why not check if the user is logged in the isAdmin() method.

@vsmelarkod this cannot be done in any way: Yii::$app->user->identity is a currently logged in user. So if user is not logged in there is no identity and we cannot call method on null.


Cedar/Bridge.c: fix NULL pointer dereference in GetEthDeviceHash()

Fixes #753.

SoftEther VPN Patch Acceptance Policy:
http://www.softether.org/5-download/src/9.patch
I choose option 1.

Let's wait for @solipse's review before merging.

Tested, runs and tap0 works fine.


Update instance types from pricing API

Issue #, if available:
#1046
Description of changes:
Updated spec files to reflect latest instance types from pricing API.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing this pull request in favor of #1048, which patches the issues in the new resource spec catalogued above.


Install QEMU emulator in arm64 variant of cpubuilder image.

Follow-up to https://github.com/iree-org/base-docker-images/pull/11.
This emulator was originally installed in https://github.com/iree-org/iree/pull/16331. We've been carrying it around since then.
I'm not thrilled with the file sitting in a cloud storage bucket (GCS or Azure). The file is 5MB so maybe we could check it in here via git LFS? Or we could build it from source via our automation.

cc @MacDue @banach-space


Remove the cast dependency.

It was only used between usize and u64 in parts of the code that were
marked with #[cfg(target_pointer_width = "64")], which guarantees
usize is 64 bits.
See #90.

Perfect, thanks!

Released as version 0.9.2.


Stress test configuration

all test projects
manual garbage collection in RunQueue after each project was
processed
test configs

Can be archived, see the test results in #95