Create dummy functions gdev 569

Title for squash and merge:
Add dummy commands for up- and download

Description for squash and merge:
- add dummy commands with complete syntax
- check if files/directories actually exist on the filesystem
- check if the api_url is actually reachable

No idea, what the github action problem is. Works fine on container rebuilds locally...

Codecov Report

Merging #2 (0d8d7de) into dev (ab9b470) will decrease coverage by 11.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev      #2       +/-   ##
==========================================
- Coverage   15.38%   4.34%   -11.04%
==========================================
  Files           6       6
  Lines          13      46       +33
==========================================
  Hits            2       2
- Misses         11      44       +33

Impacted Files
Coverage Δ

ghga_connector/__main__.py
0.00% <0.00%> (ø)

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 ab9b470...0d8d7de. Read the comment docs.

Codecov Report

Merging #2 (0d8d7de) into dev (ab9b470) will decrease coverage by 11.03%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev      #2       +/-   ##
==========================================
- Coverage   15.38%   4.34%   -11.04%
==========================================
  Files           6       6
  Lines          13      46       +33
==========================================
  Hits            2       2
- Misses         11      44       +33

Impacted Files
Coverage Δ

ghga_connector/__main__.py
0.00% <0.00%> (ø)

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 ab9b470...0d8d7de. Read the comment docs.

Codecov Report

Merging #2 (76d234b) into dev (ab9b470) will decrease coverage by 10.73%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##              dev      #2       +/-   ##
==========================================
- Coverage   15.38%   4.65%   -10.74%
==========================================
  Files           6       5        -1
  Lines          13      43       +30
==========================================
  Hits            2       2
- Misses         11      41       +30

Impacted Files
Coverage Δ

ghga_connector/__main__.py
0.00% <0.00%> (ø)

ghga_connector/cli.py
0.00% <0.00%> (ø)

ghga_connector/core/main.py
0.00% <0.00%> (ø)

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 ab9b470...76d234b. Read the comment docs.

Codecov Report

Merging #2 (d5aa84f) into dev (ab9b470) will increase coverage by 69.93%.
The diff coverage is 85.61%.

@@             Coverage Diff             @@
##              dev       #2       +/-   ##
===========================================
+ Coverage   15.38%   85.31%   +69.93%
===========================================
  Files           6        6
  Lines          13      143      +130
===========================================
+ Hits            2      122      +120
- Misses         11       21       +10

Impacted Files
Coverage Δ

ghga_connector/__main__.py
0.00% <0.00%> (ø)

ghga_connector/cli.py
80.00% <80.00%> (ø)

ghga_connector/core/main.py
91.30% <91.30%> (ø)

ghga_connector/core/api_calls.py
92.75% <92.75%> (ø)

ghga_connector/core/__init__.py
100.00% <100.00%> (+100.00%)
:arrow_up:

Continue to review full report at Codecov.

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


Add the rollback history command line option.

Provide a --rollback-history INT and a configuration setting for the PAB Webserver (named rollbackHistory) that controls how much history we retain for rollbacks.
If you don't need to test rollbacks, then setting the rollback-history to a lower value (I tested with 500) will greatly limit the amount of RAM that the PAB will consume. A value of 0 turns off this feature.
Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

I just thought of moving the option into a new Development section as this and the one in the next PR should probably not be used in production.


aiohttp-json-rpc: init at 0.13.3

Description of changes

This adds the aiohttp-json-rpc Python package.
It's my first PR to nixpkgs, sorry if something is missing!
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

I'm confused by the formatter not passing checks, I just rebased from master and nixfmt formats almost everything in maintainers-list.nix, not just my stuff. Should I just apply the format on my own new lines?

Ah, nix-shell with no arguments would have been enough. There is a shell.nix in the root directory of nixpkgs.


python312Packages.compressai: disable flaky test

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

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

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

aarch64-linux

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

x86_64-darwin

:x: 2 packages failed to build:

python311Packages.compressai
python311Packages.compressai.dist

:white_check_mark: 2 packages built:

python312Packages.compressai
python312Packages.compressai.dist

aarch64-darwin

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

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

x86_64-linux

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

aarch64-linux

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

x86_64-darwin

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist

aarch64-darwin

:white_check_mark: 4 packages built:

python311Packages.compressai
python311Packages.compressai.dist
python312Packages.compressai
python312Packages.compressai.dist


BUG: signal.lfilter sometimes works with complex coefficients and sometimes not

Describe your issue.
lfilter sometimes works with complex coefficients and sometimes not. In one example (a version of Goertzel alogrithm), the lfilter from signal computed with complex numbers.
In a further modified version, lfilter ignored the imaginary parts of the complex coefficients.
Reproducing Code Example
from scipy.signal import lfilter
import numpy as np

x_n = np.array([1, 4, 9, 16, 25])

# working example
# goertzel

k = 1
N = len(xn)

b = np.array([1, -np.exp(-2j*pi*k/N)])
b = b * np.exp(2j*pi*k/N)
a = np.array([1, -2*np.cos(2*pi*k/N), 1])

y = lfilter(b, a, xn)
y

# not working example
# sgdft

k = 1
N = len(xn)

b = np.zeros(N)
b[0] = 1
b[1] = -np.exp(2j*pi*k/N)
b[-2] = np.exp(2j*pi*k/N)
b[-1] = -1
b = b * np.exp(2j*pi*k/N)
a = np.array([1, -2*np.cos(2*pi*k/N), 1])

y = lfilter(b, a, xn)
y

Error message
<ipython-input-134-8ef5bc7772dc>:10: ComplexWarning: Casting complex values to real discards the imaginary part
  b[1] = -np.exp(2j*pi*k/N)
<ipython-input-134-8ef5bc7772dc>:11: ComplexWarning: Casting complex values to real discards the imaginary part
  b[-2] = np.exp(2j*pi*k/N)

SciPy/NumPy/Python version information
1.5.0 1.18.5 sys.version_info(major=3, minor=8, micro=3, releaselevel='final', serial=0)

Hi @hadamard-zweistein, thank you for reporting but this is not an issue with SciPy but with the code you have before even calling lfilter.
Just run this part of your example and you will get your error
b = np.zeros(N)
b[0] = 1
b[1] = -np.exp(2j*pi*k/N)

I will close this for now.

@tupui , perfect! Thank you


Rename TargetEvent Type in api side

related issue https://github.com/rancher/rancher/issues/11778

rename type to eventType for TargetEvent.Type in API side.
remove the useless Type field from TargetWorkload

Related Rancher PR rancher/rancher#12240


[BUG] Traefik v3 does not render correctly html ban page

Describe the bug 🐛
When using the provided ban.html and providing it with the option, it gets printed even through the html renders if you test it in a html renderer.
Expected behavior 👀
The html getting rendered.
Context 🔎
Configuration:
- "traefik.http.middlewares.crowdsec.plugin.bouncer.enabled=true"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.crowdseclapikey=APIKEY"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.logLevel=DEBUG"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.banHTMLFilePath=/ban.html"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.crowdsecLapiHost=crowdsec:8080"

No error in logs were found.
Version (please complete the following information):

OS: Docker
Traefik version: 3.0.0
Plugin version: 1.3.0
Redis: ?

To Reproduce
Steps to reproduce the behavior:

Put configuration.
Activate middleware.
Ban yourself.
Try to go to the service.

I suspect that this might be the change in v3 that causes this: https://doc.traefik.io/traefik/master/migration/v2-to-v3/#content-type-auto-detection

Hi @aikooo7,
I've tried to reproduce with the minimal exemple custom-ban-page updated recently to version 3 of Traefik.
make run_custombanpage

I then went to http://localhost:8000/foo/ to get my IP from whoami and banned it in crowdsec container.
after reloading /foo, I had the ban template correctly rendered.

The debug logs I have are the following:
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:22 getTLSConfigCrowdsec:CrowdsecLapiScheme https:no
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:22 cache:New initialized isRedis:false
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:22 New initialized mode:live
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:31 ServeHTTP ip:<IP_ADDRESS> isTrusted:false
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:31 cache:Get key:<IP_ADDRESS>
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:31 ServeHTTP:Get ip:<IP_ADDRESS> isBanned:false cache:miss
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:12:31 cache:Set key:<IP_ADDRESS> value:f duration:60s
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 ServeHTTP ip:<IP_ADDRESS> isTrusted:false
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 cache:Get key:<IP_ADDRESS>
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 ServeHTTP:Get ip:<IP_ADDRESS> isBanned:false cache:miss
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 cache:Set key:<IP_ADDRESS> value:t duration:60s
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 ServeHTTP:handleNoStreamCache ip:<IP_ADDRESS> isBanned:t handleNoStreamCache:banned
DEBUG: CrowdsecBouncerTraefikPlugin: 2024/05/06 16:13:50 handleRemediationServeHTTP ip:<IP_ADDRESS> remediation:t

Could you please try to test the minimal exemple custom-ban-page in the repository and see if it's working for you.
If not, I would need more information (a docker-compose and every debug logs from the plugin) to try and reproduce.

Hey @mathieuHa, I appreciate your help a lot but the problem with the demo is that it conflicts with my configuration, even when I change the name on the docker compose it just turns out to be a mess.
Any other way I could fix this?

Hey @mathieuHa, I appreciate your help a lot but the problem with the demo is that it conflicts with my configuration, even when I change the name on the docker compose it just turns out to be a mess.
Any other way I could fix this?
Here is a screenshot of my problem in case it helps:

Hey @mathieuHa, I appreciate your help a lot but the problem with the demo is that it conflicts with my configuration, even when I change the name on the docker compose it just turns out to be a mess.
Any other way I could fix this?
Here is a screenshot of my problem in case it helps:

Wanted to add on that I am experiencing this problem as well, though the demo works correctly and renders the ban.html page correctly, on my own installation, it does not and instead renders the  html as text, inspecting the page seems to shows that the html page got wrapped in between "

" tags:

Still investigating this problem though I am also using Cloudflare in front as well which could be another potential reason.
Using Traefik V3 and V1.3 Plugin.

Still investigating this problem though I am also using Cloudflare in front as well which could have other potential stuff causing an issue, but i do have all the Auto-Minify and Rocket Loader off.

Apparently cloudflare is not the problem since I am not using it.

Hey all,
@benscobie: I don't think, it's related because in the code we have:
func handleBanServeHTTP(bouncer *Bouncer, rw http.ResponseWriter) {
	rw.WriteHeader(http.StatusForbidden)
	if bouncer.banTemplateString != "" {
		rw.Header().Set("Content-Type", "text/html; charset=utf-8")
		fmt.Fprint(rw, bouncer.banTemplateString)
	}
}

We set the content type if the BanHTMLFilePath is set.
@xKhronoz: Could you share your docker-compose config, and debug logs from the launch of Traefik with our plugin.
Maybe try to put our plugin in the last middleware execution list.

Hey all,
@benscobie: I don't think, it's related because in the code we have:
func handleBanServeHTTP(bouncer *Bouncer, rw http.ResponseWriter) {
	rw.WriteHeader(http.StatusForbidden)
	if bouncer.banTemplateString != "" {
		rw.Header().Set("Content-Type", "text/html; charset=utf-8")
		fmt.Fprint(rw, bouncer.banTemplateString)
	}
}

If I understand correctly and you mean it is related to cloudflare then why would I get it?

I might have found the root cause of the issue, at least with regards to the way I setup Traefik, Crowdsec, and my other service, as I separate my services into their own docker-compose files, with the use of the 'Dynamic' Configuration File to setup a few common Middlewares that are shared across my services and the use of 'Chain' Middleware to combine these middleware into one, including this plugin, and as mentioned by @benscobie indeed that with Traefik V3, automatic content type auto detection has to be set to true in the config in order for it to be working, although I am not 100% sure if the problem happens due to there being more than 1 middleware in used for a service and somehow content type auto detection is not working or smth, but adding in the auto content type detection finally got the html page to be rendered correctly.
https://doc.traefik.io/traefik/middlewares/http/contenttype/

Hey all,
@benscobie: I don't think, it's related because in the code we have:
func handleBanServeHTTP(bouncer *Bouncer, rw http.ResponseWriter) {
	rw.WriteHeader(http.StatusForbidden)
	if bouncer.banTemplateString != "" {
		rw.Header().Set("Content-Type", "text/html; charset=utf-8")
		fmt.Fprint(rw, bouncer.banTemplateString)
	}
}

We set the content type if the BanHTMLFilePath is set.
@xKhronoz: Could you share your docker-compose config, and debug logs from the launch of Traefik with our plugin. Maybe try to put our plugin in the last middleware execution list.

@mathieuHa yes before i added the auto content type detection, I had tried to move the plugin to the last in my middleware chain but it did not work as well, however adding the auto content type detection middleware to before the plugin middleware as well did the trick.
Middleware Chain Screenshot

My sanitized dynamic config.yaml for Traefik
# Dynamic Config File

# TLS Config
tls:
  options:
    default:
      minVersion: VersionTLS12
      sniStrict: true
      cipherSuites:
        # TLS 1.3
        - "TLS_AES_256_GCM_SHA384"
        - "TLS_AES_128_GCM_SHA256"
        - "TLS_CHACHA20_POLY1305_SHA256"

        # TLS 1.2
        - "TLS_ECDHE_ECDSA_WITH_CHACHA20_POLY1305_SHA256"
        - "TLS_ECDHE_RSA_WITH_CHACHA20_POLY1305_SHA256"
        - "TLS_ECDHE_ECDSA_WITH_AES_256_GCM_SHA384"
        - "TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384"
        - "TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256"
        - "TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256"

        # Downgrade attack prevention (RFC 7507)
        - "TLS_FALLBACK_SCSV"
      curvePreferences:
        - CurveP521
        - CurveP384
      clientAuth:
        caFiles:
          # mTLS with CF Origin CA
          - /authenticated_origin_pull_ca.pem
        clientAuthType: RequireAndVerifyClientCert
    mintls13:
      minVersion: VersionTLS13
      sniStrict: true

# HTTP Config
http:
  middlewares:
    security-headers:
      headers:
        frameDeny: true
        browserXssFilter: true
        contentTypeNosniff: true
        hostsProxyHeaders: X-Forwarded-Host
        sslProxyHeaders:
          X-Forwarded-Proto: https
        forceSTSHeader: true
        stsIncludeSubdomains: true
        stsPreload: true
        stsSeconds: 63072000 # 2 Years https://hstspreload.org/
        referrerPolicy: same-origin
        customFrameOptionsValue: SAMEORIGIN
        customRequestHeaders:
          X-Forwarded-Proto: https
        customResponseHeaders:
          X-Robots-Tag: "noindex, nofollow, nosnippet, noarchive, notranslate, noimageindex"

    autodetect-contenttype:
      contentType: {}

    rate-limiter:
      rateLimit:
        average: 100
        burst: 50

    crowdsec-bouncer:
      plugin:
        crowdsec-bouncer-traefik-plugin:
          Enabled: true
          LogLevel: INFO
          CrowdsecLapiHost: crowdsec:8080
          CrowdsecLapiKey: REDACTED
          CrowdsecMode: stream
          UpdateIntervalSeconds: 10
          UpdateMaxFailure: 5
          ForwardedHeadersTrustedIPs:
            # CF IP Range
            - <IP_ADDRESS>/22
            - <IP_ADDRESS>/22
            - <IP_ADDRESS>/22
            - <IP_ADDRESS>/13
            - <IP_ADDRESS>/14
            - <IP_ADDRESS>/18
            - <IP_ADDRESS>/22
            - <IP_ADDRESS>/18
            - <IP_ADDRESS>/15
            - <IP_ADDRESS>/13
            - <IP_ADDRESS>/20
            - <IP_ADDRESS>/20
            - <IP_ADDRESS>/20
            - <IP_ADDRESS>/22
            - <IP_ADDRESS>/17
            - 2400:cb00::/32
            - 2606:4700::/32
            - 2803:f800::/32
            - 2405:b500::/32
            - 2405:8100::/32
            - 2a06:98c0::/29
            - 2c0f:f248::/32
          ClientTrustedIPs:
            - <IP_ADDRESS>/24 # LocalHost
            - <IP_ADDRESS>/8 # Private IP Range
            - <IP_ADDRESS>/16 # Private IP Range
            - <IP_ADDRESS>/12 # Private IP Range
            - <IP_ADDRESS>/10 # Tailscale IP Range
          BanHTMLFilePath: /ban.html
          RedisCacheEnabled: true
          RedisCacheHost: crowdsec-traefik-bouncer-redis:6379
          RedisCachePassword: REDACTED
          RedisCacheDatabase: "5"

    # For External Access Services
    external-secured:
      chain:
        middlewares:
          - rate-limiter
          - autodetect-contenttype
          - security-headers
          - crowdsec-bouncer

I absolutely don't know why you get it, I just said that the contentType is not the problem there.
If you want help, give me your config to reproduce it locally.
We are totally blind right now, we have two screenshots of ban.html and 5 lines of labels. How can we help you ...

I am just as lost as you, if the xKhronoz's solution doesn't work I will share my config here

Hey, the xKhronoz's solution worked, thank you so much!
I can also confirm I had multiple middlewares. If further configuration is needed please tell me and I will try to help.

Hi,
Thanks for posting.
I will keep this issue open for now as many people will upgrade to v3 might encounter this bug.
I have to dig a bit more to see in which case  autodetect-contenttype is needed.
If you have exemples of config that had this issue, please feel free to post here.
@aikooo7 It would help to have at least what middleware you use for a service
After gathering some conf, I will try to update the documentation and the examples.

@aikooo7 It would help to have at least what middleware you use for a service
After gathering some conf, I will try to update the documentation and the examples.

Here are the middlewares I am using:
- "traefik.http.routers.vaultwarden.middlewares=security-headers,autodetect,crowdsec"

Security-headers I use everywhere, autodetect and crowdsec are for crowdsec.
Here are there definition:
- "traefik.http.middlewares.security-headers.headers.accesscontrolmaxage=100"
- "traefik.http.middlewares.security-headers.headers.addvaryheader=true"
- "traefik.http.middlewares.security-headers.headers.hostsproxyheaders=X-Forwarded-Host"
- "traefik.http.middlewares.security-headers.headers.sslredirect=true"
- "traefik.http.middlewares.security-headers.headers.sslproxyheaders.X-Forwarded-Proto:https"
- "traefik.http.middlewares.security-headers.headers.stsseconds=63072000"
- "traefik.http.middlewares.security-headers.headers.stsincludesubdomains=true"
- "traefik.http.middlewares.security-headers.headers.forcestsheader=true"
- "traefik.http.middlewares.security-headers.headers.framedeny=true"
- "traefik.http.middlewares.security-headers.headers.contenttypenosniff=true"
- "traefik.http.middlewares.security-headers.headers.browserxssfilter=true"
- "traefik.http.middlewares.security-headers.headers.referrerpolicy=same-origin"
- "traefik.http.middlewares.security-headers.headers.featurepolicy=camera 'none'; geolocation 'none'; microphone 'none'; payment 'none'; usb 'none'; vr 'none'"
- "traefik.http.middlewares.security-headers.headers.customresponseheaders.X-Robots-Tag=none,noarchive,nosnippet,notranslate,noimageindex"
- "traefik.http.middlewares.autodetect.contenttype=true"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.enabled=true"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.crowdseclapikey=APIKEY"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.banHTMLFilePath=/ban.html"
- "traefik.http.middlewares.crowdsec.plugin.bouncer.crowdsecLapiHost=crowdsec:8080"

Release v1.3.1 is here, with a fix


Drop TS fields from model

CHANGELOG will be updated in a later branch (to avoid a travis build for now).


Decide which transformer_class functions are needed in an installable library

The installable library which was recently created contains a transformer_class.py file derived from base-transformer-class. This file contains functions that are different from those in the transformer_class.py file within opendronemap. The goal is to see which functions are needed.
Functions in Question
In transformer-opendronemap:

supported_file_exts
get_acceptable_files

In agpypeline:

convert_and_clean_tag
default_epsg
get_image_file_epsg
get_image_file_geobounds
generate_transformer_md

The open drone map functions in question should either be added to the library and/or the library changed to make those calls when needed.

@jvanderleeuw Can you look over this ticket and see what's still relevant and update this issue?

Both the transformer-opendronemap and the agpypeline libraries were changed when the scif workflow was implemented and variable/function names were changed
supported_file_exts became KNOWN_IMAGE_FILE_EXTS
get_acceptable_files became check_for_image_file
convert_and_clean_tag stayed
default_epsg became a parameter
get_image_file_epsg became get_epsg
get_image_file_geobounds became image_get_geobounds
generate_transformer_md stayed


Unitialized basin_id

Forcing.h gets repeated warnings on build because one of the constructors sets basin_id(basin_id), but, since no basin_id is passed, the constructor just sets the basin_id as the value it already was (which wasn't anything).
Would anyone be opposed to me setting that as 0 to ensure that it's at least getting the default value or just taking it out of the signature?

@christophertubbs, if long-term it doesn't make sense to allow default basin ids values (i.e., we eventually will need to enforce their explicit inclusion when calling constructors), I lean toward leaving it as-is to remind us there is something there that needs fixing.  I wouldn't argue too much though if that got changed regardless.
Given that, I'm fine with either adding a default argument value or an additional constructor, such that this gets initialized to 0.

A quick look shows that basin_id isn't even used anywhere by the forcing object.  Essentially, the identity is connected to the input file passed in to be read, and the forcing object is attached to the realization object.  There may come a point where we need to pass a larger forcing data set and have a forcing object pull out just the relevant information for a given identity, but if that information doesn't exist, that should be an error condition.

Also, see the TODO here: https://github.com/NOAA-OWP/ngen/blob/ce23388d96be1922d04b75b8201a9e7ca2a93427/src/core/catchment/HY_CatchmentRealization.cpp#L8
I think we should probably pass the appropriate catchment id (and rename basin_id to catchment_id to be consistent with its use across the realizations)

Yes, this is a leftover from the initial prototype. I will submit a PR now with the change to catchment_id.
I will add the 0 as default for now and put a todo for an explicit inclusion as Bobby says.


Request: Add Support for Spotify OAuth

Would love to see Spotify's OAuth sequences included... if for no other reason, to sanity check my work.  Here's a link to their docs:
https://developer.spotify.com/web-api/authorization-guide/

+1

@sahat : I have no idea who you are, but you're awesome.  :)

+1,000!  This would REALLY help me.  I created an account on Github entirely in hopes of using Satellizer for Spotify.  I would even make a donation if it would help provide Spotify authentication within a few days.

@sahat : Any update on this?

Would also love this! +1

@sahat If you haven't begun and are busy, I can take a look at this.

@justinzelinsky Thanks, I would appreciate that!

Spotify currently does not allow for users to revoke access to applications authenticated with OAuth2 (see this thread). I'm not sure if this is something we want to include in the project seeing this flaw with how Spotify implemented OAuth2.
@sahat thoughts?

@justinzelinsky I wasn't aware of that oversight on their part. However I think that's fine. Someone from Spotify did reply in that thread 2 days ago, so hopefully they will address that issue in the near future.

Just submitted my pull request here. Fingers crossed :)

hi, great work @justinzelinsky. Any chance we could get an ionic example? I'm still a little fresh to Satellizer, not sure what I'm doing wrong but getting a load error. This is what I've got in my .config
$authProvider.spotify({             clientId: SPOTIFY_CLIENT_ID,             url: HOST + 'auth/spotify',             authorizationEndpoint: 'https://accounts.spotify.com/authorize',             redirectUri: 'http://localhost:3000',             optionalUrlParams: ['state'],             requiredUrlParams: ['scope'],             scope: [SPOTIFY_SCOPE],             scopePrefix: '',             scopeDelimiter: ',',             oauthType: '2.0',             popupOptions: { width: 500, height: 530 }         });

Hi @shi11,
I'm not familar with Ionic but if you look in the example code here, you'll see how they configure other services (e.g. Google, Facebook, Twitter). If you wanted to add Spotify, you should be able to add the following code:
 $authProvider.spotify({       clientId: 'Your Spotify Client ID'   });
and then just make sure you call the Spotify authenticate in your UI.
@sahat Am I missing anything?

That looks right, Justin.
On Fri, Jul 29, 2016 at 7:28 AM, Justin Zelinsky<EMAIL_ADDRESS>wrote:

Hi @shi11 https://github.com/shi11,
I'm not familar with Ionic but if you look in the example code here
https://github.com/sahat/satellizer/blob/master/examples/ionic/www/js/app.js,
you'll see how they configure other services (e.g. Google, Facebook,
Twitter). If you wanted to add Spotify, you should be able to add the
following code:
$authProvider.spotify({
clientId: 'Your Spotify Client ID'
});
and then just make sure you call the Spotify authenticate in your UI.
@sahat https://github.com/sahat Am I missing anything?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sahat/satellizer/issues/759#issuecomment-236195305,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAhQusTCbRQGO2Ji9slljdtmugSM4CU_ks5qag4pgaJpZM4Hs4RV
.

Thanks Sahat. So I'm following the Google example and I'm running into a
load error issue after I auth. I think it's related to this inappbrowser
issue with localhost: https://github.com/nraboy/ng-cordova-oauth/issues/193

seen this before? Is the a way to compare another redirecturl?

On Sat, Jul 30, 2016 at 2:52 AM Sahat Yalkabov<EMAIL_ADDRESS>wrote:

That looks right, Justin.
On Fri, Jul 29, 2016 at 7:28 AM, Justin Zelinsky<EMAIL_ADDRESS>
wrote:

Hi @shi11 https://github.com/shi11,
I'm not familar with Ionic but if you look in the example code here
<
https://github.com/sahat/satellizer/blob/master/examples/ionic/www/js/app.js
,
you'll see how they configure other services (e.g. Google, Facebook,
Twitter). If you wanted to add Spotify, you should be able to add the
following code:
$authProvider.spotify({
clientId: 'Your Spotify Client ID'
});
and then just make sure you call the Spotify authenticate in your UI.
@sahat https://github.com/sahat Am I missing anything?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sahat/satellizer/issues/759#issuecomment-236195305,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AAhQusTCbRQGO2Ji9slljdtmugSM4CU_ks5qag4pgaJpZM4Hs4RV
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sahat/satellizer/issues/759#issuecomment-236348625,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJVQ3mE2tW2U710dRgVBP3-FSk91Tgdks5qavSYgaJpZM4Hs4RV
.

@sahat : is spotify now included ??

@gdeward Yes it is.


entc/gen: runtime/runtime.go generates wrong UUID import depending on module cache state

This is basically a rehash of #773 but with more detail and a minimal repro repo attached.
Setup:

Make a minimal module using ent
Make a schema that has a UUID field using github.com/google/uuid
Add a hook to the schema

Works:

Clean your module cache: go clean -modcache
Run ent code generation
Build your project

Fails:

Seed your module cache with github.com/gofrs/uuid:

E.g., in a separate shell, cd $(mktemp -d) && go mod init test && go get github.com/gofrs/uuid
Note that this is not changing the module under test at all!

Re-run code generation & build

Demo repo ready for the above: https://github.com/mgabeler-lee-6rs/ent-uuid-demo
Build error:
ent/runtime/runtime.go:22:20: cannot use exampleDescID.Default.(func() "github.com/gofrs/uuid".UUID) (type func() "github.com/gofrs/uuid".UUID) as type func() "github.com/google/uuid".UUID in assignment

In practice this is biting me extra because my project uses https://github.com/jackc/pgx which internally uses the problem UUID library, so even cleaning my module cache constantly won't actually work around the issue for me. For the moment I'm having to resort to a go:generate step that patches up runtime.go :frowning_face:
Using pgx isn't necessary to repro this, however. Just having some project on your machine that uses gofrs/uuid so that it ends up in the module cache is enough to cause this failure.
It sort of looks like the template for the generation of ent/runtime/runtime.go is trying to copy the imports from the schema file, but I may be misreading it, or it may simply not be working. VSCode's editor undo state sometimes captures some intermediate state of the generated runtime.go file after initial generation but before goimports is run, and it doesn't show any uuid import then.

Hey @mgabeler-lee-6rs and thanks for reporting this issue.
What version do you use for ent? It should be fixed in the latest version.
Can you please update to latest master and report if you still get this
go get github.com/facebook/ent@master

I'm using the latest release -- v0.4.3 -- but the problem reproduces exactly the same on master. I've updated my repro repo to use latest master so you can see for yourself. Please take a look (the repo is miniscule) and try the steps out for yourself to see.

Thanks for reporting this issue with a proper reproducible example. It helped me tackling this quickly.
Fixed via #901.

Thank you much for the quick fix :heart:
Confirmed it fixes my "real" app in addition to the repro demo :smiley:


[SPARK-9907][SQL] Python crc32 is mistakenly calling md5

Merged build triggered.

Merged build started.

LGTM, thanks for fixing this.

Test build #40670 has started for   PR 8138 at commit 462fa38.

Test build #40670 has finished for   PR 8138 at commit 462fa38.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
   * Set thresholds in multiclass (or binary) classification to adjust the probability of

Merged build finished. Test PASSed.


octave: 5.1.0 -> 5.2.0

See https://lists.gnu.org/archive/html/info-gnu/2020-02/msg00007.html
for release information.

Motivation for this change
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

https://github.com/NixOS/nixpkgs/blob/5e7ad66e1f462f807c8736ff2842ddd6d8e1e37d/pkgs/development/interpreters/octave/default.nix#L80
$ curl http://octave.org
<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">

<title>301 Moved Permanently</title>

Moved Permanently

The document has moved here.

So, please change to:
    homepage = "https://www.gnu.org/software/octave";

@sikmir done !
Thanks.


Equivalent to the Crossbeams select! macro?

Hi,
How would a user replace the select! macro of Crossbeam when using this library?

Right now, there are futures::select! and tokio::select! but I'm not a big fan of either of those macros. I might come up with something in the future...

Thanks, @stjepang!
So  futures::select! would be the only option if I want to create a runtime agnostic library.
Just out of curiosity: What do you not like about the futures::select! macro?
I'm not sure if you want to keep this issue open. Feel free to close it, if you like.

@d4h0 futures::select! is implemented using procedural macros (slow compile times), sometimes gives poor errors without showing the exact error location, often requires you to bump recursion_limit, and requires you to call .fuse() when it could fuse futures on its own. I also don't think complete => is a great idiom.

Thanks for the information, @stjepang.

Maybe it would be a good idea to mention futures::select! in the documentation?
I'm probably not the only async beginner who will be wondering what the equivalent to the Crossbeams select! macro is.

Nowadays you can use an or chain to emulate the select! macro.


⚠ Bofrost.fr has degraded performance

In 7044835, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1621 ms

Resolved: Bofrost.fr performance has improved in 6bd97d1 after 46 minutes.


Updating SQLite and MySQL Connector

Would there be any issues with simply updating the SQLite and MySQL connector? I would do it myself but I feel like this project is so big that maybe you might know more about any issues that may arise.

Just update the connector plugins?

Do you know if I need to the .net 4.6 or 4.7 or 6.0 or 7.0 or netcoreapp or netstandard 2.0 or 2.1?


[DNM] Use relative references for repeats only.

This introduces relative referencing for ${variables} in repeat sections.
Not sure if my interpretation is accurate, I found a super long old thread in ODK forum, not sure if it is relevant still.
I picked the option of dealing with https://opendatakit.github.io/xforms-spec/#a-big-deviation-with-xforms hoping it does address it.
Fix #4

Thank you guys for the comments and tests, I will add them in the next iteration on this PR.

Example of XForm XML with relative paths as generated by this PR. The first two are the examples @MartijnR provided above.

Choice filter with relative paths
Calculate with relative paths
For loop with relative paths
A more complex repeat structure with relative paths XML.

If you have the chance to test the above XML in ODK Collect or Enketo, let me know what's not working as expected.

Everything works as expected in Collect other than the choice filters. 🎉 I think we will have a fix for JavaRosa soon. I /think/ this is actually ok to merge though I'd like a sanity check on my thinking. The JavaRosa problem is that current() doesn't refer to the right thing in selects. As far as I can tell, this only affects two things: choice filters and the itemset nodeset. It's not very likely users are using relative paths to set the itemset nodeset. One would use that in a scenario like collecting household info in which the enumerator is asked to select the owner of each pet from a dynamically generated list of children:
repeat people
  repeat children
  repeat pets
    select itemset nodeset=current()../../children

It's cool and powerful but there's no XLSForm support for itemsets from arbitrary nodes (https://github.com/XLSForm/pyxform/issues/38) and it's pretty esoteric. I've only shown users how to do it with absolute paths but have never seen it come up with relative paths before.
The other case is choice filters. XLSForms with choice filters in repeats fail today because an absolute path doesn't make sense there. Currently in Collect we get an error about needing indexed-repeat. With this change we'd get no error but also no choices shown. Maybe a little harder for users to troubleshoot but nothing too bad. And as soon as they got patched Collect it would Just Work™.
The change we're going to make in JR would only affect users who are trying to use current() already. This PR doesn't affect them (maybe a good test to add @ukanga? Demonstrate that if a path is already relative this doesn't change anything?).
If someone else who uses Collect and understands the JR issue can run through this reasoning and validate it, I'd be comfortable merging.

You could decide to strip the current()/ part if the expression is not inside a predicate, e.g. calculate="../crop" instead of calculate="current()/../crop"

I would be in favor of this. current() is a bit of a strange bird so the more it can be avoided the better.

Thanks. I can confirm the forms work in Enketo too.  (after correcting calculation in calculate_w_relative_paths.xml to  calculate="current()/../crop"  calculate="current()/../../crop ", but that's not a pyxform issue.)
I agree best to avoid using current() when it's not required.

Hi! Just wanted to drop by and say that @lognaturel's reasoning looks good to me. I also agree that I would prefer having ../ instead of current() wherever possible.

@MartijnR @lognaturel not sure what was pending on this but I see most of what was on your comments was addressed. Any blocker to getting this merged?

Thanks! Once current() is back in choice-filters, I'll run the test forms in Enketo for a final/proper check.

@MartijnR 3c31570829e08a237656087966347bdcd21f8abd should address it.

I got some XForms consulting time with @lognaturel and this PR looks like what she wanted. That is pyxform generates relative paths with . and .. everywhere except in predicates like choice filters. In those cases, current() should be used. My only minor concern is that are there any other places besides choice filters where we could use predicates?

I confirm that this works great! Thanks!
I noticed that the calculations in Test 2 that I provided (name = ${crop}) make no sense (will always result in false), but that doesn't matter as the test still serves its purpose and the expression is valid.


Better, more consistent option name.

Kept the historical one for backward compatibility.

Oops, thanks for the follow up fix!


WIP React UI: rules filter url param

Fixes: https://github.com/prometheus/prometheus/issues/6387

API part looks good now (without having tested it yet), thanks!

@juliusv Thank's for the review. I think all the comments are addressed now but let me know if something left.

:+1: Looks great now, thanks :)


Checkbox label bug when the global scale != 1

Hi, I think I found a weird but minor bug. And given i just read you were about to push another official update, you might want to look it up beforehand.
Simply put, if you make a checkbox, when the UI scale is something different than 1, the label part will move right per a few pixels each frame until it gets out of the screen. The checkbox bitmap itself stays in place and is still functional. Changing back the UI scale to 1.0 will reset the label at its correct position and scrolling will stop.
I'm sorta assuming it's a rounding error or a = that become a += somewhere in your scaling stuff. As far as i know it's the only component doing that.
A code as simple as this is enough to trigger it (same thing happen if the checkbox is contained in a panel)
{
    var newButton = new CheckBox("Build");
    UserInterface.Active.AddEntity(newButton);
}

Weirdly enough, I have an instance of a complex settings menu with sub panels, and plenty of anchors where it's not the case. If it helps i can post the code.
Cheers! (sorry for always bringing weird bug reports)
edit: I'm on the main branch. just close if it's something you fixed in the dev one.

Hi @SerialKicked thank you for the issue report!
I think I fixed it in dev but I didn't reproduce exactly what you described, so if you could post a minimal reproduce code it would be helpful.
Or as alternative you can fetch dev and see if its fixed.
Thanks! :)

Just to let you know that the latest version got the bug fixed.
Sorry i didn't post earlier, I'm a bit overworked these days.
Thank you.

No problem mate, I felt confident enough to close the ticket knowing you'd reopen it if it didn't work :)
Thanks again for the useful bug report(s)!


pythonPackages.psutil: 5.6.7 -> 5.7.0

Motivation for this change
noticed it was out of date
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

this needs testing on Hydra unless someone does all the rebuilds.

There are tests available in Github. Shouldn't they be ran? :)

true

a lot of the tests were platform specific, or shelled out to a multitude of other utilities.

@GrahamcOfBorg build python3Packages.psutil pythonPackages.psutil python38Packages.psutil

@GrahamcOfBorg build python3Packages.psutil pythonPackages.psutil python38Packages.psutil


[DS-6655][fix] remove zkmonitor

remove zkmonitor

Codecov Report

Merging #6789 (0665b18) into dev (088a5a4) will decrease coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Current head 0665b18 differs from pull request most recent head 913025a. Consider uploading reports for the commit 913025a to get more accurate results

@@             Coverage Diff              @@
##                dev    #6789      +/-   ##
============================================
- Coverage     41.86%   41.81%   -0.05%
+ Complexity     3619     3616       -3
============================================
  Files           641      641
  Lines         25897    25897
  Branches       2795     2795
============================================
- Hits          10841    10830      -11
- Misses        14073    14088      +15
+ Partials        983      979       -4

Impacted Files
Coverage Δ

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
47.27% <0.00%> (-7.28%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

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 088a5a4...913025a. Read the comment docs.

Please also remove the backend API

ok,  I have removed all backend API


Install Issue

Go version: Go1.14
command: go get github.com/koding/kite
OS: macOS Catalina v.10.15.5
error:
pkg/mod/github.com/koding/kite@v0.0.0-20180710021347-baa1a54919e3/kite.go:177:21: not enough arguments in call to uuid.Must
	have (uuid.UUID)
	want (uuid.UUID, error)

How did you solve it?

@LBiff the same here

does this project still maintainable?

@LBiff the same here

I don't know how to slove that, and have tried "Convert project with 'Go module'", still doesn't work.

@LBiff the same here

I don't know how to slove that, and have tried "Convert project with 'Go module'", still doesn't work.

We've made a GoModule and we updated the dependency
(https://github.com/satori/go.uuid/issues/70)
And than everything worked fine :)

@LBiff the same here

I don't know how to slove that, and have tried "Convert project with 'Go module'", still doesn't work.

We've made a GoModule and we updated the dependency
(satori/go.uuid#70)
And than everything worked fine :)

Thanks, get you.


Clicking outside of inventory for inventory mode doesn't change the pages.

Title.

Are you in creative mode?

Ok, going in survival fixed it but how come the plugin does not autochange
textures anymore? It did it the other day but now everything is back to
vanilla.
On Tue, Jul 20, 2021 at 11:54 AM Nathan Wolf @.***>
wrote:

Are you in creative mode?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/elBukkit/MagicPlugin/issues/934#issuecomment-883504575,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQDLXZNSBDCFTRUHQCWXYULTYWL3FANCNFSM5AWBU34Q
.


Don't see any entity framework 7.0-* packages in nuget.org

Was there never been an upload of EF packages in nuget.org? Or the default feed api.nuget.org?
I don't see any version of 7.0.0-* packages (I included prerelease versions) in the manage nuget packages dialog (and I verified nuget.org and the packages are there)

You can find the latest EF7 nuget packages on myget: https://www.myget.org/F/aspnetvnext/.  You'll need to add this url to the list of package sources in Visual Studio:
Tools --> Options --> Nuget Package Manager --> Package Sources
I assume that the EF team will post the nuget packages to nuget.org once EF7 is released.

@PradeepKadubandi 7.0.0-* prerelease "EntityFramework" packages exist but are unlisted in nuget.org to prevent unintended upgrades from EF6 until we get a better solution for that in place. You should be able to see 7.0.0-* prerelease packages for "EntityFramework.Core" and "EntityFramework.SqlServer".


Test 841 requires memory under 2^32?

On SBSA-ref there is no memory below 2^40 address.
Test 841 wants memory in 32-bit address space:
 841 : NP type-1 PCIe supp 32-bit only                START

       BDF - 0x300
       BDF - 0x400
       BDF - 0x500
       BDF - 0x600
       BDF - 0x700
       BDF - 0x800
       BDF - 0x10000
       BDF - 0x20000
       BDF - 0x40000
       Skipping Non Type-1 headers
       BDF - 0x50000
       Skipping Non Type-1 headers
       BDF - 0x60000
       Skipping Non Type-1 headers
       BDF - 0x80000
       NP type-1 pcie is not 32-bit mem type
       Failed on PE -    0
       PCI_MM_04
       Checkpoint --  1                           : Result:  FAIL
         END

Doc says:

Systems compliant to this specification must support 32-bit programming of NP BARs on such
endpoints. This can be achieved in two ways:
Method 1: PE physical address space can be reserved below 4GB, while maintaining a one-to-one mapping between PE physical address space and NP memory address space.
Method 2: It is also possible to use a fixed offset translation scheme that creates a fixed offset in direction between PE physical address space, and PCI memory. This allows a window in PE physical address space that is above 4G to be mirrored in PCI memory space below 4G. This requires support in the PHB. Furthermore, firmware must program the PHB with the fixed offset, and to supply this information to the OS.

Looks like we would need to go with method 2 then?

Hi @hrw,
We are checking the query and provide update soon.
Thanks,
ACS team

Hello @hrw,
Can you help us with below data to debug the issue.

Qemu model and Qemu command
dump of  "pci 08 00 00 -i" command on uefi shell

Thanks,
ACS team

Log with:

exact QEMU (HEAD) command line
UEFI "pci" commmand
UEFI "pci 00 08 00 -i" command

log-pci.txt
I use QEMU master HEAD with UEFI built also with HEAD of edk2 and edk2-platforms repos. Can attach firmware image if needed.

Hello @hrw,
The failure was seen for  0x80000 device ( Bus 8 Dev 0 Func 0)
BDF - 0x80000
NP type-1 pcie is not 32-bit mem type
Failed on PE -    0
But the pci dump command list no such device, the only device at bus 8  is
.00   08   09    00 ==> .Multimedia Device - Audio device
Vendor 1274 Device 5000 Prog Interface 0
In case if same setup of QEMU was used for running BSA test and sharing pci cmd output, can you please attach bsa log with below command, to check PCIe devices found by BSA suite.

Bsa.efi -t 841 -v 1

Thanks,
ACS team

Here is output of ./boot-sbsa-ref.sh --cmd="fs0:bsa.efi -t 841 -v 1" command:
log-841.txt
(boot-sbsa-ref.sh is in https://github.com/hrw/sbsa-ref-status repo)

Thanks for looking into it.

Hello @hrw,
The support of ITS and SMMU + booting qemu with number of different PCIe devices is great.  :clap
On the failure, the PCIe device on which test is failing is a "PCI Express to PCI/PCI-X Bridge"
Pci Express device capability structure:
CapID( 0):          10            NextCap Ptr( 1):    40
Cap Register( 2):             0072
Capability Version(3:0):          0x0002
Device/PortType(7:4):             PCI Express to PCI/PCI-X Bridge
It is a Type1 header device, but is requesting 64-bit NP memory.
Shell> pci 07 00 00 -i
PCI Segment 00 Bus 07 Device 00 Func 00 [EFI<PHONE_NUMBER>]
00000000: 36 1B 0E 00 07 00 B0 00-00 00 04 06 00 00 01 00  6...............
00000010: 04 00 00 81 00 00 00 00-07 08 08 00 00 00 A0 00  ................
Bits 0 -> 0 // memory address requested
Bits 21 -> 10 // 64-bit address
Bits 3 -> 0 // NP memory
As per BSA and PCIe spec,  Type1 headers supports 32-bit addresses

I For non-prefetchable (NP) memory, Type 1 headers only support 32-bit addresses. This implies that endpoints
on the other end of a PCI-to-PCI bridge only support 32-bit NP BARs.
RPCI_MM_04 Systems compliant to this specification must support 32-bit programming of NP BARs on such endpoints.
This can be achieved in two ways:

Thanks,
ACS team

I mailed QEMU devel ML:
<EMAIL_ADDRESS>Please check the answer.

Hi @hrw,
We are having an internal discussion around the PCIe prefetchable bits and type (32/64-bit), will keep you updated.
Thanks,
ACS team

I have Arm support case 03445990 opened for it as well.

Hi @hrw,
The requirement of a PCIe device requesting NP memory should use 32-bit address only (32-bit BAR) is applicable only for EP.
The test will be updated to skip the check for Type1 headers.
Thanks,
ACS team

Hi @hrw,
Further updates on top of the last one.
As RP can only support  32-bit address for NP memory living downstream of a Type-1 header, EP which are downstream to the RP for NP memory should only have 32-bit address programmed in the BAR's.
But PCIe specification doesn't mandate BAR Type Bit to be 32-bit for NP memory. It can be 64-bit but the address programmed should be 32-bit address (below 4GB).  In case system is mapping PCI memory in above 4GB range, it can use address translation offset.
RPCI_MM_04 Systems compliant to this specification must support 32-bit programming of NP BARs on such endpoints.
This can be achieved in two ways:
Method 1: PE physical address space can be reserved below 4GB, whilst maintaining a one to one mapping
between PE physical address space and NP memory address space.
Method 2: It is also possible to use a fixed offset translation scheme that creates a fixed offset indirection
between PE physical address space, and PCI memory. This allows a window in PE physical address
space that is above 4G to be mirrored in PCI memory space below 4G. This requires support in the PHB.
Furthermore, firmware must program the PHB with the fixed offset, and to supply this information to the
OS [4].
Based on above inputs, there is no FAIL case from BSA specification perspective for current test. In case NP BAR is programmed with 64-bit address, it will be PCIe enumeration software issue but not hardware issue.
We will deprecate the test case.
Thanks,
ACS team.


Compilers should have a public API

https://github.com/sbt/zinc/blob/2529bbd58e458bc7c09fa1931faa09559e584372/internal/compiler-interface/src/main/java/xsbti/compile/Compilers.java
The interface is empty. It seems the only existing way to use it is like at https://github.com/sbt/zinc/blob/9b61c802f7c110deb4263a5971c07196746051a0/zinc/src/main/scala/sbt/internal/inc/IncrementalCompilerImpl.scala#L32-L41, which is clearly indicated as hackery.
The methods javac() and scalac() should be made publicly available.

Well ... It's worse than I thought, because the case class Compilers is private[sbt], defined at https://github.com/sbt/zinc/blob/9b61c802f7c110deb4263a5971c07196746051a0/zinc/src/main/scala/sbt/internal/inc/IncrementalCompilerImpl.scala#L206
So ... This is completely inaccessible. To get to compilers.scalac.scalaInstance, I currently have to use reflection:
type CompilersAccess = {
  def scalac(): xsbti.compile.CachedCompilerProvider
}

val compilers = (compileInputs in compile).value.compilers
val scalaInstance = compilers.asInstanceOf[CompilersAccess].scalac().scalaInstance

Fixed in #128 and #132


Can't parse response

Describe The Bug:
After using oauth login method, the plugin seems to be unable to parse shark API response.
To Reproduce:
Log in using oauth and restart homebridge
Expected behavior:
See vacuum added as new accessory
Logs:
[12/30/2024, 1:37:29 PM] [SharkIQ] Error parsing JSON response for properties: DockedStatus, Operating_Mode, Power_Mode
[12/30/2024, 1:39:29 PM] [SharkIQ] Error parsing JSON response for properties: DockedStatus, Operating_Mode, Power_Mode

Plugin Config:
{
            "name": "SharkIQ",
            "oAuthCode": "TyOpiK3QWqpX5uQHuFaCp0q4sj9ndhTm5uJcG-JUZdtkY",
            "vacuums": [
                "AC000W036853513"
            ],
            "europe": false,
            "invertDockedStatus": true,
            "dockedUpdateInterval": 30000,
            "platform": "SharkIQ"
}

Screenshots:

Environment:

Plugin Version: v1.3.2
Homebridge Version: v1.8.5
Node.js Version: v22.12.0
NPM Version: 10.9.0
Operating System: Docker

@ma-armenta What model do you have?


When I compile 'com.folioreader:folioreader:0.2.5',The book opens only chapter, no content, content is blank

When I compile 'com.folioreader:folioreader:0.2.5',The book opens only chapter, no content, content is blank.How to solve

api?

difficult to debug the problem with the info you provided.. is it possible to share ebook which not showing content?


fix non-static Datasets does not allow automatic Schema prediction (fix #41)

Fix #41

Coverage decreased (-0.04%) to 92.504% when pulling 0f31ebd87e5fe1fe62e9e9607b3cbb65c9f10a8e on fix-41 into e80d2e5219e6041d36506cdd0f4f6cb080d504f9 on develop.

:+1: LGTM


Fix slow loading when the app is off

Pull Request Information
Motivation

[x] Bug fix

Description
This PR fixes slow page loading when the app is off but sendings are enabled.
Contribution Guide

[x] I have read and followed the steps listed in the Contributing Guide.

Documentation
This PR requires Documentation update?

[x] No

Thank you  @michael-rubel


Adding new tables to DB structure

Originally reported on Google Code with ID 1091
IDE: Eclipse Luna

Hi

I would like to add some new tables to Your work as ODK Aggregate :)
And I would like to integrate it with tables creator that is running every time software
is lunched, but I cant find which class is responsible for creating tables structures
and where it is invoked :)

please help

Thank You in advice

Reported by przelewelo on 2014-11-24 13:26:12

Please ask any questions such as this in the Google group.
Thanks,
Dev Team


[WIP] Release 0.16.0

I don't see #1551 mentioned, which a user stumbled across only today.

@rade I incorporated it later, our CHANGELOG generator script wasn't detecting the new squashed merges properly.

About half the items end in a full stop; the other half don't. Is there a rationale?

About 25% of the items end in a full stop; the others don't. Is there a rationale?

Items are auto-generated from PR titles so no rationale. To be honest I don't think it's very important (and it's been inconsistent in previous releases). If we really really want to fix this, we should test PR titles or be strict about them in reviews.

To be honest I don't think it's very important

it isn't.

@jml @foot @davkal Please test. I need a LGTM before releasing (which I would like to do in less than two hours from now if everything goes well).

@pidster you too please

I've tested:

The ECS guide
Scope monitoring the service when running locally
A scope probe attached to cloud.weave.works while running (2)

Found https://github.com/weaveworks/scope/issues/1598 when testing (2) but I don't have evidence of it being part of 0.16 (plus I wouldn't say it's a blocker)
Also, we should upgrade the service to support new app-side features like https://github.com/weaveworks/scope/pull/1597

LGTM

LGTM!

LGTM

I abstain.


(580) Add family accommodation

Context

Changes in this PR

Update question copy for the family accommodation page
Adds a new page to the apply journey FamilyAccommodation

Screenshots of UI changes
Before

After

Release checklist
As part of our continuous deployment strategy we must ensure that this work is
ready to be released at any point. Before merging to main we must first
confirm:
Pre merge checklist

[ ] Are any changes required to the e2e tests?
[ ] If you've added a new route, have you added a new
auditEvent? (see server/routes/apply.ts for examples)
[ ] Are there environment variables or other infrastructure configuration which needs to be included in this release?
[ ] Are there any data migrations required. Automatic or manual?
[ ] Does this rely on changes being deployed to the CAS API?

Post merge checklist
Once we've merged it will be auto-deployed to the dev environment.
Find the build-and-deploy job in CircleCI.
Should a release fail at any step, you as the author should now lead the work to
fix it as soon as possible.

 Area - Family accommodation


Understanding how to Create and Query Data in Cosmos DB

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

`This closes #3181

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Good afternoon and thank you for submitting your article for publishing. @mercie-nyakio
Your article has been entered into our queue and should be reviewed (and published pending any changes) as soon as a peer reviewer is finished with the ones in the queue before it. :rocket:
Thank you.

Could someone kindly assign me this

@mercie-nyakio
👋 Good afternoon - and welcome to the EngEd program.
There appear to be many sentences in this article that are taken verbatim from third-party sources. We can't publish any content that is not original or properly cited/referenced. There are many free online tools where you can check this (such as quetext.com, and if you're uncertain about the definitions around plagiarism, this is a good resource to reference.
We're really looking for unique/original content
Understanding how to Create and Query Data in Cosmos DB #3628.pdf
@marienjus
cc @ahmadmardeni1


Take revision into consideration when choose warm container

Description
Need to compare the revision when choose warmed container

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Scheduler
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #5233 (031b988) into master (37f1f99) will increase coverage by 36.46%.
The diff coverage is 9.09%.

@@             Coverage Diff             @@
##           master    #5233       +/-   ##
===========================================
+ Coverage   26.29%   62.76%   +36.46%
===========================================
  Files         238      231        -7
  Lines       13965    13701      -264
  Branches      576      579        +3
===========================================
+ Hits         3672     8599     +4927
+ Misses      10293     5102     -5191

Impacted Files
Coverage Δ

...whisk/connector/kafka/KafkaProducerConnector.scala
26.82% <0.00%> (-33.18%)
:arrow_down:

...apache/openwhisk/connector/lean/LeanProducer.scala
0.00% <0.00%> (ø)

.../org/apache/openwhisk/core/connector/Message.scala
61.85% <ø> (+19.58%)
:arrow_up:

.../openwhisk/core/loadBalancer/FPCPoolBalancer.scala
0.00% <0.00%> (ø)

...enwhisk/core/loadBalancer/InvokerSupervision.scala
90.27% <0.00%> (+90.27%)
:arrow_up:

...e/loadBalancer/ShardingContainerPoolBalancer.scala
78.53% <ø> (+78.53%)
:arrow_up:

...ontainerpool/v2/FunctionPullingContainerPool.scala
80.32% <ø> (+80.32%)
:arrow_up:

...nwhisk/core/invoker/ContainerMessageConsumer.scala
78.72% <ø> (+78.72%)
:arrow_up:

...he/openwhisk/core/invoker/FPCInvokerReactive.scala
0.00% <ø> (ø)

...sk/core/scheduler/container/ContainerManager.scala
0.00% <0.00%> (ø)

... and 174 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 cbdcfe5...031b988. Read the comment docs.


MutableSideEffect doesn't seem to work

Expected Behavior
MutableSideEffect works as described in the docs.
Actual Behavior
This code:
	// load default number
	get := func(ctx workflow.Context) interface{} { return defaultNumber }
	eq := func(a, b interface{}) bool { return a.(int) == b.(int) }
	var number int
	if err := workflow.MutableSideEffect(ctx, "defaultNumber", get, eq).Get(&number); err != nil {
		panic("can't decode number:" + err.Error())
	}

works at first, but if I kill and restart the worker to force replay, I get:
2022/05/09 18:16:00 ERROR Workflow panic Namespace default TaskQueue hello-world WorkerID 2450384@pinglin@ WorkflowType Workflow WorkflowID hello_world_workflowID RunID 408db9e3-7885-4ca5-be59-0bce212c08eb Attempt 1 Error can't decode number:payload item 0: unable to decode: json: cannot unmarshal string into Go value of type int StackTrace coroutine root [panic]:
github.com/temporalio/samples-go/helloworld.Workflow({0xdfce40, 0xc00043a580}, {0xc0004360f8, 0x8})
        /home/dnr/t/samples-go/helloworld/helloworld.go:21 +0x23d
reflect.Value.call({0xbe9b60?, 0xd38768?, 0x30?}, {0xcd73c3, 0x4}, {0xc00040d1d0, 0x2, 0x18?})
        /nix/store/f2n8g9nbrxaqwyd9b3grynvwhyz3qlrw-go-1.18/share/go/src/reflect/value.go:556 +0x845
reflect.Value.Call({0xbe9b60?, 0xd38768?, 0xc000407d28?}, {0xc00040d1d0, 0x2, 0x2})
        /nix/store/f2n8g9nbrxaqwyd9b3grynvwhyz3qlrw-go-1.18/share/go/src/reflect/value.go:339 +0xbf
go.temporal.io/sdk/internal.(*workflowEnvironmentInterceptor).ExecuteWorkflow(0xc00042a730?, {0xdfce40?, 0xc00043a580?}, {0xc00042e490?, 0xc0005a86c0?}, {0xc00042e4a0?, 0x1?, 0xc0001030c8?})
        /home/dnr/t/go/pkg/mod/go.temporal.io/sdk@v1.6.0/internal/workflow.go:393 +0x1e5
go.temporal.io/sdk/internal.(*workflowExecutor).Execute(0xc00043a500, {0xdfce40, 0xc00043a580}, 0x25?)
        /home/dnr/t/go/pkg/mod/go.temporal.io/sdk@v1.6.0/internal/internal_worker.go:762 +0x2c3
go.temporal.io/sdk/internal.(*syncWorkflowDefinition).Execute.func1({0xdfcfc8, 0xc00040d170})
        /home/dnr/t/go/pkg/mod/go.temporal.io/sdk@v1.6.0/internal/internal_workflow.go:494 +0xcd

Another weird thing I noticed was what was encoded in the marker:
{
  "data": {
    "payloads": [
      {
        "metadata": {"encoding": "anNvbi9wbGFpbg=="},
        "data": "ImRlZmF1bHROdW1iZXIi"
      },
      {
        "metadata": {"encoding": "anNvbi9wcm90b2J1Zg=="},
        "data": "eyJwYXlsb2FkcyI6W3sibWV0YWRhdGEiOnsiZW5jb2RpbmciOiJhbk52Ymk5d2JHRnBiZz09In0sImRhdGEiOiJNVGs9In1dfQ=="
      }
    ]
  },
  "side-effect-id": {
    "payloads": [
      {
        "metadata": {"encoding": "anNvbi9wbGFpbg=="},
        "data": "ImRlZmF1bHROdW1iZXIi"
      }
    ]
  }
}

Decoding all of that base64 to strings just to make things clear (pseudo-json):
{
  "data": {
    "payloads": [
      {
        "metadata": {"encoding": "json/plain"},
        "data": `"defaultNumber"`
      },
      {
        "metadata": {"encoding": "json/protobuf"},
        "data": `{"payloads":[{"metadata":{"encoding":"anNvbi9wbGFpbg=="},"data":"MTk="}]}`
      }
    ]
  },
  "side-effect-id": {
    "payloads": [
      {
        "metadata": {"encoding": "json/plain"},
        "data": `"defaultNumber"`
      }
    ]
  }
}

That is, the actual data is double-base64-encoded.
Steps to Reproduce the Problem
Repro here: https://github.com/dnr/samples-go/tree/mutablesideeffect/helloworld

Run the worker
Run the starter
Kill the worker
Start the worker again

Specifications

Version: tested go sdk 1.6.0 and also 1.14.0. server is a branch somewhere near 1.16.0. go 1.18
Platform: linux

@dnr - Thanks for the report! Can you give a bit more to your snippet? Specifically I am looking to see the defaultNumber variable and where it's defined.
I will attempt to replicate and fix shortly.

Ok, I have replicated this issue. It seems we have no good tests for mutable side effects :-( I have found two issues:

We do not properly unwrap the payloads from history that we wrap. It seems #151 wrapped them but didn't unwrap them on replay/history properly and there was no test to confirm it does
We are not recording a command during replay so there is a command mismatch

This is a much harder problem than it seems, and mutable side effects have never worker properly during replay.
Basically, the Go SDK maintains a counter so it can re-obtain things deterministically. This counter, which is incremented each command, is used for command IDs, timer IDs, cancellation IDs, signal IDs, upsert search attr IDs, etc.
When mutable side effect callback result is not equal to the previous one during normal run, we record a marker command which increments the counter for all other commands. But we don't record a command if it is equal. During replay however, we can never know it is or isn't equal because we are not allowed to invoke the callback. So, do we record a command or not? If we don't get it right, the command counter mismatches causing non-determinism issues.
So, given a Go workflow that does:

WFT start (counter 5)
Mutable side effect not equal - record marker command (++counter)
Start a timer with ID 7 (++counter)
WFT end
Marker recorded
Timer 7 started
WFT start (counter 7)
Mutable side effect equal - do not record marker command
Start a timer with ID 8 (++counter)
WFT end
Timer 8 started

This is how replay looks if we always record marker during replay

WFT start (counter 5)
Mutable side effect - record marker command (++counter)
Start a timer with ID 7 (++counter)
WFT end
Marker recorded
Timer 7 started
WFT start (counter 7)
Mutable side effect - record marker command (++counter) - note, we don't know equal vs not-equal here
Start a timer with ID 9 (++counter)
WFT end
Timer 8 started

That last step causes an error because timer 8 does not exist.
This is how replay looks if we never record marker during replay (what the code does today):

WFT start (counter 5)
Mutable side effect - do not record marker command
Start a timer with ID 6 (++counter)
WFT end
Marker recorded (we don't error today if we don't know about the marker)
Timer 7 started

That last step causes an error because timer 7 does not exist.
So, what do we do here? Java seems to use a "fake" marker on every replay and just skips it when checking, but Go is not that fortunate because it has a counter that affects all things after it. Right now, here is the best solution I can think of:

Track all replay markers recorded last WFT
For every replay marker not recorded in history by the next WFT start, undo everything it did, which means I have to change past-recorded counter uses which includes:

Cancellation IDs for all pending cancel-external-workflow commands
Signal IDs for all pending signal-external-workflow commands
Upsert IDs for all pending upsert-search-attribute commands
Child workflow IDs for all pending execute-child-workflow commands. This is a bit complicated, because it only applies if they didn't provide their own workflow ID. When they don't provide it, it is <parent-run-id>_<++counter>. So I have to either check that it matches that pattern or maintain some state saying whether I auto-generated the ID so I can alter the generated value
Timer IDs for all pending timer commands
Schedule activity IDs for all pending execute-activity commands

Is there a better option here I am missing? cc @Sushisource, @mfateev, @yiminc, @alexshtin

OK, I have been able to use marker lookahead here. PR incoming...


fix social icons to make them viewable

Social icons Reddit, Telegram, Twitter & Github have been resized to make them viewable

hey sorry @CryptoLover705 i made this fix myself and forgot to push the branch. Also my bad that i did not check it better, its been very busy. And again TY for following up with the fix. I'll leave this open and check if maybe you found better icons than me

Hey @BillyG83 it's all good, It was my fault for not checking the sizes of the SVG's in the first place

@CryptoLover705 resolve conflict please

Conflicts have been resolved @locothedev

i'm going to close this for now as i have added the fix. TY


Build a shared module from Swift source

This story builds on the spike work in #1 to add polish and test coverage for building a module from Swift source.
Changes:

Remove duplication between Swift executable and Swift module plugins.
Remove duplication between C++ library and Swift module plugins.

Test coverage:

[x] Something sensible happens when the project has no source.
[ ] ~Incremental build works when source files are changed, added, removed and renamed.~ Incrementalness is handled by swiftc
[x] Stale output files are removed when source files are renamed or removed.
[x] Changes to the build directory location are respected (as was done for C++ libraries).

Left over test coverage can be reviewed as part of https://github.com/gradle/gradle/pull/2800.


Hope to bring the power of ChatGPT to Jetbrains

Hope to bring the power of ChatGPT to Jetbrains.

Me too! We were planning not to prioritize this for about another month, but recently there's been some work happening here so maybe it won't be so long. I'll keep you updated!

Good to know!

Awesome stuff!!!

@pedrosimao @Houtx @16francej @jmroblesh The JetBrains extension is making a lot of progress and we'll be releasing the official version very soon. I have an alpha version here that you can install from disk - would love to get early feedback! It should be mostly up to par with the VS Code extension, barring a few details:
https://discord.com/channels/1108621136150929458/1108621136830398496/1159049720799449119


Adds Game Development Section and Fixes Misplaced Courses in Readme

fixes #644

I have added two courses in newly created section "Game Development"
Checked whether added links are correctly working or not
Added a suitable gamepad icon from fontawesome.com website(as mentioned in the Contributing.md)
Updated README because there were some courses grouped together inside the "Beginner Course on Programming and Coding Fundamentals" section. I took out those courses because it was very clear that those courses don't belong to this section and somebody has mistakenly done that.

please resolve readme conflict


PLF_1716_00005034

This PattyFam has a product description as "No significant database matches".  New way of saying hypothetical?

hah, that’s the annotation in coreseed for those proteins.
http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143
—bob
On Feb 10, 2016, at 2:01 PM, ARWattam <notifications<EMAIL_ADDRESS>wrote:
This PattyFam has a product description as "No significant database matches". New way of saying hypothetical?
[annotation]https://cloud.githubusercontent.com/assets/11475316/12960063/35bb2b84-d007-11e5-8f49-8fa1925f3df1.png
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/578.

There were exactly 5 functional roles like that in the CoreSEED and all of them in Streptomyces coelicolor A3(2). Interestingly they had a period at the end of the role. Now those don’t exist in the CoreSEED anymore. Additionally there were 282 without a period. I tried to globally change them to "hypothetical protein", but the SEED viewer choked on them and I renamed the 282 to “hypothetical". Now it would be good if someone could rename the 290 “hypothetical” guys to "hypothetical protein” from the command line, without using the SEED Viewer. Gary must have missed all of them when he was cleaning up the hypotheticals.
Veronika

On Feb 10, 2016, at 2:42 PM, Olson, Robert D<EMAIL_ADDRESS>wrote:
hah, that’s the annotation in coreseed for those proteins.
http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143 http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143
—bob

On Feb 10, 2016, at 2:01 PM, ARWattam<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
This PattyFam has a product description as "No significant database matches". New way of saying hypothetical?
https://cloud.githubusercontent.com/assets/11475316/12960063/35bb2b84-d007-11e5-8f49-8fa1925f3df1.png
—
Reply to this email directly or view it on GitHub https://github.com/PATRIC3/patric3_website/issues/578.

Hm, search is showing them in other orgs too
I can make that change. We do have a “hypothetical” from earlier:
| prot                   | assigned_function | last_modification   |
+------------------------+-------------------+---------------------+
| fig|217.1.peg.395      | hypothetical      | 2015-11-03 13:47:57 |
| fig|296591.12.peg.2433 | hypothetical      | 2015-11-03 13:52:38 |
| fig|342108.5.peg.3976  | hypothetical      | 2015-11-03 13:54:03 |
| fig|471853.5.peg.3784  | hypothetical      | 2015-11-03 13:56:59 |
| fig|471870.8.peg.2222  | hypothetical      | 2015-11-03 13:57:05 |
| fig|573234.4.peg.13    | hypothetical      | 2015-11-03 13:58:53 |
| fig|575609.3.peg.576   | hypothetical      | 2015-11-03 13:58:59 |
| fig|657314.3.peg.422   | hypothetical      | 2015-11-03 14:00:05 |
| fig|743718.4.peg.2713  | hypothetical      | 2015-11-03 14:00:56 |
| fig|1007096.3.peg.908  | hypothetical      | 2015-11-03 14:02:19 |
+------------------------+-------------------+---------------------+
I’ll go ahead and change those as well.
—bob
On Feb 10, 2016, at 3:25 PM, Veronika Vonstein <vonstein.veronika3<EMAIL_ADDRESS>wrote:
There were exactly 5 functional roles like that in the CoreSEED and all of them in Streptomyces coelicolor A3(2). Interestingly they had a period at the end of the role. Now those don’t exist in the CoreSEED anymore. Additionally there were 282 without a period. I tried to globally change them to "hypothetical protein", but the SEED viewer choked on them and I renamed the 282 to “hypothetical". Now it would be good if someone could rename the 290 “hypothetical” guys to "hypothetical protein” from the command line, without using the SEED Viewer. Gary must have missed all of them when he was cleaning up the hypotheticals.
Veronika
On Feb 10, 2016, at 2:42 PM, Olson, Robert D. <olson<EMAIL_ADDRESS>wrote:
hah, that’s the annotation in coreseed for those proteins.
http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143http://core.theseed.org/FIG/seedviewer.cgi?page=Annotation&feature=fig|100226.1.peg.8143
—bob
On Feb 10, 2016, at 2:01 PM, ARWattam <notifications<EMAIL_ADDRESS>wrote:
This PattyFam has a product description as "No significant database matches". New way of saying hypothetical?
[annotation]https://cloud.githubusercontent.com/assets/11475316/12960063/35bb2b84-d007-11e5-8f49-8fa1925f3df1.png
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/578.

Ugly, but I don't consider it something that needs to be addressed quickly.
I guess the changed annotation will propagate when we sync the annotation next time and we can wait until then.

yes, that is correct.
On Feb 24, 2016, at 3:01 PM, mshukla1 <notifications<EMAIL_ADDRESS>wrote:
Ugly, but I don't consider it something that needs to be addressed quickly.
I guess the changed annotation will propagate when we sync the annotation next time and we can wait until then.
—
Reply to this email directly or view it on GitHubhttps://github.com/PATRIC3/patric3_website/issues/578#issuecomment-188451667.

Given that Bob has fixed the annotation in core seed, I am closing the issue.


Can not set Invariant culture in Client

Describe the bug
I tried to set the Client to English, which is not available so I chose Invariant. But that is overriden with my local culture de everytime. As a work around I set en manually in the localization config.
Expected behavior
Either select en from available culture or at least invariant should not get overriden when set manually.

The default value of the InvariantCulture is depending to the local OS settings. So it is a correct behavior if your OS is set to german. The work around looks like it is working but the ResX files contains a neutral language with english translations which works like a fall back in your case. I changed the localization config to xy and get an english localized client too.
So in my opinion en is just missing in the available localizations. I  suggest to add en and copie the texts from neutral which are already english translations. What do you think about my suggestion @Toxantron @dbeuchler ?

Your statement about the InvariantCulture is not correct. The InvarianCulture ist mapped to the english language but is not associated with any Country/Region. This is also part of the documentation at microsoft
The neutral language is part of the fallback process - by default, the invariant culture will be used. In moryx we do not change the default behaviour and therefore the neutral language is equal to the invariant culture.
For this ticket, I would prefer to have also the "en" entry within the combo box.

I am not really firm on this topic, but I definitely would like to see en in the combo box and hope there is a solution to avoid copying language content.

My fault. @dbeuchler is right about the InvariantCulture. The behavior of the neutral language and the fallback process works correctly but there are a code part which overwrites the InvariantCulture with the current ui culture. This is wrong and the reason of your problem @Toxantron . Furthermore the InvariantCulture is not realy a language which should be selectable at the UI by the user but used automatically for example in error cases. So i will fix the wrong code which overwrites the InvariantCulture, delete the InvariantCulture from the combobox and add en en entry for the user.


Support Kafka 0.9

As Kafka 0.9 is near release, we should support this new version along with new features like authentication and wire encryption.

Kafka 0.9 supports TLS/SSL (https://issues.apache.org/jira/browse/KAFKA-1684). We've been getting some customer interest in supporting this feature in our input/output plugins.

This is being actively worked on. We are targeting 2.2 for this.

That's mean is not working with Kafka 0.9 currently?
I tried with Kafka 2.11-0.9 (this one http://mirrors.ircam.fr/pub/apache/kafka/<IP_ADDRESS>/kafka_2.11-<IP_ADDRESS>.tgz), I am not seeing any errors and any messages as well ;-(

@ebuildy, In this case, "supporting 0.9" means to update the producer to depend on the latest package release, which would enable support for SSL. The current release of the plugin is still fully compatible with the wire protocol of kafka 0.9 and should continue to work against a kafka 0.9 cluster (without SSL).

Thanks you, must be an issue with my setup, so The Truth Is Out There ;-( I can see the topic is created by logstash, but I don't see any message with:
kafka-console-consumer.sh --zookeeper docker:2181 --topic logstash-raw

My logstash config is pretty simple:
 kafka
    {
        topic_id => "logstash-raw"
        bootstrap_servers => "kafka:9092"
        linger_ms => 5
        timeout_ms => 5
    }

Do I miss something important with Kafka (I am quite new on it), many thanks for your help.

Will Logstash also support SSL + SASL with Kafka 0.9.0?

+1

This is being actively worked on. We are targeting 2.2 for this.

@suyograo just wondering where is it being worked on if it still is? I have a kafka 0.9.x deployment and would love to start using new consumer and producer protocol.

FWIW, Kafka 0.10 is slated for late April, with breaking protocol changes.  Maybe better to focus on support there, instead of everything for 0.9, just to change it again soon?

@cptcanuck that's a fair point, but there will always be another version of kafka API which breaks backwards compatibility until kafka reaches 1.0, I assume.
I had a quick look and it should be fairly simple to add kafka 0.9.x producer support to logstash-output-kafka. I will give it a go.

@vaijab 0.9 for this output is work in progress here:  https://github.com/logstash-plugins/logstash-output-kafka/pull/36
As for consumer, its in this branch: https://github.com/logstash-plugins/logstash-input-kafka/tree/update/0.9
Idea is to release a beta this week.

Working on adding integration tests

Idea is to release a beta this week.

@suyograo awesome stuff. I'll definitely give it a go at testing from tomorrow. Thanks.

Hi
Thank you for supporting Kafka0.9.
I am using Kafka<IP_ADDRESS> with SASL+Kerberos.
I can't see properties related with SASL+Kerberos.
If there is, please let me know the link. i am glad to test it for this release.
or is there any plan to support it also?

@vaijab Beta 1 versions are out
bin/plugin install --version 3.0.0.beta1 logstash-output-kafka

@brouwerorange this beta does not have SASL support yet

@suyograo looks like this issue should be closed? Kafka 0.90 support is already there right?
What i see that is missing is:

SSH tests (integration and unit).
Some examples in the documentation on using SSH.
Fix old documentation to include this settings.


Microsoft.Bcl.Async without workaround

Hello.
Is there a solution for not working import of build targets of Microsoft.Bcl.Build?
We don't want to / can mix nuget and paket in our .Net 4.0 solution with over 100 projects (half of them use async/await with the library) like descriped in issue #1007.

what's the concrete issue?

Like in #1007 I got the following error message during compilation:
 The name 'TaskEx' does not exist in the current context.
I use the 2.0 version of paket.

no I mean, what do you think should be changed in Paket?

I still don't understand what we do wrong.

In #1007 at the end is a comment, that a new issue will be opend to solve this problem without the need of using nuget in parallel.
I don't found any and wanted to know, if a better way was found.

;-)
I really want to help, but I don't know what's wrong.
Is the targets file reference missing in the csproj after paket install? Or what is missing / going wrong?

Yes. The target is not in the project files.

wow just found https://github.com/fsprojects/Paket/issues/618

closing this - let's keep discussion in #1007


Train Custom Data

Thanks for sharing your excellent work! If I want to train on my own data, how to set the image input size? What conditions of input size must be met for network structure？

@LV0913 Add the following lines in your custom exp file.
https://github.com/Megvii-BaseDetection/YOLOX/blob/3bfac8f843791f007cb9cc7733e6e831f753e7d7/yolox/exp/yolox_base.py#L28-L29
14, 26 means the multiscale ranges from 1432 to 2632. The only rule is the image size can be divided by 32.

OK, thanks!


added about section in webpage

@SauravMukherjee44 please review my pr and merge it

Resolve Conflicts!

i haven't touched index or resources file, why is it conflicting then? can you help me understand. Do i have to merge in branch different from main.
i have cloned main branch only

@SauravMukherjee44 resolved merge conflicts.

Again once and share the link of hosted repository.

Because before merging my pr someone has already changed something in index.html that's why it's again causing merge conflict. I will resolve it wait

@SauravMukherjee44 fixed merge conflicts, now accept my pr

Resolve the merge conflict


Implement decorator for converter function

I'd like to use a decorator to implement a converter, similar to validator decorator.
Is something like the following possible?
    data : bytes = attr.ib( default=b'', on_setattr=attr.setters.converter )

    #! NOTE: using attrs converter as a property setter function.
    @data.converter
    def data_setter( self, attribute : attr.Attribute, value : bytes ) -> None :
        trimmed_value = value[ : 10 ]
        return trimmed_value

Maybe the API would be different?
    data : bytes = attr.ib( default=b'', on_setattr=attr.setters.converter )

    #! NOTE: using attrs converter as a property setter function.
    @data.converter
    def data_setter( self, value : bytes ) -> bytes :
        trimmed_value = value[ : 10 ]
        return trimmed_value

NOTE: I also asked question on stackoverflow :)
https://stackoverflow.com/questions/63625357/decorator-for-attrs-converter

StackOverflow should be better for questions… and it looks like you got a reply there.  Closing here.


Add default theme module

Closes #53

This change is

Reviewed 15 of 15 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.

Comments from Reviewable

Reviewed 14 of 15 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


filesize problem

v1
file size different from content length.
i need how much space it will fill up on your computer. not content length.
Content size and file size are different.
MaxFileSize function check with content length.

Thanks for opening this issue, @Prince444.
Quick question: Is it not possible to compute the file size on the server side? For example (Node.js), if you save the file on the disk, you can use fs.stat to find the file size. If you want the size before saving it, you can even use the Buffer length to compute the number of bytes (https://stackoverflow.com/a/46637826/1656944). What language are you using, perhaps I can point to the right docs?

Thanks for Reply @AnandChowdhary
i am using asp.net mvc for file save in server.
HttpFileCollection hfc = System.Web.HttpContext.Current.Request.Files;
in this line i get file in hfc variable.

again explaining
actual picture size is 1 MB but content length showing it 1.6 MB

Provide doc for implementing v2 in asp.net mvc Razor.


USHIFT-361: Fix RPM post install script not to return error when crio is not running

The problem occurs when crio has just been installed and not enabled yet.
Closes #899

@ggiguash thank you for adding the credits for @dmc5179.
/lgtm


Write up and explain concurrency models

Introduce "muted" concurrency mode where future renders are silently not run while another render is in progress.... Justification: command is running in terminal, you type and hit Enter, should that take effect? Or should it be silently dropped since you didn't know the output of the previous command

Parallel aka mergeMap (autocomplete bug): https://rxviz.com/v/A8DlYLy8
Cutoff aka switchMap (terminates existing, fixes autocomplete bug): https://rxviz.com/v/B8ZY2daJ
Serial aka concatMap (gets slower and slower): https://rxviz.com/v/RObNEYEO
Mute (denies if existing is in prograess, forces retry): https://rxviz.com/v/1o7zdGp8

https://medium.com/p/91f96d02bb61/edit


Signature of request, should also be calculated from parameters.

Currently signature is calculating only from remote url.
But this is big issue, because someone can enumerate parameters and for ex. fill up disk with cache.
Example:
http://imageproxy/200,s12312312313123/http://image/image.jpg
http://imageproxy/1,s12312312313123/http://image/image.jpg
http://imageproxy/2,s12312312313123/http://image/image.jpg
http://imageproxy/3,s12312312313123/http://image/image.jpg
http://imageproxy/4,s12312312313123/http://image/image.jpg
http://imageproxy/5,s12312312313123/http://image/image.jpg
http://imageproxy/6,s12312312313123/http://image/image.jpg

yeah, this is a known issue.  The same problem exists for why it hasn't been addressed, which is that options have to be put into a canonical form for calculating the signature, but that form has never been defined.  It exists in the imageproxy code today, but it's never been publicly documented.
And perhaps the current canonical form should be changed?  Right now it's somewhat arbitrary... options are somewhat grouped logically, but then new ones have simply been added to the end.  Maybe they should be in lexicographical order?
(I know there was discussion about this in the past at one point.  Not sure if the issue is still open.  If it is, we should probably merge this into that one)

@willnorris I think even if options can be included in the signature in the same order as they're in the URL that still will be a benefit.


Move react to dependencies

I know it is small, but I think react should be listed under dependencies not under devDependencies.
Or is this done on purpose?

It's under peerDependencies to ensure that depending apps and the router use the same instance of React and to avoid version conflicts.

Yep. And it's also in devDependencies for running tests.


add the locale keeping before formatting the output date

the result display should be changed too according with the user's locale

hi, @jkuri
I am using now "ng2-datepicker": "^1.8.3", but I see the problems with locales again. As I see, these two commits was merged in 1.7.0. Were they included into 1.8.3? Maybe I do something wrong, could you help me to find out the reason of the problem?
Thank you,
VVildVVolf


thread type annotation does not appear to have any fields/methods

Describe the bug
When I create any object of type thread and try to use its methods (e.g., resume()), the language server warns of an undefined field.
To Reproduce
Steps to reproduce the behavior:
local c = coroutine.create(function() end)
c:resume() -- Undefined field name `resume`

Similarly:
---@type thread
local t
t:status() -- Undefined field name `status`

Expected behavior
This language server should recognize that objects of builtin type thread have the methods that they do.
Screenshots

Environment:

OS:nixOS
Is WSL remote? no
Client: neovim

This is desired behaviour. In standard Lua, you can't index threads. Are you using a specific environment, where thread has a metatable? If so, add somewhere this declaration:
---@class thread : coroutinelib

Ah you're right! I have been working with some object-oriented Lua and totally misremembered the documentation about coroutines. Thanks for being helpful!


Multiplatform examples and templates. Merge all modules into one single module

We separated multiplatform project into common/android/desktop modules because we had issues with plugin compatibility ("application" plugin doesn't compatible with "com.android.application").
But now we don't use "application" plugin, so maybe we can rewrite most of our examples and templates with single module.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.