[STK-325] Add forecastle annotations

@waseem-h Chart: grafana Dry Run successful

@waseem-h Chart: grafana-storage Dry Run successful


Issue with Dockerfile

Hi all,
It seems like there is a problem with the default Dockerfile where Ubuntu cannot find the python3.7 package:
$  docker build -f Dockerfile .
[+] Building 3.6s (9/32)                                                                  docker:default
 => [internal] load build definition from Dockerfile                                                0.0s
 => => transferring dockerfile: 3.08kB                                                              0.0s
 => [internal] load .dockerignore                                                                   0.0s
 => => transferring context: 179B                                                                   0.0s
 => [internal] load metadata for docker.io/library/ubuntu:16.04                                     0.3s
 => [ 1/29] FROM docker.io/library/ubuntu:16.04@sha256:1f1a2d56de1d604801a9671f301190704c25d604a41  0.0s
 => CACHED [ 2/29] RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y     aut  0.0s
 => CACHED [ 3/29] RUN add-apt-repository -y ppa:deadsnakes/ppa                                     0.0s
 => CACHED [ 4/29] RUN wget -O - https://apt.llvm.org/llvm-snapshot.gpg.key| apt-key add -          0.0s
 => CACHED [ 5/29] RUN apt-add-repository "deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-9  0.0s
 => ERROR [ 6/29] RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y  --no-in  3.2s
------
 > [ 6/29] RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y  --no-install-recommends --force-yes     clang-9     python3.7     python3-pip     python3-setuptools:
0.274 Hit:1 http://archive.ubuntu.com/ubuntu xenial InRelease
0.274 Hit:2 http://archive.ubuntu.com/ubuntu xenial-updates InRelease
0.274 Hit:3 http://archive.ubuntu.com/ubuntu xenial-backports InRelease
0.274 Hit:4 http://security.ubuntu.com/ubuntu xenial-security InRelease
0.387 Get:5 http://ppa.launchpad.net/deadsnakes/ppa/ubuntu xenial InRelease [18.1 kB]
1.266 Get:6 https://apt.llvm.org/xenial llvm-toolchain-xenial-9 InRelease [4234 B]
1.506 Get:7 https://apt.llvm.org/xenial llvm-toolchain-xenial-9/main amd64 Packages [8828 B]
1.528 Fetched 31.1 kB in 1s (23.2 kB/s)
1.528 Reading package lists...
2.283 Reading package lists...
2.995 Building dependency tree...
3.114 Reading state information...
3.158 E: Unable to locate package python3.7
3.158 E: Couldn't find any package by glob 'python3.7'
3.158 E: Couldn't find any package by regex 'python3.7'
------
Dockerfile:32
--------------------
  31 |     RUN apt-add-repository "deb http://apt.llvm.org/xenial/ llvm-toolchain-xenial-9 main"
  32 | >>> RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y  --no-install-recommends --force-yes \
  33 | >>>     clang-9 \
  34 | >>>     python3.7 \
  35 | >>>     python3-pip \
  36 | >>>     python3-setuptools
  37 |
--------------------
ERROR: failed to solve: process "/bin/sh -c apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y  --no-install-recommends --force-yes     clang-9     python3.7     python3-pip     python3-setuptools" did not complete successfully: exit code: 100

Updated Dockerfile to use latest versions of llvm/clang/python/bear etc in commit d4fe1889416f54db54b775c754810e39b637a800


RedisTokenStore uname_to_access and client_id_to_access memory leak

Summary
When the access key is stored, there is being set several expirations on keys related to the access key.
Unfortunately there are two redis keys that are not expired correctly. These keys are with prefixes client_id_to_access: and uname_to_access:. In following example I am going to describe only the client_id_to_access: keys.
Actual Behavior
Lets imagine there is just one client - web - that is used by all the users that logs in to the application. Every time there is a new access token generated it is stored into the list (or set in newer RedisTokenStore implementation) client_id_to_access:web. The expiration is set to whole key (not just to new value added).
In case the access token expiration is 5 minutes and your app is regularly visited - there is no 5 minutes window that no one generates new access token - then this list (or set) gets bigger and bigger, containing all the tokens that were stored previously and are already expired from all the other keys (e.g. access: and so on).
Version
We're using version 2.0.16 that is somehow older. On the other hand the same issue is even in the master branch. The only main difference between 2.0.16 and master branch is in used data structure. 2.0.16 uses the list (rPush operations) and master uses set (sAdd) that has the O(1) complexity to store and remove. But the expiration is completely the same.

I am Going to create a fix PR for this.
I wonder why this was never figured out before.

Recently I have this problem too, one of our client_id_to_access list size is 4 million...
it occupies 3g memory in the redis

You can take RedisTokenStore impl from my PR and place it into your codebase. I am going to do this today.
One trouble is that when you load my implementation, already stored access tokens in those two lists are not going to be used. So e.g. if your expiration time for access token is 5 minutes, then for those five minutes those lists won't contain all the access tokens it should be containing.
It can be migrated manually, e.g. using the redis-cli, but because it is just 5 minutes and because it is on redis keys that are not normally used, then I am not going to take care of this on our product... It will just solve itself in those 5 minutes.

So, I have deployed the version from my Pull Request (just copied the class into our project so it is overriden) into out production.
In order token store to be properly cleared, you have to call method doMaintenance() once upon a time. I have created a @Scheduled method that calls it every 6 hours. Or it can be done even less.
The example with the initialization (we're using Spring Boot):
@Configuration
@EnableScheduling
public class AuthorizationServerConfiguration {

   @Autowired
    private RedisConnectionFactory redisConnectionFactory;

   @Bean
    public RedisTokenStore redisTokenStore() {
        RedisTokenStore redisTokenStore = new RedisTokenStore(redisConnectionFactory);
        // any custom settings you need
        return redisTokenStore;
    }

    @Scheduled(fixedDelay = 6 * 60 * 60 * 1000) // 6 hours
    public void doRedisStoreMaintenance() {
        logger.info("Going to do RedisTokenStore maintenance.");
        RedisTokenStore redisTokenStore = redisTokenStore(); // this actualy gets already created bean (thanks to Spring Boot proxy)
        redisTokenStore.doMaintainance();
        logger.info("RedisTokenStore maintenance done.");
    }

Hello guys.
I encountered a problem with my security implementation and want to know if this also has anything to do with this issue.
Somehow my uname_to_access on redis ended having multiple values with the same token (I have 1 year expiration tokens for default) with same expiration time. This is really necessary?
Checking directly on code those objects are really equals with each other:

Asking here because my redis key has many and many values of the same token with exacly the same values (even the expiration time).
Apparently some of them has a different expiration time, but what is the reason for having multiple values for the same token with different expiration on this database?
Thank you.

@lucasoares I think that it is not related to this issue, this was about expired tokens.
If you have more equal tokens stored in the map you have to find what component stores it more times into the token store. The equals method on OAuth2AccessToken just validates the UUID of the token. When it is stored into the RedisTokenStore, it is stored in a Redis Set - it means that it is identified by whole stored datastructure and thus if it contains e.g. different validity (or anything else), it is treated as a different item and stored again.
I would suspect the access token renewal (using refresh token) mechanism. When original access token is still valid it is left in the Token Store as it is. And if your refresh functionality e.g. returns the same existing access token as the refreshed one, it might be stored again. But it is only a hypothesis of mine...

The RedisTokenStore doesn't deal with this collection as a set and this why everytime the token is stored again.
Everytime the same token is generated using the grant type password a new value are added to this list.
I will create a new issue then.

@lucasoares I changed it from a Set to a Sorted Set. It was actually a Set before. I think that the List datastructure was used in older versions.
The Sorted Set is used because of fast operation to delete expired tokens (score is the time of expiration).
In my case I had a problem with huge amount of expired tokens still stored in Redis. And because Redis stores everything in memory, our server was slowly moving into full RAM state. This is what the PR solves, correct cleanup of expired tokens.
Your problem looks like a different situation. A still valid token is stored many times into the TokenStore. I took a quick look into the code and this case should be handled out of the TokenStore. When there is a generation of new token, the old one should be deleted (in case it exists). Open the class DefaultTokenServices and try to find whether there is not a problem.
And maybe open another issue and put a link here so we can move this conversation there :)

thanks, i have this problem too.
The value size of key client_id_to_access:xxx is about 200M, so when a token expired, will call function removeAccessToken then call redis lrem, this is a O(n) function, it will scan the list from begin to end to find the expired token and remove it.
This cause the cpu going to 100%, so sad.

Yeah I am facing the same, looks like this issue is still not fixed in master

Sorry for late reply. I was not able to accomplish my PR for this issue - https://github.com/spring-projects/spring-security-oauth/pull/1660/files
In our production we're using the method I introduced in that PR - (doMaintenance) - and it solves the issue. It is called once upon a time and because it uses the scan functionality it does not block redis on consume too much resources...
You can inspire by that and either make the same override in you app or better introduce a PR that would be accepted by the community.


embedded-time Crate

Rather than piecemeal implementing time functionality we could use the embedded-time crate which is rather complete and may help interoperability if it becomes widely adopted. Cc #136

Only blocker for this would be that it would of course be a breaking API change

The conclusion from #149 is that this should wait until embedded-time is more mature and adopted by the community.


Angular 11 - 'ngx-datatable' is not a known element

I'm submitting a ... (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
ng serve fails with
'ngx-datatable' is not a known element

Expected behavior
Should render table
Reproduction of the problem
I followed the documentation and it works as expected when <ngx-datatable> used in app.component.html but fails if I use it in any other module

Table version: 18.0.0
Angular version: 10, 11
Browser: all
Language: all

@panKt Are you importing NgxDatatableModule in your feature module?

@joeveiga yes

@panKt It works as expected on my end. Would you be able to provide a demo? Thanks.

I encounter the same issue on my side ('ngx-datatable' is not a known element) and also this error "error NG8002: Can't bind to 'sortable' since it isn't a known property of 'ngx-datatable-column'." Also I imported NgxDatatableModule in my feature module
I am on Angular 12.1.0.
Please help me on this issue, Thanks.

Same error for me , even after importing in feature module. Am using angular 12.0.1.
Error: src/app/dashboard/dashboard.component.html:349:7 - error NG8001: 'ngx-datatable' is not a known element:

If 'ngx-datatable' is an Angular component, then verify that it is part of this module.
If 'ngx-datatable' is a Web Component then add 'CUSTOM_ELEMENTS_SCHEMA' to the<EMAIL_ADDRESS>of this component to suppress this message.

349       <ngx-datatable class='bootstrap no-detail-row' [columnMode]="'force'" [footerHeight]="50" [rowHeight]="'auto'"
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
350       [rows]="rows" [count]="totalCount" [limit]="dataParams.page_size">
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Please help I'm getting same errors from last 3 days and did the same and imported NgxDatatableModule in my feature module
I am on Angular 12.1.0.

anwser plz

I'm getting this same error anhular 11 and 19.0.0 table.. why is this issue closed without proper answer?!?!?!?!

This issue persists and yet, it's still closed and no response. That's cool, I guess?

the solution is simple, Since, we are externally importing the data in the table, so we need to import and register the HttpClientModule in AppModule file.
The step we are performing in app.module.ts we should apply same on XXpage.module.ts
in my case I did for HOME
import { NgModule, CUSTOM_ELEMENTS_SCHEMA } from '@angular/core';
import { CommonModule } from '@angular/common';
import { IonicModule } from '@ionic/angular';
import { FormsModule } from '@angular/forms';
import { HomePage } from './home.page';
import { HomePageRoutingModule } from './home-routing.module';
import { HttpClientModule } from '@angular/common/http';
import { NgxDatatableModule } from '@swimlane/ngx-datatable';
@NgModule({
imports: [
CommonModule,
FormsModule,
IonicModule,
HomePageRoutingModule,
HttpClientModule,
NgxDatatableModule
],
declarations: [
HomePage
],
schemas: [CUSTOM_ELEMENTS_SCHEMA]
})
export class HomePageModule {}


change ownership to ome

this issue tracks the proposal to change the ownership of the repo to the ome organization (https://github.com/ome), or implement similar functionality in that organization.

closing as #wontfix


python38Packages.python-lsp-server: 1.2.2 -> 1.2.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.python-lsp-server is: "Python implementation of the Language Server Protocol"
meta.homepage for python38Packages.python-lsp-server is: "https://github.com/python-lsp/python-lsp-server"
meta.changelog for python38Packages.python-lsp-server is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

/nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3/bin/pylsp passed the binary check.

/nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3/bin/.pylsp-wrapped passed the binary check.

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

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

found 1.2.3 with grep in /nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3

found 1.2.3 in filename of file in /nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3

directory tree listing: https://gist.github.com/671a03076c8694374bfdc6852d763ccb

du listing: https://gist.github.com/b15ce51fc75d4169a53d1be8b1bb433a

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

53 total rebuild path(s)

14 package rebuild(s)

14 x86_64-linux rebuild(s)
14 i686-linux rebuild(s)
13 x86_64-darwin rebuild(s)
12 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
cq-editor
python38Packages.pyls-flake8
python38Packages.pyls-isort
python38Packages.pyls-spyder
python38Packages.pylsp-mypy
python38Packages.python-lsp-black
python38Packages.python-lsp-server
python38Packages.spyder
python39Packages.pyls-flake8
python39Packages.pyls-isort
python39Packages.pyls-spyder
python39Packages.pylsp-mypy
python39Packages.python-lsp-black
python39Packages.python-lsp-server

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3
ls -la /nix/store/63dypn6rq9p6pq0fj3h90lqyp1zdgnlm-python3.8-python-lsp-server-1.2.3/bin

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

Maintainer pings
cc @fabaff for testing.

closing in favor of #140596


Run CI on latest version of Windows

Since we no longer need to run our tests on Internet Explorer we can use the latest version of Windows for CI.

Gonna merge this in since it's just a small change to the CI. Let me know if there is any feedback.


[Refactor/State] Tests, basic tests

[ ] test for render for the case where you put a subscription straight on the app that has no key in the data

**render case
**  if (elem.$) {
// needs a test
let target = get(tree, elem.$)
if (!target) {
// create target on state (to do a first render)
const obj = {}
set(obj, elem.$, {})
state.set(obj)
target = get(tree, elem.$)
}
return target.[uid]
} else {
return tree.[uid]
}


[1.0] nb's unstable feature plan

embedded-hal depends on nb's unstable feature for its 1.0 release.
What is the plan for this feature regarding nb's 1.0 release?

The unstable feature will be removed for 1.0 as discussed in yesterday's EWG meeting.


Fix broken packges that do not build

Cherry-picking fixes from master
Motivation for this change
Currently these packages do not build in 18.03 do to unreachable
patches and they are fixed in master.

Please split your changes into separate commits (in most cases one commit per package), for easier review and cleaner history. If possible try to git cherry-pick -x the corresponding changes from master so we can track each change to its origin.

@xeji the problem is that it is quite tedious to track all commits that affected these packages. For example there were commits that touched all packages (like https://github.com/NixOS/nixpkgs/commit/52f53c69ce6dbc5538f7e4cd22f9d93baf1f64a2)
Cherry-picking that will not make tracking changes easier. I can definitely split this into 3 different PRs, but I am not sure whether it will be possible to do git cherry-pick -x for them

No problem. git cherry-pick -x is our preferred method for backporting changes but it doesn't  always work well. Just give it a try first before you do something else.


Data issue (Ottawa May 28)

I'm not sure if this is where to post this, but there's an issue with the vaccination data for Ottawa (region 3551) for today (May 28th).  It shows change_vaccinations of 17771 (which seems correct), but change_vaccinated is 485840 (I'm sure it's supposed to be in the range of 4,000). I'm sure it's a problem with the Ontario source data, but it's worth noting.

It might have been an error during initial data entry. Looks like it has already been collected.


Update README.md

just fixing typos (copy&paste error ?)

Thanks @snoopy0815, it must have been an auto complete error.


documentation miss: Docker label PROMETHEUS_EXPORTER_PORT should be defined in ECS Task definition

First of all: thanks, it works.
But the documentation lacks information how to set docker label(s).
I tried to to set up PROMETHEUS_EXPORTER_PORT in Dockerfile like
LABEL "PROMETHEUS_EXPORTER_PORT"="5000"

but still failed to write non-empty ecs_file_sd.yml . The log still featured
2021/06/12 04:00:51 Writing 0 discovered exporters to ecs_file_sd.yml

After poking with ECS services I managed to add
"dockerLabels": {
        "PROMETHEUS_EXPORTER_PORT": "5000"
      }

to the ECS task definition JSONs and it is what really fixed the issue.
I'm somewhat new to AWS ECS service stuff and maybe propagating real docker labels can be made, but current documentation is misleading, e.g. the words
To scrape the containers add following docker labels to them:
    PROMETHEUS_EXPORTER_PORT specify the container port where prometheus scrapes (mandatory)

Hi, I don't see where this information is missing. If you think the documentation is lacking, just make a pull request and we can review it. Thanks!

@chicofranchico done in PR #69
pls review

That's much clearer indeed. Thanks for that.

@chicofranchico @muravjov
I deployed the prometheus-ecs-discovery container in one of my AWS EC2 instance. It is running fine but getting the same "Writing 0 discovered exporters to /tmp/ecs_file_sd.yml" errors.
The confusion part is here,
I tried to setup dockerLabels in my ECS task definition but showing a huge number of existing task definition in my AWS account and multiple cluster as well.
Do I need to modify each task definition with the above dockerLabels config? or create a new one and again deploy the new container "prometheus-ecs-discovery"?
Please help me.

Sorry but this is an issue tracker, not a support request forum.  If it's getting no discovered exporters then there's probably a label matching issue.


fix: fix player struct alignment for AE

Fixes the alignment of INFO_RUNTIME_DATA, removes the padding between sleepSeconds and largeBiped

Thanks!


all diagrams

@l3r8yJ take a look please
closes #66

PR-Codex overview
This PR focuses on adding PlantUML diagrams for the register chat ID flow, add token success flow, token validation process, and notifications read flow.
Detailed summary

Added register-chat.puml diagram for the register chat ID flow.
Added puml.yaml workflow file for generating SVG diagrams.
Added add-token.puml diagram for the add token success flow.
Added token-validation.puml diagram for the token validation process.
Added notifications-read.puml diagram for the notifications read flow.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@h1alexbel thank you for your Pull Request. I'll assign someone to review it soon.
If this PR solves a todo from the code, please don't forget to remove it.

@h1alexbel please review this Pull Request. Deadline (when it should be merged or closed) is 2023-09-04T06:59:03.143679.
You should check if the requirements have been implemented (partially or in full), if there are unit tests covering the changes and if the CI build passes. Feel free to reject the PR or ask for changes if it's too big or not clear enough.
Estimation here is 30 minutes, that's how much you will be paid. You will be paid even if this PR gets rejected.

@zoeself deregister

@zoeself assign @l3r8yJ

@zoeself assign @l3r8yJ

@h1alexbel didn't understand that, sorry.

@l3r8yJ reminder

looks good to me! @rultor merge

looks good to me! @rultor merge

@l3r8yj I see you're talking about me, but I don't understand it. If you want to say something to me directly, start a message with @rultor

@rultor merge

@rultor merge

@l3r8yj OK, I'll try to merge now. You can check the progress of the merge here


Move the Travis "script" from .travis.yml to a separate file.

Fixes #693.

Codecov Report

Merging #747 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #747      +/-   ##
============================================
- Coverage     76.16%   76.14%   -0.03%
  Complexity      783      783
============================================
  Files           144      144
  Lines          3617     3617
  Branches        343      343
============================================
- Hits           2755     2754       -1
  Misses          794      794
- Partials         68       69       +1

Impacted Files
Coverage Δ
Complexity Δ

...census/implcore/trace/export/SpanExporterImpl.java
90% <0%> (-1.67%)
6% <0%> (ø)

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 62a0e7c...fcce0e8. Read the comment docs.

Coverage remained the same at 78.048% when pulling fcce0e8fd77c2db321be328f1643119693e3c054 on ubschmidt2:travis into 62a0e7c83f37ca9e78b6952fc4834eaca22285c5 on census-instrumentation:master.

Coverage decreased (-0.03%) to 78.02% when pulling 00d8cad1a51fe7acc67f2a882cebc9a7c2bbd607 on ubschmidt2:travis into 62a0e7c83f37ca9e78b6952fc4834eaca22285c5 on census-instrumentation:master.

Coverage remained the same at 78.048% when pulling 08eb73ef6842675c73ea8ae75081d170d2da0e8f on ubschmidt2:travis into 62a0e7c83f37ca9e78b6952fc4834eaca22285c5 on census-instrumentation:master.

Coverage remained the same at 78.048% when pulling d1d6a00ae2ad8b040b962a62234325887cbeb4cf on ubschmidt2:travis into 62a0e7c83f37ca9e78b6952fc4834eaca22285c5 on census-instrumentation:master.


HIP draft – Equitable Transmission Scaling

An improvement to HIP-17 to encourage movement towards optimal coverage.

This definitely raises a great point about different density rules for different environments; for example, you will naturally need more hotspots in an urban environment than in a suburban or rural one, just because of all of the RF barriers an urban environment entails.
Also, if you look at Portland, Oregon for example, almost the entirety of the urban sprawl is scaled down. With areas like this, it's a situation of extremes where you either have completely oversaturated urban areas or more suburban / rural areas that have little to no relative incentive to expand coverage, due to the current universality of PoC scaling rules.

Yes. I was looking at this just last week. In Toronto the metro area has created a saturation in Hex4, but there are plenty of areas like in Richmond Hill area (Major Mackenzie Dr/Young St) where the reward scale for res5+ is 1.00, but res4 is at 0.35. In  so the reward scale for a local HS (say Able Jetblack Sheep) is 0.35 for HIP17. So plenty of expanding urban sprawl areas are very under covered.
A way for res4 to not have such a great effect on sparse areas is desirable.

The thought here was that by scaling up and down we could reach the target faster and closer. Especially in smaller hexes. Under this HIP an unpopulated res 7 hex would grant the first hotspot there a 1.0 scaling from as little as 0.2 scaling. which I think would be more desirable for optimal coverage than starting that HS at 0.2 there is no incentive to place there.

Link to rendered view at https://github.com/Tylerlhess/HIP/blob/cd7d6f2afae72510c4a047a1d32644851f07bcd7/0038-Equitable-Transmission-Scaling.md

Is there perhaps any way to procedurally determine the average / mean reach of hotspots on the network as a whole, and use that as a metric to determine roughly how far out (zoom out, meaning higher (lower..?) res scaling should take effect? I figure there is some version of this which currently determines the validity of PoC witnessing, and there are asserted locations that determine how far away one is from another. Just spitballing here.

We could just drop the hex 4 scaling but I think that would reduce the opportunity for down scaling major cities correctly. I went back and forth on that and I do feel the resolutions are correct just that it shouldn't all be down scaling. There needs to be some mechanism to incentivize getting to target in each region equally. Otherwise you get a ripple effect on optimal hot spot placement instead of a gradual scaling as you near population centers.

@cvolkernick This is a very interesting concept. The way I interpret your comment is that the "res" is dynamically created based on reach of hotspots. So, the reach of hotspots in an area may make a res10 area (as an example) based on reach of lower range antennas. So you don't really have a local hotspot effecting a hex4 area. An interesting corollary of this, is if your location is oversaturated by other hotspots, of say low gain antennas, you can increase your range and get into an area that is less covered and improve scaling. This helps the host and the network. So, the scaling is really by how many are covering an area of interest, and not by a fixed mapped shape (hex).

@wolfenhawke the issue with this is that only a fraction of your rewards come from yourself. Its not your scaling you should be worried about but the scaling of everyone around you that you can witness.

For what it's worth, in general I'm trying to start from the first principles question(s), i.e. "what does the network want to optimize for", and then work backwards from there to think through what would the proper incentives be to encourage the behavior that leads to those optimizations.
As that applies here, my personal take is that the network wants to optimize for:

Coverage area -- straightforward; the total geographic reach
Reliability/stability/consistency of coverage -- i.e. coverage redundancies in a given area so that if one hotspot doesn't happen to hear a given broadcast signal, there are always sufficient others in the area such that at least 1 of them will receive signal.

That said, maybe what is needed is not a change to the existing scaling but an additional "layer" on top. For example, using a hotspot's witness list, calculate the average distance from the hotspot in question to all of it's witnesses. Aggregate those figures for each hotspot in a given hex, and determine a hex res average. Not sure specifically how it would be used (there are probably a lot of different ways one could imagine, with varying levels of technical complexity), but I'm thinking perhaps you could then use those hex averages to determine a coverage radius of each hex and somehow factor that radius into the scaling.
Essentially you would want to minimize the overlap while not leaving gaps in between. One would imagine that generally speaking urban hotspots are going to have a shorter average range on the whole, because of various previously mentioned environmental factors like buildings etc., as opposed to more suburban/rural areas with better line of sight. The areas with less reach you would want to have higher scaling than those with further reach, since because of the lower average coverage range you will need more hotspots to cover the same amount of area (and there generally will need to be more redundancy because of environmental blockers to "filling in the gaps" to to speak, as with large buildings which directionally obstruct said locations.

@cvolkernick There is much value in such a direction. I actually discussed something similar for a different reason with a member some months ago. Humor me for a second. Being in the semi industry and having worked with radio chips one issue I have with them is that the RSSI and SNR measured values from a silicon chip are pretty crappy. Their error is high. This can be alleviated by calibrating every chip on the line and then doing field calibrations regularly. The manufacturing costs would go up, so it's not done on the line. The product cost would go up so it's not done in the product (field support). So, I get frustrated when I see the witness table. BUT, having a list of the hotspot's witness and their RSSI and SNR range would allow for a map of valid ranges that are expected from neighboring hotspots. Thus we are using relative values which the chip is good for, and not absolute values which the chip is bad for. This allows for a more accurate way of checking for gamers (if a new HS got added, then only the legit ones would see it quickly and be able to accurately triangulate location (maybe a backend function) and roughly check values for it.
So, I like your idea of using such a system for maybe a backend calculation of what the coverage is by some definition of an area (let's say a high hex) also.
It makes the scaling distributed. Wow. The population (of hotspots) can determine based on a calculation provided if their current reach is not providing value. The user can then not just see invalids, but be able to do something about it - longer range antenna that doesn't catch the local area or shorter range to better cover near the hotspot and not oversaturate the neighboring area.

@wolfenhawke ironic timing this was just announced by the Console team: https://blog.helium.com/coverage-arrives-with-the-latest-release-of-console-46f5858b1a93

A current issue with HIP17 is the cost at which to calculate these transmit scales across the network every 30 minutes (an epoch) add another calculation on top that and it be even slower.. I like the thought of it and honestly I think an even simpler approach would be more ideal.
Even if you broke HIP17 down to it's barebones it requires a lot of calculations. As an example, let's say you wanted to calculate the reward scale of one hotspot you'd need to calculate and transverse over 49 different hexes with increasing areas and hotspot counts.
As it currently sits there's a hardcoded limit on witnesses for 300m. When comparing this to resolution 8 the max number of hotspots that can be placed simply based on this limitation is 7. One in each corner and one in the center. This is the current "max" density in the eyes of the network. If we would have a mechanism to validate each resolution 8 hex individually to the new "max" we deem necessary based on the most conservative environment we would have a more scalable approach for calculation and you as a user would only need to worry about your placement within the resolution 8 hex.
Doing it this way would also allow us to move miner rewards from the epoch to the block level which once again will move us closer to the scale needed to handle 3 million hotspots..
I too have a draft HIP that not only tries to simplify the HIP17 mechanics but also adds in anti-gaming measures and I'm not opposed to making HIP17 better but fundamentally there's already a computational problem that needs to be solved also at the very least.

A current issue with HIP17 is the cost at which to calculate these transmit scales across the network every 30 minutes (an epoch) add another calculation on top that and it be even slower.. I like the thought of it and honestly I think an even simpler approach would be more ideal.

This would be the same number of calculations just applying them in reverse order. Instead of closest to farthest it would flip total area of the 4 then the 5 then the 6, 7, 8, 9 and apply all the scaling all the way down instead of calculating it up. if you are calculating it up you would need to run these calculations the same number of times unless you are calculating it hex by hex. With this you could calculate state and only have to recalculate down the affected hexes with new nodes. And then any neighboring nodes that are affected by scaling up. of nodes reaching targets. It shouldn't be hard to have a relational graph and update that on the fly as new nodes are activated or old ones deactivated.

A current issue with HIP17 is the cost at which to calculate these transmit scales across the network every 30 minutes (an epoch) add another calculation on top that and it be even slower.. I like the thought of it and honestly I think an even simpler approach would be more ideal.

This would be the same number of calculations just applying them in reverse order. Instead of closest to farthest it would flip. Calculate the total area of the 4 then the 5 then the 6, 7, 8, 9 and apply all the scaling all the way down instead of calculating it up. if you are calculating it up you would need to run these calculations the same number of times unless you are calculating it hex by hex. With this you could calculate state and only have to recalculate down the affected hexes with new nodes. And then any neighboring nodes that are affected by scaling up. of nodes reaching targets. It shouldn't be hard to have a relational graph and update that on the fly as new nodes are activated or old ones deactivated.

It needs to be performed in a decentralized manner, using a relational graph might not be so easy. There's also no need to go up to resolution 4 if you make sure that the smallest unit provides the most ideal coverage is all I'm saying.. Also I'm not saying the number of hexes being used to calculate will change but adding another calculation in general will add to the current problem and not lessen it

closed, @Tylerlhess ...?

@Tylerlhess was this closed by mistake? I was planning to merge this morning for further discussion. Let me know if you are withdrawing it (but lgtm)

@Tylerlhess if you are indeed closing can I have permission to copy and reopen to take up the baton? 😃 I think this is an important discussion either way. :)

Oh the premise upon which this was opened was flawed and based on an incorrect understanding of HIP-17 clipping. I currently believe, with a better understanding of HIP-17, all that is needed might be to change some of HIP-17 N values. I will write up another HIP if and when we (in discord HIP proposal  channel sub topic HEX limit discussion) come to definitive conclusion and have better numbers to support actual impact of changing the N values compared to current scaling. Possibly dropping of Res 4 scaling and increasing res 5 and 6 N to a factor of N = 2


Two slides on the page

Hi, i want add 2 slider on the page.
First slider

{{ item.name }}

{{ item.comment }}

        <button class="swiper-button-next"></button>
        <button class="swiper-button-prev"></button>

Second slide
<button class="swiper-button-prev">prev</button>
      <button class="swiper-button-next">next</button>

{{ item.name }}

{{ item.comment }}

If I click the next button in the first slide, the second slider also moves and vice versa

You need to refer to Swiper documentation to see how to use it and how to configure it to not use buttons navigation.

Can you help me? Trying but it does not work

In config I have to give the right property, right?

Don't think anybody knows all the Swiper configuration options :) You just need to check them from the Swiper documentation.


Invalid address in instructions for adding to HACS

The instructions has  "https://github.com/GeorgeSG/ha-slack-user/" it should be  "https://github.com/GeorgeSG/ha-slack-user"
If using the former HACS throws an error saying:
An error occurred while processing
Exception: HacsException
Message: GitHub returned 404 for https://api.github.com/repos/GeorgeSG/ha-slack-user/
Could not add this repository, make sure it is compliant with HACS.

Thanks, fixed!


Approve Screen: Randomly not showing operations

How to reproduce
Try to approve a transaction from a DAPP, for instance https://connectors.fuel.network
Current action
Wallet is sometimes showing the contract call but other times it's not. It's sort of random
Expected action
Wallet should always show the contract call operation
Notes
https://github.com/user-attachments/assets/c9bda804-ca6a-4c0c-9dfc-1ef81fbace1f
I suspect of some things:

can it be caused because of a mismatch of versions between SDK of the dapp and connector? for instance the can be using sdk 0.88.1 while the wallet is using 0.86.0. on this way a transaction would be created in sdk 0.88.1, but sent through a sdk 0.86.0 using possibly a different sendTransaction implementation?
dryRun returning a different value depending on the contract call. can be something with gasPrice? maxFee? now that gasPrice can change in the network..before we didn't have this factor)
some detail in sdk implementation that is not detecting the operations in some specific cases

blocked by https://github.com/FuelLabs/fuels-ts/issues/2763

when https://github.com/FuelLabs/fuels-ts/pull/2782 is merged, this issue can be implemented


chromatin silencing at telomere -> subtelomere

decreased chromatin silencing at telomere
should merge into
decreased chromatin silencing at subtelomere
and similar

merged FYPO:0003880 into FYPO:0004604
renamed other terms
edit file: 1406da4f08419332f5edf314d3b32c69f3d7202f
release: e78a27623827f37533246f6b61131aff7b85e8ed


Ethereum Generator: Command generation improvements

In this PR we improve the code generator to support more functions from the contracts.
Support bool and uint*
For each of the params, we need to find out which function should be used
to parse a string argument to a type expected by the contract's Go
binding.
If any of the method's (function's) input params type is not associated with
any parsing function the function is not callable as a command (see
default path in the switch statement). This caused a lot of
functions not to be exposed in the generated Ethereum CLI commands.
In this PR we want to support more types, to have more functions
exposed in the CLI.
We add support of:

bool
uint* with bit size greater or equal to 64

To be able to use strconv.ParseUint we had to introduce the parsingFn as a format for printf function, where %s will be replaced with an argument.
Structs as functions's input params
Some of the contract's methods accept tuples (structures) as arguments.
We want to support such functions and generate ethereum CLI commands for
them.
When a param is a tuple we expect that it's string value in the CLI
execution be provided as a JSON. In the generated code we will parse the
JSON to a Go type defined for the structure.
We add _json suffix to such parameters to be printed in the CLI help
and add some context what is expected to be provided when calling it.
See the generated code in https://github.com/keep-network/keep-core/pull/3253

Exceptional work @nkuba! 🚀


Correct reference to dataShards

Quick fix to get examples/simple-encoder working -- reference to 'data' should be 'dataShards'

Thanks mate!


Questions about the pre-training data format

Hi, I want to continue pre-training based on the pre-trained model in the first stage, and then SFT,and I have some questions.
1, where is the weight of the pre-trained model in the first stage?
2, what is the format of  'path/to/pretrain/data.json', can you give a sample file similar to internvl_1_2_finetune.json?
3, To continue pre-training, what is the minimum size of data required？

mark mark

mark

@czczup

Hello, the pre-training weights from the first stage are essentially the MLP projector, and we will release them shortly. Additionally, the data format for our pre-training is consistent with the SFT stage. For continued pre-training, I recommend using at least 1M or more data.

May I ask when this pre-train weight will be released, is it for all series?
And will the pre-training data especially OCR data be released ? thanks.

Hello, we are planning to release some pre-trained OCR data, but the dataset is quite large, consisting of tens of millions of entries, so it will take some time to organize the files.

May I ask when this pre-train weight will be released, is it for all series? And will the pre-training data especially OCR data be released ? thanks.

Hello, we are planning to release some pre-trained OCR data, but the dataset is quite large, consisting of tens of millions of entries, so it will take some time to organize the files.


Cannot download Instagram video

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm requesting a feature unrelated to a specific site
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Provide a description that is worded well enough to be understood
It happened since 2024.9.13.
Using the latest version yt-dlp.
Error message is as followed:
WARNING: [Instagram] unable to extract shared data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [Instagram] Main webpage is locked behind the login page. Retrying with embed webpage (some metadata might be missing).
[Instagram] CsTwy4FBA5n: Downloading embed webpage
^C^CWARNING: [Instagram] unable to extract additional data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
[debug] Command-line config: ['--cookies-from-browser', 'chrome', 'https://www.instagram.com/reel/CsTwy4FBA5n', '--force-overwrites', '-o', 'output/1070/video.mp4', '-v']
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>[b532a3481] (pip)
[debug] Python 3.11.4 (CPython arm64 64bit) - macOS-14.2.1-arm64-arm-64bit (OpenSSL 3.0.12 24 Oct 2023)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.19.0, brotli-None, certifi-2023.07.22, mutagen-1.47.0, sqlite3-2.6.0, websockets-11.0.3
[Cookies] Extracting cookies from chrome
[debug] Extracting cookies from: "/Users/freedom/Library/Application Support/Google/Chrome/Default/Cookies"
[debug] using find-generic-password to obtain password from OSX keychain
[Cookies] Extracted 1182 cookies from chrome
[debug] cookie version breakdown: {'v10': 1202, 'other': 0, 'unencrypted': 16}
[debug] Proxy map: {'http': 'http://<IP_ADDRESS>:7890', 'https': 'http://<IP_ADDRESS>:7890'}
[debug] Loaded 1855 extractors
[Instagram] Extracting URL: https://www.instagram.com/reel/CsTwy4FBA5n
[Instagram] CsTwy4FBA5n: Downloading video info
WARNING: [Instagram] Video info extraction failed: HTTP Error 401: Unauthorized
[Instagram] CsTwy4FBA5n: Setting up session
[Instagram] CsTwy4FBA5n: Downloading JSON metadata
WARNING: [Instagram] CsTwy4FBA5n: General metadata extraction failed (some metadata might be missing).
[Instagram] CsTwy4FBA5n: Downloading webpage
WARNING: [Instagram] unable to extract shared data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
WARNING: [Instagram] Main webpage is locked behind the login page. Retrying with embed webpage (some metadata might be missing).
[Instagram] CsTwy4FBA5n: Downloading embed webpage
WARNING: [Instagram] unable to extract additional data; please report this issue on  https://github.com/yt-dlp/yt-dlp/issues?q= , filling out the appropriate issue template. Confirm you are on the latest version using  yt-dlp -U
ERROR: [Instagram] CsTwy4FBA5n: Requested content is not available, rate-limit reached or login required. Use --cookies, --cookies-from-browser, --username and --password, --netrc-cmd, or --netrc (instagram) to provide account credentials
  File "/Users/freedom/miniconda3/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 710, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/freedom/miniconda3/lib/python3.11/site-packages/yt_dlp/extractor/instagram.py", line 456, in _real_extract
    self.raise_login_required('Requested content is not available, rate-limit reached or login required')
  File "/Users/freedom/miniconda3/lib/python3.11/site-packages/yt_dlp/extractor/common.py", line 1170, in raise_login_required
    raise ExtractorError(msg, expected=True)

don't pass --cookies-from-browser; it looks like your account has been flagged for suspicious activity

Correct, I use --cookies instead, it is ok now. Thank you for supporting:-)

Correct, I use --cookies instead, it is ok now. Thank you for supporting:-)

ERROR: [Ixigua]<PHONE_NUMBER>58712786: Failed to get SSR_HYDRATED_DATA;


Route pass-through/bypass/skip/move-to-next/re-route

What problem does this feature solve?
Our site has top-level sections, for example /apples and /bananas. The list of sections can be non-trivial in length.
If we simply use a route like /:fruit?, then something like /contact would match, which we obviously don't want.
To solve this, we're using a regex route like /:fruit(apples|bananas)?, but this is very messy when there are a lot of sections.
Using nested routes might work, but it'd require things to be ordered in a specific way, and make it more difficult to import routes from multiple application "modules".
When I first saw the next callback in the documentation, I mistakenly believed that it would be similar to middleware and allow you to perform some checks on the currently-matched route, and then call next to continue matching other routes, but unfortunately it looks like that is not the case. 😞
This would have been a good/clean solution for this case. Is there a possibility that something like this would be considered?
What does the proposed API look like?
For a breaking change, maybe re-purposing next to mean "ignore this route, continue routing".
For a non-breaking change, maybe adding another value to next, like next(CONTINUE), or an additional callback.

This is unfortunately not possible because route matching is synchronous.
You can also generate static routes at runtime (which can be async)
const fruits = await getFruits() // ['banana', 'apple']
const frutRoutes = fruits.map(fruit => ({ path: '/' + fruit, component: FruitView }))

@posva,
Thanks for your reply. Perhaps I was unclear, but I'm not suggesting support for asynchronous route matching (we already have an array of section names), but instead a way of telling vue-router to continue or restart matching.
For example:
const sectionNames = ["apple", "banana"];

{
    name: "home",
    path: "/:section?",
    component: FruitView,
    beforeEnter: (to, _from, next) => {
        if (sectionNames.includes(to.params.section)) {
            return true; // Route match/confirm.
        }

        next(CONTINUE); // No match, continue to next route.
    }
},

it's not possible because resolving happens before navigation guards and cannot be intercepted. Generate an array of static routes, specially if you have the array at build time, it's the proper way to do it.


What server I should to use?

Could you provide example of any server for sse?
It would be perfect if you will show how to integrate it with alfred https://github.com/rknell/alfred

So you mean using FeatureHub for SSE on a web server?

No I would like to get work SSE on my own. I am using https://github.com/rknell/alfred


🐛 BUG:

Which Cloudflare product(s) does this pertain to?
Workers for Platforms
What version(s) of the tool(s) are you using?
wrangler 2.0
What version of Node are you using?
3.1.0
What operating system are you using?
windows
Describe the Bug
influxdb-client-js-1.33.2.zip 12.zip
Please provide a link to a minimal reproduction
No response
Please provide any relevant error logs
drive1 (1).txt

Hi @shellsperCloud1 👋 Could you clarify what specific issue with Wrangler you're running into?

We're going to close this issue as we haven't heard back from you. Please feel free to reopen if you're still experiencing the bug and can clarify the specific issue with Wrangler you're running into


Unable to show banner on apk release version

I find an app cannot show banner on apk release version, but it works fine on testing version in Android emulator. When I tried to build an app to production version and run the code on testing mode again. The following message will appear on debug console.
"Native: tried calling AdMobFree.config, but the AdMobFree plugin is not installed.
Install the AdMobFree plugin: 'ionic cordova plugin add cordova-plugin-admob-free'"
I have to run >npm install --save @ionic-native/admob-free in order to let the message above disappeared. Please help. Thank you.

Hi, is your issue resolved? I am having same problem, please help, thank you!

I am using Ionic 2 and my problem is solved, just add Admob library and function to app.component.ts not to home.ts.
Thank to this topic: https://forum.ionicframework.com/t/ionic-doesnt-work-with-admob/96985/3

My problem is resolved. I just finally tried to reinstall all AdMobFree, Android Studio, NPM, Cordova, and Ionic. The function can work no matter it is located in home.ts / app.component.ts. It should work after updating all software's version. Hope it can help. Thank you.


hotfix: Bump sp1 version in validating public inputs example

Hotfix: bump sp1 version in validating public inputs example
Description
Fixes the version in the validating public input example.
To Test

To test this on the holesky network you will need a holesky eth wallet. For reference follow the example readme here.
cd examples/validating-public-input/
In examples/validating-public-input/contracts create a .env with the parameters for the holesky testnet.

For Sp1:

Generate the sp1 proof: make generate_sp1_fibonacci_proof
Send the proof to testnet: make submit_fibonacci_sp1_proof  KEYSTORE_PATH=<LOCAL_KEYSTORE_PATH>
Deploy the validating contract make deploy_fibonacci_validator KEYSTORE_PATH=<LOCAL_KEYSTORE_PATH>
make verify_sp1_batch_inclusion FIBONACCI_VALIDATOR_ADDRESS=<FIBONACCI_VALIDATOR_ADDRESS> DATA_FILE_NAME=<DATA_FILE_NAME>.  Where FIBONACCI_VALIDATOR_ADDRESS is the address of the deployed validator contract and DATA_FILE_NAME the name of the file where the aligned verification data was saved (including the extension .json).
If successful you should see:

...
root                    <ROOT_HASH>
status                  1 (success)
transactionHash         <TX_HASH>
...

For Risc0:

Generate the risc0 proof: make generate_risc0_fibonacci_proof
Send the proof to testnet: make submit_fibonacci_risc0_proof  KEYSTORE_PATH=<LOCAL_KEYSTORE_PATH>
Deploy the validating contract make deploy_fibonacci_validator KEYSTORE_PATH=<LOCAL_KEYSTORE_PATH>
make verify_risc0_batch_inclusion FIBONACCI_VALIDATOR_ADDRESS=<FIBONACCI_VALIDATOR_ADDRESS> DATA_FILE_NAME=<DATA_FILE_NAME>.  Where FIBONACCI_VALIDATOR_ADDRESS is the address of the deployed validator contract and DATA_FILE_NAME the name of the file where the aligned verification data was saved (including the extension .json).
If successful you should see:

...
root                    <ROOT_HASH>
status                  1 (success)
transactionHash         <TX_HASH>
...

Type of change
Please delete options that are not relevant.

[x] Hot Fix

Checklist

[ ] “Hotfix” to testnet, everything else to staging
[ ] Linked to Github Issue
[ ] This change depends on code or research by an external entity

[ ] Acknowledgements were updated to give credit

[ ] Unit tests added
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.

[ ] This change is an Optimization

[ ] Benchmarks added/run

[ ] Has a known issue

Link to the open issue addressing it

[ ] If your PR changes the Operator compatibility (Ex: Upgrade prover versions)

[ ] This PR adds compatibility for operator for both versions and do not change batcher/docs/examples
[ ] This PR updates batcher and docs/examples to the newer version. This requires the operator are already updated to be compatible

It works. We should lower the max_fee or making it a parameter because it is way too high (0.1ether).

Addressed!


fix(table): pagination, toolbar toolips; (card): tooltip

Summary
Changed approach to previous issues:

#3682 Card tooltip - add throttle to scroll event listener (initial render of a tooltip caused micro-scrolling event, therefore tooltip closed itself, throttle for 10ms prevents this)
#3278 Pagination in table - decrease padding slightly between 608px and 500px instead of hiding items per page block
Carbon tooltips in table - change direction from top to bottom in table toolbar (due to overflow: hidden)

Change List (commits, features, bugs, etc)

Card tooltip: add throttling for 10ms from lodash
Table pagination: update css styles
Tooltips in table toolbar: add props to change direction to bottom

Acceptance Test (how to verify the PR)
Card tooltip

Go to this story
Resize the window vertically in order to display vertical scrollbar
Click on card or subtitle to show tooltip
Scroll windows vertically
Verify that tooltip was removed after the scrolling event

Table pagination

Go to this story
Resize the window horizontally till 500px
Verify that icon buttons to change pages do not overflow container

Table toolbar tooltips

Go to this story
Set hasFilter: true, hasSearch: true, hasAggregations: true or any other button icons in tooltip
Hover over icons in toolbar
Verify that tooltips have direction bottom

Regression Test (how to make sure this PR doesn't break old functionality)

tests here

Things to look for during review

[ ] Make sure all references to iot or bx class prefix is using the prefix variable
[ ] (React) All major areas have a data-testid attribute. New test ids should have test written to ensure they are not changed or removed.
[ ] UI should be checked in RTL mode to ensure the proper handling of layout and text.
[ ] All strings should be translatable.
[ ] The code should pass a11y scans (The storybook a11y knob should show no violations). New components should have a11y test written.
[ ] Unit test should be written and should have a coverage of 90% or higher in all areas.
[ ] All components should be passing visual regression test. For new styles or components either a visual regression test should be written for all permutations or the base image updated.
[ ] Changes or new components should either write new or update existing documentation.
[ ] PR should link and close out an existing issue

This is the acceptance test that does not work as stated.
Card tooltip

Go to this story
Resize the window vertically in order to display vertical scrollbar
Click on card or subtitle to show tooltip
Scroll windows vertically
Verify that tooltip was removed after the scrolling event

@davidicus Thanks for commenting on this! I replaced throttling with setTimeout, it seems to do the trick. Also, I added cleanup functions for timers and event listeners.


Update CIS 194 links - multiple links by year

Topics covered are different by year, and therefore listed them separately

Sorry mate, this is a no-go. We link Spring '13 because of what it covers and the point of the repository is to be selective and precise about what we point people to.

np

@ffos Appreciate your forbearance. Might change in the future, but cis194 Spring '13 is already a rough go and later implementations are even harder for beginners (more intermediate-material-focused).
If my book were free, I'd just link that instead.


Sending data from kafka topic to multiple endpoint URLs

hey Lenny,
I finally got this connector up and running. My problem before was the mis/match kafka version. I used 1.0.0 instead of 1.1.0. So upgrading to 1.1.0 solved my problem.
Could this connector send data to multiple endpoint URLs, or it can only send data to one URL?

Thalia-Long
It might be better to run a separate connector for each URL.
Error handling might be simpler that way.

lenny

Thanks for your response.


System.IndexOutOfRangeException parsing markdown with invalid HTML

Hello,
I originally logged this an issue with Markdown Edit https://github.com/mike-ward/Markdown-Edit/issues/159, but the crash is with CommonMark.NET. The following code snippet quite handily reproduces the crash using the latest version of CommonMark.NET
string source;
string output;
CommonMarkSettings settings;

settings = CommonMarkSettings.Default.Clone();
settings.AdditionalFeatures = CommonMarkAdditionalFeatures.All;

source = "Title\r\n=====\r\n\r\nSome HTML with\r\na line break<";

output = CommonMarkConverter.Convert(source, settings);

Console.WriteLine(output);

Basically, that floating < character at the end of the source is causing the crash. I experienced it when pasting HTML into a Markdown Edit window then deleting the tags as I converted it from HTML to markdown manually.
CommonMark.CommonMarkException: An error occurred during inline parsing. ---> System.IndexOutOfRangeException: Index was outside the bounds of the array.
   at CommonMark.Parser.Scanner.scan_autolink_uri(String s, Int32 pos, Int32 sourceLength) in E:\OneDrive\Programming\CommonMark\CommonMark\Parser\Scanner.cs:line 15
   at CommonMark.Parser.InlineMethods.handle_pointy_brace(Subject subj) in E:\OneDrive\Programming\CommonMark\CommonMark\Parser\InlineMethods.cs:line 851
   at CommonMark.Parser.InlineMethods.ParseInline(Subject subj, Func`2[] parsers, Char[] specialCharacters) in E:\OneDrive\Programming\CommonMark\CommonMark\Parser\InlineMethods.cs:line 989
   at CommonMark.Parser.InlineMethods.parse_inlines(Subject subj, Func`2[] parsers, Char[] specialCharacters) in E:\OneDrive\Programming\CommonMark\CommonMark\Parser\InlineMethods.cs:line 1023
   at CommonMark.Parser.BlockMethods.ProcessInlines(Block block, DocumentData data, CommonMarkSettings settings) in E:\OneDrive\Programming\CommonMark\CommonMark\Parser\BlockMethods.cs:line 300
   at CommonMark.CommonMarkConverter.ProcessStage2(Block document, CommonMarkSettings settings) in E:\OneDrive\Programming\CommonMark\CommonMark\CommonMarkConverter.cs:line 168
   --- End of inner exception stack trace ---
   at CommonMark.CommonMarkConverter.ProcessStage2(Block document, CommonMarkSettings settings) in E:\OneDrive\Programming\CommonMark\CommonMark\CommonMarkConverter.cs:line 181
   at CommonMark.CommonMarkConverter.Convert(TextReader source, TextWriter target, CommonMarkSettings settings) in E:\OneDrive\Programming\CommonMark\CommonMark\CommonMarkConverter.cs:line 294
   at CommonMark.CommonMarkConverter.Convert(String source, CommonMarkSettings settings) in E:\OneDrive\Programming\CommonMark\CommonMark\CommonMarkConverter.cs:line 314
   at ScratchPad.Program.Main(String[] args) in C:\Checkout\cyotek\source\Misc\ScratchPad\Program.cs:line 101

Regards;
Richard Moss

I will fix this as soon as possible, hopefully tomorrow.

v0.13.0 has been released to nuget with the fix

Thank you!
On Mon, Jul 4, 2016 at 8:19 AM +0100, "Kārlis Gaņģis"<EMAIL_ADDRESS>wrote:
v0.13.0 has been released to nuget with the fix
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I updated my test project from v0.12 to v0.13 but I get exactly the same crash. The DLL has today's timestamp so would appear to be the correct version (hard to tell when all the version info is 0.1 :)) but unfortunately the issue doesn't seem resolved.
Pre-submit-edit: I checked the decompiled DLL and can confirm that it's missing the commented out line you added from the commit, but it's still very much broken for me.

Ok, how did I manage to do that... That line should not have been commented out - I commented it out to check that the test actually fails without it and then did not put it back...
I will push a new release this evening, this time with more coffee preceding it...

Everyone is human :)

1.13.1 released. Hopefully I didn't mess up again :)

I hate to be the bearer of bad news... but I'm afraid something still isn't right. I grabbed the latest package and ran my test program again, and it still crashed.
I decompiled one of the 13.1 DLL's again and compared it with the commit, and it's missing the  if (pos >= sourceLength) return 0; line still. As I can see that line is correct in the source code my best guess is the updated source wasn't compiled before building the package. The files do have today's timestamp and match the timestamp of your commit, but ... well, it's broken.
Just in case something had gone wrong, I did do a forced reinstall of the package to no avail.
As a virtual messenger I'm glad I can't be shot!

I suspect I know where I didn't do proper rebuild.
Double coffee didn't help, I guess third time's the charm...

OK, 1.13.2 has been pushed to nuget. I do hope the streak of bad luck ends here...

I can confirm the streak of bad luck ends, just upgraded the test project to 13.2 and it works flawlessly. Thanks very much!

@Knagis you should've skipped right to 0.14 :)

I'll post an update tonight. Thanks.


Remove activity files that do not do anything

@optimizely/fullstack-devs
@vraja2 do you know why these files exist? the test-app doesn't build with them there

@vraja2 I dunno why this doesn't build. tests all pass locally >.<

@elliotykim it was added back in the copyright changes that were merged in earlier so that broke the test-app. Going to merge this change in to fix.

actually, it looks like there are other unneeded files that were added as well. If you can take a second pass and ensure all these files are removed, we can merge in later, but I'm going to revert this for now.

Shoot, maybe I clicked "Update branch" and Github added those files. I'll fix this

Abandon this change and instead fix https://github.com/optimizely/android-sdk/commit/ae92208b0709b7a254f7d6102a44eccf0f376cba

These are used for .gradlew test-app:connectedAndroidTest which are our
MOST valuable tests. Please revert.
On Jan 23, 2017 7:54 AM, "Elliot Kim"<EMAIL_ADDRESS>wrote:

Closed #79 https://github.com/optimizely/android-sdk/pull/79.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/optimizely/android-sdk/pull/79#event-933028309, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAOpl6MWan8AIDM8dUXAf4_EHAEfuwo0ks5rVM0ugaJpZM4LpZYz
.

@jophde I don't think that's relevant anymore

actually the test is failing
@vraja2 does the test need to be rewritten for the updated test-app?
is the test automatically run somewhere?

The test automates the UI of those activities. They are relevant.
On Jan 23, 2017 8:43 AM, "Elliot Kim"<EMAIL_ADDRESS>wrote:
actually the test is failing
@vraja2 https://github.com/vraja2 does the test need to be rewritten for
the updated test-app?
is the test automatically run somewhere?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/optimizely/android-sdk/pull/79#issuecomment-274542388,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOpl-Z5sefuN9rkCAeZWaS4llpTVf13ks5rVNjOgaJpZM4LpZYz
.


Please give sample code for text summarization

Hello @datquocnguyen ,
Can you provide a sample code to do text summarization task with this model?
Thank you.

Please see https://github.com/VinAIResearch/BARTpho/issues/2


docs(button): basic button variants and sizes

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.96%. Comparing base (dd8940d) to head (f99297f).
Report is 2 commits behind head on dev.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##              dev      #55   +/-   ##
=======================================
  Coverage   50.96%   50.96%
=======================================
  Files          17       17
  Lines         259      259
  Branches       56       53    -3
=======================================
  Hits          132      132
  Misses        127      127

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


Get the dbObject instance of the PurchaseComplete message

By passing the HTMLText object the text renders properly in the email

Good find, thanks!


A runner exists with the same name

Hi, we've been using this operator for a while now and it usually works great but sometimes the operator gets stuck.
In this case we get the error:

A runner exists with the same name

I'm not sure how it gets into this state, but it seems that when runners are idle for a while the operator seems to try and restart them, and isn't aware that the id is already registered.
Is there a way to mitigate this issue? Could the operator just remove the duplicate runner in this case?

@mattkim which version of the runner image do you use? Where do you see this message?
Maybe it was fixed by https://github.com/evryfs/github-actions-runner/commit/8bb89e3b6e0919d3255b112e5830691225c395f8 ?

We are running v0.9.3, I'll try updating to the latest version and see if that helps, thanks!

Closing due to lack of feedback.


Fix CA1835 code fix screenshot

Fixes https://github.com/dotnet/docs/issues/20820
When I created the screenshot, I didn't notice I was focusing the "expression value is never used" suggested change, instead of the actual CA1835 rule.

I was just told that all the .NET SDK analyzers documentation got moved to this repo. I don't have permission to move issue https://github.com/MicrosoftDocs/visualstudio-docs/issues/5926 to this repo, which still lives in the old visualstudio-docs repo, but if that's not an issue, we can leave it there.

closing and reopening for a fresh build.

The screenshot may need to be updated again if https://github.com/dotnet/roslyn-analyzers/issues/4235 is done.

Can I get a review, @dotnet/docs?


Added a fix for the Computation of the Pause Time % in GC for GCs by not including FGCs

BGCs shouldn't accumulate the FGCs' Pause Times to compute their corresponding % Pause Time in GC.
Also, fixes an error with the wrong index being used for the BGC.
CC: @Maoni0 @cshung

@brianrob, could you please approve and merge this PR? We found a bug in GC Stats related to BGCs. Thanks!


[FEA]Regular expression of rlike can support "$"

I wish Regular expression of rlike can support "$".
For example:
from pyspark.sql.types import StructType,StructField, StringType, IntegerType
data2 = [("abc[123]",""),
    ("xyz[123","Rose")
  ]

schema = StructType([ \
    StructField("firstname",StringType(),True), \
    StructField("middlename",StringType(),True) \
  ])

df = spark.createDataFrame(data=data2,schema=schema)
df.printSchema()
df.show(truncate=False)

simplified_regex_dollar = r"^xyz(.+)3$"
df.withColumn('newcol', expr(f"""   firstname rlike "{simplified_regex_dollar}"    """)).collect()

Not-supported-messages:
    !Expression <RLike> RLIKE(firstname#102, ^xyz(.+)3$) cannot run on GPU because line anchor $ is not supported

Output:
[Row(firstname='abc[123]', middlename='', newcol=False), Row(firstname='xyz[123', middlename='Rose', newcol=True)]

This is a duplicate of https://github.com/NVIDIA/spark-rapids/issues/4533.

Closing this one as duplicate #4533 is now closed.


Age of >18 years

Our current spec says:
    age_max:
        type: number
        description: Upper boundary of age range or equal to age_min for specific age. This field should only be null if age_min is null.

How would I capture the age range of a subject where the ages of the subjects are stratified as in
age > 18 or age <= 18
The current spec says that age_max can't be null, which prevents one possibility - age_min = 18.001, age_max = null
Perhaps age_min = 18.001, age_max = 999 and age_min = 0, age_max = 18 assuming age_unit = 'year'

I am not a big fan of requiring users to enter arbitrary data into fields just to ensure that a query succeeds. So how far can/should we deviate from what you described before? If we assume that NULL in the data indicates an unbounded interval, this could be represented by an additional
OR data_(min|max) IS NULL condition on both sides of the AND in your example.
And while we  are at it: age_* is a closed interval, yes?

And while we are at it: age_* is a closed interval, yes?

Yes,  I would think so...

I am not a big fan of requiring users to enter arbitrary data into fields just to ensure that a query succeeds. So how far can/should we deviate from what you described before? If we assume that NULL in the data indicates an unbounded interval, this could be represented by an additional OR data_(min|max) IS NULL condition on both sides of the AND in your example.

I prefer using NULL as an indicator of an unbounded limit as well, but our current spec doesn't allow this, in fact it explicitly says this is not allowed. Does anyone remember the reason for that, in particular the explicit statement that it is not allowed...

Does anyone remember the reason for that, in particular the explicit statement that it is not allowed...

It is essentially implied by the queries. Because we decided to combine single point age with age intervals into the same fields, that introduces requirements so that this query (for both a single age and for an interval) works:
30 >= age_min && 30 <= age_max

this could be represented by an additional
OR data_(min|max) IS NULL condition on both sides of the AND in your example

I think so.
30 >= age_min && (30 <= age_max || age_max is null)

This needs to ensure that we have the ability to check for null - See #298 (comment)

Separate tests have been created for null fields, so closing this issue.


Types in useAuthState are any

It seems that intellisense can't pick up the right types that are defined in the type definition of useAuthState resulting in the image above.
Am I doing something wrong here or is this some bug?

Okay, it seems it's because firebase doesn't export a named export User.
So instead of User in the type definition, it should be firebase.User

I think this is fixed here: https://github.com/CSFrequency/react-firebase-hooks/pull/86

As spotted by @mdirolf, this is fixed by #86 and will be included in the upcoming v3 release


ScalametaParser: no repeated initial implicit mod

The existing code already checks for repeated modifiers on parameters except for the implicit/using modifier on the first parameter.

combined with #3087.


revise defaults

window
dateRange
arguments in require demographics and dervied functions

remove default from requireSex etc https://github.com/OHDSI/CohortConstructor/blob/0046c78238690c041d3756125785464ccbbe9cfb/R/requireDemographics.R#L124C30-L124C39
But leave defaults for requireDemographics (as user may be only interested in one or two)

no window default and move to top


While Running Streamlit Code, It is throwing "KeyError: 'status'"

I am trying to run the streamlit app but it is throwing the error as attached in the screenshot.. Kindly provide the solution for this.

Please help me out to resolve this..!!

please me too , it doesn't work and I can't find the issue


(#364) Adjust classes from choco-theme update

Description Of Changes
Work on choco-theme has been done to generate utility classes in a more
efficient manner. In this process, a few class names were modified so
that they correlated better with the style they were providing.
This repository has been skimmed for these classes and updated
accordingly. In addition, a few inline styles have been removed and
replaced with a class.
Motivation and Context
This is a general improvement to the way choco-theme generates classes, and also clarifies what a class name might do.
Testing

Linked this repository up with the updated choco-theme
Ensured each page that has a modified class still looks the same

Change Types Made

[ ] Bug fix (non-breaking change)
[x] Feature / Enhancement (non-breaking change)
[ ] Breaking change (fix or feature that could cause existing functionality to change)
[ ] PowerShell code changes.

Related Issue

https://github.com/chocolatey/choco-theme/issues/130
https://github.com/chocolatey/chocolatey.org/issues/112
https://github.com/chocolatey/docs/issues/364
https://github.com/chocolatey/blog/issues/127
https://github.com/chocolatey/home/issues/103

Fixes #364
Change Checklist

[ ] Requires a change to the documentation
[ ] Documentation has been updated
[ ] Tests to cover my changes, have been added
[x] All new and existing tests passed.
[ ] PowerShell v2 compatibility checked.

@st3phhays thanks for getting this updated!


INFO log the config on startup

Is your feature request related to a problem? Please describe.
While debugging #294 with Chris, I kept messing up my configuration. It would be nice to log non-secret configurations so that it's easy to communicate what your env looks like.
Describe the solution you'd like
We should log the DB configuration and enabled features on startup so that it's easier to reproduce/debug issues when they come up.
Describe alternatives you've considered
N/A
Additional context
N/A

@mohnishbasha done!


feat(http-ratelimiting): add a bucket for global rate limits

This closes #650.
I opted to keep the rate limit trait as is for now, even though is_globally_locked is unused.
Additionally, the global rate limiter waits for each request to return before accepting the next request. Since 50/s isn't that many, I wanted to keep it simple. If you disagree, then I would change that.
Lastly, Discord only documents the hardcoded 50 requests per second. Nirn-proxy however seemed to have figured out that big bots have different global rate limits. We discussed this on Discord and decided not to support this due to it not being officially documented.

Let's ignore the failing http tests for now.


Preparing release v1.24.0

Signed-off-by: Ruben Vargas<EMAIL_ADDRESS>

Ready for another review.


Allow to isolate particular sources/sinks only

isolate is mostly useful for DOM isolation, and people often don't want their HTTP requests to be private to one component. Maybe isolate could take a list of driver names as third optional argument?

We already allow isolation by request 'category', I think, but why wouldn't a component want to isolate its http requests? If it wants to give the responses to its parent, it can send it as a stream from itself, to which the parent can most obviously react.

If it wants to give the responses to its parent, it can send it as a stream from itself

But that stream shouldn't be named the same way as child's http source. Quite a non-obvious restriction.

I don't understand why to avoid HTTP isolation. It's only between siblings, not between parent and child. So the parent is able to get all its children requests and responses. I mean what harm can HTTP isolation make for your application?

@Hypnosphi why should it be named the say way? the request/response may not be the intent or action that the parent component wants to react to. The component can build the intent/action stream and then expose it to the parent instead of the parent having to react to the response stream of the requests made by the child.

BTW staltz is right about isolation being between siblings...

This question was already discussed https://github.com/cyclejs/cyclejs/issues/277
I would consider  "partly" isolated component to be  antipattern.

@Hypnosphi I was looking for this exact feature when I came across this discussion. I was assuming this wasn't the current behavior but wanted complete DOM isolation for a child, but for HTTP requests to be triggered by that child (say, a click) and the response to be processed by a parent (say, a full page response).
I was under the assumption that isolation meant all requests and responses were fully isolated to only that component, but as @staltz mentioned isolation is only from siblings. You can see how this works here https://github.com/cyclejs/cyclejs/blob/master/http/src/isolate.ts#L21...
I believe it's something like this: say you have top-level component A (not isolated) which has a child component B (isolated to cycle1) which is a child C (isolated to cycle2) which two children D (isolated to cycle3) and E (isolated to cycle4) so the tree looks like so:

A
| _ B
|   | _ C
|   |   | _ D
|   |   |
|   |   | _ E

Child D's requests will have a _namespace property of ['cycle3'] which continues to get namespaces tacked on by the parent components until it's ['cycle3', 'cycle2', 'cycle1'], then finally the request is passed to A, which does nothing additional (because it wasn't isolated) and passes it to the actual driver. When the response comes back, A is going to see it because it has no isolation. Then because each child component was isolated, it's going to check if the request contained its own namespace (https://github.com/cyclejs/cyclejs/blob/master/http/src/isolate.ts#L9-L10), all of which will except for E. When E checks for 'cycle4' and it doesn't find it, it will ignore that response.
Although I'm glad this is the functionality, it seems like a pretty fair assumption to make that components will be completely isolated. I think maybe the documentation for each cycle module should have a section called isolate (assuming that module supports isolation) which describes the behavior of that particular module's isolation.

@kevinmctigue this issue with the way of http (and dom) isolation was discussed pretty much. In DOM all the events are scoped exclusively to isolated component, in http-like drivers parents parents have access to child components. It was desided to leave that way, there are aslo issue about to allow DOM source ( using options) to access child isolated events from parent.

I think maybe the documentation for each cycle module should have a section called isolate (assuming that module supports isolation) which describes the behavior of that particular module's isolation.

I agree.
Also, this issue is a bit stale as it seems it won't happen. I'll create another issue so we document how isolation works for each driver.


Parenthetical format in simulated gene trees

I am using the code below to simulate gene trees under a species tree with branches in numbers of generations and different population sizes per branch.
The simulated gene trees I get have a strange parenthetical format:
((D:40000.0)minus2:11184.0,(((C:20000.0)minus3:25551.0,(((A:10000.0)minus4:11096.0,(B:10000.0)minus4:11096.0):8904.0)minus3:25551.0):14449.0)minus2:11184.0);
Code:
using PhyloNetworks
using PhyloCoalSimulations
using PhyloPlots

tre = readTopology("(((A:10000,B:10000):20000,C:20000):40000,D:40000);")
plot(tre, showedgenumber=true, showedgelength=true);

## Population sizez (order by edge number in tre)
pop = [20000,20000,20000,20000,100000,20000]

## We now need to build a dictionary
Ne = Dict(i => pop[i] for i in 1:length(pop));

## Need to add the population at the root
rootedgenumber = PhyloCoalSimulations.get_rootedgenumber(tre)
push!(Ne, rootedgenumber => 20000);

## To chech that we built our dictionary correctly
using DataFrames, RCall
plot(tre, tipoffset=0.1, showedgelength=true, edgelabelcolor="red4",
          edgelabel=DataFrame(n=[e.number for e in tre.edge],
                              l=[Ne[e.number] for e in tre.edge]));
R"text"(x=1, y=2.5, Ne[rootedgenumber], adj=1, col="red4");
R"mtext"("red: Ne values", side=1, line=-1.5, col="red4");
R"mtext"("black: edge lengths", side=1, line=-0.5);

## Simulating the gene trees:
genetree_gen = simulatecoalescent(tre,1000000,1, Ne; nodemapping=true);
writeMultiTopology(genetree_gen, "genetrees-pop.tre")

The output gene trees would look more standard if you used the option nodemapping=false.  The node mapping creates degree-2 node in gene trees, and gives them a name that corresponds to the name of a speciation or reticulation in the species network. More specifically, in each gene tree, a degree-2 node is created each time that the gene tree passes through a node in the species tree. In the gene tree, this degree-2 node is named after the corresponding node in the species network. There are more explanations and a figure in the documentation here.
That's a great option if you want to vary things across species in your simulation. The node mapping tells you which species each gene lineage evolved into. The manual gives a example here of how this mapping can be used to vary the substitution rate of each gene lineage according to the species that this lineage is in.
So:

a pair of parentheses around a taxon name, like (D:40000.0), corresponds a degree-2 node at the time when the ancestor of D passed through the parent node of D in the species network. That was 40,000 generations prior to individual D.
weird names like minus2 in (D:40000.0)minus2 are default names given to nodes in the species network. They are different from the node numbers. If the species network had names for all nodes, then these names are used. That's the most robust thing to do, because these ancestral taxon names can be saved in the newick format for the species network. If the species network is lacking names for some nodes, then default names are assigned. For a node with number -2, the default name is minus2.

If you don't want to use these degree-2 nodes, then we can turn off the node mapping option. Or you can save these simulated gene trees with the mapping information (just in case, who knows) and then clean them up (remove the degree-2 nodes) later: see here.
If you have some use for these degree-2 mapping nodes, then the manual shows here how to assign nicer names to nodes in the species network. And then I suggest that the modified network (with these nicer node names) is saved to a file, so that the correspondance can be made later between the ancestral species names and the degree-2 nodes (named after ancestral species) in the gene trees.


Update operation defs for 0.18.0

Currently getting the error:
ReferenceError: comment_payout_beneficiaries is not defined
    at Object.<anonymous> (steem-js/src/auth/serializer/src/operations.js:346:9)

Because of this change:

On condenser's old steem-js lib, I simply reverted those lines. Is there a more appropriate fix?

Thank you for your PR. IMO the issue is just that "comment_payout_beneficiaries" variable is not defined. It seem the generated code don't include these variables and we have to type it manually the way its done here: https://github.com/steemit/steem-js/blob/master/src/auth/serializer/src/operations.js#L36-L38 for others methods using static_variant. So i would just add like this: const comment_payout_beneficiaries = types.void; next to this lines. The test pass but i'm not 100% it would be broadcasted.


Misleading / unhelpful warning tooltip on 4.0 -> 4.1 upgrade

Godot version
v4.1.stable.official [970459615]  < Steam
System information
Godot v4.1.stable - Debian GNU/Linux trixie/sid trixie - Vulkan (Mobile) - dedicated NVIDIA GeForce GTX 1060 (nvidia) - Intel(R) Core(TM) i5-8400 CPU @ 2.80GHz (6 Threads)
Issue description
After being upgraded to 4.1 via Steam (before any release announcement even! The speed!), I saw the warning marker against my game in the project manager, and noticed that the tooltip wasn't all that helpful:

It seems to think that the old version number is a deprecated feature? :/
Steps to reproduce

Have a project from Godot 4.0
Upgrade Godot to 4.1
Open project manager

Alternatively:

Create a new project in 4.1
Quit Godot
Edit the project.godot's config/features to read 4.0 instead of 4.1
Open project manager

Minimal reproduction project
default.zip

It seems to think that the old version number is a deprecated feature? :/

The wording is unfortunate for this specific case, but that's just the nature of how the version check is implemented. I guess we could somehow handle it as a special case.

Yeah, IMO the version number does seem to be a special case, separate from the explicit list of features in use.
My expectation would be that there wouldn't be any warning at all if it's only the version number which differs.
Or else the warning would explicitly state that the project is from an earlier version of the engine, so the user is informed to be on the lookout for things which are not necessarily deprecated, but may not work quite as they used to (though the confirmation dialog when opening the project may also serve that purpose?).

Here is my proposed fix: https://github.com/godotengine/godot/pull/79118


Location of config-files

I suggest moving config files to $XDG_CONFIG_HOME/streamdeck, or similar, instead of putting the config directly in $HOME.
What with the streamdeck requiring multiple resources, it would be nice with a common place to put it all.

Maybe in Fyne's App.Storage().RootURI() perhaps - that picks based on OS, based on XDG_CONFIG_HOME for linux

Resolved in v1.0.1


Feature/models filtering

Add filtering for the rest of the views. Includes little refactoring and formatting.
Refs: LIIK-40, LIIK-41, LIIK-42, LIIK-43, LIIK-44, LIIK-45

Codecov Report

Merging #21 into master will increase coverage by 0.3%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master      #21     +/-   ##
=========================================
+ Coverage    96.4%   96.71%   +0.3%
=========================================
  Files          33       33
  Lines        1864     2038    +174
  Branches       38       38
=========================================
+ Hits         1797     1971    +174
  Misses         61       61
  Partials        6        6

Impacted Files
Coverage Δ

traffic_control/tests/test_traffic_light_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_mount_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_road_marking_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_signpost_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_barrier_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/models/traffic_light.py
98.01% <100%> (ø)
:arrow_up:

traffic_control/views.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_traffic_sign_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/test_base_api.py
100% <100%> (ø)
:arrow_up:

traffic_control/tests/factories.py
96.22% <100%> (+5.75%)
:arrow_up:

... and 1 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 34dde3e...e1ee144. Read the comment docs.


Enable ConcurrentValue checking as part of Concurrency mode.

Drop the separate flag guarding this checking.

@swift-ci please test


src/main/java is hardcoded into .classpath

I am trying to use this plugin to generate eclipse project for android app however my project has a different directory structure. I do have all the source files inside the src directory with no main/java inside.
I do have overridden main.java source set in the build.gradle script, my src path is incuded in the resulting claspath however there's always redundant src/main/java added to the classpath.
It would be nice if this plugin would verify if the hardcoded path is valid before adding it or if it provided enough configuration.
Edit: gradle eclipse plugin is looking at top level sourceSets. Thus I can trick gradle eclipse plugin to add my non-standard path by adding it to the top level main.java source set.
When handling android projects sourceSets is not a top level thing but it's nested inside the android {} block. So adding both sourceSets one at the top level and the other one within the android block is somewhat redundant I think. android specific sourceSet has some more types besides src and resources, so that's another argument why to prefer android specific sourceSets configuration for android projects.
So it would be natural to take the java.srcDirs from each source set defined under android.sourceSets instead of hardcoding src/main/java .
Is this doable?
Until this is not properly addressed here is a dirty way on how I got it to work...
file.beforeMerged { classpath ->
classpath.entries.add(new org.gradle.plugins.ide.eclipse.model.SourceFolder("src", "bin"))
}
file.whenMerged { classpath ->
classpath.entries.removeAll { it.path.endsWith("src/main/java") }
}

Thanks for raising the issue, and great to see that there's a workaround!
You're right of course, this plug-in relies on projects following convention when it comes to layout.
To help me understand your use-case, what is the reasoning for having a non-standard directory structure?

Well, I would say my use case is not very special.
We are using eclipse and android ADT to build the app. The release builds are still being made with android-sdk 23 targetting API 21 formally although there might be a few API 22 calls.
I am trying to move out of Android ADT for eclipse as I am having issues with it each time I'm installing android SDK update or android platform update and similar.
Just for the convenience I like to have up to date tools and SDKs installed so I can also build other apps occassionally.
So as a first step I have added gradle based build configuration to our project which will be used side by side with the original android ADT build configuration. If my team mate developer will also find this gradle based build config usefull we might consider switching to it in the future.
If nothing else then we are at least partially prepared for using up to date SDK and build tools.
So the project layout is oldskool with flat folders
src
res
assets

You probably already know, but just in case: ADT is no longer officially supported.  See http://android-developers.blogspot.com/2016/11/support-ended-for-eclipse-android.html for details.
This plug-in is intended for Gradle projects that use a standard layout (as produced by Android Studio).  I'm sorry but your request is beyond the scope of this project.
I'm glad that you've got a workaround.  You may want to consider moving to a supported tool chain.

Another workaround:
eclipse {
  classpath {
    file.whenMerged {
      classpath -> classpath.entries.each {
        if (it.kind == "src" && it.path.equals("src/main/java")) it.path = "src"
      }
    }
  }
}


MM-40222 - show the gov references in license page when gov license

Summary
This PR adds the logic for checking for the isGovSku value in the licenses, and when found will show the Gov references in the licensing page.
Ticket Link
https://mattermost.atlassian.net/browse/MM-40222
Related Pull Requests

Has server changes (https://github.com/mattermost/mattermost-server/pull/19100)

Screenshots

Release Note
NONE

Looks great! the border radius of the license details should be rounder as seen above in the figma design @pablovelezvidal

Is the body text for the license details in open sans?

@pablovelezvidal it seems that uploading regular licenses is also labeling them as Gov as well, can you double check this @stevemudie ?

@marianunez
I'm seeing the same. Any/all licenses read Gov.

@marianunez
I'm seeing the same. Any/all licenses read Gov.

addressing it now.

@marianunez @stevemudie Fixed! good catch! thanks!
@annelieseklein your suggestions were also addressed!


Unregister a device

I would love to see the example extended to include how to unregister a device.  Not really sure where to put it, how to trigger it and so on.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8d9bfb2a-2e5d-d505-337f-919251a85de3
Version Independent ID: 695bff64-ae87-8c0f-5616-20b67893fdf0
Content: Remote Notifications with Firebase Cloud Messaging - Xamarin
Content Source: docs/android/data-cloud/google-messaging/remote-notifications-with-fcm.md
Service: unspecified
GitHub Login: @mgmclemore
Microsoft Alias: mamcle

Hi @nhwilly, thank you for the suggestion!  I'll make note to have this topic expanded to cover unregistering from FCM.


Make Camera2D's editor helper code only be compiled on editor builds

They're pointless on non-tool builds, so no reason to have them there.

I'm quite that with the recent changes, this does not break compatibility anymore.

Thanks!

Cherry-picked for 3.2.4.


Can't access active_admin.css

I have Rails 3.0.9 with active_admin 0.2.2 installed. I use sass with the config for heroku (http://devcenter.heroku.com/articles/using-sass).
When access /admin/ url, the login page shows but without css. In the log I see:
ActionController::RoutingError (No route matches "/stylesheets/admin/active_admin.css")
It seems that active_admin doesn't generate the css. There are no active_admin css in /tmp/stylesheets/

Yea, the gist is no good for me. I have this same problem but in development, not on Heroku (not even trying to use Heroku)

Rails 3.0.11, ActiveAdmin 0.3.4, and there is simply no "active_admin" CSS. Everything is all mucked up looking...


Removed a section

Removed a section as Message Analyzer is not supported and discontinued.
Problem: https://github.com/MicrosoftDocs/OfficeDocs-Enterprise/issues/622

@kelleyvice-msft - PR is ready for final review based on issue #622.
Thank you.
cc: @kenwith

Merging per @kelleyvice-msft


Default plural formula should probably be the one for English

In most programs the default language is English. Should this be reflected in the formula of the nilTranslator instead of the no-plural one?
If that is not desirable, because changing a default may have an impact on other programs, then adding an API to allow changing it would be nice.

Here is a workaround I implemented using a default FileSystem implementation that returns an empty json: https://github.com/uyuni-project/uyuni-tools/blob/main/shared/l10n/utils/defaultfs.go


Line length for paragraph text doesn't align with standard

Describe the bug
Line lengths for paragraph text doesn't align with standard
To Reproduce
Steps to reproduce the behavior:

Go to 'https://cfpb.github.io/design-system/foundation/fonts#guidelines'
Inspect paragraph text using Google inspector
Paragraph text is going full width (859px wide in the section description area, 815 in the variation group section)
Our standard on the website is 670px for paragraph text

Expected behavior

670px width for paragraph text

Screenshots

Current state
Expected behavior

@nataliafitzgerald Is this fixed? It looks like it to me. Can we close?