Build is broken on Ubuntu 20.04

Current build (commit 9e1566118) is broken on Linux Ubuntu 20.04.1 LTS.
All packages from Ubuntu repositories.
GNU Make 4.2.1
go version go1.13.8 linux/amd64
$ make
.....

go test -v ./discovery
discovery/seed.go:12:2: cannot find module providing package cqfn.org/degitx/proto/go/degitxpb
make: *** [Makefile:21: test] Error 1

@ivanbukhtiyarov have you done make proto before?

@stepanvalyavskiy , yes, with the same error message.

Maybe some documentation update on project setup is required?

Same on macOS, reading https://cqfn.org/degitx/proto/go/degitxpb?go-get=1: 404 Not Found after  go get -insecure cqfn.org/degitx/proto/go/degitxpb

@ivanbukhtiyarov cqfn.org/degitx is a local model (see go.mod) it's not supposed to fetch it remotely.

I see. @ChGen try to add the project to GOPATH, it builds successfully for me.

After executing command below I've got another error, after that I executed make and it was fully successfull.
GOPATH="$GOPATH:$PWD" make

Also, after git clean -fdx I still have successful built with regular make. I think those results still cached somewhere.
So, there's still some issue with build system.

WSL 1, ubuntu 20.04 lts, go1.13.8 linux/amd64, protobuf-compiler and all packages from the Ubuntu repo.

make proto
go mod tidy
make

was built fully successfully.
@g4s8 I suppose we need to add go mod tidy for updating dependencies to the Makefile in the beginning of "all" target  or to the build guide

@ivanbukhtiyarov I tried to reproduce you fix, but it doesn't work. I tried on clean Docker container with:
docker run -it --rm -v $PWD:/r golang:1.15
# make proto
# go mod tidy
# make
...
ok  	cqfn.org/degitx/locators	0.002s
go test -v ./discovery
go: finding module for package cqfn.org/degitx/proto/go/degitxpb
discovery/seed.go:12:2: cannot find module providing package cqfn.org/degitx/proto/go/degitxpb: unrecognized import path "cqfn.org/degitx/proto/go/degitxpb": https fetch: Get "https://cqfn.org/degitx/proto/go/degitxpb?go-get=1": x509: certificate is valid for www.github.com, *.github.com, github.com, *.github.io, github.io, *.githubusercontent.com, githubusercontent.com, not cqfn.org
make: *** [Makefile:21: test] Error 1

It seems the problem is that go can't find generated proto go files in proto/go/degitxpb/ (by package  cqfn.org/degitx/proto/go/degitxpb)

@ChGen can we close?


UpdateContact Method is updating all numbers in the contact instead of a specific number

Working on an app where we need to change the prefix of numbers from XXX to YYY.
The contact list is well populated and all is working fine. However, updating 1 specific number for a Contact is not working well. UpdateContact is updating all the numbers for this Contact with the new number.
We are able to get the list of contacts found in the contact book via this code:
//phoneContacts => all the contacts found in the address book
//contactDetails => all the contacts that have the prefix number that needs to be replaced.
//prefixNumber => is the prefix number that user wants to replace.
handleUpdate = () => {
const { phoneContacts, contactDetails, prefixNumber } = this.state;
var updated_select = 0;
var record = null;

 Contacts.getAll( (err, contacts) => {

 _.forEach(contactDetails, (contactD) => {

   if (contactD[1] == true) {

     //where contactD[5] is the specific phone number
      var num = contactD[5];
      num = num.replace(prefixNumber, '010');

     _.forEach(contacts, (cont) => {
        if (cont.recordID === contactD[0])
           //finding the record to be updated in the Contact List
           record = cont;
      });

      //Replacing the record phone numbers with the updated one.
      record.phoneNumbers[contactD[3]] = { label: contactD[4], number: num };

      //Updating the contacts as per the guide linked above
      //when we verify the values of record, it contains the correct values. That is, it has the number that has been updated as well as the other numbers for this contact.
      Contacts.updateContact(record, (err) => {
        if (err) {
          _alert('error', 'Application error: Update unsuccessful.');
          return;
        }
      });
       ++updated_select;
      }

 });

 ToastAndroid.show(updated_select + ' rows have been updated', ToastAndroid.SHORT);
  this.handleSearchContacts();
  return;
});
}

So as a concrete example. We are changing the prefix number from 555 to 010 If User X has
phoneNumbers: [
   {label: "work",number: "(555) 555-5555"},
   {label: "mobile",number: "(444) 444-4444"},
  ],

On update, the app is updating both numbers for User X to:
phoneNumbers: [
      {label: "work",number: "(010) 555-5555"},
      {label: "work",number: "(010) 555-5555"},
     ],

During debugging, we verified the value of variable record and it is showing the contact with the all its details long with the specific phone number updated correctly.
Any ideas?

sounds like a bug and something we've never tested. Are you seeing this behavior on both platforms?

@morenoh149 Didn't test on both platforms. This was done on Android. Will try on iOS and update.

@selvanr211 nope, I have higher priority issues to address and limited time (only some weekends) to work on this. PRs welcome.

@morenoh149 Can you verify #296 PR. @selvanr211 I think this code is going to work in your case if you are still finding the solution you can give this a try :) .

@AnkitKaneri  thanks! We actually did a native app in Android for our need. iOS Version was working anyway. Will give the update a try and let you know.


Look into using createcommitonbranch for restore-file

https://github.blog/changelog/2021-09-13-a-simpler-api-for-authoring-commits/
https://docs.github.com/en/graphql/reference/mutations#createcommitonbranch

I assume you're referring to replacing this bit, in which case, yes plz
https://github.com/sindresorhus/refined-github/blob/32abfaaa4696053fa956f12b0186339e80f19292/source/features/restore-file.tsx#L68-L76

Hopefully they add a restore API at some point too so we can support restoring large files.
Speaking of which, if we know a size limit, we can probably just detect this limit before sending the API request.
E.G. on click immediately show the "file too big" error


feat(#25): incorrect-package lint

In this pull I've introduced new lint: incorrect-package, that checks correctness of the +package meta tail.
closes #25
History:

feat(#25): incorrect-package
feat(#25): motive

@yegor256 take a look, please

@h1alexbel thanks!


Fix 3 incorrect key names

Hi @jpcirrus :wave:, thanks for your contribution :+1
Added your PR to the backlog for testing and code review.


translate-zh

Related Issue or Design Document

Checklist

[ ] I have read the contributing guidelines and signed the CLA.
[ ] I have referenced an issue containing the design document if my change introduces a new feature.
[ ] I have read the security policy.
[ ] I confirm that this pull request does not address a security vulnerability.
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact<EMAIL_ADDRESS>from the maintainers to push the changes.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added the necessary documentation within the code base (if appropriate).

Further comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: GWnbsp:x: CSDN4399CSDN4399 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[NUTCH-2856] Implement a protocol-smb plugin based on hierynomus/smbj

[NUTCH-2856] Implement a protocol-smb plugin based on hierynomus/smbj
Draft version of a protocol-smb plugin. Lots of todo comments still,
but it seems to work.

Moving this to DRAFT status and acknowledging the PR @HiranChaudhuri thank you.
I will try to perform a full review this coming week... thank you.

Hi @HiranChaudhuri now that you've activated the plugin test target, I suggest that we implement the unit tests using the test-containers utility. An example can be found at https://java.testcontainers.org/quickstart/junit_4_quickstart/.
For example we could use
@Rule
public GenericContainer redis = new GenericContainer(DockerImageName.parse("ndipiazza/smbj-inttest:1.1"))
    .withExposedPorts(445);

Please let me know your thoughts on this.


scan_rate failing at 2800 ms.

So, here I found a strange behavior and also I want to suggest one enhancement.
Whenever you create an ExtController you can also define the scan_rate trough the rpi parameter which by default is 100.
The deal with the connect() function is that you are assuming that the consumer want to scan the controller for the previously created Tags and add the event_emitters to them to instantly emit the events on the tags. This is very nice, but what about if I dont want to inmediately scan the variables in order to controll the reading in a more granular way.
One example to give you is: If I want to do a controller read on demand to a TagGroup I'd like to do it without loosing all this great features that you are already exposing... like the reconnecting and managing in a more abstract way the controllers instead of use the basic Controller.
Yet, this can be done, on demand, but still there is an ongoing background process happening due to "un asked" scans and emitting events on tags which possible I dont want to listen.

This abstraction is great, but we can create two different functions or allow the scan and the addTagEvents at will. This will ensure performance and not wasting resources when we dont want to listen the events of the tags.
Now.. moving on the next topic... I found that when we increate the rpi for the scan rate above 2800ms, this error is thrown:

and this dont tell us much.. but in anycase, there are specific scenariosn in which I dont want to constanlty read on the PLC (like the one I told you, I want some screenshot of the values and not interested on changed evfents)
Thanks in advance to read... I hope we can support this feature.

with a slow scan rate you would want to use unconnected messaging.
// For Controller class
let controller = new Controller(false);

//For extController class
let controller = new extController(ipaddress, slot, rpi, false)

If you don't  constantly communicate with a connected messaging controller, it will drop connection.
For custom controller setup. use the original Controller class. The controller manager was a easy to use class for standard HMI type applications


⚠ Bofrost.be has degraded performance

In 95784bc, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1842 ms

Resolved: Bofrost.be performance has improved in b1f18f5 after 36 minutes.


fix(sec): upgrade com.google.guava:guava to 32.0.0-jre

What happened？
There are 1 security vulnerabilities found in com.google.guava:guava 31.1-jre

CVE-2023-2976

What did I do？
Upgrade com.google.guava:guava from 31.1-jre to 32.0.0-jre for vulnerability fix
What did you expect to happen？
Ideally, no insecure libs should be used.
How can we automate the detection of these types of issues?
By using the GitHub Actions configurations provided by murphysec, we can conduct automatic code security checks in our CI pipeline.
The specification of the pull request
PR Specification from OSCS

Thanks for the PR. We generally update to new versions of dependencies as they become available anyway, but I don't believe this is actually a security risk for OpenRefine since we don't use FileBackedOutputStream c.f. https://github.com/google/guava/issues/2575#issuecomment-1622208055
As an aside, I've got a strong preference for linking to the canonical CVE entry (ie https://nvd.nist.gov/vuln/detail/CVE-2023-2976) as opposed to a third party site.
Reviewing the Guava release notes, it looks like we need at least 32.0.1 since 32.0 is broken on Windows, but we probably want to go straight to the latest 32.1.2.


Introduce a --multi-version preference mode

Summary
By default, we try to minimize the number of versions selected for each package when forking. This PR adds --multi-version latest which instead optimizes for choosing the latest version in each fork.
I don't know if this is actually worth doing, since you still need to tell the resolver to fork, as in:
requires-python = ">=3.8"
dependencies = [
    "numpy ; python_version >= '3.8'",
    "numpy ; python_version >= '3.9'",
    "numpy ; python_version >= '3.10'",
    "numpy ; python_version >= '3.11'",
    "numpy ; python_version >= '3.12'",
]

Closes https://github.com/astral-sh/uv/issues/7190.

To make this work "as intended", I guess we'd have to fork when we notice that a package has a requires-python on it that's higher than the current minimum?

Do you think we should do this "automatically", without the need to repeat the dependency declarations? I don't know how we could, it seems extremely hard, since we'd then need to fork after we get distribution metadata.

@konstin -- Perhaps, in addition to this... when --multi-version latest is selected, should we "automatically" fork for each Python minor?

Do you think we should do this "automatically", without the need to repeat the dependency declarations? I don't know how we could, it seems extremely hard, since we'd then need to fork after we get distribution metadata.

I think we could, with that exact caveat you mentioned that we first have to look at the wheels and then fork afterwards which changes some assumptions we currently have (fwiw requires-python doesn't help us here https://github.com/astral-sh/uv/issues/8492#issuecomment-2437194019).

Perhaps, in addition to this... when --multi-version latest is selected, should we "automatically" fork for each Python minor?

Good question.
As an example, each numpy release seems to support 4 Python minor versions, so when trying to make as few forks as possible and seeing a wider range than 4 Python minor versions from oldest to latest supported, we could do with two fork. E.g. with requires-python = ">=3.7", we could pick a version that support 3.7 - 3.10 in one fork and one that supports 3.9 - 3.13 in another fork (3.11 to 3.15 would be ideal, but there's no version that supports a higher than 3.13 yet).
Personally, I'm a fan of fewer version (less error surface, less to test) and fewer forks (less versions and better performance), but that's not universal. There can be users that explicitly want that behavior, since they test any Python minor anyway and want to get as much dependency-version coverage as possible. If their users use [uv] pip install <package> a lot, this may be more desirable than a small lockfile. CC @henryiii @cbrnr @hoechenberger you have more insight into scientific package authoring than me.

An important point is a forked resolution is never "broken" - it's just a little bigger, but a universal install can be. At best, you might be protected against a new breaking change from a library (but only due to your minimum Python version!), but you'll never get old packages on new Pythons that don't work.

In my opinion, this issue is not unique to the scientific package ecosystem or tied to whether you package an app or a library. I think the main question here is: how do we interpret requirements as specified in pyproject.toml?
Taking my example from #8492:
[project]
name = "example"
version = "0.1.0"
description = "Add your description here"
readme = "README.md"
requires-python = ">=3.9"
dependencies = [
    "scipy >= 1.13.1",
]

I have always interpreted this to mean "take the highest possible version of scipy available for any given Python version". For uv, this currently means "take the highest possible version of scipy available across all supported Python versions", which means that uv will use 1.13.1 for all Python versions >=3.9. This implies that for newer Python versions, where no wheels are available, uv will have to build the package from source.
As far as I know, pip has always used the first strategy, so I am pretty sure that most Python users will expect that to be the "correct" behavior. So even though I do see the value of having the exact same set of package versions across all Python versions, this is probably not what people expect (at least if they have been using pip, which is still the standard and official package manager in Python).
I'm not sure if I'm qualified to advise on how to implement this behavior or if it should be the default, but I like the idea of resolving everything separately per Python version (and then merging where possible). And if that was not already clear, I think this should be the default behavior simply because people will be expecting it.

I'd like to second everything @cbrnr has said. FWIW I'm pretty sure conda and mamba behave in the same way (always retrieve the newest possible version for any combination of Python and required packages).

I don't anticipate making this the default. uv lock and pip install (or uv pip install, which behaves the same way) are solving different problems. pip install is resolving for the single Python version and platform that you're using at time of execution. uv lock is resolving for all supported platforms and Python versions. In that context, you generally want to minimize the differences between environments to maximize consistency and reproducibility across users and developers (ideally you'd only have a single consistent environment, but that's often not possible). In that sense, the current behavior is the right default!

I'm happy if we get a per-Python resolution with uv lock even if it won't be the default!

+1

I don't anticipate making this the default. uv lock

The problem is that it's shared with uv run. I've really grown fond of using uv run pytest on every package, regardless of whether they have special uv configuration or not. By using dependency-groups and making sure there is a dev group, it just works, except for two problems: it makes a uv.lock file that ideally needs to be ignored (minor issue), and it resolves across the whole range, not just the Python I'm using, which has the problems above - and as noted, it's not unique to scientific packages, though it hits those a bit harder. But otherwise, it's almost what I've always wanted - a way to avoid fiddling with a virtual environment and activating it. Maybe if locking itself was opt-in; it could be opt-out for projects (no build-system) and opt-in for packages (a build-system)?  People wanting locking could add tool.uv.autolock = True/False? The problem is that setting requires-python is fundamentally different then setting the range you want to lock over. Often that's just "current-version" unless you are trying to distribute the lock file! Maybe tool.uv.lock-current-only, for example? Again, since there's no reason to set python-requires on a "project" except to specify the locking range, it would make sense to have the default depend on [build-system] being present. But just because you have >=3.8 doesn't mean you want dependencies that supported 3.8 only. Libraries must support a range of dependency versions anyway, while projects don't need to.
Also, again, finding a unique set of packages that "works" over the whole range doesn't help if you get an old version of a package that's actually broken on newer Python versions but you don't know it. It's quite common that packages ship fixes for newer Python versions! Personally, in the Python ecosystem at least, I believe it's best to assume newer versions of packages are better, most library authors try to have reasonable backward compatibility and deprecation periods/warnings, and mostly ship fixes and improvements.

I agree with @henryiii, as I'm also using uv run instead of uv pip, but I was assuming that there would be a flag that can be set for uv run, uv sync, uv lock, and wherever else it might make sense. Would that not solve the issue (provided that all of this forking happens automatically without any additional user input, which is still required even after this PR is merged)?

Yeah the current iteration of this PR is such that (1) we'll solve for each Python minor automatically, no additional user input required; and (2) you can set this as an environment variable or in your uv.toml or whatever.

The problem is that setting requires-python is fundamentally different then setting the range you want to lock over. Often that's just "current-version" unless you are trying to distribute the lock file!

Do you just want a way to specify the locking range separately from the requires-python range?

I've always wanted that, and I've argued that packages like poetry. But we already have this in uv:
[tool.uv]
environments = ["python_version >= '3.10'"]

But I'm really hoping here is that there will be some sort of reasonable default that will be friendly toward libraries, at least in library mode. Most tools with locking solvers optimize really heavily on the project case, and have hidden issues for the very common case of working on libraries. I do a ton of teaching of python packaging, and it would be nice if the defaults were reasonable.
regardless of what the solution might be, it would be very nice if depending on a package like NumPy or Sphinx worked out of the box and didn't resolve old versions on newer pythons when working on libraries!
(Also, for anything that supports a Python range, I think you need to test at least at the lowest and highest versions of that range, I don't think you can get away with saying a single set of dependencies was found so it all will work)

@henryiii -- I'm just trying to understand -- are you arguing that we should have defaults that are framed around library usage? Or you want straightforward settings that work well for libraries?

I'd like both. :) Having a setting would be good, good defaults would be fantastic. :)

Sharing an alternative pitch of what we could do:
The goal is to use the latest version of each package for each python (minor) version.
Whether we would use a different version of a package depends on the requires-python of the packages. We can use this by forking each time we didn't select a package due to the requires python bound being to high, forking with the python version we didn't select.
Example:
We have requires python >=3.8 and a plain numpy requirement. We have to select 1.24.4 since this is the latest version matching our constraint. We know there's a new version with >=3.9 we ignored, so we create a fork for 3.9. In the >=3.9 fork, we select numpy 2.0.2, and create a fork for >=3.10. In the 3.10 fork, we select 2.1.2 (latest) and are done: We know covered all python versions, each newer python version would only select the same versions again. With this method, we let the actual requires python bound determine the forks.
Assuming that newer versions only increase the upper bound of python versions supported (by having built wheels for later versions), and us selecting the latest version, we get the most python version support possible. (For simplicity, i'm ignoring the no source dist cases here)
Implementation-wise, we would add a second fork point. Currently, we're only forking after retrieving and before adding dependencies, now we also fork in the version selection. Version selection is a hot path, so we need to be efficient in tracking whether we skipped a version due to requires python.
numpy requires python reference: https://gist.github.com/konstin/4d97de8458999f874912d628e88a0c49

I think what we have here is "fine" though it has the major disadvantage that we have to encode the maximum "known" Python version. Konsti's solution is definitely better, though more work to implement.

I'd also really like to see this mode be the default, particularly for libraries. But I think it can also make sense as the default for applications as well, because applications have a couple other tools to avoid unintentional forking: upper bounds and checked-in lockfiles.
If an application wants to support fewer (or no) version forks, they can restrict the python version upper bound. Unless the application is being packaged up for consumers to run themselves, or is in the middle of a transition between Python versions and needs to support both for a little while, I'd highly encourage restricting the Python version in an application as much as possible.
Applications also should check in their lockfiles. If the resolver prefers to avoid upgrading versions in the lockfile unless explicitly instructed to (the intended behavior, if I've understood correctly), then it will generally stay on an older version, which in many cases (like a new version of a library having a higher minimum Python version) will avoid forking for a newer Python.
Given those two capabilities together, I feel that this behavior could reasonably be not just the default, but the only option: if forking minimization weren't already implemented I think it would likely be a low priority to add it as an option. That said, I can imagine some application authors preferring this resolution strategy, and many having no preference. So unless you had a strong desire to limit the code paths, I think it would make sense to have some tool configuration in the pyproject.toml to instruct the resolver to minimize forking.
Perhaps that could be a setting that tells uv whether this is an application or a library. Maybe we can autodetect it from the presence of a build-system as @henryiii suggested. Or perhaps based on the presence of the uv.lock in the .gitignore if we want to have a little bit of magic around the VCS.

Until the new feature becomes available, would someone have a quick look at https://github.com/MIT-LCP/wfdb-python/pull/511 and help me figure out how to implement a per-Python-version resolution strategy using uv venv and uv pip? I tried some things, but couldn't make it work 😢...


UI - "Add Container" option missing in host view when Network Agent is deleted and purged.

Server version - V0.58.0-rc2
On a host that has few containers running , delete Network Agent instance.
Wait for Network Agent instance to be purged.
"Add Container" option missing in host view when Network Agent is deleted and purged.

version - master 2/11
Verified fixed


Create VM Wizard - VM time zone format should be compatible with VM OS and also displayed on the review report step

This includes two parts for handling the VM timezone:

Apply the fix done for previous create VM  #1118, on existing create VM flow:
If template's time zone format or VM's time zone format is not compatible with VM OS then it should change time zone to default. For Windows default time zone if: GMT Standard Time, for other OSs: Etc/GMT

Consider displaying current timezone in the review report step (not just for windows but for all OS types) so that the user will know which timezone was actually set.

Just a note that this is because Windows and Linux use different time zone names and the (default or otherwise) template time zone could be in the wrong format.

@sgratch What time zone should I choose when user sets ISO as a Provision Source? I would not set any time zone and let the backend to choose. Creating VM works fine when I don't choose any time zone in this case.

@sgratch What time zone should I choose when user sets ISO as a Provision Source? I would not set any time zone and let the backend to choose. Creating VM works fine when I don't choose any time zone in this case.

You should choose the default based on the OS, as done on current 1220 PR fix.


KVM: Enhancements for direct download feature

Description
Extend improvements on PR #2995 and #3075

Add an API method to revoke certificates from hosts
Add a background task to sync missing certificates to hosts (which for some reason could not be uploaded before)

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):
How Has This Been Tested?

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2844

@blueorangutan test

@nvazquez can you fix the conflicts?

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2872

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3678)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 27955 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3374-t3678-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_direct_download.py
Smoke tests completed. 70 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_02_upload_direct_download_certificates
Failure
0.00
test_direct_download.py

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-2883

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-3687)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 33489 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3374-t3687-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Smoke tests completed. 70 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_05_rvpc_multi_tiers
Failure
408.25
test_vpc_redundant.py

test_05_rvpc_multi_tiers
Error
434.76
test_vpc_redundant.py

Fixed merge conflict please check @nvazquez

Thanks @rhtyd
@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-10

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@anuragaw @shwstppr @DaanHoogland please review, thanks.

 I'll keep you posted as I make progress.

Packaging result: ✖centos6 ✖centos7 ✖debian. JID-31

Fixed conflicts @nvazquez can you check?
@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-42

@nvazquez can you add support for ipv6 downloads

Ping @nvazquez can you see https://github.com/apache/cloudstack/issues/3047

@rhtyd yes, but couldn't start working on it yet, rather than simple verification. The apache http client seems to be preferring ipv4

@blueorangutan package

@borisstoyanov a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-94

@rhtyd as issue #3047 has been moved to the next minor release and can be treated on a separate PR

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-115)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 28455 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3374-t115-kvm-centos7.zip
Smoke tests completed. 71 look OK, 1 have error(s)
Only failed tests results shown below:

cc @borisstoyanov

I'm having a look @nvazquez @rhtyd @anuragaw

@borisstoyanov please find the latest fix for the issues we've been through
@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-119

@blueorangutan test

@rhtyd a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-152)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 35108 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3374-t152-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_internal_lb.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_host_maintenance.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 70 look OK, 2 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_cancel_host_maintenace_with_no_migration_jobs
Failure
0.16
test_host_maintenance.py

test_02_cancel_host_maintenace_with_migration_jobs
Error
2.55
test_host_maintenance.py

test_hostha_enable_ha_when_host_disabled
Error
1.67
test_hostha_kvm.py

test_hostha_enable_ha_when_host_in_maintenance
Error
302.87
test_hostha_kvm.py

@borisstoyanov fixed, please consider rebuilding your test environment as I have slightly modified the database schema

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Ping @borisstoyanov @PaulAngus

@rhtyd @PaulAngus @borisstoyanov please also check: https://github.com/apache/cloudstack-documentation/pull/54

Thanks @borisstoyanov
@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: ✔centos6 ✔centos7 ✔debian. JID-143

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-173)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 34208 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr3374-t173-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_vpc_redundant.py
Intermittent failure detected: /marvin/tests/smoke/test_hostha_kvm.py
Smoke tests completed. 76 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_hostha_enable_ha_when_host_in_maintenance
Error
303.82
test_hostha_kvm.py


Update scribe, scribe-file to 3.8.1

Updates

com.outr:scribe
com.outr:scribe-file

from 3.5.5 to 3.8.1.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.outr" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #441.


Error: Couldn't get access token: failed to decode response: invalid character 'e' looking for beginning of value

docker部署，正常对接telegram，交互报错

同样的问题，Error: Couldn't get access token: failed to decode response: invalid character '<' looking for beginning of value

这是什么问题？

同样的问题，是不是使用api应该能稳定些

同样 碰到 , 请问 是chatgpt服务端的问题吗?

me too!

me too

me too!!!

me too!!!!

有哪位帅哥解决了吗

me 2,

+1

me 2。

:( Me too

Error: Couldn't get access token: failed to decode response: invalid character '<' looking for beginning of value
same question : (


Feign inheritance

I have been trying to get Spring cloud Feign inheritance working, but upon looking at the latest documentation http://cloud.spring.io/spring-cloud-static/spring-cloud.html#spring-cloud-feign-inheritance
I see the following note has been added.

It is generally not advisable to share an interface between a server and a client. It introduces tight coupling, and also actually doesn’t work with Spring MVC in its current form (method parameter mapping is not inherited).

I saw Feign as a simple way to share the connection contract between a client and server app. I would just add the shared interface into another jar which the client and server would both depend upon.
But from the note it appears the functionality I desire is actively discouraged due to 'tight coupling'. Surely having a shared defined contract between client and server is a good thing as it will lead to less errors.

There are a couple of issues. First there is tight coupling, the client and server may need/want to change independently of each other. Second, some things, the client only cares about one representation ([json/form/etc...) where the server needs to know about all of them. @adriancole can articulate it better then I can :-). It's just our warning. If you come asking for features based on sharing, we're likely to say no without a compelling reason.
Beyond the comment and discussion, was there a problem doing what you wanted?

I understand your point but I think it too idealistic for me. I would prefer the safety of tight coupling with a shared contract to avoid programmer errors.
I am wondering what is what is the point in offering feign inheritance as an option? As it is not advised due to your previous comment, and it does not work anyway due to the current Spring MVC implementation.

We're not forcing you to not use inheritance as you want. The point was to share common methods (like CRUD) across clients.

I also see many advantages sharing it. It works with some restriction.
Checkout this
https://jmnarloch.wordpress.com/2015/08/19/spring-cloud-designing-feign-client/
Spencer Gibb<EMAIL_ADDRESS>schrieb am Mi., 6. Apr. 2016 15:55:

We're not forcing you to not use inheritance as you want. The point was to
share common methods (like CRUD) across clients.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/spring-cloud/spring-cloud-netflix/issues/951#issuecomment-206383161


feat: implement case sensitive search queries on string field

Codecov Report

Merging #288 (f776180) into main (5a0f3de) will decrease coverage by 0.30%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##             main     #288      +/-   ##
==========================================
- Coverage   29.12%   28.82%   -0.31%
==========================================
  Files          52       52
  Lines        5297     5332      +35
==========================================
- Hits         1543     1537       -6
- Misses       3557     3598      +41
  Partials      197      197

Impacted Files
Coverage Δ

query/filter/logical.go
27.27% <0.00%> (-4.31%)
:arrow_down:

server/metadata/tenant.go
37.30% <0.00%> (ø)

server/services/v1/query_runner.go
0.00% <0.00%> (ø)

server/services/v1/row_reader.go
0.00% <0.00%> (ø)

server/services/v1/search_indexer.go
0.00% <0.00%> (ø)

query/filter/selector.go
14.70% <7.69%> (-6.13%)
:arrow_down:

query/filter/filter.go
51.33% <9.52%> (-6.57%)
:arrow_down:

schema/collection.go
76.69% <87.50%> (ø)

query/filter/key_builder.go
89.69% <100.00%> (ø)

query/search/search.go
50.00% <100.00%> (-30.00%)
:arrow_down:

Continue to review full report at Codecov.

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


Fixes issue where blueprints only ever took one item from player

The 'CheckTakeItems' function would attempt to take the same item from the player x amount of times.

It'd probably be better to fix the source function:
https://github.com/CloudSixteen/Clockwork/blob/master/Clockwork/garrysmod/gamemodes/clockwork/framework/sv_kernel.lua#L5391
self:TakeItem(self:FindItemByID(itemTable));

I made my own fix to CheckTakeItems that still uses the if statements properly. Not really my best work, as it was a hasty patch to get crafting working as quick as possible for a server, but maybe you all can fix it up.
`function Clockwork.crafting:CheckTakeItems(player, item, amount)
if (player and item and amount) then
if (amount > 1) then
local itemsToTake = {};
local itemList = Clockwork.inventory:GetItemsByID(player:GetInventory(), item);
local iterator = 1;
		for i, v in pairs(itemList) do
			if iterator <= amount then
				itemsToTake[#itemsToTake + 1] = v;
				iterator = iterator + 1;
			end;
		end;

		player:TakeItems(itemsToTake);
	else
		player:TakeItem(player:FindItemByID(item));
	end;
end;

end;`

Yeah, I did a bunch of testing by printing out the values to the console and saw the issue. I wrote a few fixes but this was the one I ended on. I originally just wanted to do a normal for loop for the amount and do itemsToTake[#itemsToTake + 1] = itemList[i]; but the indexes of itemList are the itemIDs of the item, so couldn't do it like that. Ended up with that workaround, though I think an else statement and a break would make it a bit more efficient, but not a final solution.

I just finished the code now and actually made it the way you described already haha. I think what I got written should resolve issue, I just changed around some of the variable names to be a bit more descriptive and added the else break ofc. Commit coming soon.

Sweet, glad to help!


java.lang.zip.ZipException: No entries error while converting Minecraft

Version of Boardwalk: 1.0
Version of Android: 4.2
Device model: Asus MeMoPad HD 7, Lenovo A3000
Description of the problem:
"java.lang.zip.ZipException: No entries error while converting Minecraft" displays in runRename.
https://twitter.com/egyptiansanta/status/521516515418333184
https://twitter.com/joshuaplaysmcpe/status/521530493309698049
Thanks to @egyptiansanta and @joshuaplaysmcpe for reporting this.

I can't play

No play game

Version of Boardwalk: 1.1
Version of android: 4.2.2
Device model: BLU BLU DASH 5.0
Description of the problem:
"java.lang.zip.ZipException: Dex returned 11

Version of Boardwalk: 1.1
Version of android: 4.2.2
Device model: BLU BLU DASH 5.0
Description of the problem:
"java.lang.zip.ZipException: Dex returned 11 with error:


firefox-*-bin: updates

firefox-beta-bin: 51.0b8 -> 54.0b13

firefox-devedition-bin: init at 54.0b14

Firefox DevEdition became a new product of Mozilla and is "repackaged"
Firefox Beta with its own release channel and six weeks release cycle as
other channels. It is no longer being built on nightly basis

updated the update.nix script to facilitata firefox-devedition-bin

disabling automatic updates by pointing to non existing channel

Motivation for this change
Things done

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

[x] NixOS
[ ] macOS
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

LGTM but I haven't tested it. I will do that later today if the PR is still open :)

I found the error which was already present before this update, but nevertheless i might ask here how fix it. If you open any Browser button where file selector shows up, then all channels of firefox-bin crash. I'm probably missing some dependency which is loaded dynamically, but I'm yet to learn how to detect this.
And for those who are looking for Firefox Nightly, look no further then github.commozilla/nixpkgs-mozilla :) and wow does nightly works faster.

hmm ... having a browser dialog works on nightly. i wonder what am i missing here.

Hm… Does firefox-bin set gtk3 attribute on the unwrapped version? I think wrapFirefox sets XDG_DATA_DIRS needed by gtk3 only when browser.gtk3 attribute exists.

@7c6f434c tnx for the pointer. i just inherit gtk3 in firefox-bin/default.nix (see last commit in this PR) and that fixed the problem. would this be ok solution?

Well, the main derivation adds it to passthru, but as it is a dependency already the difference is negligible.


New branch

Yeah pygments I'll remove.
And django-vote I am using for maintaining a count of the stars for a file.(i.e. StarFileView and UnStarFileView).
And I am still figuring out how to sort by popularity for which I thought that sort() function in django-vote may help.

Sorry I mean the count() function in django-vote.


Add a workaround for RuntimeException when checking self permissions

see #107

Thx!

Fixed and squashed.

:100:

Thanks!


Javadoc creates build issues

When I run "./gradlew build" I get build errors in the javadoc section. Obviously this doesn't affect much, but it would be nice if the entire build process worked.
I am compiling on Ubuntu 14.04.1 LTS with the Oracle Java 8 JDK (not openjdk). I've been out of the loop with Java for about 4 years, so I suspect this is a situation of PEBCAK.
===========gradlew output=================
:libraft-core:compileJava UP-TO-DATE
:libraft-core:processResources UP-TO-DATE
:libraft-core:classes UP-TO-DATE
:libraft-core:jar UP-TO-DATE
:libraft-agent:compileJava UP-TO-DATE
:libraft-agent:processResources UP-TO-DATE
:libraft-agent:classes UP-TO-DATE
:libraft-agent:jar UP-TO-DATE
:libraft-core:javadoc
/home/bdeetz/libraft/libraft-core/src/main/java/io/libraft/algorithm/Log.java:51: error: bad use of '>'
* @param index index >= 0 of the {@code LogEntry} to get
^
/home/bdeetz/libraft/libraft-core/src/main/java/io/libraft/algorithm/Log.java:54: warning: no @throws for io.libraft.algorithm.StorageException
@Nullable LogEntry get(long index) throws StorageException;
...
...
...
75 errors
21 warnings
:libraft-core:javadoc FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':libraft-core:javadoc'.

Javadoc generation failed.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

BUILD FAILED
Total time: 11.978 secs

i like libraft, but there is no more development, it's petty. I will help you maintain and continue to develop lib raft.

I'm happy to take a PR...
Terminal Musings: http://www.allengeorge.com/
Raft in Java: https://github.com/allengeorge/libraft/
Twitter: https://twitter.com/allenageorge/
On Fri, Sep 2, 2016 at 11:44 PM, zhuguangxiang<EMAIL_ADDRESS>wrote:

i like libraft, but there is no more development, it's petty. I will help
you maintain and continue to develop lib raft.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/allengeorge/libraft/issues/48#issuecomment-244524430,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEO9FLQdvfeC6aRLmsvm6UzLqFOQI_8ks5qmO0KgaJpZM4DFMQe
.


Invoke-WebRequest: The error message is damaged beyond repair

Steps to reproduce
iwr 'http://github-production-release-asset-2e65be.s3.amazonaws.com/'

Expected behavior
Invoke-WebRequest: Forbidden
Error:
Code: AccessDenied
Message: Access Denied
RequestId: 229E67C9F0B26053
HostId: kUI8bnH/eZxSAjznVy7+RQpaBv1yHVAP7w9Vh/Z2LYdrrPfn1E8E3y176OHzhKwwdkRMZ0RygbQ=

Actual behavior
Invoke-WebRequest:
AccessDeniedAccess Denied4GEN7KFG5Q8QAT8J/8ZJ9JizWnxWHeo0m3wU/rl7Kh7xzbV89arVlDlaSbYwEGELtFblhWEQAKxeaWmeY61sUti5ISk=

Environment data

Name                           Value
----                           -----
PSVersion                      7.0.3
PSEdition                      Core
GitCommitId                    7.0.3
OS                             Microsoft Windows 10.0.18363
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

PS Temp:\> (iwr 'http://github-production-release-asset-2e65be.s3.amazonaws.com/' -SkipHttpErrorCheck).Content
<?xml version="1.0" encoding="UTF-8"?>
<Error>AccessDenied<Message>Access Denied</Message><RequestId>AY9Q4P8SDQCRCNAJ</RequestId><HostId>teel4p+MFOck+dIyvQtwFzXV+jYfLOnyyp6wJBq1wTo9SUHLU+fG+lQtr3WYP9B/4ZND/nP9SzQ=</HostId></Error>

Then it helpfully removes the XML tags and leaves only the content: https://github.com/PowerShell/PowerShell/blob/dd73c303a86ee04afca54a2be949c18299f2d8ed/src/Microsoft.PowerShell.Commands.Utility/commands/utility/WebCmdlet/Common/WebRequestPSCmdlet.Common.cs#L1548-L1549
Perhaps that should trigger only if Content-Type is text/html or application/xhtml+xml. This response has application/xml.

I expect XML to be handled in a user-friendly way.  Blindly removing tags is not user-friendly and leaving them as they are is not either.  Remember that XML is an application format, not a text format.

Sure. Error messages, however, come in text format. 🤷

And that is why we must use a formatter to get something we can throw.

Microsoft.Rest.HttpResponseMessageWrapper referenced in PowerShellCore_format_ps1xml.cs unfortunately does not have code for converting the HTTP response body to properties that could then be nicely formatted.

I would not expect them too, consider the error page response body can be in any form, including Adobe Illustrator, because why not.  However, we should be prepared handle the cases that are common, at least HTML and XML.
The right way to format HTML is to render it to a text document with scripts disabled.
The right way to format XML is as indicated in the issue.

The right way to format HTML is to render it to a text document with default style

I'll be surprised if an HTML renderer is added to PowerShell, especially if it is then used for errors only.
On the other hand, ConvertFrom-Markdown surprised me as well.

format as YML

Do you mean YAML? Your "Expected behavior" does not have any indentation to indicate nesting of elements.

That still means we have to honour inline DTDs, including text entities.

Wasn't there some security problem with inline DTDs, too? Perhaps it was just a denial of service that could be mitigated by setting XmlReaderSettings.MaxCharactersFromEntities. Regardless, a web server that required DTD processing of error responses would only hurt its compatibility with client software, so I don't believe many servers do that.

I know, some XML formats do not think much of this theory—it will be their fault.

Pragmatically though, the error summary displayed by PowerShell should try to help the user solve the error.

format as YML

Do you mean YAML? Your "Expected behavior" does not have any indentation to indicate nesting of elements.

I am sorry, I am not a DevOps guy.  Is it better now?

That still means we have to honour inline DTDs, including text entities.

Wasn't there some security problem with inline DTDs, too? Perhaps it was just a denial of service that could be mitigated by setting XmlReaderSettings.MaxCharactersFromEntities. Regardless, a web server that required DTD processing of error responses would only hurt its compatibility with client software, so I don't believe many servers do that.

Yes, you can very easily go exponential with DTD entities.  DOS can (and should) be mitigated by limiting the total length of the result.

I know, some XML formats do not think much of this theory—it will be their fault.

Pragmatically though, the error summary displayed by PowerShell should try to help the user solve the error.

Pragmatically, putting relevant content into attributes is actively preventing users from seeing it.  The purpose of PowerShell is not to save the world.

If the HTTP server returns an error response in XML or JSON format, then the provider of the HTTP server presumably intends the response to be parsed by dedicated client software, rather than a generic web browser.
If the user of PowerShell is writing a script that sends requests to such an HTTP server, then the user presumably has access to documentation of the error response format and would benefit from seeing the response in that format. Raw XML or JSON would be fine in that case, optionally indented by PowerShell.
However, if the user of PowerShell is running a script provided by someone else and is not familiar with the intended API, then it might be better to display the response in a different format, like YAML.
I don't think the original response body should be stored in the TargetObject, if it is large. Errors accumulate in the $Error variable and could take up too much memory.
Likewise, when PowerShell shows the error response to a user, it should not blast out many pages of text and should escape or filter out terminal control characters. A user who wants the exact error response can rerun the request with Invoke-WebRequest -SkipHttpErrorCheck. That is also what scripts should do if they need to parse the error response.

You assume that re-running the request is cheap, which need not be the case.  I have already reported one issue with excessive error messages.  We have agreed that TargetObject is the way to go.  If $ERROR takes too much memory, there should be a way to GC it.

We have agreed that TargetObject is the way to go.

The decision was apparently in https://github.com/PowerShell/PowerShell/issues/2193#issuecomment-280187451. But it turns out that ErrorRecord.TargetObject is the HttpRequestMessage, rather than HttpResponseMessage. ErrorRecord.Exception may be an HttpResponseException that then carries the response.


Google Analytics

Hi there,
I followed the instructions to use google analytics, and GA does not see any connection to the website, any view, etc.
Could you tell me if the publisehd process is not valid anymore ?
Thanks

I checked the Analytics dashboard for manual installation instructions for Google Analytics. It provided me with a snippet to include in my site. I compared it to _includes/google-analytics.html included in this repo and it did not match. I replaced this file by creating a file called google-analytics in my local directory _includes with the snippet that was provided to me, which is at time of writing:
<!-- Google tag (gtag.js) -->
<script async src="https://www.googletagmanager.com/gtag/js?id=G-BD3KKE8T6Y"></script>
<script>
  window.dataLayer = window.dataLayer || [];
  function gtag(){dataLayer.push(arguments);}
  gtag('js', new Date());

  gtag('config', 'G-BD3KKE8T6Y');
</script>

This caused data to start flowing into my Google Analytics instance. If you need to troubleshoot, further check out this documentation: https://support.google.com/tagassistant/answer/10039345?sjid=15349293314530816036-EU.

@iTiamo Will you be able to submit a pull request to update the analytics code provided by Minima?

site.google_analytics : analytics ID should be added to config.yml right ?

Yes, this is already detailed in the README.
https://github.com/jekyll/minima/tree/2.5-stable?tab=readme-ov-file#enabling-google-analytics

Yes indeed sorry. Still not working. Maybe a "production"environment variable issue on github. Analytics is only working in production mode according to the README
This is a bit of an annoying struggle.

Yes, I had to include the environment variable in my GitHub pages deployment. Additionally the Google Analytics tag should be in the root of the config file. Note that the site object is populated from there. So don't create a site object in the config file yourself.

How do you include it ?
From what I got, I thought github was automatically putting the environment variable to production by default.

If you are using GH pages, you can find the environment variables in your repository's settings, environments, github-pages. I hope that helps.


[BUG] Find available cash and account value inside strategy.

Been trying for a while now to figure out how to get the cash and account value information from inside the strategy. Been looking through the library code for a while and these functions seem to not exist?
Is there anyway we could get these two features added? Or maybe they already do but it's not obvious?

You can get account details like AvailableFunds etc., by
class Teststrategy(Algo):
def on_start(self):
account = self.get_account()
print(account.get("AvailableFunds"))
get_account method is not documented, it returns dict key value pairs.


[Question] NPC vs PC Character Sheet

You mention in the settings area of this module, that using a PC Character Sheet resolves some potential incompatibilities.
Could you elaborate further what you mean by this?
Also on a more fundamental note, is there any reason to use NPC sheets at all in Foundry? I could not think of any consequence but for modules that require NPC sheets to function properly (e.g. loot modules).
Thank you for this incredibly useful tool!

If I remember correctly npc sheets do not have resources for example, and some character stats. I have no idea why you would use the NPC sheet except for the combat rating field


Update docstrings for 0.9

Current coverage is 98.40% (diff: 100%)

Merging #228 into master will not change coverage

@@             master       #228   diff @@
==========================================
  Files             9          9
  Lines           188        188
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            185        185
  Misses            3          3
  Partials          0          0

Powered by Codecov. Last update 554a4a3...4c672fa


Add proper backward compatible cases when unmarshaling authType

Seems like this is the correct mapping for older option names but if any data source is already using this they may rely on the older mapping in some way. But I assume only other one that needs to be checked besides x-ray is timescale.

I'm going to merge so I can update in sitewise & timestream


ARM "native" logic creating invalid images

Attempting to build an ARM image from an ARM64 device results in an unusable output image.  This appears to be a result of #30.
The relevant code is here: https://github.com/solo-io/packer-plugin-arm-image/blob/52faad571c70656187238268e2032160b74fe695/pkg/builder/builder.go#L304-L310
When building an ARM image on an ARM64 device, qemu is never initialized.  As a result, the image sees the host's aarch64 machine type and installs packages for the wrong architecture.
Even when building for the same architecture as the target image, not using qemu can result in breaking encapsulation.  For example, chroot alone isn't enough to prevent changing the host machine's hostname: https://unix.stackexchange.com/q/588789

I did some poking around and unfortunately, I don't think we can determine from the provided image if it's supposed to be for ARM or ARM64.  Specifically, the RaspiOS boot partition includes all kernels, not just one or the other.
I'm thinking the best approach might be adding an option for the target image architecture and have it default to ARM.  If the target arch matches GOARCH, then it can skip qemu.
I'll see if I can whip up a PR for this.  It might take me a little bit because I need to learn go first. 😁

i like the approach, just keep in mind that we should try and make the PR backwards compatible - if the new target option is missing, fallback to current logic

fixed by #146


Push first additions to create extra flexible code.

Finished testing new components but now making flexible code for mutiple input types.


Do not fail on write error

Fixes #1505
This issue occurred to us too. Found existing issue about it and made this PR.

Moved offset parsing and message parsing to single try/catch block
Reused fail backoff logic which was implemented in parse block
Made fail threshold configurable
Split consumeNextMessage to multiple smaller methods to make it more readable.

The refactoring makes the code diff difficult to read, I wasn't sure the purpose of making those new methods. Looks like the main purpose of the PR is to add mBasMessages, you can just insert this logic in place. Also what kind of errors do you see during messageWriter.write and you want to skip those errors?

Hey Henry, thanks for taking time to review my  Pull Requests :). I do understand that it is hard to review this code, since the method was not that small.
mBadMessages was introduced to make it possible to ignore all failing messages, but it is not the main purpose of this PR.
To simplify the idea, before code looked like this:
//...
try {
  // parse partition
} catch (Exception e) {
  // increment consumer.message_errors.count metric
  // handle fail, re-throw if there were > 1000 errors
}

try {
  // write message
} catch (Exception e) {
  // just re-throw
}

After this change there is only one try block:
try {
  // parse partition
  // write message
} catch (Exception e) {
  // increment consumer.message_errors.count metric
  // handle fail, re-throw if there were > 1000 errors
}

This allows us to configure secor to not fail on writes and also onsumer.message_errors.count is more correct.

Also what kind of errors do you see during messageWriter.write and you want to skip those errors?

Our error is very similar to one, described in #1505. We use custom ReaderWriterFactory implementation with custom Writer to write json messages to parquet and validate schema, so exception is a bit different though, I thought it would make sense to catch exception in our Writer, but then I saw that others have the same problem on #1505. And consumer.message_errors.count metric is increased on Consumer class, so i decided to keep on place responsible for handling this kind of errors.

Your explanation makes sense, I will merge in your PR, thanks.


It was vertically centered (regardless of the body), which does not make sense on small devices.
Now the tooltip is at the bottom, on top of the bottom action bar

it's visible
it's in a place that the user is expected to see (because of the "previous" / "next" buttons)
we avoid the previous confusion displaying a de-facto non clickable "next" button (if you saw the tooltip and the "next" button, a click on the "next" button actually meant a click on the tooltip)

Screenshot

Fixes bug(s)

Fixes: #2863

Codecov Report

Merging #2889 (df30507) into develop (ac19a9c) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           develop   #2889      +/-   ##
==========================================
- Coverage     7.04%   7.03%   -0.02%
==========================================
  Files          219     219
  Lines        10874   10895      +21
==========================================
  Hits           766     766
- Misses       10108   10129      +21

Impacted Files
Coverage Δ

...p/lib/generic_lib/dialogs/smooth_alert_dialog.dart
15.70% <ø> (ø)

packages/smooth_app/lib/pages/image_crop_page.dart
1.92% <0.00%> (-0.30%)
:arrow_down:

...ages/onboarding/knowledge_panel_page_template.dart
0.00% <0.00%> (ø)

...ib/pages/preferences/user_preferences_connect.dart
6.81% <0.00%> (-1.08%)
:arrow_down:

.../lib/pages/product/confirm_and_upload_picture.dart
0.00% <0.00%> (ø)

...app/lib/pages/product_list_user_dialog_helper.dart
0.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I'm not really fan of it, because it forces to click the tooltip to continue.
Buttons are not accessible anymore (according to your screenshot).

I'm not really fan of it, because it forces to click the tooltip to continue.

You're right, but it was already the case before. Actually you just had to click anywhere on the screen to dismiss the tooltip, not specifically the tooltip.

Buttons are not accessible anymore (according to your screenshot).

Again, the buttons were NOT accessible. They were visible, but could not be tapped on, unless you clicked before somewhere on the screen to dismiss the tooltip.
Something did not work before, and it still doesn't: I haven't changed that, I've just moved the tooltip downwards.
I could move the tooltip upwards so that the buttons are visible too if you prefer, but still, the first click is for the tooltip.
A solution would be to:

make the tooltip clickable (click = dismiss), and remove the previous full screen click capture
move the tooltip upwards, just on top of buttons => visible and active buttons

What do you think of that?

I'm not really fan of it, because it forces to click the tooltip to continue.

You're right, but it was already the case before. Actually you just had to click anywhere on the screen to dismiss the tooltip, not specifically the tooltip.

I'm note sure of that, clicking on Next would prevent this.
The real problem here is that the buttons are hidden by it.
What about highlighting the nutriscore card instead? Whatever the screen size, there would be enough place for the bottom buttons.

@g123k

I'm note sure of that, clicking on Next would prevent this.

Then please try again: this happens when we run the test+screenshots, and this happened when I coded this PR.

The real problem here is that the buttons are hidden by it.

That's why I suggested to:

make the tooltip clickable (click = dismiss), and remove the previous full screen click capture
move the tooltip upwards, just on top of buttons => visible and active buttons

What about highlighting the nutriscore card instead? Whatever the screen size, there would be enough place for the bottom buttons.

You mean the tooltip landing just below the nutriscore card?
Please consider my suggestion too.

@g123k I've raised a little the tooltip so that the next / previous buttons are visible.
I've also removed the inkwell on top that prevented a direct tap on an item.


rework generateFolders()

generateFolders() creates one or more folders within a specified directory, all of which will contain identical subfolders.

Current functionality:

If one of the folders specified in the folders parameters already exists and is not empty, it will be skipped and a warning stating which folder was skipped is thrown.

Desired functionality:

If one of the folders specified in the folders parameter already exists, it will attempt to write the subfolders to the existing folder.

If any subfolder(s) is already present, skip it and throw a warning making note of which subfolder(s) was skipped.
If all subfolders are already present then skip the entire folder, and throw a warning making note of which folder was skipped.
If there are additional subfolders already present that do not match any names specified in the subfolder parameter, throw a warning which notes these subfolders.

Any time a warning is thrown which references a subfolder, specify its parent directory within the message.
Create at minimum 1 test for each of the following scenarios:

A folder specified in the folder parameter already already exists with at least 1 but not all subfolders already present
A folder specified in the folder parameter already exists with all subfolders already present
A folder specified in the folder parameter already exists and contains subfolders which weren't specified within the subfolder parameter.

Review existing tests to ensure they are still applicable given this new behavior.
One other small change, modify the return statement as follows: return(NULL) to return(invisible(NULL)) so NULL isn't printed every time the function is called within the console.

I have some code prepared to address this issue and plan to push it to my fork once I wrap up a few tests. Don't think I'm able to formally assign this issue to myself but happy to take this on!

Great, thanks EmmaLi!  I have you assigned now, feel free to submit a pull request whenever you're wrapped up.


Distroless Support

Distroless container images are frequently used by many projects (notably the Kubernetes project uses them for almost all container images). Despite the name they are a derivative of Debian, however they lack built-in package manager support. It is however possible to add additional packages to them, see the fluent-bit images for an example of how this is usually done with multi-stage builds. Additionally they are already built in a fully reproducible fashion, so extending them with reproducible additional packages seems a good match to me.
Not sure what this all entails as I'm still getting my head around the tool, but it would be an interesting thing to consider supporting.

When creating a debianless or cryptoless image, one normally just uses debian packages for the builder image.

gcr.io/distroless/base-debian11

gcr.io/distroless/base-nossl-debian11

The "distroless" only means that the shell and package manager is removed, in the output/deployment images...


Tenant in the selection lists should be represented with the name, not ID

Displaying the generated tenant ID in the selection lists (e.g create organization, create marketplace pages) is not very user-friendly. The tenant name should be used instead.
@mdehn what do you think about this?

Please check if the issue can be fixed for 16.1.2

In create/update marketplace page we have autocomplete component for selecting the tenant. Should we adapt also selecting the tenant by matching name (now it is matched with id) or only displaying the name here (for example  tenant_id (tenant_name) - like in organization autocomplete component) is enough ?

The latter option is fine. Despite of less effort, it's better to be consistent with the organization autocomplete component.

Can be verified with BES_MASTER_Unit - 1619


soil: fix build

Motivation for this change
noticed it was broken
Things done

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

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

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

Result of nixpkgs-review pr 92278 1

1 package marked as broken and skipped:
- openspace

1 package built:
- soil


DB should contain the number of commits per user per month for each repo

So that the front end can show a timeline.
e.g. at https://github.com/ghuser-io/db/blob/a9c116e/data/repos/reframejs/reframe.json#L49
  "contributors": {
    "AurelienLourot": 11,
    "tdfranklin": 22,
    "brillout": 2038,
    "yuxal": 4
  },

should become something like
  "contributors_2": {
    "AurelienLourot": {
      "2018-01": 1,
      "2018-03": 5,
      "2018-04": 5,
    }
    "tdfranklin": {
      "2018-02": 1,
      "2018-04": 10,
      "2018-05": 11,
    },
    ...
  },

Note: it makes sense to work on this and #6 at the same time.
@brillout would you rather have the data per week instead? Weeks and months aren't aligned and I think you might have a harder time on the frontend if the data is per week.

@brillout the data is ready 🎉 e.g. https://s3.amazonaws.com/ghuser/data/repoCommits/reframejs/reframe.json

Neat 😍


Logs, errors, etc.
...
Your environment
zeek-3.0.12
zeek-kafka version 0.3.0 (whatever is installled over zkg)
librdkafka ~1.4.2
Ubuntu 20.04.02 (Linux 5.4.0-70-generic x86_64)
pf_ring 7.8

This isn't crashing, it is handling the reply from RDKafka.  Can you change the title?
This error means that zeek is writing more than RDKafka can handle I believe.

Well, the logger crashes, this is the output I get from "diag logger".  What is a better title?
Is the problem that kafka can't handle the load, or that the receiver isn't pulling fast enough?

Well, there is no "crash" here that I can see.  The plugin is handling the error and logging a message.  Can you add the crash information?  I need to check to see if we are returning the correct value from the method based on what is happening API-wise.  It is possible that we are, and that the 'logger' issue will have to be handled by zeek itself.
Kakfa has an Internal queue it uses.  when you call produce messages are put there and usually sent right away.  That queue is also where delivery notifications are kept as well, why we call poll() as well as produce.  Between those two things ( what you are putting in the queue and the delivery notifications ) may be why the queue is filling up.
I'm not sure why the queue is not emptying.   Network issues?  What is your plugin configuration?  Are there other errors before this?  How much data are you trying to send?  Issues on the brokers?
This is a common kafka problem it looks like.  In the plugin, what we need to do it:

Make sure we expose any settings for RDKafka that may help this
Make sure we are calling poll the right way ( we only call it if the produce succeeds, I'm not sure that is right
Think about what we do when this happens ( should we retry?, change behavior if this specific error comes up )?

If all those things were to be true, the behavior would be better defines, and you would be able to tune it for your needs.  But in the end you will need to understand your kafka system.
@JonZeolla ?

Well, as far as the crash info, zeekctl says that the logger is crashed, but I don't find any core file, despite ulimit -c and kernel.core_pattern = core.%e-%t-%p.  I suppose I should ask the slack channel about that.
Plugins: I don't have any other plugins.  Network issues: maybe, the producer and the broker are on opposite coasts.
No other errors that I see.  Data: I'm capturing two 10 Gbs links on a 48 core Intel E5-2650 with 128 MB RAM, 22 cores for each interface (the node.cfg).  The capturing machine (producer) is running at about 43 load.  The broker is a 80 core Intel 6248, 192 MB running with a load of about 3, thus barely utilized.  There is one broker on the 80 core machine.  I've run the experiment with a consumer (logstash) and without, and it seems to make no difference.
So you're probably right that I will need to understand my kafka system, and the plugin is not the problem.

Well, I'm not saying that there isn't some problem with the plugin.  Please don't get me wrong, I'm not trying to brush this issue off.
I don't see that we do anything in the plugin to handle this situation, and it would make a lot of sense if we did handle it per best practices for Kakfa producers.

Justin Azoff pointed out on the zeek slack channel that the OOM killer on linux killed the logger process, not the zeek-kafka plugin.  So this is not a bug (for zeek-kafka at least).
I also ran the system with just 10 Gbps overnight and it handled the load just fine.  So I need to do some load testing to figure out how far I can push it.

If there is any librdkafka tuning that makes sense for your environment you can pass it using https://github.com/SeisoLLC/zeek-kafka#kafka_conf

@ottobackwards I agree that it makes sense to look into a few things to confirm it's optimal.

The errors are pretty common, there are suggestions abound on setting batching or other things to help it out, those could be passed in the way that Jon says.


Cancelable transitions

If user interaction occurs before the transition has reached a minimum completeness threshold (e.g. is at least 90% through), rewind the transition.  Interaction during the rewind is ignored.
@schlem's idea.

Bonus challenge: allow interaction with the rest of the software while the transition is being canceled. (this is presently hard to do on iOS, for example).

Reminds me of @andymatushak's React 2016 talk about being able to drag a stack of cards around with one finger and add new cards to the stack by tapping them with another finger during the drag.  Pretty sure this is in the litmus doc.

Augh we can so easily build a thing like that :)
CardStack: Director {
  var stackOfCards = []

  function setUp() {
    foreach card {
      transaction.add(new Grabbable(delegate: self), card)
    }
  }

  function didGrab(card) {
    transact(function(transaction) {
      if (stackOfCards.length == 0) {
        transaction.add(new Draggable(delegate: self), card)
      } else {
        transaction.add(new SpringAnchoredTo(stackOfCards[0]), card)
      }
    })
  }
}

Augh we can so easily build a thing like that :)
CardStack: Director {
  var grabbedCard

  function setUp() {
    foreach card {
      transaction.add(new Grabbable(delegate: self), name:"interaction", card)
    }
  }

  function didGrab(grabbedCard) {
    transact(function(transaction) {
      foreach card {
        transaction.add(new Tappable(delegate: self), name:"interaction", card)
      }
      transaction.add(new Draggable(delegate: self), grabbedCard)
    })
  }

  function didTap(tappedCard) {
    transaction.add(new SpringAnchoredTo(grabbedCard), card)
  }
}

All plans made up now just for sake of exploring an implementation; not specifically happy with any of them.

/tangent

you've broken me

Copy/pasting some notes from an old brainstorm I did on this (before @schlem suggested treating all interactions during a transition as a cancel):

Consider these routes:

/listen-now/ (or /search/strfkr/)
/strfkr/miracle-mile/
/hot-chip/
/bloc-party/silent-alarm/

If someone clicked through them in rapid succession (e.g. before onRest), you’d want:

Miracle Mile cover art starts to fly to hero slot
Hot Chip label starts to fly to hero position, while Julius cover art goes back to destination

The actual transition between listen-now and artist is not nec. defined.  Neither labels nor band photos transition in the current version of the app.

Silent Alarm cover starts to transition to hero position, while Hot Chip and Miracle Mile continue to retreat to their homes on Listen Now

To handle this correctly implies:

View duplication for shared elements.  Otherwise, Silent Alarm could replace Miracle Mile before it completes its transition.  View duplication is seriously hard in a world with inheritable CSS - it would need some recursive getComputedStyle sorcery that is unlikely to be fast enough to suffice.  If you ignore the cascading problem, inducing authors to use styles that don’t cross the transition boundary, it gets less-hard.

The from elements already exist with the correct positions in the cascade, but likely at the wrong resolution and z index.
How do you handle circle-to-square transitions, eg artist?
Only shared elements should be duplicated. Otherwise, you’ll end up stacking redundant views on top of each other, which will throw off the shadows and transparency.

Independent springs for each transition to handle multiple aborts resiliently.

The simple and common (though less than ideal) solution is to disallow interactivity during a transition.  How would this affect {stop,back,forward} browser buttons?
If you can put a scrim between the From and To views, does that change the complexity here?

Maybe wrap the From view in a scrim that uses event capturing to history.back()

Should probably reverse the timeline and ignore taps after that until the timeline comes to rest(ish).  Rest on a spring is felt in the .9s, even though it doesn’t trigger until 1.0

Closing this until we begin pursuing transitions.


Passing Values to Functions with Arguments

Challenge Name

Passing Values to Functions with Arguments
https://www.freecodecamp.org/challenges/passing-values-to-functions-with-arguments
Issue Description

I am a old hand for javascript .
first of all；
I used
const functionWithArgs = ()=> { //something; }
the first time didn't passed;
and then I wrote the right code,bug   I can't passed it from here; I was so confused !
I saw the hint and github issue ,but cant found why;
sometimes later, I got the answer!
this is the reason:
the dev console had an error for Uncaught SyntaxError: Identifier 'functionWithArgs' has already been declared
and you will never passed it from then;
unless you create a new tab in the browser to copy the url to it;
it's all because of the const , oh my god~!
is it a bug ?hia hia hia;
:)
Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

// Example
function ourFunctionWithArgs(a, b) {
  console.log(a - b);
}
ourFunctionWithArgs(10, 5); // Outputs 5

// Only change code below this line.
function functionWithArgs (a,b) {
  console.log(a+b);
  return a+b;
}
functionWithArgs(1,2);

functionWithArgs(7,9);

Screenshot

Your example uses ES6 Arrow Functions but does not declare it in the js hint comment. Here's my code that I used to fix your "const" issue
//jshint esversion: 6
function ourFunctionWithArgs(a, b) {
  console.log(a - b);
}
ourFunctionWithArgs(10, 5); // Outputs 5

// Only change code below this line.
const functionWithArgs =  (a,b) => {
  console.log(a+b);
  return a+b;
};
functionWithArgs(1,2);

functionWithArgs(7,9);


OnDemandKorea Cannot Pull Videos

Checklist

[✓] I'm reporting a broken site support
[✓] I've verified that I'm running youtube-dl version 2020.09.20
[✓] I've checked that all provided URLs are alive and playable in a browser
[✓] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[✓] I've searched the bugtracker for similar issues including closed ones

Verbose log

youtube-dl.exe --verbose https://www.ondemandkorea.com/nokdu-flower-e02-1.html
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.ondemandkorea.com/nokdu-flower-e02-1.html']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2020.09.20
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: none
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (CA) as X-Forwarded-For.
[OnDemandKorea] nokdu-flower-e02-1: Downloading webpage
ERROR: Unable to extract jw config; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpksi3o1r1\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpksi3o1r1\build\youtube_dl\extractor\common.py", line 532, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpksi3o1r1\build\youtube_dl\extractor\ondemandkorea.py", line 52, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpksi3o1r1\build\youtube_dl\extractor\common.py", line 1010, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract jw config; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description

Is the error I am getting, I believe this is due to the regex being updated, but I am not knowledgeable on the subject.

use generic extractor >  youtube-dl  --force-generic-extractor https://www.ondemandkorea.com/nokdu-flower-e02-1.html

Thanks that got me further, however I am now getting this error message
[generic] nokdu-flower-e04-1: Requesting header WARNING: Forcing on generic information extractor. [generic] nokdu-flower-e04-1: Downloading webpage [generic] nokdu-flower-e04-1: Extracting information ERROR: Unsupported URL: https://www.ondemandkorea.com/nokdu-flower-e04-1.html I thought this was due to needing a premium account when I tried putting in both parameters got the same message  youtube-dl.exe -u USERNAME -p PASSWORD  --force...  any suggestions?

do you have ffmpeg installed ??

Yes, I managed to get that bit working but now I am back at the original problem   C:\Users\NAME >C:\Users\NAME\Downloads\youtube-dl\youtube-dl.exe --username USERNAME --password PASSWORD --force-generic-extractor https://www.ondemandkorea.com/nokdu-flower-e04-1.html [generic] nokdu-flower-e04-1: Requesting header WARNING: Forcing on generic information extractor. [generic] nokdu-flower-e04-1: Downloading webpage [generic] nokdu-flower-e04-1: Extracting information ERROR: Unsupported URL: https://www.ondemandkorea.com/nokdu-flower-e04-1.htm l
I appreciate the help though
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From<EMAIL_ADDRESS>Sent: Wednesday, October 7, 2020 4:51 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [ytdl-org/youtube-dl] OnDemandKorea Cannot Pull Videos (#26818)
do you have ffmpeg installed ??
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/ytdl-org/youtube-dl/issues/26818#issuecomment-704761413, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHRM6ET2X3PSYPSWKF34W4DSJQMYDANCNFSM4SF6F75A.

don't use the username & password option just use youtube-dl --force-generic-extractor https://www.ondemandkorea.com/nokdu-flower-e04-1.html
the video downloads just fine this way

Thanks October, that actually worked! :)

This is already fixed in #25877 to download it without the need for an additional command, but they have not done the merge


Adjust text anchors & offsets so labels are below icons

Uses anchor: bottom and the new zoom interpolated offset to properly position label text below their icons/POIs.
This is a bit messy to do because the text and icon are unaware of each other, and while we can set anchor: bottom on the text, that only anchors it to the bottom of the actual lat/lng point for the POI, it doesn't account for the actual icon height. Meanwhile,  the icon size itself is changing across zooms! :)
So the current solution was to use an interpolating offset that tracks to the size of the icon, so the text is always positioned down an additional half-icon-height. The longer term solution is a unified label style that can do layout for text and icon together.
With correct text placement:

Note, this offset syntax works with the new 0.4.4 JS build, and will work on ES as soon as this PR is merged: https://github.com/tangrams/tangram-es/pull/429, but is safe to merge in the meantime (the new syntax will just be ignored, leaving the current incorrect offset).

The populated places offsets need to be tightened to reduce gap, I'll land a fix in a few minutes.

It's possible that is caused by JS moving the point location when near a
tile edge too (though also possible I just got the offset math wrong :)
On Thursday, December 10, 2015, Nathaniel V. KELSO<EMAIL_ADDRESS>wrote:

The populated places offsets need to be tightened to reduce gap, I'll land
a fix in a few minutes.
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/eraser-map/pull/115#issuecomment-163775848.

It's probably that the assets have too much transparency color around them, too? At any rate, tighter is better with circles and labels.

Ah yeah, I was gonna say, I was having problems getting some of these
combos to show up on ES because of overlap (which actually led to a
separate ES bug Karim is fixing). But with current ES behavior (assuming
bug is fixed), any overlap between the two will cause one to get culled
from collision. So we may need to bring down the transparent buffer in the
icon size to get them to overlap properly, as you say (and/or add an option
to allow overlap in collisions, but probably not the best solution here and
I'm not sure if it's a generally desirable feature).
On Thu, Dec 10, 2015 at 6:01 PM, Nathaniel V. KELSO <
<EMAIL_ADDRESS>wrote:

It's probably that the assets have too much transparency color around
them, too? At any rate, tighter is better with circles and labels.
—
Reply to this email directly or view it on GitHub
https://github.com/tangrams/eraser-map/pull/115#issuecomment-163780408.

Let's treat refactoring the townspot artwork assets as a separate issue. I've cleaned this up so it looks good on WebGL and any ES optimizations


Webhooks for git integration

Most git server have webhooks that can send a POST to  a URL when content is updated. This could be added to the API.

Added documentation for Gitea self-hosted. https://github.com/DavesCodeMusings/container-central/blob/main/docs/Webhooks.md Supporting other git hosting services will require coding changes to accept a POST rather than a GET.


Enhance Unified Summaries

I think unified summary pages like that site page are great!
I would suggest that any of these stats would link to a filtered view of these stats  in the corresponding page.
I suggest we could introduce this across other unified pages such as:
1.) Manufacturer:

total number of platforms
total devices breakdown by device type
total devices breakdown by platform (recommended: a manufacturer foreign key to be added to platform)
total devices breakdown by device roles (I think data validation here may be over kill)

*Note: I recommend the new foreign key to allow for data validation fields when selecting a platform for a device.
2.) Rack Groups:

total devices
total free RUs
total used RUs
total circuits (optional, may be over kill)

3.) Device Types:

total devices

4.) Prefixes:

% of IP space used or available?
VLAN (if associated)

4.) VRFs:

Number of prefixes

5.) RIRs:

Number Prefixes
% of IPs used/available?

5.) Providers:

Number of circuits

Thoughts? Other suggestions?

This feature request has languished for quite a while. Some of these have probably been addressed since it was open; others may or may not be addressed. I'm going to close this out but encourage people to open new requests for any specific details they'd like to see added (please limit the scope of a request to a single model).


Fixes Multiple Issues / Refactors Code / Adds Additional Op-Code Support

Fixes issues on DS conversion for LIKE, EXTNAME, EXTFMT, and use of special reserved words (*PROC, *STATUS, etc.). Specific issues fixed:

Fixes BrianGarland/vscode-rpgfree#48
Fixes BrianGarland/vscode-rpgfree#52
Fixes BrianGarland/vscode-rpgfree#62
Fixes BrianGarland/vscode-rpgfree#77
Fixes BrianGarland/vscode-rpgfree#78

Additional D-Spec clean up.  Specific issues fixed:

Fixes BrianGarland/vscode-rpgfree#32
Fixes BrianGarland/vscode-rpgfree#35
Fixes BrianGarland/vscode-rpgfree#45
Fixes BrianGarland/vscode-rpgfree#50
Fixes BrianGarland/vscode-rpgfree#61
Fixes BrianGarland/vscode-rpgfree#81
Fixes BrianGarland/vscode-rpgfree#82

Corrected indentation of procedure interface, parameters, and data definitions inside a procedure.  This fixes the issue:

Fixes BrianGarland/vscode-rpgfree#83

Additional issues addressed in this pull request include:

Resolves BrianGarland/vscode-rpgfree#16
Fixes BrianGarland/vscode-rpgfree#53
Fixes BrianGarland/vscode-rpgfree#70
Fixes BrianGarland/vscode-rpgfree#85
Fixes BrianGarland/vscode-rpgfree#86
Fixes BrianGarland/vscode-rpgfree#88
Fixes BrianGarland/vscode-rpgfree#89
Fixes BrianGarland/vscode-rpgfree#90
Fixes BrianGarland/vscode-rpgfree#91

Additional patches incorporated from changes made by @DavidShears:

Fixes BrianGarland/vscode-rpgfree#56

@BrianGodsend Because of the massive pull request I just merged I need your changes to be made based on that version to merge them.    Can you pull the new code and make your changes there?

@BrianGodsend There is also a test suite now, so it'd be cool to ensure that after main has been merged into this branch that the tests still pass (rpm run test) and then you can also add your own tests!

Hi there,
I'm new to VS Code.  I see this Pull will fix some issues I'm having.  How long will it take for this to get promoted so us end-users can get the fix?
Thank you.

@GlennGundermann The complication with this pull request is that it is based on code from before some major changes and needs to be merged with those changes before I can accept it.   I have asked @BrianGodsend to do this but have not heard back yet.

It's been over 6 months with no response.  I'm closing this pull request as it looks like the author is no longer active.


Ultrasound support?

This is a great project!  Thank you for sharing!
What would be required to use this library in the inaudible ultrasound range?
I have been evaluating a few different libraries and most provide a way to configure the frequency range and support inaudible communication.  They all have down sides, however, so I am trying to find the solution with the most robust implementation and I find qrtone to be promising in this regard.
I would prefer to use the wasm library so if I need to make a custom build I would appreciate any build instructions you may have for this.
Also is there a limit to the payload size that can be sent?  I would like to be able to basically send files between devices.
Thank you!!

Hi,
Thanks for your interest in QRTone.
The frequencies are defined in the qrtone.c file, you could use the 16khz-20khz range for mostly inaudible communication.
The transfer rates of data over audio (through air not wires) are quite low (1 byte each 85 ms).
When using data-over-audio, only the "link" should be sent, the receiving phone would use internet in order to download the file content.
There is another library with javascript support:
https://quiet.github.io/quiet-js/


Video/audio integration opens in default browser, not chrome

If you have a different default browser than Chrome (e.g. Firefox), the medias will be opened in a different browser and it wont show. There's a blank page.

@jfcartier Windows?

@survivor303 Thanks but no, I'm on OSX.


Issue #DP-543 feat: pagination for nomination list shown to sourcingO…

…rg..reducedd api load

closing this one as we have rewritten the program-nominations view, to call the APIs needed for the tab only after we click the tab


16/08/12 master run convnet benchmark googlnet failed on CPU only version

CPU only
simply excute https://github.com/soumith/convnet-benchmarks/blob/master/mxnet/gnetv1.py
Failed log
(mxnet_python) [ mxnet]$ python gnetv1.py
[00:30:27] src/engine/engine.cc:36: MXNet start using engine: NaiveEngine
[00:30:27] mxnet/dmlc-core/include/dmlc/logging.h:235: [00:30:27] src/operator/./pooling-inl.h:198: Check failed: param_.kernel[0] <= dshape[2] + 2 * param_.pad[0] && param_.kernel[1] <= dshape[3] + 2 * param_.pad[1] kernel size exceed input
Traceback (most recent call last):
File "gnetv1.py", line 88, in
g_exec = loss3_classifier.simple_bind(ctx=dev, grad_req="write", data=dshape)
File "mxnet-0.7.0-py2.7.egg/mxnet/symbol.py", line 671, in simple_bind
arg_shapes, _, aux_shapes = self.infer_shape(*_kwargs)
File "mxnet-0.7.0-py2.7.egg/mxnet/symbol.py", line 453, in infer_shape
return self.infer_shape_impl(False, *args, *kwargs)
File "mxnet-0.7.0-py2.7.egg/mxnet/symbol.py", line 513, in infer_shape_impl
ctypes.byref(complete)))
File mxnet_python/lib/python2.7/site-packages/mxnet-0.7.0-py2.7.egg/mxnet/base.py", line 77, in check_call
raise MXNetError(py_str(LIB.MXGetLastError()))
mxnet.base.MXNetError: InferShape Error in pooling4: [00:30:27] src/operator/./pooling-inl.h:198: Check failed: param.kernel[0] <= dshape[2] + 2 * param.pad[0] && param.kernel[1] <= dshape[3] + 2 * param.pad[1] kernel size exceed input

https://github.com/soumith/convnet-benchmarks/blob/master/mxnet/gnetv1.py
that script is for GPU, can you have a try to replace gpu with cpu?

The problem is we change pooling shape recently. We will release a new
version code of Deepmark soon.
A quick fix is: Add padding 1 to the pooling branch in last tower of each
branch
On Fri, Aug 12, 2016 at 07:28 Chen Xiaoming<EMAIL_ADDRESS>wrote:

https://github.com/soumith/convnet-benchmarks/blob/master/mxnet/gnetv1.py
that script is for GPU, can you have a try to replace gpu with cpu?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/3014#issuecomment-239461373, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABM13vK7WvoSYv6M-0JoIOnTSwwcnyDqks5qfIMSgaJpZM4JjFmc
.

--
Sent from mobile phone

('Avg forward per batch: ', 25.5602810382843)
[02:14:22] mxnet/dmlc-core/include/dmlc/logging.h:235: [02:14:22] include/mxnet/././tensor_blob.h:696: Check failed: (this->shape_.Size()) == (shape.Size()) TBlob.get_with_shape: new and old shape do not match total elements
[02:14:22] mxnet/dmlc-core/include/dmlc/logging.h:235: [02:14:22] src/engine/./threaded_engine.h:306: [02:14:22] include/mxnet/././tensor_blob.h:696: Check failed: (this->shape_.Size()) == (shape.Size()) TBlob.get_with_shape: new and old shape do not match total elements
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.
terminate called after throwing an instance of 'dmlc::Error'
what():  [02:14:22] src/engine/./threaded_engine.h:306: [02:14:22] include/mxnet/././tensor_blob.h:696: Check failed: (this->shape_.Size()) == (shape.Size()) TBlob.get_with_shape: new and old shape do not match total elements
An fatal error occurred in asynchronous engine operation. If you do not know what caused this error, you can try set environment variable MXNET_ENGINE_TYPE to NaiveEngine and run with debugger (i.e. gdb). This will force all operations to be synchronous and backtrace will give you the series of calls that lead to this error. Remember to set MXNET_ENGINE_TYPE back to empty after debugging.

get above error after add padding, and I found this error in Tag V0.7,  Tag16016

@antinucleon Any progress on this issue?

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!