Test additional Python versions and dependency versions

This PR introduces tests for Python 3.6 and PyPy 3.7 and 3.8, as well as tests with the minimum declared versions of dependencies.
This depends on #22.

Apparently this fails tests on PyPy 3.7 so we'll have to figure out some way to work around that.

Yeah, I should install Pypy and see how well some of my other stuff works.  While not as commonly used, from what I understand it's a very fast implementation of Python.

That's what I hear as well. I've never really found a need to use it myself, but I can see how it would be beneficial with some particularly complex scripts or long-running operations. Some people used it for a few things at my prior-to-current job, and for the most part it was a drop-in replacement for CPython, but there are definitely some edge cases.

Based on the latest test results I think this is going to be blocked by #20, so I'll work on that soon.


metadata for a few 2003 sequences that are not in the nextstrain tree

Also includes the metadata and lat_longs and (unknown collection date) correction for the new French and Netherlands sequence
The USA 2003 sequences include the only squirrel in the tree, so worth to add it !

I'll merge this. But one question @babarlelephant: did you add the lat-longs because they were missing? Augur should have most of these countries already covered:
https://github.com/nextstrain/augur/blob/master/augur/data/lat_longs.tsv


Fixing a security issue which is banned by google play.

Please look issue #1260 .

Also, I fixed travis-ci issues.

Also , I fixed travis ci build tools version issues. Please check the fixes.

@BahadirBulduk , @miiant your commits were merged into https://github.com/Vorlonsoft/AndroidAsyncHTTP


Recieved corrupt message when running locally

Hi, thank you for your work on this project, it's been a real help using Prisma with Deno.
However, when trying to use the proxy I get an error:
Cannot fetch data from service:
error sending request for url (https://localhost:3851/4.7.1/b7b4fb7095922bd7b7c1126f4ffb522c4a94a7c7baa9310b74ef3789231d8756/graphql): error trying to connect: received corrupt message
    at wn.handleRequestError (file:///.../backend/generated/client/runtime/edge-esm.js:80:18717)
    at wn.handleAndLogRequestError (file:///.../backend/generated/client/runtime/edge-esm.js:80:18249)
    at wn.request (file:///.../generated/client/runtime/edge-esm.js:80:18087)
    at async t._request (file:///.../backend/generated/client/runtime/edge-esm.js:97:2022)
    at async file:///.../backend/src/prisma.ts:27:15
Watcher Process finished. Restarting on file change...

First thing I noticed was that it was trying to use https://. Is this perhaps the issue?
I put a console.log in the beforeMiddleware.js to log every request. Requests sent manually via curl to the proxy-alt server log to the console, but when using the Prisma client nothing gets logged, possibly indicating it never actually reached the server. I have a feeling it's expecting an https response and getting an http one, or something along those lines.
I'd like to run the proxy on localhost for integration tests :)

See https://github.com/aiji42/prisma-data-proxy-alt#launch-proxy-server

This will bring up the proxy server, but it must be SSL-enabled to connect from @prisma/client.
So here are the steps to establish a local connection with SSL using docker-compose and https-portal with a self certificate.

@prisma/client communicates over https when in data proxy mode, even if local.

@jeremyjacob Hello, please, did you manage to run this on deno deploy by any chance?

At the time I was running this locally—not on Deno Deploy.


suddenly getting this message

Repro steps:

Action: cosmosDB.executeMongoCommand
Error type: Error
Error Message: Could not find unique name for new file.
Version: 0.11.0
OS: darwin
Product: Visual Studio Code
Product Version: 1.40.2
Language: en

Call Stack
extension.bundle.js:105:19104
s extension.bundle.js:105:18035

Please provide repro steps.


feat(metric-provider): use explore v4 and analytics bridge [MA-2530]

BREAKING CHANGE: new query endpoint; requires analytics bridge

Move to using the analytics bridge for queries
Move to explore v4 query format and response format
Support passing in an AbortController
Use promise resolve / reject to determine error states, rather than Axios response objects
Add cacheKey method to timeframes MA-1960
Add v4Query method to timeframes for better explore v4 support

Summary

Resources

Creating a package docs

Setting back to draft to split out the analytics-utilities changes.


clarification about steps to be skipped

#5243

@nenonix @JohanFreelancer9 - kindly do the copy editor review for this PR. Thanks.

@mapalko - PR has been copy edited and is ready for final review based on issue #5243.
Thank you.
cc: @kenwith

@mypil : Document author has approved the changes, but the PM/DM is needed for the final merge.

@Dansimp - Please assist to merge this PR as approved by @mapalko.
Thank you.
cc: @kenwith


ApplicationCharge endpoint support. Tests for the #91.

This PR adds ApplicationCharge endpoint support. It also includes #91 (original commit) with extensive tests as it was requested which, in effect, renders #91 resolved.
cc: @orian

Codecov Report

Merging #107 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #107   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          19     20    +1
  Lines         645    699   +54
=====================================
+ Hits          645    699   +54

Impacted Files
Coverage Δ

applicationcharge.go
100% <100%> (ø)

goshopify.go
100% <100%> (ø)
:arrow_up:

recurringapplicationcharge.go
100% <100%> (ø)
: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 f5e2c87...e9f7834. Read the comment docs.

I think that I might have accidentally overwritten changes from #102 with this commit (CI failure). This is now fixed. I think it might be more convenient to use something like https://godoc.org/github.com/stretchr/testify/assert#Equal instead of DeepEquals.

@amwolff testify is indeed a nice library. I can't remember why I didn't use it when I created go-shopify to be honest, but it was either I didn't know about the library at the time or I wanted to reduce dependencies :-) Thanks for the PR btw! I'll review it later today when I have a bit more time, unless someone beats me to it :beers:

Also, sorry for the delay. Yesterday was shorter than I thought :thinking: :slightly_smiling_face:


fix minor spelling to fix-minor-spelling

Contributing to Twilio

All third party contributors acknowledge that any contributions they provide will be made under the same open source license that the open source project is provided under.

[x] I acknowledge that all my contributions will be made under the project's license.

Pull Request Details
JIRA link(s):

AHOYAPPS-0000

Description
A description of what this PR does.
Burndown
Before review

[ ] Updated CHANGELOG.md if necessary
[ ] Added unit tests if necessary
[ ] Updated affected documentation
[ ] Verified locally with npm test
[ ] Manually sanity tested running locally
[ ] Included screenshot as PR comment (if needed)
[ ] Ready for review

Before merge

[ ] Got one or more +1s
[ ] Re-tested if necessary

Hi @t4nmoy - thanks for the pull request!
I think the PR looks good, but unfortunately I can't merge it because our CircleCI tests aren't allowed to run on third-party pull requests due to security concerns. I created a new PR (#528) that contains your commit, so when I merge it you'll still be listed as a contributor.
I'll close this PR and merge #528 in instead.
Thanks!

Thanks!


wip show threads

https://github.com/radio-t/radio-t-site/issues/94

@Reeywhaar

Wow, thank you so much!
At first glimpse I see problem: Comments on level 5 and below should not be indented (think of mobile), yet somehow show that they are replies 🤔

I think the line should be aligned to the left side of the first symbol. I know Reddit does it in the middle of the first one, but they have arrows here.

Comments on level 5 and below should not be indented (think of mobile), yet somehow show that they are replies 🤔

I think the current indication on a deep level is fine but will be nice to fix it to actually jump to the parent comment

this is an example as NYT shows threads

fixed alignment

@sw-double & @Reeywhaar - what is the status of PR? is it still WIP or good to be merged?

not sure that it handles dark theme in an idiomatic way
otherwise LGTM on my end

@sw-double cool, thx!
@Reeywhaar - need your blessing

I'm working on ui and stream api, plan to end in next 3 hours, then I'll watch this PR.
@umputun hope you don't rush to make it before the show :-)
@sw-double, thank you so much 👍

sure, take your time. thx

In general I see no crimes here, concise work, @sw-double, thank you, once again! So, @umputun, if you like it then great, I can fix some minor issues myself.

I think we should make something with mobile layout

Generally, I like it, thx @sw-double . Two issues to address:

All internals "levels" don't need any visible thread lines. On such a deep level it is already impossible to understand the tree and adding zebra doesn't make it any better
I agree with @Reeywhaar - those lines are too visible. Your dotted variant looks better to me

I'm afraid without the lines it becomes even more unclear

... I can make level5+ lines even more muted

Is there a reason why 5+ level indentations are any different? Or why we even have any for such deep levels?

Not sure I follow. 5+ are indented less mainly for mobile and for big discussions. In old version, as you may remember, levels 5+ were unindented at all. Dunno what helps there actually, I think comments this deep are mess no matter what. What do you suggest?

One another solution is to set min-width on thread, and if some discussiton breaks this limit then we get horizontal scroll

My suggestion is to revert 5+ to unindented

ok, agree


const expression bug

const SCREEN_WIDTH = 360
const LIST_ITEM_WIDTH = 300
const INIT_OFFSET = (SCREEN_WIDTH - LIST_ITEM_WIDTH) / 2 # expected constant expression

Would it be possible to extend the evaluation for expression like this?
const unit = Vector3(1, 1, 1).normalized()


Skip “Percy screenshots” when secrets are unavailable on forks (fix)

Looks like our "local branches only" check for Percy screenshots didn't work
I'd forgotten that workflows for fork PRs identify our repo making this check always pass: 🙈
❌  Use the github.repository_owner context
if: ${{ github.repository_owner == 'alphagov' }}

✅ Use the github.event context to identify forks
if: ${{ !github.event.pull_request.head.repo.fork }}

For example, this PR (fork) has passed the incorrect check and tried to run Percy

https://github.com/alphagov/govuk-frontend/pull/3074

But others (my fork) use the new check and Percy has been skipped:

https://github.com/alphagov/govuk-frontend/pull/3129
https://github.com/alphagov/govuk-frontend/pull/3131

Unlike the old fix, Percy is not skipped when run directly from the fork:

https://github.com/colinrotherham/govuk-frontend/actions/runs/3749530406
https://github.com/colinrotherham/govuk-frontend/actions/runs/3749529213

This allows contributors to add their own ${{ secrets.PERCY_TOKEN }} but with a warning if not provided:

Ready for review again @domoscargin


Manual filtering without having to write the whole query

Is your feature request related to a problem? Please describe.
It's related to a bug report i already submitted, but do not know how to link it.
Describe the solution you'd like
Add option to manually filter in tables.
For example instead of using the predefined filter you should also be able to write something like " table_name like 'some%text' " in an input text field.

Please can you explain to me in more detail what you mean?

When you click on the magnifying glass you get filter on column names (which is awesome), but instead of clicking on several filters when i need to do complex filtering i'd preffer to write the filter manually in a textfield.
Pretty much like when you write raw query, but without the boilerplate of it.
Here is example:

Real query:  " select * from table_name where condition_1 && condition2 || condition 3 && condition4"
what i expect to use instead the exisitng filtering or the long raw query is  just "condition_1 && condition2 || condition 3 && condition4"" in a simple text input row.

For reference other tools like DBeaver have this feature.
I cannot provide screenshot at this time.


Switching between tabs reset zoom

It is quite annoying that when you switch of tabs we reset the from / to and I was thinking about the more intuitive behaviour. What is sure is that resetting is not intuitive.
For now I would keep the same from / to as when we left the tab so that going back to the tab we see the same zoom level.

@lpatiny
Actually, we change the domain every time we switch between tabs (Nucleus) because we have different spectra, What you request to save the zoon level when switching between tabs?

@lpatiny
Actually, we change the domain every time we switch between tabs (Nucleus) because we have different spectra, What you request to save the zoon level when switching between tabs?


ARTEMIS-4436 Artemis is logging warnings during clean shutdown of server in cluster.

When we know that a node leaves a clustercleanly we shouldn't log WARN messages about it.
Issue: https://issues.apache.org/jira/browse/ARTEMIS-4436

This looks like it aims to suppress a [warning] notice that the other broker has gone away, is that correct? Its not clear to me that entirely makes sense. I could certainly see reducing the notice to Info level, but it still seems like one should be given? I don't necessarily care that the other broker was shut down cleanly, often just that it isn't there any more and was expected to be.
(I've also seen several times where people 'cleanly' shut down things without realising they are doing it, then come asking why it has 'crashed' or 'shut down by itself')

@gemmellr Yes that's correct. Moving the message to info level is another solution. Currently people removing a node from a cluster and seeing warning is also a problem. They are doing something completely fine so WARN is a bit too high

@gemmellr do you have any concerns on this PR? If not I think it should be merged.

Yes, per my comment, I dont think it makes sense to remove notice that clients/brokers-acting-as-clients lost connection just because the other side was shut down 'cleanly'. This side still expected to be connected and no longer is, thats detail they'll often want/need to know, regardless of whether the other side went away intentionally (which, per my comment, I've seen people do unwittingly on several occasions).
I agree a warning level message is excessive, but suppressing notice entirely seems the wrong fix to that...I'd still report it happened, but at Info level.

@gemmellr @clebertsuconic  @jbertram could you please take a look now that I've updated the PR

My last feedback was that I didnt know enough to say whether it was ok or not, and to ask for others to finish reviewing, so at this point I'll defer to those that do.


python311Packages.opower: 0.0.32 -> 0.0.33

Diff: https://github.com/tronikos/opower/compare/refs/tags/v0.0.32...v0.0.33
Changelog: https://github.com/tronikos/opower/releases/tag/v0.0.33
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 252518 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.opower
python310Packages.opower
python310Packages.opower.dist
python311Packages.opower
python311Packages.opower.dist


Byfn update

Added support for deploying any contract via byfn.sh, plus helper scripts.

opened in error


Switch shebang to /usr/bin/env python3

PR for https://github.com/The-Compiler/nasebohr/issues/12

Thanks and congrats on the first PR in this project! :tada: :sparkles:


Merge: https://github.com/rancher/rancher/issues/4323

Merge fix for https://github.com/rancher/rancher/issues/4323.

Tracked in #4722


RuntimeError: cuda runtime error (30)

An error occurred:
RuntimeError: cuda runtime error (30) : unknown error at /pytorch/aten/src/THC/THCTensorRandom.cu:25
My env is ubuntu 18.04, cuda 8.0, torch-0.4.0, python 3.6. Which one is wrong？Or what's the reason?
In the document "readme" shows that:
PyTorch is included in the requirements.txt file, and will currently download PyTorch version 3.1 for CUDA 8.0. This is the setup we've used for testing Distiller.
But the requirements.txt shows that the torch version is 0.4.0.
What are the final versions? Which cuda and which torch and the others.

The PyTorch version currently required is 0.4.0. I'll update the README to reflect that (thanks for catching it).
Other than that, as stated in the readme we've tested Distiller on Ubuntu 16.04 + CUDA 8.0 + Python 3.5, and that's the only setup we can provide support for at the moment. So we won't be able to re-produce your error on our end.
The error you're getting is a generic CUDA error, so it's hard to say exactly what's happening here. Are you able to run PyTorch with GPU successfully without Distiller (for example, their ImageNet sample here)? If that works, then please provide more details on the error you're getting with Distiller - where exactly does it occur - and we'll try to help.

Thank you. I have solved this problem. It need root permission to read the GPU device number. "sudo" can solve it.


Cache graph parsing when using from Javascript

Hello,
I'm wondering if there's a way to cache the graph loading and parsing when using from Javascript.
So I would be able to load the graph once, and perform multiple queries on it
https://github.com/logseq/nbb-logseq/tree/main/examples/from-js
const results = await loadString(`
(ns query
(:require [logseq.graph-parser.cli :as gp-cli]
[logseq.db.rules :as rules]
[datascript.core :as d]
[clojure.edn :as edn]))
(defn- main [graph-dir query*]
(let [{:keys [conn]} (gp-cli/parse-graph graph-dir {:verbose false})
query (into query* [:in '$ '%]) ;; assumes no :in are in queries
results (map first (apply d/q query @conn [(vals rules/query-dsl-rules)]))]
(clj->js results)))
(main "${args[0]}" '${args[1]})
`);

yeah, there's probably a way

Hi. The @conn in the script is your graph's database. Writing multiple queries in the .mjs is one way you could do this.
If you'd like to cache the database between script calls it's possible but will require you to use babashka and only clojurescript. Larger cached graphs like logseq's docs repo still take 1-2s to load. See https://github.com/logseq/bb-tasks#logseqbb-tasksnbbcached-db for more
Fwiw db graphs will not have this parsing step so this won't be an issue for them

hey @logseq-cldwalker, I really appreciate the feedback. Doing multiple queries is definitely a good option, unfortunately in my case they depend on each other. From what you're saying, it sounds like it's not straightforward to cache the parsing across calls via Javascript.
I considered doing the caching via nrepl, but seemed a bit too complex.
https://github.com/babashka/nbb/blob/main/README.md#repl-api
If the new db doesn't require the parsing, seems like that would solve my problem, so I'll hold off for now and re-evaluate when the Logseq db version is further along.
Thanks!

Fwiw, most of the complexity around the caching link I sent is around incremental caching.  But waiting for db version is also fine

Ah, I might have misunderstood what you said earlier. I just got it working based on your suggestions in cache read and cache write. Just reading and writing the parsed db to a file via cljs, and calling that via loadFile or loadString in javascript
I just need fast queries for exporting, not writes, and querying from the cached file is quite fast.
Here's my quick solution, which is a mix of the from-js example and the bb-tasks cache loading, in case anyone else is trying to use this from Typescript.
https://gist.github.com/briansunter/b7843ada6b8460ba61dd32644d5ecbbe


[PE-D][Tester A] Remark Command Does Not Display Remark in UI

When using the remark command to add a remark to a person's profile, the UI does not display the added remark. Although the command successfully adds the remark to the person’s data, it does not appear in the UI immediately, leading users to believe that the remark was not saved.

Steps to Reproduce:

Open the application and select a person to whom you want to add a remark.
Use the remark command to add a remark (e.g., remark 1 r/Financial Issue).
Save and view the person’s details in the UI.

Expected Result:
The remark should be displayed in the UI under the person’s details once the command is executed, allowing users to verify that the remark has been added successfully.
Actual Result:
The UI does not display the remark, even though it has been added to the person’s profile data.
Suggested Improvement:
Update the UI to include the full view of a user upon adding a remark to allow users to verify that a remark has been added

Labels: severity.Low type.FunctionalityBug
original: mattisongjj/ped#7

Duplicate of #291


Compilation final error

This target is used to compress a snapshot zip, you can still get compiled exe from folder bin\Release even this target is failed.


Rename cluster default env name when empty

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#777  👈
#762
#761
#760
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @carolineechen and the rest of your teammates on  Graphite


[WIP][5.10] Significantly improve performance of SyntaxVisitor

There are two core ideas here:

Before this change, we were allocating Syntax.Info for every visited node. We need a heap-allocated object here because Syntax can access its parent. In most cases, however, the syntax node was not stored by the visitor and we would just destroy the Syntax.Info after finishing the visit call. Instead, check if Syntax.Info is uniquely referenced, which means that the the node was not stored. In that case, don’t deallocate the memory but place it into a reuse pool. I have seen reductions in malloc calls from ~50,000 to ~100 (500x) in testEmptyVisitorPerformance.
Now retain and release calls made up a significant portion of the visitor’s performance. Add some annotations to eliminate reference counting. I added comments for the remaining retain calls.

I measured that this improves the performance of the EmptyParametersRule in SwiftLint by ~35% compared to 509.0.0 (and by ~50% compared to 510.0.0).
Fixes #2091
rdar://114330163

@swift-ci Please test

Assessed that the risk of taking this in a patch release of 510 is too big.


Incorrect searching URL

Hi, great piece of work you got, I found a problem while trying to search in the retrieved links as their format is incorrect:

Thanks, just fixed this issue. Google's formatting of search results must vary between machines, as it worked perfectly on mine.


Make metadata accessible

Hi, I was wondering if you could make metadata accessible as well -- I.e., what you get from s3_client.head_object(Key=key, Bucket=bucket)['Metadata'].
Perhaps stored at some_s3_f_obj.metadata -- though, it might need to be given before writing, either at object open time, or assigned before the first write (depending on how you handle writes).
Thanks for the great tool!

I think that would involve duplicating functionality that is already in boto3, as you've demonstrated in your example.
What would be the benefit of doing this over accessing the metadata via boto3 directly?

Upload is the only critical one, but implementing both would have this
primary and critical benefit:

Allow setting of metadata when using smart_upload which, with some
weird exceptions, is currently unsettable with smart_open

Weird exception: Uploading the wrong name, then using the AWS API to
copy the file to the right name, with the right metadata
..and these tertiary benefits:

Be a full abstraction of an S3 object instead of a partial one
Consistency in API -- what is readable is also writeable, where the
underlying medium supports it
..and, if S3/boto3 supports it (which it may not):
get the metadata at download time, without a separate head call

On Wed, Nov 14, 2018 at 7:31 PM Michael Penkov<EMAIL_ADDRESS>wrote:

I think that would involve duplicating functionality that is already in
boto3, as you've demonstrated in your example.
What would be the benefit of doing this over accessing the metadata via
boto3 directly?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/RaRe-Technologies/smart_open/issues/249#issuecomment-438870395,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACXHz7VpQAt1U3Kigw-hnmm4Zp8CAQ42ks5uvLXxgaJpZM4Ye1cD
.

Upload is the only critical one, but implementing both would have this
primary and critical benefit:

Allow setting of metadata when using smart_upload which, with some
weird exceptions, is currently unsettable with smart_open

Weird exception: Uploading the wrong name, then using the AWS API to
copy the file to the right name, with the right metadata
..and these tertiary benefits:

Be a full abstraction of an S3 object instead of a partial one
Consistency in API -- what is readable is also writeable, where the
underlying medium supports it
..and, if S3/boto3 supports it (which it may not):
get the metadata at download time, without a separate head call

We don't want to do this, by design.


Wrong state.json is being used

This was initially reported by @biilmann.
He created a completely fresh project with just a public folder, a functions folder and a netlify.toml file and then did a netlify deploy --prod and instead of creating a new site or prompting for an existing one, it deployed to the last project he worked on and overwrote the product deploy there.
It seems the CLI is writing the site state.json to the .netlify folder in my home directory instead of in the site specific directory.
He is not sure how he got into that state because it is so hard to debug. For some reason, setting up a project ended up writing to his home directory instead of the project dir, and then any new project travels up and finds that state file.
Once he deleted that file things go back to working as expected.

It seems like the only place retrieving state.json is here:
https://github.com/netlify/cli/blob/f88b460882a43b63d4997f930722519e9fbdd8bb/src/utils/state-config.js#L17-L25
So, it looks like the situation above would happen for any user who has a ~/.netlify/state.json file on their machine, but no ./.netlify/state.json (e.g. when creating a new Site directory). The question would then become: how did @biilmann end up having a ~/.netlify/state.json file? Maybe he ran some netlify CLI commands with the home directory being the current directory: based on the logic above, the current directory is the default value, so maybe that command created it?
@erezrokah What are your thoughts on this issue?

Thanks @ehmicky for looking into it.
I think it would make sense to ignore the home directory when looking for state.json.
I don't think it is a valid use case to have a site in the home directory, WDYT?

Yes, I agree.
When a user is running a netlify CLI command and cwd is the home directory, what should happen?
The current code is assuming that there is always a valid .netlify/state.json that can be read/written for any netlify CLI command invocation.
Several possible solutions:

Throwing when the current directory is the home directory (and maybe also the root directory).
Throwing when .netlify/state.json is about to be written to, and the current directory is the home directory. This seems to happen only with the npm init and npm link commands.
Using a temporary (in-memory), empty state.json for the duration of the command

What do you think?

We can divide the logic:

What happens when we find a state file in the home directory? Possible solution - warn the user but allow to continue via a prompt. On CI we can throw.
What happens when the user tries to initialise a project in the home directory? Possible solution - warn but allow via prompt. On CI we can throw.

Once I created a new site via a globally installed netlify-cli. ~/.netlify/state.json was generated silently; I was not aware of that.
Later, I found that one of my tests was failing locally. This time, netlify-cli was installed via the project's package.json.
Comparing --debug output from local vs CI showed me this:

baseRelDir was different. This led to the edge_functions path being incorrect.
Somehow, I was able to guess that I needed to remove ~/.netlify/state.json. It was very confusing 😅


Reminder to 'create Object' in json api documentation?

Hello
I'm wondering if telling the user to don't forget to generate content of factories ( like server.createList('user', 2);) in the test when speaking about working with json api is a good idea ? Or completely useless.
Thanks in advance.

Good suggestion! But I think I'll punt since I'm focused on 0.2.x docs, and when that lands this section won't really be relevant anymore.
Thank you :)

Ok Sam, thanks.


#define description

Feature description
What do you want to see added to the extension?
Maybe you just didn't add it, but I think it's a mistake that the extension thinks #define can only do one line. The \ character has always been used for hyphenation, but for some reason the extension breaks the demonstration on the first line.

I would also like to see the function code in #define and description.

How would it help you?
This will help me better navigate my own and other people's code.
How often would you use this feature?
Always when I forget or don't know what this or that #define is responsible for :)

Fixed in #221.


Can't import Bestiary 3 - Aghash (Div)

Can't import statblock:
Aghash (Div)      CR 4
XP 1,200
Male aghash (Pathfinder RPG Bestiary 3 83)
NE Medium outsider (div, evil, extraplanar)
Init +2; Senses darkvision 60 ft., see in darkness; Perception +11
Defense
AC 18, touch 12, flat-footed 16 (+2 Dex, +6 natural)
hp 42 (5d10+15)
Fort +4, Ref +6, Will +7
DR 5/good or cold iron; Immune fire, poison; Resist acid 10, electricity 10; SR 15
Offense
Speed 30 ft.
Melee 2 claws +8 (1d6+2)
Special Attacks cursed gaze, sandstorm
Spell-Like Abilities (CL 6th; concentration +10)
At will—bestow curse (DC 18), detect good, detect magic, dimension door, minor image (DC 16), spectral hand
1/day—suggestion (DC 17), summon (level 3, 1d2 dorus 25%)
Statistics
Str 14, Dex 15, Con 16, Int 13, Wis 13, Cha 18
Base Atk +5; CMB +7; CMD 19
Feats Alertness, Iron Will, Weapon Focus (claw)
Skills Bluff +10, Disguise +12, Intimidate +10, Knowledge (arcana) +7, Knowledge (planes) +7, Perception +11, Sense Motive +9, Spellcraft +9, Stealth +10
Languages Abyssal, Celestial, Infernal; telepathy 100 ft.
Special Abilities
Cursed Gaze (DC 16) (Su) Gaze in 30 ft, choice of confuse/shaken/stun 1 rd, or 1d4 dam (Fort neg).
Damage Reduction (5/cold iron or good) You have Damage Reduction against all except Good or Cold Iron attacks.
Darkvision (60 feet) You can see in the dark (black and white only).
Energy Resistance, Acid (10) You have the specified Energy Resistance against Acid attacks.
Energy Resistance, Electricity (10) You have the specified Energy Resistance against Electricity attacks.

Immunity to Poison You are immune to poison.
Sandstorm (1/day) (Su) Can create 100 ft radius sandstorm centered on self for 1 min/HD.
See in Darkness See perfectly in darkness of any kind, including magical darkness.
Spell Resistance (15) You have Spell Resistance.
Summon (level 3, 1d2 dorus 25%, 1/day) (Sp) Summon creatures as though with summon monster spell but with limited chance of success.
Telepathy (100 feet) (Su) Communicate telepathically if the target has a language.

I was wrong.  It imports with warning messages.

Actually, this is a good representative of NPC statblocks that produce parsing errors.  I've attached the messages.
![warning messages with parsing errors.  I'm seeing a lot of them for other NPCs, also.

Used your latest module that was available as of this afternoon at about 2:30pm.  I used Hero Lab to produce this statblock. For greater accuracy, you might want to use one of the online resources. Hero Lab can truncate some of the text. All the other statblocks from Bestiary 3 imported -- a total of more than 86% imported!

I've included these Notifications in the spell parsing section, because not all Spells are included in the compendium yet. But here it seems the Summons Feature is acting up.


Stack v2

Hi Tim, today I propose a 're-envisioning' of Stack.
My intention with this PR is two fold.  First, to preserve your original work on Stack. Secondly, to kindly request the keys to the "stack" namespace on npm.
Please have a quick skim through the new stack to get an idea of what I'm trying to create.  And if you're OK with this new direction you may accept this PR and enable me (drschwabe on Github and npm) with write access to the repo on npm and Github.
Otherwise, if you have any questions or concerns please share them!
Oh, I've taken the time to merge our histories so that your commits will never be lost, and have noted on the README specifically how users can access your original stack (via branch or npm install).
p.s. - aside from this module on it's own merit - a key reason I am hopeful to leverage the "stack" namespace is the series of other related modules I have cooking up; stack-element, stack-element-ejs, stack-page, stack-socket all underway.  Conceivably if this catches on there can be other contributors who leverage the stack- naming pattern too.  But I won't get too far ahead... world domination doesn't happen overnight :)

It's a neat framework, but it's very different from the minimal style of stack.  In fact, the whole point of stack was to be a minimal version of connect.  There are 24 dependents on this library and it's been downloaded over 1000 times in the last month. (Not huge, but significant)  Looking at the dependents and the authors of them, I recognize many authors who I know to love the minimal design and are minimalists themselves.
I feel it would unfair to them to give away the namespace for another project that is vastly larger in scope and doesn't preserve the spirit of minimalism.
You do know about npm's namespace support now right? https://docs.npmjs.com/getting-started/scoped-packages You can publish your own stack to npm under your username.  Another option is to think of another name for your experiment.
If nobody used this, I wouldn't mind giving it away, but I don't want o break their trust in me as a provider of minimal libraries.
Best of luck on your adventures.

Hi Tim, thanks for your consideration.    Stack is your namespace and I respect your intentions to keep the integrity of your work and the original scope of the project.     I will proceed to use npm's namespace feature as you have suggested.
Perhaps if it is OK I will ping you again in the near future if there is traction (and after there are more docs, examples, and tests fleshed out).  If @drschwabe/stack can prove itself then it may be worthy of another consideration.   If you are open to this, I would be highly appreciative as it would be even more inspiration to make something great.
Oh one more thing,  I do share your intent to create minimalistic libraries - my plan for the main repo is to simply host the very core of it (an express-like router for control flow / state management); keeping it as fast and as lightweight as possible - and accommodate for other modules in it's theoretical ecosystem (ex: stack-socket, stack-element, etc) to require it and implement/extend it's API to create new commands (routes) to take advantage of this convenient development pattern.
Anyway, I'll keep working on it.   Thanks again for your response and open dialog on this.  Peace!

Good luck.  Feel free to ping me again when you've got it more fleshed out.


feat(SDK): add Ximmerse SDK as a new SDK option

This integration is compatible with XimSDKv2.0.1
(https://github.com/Ximmerse/SDK/tree/master/Unity)
Setup up process is very similar to any other SDK:

Import XimSDKv2.0.1
Add Scripting Define Symbols: VRTK_SDK_XIMMERSEVR
Drag VRCameraRig to the scene
Use quick switch to set SDK to XimmerseVR
Actual Boundaries = VRCameraRig(scene object)
Actual Headset = CenterEyeAnchor(scene object)
Actual Left Controller = LeftHandAnchor(scene object)
Actual Right Controller = RightHandAnchor(scene object)
ModelAliasLeftController = _VisibleObject(scene object)
ModelAliasRightController = _VisibleObject(scene object)

There were the following issues with your Pull Request

Commit: ba161d183a8df86ddfba1ff526b8f644588f17a1
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

oh dear, it looks like the commit has gone haywire :(
were you trying to squash the commits and ended up doing a merge?
You may be better taking your latest code and raising a new pull request, unless you want to delve in to your git reflog and resolve the merge commits

No problem! I will do it tomorrow :) thanks for reviewing it.
I will make a new pull request. My changes are relatively simple.


Add BAT tests for port forwarding

Now that we support port forwarding, we should add functional tests for it.

@runcom @mrunalp Please feel free to assign that one to me.

@sameo Thanks!

@sameo did you complete this?

@sameo @runcom @mrunalp Do we have tests for this now?

Since this was never answered and a year old, I am going to close.


Property Aliases documentation

Describe the Enhancement:
Currently, property aliases are not documented for chef/chef. (See this example of aliases.) Ideally, this gap in documentation should be fixed.
Describe the Need:
Property aliases may be something that users use, yet users are not able to cross-reference within the docs.
Current Alternative:
Currently, property aliases can be found in the code, but not in the docs. It's just something that magically works for folks.
Can We Help You Implement This?:
Please determine if documenting property aliases is something to invest time in. Another factor in this decision is how to automate the documentation of property aliases if the answer is yes.

That isn't a "property alias", that is just a ruby alias of the method.  Maybe the aliases could be added to the property itself, but that particular ruby statement can't really be extended itself in any way.

Yeah the point was that we're just using the method aliases and those aren't known by chef in any way. We probably need a "chef way" of doing this in the resource class that allows us to bubble up aliases in the docs.

Closing this out since we don't really have any way to fix it. In general if we have an alias we should create a cookstyle rule to get folks off the old name. I've done that for yum_repository that had some historic names.


[Enhancement] pub.dev scores

Now that package:media_kit is available on pub.dev, I'm noticing that we're losing few scores.
From what I've seen so far, primarily due to:

Missing example(s) in various packages.
Having too short description in pubspec.yaml of various packages.

A solid 140/140 on all packages will be good to have. Current scores:

[x] https://pub.dev/packages/media_kit
[x] https://pub.dev/packages/media_kit_libs_ios_video
[x] https://pub.dev/packages/media_kit_libs_linux
[x] https://pub.dev/packages/media_kit_libs_macos_video
[x] https://pub.dev/packages/media_kit_libs_windows_audio
[x] https://pub.dev/packages/media_kit_libs_windows_video
[x] https://pub.dev/packages/media_kit_native_event_loop
[x] https://pub.dev/packages/media_kit_video

The 'Package layout convention' page suggests that we can have examples at following places. It is important to consider because few of the packages don't have any Dart code in them (at all) & only native implementation or build steps.

example/example[.md]
example[/lib]/main.dart
example[/lib]/package_name.dart
example[/lib]/package_name_example.dart
example[/lib]/example.dart
example/README[.md]

And perhaps some screenshots too.

All packages are 140/140 now.


How can i get the edit_password_url url from console ?

This is what i have into a mailer file
  def reset_password_instructions(record, token, opts = {})
    @token = token
    @resource = record

    if @resource.email.present?
      roadie_mail(to: @resource.email, subject: "Reset password instructions") do |format|
        format.html { render layout: 'mailer' }
      end
    else
      **url = edit_password_url(@resource, reset_password_token: @token)**
      su = ShortUrl.create({url: url})
      reset_url = short_urls_url(token: su.token)
<EMAIL_ADDRESS>"Reset your password from this link #{reset_url}")
    end
  end

And this is the error
undefined method `edit_password_url' for #NotificationMailer:0x000000000289d8
Thank you

Hello, if you need to access URL helper from outside of controllers and views, you can use: Rails.application.routes.url_helpers.edit_password_url(...)

Yes, accessing via the app routes url helpers is how you'd do it outside of a controller or view. Thanks @jgigault-pa!


Bump quarkus-resteasy-problem extension to v1.0.0

No functional changes between 0.9.4 and 1.0.0 that would be relevant for generator-jhipster-quarkus
We've started preparation for Quarkus 2.0 in our extension project - quarkus-restreasy-problem:1.X.Y is supposed to work with Quarkus 1.X, and - once Quarkus 2.0 is available - we will also bump our extension version to 2.0

@lwitkowski You should add a reference to your library in Zalando problem project like it was done with Micronaut ;)
https://github.com/zalando/problem#credits-and-references

Very good idea @DanielFran, thanks! First version of our extension heavily depended on zalando's lib, at some point we introduced our own RFC7807 model, but we still support zalando problem classes if present in classpath.


Changes are made to prevent SQL injection

Before the changes, SQL injection could be performed with the following path:
/users?id=0 OR 1=1;--
http://localhost:3000/users?id=0%20OR%201=1;--

Nota: 5


Sprint 2: Develop an additional Windows persistence method with root

https://github.com/Nekrosis-ISS-Capstone/Nekrosis/blob/cd927c1f7ec30f6c23fd118acd417ff99743adb0/nekrosis/support/windows_utilities/regrun.py#L23-L36


Adding code from GSOC starter project to official repo

Can we add a control-plane label so that as I make more commits it is easier to distinguish what commits are control-plane related? Also, I wrote some tests for the code I had written, but I need some more guidance on what we are looking for in terms of tests.

Hi @haroonmoh, thank you for your PR.
A couple general things I'd like you to look into:

remove all project env specific configuration (allow for this to be deployed to any Firebase project we choose to and add instructions how to do that)
remove all cached / generated files that shouldn't be checked in (exceptions are files with hashes of dependencies)
use a templating engine / web framework that supports templates to not have HTML code in your .js files (have a look at Angular or react.js).

The last thing to do is to switch to a templating web framework. I will hopefully be able to get that done by the end of today

I have added react.js to my code as recommended by Robert

I have added some information to the readme. Let me know what you guys think about it and what I should remove

@hostirosti Sure, will do.

One more thing I just noticed, please check that all files have the appropriate license headers where applicable (files you wrote / created need the Apache 2.0 license header, files you copied / imported need the license headers they where published with)

LGTM, thank you!


tool for converting a wav voice to ogg

to reduce file size.
should be done on a per-voice basis. quality should be selectable.

maybe have a "project cleanup" dialog. including removing unused voices, wav -> ogg conversion, maybe stuff like unused/unneeded events (e.g., 2 volume events in a row with the same value)?


Basic client scaffold

Goal
The aim of this PR is to set up the basic organization around the Client abstraction. Specifically, the following requirements should be satisfied before merging:

Instantiate a client with a provided config
Authenticate via SSL
Successfully make a single API call

This should provide a platform on which to build out additional functionality. For example: resource discovery, multiple group-version support for clients, other auth methods (exec, auth-provider, etc.).

This is in a state where I think there's enough new stuff (wiring up Client/RESTClient to make an actual call against a local KinD cluster). I'll merge this and start thinking about the next steps I need to make.


"Automatically set modal's width for larger viewports" causes overflow X (v4)

.modal-dialog gets set to width: 600px at media-breakpoint-up(sm), causing it to be wider than the browser window between 544px and 599px. I overrode the var $modal-md with a value of map-get($grid-breakpoints, sm) !default; to resolve the issue.
The issue is on line 133 of _modal.scss:
// Scale up the modal
@include media-breakpoint-up(sm) {
  // Automatically set modal's width for larger viewports
  .modal-dialog {
    width: $modal-md;
    margin: 30px auto;
  }
...

Testcase: https://jsfiddle.net/x1qpe0mm/

Duplicate of #17581, I believe.

@cvrebert The testcase uses the latest master branch. #17581 is supposedly fixed. This issue isn't. Please reopen.

@RyanZim The width declaration that OP referenced no longer exists. It's a max-width now: https://github.com/twbs/bootstrap/blob/v4-dev/scss/_modal.scss#L119
Also, couldn't repro with your testcase (in OS X Chrome):

@cvrebert Perhaps it is a different issue, but your screenshot demonstrates it. The modal comes right up to the edge of the viewport. If you make the preview window smaller, it will have a nice margin. I doubt this is intended behavior.

@RyanZim Filed #19973.


solitary flower anatomical entry?

The inflorescence plant anatomical entry (PO:0009049) states it must have at least two flowers. What is a single flower reproductive shoot system called? I don't find any entry under:
plant structure
collective plant organ structure
shoot system
reproductive shoot system
thanks.

You are correct that an inflorescence (PO:0009049) is defined as having 2 or more flowers.
I am a bit confused why you had an issue finding it, it is a direct child of reproductive shoot system (PO:0025082).

You could also get to it by typing "flower" into the search box on any page.
https://browser.planteome.org/amigo/term/PO:0009046

I think I understand the misunderstanding - I'm referring to a single or solitary flower shoot system - flower, peduncle, bract(s), etc.
thanks.


Modulus Status error on staging

I get the following error when I run modulus status while logged into staging:
Welcome to Modulus
You are logged in as staging-ops
/usr/local/lib/node_modules/modulus/lib/commands/status.js:41
    body.components.forEach(function (component) {
                   ^

TypeError: Cannot read property 'forEach' of undefined
    at Request._callback (/usr/local/lib/node_modules/modulus/lib/commands/status.js:41:20)
    at Request.self.callback (/usr/local/lib/node_modules/modulus/node_modules/request/request.js:129:22)
    at emitTwo (events.js:106:13)
    at Request.emit (events.js:191:7)
    at Request.<anonymous> (/usr/local/lib/node_modules/modulus/node_modules/request/request.js:873:14)
    at emitOne (events.js:101:20)
    at Request.emit (events.js:188:7)
    at IncomingMessage.<anonymous> (/usr/local/lib/node_modules/modulus/node_modules/request/request.js:824:12)
    at emitNone (events.js:91:20)
    at IncomingMessage.emit (events.js:185:7)

onmodulus/modulus-cli#144


Inserting task number into commit message

prepare-commit-msg hook was added to custom .git-hooks folder. The script searches for task number in branch name and puts it to the end of the commit message.
Also added checking SHOULD_LINT_ON_PRECOMMIT value before linting files on pre-commit hook

!squash


Add Option to Limit File Size?

Hello Manu,
You may not know me, but I have been very grateful for this Lightroom Plugin that you built. I saved gigs of storages by re-export my library to HEIC, instead of JPG, while keeping the same quality. Thank you!
While I'm using this plugin, I found that I missed the "Limit File Size To" feature that Lightroom has built for JPG output. I wish your plugin could have it too!
You may or may not have the bandwidth for this feature... If you don't, I'm happy to help and to contribute to your plugin. A naive approach that I have to let the ExportHEICCommand.swift file call ctx.writeHEIF(10)Representation() multiple times, with different quality, to find the best approximation. If you are open to community contribution, do you feel good about it?
If you are happy about me contributing to your project, it will be great if you write down the instructions that someone can use to set up the environment. For example, clone your code, install certain SDK, compile, set this or that settings, check debug logs, etc. They will help a lot people like me who want to ramp up with this project.
Thank you in advance. Look forward to your reply!
YoungCat

Hey, apologies for the delay in response, I wasn't getting email notifications for this repo from GitHub. I'll take a look at your PR

I merged your CR, thanks again for your contribution. Happy that this is helping someone out there.


Abstracted API methods used by tft.js into util file tftApi.js

Works easily enough

Please remove hauned's id
 20    const entries = await getLeagueEntries(summoner.id);//UtcMpsT00sv70OmKiRsdZY9mJTBOF7tjhvczP9zIzoHw80w

Please remove hauned's id
 20    const entries = await getLeagueEntries(summoner.id);//UtcMpsT00sv70OmKiRsdZY9mJTBOF7tjhvczP9zIzoHw80w

You moved too much logic to the tftApiUtil file. Please see branch lolwr, file lolApiUrl and follow that standard.
Ty <3

You moved too much logic to the tftApiUtil file. Please see branch lolwr, file lolApiUrl and follow that standard.
Ty <3


Linter does not work with two default function arguments

I'm seeing weird behavior when passing two arguments which default to a fat arrow function. This is linting correctly for me when using vanilla eslint, so my guess is that something between standard and eslint is causing this to fail (though there is a chance that it is an eslint problem if some config is triggering it). Opening this to hopefully track it down further and see if anyone has an idea of what's wrong.
// Lints correctly:
var workingLinter = ({
  foo,
  bar = x => x
}) => window.alert('it worked.')

workingLinter()

// Lints incorrectly:
// Error: Parsing error: Unexpected token =>
var brokenLinter = ({
  foo,
  bar = x => x,
  baz = x => x
}) => window.alert('it broke.')

brokenLinter()

This works fine on the v6 branch, so it sounds like the eslint AST parser has gotten better and can handle this now.
standard v6 will be released very soon.


Virtually no options available? Broken by 2022.11

Checklist:

[X] I updated to the latest version available
[X] I cleared the cache of my browser

Release with the issue:
1.0.2
Last working release (if known):
Same
Browser and Operating System:
Microsoft Edge Version 107.0.1418.35 (Official build) (64-bit) // Windows 11
Description of problem:
Trying to add the card, and getting very few options available to configure even with 2 weather integrations installed.  See...

Additional information:
Happy to provide anything additional, though you may need to direct me where to find it.

HA 2022.11 breaks loading the picker elements and therefore are simply not there. I have asked the devs for advice/assistance but they so far have not been at all helpful.
As a workaround if you add a button and an entity card on the same view it will work properly.

This is marked closed. But it’s not fixed!

It was closed by the person with the original issue. I guess as they were ok with the workaround.
@chicknlil I am reopening as the problem isn't actually fixed.
I have a potential way forward thanks to another card author that has suggested a solution. I won't get a chance to try it until the weekend.

Hi
What do you mean with "As a workaround if you add a button and an entity card on the same view it will work properly." ?
Where and what add ?
Phil

On the same view where your weather card is add another button or entity card. It doesn't matter what it will contain.

Hi
I have installed your card in a view where there are already several cards.
I tried to add another one without success.
When I open PWC, I don’t have any mean to introduce existing weather input
Phil

A pre-release 1.0.3b0 has been published to try and fix everything that HA 2022.11 broke.
Please try it out and let me know how it goes.

Hi
Sorry but I still don’t have any possibility to enter a weather forecast entity with this prerelease Version
Phil

A pre-release 1.0.3b0 has been published to try and fix everything that HA 2022.11 broke. Please try it out and let me know how it goes.

Works perfect for me, thanks.

Hi Sorry but I still don’t have any possibility to enter a weather forecast entity with this prerelease Version Phil

Make you you have done a hard refresh of the card. If that doesn't work for you I will need more information, screenshots etc.

1.0.3 has been released that fixes this.


Request never returns - Broadcasting transaction to the blockchain...

Using keychain to perform any action other than logging in, the request never returns
Broadcasting transaction to the blockchain...
returns no error, returns no success, permanently waiting for a response
Browsers: Brave, Firefox

Which dApp? Code? Node?
I've just tested authentication, post, vote and memoDecode on PeakD, it s all working as intended.

peakd.com
code?
Node: default
I uninstalled and reinstalled on both Brave and Firefox.
Brave now works again.

Clicking back to account set up, everything is overlayed on the donation screen.
Screenshots attached.

Hmm weird . Please give me your os and keychain version as well, i can't
reproduce the issue so far
On Tue, Feb 23, 2021, 18:08 leoplaw<EMAIL_ADDRESS>wrote:

peakd.com
code?
Node: default
I uninstalled and reinstalled on both Brave and Firefox.
Brave now works again.

Clicking back to account set up, everything is overlayed on the donation
screen.
Screenshots attached.
[image: Screenshot_20210223_110442]
https://user-images.githubusercontent.com/520581/108828580-4e727e00-75c7-11eb-9daf-194ff54390da.jpg
[image: Screenshot_20210223_110548]
https://user-images.githubusercontent.com/520581/108828585-503c4180-75c7-11eb-8854-a2c4cf434c80.jpg
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/stoodkev/hive-keychain/issues/87#issuecomment-784060732,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHXAF3IXUEKKG4UTRX4FR2TTAN5APANCNFSM4YCEQBAQ
.

Update: I uninstalled and reinstalled the keychain on Firefox and it worked again, flawlessly until today.
I was logged into PeakD, commenting and voting. I opened a PeakD link in a new tab, only for it to say I needed to log in, which I did. Voting and commenting was broken again. So, logout, and now cannot log in again as the plugin window never returns.
Screenshot attached.

Operating System: openSUSE Tumbleweed 20200819
KDE Plasma Version: 5.19.4
KDE Frameworks Version: 5.73.0
Qt Version: 5.15.0
Kernel Version: 5.8.0-1-default
OS Type: 64-bit
Processors: 8 × Intel® Core™ i7-2670QM CPU @ 2.20GHz
Memory: 15.6 GiB of RAM
Graphics Processor: NVC4
Firefox: 85.0.2 (64-bit)
Hive Keychain: <IP_ADDRESS>


[PluggableDevice] Add DEVICE_DEFAULT registration for int32 Const

Add DEVICE_DEFAULT registration for int32 Const so that pluggable devices don't have to register it themselves.

Can we remove the registration for DEVICE_GPU?
CC @penpornk

Thanks for the review! I removed the GPU registration.

@sanjoy Do I need to do anything to satisfy the import/copybara check?

@rthadur Do you know what's going on with the import/copybara failure?

@sanjoy sorry I missed it


Vaadin 14 support with npm packaging

Migrate to npm packaging to be compatible with Vaadin > 14. Since Vaadin 14 Bower packaging is deprecated.

Is this planned any time soon?

@markusherr I planned to begin with it today. But first I will port the ApexCharts addon to get some experience with npm.
The app-layout is planned after that.

That sounds great. I really appreciate it.

I tried to use the migration too myself today and had these issues:
On windows 10 bower did not find GIT although it was installed and in the PATH. I  worked around that by starting the migration using the GIT Shell
I got this error from the migration tool:
"FATAL ERROR: Ineffective mark-compacts near heap limit Allocation failed - JavaScript heap out of memory"
Worked around it by setting the NODE_OPIONS environment variable before doing the migration in GIT Shell like so:
export NODE_OPTIONS=--max_old_space_size=4096"
See https://github.com/vaadin/flow/issues/6250
I posted that workaround in the mentioned bug.
After that I got another error:
Unhandled Rejection at: Promise  Promise {   <rejected> TypeError: url.replace is not a function       at Object.replaceHtmlExtensionIfFound (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\urls\util.ts:32:14)       at Object.generatePackageJson (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\package-manifest.ts:205:24)       at Object.<anonymous> (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\convert-package.ts:133:23)       at Generator.next (<anonymous>)       at fulfilled (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\lib\convert-package.js:17:58) }  reason:  TypeError: url.replace is not a function     at Object.replaceHtmlExtensionIfFound (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\urls\util.ts:32:14)     at Object.generatePackageJson (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\package-manifest.ts:205:24)     at Object.<anonymous> (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\src\convert-package.ts:133:23)     at Generator.next (<anonymous>)     at fulfilled (C:\work\GitRepositories\vaadin-app-layout\app-layout-addon\target\migration\node_modules\polymer-modulizer\lib\convert-package.js:17:58)
See https://github.com/vaadin/flow/issues/6208
So migration is far from beeing smooth. In fact it didn't work at all for me.
Hope this helps a little.

@markusherr this is not really an issue, the manual migration did not take that long an is mostly done.

Closed by af0e31c16f5b319fcce3bf158c9a6c736ae4fdea


Export Logs to Different Formats:

Is your feature request related to a problem? Please describe.
It is usually frustrating to manually copy logs and convert to different formats for analysis or sharing, which is time-consuming and prone to errors. Having an integrated export feature for logs in various formats like JSON, CSV, and plain text would greatly enhance usability and streamline the process for developers.
Describe the solution you'd like
I would like LogSuite to include an option to export logs directly from the application in various formats, such as JSON, CSV, and plain text. This feature should allow developers to select the desired format and easily save the logs to a file, making it more convenient to share and analyze logs without manual conversion.
Describe alternatives you've considered
Using third-party logging libraries that already support multiple export formats, but this would require integrating and managing additional dependencies, potentially complicating the project.
Additional context
I'm a GSSoC'24 contributor and I want to work on this. So can you please assign it to me @avinashkranjan

Go Ahead @sanskriti-lal


Consider renaming serverFetch to externalFetch

Describe the problem
serverFetch name is unclear. That the docs need to say in bold that it's external is a bit of a code smell. Even still, we've had people get confused (https://github.com/sveltejs/kit/issues/1823)
Describe the proposed solution
Rename to externalFetch. That it runs on the server is already implied by it being located in hooks
Alternatives considered
status quo
Importance
nice to have
Additional Information
No response

As, author of the original idea and PR I agree with you, this name will be better. Sorry for what I didn't find this name.


Why usleep(100000) is used in rclc_executor_spin_some() function?

Hardware description: ESP32
RTOS: FreeRTOS
Installation type: micro_ros_setup
Version: micro-ROS running on ROS2 Foxy

Hello @pablogs9  ,
I am using the below mentioned code of int32_subscriber (below link).
[](https://github.com/micro-ROS/freertos_apps/blob/humble/apps/int32_subscriber/app.c)
I am curious , why the delay of usleep(100000) is used inside the rclc_executor_spin_some() function?
Is there any specific reason?
As after removing the delay the performance is better.
Thanks in advance!!

Hello, @SiddharthZokarkar I do not see any usleep inside rclc_executor_spin_some. Could you send a link to the line?

Hello @pablogs9 ,
I think i phrased the question incorrectly . So sorry for that.
I am talking about the usleep(100000) used inside the while loop .
while(1){
rclc_executor_spin_some(&executor, RCL_MS_TO_NS(100));
usleep(100000);
}
Line 48 in the below link:
[]((https://github.com/micro-ROS/freertos_apps/blob/humble/apps/int32_subscriber/app.c)
I am curious , why the delay of usleep(100000) is used inside the while loop?
Is there any specific reason?
This code (int32_subscriber) came as an example when I gitclone the repository
git clone -b $ROS_DISTRO https://github.com/micro-ROS/micro_ros_setup.git src/micro_ros_setup

It depends on the board and toolchain that you are using, but normally the user application shall have a priority between the inner tasks (timers, ethernet...) and the idle one.

Ok Thank you so much !!


Issue with run polling(sleep)

Confirm this is an issue with the Python library and not an underlying OpenAI API

[X] This is an issue with the Python library

Describe the bug
Hi OpenAI team,
I'm reaching out to report a possible issue or improvement regarding the polling mechanism in asynchronous functions.
Problem Description
The current implementation of the poll method uses the time.sleep() function, which is blocking and halts the execution of other asynchronous tasks during polling. This behavior affects concurrency, preventing other coroutines from running efficiently.
To Reproduce

Code snippets
import asyncio

async def poll(
    self,
    run_id: str,
    thread_id: str,
    extra_headers: dict | None = None,
    extra_query: dict | None = None,
    extra_body: dict | None = None,
    timeout: float | 'httpx.Timeout' | None = 'NOT_GIVEN',
    poll_interval_ms: int | 'NotGiven' = 'NOT_GIVEN',
) -> 'Run':
    extra_headers = {"X-Stainless-Poll-Helper": "true", **(extra_headers or {})}

    if is_given(poll_interval_ms):
        extra_headers["X-Stainless-Custom-Poll-Interval"] = str(poll_interval_ms)

    terminal_states = {"requires_action", "cancelled", "completed", "failed", "expired"}
    while True:
        response = await self.with_raw_response.retrieve(
            thread_id=thread_id,
            run_id=run_id,
            extra_headers=extra_headers,
            extra_body=extra_body,
            extra_query=extra_query,
            timeout=timeout,
        )

        run = response.parse()
        if run.status in terminal_states:
            return run

        if not is_given(poll_interval_ms):
            from_header = response.headers.get("openai-poll-after-ms")
            if from_header is not None:
                poll_interval_ms = int(from_header)
            else:
                poll_interval_ms = 1000

        await asyncio.sleep(poll_interval_ms / 1000)

OS
macos
Python version
Python v3.11.4
Library version
openai v1.26.0

Hey just curiosity, do you have a alternative to avoid asyncio.sleep?

There are likely a few alternatives available, but they would involve significant changes to the implementation, such as utilizing call_later or queues. If the current implementation already includes a sleep function and there's no specific reason to use a blocking sleep here, it should be simple to switch to a non-blocking sleep.
However, I'm not sure if this was intentional or just an oversight in the implementation.

This will be fixed in the next release: https://github.com/openai/openai-python/pull/1414


Custom branch support for on-demand clusters

TODO: This currently duplicates the deployer pipeline, we need to DRY it out.
The AWSSSMAgent block is added to default values as it's something the release
pipeline would normally package up for us (as of #726), but in this case we
don't get that.

@samcrang Proposed changes for resolving drift between deployer.yaml and ondemand-deployer.yaml:

gsp vs. platform naming - can standardise on one in both pipelines
Removal of verification of helm templates, removal of unpack-gsp-release commands, change of helm template source - we can just put a bash condition in on the basis of a parameter from Concourse
Removal of unpack-gsp-release tasks - conditions in the task that make it check the parameter
Self-update path changes (deployer.yaml and deployer.defaults.yaml from root of repo to under pipelines/deployer/) - maybe the release should pack this at the same path? self-update would be able to unpack-release
Pipeline name - can come from cluster-config (hack/set-deployer-pipeline currently gets it from env on bootstrap), defaulting to deployer.yaml
Change of the platform resource from a github-release type to a git type - cunning use of parameters - if we can set all of the source parameters for both github-release and git resource types (and neither will be upset that we tried to specify parameters that they didn't expect, i.e. that would be used for the other resource type), we can have a parameter that just changes the type, and set the type to git for on-demand clusters


Metadata url

@annatisch @johanste This is exporting the URL used in Autorest outside of the operation method. This will allow Profiles matching to be done based on the Url (by looking at method.metadata['url']. This does not require msrest change and is 100% backward compatible (inspect will still give the same results, etc.)
This could be enhanced in the future with more metadata (return type, HTTP verb, etc.). At the end, the method itself could become an empty shell where every important information is inside metadata (not there yet :)).

🤖 AutoRest automatic publish job 🤖
> will publish once PR gets merged

🤖 AutoRest automatic feature coverage report 🤖
feature set version 2.3.25
❌ General: 95%
24 out of 535 features are not covered by tests
HttpRedirect300Get, additionalPropertiesInProperties, additionalPropertiesInPropertiesWithAPTypeString, additionalPropertiesTrue, additionalPropertiesTypeObject, additionalPropertiesTypeString, allowedValueEnum, animalNotFoundError, expectedEnum, expectedNoErrors, expectedPetHungryError, expectedPetSadError, getDateInvalid, getDictionaryNullkey, getIntegerOverflow, getIntegerUnderflow, getLongOverflow, getLongUnderflow, intError, linkNotFoundError, putComplexPolymorphismNoDiscriminator, roundTripEnum, stringError, unexpectedEnum
❌ Azure: 99%
1 out of 118 features are not covered by tests
LRONonRetryPut201Creating400InvalidJson

🤖 AutoRest automatic feature coverage report 🤖
feature set version 2.3.25
❌ General: 95%
24 out of 535 features are not covered by tests
HttpRedirect300Get, additionalPropertiesInProperties, additionalPropertiesInPropertiesWithAPTypeString, additionalPropertiesTrue, additionalPropertiesTypeObject, additionalPropertiesTypeString, allowedValueEnum, animalNotFoundError, expectedEnum, expectedNoErrors, expectedPetHungryError, expectedPetSadError, getDateInvalid, getDictionaryNullkey, getIntegerOverflow, getIntegerUnderflow, getLongOverflow, getLongUnderflow, intError, linkNotFoundError, putComplexPolymorphismNoDiscriminator, roundTripEnum, stringError, unexpectedEnum
❌ Azure: 99%
1 out of 118 features are not covered by tests
LRONonRetryPut201Creating400InvalidJson


i got an empty pdf

first i meet this problem: AttributeError: 'list' object has no attribute 'grad_fn', then i concatenate the output:
    from torchviz import make_dot
    x = torch.rand(8, 3, 112, 112).cuda()
    y = model(x)
    for i in range(len(y)):
        if i == 0: c = torch.cat((y[0], y[1]), 0)
        elif i >= 2 and i <= len(y)-2:
            c = torch.cat((c, y[i+1]), 0)
    g = make_dot(c)
    g.render('net_arch', view=False)

but at last it only saved an empty PDF for me, why ?
net_arch.pdf

finally i solved it by only using the model class, and do not load pretrained weights.
    from torchviz import make_dot
    from arcface import model
    net = model.Backbone(num_layers=50, drop_ratio=0.6)
    x = torch.rand(8, 3, 112, 112)
    y = net(x)
    for i in range(len(y)):
        if i == 0: c = torch.cat((y[0], y[1]), 1)
        elif i >= 2 and i <= len(y)-2:
            c = torch.cat((c, y[i+1]), 1)
    g = make_dot(c)
    g.render('net_arch', view=False)


crypto: make ecdsa verification faster

It was dog-ass bloaty before. It allocated so much it broke iOS.

Test failure is from deps test. Easily fixed, but will have to wait for tomorrow.


Client stream does not support selector

This is not a very helpful title. Please provide more info.

I see begin Stream ，have select node:
func (r *rpcClient) Stream(ctx context.Context, request Request, opts ...CallOption) (Stream, error) {
	// make a copy of call opts
	callOpts := r.opts.CallOptions
	for _, opt := range opts {
		opt(&callOpts)
	}

	next, err := r.next(request, callOpts)
	if err != nil {
		return nil, err
	}
        ...

but , how I can in stream handler function， send one msg to other stream obj

The stream is created by connecting to a node. To one node. We use the selector to find that node. See the next method for that call to the selector. The stream has Send/Recv methods to talk to that node.


fix(fiori-freestyle-writer) align formatter name with JS template

fix https://github.com/SAP/open-ux-tools/issues/880

Thanks all.


pam_mount: 2.15 -> 2.16

Semi-automatic update. These checks were performed:

built on NixOS
ran /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin/mount.crypt --help got 0 exit code
ran /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin/mount.crypt --help and found version 2.16
ran /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin/pmt-ehd --help got 0 exit code
ran /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin/pmt-ehd --help and found version 2.16
ran /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin/pmvarrun -h got 0 exit code
found 2.16 with grep in /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16
found 2.16 in filename of file in /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16

cc "@tstrobel"

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/sbin/pmvarrun
shrinking /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/lib/libcryptmount.so.0.0.0
shrinking /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/lib/security/pam_mount.so
gzipping man pages under /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/lib  /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/sbin
patching script interpreter paths in /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16
checking for references to /tmp/nix-build-pam_mount-2.16.drv-0 in /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16...
moving /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/sbin/* to /nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16/bin
/nix/store/ylk3vdmg5vk7c21fxg54yvrclvj5cpx1-pam_mount-2.16

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘pam_mount-2.16’ in /Users/graham/nix-borg/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/grahamc-ndnd/pkgs/os-specific/linux/pam_mount/default.nix:32 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/sbin/pmt-ehd
shrinking /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/sbin/pmvarrun
shrinking /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/sbin/mount.crypt
gzipping man pages under /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/share/man/
strip is /nix/store/viv0il4zpdqj4p7yamwbmn2k9pb8dcnv-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/lib  /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/sbin
patching script interpreter paths in /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16
checking for references to /build in /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16...
moving /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/sbin/* to /nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16/bin
/nix/store/iw083yjrxljcf17a3slnxrmrrz7r0yzr-pam_mount-2.16


Java Version Incompatibilities / Documentation

I'm on Pop!_OS 22.04 LTS (based on Ubuntu)
I've locally tried

:x: OpenJDK Runtime Environment (build 11.0.17+8-post-Ubuntu-1ubuntu222.04)
:x: OpenJDK Runtime Environment (build 17.0.5+8-Ubuntu-2ubuntu122.04)
:heavy_check_mark: OpenJDK Runtime Environment Temurin-17.0.5+8 (build 17.0.5+8)

The first two have different failure modes, (the last starts fine):

OpenJDK Runtime Environment (build 11.0.17+8-post-Ubuntu-1ubuntu222.04) causes

Exception in thread "main" java.lang.UnsupportedClassVersionError: dev/necauqua/scm/Agent has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0
	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
	at java.base/java.lang.ClassLoader.defineClass(ClassLoader.java:1017)
	at java.base/java.security.SecureClassLoader.defineClass(SecureClassLoader.java:174)
	at java.base/jdk.internal.loader.BuiltinClassLoader.defineClass(BuiltinClassLoader.java:800)
	at java.base/jdk.internal.loader.BuiltinClassLoader.findClassOnClassPathOrNull(BuiltinClassLoader.java:698)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClassOrNull(BuiltinClassLoader.java:621)
	at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(BuiltinClassLoader.java:579)
	at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(ClassLoaders.java:178)
	at java.base/java.lang.ClassLoader.loadClass(ClassLoader.java:522)
	at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndStartAgent(InstrumentationImpl.java:431)
	at java.instrument/sun.instrument.InstrumentationImpl.loadClassAndCallPremain(InstrumentationImpl.java:525)
*** java.lang.instrument ASSERTION FAILED ***: "result" with message agent load/premain call failed at ./src/java.instrument/share/native/libinstrument/JPLISAgent.c line: 422
FATAL ERROR in native method: processing of -javaagent failed, processJavaStart failed
Aborted (core dumped)

That doesn't seem fixable from an End-User perspective, unless you can compile it for an older version I assume. Not sure if it works even if you could, though.

OpenJDK Runtime Environment (build 17.0.5+8-Ubuntu-2ubuntu122.04) causes

Inconsistency detected by ld.so: dl-lookup.c: 106: check_match: Assertion `version->filename == NULL || ! _dl_name_match_p (version->filename, map)' failed!

This seems similar to the one posted in #1 - I found no solution to this online for this java version and according to https://github.com/necauqua/starsector-fixes/issues/1#issuecomment-1345484888 you also cannot say what could be the causes.

yeah I had to compile it myself aswell when I tried with JDK11 because the prebuilt binary is built with JDK15 (as far as I remember)
has nothing todo with starsector-fixes specifically, it's an issue of libgdx and lwjgl (the game engine starsector uses) - see this post on stackoverflow

Oh wow yeah I've just compiled it with some jvm without thinking about it too much, I should downgrade the code to java 8 and compile it with that

well I failed to close the second issue with a commit message, but yeah I've compiled it with jdk 11 (8 does not work because the shim depends on a java 11 library that we're also adding to starsector classpath) and made a new release with that


No image specified in ctr list

When listing containers with ctr there is no image shown:
ID        IMAGE     PID       STATUS
shell               0         STOPPED

#dibs


Fix broken disconnect.html and faq.html pages

Because these two pages were in generic_ui/polymer/, after Polymer vulcanization, they were no longer being copied into the dev folders.
Moving these non-Polymer files out of generic_ui/polymer/ and into generic_ui/ seemed appropriate.
ran grunt test
(Note that faq.html is still not working for Firefox and there is an issue for this.)

:+1:


Clone of this repo fails to build Node 12, node-gyp 3.2.1, gcc

➜  node-xpc-connection git:(master) npm install

><EMAIL_ADDRESS>install /Users/brianfeister/dev/temp/node-xpc-connection
> node-gyp rebuild

  CXX(target) Release/obj.target/binding/src/XpcConnection.o
  SOLINK_MODULE(target) Release/binding.node
ld: library not found for -lgcc_s.10.5
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [Release/binding.node] Error 1
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:269:23)
gyp ERR! stack     at ChildProcess.emit (events.js:110:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:1067:12)
gyp ERR! System Darwin 14.3.0
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/brianfeister/dev/temp/node-xpc-connection
gyp ERR! node -v v0.12.0
gyp ERR! node-gyp -v v2.0.2
gyp ERR! not ok

npm ERR! Darwin 14.3.0
npm ERR! argv "node" "/usr/local/bin/npm" "install"
npm ERR! node v0.12.0
npm ERR! npm  v2.14.0
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! This is most likely a problem with the xpc-connection package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls xpc-connection
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/brianfeister/dev/temp/node-xpc-connection/npm-debug.log
➜  node-xpc-connection git:(master) node -v
v0.12.0
➜  node-xpc-connection git:(master) node-gyp -v
v3.2.1
➜  node-xpc-connection git:(master) node -v
v4.2.3
➜  node-xpc-connection git:(master) gcc -v
Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Apple LLVM version 6.1.0 (clang-602.0.53) (based on LLVM 3.6.0svn)
Target: x86_64-apple-darwin14.3.0
Thread model: posix
➜  node-xpc-connection git:(master) which gcc
/usr/bin/gcc
➜  node-xpc-connection git:(master) echo $PATH
/Users/brianfeister/.nvm/versions/node/v4.2.3/bin:/usr/local/heroku/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/bin:/opt/X11/bin

These config details all seem fine according to other open issues.  Any ideas?

Also it fails on node v0.12 which I initially tried with

The fix for this ended up being fairly involved.  I had Yosemite installed (long after release of El Capitain) and Xcode 6.  I had installed Node via brew when Xcode 6 was active, but it was necessary to update to Xcode 7, then update my toolchain to use Xcode 7, then brew rm node and re-install brew install node.  Before installing Node via Homebrew, I needed an Xcode version 7 or better in clang -v output like this:
➜  clang -v
Apple LLVM version 7.0.2 (clang-700.1.81)
Target: x86_64-apple-darwin14.5.0
Thread model: posix

Here's a more thorough walkthrough of that part:
https://github.com/Homebrew/homebrew/issues/40653


Exported variable throws error while on the editor when trying to access its variable

Tested versions
Reproducible in 4.2.1 stable
System information
Windows 10 - Godot 4.2.1 stable - Forward+
Issue description
When you try to access a variable from an exported variable inside the script, the console throws an error while in the editor. When pressing play it does not matter, but everytime you go to the editor you will have the error there.
The error says "Invalid get index"
Steps to reproduce
Create script A, script B that extends script BB.
scripts B and BB are @tool
export a variable of type A in script B, then try to use a variable
Minimal reproduction project (MRP)
test.zip

It was a mistake from me, I was using the @tool keyword and I must ask for when i'm not in the editor


componentWillMount is not called for any child component

Add componentWillMount to any component

Result: it is never get called, it is called only for app.react.js
But componentDidMount called correctly.

Tested on notfound.react and home index. It works. There is nothing special in Este.


Pcolormesh shading

Description
The pcolormesh shading is different between Matplotlib and ProPlot figures. I'm wondering if anyone has come across this; I can't figure out what parameter is being changed. The issue doesn't show up for imshow.
Steps to reproduce
import numpy as np
import matplotlib.pyplot as plt
import proplot as pplt

X = np.random.multivariate_normal(mean=[0., 0.], cov=[[1.0, 0.5], [0.5, 1.0]], size=100000)
lim = (-3.5, 3.5)
bins = 50
Z, xedges, yedges = np.histogram2d(X[:, 0], X[:, 1], bins, (lim, lim))

fig, ax = plt.subplots(figsize=(3, 3))
ax.pcolormesh(Z.T, cmap='dusk_r', shading='flat')
plt.show()

fig, ax = pplt.subplots(figsize=(3, 3))
ax.pcolormesh(Z.T, cmap='dusk_r', shading='flat')
plt.show()

Matplotlib version: 3.4.3
Proplot version: 0.9.1

Hi, this is a feature -- not a bug. Please see the documentation section on distinct levels and DiscreteNorm. I find it is very often useful to break colormaps into a subset of color levels like contourf rather than giving the perception of "smooth" color transitions, because it helps pick out numeric values in a colorbar.
Distinct levels can be turned off for particular pcolor plots by passing discrete=False to the plotting commands, or turned off persistently by setting rc['cmap.discrete'] = False.

Ah, I missed that. Thanks for the help.


Update TSI max log file size

Would it be useful to mention that the size can be configured and link to the config option?

Probably yes, I'd say. I'll look into this— 👍

closes #2668


Track item effect

Hello. Is there a way to track an item effect like the Soul Preserver trinket ?

i want this to trigger the same effect as Omen of clarity.
I tried a few Weak aura but it dont feel the same as the addon.

You can add this effect, granted you are brave enough to edit the source code.
I will give you the steps to do so, but you have to know 3 things;

the effect may overlap with existing effects, so it may get confusing depending on other effects that already exist
I haven't tested the code because I don't have this trinket myself
this code shall be reset when you download an update of the addon, in which case you will need to re-do these steps

The goal is pretty simple: edit the addon source code (with a text editor like VS Code, or even Notepad), save the modified file, and reload the addon (e.g., reload UI, log out / log in, etc.)
You will need to edit the source code of the class you're playing with. If you have the trinket on multiple characters, simply edit as many files as needed.
The source code is in the addon folder, in WoW/Interface/AddOns/SpellActivationOverlay
All modifications happen in the registerClass function of each class.
Druid
You must edit classes\druid.lua and add this line inside registerClass:
self:RegisterAura("healing_trance", 0, 60512, "natures_grace", "Left + Right (Flipped)", 1, 255, 255, 255, true);

Paladin
You must edit classes\paladin.lua and add this line inside registerClass:
self:RegisterAura("healing_trance", 0, 60513, "natures_grace", "Left + Right (Flipped)", 1, 255, 255, 255, true);

Priest
You must edit classes\priest.lua and add this line inside registerClass:
self:RegisterAura("healing_trance", 0, 60514, "natures_grace", "Left + Right (Flipped)", 1, 255, 255, 255, true);

Shaman
You must edit classes\shaman.lua and add this line inside registerClass:
self:RegisterAura("healing_trance", 0, 60515, "natures_grace", "Left + Right (Flipped)", 1, 255, 255, 255, true);

PS. even though all the above lines look the same, there is one thing that differs for each class: the spell ID (druid is 60512, paladin is 60513, etc.)

Coming a bit late, but this has been implemented as a separate spell alert, similar to Mage's Clearcasting effect.

Hi, closing this issue as it was implemented in #149 and released in v0.9.5. Thanks for the feedback!


Adding project to GitHub repo

Function to take 2 numbers as parameters and add them . Then print out the result>

I have edited the file.


Make keyboard dark on dark themes (iOS only)

Summary
This will pass the keyboardAppearance prop into every TextInput that is themed. It's auto-detected from the current theme's centerChannelBg using the same approach used for the satus bar color. It makes the keyboard 'light' or 'dark' on iOS. React Native hasn't implemented this effect on Android yet as of 0.60. (ref) I'm still passing the props into Android-only components/files in case they start supporting it (lmk if not including is a better practice for mattermost)
Checklist

[x] Has UI changes

Device Information
This PR was tested on: [
iPhone SE iOS 12.2,
iPhone 5s iOS 12.2 Simulator,
Google Nexus 5X Genymotion Simulator API Version 24
]
Screenshots

Light Theme (iOS)
Dark Theme (iOS)

@cardoso Looks good.  One thing I observed, if I changed theme from other client (like browser), the UI gets updated except for the keyboard.  New theme is applied only when I closed then opened the keyboard - https://youtu.be/HkNJHTKPekY

Saturnino, that is controlled by the OS and I doubt it will ever be possible to work unless we “auto” close/open the keyboard when that happens which to me is even weirdest

@esethna can you please review


netskope_privateapps does not respect api rate limits

When attempting to create / update / delete more than a couple of private apps at any one time, the netskope_privateapps resource throws the below API rate limit exceeded error:

│ Error: API rate limit exceeded
│
│   with module.netskope_fqdn_privateapps["PLACEHOLDERDOMAIN"].netskope_privateapps.privateapp,
│   on privateapp-module\main.tf line 24, in resource "netskope_privateapps" "privateapp":
│   24: resource "netskope_privateapps" "privateapp" {
│

For the creation/deletion of private apps you can re-run the terraform apply until all apps are created, so though annoying, in this case the issue fails safe.
This is not true for private app updates.
If you're updating an existing app (e.g. updating port list), when the rate limit is hit the provider will update the state as if the change was successful, even when not.
This isn't appropriate terraform behaviour as the mismatch negates the whole purpose of using terraform.
Relevant resource reference from module example:
resource "netskope_privateapps" "privateapp" {
  app_name                = local.validated_app_name
  host                    = join(", ", var.hosts)
  use_publisher_dns       = var.publisher_dns
  clientless_access       = var.clientless_access
  trust_self_signed_certs = var.trust_self_signed_certs

  dynamic "protocols" {
    for_each = var.protocol_map
    content {
      type = protocols.key
      port = join(", ", protocols.value)
    }
  }

  dynamic "publisher" {
    for_each = var.publisher_list
    content {
      publisher_id   = publisher.value.id
      publisher_name = publisher.value.name
    }
  }
}

Environment information:
PS C:\git\netskope_config> terraform -v
Terraform v1.3.2
on windows_amd64
+ provider registry.terraform.io/netskopeoss/netskope v0.2.1

@Taoquitok Thanks for reporting this. I should be able to take a closer look in the next few days and I will let you know what to expect for a timeline on fixing once I dig into the issue.

@ns-sbrown Thanks for checking. Good to know the issues are monitored too.
Before I commit some development time to it, are you open to pull requests too?
Though I may not have time to investigate this specific issue, there's other minor bugs/inconsistent behaviours that I'm interested in improving, and at least one resource I'd like to add to manage the refreshing of publisher API tokens that would be beneficial to using this provider at scale.
(All assuming I'm not stepping on the toes of existing in-house developement plans?)

@Taoquitok In general I am am open to pull requests. If it's a small fix or something quick and easy please go ahead and submit a pull request and I will happily review. If it is a larger effort on your part, I'd reach out and coordinate just to make sure you aren't duplicating effort.
On this specific issue, I'd hold off as I think I know how I want to handle this. In short I am likely going to fix this in the nsgo client by implementing retry logic. This way other project which use the client will also benefit.  This will probably take me a few weeks between implementation and testing but I am close to having a plan ready to go. I do still need to investigate why you are seeing the issue are for what should be a failure in Apps so this may be a separate effort though may be masked when retry logic gets added.

@Taoquitok I released v0.2.2 this morning which uses new retry logic from the client. In my testing this has resolved issues with rate limiting. Would be good to get feedback before I close this issue.

@ns-sbrown
Thanks for the quick change. I've run a few applies to add/remove ports to 20 apps a few times and it's consistently made the change. No noticeable slowdown either, so the behaviour is looking good.
I noticed there's an additional change to include tags? I assume this is a new feature that'll come to private apps?
Is it something I could start including without risk, or best to wait?

Great to hear and thanks for testing. Tag functionality is partially implemented in the product right now so I slipped some support into the provider. That said this is not turned on in all tenants yet, so I'd probably hold off a bit.


Move state summary to a table caption

Relocate the overall state summary information to a <caption> instead of a <th>, which causes accessibility issues per #127.
Resolves #127
Before:

After:

Motivation
Changes

[X] Ensured that relevant issues are linked, if this PR resolves any outstanding.
[X] Added a screenshot for all UI changes (you can drag the file into this edit box and it will be uploaded).
[X] Ensured that changes to auto-generated files have not been committed; in particular, *.html, *.csv and *.json files.

Thanks for the PR! We've already resolved #127 with a slightly different change, and are trying to avoid too much churn on the relatively fragile headers (we've accidentally broken them a few times already), so I'm going to close this.


😇 Question about naming

I personally suggest common naming convention such as life init (or life i as shorthand),  instead of life --init or life -i. I think it's easier to type and allow more option parsing. Because as the program grows, you might need more options for a single command, such aslife i --student or life log --detail.
(Not an serious issue, just one of my questions 😆)

I like this especially considering it's how Git command line works. :+1:

I followed the example format in https://github.com/sindresorhus/meow previously.
But @chiunhau 's idea sounds great. I will take it into consideration!

meow looks cool. There's also commander if meow won't do what you want.

@sirlancelot OMG it's another tj repo!
It looks fantastic. Thanks.


Error [ERR_REQUIRE_ESM]: require() of ES Module >=7.3.0

When I update from 7.2.6 to >=7.3.0 I got this error Error [ERR_REQUIRE_ESM]: require() of ES Module
I use it in electronjs so node.
I found that you remove commonJS but all your node examples use commonJS
What is the solution here ?

@castroCrea  it seems your platform still needs CJS support, similar to the issue #359 and extractus/oembed-extractor#177
Please try the version v7.3.1 I've just released.

Thank it works perfectly 🦾
I'll add to my Roadmap to pass to ESM


Invite link doesn't work correctly

When I log into Jitsi Meet (I use the web wallet) invite link does not work correctly: https://wallet.superhero.com/sign-message?x-success=https%3A%2F%2Fmeet.aepps.com%2FGladDaysRemindOnce%3Fresult%3Dsuccess%26signature%3D{signature}&x-cancel=https%3A%2F%2Fmeet.aepps.com%2FGladDaysRemindOnce%3Faddress%3Dak_cEjh5on5RvoBw8zqCGS4VWeY2bmNYNPjn3SCZutYyVLWKf7X1&message=I would like to generate JWT token at Fri, 28 Aug 2020 12%3A30%3A56 GMT

related to #207

Seems that invite link works correctly, but options tip user via a button on avatar/text message are not available (Tip button is invisible):


Appears under home indicator on iPhone X.

Do you use the latest version? The latest version should fix the issue. Otherwise: Can you some more information? Like a screenshot and the code you use to present the RMActionController.

My mistake! We were pinned on a past update.


Passphrase screens have content covered by keyboard on iOS

On iOS, the keyboard covers up the action buttons on both the CreatePassphraseScreen and EnterPassphraseScreen.
These screens do seem to work properly on Android, so it might be an iOS-only issue with our React Native components?

@shamilovtim can you sort this bug out?

@shamilovtim can you sort this bug out?

Yeah I've got a great long term solution / abstraction for this


 Do I need to pass the instance of polka in order to extend the routing to incorporate route definitions in another file? Do you have any examples?

Hey, I'm not entirely sure I understand the question, but you can always add new routes to the Polka instance, from anywhere. You just need access to the Polka instance itself.
For example:
// routes/users.js

exports.param = 'userid';

exports.index = async (req, res) => {};
exports.show = async (req, res) => {};
exports.post = async (req, res) => {};
exports.put = [
  middleware1,
  middleware2,
  async (req, res) => {}
];

// builder.js
const { readdirSync } = require('fs');
const { join, basename } = require('path');
const app = require('polka')();

const dir = join(__dirname, './routes');

const GET = ['index', 'show'];
const ROOT = ['index', 'post'];

readdirSync(dir).forEach(file => {
  const base = basename(file);
  const param = mod.param || 'id';
  const mod = require( join(dir, file) );

  for (let k in mod) {
    if (k == 'param') continue;
    let method = GET.includes(k) ? 'get' : k;
    let uri = ROOT.includes(k) ? base : `${base}/:${param}`;
    // add the route handler(s)
    app.add(method, uri, mod[k]);
    // or:
    // app[method](uri, mod[k]);
    // ^ does not support multiple handlers
});

This will create all routes to the /users endpoint, and construct URLs (and methods) based on the export key name. It'd be equivalent to this:
const app = require('polka')();

app.get('/users', async (req, res) => {});

app.get('/users/:userid', async (req, res) => {});

app.post('/users', async (req, res) => {});

app.put('/users/:userid', middleware1, middleware2, async (req, res) => {});

FWIW, you're much better off using a sub-application (or producing one similarly).
const subapp = polka().get('/', ...).get('/:userid', ...).post('/', ...).put('/:userid', ...);

app.use('/users', subapp);

Hope that helps! Certainly possible. About half of my applications create routes dynamically thru some kind of file structure basis.

Certainly more than I had anticipated with that half-assed hastily posted question and non-issue. I appreciate all the information, that's a big help.

Haha no problem, glad I was on the same page.


Server turned off after client not safe disconnection

When client disconnect from websocket server and don't close connection safely, I get error "Socket error: an existing connection was forcibly closed by the remote host" and server closed. How to fix this problem?

I don't think it's a fatal problem?

When client disconnect by wifi disconnected or shut-downed suddenly (for example), my server is suspended and doesn't respond anymore. How to solve this situation?