now help no longer lists available subcommands

From @darknoon on August 24, 2017 22:47
I find the beta version significantly harder to use, b/c I can't see what the available subcommands are like ls, etc, especially since they might vary among providers (aws, sh, …)
Old:
/ $ now help

  𝚫 now [options] <command | path>

  Commands:

    Cloud

      deploy               [path]      Performs a deployment (default)
      ls | list            [app]       List deployments
      rm | remove          [id]        Remove a deployment
      ln | alias           [id] [url]  Configures aliases for deployments
      domains              [name]      Manages your domain names
      certs                [cmd]       Manages your SSL certificates
      secrets              [name]      Manages your secret environment variables
      dns                  [name]      Manages your DNS records
      logs                 [url]       Displays the logs for a deployment
      scale                [args]      Scales the instance count of a deployment
      help                 [cmd]       Displays complete help for [cmd]

    Administrative

      billing | cc         [cmd]       Manages your credit cards and billing methods
      upgrade | downgrade  [plan]      Upgrades or downgrades your plan
      teams                [team]      Manages your teams
      switch                           Switches between teams and your account
      login                            Login into your account or creates a new one
      logout                           Logout from your account

  Options:

    -h, --help                Output usage information
    -v, --version             Output the version number
    -n, --name                Set the name of the deployment
    -c FILE, --config=FILE    Config file
    -d, --debug               Debug mode [off]
    -f, --force               Force a new deployment even if nothing has changed
    -t TOKEN, --token=TOKEN   Login token
    -L, --login               Configure login
    -l, --links               Copy symlinks without resolving their target
    -p, --public              Deployment is public (`/_src` is exposed) [on for oss, off for premium]
    -e, --env                 Include an env var (e.g.: `-e KEY=value`). Can appear many times.
    -E FILE, --dotenv=FILE    Include env vars from .env file. Defaults to '.env'
    -C, --no-clipboard        Do not attempt to copy URL to clipboard
    -N, --forward-npm         Forward login information to install private npm modules
    --session-affinity        Session affinity, `ip` (default) or `random` to control session affinity.

  Enforcable Types (when both package.json and Dockerfile exist):

    --npm                     Node.js application
    --docker                  Docker container
    --static                  Static file hosting

  Examples:

  – Deploys the current directory

    $ now

  – Deploys a custom path `/usr/src/project`

    $ now /usr/src/project

  – Deploys a GitHub repository

    $ now user/repo#ref

  – Deploys a GitHub, GitLab or Bitbucket repo using its URL

    $ now https://gitlab.com/user/repo

  – Deploys with ENV vars

    $ now -e NODE_ENV=production -e MYSQL_PASSWORD=@mysql-password

  – Displays comprehensive help for the subcommand `list`

    $ now help list

New:
$ ../node_modules/.bin/now help

  Now: universal serverless deployments.

  To deploy, run in any directory of your choosing:

    `now`

  The deployment backend provider is fully configurable.
  The following are supported:

    - sh	 now.sh
    - aws	 AWS Lambda
    - gcp	 Google Cloud Platform

  For example, to setup AWS Lambda functions run:

    `now aws login`

  Some useful subcommands:

    - `now ls`
    - `now rm`
    - `now alias`

  To read more in-depth documentation, run:

    `now [provider] [subcommand] help`

  For more information: https://github.com/zeit/now.

Copied from original issue: zeit/now#58

From @darknoon on August 24, 2017 22:51
I would expect now sh help to at least list the available subcommands for the sh provider, but it seems to do the same thing as now help.

✔ This is back now. Shipping with the next stable release!

@leo This is what I am getting on 8.0.1
now help
> Error! Could not resolve deployment target "help"


default directories separators not working

When trying to create a new Plugin it provides defaults for the vst dirs, separating them with the colon (:) ... this notation then isn't accepted when clicking ok

As far as I understood this issue, you have configured the VST_PATH environment variable with multiple directories, separated by the colons. In the current implementation only one "favorite" VST directory is supported ("VST location" in the settings dialog). I think I can use the first path from the VST_PATH if it has multiple values. I will fix this in the next version.

Exactly, that is, Ubuntu did this probably and I actually thought you did it :) hehe ...
Yeah, but you read right in between the lines, getting there, thx!

I totally forget about that one, I'm sorry for that. I think I fixed it in the develop branch.


How to add modern e2k processor architecture?

Hi! MCST produces its own microprocessors based on the perspective e2k architecture, how can we add support for this architecture to the Golang?
Thanks!
http://mcst.ru
http://mcst.ru/programmnoe-obespechenie-elbrus
https://en.wikipedia.org/wiki/Elbrus_2000#:~:text=The Elbrus 2000%2C E2K (Russian,instruction set architectures (ISA)%3A

See https://golang.org/wiki/PortingPolicy.
In general we do not use the issue tracker for questions.  You will get better and faster answers if you use a forum.  See https://golang.org/wiki/Questions.  Thanks.


BUG: Heroku Deployment

Describe the bug:
Heroku Deployment is not working due to Python Version and other issues
To Reproduce:
Steps to reproduce the behavior:

Deploy to Heroku

Expected behavior:
The deploy and build should pass
Actual Behavior:
The deploy, build, and GitHub Action fails
Screenshots:
remote: -----> Read Python version from poetry.lock
remote: -----> ^3.11.7 is not valid, please specify an exact Python version (e.g. 3.8.1 or ==3.8.1) in your pyproject.toml (and thus poetry.lock)
remote:  !     Push rejected, failed to compile Python Poetry app.
remote:
remote:  !     Push failed
Possible Fix
Change the toml and lock files

Heroku Deployment is working


Combine Appointment and LakeAppointment class.

Describe what needs to be refactored
Currently, the app uses two different classes for keeping track of appointments: LakeAppointment roughly corresponds to the appointments in Firebase, and the SFCalendar uses Appointment. These classes get confusing when used widely across the app, and it becomes necessary to convert between the two constantly.
Suggest possible ways in which the project could be refactored
Instead, we should modify the LakeAppointment class so that it extends the Appointment class. This would greatly simplify the app and allow the calendar to integrate smoothly with the Firebase database.

Appointment class is removed with new calendar library


include_in_copy not working as documented (breaks asdf)

Describe the bug
The include_in_copy attribute for the terraform block is not working as documented. It fails to copy my .tool-versions file into the download_dir, such that asdf can pick up the desired version from it when the terraform command is run.
To Reproduce
Steps to reproduce the behavior, code snippets and examples which can be used to reproduce the issue.
$ tree -a
.
├── live
│   └── prod
│       └── mystack
│           └── terragrunt.hcl
├── stacks
│   └── mystack
│       └── main.tf
├── terragrunt.hcl
└── .tool-versions

# .tools-versions
terraform 1.7.4
terragrunt 0.54.5

# terragrunt.hcl

# this code works when uncommented
# generate "asdf_tool_versions" {
#   path      = ".tool-versions"
#   if_exists = "overwrite_terragrunt"
#
#   contents = file("${get_repo_root()}/.tool-versions")
# }

# this code does not work
terraform {
  include_in_copy = ["${get_repo_root()}/.tool-versions"]
}

download_dir = pathexpand("~/.terragrunt.d/downloads")

include "root" {
  path = find_in_parent_folders()
}

terraform {
  source = "${path_relative_from_include("root")}/stacks/${basename(get_terragrunt_dir())}"
}

# stacks/mystack/main.tf
resource "null_resource" "default" {
  provisioner "local-exec" {
    command = "echo 'Hello World'"
  }
}

Expected behavior
I expect that the following should work:
git init
cd live/prod/mystack
terragrunt init

Instead, I get the following. Note that the .tool-versions file was NOT copied to the download_dir.
~/terragrunt/live/prod/mystack$ terragrunt --terragrunt-log-level debug --terragrunt-source-update init
DEBU[0000] Terragrunt Version: 0.54.5
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] [Partial] Included config /home/nhavens/terragrunt/terragrunt.hcl has strategy shallow merge: merging config in (shallow).
DEBU[0000] Running command: terraform --version          prefix=[/home/nhavens/terragrunt/live/prod/mystack]
DEBU[0000] terraform version: 1.7.4
DEBU[0000] Reading Terragrunt config file at /home/nhavens/terragrunt/live/prod/mystack/terragrunt.hcl
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] [Partial] Included config /home/nhavens/terragrunt/terragrunt.hcl has strategy shallow merge: merging config in (shallow).
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Included config /home/nhavens/terragrunt/terragrunt.hcl has strategy shallow merge: merging config in (shallow) for dependency.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] Did not find any locals block: skipping evaluation.
DEBU[0000] git show-toplevel result:

DEBU[0000] Included config /home/nhavens/terragrunt/terragrunt.hcl has strategy shallow merge: merging config in (shallow).
DEBU[0000] The --terragrunt-source-update flag is set, so deleting the temporary folder /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE before downloading source.
DEBU[0000] Downloading Terraform configurations from file:///home/nhavens/terragrunt/stacks/mystack into /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE
DEBU[0000] Copying files from /home/nhavens/terragrunt/live/prod/mystack into /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE
DEBU[0000] Setting working directory to /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE
DEBU[0000] Running command: terraform init               prefix=[/home/nhavens/terragrunt/live/prod/mystack]
No version is set for command terraform
Consider adding one of the following versions in your config file at
terraform 1.3.5
terraform 1.3.9
terraform 1.7.4
ERRO[0000] terraform invocation failed in /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE  prefix=[/home/nhavens/terragrunt/live/prod/mystack]
ERRO[0000] 1 error occurred:
	* [/home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE] exit status 126

~/terragrunt/live/prod/mystack$ tree -a /home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE
/home/nhavens/.terragrunt.d/downloads/aAanXV8z0RsnE0e8Pd50fQjfviM/bw8b_YNr-FzBqiXrg_Tlfcll4sE
├── main.tf
├── terragrunt.hcl
├── .terragrunt-init-required
├── .terragrunt-module-manifest
├── .terragrunt-source-manifest
└── .terragrunt-source-version

0 directories, 6 files
~/terragrunt/live/prod/mystack$

Versions

asdf version: v0.14.0-ccdd47d
Terragrunt version: v0.54.5
Terraform version: v1.7.4
Environment details: Linux Mint 21.3

Additional context
A key part of the problem is that the download_dir is outside the git repo's directory structure, so as to nullify the effect of the in-repo .tool-versions file.
If I uncomment the generate block in my top-level terragrunt.hcl, the .tool-versions file is generated and terragrunt init works properly.

Hi,
AFAIK, hidden files aren't copied to download_dir, can be used include_in_copy to pass configurations to copy hidden files
https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/

@denis256 Yup, The include_in_copy documentation uses .python-versions (a hidden file) as its example.

Hi,
I'm having the same issue with include_in_copy, wanting to copy .git hidden folder.
In my case however I noticed the .git folder was included when the source was remote (source = "git::https://MYURL) so this was a workaround for me, but even without the use of include_in_copy
If I'm not mistaken according to documentation I should be able to use include_in_copy attribute to copy the .git folder from my local repo .

By default, Terragrunt excludes hidden files and folders during the copy step. This feature allows you to specify glob patterns of files that should always be copied from the Terragrunt working directory
For example, if your terraform module source contains a hidden file that you want to copy over (e.g., a .python-version file), you can specify that in this list to ensure it gets copied over to the scratch copy (e.g., include_in_copy = [".python-version"]).

https://terragrunt.gruntwork.io/docs/reference/config-blocks-and-attributes/

Hi there, can this issue be reopened, include_in_copy doesn't seem to work at all. I am using version 0.68.4
terraform {
  source = include.locals.code_source

  include_in_copy = [
    ".yamllint"
  ]

  extra_arguments "k8s_concurrency" {
    commands = [
      "apply",
      "destroy",
      "plan"
    ]
    arguments = [
      "-parallelism=100"
    ]
  }
}

there is no mention of include_in_copy or .yamllint in debug logs


trying to use lora loader, get ERROR lora diffusion_model.single_blocks.37.linear1.weight 'GGMLTensor' object has no attribute 'tensor_shape'

i get a different error
ERROR lora diffusion_model.double_blocks.18.img_attn.qkv.weight The size of tensor a (1728) must match the size of tensor b (3072) at non-singleton dimension 1 ERROR lora diffusion_model.double_blocks.18.img_attn.proj.weight The size of tensor a (1728) must match the size of tensor b (3072) at non-singleton dimension 1 ERROR lora diffusion_model.double_blocks.18.txt_attn.qkv.weight The size of tensor a (1728) must match the size of tensor b (3072) at non-singleton dimension 1 ERROR lora diffusion_model.double_blocks.18.txt_attn.proj.weight The size of tensor a (1728) must match the size of tensor b (3072) at non-singleton dimension 1

Got the same error :
Loading 1 new model
ERROR lora diffusion_model.double_blocks.0.img_attn.qkv.weight 'GGMLTensor' object has no attribute 'tensor_shape'
ERROR lora diffusion_model.double_blocks.0.img_attn.qkv.weight 'GGMLTensor' object has no attribute 'tensor_shape'
ERROR lora diffusion_model.double_blocks.0.img_attn.qkv.weight 'GGMLTensor' object has no attribute 'tensor_shape'
ERROR lora diffusion_model.double_blocks.0.img_attn.proj.weight The size of tensor a (3264) must match the size of tensor b (3072) at non-singleton dimension 1

yeah same error as Danamir & Nelathan not sure if it's a problem with the version of pytorch we're using

Should be fixed

Fixed it for me too.
Sadly even without the errors, the LoRA has almost no effect the output, compared to the full model. I'll do a separate ticket.

@Danamir sounds good, include a link to a LoRA if possible so I can test it.

works great now, only a 5% speed malus compared to no lora. Thank you! (used Q4_0 with FaeTasticDetails)

thansk for the fixed!


I have not used any external library, i have used acrobat reader to annotate, insert and delete. I had taken the screenshot form your demo link only. Here by i will attach my pdf file as well as screenshot of demo link which you have given in the GitHub.

I have not used any external library, i have used acrobat reader to annotate, insert and delete. I had taken the screenshot form your demo link only. Here by i will attach my pdf file as well as screenshot of demo link which you have given in the GitHub.
Passasjeren_strukturert (4) 11.pdf

Originally posted by @rakeshkasinathan in https://github.com/mozilla/pdf.js/issues/10672#issuecomment-476064327

An idea bug report contains three things: what you were doing, what you were expecting, and what happened instead.
What were you expecting?

Please don't open multiple duplicate issues, but update the original one with all the necessary information instead and then that may be re-opened when enough information has been provided.

I agree with the comments above: what are you expecting? From the screenshots I really can't tell what the problem is. Is there text in the annotations that should not be there? Should there be more/less annotations? What is the rendering defect you're reporting?

There are two issues:

The comment in the insert mark (blue caret symbol) is not showing up in PDFjs. If you open the attached PDF in the Acrobat Reader, you would notice that both the insert icons have comments in them. These completely missing in PDFjs.

In this sticky note comment at the start of the PDF, the original comment was "This is a test" and it has reply to comment "My reply". Only "My Reply" is showing up.

I have attached the PDF that I'm using to test for your reference.
Passasjeren_strukturert (4) 11.pdf

Hi, there is missing implementation of some annotation features like InReplyTo (your 2. issue) and groups (your 1. issue).
This requires quite a lot of coding on both worker part and display part of pdfjs. I tried to implement the worker part, so I can provide that as pull request (https://github.com/vlastimilmaca/pdf.js/tree/annotations-enhanced-parsing), but there's gonna be some extensive coding work on display part too. I think, there will have to be some kind of side-bar (like bookmarks, attachments etc) for displaying markup annotations as discussion threads. In page area, on mouseover, there should be displayed first comment + info about how many replies and state it has (which will probably need some localized text ... not easy ...). On click, it should open discussion sidebar and focus to correct annotation thread. But that is how I see it and it will be quite a lot of work to get it done right. But worker part is probably first step ... It should not break display part, so display part work can start later.

Do you need any more clarifications from my side.

I understand that the above solution of for enhancing annotations with multiple response to a comments. We still have another outstanding issue. The insert comment with the blue caret symbol still does not display the comment it. How can we get to see the comment inside the insert comment in PDFjs.

That should do the implementation of worker part I guess … It should correctly fill-in the master annotation with content and current display layer should display it just fine. I'll try to prepare merge request over weekend and we will se where we can get :).

Hello,
How can i get CaretAnnotation,StrikeOutAnnotation position and text ranges.

For caret, use the /Rectangle entry. For strikeout, use the /QuadPoints entry. See also the PDF specification.
https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf
however the order of the quadpoints is wrong, see
https://stackoverflow.com/questions/9855814/pdf-spec-vs-acrobat-creation-quadpoints

The functionality is now in the API, so since the other points here are fixed I'm closing this with #10982 as a follow-up.


Static this is undefined during recursive call

While using this in a static class method to recursively call itself works in the browser, it does not work with compiled code:
static buildTagCache(node, idCache) {
	if (!node || (node.nodeType !== 1)) return; // not an element node
	var nn = node.nodeName;
	if (nn === 'point' || nn === 'link') {
		// do some stuff
	}
	// search down
	util.do4children(node, n => {
		this.buildTagCache(n, idCache);
	});
}

has an undefined this at the child call.
Originally posted by @ray007 in https://github.com/google/closure-compiler/issues/3553#issuecomment-590834733

Some question to help us debug:

What optimizations are you using?

Devirtualization?
Optimize Calls?

Are you using class-side inheritance?
What does the compiled code look like?

An example a bit closer to my source:
class C {
  /** @param {Element} n */
  static method(n) {
    if (!(n instanceof Element)) return;
    console.log(n);
    [].forEach.call(n.children, (c) => {
    	this.method(c);
    });
  }
}
C.method(document.body);

becomes (with prettyprint)
var C = function() {
};
C.method = function(a) {
  var b = this;
  a instanceof Element && (console.log(a), [].forEach.call(a.children, function(a) {
    b.method(a);
  }));
};
C.method(document.body);

If C.method (or the call to it) was bound to C, things would work.
Testlink to closure-compiler.appspot.com.

Thanks for point this out! This is a compiler option that would solve this, as @brad4d mentioned in the other thread, but it hasn't be exposed as a flag yet. There's a field in CompilerOptions called assumeStaticInheritanceRequired that just needs to be set true.
This option hasn't been exposed  mostly because it can really hurt RemoveUnusedCode, and also to try to reduce the number of compiler flags. However, issues like this may prioritize creating the flag.
Would you use such a flag?

If it fixes the bug I'd probably use the flag.
But it feels funny to require a flag to avoid a class of bugs...


Bug: infinite loop in CalendarDateUntil calculation

While migrating Boa to temporal_rs in boa-dev/boa#3694, an infinite loop was discovered in the calendar.prototype.dateUntil test suite.
Cause: most likely occurs here in the diff_iso_date function.
Test 262 tests: The loop occurs in the largest-unit-year.js tests.

Would be good to port the test from JS to native Rust


issue #290 - Warning: Formsy: isMounted is deprecated. Instead, make sure to clean up subscriptions and pending requests in componentWillUnmount to prevent memory leaks.

formsy-react updated to version 0.19.5 which fixes issue.
I've tested connect-app with the updated version of the formsy-react and it looks like works good without any new issues.

Awesome. Thanks @maxceem


TP-770-posthumous-review

This PR has been created in order to apply comments left posthumously against the now merged https://github.com/uktrade/tamato/pull/328.

Codecov Report

Merging #340 (d49ca4a) into master (6b955dc) will increase coverage by 0.01%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master     #340      +/-   ##
==========================================
+ Coverage   89.47%   89.49%   +0.01%
==========================================
  Files         276      276
  Lines       14146    14142       -4
  Branches     1031     1030       -1
==========================================
- Hits        12657    12656       -1
+ Misses       1271     1267       -4
- Partials      218      219       +1

Impacted Files
Coverage Δ

additional_codes/forms.py
70.73% <50.00%> (+2.49%)
:arrow_up:

additional_codes/views.py
77.31% <100.00%> (+0.78%)
:arrow_up:

measures/business_rules.py
86.49% <0.00%> (-0.87%)
:arrow_down:

quotas/business_rules.py
98.42% <0.00%> (+1.57%)
:arrow_up:

Continue to review full report at Codecov.

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

Yes as mentioned my comment strictly applies to those models that do use
sid, not the ones that use as identifying fields things such as code,
item_id, area_id, etc. Stuart’s response seems to hint at what we can prefer
On Tue, 27 Jul 2021 at 11:58, Paul Pepper @.***> wrote:

@.**** commented on this pull request.
In additional_codes/forms.py
https://github.com/uktrade/tamato/pull/340#discussion_r677340524:

   workbasket = WorkBasket.current(self.request)

   tx = None

   if workbasket:

       tx = workbasket.transactions.order_by("order").last()

   tx = WorkBasket.get_current_transaction(self.request)

   highest_sid = (

@kintisheff https://github.com/kintisheff Where a model does use a sid
then perhaps it makes sense to factor that out - though not all Models use
a sid as sole attribute of identifying_fields.
@stuaxo https://github.com/stuaxo
we use could get the next sid_counter from the database
Is the suggestion to continue caching the value, but push cache storage
out of the instance (of a Measure in your test cases?) and into the
database, using something like django.core.cache.backends.db.DatabaseCache
?
Perhaps it's worthwhile putting @kintisheff
https://github.com/kintisheff's and @stuaxo https://github.com/stuaxo's
concerns on a separate tech debt ticket? I only created this PR to address
messages that were posthumously left against another, now closed, PR :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uktrade/tamato/pull/340#discussion_r677340524, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWMJEHR7VO26VJDX2YNHS3TZ2GLXANCNFSM5AUC3QAQ
.

--
Tsvetan Kintisheff
cell: +44 74 88 55 88 48
email: @.***

Agree these concerns are out of the scope of this ticket, though
On Tue, 27 Jul 2021 at 16:30, Tsvetan Kintisheff @.***>
wrote:

Yes as mentioned my comment strictly applies to those models that do use
sid, not the ones that use as identifying fields things such as code,
item_id, area_id, etc. Stuart’s response seems to hint at what we can prefer
On Tue, 27 Jul 2021 at 11:58, Paul Pepper @.***>
wrote:

@.**** commented on this pull request.
In additional_codes/forms.py
https://github.com/uktrade/tamato/pull/340#discussion_r677340524:

   workbasket = WorkBasket.current(self.request)

   tx = None

   if workbasket:

       tx = workbasket.transactions.order_by("order").last()

   tx = WorkBasket.get_current_transaction(self.request)

   highest_sid = (

@kintisheff https://github.com/kintisheff Where a model does use a sid
then perhaps it makes sense to factor that out - though not all Models use
a sid as sole attribute of identifying_fields.
@stuaxo https://github.com/stuaxo
we use could get the next sid_counter from the database
Is the suggestion to continue caching the value, but push cache storage
out of the instance (of a Measure in your test cases?) and into the
database, using something like
django.core.cache.backends.db.DatabaseCache?
Perhaps it's worthwhile putting @kintisheff
https://github.com/kintisheff's and @stuaxo https://github.com/stuaxo's
concerns on a separate tech debt ticket? I only created this PR to address
messages that were posthumously left against another, now closed, PR :-)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/uktrade/tamato/pull/340#discussion_r677340524, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABWMJEHR7VO26VJDX2YNHS3TZ2GLXANCNFSM5AUC3QAQ
.

--
Tsvetan Kintisheff
cell: +44 74 88 55 88 48
email: @.***

--
Tsvetan Kintisheff
cell: +44 74 88 55 88 48
email: @.***


Bunch of usability improvements, some bug fixes and functional enhancements

Thanks for these scripts! I found them quite useful and I was glad not to have to write them from scratch myself!
I'm submitting some changes which I believe are improvements. I've tried to isolate them logically in separate commits.
The only change I'm unsure about is 583f17c. Your code uses the "storeId" field, if it is returned by the API, rather than the "id" field. I found that none of the songs for which the API returned a "storeId" were actually being added to my playlists. When I modified the code to always use "id", all of those songs were imported properly into my playlists.
Since I don't understand the API or your usage of it well enough to understand why "storeId" was being used in your code, I don't know what the global ramifications of my change are. All I know is that, for me at least, the import worked properly without "storeId", and improperly with it.
I've mentioned this change first because as I said it's the one change I can't say for certain is correct. Here are the other changes, which I'm much more confident about:

Use argparse for parsing the command line in both ImportList.py and ExportLists.py.
Add support for an INI file.
Make the username be specified on the command line or in the INI file rather than hard-coding it in preferences.py.
Make it possible to specify the android ID on the command line or in the INI file or have the script automatically generate and save a random android ID into the INI file for future use (necessary because the script wouldn't work for me when I used MobileClient.FROM_MAC_ADDRESS, and I assume others will have the same problem).
Add the ability to automatically replace existing playlists with the same name when importing one, so the user doesn't end up with many playlists with the same name.
Allow the password to be specified in the INI file. Not recommended for long-term use, I know, but it's awful useful in the short term when you need to run the script over and over to import a lot of playlists and/or debug issues with the script.
Be smarter about picking the best match for a track and about dropping duplicates. In particular:

When there's more than one match, and some of the matches have already been selected and others haven't, then select one of the matches that hasn't been selected yet, rather than selecting the first match.
When there's more than one match, prefer exact matches for title, album, or artist, over substring matches.

if a song is not found then i get:

Traceback (most recent call last):
  File "ImportList.py", line 301, in <module>
    search_result = search_for_track(smart_details)
  File "ImportList.py", line 125, in search_for_track
    top_result = best_match(details, search_results)
  File "ImportList.py", line 86, in best_match
    return pairs[0][0]

Fixed in https://github.com/soulfx/gmusic-playlist/pull/56/commits/17d8ede749c2fc7f9bac5d0c43438186a38cd707, I hope.


How do i use a 3rd psmove controller for orientation in normal games or steamvr

Hi,i have a Doogee x55 cellphone,and sadly it doesnt have a gyroscope,and i was wondering if i could use a 3rd psmove controller for orientation and if a can ,please tell me how,thanks.

Yes, you could use 3rd controller. You can use freepie (Read FREEPIE USERS on realise page)

what releases?


enableAllSpaceWindows doesn't work on multiple screens

Thanks for the nice tool
With multiple screens, is it possible to have the switch window shown either on all screens or on the main screen (rather than the active one)?
Right now, it shows only on the active screen and require me to hunt for it. The enableAllSpaceWindows doesn't seem to have an effect with me.

You want to always show the HotSwitch-HS panel on main screen, aren't you?
If you mean it, there is not the option yet.

Then all screens?

It is somewhat difficult to show on all screens.
But I'll consider implementing that it always shows on main screen.

Thanks, you can close this if you like.
Cheers

I have added the option to show the panel not on main screen but on primary screen.
You can use it on v2.2.5
Usage: https://github.com/oniatsu/HotSwitch-HS#always-showing-the-panel-on-primary-screen


⚠ Samsung has degraded performance

In 8a88202, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2569 ms

Resolved: Samsung performance has improved in ec7cc13.


ValueError: bucket_boundaries must not be empty

I am having the above error when running the "train" module. Any lead will be highly appreciable. Thanks in advance.

Hey yes it is taking me a long time, like 12 hours plus just for train.py. Please let me know if you are able to get the code running within a day's time.

Hey yes it is taking me a long time, like 12 hours plus just for train.py(I quit it in middle of code execution). Please let me know if you are able to get the code running within a day's time.

My train.py is not even running. I am getting the above error while running the train.py module. Please let me know how this can be resolved.

make sure you download LJ Speech and point the directory variable in hyperparams.py to that folder. Rename metadata to transcript, since training needs that

I have done all the modifications to fir the model. But I think there is some problem while batching. I am getting the below error while running the train.py module.

ValueError                                Traceback (most recent call last)
 in
142     num = 1
143
--> 144     g = Graph(num=num); print("Training Graph loaded")
145
146     logdir = hp.logdir + "-" + str(num)
 in init(self, num, mode)
40         ## mags: Magnitude. (B, T, n_fft//2+1) float32
41         if mode=="train":
---> 42             self.L, self.mels, self.mags, self.fnames, self.num_batch = get_batch()
43             self.prev_max_attentions = tf.ones(shape=(hp.B,), dtype=tf.int32)
44             self.gts = tf.convert_to_tensor(guided_attention())
 in get_batch()
129                                             num_threads=8,
130                                             capacity=hp.B*4,
--> 131                                             dynamic_pad=True)
132
133     return texts, mels, mags, fnames, num_batch
E:\Python Anaconda\lib\site-packages\tensorflow\contrib\training\python\training\bucket_ops.py in bucket_by_sequence_length(input_length, tensors, batch_size, bucket_boundaries, num_threads, capacity, bucket_capacities, shapes, dynamic_pad, allow_smaller_final_batch, keep_input, shared_name, name)
372         bucket_boundaries)
373   if not bucket_boundaries:
--> 374     raise ValueError("bucket_boundaries must not be empty")
375   for (s, e) in zip(bucket_boundaries[:-1], bucket_boundaries[1:]):
376     if not isinstance(s, int) or not isinstance(e, int):
ValueError: bucket_boundaries must not be empty

This issue has been resolved. This error was due to my dataset in which maxlen+1 and minlen-1 were equal and hence the for loop was returning 0 elements in the list for the parameter bucket boundaries in bucket_by_sequence_length function.


If default filer is disabled, nothing is put on map

enabled: false,
creates the problem

Added branching had left  function calls db.applyAllNoFilter and psgeoInitMarker in the wrong branch. Markers were thus not initialized at all in some cases.


Use JGitVer as input to versions plugin

The tycho versions plugin pushes versions down into a Tycho build. It is quite fast. I was wondering how if it can be linked to work with JGitVer in some way. It would be great to cause it to fire on builds that receive a tag.

isn't that an enhancement request for the tycho-plugin instead?
What kind of integration do you have in mind?

You are probably right. Let me think this through properly. I think I have
spoken before I had my needs defined and generalized well enough. Please
leave the ticket open for now and then I will come back to you.
On Sat, 6 Mar 2021 at 18:14, Matthieu Brouillard<EMAIL_ADDRESS>wrote:

isn't that an enhancement request for the tycho-plugin instead?
What kind of integration do you have in mind?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jgitver/jgitver/issues/122#issuecomment-791893869,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAF7DEJLJ3OLV2C2DYJ4OULTCHP4ZANCNFSM4YVB7JUA
.

--
730 Hawkesbury Road
Anstead, QLD 4070
Australia
email<EMAIL_ADDRESS>


Treesitter support broken

To actually have treesitter support, this file is required.
I hereby request to support treesitter without this workaround.

Thank you for your suggestion, I'll use it.

To be honest, I am confused. Instead of fixing the issue, you integrate the workaround to your repo? I mean, this is totally appropriate as a temporary hotfix. However, I feel like the actual issue is not resolved. There is deprecated treesitter API and ts-fix.lua acts as an adapter to define hlgroups the old way and make them comply with new API. Is there someone who can do that API upgrade?

Yes, I have it there until I implement it in the proper way, using the new highlight groups names when defining the colors.

Out of curiosity: why are people downvoting?


autocomplete: Introduce topic input autocomplete

Fixes: #310
https://github.com/zulip/zulip-flutter/assets/63330019/d47b388d-f879-4ee5-b290-62c0342f61d3

Thanks @sm-sayedi! you're comments are really helpful!
I've pushed a new revision PTAL and please refer to this discussion regarding naming (streams / topics).

Thanks @sm-sayedi! I've pushed a new revision PTAL.

@Khader-1 Could you please add screenshots that show your changes to the PR description?

Thanks for the review @chrisbobbe! And Thanks @alya!
Please check the latest revision and the updated video in PR description.

I noticed that in the video, the cursor is initially in the message area. Where do we initially put the cursor when the compose box is opened?

What's the topic ordering? Is it the same as in the web app, or a different algorithm?

Please check the updated video now, @alya. It shows a clearer life cycle. Regarding the order, there is currently no sorting algorithm. I'll look into the web repo to copy its algorithm.

Thanks @chrisbobbe! I've pushed a new revision PTAL.

Thanks @chrisbobbe! I've pushed a new revision PTAL.

Thanks @chrisbobbe! I've pushed a new revision PTAL.

Thanks! Looks like there's a conflict from the just-merged #805. I think you can resolve by skipping this commit:
autocomplete test [nfc]: Refactor widget setup in `setupToComposeInput`

and using TestZulipApp instead of ZulipApp in test/widgets/autocomplete_test.dart in the main commit.
Also, if you get some test failures about TextDirection.ltr and TextDirection.rtl, run flutter run and rerun the tests, and they should succeed. (There's an autogenerated file that should get updated; let me know if that doesn't work.)

Thanks @chrisbobbe! I've resolved with a new revision. PTAL

Thanks! Marking as ready for Greg's review.

Thanks @gnprice! I've pushed a new revision PTAL.


Add more dartdoc to widgets.dart

This patch includes documentation for transitions.dart and for
scrollable.dart.

@Hixie

lgtm


  Applying "js-tobigint64-overflow.patch"...
  thread 'main' panicked at 'Could not apply patches: Error { kind: NotFound, message: "program not found" }', C:\Users\kreij\.cargo\registry\src\github.com-1ecc6299db9ec823\libquickjs-sys-0.9.0\build.rs:133:14
  note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
warning: build failed, waiting for other jobs to finish...
error: failed to compile `pandoc-katex v0.1.9`, intermediate artifacts can be found at `C:\Users\kreij\AppData\Local\Temp\cargo-installlTv7bX

I saw that you try to build the project on Windows. What’s your build environment? Windows MSVC or MinGW?
If you simply want to install the tool, you could download the pre-built binaries from releases.

I saw that you try to build the project on Windows. What’s your build environment? Windows MSVC or MinGW?
If you simply want to install the tool, you could download the pre-built binaries from releases.

it was mingw64

Please check to ensure gcc and patch are installed and available in path.


400 Bad request on anything except for GET

If i try a POST or DELETE i end up with a 400. Any ideas? Its a pretty straightforward flask app
Traceback (most recent call last): File "/var/task/handler.py", line 137, in lambda_handler return LambdaHandler.lambda_handler(event, context) File "/var/task/handler.py", line 51, in lambda_handler return cls().handler(event, context) File "/var/task/handler.py", line 132, in handler raise Exception(exception) Exception:
<!DOCTYPE html>400<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2 Final//EN">
<title>400 Bad Request</title>

Bad Request

The browser (or proxy) sent a request that this server could not understand.

Not quite sure what's going on there, but I don't think it's related to the type of request.
For instance, in a test Flask app with the current Zappa version in PyPi: https://vna1tr18s7.execute-api.us-east-1.amazonaws.com/test/post
Seems to work fine.
Could it be something else about the function? Do you see anything else if you do 'zappa tail ((your env))'

Other things that might help me debug -
Where was your forms action pointing at? If you point to "/" on the server without using a domain, you'll have a problem there.
What was the content-type of the POST message, and of the response?

Seems to work fine on domains as well: https://flask.gun.io/post

Any updates on this @7wonders? I'd love to fix this if there is a bug in the code, but unfortunately I'm unable to reproduce it, so I can't fix it.

I just pushed 0.16.0 which may help to debug this.

I'm going to close this if I can't get any more information, unfortunately. Feel free to open if you wan to provide more information.


User Guide Needed

Had an issue that I helped debug that points towards the fact that we could use a user guide so that people understand how to:

create an HDC that will work with the system
walk through how to run a combat (selecting characters, selecting target, etc)

Agreed, perhaps several short YouTube videos.

https://www.youtube.com/channel/UCcmq0WFFNZNhRSGwuEHOgRg


Get the cropped image size

Can I get the cropped image size directly? or I have to change the base64 type "imageDate" to image and get its size ?  Thanks

The cropped image size should be previewSize * exportZoom, which supposedly you should know when initiating cropit.


Initial test run fails when mutating code related with angular module lazy loading

Summary
As a followup to https://github.com/stryker-mutator/stryker-js/issues/2736, I'm leaving a small reproducer example of the mentioned problem.
https://github.com/Zialus/angular-cnzrtw
➜  angular-cnzrtw git:(main) ✗ npm t

><EMAIL_ADDRESS>test
> ng test

✔ Browser application bundle generation complete.
28 07 2022 03:17:52.138:INFO [karma-server]: Karma v6.4.0 server started at http://localhost:9876/
28 07 2022 03:17:52.140:INFO [launcher]: Launching browsers ChromeHeadless with concurrency unlimited
28 07 2022 03:17:52.149:INFO [launcher]: Starting browser ChromeHeadless
28 07 2022 03:17:53.358:INFO [Chrome Headless 103.0.5060.134 (Mac OS 10.15.7)]: Connected on socket lW5sqFcGnb9yNn3GAAAB with id 82348800
Chrome Headless 103.0.5060.134 (Mac OS 10.15.7): Executed 1 of 1 SUCCESS (0.031 secs / 0.027 secs)
TOTAL: 1 SUCCESS

=============================== Coverage summary ===============================
Statements   : Unknown% ( 0/0 )
Branches     : Unknown% ( 0/0 )
Functions    : Unknown% ( 0/0 )
Lines        : Unknown% ( 0/0 )
================================================================================

➜  angular-cnzrtw git:(main) ✗ npm run stryker

><EMAIL_ADDRESS>stryker
> stryker run

03:18:06 (16955) INFO ProjectReader Found 11 of 300 file(s) to be mutated.
03:18:06 (16955) INFO Instrumenter Instrumented 11 source file(s) with 25 mutant(s)
03:18:07 (16955) INFO ConcurrencyTokenProvider Creating 1 test runner process(es).
03:18:22 (16955) INFO DryRunExecutor Starting initial test run (karma test runner with "perTest" coverage analysis). This may take a while.
03:18:23 (16955) ERROR DryRunExecutor One or more tests failed in the initial test run:
	Router Module should navigate to view page
		Error: Cannot find module './customers/customers.module'
error properties: Object({ code: 'MODULE_NOT_FOUND' })
    at apply (src/app/ lazy namespace object:5:1)
    at _ZoneDelegate.invoke (node_modules/zone.js/fesm2015/zone.js:372:26)
    at ProxyZoneSpec.onInvoke (node_modules/zone.js/fesm2015/zone-testing.js:287:39)
    at _ZoneDelegate.invoke (node_modules/zone.js/fesm2015/zone.js:371:52)
    at Object.onInvoke (node_modules/@angular/core/fesm2020/core.mjs:26376:33)
    at _ZoneDelegate.invoke (node_modules/zone.js/fesm2015/zone.js:371:52)
    at Zone.run (node_modules/zone.js/fesm2015/zone.js:134:43)
    at apply (node_modules/zone.js/fesm2015/zone.js:1275:36)
    at _ZoneDelegate.invokeTask (node_modules/zone.js/fesm2015/zone.js:406:31)
    at ProxyZoneSpec.onInvokeTask (node_modules/zone.js/fesm2015/zone-testing.js:318:39)
    at _ZoneDelegate.invokeTask (node_modules/zone.js/fesm2015/zone.js:405:60)
    at Object.onInvokeTask (node_modules/zone.js/fesm2015/zone.js:284:85)
    at _ZoneDelegate.invokeTask (node_modules/zone.js/fesm2015/zone.js:405:60)
    at Object.onInvokeTask (node_modules/@angular/core/fesm2020/core.mjs:26363:33)
    at _ZoneDelegate.invokeTask (node_modules/zone.js/fesm2015/zone.js:405:60)
    at Zone.runTask (node_modules/zone.js/fesm2015/zone.js:178:47)
    at drainMicroTaskQueue (node_modules/zone.js/fesm2015/zone.js:585:35)
03:18:23 (16955) ERROR Stryker There were failed tests in the initial test run.
➜  angular-cnzrtw git:(main) ✗

@Zialus thanks for the clear bug report with reproduction example.
I think what is happening here is that the webpack plugin that is responsible for creating the different (lazy-loaded) js bundles per module can't handle the instrumented code by Stryker.
I think the way to fix it for all angular projects is with #3229 and then ignoring these mutants using heuristics.
In the meantime, there are 2 workarounds. We  can also add "!src/**/*-routing.module.ts" in the mutate property for the angular template.

Disable the mutants for the lines responsible for `loadChildren`
const routes: Routes = [
  {
    path: "customers",
+    // Stryker disable all
    loadChildren: () =>
      import("./customers/customers.module").then((m) => m.CustomersModule),
+    // Stryker enable all
  },
  {
    path: "orders",
+    // Stryker disable all
    loadChildren: () =>
      import("./orders/orders.module").then((m) => m.OrdersModule),
+    // Stryker enable all
  },
  {
    path: "",
    redirectTo: "",
    pathMatch: "full",
  },
];

Prevent the routing module from ever being mutated
// @ts-check
/** @type {import('@stryker-mutator/api/core').PartialStrykerOptions} */
const config = {
  _comment:
    "This config was generated using 'stryker init'. Please see the guide for more information: https://stryker-mutator.io/docs/stryker-js/guides/angular",
  mutate: [
    "src/**/*.ts",
    "!src/**/*.spec.ts",
    "!src/test.ts",
    "!src/environments/*.ts",
+    "!src/**/*-routing.module.ts"
  ],
  testRunner: "karma",
  karma: {
    configFile: "karma.conf.js",
    projectType: "angular-cli",
    config: {
      browsers: ["ChromeHeadless"],
    },
  },
  reporters: ["progress", "clear-text", "html"],
  concurrency: 1,
  concurrency_comment:
    "Recommended to use about half of your available cores when running stryker with angular",
  coverageAnalysis: "perTest",
};
export default config;

the problem still happening, is this in a roadmap for this or I should just apply the workaround ?


Remove usage of CompilationStrategy from Config

This commit removes the public API usage of the internal
CompilationStrategy enumeration from the Config type in the
wasmtime crate. To do this the enum was copied locally into the
crate and renamed Strategy. The high-level description of this change
is:

The Config::strategy method now takes a locally-defined Strategy
enumeration instead of an internal type.

The contents of Strategy are always the same, not relying on Cargo
features to indicate which variants are present. This avoids
unnecessary downstream #[cfg].

A lightbeam feature was added to the wasmtime crate itself to
lightbeam compilation support.

The Config::strategy method is now fallible. It returns a runtime
error if support for the selected strategy wasn't compiled in.

The Strategy enum is listed as #[non_exhaustive] so we can safely
add variants over time to it.

This reduces the public crate dependencies of the wasmtime crate
itself, removing the need to reach into internal crates even more!
cc #708

Note: This is based on https://github.com/bytecodealliance/wasmtime/pull/763 currently and the first commit doesn't need to be reviewed here. Additionally the change to Config::strategy being fallible had a bit more fallout than intended, but overall pushed the usage of anyhow and ? into more places so felt somewhat appropriate to be here. That ended up cascading in a number of otherwise-unrelated changes though.

Oh dear I don't mean to generate more work for you @peterhuene! If anything looks weird I'd be happy to help out rebasing

No worries, this is pretty straight forward to resolve.  Thanks!


style: bump up project organization

This PR might be too opinionated, so feel free to comment everywhere, or reject the changes.
But.. my goal with this PR was to

Just upgrade everything to make sure we're following the trends (in a good way)
Reset config files to latest recommended versions (and omit the over configuration that we had)

I believe in minimal configuration that makes sense rather than having the best airbnb supported config that we don't fully utilise or understand.
This should -maybe- minimise the project size, and makes it easier to understand/modify everything in the future.
Updating everything to latest version as well should help us newbies as what we read in a library docs would match what we see in the code without doing much research! (it's also cool!)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3269

Relevant Lines:
5696

💛  - Coveralls

+1 on the .gitignore I totally forgot about the .env file
no opinion on the rest though, I don't know the project enough yet to be able to decide this


BUG extended Group can() methods will only ever respect false result

The can() methods in Group seem to be a one way street when it comes to granting/denying permissions for that particular class. Permission can only ever be denied to someone who already has access, permission can't be granted to someone who by default doesn't have permission.
The can() methods in Group only seem to utilize results with a false value, differently than how Member handles it's can() methods.
Formatting has changed slightly in the 4 branch, but the code is mostly unchanged. Group also seems to be inconsistent with how those extended can calls are made compared to Member, which utilizes extendedCan().

I agree that we should allow extend('can*()') calls to override permissions both ways (true and false). This is a sensitive issue though, people could rely on the current behaviour (even by accident). So I think this is a 5.x level API change. @tractorcow thoughts?

The only risk I can see is canEdit() allowing non-admins the ability to promote themselves to admin if a malicious extension returned true for canEdit.
I would suggest:
$extended = $this->extendedCan('canEdit', $member);
if ($extended === false) {
	return false;
}
// Refactor check from canEdit()
if (!$this->denyNonAdminModifyingAdmin()) {
	return false;
}
if ($extended === true) {
	return true;
}
return true;

As a safe middle-ground to prevent this occurring. :)
Use same pattern for canDelete() calling the refactored method out, rather than calling canEdit()
I'd put this into 4.1.0 as an enhancement, personally "Allow extensions to allow write permission on groups"

@chillu I'll let you check my reasoning before I re-triage this.

I believe this has been fixed since then - if someone else runs into this we can reopen it.


stage1:init Fix forwardedPorts nil pointer dereference

Fixes #989

@yifan-gu can you review please?

related to #975

Tests fail on Semaphore. If you rebase your branch to master you'll get better logs when a test fails.

The test on Semaphore only fails with the src flavor built with systemd from git master. It was probably the double-free bug in nspawn which is now fixed by https://github.com/systemd/systemd/pull/118

@kibra Thanks for the PR!
I guess you hit the panic when using --pod-manifest? (sadly my tests haven't covered port stuff yet..)
I can think of other approaches, that we can actually change validatePodManifest() in (stage0/run.go)[https://github.com/coreos/rkt/blob/master/stage0/run.go#L172] to make the app.App non-nil. And we can further change appToSystemd() in (stage1/init/pod.go)[https://github.com/coreos/rkt/blob/master/stage1/init/pod.go#L164] so that it always use the RuntimeApp.App

However the idea above requires us to encode the pod manifest again.
Maybe we can just make sure the app.App is non-nil in LoadPod() in stage1() ?

@yifan-gu thanks for the review

Maybe we can just make sure the app.App is non-nil in LoadPod() in stage1() ?

This sounds like a good idea to me. I'll update my PR to try that approach.
@iaguis @alban I'll see if I can get the tests to pass as well...

@kibra there's one test that replies on the function signature of appToNspawnArgs. It's ok to just delete that test If you change the signature since it's already covered by pod-manifest tests https://github.com/coreos/rkt/pull/980

@yifan-gu I've taken a pass implementing your suggestions.

Another approach I can think of is that we can change validatePodManifest() in stage0/run.go to make the app.App non-nil.

I've thought a little more about that approach, and I think it makes more sense for stage1 to ensure the RuntimeApp.App is non-nil when appropriate. i.e. we should allow /var/lib/rkt/pods/$uuid/pod to be any valid PodManifest, including manifests with no app overrides.

I've thought a little more about that approach, and I think it makes more sense for stage1 to ensure the RuntimeApp.App is non-nil when appropriate. i.e. we should allow /var/lib/rkt/pods/$uuid/pod to be any valid PodManifest, including manifests with no app overrides.

Yes. I agree.
Thank you @kibra! this LGTM!


Support for multiple images per post

Hi,
Many tumblrs like let's say Steve Argyle's contain multiple images per post.
It's easy enough to get them all from API, just replace this:
      doc.xpath("//posts/post/photo-url[@max-width='1280']").map { |url| url.text.strip }

with this:
      doc.xpath("//posts/post//photo-url[@max-width='1280']").map { |url| url.text.strip }.uniq

(.uniq necessary as primary image is listed twice, one as itself, second time as part of photoset together with others).
This fetcthes the right images, unfortunately it also screws up the progress bar, as now number of images > number of posts (well, in this case), so this one like is not a proper pull request :-)

Hi @taw,
Nice catch! Tumblr API is evolving or perhaps I completely missed the photoset.
Maybe to avoid using unnecessary uniq, a cleaner solution would be:
require 'net/http'
require 'nokogiri'
uri_endpoint = URI.parse "http://steveargyle.tumblr.com/api/read?id=116490605653"
response = Net::HTTP.get_response uri_endpoint
doc = Nokogiri::XML.parse response.body
pp doc.xpath("//posts/post/photoset//photo-url[@max-width='1280']").map(&:text)

I'll look into your PR soon. Thx! :+1:
This project was a bit abandoned unfortunately by lack of time, but I have a old branch improve-threading which is waiting its time!

1-image posts have just photo, multi-image post have photoset and photo with repeated first image.
The complex part was just getting progress bar to do something sensible, as threads download images, but the total it has is number of posts, not images.
The PR has test cases, so if you want to clean up the code, just rerun tests and see if it still works.

My bad, weird API, photoset is not always there, ok... Deleting the first image seems the best way, right.
Ok about PR I'll do.


Some fixes to PyHeron (#1264)

Mainly to improve the compatibility with the Streamparse API.

Add NotCompatibleError for the streamparse APIs that are invalid for PyHeron
Add process_tick()

👍


Copy-pasted image

The steps 1 and 2 of https://docs.microsoft.com/en-us/azure/dns/dns-custom-domain#app-service-web-apps include the same image while the second step should show an image about DNS zones but it doesn't. Incorrect copy-paste?

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

ID: 806c0eb1-6e39-d04f-f870-9bab02bd168d
Version Independent ID: 6b936c21-52c3-92b5-1410-70c73a7157e2
Content: Integrate Azure DNS with your Azure resources - Azure DNS
Content Source: articles/dns/dns-custom-domain.md
Service: dns
GitHub Login: @rohinkoul
Microsoft Alias: rohink

@qrilka
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@rohinkoul
Can you please check and add your comments on this doc update request as applicable.

Apologies for the long delay fixing this. It is indeed just a typo in the image that is linked. I've fixed this and the article will republish this afternoon. #please-close


Open Collapsible programatically

Hi,
How to do this? I can't find it in the issues.
I'm trying this but it's not working:
 var col = $('.collapsible').collapsible({
                    accordion: false
});
if (this.filteractive) {
      var test = $('.collapsible-header');
      col[0].accordionOpen(test);
}

accordionOpen() doesn't exist but it's there in the source
Thanks

Simply programmaticaly click on the row you want in the accordion.
For example :$('#rowTwo').click();

ha, i forgot i can do that :)
thanks Clemzd


Visual highlight is not shown underneath a blinking cursor

This is just a minor annoyance of mine. In terminal vim, when the cursor blinks off, the character underneath the cursor appears as though it is not visually selected.
Here are some GIFs. (Please ignore the different blink rates).
This is terminal vim (bad):

This is MacVim (good):

I'm on Linux, and here the highlight is visible in gvim but not in Vim in xterm or in konsole. To me it's no big problem because 'selection' is inclusive and I know it (I wrote it into my vimrc to make sure).
Best regards,
Tony.

FWIW, neovim does not have this issue anymore.


[BEAM-1211] Make Coders more efficient by removing unnecessary nesting

See BEAM-469 for more information about why this is
correct.

R: @lukecwik
This is a first prototype to see if we should generalize the discussion we had on [BEAM-469] about NullableCoder

I'm hoping all the ITs will pass. Because nothing should depend on the value coder being force-nested, I think

retest this please

There are some wire format tests that ensure the encoding matches a predetermined value that is hardcoded.

(the "stash work" commit is meant to be ignored). KVCoder's tests didn't break because they used a valuecoder that isn't context-sensitive.

Filed a proper JIRA issue, ready for review.
I may make a test-improvement pass next week.

retest this please

Which coders are cross-Fn-API and need to stay fixed? Should have a list. E.g., is WindowedValueCoder okay to change?

No point in worrying about binary representations right now. Someone will have to make a pass through on a coder by coder basis per SDK to make them align.

Thanks Luke! Fixed nits and merging.


remove layers of abstraction

Currently we have ember-cli/ember-cli-app-version which uses cibernox/git-repo-version which uses rwjblue/git-repo-info. I think this is too much complexity for this problem.
Also I think more and better configuration would be nice for ember-cli-app-version.
Also it seems (#49, #50, cibernox/git-repo-version#9) that we need some configuration.
Ideally I think would be to have some kind of template string the user can provide, to specify which information in which order should be shown. At least we should have a way to display everything provided by rwjblue/git-repo-info.
My suggestion is to remove cibernox/git-repo-version and directly rely on rwjblue/git-repo-info. Then we could allow the user to provide a format function that takes the info from rwjblue/git-repo-info and have a sane default. Also we could provide the prefix, which calculation is currently the most complex thing git-repo-version/index.js does.
something like this looks like a possible solution in my opinion:
formatFunction: ({prefix, sha, authorDate}) => `${prefix}${sha ? `+${sha}` : ''} ${authorDate}`,

any thoughts?

whats the downside?

what about complexity and boilerplate code?
I've had a chat in slack with @twokul about #49 and we've encountered this confusion. But I also got confused about which repo is which far to often myself.
Also for most changes this means we have to change 2 or maybe even 3 repos. This makes contribution harder.

@luxferresum i think you misunderstand, I am agree with you. I am merely asking if you have identified any downsides?

@stefanpenner yes, sorry, I totally misunderstood!
I only see one downside: We probably will loose maintainers for cibernox/git-repo-version, and if any other project then ember-cli-app-version uses it, they have to maintain it themselves or drop it as we do.

@luxferresum i think thats fine. cc @cibernox thoughts?

sure, if you find it useful let's cut the middleman

I'm happy to add more maintainers to rwjblue/git-repo-info also. It has at least @hjdivad and myself, but can also add @stefanpenner and @cibernox if they want...


You promise it's going to be awesome. How can I be sure you'll maintain your promise?

You can be sure by supporting the project financially! Otherwise you will just have to wait and see :).


Add different gradient types

Not only provide linear gradient, but also radient etc..

Commit 5641ea3e7d6027d38ba1c8944e546af4c2ead558 implemented this feature


Add support for mounting custom files during image builds

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

/approve


Cydia not working correctly

When I open Cydia on the main page I can't access anything pike featured or themes. When I choose them it colors grey.

#162 Use Reddit don't post questions here like this.
Try swiping slightly from the right to left while tapping should work or get the cydia beta.

Ok thanks

What worked for me, download from cydia: "ifile", "mobileterminal" (lordscotlands fork works for me)
Double click home and close cydia.
Then download these in safari, open them in ifile and use the install option. Do them one at a time, in the order I give:

http://apt.saurik.com/beta/cydia-arm64/tar_1.29-10_iphoneos-arm.deb

http://apt.saurik.com/beta/cydia-arm64/dpkg_1.18.10-12_iphoneos-arm.deb

http://apt.saurik.com/beta/cydia-arm64/cydia_1.1.28~b12_iphoneos-arm.deb

(All taken from this page: http://apt.saurik.com/beta/cydia-arm64/ )
Open cydia again, refresh data... possibly there are language packs needed.... I didn't need them, you may need to do some research
LTF

congrats for not being able to google


DevOps Dojo: Build & Run (Phase 2)

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I've agreed with the maintainers that I can plan this task.

Description
DevOps Dojo: Build & Run will separate the development and production environments by using Development Environment (DE) and Production Environment (PE) containers. This will be done by introducing a multi-stage build system and potentially using Docker Compose.
Phase 1 - Motivation

https://github.com/autowarefoundation/autoware.universe/issues/2654

Phase 2 - Definition

(Add link to Phase 2)

Phase 3 - Execution

(Add link to Phase 3)

Phase 4 - Integration

(Add link to Phase 4)

Purpose
Phase 2 - Definition

define structure and milestone(s)
identify gaps in current software and developers’ skills
produce actionable items
consensus on solution

Possible approaches
The Twelve-Factor App:

Codebase
Dependencies
Build, release, run

Dockerfiles
Docker Compose
Ansible
Definition of done
Consensus on Definition.

Find discussion on how to implement the dojo here: https://github.com/orgs/autowarefoundation/discussions/3651

2023-10-12 Open AD Kit Weekly
Related to the Build & Run dojo:

https://github.com/autowarefoundation/autoware.universe/issues/4497

build-and-test

builds the entire Autoware software stack once a PR is merged
sanity check to ensure all of Autoware builds

build-and-test-differential

runs once the PR is opened, builds a large part of Autoware
should reuse the artifacts from build-and-test
only builds the modified packages and the ones which are directly dependent on the modified package
this approach reduces the time to build from ~8h to less than an hour

using approach 1 is preferable

Scenario simulator is not building on Arm:

https://github.com/autowarefoundation/autoware/issues/2413
@kaspermeck-arm @ambroise-arm @mitsudome-r @oguzkaganozt  to collaborate on how to build embree on Arm

it seems to be an incompatible compiler flag but could potentially be more complicated

Containerization of Autoware

Luke has an interesting proposal https://github.com/orgs/autowarefoundation/discussions/3651#discussioncomment-7212357
@kaspermeck-arm to reach out to Leo and Luke on Ansible and immutable OS

2023-11-02 Open AD Kit Weekly
Find the WIP modular approach in the LeoDrive repo:

https://github.com/leo-drive/avte_autoware/tree/main


Update workshop to use 0.21

This PR updates the workshop to use the 0.21 binary directly.
It also updates the notebooks so that they are aligned with the changes that have been made to the BIPs including:

Change from 33 byte encoding of public key to 32 byte encoding
Change from assuming that a nonce point has a y-coordinate that is quadratic residue to a y-coordinate that is even
Change in the elements of the control block

Thanks for the review @glozow 😄 I will update accordingly later this week 👍

Updated since your review @glozow :) ready for re-review. (For some reason the 'request re-review' button does not seem to be working)

Thanks @glozow  🚀

I've pushed a bunch of very minor edits:

making sure that the exercises are fully sync'ed between the worksheets and solutions sheets
updating to use bech32m for segwit v1
standardizing on the optech style of BIP340 (instead of bip-0340)

This is great work. Thanks so much for implementing it @ellemouton and for review @glozow. Apologies that it took me so long to get round to merging.
@ellemouton - there's just one minor requested change here: https://github.com/bitcoinops/taproot-workshop/pull/168#discussion_r673864291. Totally non-blocking, and not a problem at all if you don't have time to make the suggested change.


set up aarch64-linux runner for github actions

I'd like some level of testing on ARM64 Linux if we're going to start shipping aarch64-linux native gems (see #2332).
Azure / Github Actions don't appear to support ARM64 Linux, so I guess AWS EC2 Graviton instances are the best bet?
Some docs about setting up an EC2 self-hosted runner are here:

https://github.com/machulav/ec2-github-runner

though there may be other (more official?) ways to do it.

I made good experience by using docker with qemu-user-static like here in rake-compiler-dock: https://github.com/rake-compiler/rake-compiler-dock/blob/8b75537c41fc4287211f8780cc05dc125eedcfa3/.github/workflows/ci.yml#L170
The big advantage is that you can build and execute many architectures on your local computer. Not just on some specialized CI environment. And it is almost as fast as native.

@larskanis :exploding_head: Thanks for that pointer, I think this may solve a whole class of problems for me.

For posterity: a kind note from an Actions PM with advice on how he does this for his own project: https://github.com/actions/runner/issues/1491#issuecomment-970418408

PR #2425 adds this test coverage.


Taylor - Wireframe app aspects - Crud Action - CREATE community for authenticated manager

Here are the CRUD actions for the Manager. I based these wireframes on @istoga 's flowchart but changed a few of the suggested modals. I believe the user would need more space to add/edit a community so made those separate pages. I did keep the unpublish/delete/payment page as a modal. In the Add/Edit Community, I based those on @ibimina's single-page listing https://github.com/AritDeveloperCircle/findyourpeople/issues/22

Edit Page - I grouped the sections into 3 separate forms, the manager can click the icon for the form to open to make changes. The changes automatically reflect on the main page when the manager clicks the icon and closes the individual form.

UnPublish/Delete - Pop up as warning modals before the manager confirms the action

Payment - Also a modal, but wasn't sure if we should add a short description of what's included in the Upgrade Feature. Like the length of the upgrade?

Let me know your thoughts, what could be added, changed or deleted. Thanks!

Awesome work @TaylorDJones11! 🎉
Seeing as you have not updated this ticket with any diagrams or descriptions, please update the ticket status in the Project Board to "In Progress". Thanks!

@msarit Updated

With your use of the workflows and this wireframe, I can start to see the progress of what this app can really look like, it's exciting!

Phenomenal work @TaylorDJones11 - you clearly have a solid eye for design and UI! 🎉
The App Arch meeting will demystify a lot of aspects for us, so these may be updated after that meeting. Great work!

@ibimina here above "Edit Community" Some things will need to be taken out of course but it's a good start for Irene


Checklist about HQMTool

This serves as a meta issue on the interactive HQMTool capabilities and implementation. It follows the "make it work, make it nice" philosophy. The phrase "make it nice" can be interpreted as make the implementation more Julian.
Make it work:

[ ] Incorporate functionality and write corresponding tests #15

Make it nice:

[ ] Improve export and only send out the necessary functions
[ ] Fix the license. Currently year is set to 2010 which is incorrect. Possibly remove license from all files and let the main license cover everything
[ ] Clean up include statements as some are now redundant
[ ] Do not need explicit variable type statements in most functions because they have unique names
[ ] Function names are written in camel case whereas snake case is the Julia standard. Change this globally so, for example, getBackgroundGridSize becomes get_background_grid_size
[ ] Keep the docs consistent with all these updates
[ ] Convert all the warnings ending in "Try again." to either (i) proper errors with @error such that way we can use @ test_throws ErrorException for testing or (ii) proper warnings with @warning
[ ] If possible, increase the "power" of the generic remove! command to encompass more routines

Minor stuff:

[ ] Indentation and spacing conventions
[ ] Plotting constants like MODEL and REFINEMENTS in Viz/VizProject.jl little unwieldy. Is there a better way?

The last one is the standard way options are done in C frameworks, using binary add to add options together. Makes it easy from the user end just to add numbers together to combine features.

The word unwieldy was not the best to use. What I meant is that these constants are not exported currently so in the demos we can write MODEL but a user playing around in a Julia REPL would need to write HOHQMesh.MODEL. This is something that should be made simpler

Great tool! I haven't looked at the PR in detail, but I watched another nice demo of @andrewwinters5000 today. You could think about renaming some functions to be in accordance with interfaces from Base. For example, push! and pop! might be the kind of interface used instead of add! and remove!.
Take this with a grain of salt - I haven't really looked at the PR yet.

Hi @ranocha. You are assuming that the users will be Julia programmers. Presumably the users could be complete non-programmers, or programmers from other languages who just want to generate a mesh, so terms like push! and pop! have no meaning. User facing routines should be descriptive, usually with a verb plus noun like addCurve, etc. I did push^H^H^H^H add generics (since at the moment there are no good GUI frameworks as far as I can tell) so that a more familiar user could avoid typing so much. But ultimately, all (user facing) commands should be "guessable" and "rememberable".

But ultimately, all (user facing) commands should be "guessable" and "rememberable".

:+1:


[Feature Request] SSH port forwarding in Connections settings

Hello!
I think Terminus has great potential and I would be great to have the ability to forward ports on the SSH connections tab; more or like this functionality of bitvise. Specially the client-to-server port forward function; like this:
.

I would love to have SSH forwarding as well. Unless it's already implemented and I've missed it.)

Just an update for this issue, port forwarding is implemented now https://github.com/Eugeny/terminus/releases/tag/v1.0.98

Reaching out on this old item, I have some connections that always need an active port forward, at this time I need to manually create it each time after I connect, that is a very useful capability however is there anyway to make them persistent and automatic each time that connection is opened...


DBZ-7139 MongoDB data collection filter requires replica set specification on blocking/initial snapshot execution

Use namespace instead of identifierto match collections to be snapshotted

@mfvitale Applied, thanks!
@jcechace Yes, let's remove it as replica_set is removed in the next release. I'll just highlight the change in the release notes.


There is no "empty.html" file

In documentation there is instruction:

Use the empty.html and start right away building your own pages.

But there is no empty.html page in repository!

There is a ````plain_page``` template. Please, make a PR.

Documentation it's also up to date from that point of view ( plain_page.html )


assert metadata when passing EventMessages in ResultValidator#expectEvents()

This makes asserting expected events and metadata easier than using the ResultValidator#expectEventsMatching(..)

Metadata is now checked with a dedicated Matcher for Map<String,Object>.

@smcvb do you want the overloaded function on the same branch?


v0.3.11 error win32 x64

:warning:Please attach %TEMP%\vscode-java-installer\log.txt and %TEMP%\vscode-java-installer\AdoptOpenJDK-MSI.log for more details.
version: 0.3.11
env:
JAVA_HOME: C:\Program Files\Java\jdk-14.0.2
PATH:
C:\Users\i\AppData\Local\Google\Chrome\Application
C:\Windows\System32
C:\Program Files\Java\jdk-14.0.2\bin
C:\Program Files\Git\cmd
F:\Microchip\xc8\v2.31\bin
F:\flutter\bin
C:\Program Files\Java\jdk-14.0.2\bin
JDKs:
C:\Program Files\Java\jdk-14.0.2	env.JAVA_HOME,env.PATH,env.PATH,DefaultLocation

log:
[6:30:41 PM] arch:x64
[6:31:15 PM] Download Visual Studio Code ... [Completed]
[6:31:15 PM] Install Visual Studio Code
[6:32:19 PM] Install Visual Studio Code ... [Completed]
[6:32:19 PM] Configure Java Settings
[6:32:19 PM] Error occurred
[6:32:19 PM] Error:  Please check if you have access to Windows Registry.

vscode detection was failed because you don't have access to windows registry. We've fallback to check default locations in 0.3.13, you can try the latest release . Let me know if it works.


Time signatures

Adds customizable time signatures from 1/2 to 16/16. Default is 4/4.
(The first number indicates how many beats in a section. The second indicates the step duration, the higher the shorter & faster)
EDIT: Now includes mid-song time signature changes.
Resolves #104

nice, this can help people do swing covers of songs now, thanks for committing this!

ooohh..

just checked it does support custom signatures, I love it!!

It will be useful for MANY chartings! It should be merged ASAP. Let's hope that @ShadowMario or anyone else in the Dev team merges it. :)

rlly hoping this gets merge it'd be good if it did

would it be possible to have it so that you can change time signatures mid-song, similarly to how you can do that with bpm?

would it be possible to have it so that you can change time signatures mid-song, similarly to how you can do that with bpm?

Probably, I haven't gotten around to trying it out yet

I want to pull so badly but shads fixin some bugs. maybe after he does i can poop da scoop

i have nothing to comment on this pr other than the fact that i love this commit name.

can't wait for this to be pulled, literal game changer for charting

just got around to testing this PR since i wanted to see it with Fresh (Week 7 Update)
it seems the sections just speed up instead of following the time signature (look at icons and gf)
https://user-images.githubusercontent.com/15317421/155064810-ab712d5e-3403-4d6c-9ffa-8f79641e70fd.mp4
the sustains also seem to break after a time signature change

here's the chart file modified for this pr's time sig stuff if you wanted to check it out
fresh-(week-7-update).zip

I think the speed happens cause the number of beats per measure (4) doubles

The beats go twice as fast because the denominator was doubled, due to that the step duration is halved
I'll look into the sustains

can you uhhh make it work with 0.5.2h

i know this is closed and everything but i only just saw this PR, this is super cool and im glad you put in the work to add a little feature i recommended back in september LOL. seriously good stuff though, would love to see this get updated/picked up and added to psych now


Zeros on longitude variable

MCVE Code Sample

# Your code here

Expected Output
Problem Description

Output of xr.show_versions()

# Paste the output here xr.show_versions() here

Hi,
I would like to use xarray for our Copernicus Sentinel-2 data in satellittdata.no which are in NetCDF-4/CF. However, the longitude variable gives a lot of zeros when using xarray, but the correct values using netCDF4 Dataset reader:
ncin_s2 = xa.open_dataset('http://nbstds.met.no/thredds/dodsC/NBS/S2A/2019/08/27/S2A_MSIL1C_20190827T123701_N0208_R095_T39XVL_20190827T143509.nc')

Reading using OPeNDAP.

That URL doesn't seem to work: http://nbstds.met.no/thredds/dodsC/NBS/S2A/2019/08/27/S2A_MSIL1C_20190827T123701_N0208_R095_T39XVL_20190827T143509.nc

It is an OPeNDAP URL. If you add ".html" or ".dds" you should get some
information in a web browser. xarray and netCDF4 reads the url as given..
tor. 26. sep. 2019 kl. 21:06 skrev Deepak Cherian<EMAIL_ADDRESS>
:

That URL doesn't seem to work:
http://nbstds.met.no/thredds/dodsC/NBS/S2A/2019/08/27/S2A_MSIL1C_20190827T123701_N0208_R095_T39XVL_20190827T143509.nc
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/issues/3347?email_source=notifications&email_token=AF7PQWQ2PDJLPNCOYCGDIADQLUB2NA5CNFSM4I25ZB6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7WUNWY#issuecomment-535643867,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF7PQWTVCNZH3XGGBYX7D5DQLUB2NANCNFSM4I25ZB6A
.


训练弹窗中止

问题类型：模型训练
问题描述
export CPU_NUM=4 # for example, set CPU_NUM as number of physical CPU core which is 4.
!!! The default number of CPU_NUM=1.
2021-06-25 17:40:48,234-INFO: If regularizer of a Parameter has been set by 'fluid.ParamAttr' or 'fluid.WeightNormParamAttr' already. The Regularization[L2Decay, regularization_coeff=0.000500] in Optimizer will not take effect, and it will only be applied to other Parameters!
2021-06-25 17:40:55,698-WARNING: /Users/RoyGuo/desktop/paddlex_workspace/projects/P0009/T0009/output/epoch_30.pdparams not found, try to load model file saved with [ save_params, save_persistables, save_vars ]
/Users/RoyGuo/Downloads/paddlex_gui_mac_v1.1.7/Contents/MacOS/paddle/fluid/executor.py:1093: UserWarning: There are no operators in the program to be executed. If you pass Program manually, please use fluid.program_guard to ensure the current Program is being used.
Process Process-1:2:
Traceback (most recent call last):
File "multiprocessing/process.py", line 297, in _bootstrap
File "multiprocessing/process.py", line 99, in run
File "paddlexui/pms/model_tasks/tasks.py", line 68, in _call_paddlex_train
File "paddlexui/pms/model_tasks/train/detection.py", line 216, in train
File "site-packages/paddlex/cv/models/ppyolo.py", line 350, in train
File "site-packages/paddlex/cv/models/base.py", line 250, in net_initialize
File "site-packages/paddlex/utils/utils.py", line 313, in load_pretrain_weights
Exception: Optimizer parameter /Users/RoyGuo/desktop/paddlex_workspace/projects/P0009/T0009/output/epoch_30/@LR_DECAY_COUNTER@ doesn't exist, the training process can not be resumed due to optimizer set now and last time is different. Recommend to use pretrain_weights instead of resume_checkpoint
====================
请在这里描述您在使用过程中的问题，如模型训练出错，建议贴上模型训练代码，以便开发人员分析，并快速响应

/Users/RoyGuo/desktop/paddlex_workspace/projects/P0009/T0009/output/epoch_30/@LR_DECAY_COUNTER@ doesn't exist, the training process can not be resumed due to optimizer set now and last time is different. Recommend to use pretrain_weights instead of resume_checkpoint

因为这次训练的优化器设置跟上次不一样，所以无法恢复训练


Curie IMU module gives incorrect values

The Curie IMU module seems to have issues with incorrect bit-masking applied to register values. According the to the datasheet it is a 16-bit IMU but the module seems to be masking for 8 bits which means values are always positive and jump from 0 to 16000 when it should be going into negative values.
I have confirmed this using the latest MRAA and UPM versions.

Hi @srware thanks for pointing this out. I can add it to our todo list however, if you only need the data and no advanced features, take a look at the bmi160 driver we have. Patching the Firmata sketch as mentioned in #505 should enable this alternate driver too.


score counter not working

Describe the Bug
The rank and percentage isn't updating. It's locked at 100% SS.
Screenshot

Some attachments
CountersPlus.ini and log:
2<IP_ADDRESS>.50.40.log
CountersPlus.ini.txt
Other mods
I used Modassistant to install CountersPlus besides these other mods:

ScoreSaber, Songcore, SongDataCore
CameraPlus
CustomAvatar
Mapping Extension
LevelPackPanelMover
SongBrowser
TheTrashMan
All the dependencies needed

Thanks.

Hi,
Seems like a bit more information is needed. Can you grab and upload 2<IP_ADDRESS>.50.40.log from the general Logs folder rather than the Counters+ specific folder? It's just a directory up from where the Counters+ log was.

Ok there is a NullReferenceException thrown by CountersPlus in ScoreCounter.CreateText ().
general_2<IP_ADDRESS>.50.21.log

Ah, I think I know what's up. I'll get to it as soon as I can.

Fixed in commit 644bdcca0040fa71c58edaa1cb800008b6401c97. This fix will appear in the next update to Counters+.


Deluge not downloading

Hi there,
I hope I'm not bothering you here...
I followed every step (without VPN) but somehow deluge won't download anything.
This is the log from Docker.
deluge  | 20:34:49 [INFO    ][deluge.core.rpcserver         :1622] Deluge Client connection made from: <IP_ADDRESS>:34448 deluge  | 20:34:49 [WARNING ][deluge.config                 :477 ] Unable to open config file: /config/label.conf because: [Errno 2] No such file or directory: '/config/label.conf' deluge  | 20:34:49 [WARNING ][deluge.config                 :502 ] Unable to backup old config: [Errno 2] No such file or directory: '/config/label.conf' deluge  | 20:56:28 [INFO    ][deluge.core.torrentmanager    :1622] Torrent testfile from user "localclient" added deluge  | 20:56:28 [WARNING ][deluge.core.torrentmanager    :1622] Torrent id not in torrents loading list: 'xxxxxxxxxxxxx' deluge  | 20:56:45 [WARNING ][deluge.httpdownloader      :1622] Error occurred downloading file from "http://xxxxxx/": DNS lookup failed: no results for hostname lookup: xxxxx. deluge  | 20:56:50 [WARNING ][deluge.httpdownloader      :1622] Error occurred downloading file from "http://all/": DNS lookup failed: no results for hostname lookup: all.
I've tried this setup on a VM, Remote server and local. but get this error in all cases. Hoping you can help a bit here.
Kind regards.

Hi, are you able to open the deluge UI and see the torrents?

Hi, are you able to open the deluge UI and see the torrents?

Yes. I can access the UI and the torrent shows. I've also checked if the tracker is actually online which it is.

that looks like the DNS lookup does not work, I have some issues like that one without VPN if I use the DNS from my ISP, as they block all the major torrent sites. are you using it or a DNS like google or Cloudflare?
are you able to download something like the Ubuntu distro like https://releases.ubuntu.com/22.04/ubuntu-22.04.3-desktop-amd64.iso.torrent?


Invalid type errors

I'm seeing errors like
14:53:28 [ERROR] caused by: invalid type: null, expected f64 at line 1 column 159

while trying to construct a custom Criterion. It seems the benchmark runs properly anyways, but I'm not sure and anyways if it's not a problem, there should be no error (and maybe, if it's really an error, the error message could be more telling).
Any idea what this is about?

My first guess is that you've got a corrupted JSON file somewhere, so I'd suggest you delete the target/criterion directory.
If that doesn't help, could you post your code somewhere?

Yes, that was it, thanks.
Looks like this match leads to simply logging the serde json error without telling what's going on.
Would you be open to extending error to improve the error message here? I'd give it a try.

Certainly! Better error messages are always welcome.


[Features?] Parallel installation of multiple packages from source

Please always follow these steps:

[ x] Confirmed this is a problem with running a brew command and not brew installing or the post-install behaviour of one or more formulae? If it's a formulae-specific problem please file this issue at the relevant tap e.g. for Homebrew/homebrew-core https://github.com/Homebrew/homebrew-core/issues/new
[ x] Ran brew update and retried your prior step?
[ x] Ran brew doctor, fixed all issues and retried your prior step?
[ x] Ran brew config and brew doctor and included their output with your issue?

To help us debug your issue please explain:

What you were trying to do (and why)

brew install or brew upgrade different packages in different bash session.
running multiple brew installation at the same time.

What happened (include command output)

a. xz is locked or used by another process
b. pkg-config is locked  or used by another process
c. Error: Another active Homebrew update process is already in progress.
Please wait for it to finish or terminate it to continue.

What you expected to happen

Adding new feature for "--ignore-locks" , and "--skip-updates", which means, the user knows the consequence and wants to run the brew upgrade parallel.

Step-by-step reproduction instructions (by running brew commands)

Features
Please replace this section with:

a detailed description of your proposed feature
Adding new feature for "--ignore-locks" , and "--skip-updates", which means, the user knows the consequence and wants to run the brew upgrade in parallel.

the motivation for the feature
**--ignore-locks : To speed up automation script to pull or install multiple bottles or initiate several builds for high speed networks/ high end servers.
**--skip-updates : Sometimes, the user may want to reduce the numbers of calls for updates when there is a need to build several packages at the same time.

how the feature would be relevant to at least 90% of Homebrew users
**As we know, by using 'brew deps --tree ' feature,  experienced user may avoid conflicts in the parallel installation of packages. Once such feature is available, those users who handles stronger server may have chance to create multi-installation recipe, instead of sequential-installation, which may fruther reduce overall fresh installation runtime.

what alternatives to the feature you have considered

To speed up automation script to pull or install multiple bottles

Do this with brew fetch.

initiate several builds for high speed networks/ high end servers.

We already maximise the build process for the number of cores on your machine. Running multiple in parallel in this way, as a result, would be slower rather than faster.
Passing on this but thanks for the write-up.

Thanks, Mike for your feed back. I tested it out, fetch does help download multiple bottles at a time and help speed up installation. A better logic maybe applied for further need.
brew fetch glassfish& brew fetch jenkins& brew fetch dex2jar& brew fetch tomee-plus
brew install glassfish jenkins dex2jar tomee-plus


Declaration emit is incorrect when type parameter names collide

function f<T>(p: T) {
    let g: <T>(x: T) => typeof p = null as any;
    return g;
}

Calling f(10) will produce something of type <T>(x: T) => number, however, the declaration emit is the following:
declare function f<T>(p: T): <T>(x: T) => T;

But calling that f produces the type <T>(x: T) => T.

I'm looking forward for a solution to this issue. It would be a big problem when polymorphic unification gets implemented, because generic parameters must be instantiated with a copy of their type parameters for inference to properly work.
declare function id<X>(x: X): X;
declare function mapPair<A, B, C, D>(
    fx: (a: A) => B,
    fy: (c: C) => D
): (a: A, c: C) => [B, D];

mapPair(id, id) // = mapPair<X, X, X, X>, but should be mapPair<X1, X1, X2, X2>


feat: add matriculation number param

fixes #89
fixes #134
Added an optional matriculation number encapsulated with the Matric class.

Merge conflict is a simple fix, the lines conflicting are exactly the same except that one has been wrapped around due to the line length.

The updated code looks clean and neat! I see that comprehensive testing has also been added for this new parameter. Merge conflict with #136 should also be easily resolved.


[proxyd] Blast API backend provider not working with consensus aware enabled

Describe the bug
Blast API backend provider not working with consensus aware enabled.
To Reproduce
Sample configuration (without credentials):
#############################################
# Configuration file for proxyd            #
#############################################

#############################################
# Server configuration                      #
#############################################
[server]
rpc_host = "<IP_ADDRESS>"
rpc_port = 8080
ws_host = "<IP_ADDRESS>"
# Set the ws_port to 0 to disable WS
ws_port = 0
# Maximum client body size, in bytes, that the server will accept.
max_body_size_bytes = 10485760
max_concurrent_rpcs = 1000
# Server log level
log_level = "debug"

#############################################
# Metrics configuration                     #
#############################################
[metrics]
# Whether or not to enable Prometheus metrics.
enabled = false
# Host for the Prometheus metrics endpoint to listen on.
host = "<IP_ADDRESS>"
# Port for the above.
port = 9761

#############################################
# General backend configuration             #
#############################################
[backend]
# How long proxyd should wait for a backend response before timing out.
response_timeout_seconds = 2
# Maximum response size, in bytes, that proxyd will accept from a backend.
max_response_size_bytes = 5242880
# Maximum number of times proxyd will try a backend before giving up.
max_retries = 3
# Number of seconds to wait before trying an unhealthy backend again.
out_of_service_seconds = 600
# Maximum latency accepted to serve requests, default 10s
max_latency_threshold = "30s"
# Maximum latency accepted to serve requests before degraded, default 5s
max_degraded_latency_threshold = "10s"
# Maximum error rate accepted to serve requests, default 0.5 (i.e. 50%)
max_error_rate_threshold = 0.3

#############################################
# List of backends                          #
#############################################
[backends]
[backends.infura]
rpc_url = "https://sepolia.infura.io/v3/XXXX"
max_rps = 3
max_ws_conns = 1

[backends.blast]
rpc_url = "https://eth-sepolia.blastapi.io/XXXX"
ws_url = ""
username = ""
password = ""
max_rps = 3
max_ws_conns = 1

#############################################
# Backend groups                            #
#############################################
[backend_groups]
[backend_groups.main]
backends = ["blast", "infura"]
consensus_aware = true

#############################################
# Mapping of methods to backend groups      #
#############################################
[rpc_method_mappings]
eth_getBalance = "main"
eth_chainId = "main"
eth_blockNumber = "main"

Expected behavior
proxyd working with Blast API. I don't know exactly what is missing or behaving wrongly, it may be an issue with Blast API.
System Specs:

OS: macOS 11.6 (M1)
Package Version (or commit hash): 918459c4428bafb110e2f89a4b9d79b6bd1dd27c

Additional context
Logs:
{"lvl":"info","msg":"creating poller for consensus aware backend_group","name":"main","t":"2023-12-07T09:53:12.988209+01:00"}
{"consensusBackends":"","filteredBackends":"blast, infura","lvl":"dbug","msg":"group state","proposedBlock":"0x0","t":"2023-12-07T09:53:12.988229+01:00"}
{"addr":"<IP_ADDRESS>:8080","lvl":"info","msg":"starting HTTP server","t":"2023-12-07T09:53:12.988286+01:00"}
{"lvl":"info","msg":"started proxyd","t":"2023-12-07T09:53:12.999266+01:00"}
{"err":"backend returned an invalid response","lvl":"warn","msg":"error updating backend sync state","name":"blast","t":"2023-12-07T09:53:13.048915+01:00"}
{"finalizedBlockNumber":"0x49d6bb","inSync":"false","lastUpdate":"0001-01-01T00:00:00+0000","latestBlockHash":"0x6a7bf0f90e6b8c615782d9b3aa6db4d027b6f70516b298464259ac5f43a05e0b","latestBlockNumber":"0x49d6f6","lvl":"dbug","msg":"backend state updated","name":"blast","peerCount":50,"safeBlockNumber":"0x49d6d8","t":"2023-12-07T09:53:13.236293+01:00"}
{"finalizedBlockNumber":"0x49d6bb","inSync":"true","lastUpdate":"0001-01-01T00:00:00+0000","latestBlockHash":"0x6a7bf0f90e6b8c615782d9b3aa6db4d027b6f70516b298464259ac5f43a05e0b","latestBlockNumber":"0x49d6f6","lvl":"dbug","msg":"backend state updated","name":"infura","peerCount":100,"safeBlockNumber":"0x49d6d8","t":"2023-12-07T09:53:13.723187+01:00"}
{"lowestLatestBlock":"0x49d6f6","lvl":"dbug","msg":"validating consensus on block","t":"2023-12-07T09:53:13.989265+01:00"}
{"err":"backend returned an invalid response","lvl":"warn","msg":"error updating backend sync state","name":"blast","t":"2023-12-07T09:53:14.024626+01:00"}

Same issue with alchemy + base. They don't expose eth_syncing for base. Can we have an option to skip sync check just like we have skip peer count check?


support standard credential interfaces

Operating System version: Multiple
Firebase SDK version: All
Library version: All
Firebase Product: multiple (auth, database, storage, etc)

Feature request to support standard Google credential interfaces to support more custom credential integrations.
https://github.com/google/google-auth-library-nodejs
Currently this library uses bespoke approaches such as:
https://github.com/firebase/firebase-admin-node/blob/5d43af1e29c2afcbb84fa168e720874fa4211e4f/src/firestore/firestore.ts#L90

I ran into several issues last time I tried supporting google-auth credentials (can't remember what those were exactly right now). But if you want to support more custom credentials, you can do so by implementing the Credential interface provided by the Admin SDK: https://firebase.google.com/docs/reference/admin/node/admin.credential.Credential

@ptone can you comment on how you think this should be exposed to the developers? What should the admin.initializeApp() method accept? I'm making some progress towards natively supporting google-auth-library, but what they consider to be "standard" Google Credentials are quite, umm, unusual (at least compared to other languages).
These are the options I can think of:
Option 1
This is what other Node.js GCP libraries support. But this doesn't support refresh token credentials, which is something we do support today.
// service account
admin.initializeApp({
  keyFilename: 'path/to/serviceAccount.json',
});

// or
admin.initializeApp({
  credentials: {client_email, private_key},
});

// ADC
admin.initializeApp();

Option 2
This has more in common with how the credentials are exposed in other languages (Java, Go, C# etc). Notice that this calls a bunch of async APIs at the initialization, which causes problems when trying to initialize the app in the global scope.
import {auth} from 'google-auth-library';

// service account OR refresh tokens
const file = require('path/to/file.json');
admin.initializeApp({
  credentials: auth.fromJSON(file),
});

// service accounts ONLY
const credentials = await auth.getClient({
  keyFilename: 'path/to/serviceAccount.json'
});
admin.initializeApp({credentials});

// or
const credentials = await auth.getClient({
  credentials: {client_email, private_key},
});
admin.initializeApp({credentials});

// ADC
admin.initializeApp();

// or
const credentials = await auth.getClient();
admin.initializeApp({credentials});

Option 3
import {GoogleAuth} from 'google-auth-library';

// service account
admin.initializeApp({
  credentials: new GoogleAuth({
     keyFilename: 'path/to/serviceAccount.json',
  }),
});

// ADC
admin.initializeApp({
  credentials: new GoogleAuth(),
});

// this might work for refresh tokens
const credentials = require('path/to/file.json');
admin.initializeApp({
  credentials: new GoogleAuth({credentials});
})

GCP Node.js libraries seem to follow the configuration mechanism outlined here: https://github.com/googleapis/google-cloud-node/blob/master/docs/authentication.md

Thanks for looking at this @hiranya911
I recall Python libs having the cleanest interface definition of a token, basically duck type so long as you have a refresh method, and a access_token and expiration set of attributes. The client library will call your refresh method as needed.
So I see this less about initializing with the well-known credential data, and more about providing an interface.  In go with would be the tokensource https://godoc.org/golang.org/x/oauth2#TokenSource
Anything that can provide a token, can be used with the client-libs.

Thanks @ptone. I think in our case that interface is Credential: https://firebase.google.com/docs/reference/admin/node/admin.credential.Credential. Admin SDK already accepts any object that implements this interface.
However, it seems all the Node.js GCP libraries are designed to work with only 2 types of credentials:

service account credentials
application default credentials

This means APIs like Firestore and GCS will not work with custom credentials. I will follow up with the GCP Node.js team and try to figure out what their thinking here is.