mkShell alternative that doesn't include stdenv, for use with direnv or impure nix develop

Problem you want to solve
I want to use direnv (using nix print-dev-env) or nix develop (or nix-shell even) to give me an environment with a set of packages but I don't want to pull in anything from stdenv (unless I requested it). As it is, using mkShellNoCC on my system includes bashInteractive, coreutils, findutils, diffutils, gnused, gnugrep, gawk, gnutar, gzip, bzip2, gnumake, bash, patch, and xz, all in addition to the packages I requested.
Using a shell to reproduce the build environment of an existing package should include all this, but when I'm using mkShell it's because I want to add specific dependencies to my existing environment. A pure shell probably needs stdenv, but it just gets in the way with my impure shell.
The fundamental problem here is that mkShell/mkShellNoCC depends on stdenv, and so tools like nix print-dev-env pull in all of the stdenv packages too.
Proposed solution
Add a new library function that acts like mkShell except it produces a derivation that itself does not depend on stdenv. nix develop should only add the packages that are listed explicitly.
If possible, it should consider doing this only for "impure" shells, and still pull in stdenv for "pure" shells.

You still want to run the setup hooks though? Those do rely on stdenv.

I'm not certain which packages rely on them and which don't. I'm not pulling in compilers. I am doing things like autoconf, jq, etc. One project does use pkg-config and openssl, which really just needs a particular env var set, and I'm not sure if that's constructed with a setup hook or some other mechanism.
I suppose what I really care about here is just ensuring the PATH var in my shell contains only what I requested. In theory that means "save PATH before doing anything, run setup, then reset PATH and rebuild it using just the declared dependencies without stdenv". This approach also provides a convenient spot to say "don't do this last part if it's a pure shell".

If you can work without the shell hooks, there is already nix shell:
# env -i $(which nix) shell nixpkgs#hello -c /usr/bin/env | grep PATH
PATH=/nix/store/lj6qjfzhk25ynvyk7d6i6a2v5k9aa5gp-hello-2.10/bin

Otherwise, unsetting of unwanted paths can be done in the shellHook.

Using nix shell to provide a whole set of dependencies at once seems rather awkward. And of course if I need anything else like PKG_CONFIG_PATH then this doesn't work (presumably this var is populated via a shell hook).
I actually do have a cleanStdenv wrapper I wrote that basically uses a preHook to record the initial PATH provided by stdenv, then a shellHook that removes this from the new resulting PATH, but it feels a bit fragile and I'm unsure whether it will work if I explicitly request any of the stdenv packages. I'm also unsure if there's anything else from stdenv that I accidentally inherit from this (I haven't done a detailed inspection of all resulting env vars yet).
Having something in nixpkgs that did this officially would feel a lot more reliable. And this also feels like something that should be generally useful for others.

https://github.com/NixOS/nixpkgs/pull/132617 this PR adds a shell that only relies on bash.

Interesting. That looks like it does something very similar to what I want. It does mean I can't trivially pull in openssl + pkg-config without managing PKG_CONFIG_PATH myself unfortunately, and I'd prefer to not even have bash in my path, but it still does most of what I want.
The openssl + pkg-config thing though is something that would be very helpful to be able to solve properly. I don't know if that's "do another mkShell variant that runs setup like normal and then fixes up PATH to omit stdenv packages" like what my local cleanStdenv wrapper attempts, or maybe something like a wrapper that runs a mkShell variant that records all the resulting env vars, and then pulls in the non-PATH vars into the shell and reconstructs PATH based on the selected packages. There's no immediately obvious answer here.


Fix Heise

Heise had a minor layout update one week ago. This PR fixes it by changing a minor detail. As it turns out Heise has a gadget blog called techstage and I added support for that as well. Example link: https://www.techstage.de/test/mini-pc-fuer-190-euro-mit-seltenem-extra-ouvis-gk3-plus-mit-vga-anschluss-im-test/lllvhkg

Since the layout change a new version would be necessary to parse previously crawled articles.

Actually, I did add a new version at first, but then I did decide against it, since the selectors now are just a generalized version of the previous ones and should be completely backwards compatible. The subheadline and summary selectors were just extended and the paragraph selector was also just extended and the existing part generalized.


managing additional layers in fullscreen

Are there any examples of applying different layers present on the original map to the fullscreen version, i.e. heatmap.js?

There is no fullscreen map, it's just a map displayed differently, no bug here, closing ;)


back to back restarts with restart grace overlap each other

I have a config running 2 pumas with a restart grace so that only one is stopped and started at a time.  If restarts are stacked up, things happen like this:

restart1 stops puma1
restart1 starts puma1
restart1 stops puma2 AND restart2 stops puma1 # Now I'm down!
restart1 starts puma2 AND restart2 starts puma1
restart2 stops puma2
restart2 starts puma2

eye doesn't seem to keep track of the restart sequence as a whole in order to properly queue individual restarts separately.

show processes tree, and command what you call

eye config:
RACK_ENV  = ENV['RACK_ENV']  || "staging"
RACK_ROOT = ENV['RACK_ROOT'] || "/export/web/myapp/current"

def puma_factory(proxy, instance)
  name = "puma-#{instance}"
  proxy.process(name) do
    pid_file "/var/run/myapp/#{name}.pid"

    sock_file  = "/var/run/myapp/#{name}.sock"
    state_file = "/var/run/myapp/#{name}.state"

    flags = [ "--dir #{RACK_ROOT}",
              "--environment #{RACK_ENV}",
              "-C #{RACK_ROOT}/config/puma.rb",
              "--pidfile #{pid_file}",
              "--tag puma#{instance}",
              "--bind 'unix://#{sock_file}'" ]

    start_command "bin/puma #{flags.join(' ')}"
    start_grace 30.seconds
    start_timeout 45.seconds

    stop_signals [:TERM, 10.seconds, :KILL]
    stop_timeout 30.seconds
    stop_grace 2.seconds

    stdall  "log/#{name}.log"

    check :cpu, every: 37.seconds, below: 90, times: 3
    check :memory, every: 27.seconds, below: 1000.megabytes, times: [3, 5]
  end
end

Eye.application 'myapp'.to_sym do
  working_dir RACK_ROOT
  env "RACK_ENV"  => RACK_ENV,
      "RAILS_ENV" => RACK_ENV,
      "RACK_ROOT" => RACK_ROOT

  group 'puma' do
    chain :action => :restart, :grace => 40.seconds
    chain :action => :start, :grace => 30.seconds
    trigger :flapping, :times => 10, :within => 1.minute

    1.upto(2) do |instance|
      puma_factory self, instance
    end
  end
end

I've attached info about going through two restarts back to back.
puma_restarts.txt

try:
chain :action => :restart, :grace => 40.seconds, :type => :sync

That helped :)


⚠ Bofrost.be has degraded performance

In 82af19b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1585 ms

Resolved: Bofrost.be performance has improved in e8861b1.


Condition doesn't work if message has an attachments

In file /src/ImageEmbedPlugin.php:21 there is an condition:
if ($message->getContentType() === 'text/html') {         $message->setBody($this->embedImages($message)); }
But if message got an attachments, then returned content type will be multipart/mixed and plugin will not work

Hi @ftw-soft. Thanks for the report.
I added a specific test for the case you describe but the plugin is working as expected.
In a multipart/mixed message, the text/html part will be catch during the following foreach : https://github.com/Hexanet/swiftmailer-image-embed/blob/master/src/ImageEmbedPlugin.php#L25.


[PL-49136] SMP 0.16.0 Release Notes

PL-49136
https://harness.atlassian.net/wiki/spaces/PD/pages/21750284289/Release+Notes+tickets+for+v0.16.0+release
Discussion about this RN can be found at - https://harness.slack.com/archives/C0501BCG1EF/p1713365464426289
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://662274b8a9cb2513cbe1235c--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66229401d7fcda3427909088--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66229435d7fcda34279091b9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6622a5c9cbe67e0cf36ce5a4--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66282eb1b553c2204cb0bb77--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66313aa02cb0ff0c65aabfe9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66313b73b5bd6211651fe5e5--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66313bf3a9c5cb0e2adc6155--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://66313d3a9e2b8e0f390eb0c2--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6632df1ea4e2893d769a96e9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6632df979a91de40df09f371--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6632e493fa96bf3d3e08fbcf--harness-developer.netlify.app


bug/Fix (dead) fonts

Problem

1 we attempt to use UTS themed fonts which we don't have
2 we apply font styling at a local (stylesheet) level, these should be applied globally by default
3 the global IBM font pack from CBS doesn't bloody work

Fix

1, 2 are fixed by simply removing the style usages
3 requires two steps

removing our own resets + normalisations (CBS includes their own)
FIX THE BLOODY THING

Working fonts,

and logs have cleared up!

Renders


Find "llvm-profdata" for the current rust toolchain.

Find "llvm-profdata" for the current rust toolchain.

Thanks for the PR! I'll fix up the lints and get this merged.


Suggestion: Add example scripts

Example scripts allow to quickly see how the library can be used, and what the difference is between the API and the library (advantages/disadvantages).

That's a good idea. Aside from usage examples for specific functions, actual working examples using various parts of flib, with a comparison to the same functionality without flib, would allow people to decide how they will use it. We could stick these in an examples subfolder.

Ideally examples would be generated from the header.
Whoever writes a function could do all documentation work in one place.

Concise usage examples can be included in the documentation, but I think Bilka was suggesting more elaborate examples with real working code?

This is what I imagine Bilka had in mind. Fully integrated examples that modders can download and mess with themselves.

Yep, you got it Raiguard.
Looking at the example, it seems that your library doesn't do valid checking for events it dispatches (you do "entity.valid" checks). If that is the case, it should definitely be documented.

It actually does. If any user data gets invalidated by one of the handler functions, it immediately ceases dispatch.

Some example scripts would be wonderful, especially for the GUI part. I've been trying to replicate the GUI from Rate Calculator because it looks great and it uses flib of course. But I'm having a hard time getting something similar in my mod. Granted I also have no experience with writing GUI's with only the base mod.
All the needed functions are documented well, but it is not clear how they should be used together. Documentation of the base mod also suffers from this. Some basic "Getting Started" would be greatly appreciated.

I have not had an opportunity to work on this project as much as I would have liked recently, so the full-fledged examples aren't in yet. However, I did write a test GUI when I was developing the GUI module.
I hope it helps!

That test demo helped a lot! Together with that and more reading of the Factorio and flib docs I understand the GUI part much better.
I might spend some time writing examples and create a pull-request if you want.

I added example scripts for event, gui, migration, and translation a while ago. I don't believe the other modules would benefit from full-fledged examples, but if I have a spare moment I'll write some for them.


Moving Solo.io Istio to revisions

Issue #,https://github.com/aws-samples/eks-anywhere-addons/issues/252
Description of changes:
This PR:

Transfers the Solo Istio Distro to the solo-istio-system namespace.
Introduces revision support to enable multiple Istio Control Planes to coexist within the same cluster.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thankyou @PetrMc for a rapid quick PR. We will get this validated soon.

Hi @PetrMc , Apologies for the change here but we needed to revert the PR based on a conflict that we suspect it was causing with the other Istio add-on that we are attempting to run/test alongside Solo. We believe that the use of discoverySelector to isolate solo would remedy the issue. If you could please resubmit the PR with discoverySelector implemented to focus on just the Solo solution that would be appreciated. Thanks, and let me know if you have any questions!

@mikemcd3912 / @elamaran11 - please review https://github.com/aws-samples/eks-anywhere-addons/pull/261/


Hyper Majority / SkipDay Vote / Lock Vote

what should be the case:

Have a lock vote mechanic where the day would end when hyper majority of >50% is achieved with locked votes

Have a Hyper Majority mechanic where after reaching hyper majority of (66%?) a 1 min timer would run till end of phase

Skip day vote mechanic where if (66%?) vote this the day would end early

2nd


Update zio, zio-test, zio-test-sbt to 2.0.19

About this PR
📦 Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 1.0.18 to 2.0.19 ⚠
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "dev.zio" }
}]

labels: library-update, early-semver-major, semver-spec-major, commit-count:1

Superseded by #275.


Unable to load extension 'storage-preview' for azure-cli 2.0.71

If the issue is to do with Azure CLI 2.0 in-particular, create an issue here at Azure/azure-cli

Extension name (the extension in question)
storage-preview
Description of issue (in as much detail as possible)
After upgrading to azure-cli 2.0.71, commands that I run gives me a
lphan@thatguylong-2 ~ $ az account set -s < subscription >
Unable to load extension 'storage-preview'. Use --debug for more information.
error message.

The only one that does not seem to produce that error is:
lphan@thatguylong-2 ~ $ az --version
azure-cli                         2.0.71

command-modules-nspkg               2.0.3
core                              2.0.71
nspkg                              3.0.4
telemetry                          1.0.3

Extensions:
storage-preview                    0.1.6

Python location '/usr/local/Cellar/azure-cli/2.0.71/libexec/bin/python'
Extensions directory '/Users/lphan/.azure/cliextensions'

Python (Darwin) 3.7.4 (default, Aug 15 2019, 06:16:55)
[Clang 11.0.0 (clang-1<IP_ADDRESS>)]

Legal docs and information: aka.ms/AzureCliLegal

Your CLI is up-to-date.

Here is the debug:
Found 1 extensions: ['storage-preview']
Extensions directory: '/Users/lphan/.azure/cliextensions'
Extension compatibility result: is_compatible=True cli_core_version=2.0.71 min_required=2.0.43 max_required=None
Event: CommandLoader.OnLoadCommandTable []
Event: MainLoader.OnFailedExtensionLoad []
Unable to load extension 'storage-preview'. Use --debug for more information.
Traceback (most recent call last):
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/__init__.py", line 202, in _update_command_table_from_extensions
    _load_extension_command_loader(self, args, ext_mod)
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 958, in _load_extension_command_loader
    return _load_command_loader(loader, args, ext, '')
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 944, in _load_command_loader
    command_table = command_loader.load_command_table(args)
  File "/Users/lphan/.azure/cliextensions/storage-preview/azext_storage_preview/__init__.py", line 33, in load_command_table
    load_command_table(self, args)
  File "/Users/lphan/.azure/cliextensions/storage-preview/azext_storage_preview/commands.py", line 36, in load_command_table
    custom_command_type=storage_account_custom_type) as g:
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/__init__.py", line 446, in command_group
    return self._command_group_cls(self, group_name, **kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 1058, in __init__
    merged_kwargs = self._merge_kwargs(kwargs, base_kwargs=command_loader.module_kwargs)
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 1082, in _merge_kwargs
    return _merge_kwargs(kwargs, base, CLI_COMMAND_KWARGS)
  File "/usr/local/Cellar/azure-cli/2.0.71/libexec/lib/python3.7/site-packages/azure/cli/core/commands/__init__.py", line 1033, in _merge_kwargs
    raise TypeError('unrecognized kwargs: {}'.format(unrecognized_kwargs))
TypeError: unrecognized kwargs: ['min_profile']

Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x1108a1170>]

@Juliehzl Please take a look

add to S173

Hi @thatguylong, sorry for late response because it is missed during transition. There is breaking change in storage-preview extension. please use az extension update -n storage-preview to upgrade your extension. otherwise you have to downgrade your azure-cli because the breaking change in azure cli 2.0.67 see here https://github.com/Azure/azure-cli/pull/9635.


Adding optional support for setting the cookie domain for JWT authentication tokens

Users using SPA commonly set their backend in a separate subdomain like for example api.example.com. When the JWT auth token cookie is set, it will only be set on that domain (api.example.com).
With this update users can have the option to change the domain where the auth cookies are set thus allowing the cookies to be set for all subdomains. It also fixes issues when Server Side Rendering pages in frameworks like NextJs where the SSR part of the application has no access to the auth cookies thus can't forward them to the Django backend.
Fixes issue #238
Thanks for your time.
PS: It's my first contribution on GitHub :)

Thanks for your contribution @adrenaline681

I see that JWT_AUTH_COOKIE_DOMAIN is merged to master. But I can't see it in https://dj-rest-auth.readthedocs.io/en/latest/configuration.html#. Is readthedocs updated?


Automatic conflict resolution

Right now, it's common for any two concurrent update/delete transactions to have one return CommitConflict error. This makes it hard to make many updates. The best workaround for now is to do updates serially, but it would be nice if it wasn't required.
We have a retry loop for if transactions are compatible. If they need resolution, however, we just return an error. What if instead we can automatically resolve the conflicts?
Resolving conflicts
We would like to have an API that can take a sequence of transactions, and merge them into a single transaction consolidating all changes:
fn resolve_conficts(
    dataset: &Dataset,
    transactions: &[Transaction],
) -> Result<Transaction>;

In the commit loop, transactions would just be a single transaction. But in other use cases, we might want to be able to consolidate all the changes into a single transaction.
This loop would need to load all the previous transactions since each pending transactions read_version. To make this faster, we should consider the caching described in https://github.com/lancedb/lance/issues/3057
A few examples of what this could do can be illustrative:

Two concurrent deletes: each delete transaction will have their own deletion files, and potentially removed fragments. Should union set of removed fragment ids. Should write a new deletion file that is a union of the two.
A concurrent append and delete: should transform in to a transaction that has both
Two concurrent appends: Should handle modified fragments the same way as deletions. And then combine appends the same way.
Add column and append: if new column is nullable, we will be able to append subschema.

What if it is not nullable? Then it will fail?

Overwrite and append: append can be ignored. In fact, any other transaction could be ignored.

This makes sense.  Both A and B are happening at the same time.  We should consider what would happen if they were not concurrent.  I think there are several situations:

Both A then B and B then A have the same output: We can safely merge
A then B has different output from B then A and both are valid: We should have some kind of user-flag here (I think databricks calls this WriteSerializable isolation level).  If the flag is set then we can pick one of the outputs and merge.  If not, we should raise an error.
A then B fails but B then A succeeds: We could maybe use the same flag.  Or we could just force the B then A order.
A then B fails and B then A fails: We should fail in this situation

Two concurrent deletes

Both orders have the same output.  We can merge.

A concurrent append and delete

Append then delete would have output X
Delete then append would have output Y
We should only merge is user flag is set to allow this kind of thing.  Probably just merge to "Delete then Append" because that's the only one we can calculate (since we don't store delete filter on transaction).

Two concurrent appends

Both orders have the same output.  We can merge.

Add column and append: nullable column

Both orders have the same output.  We can merge.

Add column and append: non-nullable column

Both directions fail.  We can raise an error.

Overwrite and append

Overwrite then append would fail (or at least could, if schema was changed)
Append then overwrite would succeed
Probably don't need to check user flag here and just assume it was "append then overwrite"

What if it is not nullable? Then it will fail?

Yes, I think failure makes sense here.  In the other cases we are just restoring what would happen if the two operations ran serially without conflict.  e.g. if the two deletes conflict and get merged you get the same result as you would if the second delete had the result of the first delete as the read version.
So if we apply that same logic here we get an error in both cases.  Either "you ran add_columns and didn't give a value for all rows" or "you ran append after the add columns and the append schema was missing nullable columns".


Use parse_version to compare versions conforming to PEP-440

from pkg_resources import parse_version and use to to compare structured versions conforming to https://www.python.org/dev/peps/pep-0440/.
This will provided the following ordering:
'i.j(.|-)dev[N]' < 'i.j.rc[N]' < 'i.j.k' < i.j(.|-)post[N]
So we can use 'i.j.dev' for the master branch, before it's forked to branch-i.j that will initially versioned as i.j.rc0 and so on until it becomes i.j.0.
Cc @avikivity @penberg @fruch
See also usage in https://github.com/bhalevy/scylla-dtest/commits/parse_version

Both branches passed next-gating on my desktop until it ran out of space.
https://jenkins.scylladb.com/job/scylla-master/job/releng-testing/job/byo/job/byo-dtest/66 tests full dtest-release.

@bhalevy did you test it with 2019.1 version strings ?

@bhalevy did you test it with 2019.1 version strings ?

yes, manually...
$ python
Python 3.8.5 (default, Aug 12 2020, 00:00:00)
[GCC 10.2.1 20200723 (Red Hat 10.2.1-1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from pkg_resources import parse_version
>>> parse_version('2019.1.0') > parse_version('3.0')
True
>>> '2019.1.0' > '3.0'
False


Update zIndex tokens

Update zIndex tokens - use camel case and rename tooltip to flyover

needs changeset, will add when i can


Harden securityContext of Deployments and StatefulSets

Issue
Check that the deployed resources have the maximum security context needed to run on CaaS Clusters.

The bug was falsely reported and after deploying caas-project-monitoring (the 1.2.3 version) to an OTC, AWS and FCI cluster. All deployments and statefulsets started successfully. I'll check whether the next audit catches them, but deployment seems to be working fine.

Closing this for now, as the helm chart works as intended.


Separate BPG::Context

Codecov Report

Merging #36 into master will decrease coverage by 0.07%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #36      +/-   ##
==========================================
- Coverage   98.23%   98.15%   -0.08%
==========================================
  Files           5        6       +1
  Lines         170      163       -7
  Branches        6        5       -1
==========================================
- Hits          167      160       -7
  Misses          3        3

Impacted Files
Coverage Δ

HCImage+BPG/Internal/CG/DataProvider.cpp
100% <ø> (ø)

HCImage+BPG/HCImage+BPG.mm
97.19% <ø> (-0.35%)
:arrow_down:

HCImage+BPG/Internal/CG/Image.cpp
100% <ø> (ø)

HCImage+BPG/Internal/CG/ColorSpace.cpp
100% <ø> (ø)

HCImage+BPG/Internal/BGP/Context.cpp
100% <100%> (ø)

Continue to review full report at Codecov.

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


Refactor database models to function same as other models.

I was running into issues trying to write tests for User and needing to mock Event, and not being able to since they were all loaded from index.js.
This should simplify it substantially, each database model is now a single file and can be required separately from the others.

Pull Request Test Coverage Report for Build 68

53 of 70 (75.71%) changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.5%) to 80.408%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

models/event.js
15
22
68.18%

models/user.js
26
36
72.22%

Totals

Change from base Build 66:
1.5%

Covered Lines:
143

Relevant Lines:
168

💛  - Coveralls

This will force changes to #4 but I can deal with those conflicts


Add support for memray --version and maybe memray -V

Allow users to ask Memray to report its own version.

Note that, until we implement this feature, python -c "import memray; print(memray.__version__)" is a workaround.


Include RKE2 charts and images in air-gap setup

There are new images (like rancher-machine) and new charts (like RKE CNI charts) that need to be included for air-gapped environments. It is likely that the images and charts should be included in KDM and pulled during the build process.

This is dependent on RKE2 team creating an "all-in-one" images.txt. Then Rancher team can grab it based on given version of RKE2.

This exists now. We need to add a setting for which version of rke2 to use, and given that setting we can download the rke2 images.txt and append it to ours: https://github.com/rancher/rke2/releases/download/v1.21.0-alpha3%2Brke2r1/rke2-images-all.linux-amd64.txt
This should also ensure the rancher-images-sources.txt list has the source set as rke2.

Consider adding milestone 2.5.9 to this issue.

@ron1 the solution going into 2.6 will likely be quite different than solution going into 2.5. If thats true its less likely for us to formulate a path for 2.5, since it will essentially be a deprecated path.

Dependent on https://github.com/rancher/rke2/issues/1077 being resolved. Once this issue is resolved, the following images will be mirrored.
docker.io/calico/cni
docker.io/calico/kube-controllers
docker.io/calico/node
docker.io/calico/pod2daemon-flexvol
docker.io/calico/typha

The difference in CI log output regarding k3s-upgrade images being in the list of generating images was because we only install golangci-lint for amd64 (because the checks are not arch dependent so it only has to run on one arch), and in our scripts/validate we have the following snippet:
if ! command -v golangci-lint; then
	echo Running: go fmt
        echo Skipping validation: no golangci-lint available	test -z "$(go fmt ./... | tee /dev/stderr)"
        exit
fi

Where we run go fmt for the entire repository, but we just exit after which ends with exit 0 (regardless if go fmt returns errors). Because of the dirty repo, scripts/version marks the TAG as commit_hash-dirty which makes the version comparison pass for when KDM data is parsed and compared to the min/max versions specified in KDM data and the images get added to the list. On amd64, we end up with just the commit_hash as the TAG and then the version comparison failed and the images did not get added. The version parsing was not specifically tested/validated but based on the test it seems that that was the reason.
The fix will be to make sure go fmt runs and actually fails when something is found. In the process of doing that we also will upgrade golangci-lint to latest version and update to go-skel latest standards.

Nice writeup! One piece I am still confused about though is why its running on both amd64 and arm64, and which one actually gets attached as the asset (or if its whichever finishes last). Can't we just run it on amd64 since it is OS agnostic.

RE: https://github.com/rancher/rancher/issues/32403#issuecomment-853443837 from @cbron
Spoke to @StrongMonkey and an ARCH check has been added for image generation since it only needs to run on amd64.

QA Validation
Pre-requisite changes have been merged, but this will require two things:

[ ] Rancher v2.6 RC (cannot be tested with dev branch unless generated manually)
[x] RKE2 1.21+ Tag that includes the rke2-images-all.linux-amd64.txt file and has all images prefixed with rancher/

For the merged solutions to be valid, the rancher-images.txt file must include all images in rke2-images-all.linux-amd64.txt for the Rancher RC.

Available to test in v2.6.0-rc1

blocked by https://github.com/rancher/rancher/issues/34106

blocked by this issue https://github.com/rancher/rancher/issues/34265 on 2.6.0-rc3

For more context on https://github.com/rancher/rancher/issues/32403#issuecomment-900417364: Fleet v0.3.6-rc11 was a dead-on-arrival RC due to Windows automation issues. The manifest for rc11 DockerHub images was not published as a result. It has been fixed with rc12, which requires another Rancher RC.

Available for testing in 2.6.0-rc4

blocked by: https://github.com/rancher/rancher/issues/32600#issuecomment-901341667 (unable to provision rke2 cluster in airgap setup)

this image rancher/system-agent-installer-rke2:v1.21.3-rc6-rke2r2 is in rancher-images.txt and is tagged in the registry like all the other images, <registry>/rancher/system-agent-installer-rke2:v1.21.3-rc6-rke2r2.  But when creating an RKE2 cluster in an airgap setu (2.6.0-rc5) the pull fails in rancher-system-agent , as it is expecting <registry>/docker.io/rancher/system-agent-installer-rke2:v1.21.3-rc6-rke2r2

on v2.6.0-rc6, I am able to:

bring up rke2 cluster v1.21.3-rc7+rke2r2 successfully -- pass
briing up rke1 cluster v1.21.4 successfully -- pass


Mempool locking fixes

Fix some mempool locking issues mentioned in the #526 and related to the #533.

Vote for having tests for that.

I don't think you can have reliable unit tests for such problems, we have TryAdd and TryGetValue mostly covered but that didn't prevent this in any way.


Is it possible to load the prompt from a file ?

Instead of typing out the whole query in terminal, is it possible to write the query in a text file and have the aider read from it?

Thanks for trying aider and filing this issue.
Yes, you can use --message-file <filename> to do this. Also try running aider --help for a list of all the options.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Delete button: method is null

I am trying to simply delete a model listed in a regular/common screen.
These are the routes:
// routes/platform.php file:

Route::screen('phase-templates', PhaseTemplateScreen::class)
    ->name('platform.phase-templates');
Route::screen('phase-templates/create', PhaseTemplateEditScreen::class)
    ->name('platform.phase-templates.create');
Route::screen('phase-templates/{phaseTemplate}/edit', PhaseTemplateEditScreen::class)
    ->name('platform.phase-templates.edit');
// This is the remove route:
Route::screen('phase-templates/remove', PhaseTemplateEditScreen::class)
    ->name('platform.phase-templates.remove');

The button is declared like this:
// app/Orchid/Layouts/PhaseTemplate/PhaseTemplateListLayout.php file:
 /**
     * Get the columns elements to be displayed.
     *
     * @return TD[]
     */
    protected function columns(): iterable
    {
        return [
          // other lines and...
            TD::make(__('Actions'))
                ->align(TD::ALIGN_RIGHT)
                ->width('100px')
                ->render(function (PhaseTemplate $template) {
                    return DropDown::make()
                        ->icon('options-vertical')
                        ->list([
                            Link::make(__('Edit'))
                                ->route('platform.phase-templates.edit', $template->id)
                                ->icon('pencil'),
                            Button::make(__('Delete'))
                                ->icon('trash')
                                ->confirm(__('Once the template is deleted, all of its resources and data will be permanently deleted.'))
                                ->method('remove', [
                                    'id' => $template->id,
                                ]),
                            ]);
                    }
                ),
        ];
    }

So, essentially, if I write the route as above, the error is "method on Screen is null":
Orchid\Screen\Screen::callMethod(): Argument #1 ($method) must be of type string, null given, called in /project/vendor/orchid/platform/src/Screen/Screen.php on line 220

If I change the route to be like this:
Route::screen('phase-templates/{phaseTemplate}/remove', PhaseTemplateEditScreen::class)
    ->name('platform.phase-templates.remove');

The error changes to:
The POST method is not supported for this route. Supported methods: GET, HEAD.

Which makes no sense at all, since I'm registering a Screen route. So, POST should be already in place. Either way, the URL being called by the button click is POST to:
http://localhost:8000/admin/phase-templates/remove?id=6

So, my question is: how should I implement a delete button? What am I doing wrong?
Composer infos:
"laravel/framework": "^9.0",
"orchid/crud": "^3.7",
"orchid/platform": "^12.6",
Thank you all for any help/assistance.

if you are in a model edit screen, the route should be:
Route::screen('phase-templates/{phaseTemplate}/remove', PhaseTemplateRemoveScreen::class)
    ->name('platform.phase-templates.remove');

Be sure to have the remove method in your class, and if I am not wrong, the button should be like this:
Button::make(__('Delete'))
    ->icon('trash')
    ->confirm(__('Once the template is deleted, all of its resources and data will be permanently deleted.'))
    ->method('remove')
    ->parameters(['id' => $user->id])

Also, remember to run the command php artisan clear:optimize

Documentation https://orchid.software/en/docs/screens/#routing-screens and https://orchid.software/en/docs/screens/#screen-actions
No need to add an extra route, that is, this
Route::screen('phase-templates/remove', PhaseTemplateEditScreen::class)
    ->name('platform.phase-templates.remove');

no need.
You need to add method "remove" to screen "PhaseTemplateScreen" (and also in "PhaseTemplateEditScreen", if the edit screen also has a delete button).
Example of a layout that has a button - https://github.com/orchidsoftware/platform/blob/b619ec0bf938483ecfac435bd6e3a0fd619ba251/stubs/app/Orchid/Layouts/User/UserListLayout.php#L63-L68
Screen example with method - https://github.com/orchidsoftware/platform/blob/b619ec0bf938483ecfac435bd6e3a0fd619ba251/stubs/app/Orchid/Screens/User/UserListScreen.php#L112-L117
If the method is used in several screens, for example, "list" and "edit", then it is better to make a base screen class that will have a delete method, and a screen "list" and "edit" through inheritance from the base one (although all this is in the documentation).


Fix time series

fixes #2851

@anthonydresser I would like to test this, so I'll merge now.  Change looks good to me.


feat: proof parser + query-wise verifier

This PR proposes a draft implementation of a FRI verifier which runs the query phase of the FRI protocol in a query-by-query fashion. To do this, this PR also includes a potential modification to the verifier part of the channel so that the proof generated by the prover, which looks at all the queries in certain layer in one go, can be parsed for the new verifier implementation in this PR.
The interface of both the unbatch function as well as the verify_generic_query function is, in my opinion, suboptimal. The reason being that, in order to parse the proof, the unbatch function requires the array of queried positions and this makes the design of the interfaces, based on the current one, a challenge.

One more thought regarding verify query: instead of passing a Merkle path per layer, let's pass a struct which closely mimics the mtree_get instruction. That is, it exposes a function with roughly this interface:
fn get_node(root: Digest, depth: usize, index: usize) -> Digest

The above function would return the value of the node from the tree with the specified root. The node would be located in the tree at the specified index and depth. If the tree with the specified root is not present in the struct, or the specified node is not present in the tree - the function should panic.
To implement this struct, you can probably adapt the code form AdviceProvider and MerklePathSet structs.

Thank you Bobbin!
Unbatch PR: It makes sense, I have gathered the code as well as relevant tests into a separate PR.
Verify query: I have factored the portion you mentioned into a separate function let me know if it is what you were thinking.
FRI changes: That should be straightforward, I'll work on it.

One more thought regarding verify query: instead of passing a Merkle path per layer, let's pass a struct which closely mimics the mtree_get instruction. That is, it exposes a function with roughly this interface:
fn get_node(root: Digest, depth: usize, index: usize) -> Digest

The above function would return the value of the node from the tree with the specified root. The node would be located in the tree at the specified index and depth. If the tree with the specified root is not present in the struct, or the specified node is not present in the tree - the function should panic.
To implement this struct, you can probably adapt the code form AdviceProvider and MerklePathSet structs.

Here I am bit not sure about what you mean. More specifically:

Should get_node return a Word instead?
Should the data structure in question include all Merkle paths for all layers?
How does this change the way we do the verification of the Merkle proofs?
Thank you!

@Al-Kindi-0 Thank you! I'll leave some comments in the other PRs. To answer the above questions:

Word and Digest are basically the same thing for Rescue Prime (a digest consists of 4 field elements). I think it should be relatively easy to convert from one to the other.
Yes, it would contain all paths for all layers. We should probably organize it similar to AdviceProvider where we have AdviceSets indexed by root, and then within each advice set we have a collection of paths. To simplify things, we could probably just use MerklePathSet struct - since we know that in our case, we won't have any other types of advice sets.
I think we should assume that if get_node returns a value and doesn't panic, then the path for the node in question has been verified. But maybe I misunderstood the question?


(Optionally) Use ZooKeeper instead of MySQL for sender election

Use ZK with Kazoo instead of MySQL as we had problems with our Galera setup

Catch kill signals to immediately remove from cluster

Bring back support for hard coding in master and slave configuration in
config files. This new ZK support is 100% optional and does not give us a hard
dependency on ZK.

Add new NonClusterCoordinator class to provide for hard coded cluster configs
easily

so production percona test didn't go well?

@houqp it didn't work out in prod. Master status kept being switched through nodes even though none were down.

Github wouldnt let me update this PR so I made this new PR instead. https://github.com/linkedin/iris/pull/231


Can not add repo

I am getting the following error any help?
21-01-05 21:52:15 ERROR (MainThread) [supervisor.store] https://github.com/LaggAt/hacs-govee is not a valid add-on repository
21-01-05 21:52:15 ERROR (MainThread) [supervisor.utils.json] Can't read json from /data/addons/git/217cf69b/repository.json: [Errno 2] No such file or directory: '/data/addons/git/217cf69b/repository.json'

Same issue here. Though I'm trying to add this first time and get Invalid Add-on repository!.

Same issue here. Though I'm trying to add this first time and get Invalid Add-on repository!.

I found out that you have to install HACS then add the repo from there.

I found out that you have to install HACS then add the repo from there.

I was being a numpty. Adds the repo fine.

I was being a numpty. Adds the repo fine.

I was being a numpty. Adds the repo fine.

Yup

I was being a numpty. Adds the repo fine.

Yup

yes, it's a hacs repo. Glad you figured that.


StopIteration: points is not a numpy array, neither a scalar

Hi again,
I want to train on VOCpascal and get this error message. From an earlier, similar issue, it had something to do with OpenCV, but this one seems different.
Sharing experiences or ideas will be highly appreciated !!

.
.
Total params: 36,213,592
Trainable params: 36,107,352
Non-trainable params: 106,240

None
Epoch 1/50
Traceback (most recent call last):
File "examples/train_pascal.py", line 105, in
keras.callbacks.ReduceLROnPlateau(monitor='val_loss', factor=0.1, patience=10, verbose=1, mode='auto', epsilon=0.0001, cooldown=0, min_lr=0),
File "/usr/local/lib/python2.7/dist-packages/keras/legacy/interfaces.py", line 87, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/keras/engine/training.py", line 2046, in fit_generator
generator_output = next(output_generator)
File "/usr/local/lib/python2.7/dist-packages/keras/utils/data_utils.py", line 518, in get
raise StopIteration(e)
StopIteration: points is not a numpy array, neither a scalar

versions:
python 2.7.6
keras 2.0.9

What is your OpenCV version?

2.4.8

Could you try OpenCV 3+?
On Nov 7, 2017 18:29, "nealzGu"<EMAIL_ADDRESS>wrote:

2.4.8
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/delftrobotics/keras-retinanet/issues/43#issuecomment-342559254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AArtav_ZJBCvcsXVJsonz1p7k1CKh3R6ks5s0JOKgaJpZM4QVJ75
.

having quite some problems with this ... try to fix it tomorrow!

Alternatively, if you change the code there such that it is not using OpenCV (at the point where the error comes from) and make a PR out of it I'd be happy to merge it.

Luckily that's not necessary - it works now!
For everybody who is experiencing the same error, this worked for me:
remove existing openCV with
sudo apt-get autoremove opencv-doc opencv-data libopencv-dev libopencv2.4-java libopencv2.4-jni python-opencv libopencv-core2.4 libopencv-gpu2.4 libopencv-ts2.4 libopencv-photo2.4 libopencv-contrib2.4 libopencv-imgproc2.4 libopencv-superres2.4 libopencv-stitching2.4 libopencv-ocl2.4 libopencv-legacy2.4 libopencv-ml2.4 libopencv-video2.4 libopencv-videostab2.4 libopencv-objdetect2.4 libopencv-calib3d2.4
then just do:
pip install opencv-python
there are a lot of other complicated descriptions out there that didnt work for me, this is the simplest one and worked fine. Will close this issue, thanks again!


ci: use .go-version file

Even if folks don't use goenv to manage their Go versions, this means CI is reading from a single file making it easier to update the version of Go being used across all workflows.

Codecov Report

Merging #564 (36db95b) into main (12331be) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #564   +/-   ##
=======================================
  Coverage   78.64%   78.64%
=======================================
  Files          77       77
  Lines        5273     5273
=======================================
  Hits         4147     4147
  Misses        963      963
  Partials      163      163


[PLAT-844] Modify language on confirmation modal for making registration

Purpose
This is a language fix that makes it obvious that registrations are immutable when registering a draft.
Changes

adds immutability prompt to modal
removes "tags" prompt

QA Notes
create a draft and register it, you should see this:

Documentation
Shouldn't be necessary
Side Effects
None that I know of.
Ticket
https://openscience.atlassian.net/browse/PLAT-844

Cherry-picked into a new branch and merged into release/next-platform.


Remove coins parameter

massa-web3 version needs to be bumped as well for this to work. This can be done once https://github.com/massalabs/massa-web3/issues/62 is tackled.

We also need to update the dependencies and the unit tests


Tooltip which displays time up to seconds precision

💻 Environment
Platform: macOS/Windows
📒 Description
When hovering over Start time/End time fields it would be nice to see values with more precision, up to seconds. Hovering over these fields currently does nothing anyway.
⭐ Why do you want this?
As a user, I'd like to view more detailed information about my time entries.

Moved Linux part to a separate issue #3366


Port/combinatorics.quiver.single obj

There's one error left about deriving Unique. I also needed to add two minor lemmas to be able to fix some proofs, ext and path_to_list_cons. Should I backport them to mathlib3 or just add a porting note?

Note that the start_port.sh posts a URL that will allow you to open a PR with a title that's already pre-formatted.

Thanks!
bors r+


Don't escape all fixes 3315 without breaking Amazon URLs

This fixes #3315 without breaking other URLs.

Frontend tests were OK :+1: (details)

:+1: apart from the comment/idea

Frontend tests were OK :+1: (details)


memcached: 1.6.5 -> 1.6.6

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/memcached/versions.
meta.description for memcached is: "A distributed memory object caching system"
meta.homepage for memcached is: "http://memcached.org/"
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.6.6 with grep in /nix/store/n8f8bh3wp1sxaxx6szrysx039aqi6j8z-memcached-1.6.6

directory tree listing: https://gist.github.com/b089af8b95a4258050fc6f5f8d9a8d77

du listing: https://gist.github.com/44c29707f90960e1af7e10ae23356fab

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
memcached

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/n8f8bh3wp1sxaxx6szrysx039aqi6j8z-memcached-1.6.6 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A memcached https://github.com/r-ryantm/nixpkgs/archive/bc8432f6411eb2de66001f37ebde6c41569087a1.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/n8f8bh3wp1sxaxx6szrysx039aqi6j8z-memcached-1.6.6
ls -la /nix/store/n8f8bh3wp1sxaxx6szrysx039aqi6j8z-memcached-1.6.6/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- memcached

Maintainer pings
cc @coreyoconnor for testing.

@GrahamcOfBorg test memcached


iRate SKStoreReviewController

Would be great if there is an option to support iOS 10.3+ SKStoreReviewController.RequestReview() by the iRate component. Otherwise this component is not allowed by the Apple AppStore Guidelines.

1.1.7 App Store Reviews:
Use the provided API to prompt users to review your app; this functionality allows customers to provide an App Store rating and review without the inconvenience of leaving your app, and we will disallow custom review prompts.

Source: https://developer.apple.com/app-store/review/guidelines/
Currently I use a overwrite for the ShouldPromptForRating Delegate but that seems to be not a clear solution because it prompts on TestFlight users with the wrong dialog because SKStoreReviewController.RequestReview() does not prompt for TestFlight.
My current implementation:
iRate.SharedInstance.ShouldPromptForRating = (arg1, arg2) =>
	{
#if DEBUG
		return false;
#endif
		if (UIDevice.CurrentDevice.CheckSystemVersion(10, 3))
		{
			try
			{
				SKStoreReviewController.RequestReview();
			}
			catch (Exception e)
			{
				Crashes.TrackError(e);
			}

			return false;
		}

		return true;
	};

@Sebastian1989101  Thank you for your feedback and this is something we can look into.  In the meantime, if you have a work around you could submit a Pull request with the changes.
Thanks

iRate has now been deprecated and there will not be any further updates for the package.


Tor issues

On the contact page, copyright is offcenter and email displays as "Email Protected"

Looks like email protection is coming from Cloudflare, claiming protection from spammers. This can probably be solved with an email form rather than just a button.

Can't find any other bugs at the moment


LXD support on Centos

Hello,
it s not really an issue. I need to set up lxd on centos server. This package is not on the standard repository. So , my question is : does LXD support on Centos. If yes, i want to know how setting up ?
Best regards

I'm not sure how interoperable Fedora and CentOS are but @ganto has a Fedora repo (https://github.com/ganto) and he might also now more.
ping @stgraber, you recently had a list of distros that have a package.

Ok , i m gonna look it.
My purpose  :
i want a centralized server which backup LXD containers ( containers to tarball). To do it, i need LXD dameon and commands ( lxc snapshot, publish and export). I got a server under centos and i don't want to change it's distro.

If you are using Fedora, you can install LXD from my ganto/lxd COPR repository.
Originally I intended to also build the packages for CentOS 7. But unfortunately CentOS is missing a lot more golang package dependencies of LXD. And since I don't intend to use LXD on CentOS, I wasn't happy about the additional effort and therefore dropped the idea again. Also I wouldn't feel comfortable offering packages that I don't personally use and care.
As the lxc tools (/usr/bin/lxd, /usr/bin/lxc) are technically statically linked executables, it might be possible to use the RPM from Fedora. lxcfs might already (or with minimal changes) build for CentOS 7 with the spec file from my repository.

I've helped someone sort out LXD on Centos 7 before and the following kinda worked:
yum install wget rsync libseccomp golang sqlite-devel git dnsmasq squashfs-tools
wget https://kojipkgs.fedoraproject.org//packages/lxc/2.0.7/1.fc24/x86_64/lxc-libs-2.0.7-1.fc24.x86_64.rpm
wget https://kojipkgs.fedoraproject.org//packages/lxc/2.0.7/1.fc24/x86_64/lxc-devel-2.0.7-1.fc24.x86_64.rpm
rpm -ivh lxc-libs-2.0.7-1.fc24.x86_64.rpm
rpm -ivh lxc-devel-2.0.7-1.fc24.x86_64.rpm
export GOPATH=/root/go && export PATH=/root/go/bin/:$PATH
vi ~/.bash_profile (update with GOPATH & new PATH)
go get -v -x -tags libsqlite3 github.com/lxc/lxd/lxc
go get -v -x -tags libsqlite3 github.com/lxc/lxd/lxd
grubby --args="user_namespace.enable=1" --update-kernel="$(grubby --default-kernel)"
reboot
lxd --debug (in a separate terminal)
lxd init

Note that this is obviously very hackish, but it did result in a working LXD.

Thanks four your answers. When i get time, i ll set LXD on my Centos.
I ll give you my feedback.

I've been working up some installation instructions that include installing LXD on Centos. You  might find this useful as well: https://gist.github.com/AdamIsrael/e6b4a540233a7be6d28decd917b60c3d

@AdamIsrael
The gist on your link worked for me on CentOS 7, with the minor fix of the user and group missing.

Any chance of LXD supporting CentOs better? I ask because CentOs is the defacto install  for so many Enterprise environments. I don't mind there being more packages to install or something. Just a cleaner install with all the features working would be nice. The one that I see is a hack....

I've made an script based on some online solutions. It worked for me. I hope it would be useful for you.
centosLXD.sh.txt
#!/bin/sh

#LXD on CENTOS 7
# Made by Jeovane : 29/09/2017
# jeovanevs at gmail
# Based on:
# https://www.hiroom2.com/2017/03/24/centos-7-run-containers-with-lxd-lxc/
# https://github.com/lxc/lxd/issues/2984
# https://gist.github.com/AdamIsrael/e6b4a540233a7be6d28decd917b60c3d

# find /lib/systemd/system -maxdepth 1 \
# -type f -exec sed -e 's/umount\.target//g' -i {} \;
# systemctl --system daemon-reload

sudo yum install -y epel-release wget
###############################################################################
# 							LXC install methods:
# ####################################[1]######################################
# #[Install LXC 2.0 from fedora repo]
# #Download packages
# sudo yum install -y rpmdevtools rpm-build graphviz
# F=https://dl.fedoraproject.org/pub/fedora/linux/releases/26
# L=${F}/Everything/source/tree/Packages/l
# RPM = lxc-2.0.8-2.fc26.src.rpm
# wget -q ${L}/${RPM}
# # Rebuild deps
# sudo yum-builddep -y ${RPM}
# rpmbuild --rebuild ${RPM}
# # Install package.
# # shellcheck disable=SC2046
# sudo yum localinstall -y $(find ~/rpmbuild/RPMS -type f -a ! -name "*debuginfo*")
#
# ####################################[2]######################################
# #[Install LXC 2.0 from fedora Koji server]
# # Koji is the official package build server(s) for the Fedora Project. Every rpm
# # package that ends up in the official Fedora repositories (3rd party rpms such as
# # RPM Fusion excluded) is built on the koji build server. The only difference
# # between the packages on koji and those in the yum repositories are that the
# # ones on koji have not been "signed" with the gpg key. Search packages in:
# # https://koji.fedoraproject.org/koji/packageinfo?packageID=8889
#
# # download packages
# # base url
# B=https://kojipkgs.fedoraproject.org//packages/lxc/
# # lxc version
# V=2.0.8
# # Fedora releases x.fc, x=1 (alpha), x=2 (beta)
# R=2.fc27
# # Arch
# A=x86_64
# # download Lib and devel packages
# wget ${B}/${V}/${R}/${A}/lxc-libs-${V}-${R}.${A}.rpm
# wget ${B}/${V}/${R}/${A}/lxc-devel-${V}-${R}.${A}.rpm
# # Install packages.
# rpm -ivh lxc-libs-${V}-${R}.${A}.rpm
# rpm -ivh lxc-devel-${V}-${R}.${A}.rpm
#
####################################[3]########################################
#[Install LXC 2.0 from COPR repositories]
# Copr (Cool Other Package Repo) is a Fedora project to help make building and
# managing third party package repositories easy. The instance to be installed
# within Fedora Infrastructure provides Fedora maintainers with the ability to
# create repos of packages to build against and share with others.
sudo yum -y install python-requests
# install copr plugin:
# https://copr.fedorainfracloud.org/coprs/alonid/yum-plugin-copr/
rpm --nodeps -i \
https://copr-be.cloud.fedoraproject.org/results/alonid/yum-plugin-copr/\
epel-7-x86_64/00110045-yum-plugin-copr/\
yum-plugin-copr-1.1.31-508.el7.centos.noarch.rpm
# enable thm/lxc2.0 repository: https://copr.fedorainfracloud.org/coprs/thm/lxc2.0/
sudo yum -y copr enable thm/lxc2.0
# Install packages.
sudo yum -y install lxc lxc-devel
###############################################################################

#[Install LXD methods: download packages or compile from source]
sudo yum install -y rsync libseccomp golang sqlite-devel git dnsmasq squashfs-tools golang-bin make libacl-devel
# make go packages dir
sudo mkdir /root/go
#Add to path
export GOPATH=/root/go && export PATH=${GOPATH}/bin/:$PATH

# # Method 1: [get LXD packages from github]
# go get -v -x -tags libsqlite3 github.com/lxc/lxd/lxc
# go get -v -x -tags libsqlite3 github.com/lxc/lxd/lxd

# Method 2: [Build LXD from source]
go get github.com/lxc/lxd
cd $GOPATH/src/github.com/lxc/lxd
make

#Add execs to /usr/bin (copy or link)
# sudo cp "${GOPATH}"/bin/* /usr/bin/
sudo ln -s $GOPATH/bin/* /usr/bin

# Running container needs user_namespace.enable=1.
# # method 1: need to reboot
# grubby --args="user_namespace.enable=1" --update-kernel="$(grubby --default-kernel)"
# reboot

# method 2: install grub2-tools and update config
sudo yum install -y grub2-tools
. /etc/default/grub
V="$GRUB_CMDLINE_LINUX user_namespace.enable=1"
sudo sed -e "s;^GRUB_CMDLINE_LINUX=.*;GRUB_CMDLINE_LINUX=\"$V\";g" \
-i /etc/default/grub
sudo grub2-mkconfig -o /boot/grub2/grub.cfg

# Start LXD manually or create a service
# # manually
# lxd --debug #(in a separate terminal)

# Create systemd service and socket.

sudo su -c '
cat <<EOF > /usr/lib/systemd/system/lxd.service
[Unit]
Description=LXD Container Hypervisor- main daemon
After=network.target
Requires=network.target lxd.socket network.service
Documentation=man:lxd(1)

[Service]
EnvironmentFile=-/etc/environment
ExecStart=/usr/bin/lxd --group lxd --logfile=/var/log/lxd/lxd.log
ExecStartPost=/usr/bin/lxd waitready --timeout=600
KillMode=process
TimeoutStartSec=600
TimeoutStopSec=40
KillSignal=SIGPWR
Restart=on-failure
LimitNOFILE=infinity
LimitNPROC=infinity
#LimitNOFILE=-1
#LimitNPROC=-1

[Install]
WantedBy=multi-user.target
Also=lxd.socket
EOF
'
sudo su -c '
cat <<EOF > /usr/lib/systemd/system/lxd.socket
[Unit]
Description=LXD - unix socket
Documentation=man:lxd(1)

[Socket]
ListenStream=/var/lib/lxd/unix.socket
SocketGroup=lxd
SocketMode=0660
Service=lxd.service

[Install]
WantedBy=sockets.target
EOF
'
#[Start LXD daemon]
#Once you've saved that file, enable, start, and set the service to start on boot:
sudo adduser --system lxd --home /var/lib/lxd/ --shell /bin/false
sudo groupadd --system lxd
sudo mkdir -p /var/log/lxd
sudo chown root:lxd /var/log/lxd
sudo systemctl --system daemon-reload
sudo systemctl enable lxd
sudo systemctl start lxd

# Run LXD for initialization.
cat <<EOF | sudo lxd init
yes
default
dir
no
yes
yes
lxdbr0
auto
auto
EOF

# Add user to lxd for running lxc command without privilege.
USER_ADDED_TO_LXD_GROUP="${USER}"
sudo gpasswd -a "${USER_ADDED_TO_LXD_GROUP}" lxd

# Reboot.
sudo reboot

# #[Testing LXD]
# lxc launch ubuntu:16.04
# lxc list

Only that solution worked for me:
https://github.com/aleph-zero/lxd-centos-7#running-a-container


Variable-size input and output  to convolutional layer

Hi,
I have sequence of different lengths: some of them is about 20 sequence length while some is about 2000 lengths the same is for its output"Lables" have the same input length. Depending on this github problem "https://github.com/fchollet/keras/issues/1920" I could solve entering problem, but now I am phasing the output problem now. I made my data have same sequence length for every batch, but when the model move to the next batch it gives me this error:
ValueError: Error when checking target: expected dense_1 to have shape (1, 20) but got array with shape (1, 21)
If i stop using Dense layer the GlobalMaxPooling layer gives same error
do you have any suggestion for solving this problem??
thanks

@shay86 please can you share the code or some useful link so that i can implement  i am new to deep learning.
Thanks in advanced


Add some functional tests

Codecov Report

Merging #259 (2eeb172) into master (a82a39e) will increase coverage by 5.89%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #259      +/-   ##
============================================
+ Coverage     72.80%   78.70%   +5.89%
  Complexity      529      529
============================================
  Files            44       44
  Lines          1379     1385       +6
============================================
+ Hits           1004     1090      +86
+ Misses          375      295      -80

Impacted Files
Coverage Δ

src/ContentSecurityPolicy/Violation/Log/Logger.php
0.00% <0.00%> (ø)

src/Controller/ContentSecurityPolicyController.php
0.00% <0.00%> (ø)

...licy/Violation/Filter/CustomRulesNoiseDetector.php
0.00% <0.00%> (ø)

src/ContentSecurityPolicy/DirectiveSet.php
98.68% <0.00%> (+1.31%)
:arrow_up:

src/EventListener/ExternalRedirectListener.php
80.35% <0.00%> (+1.78%)
:arrow_up:

src/DependencyInjection/Configuration.php
98.14% <0.00%> (+2.97%)
:arrow_up:

...rc/EventListener/ContentSecurityPolicyListener.php
83.51% <0.00%> (+5.25%)
:arrow_up:

src/EventListener/ClickjackingListener.php
95.83% <0.00%> (+12.50%)
:arrow_up:

...endencyInjection/Compiler/UAParserCompilerPass.php
50.00% <0.00%> (+50.00%)
:arrow_up:

...rc/DependencyInjection/NelmioSecurityExtension.php
51.78% <0.00%> (+51.78%)
:arrow_up:

... and 2 more

Continue to review full report at Codecov.

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


Update webauthn to v0.10.2

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.805%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
192

Relevant Lines:
338

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 56.805%

Totals

Change from base Build<PHONE_NUMBER>4:
0.0%

Covered Lines:
192

Relevant Lines:
338

💛  - Coveralls


2.0.5 breaks with .NET Framework 4.8 / Non-SDK-style / Mixed Platforms: Unable to load DLL 'e_sqlite3'

Hello,
I am quite probably creating an issue very related (or even a duplicate?) to #389, #444 or #446. My hope is I will be able to provide some additional info that will try to fix a problem introduced in 2.0.5.
2.0.5 is actually the only thing I am 100% sure about. It connects the above issues and it was the version when I discovered the bug. When I try 2.0.6 issue is still reproducible. If I however rollback to 2.0.4 all is fine. The day I discovered this issue there was no any newer version of Frank Krueger's sqlite-net-pcl, it was on the same 1.7.335 version. My Xamarin based Android project is not affected by the same update. There it runs fine even after 2.0.5 upgrade.
I also figured out how to 'fix it', or work it around better said. Following the error message it is clear that the app cannot locate e_sqlite3.dll. My project is couple of years old and all those years the concrete platform specific e_sqlite3.dll was used from .\runtimes\win-x86\native\ or .\runtimes\win-x64\native\ subfolder. If I manually find the e_sqlite3.dll which I find in this subfolder and copy it 3 levels up in the folder hierarchy, i.e. next to the main executable .exe file the application runs as it used to do before.
To my (not very deep) understanding the 'old' way is the correct intended way. I can compare it for example to WebView2Loader.dll which my project also utilizes and there it is loaded correctly from the above named subfolder.
Platform: The old-fashioned classic .NET 4.8 of the main .exe. My solution has number of Class Libraries built with .NET Standard 2.0 target framework.
OS: Windows 10
Debug, Mixed Platforms (When I try to build for x86, x64 or Any CPU the issue still reproduces)
Exception: The type initializer for 'SQLite.SQLiteConnection' threw an exception.
Inner exception: Unable to load DLL 'e_sqlite3': The specified module could not be found. (Exception from HRESULT: 0x8007007E)
Source: SQLitePCLRaw.provider.e_sqlite3
Stack:
   at SQLitePCL.SQLite3Provider_e_sqlite3.NativeMethods.sqlite3_libversion_number()
   at SQLitePCL.SQLite3Provider_e_sqlite3.SQLitePCL.ISQLite3Provider.sqlite3_libversion_number()
   at SQLitePCL.raw.SetProvider(ISQLite3Provider imp)
   at SQLitePCL.Batteries_V2.Init()
   at SQLite.SQLiteConnection..cctor()

   at SQLite.SQLiteConnection..ctor(String databasePath, SQLiteOpenFlags openFlags, Boolean storeDateTimeAsTicks)

Please let me know whether I can provide more usable info.

Try 2.0.7-pre20210929171745 ?

Hi @ericsink,
My apologies for my delayed answer. I tried this new version and yes, this problem problem is fixed! Better tested the android solution as well and that one stays working correctly as well.
If this change accepted should I expect 2.0.7 version in the following days / weeks? Best regards, Jan

I was experiencing the same issue, after upgrading the problem was solved as well.
no strange side-effects until now

I also want to add some information as well. As an example of our structure, we have a project and its dependents, where one is a test project. Both are targeting .NET Framework 4.8 and .NET 5. It looks something like this:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFrameworks>net5.0;net48</TargetFrameworks>
    <AssemblyName>ProjectUsingSQLCipher</AssemblyName>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Data.Sqlite.Core" Version="5.0.10" />
    <PackageReference Include="SQLitePCLRaw.bundle_e_sqlcipher" Version="2.0.4" />
  </ItemGroup>
</Project>

<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <TargetFrameworks>net5.0;net48</TargetFrameworks>
    <IsPackable>false</IsPackable>
    <AssemblyName>ProjectUsingSQLCipher.Test</AssemblyName>
    <IsTestProject>true</IsTestProject>
  </PropertyGroup>

  <ItemGroup Condition="'$(TargetFramework)' == 'net48'">
    <!-- See https://github.com/dotnet/efcore/issues/19396 why we need to do this for .NET Framework-->
    <PackageReference Include="SQLitePCLRaw.bundle_e_sqlcipher" Version="2.0.4" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.11.0" />
    <PackageReference Include="xunit" Version="2.4.1" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.3">
      <PrivateAssets>all</PrivateAssets>
      <IncludeAssets>runtime; build; native; contentfiles; analyzers; buildtransitive</IncludeAssets>
    </PackageReference>
  </ItemGroup>

  <ItemGroup>
    <ProjectReference Include="..\ProjectUsingSQLCipher\ProjectUsingSQLCipher.csproj" />
  </ItemGroup>

  <ItemGroup>
    <Content Include="xunit.runner.json" CopyToOutputDirectory="PreserveNewest" />
  </ItemGroup>
</Project>

For good measure, the xunit.runner.json file disables shadow copying. The setup above works well in both frameworks with SQLitePCLRaw.bundle_e_sqlcipher 2.0.4, though it is somewhat clunky that the dependency to SQLitePCLRaw.bundle_e_sqlcipher is not working transitively for .NET Framework 4.8. However, it works. When I tried to upgrade to version 2.0.5 or 2.0.6 the tests started to fail on .NET Framework 4.8 with the error described here and in the linked issues. I have tried to upgrade to 2.0.7-pre20210929171745 and can report that everything works. I can even delete the clunky
  <ItemGroup Condition="'$(TargetFramework)' == 'net48'">
    <!-- See https://github.com/dotnet/efcore/issues/19396 why we need to do this for .NET Framework-->
    <PackageReference Include="SQLitePCLRaw.bundle_e_sqlcipher" Version="2.0.4" />
  </ItemGroup>

and it still works on both .NET Framework 4.8 and .NET 5. That is fantastic!
I just want to finish with saying thanks for the great work, and I hope that 2.0.7 gets released soon. :-)


Add files via upload

Fixes for using time steps T > 1. Also, automatic draw after 40 (full) moves is implemented.

@Zeta36, what do you think...?!


Automatically derive module name from path

There's no reason to have Ravel.module('cities', './modules/cities.js') when the name can be derived automatically from the filename for the module.

Fixed by #30


generic-dex: issues after upgrading TimescaleDB to latest-pg13

could not load library "/usr/local/lib/postgresql/timescaledb-2.7.2.so": Error loading shared library libssl.so.1.1: No such file or directory (needed by /usr/local/lib/postgresql/timescaledb-2.7.2.so)

Workaround: lock image to 2.7.2-pg13

Oh, thanks! So, one of our images is broken? And what do you mean by "transferring a checkpoint"?

It doesn't seem like a corrupted registry cache. I've checked twice; a legit latest-pg13 image. So... I went to the running container and ran apk add libssl1.1. Bingo!
Possibly related:

https://github.com/timescale/timescaledb/issues/1073
https://github.com/timescale/timescaledb/issues/732

I encountered the following as well
could not load library "/usr/local/lib/postgresql/timescaledb-2.4.0.so": Error loading shared library libssl.so.1.1: No such file or directory (needed by /usr/local/lib/postgresql/timescaledb-2.4.0.so)
My volume was originally with 2.4.0-pg12 but since then upgraded to 2.8.1-pg12 and then to 2.9.0-pg12.  I wonder if this has anything to do with the 2.9.0 image?

I went to the running container and ran apk add libssl1.1. Bingo!

I did the same and it solves the problem.

We haven't met this issue in a wild for a long time; closing.


Asynchronous send PDU

I used the library to build an smpp server. However when I want to send an asynchronous pdu to the connected esme it hangs the server

Please provide a code example.


[Backport 2.x] Add support for dependencies in plugin descriptor properties with semver range #11441

Description
Backport https://github.com/opensearch-project/OpenSearch/pull/11441 to 2.x.
Following are the additional changes:

Updated PluginInfo version check for serialization of semver range list to use V_2_13_0.
Updated PluginInfo version check of Version.V_1_1_0 for custom folder name and related test.
Resolved minor conflicts in ListPluginsCommandTests.java

Related Issues
Resolves https://github.com/opensearch-project/OpenSearch/issues/1707
Check List

[X] New functionality includes testing.

[ ] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[ ] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[ ] Public documentation issue/PR created

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

Tests with failures:

org.opensearch.cluster.coordination.AwarenessAttributeDecommissionIT.testConcurrentDecommissionAction
org.opensearch.remotestore.SegmentReplicationUsingRemoteStoreIT.testRestartPrimary_NoReplicas

@abseth-amzn This is resulting in a breaking change within 2.x as GET /_cluster/stats and GET /_nodes both contain the PluginInfo object in their response bodies, and the opensearch_version field has changed from a string to an array.

@Xtansia Can you please open a GH issue if you haven't yet and assign it to @abseth-amzn?

I opened https://github.com/opensearch-project/OpenSearch/issues/12528, @abseth-amzn will you please take a look?


[2pt] Create a schema.sql

Create a schema.sql file with tables that will be analogical to the structure of the classes that you created:

books table (add all properties and associations from the parent Item class as table columns)
labels table

Completed in books branch


Update Continuous Integration against the opencflite Project Dependency from v476 to v635

Update the workflow continuous integration against the opencflite project dependency from v476 to v635.

Closed with #17.


sim pass assertion failure on unflattened design with unused port

Steps to reproduce the issue
To reproduce, create the following Verilog file,
module check(clk, rst, value, out);
	input	wire	clk, rst, value;
	output	wire	out;

	submod m1(.clk(clk), .rst(rst), .value(value), .out(out), .count());
endmodule

module submod(clk, rst, value, out, count);
	input	wire	clk, rst, value;
	output	wire	out;
	output	wire	[7:0]	count;

	always @(posedge clk)
	if (rst)
		count <= 0;
	else
		count <= count + 1;

	assign	out = value;
endmodule

Together with the Yosys script,
read_verilog -sv check.v
prep -top check
sim -d -clock clk -reset rst -rstlen 1 -n 1 -w check

Then run yosys -s script.ys.
Expected behavior
I expect the tool to simulate a single step without errors.
Actual behavior
The core crashes with an internal log_assert() error in passes/sat/sim.cc:set_state()
At issue is the null reference to the count output of the submodule.  Inside set_state(), the simulation tries to set a zero length value--associated with the count output of the submodule that ... doesn't go anywhere.
If the script file is adjusted to include flatten, the assertion failure vanishes.

This fixes the users issue.  Feel free to close the request when merged.


import image_classification from tensorflow legacy

Tensorflow moved this image_classification dependency to legacy breaking imports for the bit data pipeline
https://github.com/tensorflow/models/commit/29fa8c42326158424236efbe2a9f9f29c376ba86

I agree that's better since it will be maintained, I made this change
And no problem, happy to help!


Time and query limits

Refs #33. Still needs:

[ ] Configuration
[ ] Tests
[ ] Documentation

Two configuration options:

total time allowed for queries during a single GraphQL execution
total number of table SQL queries allowed during a single GraphQL execution

plugins:
  datasette-graphql:
    total_time_limit_ms: 2000
    num_query_limit: 200

Setting these to 0 should disable the limits.


Added support for wchar_t and std::wstring for C#

With the help of @tritao:

added support for wchar_t
added support for std::wstring
unified the way wide character strings are handled between C# and
C++/CLI
changed the way strings are handled, using 'IntPtr' instead of 'string'
with marshalling attributes on the P/Invokes

Has this been abandoned? Seems like it would be a nice improvement!

Frankly I have to really apologize for leaving this behind. I had other priorities at work so I couldn't give it the final push it needed.
I think I could be able to get back at this in my free time since not far from now I'll also need this at work.
If @tritao or @ddobrev could give me a little refresh on what was missing and/or what should be done I can give it a try.
Sorry but it's been almost 3 years and things most likely could have changed that impacted the work I did in this PR too.

I don't recall anymore either, but think it was pretty much good to go.
Please rebase it on top of latest master, make sure all tests still pass and lets take it from there.

 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: zillemarco sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

It's not compiling anymore:
Passes\IgnoreSystemDeclarationsPass.cs(60,49): error CS0103: The name 'GetCharSpecializations' does not exist in the current context [C:\projects\cppsharp\src\Generator\CppSharp.Generator.csproj]

Guess some code moved around or was renamed, and needs to be updated in this PR.

Yeah I know, during the rebase I had quite a bit of conflicts. I was thinking about drafting the PR until I get it back to work.


not correctly dismissed when executing Alert.alert(...) in onPress

ios simulator Version 10.0, iphone6
"react-native": "^0.38"
"react": "15.3.2"
"react-native-actionsheet": "2.4.2"
Alert and actionsheet button disappeared, but the mask layer did not disappear.

就是中国人

这情况还是自己把项目 clone 到本地后调试下吧

@yezhichao 帅哥，解决掉了吗？我也遇到了，无从下手

For users having the same problem, and non-chinese speaking, it's a bug in react-native:
https://github.com/facebook/react-native/issues/22237


Hands - slow down to 1-3 fps with Chrome 88.0.4324.150

Found it today with version 0.1.1613506466
Check it: https://mediapipe.dev/demo/hands
Face works as usual fine: https://mediapipe.dev/demo/face_mesh

Thanks for sharing the update. We are closing this issue now.


fix: empty content bug

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


Visible values are not currently supported

Spreadsheet Script FilterCriteriaBuilder in setVisibleValues(values)

I'm not sure what the problem is here. That function isn't used anywhere in these samples.


columns get somehow shuffled when there are more than 8 columns in ColumnDefinition

Griddle version
1.1.0 / 1.2.0
chrome & safari (others not tested)
Expected Behavior
Columns should always be in the same order as defined in <RowDefinition><ColumnDefinition .. /></RowDefinition>
Actual Behavior
if there are more than 8 columns, the order is different. It's always the same, but i did not found the logic how it gets shuffled.
Steps to reproduce
add more than 8 columnDefinition
Pull request with failing test or storybook story with issue
I hope i can do this later, but maybe someone has an idea why this is happening. its really odd ;-o

I have the same problem, here is a test case:
<Griddle
	data={[{x: 1}]}
>
	<RowDefinition rowKey="x">
		<ColumnDefinition id="s" title="First" />
		<ColumnDefinition id="r" title="2" />
		<ColumnDefinition id="h" title="3" />
		<ColumnDefinition id="e" title="4" />
		<ColumnDefinition id="d" title="5" />
		<ColumnDefinition id="d2" title="6" />
		<ColumnDefinition id="r2" title="7" />
		<ColumnDefinition id="e2" title="8" />
		<ColumnDefinition id="a" title="Last" />
	</RowDefinition>
</Griddle>


Add a Scaleway secrets provider

Please have a look at the individual commit messages for the details.

Note that we need this urgently, and thus no sophisticated tests are added that do not work without a real Scaleway account. But I've run the tests with my credentials and they pass.
There probably also is not much point in capturing responses via WireMock of so, as the Scaleway API is still in beta, and things might change. So we'll best notice this in production 😬
Anyway, this implementation does not affect anyone who's not actively using the Scaleway secrets manager. So the risk to merge should be very low.

You have to add the config properties to the secretsProvider sections in all application.conf files that have this section, otherwise there is no way to configure the new provider. If you are touching this anyway, maybe take the chance and add some comments for better overview, like this:
secretsProvider {
  name = "vault"
  name = ${?SECRETS_PROVIDER_NAME}

  # Properties for the vault provider.
  vaultUri = ${?VAULT_URI}
  vaultRoleId = ${?VAULT_ROLE_ID}
  vaultSecretId = ${?VAULT_SECRET_ID}
  vaultRootPath = ${?VAULT_ROOT_PATH}
  vaultPrefix = ${?VAULT_PREFIX}
  vaultNamespace = ${?VAULT_NAMESPACE}

  # Properties for the file based provider.
  fileBasedPath = ${?FILE_BASED_PATH}

  # Properties for the scaleway provider.
  ...
}

You have to add the config properties to the secretsProvider sections in all application.conf files that have this section

Ok, this means I should use a prefix to group / disambiguate property names in the config, which in turn means config properties are named differently than class properties, and thus my kontlix.serialization approach would not have worked anyway.

You have to add the config properties to the secretsProvider sections in all application.conf files that have this section

Ok, this means I should use a prefix to group / disambiguate property names in the config, which in turn means config properties are named differently than class properties, and thus my kontlix.serialization approach would not have worked anyway.

Right, that might well be the case. I'm only just getting a deeper look into the configuration code because of the Slack discussion we had today.

Could you then add a disclaimer to the class that it accesses a preview API and use on your own risk or something like that. Also the test should maybe have some TODO comments that it should be reworked later. Would it make sense to add the code to an experimental package?

Could you then add a disclaimer to the class that it accesses a preview API and use on your own risk or something like that. Also the test should maybe have some TODO comments that it should be reworked later.

Done.

Would it make sense to add the code to an experimental package?

I'd prefer not to, to keep the refactoring effort a bit lower once the API becomes stable. Also, I'd not consider the client implementation per se to be experimental, but the server-side API it talks to.

Done.

Would it make sense to add the code to an experimental package?

I'd prefer not to, to keep the refactoring effort a bit lower once the API becomes stable. Also, I'd not consider the client implementation per se to be experimental, but the server-side API it talks to.

Ok, thanks. For me it's fine now. Leaving the approval to @mnonnenmacher .


Avoiding require('../../../..

browserify-handbook : avoiding ../../../../../../..
Works well for https://github.com/CartoDB/cartodb.js/commit/898beed7827a59f3ceddc923dcefb047d7a91072
cc @xavijam

Caused more trouble that it solved in other repos in the end, so closing it until we can come up some better/more stable way


[Feature Request] Make Filter More Obvious in the Manga Info/Chapter Pg

Why/User Benefit/User Problem
Be able to instantly see if there was a filter used so it is easier to tell that a filter is on/which filter is on.
What/Requirements
Text on the top or next to the number of chapters. Ex: Show unread chapters, Scanlation group name
or make it super obvious with some change in the filter icon that there is some sort of filter on
or just have the text "Filters applied" or something

in newest release


Refactoring code to use bower/lib/config instead and fix wrong tests.

Your current logic doesn't account for the merging strategy described in the config docs.

Test failed.

Oops. I have just fixed it.
Can you run again the AppVeyor test for 0.10? It seems to be a temporary issue with their system. https://ci.appveyor.com/project/ShinnosukeWatanabe/bower-directory/build/25/job/lruasxu4803ytu5l

Everything is passing now.

@shinnn, I have wrapped the call with a setTimeout on a new commit (d8583498155aa55b0d54231f13af11ae9eac066e) to make it non-blocking as you want.

Any news?


Full-stack capstone project

Greetings 👋
We are a team of 5 members: @osmanwako, @prius2055, @eudondian, @massazoum, @anbehindY
We have distributed our workload according to the project requirements
Please kindly check our Kanban Board

Hi @anbehindY
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that does not contain a screenshot of the kanban board.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @anbehindY,
Wow, you did it 🎉

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
To highlight

All cards have descriptions, pointings and assignees ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


ParticleOrbitalState - advice on inheriting from ParticleState

I'm trying to create a new class from ParticleState and OrbitalState. I found this almost hilariously easy when combining GaussianState with OrbitalState to run derivations of Kalman filters on orbital scenarios.
I've come somewhat unstuck when trying to to the same thing with particle states though. Because the parameters don't match, ParticleState initiates on a list of Particles (weighted state vectors), OrbitalState on a StateVector, I get an error in super.__init__(). Before I go down a rabbit hole for a solution, does anyone have an idea on how this might be solved?

Maybe something like the Update types, where you have a generic Astro and then construct State based types from it: https://github.com/dstl/Stone-Soup/blob/ee2d6834ce144163d03332e9452de24af81bd5c5/stonesoup/types/update.py#L54-L109


test: adding test for runAdAuction

Adding e2e test for runAdAuction

Codecov Report

Merging #54 (cc6d4e2) into main (6db7686) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #54   +/-   ##
=======================================
  Coverage   64.45%   64.45%
=======================================
  Files           9        9
  Lines         166      166
  Branches       30       30
=======================================
  Hits          107      107
  Misses         47       47
  Partials       12       12

Flag
Coverage Δ

unittests
64.45% <ø> (ø)

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

Continue to review full report at Codecov.

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


When a consumer loses connection from a broker due to rebalancing, offsets are not synced upon reconnect, and messages are consumed multiple times.

Maybe this is just a wontfix due to Kafka version and the client API being super alpha, but it was surprising behavior to me.
If a second consumer attempts to consume a partition already being serviced by another consumer in the same group, it causes the broker to rebalance (whether that is the correct behavior on the part of Kafka is unknown to me), and one of the consumers will get chosen to continue servicing the partition, while the other is kicked out. The problem is that when this happens, any offsets not already committed by the original consumer will not be committed even if it is chosen by the broker to continue servicing the partition, so those messages are re-fetched. What's especially strange in this case, is that even in this case, only the first message of the re-fetched batch is committed, so the remaining ones are fetched again, with only the first being committed each fetch, until the partition is finally caught up.
It's something that could happen accidentally, or when spinning up new hardware, or whatever.
This can be worked around by setting offset_commit_threshold to 1 to enforce offset commit after every message processed, but I still think it should be handled at some lower level to prevent the messages from being processed so many times each.

Version of Ruby: ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-darwin15]
Version of Kafka: <IP_ADDRESS>
Version of ruby-kafka: 0f2ae75a712b00a94eb01904582ab397280a6758 (i.e. current HEAD)

Steps to reproduce
consumer.rb:
#!/usr/bin/env ruby

require 'ruby-kafka'

TOPIC = 'mpd-topic'

$kafka = Kafka.new(
  seed_brokers: ['localhost:9092'],
  client_id: 'mpd1'
)

consumer = $kafka.consumer(group_id: 'mpd-consumers', offset_commit_threshold: 0)

consumer.subscribe(TOPIC)

begin
  consumer.each_message do |msg|
    p msg
  end
rescue RuntimeError
  retry
end

producer.rb:
#!/usr/bin/env ruby

require 'ruby-kafka'

TOPIC = 'mpd-topic'

$kafka = Kafka.new(
  seed_brokers: ['localhost:9092'],
  client_id: 'mpd1'
)

producer = $kafka.async_producer

20.times do
  producer.produce(
    'sup?',
    topic: TOPIC,
    partition_key: 'mah-key'
  )
end

producer.deliver_messages

at_exit do
   producer.shutdown
end

Then in separate shells to keep the output easy to read:
first shell:
$ ruby consumer.rb

another shell:
$ ruby consumer.rb

third shell:
$ ruby producer.rb

Then watch the output of the consumer.rb shells.
Expected outcome
My consumer group wouldn't consumer messages more than twice.
Actual outcome
My consumer group consumed messages more than twice.

Can you try without setting offset_commit_threshold to 0? I'm not sure what effect it has, but it's certainly not intended...

@dasch It's the same effect. offset_commit_threshold defaults to 0.

Hmm, indeed it is. I think this might be a complex case, I'll have to do some digging.

Note that offset commits are unrelated to fetches – the reason there's a commit after processing the first message in a batch is that the time-based commit trigger fires immediately the first time it's checked.

@dasch I'm not sure I understand what you mean here. The time-based trigger does not happen on a timer. It fires after a subsequent message is processed as long as offset_commit_interval seconds have elapsed (defaults to 10) since the previous commit, and does commit all the stored offsets.

Let me clarify: the offset commits trigger upon message processing, but is not related to message fetching. That may explain this:

only the first message of the re-fetched batch is committed, so the remaining ones are fetched again, with only the first being committed each fetch, until the partition is finally caught up.

Or are you seeing duplicate processing even with a stable consumer group?
An unstable consumer group cannot be guaranteed to not process any messages twice, but some measure can be taken to limit the problem. As you correctly noticed, a consumer disregards its own in-memory checkpoints when the group re-syncs, starting over from the last committed ones. This was done during development to err on the side of at-least-once processing, but it's something that can probably be changed – especially if someone would be willing to put in the time to create tests and/or an implementation.
As an extension of this, I've also looked into ways of trying to keep as many partitions as possible from being re-assigned during a re-sync. It's difficult to balance this with assignment fairness, though. See #103 for some details.
I'd love to get some help with this as I'm quite swamped with other stuff these days.

I'm not seeing duplicates with a stable group, but with an unstable group, I would not expect a message to be processed more than twice, but some of them are being processed many more times than that.
The naive approach would be commit uncommitted offsets on connect if there are any.

@xxx I'm trying to think through whether it would be safe to keep a consumer's offsets across a group re-sync – I worry that there may be edge cases where a consumer is cut off from the group for some time before rejoining, then committing offsets for a partition that had been handled by another member in the meantime. I may be able to check whether the new group generation is the immediate successor to the old one, in which case it should be safe to keep the offsets for partitions that are still assigned to the consumer...

@xxx I opened #248.

Should be fixed in the latest releases.


README: Update architecture image

The current image was made in draw.io in the early stages of development and is a quite rudimentary interpretation of what's happening. A better graphic is needed.
Also maybe a different color would be better.

Updated, with new architecture and design.