OverworldSkyOverride not showing up.

[Please fill out the form below and delete the sections in square brackets after reading them]
Version of Advanced Rocketry

Build 13 for 1.12.2

Have you verified this is an issue in the latest unstable build

[x] N/A

Version of LibVulpes

1.12.2-0.4.2-25

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] N/A

config file issue more or less i'm assuming.

Crash report or log or visualVM (if applicable)
Description of the problem

Opening the config file. I haven't seen this issue in the rest of the issues so it could be just me. I came across this when i was trying to disable the skybox for the OVERWORLD SPECIFICALLY. The code lines show up for Planets and for the station, just not the overworld. i will have an image attached. (side note, disabling all of them does not fix it, i've looked through the entire config file and could not find the overworld option. it works when disabled for the proper biome, I.e planets and space station, just not the overworld.)

It's hidden below the Electric Mushrooms option, around 2/3rds of the way down the image. Took me a while to find it too


Use a custom wamp_error exception type to propagate args and kw_args from wamp invocations

This looks good. Still not sure about the shared_ptr stuff, we will see what they have to say about it upstream.


Ghost redirect doc suggestions

Based on 1) experience of cloudflares page rules guide and 2) John asking whether multi-wildcard was supported in Ghost and 3) planning an update to our redirects feature and not finding all the features documented, I would like to propose some changes to the redirects guide.
I'm starting by proposing problems 😬
Issues I think exist:

"regex" could be considered a bit vague (there are different flavours)
there's no clarity around double escaping (I still am not sure myself what the rules are)
We don't make it clear that "multi wildcard" is supported E.g. you can use $1, $2
Case insensitive matching with the i flag is not mentioned
It's not clear that temporary redirects are the default
Text around the required JSON format could be more explicit
Minor Typo: the from field text has an additional word "you"

Possible solutions?
I think there are a couple of small changes that could be made to the text to resolve these things. E.g. using more explicit technical language in a summary somewhere like:
"The from and to fields work exactly the same as javascript's replace function, with full support for regex patterns, the i case insensitivity flag,  and multiple matches using $1, $2. "
and also including the phrase "The file must be valid JSON", with the words "valid JSON" being a link to https://jsonlint.com/ (This linter gives clarity on why a file is not uploading). I think some of the text about how JSON works could also be removed or grouped together.
I would also like to propose that we include one advanced example. I'm happy to be shot down for this, but I think a lot of us like to see a super complex example to see what we can achieve (thinking about the trouble with the page rules guide)
My proposed advanced example is similar to our docs redirect, but we could probably find a better one:
{
   "from": "^/(v.*?)/docs/([a-z0-9_\-]+)(\\/)?$/i",
    "to": "/$1/$2/"
}

It includes: advanced pattern matching in JS, the need for double-escaping when using regex, 2 wildcards and the i flag all in one fell swoop, but note that I'm not really sure where and when I need to use a double backslash escape here.
It maybe that all this is waaaay to technical for the 90%. My main suggestion is really to either use more explicit language to be clearer what does and doesn't work e.g. say "valid JSON" and "javascript regex" and let people use the rest of the internet to determine what those phrases mean.
Happy to have a stab at a PR for this if folks agree.

@ErisDS
I think there’s definitely room for more technical information about redirects in Ghost on the concepts page - seems like there is some missing information based on your list of issues. I rewrote this page during the docs project to the best of my knowledge using old docs as reference. I don’t fully understand the missing issues - so do you want to submit a PR and we can work on improving it together?
We do also have a tutorial for redirects, which is more of a getting started/implementation guide. The two should probably link to each other, but the article in Concepts should include any technical references that cover a wide range of use-cases.
If you have any improvement suggestions for the tutorial then you can send those to me either in this issue or another channel, the content exists in Ghost.

o_0 I totally missed that there was a tutorial! I was referring only to the concepts page. Seems like a lot of the info that I thought was missing is actually in the tutorial. l
I'll review both properly, and then come back with any recommendations 👍


[Test] Generate TensorV2 in unit test

This PR includes the implementation of test util functions to create a tensor filled with values to utilize in unit testing.
Self-evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test:   [X]Passed [ ]Failed [ ]Skipped

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2420. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Typing "code ." from command-prompt shows brings up cli.js and starts logging message to the locked command-prompt window

Yes

VS Code Version: 1.72.2
OS Version: Windows 10

Steps to Reproduce:

Go to Admin command prompt.
Type "code ."
VS Code launches with current folder, but also opens cli.js open.
The command prompt window (cmd.exe) is showing log output from vscode
The command prompt window is locked -- waiting for vscode to end before you gain back control.

This happens 1 out 50 or so times.   But it just happened twice in a row.  When it happens, it's consistent.   But I don't know the steps to reproduce it.
Also, I have this configuration:

For this EXE: "C:\Program Files\Microsoft VS Code\Code.exe", I've set the file's properties' Compatibility tab to "Run this program as an administrator"

I had to do to work around a different problem:
The problem:
If I am in command prompt that's elevated, and I do "code .",then that VS Code will run elevated (which is correct).   I may have that editor open for hours while I work on my project.
Later, I may be in a different command prompt that's not elevated, and I may want to edit some files in there.  When I do "code ." there, it won't work -- vscode will complain that it cannot open as non-admin because there's already another session of vscode running as admin 🙄.
The solution I was recommended was to set vscode itself to always run as admin that way, no matter where I launch it from, it will run as admin and won't run into that problem.
I don't know if this is related or not, but I thought I'd give that info as well.   As I said, I don't run into this problem all the time.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.73.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

We generally do not support running VS Code as admin.
Anyway, @deepak1556 anything that you could think of involved here, I have unfortunately no real clue.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Fix relative links on minikube docs

The getting started guide was lifted from minikube's README.md on
github and included relative links.  They have been updated to point to
the absolute path to the latest release.
ref https://github.com/kubernetes/kubernetes/issues/39645#issuecomment-271527761

This change is

Looks like you need to rebase.

Thanks.  I fixed this in a separate PR.


node example server fails to start

Hello,
I am just trying to get introduced to the node sdk and I am having some trouble.
In Flux SDK Node Example repo when I run npm start I receive an error saying that flux-sdk-node module cannot be found. When I look in node_modules, it is there. Can you help please?

Heya! What version of npm/node do you have? We satisfy the "flux-sdk-node" dependency with a feature that was only introduced in npm v2.

I currently have npm v4.0.3 and node v7.2.0 installed.

Hmm, I can't reproduce so far... What version of the SDK are you using?

Latest version I think. 4.7

Here is the Debug.log file.
1 verbose cli [ '/usr/bin/nodejs', '/usr/bin/npm', 'start' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'prestart', 'start', 'poststart' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 silly lifecycle<EMAIL_ADDRESS>no script for prestart, continuing
7 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
9 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/lib/node_modules/npm/bin/node-gyp-bin:/home/bigdoods/yano/flux-sdk-js/flux-sdk-node/example/node_modules/.bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/snap/bin
10 verbose lifecycle<EMAIL_ADDRESS>CWD: /home/bigdoods/yano/flux-sdk-js/flux-sdk-node/example
11 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'node ./bin/www' ]
12 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
13 info lifecycle<EMAIL_ADDRESS>Failed to exec start script
14 verbose stack Error<EMAIL_ADDRESS>start: `node ./bin/www`
14 verbose stack Exit status 1
14 verbose stack     at EventEmitter.<anonymous> (/usr/lib/node_modules/npm/lib/utils/lifecycle.js:279:16)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at EventEmitter.emit (events.js:191:7)
14 verbose stack     at ChildProcess.<anonymous> (/usr/lib/node_modules/npm/lib/utils/spawn.js:40:14)
14 verbose stack     at emitTwo (events.js:106:13)
14 verbose stack     at ChildProcess.emit (events.js:191:7)
14 verbose stack     at maybeClose (internal/child_process.js:885:16)
14 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
15 verbose pkgid<EMAIL_ADDRESS>16 verbose cwd /home/bigdoods/yano/flux-sdk-js/flux-sdk-node/example
17 error Linux 4.4.0-51-generic
18 error argv "/usr/bin/nodejs" "/usr/bin/npm" "start"
19 error node v7.2.0
20 error npm  v4.0.3
21 error code ELIFECYCLE
22 error<EMAIL_ADDRESS>start: `node ./bin/www`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>start script 'node ./bin/www'.
23 error Make sure you have the latest version of node.js and npm installed.
23 error If you do, this is most likely a problem with the flux-sdk-node-example package,
23 error not with npm itself.
23 error Tell the author that this fails on your system:
23 error     node ./bin/www
23 error You can get information on how to open an issue for this project with:
23 error     npm bugs flux-sdk-node-example
23 error Or if that isn't available, you can get their info via:
23 error     npm owner ls flux-sdk-node-example
23 error There is likely additional logging output above.
24 verbose exit [ 1, true ]

I'm still having trouble reproducing: can you delete node_modules and rerun npm install?

Ok - I have solved it. It turns out that I did not have correct permissions set for npm.
FYI - Something else came up when I deleted everything to start from scratch. When I ran npm install from flux-sdk-node/example I got another error : sh: 1: rimraf: not found. I returned to the flux-sdk-node directory and ran npm install and then tried again and it was fine.
Thanks for your time Felix :+1:

Whew! Yeah, we might install rimraf as a dev dependency in the example directory to fix that. Good luck!


When server returns a 500 error resource is removed from state but cannot be recreated when server is back up

Describe the bug
If the harbor server goes down when terraform is applying and returns a 500 error (in particular 502 and 504) those resources are removed from the statefile and the provider attempts to recreate them on the next apply. This causes issues as those resources still exist in harbor they just weren't accessible before due to networking or load balancer issues.
If the server returns anything but a 404 error for a particular ID then IMO the provider should just return an error instead of trying to recreate the resource.
To Reproduce
Steps to reproduce the behavior:

Create a project using a project resource. Run apply
Get the service to return a 504 (easy way is to remove the service resource in K8s)
Reapply and notice how it identifies those resources are "gone" and it removes them from the statefile, and then attempts to recreate them.
Recreate the service so harbor is up again and rerun the apply job and notice how the creation of the resources fail because they already exist.

Expected behavior
The provider should fail if anything but a 404 or 200 is returned from the API. As anything other than these indicate that the request wasn't processed properly.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

we're also running into these issues when using the provider - especially when connecting to Harbor instances that are geographically "far away" (on the other side of the world)...
Since the issues are appearing so sporadically, we weren't yet able to figure out the exact nature of the problem, but I highly suspect some non-standard HTTP return codes or some connection timeouts/aborts.
Here's an excerpt from such a failing terraform apply run:
[...]
- Using bestseller/harbor v3.5.0 from the shared cache directory
[...]
module.abc.harbor_robot_account.docker_pull: Refreshing state... [id=/robots/154]
[...]
# module.abc.harbor_robot_account.docker_pull has been deleted
  - resource "harbor_robot_account" "docker_pull" {
      - disable     = false -> null
      - duration    = -1 -> null
      - robot_id    = "154" -> null
      - secret      = (sensitive value)
      - permissions {
[...]
# module.abc.harbor_robot_account.docker_pull will be created
  + resource "harbor_robot_account" "docker_pull" {
      + disable     = false
      + duration    = -1
      + full_name   = (known after apply)
      + id          = (known after apply)
      + level       = "system"
[...]
│ Error: [ERROR] unexpected status code got: 409 expected: 201
│  {"errors":[{"code":"CONFLICT","message":"robot account 0:docker-pull already exists"}]}
│
│
│   with module.abc.harbor_robot_account.docker_pull,


Make timestamp expression native SQLAlchemy element

SUMMARY
The functional part of this PR makes the timestamp expression a native SQLAlchemy element that respects quoting rules for column names (based largely on https://docs.sqlalchemy.org/en/latest/core/compiler.html) . This is a continuation of what was started in #6897, which was really just a hack to make Postgres work, but didn't address e.g. reserved keywords or similar problems that might arise in other dbs. This PR adds a new class TimestampExpression, which can be used in a SQLAlchemy query, which keeps the target column as a native Core element. The column name is only rendered to text once the query is compiled, which ensures that engine specific quoting rules are respected.
Refactoring:

Added mypy typing where code was changed.
Moved some functional parts of sqla/models/get_timestamp_expression() to db_engine_specs/get_timestamp_expr().
Removed models/core/grains() which was no longer needed.

TEST PLAN
Added unit tests that test the central features and tested timeseries graphs using both column and expression, both with and without time grains.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration. Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API
[x] Fixes bug
[x] Refactors code
[x] Adds test(s)

REVIEWERS
Comments much appreciated @betodealmeida @john-bodley. Also @agrawaldevesh: I had to change the Pinot logic for this PR. Do you have the opportunity to test if this works on your Pinot deployment (I don't have a Pinot installation handy right now)? I also added a few grains while at it, do they work?

Thanks @villebro, Happy to "test this in prod" at Uber and report back to you. The code changes look okay to me and I think they should work.
I have just one question: I am wondering if the TimestampExpression should be a good vehicle to fix the TZ issues plaguing superset: https://github.com/apache/incubator-superset/issues/6768 and https://github.com/apache/incubator-superset/issues/1149 for example.
The problem is that Superset is essentially TZ unaware, and it assumes that the stored data is in the local TZ. I was curious if TimestampExpression could have a ".tz" field in it eventually ? And for databases that do support TZ's, that tz field is plumbed in when generating the timestamp expression sql.

Codecov Report

Merging #7131 into master will increase coverage by 0.03%.
The diff coverage is 94.73%.

@@            Coverage Diff             @@
##           master    #7131      +/-   ##
==========================================
+ Coverage   64.61%   64.64%   +0.03%
==========================================
  Files         422      422
  Lines       20605    20605
  Branches     2253     2253
==========================================
+ Hits        13313    13320       +7
+ Misses       7170     7163       -7
  Partials      122      122

Impacted Files
Coverage Δ

superset/models/core.py
83.46% <ø> (-0.11%)
:arrow_down:

superset/connectors/sqla/models.py
81.98% <100%> (+0.11%)
:arrow_up:

superset/db_engine_specs.py
58.45% <93.1%> (+1.08%)
:arrow_up:

Continue to review full report at Codecov.

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

Rebased to fix confict. @agrawaldevesh thanks for offering to help verify this PR! Regarding the timezone issue, I think it's something that runs slightly deeper in the code, so would probably require some additional planning (perhaps a SIP). Perhaps something along the lines of a setting in superset_config.py to specify a timezone in which all timestamps with timezone info would be cast to (naive timestamps would probably be kept unchanged)? Anyway, a really good idea worth investigating (committer guidance would be helpful).

kind reminder @john-bodley @agrawaldevesh for final comments on this PR.

Hi @villebro , I left some comments inline. LGTM to me otherwise.

Hi @villebro , I left some comments inline. LGTM to me otherwise.

Thanks @agrawaldevesh , great comments as always! Were you able to get this working? And was the unit test ok?

Hi @villebro , I left some comments inline. LGTM to me otherwise.

Thanks @agrawaldevesh , great comments as always! Were you able to get this working? And was the unit test ok?

Hi Ville ... Everything works. The unit test is proper and also I was able to merge this branch and test it out in my prod environment and superset had no problems. All charts and everything rendered fine. The generated queries remained the same.
So this PR is totally fine from Pinot perspective.

@mistercrunch @john-bodley this one should be ok to ship.

Hi Max ... I tested this with Pinot natively and it works. Presto+pinot wouldn't really test the pinot side of superset (since superset would just speak presto-sql then). Having tested it, I am confident that this PR would work with Pinot.

Hi,
Need help on similar requirement.
We have many timestamp with time zone fields in Oracle database which is the source for our Superset (0.36 version) reports.
And we need to show the data in EST or PST or some other time zones based on customer need.
Ex:
Data in Oracle database table is like '2021-01-04 21:48:56.66 +05:30'
If we want to display in EST, report should show it like : 2021-01-04T11:18:56.660000
If we want to display in PST, report should show it like : 2021-01-04T08:18:56.660000
So, is there any configuration parameter that we can modify to display the data in desired time zone (like EST/PST or some other)?
Please advise. Thanks in advance.

Hi,
Need help on similar requirement.
We have many timestamp with time zone fields in Oracle database which is the source for our Superset (0.36 version) reports.
And we need to show the data in EST or PST or some other time zones based on customer need.
Ex:
Data in Oracle database table is like '2021-01-04 21:48:56.66 +05:30'
If we want to display in EST, report should show it like : 2021-01-04T11:18:56.660000
If we want to display in PST, report should show it like : 2021-01-04T08:18:56.660000
So, is there any configuration parameter that we can modify to display the data in desired time zone (like EST/PST or some other)?
Please advise. Thanks in advance.

Also, we need to show the data displayed in time zone per user profile or preference if there is a way.

Also, we need to show the data displayed in time zone per user profile or preference if there is a way.

It is observed that Superset is automatically converting the timestamp fields to EST time while displaying in the SQL Lab or Chart.
Anyone knows where exactly we can change this setting to show it in different time zone or customize it to work using a config parameter?

It is observed that Superset is automatically converting the timestamp fields to EST time while displaying in the SQL Lab or Chart.
Anyone knows where exactly we can change this setting to show it in different time zone or customize it to work using a config parameter?


Global pool with expressjs

I am trying to create the cluster pool with express.
Any known way to create cluster at global scope and accessing it inside app.get or app.post?

There is a sample application in the examples folder.


Autocomplete Options not working

Hi @jharding  ,
I am new to typeahead.js and I am trying to create my first demo based on the examples you have given but not getting the auto complete options.
what is the error I am unable to figure out.

There's not enough info here for me to help debug. If you post more information (maybe a jsfiddle?), I can help.


Update README.md to include install instructions

Add brief instructional step for installing this module.
Add notice about potential need to install plugins used in the example implementation.

Thanks :)

Fixes #60


Use dataclasses for all objects returned by HfApi

Currently, we use TypedDict to represent some of the objects' attributes returned by the HfApi methods. To be consistent, we should only use dataclasses, as suggested in https://github.com/huggingface/huggingface_hub/pull/1809#discussion_r1387635398:

I think we should open a separate PR to switch all of them (BlobLfsInfo, LastCommitInfo, BlobSecurityInfo, TransformersInfo, SafeTensorsInfo) to dataclasses, in a backward compatible way (and with a deprecation warning when dict-only method is used). This way we'll finally have a single type to represent data returned by the server (now that you've removed ReprMixin).

This work can be split in several PRs to make it easier to implement and review.

Hello mario, i would like to do this task, is it possible ?

Hi @NouamaneELGueddarii! Thanks for proposing yourself! It is possible to take this task yes. Just to be sure we are aligned, the goal is to update a bunch of objects in hf_api.py from TypedDict to proper dataclasses. At first, typed dict were used only for static type checking. However now we want to convert everything to dataclasses that have more features. So for each object, you would need to:

inherit from dict like in this one
decorate with @dataclass like in this one
accept a dict as input in __init__.py (see here)
update self.__dict__ (see here) for backward and future compatibility (if server response gets extended)
update how objects are instantiated

What I would suggest you is to start working on just 1 or 2 classes and open a PR for them. This way we can iterate on it and answer your questions (if any). What do you think?
Thanks in advance!

@Wauplin is it fine if I start working on it? I want to start making open source contribution to github.
Thanks

Hi @Ahmedniz1, yes it is! Thanks for proposing you. Please refer to https://github.com/huggingface/huggingface_hub/issues/1911#issuecomment-1881212204 on how to deal with it.
Since @NouamaneELGueddarii mentioned earlier they would like to work on it as well, could you share on which dataclass you would like to focus first before starting implementing something?  A simple message on this thread is enough (just want to avoid duplicate work).

I want to start making open source contribution to github.

:rocket: :rocket: :rocket:

Sounds like a plan!

@Wauplin i will be working on the following class "TransformersInfo".  I will make a PR and wait for approval then proceed to work on the others.

Great! Thanks for confirming @NouamaneELGueddarii!

@Ahmedniz1, saw it and just reviewed it! Thanks!

@Ahmedniz1 @NouamaneELGueddarii I have updated the instructions on how to proceed with the issue. I'm sorry about this late change but I realized the previous recommendations were working correctly. I've commented on @Ahmedniz1's PR here (https://github.com/huggingface/huggingface_hub/pull/1974#pullrequestreview-1827419034) and updated the comment above (https://github.com/huggingface/huggingface_hub/issues/1911#issuecomment-1881212204). Hopefully, this will make it easier to integrate new dataclasses while keeping backward compatibility.
Sorry about the change and thanks in advance for adapting!

@Wauplin I'm trying to update in the previous PR but it appears to be closed. I can push the updates for other classes(apart from transformerinfo) once the original PR is open again.
Thanks

I'm trying to update in the previous PR but it appears to be closed.

Yes this is expected! You have to checkout to main, pull the latest changes, create a new branch from main and start a separate PR from there. This way we have small PRs that can easily be reviewed. The work you've already done is now on the main branch.


Does this work with any exposed endpoints?

For instance, would this work with https://wordpress.org/plugins/acf-to-rest-api/

You'd have to extend the schema for it to work with non-standard REST endpoints.
Just a warning up front though: WordPress changed their API to now only allow access to authenticated write methods from admin users. So regardless if you use a key-based authentication, if a user is logged in with non-admin credentials, the API will fail silently.
Our team stopped using the REST API because of this.

WordPress changed their API to now only allow access to authenticated write methods from admin users.

Do you happen to have any more resources on this? I'd like to read the debate and specific implications of this.

Our team stopped using the REST API because of this.

Out of curiosity, what did you move to?

Also, did you guys investigate Rooftop for this? https://rooftopcms.readme.io/docs/authentication

Do you happen to have any more resources on this? I'd like to read the debate and specific implications of this.

Unfortunately, no. Their REST page isn't very clear on this either.
To give you an idea of why it failed to meet our needs:
We specifically created this repo for a circumstance where we needed to create a custom dashboard for a user to manage subordinates (we were/are using React for this). The dashboard needed to pull in various meta fields from each of their subordinate's usermeta and display it on the dashboard. The issue though is that because the managing user is not an "Administrator" user, they can't pull in other user's metadata (including read-only data!), even if the API is called programatically using basic authentication using an administrative-level API token.
 This fundamentally made the API useless for us because you are unable to access even the most basic of data without yourself (the logged in user) being an administrator.

Out of curiosity, what did you move to?

For non-secure data, we're just dumping it to the global scope using wp_localize_script. For secure data, we're accessing it using ajax.

Also, did you guys investigate Rooftop for this?

Haven't heard of that before, so no. But that looks interesting from first glance. I'm wondering how they're able to bypass the issues we've ran into.
We used Basic Auth.

Thanks for the context!

@dsifford Do you care if I email/Discord you for more context? I'm trying to pick a good default stack, clients love WordPress, and just have a bunch of small questions. 😬

Sure. Happy to help clarify where I can.
<EMAIL_ADDRESS>


Make number formatting more forgiving so that a number is displayed

If phony incorrectly detects a wrong prefix, then e.g. formats it using split(3,3), instead of e.g. split(4,4) it will format it incorrectly.
However, much worse, it will lose 2 digits. This will usually only be detected at (phone) call time, with very bad results.
I suggest making the split method more forgiving. Hackish way: Adding X to the last number passed in, where X is an arbitrary number, e.g. 5.

I think that this makes sense.
Worth nothing that the primary reason we've been able to find issues with the formatting of numbers (and subsequently fix them) on our site is because of cases where the last number has been chopped off - users are much more likely to report a missing digit rather than just incorrect formatting.

Yes, I thought so. And this makes me feel queasy, even though making phony more lenient means improving it less quickly.
The thought of someone standing out in the rain, trying to call someone and not getting through is not a good one ;)
If there are no surprises or obstacles I haven't thought of to this, expect this API change in the next few days. Thanks for your feedback!

Phony is now more lenient. 1.6.0 has been released.
I hope Phony still gets bug reports even if the likelihood of cut-off numbers is very low now.


Update Go (golang) handling

Use https://golang.org/s/generatedcode in addition to existing rule for generated files.
Do not treat vendored Go code as generated.

Closes #3719.
Refs #2724.
/cc @lildude @cespare

I'm conflicted about the second point. Vendored files are not generated, they are vendored, that is clear. But why all vendored files are not suppressed in diffs?

Nudge

No response in three months. Marking as stale.

Vendored files are not generated, they are vendored, that is clear. But why all vendored files are not suppressed in diffs?

That is the important question, and answering may answer your questions. It looks like different parts of linguist treat "vendored" and "generated" words differently.

But why all vendored files are not suppressed in diffs?

Hmmm, vendored files generally aren't suppressed in diffs. Only generated should be.  That said, very large diffs are minimised in diffs, regardless of whether they're vendored or not. Do you have examples of where you're seeing discrepancies that I can see and try explain?

Hmmm, vendored files generally aren't suppressed in diffs. Only generated should be.

PTAL: https://github.com/percona/pmm-managed/commit/6e3e7d2ef368cc0c7a5f1d93bf1bef583b54bc1f Here vendored files are not rendered, but the message says "Some generated files are not rendered by default." But those files are vendored, not generated.

There's some background to this in #2843 and #2620.
As far as I understand, we mark these files as generated---even though they're not really---precisely because we want to hide them by default in diffs.

Nudging this for one last time.

@AlekSi Are you planning to continue to work on this PR?
If not, I can send a new PR.

@dmitshur Feel free to work on it. And thank you.

As far as I understand, we mark these files as generated---even though they're not really---precisely because we want to hide them by default in diffs.

That feels right. Right now README.md provides a good description of what is considered vendored and what is generated. So what we call "vendored" in Go is "generated" in terms of Linguist.
@dmitshur While you work on that PR, may you please add a few comments in code to mention this vendored/generated terms clash? They could save time for the next contributor. 😄


Postgres, javalin and junit are vulnerable to security threats

Checked for duplicates
Yes - I've already checked
Is this a regression?
No - This is a new bug
Version
1.11.0
Describe the bug
An intellij plugin gave me warnings about security vulnerabilities of some of our dependencies:
Javalin 5.1.2
Provides transitive vulnerable dependency org.eclipse.jetty:jetty-http:11.0.12
CVE-2023-26048 5.3 Allocation of Resources Without Limits or Throttling vulnerability pending CVSS allocation
CVE-2023-26049 5.3 Exposure of Sensitive Information to an Unauthorized Actor vulnerability pending CVSS allocation

junit 4.12
CVE-2020-15250 5.5 Incorrect Permission Assignment for Critical Resource vulnerability pending CVSS allocation
postgresql 42.2.5
Dependency org.postgresql:postgresql:42.2.5 is vulnerable
CVE-2020-13692 7.7 Improper Restriction of XML External Entity Reference vulnerability pending CVSS allocation
Cx06bea32e-26f2 9.8 Improper Control of Generation of Code ('Code Injection') vulnerability with high severity found
CVE-2022-21724 9.8 Improper Initialization vulnerability pending CVSS allocation
CVE-2022-31197 8.0 Improper Neutralization of Special Elements used in an SQL Command ('SQL Injection') vulnerability pending CVSS allocation
CVE-2022-41946 5.5 Exposure of Sensitive Information to an Unauthorized Actor vulnerability with medium severity found

Reproduction
No reproduction needed.
Logs
No response
System Info
MacOS, IntelliJ

Severity
Moderate

We should look into adding https://github.com/marketplace/actions/gradle-dependency-submission to catch these eariler
CC @skovati


_86Box: darwin support, add maintainer, derivation cleanup (with lib; and RFC format)

Description of changes

Added better support for Darwin

Added $out/share/86Box/roms as an additional ROM path, so ROMs - if present - are picked up automatically

Cleaned up meta as per #208242
Formatted the derivation with nixfmt-rfc-style.
Added myself as a maintainer for Darwin

To test this with ROMs (on Darwin):

Checkout my branch and cd into it
Run NIX_PATH="nixpkgs=$PWD" NIXPKGS_ALLOW_UNFREE=1 nix-shell -p _86Box-with-roms
Once built and inside the Nix shell, run 86Box - it should launch the GUI if it can find the roms within the Nix Store.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

@NixOS/darwin-maintainers could anyone test this on x86-64_darwin please? 🙏🏻

Result of nixpkgs-review pr 335384 run on x86_64-darwin 1

2 packages failed to build:

_86Box
_86Box-with-roms

@jchv @DontEatOreo reverting this back to draft - just realised it has a weird behaviour on Darwin - the configuration file is not set correctly.
It should default to $PWD/86Box.cfg, but it only works like that on Linux.
On Darwin, it defaults to $out/bin/86Box.cfg for some reason.

Ready for review again @jchv @DontEatOreo @reckenrode  🙏🏻
Re this:

@jchv @DontEatOreo reverting this back to draft - just realised it has a weird behaviour on Darwin, the configuration file is not set correctly.
It should default to $PWD/86Box.cfg, but it only works like that on Linux. On Darwin, it defaults to $out/bin/86Box.cfg for some reason (wrapper, perhaps?)
Also, overriding the configuration with -C doesn't seem to have effect at all on Darwin 🤔

This only affects Darwin (tested on aarch64)  when using relative paths - the output bin folder is prepended to the config name

86Box -> cfg file is /nix/store/hwqbm4yv0mg8mdzzhysdggap9qhb3dmf-86Box-4.1.1/bin/86box.cfg
86Box ./some.cfg -> cfg file is /nix/store/hwqbm4yv0mg8mdzzhysdggap9qhb3dmf-86Box-4.1.1/bin/./some.cfg
86Box "$PWD/some.cfg" -> cfg file is correct

Hard to tell if it's a bug from upstream or us at this stage - going to have a look after this PR.

Result of nixpkgs-review pr 335384 run on x86_64-darwin 1

2 packages built:

_86Box
_86Box-with-roms

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1910

Friendly ping for a merge 🙏🏻  @toonn @K900 @SuperSandro2000 @NickCao


added basic embed message example

This exists, just in the wiki rather than an example.
https://github.com/bwmarrin/discordgo/wiki/FAQ#sending-embeds

This exists, just in the wiki rather than an example.
https://github.com/bwmarrin/discordgo/wiki/FAQ#sending-embeds

I didn't know that, thank you for pointing it out. However, I think it would make more sense to put the example in the examples directory rather than the FAQ.

I don't think so. Having it in the examples makes sense, but... message embeds are the core functionality of the Discord API. And all core functionality may have a separated example, but in this case - creating a new example just for embeds - doesn't make any real sense. Since only major sets of API functionality had a separate example: slash commands, components (coming soon btw), audio, etc.

The reason why I made this PR is because I was struggling to figure out how to use embeds. I tried looking through the examples directory but there was no example. It never occured to me to look through the FAQ because, quite frankly, putting examples in the FAQ makes no sense because you've already got the examples directory. I think the future users of this library will find this example useful. If I struggled with this I'm willing to bet that more people did too, and merging my example could save people a ton of time and confusion.

I think having this in the examples directory is a good idea, and creating examples for each major feature rather than each core feature would be better in my opinion

I believe adding this to the examples directory would be a good course of action, for reasons stated by @AverageComet250

Thanks God for this... believe me, I was this giving up on embed with discordgo when I saw this... it needs to be in the example dir

This exists, just in the wiki rather than an example.
https://github.com/bwmarrin/discordgo/wiki/FAQ#sending-embeds

@jolheiser the helper func with newEmbed(). please where is this comming from? it not on discordgo struct itself

This example provides a link to the https://gist.github.com/Necroforger/8b0b70b1a69fa7828b8ad6387ebb3835. Which wraps discordgo.MessageEmbed and provides this functionality


Add module for todo-txt

Add useful aliases for todo-txt.
We can also include alias t recommended in https://github.com/todotxt/todo.txt-cli/wiki/tips-and-tricks, if you agree @belak.

Sorry for the massive delay! This looks good to me!


Refactor GCEClient: wrap compute.Service in an interface for mocking …

…GCP compute
This change creates a ComputeService interface which has a runtime
implementation that wraps the compute.Service. This will enable creating
tests that mock GCP Compute Service calls.
What this PR does / why we need it:
It is difficult to test changes that interact with GCP without manually running integration tests. This change will enable developers to write unit tests that mock GCP and test that their code correctly handles various scenarios.
Release note:
NONE

@kubernetes/kube-deploy-reviewers

In fact, the Go standard library fell into this trap. When https://golang.org/pkg/compress/flate/ first started out, they returned io.ReadCloser. Then they needed to add a Reset optimization, but couldn't break the API contract. They had to return an io.ReadCloser. Since they couldn't update the return type, they had to do a less than optimal design. They had to add (flate.Resetter)[https://golang.org/pkg/compress/flate/#Resetter] so you could type assert to it, and then call Reset as they do in the reset example.
If they had just returned a concrete type that had Read and Close, users could have still assigned it to an io.ReadCloser, and they could have added Reset later without breaking the API contract (because they'd still be returning the same struct).

I think i'm coming from the world where creating an interface is expensive, not cheap like Go. Thanks for the justification Kris, I can see how it is much better not have a single 'master' interface but to instead allow consumers to define the interface and then testers of those consumers can implement their own mocks.
Updating as per comments.

I moved the interface into the machineactuator.go file and renamed it to GCEClientComputeService. I pushed a new version of the PR which has an updated commit message to reflect the changes.

@krousey - why does this need migration? It is squarely in the cloud directory.

@jessicaochen we are going with a single repository now.

@spew - do we still need this change (in the cluster-api repo)?
As this PR is no longer relevant I'm going to close it.

Already completed in the new repo.


Docker cannot connect to Redis

Hi,
Im trying to run docker container with Hydra Router, but get this error:
 docker run -p 5353:5353 --add-host host:MY_HOST --name hydra-router flywheelsports/hydra-router:1.3.3
Redis reconnection attempt 2 of 5...
Redis reconnection attempt 3 of 5...
Redis reconnection attempt 4 of 5...
Redis reconnection attempt 5 of 5...
Error: Max reconnection attempts reached. Check connection to Redis.
    at Promise (/usr/src/app/node_modules/hydra/lib/redis-connection.js:109:18)
    at Promise._execute (/usr/src/app/node_modules/bluebird/js/release/debuggability.js:300:9)
    at Promise._resolveFromExecutor (/usr/src/app/node_modules/bluebird/js/release/promise.js:483:18)
    at new Promise (/usr/src/app/node_modules/bluebird/js/release/promise.js:79:10)
    at Object.until (/usr/src/app/node_modules/hydra/lib/redis-connection.js:107:40)
    at Timeout.setTimeout [as _onTimeout] (/usr/src/app/node_modules/hydra/lib/redis-connection.js:115:30)
    at ontimeout (timers.js:386:14)
    at tryOnTimeout (timers.js:250:5)
    at Timer.listOnTimeout (timers.js:214:5)

Redis is runnign on 6379 on the same host
What am I doing wrong?
Thank you for help.
A.

@intermund the most common reason for the issue above is that your hydra-router config/config.json isn't properly setup to point to your instance of Redis.  Looking at your docker run command above is MY_HOST an export with your machine IP?  If so, that IP can't be localhost or <IP_ADDRESS>
On my mac I use this to grab my current IP:
export HOST_IP=`echo "show State:/Network/Global/IPv4" | scutil | grep PrimaryInterface | awk '{print $3}' | xargs ifconfig | grep inet | grep -v inet6 | awk '{print $2}'`
export HOST_IP=${HOST_IP:='<IP_ADDRESS>'}

On a GNU/Linux box I do this:
HOST=`ifconfig eth0 | grep "inet addr" | cut -d ':' -f 2 | cut -d ' ' -f 1`

@intermund let us know if the above fixed your issue?

@PraveenShrivastava11jan Is this still an issue for you? I've chalked this up to user error.

Thank you, I changed my redis connection inside my config file so it working.

@PraveenShrivastava11jan what config change worked for you? I have a Redis Pod on Kubernetes in Google Cloud and cannot connect to it from other pods in same cluster, with same error. I built my own image to wrap Redis' per their recommendation and overrode the bind config (first commenting and then trying <IP_ADDRESS>) and nothing seems to work. I cannot telnet to redis on 6379 within same cluster.
Dockerfile
FROM redis

# Override config with local version
COPY redis.conf /usr/local/etc/redis/redis.conf

# Start server with new config
CMD [ "redis-server", "/usr/local/etc/redis/redis.conf" ]

redis.conf (relevant settings)
################################## NETWORK #####################################

# By default, if no "bind" configuration directive is specified, Redis listens
# for connections from all the network interfaces available on the server.
# It is possible to listen to just one or multiple selected interfaces using
# the "bind" configuration directive, followed by one or more IP addresses.
#
# Examples:
#
# bind <IP_ADDRESS> <IP_ADDRESS>
# bind <IP_ADDRESS> ::1
#
# ~~~ WARNING ~~~ If the computer running Redis is directly exposed to the
# internet, binding to all the interfaces is dangerous and will expose the
# instance to everybody on the internet. So by default we uncomment the
# following bind directive, that will force Redis to listen only into
# the IPv4 lookback interface address (this means Redis will be able to
# accept connections only from clients running into the same computer it
# is running).
#
# IF YOU ARE SURE YOU WANT YOUR INSTANCE TO LISTEN TO ALL THE INTERFACES
# JUST COMMENT THE FOLLOWING LINE.
# ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# bind <IP_ADDRESS>

# Protected mode is a layer of security protection, in order to avoid that
# Redis instances left open on the internet are accessed and exploited.
#
# When protected mode is on and if:
#
# 1) The server is not binding explicitly to a set of addresses using the
#    "bind" directive.
# 2) No password is configured.
#
# The server only accepts connections from clients connecting from the
# IPv4 and IPv6 loopback addresses <IP_ADDRESS> and ::1, and from Unix domain
# sockets.
#
# By default protected mode is enabled. You should disable it only if
# you are sure you want clients from other hosts to connect to Redis
# even if no authentication is configured, nor a specific set of interfaces
# are explicitly listed using the "bind" directive.
protected-mode no

# Accept connections on the specified port, default is 6379 (IANA #815344).
# If port 0 is specified Redis will not listen on a TCP socket.
port 6379

What is the solution for this ?

@mikesparr @mkrishna2025 Sorry - I'm not a Kubernetes user. @jkyberneees would you happen to know the answer to this?

@mikesparr @mkrishna2025 Sorry for late reply just try # bind <IP_ADDRESS> to <IP_ADDRESS> and protected-mode no to yes in redis config setting

@PraveenShrivastava11jan is correct. I created my own Dockerfile to edit the config file and commented out the bind params and it worked. I was also missing the containerPort param in my Kubernetes manifest. Those two solved and it works fine for a single instance (not clustered).
Working image

mikesparr/docker-redis
https://hub.docker.com/r/mikesparr/docker-redis/ (image)
https://github.com/mikesparr/docker-redis (source)

Working manifest (service + deployment)
apiVersion: v1
kind: Service
metadata:
  name: redis
  labels:
    component: redis
spec:
  ports:
  - port: 6379
  selector:
    component: redis

---

apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: redis
  labels:
    component: redis
spec:
  template:
    metadata:
      labels:
        component: redis
    spec:
      containers:
      - name: redis
        image: mikesparr/docker-redis
        imagePullPolicy: Always
        volumeMounts:
          - name: redisdata
            mountPath: /data
            subPath: redisdata
        ports:
        - containerPort: 6379
          name: transport
          protocol: TCP
      volumes:
        - name: redisdata
          persistentVolumeClaim:
            claimName: redisdata
---
# Request a persistent volume from the cluster using a Persistent Volume Claim.
kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: redisdata
  annotations:
    volume.alpha.kubernetes.io/storage-class: default
spec:
  accessModes: [ReadWriteOnce]
  resources:
    requests:
      storage: 5Gi

I downloaded GIT and updated the Redis manually... Then I created image and deployed.. It works but any alternative option to configure in Portainer itself ?


Credits?

Hi @RokKos,
I think it is great that (1) you credit your contributors, and (2) you try to create minimal Travis CI examples, and (3) you are inspired from my examples.
Perhaps you'd like to mention my work as the source of inspiration?
Feel free not to, I just would feel that would be more honest.

Hi @richelbilderbeek
first I must thank you because when I did research for my own personal project with Travis your repositories helped me immensely. And from my personal project, I created this example. I am kinda ashamed that I forgot to give you at least some credit. I also follow you and you publish amazing work :+1: .
Is it okay to creates a special paragraph in README.md with title inspiration and more reference (or additional reading). With the link to this page or even to your profile?
And again thanks for contributions to open source community and keep on the good work (for some of us you are the inspiration and someone to look up to) :smile:

@RokKos thanks for this friendly post! Just a simple mention is fine.
Perhaps it would be a nice idea to let our GitHubs link to each other, for example my C++11 CMake example.
Good idea? No = fine.

Yea that great idea! :tada:
I will also link to your page with all Travis examples because I think that is a great start if someone wants to start with Travis and some other language and requirements.


Add example of scriptable pie chart

This PR adds a pie/doughnut example doc demonstrating scriptable options to augment the previously merged #5966.  (I spoke to @simonbrunel about this over slack).
I keep to the same format in the document as the other examples found in the scriptable folder with a few changes:

I center the buttons underneath the chart.

I add a scriptable hoverBackgroundColor as well as scriptable backgroundColor.

I add a button that will toggle the view to a Doughnut Chart by changing the cutoutPercentage property.

Let me know if there's anything I can improve on.

@janelledement could you add a fiddle that demonstrates your change? Example template: https://codepen.io/pen?template=JXVYzq

@janelledement you also need to add this new example in samples/sample.js, else it will not show up on the home page.

@simonbrunel Working on adding line and polarArea examples.


help/index.md: Add names of articles to create.

Automated message from Dropbox CLA bot
@rishig, it looks like you've already signed the Dropbox CLA. Thanks!

Cool, merged, thanks @rishig!


installation issue

I have some installation problems：
Before installing the nilm-contrib package, my virtual environment is as follows:
The python version：3.8
cuda version：11.6
cudnn version：8.6.0
nilmtk：0.4.0 (However, in the conda environment, it will be displayed as: nilmtk-0.4.0.dev1+git.303d45b)
pandas：0.25.3, numpy：1.13.3, Keras：>=2.4, tensorflow-gpu：2.4.0
There is a package conflict problem when installing nilm-contrib. I have to download the package offline from anaconda.org and install it with the command conda install nilmtk-contrib-0.1.1-py_0.tar.bz2 (for me it is only normal through this method Install, but some compatibility errors will still be reported after the installation is complete)
The official recommended command: conda install -c conda-forge -c nilmtk nilmtk-contrib does not solve the problem, even if I recreated a virtual environment and the environment only contains python3.8 and some basic packages. This process will take a long time and eventually report the following error message, but it does not show which modules are conflicting. I've been stuck with this problem for a week now, hope it gets fixed

The environmental problem was finally solved.
I reinstalled a virtual environment using conda, and the python version is 3.7 (the 3.8 I used before). Then install the older version of tensorflow first, then install nilmtk, and finally nilm-contrib. In addition to installing nilm-contrib, the error about Keras is reported during the whole process (but it seems to have no effect). Then check if the GPU is available (False). Then the WindowsGRU library was run again, but the CPU was used, and the code was executed for a long time, but fortunately, the environmental problem was solved. The next step may be the problem of cuda and cudnn, I will update it later
My environment configuration is as follows:：
python：3.7
tensorflow-gpu：1.14.0
Keras：2.3.1
pandas：0.25.3
numpy：1.19.5
nilmtk：0.4.0.dev1+git.303d45b
nilmtk-contrib：0.1.1

It took a lot of time to finally install all the environments. The installation process is as follows😀😀:

First create the virtual environment of python3.7 in conda
Use conda install tensorflow-gpu=1.14.0 to install the tensorflow package
Use conda install -c nilmtk nilmtk to install the nilmtk package (it is recommended to add conda-forge to conda channels)
Download the nilm-contrib 0.0.1 version package from the anaconda.org official website, and use conda install --offline nilmtk-contrib-0.1.1-py_0.tar.bz2 to install the package offline after activating the python environment
If you do not have the corresponding cuda and cudnn environment, first download and install the corresponding version of the installation package (python3.7 and tensorflow-gpu1.14.0 require corresponding versions of cuda10.0 and cudnn7.4.2)
Run the deep learning algorithm corresponding to nilm-contrib. If you find that the saving.py file in the Keras package will report an error, open the file and delete all the codes with the suffix .decode() (this problem may be due to python2.x Caused by incompatibility with python3.x)

Hi, I am trying to work with this libraries in Google Colab. Did you by any chance manage to get them to work there? Can you advice on how to do that?
I managed to install and use the NILMTK and NILM_Contrib following the standard installation procedures in their repos (with independent environments) in my computer, but my experiments take too long and want to try the GPUs in cloud.
I would appreciate any support.

How did you manage the Installation without Google Collab? I would most certainly appreciate any help

I did not do anything sophisticated. Just followed the instructions in the repos for both environments (nilmtk & nilm-contrib). I am trying to install contrib in another machine to run some experiments, I will let you know if I encounter any issues now.

Hi, I am trying to work with this libraries in Google Colab. Did you by any chance manage to get them to work there? Can you advice on how to do that?
I managed to install and use the NILMTK and NILM_Contrib following the standard installation procedures in their repos (with independent environments) in my computer, but my experiments take too long and want to try the GPUs in cloud.
I would appreciate any support.

How did you manage the Installation without Google Collab? I would most certainly appreciate any help

I did not do anything sophisticated. Just followed the instructions in the repos for both environments (nilmtk & nilm-contrib). I am trying to install contrib in another machine to run some experiments, I will let you know if I encounter any issues now.

Would you mind telling me exactly what you did for installing the two repos? Did you install them via conda-forge or pip install git+url?
Thanks in advance

Hi, I am trying to work with this libraries in Google Colab. Did you by any chance manage to get them to work there? Can you advice on how to do that?
I managed to install and use the NILMTK and NILM_Contrib following the standard installation procedures in their repos (with independent environments) in my computer, but my experiments take too long and want to try the GPUs in cloud.
I would appreciate any support.

How did you manage the Installation without Google Collab? I would most certainly appreciate any help

I did not do anything sophisticated. Just followed the instructions in the repos for both environments (nilmtk & nilm-contrib). I am trying to install contrib in another machine to run some experiments, I will let you know if I encounter any issues now.

Would you mind telling me exactly what you did for installing the two repos? Did you install them via conda-forge or pip install git+url?
Thanks in advance

I can try to summarize it like this:
For NILMTK:

Install a fresh new anaconda
Open anaconda prompt
Create a new environment: conda create --name nilmtk-env
Install conda: conda config --add channels conda-forge
Update conda version if needed (this was necessary in my case)
Activate the environment you just created: conda activate nilmtk-env
Install NILMTK: conda install -c nilmtk nilmtk (I used this line because the line provided in the repo did not worked, you can try different commands from here if needed: link. However, that line worked smoothly for me on 2 different machines already
The installation ran for about 6-8 hours in my case.
Run your IDE and test it. I am using Jupyter notebooks and it worked.

For the NILM-Contrib I did the same, just changed the line to create the a dedicated environment using the anaconda prompt as adviced in the repo. However, I am trying to install it in a different computer and is not working so far.
Hope I am able to help you. Good luck!


push parse step of parseEvalNumericManyList to C++

This PR moves a costly step in lots of model processing into C++.  On one example model, configureMCMC with useConjugacy = FALSE saves 40% from this change.

This passed testing.  I merged something else into devel and decided to merge devel into this PR branch to initiate new testing.

@paciorek Here is an additional speed improvement that could be included in 0.6-10 if not too late.


Aaru sidecar files

Aaru produces various sidecar files; not sure all are needed. Might be possible to fine-tune this, needs some further investigation.

I can see the following files:

e03214b4-294d-11ed-8943-b526c822f2c0.cicm.xml - metadata file. Contains various checksums, filesystem and fle-level metadata, media type etc.
e03214b4-294d-11ed-8943-b526c822f2c0.resume.xml - partially subset (but not quite) of cicm.xml file.
e03214b4-294d-11ed-8943-b526c822f2c0.log - extraction log
e03214b4-294d-11ed-8943-b526c822f2c0.error.log - error log
e03214b4-294d-11ed-8943-b526c822f2c0.ibg - apparently some ImageBurn-specific format
5 .bin files - no idea?


[BUG] Failed BrokerService#getTopic cause metadata request always fail

Describe the bug
Sometimes the Kafka Client failed with

Topic  not present in metadata after 60000 ms.

Obviously, it's because METADATA request failed. After checking the logs, I found that it's caused by the first failed BrokerService#getTopic.
Here's the logs:
12:14:28.472 [zk-metadata-store-callback-20-1] ERROR io.streamnative.pulsar.handlers.kop.KafkaTopicManager - [[id: 0x65c5c076, L:/<IP_ADDRESS>:9092 - R:/<IP_ADDRESS>:58652]] Failed to getTopic persistent://public/fexfiusuynjxuxrs/my-kafka-topic-partition-0. exception:
org.apache.pulsar.broker.service.BrokerServiceException$PersistenceException: org.apache.bookkeeper.mledger.ManagedLedgerException: org.apache.bookkeeper.mledger.ManagedLedgerException$BadVersionException: org.apache.pulsar.metadata.api.MetadataStoreException$BadVersionException: org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion for /managed-ledgers/public/fexfiusuynjxuxrs/persistent/my-kafka-topic-partition-0
        at org.apache.pulsar.broker.service.BrokerService$3.openLedgerFailed(BrokerService.java:1122) ~[org.apache.pulsar-pulsar-broker-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl.lambda$asyncOpen$9(ManagedLedgerFactoryImpl.java:426) ~[org.apache.pulsar-managed-ledger-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at java.util.concurrent.CompletableFuture.uniExceptionally(CompletableFuture.java:884) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture$UniExceptionally.tryFire(CompletableFuture.java:866) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:488) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:1990) ~[?:1.8.0_275]
        at org.apache.bookkeeper.mledger.impl.ManagedLedgerFactoryImpl$2.initializeFailed(ManagedLedgerFactoryImpl.java:421) ~[org.apache.pulsar-managed-ledger-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl$1.operationFailed(ManagedLedgerImpl.java:364) ~[org.apache.pulsar-managed-ledger-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at org.apache.bookkeeper.mledger.impl.MetaStoreImpl.lambda$null$1(MetaStoreImpl.java:81) ~[org.apache.pulsar-managed-ledger-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at java.util.concurrent.CompletableFuture.uniExceptionally(CompletableFuture.java:884) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture$UniExceptionally.tryFire(CompletableFuture.java:866) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture.postComplete(CompletableFuture.java:488) ~[?:1.8.0_275]
        at java.util.concurrent.CompletableFuture.completeExceptionally(CompletableFuture.java:1990) ~[?:1.8.0_275]
        at org.apache.pulsar.metadata.impl.zookeeper.ZKMetadataStore.lambda$null$10(ZKMetadataStore.java:186) ~[org.apache.pulsar-pulsar-metadata-<IP_ADDRESS>.jar:<IP_ADDRESS>]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) [?:1.8.0_275]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) [?:1.8.0_275]
        at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) [io.netty-netty-common-4.1.51.Final.jar:4.1.51.Final]
        at java.lang.Thread.run(Thread.java:748) [?:1.8.0_275]
Caused by: org.apache.bookkeeper.mledger.ManagedLedgerException: org.apache.bookkeeper.mledger.ManagedLedgerException$BadVersionException: org.apache.pulsar.metadata.api.MetadataStoreException$BadVersionException: org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion for /managed-ledgers/public/fexfiusuynjxuxrs/persistent/my-kafka-topic-partition-0
Caused by: org.apache.bookkeeper.mledger.ManagedLedgerException$BadVersionException: org.apache.pulsar.metadata.api.MetadataStoreException$BadVersionException: org.apache.zookeeper.KeeperException$BadVersionException: KeeperErrorCode = BadVersion for /managed-ledgers/public/fexfiusuynjxuxrs/persistent/my-kafka-topic-partition-0
12:14:28.474 [zk-metadata-store-callback-20-1] WARN  io.streamnative.pulsar.handlers.kop.KafkaRequestHandler - [[id: 0x65c5c076, L:/<IP_ADDRESS>:9092 - R:/<IP_ADDRESS>:58652]] findBroker: Failed to getOrCreateTopic persistent://public/fexfiusuynjxuxrs/my-kafka-topic-partition-0. broker:PLAINTEXT://<IP_ADDRESS>:19092, exception:
12:14:28.474 [zk-metadata-store-callback-20-1] WARN  io.streamnative.pulsar.handlers.kop.KafkaRequestHandler - [[id: 0x65c5c076, L:/<IP_ADDRESS>:9092 - R:/<IP_ADDRESS>:58652]] Request RequestHeader(apiKey=METADATA, apiVersion=6, clientId=producer-1, correlationId=1): Exception while find Broker metadata
12:14:28.474 [zk-metadata-store-callback-20-1] WARN  io.streamnative.pulsar.handlers.kop.KafkaRequestHandler - Failed find Broker metadata, create PartitionMetadata with NOT_LEADER_FOR_PARTITION
12:14:28.588 [pulsar-web-42-4] INFO  org.eclipse.jetty.server.RequestLog - <IP_ADDRESS> - - [15/Jan/2021:12:14:28 +0000] "GET /admin/v2/persistent/public/fexfiusuynjxuxrs/my-kafka-topic/partitions HTTP/1.1" 200 16 "-" "Pulsar-Java-v<IP_ADDRESS>" 4
# Above 3 lines repeated until client disconnected

The first error is caused by https://github.com/streamnative/kop/blob/694fd2d35531ef8eccaf80e399cb5a9afd9e82b1/kafka-impl/src/main/java/io/streamnative/pulsar/handlers/kop/KafkaTopicManager.java#L298-L304
They we can see the repeated error is caused by KafkaTopicManager#getTopic: https://github.com/streamnative/kop/blob/694fd2d35531ef8eccaf80e399cb5a9afd9e82b1/kafka-impl/src/main/java/io/streamnative/pulsar/handlers/kop/KafkaRequestHandler.java#L1355-L1361
To Reproduce
The KoP version is v<IP_ADDRESS>-rc-202101150854 (branch-2.7.0). It's not easy to reproduce. It's my private integration test environment, which will set 1 ZK, 1 BK, 1 Broker and KoP for simple produce test.
Expected behavior
A failed BrokerService#getTopic invocation should not affect followed the METADATA request attempts.

I'm not sure if #267 fixed the bug, I'll cherry-pick this commit and test again. Currently the test has about 50% possibility to fail.

I'm not sure if #267 fixed the bug, I'll cherry-pick this commit and test again. Currently the test has about 50% possibility to fail.

Confirmed that #267 works

Confirmed that #267 works


Remove finished todo from README.md

Remove todo for bitwidth>8. This has been fixed in 6001b5e777d3b09faa418e597baa502685be20c9

Updated version of https://github.com/jcrobak/parquet-python/pull/14


begin new_alignment.SequenceCollection

Waiting on #1866

[x] Create a SequenceCollection class
[x] SequenceCollection(seq_data=, moltype=..)
[x] type hints the constructor
[x] Assigns seq_data to a _seq_data attribute
[x] Assigns self._seq_data.seq_maker = self.moltype.make_seq
[x] Implements a seqs property which returns the _seq_data
[x] Ports relevant tests from test_alignment.py into test_new_alignment.py, converting them into pytest style (sorry!)
[x] Implements a named_seqs property, which just returns .seqs . Deprecate this method and point to just using .seqs (give this 12 months).

@GavinHuttley will the new class be inheriting from _SequenceCollectionBase?

No. _SequenceCollectionBase is an artefact of needing core methods on both SequenceCollection (from which Alignment directly inherits) and ArrayAlignment (which inherits directly from _SequenceCollectionBase).
In new_alignment.py, define a SequenceCollection class without any reference to alignment.py classes. All it has to begin with is a constructor and .seqs attribute.

closing this as there are now specific issues that are more granular


Profiler % metrics on small sampled data are greater than 100%

Affected module
Profiler
Describe the bug
When running the profiler on a small sample set of data, the % metrics in the profiler can ended being greater than 100%.
To Reproduce
Run the profiler against a small dataset and add a sample_profile value
Expected behavior
percentage metrics should never be greater than 100
Version:

OpenMetadata version: latest (0.11.0)

This is caused by  a non-deterministic random sample function.


Is the reinforced learning like AlphaGo successed?

Regards...

You mean the method described in the Nature paper?
I think most of the strong programs are adopting that method to generate the value network.

Yes I know...
But as I know, the ray isn't adopted yet.
I would like to know about AQ.
Regards...

AQ adopts the method.
Please wait until the learning part is released to know the detail.

Thanks...


Dummy net lines are absent

And again thank you very much for your script!
There is another glitch I found I think.
Shapes with "dummy nets" from etch are exported but lines are not. Is it possible to add them to export? I have some text made with lines of dummy net, it would be great to have it exported to graphics.

Please see file attached to this message.

Further progress is discussed in this issue.


Add transport data from maps.yandex.ru api

Breaking Change:

Description:
This feature allow to obtain bus departure time at transport stops in many russian cities
Related issue (if applicable): fixes #
Pull request with documentation for home-assistant.io (if applicable):
https://github.com/home-assistant/home-assistant.io/pull/10375
ya_ma library version updated to 0.3.7
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: yandex_transport
    stop_id: 9639579

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Thanks

Thanks

No problem


Support reward notifications for more scenarios

I believe that if you follow the best practice of using a separate (offline) wallet (describe here https://github.com/Chia-Network/chia-blockchain/wiki/Good-Security-Practices-on-Many-Machines#keep-your-wallet-separate) you won't get notified (because the wallet logs won't contain the lines for the correct wallet)? Maybe it's a good idea to have a separate notification for when you have won and another one that watches the wallet logs?

I'll hijack that issue and generalise it a little so that we have the full context in the same place.
The XCH notifications currently work only in the following scenario:

Running full node + harvester + wallet on the same machine and receiving rewards on that same wallet.

These notifications are just a bonus on top of critical alerts in monitoring your harvesters, so resolving that won't be a priority from my side but contributions are welcome.

The low hanging fruit here is to bring back the "Found proof" notifications. I'll do that even though a found proof doesn't guarantee an actual reward. It's the best we can do that works for all possible setups.

Few notes for anyone possibly interested in tackling this:

If harvester is running standalone, only "found proofs" events can be parsed out from the logs. However, finding a proof doesn't guarantee a reward.

Having the "found proof" notifications are enough for me for now. Thanks!

Hi,
Since we have the official pool protocol, finding proofs doesn't give much more info if we have found a block with a reward destination inna separated wallet.
The only usefull information in this case I have found in the debug log was this line :
2021-07-24T16:01:42.245 full_node chia.full_node.full_node: INFO     Starting to make the unfinished block
2021-07-24T16:01:42.255 full_node chia.full_node.full_node: INFO     Made the unfinished block
2021-07-24T16:01:42.328 full_node chia.full_node.full_node: INFO     🍀 Farmed unfinished_block 409126291baa2debe79099cff716ac0aedc451354105ed2667dc1ddfed3d8be0, SP: 42, validation time: 0.04041004180908203, cost: 202394374

Could we consider to add such parser?

Actually the answer in https://github.com/martomi/chiadog/issues/252#issuecomment-878092188 would solve this issue.
Hope this will come soon :)


asyncio.exceptions.TimeoutError  when enable 2 or more jbd bms in configuration

ERROR    [sampling.py:125] xiaoxiang_BMS_8KWT error: <class 'asyncio.exceptions.TimeoutError'>
INFO     [sampling.py:51] bms debug data: %s
ERROR    [sampling.py:125] xiaoxiang_BMS_11KWT error: <class 'asyncio.exceptions.TimeoutError'>
INFO     [sampling.py:51] bms debug data: %s
ERROR    [main.py:274] 2 exceptions occurred fetching BMSs
ERROR    [main.py:85] Error (num 6) reading BMS:
ERROR    [main.py:86] Stack: Traceback (most recent call last):
File "/usr/lib/python3.10/asyncio/tasks.py", line 456, in wait_for
return fut.result()
asyncio.exceptions.CancelledError
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/app/main.py", line 81, in fetch_loop
await fn()
File "/app/main.py", line 275, in fn
raise exceptions[0]
File "/app/main.py", line 270, in fn
await t
File "/app/bmslib/sampling.py", line 47, in call
return await self.sample()
File "/app/bmslib/sampling.py", line 69, in sample
sample = await bms.fetch()
File "/app/bmslib/jbd.py", line 75, in fetch
buf = await self._q(cmd=0x03)
File "/app/bmslib/jbd.py", line 69, in _q
return await self._fetch_futures.wait_for(cmd, self.TIMEOUT)
File "/app/bmslib/init.py", line 44, in wait_for
return await asyncio.wait_for(self._futures[name], timeout)
File "/usr/lib/python3.10/asyncio/tasks.py", line 458, in wait_for
raise exceptions.TimeoutError() from exc
asyncio.exceptions.TimeoutError

Thats odd. Did you try toggle keep_alive ?

Please re-open if issue still exists


dry-run does not work with artifactory move and copy

In Version 1.10.1 it worked, then I updated to 1.15 and it does not work anymore for copy and move, but still for delete.
[ahu jfrogtest ]$ jfrog rt mv --dry-run ahutest/othertestfile/test2 ahutest/testfile/
[Info] Searching artifacts...
[Info] Found 1 artifact.
[Info] [Dry run]  Moving artifact: ahutest/othertestfile/test2 to: ahutest/testfile/othertestfile/test2
[Info] Moved 1 artifacts.

[ahu jfrogtest ]$ jfrog --version
jfrog version 1.15.1
[ahu jfrogtest ]$ jfrog rt mv --dry-run ahutest/testfile/othertestfile/test2 ahutest/testfile/
[Info] Searching artifacts...
[Info] Found 1 artifact.
[Info] Moving artifact: ahutest/testfile/othertestfile/test2 to: ahutest/testfile/testfile/othertestfile/test2
{
  "status": "success",
  "totals": {
    "success": 1,
    "failure": 0
  }
}

@peshay,
Thank you for reporting this issue.
We are planning to fix it in the next release 1.16.0

@peshay,
JFrog CLI v1.16.0 is released with a fix for this issue. Feel free to upgrade.
We'll appreciate your feedback for that.

Thanks for the quick fix! But I would also really appreciate if someone could made this copy/move to different paths possible #163 :)


[next] feat: remove network package

Refactor 🚂

Removes @stacks/network package

Moves most functionality to transactions package (this will become the main package allowing most folks to use it and not need to worry about other imports)
Move to a network OBJECT approach (similar to bitcoin-js, micro-btc-signer, and others). The network will not be an instantiated class, but a static exported object with all information that might be relevant (txVersion, magicBytes, etc.)
Remove fetch functions from network and introduce StacksApi, which can derive it's URL from a network, but should be treated separately -- more closely mirroring to what's actually happening in the background.
The majority of users doesn't realize network is also setting changes for "networking"

Most users don't customize fetchFn or URL. For users hosting their own node the only change becomes { api: { url: "my-node.com" }, ...} in the params of most relevant tx functions.

Some changes might irritate users at first, but I think it's an approach that follows the structure of the blockchain and in a way educated while using the library. More to follow

Curious to get feedback 🙏 @hugocaillard @zone117x @rafaelcr

Thoughts on param naming?

Should url be baseUrl, or nodeUrl
Should StacksApi be StacksNode or StacksNodeApi

I like it short, but "node" could be helpful (even though most users wrap their node with the API). We could also add a "node-only mode" which ensures the requests it's trying to make exist on the v2 rather than extended.

What should live where?

Should we move most things into @stacks/transactions (is there a case for having users import from @stacks/common for basic interactions or should it stay for helpers used by all @stacks/ packages?)

Examples
Diff for most methods
const transaction = await makeSTXTokenTransfer({
  recipient,
  amount,
  senderKey,
  memo,
- network: new StacksTestnet({ url: "mynode-optional.com" }), // optional options
+ network: STACKS_TESTNET,
+ api: { url: "mynode-optional.com" } // optional param
});

String enum params still work (e.g. 'testnet') ☝

New API class
const api = new StacksApi({ network: STACKS_TESTNET });
await api.broadcastTransaction(tx); // bundled, discoverable API maintained endpoints/methods (with helper processing)

Hmm good arguments for both options. I don't really have a strong opinion. Kinda lean towards keeping the common package so that other packages aren't transitively dependent on the transaction package for helpers that are unrelated to transactions

Splitting this out into separate PRs

closed in favor of #1622


⚠ SceneTime has degraded performance

In 2963b06, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2326 ms

Resolved: SceneTime performance has improved in f1c9076 after 13 minutes.


bug: need accessibility improvements

Description
The accessibility of this site can be improved. For example, there is

the lack of lang attribute in html element
link icons don't have alt text / title
headings are not correctly ordered (sequentially-descending order)
Color contrast issues
Empty links
etc etc
make sure to do these accessibility improvements in global components too that are present in the page you are working on (for example ScrollToTop button or Header component)

You can use https://wave.webaim.org/report#/https://walkingpal-6tqqs5cew-walkingpal.vercel.app/ to do all your accessibility analysis. It's the latest Development build.

Checkout the file changes and PR review comments in this PR to get more clear view on what changes you need to do.

There are 5 pages. We need to do this in all 5 pages.

Progress

[x] Home page (@satnaing   )
[ ] About page (@Asmi-1 )
[ ] Company page (@satnaing   )
[ ] FAQs page
[ ] Register page

⚠Contributors who are interested, please tell beforehand that which page you are going to start working on. This will avoid clashes (possibility of 2 people working on the same page).

headings are not correctly ordered (sequentially-descending order)

We have never thought of this before. We have not used headings in sequential order. We just followed what was heading was given in the figma design board.
I don't think it can be changed without altering the figma designs too.

@satnaing I have assigned you Home Page.

I want work on About page

@satnaing assigned you company page. Make sure cover all global components used in company page too

@ashuvssut I would love to work on FAQs Page


Update report to discuss extending analysis to other months and years

I'm planning to ask a TA how to approach this in the lab this week.

As per Tiff: we don't have the method yet to analyze multiple p-values or time series. Instead of updating analysis, update the report to mention that we would be interested in exploring that in future studies. It might also be good to refer to the seasonal trends in the histogram for the months and CIs for the years.

Added the new references to the Rmd. to match .bib


Add a "Clear" link like "Browse" to the label

How do I add a "Clear" link like "Browse" to the main label to clear the input?

Hi, there is currently no way to add buttons to the drop area. You might be able to change the label and add HTML elements to it, then you could try adding event listeners, not sure if that will work.


Honor kubelet version when creating GCE VMs.

Part of https://github.com/kubernetes/kube-deploy/issues/352.

This change is


Wrong parsing of CWMP method name for XML Self Closing Tags

If the CWMP XML tag is self closing without a space before /, then the '/' character is also considered in the method name.
E.g.<cwmp:GetParameterNamesResponse/>could be taking "GetParameterNamesResponse/" as the method name. The last character / should be excluded.
2016-06-23 19:18:01.420 ERROR  xxxxxUnit                           An error ocurred: Could not process HTTP-request (from TR-069 client)
class com.owera.xaps.tr069.UnknownMethodException:The method GetParameterNamesResponse/ was not not recognized by the server.
    com.owera.xaps.tr069.methods.HTTPRequestProcessor.processRequest(HTTPRequestProcessor.java:477)
    com.owera.xaps.tr069.Provisioning.doPost(Provisioning.java:210)
    javax.servlet.http.HttpServlet.service(HttpServlet.java:650)
    javax.servlet.http.HttpServlet.service(HttpServlet.java:731)
    org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:303)
    org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
    org.apache.tomcat.websocket.server.WsFilter.doFilter(WsFilter.java:52)
    org.apache.catalina.core.ApplicationFilterChain.internalDoFilter(ApplicationFilterChain.java:241)
    org.apache.catalina.core.ApplicationFilterChain.doFilter(ApplicationFilterChain.java:208)
    org.apache.catalina.core.StandardWrapperValve.invoke(StandardWrapperValve.java:221)
    org.apache.catalina.core.StandardContextValve.invoke(StandardContextValve.java:122)
    org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:505)
    org.apache.catalina.core.StandardHostValve.invoke(StandardHostValve.java:169)
    org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:103)
    org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:956)
    org.apache.catalina.core.StandardEngineValve.invoke(StandardEngineValve.java:116)
    org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:436)
    org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:1078)
    org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:625)
    org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:318)
    java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
    java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
    org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61)
    java.lang.Thread.run(Thread.java:745)

Could be solved by making XML parsing of the input, instead of raw String operations. Will involve in the fixing and commit if time permits.

fixed in latest release candiate of rebooted project located at https://github.com/freeacs/freeacs if anyone wonders. https://github.com/freeacs/freeacs

Just mentioning that this problem has been fixed once and for all. I have made several improvements in the last months, one of them being getting rid of all regex ugly stuff code to retrieve for example cwmp methods. Now the cwmp method is found in a future proof, type safe and functional manner. TLDR; it should just work.
New release at https://github.com/freeacs/freeacs/releases/tag/v3.0-RC1
I have moved away from deb and rpm files, so the release is a distribution zip of multiple zips. Each module contains a start script, jar files, configuration etc.

Artig å se at du jobber videre, håper du lykkes!
mvh
Morten S
tir. 30. apr. 2019 kl. 19:49 skrev Jarl André Hübenthal <
<EMAIL_ADDRESS>
Just mentioning that this problem has been fixed once and for all. I have
made several improvements in the last months, one of them being getting rid
of all regex ugly stuff code to retrieve for example cwmp methods. Now the
cwmp method is found in a future proof, type safe and functional manner.
TLDR; it should just work.
New release at https://github.com/freeacs/freeacs/releases/tag/v3.0-RC1
I have moved away from deb and rpm files, so the release is a distribution
zip of multiple zips. Each module contains a start script, jar files,
configuration etc.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mortensimon/tr069/issues/4#issuecomment-488049230,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLPTRZXSO4FTZYHRG33KWDPTCBCJANCNFSM4CH3JIBQ
.


pull request for e2e test at 01/21/2017 17:56:16

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


Fix sim speed benchmark

The benchmark is out of date with the new reset and data loading changes. This PR fixes it.

cool, will run this and check new perf soon!

@jsuarez5341 I dont think you would see any perf changes. I am only bringing the benchmark script up to date with a couple of API changes.


git-checkout-pull-merge.sh does not run git submodules

Describe the bug
git-checkout-pull-merge.sh does not run submodule installations
Opposed to - git-checkout-pull.sh
To Reproduce
Use Pullrequest environments for projects that have submodules
Expected behavior
Submodules are installed after the merge of the source branch

@Schnitzel is the git submodule --init not in there by accident or is this by design?

Fix: https://github.com/amazeeio/lagoon/pull/558

Looked at changing it and moving the git submodule operation outside the two git files but that makes the whole thing a bit harder to understand. whereas now we have all the git operations in the files that run trough.


Does the webpack prod build exclude all spec.js files from bundle.js?

Does the webpack prod build exclude all spec.js files from bundle.js?

I believe it does because webpack only compile the files that are references via require or import. You can test this by creating a new file without referencing it and you should not see it in the list of source mapped files in your browser.

Well said @kwelch. Thanks!


tern.el not working with emacs latest version

I'm trying to use tern with the latest version of emacs (27 : master branch) and I'm getting the following error :
Request failed: ((wrong-type-argument Need a pre-parsed URL. [cl-struct-url http nil nil <IP_ADDRESS> 39585 / nil nil nil nil t] url))

It seems to come from tern-req function, this line :
(url (url-parse-make-urlobj "http" nil nil tern-server port "/" nil nil nil))

But I don't know how to fix it, 'cause I'm not familiar with lisp development

Yes the same of me .
emacs version: 26.1-rc1 and 27.0.50
(url (url-parse-make-urlobj "http" nil nil tern-server port "/" nil nil nil))

Does the maintainer didn't have the willing or be out of steam of it?
Hope for fixing and Regards 😄

tern is no longer maintained https://github.com/ternjs/tern/commit/ae83725819286d3f134a6b9956f1117b341e1040

Hi again,
I asked for help on emacs stackexchange and found it :)
There is no problem with tern itself, you just need to re-compile it with your version of Emacs.
Here is the link to the solution.


Automated Package Upgrade

:tada: This PR is included in version 12.3.12 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Machine controller upgrade documentation

This should consider the different architectures where the controller might be running.

This issue was moved to kubernetes-sigs/cluster-api#38


Update progress95.schema.json

Minor proposed change to title:
before: 95% Progress Marker
after: Video Play 95% Progress Marker

This schema was added initially to support Media Analytics data, which includes both video and audio streams. Changing the name to include "Video" will exclude the idea that this metric is also exposed for audio streams.


iOS GS1 is not recognize

Does this plugin supports GS1 data bar?
So far what's working in GS1 Databar is the is GS1-128.
To test this barcodes just go to this barcode generator.
Thanks,
Moiz

@amoizesmailacn no, it is not supported.


Allow unicode surrogates to be stored in str

Related to #4470 .
This pull request attempts to allow RustPython to store unicode surrogates in a str.

This is potentially not safe. Rust will fill the unused char ranges with other values when it can do. (if not yet, it can do those optimization in future)
To correctly fix this problem, #3574 need to be finished.
are you interested in it?

I'm still looking for a better solution for this. Sure. I'll take a look at #3574. Thanks.

@garychia did you get any idea about it?

Currently the parser uses Rust chars and strings to handle Python string literals. This does not work with Unicode surrogate characters. We may need to change the way we store Python string literals. My suggestion is to replace Rust strings with our custom class (such as PyStr, I guess). But I guess this may require a lot of work to be done.

Perhaps the bstr crate can be useful.


fix wording

based on user in #general chat

Tom is not around so I'll just merge this myself. Very straight forward typo.
Thanks

@nikooo777 thanks Niko... Tom on vacation right now


[Concurrency] Un-gate implicit nonisolated access to struct vars and isolated subclassing.

These features are additive, and they don't need to be gated behind the GlobalActorIsolatedTypesUsability upcoming feature. The other inference changes, including @Sendable inference for global-actor-isolated function types, and global-actor inference on protocol refinements, remain gated behind the upcoming flag.

@swift-ci please smoke test

@swift-ci please smoke test


Pip install fails

macbookpro:~ sander$ pip install invoice2data
Collecting invoice2data
  Using cached invoice2data-0.2.47.tar.gz
Collecting pytesseract (from invoice2data)
  Using cached pytesseract-0.1.6.tar.gz
Collecting pillow (from invoice2data)
  Using cached Pillow-4.0.0-cp27-cp27m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
Collecting pyyaml (from invoice2data)
  Using cached PyYAML-3.12.tar.gz
Collecting dateparser (from invoice2data)
  Using cached dateparser-0.6.0-py2.py3-none-any.whl
Collecting unidecode (from invoice2data)
  Using cached Unidecode-0.04.20-py2.py3-none-any.whl
Collecting pdfminer.six (from invoice2data)
  Using cached pdfminer.six-20160614.zip
Collecting olefile (from pillow->invoice2data)
  Using cached olefile-0.44.zip
Collecting tzlocal (from dateparser->invoice2data)
  Using cached tzlocal-1.3.tar.gz
Requirement already satisfied: pytz in /System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python (from dateparser->invoice2data)
Requirement already satisfied: python-dateutil in /System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python (from dateparser->invoice2data)
Collecting ruamel.yaml (from dateparser->invoice2data)
  Using cached ruamel.yaml-0.13.14.tar.gz
    Complete output from command python setup.py egg_info:
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:6:8: warning: explicitly assigning value of variable of type 'yaml_parser_t' (aka 'struct yaml_parser_s') to itself [-Wself-assign]
    parser = parser;  /* prevent warning */
    ~~~~~~ ^ ~~~~~~
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:6:10: warning: variable 'parser' is uninitialized when used here [-Wuninitialized]
    parser = parser;  /* prevent warning */
             ^~~~~~
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:5:1: note: variable 'parser' is declared here
    yaml_parser_t parser;
    ^
    2 warnings generated.
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:6:8: warning: explicitly assigning value of variable of type 'yaml_parser_t' (aka 'struct yaml_parser_s') to itself [-Wself-assign]
    parser = parser;  /* prevent warning */
    ~~~~~~ ^ ~~~~~~
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:6:10: warning: variable 'parser' is uninitialized when used here [-Wuninitialized]
    parser = parser;  /* prevent warning */
             ^~~~~~
    /var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/tmp_ruamel_J1xp6L/test_ruamel_yaml.c:5:1: note: variable 'parser' is declared here
    yaml_parser_t parser;
    ^
    2 warnings generated.
    sys.argv ['-c', 'egg_info', '--egg-base', 'pip-egg-info']
    test compiling test_ruamel_yaml
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/pip-build-VrgoKR/ruamel.yaml/setup.py", line 854, in <module>
        main()
      File "/private/var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/pip-build-VrgoKR/ruamel.yaml/setup.py", line 843, in main
        setup(**kw)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/core.py", line 111, in setup
        _setup_distribution = dist = klass(attrs)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/setuptools/dist.py", line 272, in __init__
        _Distribution.__init__(self,attrs)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/distutils/dist.py", line 287, in __init__
        self.finalize_options()
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/setuptools/dist.py", line 326, in finalize_options
        ep.require(installer=self.fetch_build_egg)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 2385, in require
        reqs = self.dist.requires(self.extras)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 2617, in requires
        dm = self._dep_map
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 2606, in _dep_map
        if invalid_marker(marker):
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 1424, in is_invalid_marker
        cls.evaluate_marker(text)
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 1549, in _markerlib_evaluate
        env = cls._translate_metadata2(_markerlib.default_environment())
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 1537, in _translate_metadata2
        for key, value in env
      File "/System/Library/Frameworks/Python.framework/Versions/2.7/Extras/lib/python/pkg_resources/__init__.py", line 1536, in <genexpr>
        (key.replace('.', '_'), value)
    ValueError: too many values to unpack

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/kl/c7vdfbm92sz3j_frpf8m5tfm0000gn/T/pip-build-VrgoKR/ruamel.yaml/

Thanks for writing in. This error is probably related to you trying to install directly in macOS' system python or the external package ruamel.yaml you're installing rather than this project's code.


Fixed "Error parsing [float] as 64-bit integer" in Model::freshTimestamp()

Fixes an error when round(microtime(true) * 1000) works out to be a float. Running in HHVM 3.15.4
https://github.com/jenssegers/laravel-mongodb/issues/712

Can you change the casting to (int)?

Can you change the casting to (int)?

Absolutely - done!

@jenssegers @macdabby Overflow?!
microtime(true) =<PHONE_NUMBER>.2899
microtime(true) * 1000 =<PHONE_NUMBER>289.9
round(microtime(true) * 1000) =<PHONE_NUMBER>290
round(microtime(true) * 1000) =<PHONE_NUMBER>290
(int) round(microtime(true) * 1000) =<PHONE_NUMBER>

new UTCDateTime((int) round(microtime(true) * 1000)) will evaluate to a date in 1970-ish.

Interesting - php version?

This function should also accept null to use the current time. @jenssegers should we use that instead?

@macdabby Wow, why didn't we see that.