Latte: Fixed exponential backtracking

Is the detector improving or this was just not dealt with before?

This is one of the very few regexes that aren't checked by our detector. This was found using a CodeQL query.

I'll assume it's something weird about how markup templating works or something that it's "special"... so much of our regex are generated at run-time but since the tests are running at run-time we just hook into the FINAL prepared regex and run them thru the checker.

Yeah, we just check the regexes of all grammars. Markup templating also needs one additional regex to detect the embedded language and that regex isn't checked.
One way around that might be to override the exec/test methods in RegExp.prototype to run the detector on all patterns used to tokenize Prism's test cases.


[build-script] XCTest CI builds on OS X

What's in this pull request?

Fix the OS X XCTest CI preset:

Installation of XCTest isn't supported on OS X. After all, the library is meant to be used on Linux, and not intended to be used or installed on OS X. Therefore, no longer pass an INSTALL_PATH to the xcodebuild invocation on OS X.
xcodebuild expects swift-stdlib-tool to be present in the same directory as the SWIFT_EXEC it's given. Re-use some shellscript from the toolchain installation script to copy the stdlib-tool substitute.

This allows the XCTest CI to work on all platforms, and should unblock https://github.com/apple/swift-corelibs-xctest/pull/77. That pull request will make it much easier for new contributors to start working on swift-corelibs-xctest.
Resolved bug number: None

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci test

Looks like some Swift package tests are failing due to something related to SwiftPM. Doesn't seem related to this PR. @kostiakoval @aciidb0mb3r are you aware of any ongoing CI issues in SwiftPM?

Also, do we have ci tests in GitHub for swiftpm? How did these breakages escape at least this sort of testing?
Sent from my iPhone

On Apr 2, 2016, at 2:07 PM, Brian Gesiak<EMAIL_ADDRESS>wrote:
Looks like some Swift package tests are failing due to something related to SwiftPM. Doesn't seem related to this PR. @kostiakoval @aciidb0mb3r are you aware of any ongoing CI issues in SwiftPM?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

@gottesmm I believe the problem is that package tests are only run for apple/swift CI. The preset used by SwiftPM CI doesn't include them. That may be reasonable--they are quite expensive.
/cc @shahmishal for CI wisdom.

@modocache We are trying to move towards a model where all commits have some sort of CI testing. SwiftPM is no exception to this rule.
If the tests are too expensive then the tests should be sped up or a reasonable subset should be provided that can be run.

@gottesmm I can definitely sympathize with your argument, especially because I've accidentally broken the package tests before in #1704. Again, I'll leave it up to @shahmishal to determine what's the best approach here.
@shahmishal Any thoughts on @gribozavr's concerns about derived data above? I'm excited to get this merged, it'll be a big improvement in XCTest's CI.
Also, if someone could try the tests again, that'd be much appreciated! :bow:

@modocache I agree with @gribozavr we should specify derived data dir, so we can reliably clean the workspace.
What is the error are you seeing with DSTROOT="${XCTEST_BUILD_DIR}"?

@swift-ci Please test

What is the error are you seeing with DSTROOT="${XCTEST_BUILD_DIR}"?

Oops, sorry, the error I was seeing must have been due to INSTALL_PATH and the other parameters I removed. I added DSTROOT and everything works fine. Thanks!!
I believe this is now good to merge? The CI passed last time it was run.

@shahmishal Friendly ping! I think this is a big improvement for corelibs-xctest development.

@swift-ci Please test and merge

Just my luck: two CI failures and both appear unrelated:

OS X #782: The XCTAssert failure fixed in https://github.com/apple/swift/commit/2d50e813478f5962b6942a7eb581f4d70e2ab3eb.
Linux #1003: ClangModules/attr-swift_name_renaming.swift test failure, seems unrelated to this.

Excellent!! Thanks!


Make Tree:from_dom() public

I'd like to be able to parse svgdom::Document myself, modify it programmatically (as it's easier than modifying a usvg::Tree, and then convert into a Tree.
Currently I have to do this:
let document = Document::from_str(string).unwrap();
// modify document
let opt = resvg::Options::default();
let tree = Tree::from_str(&document.to_string(), &opt.usvg).unwrap();

If Tree::from_dom() was public, I would be able to get rid of extra serialization and deserialization.

svgdom not used for many versions now.

svgdom not used for many versions now.

Thanks, I see now. I was using an outdated version, then checked the repo and saw that from_dom() is still private, but didn't notice that it was using svgtree::Document. Maybe because the comment still says that svgdom::Document is used.
Still, wouldn't it make sense to make that function public?

No. It's an internal implementation.


[BUG] Incorrect typehintings for HTTPClient

Describe the bug.
The .pyi for the client in interactions/api/http has incorrect typehintings for HTTPClient, and most of the HTTP methods will not be typehinted because of it.
This is likely due to the fact that in the .pyi file, the HTTPClient does not inherit from the request classes, unlike in the regular file. This means Python has no clue that HTTPClient even has those methods.
While the HTTP client is an underscored attribute, it is used so often as of right now that it is worth having accurate typehintings. It's also nice when developing the library.
List the steps.
Do something like this in pretty much any modern-day text editor with type checking support. I'll be using Pylance since I use Visual Studio Code:
client = interactions.Client(...)

async def func():
    await client._http.get_member(GUILD_ID, MEMBER_ID)

What you expected.
For the get_member function to be typehinted correctly.
What you saw.
Pylance reports:
Cannot access member "get_member" for type "HTTPClient"
  Member "get_member" is unknown

What version of the library did you use?
unstable
Version specification
No response
Code of Conduct

[X] I agree to follow the contribution requirements.

Judging off of the poll ran going over the type hinting debacle, I'm not sure if this issue should be considered with fixing .pyi files, or ditching them in favor of typing.Protocol as voted out by the community.

Judging off of the poll ran going over the type hinting debacle, I'm not sure if this issue should be considered with fixing .pyi files, or ditching them in favor of typing.Protocol as voted out by the community.

For a minor issue that can be fixed pretty simply, I'd argue the former makes sense for now. The latter will require quite a bit of work and will need to be addressed separately, but just because that was voted for doesn't mean we immediately drop support for .pyi files.


Fix linode-cli linodes create operation

In testing an unrelated CLI change, I noticed this:
$ linode-cli linodes create --help
linode-cli linodes create
Linode Create

Arguments:
  --root_pass: This will set the root user's password on the newly-created Linode.
  --stackscript_id: A StackScript ID that will cause the referenced StackScript to be run duringdeployment of this Linode.
  --authorized_keys: A list of public SSH keys that will be automatically appendedto the root user's `~/.
  --authorized_users: A list of usernames.
  --stackscript_data: This field is required only if the StackScript being deployed requires inputdata from the User for successful completion.
  --booted: This field defaults to `true` if the Linode is created with an Image or from a Backup.
  --image: An Image ID to deploy the Disk from.

That's not good - the linodes create operation is missing many
parameters, including type and region that are required.  After some
digging, I determined that it was because the allOf in
/linodes/instances.post.requestBody.content.application/json.schema
section was not nested properly; the schema should be all of the
references LinodeRequest and the given properties.  Having an
allOf alongside properties isn't really correct (although it's
apparently not invalid).
Built off this spec, the CLI now see the full request:
$ python -m linodecli linodes create --help
linode-cli linodes create
Linode Create

Arguments:
  --region: (required) The [Region](/docs/api/regions/#regions-list) where the Linode will be located.
  --type: (required) The [Linode Type](/docs/api/linode-types/#types-list) of the Linode you are creating.
  --interfaces.label: The name of this interface.
  --root_pass: This will set the root user's password on the newly-created Linode.
  --backup_id: A Backup ID from another Linode's available backups.
  --group: A deprecated property denoting a group label for this Linode.
  --stackscript_id: A StackScript ID that will cause the referenced StackScript to be run duringdeployment of this Linode.
  --authorized_keys: A list of public SSH keys that will be automatically appendedto the root user's `~/.
  --stackscript_data: This field is required only if the StackScript being deployed requires inputdata from the User for successful completion.
  --image: An Image ID to deploy the Disk from.
  --tags: An array of tags applied to this object.
  --backups_enabled: If this field is set to `true`, the created Linode will automatically beenrolled in the Linode Backup service.
  --label: The Linode's label is for display purposes only.
  --interfaces.purpose: The type of interface.
  --private_ip: If true, the created Linode will have private networking enabled and assigned a private IPv4 address.
  --swap_size: When deploying from an Image, this field is optional, otherwise it is ignored.
  --interfaces.ipam_address: This Network Interface's private IP address in Classless Inter-Domain Routing (CIDR) notation.
  --authorized_users: A list of usernames.
  --booted: This field defaults to `true` if the Linode is created with an Image or from a Backup.

I could be persuaded that the CLI's spec parser is incorrectly merging
the allOf with the same-level properties, but I do not believe this
is the case.  The OpenAPI Specification is light on details here, but
all examples using allOf do not include sibling nodes like this.

I pointed this at master since the CLI's linode create operation is presently broken - I could point it back at development if you prefer.


[RFC - do not merge]Add organisations displayed first to details hash

Currently, there are three types of organisations stored in the publishing-api database:

lead organisations
organisations
supporting organisations
Where: organisations = lead organisations + supporting organisations

GOV.UK publishing apps do not send to the publishing-api the same organisation
types consistently. We are going to modify all applications to send at least
the organisations content ids in the links hash and the "organisations that should be displayed first" in the details hash.
Frontend applications will then decide what and how to display the organisations by reading the details and links hashes of a piece of content.
We don't know yet how to call the new field but we know that:

organisations field in the links hash will contain all the item organisations
the new field will contain the most important organisations for a piece of content, those that should be displayed first.
organisations that are are not mentioned in the new field but are present in the organisations field in the links hash will be displayed last

We thought about the following names:

most_important_organisations
primary_organisations
highlighted_organisations
organisations_that_should_be_displayed_first
firstly_displayed_organisations
prominently_displayed_organisations
lead_organisations
first_displayed_organisations
organisations_displayed_first
protruded_organisations

What do you like the most? Any other you would like to suggest?
ticket: https://trello.com/c/us3MI1n9/602-fix-organisation-tagging
related to: https://github.com/alphagov/whitehall/pull/2585
cc: @boffbowsh @tijmenb

I think I prefer primary_organisations, or maybe lead_organisations should be retained as it matches Whitehall's UI.

Thanks @boffbowsh. I agree that it's important to keep UI and data naming in sync.
However in this case it may be better to give it an (overly) specific name, because people could interpret "lead_organisations" as having some sort of real meaning in the context of government.
We also want to use the same mechanism in policy-publisher, where the UI is different:

In short, lead_organisation could be misinterpreted and lead to people using it in different incompatible ways. Something like organisations_displayed_first is easier to change to something more broad, while the reverse is not true. Thoughts?

Could just do organisation_display_order and list all orgs?

In short, lead_organisation could be misinterpreted and lead to people using it in different incompatible ways. Something like organisations_displayed_first is easier to change to something more broad, while the reverse is not true. Thoughts?

Sounds like we need some user research.

How about "Boaty McBoatface" "emphasised organisations".
It implies the distinction is purely presentational, but isn't overly specific, and allows us to change the things on the frontend without changing the publishers. I don't think anyone would infer anything about the structure of government from this.
I think the key improvement is moving it to the details hash. The links hash should describe semantics of how things relate to each other, and lead organisation doesn't have a single meaning at the moment. 👍

Please try out options with users.

This is an internal name @fofr, not for displaying in admin UIs

emphasised_organisations also works well imo

Just thinking a little bit more about this -
I'm assuming this means content tagger will have to know about details, which it doesn't already? Is this correct? Could we do implement this in such a way that works for all link types?
Regarding user research: I'm aware there has been some research already on publishing workflow, so that may be relevant to helping ensure editors understand the effects of tagging.
I think our (my?) assumptions right now are:

We're not aware of a user need to order things in an arbitrary way at publishing time.
The lead/supporting distinction arose because the way we present long lists of organisations looks very odd if you show all of them alphabetically, e.g. the 2012 olympics had lots of organisations tagged to it that people are unlikely to associate with the olympics
We don't know if there is any end user benefit to this
We don't have a lot of evidence these organisation links are even useful to end users at all...

Content Tagger won't need to know about details, only the Publishing apps (mainly Whitehall) would need to know about that in order to add the organisations in the correct order.

Ah, right forgot organisations is not in content tagger.

organisations is in content-tagger, but setting the emphasised_organisations is not its responsibility.
To recap (a consensus):

We won't change the UIs
We use emphasised_organisations in details hash to allow the frontend to display the things in a certain way
We will use this pattern for policies (policy-publisher) and whitehall content.

Makes sense?

👍  to summary

👍  thank you all. Updating.

This is an internal name @fofr, not for displaying in admin UIs
Internal names have a habit of making themselves visible, usually for the sake of consistency ;)

Renamed and added deprecation messages.

Should lead_organisations still be a required field on some of the formats?
eg https://github.com/alphagov/govuk-content-schemas/blob/master/formats/detailed_guide/publisher/links.json#L6


test runner example doesn't work

$ cargo test --package test-runner-example --lib -- tests::test_runner --exact --nocapture
    Finished test [unoptimized + debuginfo] target(s) in 1.08s
     Running unittests (target/debug/deps/test_runner_example-7743848064d85300)

running 1 test

thread 'tests::test_runner' has overflowed its stack
fatal runtime error: stack overflow
error: test failed, to rerun pass '-p test-runner-example --lib'

Caused by:
  process didn't exit successfully: `/Users/xiliangchen/projects/substrate/target/debug/deps/test_runner_example-7743848064d85300 'tests::test_runner' --exact --nocapture` (signal: 6, SIGABRT: process abort signal)

This should be checked by CI as well.

substrate test runner is deprecated


Why would the CrashPlan backup not exist?

Hi, not on OS X but I don't think that'd make a difference. I'm experiencing the issue below:
.# locate cp.properties
/usr/local/crashplan/cache/42/cp.properties
.# ./check_crashplan.sh
CRITICAL - the CrashPlan backup you pointed to does not exist! - /usr/local/crashplan/
I also tried /usr/local/crashplan/cache and /usr/local/crashplan/cache/42/ but both failed.
But it's there, is it not?

I'm not using OSX either and was able to get this working. Does nagios/nrpe not have permissions to this folder?

Does it need r/w perms? I have 755 (root is owner).

I modified this script to work with CrashPlan pro.
https://github.com/jordantrizz/nagios-additional-plugins/blob/master/check_crashplan_pro

Thanks Jordan - still doesn't work though.
.# cd /usr/local/crashplan
.# ls
bin    conf                 ~custom  install.vars  lang  libjniwrap64.so libjtux64.so libmd564.so  log   upgrade
cache  CrashPlanEngine.pid  doc      jniwrap.lic   lib   libjniwrap.so libjtux.so libmd5.so    skin
.# /usr/lib64/nagios/plugins/check_crashplan_pro
CRITICAL - the CrashPlan backup you pointed to does not exist! - /usr/local/crashplan

Anyone?

Shoot me an email (on my profile) and we can try to link up and test things!


add support for "Always Use HTTPS" zone settings

Description
Adds support for getting and setting "Always Use HTTPS" zone setting with new methods:

GetAlwaysUseHTTPS
SetAlwaysUseHTTPS

I'm trying to fully automate my zone provisioning process, this was a part that I had to do manually.
I tried to follow the style of authenticated_origin_pulls.go (another simple on/off zone setting) as much as possible.
Open to file+struct naming suggestions of course -- this one is a doozy lol.
Has your change been tested?
I've already used this code for provisioning a new domain with "Always Use HTTPS" enabled.
Types of changes
What sort of change does your code introduce/modify?

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have added tests to cover my changes.
[X] All new and existing tests passed.
[X] This change is using publicly documented (api.cloudflare.com or developers.cloudflare.com) and stable APIs.

this is already possible using the existing zone settings. if you're after an example of how to use it, check out the zone settings in the Terraform provider repository.

this is already possible using the existing zone settings. if you're after an example of how to use it, check out the zone settings in the Terraform provider repository.

I'm not using terraform but I see the usage of ZoneSingleSetting here and can work with that for what I need.
I guess I thought this wasn't possible because the name of the function ZoneSingleSetting isn't exactly obvious that it allows you to set a "setting" for a zone. I expected the experience for individual settings to be consistent with the "Authenticated Origin Pulls" (calling client.SetAuthenticatedOriginPullsStatus(ctx, zone, true) ).


Add dotnet/symreader back to source-build: runtime (ILCompiler, crossgen2, dotnet-pgo) now depends on it

symreader was removed by https://github.com/dotnet/source-build/pull/518 because it was no longer necessary, but there are some new dependencies on it now that are showing up as prebuilts.
  <AnnotatedUsage Id="Microsoft.DiaSymReader" Version="1.3.0" File="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/artifacts/obj/coreclr/ILCompiler.ReadyToRun/project.assets.json" Project="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/" />
  <AnnotatedUsage Id="Microsoft.DiaSymReader" Version="1.3.0" File="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/artifacts/obj/coreclr/ILCompiler.TypeSystem.ReadyToRun/project.assets.json" IsDirectDependency="true" Project="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/" />
  <AnnotatedUsage Id="Microsoft.DiaSymReader" Version="1.3.0" File="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/artifacts/obj/coreclr/crossgen2/project.assets.json" Project="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/" />
  <AnnotatedUsage Id="Microsoft.DiaSymReader" Version="1.3.0" File="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/artifacts/obj/coreclr/dotnet-pgo/project.assets.json" Project="src/runtime.bf456654f9a4f9a86c15d9d50095ff29cde5f0a4/" />

Maybe there's some other way, but it does seem very reasonable to me that crossgen and PGO-related projects would be using symreader libraries.

Note from triage: we aren't building ILCompiler or dotnet-pgo now, but crossgen2 is used during the dotnet/runtime build, so we need to do this.

we aren't building ILCompiler or dotnet-pgo now, but crossgen2 is used

I think this is wrong (as far as ILCompiler goes), it's contradicted by the project references crossgen2 takes on ILCompiler libraries (which is where the dependency actually comes from):
  <ItemGroup>
    <ProjectReference Include="..\ILCompiler.DependencyAnalysisFramework\ILCompiler.DependencyAnalysisFramework.csproj" />
    <ProjectReference Include="..\ILCompiler.TypeSystem.ReadyToRun\ILCompiler.TypeSystem.ReadyToRun.csproj" />
    <ProjectReference Include="..\ILCompiler.ReadyToRun\ILCompiler.ReadyToRun.csproj" />
  </ItemGroup>

https://github.com/dotnet/runtime/blob/v5.0.0-rc.1.20451.14/src/coreclr/src/tools/aot/crossgen2/crossgen2.csproj#L26-L30

Added by https://github.com/dotnet/source-build/pull/1841.


[All] (Centralized deployment) Note Mac supported

Note Mac support based on Centralized Deployment article

Docs Build status updates of commit 1717322:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/publish/centralized-deployment.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Addresses issue https://github.com/OfficeDev/office-js-docs-pr/issues/895


WordCounter for "Anna-Maria" returns 3

The WordCounter produces unexpected results...
require DIR . '/vendor/autoload.php';
use \Org_Heigl\TextStatistics as Ts;
$tsg = new Ts\TextStatisticsGenerator();
$tsg->add('word_count', Ts\Service\WordCounterFactory::getCalculator());
print_r($tsg->calculate(new Ts\Text("Anna-Maria")));
... returns a [word_count] => 3, but I'd expect 2.

Thanks for noticing! I'm digging into it!

You will need to call composer update org_heigl/hyphenator to fix this issue.
The reason was an error in the tokenizer of the hyphenator-library that was fixed with https://github.com/heiglandreas/Org_Heigl_Hyphenator/releases/tag/v2.0.6

BTW: There's now a test that checks for Anna-Maria: https://github.com/heiglandreas/TextStatistics/blob/0.1.0/tests/Calculator/WordCounterTest.php#L54

Super - Problem gelöst!
Besten Dank
Mit freundlichem Gruß,
Daniel Berlin

Am 20.03.2017 um 21:19 schrieb Andreas Heigl<EMAIL_ADDRESS>Closed #2 https://github.com/heiglandreas/TextStatistics/issues/2.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/heiglandreas/TextStatistics/issues/2#event-1007435172, or mute the thread https://github.com/notifications/unsubscribe-auth/ABKp4_jgx41dMeFCgeFT4G2HXPRGpOmuks5rnt9lgaJpZM4Mi7C5.


Proper error wrapping

context:

While working on this we realized we're using errors.Wrap all over the place on owncloud.go. GetMD(). I believe we should wrap on the lowest layers, in the context of this PR that would be getPath() and unwrapping on the top layers to assert on the underlying error type.

more context:
https://github.com/cs3org/reva/pull/1064

We should also take the opportunity to use the std new error library:
https://golang.org/pkg/errors/

@labkode YES PLEASE. I was on talks about that with Jorn and Vincent yesterday. Dave's Chenney spoke his mind in moving away from gh/pkg/errors to the stdlib.

@labkode does it make sense to add technical debt label to the repo, to capture all this?

Done


time difference between upload and publish

How long is the time difference between the key upload time and the time they are published?
Or more precise:
If keys are uploaded 23:00:00-23:59:59, will they be added to the 23:00-hour-file (assume more than 140 keys exists)
or is it possible they will be added to the 0:00-hour-file, if they come later for example after 23:55:00.
Then the time difference would be 5 min.
Reason for the question:
On 2020-07-10-07 the hour-00 file looks like all keys have been uploaded on 2020-07-09
45 keys from 18:00 to 19:00
65 keys from 22:00 to 23:00
60 keys form 00:00 to 01:00  << this one looks as if it has been uploaded before 0:00 but added after 0:00
the 60 keys are:
5x6 08.07
5x8 07.07 5x8 06.07 5x8 05.07
5x5 04.07
5x3 03.07
5x1 02.07 5x1 01.07 5x1 30.06 5x1 29.06 5x1 28.06 5x1 27.06

key
risk
date
period

...83d
1
26.06.
2655216

...1d1
1
26.06.
2655216

...601
1
26.06.
2655216

...224
1
26.06.
2655216

...f7b
1
26.06.
2655216

...ccf
1
27.06.
2655360

...1ac
1
27.06.
2655360

...798
1
27.06.
2655360

...827
1
27.06.
2655360

...0c2
1
27.06.
2655360

...295
1
28.06.
2655504

...204
1
28.06.
2655504

...c71
1
28.06.
2655504

...ebe
1
28.06.
2655504

...c8b
1
28.06.
2655504

...b02
1
29.06.
2655648

...d13
1
29.06.
2655648

...278
1
29.06.
2655648

...79a
1
29.06.
2655648

...e62
1
29.06.
2655648

...c06
1
30.06.
2655792

...97d
1
30.06.
2655792

...9d2
1
30.06.
2655792

...534
1
30.06.
2655792

...d79
1
30.06.
2655792

...432
1
01.07.
2655936

...a9e
1
01.07.
2655936

...873
1
01.07.
2655936

...38c
1
01.07.
2655936

...4e4
1
01.07.
2655936

...643
1
02.07.
2656080

...020
1
02.07.
2656080

...604
1
02.07.
2656080

...94b
1
02.07.
2656080

...ff8
1
02.07.
2656080

...32c
3
03.07.
2656224

...045
3
03.07.
2656224

...739
3
03.07.
2656224

...d07
3
03.07.
2656224

...dd3
3
03.07.
2656224

...c7b
5
04.07.
2656368

...087
5
04.07.
2656368

...86c
5
04.07.
2656368

...fc4
5
04.07.
2656368

...1c4
5
04.07.
2656368

...e37
6
08.07.
2656944

...085
6
08.07.
2656944

...8c3
6
08.07.
2656944

...082
6
08.07.
2656944

...30c
6
08.07.
2656944

...85f
8
05.07.
2656512

...806
8
05.07.
2656512

...1f4
8
05.07.
2656512

...a3e
8
05.07.
2656512

...7c7
8
05.07.
2656512

...eaf
8
06.07.
2656656

...117
8
06.07.
2656656

...6d4
8
06.07.
2656656

...4b3
8
06.07.
2656656

...301
8
06.07.
2656656

...0c0
8
07.07.
2656800

...716
8
07.07.
2656800

...ffc
8
07.07.
2656800

...6e0
8
07.07.
2656800

...0ac
8
07.07.
2656800

Hello @Tho-Mat ,
The diagnosis keys need to be expired for at least two hours, before they can be distributed. This means that if the key has been submitted within two hours after the expiry date, it cannot be published immediately. Therefore a distribution timestamp is calculated, which is either the submission timestamp, or, if the submission timestamp is within two hours after the expiry date, the expiry date plus two hours. This ensures compliance with the specification from Google and Apple.

@EvgeniiSkrebtcov
Sorry I do not understand what you mean with "expiry date". (Isn't this 14days?)
Could you make an example for each case with submitting time and date in UTC?

Hello @Tho-Mat,
I referred to an expired Diagnosis key in the sense that it is no longer used by the mobile device.
For each submitted key Distribution service calculates the earliest point in time at which a key can be distributed according to the following rule:

Before a key is allowed to be distributed, it must be expired for a configured amount of time (currently 2 hours).

See ProdDiagnosisKeyBundler.java for implementation details.

a still valid Diagnosis key will be embargoed and not included it in any batches, until after it has expired + 2 hours to handle skew
Diagnosis keys that already expired at least 2 hours back, will be picked up by the next Distribution service run, that is scheduled to run every hour
  schedule: '5 * * * *'
and added to the hourly package (considering that there are enough keys to create a package, otherwise shifting policy applies)

Hello, @Tho-Mat
Kindly let me know in case further clarification needed.


Firefox Nightly (Deb) 132.0a1 Crashes On New Power Automate Instant Button Flow

Describe the Bug with repro steps

New Instant Cloud Flow
Name it
Select 'Manually trigger a flow'
Tap to continue ... and crash

What type of Logic App Is this happening in?
Power Automate
Which operating system are you using?
Linux
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Firefox Nightly 132.0a1 (Mozilla Firefox Debian Package | mozilla-deb - 1.0)
(just updated)
I ran this on a completely fresh profile with zero modifications and on my existing main profile ... it crashed both.
Additional context
I have also logged this in the Mozilla forum, here: https://support.mozilla.org/en-US/questions/1465140
This works fine in the following browsers with zero modifications to the browsers:

Ungoogled Chromium - Absolutely fine, no add-ons
Firefox mozilla-deb - 1.0 130.0.1 (64-bit) - Absolutely fine, no add-ons

I have no screenshots of a crash and no flow to send details of.
I am forced to use the new designer by way of the creation flow, there is no way to use the old designer for creating a new flow. At least, I don't know what URL / JS I'd need to manipulate to get that.

This isn't a piece of UI we own here but I can make sure this gets sent to the right people to take a look at on the power automate side. It's not uncommon for nightly firefox builds to have issues like this though. In the past, the Mozilla team has been good about addressing them when they're reported.

Hello @influential-eliot, Power Automate side here!
+1 to what @hartra344 said. For this kind of issue, the Power Automate team would usually be the ones to take a look.
However, in this case, I was not able to repro with Firefox release build 130.0.1 x64 on Windows 11, which would be our primary focus for support. If you're still seeing this issue when Firefox 132 hits public release, feel free to ping me in this thread and we'll carry forward the issue with the Power Automate team. Thanks!

Cheers, both ... but I would imagine that if it crashes in PA, it will crash in LA. Which I will now test. 😩
But I wasn't really looking for help ... but it's a shame to read the above. Either way, good luck.
Also, @ek68794998 ... I mean ... I cannot use Windows ... so ... if this bug filters down to mainline Firefox on Linux will that basically be it for me and my (actual) job?
I logged it because:

... I presume that when you are testing the GUI (used in both, hence the next point) you test not just with current versions of browsers, but with future versions like DEV and Canary for EDGE
... there was literally an option to log PA UI issues here ... so ... I have

... but ... Whatever works. 🤷 😅

Also, @ek68794998 ... I mean ... I cannot use Windows ... so ... if this bug filters down to mainline Firefox on Linux will that basically be it for me and my (actual) job?

No, it just happens in this case that I can't easily test out Firefox's public release with a Linux machine. But if we somehow get a Linux-specific bug on stable release, we'll figure that out at the point in time when it happens.

... there was literally an option to log PA UI issues here ... so ... I have

Yeah, no problem with the issue being posted. Would rather be aware of it than not, even if we're not looking deeper right now. Since we do share a good chunk of code between LA and PA, a problem in this code base might appear in both—but it also might not. 😊

edit Also, where do we publicly raise issues with the V3 designer for Power Automate if not here?

Generally, you would go through either the official support channel or through the Power Automate community forum.

Cheers, @ek68794998 👍
Oh and ... 'the official support channel' isn't public and the Power Automate community forum is a different resource ( which currently doesn't work - have sent all issue to italent, yep 😉 ) not well equipped (like this).
Thanks anyway. 👍

@influential-eliot feel free to post any issues here if you're in doubt. If it's not shared code or caused by our code, we can always do what we just did and make sure it does get put in front of the right people.

Many thanks!


Fixed some issues I had when using SassyJSON

I ran into a couple of issues when using SassyJSON, the commits in this request fixed these issues.

Regarding the 'deprecation warning: Unescaped multiline' commit, I have to admit I'm unsure why a new line was initially used here, but an empty quote here has yielded me perfect to JSON results.


Logic bug in WriteStreamSubscriberImpl skipping onNext when NOT done.

In trying to use WriteStreamSubscriber I found my items were not being delivered.  On debug they were being skipped in onNext with "if (!done.get()) return"  It seems this is the wrong way around and we want to skip and return immediately if done is true.  I will add a pull request.

Fixed by #159

@cescoffier you can close it :-) (I don't have permissions)


Add en-sync-check.sh for daily-check

Description
Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

Summary by CodeRabbit

New Features

Introduced a synchronization check tool for Klaytn nodes, enabling users to verify the sync status between local and public nodes.

@jay-kim-kor I think it would be worth to check the comments from @coderabbitai. Do you think they are irrelevant?

@jay-kim-kor I think it would be worth to check the comments from @coderabbitai. Do you think they are irrelevant?

I've checked that comments but, I didn't apply they recommendation.
I think it's not irrelevant. But I just don't agree to it's need double quote in echo.


Addresses #39

Converts the embeded EXIF orientation into UIImageOrientation to be used when creating the resulting UIImage

This is awesome! Thanks for taking the time to build this. Also of note, iOS 8 looks to have added
kCGImagePropertyOrientationDown
kCGImagePropertyOrientationDownMirrored
kCGImagePropertyOrientationLeft
kCGImagePropertyOrientationLeftMirrored
kCGImagePropertyOrientationRight
kCGImagePropertyOrientationRightMirrored
kCGImagePropertyOrientationUp
kCGImagePropertyOrientationUpMirrored
Could you check if those are defined and if not define them? Then use those instead of one two three, etc.? You can see this in CGImageProperties.h

Those constants are defined a CF_ENUM.  Do you have any recommendations on how to determine whether those exist already?  A bit of googling has me stumped.   Ideally we can find something better than checking the SDK version, but from my research it's not looking promising.
As a fallback, i could add an inline comment for each case:
case 1: /*kCGImagePropertyOrientationUp*/

Hmm, I think you're right, I'm not sure of a good way either. Lets go with the comments for now. Thanks for doing this!

Awesome thanks for your work on this!


make docker simpler

assets of frontend is built by the last RUN,
so we just need to host the built assets.
in node, python is required, so use python simple server to do this.

That makes more sense, thanks.
Also welcome to Maputnik @morlay and thanks for contributing.

Since we don't use the dockerfile in appveyor I'm assuming appveyor was having a bad day. Merging anyway, hopefully it'll pass... else I'll look into the issue once merged.

There is something wrong with appveyor in general: https://ci.appveyor.com/project/lukasmartinelli/editor/builds/20372093 (@orangemug could you cancel the test there?)
There was recently an image update, hopefully https://github.com/maputnik/editor/pull/470/commits/7004259867741dc793c88b6066449ea1124eba08 helps.

...could you cancel the test there?

Unfortunately I don't seem to be able to, there's no UI for me to do that... which is odd. Maybe because @lukasmartinelli is the owner of the CI env still. It's odd though because I can trigger a rebuild


onChange function prop BasePicker not called

Environment

Tech
Version

material-ui-pickers
2.0.1

material-ui
3.5.1

React
16.6.3

Browser
Chrome

Peer library
date-fns

Steps to reproduce

Create a static picker using <BasePicker> and <Calendar> as per the documentation example
Add a function to the onChange prop to the BasePicker
Function is not called

Expected behavior

onChange prop on BasePicker should call the function passed with the changed date as variable.
Actual behavior

The function inside the onChange handler is not called. The onChange handler on the Calendar component is fired as you can see in the sandbox below.
Live example

Check here for a live example:
https://codesandbox.io/s/xp347yn3pw

On change will be called if autoOk is passed or handleAccept was called https://codesandbox.io/s/k205qokwxr


heavy CPU usage, even with empty project.

Godot version: godot-bin-mono 3.2.1 (this one: https://downloads.tuxfamily.org/godotengine/3.2.1/mono/Godot_v3.2.1-stable_mono_x11_64.zip)
OS/device including version: archlinux, updated. Running Xorg.
Hardware: i5-3570 CPU (3.40GHz), nvidia geforce 460 GTS with nvidia proprietary drivers.
Issue description: heavy CPU usage. Around 318% with vsync enabled. If i remove vsync and limit FPS (debug settings) to 60, i get around 60% of cpu usage.
Steps to reproduce: open new project, creates node2d, run.
Minimal reproduction project:
stress_test.zip

Does it happens with non-mono version too?

If you turn on the update spinner in editor settings, does it keep spinning? (that's often a good indicator to check if the editor constantly redraws because something pings the renderer)

Sorry, i had a setup problem with my config.
As it's an old PC (my Geforce 460GTX is 10 years old), the driver support has been removed from the main nvidia driver, and even from the community driver. I have to use AUR (arch user repo) for finding the driver which recognize my card.
That said, you have all guessed that after my last reboot (about three per year, i use hibernation and it's rock steady now), i had lost my nvidia driver after a kernel upgrade.
And … and i was then using the horrible MESA driver instead of nvidia DRM module.
I'm sorry for the waste of time. Thanks to Zylann with his magic spinner, it helped me to understand that something was wrong with my setup. As i'm only typing text, reading doc and surfing most of the time, the 2D FB support was enough and i hadn't noticed something fishy.
Thank for your help.


PPVS Subscriber UPS element display doesn't provide tree with sequences and sequence items

The PPVS Subscriber UI only shows top level elements and specific extracted values.
dcmQTreePy has full tree display capability, so incorporating that type of tree view would be nice to have

closed with #66


🐛 Fix unexpected behaviour in the saturate_cast function

In the saturate_cast function, I observed an anomaly in the behaviour when converting a negative integer to a unsigned integer:
saturate_cast<u64>(i32(-1)) =<PHONE_NUMBER>3709551615

The new implementation, adapted from GCC (https://github.com/gcc-mirror/gcc/blob/af64af69c3cc85dbe00c520651a54850bf5cadc1/libstdc%2B%2B-v3/include/bits/sat_arith.h#L107-L142), gives
saturate_cast<u64>(i32(-1)) = 0

@ThibFrgsGmz, thank you very much for finding an issue and making the effort to provide a fix!
This repo is released under the Boost Software License 1.0, a permissive license. We cannot take any pull requests from code that is derived from a GPL project.
Thanks again for your effort! We will have to fix this without looking at this PR any further.


npm ci fails when repository contains lockfile version 1

https://github.com/ArtiomTr/jest-coverage-report-action/issues/11#issuecomment-801921789
New version of npm generates lockfile v2, so npm ci fails, because of incompatible lockfile versions.

@dadayama Would you like to work on this issue?

I think that solution would be not to use npm ci, because currently jest-coverage-report-action uses it to prevent cache issues.
Similar functionality will be just to remove node_modules folder if it exists and run default npm install command

I wish I could handle this one as well.
I understand the problem and your suggestion.
I am thinking of modifying it as follows, what do you think?
// getRawCoverage.js
...

// NOTE: Use the `recursuve` option to delete files recursively. In addition, use this option to avoid throwing exceptions, so you don't need to use the `try ~ catch` block.
fs.rmdirSync('node_modules', { recursive: true });
await exec('npm i');

...

The solution looks good, thank you!


SystemJs Builder - Error calculate canonical name

We are trying to build our solution with SystemJs Builder without success.
RxJs version: 5.0.0-beta.6
We have problems and we had already some help from SystemJs guy: https://github.com/systemjs/builder/issues/587#issuecomment-220835913
We have developed a simple project to test build process:
https://github.com/meriturva/SystemJsBuilder-587
Just clone it and run gulp build and you will see problems related to RxJs package:
error { [Error: Error tracing ./node_modules/rxjs/Subject.js at file:///H:/Projects/github.com/SystemJsBuilder-587/node_modules/rxjs/Subject.js
        Loading<EMAIL_ADDRESS>        Loading app/main.js
        Unable to calculate canonical name to bundle file:///H:/Projects/github.com/SystemJsBuilder-587/node_modules/rxjs/Observable. Ensure that this module sits within the baseURL or a wildcard path config.]
  originalErr: [Error: Unable to calculate canonical name to bundle file:///H:/Projects/github.com/SystemJsBuilder-587/node_modules/rxjs/Observable. Ensure that this module sits within the baseURL or a wildcard path config.] }

We also would like to use umd version but we need WebSocket class that it is not already included https://github.com/ReactiveX/rxjs/issues/1506
Any ideas how to solve it?
Thanks.
Diego

Related PR #2381

I don't think this is any longer an issue.


ListBox strange behavior

I have a list of items, the parts of which I set IsVisible = false, depending on the current item type.
Items are displayed in the Popup.
If I display them in a ListBox, then the popup behaves strangely. Some of the elements are not visible at all

I tried to expose:
ScrollViewer.HorizontalScrollBarVisibility = "Disabled"
ScrollViewer.VerticalScrollBarVisibility = "Disabled"

but it did not help
If I display them in an ItemControl then everything works as expected:

When I set
ScrollViewer.HorizontalScrollBarVisibility = "Disabled"
ScrollViewer.VerticalScrollBarVisibility = "Disabled"

Hi @RomanSoloweow
I saw the same with DataGrid. Instead of hiding the items I ended up adding / removing them on demand, eg. via a filtered Collection (I use DynamicData). I've heared that ItemsRepeater handles this in a better way, but I cannot promise.
Happy coding
Tim


[AVM Module Issue]: db-for-postgre-sql private deployment no publicNetworkAccess param

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Bug
Module Name
avm/res/db-for-postgre-sql/flexible-server
(Optional) Module Version
0.2.0
Description
Unable to deploy a private server, get a conflict config error if vnet integration and private dns configured in the template. publicNetworkAccess param on latest api https://learn.microsoft.com/en-us/azure/templates/microsoft.dbforpostgresql/flexibleservers?pivots=deployment-language-bicep not available for the avm module https://github.com/Azure/bicep-registry-modules/blob/main/avm/res/db-for-postgre-sql/flexible-server/main.bicep

(Optional) Correlation Id
No response

The currently deployed API version does not support public endpoints. We will need to upgrade the API version to make this feature available.

@arnoldna is there any ETA for this? My team is looking forward to the API update.


Feature request: Add event object sent from Cloudwatch Alarms

Use case
AWS has added a trigger for Cloudwatch Alarms that can trigger lambdas. Thus, it would be helpful to include this in the available data classes.
More on the event object can be found here: https://docs.aws.amazon.com/AmazonCloudWatch/latest/monitoring/AlarmThatSendsEmail.html#alarms-and-actions
Solution/User Experience
The required data classes are added to AWS Powertools
Alternative solutions
No response
Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (Python) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Java, TypeScript, and .NET

Hi @jaydm26! Thanks for opening this issue and remembering that we need to add this new event source data class.
Do you want to send a PR to implement this? We'd love to have your first contribution here 🚀.

Hi @leandrodamascena
I would be happy to. Might take some time to implement it.
If someone takes it up before I take it up, go ahead. I will update the thread again when I begin.

I have opened a pull request (still on draft) for this, as I've already implemented something similar in my team. I'd be more than happy to hear your feedback/comments/opinions on the changes! 😃

@par6n awesome work! Feel free to publish the PR and I'll be happy to review and help you merge your contribution!

Closing as completed!


Changed system console schema to use full key for settings

This allows us to mix settings from different sections of the config on a single system console page and it'll also be used for https://mattermost.atlassian.net/browse/MM-9849.
@jwilander Are there tests in place for plugin settings or can you point me to a test plugin that I can use that has system console settings? I want to make sure I didn't break anything with those by making this change

There is tests/components/admin_console/custom_plugin_settings.test.jsx which seem to be passing

I've switched it to Elias so we can get it in on time. This is mostly Jesús's code, so I'll still grab him for a followup review once he's back


How can i run this without docker?

Hi there, for my final project at university i would like to create a GUI for vqgan+clip, i would like to run it locally on my machine without using docker, and connect cuda to a python application rather than jupyter?, is this possible?
Thanks

I won't recommend doing it without docker. First, you can search how to run GUI apps with docker and then make your Dockerfile importing my container.
But, if you want to run without docker, I recommend you to read the Dockerfile and follow the instruction to install the dependencies.
https://github.com/sborquez/VQGAN_CLIP_docker/blob/4b082382feead9a61afb451b6a932c41a6b1cdea/Dockerfile#L1-L15
Make sure to use the correct version of CUDA.
It is possible to create a new GUI app without Jupyter. Just use this function as inspiration 😉:
https://github.com/sborquez/VQGAN_CLIP_docker/blob/4b082382feead9a61afb451b6a932c41a6b1cdea/generate_images.py#L228-L330


Discussions on Pagination hasNext() and hasPrevious()

If current page > 1 , then it should show "previous page".If page count  > current page , then it should show "next page".
I think this is more reasonable. Such as google search paging style.

Since you brought this up on the forums also, I'm closing this and discussion can continue there.

ok


Copying live to master

Docs Build status updates of commit 3c45113:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Move evaluation to its own structure

Right now most of the evaluation happens inside the lir and lir::eval modules and it is completely stateless.
However in the future we might require to store state (for call-by-need evaluation or interact with the outside world). So it would be better to have a single Machine structure to take care of all the evaluation.
For this we need to:

[x] Create a new top-level module machine.
[x] Create an empty (for now) Machine structure.
[x] Move the lir::Term::step, lir::Term::step_inplace, lir::Term::evaluate and all the functions inside lir::eval to the machine module and change them to be methods of Machine.
[ ] Add a machine::evaluate public function that takes a Term, creates a new Machine and evaluates the term using the new Machine::evaluate method.
[x] Plug this machine::evaluate function into the run method at the top of the library.

For example the lir::Term::step method looks like this
impl Term {
    ...
    fn step(mut self) -> (bool, Term) { ... }
}

after these changes it should look like this
impl Machine {
    ...
    fn step(&mut self, mut Term) -> (bool, Term) { ... }
}

I will extract my Machine struct from #47, remove unnecessary things will provide it as new PR if its ok?

Sounds great! Thanks! Keep it minimal though

closed via https://github.com/christianpoveda/pijama/pull/57


Makefile add optional CONTAINER_PLATFORM flag to container builds

Explain the changes

See https://lima-vm.io/?file=docs/multi-arch.md
See https://github.com/containerd/nerdctl/blob/main/docs/multi-platform.md
used CONTAINER_PLATFORM=amd64 for building x86_64 images on Mac arm64.
In a lima shell:

make noobaa CONTAINER_ENGINE=nerdctl CONTAINER_PLATFORM=amd64

CC @utkarsh-pro
Issues: Fixed #xxx / Gap #xxx

NA

Testing Instructions:

NA

[ ] Doc added/updated
[ ] Tests added

@guymguym, docker supports multi platform builds as well.
$ docker run --platform linux/amd64 busybox uname -a
Linux a7476d751565 5.15.82-0-virt #1-Alpine SMP Mon, 12 Dec 2022 09:15:17 +0000 x86_64 GNU/Linux

$ docker build --platform linux/arm64,linux/amd64,linux/ppc64le ...` # This requires a custom docker builder  though - docker buildx create ...

docker supports multi platform builds as well.

@utkarsh-pro even better then! I just tried with lima and nerdctl, but I assumed this flag could be applicable for other container engines as well.

@guymguym @utkarsh-pro
we need to see that podman support that too.
Currently, we all should move to podman.

@liranmauda it was very simple for me to check that podman has the exact same --platform flag too. However I am not trying to make any suggestions about running this downstream. But if you want to try, now you can with a simple arg.

@liranmauda it was very simple for me to check that podman has the exact same --platform flag too. However I am not trying to make any suggestions about running this downstream. But if you want to try, now you can with a simple arg.

I remember that at some point we could not do it as podman was behind.
In any case, those flows are not triggered downstream, so we are safe there. (we are not using our makefile there).


From private repo

@ : Thanks for your contribution! The author, @rkarlin, has been notified to review your proposed change.


Add option to auto adjust trail speed based on instance parameters

my fundamental approach is probably a bit scuffed but it works.

I believe there is still a progressive desync at extreme low and high scroll speed probably due to the arbitrary 40600f value. Assuming that people never play with those extreme values it shouldn't be a big deal. I haven't tested other stuff so Ill send that over to the beta testers and merge if no issues are found.

I did notice that in testing. I leaned towards the lower scroll speeds when choosing the arbitrary number because its much more noticable there if its slightly off. I believe it gets about 3-4 ish pixels off between 20% and 250% scroll speed with spacing override at 300.

This is already pushed to thunderstore, merging now


Reduce number of automatic resend retries

There are two loops that occurs when perform a resend. The first will try 20 times, with a backoff period set by the poller rate (user configurable). The intention is that a resend will occur eventually if one of the nodes was offline, but not to try indefinitely.
The second loop occurs 5 times for each of the 20 iterations mentioned prior. This was intended to catch ephemeral issues, but causes more harm than good. Any ephemeral issue will be caught during the 20-loop, and even so, after 5 failed attempts it would be pretty clear the issue is persistent.
This PR removes the 5 attempts per loop, greatly reducing the number of retries made from 100 to 20, and simplifying the flow.

Codecov Report

Merging #993 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #993      +/-   ##
============================================
- Coverage     99.31%   99.31%   -0.01%
+ Complexity     2235     2233       -2
============================================
  Files           337      337
  Lines          6106     6100       -6
  Branches        364      363       -1
============================================
- Hits           6064     6058       -6
  Misses            5        5
  Partials         37       37

Impacted Files
Coverage Δ
Complexity Δ

.../com/quorum/tessera/sync/ResendPartyStoreImpl.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

.../quorum/tessera/sync/TransactionRequesterImpl.java
100% <100%> (ø)
6 <1> (-2)
:arrow_down:

Continue to review full report at Codecov.

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


Blu Proofs when Pooling

Is there a way that for pooling the proofs can be limited to only show when the farmer itself wins the block rather than all the partials?

Or maybe create a new color category (green maybe) for the partials and keep blue for an actual proof!

Or maybe create a new color category (green maybe) for the partials and keep blue for an actual proof!

Even better idea +1

Can you add a screenshot of what you see currently, and an example log file
that has those pooling proofs in them? Thx.
On Fri, Jul 9, 2021 at 4:57 AM BigPeteL @.***> wrote:

Or maybe create a new color category (green maybe) for the partials and
keep blue for an actual proof!
Even better idea +1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stolk/chiaharvestgraph/issues/40#issuecomment-877133146,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPE4ED7C5VBXMF4G33YITTW3P3PANCNFSM474G7KQQ
.

--
Owner/Director of Game Studio Abraham Stolk Inc.
Vancouver BC, Canada
@.***

debug1.log

Thank you for the debug.log
To differentiate cases, I now need a debug.log that has a real proof in it, not a pooled one. I understand that log is harder to obtain.

Thank you for the debug.log
To differentiate cases, I now need a debug.log that has a real proof in it, not a pooled one. I understand that log is harder to obtain.

I found an "OG" proof 20 minutes ago, how can I provide you the log? :)

Thx. Email to b.stolk at gmail.com
On Fri, Jul 9, 2021 at 9:59 AM extrasolarCHIA @.***>
wrote:

Thank you for the debug.log
To differentiate cases, I now need a debug.log that has a real proof in
it, not a pooled one. I understand that log is harder to obtain.
I found an "OG" proof 20 minutes ago, how can I provide you the log? :)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stolk/chiaharvestgraph/issues/40#issuecomment-877325826,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPE4HMN2URGQHQ2MSDJLDTW4TG3ANCNFSM474G7KQQ
.

--
Owner/Director of Game Studio Abraham Stolk Inc.
Vancouver BC, Canada
@.***

Try pulling latest version, rebuild, and run it again.
It should now show pool proofs in cyan, and solo proofs in blue.

Try pulling latest version, rebuild, and run it again.
It should now show pool proofs in cyan, and solo proofs in blue.

Works a treat thanks!

Here it shows that I found an actual proof about 12h ago but nothing in my wallet indicates so :v

debug.log


Add version used to telemetry data

Coverage decreased (-0.3%) to 77.039% when pulling bf090bf0fd9e0009649b99267a53aac59bfe09cf on version-telemetry into 98f689f92f820b54eff1e5bff1ca64e29a9cac96 on main.

Coverage decreased (-0.3%) to 77.039% when pulling bf090bf0fd9e0009649b99267a53aac59bfe09cf on version-telemetry into 98f689f92f820b54eff1e5bff1ca64e29a9cac96 on main.


Fix handling of temporary files

NamedTempfile is removed when it is close, isn't it?
https://docs.python.org/2/library/tempfile.html#tempfile.NamedTemporaryFile

OK, delete=True is not supported on Windows. Please write a comment about it.


`normalize_timestamp': Invalid time value<PHONE_NUMBER>

hello guys i just install the google api credentials under ruby  and when i try to run my code or try run again ruby quickstart.rb i got the following error:
/home/user/.rvm/gems/ruby-2.1.6/gems/signet-0.7.2/lib/signet/oauth_2/client.rb:1183:in `normalize_timestamp': Invalid time value<PHONE_NUMBER> (RuntimeError)
looks like that value is no a valid date but i dont know that to do that value come from clien_secrent.json generated in the google console

hi guys any advice comment? please

@zinzinzan I think I just ran into this issue. Could please run Time.now.to_i.class in irb to confirm that it returns a Bignum on your system? Here is how it should look if we are talking about the same problem:
mikhail$ irb
irb(main):001:0> Time.now.to_i.class
 => Bignum

I've opened a PR (#71) to address the issue. As a quick workaround, you can add this patch to your project. In essence, it will replace original implementation of the normalize_timestamp with a new version that supports Bignums.

thats right bignum is the output
2.1.6 :001 > Time.now.to_i.class
=> Bignum
2.1.6 :002 >
where i add that code?? or what i do ?? ruby patch.rb? inside de root folder?

i edit the ruby class located in /home//.rvm/gems/ruby-2.1.6/gems/signet-0.7.2/lib/signet/oauth_2/client.rb
when Fixnum
Time.at(time)
else
to
when Fixnum, Bignum
#Adding Bignum ^ here as timestamps like<PHONE_NUMBER> are bignums on 32-bit systems
Time.at(time)
else
and still give the same error

i edit the ruby class located in /home//.rvm/gems/ruby-2.1.6/gems/signet-0.7.2/lib/signet/oauth_2/client.rb

That certainly works, though I should probably warn you that it will work only until you deploy your code somewhere else as you've patched your local copy of the lib. A better option is to place this patch side by side with the rest of your code and include it before you use the library:
require 'my_awesome_project/patches.rb'

Hi, I rewriting parts of "issued_at" on credential cache json file.
credential = "#{path}credential.json"
##
# Google gem bug support.
# signet/client.rb:1172 Bignum is unsupport on normalize_timestamp.
# 'issued_at' rewrite to Time Object String from Bignum.
if File.exist?(credential)
  json = JSON.load(File.read(credential))
  issued = json["issued_at"]
  json["issued_at"] = "#{Time.at(issued)}" if issued.is_a?(Bignum)
  File.open(credential, "w"){|f| f.write JSON.dump(json) }
end

@alexander-js Yeah, tried that as well. Turned out that in some cases there is no 'issued_at' field in the credentials file and as a result, you can't reliably workaround the issue by patching it.
So far I haven't found anything better than fixing normalize_timestamp at runtime. Hopefully the PR #71 will be merged soon and there will be no need to hack the lib.
Here is how my credentials file looks:
cat ~/.credentials/com.samolinov.pixtream
---
default: '{
   "client_id":<redacted>,
   "access_token":<redacted>,
   "refresh_token":<redacted>,
   "scope":[
      "https://www.googleapis.com/auth/calendar.readonly"
   ],
   "expiration_time_millis":1453986354000
}

Fixed by #71


ValueError: could not convert string to float:

Hello,
Thank you for the repository. It is a great help.
However, please tell me why I am getting this value error. I have tried the code with histogram format genuine scores and imposter scores but I get the following error.
Following is the command I have used and the error I get :
!geteerinf -p "/content/drive/MyDrive/data/" -i "/content/drive/MyDrive/data/imposter.txt" -g "/content/drive/MyDrive/data/genuine_score.txt" -e "exp2" -ht -sp "/content/drive/MyDrive/data/EER1/"
exp2: Loading genuine scores file...
Traceback (most recent call last):
  File "/usr/local/bin/geteerinf", line 8, in <module>
    sys.exit(get_eer_info_cmd())
  File "/usr/local/lib/python3.6/dist-packages/pyeer/eer_info.py", line 127, in get_eer_info_cmd
    gen_scores = [__get_score(line) for line in tf]
  File "/usr/local/lib/python3.6/dist-packages/pyeer/eer_info.py", line 127, in <listcomp>
    gen_scores = [__get_score(line) for line in tf]
  File "/usr/local/lib/python3.6/dist-packages/pyeer/eer_info.py", line 45, in __get_score
    return float(sline[-1])
ValueError: could not convert string to float:

please help, thank you in advance.

Hi, it seems that the format of your scores file is incorrect. Please verify that the format corresponds to the format of the example files in https://github.com/manuelaguadomtz/pyeer/tree/master/pyeer/example_files/hist. Particularly, it seems that you have some not float values in the last column of your score files.

Hi, it seems that the format of your scores file is incorrect. Please verify that the format corresponds to the format of the example files in https://github.com/manuelaguadomtz/pyeer/tree/master/pyeer/example_files/hist. Particularly, it seems that you have some not float values in the last column of your score files.

Thank you for your consideration. I have figured it out .

Thank you for your consideration. I have figured it out .


[DEPENDENCY]: update ember-cli and all other dependencies

The only part of this I'm a little nervous about is the ember-try addition. This may not be 2.x friendly yet (as my tests use controllers)


ng xi18n isn't extracting localized messages when using $localize inside component ts files

Has this issue been solved?
https://github.com/angular/angular-cli/issues/16375
The issue was closed due to inactive, but I'm experiencing the same problem, the tool generates the translations for the html tags, but not for the strings marked with $localize in the typescript code.

No it has not been resolved yet. It will be fixed by https://github.com/angular/angular/pull/32912

Please follow that PR for updates. By the way $localize has not yet been published as a public API.

@petebacondarwin is there an alternative recommended way to translate code within typescript? I need this to translate the metatags, so the html template is not sufficient here as far as I know.

You can use $localize it is just not officially supported and you must work out the message IDs yourself because the current extractor will not extract them. One workaround for this is to use a custom id which you place at the beginning of the $localize tagged string:
$localize `:@@custom-id:my message text`;

In this case the id for this message will be custom-id.


Fixed crash on dynamic import traversal

This is a straightforward fix for #344
Signed-off-by: Antoine Salon<EMAIL_ADDRESS>

I don't know what to test for exactly since that particular code depends on many other code locations.
On the stacktrace the context is populated in visit_Call (node_visitor.py l.90), the call args property access triggers a call to _get_literal_value (context.py l.196).
Would it then make more sense to have it as a functional test? Since there is no linting rule yet it could simply be added as:
    def test_dynamic_imports(self):
        '''Test for dynamically constructed importlib calls.'''
        expect = {
            'SEVERITY': {'UNDEFINED': 0, 'LOW': 0, 'MEDIUM': 0, 'HIGH': 0},
            'CONFIDENCE': {'UNDEFINED': 0, 'LOW': 0, 'MEDIUM': 0, 'HIGH': 0}
        }
        self.check_example('dynamic-imports.py', expect)
`

@evqna Yes, a functional test would also be fine.  Your example above looks good.

Rebased onto master

I looked into implementing a blacklist unit test but the parameters (context and config) are quite complex and mocking them would require a lot of setup (generating an AST and a correct blacklist). I am not sure how to go about that.

hi there,
any chance to release this fix in a 1.5.1 version ?

@calve Sure, I can spin a new tag/release for this.


[MM-29217] Remove Posts.ParentId

Summary
Removing the deprecated Posts.ParentId.
Ticket
https://mattermost.atlassian.net/browse/MM-29217
Release Notes
Removed deprecated Posts.ParentId in favor of the semantically equivalent Posts.RootId

Related PRs
https://github.com/mattermost/mattermost-server/pull/17923
https://github.com/mattermost/enterprise/pull/1019
https://github.com/mattermost/mattermost-api-reference/pull/648
https://github.com/mattermost/mattermost-mobile/pull/5541

Changed base branch, otherwise I cannot delete release-6.0 branch.

@streamer45 Needs cherry-picking to cloud.

I think this is already included in cloud. It's been merged several days ago.


can't install drive : cannot install printer %1 is not a valid win32 application

I managed to build XPS sample project but it keeps poping this error when I try to the add printer from wizard

those are the output folder content

what could be the issue ?

I have the same problem, I built the driver but I can't install with printer wizard in Windows 10. Someone found a solution?

Encountered the same problem, managed to solve it by changing the target platform of the project.
Turned out that I was trying to install the driver on a x64 machine when it was instead compiled for x86.
Simply changing the target platform to x64 worked for me.


Remove private field as it's now deprecated

Fixes #772

hi @stevenfranks !! really sorry about the lack of reply on this PR. we really appreciate this- so please don't let our inaction make you think we don't care. because this PR is a bit old, the merge conflict has gotten pretty hairy. i'll be closing this in favor of a new PR https://github.com/cloudflare/wrangler/pull/804, but we're going to cite you in the changelog as in spirit this way truly your contribution. sorry again about the delay on our part- please keep contributing to wrangler! we'll make sure things go smoother on our end the next time <3


update documentation with the new namespace visibility annotations

Feature Request
Short Description
PR #1088 , we have introduced the ability to select what kind of event to trace "file/network/process/capabilities".
How to leverage this feature is not yet documented.
The document should explicitly mention:

[ ] if telemetry is disabled, the user wont get audit event even if there is an audit rule

Would love to work on this. I think you have mentioned the wrong PR??

Hi @jatinagwal, Sure. Ill asign the issue to you.
you are right, I linked to the issue. This is the PR in question #1087, It is not merged yet.

Hey @achrefbensaad , Will submit first draft by thursday

Hey @jatinagwal The PR is merged now.


Redesign how object methods have been modeled in object typedesc

Currently in the semantic API, an object typedesc maintain a list of references to the method symbols of its methods. Since our design approach in the semantic API had been to get the typedesc from the symbol, there's a mismatch in that story here. Also, it present numerous complexities when implementing it in that way. e.g., cyclic dependencies between classes. Ideally we should focus on keeping just the necessary stuff for the typedesc to perform typechecking and move the rest of the details to the symbol.

This was addressed in #26684, #26698. Hence closing this.


webconsole refactoring with react 18

webconsole refactoring with react 18 and typescript and material ui.

I can't login. Can you fix it?

@kishiguro
I think the UI is beautiful, but there's an error when test with UERANSIM

UE's setting:

@brianchennn are you able to login with User: free5gc  Pass: admin ?

There are some issues:

How can I create 100 UEs simultaneously ?
When I add Flow Rules, error happened.
It isn't able to display the UEs information created by old webconsole.

@brianchennn

Yes, I can login now.

nice!

I think the UI is beautiful, but there's an error when test with UERANSIM.

Let me double check with the same configuration

@kishiguro
Panic when add Flow Rules:
2023-08-01T11:34:35.547912779Z [INFO][SMF][Consumer] SendNFDiscoveryServingAMF ok
2023-08-01T11:34:35.548042234Z [INFO][SMF][CTX] Allocated UE IP address: <IP_ADDRESS>
2023-08-01T11:34:35.548063530Z [INFO][SMF][CTX] Selected UPF: UPF
2023-08-01T11:34:35.548080809Z [INFO][SMF][PduSess][pdu_session_id:1][supi:imsi-208930000000997] Allocated PDUAdress[<IP_ADDRESS>]
2023-08-01T11:34:35.548279120Z [INFO][NRF][DISC] Handle NFDiscoveryRequest
2023-08-01T11:34:35.550266626Z [INFO][NRF][GIN] | 200 |       <IP_ADDRESS> | GET     | /nnrf-disc/v1/nf-instances?preferred-locality=area1&requester-nf-type=SMF&target-nf-type=PCF |
2023-08-01T11:34:35.551690950Z [INFO][PCF][SMpolicy] Handle CreateSmPolicy
2023-08-01T11:34:35.555088331Z [INFO][UDR][DataRepo] Handle PolicyDataUesUeIdSmDataGet
2023-08-01T11:34:35.556130071Z [INFO][UDR][GIN] | 200 |       <IP_ADDRESS> | GET     | /nudr-dr/v1/policy-data/ues/imsi-208930000000997/sm-data?dnn=internet&snssai=%7B%22sst%22%3A1%2C%22sd%22%3A%22010203%22%7D |
2023-08-01T11:34:35.557666677Z [ERRO][PCF][GIN] panic: interface conversion: interface {} is nil, not float64
goroutine 653 [running]:
runtime/debug.Stack()
        /usr/local/go/src/runtime/debug/stack.go:24 +0x65
github.com/free5gc/util/logger.ginRecover.func1.1()
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:284 +0x1a5
panic({0xbf8fc0, 0xc0006177a0})
        /usr/local/go/src/runtime/panic.go:1038 +0x215
github.com/free5gc/pcf/internal/sbi/producer.createSMPolicyProcedure({0x0, 0x0, {0x0, 0x0}, {0xc0005d20c0, 0x14}, {0x0, 0x0, 0x0}, 0x1, ...})
        /home/brian/free5gc-github/NFs/pcf/internal/sbi/producer/smpolicy.go:226 +0x2c45
github.com/free5gc/pcf/internal/sbi/producer.HandleCreateSmPolicyRequest(0xc000371b60)
        /home/brian/free5gc-github/NFs/pcf/internal/sbi/producer/smpolicy.go:37 +0x145
github.com/free5gc/pcf/internal/sbi/smpolicy.HTTPSmPoliciesPost(0xc000362700)
        /home/brian/free5gc-github/NFs/pcf/internal/sbi/smpolicy/api_default.go:56 +0x29a
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/free5gc/util/logger.ginRecover.func1(0xc000362700)
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:316 +0x7b
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/free5gc/util/logger.ginToLogrus.func1(0xc000362700)
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:251 +0x82
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/gin-gonic/gin.(*Engine).handleHTTPRequest(0xc000242340, 0xc000362700)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/gin.go:620 +0x671
github.com/gin-gonic/gin.(*Engine).ServeHTTP(0xc000242340, {0xe8e318, 0xc000534028}, 0xc000362600)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/gin.go:576 +0x1dd
golang.org/x/net/http2.(*serverConn).runHandler(0x13ea320, 0x0, 0x0, 0xc0003faf00)
        /home/brian/go/pkg/mod/golang.org/x/net@v0.7.0/http2/server.go:2311 +0x8c
created by golang.org/x/net/http2.(*serverConn).processHeaders
        /home/brian/go/pkg/mod/golang.org/x/net@v0.7.0/http2/server.go:2025 +0x5f4
2023-08-01T11:34:35.557825062Z [INFO][PCF][GIN] | 500 |       <IP_ADDRESS> | POST    | /npcf-smpolicycontrol/v1/sm-policies |
2023-08-01T11:34:35.558279343Z [ERRO][SMF][GIN] panic: interface conversion: interface {} is nil, not models.ProblemDetails
goroutine 338 [running]:
runtime/debug.Stack()
        /usr/local/go/src/runtime/debug/stack.go:24 +0x65
github.com/free5gc/util/logger.ginRecover.func1.1()
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:284 +0x1a5
panic({0xaad1a0, 0xc0000e3380})
        /usr/local/go/src/runtime/panic.go:1038 +0x215
github.com/free5gc/smf/internal/sbi/producer.HandlePDUSessionSMContextCreate({0xc00059c480, {0xc0006fec00, 0x12, 0x3e8}})
        /home/brian/free5gc-github/NFs/smf/internal/sbi/producer/pdu_session.go:169 +0x16fd
github.com/free5gc/smf/internal/sbi/pdusession.HTTPPostSmContexts(0xc0006d0700)
        /home/brian/free5gc-github/NFs/smf/internal/sbi/pdusession/api_sm_contexts_collection.go:54 +0x265
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/free5gc/util/logger.ginRecover.func1(0xc0006d0700)
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:316 +0x7b
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/free5gc/util/logger.ginToLogrus.func1(0xc0006d0700)
        /home/brian/go/pkg/mod/github.com/free5gc/util@v1.0.5-0.20230511064842-2e120956883b/logger/logger.go:251 +0x82
github.com/gin-gonic/gin.(*Context).Next(...)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/context.go:174
github.com/gin-gonic/gin.(*Engine).handleHTTPRequest(0xc0000836c0, 0xc0006d0700)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/gin.go:620 +0x671
github.com/gin-gonic/gin.(*Engine).ServeHTTP(0xc0000836c0, {0xc99d28, 0xc00052c1b0}, 0xc0006d0600)
        /home/brian/go/pkg/mod/github.com/gin-gonic/gin@v1.9.0/gin.go:576 +0x1dd
golang.org/x/net/http2.(*serverConn).runHandler(0xc000524930, 0xc0005757d0, 0x8a24a6, 0xc0004d9080)
        /home/brian/go/pkg/mod/golang.org/x/net@v0.7.0/http2/server.go:2311 +0x8c
created by golang.org/x/net/http2.(*serverConn).processHeaders
        /home/brian/go/pkg/mod/golang.org/x/net@v0.7.0/http2/server.go:2025 +0x5f4
2023-08-01T11:34:35.558543765Z [INFO][SMF][GIN] | 500 |       <IP_ADDRESS> | POST    | /nsmf-pdusession/v1/sm-contexts |
2023-08-01T11:34:35.558940606Z [ERRO][AMF][Gmm][amf_ue_ngap_id:RU:8,AU:8(3GPP)][supi:SUPI:imsi-208930000000997] CreateSmContextRequest Error: undefined response type

@kishiguro
After editing Flow Rules, these fields still empty except IP Filter:

@kishiguro
I want to let you know that another contributor is also modifying the webconsole in PR #43. Free5gc team will choose better one to merge.

@brianchennn Seems there is a bug of setting flow rules.  Let me fix it.

@brianchennn FlowRule (and QosFlows) handling has been fixed.  Would you mind to try again?

@kishiguro
Thanks for fixing the bug. Now I can't add second DNN:
https://youtu.be/Rj9iRkLBZ6A

@kishiguro Thanks for fixing the bug. Now I can't add second DNN: https://youtu.be/Rj9iRkLBZ6A

Thanks for pointing out.  Just fixed the issue.  Would you mind to check it out?

@kishiguro
That's great!
Next, can you help me to add this field for adding multiple UEs? Sometimes we may do that for testing. Thanks a lot !

@brianchennn

Next, can you help me to add this field for adding multiple UEs by increment SUPI? Sometimes we may do that for testing.

sure, wait a sec.

@brianchennn just added support for multiple UEs by increment SUPI.  would you mind to try?

@kishiguro  It works well !
Now, could you make the default S-NSSAI setting be 2 S-NSSAI Configuragtions as below 2 figure?

@brianchennn

Now, could you make the default S-NSSAI setting be 2 S-NSSAI Configuragtions as below?

Now default S-NSSAI configuartion will be set.  Would you mind to double check?

@kishiguro
Could you add the confirm password field when creating/modifying user of tenant?
And the password field should be concealed in browser view.
I just added these features  in PR #44.

I also found that when creating a tenant, the hint of this field is tenant ID, but it should be tenant name.

@brianchennn got it let me integrate the confirm password feature.

@brianchennn I have integrated Password Confirmation feature.  "Tenant ID" is also fixed as "Tenant Name".

@tim-ywliu
I think we can merge this PR.


Learning rate is not changed in accordance with the scheduler after resuming [FasterRCNN example]

Bug Reports: PLEASE always include

What you did. (command you run if using examples; post or describe your code if not)

Resume a model from a checkpoint.

What you observed, e.g. logs.

The learning rate is not changed according to the scheduler.

What you expected, if not obvious.

Learning rate is scheduled to change from 0.01 to 0.001 at epoch 500; however, after 500 epoch, the learning rate is still 0.01.

Your environment (TF version, cudnn version, number & type of GPUs), if it matters.

TF 1.4.0, cudnn 7.0.3, 8 v100 GPUs., Tensorpack version 0.8.0
This is related to the examples of FasterRCNN, in my case, warmup_epoch= 5, factor=1, st3pnum=300.
I resume the model at global_step=147000. According to my scheduler, the learning rate should change to 0.001 after global_step=150000 a.k.a. epoch=500
callbacks=[
				PeriodicTrigger(ModelSaver(), every_k_epochs=5),
				ScheduledHyperParamSetter(
					'learning_rate',
					[(warmup_epoch * factor, 1e-2),
					 (150000 * factor // stepnum, 1e-3),
					 (230000 * factor // stepnum, 1e-4)]),
				EvalCallback(),
				GPUUtilizationTracker(),
			]

Please ignore it now... something wrong from my side....

okay, my bad. but the start epoch is read from stat.json rather than deriving from global_step and step_num