hacker_news_pwa: Publish on Firebase/Github.io

Just fun :)

I'd like to set this up via travis, but we aren't green right now.
For now I'll do a single manual push.

Done as https://dart-lang.github.io/angular/ for now.
... can find a better home later on :)

That's pretty awesome.

The page loads & paints & gets interactive in 2780ms, even on the simulated slow connection. That's really good for something like AngularDart, which doesn't even try to optimize for microbenchmarks like this. Just look at https://hnpwa.com/, the latest React score is 91 and TTI is 2570ms, AngularJS gets 4-6 seconds, etc.
Ignoring SEO (that's another discussion altogether), there are only a couple of issues.

We lose some perf points by not caching angular/main.dart.js and images/logo.svg for long enough. I don't think there's anything we can do on github.io, but on Firebase Hosting, you just increase TTL to something like 24 hours and Lighthouse is happy (because that's when ISPs start to cache things, which makes things much faster for the end user). So: impossible fix on github.io, easy fix on firebase hosting.

The rest of points lost is "perceptual speed index". This is best explained here. Basically, the earlier we can show the UI (even if it's only a placeholder) the better. Maybe show the top bar in the static HTML? Even better, "make room" for the loaded articles with white divs? That alone makes a lot of the initial pixels correspond to the final result.
And this is not just about "beating a score", by the way. The apps that do this kind of optimization do perceptually load faster. It's a very different experience looking at a gray screen for 2 seconds than at the beginnings of a UI.

Congrats to all!
In summary, consider:

Moving this to Firebase Hosting and cache the two files more aggressively.
Submitting the app to https://hnpwa.com/.
Adding the most visually prominent parts of the UI (app bar, white area with list of articles) to the static HTML + CSS.

Yeah, @leonsenft had the original version on Firebase Hosting. This is definitely temporary; I'd like to set it up to auto-push to Firebase Hosting every successful merge:

[ ] Move to Firebase Hosting
[ ] Submit to hnpwa.com
[ ] Add visually prominent parts of the UI to the static HTML/CSS if possible

We already have it hosted on Firebase: https://dart-hn.firebaseapp.com/, although this is an older version. Now that it's open-sourced, it should be easier to keep in sync.
Are we happy with this location?

@leonsenft Seems fine to me unless @kevmoo has other ideas.

Let's run with it!

I have push access, so I'll try and setup an auto-push this week.

(We need our travis to be green, ideally)

Similar numbers with Firebase @filiph, but thanks for suggesting that!

I think you need to bump up the max-age to something like 86400 for this to register. I can help with that, just point me to your firebase.json.
Note that yes, updates to main.dart.js will take up to 24 hours (86400 seconds) to show up without force refresh. In a production app, you'd have a build step that does things like main.949234.dart.js and then caches forever.

I agree it's pretty low priority, but perhaps we could automate bumping the number (in main.#.dart.js) in the Travis deploy script? Of course we'd need to do this for each script (including all of the deferred parts) as well as the list of offline URLs used by the service worker itself.

@filiph – if you have specific ideas/feedback, please file issues and add to this milestone: https://github.com/dart-lang/angular/milestone/7

@kevmoo I added https://github.com/dart-lang/angular/issues/963 (can't add it to milestone because I don't have rights here — should I be offended?).

I think @filiph's suggestion of just a ~10 minute cache is fine here.

Didn't @filiph say it should be 24 hours though to improve caching through ISPs?

I'm closing this issue for now, since we have something.
Please open additional issues for tracking.

Yeah, I meant longer than 10 minutes. 24 hours seems to be the customary max-age for js/image assets.


About Model Figure?

@policeme Hi，In the model figure,
1.Multi-head Attention Distribution part:
what's '1 2 3 ... k-2 k-1 k'? Do they represent different head attention？
2.Final Distribution: Why is Vocabulary distribution on top, copy distribution below?

first question, I don't understand u question, for my opinion, different head attention means different feature, in the attention final, we concat them.
second question, I don't understand, first, I don't concat the vocabulary distribution and attention, just add them by vocab index.

@policeme look the picture above, especially the content of red circle, you will understand my questions.

not concat.just add. See function final_dist
from Alimail iPhone
------------------Original Mail ------------------
From:yjyGo<EMAIL_ADDRESS>Date:2019-07-18 20:50:24
Recipient:policeme/transformer-pointer-generator<EMAIL_ADDRESS>CC:Cally<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject:Re: [policeme/transformer-pointer-generator] About Model Figure? (#12)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

can I add you wechat number?

只是简单的画图，实际是想加，可以看代码，一步步调试，你就会知道。
from Alimail iPhone
------------------Original Mail ------------------
From:yjyGo<EMAIL_ADDRESS>Date:2019-07-18 20:55:32
Recipient:policeme/transformer-pointer-generator<EMAIL_ADDRESS>CC:Cally<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject:Re: [policeme/transformer-pointer-generator] About Model Figure? (#12)
我的意思是你的最终分布图形里边蓝色在上，红色在下，蓝色表示词汇表分布，红色代表拷贝分布，我可不可以把它颠倒过来，红在下蓝在上，只是单纯的画图而已。
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

我知道相加，论文里也有，你没明白我的意思

详细请阅读 All Attention Is You Need
from Alimail iPhone
------------------Original Mail ------------------
From:yjyGo<EMAIL_ADDRESS>Date:2019-07-18 20:56:54
Recipient:policeme/transformer-pointer-generator<EMAIL_ADDRESS>CC:Cally<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject:Re: [policeme/transformer-pointer-generator] About Model Figure? (#12)
多头注意力分布图形的每个数字表示什么
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

你没描述清楚，请详细描述问题
from Alimail iPhone
------------------Original Mail ------------------
From:yjyGo<EMAIL_ADDRESS>Date:2019-07-18 20:57:23
Recipient:policeme/transformer-pointer-generator<EMAIL_ADDRESS>CC:Cally<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject:Re: [policeme/transformer-pointer-generator] About Model Figure? (#12)
我知道相加，论文里也有，你没明白我的意思
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

不是，下次问的时候请阅读代码后提问

不好意思，言语上表达不清楚很抱歉，可以加微信吗？方便交流。


Support VersionNumber

Fixes #200 by adding round-trip support for VersionNumber. Requires #201.

CI expected to fail due to #202

CI is disabled for branches not based off of main. Will rebase once cv/empty-tuple is merged.

now that #201 is merged looks like GitHub auto switched this PR's base branch to main - will closing/reopening trigger CI? let's see


ඞ You know the rules and so do I ඞ

Say goodbye ඞ
https://github.com/Rick-Lang/rickroll-lang
Feature: you can also reproduce the audio in ඞ.rickroll, with Rick singing!

ඞ


Feedback send file

You now get feedback when uploading a file to the webserver

Looks good, second member can merge once it passes.

Looks good. Merging.


Add integration tests to CLI

Issue Number: 583
Summary
Add integration tests for the CLI
Checklist

[ ] Local Tests Passing?
[ ] CICD and Pipeline Tests Passing?
[x] Added any new Tests?
[x] Documentation Updates Made? NO
[x] Are there any API Changes? If yes, please describe below. NO
[x] This is not a breaking change. If it is, please describe it below. IT IS NOT

Anything else?
Tests currently failing because of a known issue with the JSON Data Source! This is what these tests are literally for :) I will wait to merge until they are passing because the fix has been merged

Codecov Report

Merging #144 (99932c4) into dev (49d56cf) will increase coverage by 7.03%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##              dev     #144      +/-   ##
==========================================
+ Coverage   78.70%   85.73%   +7.03%
==========================================
  Files          59       52       -7
  Lines        1864     1711     -153
  Branches      205      194      -11
==========================================
  Hits         1467     1467
+ Misses        344      191     -153
  Partials       53       53

Impacted Files
Coverage Δ

src/CarbonAware.CLI/src/Program.cs

...are.CLI/src/Commands/Emissions/EmissionsCommand.cs

...LI/src/Common/CommonLocalizableStrings.Designer.cs

...I/src/Extensions/ConfigurationBuilderExtensions.cs

...CLI/src/Extensions/CommandLineBuilderExtensions.cs

src/CarbonAware.CLI/src/Common/CommonOptions.cs

.../Commands/Emissions/LocalizableStrings.Designer.cs


entrypoint: Remove token file after use

File named token was a left over from mirroring setup done via entrypoint.sh

I'm a little unclear on what the change is accomplishing? Is it just a cleanliness thing?

Yeah, token file existed after the execution of test-containers-test target.

One advantage of leaving the token behind is that it can be reused later by a human doing debugging / special stuff. Maybe printing the content of token would be a better way to accomplish that and be cleaner?

Sure, we'll use the other way to provide token via standard input as mentioned in docs.


set the thermostat to cooling - wrong execution command

Hello,
I have noticed a strange behavior on set the thermostat to cooling.
When you send this command (set the thermostat to cooling) i get "action.devices.commands.ThermostatTemperatureSetpoint" on the execute call, but when I send "set thermostat to cooling" (without "the"), I am getting "action.devices.commands.ThermostatSetMode".
For "set the thermostat to heating" and "set thermostat to heating" I am getting the same execution command, "action.devices.commands.ThermostatSetMode", so only the cooling + the is with a problem.
I want to add that the response on the query before the execute is exactly the same for both commands (with or without "the").
Thank you

When u get ThermostatTemperatureSetpoint i also see a Query before that execute, is the same as in the case of  "set thermostat to cooling", and i respond with the same result. The temperature that i get in this ThermostatTemperatureSetpoint i think is the original one minus 3( i still have to double check this), but the problem is that i don't want to execute any setPoint for this commands ( I return "Mode not avaialbe error" for this commands).

Okay, so it's probably a mismatch, thinking that you're trying to make the room "A little cooler" and not interpreting that as the name of the mode.

Cannot reproduce.


Accessibility Voice Over Cannot Sort

Hello,
I have set up a LazyColumn and used ReorderableItem with .draggableHandle() on an Image within the Card that is inside the ReorderableItem, but I am unable to sort the items when voice over is enabled.
Is there already something existing for voice over sorting control or would you be able to allow a double click to active the ReorderableItem so the user can drag it then on release drop the item?
Let me know!
Thank you!

Any examples of how this is handled in other apps? Currently on a plane but I will be looking into this after my long flight.

I went around my Google Pixel phone to see how Google handles accessibility for drag and drop reordering.

In System > Languages > System Languages, if there are multiple languages, it shows a handle for drag and drop reordering for every item. The drag and drop handle is completely invisible to TalkBack. There are no additional actions in the TalkBack menu.
In Gboard Settings > Languages, after clicking the Edit button, it shows a handle for drag and drop reordering for every item. The drag and drop handle is completely invisible to TalkBack. There are no additional actions in the TalkBack menu.
In the Notification Drawer > Quick Settings > Edit Mode, there's a "Move tile" action in the TalkBack menu of every item in the grid, clicking it enables a mode where you can select another item in the grid to swap the current item with.

I think a good solution would be to add an action to the TalkBack menu to move an item up or down by swapping the selected item with the next reorderable item. I also think it should be up to the user of this library to add them (partly because of i18n, I can't translate "Move up" into all the languages) but I will add them to the example app.

Yeah I ended up adding a move up and move down option in the talk back menu which works good. In the future I might switch out the drag handle for and up and down buttons on the row itself to make it easier for a talk back user to sort items. Thank you though!

I have added "Move Up"/"Move Down"/"Move Left"/"Move Right" actions to the TalkBack menu for list items in the demo app and a section on Accessibility in the README.


Multiple databases in a multi-tenant architecture

I note from the FAQ that the recommended architecture is a single PostgreSQL database per cluster.
My application is SaaS with a relatively small number of customer instances (10s currently).
Due to various constraints (that I can't change in the short-term) I'm stuck with each customer instance needing a relatively small (>4Gb) and fairly lightly used (1-2 active users at any time) PostgreSQL database - I'm planning for around 100 databases, all with the same data-schema.
Much of the rest of the application is hosted in Kubernetes, and the bits that are not we'd like to move that way.
I'm thinking in these circumstances I treat these 100 identical databases as if they were just one - and pool them on a single cluster so we can more easily manage resilience backups upgrades etc.
Can you see any particular pitfalls with this approach that I'd need to be aware of and any particular limits you'd advise?
Assuming this is not a big no-no:
It would be helpful for adopters in a position such as this if it was possible via the operator to optionally define databases (with roles) separately from the creation of the cluster - we can then more easily manage them alongside the other provisioning processes when a new customer instance is defined.
Thanks!

Hi @AJK74
Can you be a bit more clear on what you one to do? you want to have a 100 databases in one instance or merge all those 100 database into one? is not clear your final goal.
On the PostgreSQL, it's possible but not recommended, since if one database goes down, all of them will go down, instead, having one cluster with one database, it's harder to have a big outage when something goes wrong, that's why we strongly recommend to have one database per cluster and not all of them in just one cluster.
I'm also turning this into a discussion since it's not an issue.
Regards!


JVM Crashing in Dockerized Alpine Environment

This is the code I'm using to reproduce the problem:
import com.uber.h3core.H3Core;

import java.io.IOException;

public class Main {
    private static final double[][] coordinates = new double[][] {
            {0.0    , 90.0    },
            {13.5467, -19.2347},
            {13.5467, -30.2347},
            {17.5467, -30.2347}
    };

    public static void main(String[] args) throws IOException {
        H3Core h3Core = H3Core.newInstance();
        for (double[] coordinate: coordinates) {
            long h3Id = h3Core.geoToH3(coordinate[0], coordinate[1], 12);
            System.out.printf("Lat: %s Lng: %s H3: %d%n", coordinate[0], coordinate[1], h3Id);
        }
    }
}

I'm using the gradle docker plugin to build the docker images. The project with the crashing scenario and setup is available here.
To build the image, run ./gradlew dockerBuildImage. Then run docker run -it divinomenezes/h3-issue-test:latest to run the code. You should be able to run normally.
If you change the build.gradle:27 as described in the comment and rerun the steps above, you should see the following error:
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x0000000000003b36, pid=1, tid=0x00007fbaf7e76ae8
#
# JRE version: OpenJDK Runtime Environment (8.0_171-b11) (build 1.8.0_171-b11)
# Java VM: OpenJDK 64-Bit Server VM (25.171-b11 mixed mode linux-amd64 compressed oops)
# Derivative: IcedTea 3.8.0
# Distribution: Custom build (Wed Jun 13 18:28:11 UTC 2018)
# Problematic frame:
# C  0x0000000000003b36
#
# Failed to write core dump. Core dumps have been disabled. To enable core dumping, try "ulimit -c unlimited" before starting Java again
#
# An error report file with more information is saved as:
# //hs_err_pid1.log
#
# If you would like to submit a bug report, please include
# instructions on how to reproduce the bug and visit:
#   http://icedtea.classpath.org/bugzilla
#

The hs_err_pid1.log is also available here.

Thanks for providing detailed steps for reproducing. I was able to reproduce the issue with the openjdk:8-alpine image.
After taking a look at the core dump, it looks like the issue is inside the function h3ToGeo. I noticed it calls a C core library function isfinite, and I was able to move the crash up in the call stack by adding a call to isfinite in jniapi.c. The issue is that the linux-x64 cross-compile build links to glibc, which is not available on Alpine Linux.
Since installing glibc on Alpine or statically linking with glibc are not recommended, it looks like the best approach here might be to change the linux-* cross-compile builds to statically link to musl.

If this is the path we'll follow, I'd really enjoy opening a PR to change that during the weekend.
Are you aware of any dependency from glibc that is not available in musl?

No, we shouldn't have any dependency on glibc itself. The core library has been tested with Alpine Linux and musl itself, and with MSVC.
cc @dfellis @isaachier @nrabinowitz in case there's something I'm not considering about statically linking to a different libc.

@isaacbrodsky statically linking libc is generally a bad idea, and I think affects the licensing of the resulting binary (https://lwn.net/Articles/117972/), so H3 doesn't do that.

@isaacbrodsky I believe the idea is to statically link musl, actualy. musl has a MIT license

I don't really like the idea of statically linking Musl to support one uncommon Linux variant. If we're going to support this, I'd rather we do some autoconf-like detection of which library, glibc or musl, to link to, and to use the OS-provided one. Otherwise we're opening ourselves to an upgrade treadmill for all security updates to musl.

@isaacbrodsky somewhat related question: Would it make sense to use -DCMAKE_BUILD_TYPE=RelWithDebInfo so the stacktraces aren't hex values?

I don't want to get into a Linux distro flamewar (Linux is my daily driver, I've had enough of those over the years), I wanted to point out that we could easily tangle ourselves into a support nightmare when we look beyond the top 10 Linux distros (Alpine is currently 65th according to Distrowatch, and even in the world of Docker, Alpine has half the stars of Ubuntu).

@dfellis I understand your position, but you'd be surprised about how many docker images use Alpine as a base. Stars aside, I wonder if there is a way to search for base image "spinoffs."

Yes that would work but you will have a lot of package systems to work with (debian, rpm, etc.).


Support Cross chain and Extensions

Two test cases currently are failing that were added to Uniswap's repo:
https://github.com/Uniswap/token-lists/blob/main/test/schema/example-crosschain.tokenlist.json
https://github.com/Uniswap/token-lists/blob/main/test/schema/extensions-valid-object.tokenlist.json

See: https://github.com/fubuloubu/py-tokenlists/runs/4835408461?check_suite_focus=true

I'm not sure this was fully resolved.
A. There are absolutely no docs how to use this in a multi-chain example, I am so lost.
B. (this could be an issue in Ape framework): When I am connected to Optimism and I try to get a token, I still get a key error. But perhaps I am lacking information that could be solved from item A.

Re-closing, most of issue is on not understanding lists as well as just having really limited lists out there... But some mult-chain docs would still be appreciated.
If anyone is looking to see how to work with multi-chain, see this repo: https://github.com/ApeWorX/ape-tokens


Bugfix/yousong vm changeip

这个 PR 实现什么功能/修复什么问题:
feat(cloudcommon): db: jointbase: add list:"user" for RowId
fix(vpcagent): models: guestnetworks: accomodate change_ipaddr

是否需要 backport 到之前的 release 分支:

release/3.3

/area region
/area vpcagent
/cc @swordqiu @zexi @ioito @wanyaoqi

Codecov Report

Merging #8942 (9beb370) into master (dc3e2a7) will increase coverage by 11.94%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #8942       +/-   ##
===========================================
+ Coverage    5.88%   17.82%   +11.94%
===========================================
  Files         825       88      -737
  Lines      128694     8857   -119837
===========================================
- Hits         7573     1579     -5994
+ Misses     120441     7105   -113336
+ Partials      680      173      -507

Flag
Coverage Δ

aFlag
17.82% <ø> (+11.94%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/compute/models/guesttemplate_resource.go

pkg/compute/guestdrivers/kvm.go

pkg/compute/models/instance_snapshots.go

pkg/compute/models/cloudprovidercapacities.go

pkg/multicloud/esxi/datacenter.go

.../compute/models/loadbalanceropenstackcachedlbbg.go

pkg/multicloud/aws/instance.go

pkg/compute/models/natgateways.go

pkg/compute/models/loadbalancercachedacls.go

pkg/compute/models/dbinstance_accounts.go

... and 726 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 dc3e2a7...9beb370. Read the comment docs.

/lgtm

/approve


Product by CVE

Here's the draft PR for a replacing mock data with data from Guac.
It needs more testing, but for rhel-7.9.z.json sbom and cve-2022-2284.json vex it should return something like
{
  "id": "cve-2022-2284",
  "products": {
    "KnownAffected": [
      {
        "sbom_id": "rhel-7.9.z",
        "packages": [
          {
            "purl": "pkg:rpm/vim@7.4.629-8.el7_9?arch=src&epoch=2",
            "type": "Direct"
          }
        ]
      }
    ]
  },
  "advisories": [],
  "details": []
}

We should start testing it further and see what's still need to fixed for this.
To reproduce:

Start trustification with with bombastic-api validator turned off (remove --validator sbom)
Obtain the token

TOKEN=$(curl -s -d "client_id=walker" -d "client_secret=ZVzq9AMOVUdMY1lSohpx1jI3aW56QDPS" -d 'grant_type=client_credentials' \
  'http://localhost:8090/realms/chicken/protocol/openid-connect/token' | jq -r .access_token)

Ingest SBOM

curl -v -H "transfer-encoding: chunked" -H "content-encoding: bzip2" --json @rhel-7.9.z.json.bz2 --oauth2-bearer $TOKEN  "http://localhost:8082/api/v1/sbom?id=rhel-7.9.z.json.bz2"

Ingest VEX

curl -v -X POST --oauth2-bearer $TOKEN --json @cve-2022-2284.json http://localhost:8081/api/v1/vex

Make a query

curl -v -X GET --oauth2-bearer $TOKEN "http://localhost:8083/api/v1/cve/cve-2022-2284/related-products" | jq

@dejanb thanks for this awesome PR, could you please just fix the conflicts so we can merge it?


Make padding optional in object detection data extension

This change makes it optional to pad images when creating an object detection dataset.
A form validator is added to verify that either both or none of width/height are specified when validating the form.
Doing this in the form validator instead of (like formerly) the extension instantiation helps drop a more user-friendly error.
Template updated to show width/height on same row.

Depends on #833

Looks fine and should make DetectNet more generally useful.


Linking files fails

It looks like it tests for the linked file but never tries to create the link. This is the output from cap vagrant deploy --trace
** Invoke deploy:check:linked_files (first_time)
** Execute deploy:check:linked_files
DEBUG[7780bb35] Running /usr/bin/env [ -f /srv/photo_album/shared/.env ] on <IP_ADDRESS>
DEBUG[7780bb35] Command: [ -f /srv/photo_album/shared/.env ]
DEBUG[7780bb35] Finished in 0.003 seconds with exit status 1 (failed).
ERRORlinked file /srv/photo_album/shared/.env does not exist on <IP_ADDRESS>

Is it possible to configure this such that it doesn't fail if the file doesn't exist (i.e. Optional linked files)? e.g. My app may not require the file, but if the file does exist, it will use it and should still be retained between releases.

@patricknelson no, there is not such a feature. Could you help me understand your use-case a bit more? Why are the files optional?

In this case, it's because I deploy to about 6 environments which vary depending on their use case (Prod + DR, UAT, QA, Dev and agency). In most situations the private key file I'm symlinking will exist, but it's only optionally needed depending on other configuration (in this case: SSO). If SSO is disabled, you can login normally via user/pass (which you can do even with SSO enabled in the off chance SSO is failing).
In this way, I have the ability to be more robust and deploy to environments during my transition period as we dev/debug SSO. It's not a huge deal since it's easy to set up, I was simply researching if it was possible since I'd like to not have to create an empty file for an environment which may not ever be setup with SSO.

I see.  Thanks for the explanation! We don't have plans to support optional linked files in Capistrano itself any time soon, but it sounds like you have some viable workarounds.

@patricknelson I came across this issue while Googling for the same thing.  I ended up making a custom task using the deploy:symlink:linked_files task as a starting point and just added a line to skip creating a symlink unless the file exists.
# lib/capistrano/tasks/optional_linked_files.rake
namespace :deploy do
  namespace :symlink do

    task :shared do
      invoke "deploy:symlink:optional_linked_files"
    end

    desc 'Symlink optional linked files'
    task :optional_linked_files do
      next unless any? :optional_linked_files
      on release_roles :all do
        execute :mkdir, "-p", linked_file_dirs(release_path)

        fetch(:optional_linked_files).each do |file|
          target = release_path.join(file)
          source = shared_path.join(file)
          next unless test "[ -f #{source} ]"
          next if test "[ -L #{target} ]"
          execute :rm, target if test "[ -f #{target} ]"
          execute :ln, "-s", source, target
        end
      end
    end

  end
end

# config/deploy.rb or config/deploy/<stage_name>.rb
set :optional_linked_files, %w{
  .env
}

Brilliant, thanks @speckins for the quick snippet! I wasn't sure how much demand there'd be for this so I figured someone had to ask first.
n00b question: Is there any way to incorporate this locally into my project repo (e.g. somewhere adjacent to deploy.rb) instead of dropping into my capistrano install directory (i.e. all the way under /usr/local/share/gems/gems/capistrano-VERSION/lib/capistrano/tasks/ on my CentOS local dev server)?

@patricknelson Yes, there is a place for you to add custom Capistrano tasks to your project, usually {Rails.root}/lib/capistrano/tasks/ -- look at the end of your Capfile.  Take the entirety of the code I posted above (minus the set :optional_linked_files, ... part that goes in config/deploy.rb) and put it in a file under that directory (such as lib/capistrano/tasks/optional_linked_files.rake).

@speckins Thanks for the tip! I tested this and of course it works beautifully. Thanks for making it easy plug-n-play 👍


Basepairs in 34 b.p. cell barcode are being re-ordered

Hi, I'm using kallisto and bus tools to process split pool scRNA-seq data that has a 34 base pair long cell barcode and a 10 base pair UMI.
In the final output, the cell barcode base pairs are returned in the wrong order. I.e. instead of returning the cell barcode cbc the software is returning cbc[2:34] + cbc[23:34] (with 0-indexing). I've confirmed this by manually inspecting the first 34 base pairs of the cell barcodes in my R1 fastq files.
Example code that generates these outputs is as follows. Note I do not run bustools correct because the cell barcodes in my R1 fastqs are already corrected by an upstream step in my pipeline.
kallisto bus         -i /path/to/kb.idx        -x 0,0,34:0,34,44:1,0,0    -t 60      -o output_dir /path/to/R1.fastq.gz  /path/to/R2.fastq.gz
bustools sort         -t 60         -o output_dir output.bus.sorted        output.bus
mkdir -p output_dir/genecounts
bustools count         -o output_dir/genecounts/         --genecounts         -g /path/to/t2g_gene_names.txt         -e output_dir/matrix.ec         -t /path/to/transcripts.txt         output_dir/output.bus.sorted

Barcodes >= 32 bp are not supported by bustools.

Thanks for the clarification, I'll try another approach.


Update Doc links

This request contains changes to doc links within the Markdown files. Links are updated to direct to correct path, either by updating the link's filename or by replacing a link with the absolute URL (for cross-repo doc links). These errors originally occurred during Doc migration.

@chadpowers Looks good except for the merge conflicts.

@chadpowers Chad - we'll need to merge the conflicts in the READMEs here to complete this PR.  Can you please follow the instructions above to 'use the command line' (click to expand the instructions).
Thanks!


Should train_ratio of Atari 200M be 128?

Hi, I noticed that in the paper, the replay ratio for Atari 200M experiment is 32, and from my understanding, we have
replay_ratio / (action_repeat * batch_steps) ==  num of gradient steps / num of environment steps

where action_repeat=4, batch_steps=2^10.
However, replay ratio is not directly defined in this repo. In config.yaml, I find train_ratio which is defined as, from what I believe,
train_ratio /  batch_steps ==  num of gradient steps / num of environment steps

So, we have
train_ratio (in code) == action_repeat * replay_ratio (in paper)

For Atari 200m, since action repeat is 4, and replay ratio, as reported in the paper, is 32 (see here), so I think the train_ratio should be 128. But in the code it's set to be 32, which makes me confused.

The agent doesn't know about action repeat, that's only applied inside the environment and inside the logger.

The agent doesn't know about action repeat, that's only applied inside the environment and inside the logger.

Thanks for your reply! So can you tell me the train_ratio value to reproduce the Atari 200M results in the DreamverV3 paper?

Yes, train ratio 32 is correct as in the code. The train ratios in the paper are also in terms of policy frames not agent frames.

Yes, train ratio 32 is correct as in the code. The train ratios in the paper are also in terms of policy frames not agent frames.

I see. May I also ask about the correct train_ratio value for atari100k? Because in paper it's 128. while in code it's 256 (see here).


Firebase Analytics & Crashlytics integration

Implement Firebase Analytics to get user behaviour data and Firebase Crashlytics to get crash report

I can do this! Please assign

Okay then


call lua function in action

hey, I have a setup using fzf.vim and nvim-lspfuzzy, with a custom function to send fzf selections to q qflist.
qf_populate = function(lines, mode)
    if mode == nil then
        lines = core.foreach(lines, function(item) return { filename = item, lnum = 1, col = 1 } end)
        mode = "r"
    end

    vim.fn.setqflist(lines, mode)
    vim.cmd [[
        copen
        setlocal statusline=%!v:lua.StatusLine('QuickFix')
        setlocal nobuflisted
        wincmd p
        echo hello
    ]]
end

and my fzf_actions is set as:
vim.g.fzf_action = {
    ["ctrl-q"] = qf_populate,
    ["ctrl-x"] = "split",
    ["ctrl-v"] = "vsplit"
}

this setup works for fzf.vim, and a few custom LSP handlers I wrote (using qf_populate for multi-file refactors).  How can I get lspfuzzy to call this function, with the first param as the lines. ?

Hello,
If you use  to select multiple entries with lspfuzzy, then select one entry with , the plugin jumps to the selected entry and put the remaining entries into the quicklist. Does that suit the workflow you're requesting?

Yes its the functionality i want, but i want to set the statusline and also want the ability to extend it . Is it possible to call the lua function? Fzf.vim is able to.

It should work like you want in the latest commit. Can you try and tell me if it does?

@ojroques yes it is perfect! I really appreciate you responding and reacting so quickly on this amazing project, it just shows the commitment, and effort you have put into making so many of our developer experiences better! Thank you :)

okay I am noticing that even after calling the action function, it automatically sets the qflist and opens that split, which defeats the purpose of my qf_list function. I we can let action take complete control over what happens with the selections, which makes it more like the last callback.
I suggest letting the function return calling functional action. What do you think?


Confusing behavior on duplicate keys?

demo at https://codesandbox.io/s/PlWREDon
to reproduce the issue:

type '1212' then click on Patch! button

type '121211' then click again on Patch! button

Actual output

For info here is a comparison with other virtual dom libs (demo)

console output

React errors ~and breaks on the first duplicate key encountered but the UI stays frozen on the the last 'correct' state~. ~Actually react doesnt break on duplicate keys. it warns for each duplicate key encountered and just skips them~. react v16 no longer skips duplicate keys. see https://twitter.com/trueadm/status/893491470409154560
react v16 output

react v15 output

Inferno warns  but doesn't break. Other libs handle duplicate keys without warning.
EDIT
Inferno warns early at the time the element with duplicate keys is created. While react error is not thrown until reconciliation.

We now print a warning when children contain duplicate keys. It's unlikely that we'll introduce a workaround to somehow guessing the correct keys due to our strict size constraints ✔


Add new animations and improve animation blueprint

Add new animations for armed and unarmed states.
Improve player animation blueprint and locomotion to work based on movement and armed states.

Done in commit 91daf9822afdce0cfc03be8cdbb361696b5d1261.


fix: duplicated tags when using x-go-custom-tag extension

Go-swagger would generate duplicated tags while using the x-go-custom-tag extension.
for example:
    properties:
      tags:
        items:
          type: string
        type: array
        x-go-custom-tag: '"json:",inline"'

The output looks like this:
Tags []string `json:"tags" json:",inline"`

After this PR merged, the output will be:
Tags []string `json:",inline"`

Hi @casualjim
Sorry for disturbing you.
Could you please help to review this PR or assign the committer to review it?
Many thanks.

If everything looks ok, I will help to update the documentation.

@casualjim
Sorry for disturbing you again.
I just want to make sure this PR is ready to be merged.
Thank you.

It's good to be merged, but I have 1 comment

@casualjim
The PR is updated, please help to check it.
Tks!

Hi @casualjim
Sorry for disturbing you again.
Please let me know if have anything I need to do for merging this PR.
Also, I'm interested to become a maintainer of the go-swagger.
If you have any suggestion, please let me know.
Thank you.

Sorry, for the long wait

@casualjim @ianchen0119 This PR unfortunately introduces a bug. Prior to this PR, this swagger (https://gist.github.com/dadgar/6c16479180f94b76789ac84fd886e962) could be generated and now it fails with the error I described in https://github.com/go-swagger/go-swagger/issues/3142.


CLI equivalents for Azure DevOps Deployments?

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

azure-devops
CLI equivalents for Deployments under the "Release" service

What is the CLI equivalent for getting a list of deployments (REST)  for a given definition or is there none yet?

route to service team


CircleCI is broken

:-(
https://circleci.com/gh/ozkriff/zemeroth/194
See: https://github.com/tomaka/android-rs-glue/issues/162

I don't know what it was, but now everything seems to work fine again. :-/


ci: Switch to GitHub Actions

You should probably update the badges in the readme as well?

Nope I'll do that separately


URL does not link to DuckDuckGo

https://github.com/sundowndev/dorkgen/blob/8d0b63cafeac9e1cf0bd35f1ad3c0a24721eb823/duckduckgo/duckduckgo.go#L9

Thanks for pointing this out, issue has been fixed in v1.3.1.


Process explorer does not show memory usage on remote server

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.84.1
OS Version: Windows_NT x64 10.0.19045(local) Linux x64 4.15.0-208-generic(remote)
Steps to Reproduce:

open process explorer

the CPU(%) is working (it shows 0 only because it's not working right now) but the memory usage on remote server stays at 0 all the time

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.84.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

updated to 1.84.1 and nothing seems to change

Confirmed this with a codespace


Theme song is x10 louder than all other audio

Love your plugin, it works basically flawlessly
But could you please add a volume setting to your plugin?
It doesn't need to be anything fancier than a % slider in the plugins menu, and I would be 1 happy camper
Thanks

Confirmed, this happens for me as well.
Maybe not 10x louder, but about double what it needs to be.
I end up turning down the amp while picking an episode, just to have to turn it up again once we play an episode.

Do we have any solution to this yet? It would be perfect if the theme song volume would be at the same level as the video itself.

Do we have any solution to this yet? It would be perfect if the theme song volume would be at the same level as the video itself.

I haven't had any problems with it for awhile.  From a quick look, it looks like the theme songs are getting downloaded are lower volume now. I was running an mp3gain app to check and bulk update the theme songs, but it seems like somethinng has changed. It could be that theTVDB is serving up quieter mp3s now, I'm not sure as I haven't done much testing at all.

Facing the same issue, I created a simple python script to make the theme songs more atmospheric. I am not savvy enough to create a pull request in this project, but maybe it helps or inspires someone else:
https://github.com/traptegies/atmosphere_adjuster/tree/main

Please add a volume setting for the theme song playback or a way to mute it in the interface.

Hi, this plugin only downloads the theme songs, the actual implementation of playing them is in jellyfin. Please, open a an issue on https://github.com/jellyfin/jellyfin/issues or create a feature request on https://features.jellyfin.org/

Seems like there is already a request for this as a feature for JellyFin https://features.jellyfin.org/posts/1749/volume-setting-for-theme-songs


Using Glide instead of Picasso

I noticed the images stretch sometimes and do not re-size properly. I want to use Glide, how can I override Picasso so that I can use Glide?

+1 , same question

do this
public class GlideSliderView extends BaseSliderView {
protected void bindEventAndShow(final View v, ImageView targetImageView) {
v.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v) {
if (mOnSliderClickListener != null) {
mOnSliderClickListener.onSliderClick(GlideSliderView.this);
}
}
});
    if (targetImageView == null)
        return;

    if (mLoadListener != null) {
        mLoadListener.onStart(this);
    }
    RequestManager p = (mGlide != null) ? mGlide : Glide.with(mContext);
    DrawableTypeRequest rq = null;
    if (mUrl != null) {
        rq = p.load(mUrl);
    } else if (mFile != null) {
        rq = p.load(mFile);
    } else if (mRes != 0) {
        p.load(mRes);
    } else {
        return;
    }
    if (rq == null) {
        return;
    }
    if (getEmpty() != 0) {
        rq.placeholder(getEmpty());
    }
    if (getError() != 0) {
        rq.error(getError());
    }
    v.findViewById(com.daimajia.slider.library.R.id.loading_bar).setVisibility(View.INVISIBLE);
    rq.diskCacheStrategy(DiskCacheStrategy.ALL).thumbnail(0.1f).skipMemoryCache(true).centerCrop().into(targetImageView);

}
}

This works only if you download the library, because you need to change some super properties from private to protected. But thanks anyway.

public class GlideSliderView extends DefaultSliderView {

    private View progressBar;

    public GlideSliderView(Context context) {
        super(context);
    }

    protected void bindEventAndShow(final View v, ImageView targetImageView) {
        if (progressBar == null) {
            progressBar = v.findViewById(com.daimajia.slider.library.R.id.loading_bar);
        }
        if (progressBar != null) {
            progressBar.setVisibility(View.INVISIBLE);
        }

        Glide.with(getContext())
                .load(getUrl())
                .centerCrop()
                .crossFade()
                .into(targetImageView);
    }
}

its not working.images not loaded from url :(

can i load images from json?
i've tried but the images not appear

Pull request #289 is an implementation that splits the library into a "picasso" and "glide" product type so that you can choose which image library to use. The demo app in that PR was changed over to use the glide implementation for testing/reference.

You can check this library https://github.com/OpenSooq/Pluto
it has the ability to have full control over your views and layout and not depending on any image library


_this.packager is not a function

I'm not sure if this is a bug with meteor-desktop or electron-packager - or if I'm doing something wrong.
I run my app locally with:
meteor --mobile-server=<IP_ADDRESS>:3000 in one tab
and in another tab
npm run desktop -- --scaffold
And everything works as you'd expect.
But when I try to package my application to .app/.exe using
npm run desktop -- package
I get an error:

ERROR  index:  while trying to build a package an error occurred: TypeError: _this.packager is not a function

Am I missing something?

Hello @gunnartorfis,
I fixed the problem with import, but I don't know when it will be released


Use window instead of self

Closes #631.

Please review #690. The intent is to retain support for windowless JavaScript contexts.


fix: fix integ test errors when running with py2

this change fixes module import errors in the test directory when running with Python2.7
reduce max training steps in the mnist test from 1000 to 200 in order to shorten test runtime

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Favicon

Could be useful: https://github.com/barryclark/jekyll-now/pull/642

Thanks Petr! :)

Fixed in ac7f247682a3dddc8618fc907b37428cd53bfeaa


Unexpected Behavior: Ghost elements on rerender of Array of Elements

The Problem
I'm rendering a table of values, with rows that are loaded in async, and are updated throughout the life of the app. When the list is updated, however, it merely prepends the new list and keeps the old list in place. From my research, similar problems have been identified, but they mainly refer to the Render() function and not my specific case.
Steps to Recreate

Go to https://mayormonty.github.io/skills/ (a public instantiation of the app)
Wait for the table to appear (Loading... should disappear)
Open the developer console
Type the following to update the filter, and therefore rerender the list:

store.set("filters.country", "United States")

Wait for the new data to be loaded in
Observe as the new data is rendered above the old data (which now doesn't have checkboxes for some reason)

Extra Information

In my file to render the table, I'm not including keys because having them didn't change the problem. Should I be using them?
The aforementioned DataTable component is stateless
I'm using Preact CLI (latest)
I'm using Preact Material Components
My entire codebase can be found at https://github.com/mayormonty/skills

Questions

Is this a diffing issue? Can I resolve it by disabling it (via componentShouldUpdate()) somewhere?
Can I mess directly with the HTML in some way (via this.base.innerHTML) that might resolve this issue?
Should I include keys?
I've experienced similar problems before. Is this a problem with me, or with preact?

Thanks for your insight!

D'oh! In my previous implementation of keys, I had many, many duplicates. This isn't an issue with Preact


Credit Transactions | Adjust Confirmation alert message for Recall as Draft button |1|

Details:
Transaction: PVR
Role: Government Analyst
Status: Reviewed
Button: Recall as Draft
Current messaging:
"Are you sure you want to pull this transfer back?"
Revised messaging:
"Are you sure you want to recall this credit transaction? This will return the transaction to a draft state where it will no longer be visible to the Director."

 Credit Transactions | Adjust Confirmation alert message for Recall as Draft button |1|


Add ability to return to zero move

When adding new variation right after game start (i.e. another possible 1st move) it is shown incorrectly - at the bottom of the current line. @SmekhMaksimgithub


[BUG]DefaultAzureCredential / EnvironmentCredential  doesn't have an option to set ClientCertificateCredentialOptions.SendCertificate

Describe the bug
When we use DefaultAzureCredential and then set the AZURE_CLIENT_CERTIFICATE_PATH,  it fails to authenticate to AAD because the Application is only enabled Subject Name + Issuer authentication and the hardcoded behavior of EnvironmentCredential is set ClientCertificateCredentialOptions to null.
Expected behavior
We should have an option to allow ClientCertificateCredentialOptions.SendCertificate to be set to true when using DefaultAzureCredential / EnvironmentCredential.
Actual behavior (include Exception or Stack Trace)
There's no such code path that we can do it today.
To Reproduce
As is.
Environment:

Azure.Identity, Version="1.3.0"
Service Fabric Local dev + Windows 10, .Net Core App 3.1
Visual Studio 16.5

Hi @yangyud-cn - Could you try constructing a ChainedTokenCredential that includes a ClientCertificateCredential (among the other credentials you prefer)? This would allow you to configure the options fully.

@christothes, as a work around I can do it but the idea here is to make sure we could reuse existing things as much as possible.

Hi @yangyud-cn DefaultAzureCredential is meant to be used as a simple default option that works for most scenarios. You can think of ChainedTokenCredential as an advanced version of DefaultAzureCredential that allows you to configure it with full control.
In your case, you wouldn't add DefaultAzureCredential to the ChainedTokenCredential, rather each specific credential you want in the chain. For example:
var credential = new ChainedTokenCredential(
    new ManagedIdentityCredential(),
    new EnvironmentCredential(),
    new AzureCliCredential(),
    new ClientCertificateCredential(new ClientCertiificateCredentialOptions{ <...>}),
    new VisualStudioCredential());

@christothes,  sorry for the late reply.  ChainedTokenCredential requires each credential to be not null, and ClientCertificateCredential requires all basic arguments to be valid on ctor, while EnvironmentCredential provides a safe fall back in the case where env variables are not set.    So I will have to write my own EnvironmentCredential to not throw in the case of missing env variable and to return ClientCertificateCredential  when we have the env variables.

@yangyud-cn - I think you may get the behavior you want with ChainedTokenCredential using just these credentials:
var options = new ClientCertificateCredentialOptions { SendCertificateChain = true };
var credential = new ChainedTokenCredential(
    new ManagedIdentityCredential(),
    new EnvironmentCredential(options));

The logic of the EnvironmentCredential already has conditional logic to use the certificate credential if the AZURE_CLIENT_CERTIFICATE_PATH environment variable is set, and the ChainedTokenCredential has the logic to try the MSI first and fallback to the EnvironmentCredential when it is not available.

@christothes Thanks for the update. I missed that EnvironmentCredential can take options :)

+1.

Basically one the most important for each of them. But it doesn't expose anything for certificate based auth, and SendCertificateChain is exactly one.
I'm willing to work on a pull request if it'll be considered.
Otherwise I'll file a pull request to remove all custom properties, because it's black and white: either they
all belong to DefaultAzureCredentialOptions or none do.

@yangyud-cn please reopen the issue.


add graphql schema indexing to search

Codecov Report

Merging #158 (4080624) into main (be03907) will increase coverage by 0.00%.
The diff coverage is 66.66%.

:exclamation: Current head 4080624 differs from pull request most recent head 578cbf5. Consider uploading reports for the commit 578cbf5 to get more accurate results

@@           Coverage Diff           @@
##             main     #158   +/-   ##
=======================================
  Coverage   88.88%   88.89%
=======================================
  Files          88       88
  Lines        4788     4790    +2
  Branches      679      679
=======================================
+ Hits         4256     4258    +2
  Misses        475      475
  Partials       57       57

Impacted Files
Coverage Δ

...ckages/plugins/fuse-graphql/src/indexer/indexer.ts
89.62% <66.66%> (+0.15%)
:arrow_up:

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


trying ruby bullseye

changing image to a mac os compatible image

We can close this PR, as we already updated the image to ruby3


Add ShiftedDatum

closes #41

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

Project coverage is 94.60%. Comparing base (20f7d59) to head (29d7838).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #58      +/-   ##
==========================================
+ Coverage   94.58%   94.60%   +0.01%
==========================================
  Files          21       22       +1
  Lines         979      982       +3
==========================================
+ Hits          926      929       +3
  Misses         53       53

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


Redirecting to wrong link

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Both the GitHub and Linkedin logo are redirecting to the Career Zunction website only and not the respective platforms. I want to work on this issue under GSSoC'24. Please assign me this.
Expected behavior
The logos should redirect to the respective websites and not to the Career Zunction website.
Add ScreenShots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

I want to work on this issue under GSSoC'24. Please assign me this.

@HimanshuNarware can you please assign me this issue under GSSoC'24.

Please assign this issue to me under GSSOC"24


Implement serde modules for FeeRate

Implement and enforce explicit unit when serializing. This is as we do for Amount (see #3672 for similar).
To test it, and as part of the 1.0 effort; add regression tests for serde stuff in units.
With this applied one must use attributes to serialize FeeRate.
    #[derive(Serialize, Deserialize)]
    pub struct Foo {
        #[serde(with = "bitcoin_units::fee_rate::serde::as_sat_per_kwu")]
        pub fee_rate: FeeRate,
    }

CI failures look real.

Why do we have special code for serde for Option?

@sanket1729 if a type Type does not have a default serde impl, then when it appears in a struct field you need to annotate it with something like
#[derive(Serialize)]
struct Struct {
    #[serde(with = "serde_module")]
    fe: Type,
}

But if fe is an Option<Type> then there's nothing you can do (well, ok, you can explicitly implement the serde impls for Struct), unless the library authors for Type kindly provided a separate module specifically for Option<Type>.
The same is true for Vec<Type>, Cell<Type>, etc., but in practice Option is by far the most common case, because in de/serialization code it represents "this field is allowed to be missing".

Did you want to weigh in on this again @sanket1729? Otherwise merge please when you get a chance.

I can't merge this because it causes serde regression tests to fail (I think from #3659).

Need to rebase and update those tests (assuming they are supposed to break)

Thanks, I did expect them to break but had totally forgotten about it.

Will need #3721

I'm willing to bet that if #3704 goes in first this will have lint issues.

Needs rebase.


JRuby Nokogiri indents XML if :indent is specified even if :save_with => 0 specified

I'm using Nokogiri 1.5.6 with JRuby 1.7.2 (1.9 mode):
ruby --version
jruby 1.7.2 (1.9.3p327) 2013-01-04 302c706 on Java HotSpot(TM) 64-Bit Server VM 1.6.0_37-b06-434-11M3909 [darwin-x86_64]

Test script:
require 'nokogiri'

x = Nokogiri::XML("<foo><bar>baz</bar></foo>")

puts "INDENT:"
puts x.to_xml(:indent => 2, :save_with => 0)

puts "NO INDENT:"
puts x.to_xml(:save_with => 0)

In JRuby:
INDENT:
<?xml version="1.0"?>
<foo>
  <bar>baz</bar>
</foo>
NO INDENT:
<?xml version="1.0"?>
<foo><bar>baz</bar></foo>

In MRI 1.9.3:
INDENT:
<?xml version="1.0"?>
<foo><bar>baz</bar></foo>
NO INDENT:
<?xml version="1.0"?>
<foo><bar>baz</bar></foo>

It seems like in the JRuby version, specifying :indent forces indentation even if :save_with => 0 is specified (which should turn off indenting since it's missing the FORMAT flag).

Ugh, this is so complicated and has so much history in it ...
https://github.com/sparklemotion/nokogiri/issues/415 led us to the decision to turn OFF JRuby formatting by default (but it's ON by default in CRuby).
As a compensating factor, we then made JRuby turn formatting back on if indentation is provided (assuming reasonably that people probably want formatting if they're providing indentation).
However, if you explicitly say "no formatting" and also provide indentation, then yes the behavior will be different between the implementations, because CRuby (libxml2) will only use indentation if formatting is on.
The right thing to do here might be to have CRuby also implicitly turn formatting on if indentation is provided, but this would likely surprise people who have explicitly asked for formatting to be turned off.
While I acknowledge this is not working as expected, the obvious workaround is to not provide indentation options if you don't want it formatted, and so I'm going to mark this "wontfix".
Thanks for your patience, and sorry I didn't respond on this sooner.


feature: Zap support

I have not done too much thinking about this but adding it so I, or someone else, can take a look at it later. Does it make sense to support Zaps in nostr-tool?

As far as i digged, Zap is not supported yet natively in the nostr-sdk. I couldn't find any reference to NIP-57, be it in the issues or in the code.
I guess there is a way to implement its support, but it would make sense to design it for the SDK first and use the implementation rather than creating an external implementation.

I used nostr-sdk for https://github.com/thesimplekid/cln-zapper-rs and was thinking it would make sense to upstream some of it didn't just to get it working. But plan to take another look.

https://github.com/rust-nostr/nostr/pull/69

https://github.com/rust-nostr/nostr/pull/72

Fixed in #47.


Upgrade Checkstyle 10.19.0 -> 10.20.0

This PR contains the following updates:

Package
Type
Update
Change

Checkstyle (source)
build
minor
10.19.0 -> 10.20.0

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Checkstyle 10.19.0 -> 10.20.0

See:
- https://checkstyle.sourceforge.io/releasenotes.html
- https://github.com/checkstyle/checkstyle/releases/tag/checkstyle-10.20.0
- https://github.com/checkstyle/checkstyle/compare/checkstyle-10.19.0...checkstyle-10.20.0


Configuring Backups in a ManagedSeed with Gardenlet does not work

How to categorize this issue?

/area backup
/area control-plane-migration
/kind bug
/priority 1
What happened:
I am deploying ManagedSeeds with the GardenletConfig in it. I also configure the spec.backup part of it. However, the resulting Seed does not get the required Backup entries created. As a consequence, the BackupBucket is not created and no Shoot is backed up.
What you expected to happen:
Seedgets Backup entry from ManagedSeed configuration.
How to reproduce it (as minimally and precisely as possible):
apiVersion: seedmanagement.gardener.cloud/v1alpha1
kind: ManagedSeed
metadata:
  name: seed-test
  namespace: garden # Must be garden
spec:
  shoot:
    name: seed-test
  gardenlet:
    mergeWithParent: true
    config:
      apiVersion: gardenlet.config.gardener.cloud/v1alpha1
      kind: GardenletConfiguration
      featureGates:
        APIServerSNI: true
        Logging: true
        ManagedIstio: true
        ReversedVPN: true
        MountHostCADirectories: true
      seedConfig:
        provider:
          type: aws
          region: me-south-1
        backup:
          provider: aws
          region: me-south-1
          secretRef:
            name: seed-test-seed-backup
            namespace: garden

results in this Seed:
Name:         seed-test
Namespace:    garden
Labels:       gardener.cloud/role=seed
Annotations:  seed.gardener.cloud/ingress-class: nginx
API Version:  core.gardener.cloud/v1beta1
Kind:         Seed
Metadata:
  Creation Timestamp:  2021-08-27T10:50:05Z
  Finalizers:
    core.gardener.cloud/controllerregistration
    gardener
  Generation:  1
    Manager:         gardenlet
    Operation:       Update
    Time:            2021-08-27T10:55:28Z
  Resource Version:  16985963
  UID:               84e15c2b-68b2-4cdd-8bf6-e536696953e8
Spec:
  Dns:
    Ingress Domain:  <redacted>>
  Networks:
    Nodes:     <IP_ADDRESS>/16
    Pods:      <IP_ADDRESS>/17
    Services:  <IP_ADDRESS>/17
  Provider:
    Region:  me-south-1
    Type:    aws
  Settings:
    Excess Capacity Reservation:
      Enabled:  true
    Scheduling:
      Visible:  true
    Shoot DNS:
      Enabled:  true
    Vertical Pod Autoscaler:
      Enabled:  false
Status:
  Allocatable:
    Shoots:  250
  Capacity:
    Shoots:          250
  Cluster Identity:  <redacted>
  Conditions:
    Last Transition Time:  2021-08-27T10:53:56Z
    Last Update Time:      2021-08-27T10:53:56Z
    Message:               Gardenlet is posting ready status.
    Reason:                GardenletReady
    Status:                True
    Type:                  GardenletReady
    Last Transition Time:  2021-08-31T14:22:19Z
    Last Update Time:      2021-08-31T14:22:19Z
    Message:               Seed cluster has been bootstrapped successfully.
    Reason:                BootstrappingSucceeded
    Status:                True
    Type:                  Bootstrapped
    Last Transition Time:  2021-08-27T11:02:33Z
    Last Update Time:      2021-08-27T11:02:33Z
    Message:               All extensions installed into the seed cluster are ready and healthy.
    Reason:                AllExtensionsReady
    Status:                True
    Type:                  ExtensionsReady
  Gardener:
    Id:                 96c6fef83bdee5d8559cd4e87a842ac485a840dfb0cfd24cf58247ec0ee51781
    Name:               gardenlet-5d76cffc6-bj29s
    Version:            v1.29.0
  Kubernetes Version:   v1.20.10
  Observed Generation:  1
Events:                 <none>

only after editing the Seedand manually adding the backup section from above, it does pick it up:
Name:         seed-test
Namespace:    garden
Labels:       gardener.cloud/role=seed
Annotations:  seed.gardener.cloud/ingress-class: nginx
API Version:  core.gardener.cloud/v1beta1
Kind:         Seed
Metadata:
  Creation Timestamp:  2021-08-27T10:50:05Z
  Finalizers:
    core.gardener.cloud/controllerregistration
    gardener
  Generation:  2
    Manager:         gardener-controller-manager
    Operation:       Update
    Time:            2021-08-31T17:41:43Z
  Resource Version:  17089085
  UID:               84e15c2b-68b2-4cdd-8bf6-e536696953e8
Spec:
  Backup:
    Provider:  aws
    Region:    me-south-1
    Secret Ref:
      Name:       seed-test-seed-backup
      Namespace:  garden
  Dns:
    Ingress Domain:  <redacted>
  Networks:
    Nodes:     <IP_ADDRESS>/16
    Pods:      <IP_ADDRESS>/17
    Services:  <IP_ADDRESS>/17
  Provider:
    Region:  me-south-1
    Type:    aws
  Settings:
    Excess Capacity Reservation:
      Enabled:  true
    Scheduling:
      Visible:  true
    Shoot DNS:
      Enabled:  true
    Vertical Pod Autoscaler:
      Enabled:  false
Status:
  Allocatable:
    Shoots:  250
  Capacity:
    Shoots:          250
  Cluster Identity:  shoot--garden--seed-test-40f3bbef-4dd0-4741-9dcf-8f061e303abe-finleap-cloud-garden
  Conditions:
    Last Transition Time:  2021-08-27T10:53:56Z
    Last Update Time:      2021-08-27T10:53:56Z
    Message:               Gardenlet is posting ready status.
    Reason:                GardenletReady
    Status:                True
    Type:                  GardenletReady
    Last Transition Time:  2021-08-31T21:14:58Z
    Last Update Time:      2021-08-31T21:14:58Z
    Message:               Seed cluster has been bootstrapped successfully.
    Reason:                BootstrappingSucceeded
    Status:                True
    Type:                  Bootstrapped
    Last Transition Time:  2021-08-27T11:02:33Z
    Last Update Time:      2021-08-27T11:02:33Z
    Message:               All extensions installed into the seed cluster are ready and healthy.
    Reason:                AllExtensionsReady
    Status:                True
    Type:                  ExtensionsReady
    Last Transition Time:  2021-08-31T17:41:42Z
    Last Update Time:      2021-08-31T17:41:42Z
    Message:               Backup Buckets are available.
    Reason:                BackupBucketsAvailable
    Status:                True
    Type:                  BackupBucketsReady
  Gardener:
    Id:                 96c6fef83bdee5d8559cd4e87a842ac485a840dfb0cfd24cf58247ec0ee51781
    Name:               gardenlet-5d76cffc6-bj29s
    Version:            v1.29.0
  Kubernetes Version:   v1.20.10
  Observed Generation:  2
Events:                 <none>

Only then a BackupBucket is created and backups are written.
Anything else we need to know?:
Environment:

Gardener version: 1.28.x and 1.29.0
Kubernetes version (use kubectl version): 1.20.10
Cloud provider or hardware configuration: AWS
Others:

cc @stoyanr @timebertt

A little addition here:
the initial ManagedSeed manifest is how it is rendered. I just saw the the "backup" block is not actually written to the etcd apparently. If I get the YAML from the cluster, the backup part is not actually present.

Ok, I am sorry. I totally did not see that the .seedConfig requires a .spec sub field. With that added and everything shifted right, it works just fine. Weird though, that I did not receive any kind of validation error.


Error while training system

I got "MemoryError" while training system.

The output of the error is attached above.
What can be the reason to get this error? Also I changed memory amount of vagrant file from "1024" to "2048".

You probably need more memory then 2048 MB.  It's been some time since I've run the training so don't remember how much off hand -- but give your vagrant VM as much memory as you can and try again.

I think the problem is not the memory only. While I was trying to install pylab, after finishing matplotlib installation, the same error was appeared. The problem might be storage or version inconsistency.

This is almost certainly due to not enough memory in your vagrant VM.  Did you try to increase the memory and re-run?  I just now verified that extract_gold_standard_all_training_data(rootdir) runs without problems on a fresh Vagrant VM with 8192 MB of memory.


[Help] rgbd_pcl low FPS

Hi, I have been trying to get depth and rgb images published simultaneously from a Turtlebot4 with an Oak-D-Lite. Currently I'm using rgbd_pcl.launch.py, but am having frame rate issues (~4-5 FPS). I have tried setting i_low_bandwidth to true for everything from camera.yaml in depthai_ros_driver/config/ as well as lowering the i_low_bandwidth_quality to 20. However, nothing seems to improve the performance. It looked like I should be editing camera.yaml, but it was mentioned in another issue that rgbd_pcl has its own .yaml file (which appears to now be gone) https://github.com/luxonis/depthai-ros/issues/329.
Any insight would be appreciated. Not sure if I've modified the low_bandwidth setting incorrectly or if there's a better way to output lower resolution images.

Hi, are you subscribing to the images from other PC? If yes, framerate can be influenced by network/DDS settings, here you can find a guide on tuning DDS. If not, please share your config file and exact setup according to the bug template.

I had the same issue, until I realize it was the cable... I tested 3-4 cables until one gave me the proper FPS using the rgb_stereo_node.launch , I used to have 7-8 fps and now with a proper cable I have 30 on the color image


mac check in GCM failed

This is the error or issue the occurs when i try the statements
"the error is within the attachment"
error output.txt
And this is the program it self
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using CockyGrabber.Grabbers;
using CockyGrabber;
using System.IO;

namespace Test_Ground
{
    internal class Program
    {
        private const string LogFilePath = "AAA.txt";
        static void Main(string[] args)
        {
            Console.Title = "Test Ground - Console Apllication";
            Console.Write("Starting Ground Work");
            Console.WriteLine("");

           // UniversalGrabber g = new UniversalGrabber(); // Create Grabber
            OperaGxGrabber g = new OperaGxGrabber();

            File.AppendAllText(LogFilePath, g.GetLogins().ToString()); // Append the grabbed Logins to the log file
            Console.ReadKey();

        }
    }
}

And yes i installed the nugget packages

Are you using the latest version? Is your project 86x (32bit) or 64x?

Yes

Is the problem only in OperaGxGrabber or other grabbers too?
Have you tried to make it work on other systems and if not can you try?

i tried it on OperaGx but as it shows fails with the error but with the other not sure i dont have them installed in the mean time

Are you sure you don't have edge installed? It's usually preinstalled on windows 10. What OS are you using as as I asked: Can you test it on an other system like a virtual Maschine?

I got edge installed, Windows 11 Workstations, Same error

im gonna try older .net framework versions ill update you later on

the chrome cookie grabber works but on opera gx same error

my main goal is to get OperaGx login and Cookies and History
can you make a explanitory event that does the specific command?

I didn't test it on Windows 11 so I don't know it it does not work on it. Can you test it on win 10?

hmm i see, let me check it

but anyways what does the OS environment affect the decryption key?
tested on windows 10 same error

Could it be that OperaGx took notice of the grabber?

also did it work for you?

cause if that's the prob then it might be because the unsupported version
if that's case you should add it to the todo list

Hmm. Idk if win11 encrypts values differently. CockyGrabber uses the WinAPI so It decrypts values based the API of windows.
However: I tested it and it works for me. I can later test it on a virtual machine with win 11 and see for myself.
But maybe it's because of Opera itself as you say. What version do you use?

Hmm. Idk if win11 encrypts values differently. CockyGrabber uses the WinAPI so It decrypts values based the API of windows.
However: I tested it and it works for me. I can later test it on a virtual machine with win 11 and see for myself.
But maybe it's because of Opera itself as you say. What version do you use?

It seems like it's a local problem. I tested it on win11 on a VM and it works. The same goes for my current PC which is win10.
I will close this issue for now but if there are more problems like this I will reopen it.


Fix fill.Fill

This never worked as checked in.
I brought this from a bigger branch and clearly didn't test it at all.  :-/ :-/
Sorry, too many things.  I'll do better next time.

+1
On Fri, Jul 27, 2018 at 6:51 AM Tom Swirly<EMAIL_ADDRESS>wrote:

This never worked as checked in.
I brought this from a bigger branch and clearly didn't test it at all. :-/
:-/
Sorry, too many things. I'll do better next time.
You can view, comment on, or merge this pull request online at:
https://github.com/ManiacalLabs/BiblioPixel/pull/897
Commit Summary

Fix fill.Fill

File Changes

M bibliopixel/animation/fill.py
https://github.com/ManiacalLabs/BiblioPixel/pull/897/files#diff-0
(3)

Patch Links:

https://github.com/ManiacalLabs/BiblioPixel/pull/897.patch
https://github.com/ManiacalLabs/BiblioPixel/pull/897.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ManiacalLabs/BiblioPixel/pull/897, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6a6l_qIq5SpVPN6mqHzUi8RXpFdGdpks5uKvChgaJpZM4VjTWe
.


brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1)

[The issue template mentioned on https://ibotpeaches.github.io/Apktool/contribute/ does not link, so I could not use it].
Information
Apktool Version: 2.5.0
Operating System: Tried Centos 7 and Ubuntu 20.04.
APK From?: OpenGapps (Google Play Store APK)
I consistently get this error just rebuilding the APK without making any changes. I tried several versions of apktool, and hit the same issue each time. I've attached a script that downloads the APK and reproduces the issue.
Stacktrace/Logcat
brut.androlib.AndrolibException: brut.common.BrutException: could not exec (exit code = 1): [/tmp/brut_util_Jar_22331353039922968304304641883857725150.tmp, p, --forced-package-id, 127, --min-sdk-version, 21, --target-sdk-version, 29, --version-code, 82371110, --version-name, 23.7.11-21 [0] [PR] 352600257, --no-version-vectors, -F, /tmp/APKTOOL7682094809676514826.tmp, -e, /tmp/APKTOOL6496528476982564118.tmp, -0, arsc, -I, /home/johnjaco/.local/share/apktool/framework/1.apk, -S, /tmp/apktoolFfN8/Phonesky/./res, -M, /tmp/apktoolFfN8/Phonesky/./AndroidManifest.xml]

Steps to Reproduce

Decompile the APK with apktool d.
Recompile the APK with apktool b.
Here is a script to reproduce the issue. It uses wget to download the APK.

#!/bin/bash
TMPDIR=`mktemp -d apktoolXXXX` || exit 1
pushd "${TMPDIR}"
wget https://bitbucket.org/iBotPeaches/apktool/downloads/apktool_2.5.0.jar || exit 1
mkdir zipdir
pushd zipdir
wget https://netactuate.dl.sourceforge.net/project/opengapps/arm64/20210127/open_gapps-arm64-10.0-pico-20210127.zip || exit 1
unzip open_gapps-arm64-10.0-pico-20210127.zip
tar -xvf Core/vending-arm.tar.lz -C .
mv vending-arm/nodpi/priv-app/Phonesky/Phonesky.apk ../
popd
rm -rf zipdir
java -jar apktool_2.5.0.jar d Phonesky.apk
pushd Phonesky
java -jar ../apktool_2.5.0.jar b .
popd
popd
rm -rf ${TMPDIR}

Frameworks
N/A
APK
Google Play Store APK extracted from OpenGapps.
Questions to ask before submission

Have you tried apktool d, apktool b without changing anything? yes
If you are trying to install a modified apk, did you resign it? N/A
Are you using the latest apktool version? yes

There is not really an error with your crash, just that aapt failed to work. Try executing it on your own - does it work?
I don't really have anything to go on here, sorry.

[The issue template mentioned on https://ibotpeaches.github.io/Apktool/contribute/ does not link, so I could not use it].

fixed thanks - https://github.com/iBotPeaches/Apktool/commit/d2fa4049fa6fd5637ed0943a59252ea35cb00fa8

There is not really an error with your crash, just that aapt failed to work. Try executing it on your own - does it work?
I don't really have anything to go on here, sorry.

[The issue template mentioned on https://ibotpeaches.github.io/Apktool/contribute/ does not link, so I could not use it].

fixed thanks - https://github.com/iBotPeaches/Apktool/commit/d2fa4049fa6fd5637ed0943a59252ea35cb00fa8

I've this same issue using ubuntu 18.04. I don't know how to solve this


Installation

How i can setup the panel
i did everything but there isn't login path or login file

The login path is "/login"


Update getting started with rkt guide

The guide should include examples how to build aci images from Ubuntu, debian and etc rootfs
As right now it only covers go applications.

@rimusz Currently Docker is used to handle the cases you describe. What would a non-docker workflow look like to you?

@rimusz I'm also a big +1 on showing people how to build images "manually" using the Debian build tools, which should provide the necessary "how-to" for full automation down the road.

@kelseyhightower rkt way workflow should be very similar to the docker one, just adopted to rkt features

@kelseyhightower I would like totally switch to rkt, but so many things are not clear there yet:
building based aci images
reusing them for you app aci
using ports, specially with flannel is complete unclear
data volumes

For now, there is no easy way to build aci images. But hopefully https://github.com/appc/acbuild will help.

@alban acbuld is looking as not a bad option

Since this commit https://github.com/coreos/rkt/commit/9e6e1f67fa0da5e268e5ca699fdd89ba7845ceb5 there seems to be examples using acbuild. Does this not suffice for this issue?

Yep, good catch!


[MRG] Use python -m pip in installation instructions and add integration with other software page

This PR:

replaces pip install with python -m pip install in README and in the documentation
adds a new page in the documentation illustrating the integration with other Python packages (how to transform time series from tslearn to another package and vice versa).

This is great!
Also, pandas is a required dependency to cast to / from the following packages:

pyflux
sktime
tsfresh

I guess this information could be added to the page, no?

Yes you're right! I'm going to add this information.

Codecov Report

Merging #199 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #199   +/-   ##
=======================================
  Coverage   93.93%   93.93%
=======================================
  Files          25       25
  Lines        3432     3432
=======================================
  Hits         3224     3224
  Misses        208      208

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 abe9724...f7607fd. Read the comment docs.

Hey, by the way, if we want this PR to be included in 0.3.1, we'd better merge it into master, I think. @johannfaouzi could you change that?

I guess we should close this one, no?


Building a fastp binary on Apple Silicon

Hello, I am trying to use fastp for an RNAseq pipeline on an M2 Pro Mac Mini. When installing with brew install fastp homebrew is giving an invalid architechure message. Has anyone sucessfuly built fastp from source on Apple Silicon? Also has anyone got libisal to compile on Apple Silicon? Trying to avoid using andaconda.

These instructions provide a way to install homebrew x86 and run it alongside Apple Silicon brew to install fastp. Does not require conda.


Add No-Adviser plugin

Disables the Activity Adviser button next to the minimap when in combat, or optionally, all the time.

First time PR. Changing from com.example to com.noadviser broke it somehow. Will look into it later.

You should be using plugins=com.noadviser.NoAdviserPlugin, not just com.noadviser.

Side note, can't the activity adviser be disabled ingame? I'm not sure who this plugin caters to

It should be com.noadviser.NoAdviserPlugin in runelite-plugin.properties

Yes, it is possible to remove the activity adviser orb in the in-game settings

Thanks for this, I wasn't aware of this feature.
By default the plugin only disables the activity adviser when the player is in combat, so there is still some merit to using this if you don't want to re-enable the button every time you want to use it. For me that's the largest annoyance since it can get you killed if you misclick it during a raid, for example. It's a small benefit, but it's also a small plugin. So I'll leave it up to whoever decides these things if they think it's worthy of a spot or not.
Thanks for the help everyone!


Client fails in a confusing manner when token contains whitespace

Description
I load my API token from a file. The default behaviour of most file editors is to place a newline at the end of a file. Reading the API token from a file with that trailing newline results in failures that are not obviously related to the token containing that whitespace.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
slackclient version: 2.5.0
python version: 3.6.8
OS version(s): any
Steps to reproduce:

Open ~/.slacktoken in nano
Paste API token into file and save
Execute this Python code:

import slack
with open(expanduser('~/.slacktoken'), 'r') as f:
  slackclient = slack.WebClient(token=f.read(), run_async=True)
  slackclient.chat_postMessage(channel="somechannel", text="test")

Expected result:
{'ok': True, 'channel': 'somechannel', 'ts': '1579545629.000300', 'message': {'type': 'message', 'subtype': 'bot_message', 'text': 'test', 'ts': '1579545629.000300', 'username': 'Slack API Tester', 'bot_id': 'somebotid'}}

Actual result:
SlackApiError: The request to the Slack API failed.
The server responded with: {'ok': False, 'error': 'channel_not_found'}

Attachments:

Thanks for filing the issue and sending the PR!


Possibility to edit complete data structure in Field Plugin Sandbox

Is your feature request related to a problem? Please describe.
Currently the Field Plugin Sandbox only displays the content and the data.
I would suggest for a possibility to enter data.
Not a problem, but would help to develop plugins, that access content of the current story.
Describe the solution you'd like
The complete data structure (incl. content) should be editable and provided to the field plugin as storyblok would do it in real life.
Describe alternatives you've considered
Currently the data will be taken from an .env.local file and "simulated" that this data comes from stroyblok

Hi @RoettingerJ thanks for reaching out to us.
If I understand your request correctly, do you want to have an actual (or simulated) payload of whole story within the Sandbox? I'm curious to hear your example / experience on how it could make difference in your workflow :)

Initially it would be enough to have just a simulated payload that can be entered (or pasted in) manually.
It's used for manual testing the behavior of the plugin entering the first time (with no plugin specific data set) and the further times (when the user has changed data in the plugin)
To have an actual payload, that can be selected would make life easier - sure ;-)

@RoettingerJ thanks for your input. We could improve field-plugin.config.js file to include a dummy story object :) I've just put it in our backlog.


runtime: killed threads do not crash whole process

What version of Go are you using (go version)?

$ go version
go version go1.12 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ivan/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ivan/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build580235778=/tmp/go-build -gno-record-gcc-switches"
What did you do?
I have a program running under systemd and with SystemCallFilter applied. My list of allowed syscalls does not include madvise, which eventually leads to:
$ sudo strace -f -p $(pidof tracefwdr) 2>&1 | fgrep SIGSYS -C5
[pid 125587] <... futex resumed> )      = 0
[pid 125588] <... read resumed> "\202\201\0\temitBatch\34\34\30\10nginx-fl\31<\30\16jae"..., 65000) = 7271
[pid 125587] madvise(0xc000400000, 2097152, MADV_NOHUGEPAGE <unfinished ...>
[pid 125588] read(3, 0xc000438000, 65000) = -1 EAGAIN (Resource temporarily unavailable)
[pid 125587] <... madvise resumed>)     = ?
[pid 125587] +++ killed by SIGSYS +++
[pid 125588] sched_yield()              = 0

Here one thread is killed by SIGSYS for seccomp violation.
What did you expect to see?
Whole process panics, clearly indicating failure.
What did you see instead?
One thread is dead and random part of my program is not working anymore (either reads from channel or udp socket, not so sure).

/cc @aclements @ianlancetaylor

I don't know what we can do here.  I'm open to suggestions.

If it's not possible to detect silently crashed / killed threads from Go runtime, I guess I can ask systemd to implement SECCOMP_RET_KILL_PROCESS  as an option or return an errno:
       SECCOMP_RET_KILL_PROCESS (since Linux 4.14)
              This value results in immediate termination of the process,
              with a core dump.  The system call is not executed.  By con‐
              trast with SECCOMP_RET_KILL_THREAD below, all threads in the
              thread group are terminated.  (For a discussion of thread
              groups, see the description of the CLONE_THREAD flag in
              clone(2).)

Feel free to close if it's not possible.

Thanks.  I don't know that it's not possible.  I just don't know of a way to do it.

Killing just a thread like this seems like it would break all sorts of things, not just Go. Is this really standard behavior for systemd?
I'm also confused because your strace output indicates a SIGSYS, but Go is supposed to trap SIGSYS and turn it into a fatal runtime panic. Does seccomp somehow produce a SIGSYS that can't actually be trapped?

Yes, by default seccomp's SIGSYS is untrappable:

http://man7.org/linux/man-pages/man2/seccomp.2.html

              The process terminates as though killed by a SIGSYS signal.
              Even if a signal handler has been registered for SIGSYS, the
              handler will be ignored in this case and the process always
              terminates.  To a parent process that is waiting on this
              process (using waitpid(2) or similar), the returned wstatus
              will indicate that its child was terminated as though by a
              SIGSYS signal.

It can be trapped if SCMP_ACT_TRAP is used:

http://man7.org/linux/man-pages/man3/seccomp_init.3.html

       SCMP_ACT_TRAP
              The thread will be sent a SIGSYS signal when it calls a
              syscall that does not match any of the configured seccomp
              filter rules.  It may catch this and change its behavior
              accordingly.  When using SA_SIGINFO with sigaction(2), si_code
              will be set to SYS_SECCOMP, si_syscall will be set to the
              syscall that failed the rules, and si_arch will be set to the
              AUDIT_ARCH for the active ABI.

I agree that killing a thread seems a bit weird in context of systemd and mentioned it here:

https://github.com/systemd/systemd/issues/11967

So currently it looks like systemd can do 2 things when it comes to not allowed syscalls ( more detailed explanation can be found at https://www.freedesktop.org/software/systemd/man/systemd.exec.html#System Call Filtering:

by default, it does SCMP_ACT_KILL, which does:

              The thread will be terminated by the kernel with SIGSYS when
              it calls a syscall that does not match any of the configured
              seccomp filter rules.  The thread will not be able to catch
              the signal

This doesn't look like what we want ever.

When armed with SystemCallErrorNumber, it returns the error number instead, which maybe what we want here.

One interesting thing i noted out of this is that we call madvise without checking return code ( https://github.com/golang/go/commit/295a4d8e6433e8a8b6df25375fb780b0f75ff4e6 ), so returning EPERM errors doesnt signal failures in those cases. Should we do more thorough checks for madvise ? I'm not sure if there are any other occurrences.

I don't think it matters if madvise fails.  The program won't return memory to the OS but will otherwise run unchanged.


Error in copying frida snippet

当hook构造函数的时候，返回时应该为init，不是new
let ret = this.$init("哎呦", amount);

Google Translate:
When hooking the constructor, the return should be init, not new let ret = this.$init("Whoops", amount);

@335158573 I think this issue already fixed in PR #1605 by replacing $init with $new, so please upgrade jadx to the latest version. If this is not your case, please add more details to your report.


Filter pushed down too far

Which issue does this PR close?
Closes #12762.
Rationale for this change
Fix the bug
What changes are included in this PR?
No fix yet, just testing in CI.
Are these changes tested?
Yes
Are there any user-facing changes?
When complete, more queries should return correct results

Tests passed, proving it was my "rusty" recollection of SQL tri-state logic that was the issue.


[Core] Nomenclature

Derived from https://github.com/callemall/material-ui/pull/3682
There are a number of inconsistencies with naming in the current library.  Some examples include <RaisedButton /> and <FlatButton /> vs <TextField /> and <TextField multiLine={true} />.
We should agree on some guidelines to answer the questions "When is a new property/feature on a component a new component?" and "What should this component be called?"
Some suggestions to trigger further discussion.

Formatting is a component concern, therefore <FlatButton /> and <RaisedButton /> should probably be combined.  And the existence of <IconButton /> should be questioned.
Behaviour changes should often make new components.
a) TextField Multiline should be a different component as the number of lines, and autogrowth probably warrants separation.  Most TextField uses won't need this functionality.
b) Also, <FloatingActionButton /> perhaps has different behaviour, (though we don't currently implement much of that) so should be a separate component.
c) <ListItem /> has nestedItems and children, which by proxy <MenuItem /> then has.
It would be nice to scope naming of similar components... I quite like <Progress.Linear />, and <Progress.Circular /> as the end product for users of material components.  The implementation of this might be too hacky, so <LinearProgress /> vs <ProgressLinear /> should be considered.  Also, should <DatePicker /> and <TimePicker /> fall under this categorisation like the material spec?
Material Design Spec has neither <DropDownMenu /> nor <SelectField /> nor <Badge /> nor <LeftNav />

@chrismcv Thanks mate! Better than the issue I would have cooked up today ;)

100%
I agree, one of the problems with the current TextField that we all want to address (and we will revisit after 0.15 is that it feels as though you're bringing the entire kitchen sink with you for a simple input.
How do you do that scoping without attaching components as a static property on other components?

About the SelectField, I don't really understand the difference between SelectField and DropDownMenu in terms of "features" ?
Also, AutoComplete component is like a SelectField with auto-select feature... I've missed something ? Isn't it possible to make it an option/props of SelectField ?

We have done our best to improve the nomenclature on the v1-alpha branch. Most of the time, we try to the Material Design Specification nomenclature. If you find anything disturbing on that branch, please raise your voice before it's too late with the official release.


fix(build): do not halt the build on missing rules repository

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

What this PR does / why we need it:
Not all rulesfile that have a rules_url have an OCI repository for rules that is called <plugin_name>-rules. In that case, do not halt the build process but proceed and emit a warning instead.
I believe the proper fix for this would be to remove this job entirely and integrate it with release but we have more cleanups to do before we can do that. In the meantime, this should unblock the current release.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: LucaGuerra
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr, LucaGuerra
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: e5ea3657f554e68c83524aa6cfb8e65c61c55a66


[FEATURE][DocBot] Add RAG Search Pipeline

Is your feature request related to a problem?
We need a RAG Search Pipeline that will be used with our cohere-index.
What solution would you like?
In our cluster_bootstrap.py we need to create a search pipeline that will be used as the default for our cohere-index. This search pipeline will need to be created after our models are initiated and before the cohere-index is created.
We will need to use the new LLM_MODEL_ID from our model state as the model for this. See #62
Here is the reference for creating a RAG search pipeline in OpenSearch: https://opensearch.org/docs/latest/ml-commons-plugin/conversational-search/#setting-up-the-pipeline

Closed with #98


Red Keep - Contributing STR

Had a slight issue with the Red Keep giving +2 to POW challenges, I found that when attacking with characters which denied STR (Balon specifically) the Red Keep would simply add +2 to any participating character, rather than the controlling player's side in general.
e.g.  I am attacking with Balon alone in a POW challenge, opponent defends with Melissandra - I should win by 5-2 (assuming no other modifiers) but the STR was decided at 5-6 (Mel + Red Keep).  I'm not certain if this issue is with Balon or The Red Keep tbh.

This is just down to Balon not being implemented :)

OK cracking, wasn't certain where the issue was, should have guessed it was in my ability to check the implementation list ;)


Feature: add loader for graph execution

Summary

Add a spinner loader when the graph execution happens
Add better error and success messages
Update optimization policy select

Screenshots

https://github.com/user-attachments/assets/f2975410-f59a-4c7c-a8d7-6d6af4d77ce2

Resolves #9

Great! Will tryout ASAP.


Fixing space issue

Fixing space issue in the entity default dashboard
Relevant information
Error:
There was a problem…
NRQL Syntax Error: Error at line 1 position 362, unexpected 'name'
There was a space/syntax issue so fixed it.
Before:
FROM Metric SELECT latest(kentik.snmp.IfInUtilization) AS 'RX %', latest(kentik.snmp.IfOutUtilization) AS 'TX %', latest(if_OperStatus) AS 'Operational Status', latest(kentik.snmp.ifInErrorPercent) AS 'RX Error %', latest(kentik.snmp.ifOutErrorPercent) AS 'TX Error %' FACET if_interface_name AS 'Interface', if_Speed AS 'Interface Speed' WHERE instrumentation name = 'cisco-asa' LIMIT MAX
After:
FROM Metric SELECT latest(kentik.snmp.IfInUtilization) AS 'RX %', latest(kentik.snmp.IfOutUtilization) AS 'TX %', latest(if_OperStatus) AS 'Operational Status', latest(kentik.snmp.ifInErrorPercent) AS 'RX Error %', latest(kentik.snmp.ifOutErrorPercent) AS 'TX Error %' FACET if_interface_name AS 'Interface', if_Speed AS 'Interface Speed' WHERE instrumentation.name = 'cisco-asa' LIMIT MAX

Checklist

[ Yes] I've read the guidelines and understand the acceptance criteria.
[ ] The value of the attribute marked as identifier will be unique and valid.
[ ] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

 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.


Testing: Use pytest

Tests appear to be written as prints as part of guarded main functions. Switch to something like pytest with travis CI auto-testing.

Repository is now using pytest.


Add Python module to requirements.txt

ffmpeg==3.2.4

Hi!
I had troubles with the script 'process_all.py' without knowing what the issue was. When I looked into the 'Dockerfile' I saw that I missed the ffmpeg module. Maybe you want to add this to the 'requirements.txt' file for people who do not use Docker?
Cheers,
Urs

I don't think that you can install ffmpeg 3.2.4 using pip. However, it should definitely be listed as a requirement in the readme so I've added it there (https://github.com/anibali/h36m-fetch/commit/383c24d1651920f9d213fa15b83dcbe07641aa77).

You can pip install ffmpeg. But yes, I also used conda install --no-update-deps -y -c conda-forge ffmpeg=3.2.4.

I think this is done by your comment in the readme.


Fedora - Install from Repository incomplete

File: install/linux/docker-ce/fedora.md
The steps to install docker from repository on fedora f28 seam to be incomplete I tried to help someone to install docker on her notebook, but she was using fedora and the official steps within the docs did not yield success.
systemd status docker.service saied running as did containerd.service
But when trying to docker run hello-world it throws:
Error response from daemon: OCI runtime create failed: container_linux.go:345: starting container process caused "process_linux.go:430: container init caused \"write /proc/self/attr/keycreate: permission denied\": unknown.

The user joined the docker group and also rebooted the system after installing docker.
Steps to reproduce on fedora 28:
su
dnf -y install dnf-plugins-core
dnf config-manager \
    --add-repo \
    https://download.docker.com/linux/fedora/docker-ce.repo
dnf install docker-ce docker-ce-cli containerd.io
systemctl start docker
systemctl enable docker
docker run hello-world

Maybe this link can help you: https://stackoverflow.com/questions/56870478/cannot-start-docker-container-in-docker-ce-on-oracle-linux

I found it. after disable selinux the problem solved. selinux prevent docker to write on /proc/self/attr/keycreat
For disable selinux:
Edit the /etc/selinux/config file to set the SELINUX parameter to disabled, and then reboot the server

Also witnessing this on RHEL 7. I've been using SELinux successfully - currently 18.09.9-ce works but 19.03.6-ce fails with the error mentioned in the OP.


MIC-384 add hibernate envers for auditing

Codecov Report
Attention: Patch coverage is 95.53571% with 5 lines in your changes missing coverage. Please review.

Project coverage is 89.31%. Comparing base (a9f1588) to head (5622765).

Files
Patch %
Lines

...ventionplanapi/entity/audit/AuditRevisionEntity.kt
86.84%
4 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #100      +/-   ##
============================================
- Coverage     89.43%   89.31%   -0.12%
  Complexity      696      696
============================================
  Files           117      118       +1
  Lines          2129     2181      +52
  Branches        113      124      +11
============================================
+ Hits           1904     1948      +44
- Misses          178      185       +7
- Partials         47       48       +1

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