Testset

Thank you for recently providing the test set files. I have a few questions regarding them:

Do these files require the Data Processing Pipeline?

In the dataset.py file, you defined the StorySalonDataset class for the training data, which includes prev_images and prev_prompts. Is there an equivalent file for the test set, or do we need to create something similar ourselves?

These data have been processed by the data processing pipeline and can be directly used for model performance testing.
Please refer to the experimental section and appendix of our paper. When conducting quantitative evaluation, we use one frame of prev_image and prev_prompt as conditions to generate the current frame. You should change the 3 frames in dataset.py to the corresponding 1 frame as conditions.

Thank you for your reply. I have two more questions:
In the experiments mentioned in the paper, there are no results for the PororoSV and FlintstonesSV datasets. Have you tested your model on these two datasets?
While examining your training dataset, I noticed that many prev_image and prev_prompts entries do not seem to be related to the current prompt (as shown in the example below). What is the reason for this discrepancy, and will this lack of coordination affect the results?
Thank you for your attention and response.

@haoningwu3639

We did not test on the PororoSV and FlintstonesSV datasets. This is because our goal is ‘open-ended visual storytelling’, and the above datasets have only about 10 characters, which cannot meet the training requirements of our task. It would be somehow unfair to directly zero-shot compare our StoryGen with the model trained on these data, but during the writing of the paper, we did not consider fine-tuning on these data.

For our test set, we randomly sampled from the entire StorySalon dataset to ensure fairness. Therefore, the above situation sometimes occurs when only one previous frame is used as condition. Although it cannot provide the character's appearance, the previous frame can still provide a reference for style. Honestly, it isn't easy to obtain a coherent story with specific characters with public copyright.


Proposed 1.2.0-b6

When merged this:

Closes #2730
Closes #2734
Closes #2737
Closes #2739
Closes #2742
Closes #2743
Closes #2747
Closes #2749
Closes #2752
Closes #2754

Jenkins Build Summary
Built from this commit
Built at 20181104 - 05:40:58
Test Results

Build Type
Log
Result
Status

msvc.Debug
logfile
1137 cases, 0 failed, t: 619s
PASS :white_check_mark:

gcc.Release -Dassert=ON, MANUAL_TESTS=true
logfile
915 cases, 0 failed, t: 6m7s
PASS :white_check_mark:

docs, TARGET=docs
logfile
1 cases, 0 failed, t: 0m1s
PASS :white_check_mark:

gcc.Debug -Dcoverage=ON, TARGET=coverage_report, SKIP_TESTS=true
logfile
1140 cases, 0 failed, t: 17m3s
PASS :white_check_mark:

clang.Debug
logfile
1140 cases, 0 failed, t: 3m10s
PASS :white_check_mark:

msvc.Debug, NINJA_BUILD=true
logfile
1137 cases, 0 failed, t: 636s
PASS :white_check_mark:

rpm
logfile
1140 cases, 0 failed, t: n/a
PASS :white_check_mark:

gcc.Debug
logfile
1140 cases, 0 failed, t: 3m28s
PASS :white_check_mark:

clang.Debug -Dunity=OFF
logfile
1140 cases, 0 failed, t: 10m21s
PASS :white_check_mark:

clang.Release -Dassert=ON
logfile
1140 cases, 0 failed, t: 5m38s
PASS :white_check_mark:

gcc.Debug -Dunity=OFF
logfile
1140 cases, 0 failed, t: 11m6s
PASS :white_check_mark:

msvc.Debug -Dunity=OFF
logfile
1137 cases, 0 failed, t: 1271s
PASS :white_check_mark:

gcc.Debug -Dstatic=OFF
logfile
1140 cases, 0 failed, t: 3m28s
PASS :white_check_mark:

gcc.Release -Dassert=ON
logfile
1140 cases, 0 failed, t: 6m6s
PASS :white_check_mark:

gcc.Debug, NINJA_BUILD=true
logfile
1140 cases, 0 failed, t: 3m2s
PASS :white_check_mark:

gcc.Debug -Dstatic=OFF -DBUILD_SHARED_LIBS=ON
logfile
1140 cases, 0 failed, t: 3m27s
PASS :white_check_mark:

msvc.Release
logfile
1137 cases, 0 failed, t: 415s
PASS :white_check_mark:

clang.Debug -Dunity=OFF -Dsan=address, PARALLEL_TESTS=false, DEBUGGER=false
logfile
no test results, t: 10m44s
FAIL :red_circle:

clang.Debug -Dunity=OFF -Dsan=undefined, PARALLEL_TESTS=false
logfile
1140 cases, 0 failed, t: 12m9s
PASS :white_check_mark:

Codecov Report

Merging #2755 into develop will decrease coverage by 0.27%.
The diff coverage is 97.28%.

@@            Coverage Diff             @@
##           develop   #2755      +/-   ##
==========================================
- Coverage    70.37%   70.1%   -0.28%
==========================================
  Files          710     701       -9
  Lines        68399   62392    -6007
==========================================
- Hits         48137   43739    -4398
+ Misses       20262   18653    -1609

Impacted Files
Coverage Δ

src/ripple/crypto/csprng.h
100% <ø> (ø)
:arrow_up:

src/ripple/app/main/Main.cpp
40.16% <ø> (-0.5%)
:arrow_down:

src/ripple/json/json_value.h
94.44% <ø> (ø)
:arrow_up:

src/ripple/rpc/impl/Handler.cpp
99.08% <ø> (-0.2%)
:arrow_down:

src/ripple/crypto/impl/csprng.cpp
93.42% <ø> (-5.36%)
:arrow_down:

src/ripple/beast/core/LexicalCast.h
98.5% <ø> (-0.31%)
:arrow_down:

src/ripple/protocol/impl/BuildInfo.cpp
79.16% <ø> (-2.89%)
:arrow_down:

src/ripple/core/Config.h
100% <ø> (ø)
:arrow_up:

src/ripple/rpc/handlers/Fee1.cpp
81.81% <ø> (-9.85%)
:arrow_down:

src/ripple/protocol/Feature.h
100% <ø> (ø)
:arrow_up:

... and 275 more

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 cb71d49...2a8b0e4. Read the comment docs.


fix(ModelColocation): Fixed verbose mode output for models with compo…

…nent meshes points colocated but on the same unique vertex.

:tada: This PR is included in version 2.6.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


auto-fill inside comments does not work

I've set   (setq comment-auto-fill-only-comments t) and I've noticed that when structured-haskell-mode is enabled, comments aren't getting auto-filled by default. I'd have to manually select a long comment and execute fill-region.

 (I might add it to pull request.
(defun shm-fill-comment ()
"Fill the comment(s)."
(cond ((shm-in-comment)
(fill-paragraph))
((when-let ((beg (save-excursion
(comment-beginning))))
(string= "-- " (buffer-substring-no-properties beg (point))))
(let* ((start (save-excursion (comment-beginning)))
(commentStart (save-excursion
(goto-char start)
(let ((back start))
(while back
(goto-char (- back 1))
(setq back (comment-beginning))))
(point)))
(commentEnd (save-excursion
(goto-char start)
(while (comment-forward))
(point))))
(fill-region commentStart commentEnd)))))

@emmanueldenloye  can you tell me how I can test this function out? I tried adding advice to hindent-reformat-decl-or-fill and fill-paragraph but could not get the desired feature.
What I want is auto-fill-mode  to behave like it does with other modes when comment-auto-fill-only-comments set to t
PS. I can also press return and the comment will get auto filled to multiple lines as necessary. No need press M-q.


Update README.md

Updated details as per my discussion with PG:
User must unmount all Azure Blob NFS share mount points before setting the AZNFS_IP_PREFIXES environment variable and wait for 5 minutes to clear the entry from mountmap file if any before remounting it.

We should cherry pick this to release/aznfs2.0 too
@palashvij-msft


error in logs: "TypeError: Cannot read property 'daysRemaining' of null"

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

Description
I can see this error in my logs from uptime-kuma every few minutes:
TypeError: Cannot read property 'daysRemaining' of null
at Prometheus.update (/app/server/prometheus.js:63:91)
at beat (/app/server/model/monitor.js:396:24)
at async Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:419:17)
FYI I'm not using Prometheus, just the base docker image with just a few http monitors configured, no special settings or anything
👟 Reproduction steps
deploy uptime kuma, add a monitor, wait and check the container logs
👀 Expected behavior
we should not have this error every few minutes in the logs
😓 Actual Behavior
I can see this error in my logs from uptime-kuma every few minutes:
TypeError: Cannot read property 'daysRemaining' of null
at Prometheus.update (/app/server/prometheus.js:63:91)
at beat (/app/server/model/monitor.js:396:24)
at async Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:419:17)
🐻 Uptime-Kuma Version
1.11.1
💻 Operating System and Arch
Ubuntu 20.04 - x64
🌐 Browser
Chrome
🐋 Docker Version
No response
🟩 NodeJS Version
No response
📝 Relevant log output
TypeError: Cannot read property 'daysRemaining' of null
at Prometheus.update (/app/server/prometheus.js:63:91)
at beat (/app/server/model/monitor.js:396:24)
at async Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:419:17)

Can you check if any of your HTTP monitors has missing "certificate info" shown in the monitor page? While this is an easy fix I wonder if there is something else going on.

I have checked quickly the 26 monitors I have not trace about that
some of my monitors are on HTTP some others on HTTPS and also some more with PING

Just wanted to let you know that the issue is still present in the latest version 1.11.3
every minute I have this in my log
Jan 12 08:06:52 uptime-u3 19a2273a2382[788]:     at Prometheus.update (/app/server/prometheus.js:63:91)
Jan 12 08:06:52 uptime-u3 19a2273a2382[788]:     at beat (/app/server/model/monitor.js:420:24)
Jan 12 08:06:52 uptime-u3 19a2273a2382[788]:     at async Timeout.safeBeat [as _onTimeout] (/app/server/model/monitor.js:443:17)


Keybase app shows a blank screen under Debian 9

As the title suggests, opening Keybase resulted in a blank screen. Elements on the page still seem to be clickable, as the cursor changes when I hover over parts of it. I tried running run_keybase again, to no avail. I also rebooted my PC, and tried manually killing the Keybase processes (which was probably a bad idea in hindsight), but still no dice.
Also, I discovered this issue last night. I haven't tried opening Keybase on this device in quite some time before then.
my log id: 3dc5d53a77e5ca082962d11c
My previous issues:
https://github.com/keybase/client/issues/6516
https://github.com/keybase/client/issues/8776
Screenshot of blank Keybase app:

Opened the app today, and it was no longer blank. Guess it fixed itself? I can send another log if that's of any interest.


extensions.enabledScopes=1 blocks privacybadger installed as archlinux package under FF62

Under FF62 with extensions.enabledScopes=1 extensions from /usr/lib/firefox/browser/extensions/ are not loaded at all. Setting extensions.enabledScopes to 4 fixes that.
https://www.archlinux.org/packages/community/any/firefox-extension-privacybadger/ is installed at /usr/lib/firefox/browser/extensions/ so it doesn't work with extensions.enabledScopes=1
Here's some documentation https://developer.mozilla.org/en-US/docs/Mozilla/Add-ons/WebExtensions/Alternative_distribution_options/Add-ons_in_the_enterprise#Firefox_settings

thanks for the info

Yup, thanks for the info. BTW, we don't recommend Privacy Badger (in fact we explicitly said it was not worth it). That said, what is so special about EFFs extension that it uses a different path on Arch?. Is it just PB ? what happens with other extensions on Arch? What happens on other Linux packages? I am not a Linux expert, so excuse my ignorance.
Do we need to add some more info to the user.js? eg
/* 2660: lock down allowed extension directories
 * [WARNING] This will break extensions that do not use the default XPI directories
newline-> * e.g. some Linux packages might require setting enabledScopes at `4`
newline-> * 1=profile 2=user 4=application 8=system
 * [1] https://mike.kaply.com/2012/02/21/understanding-add-on-scopes/
 * [1] archived: https://archive.is/DYjAM ***/

wait, now I'm confused. The MDN link says those values (1,2,4,8) are for extensions.autoDisableScopes which to me is incorrect. https://mike.kaply.com/2012/02/21/understanding-add-on-scopes/ says these values are meant for extensions.enabledScopes :tired: :needsleep: :zzzzz:

You're right, no need to sleep. I find this useful.


Chat temporarily disappears in LHN when last message is updated to "0"

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Actions Performed:

Go to any chat
Send any message
Edit the message to just "0"
Save changes

Expected Result:
0 is displayed on LHN as the last message
Actual Result:
Email address is displayed on LHN if that is the only message in that chat, otherwise, the message before the last message is displayed
Workaround:
unknown
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number: 1.2.28-0
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/202240620-ac7e61da-1aa7-4c95-955c-9c364ae2a0f3.mp4
https://user-images.githubusercontent.com/43996225/202240393-446019ca-4196-4ee6-a79b-0dfdd0c3f428.mov
Expensify/Expensify Issue URL:
Issue reported by: @adeel0202
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1668587954165379
View all open jobs on GitHub

https://www.upwork.com/jobs/~01f71071f960badc10

Taking this issue and fixing it myself, PR incoming.

This is on prod 👍 making it a weekly now to close out regression steps

[ ] @iwiznia  The PR that introduced the bug has been identified. Link to the PR:
[ ] @iwiznia The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
[ ] @iwiznia  A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@laurenreidexpensify] A regression test has been added or updated so that the same bug will not reach production again. Link to the GH issue for creating the test here:
[x] [@laurenreidexpensify] Applicable payments have been made. This potentially includes the issue reporter, the contributor that fixed the issue, and/or the contributor+ that helped on the issue.

Following up here to close out regression testing step here - aiming to close this today

https://github.com/Expensify/Expensify/issues/247789

@adeel0202 HUGE Apologies - I've sent the offer in Upwork now

Now paid!


Add a simple document about the format of the BM file

As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

/approve
thanks for that doc update! Be sure to advertise it downstream for QE and other consumers once the feature is fully in!

/approve
thanks for that doc update! Be sure to advertise it downstream for QE and other consumers once the feature is fully in!

Luckily for them them we will directly showcase this the NFV VA, so we will provide them a real example :)

/lgtm

/recheck

/retest


New structure to store form data into DB

DCO 1.1 Signed-off-by: {{Ayush Gupta}<EMAIL_ADDRESS>

@Snugug @scottnath
It fails one test


[BUG]: [charts/karavi-metrics-powerflex]: Use yaml multiline for ConfigMap files

Describe the bug
The current version put config files as quoted-string which makes it difficult to modify in Kubernetes manually since indentation is part of the language. It will be better to use yaml mutltiline

@coulof: Thank you for submitting this issue!
The issue is currently awaiting triage. Please make sure you have given us as much context as possible.
If the maintainers determine this is a relevant issue, they will remove the needs-triage label and assign an appropriate priority label.

We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at<EMAIL_ADDRESS>

@coulof: Thank you for submitting this issue!
The issue is currently awaiting triage. Please make sure you have given us as much context as possible.
If the maintainers determine this is a relevant issue, they will remove the needs-triage label and assign an appropriate priority label.

We want your feedback! If you have any questions or suggestions regarding our contributing process/workflow, please reach out to us at<EMAIL_ADDRESS>

Done in PR #71

Done in PR #71


Windows Boot

I have two ssd one with the hackingtosh and the other with windows, if I try to boot to windows through clover boot menu, I get a blue screen, but when I use F12 and boot from the windows drive directly it works fine, help please... I want to use the Bluetooth

Try linux using mac VM method  first? Sound better than a double boot just for Bluetooth...

The VM method worked for the bluetooth, side note, I tried it with virtual box initially and it didn't work, it worked with VmWare Fusion as suggested in the article.
Haven't found a solution to the bluescreen issue though


add search functionality

It'd be cool if this also searched github issues

Added searching through issues in f8bc887


[KOA-4829]: Add new typography tokens

Remember to include the following changes:

[x] UNRELEASED.md (See How to write a good changelog entry)
[ ] README.md
[ ] Tests
[ ] Docs (either update backpack-docs now, or create a follow up ticket)

LGTM!
One question are we not adding the new bigger sizes 5xl - 8xl in this PR like app already have done? Or have I missed something?
https://github.com/Skyscanner/backpack-foundations/blob/main/packages/bpk-foundations-common/app/typography.json#L27

I remember we discussed a while ago that there are no use cases for those sizes and while we do support them, there's no need to add them right now. Let me double check with Adam, as it might be good to just add them now to make sure there's no confusion in the future if someone wants to use them!

I remember we discussed a while ago that there are no use cases for those sizes and while we do support them, there's no need to add them right now. Let me double check with Adam, as it might be good to just add them now to make sure there's no confusion in the future if someone wants to use them!

Sure thing! I guess if we want to go down that path we need to sync with @frugoman and @marianeum with this as if we don't add them maybe we should remove them from the apps typography also just to avoid confusion and keep the platforms aligned

I remember we discussed a while ago that there are no use cases for those sizes and while we do support them, there's no need to add them right now. Let me double check with Adam, as it might be good to just add them now to make sure there's no confusion in the future if someone wants to use them!

Sure thing! I guess if we want to go down that path we need to sync with @frugoman and @marianeum with this as if we don't add them maybe we should remove them from the apps typography also just to avoid confusion and keep the platforms aligned

Think it's better to just add them! Otherwise we risk engineers thinking the style in their design doesn't exist and even use tokens instead. Will add them now!


Allow class_of to operate on generics

Problem
I am trying to write a well-typed event bus, where the #on method takes an event class and calls the given block with instances of that class.
Simplified example
Proposed solution
I guess what I would expect is for the computation of T.class_of to be deferred until there is a concrete type available (in this case, at call-sites for EventBus#on).
Alternatively, it might be useful to have a T.instance_of(Foo) helper, which results in the return type of Foo.new, but in any case one would still need to defer the resolution to the call site of the generic method.

I believe I also have this issue as I attempt to add type-checking to my mocking library Mocktail as @paracycle & @Morriar were teaching me Sorbet at their RailsConf workshop.
Because Mocktail provides functionality to return fake instances of provided types, we need to be able to refer to a generic type's instance & class in the type signature. So I was imagining something like this:
sig do
  type_parameters(:E)
    .params(type: T.class_of(T.type_parameter(:E)))
    .returns(T.type_parameter(:E))
end
def self.of(type)
  # …
end

Which fails with:
T.class_of needs a class or module as its argument

Here's a runnable example.
Being able to add meaningful type information to Mocktail would really supercharge its usefulness in test-driven development and providing great messages as invocations are used to drive the creation of new types and methods, so it'd be really awesome to see support for this!
Thanks for all your hard work 💖

Possible dupe of #62

Yes, this looks like a dupe of #62. Thanks for finding.


Add kafka source json to upgrade path for flyway migration

https://github.com/hortonworks/streamline/pull/1053


event (firstDotCall) leak at xdot.c

Cause "out of memory" errors with intel opencl drivers.
Here's the forum thread:
https://software.intel.com/en-us/forums/opencl/topic/701907
And bugfix:
https://github.com/ache7/clBLAS/commit/959372fb480225e6ca7c5f2812c5ab3b524650ef

Hi @ache7
Can you make a PR targeting our /develop branch?  I'll merge this.

Here https://github.com/clMathLibraries/clBLAS/pull/297

Maybe it's better to initialize firstDotCall to NULL and release firstDotCall event before freeSolutionSeq(&seq). Differs if first makeSolutionSeq or executeSolutionSeq failed.

Also, when I search for "eventWaitList, &first" in sources - there is more functions with unreleased events, all these events come to "step->event = events + j;" in makeSolutionSeq, so can be freed automatically somewhere in freeSolutionSeq -> freeSolutionStep -> releaseStepImgs, but I don't see clReleaseEvent() calls. Someone must check this, I don't know code well enough.

Hi @ache7
I have made a new branch in my fork that fixes the many events you mention in this thread.  It passes my unit tests.
If you have time, would you mind testing in your environment?  If not, I'll merge my branch as it's already passing my tests.

Sorry, I don't use library directly, only through other libraries (from cltorch), so can't test it thoroughly. Seems that neural nets use only xdot() from functions with event leaks.

Thanks for replying; I'll close your PR and merge in my fixes.  I think it should square you away.


perf(p2p)!: Remove PeerSendBytesTotal metric

Component of #2840. As we discussed in the issue, we want to deprecate the peer send bytes total metric because:

This is a costly metric (see pprof)
This increases the cardinality of labels in Prometheus, which can slow things down in Prometheus significantly.
Its not really useful at this level of abstraction. We don't have reported usecases of this being used yet. If we want to do it in the future, we should more likely architect it as updating a live go variable, and then sending to our metrics instance on a polling ticker (ala PeerSendQueue metric). The right levels that are interesting for outbound traffic could be the consensus gossip routines or blocksync.

To see its costly, 21% of all CPU time for live osmosis nodes in the latest Osmosis patch release is in peer.Send. over half of that time is in these metrics:

Please advise for how we should go about deprecating / removing this metric? Feel free to close this PR / say you want to handle it separately.

PR checklist

[X] Tests written/updated - N/A
[x] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments - I didn't find any relevant docs
[x] Title follows the Conventional Commits spec

Concerning the fact that this is going to be removed and not available in the future, it would be interesting to ensure this is not used in our QA or Infra tests, I briefly looked at the logic there and couldn't find references to this particular metric, but @hvanz or @sergio-mena might have more insights.
If we find out that some logic critically depends on this metric, probably a better approach would be to make it a configurable option if possible (e.g. disable it through a config value).

ah thanks, didn't think to search the docs with underscores! (Explains why I didn't find it earlier haha)

Thoughts on merging, and then reverting if we discover some metric consumer used this?

That's a bad dev UX 😅

If we find out that some logic critically depends on this metric, probably a better approach would be to make it a configurable option if possible (e.g. disable it through a config value).

please no more config options. IMO we have too many config options

If we find out that some logic critically depends on this metric

Even if some external software depends on this metric, it shouldn't prevent us from removing it. CometBFT's mission and what it should do best in the world is fast and reliable consensus. And if some metric is significantly slowing CometBFT down, we should eliminate it!

Ive tested removing it, and the prometheus time here went down ~70%! TBH that still justifies us removing the other metric or doing that more "asynchronously", e.g. via the updateStatsRoutine

@mergify backport v1.x


[BUG] TS http-hello-world example fails to build

Affected project(s)

[ ] documentation
[ ] examples
[ ] wasmCloud host
[X] wasmCloud CLI (wash)
[ ] wasmCloud dashboard UI (washboard)
[ ] capability providers
[ ] provider bindgen
[ ] control interface client
[ ] other / not sure

Describe the bug
I made a repo to help reproduction: https://github.com/atroche/wasmcloud-ts-issue-repro
I've been going through the tutorial, using typescript, trying to follow the instructions closely.
I'm at the step where I'm adding logging -- so far everything's worked well. I've added the import in world.wit and then in the TS src file:
  import<EMAIL_ADDRESS>
//@ts-expect-error -- these types aren't currently generated by JCO
import { log } from 'wasi:logging/logging';

but I'm seeing this (cleaned up linebreaks and removed my username):
❯ wash dev
ℹ   Resolved wash session ID [D3Q126]
🚧 Starting a new host...
ℹ   NATS server successfully started, using config @ [~/.wash/downloads/nats.conf]
ℹ   NATS server logs written to [~/.wash/dev/D3Q126/nats.log]
👀 Found wadm version on the disk: wadm-cli 0.18.0
✅ Using wadm version [v0.18.0]
🔧 Successfully started wasmCloud instance
✅ Successfully started host, logs writing to ~/.wash/dev/D3Q126/wasmcloud.log
🚧 Building project...
⛔ Failed to build project:

failed to build component with custom command:
"Error opening file ./wasi__logging$logging.js (resolved to /var/folders/1v/rtt8x01j3tn_5k2w7x0zj8b00000gn/T/42aee0ad3a85/sources/wasi__logging$logging.js): No such file or directory

Error while evaluating top-level script, but no exception is pending. Aborting, since that doesn't seem recoverable at all.

Error: the `componentize.wizer` function trapped

Caused by:
    0: error while executing at wasm backtrace:
           0: 0x78f2b2 - <unknown>!<wasm function 12768>
           1: 0x78fc25 - <unknown>!<wasm function 12784>
           2: 0x217a6 - <unknown>!<wasm function 181>
           3: 0x20759 - <unknown>!<wasm function 179>
           4: 0xb8ae  - <unknown>!<wasm function 161>
           5: 0x226a6c - <unknown>!<wasm function 5087>

    1: Exited with i32 exit status 1

Error: Failed to initialize the compiled Wasm binary with Wizer:
Wizering failed to complete
    at componentize (file:///path/to/node_modules/@bytecodealliance/componentize-js/src/componentize.js:271:13)
    at async componentize (file:///path/to/node_modules/@bytecodealliance/jco/src/cmd/componentize.js:11:25)
    at async file:///path/to/node_modules/@bytecodealliance/jco/src/jco.js:200:9
"

any clues? if it's relevant, I am on latest node (v23.5), and on Mac OS 15.2.
I've uploaded my package-lock.json just in case, along with the current contents of world.wit and the TS src file:
https://gist.github.com/atroche/3f942030488fd8f3956f52d2d7d5dc93
Steps to reproduce
git clone https://github.com/atroche/wasmcloud-ts-issue-repro
cd wasmcloud-ts-issue-repro
wash dev

Expected behavior
I expected to see an info log in the wash dev output corresponding to the call to the log function imported in src/http-hello-world.ts.
Environment

OS: Mac OS 15.2
Shell: fish
Node: v23.5
npm: 11.0.0

wash          v0.37.0
├ nats-server v2.10.20
├ wadm        v0.18.0
└ wasmcloud   v1.4.2
Screenshots / Logs / Additional context
No response

Looks like this is a bug in the example code! Thanks for the issue!
Quickest fix is to change the line in src/http-hello-world.ts like this so that the import statement matches the version in the wit file:
- import { log } from 'wasi:logging/logging';
+ import { log } from<EMAIL_ADDRESS>
Give that a go and let us know how you go.

thanks! working well now


Logging in using header method

Description
I'm having trouble when authenticating with HTTP Headers . I've tried passing token instead of password but I think it's not passing the correct header.
Error logs:
2024-07-18 17:21:00,807 - ERROR - Failed to backup ActualBudget: 2 validation errors for LoginDTO
status
  Input should be 'ok' [type=enum, input_value='error', input_type=str]
    For further information visit https://errors.pydantic.dev/2.8/v/enum
data
  Field required [type=missing, input_value={'status': 'error', 'reason': 'invalid-header'}, input_type=dict]
    For further information visit https://errors.pydantic.dev/2.8/v/missing

Ah, I did not know they supported this, I quickly implemented it in https://github.com/bvanelli/actualpy/pull/44

Thanks! Tested using pip install git+https://github.com/bvanelli/actualpy.git@43-header-login and it seems to be working!


Make useUploadFile hook resumable

Return [number, number] state instead "loading"

Decided as separate hook


Fix: broken pathname

We refer to the model file superannuation_fund but named our model file superannuation.rb.  This PR updates that file to be superannuation_fund.rb

@ghiculescu are you able to have a look over this one, please?

I can push a new gem version tomorrow, in the meantime just point to the repo on GitHub for development.

pushed 0.10.5


SquenceDiagram does not support actors with a dash

I try to create a sequencediagram with an actor that includes a dash. It does not work because the dash is also included in the line definition. I just wanted to ask if it is already possible to add actors with dasehs or if we could add this feature.
I tried the following diagram on the mermaid online editor:
sequenceDiagram
foo-test->>+bar:test

Output
Parse error on line 2:
sequenceDiagramfoo-test->>+bar:test
------------------^
Expecting 'NL', 'AS', ',', 'SOLID_OPEN_ARROW', 'DOTTED_OPEN_ARROW', 'SOLID_ARROW', 'DOTTED_ARROW', 'SOLID_CROSS', 'DOTTED_CROSS', 'TXT', got '-'

maybe you want:

sequenceDiagram
foo#8211;test->>+bar:test


Add https://population.un.org/wpp/ Data Source

@ageddesi

Hi @GarrickBee This ticket was a placeholder to look into that data source and see what type of endpoints we could make using the data points.


Subcommands prefixed with a forward slash are seemingly ignored

I don't actually have any clue what's going on here, but I've tracked the misbehavior down to a specific commit, with a failing test fails on this commit, but succeeds on the previous commit.
Let's start with the test case:
def test_bad_command(runner):
    @click.group()
    def cli():
        pass

    @cli.command()
    def wups():
        pass

    result = runner.invoke(cli, ['other'])
    assert 'No such command' in result.output

    result = runner.invoke(cli, ['/other'])
    assert 'No such command' in result.output

This probably explains the issue better. In normal situations, we'd expect some call to cli other to come back and error with "No such command ...", but in the case if prefixing with a forward slash, it somehow tricks the parser into ignoring the entire argument and instead, we get the help of the parent command.
This behavior seems to be broken starting with commit 0a2919f34fcbc635d8530b4c5b60bf119b2bcedb, and the tests fail when applied here, but pass when applied against e68766c35ab134af2db2abf7cbd6f31e9b25d150.
Let me know if there's more information I can provide to help find the issue.
To add to it, in terms of releases, this means the behavior worked against Click 5.1, but has failed since Click 6.0 and continues to not work through latest master.

Related to #590


Cannot install cutorch

luarocks install cutorch gives me this error message :
nvcc fatal   : Unsupported gpu architecture 'compute_52'
CMake Error at THC_generated_THCTensorTopK.cu.o.cmake:206 (message):
  Error generating
  /tmp/luarocks_cutorch-scm-1-6919/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCTensorTopK.cu.o

nvcc fatal   : Unsupported gpu architecture 'compute_52'
CMake Error at THC_generated_THCBlas.cu.o.cmake:206 (message):
  Error generating
  /tmp/luarocks_cutorch-scm-1-6919/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCBlas.cu.o

nvcc fatal   : Unsupported gpu architecture 'compute_52'
make[2]: *** [lib/THC/CMakeFiles/THC.dir/./THC_generated_THCTensorTopK.cu.o] Error 1
make[2]: *** Waiting for unfinished jobs....
make[2]: *** [lib/THC/CMakeFiles/THC.dir/./THC_generated_THCBlas.cu.o] Error 1
CMake Error at THC_generated_THCReduceApplyUtils.cu.o.cmake:206 (message):
  Error generating
  /tmp/luarocks_cutorch-scm-1-6919/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCReduceApplyUtils.cu.o

What am I doing wrong?
Some info :
$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2014 NVIDIA Corporation
Built on Thu_Jul_17_21:41:27_CDT_2014
Cuda compilation tools, release 6.5, V6.5.12

$ uname -va
Linux rhea 3.13.0-32-generic #57-Ubuntu SMP Tue Jul 15 03:51:08 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux

$ lsb_release -a
No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 14.04.4 LTS
Release:        14.04
Codename:       trusty

You have an nvcc that's too old?

So, time to reinstall CUDA?

Shouldn't CUDA 6.5 work?

Won't work for 5.2

I just installed 7.5. But it should work with 6.5 because :
Found CUDA: /usr/local/cuda-7.5 (found suitable version "7.5", minimum required is "5.5")

Also
th> print(_VERSION)
Lua 5.1

And now my installer is stuck at
$ luarocks install cutorch
Installing https://raw.githubusercontent.com/torch/rocks/master/cutorch-scm-1.rockspec...
Using https://raw.githubusercontent.com/torch/rocks/master/cutorch-scm-1.rockspec... switching to 'build' mode
Cloning into 'cutorch'...
remote: Counting objects: 105, done.
remote: Compressing objects: 100% (100/100), done.
Receiving objects: 100% (105/105), 156.21 KiB | 0 bytes/s, done.
remote: Total 105 (delta 8), reused 47 (delta 3), pack-reused 0
Resolving deltas: 100% (8/8), done.
Checking connectivity... done.
cmake -E make_directory build && cd build && cmake .. -DCMAKE_BUILD_TYPE=Release -DCMAKE_PREFIX_PATH="/home/nicholas14/torch/install/bin/.." -DCMAKE_INSTALL_PREFIX="/home/nicholas14/torch/install/lib/luarocks/rocks/cutorch/scm-1" && make -j$(getconf _NPROCESSORS_ONLN) install

-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.8.4
-- Check for working C compiler: /usr/bin/cc
-- Check for working C compiler: /usr/bin/cc -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Found Torch7 in /home/nicholas14/torch/install
-- Found CUDA: /usr/local/cuda-7.5 (found suitable version "7.5", minimum required is "5.5")
-- MAGMA not found. Compiling without MAGMA support

@soumith Please re-open. Nothing is working!

I guess you are using a rather new GPU (TitanX?) that supports the latest compute capability. Since we detect this new card, we assume that you have a CUDA version that allows you to compile for the compute capability that will give you the best performance for your card. So indeed the idea here was to upgrade to CUDA 7.5
Now for the installer hanging, you should make sure that there is no remaining link to the old CUDA installation and that the CUDA samples actually work before running the installer.
If the samples work properly but the installation hang, does installing with this CUDA_ARCH_BIN=2.0 luarocks install cutorch work as expected  ?

@albanD I am using Titan Black.

 CUDA Device Query (Runtime API) version (CUDART static linking)

Googling around, it seems I am in the same boat as https://groups.google.com/forum/#!topic/torch7/kLusyLEj4oc .
I am trying the posted solution.

I was able to install and require cutorch, but now it hangs when I try to exit TREPL:
th> cutorch.getDeviceProperties(1)
{
  computeMode : 0
  memPitch :<PHONE_NUMBER>
  canMapHostMemory : 1
  warpSize : 32
  pciDeviceID : 0
  pciBusID : 2
  totalConstMem : 65536
  pciDomainID : 0
  integrated : 0
  deviceOverlap : 1
  maxThreadsPerBlock : 1024
  clockRate : 1071500
  maxTexture1D : 65536
  minor : 5
  name : "GeForce GTX TITAN Black"
  freeGlobalMem :<PHONE_NUMBER>
  maxTexture1DLinear : 134217728
  kernelExecTimeoutEnabled : 0
  sharedMemPerBlock : 49152
  major : 3
  totalGlobalMem :<PHONE_NUMBER>
  regsPerBlock : 65536
  textureAlignment : 512
  multiProcessorCount : 15
}
                                                                      [0.0012s]
th> exit
Do you really want to exit ([y]/n)? y
[hangs]

Hello. I am having the same problem, my install is stopping with the same exact errors as nicholas. Specifically, I am getting the error nvcc fatal   : Unsupported gpu architecture 'compute_52'. I have a GTX 980, and have read around some places that the gpu architecture compute_52 only applies to the GTX 970 and 980 gpus.

@Maurdekye update your cuda.
i'm also closing this, as this is some kind of a nvidia device driver issue as specified in https://groups.google.com/forum/#!topic/torch7/kLusyLEj4oc

My cuda is already updated to 6.5. Here is my nvcc;
nvcc: NVIDIA (R) Cuda compiler driver Copyright (c) 2005-2014 NVIDIA Corporation Built on Thu_Jul_17_21:41:27_CDT_2014 Cuda compilation tools, release 6.5, V6.5.12
If there's a newer version of cuda available, I don't know about it.

check the nvidia website and get cuda 7.5.

This is where I originally installed cuda from; https://developer.nvidia.com/cuda-downloads to install I followed the instructions to install the 64 bit network deb file for 15.04, but it seems that I ended up with version 6.5 instead of 7.5 somehow.

It actually worked after I removed nvidia-cuda-toolkit. I don't know why it was causing issues, but it was preventing me from installing cutorch.

@Maurdekye it seems that installing nvidia-cuda-toolkit via apt-get forces system to use CUDA v5.5. Just removing it makes system use 7.0 again. :)

I'm getting the following:
-- Found Torch7 in /Users/mariogintili/dev/torch/install
CMake Error at /usr/local/Cellar/cmake/3.0.2/share/cmake/Modules/FindCUDA.cmake:568 (message):
  Specify CUDA_TOOLKIT_ROOT_DIR
Call Stack (most recent call first):
  CMakeLists.txt:7 (FIND_PACKAGE)

-- Configuring incomplete, errors occurred!
See also "/tmp/luarocks_cutorch-scm-1-8206/cutorch/build/CMakeFiles/CMakeOutput.log".

Error: Build error: Failed building.

No idea how to debug that :( help?

looks like you need to install cuda


Implement a base class for the Storage abstraction

It should automate: caching, late initialization, disposing of instances, storage instance-level singletones, and factories

Closed by #119


WIP: Portfolio assets selection

Branches off https://github.com/TokelPlatform/tokel_app/pull/27
And should only be merged after.

Too many conflicts, I m moving it to another branch


Try to config access Grafana without password

Currently when user wants to login grafana, user needs to input account and password. This pr try to make user login without password.

This pr is copy from the metric: https://stackoverflow.com/questions/33111835/how-to-set-up-grafana-so-that-no-password-is-necessary-to-view-dashboards
It may cause security problems, so it still needs some checkings.


PhantomJS cannot run tests

When karma runs the tests using Chrome, then the tests run successfully. When karma runs them using PhantomJS, however, the following error will be triggered:
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  SyntaxError: JSON Parse error: Unexpected identifier "undefined"
  at undefined:16

My package.json file:
...
    "phantomjs": "^2.1.1",
    "karma": "^1.1.1",
    "karma-chai-plugins": "^0.7.0",
    "karma-coverage": "^1.1.0",
    "karma-jasmine": "^1.0.2",
    "karma-mocha": "^1.1.1",
    "karma-mocha-reporter": "^1.1.1",
    "karma-phantomjs-launcher": "^1.0.1",
    "karma-phantomjs-shim": "^1.4.0",
    "karma-sourcemap-loader": "^0.3.7",
    "karma-webpack": "^1.7.0",
...

My karma.conf.js file:
...
    browsers: ['PhantomJS'],
    files: [
      'test/loadtests.js'
    ],
    port: 9090,
    captureTimeout: 60000,
    frameworks: ['phantomjs-shim', 'mocha', 'chai', 'chai-as-promised'],
...

Wherever you are parsing JSON, seems you have a value that is specified as undefined which is not a valid JSON value. PhantomJS may be handling this differently than Chrome?
This SO Q/A discusses JSON and undefined: http://stackoverflow.com/questions/13796751/json-undefined-value-type

There is a solution for this problem?
I have the same issue, when I run using PhantomJS browser i get error, but when I run using ChromeHeadless tests run ok.
Help!
Regards,


Can't bind to nested properties

So it came to my knowledge you can't do something like this bind Obj.Prop.Nested because for some reason it translates to this XAML: <Binding Path="Obj.PropNested/> (notice the lack of second dot).

Hm, can you try bind "Obj.Prop.Nested"?

Yep, that works.


Failed to connect

Hello, after installation in HA, i use menu integration, but when i complete all folder, when i submit i receive message to Failed to connect. thank a lot for your help.

Hello, can you try with the new version : https://github.com/Aohzan/ecodevices/releases/tag/2.0.0

hello, same issue.... I use ECODEVICE RT2, it is compatible ?

(it's already you the first post :grin: )
No it's not compatible, it's not the same API

thank a lot. I dream for RT2 in HA ready to use !

Here : https://github.com/pcourbin/ha-ecodevices-rt2

ohhhh big thank you, you make my day !


Trying to connect two PSEyes with the CLEye driver but not working.

I am trying to add two cameras to psmoveservice and it is not working. I have followed the steps to set up the cameras but its still not working. when I launch psmoveservice it says it detected both of them but only opens one.

I have also bought every thing that was needed to use cleye.

On the wiki you'll see that the CL Eye Driver only supports one camera.

even with the cleye platform sdk?

I believe multiple cameras should work if you have the cleye platform sdk, but you have to copy the cleye dll in the psmoveservice directory with the one in the SDK I think. Someone mentioned it in another issue thread but I can't find it.

couldn't find the .dll file in the sdk.

It's called CLEyeMulticam.dll. It should be enough to delete the one that comes with PSMoveService because the one installed by CL Eye Platform SDK will be put on the path (I believe in C:\Windows\sys or C:\Windows\sysWoW64). Be sure to use the CL Eye Platform SDK install tool to choose the "developer libraries" instead of the "redistributable libraries". I am not on my Windows machine regularly so I would appreciate it if you can take some screenshots of that process and update the Wiki.

It is still only opening one of the the cameras in psmoveservice even when i put the new .dll in to it.

sorry bad picture.

That last log you posted sure looks like it's opening both (though the indexing is goofy). It found both cameras and opened camera index 0 and camera index 1. You can test that they are both open if you launch the config tool > tracker settings. There should be two trackers listed.

If you have Alpha4, then run test_camera.exe to confirm both are working and providing a feed. Then launch PSMoveService.exe and PSMoveConfigTool.exe. In the tool go to Tracker Settings > Test Tracker Video Feed and then return and click the [ >] at the top of the floating box which should change Tracker : 0 to be Tracker: 1. Testing the video feed should show you the perspective from the second camera.

Yea, I'm having issues as well. Both trackers are only showing green in Test_camera. I bought the CL-eye sdk for multicamera.

If its green then i've found that to be bandwidth-related. Try plugging them into different ports, or restarting your computer. I sometimes have mine showing up perfectly at first but then intermittent green screens when Steam VR is running as well. Try to make sure you're not using USB hubs and connect directly to your mobo USB slots or case ones. I ended up shuffling around my ports and connections and after connecting to 2 separate USB3.0 ports, both cameras seem to be working well-enough.

Thanks for the reply Greg. One camera was working fine on usb2, but after installing the CL-Eye SDK non of the cameras seem to do anything but green screen. I'm trying to get one camera working with the sdk now and I've used every usb3 port on my motherboard and also on my usb3 pci-e card. The on on my pci-e card I'm seeing the video feed but ever other frame it seems to be green. I'll look at buying another usb3 card. Thanks

Okay so I got the cameras to work but PSmoveservice keeps crashing on me.


Never mark spam on restore

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Back up from one account
Restore to another account

Expected outcome (what are you trying to do?): All messages restored with similar structure.
Actual outcome (what errors or bad behavior do you see instead?): Thousands of legitimate messages from the first account classified as spam in the second account.
Unfortunately Gmail won't let me bulk mark them all "not spam", either, so this is a whole lot of repetitive clicking to rectify.
I see the Gmail API has a neverMarkSpam option on some endpoints, but I can't tell if it's available on the endpoint you're using because I can't read Python. 🙃

Yes, GYB sets this parameter:
https://github.com/jay0lee/got-your-back/blob/main/gyb.py#L1971
so this shouldn't be happening. Can you provide sample messages or a sample backup that is showing this problem?

I'm up to several thousand messages in spam, but thankfully I found a workaround:

Search label:spam -label:inbox
Select all conversations that match this search
Move to inbox

Each message in spam shows the same reason for being there:

Why is this message in spam? It is similar to messages that were identified as spam in the past.

So it doesn't seem like it would be a phishing filter thing (my old and new addresses are similar, which had me wondering).
Here is a cleaned-up version of the commands I used:
cd
bash <(curl -s -S -L https://git.io/gyb-install)

mkdir<EMAIL_ADDRESS>cd<EMAIL_ADDRESS>~/bin/gyb/gyb --email<EMAIL_ADDRESS>--action quota
~/bin/gyb/gyb --email<EMAIL_ADDRESS>--action backup

mkdir<EMAIL_ADDRESS>cd<EMAIL_ADDRESS>~/bin/gyb/gyb --email<EMAIL_ADDRESS>--action create-project
~/bin/gyb/gyb --email<EMAIL_ADDRESS>--action restore --local-folder ../first<EMAIL_ADDRESS>--label-restored "firstlast.ca"

The messages going to spam are decidedly the "spammier" ones, it's almost exclusively newsletters and notification emails; so it does seem like the spam filter is somehow processing each inbound message despite being asked not to.

Can you provide sample messages or a sample backup that is showing this problem?

Let me know if any of the information above helps. If not, after my restore finishes running, I can try reproducing the problem with a small backup that I'd be comfortable sharing.

I'm seeing this issue as well, backing up a Workspace account and restoring to a free Gmail account.
I have a total of 53,001 messages in the backup, and on restore there was ~7,200 messages in the Spam folder.
My workaround was to move all of those messages in Spam back to Inbox (by selecting, 100 messages at a time and clicking the Not Spam button in the Gmail UI).
If you're going to do this, please ensure that you have 0 spam messages in the target Gmail account, otherwise you could end up moving genuine spam into your Inbox.

I have this issue too 15000 msgs in spam. Mainly very old messages.
Also many seem gotten the date set to the restore time instead of the original date it was sent.

I just released GYB 1.55 which adds a --cleanup option on restore. This tells GYB to confirm the message has a valid From:, Message-ID: and Date: header on it before restoring. This should prevent the message from landing in Spam.
Can a few people do some testing and confirm it works for them? See the 1.55 release details for more info:
git.io/gyb-releases

Hm.. I though if I emptied that spam folder and then did a restore it would restore all those messages again but it doesn't seem so.
What should I do ?

You need to tell GYB to try restoring all messages again with --noresume.
Jay
On Wed, Jan 26, 2022, 5:26 PM Hans-Henrik Jensen @.***>
wrote:

Hm.. I though if I emptied that spam folder and then did a restore it
would restore all those messages again but it doesn't seem so.
What should I do ?
—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/got-your-back/issues/342#issuecomment-1022660251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMGDTP7LIKVKEJQEOPDUYBYINANCNFSM5MOEUPLQ
.
You are receiving this because you commented.Message ID:
@.***>

Hm it would be nice to be able to label the messages that were cleaned up though. I tried to use label-restored but it labels everything now.

Traceback (most recent call last):
File "gyb.py", line 2532, in
File "gyb.py", line 2007, in main
File "gyb.py", line 1769, in message_hygiene
File "gyb.py", line 1713, in cleanup_from
File "email\utils.py", line 215, in parseaddr
File "email_parseaddr.py", line 513, in init
File "email_parseaddr.py", line 256, in getaddrlist
TypeError: object of type 'Header' has no len()
[29748] Failed to execute script 'gyb' due to unhandled exception!

Still got about 300 in spam of the 6000 restored before the exception

I can no longer reproduce the issue with the sample from the issue tracker and --cleanup. Can you share examples of messages that went to Spam?

I'd need to see the full headers as described at:
https://support.google.com/mail/answer/29436?hl=en

Yes, that's fine. You can post it here or email it to me.

Ok I sent an email.

Oh by the way I saw that all the mails that had the now() date after restore seems to have a correct date in msg-db.sqlite so maybe that could be used for --cleanup

I am in the same boat (185,000 email to transfer though). I was watching my Spam as the transfer was happening and saw some go in and then automatically go out of Spam. I was nervous the "older than 30 days will be deleted" thing was happening faster than I was moving them out of Spam.
I am redoing my restore but put this filter in place:

I have not seen anything to to Spam. When the restore is done I'll turn off that filter.
I don't know enough about how quickly "older than 30 days" gets removed from Spam, and makes a data hoarder nervous.

Has anyone else tested with --cleanup to see if that helps?
On Mon, Jan 31, 2022, 5:56 PM brechmos @.***> wrote:

I am in the same boat (185,000 email to transfer though). I was watching
my Spam as the transfer was happening and saw some go in and then
automatically go out of Spam. I was nervous the "older than 30 days will be
deleted" thing was happening faster than I was moving them out of Spam.
I am redoing my restore but put this filter in place:
[image: image]
https://user-images.githubusercontent.com/887675/151886623-b1de6315-e273-424e-bd08-ba16df4aefeb.png
I have not seen anything to to Spam. When the restore is done I'll turn
off that filter.
I don't know enough about how quickly "older than 30 days" gets removed
from Spam, and makes a data hoarder nervous.
—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/got-your-back/issues/342#issuecomment-1026293288,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDIZMG2Q44TIVYGJUXROYTUY4HTNANCNFSM5MOEUPLQ
.
You are receiving this because you commented.Message ID:
@.***>

Did my EML files work fine for you?
man. 31. jan. 2022 23.59 skrev Jay Lee @.***>:

Has anyone else tested with --cleanup to see if that helps?
On Mon, Jan 31, 2022, 5:56 PM brechmos @.***> wrote:

I am in the same boat (185,000 email to transfer though). I was watching
my Spam as the transfer was happening and saw some go in and then
automatically go out of Spam. I was nervous the "older than 30 days will
be
deleted" thing was happening faster than I was moving them out of Spam.
I am redoing my restore but put this filter in place:
[image: image]
<
https://user-images.githubusercontent.com/887675/151886623-b1de6315-e273-424e-bd08-ba16df4aefeb.png
I have not seen anything to to Spam. When the restore is done I'll turn
off that filter.
I don't know enough about how quickly "older than 30 days" gets removed
from Spam, and makes a data hoarder nervous.
—
Reply to this email directly, view it on GitHub
<
https://github.com/GAM-team/got-your-back/issues/342#issuecomment-1026293288
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABDIZMG2Q44TIVYGJUXROYTUY4HTNANCNFSM5MOEUPLQ
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/GAM-team/got-your-back/issues/342#issuecomment-1026294593,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACJV5MVQON6764JT5HET3YLUY4H4BANCNFSM5MOEUPLQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

FWIW, I am also experiencing emails going into Spam (I have not yet tried --cleanup).

Has anyone else tested with --cleanup to see if that helps?

Yes I am. It doesn't work.
My numbers on restoration:

without --cleanup
200+ messages in SPAM out of 1500

with --cleanup
123 messages in SPAM out of 604

I was restoring different accounts so absolute numbers are different but you can easily calculate the percentage, it's nearly the same.

I have been doing an import moving 69k emails from a workspace account to a personal gmail account
I have been running into this same issue.

The messages going to spam are decidedly the "spammier" ones, it's almost exclusively newsletters and notification emails; so it does seem like the spam filter is somehow processing each inbound message despite being asked not to.

This has been my experience as well. Lots of receipts, newsletters, etc.
I put in the same filter @brechmos did and that has helped eliminate messages going to spam.


From moment to data fns

Duplicate of #2129. Duplicate of #2128.
Please stop creating spam pull requests or you’ll need to be blocked from the repo.


Need to stop trace route

When trace route is running, you need to force-quit the application to quit

fixed. trace route can now be aborted


regen after adding otx via build.sh

massive number of changes to JSON

I believe the JSON get's autogenerated by a bot, so no need to run build.sh?
Would need @davemachado's input for this one :)

Ah ha! Just seen this commit.
Mind removing the json run commit please? :)

given that, i'm going to close this PR - #423 is this PR minus the regen step. it only adds a link to README.md.


[Binary Client/Source.] Missing NPC Towers Colony Zone.

The towers are missing on both nation's in colony zone.

They're just not in the DB.

This issue was moved to ApexKO/OpenKO#52


Multiline comment support

For following code, GProlog, SWI and YAP runs successfully:
/*
        foo1
        foo2
*/

:- initialization(main).

main :-
        write(hi /* bar1 */), nl, /* bar2 */ halt. /* bar3 */

Trealla will failed with following error:
% tpl foo.pro
ERROR: unknown operator: '/*'
ERROR: consult 'foo.pro', line=1, '/*'

According to SWI-Prolog manual, multiline comment is specified in ISO Prolog, so it would be nice if Trealla can support that.

Ok, didn't know that!
On Fri, May 26, 2017 at 12:01 PM, Xin Wang<EMAIL_ADDRESS>wrote:

For following code, GProlog, SWI and YAP runs successfully:
/*
foo1
foo2
*/
:- initialization(main).
main :-
write(hi /* bar1 /), nl, / bar2 / halt. / bar3 */
Trealla will failed with following error:
% tpl foo.pro
ERROR: unknown operator: '/'
ERROR: consult 'foo.pro', line=1, '/'
According to SWI-Prolog manual
http://www.swi-prolog.org/pldoc/man?section=isosyntax#sec:<IP_ADDRESS>,
multiline comment is specified in ISO Prolog, so it would be nice if
Trealla can support that.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/trealla-lang/trealla/issues/113, or mute the thread
https://github.com/notifications/unsubscribe-auth/AVqpEmE_f1T2STBbAkCCnyCXWuLG2Rnrks5r9jKOgaJpZM4NnFyQ
.

Trailing whitespace after */ will cause parsing error, I have updated test case in trealla-tests.
Reopen this issue.

Thanks!

Still has problem for following code:
:- initialization(main).

main :-
        A = foo, /*foo*/
        write(A), nl, halt.

I have add this code to trealla-tests.
Reopen this issue.

Add another test in trealla-tests repo:
/*
************************************/

/*
foo bar foo bar foooooooooo bar
************************************/

Fix confirmed, thanks!


djcs_t_share_decrypt needs private_key

Hi,
we'd like to use this library to implement a secure multi party computation protocol for a paper. Looking at the source code, I stumbled over the djcs_t_share_decrypt method which needs the private key.
https://github.com/tiehuis/libhcs/blob/0e1deeaca38617b7908b462747dbb80ae9f29d44/include/libhcs/djcs_t.h#L216-L228
This is in contrast to the equivalent method for the standard Paillier implementation here:
https://github.com/tiehuis/libhcs/blob/0e1deeaca38617b7908b462747dbb80ae9f29d44/include/libhcs/pcs_t.h#L362-L374
which says private key in the doc string, but actually takes a public key.
I've looked at the local_ecample.c and there paillier is used with the share_decrypt method and the public key as a parameter. Intuitively this seems to be the correct way to me, since share_decrypt is intended to be used to 'partially' decrypt a ciphertext with the secret key share which the auth server holds, right?
Is my understanding of the intended functionality wrong and there is a reason why djcs_t_share_decrypt needs a private key, while for the Paillier equivalent the public key suffices?
Thanks for the library :)

If anyone stumbles over this: the fork at https://github.com/lemonviv/libhcs seemingly contains a fixed version of djcs.
I've also started my own fork where I've mainly fixed up some of the C++ interfaces https://github.com/robinhundt/libhcs and will likely also incorporate the fix for djcs.


Delayed battery sensor values

When I drive my car, the battery percentage is (of course) dropping during the trip.
This is more or less directly visible in the official Polestar app, but not in the Polestar API.
Sometimes there is a delay upto 1,5 hours before sensor value of battery percentage is changing.
So when I drive home between 17:00 and 18:00, I see the battery percentage change during a period between 18:20 to 19:20. So I see my battery percentage dropping when I am sitting on the dinner table...
For the ODO meter there is no delay btw.

i notice that there is a delay. in the website the  value is also not changed. only on the app i see a difference. i think this is not from the code issue, but issue on the polestar side (i think they use some caching to prevent DDoS)

Did not check the website, but I agree: seems to be caused by Polestar side.
Thanks for the confirmation.


Add exemptions to word replacement

This is a partial implementation of https://github.com/letsbuilda/imsosorry/issues/29 since it only exempts words. Should probably update it for full strings but that'd be more complex.

exemplifications should be made with a regex, such as checking :(.+?):. in places where word-exemplification is necessary, the exemplification should be done in the nyaify and char_replace functions, such as replacing "lol" to "wow"


Support for OCR confidence data?

It would be great to be able to access data on the confidence of some OCR result.  I'm not super familiar with the C++ API but it looks like this can be obtained by this API.
Would it make sense to adapt this to an impl Iterator?  One difficulty is that the next, etc. functions in the C++ API are all parameterized by a level parameter, while Iterator::next doesn't take any parameters.  One way to get around this would be to have an internal Iter struct that wraps a C++ ResultIterator with a set level, and provide multiple functions like iter_blocks() -> impl Iterator<Item = ...>, iter_words() -> impl Iterator<Item = ...> that internally construct an Iter with the appropriate level.
The Item would need to be some type, say OcrResult, that exposes all of the result data to be passed through.  The C++ functions are on the iterator itself, meaning that they can only be called for the current item.  Because the Iterator trait doesn't impose any lifetime constraints on the Items, I think that this means that all the data in OcrResult has to be populated upfront when it's constructed in the next() call, because it can't be retrieved later.  But the upshot would be that the OcrResult could just be a plain struct that people can pattern match on and destructure. For a start maybe just the confidence value, the text, and the rect would be enough.

I'll stick it on the radar, but for now I'm focussed on improving what's already here.

I'd be happy to implement it, as long as something like that design sounds OK to you and you would have time to review it.

That sounds good to me.
Do you mind if I get my refactor merged in before you start though?
I've got a free evening tomorrow, so I'm hoping to open a pull request then.

I've got a free evening tomorrow, so I'm hoping to open a pull request then.

Not tonight, I'm falling asleep. Soon though, all that's left is finishing error types, the builder pattern and a git history cleanup.

The refactor has been merged in. You're good to start whenever you're free.

Any progress on this?

@Gearme no progress as far as I'm aware.
Would you like to pick this up?


Compiling error in project slic3r with cenv.hpp in line 104 with MinGW toolchain on windows 32 bit XP/7

The error was reported to the project, the mantainers refferd me to your project for a permanent fix:
The original issue:
perl Build.PL stops with the following error:
file \xs\src\boost\nowide\cenv.hpp
Error: cenv.hpp:104:41: error: ISO C++ forbids comparison between pointer and integer [-fpermissive]
in cenv.hpp line 104/41
if I change from
while(*key_end!='=' && key_end!='\0')
to
while(*key_end!='=' && key_end!="\0") //changed '\0' to "\0"
it works as expected.
Infos on build environment perl 5.26 compiled from source on WinXP SP3
Mingw-w64 v5.02 toolchain
Problem could be reproduced on win 7 ultimate 32bit with the same toolchain - it just popped up on XP
the compiler is the latest stable version.
So hopefully there is an possibility to get rid of the error globaly.
Link to the slic3r project: https://github.com/alexrj/Slic3r/
Thanks in advance
red007

Duplicate of #14

Yes I noticed issue 14 before, but here comilation stops the hard way - just to let you know this is serious trouble for some projects.
Thanks for careing in advance


Room directory: stuck after the 20 first items

Since the Merge Room Directory into Rooms feature, the user isn't able to retrieve more than 20 items in the Room directory section.
To reproduce: Hit the rooms tab, then the "ROOM DIRECTORY" section. Scroll in the list.
Note that if you choose Matrix in the Network picker, it will work.
This bug doesn't exists in the Unified Search, Browser directory item.

@jeangb: can you check the fix, please?

@manuroe It works the first time but I when I clear the cache, and goback to it, it doesn't scroll anymore after the 20 first elements.

@jeangb: is Build #1969 make you happy?

Yes, seems good.


Functionality to interconvert roads labels between competition and geojson formats

@nrweir commented on Thu Apr 04 2019
@avanetten , If you get a chance to drop me a sample submission from SN3 I'd appreciate it - I just want to make sure I know what the exact submission format is so I can get the interconversion right. Thanks!

I think this is resolved - the functionality exists using sol.vector.polygon.reproject()


Final breadcrumb's link in search note preview is incorrect

Describe the bug
When using the search popup, the final breadcrumb link's href in the note preview is always "", causing the link to go to the current page rather than the note's page. This bug can be observed on the official Quartz documentation site.

To Reproduce
Steps to reproduce the behavior:

Search for a note
Click the note's name in the preview breadcrumbs
Link takes you to the current page rather than the note's page

Expected behavior
The link should take you to the note's page.
Desktop (please complete the following information):

Quartz Version: 7d4bed6
node Version: 20.18.1
npm version: 11.0.0
OS: Arch Linux
Browser: Firefox

I just noticed that the same issue exists for hover previews.


Add VirtualMachineCRCErrors runbook

Runbook for alert in PR https://github.com/kubevirt/kubevirt/pull/9741

/cc @apinnick

Is this runbook for 4.14.0?

@apinnick

Is this runbook for 4.14.0?

yes

Is there a downstream epic for this feature? I will have to create some doc tasks.

/lgtm


Reactor-Netty HttpClient changes

Changes:

Used Reactor-Core and Reactor-Netty latest release.
New TCP Configurations with idle connection timeout, connect timeout, connection keep alive, etc.
Used Reactor-Netty's response APIs to convert Flux / Mono to Flux / Mono

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests

Merging after conversation with @kushagraThapar


fix(ast): FinallyClause won't get visited as BlockStatement anymore.

#2881  👈
main

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

Join @rzvxa and the rest of your teammates on  Graphite


Incorrect PB_STATIC_ASSERT for bytes inside oneof

My .proto file describing the message looks like this:
syntax = "proto2";

import "nanopb.proto";

message BodyMessage {
    oneof body_type {
        bytes device_data_crypted = 1 [(nanopb).max_size = 256];
        bytes device_config_crypted = 2 [(nanopb).max_size = 256];
    }
}

Сompiling .proto, I got this structure:
typedef struct _BodyMessage {
    pb_size_t which_body_type;
    union {
        BodyMessage_device_data_crypted_t device_data_crypted;
        BodyMessage_device_config_crypted_t device_config_crypted;
    } body_type;
/* @@protoc_insertion_point(struct:BodyMessage) */
} BodyMessage;

There is a STATIC ASSERT in .с file:
/* Check that field information fits in pb_field_t */
#if !defined(PB_FIELD_32BIT)
/* If you get an error here, it means that you need to define PB_FIELD_32BIT
 * compile-time option. You can do that in pb.h or on compiler command line.
 *
 * The reason you need to do this is that some of your messages contain tag
 * numbers or field sizes that are larger than what can fit in 8 or 16 bit
 * field descriptors.
 */
PB_STATIC_ASSERT((pb_membersize(BodyMessage, device_data_crypted) < 65536 && pb_membersize(BodyMessage, device_config_crypted) < 65536), YOU_MUST_DEFINE_PB_FIELD_32BIT_FOR_MESSAGES_BodyMessage)
#endif

This ASSERT uses such a macro from a pb.h:
/* Size of a structure member, in bytes. */
#define pb_membersize(st, m) (sizeof ((st*)0)->m)

When I try to build a project, I get errors:
'BodyMessage {aka struct _BodyMessage}' has no member named 'device_config_crypted'
'BodyMessage {aka struct _BodyMessage}' has no member named 'device_data_crypted'

I think the problem is that ASSERT is generated incorrectly. Since the elements need to be accessed through union. Like this:
PB_STATIC_ASSERT((pb_membersize(BodyMessage, body_type.device_data_crypted) < 65536 && pb_membersize(BodyMessage, body_type.device_config_crypted) < 65536), YOU_MUST_DEFINE_PB_FIELD_32BIT_FOR_MESSAGES_BodyMessage)

Yeah, looks like that's what is happening. Bug is here:
https://github.com/nanopb/nanopb/blob/maintenance_0.3/generator/nanopb_generator.py#L620
It has separate path for doing it correctly for submessages, but the same thing with member name should be done for any field type.
It looks like this logic will be changed anyway for 0.4.0 release, but it might be worth it to fix in the maintenance branch. For a temporary workaround, you can also define PB_NO_STATIC_ASSERT to disable all the checks.

Fixed in maintenance_0.3 now, fix will be included in <IP_ADDRESS>.

Fix released in nanopb-<IP_ADDRESS>.


implement sys_msgq that reside in user memory

Message queues are a basic IPC mechanism currently offered by k_msgq, which is a kernel object. We could implement these in userspace based on the wake/wait APIs and sys_mutex() instead, allowing the message queue data structures to reside in user memory.

Depends on completion of #14493

Not exactly memory protection related but more about the userspace counterpart of k_msgq. Someone who is more familiar with k_msgq would be more appropriate to implement this.

I think this is asking for something equivelant to sys_sem and sys_mutex, I would just close it in this case until the need really arises.


Update theme to use Firefox Design System

This is a WIP for a theme update.

This is just the first step using the dataset for mozilla-central.
Once we land this, I'll add two more "milestones" - for startup, and for browser.xhtml. This will make the list.html view more useful.


[WIP] Add an PM from welcome bot on user creation.

Under work in progress.

@rishig You can test this easily by adding a first_ten_minutes_tutorial() call to initialize function in tutorial.js. This will make the message to be send after each reload. Otherwise you have to create a new user for seeing the PM.

@rishig Updated the code.

Codecov Report

Merging #5342 into master will increase coverage by 0.04%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master    #5342      +/-   ##
==========================================
+ Coverage   86.57%   86.62%   +0.04%
==========================================
  Files         430      430
  Lines       53219    53230      +11
  Branches     2070     2069       -1
==========================================
+ Hits        46077    46109      +32
+ Misses       7142     7121      -21

Impacted Files
Coverage Δ

zproject/settings.py
84.81% <ø> (ø)
:arrow_up:

static/js/tutorial.js
17.14% <0%> (ø)
:arrow_up:

zerver/tests/test_signup.py
99.52% <100%> (ø)
:arrow_up:

zerver/lib/actions.py
95.38% <100%> (+0.18%)
:arrow_up:

zerver/tests/test_home.py
99.04% <100%> (ø)
:arrow_up:

zerver/views/registration.py
100% <100%> (ø)
:arrow_up:

zerver/tests/test_decorators.py
99.53% <0%> (ø)
:arrow_up:

zerver/forms.py
96.15% <0%> (+0.07%)
:arrow_up:

zerver/lib/logging_util.py
75.43% <0%> (+31.57%)
:arrow_up:

Continue to review full report at Codecov.

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

@rishig Please take a look when you are free. I have updated the code.

Left a few more comments, and one additional thing is to fixup the commit messages a bit (let me know if you're not sure what I mean).
This will be a huge improvement to the onboarding, thanks @hackerkid!

Working for me now; maybe I hadn't rebased or something. Sorry about that!
The commit messages could use a file/area indication in front, like
actions.py: Create internal_send_private_message function..
Otherwise ready for merge on my end!

@showell, first four commits ready for merge from my end

I merged the first four commits here.  @rishig Is further action needed for the fifth?

This can be merged as well. Thanks @hackerkid!

@hackerkid can you rebase the remaining commit?

@timabbott

Cherry-picking this into another set of commits, that would otherwise cause a merge conflict. Closing, thanks @hackerkid!


To band return str

Please review the guidelines for contributing to this repository.

[ ] Check the commit's or even all commits' message styles matches our requested structure.
[ ] Check your code additions will fail neither code linting checks nor unit test.

Could not check any of the boxes above as there is no commit message style in the guidelines and no test suite.
Issue #138
Thank you!

Hi !
Thanks to this proposal 😄
I'd prefer to change the output type only if a keyword is added (i.e. as_list defaulting to True)
This behaviour should be also replicated to the to_str function, to keep it symmetric.

Thanks!
Could you now add a line in CHANGES.md with a small text and a reference to this PR ? (see other lines to get inspiration)
Don't hesitate to cite yourself 😉

Suggested changes made and added the changes to CHANGES.md. Sorry for the mess!

Nice! Can I merge?


Move runtime activity into mode parameter

While we're doing a breaking release, what are folks thoughts -- specifically @vchuravy
For now we still will have to do an Enzyme.API.runtimeActivity!(true/false) inside at the start of the generated function, until the Enzyme API for changes.
We'll also probably need some way to query this in gutils, and also pass this as a config to rules.
Which means all rules need a config now [not just forward], which is breaking for fwd mode rules in the ecosystem.

We can arguably forgo the rules change for now, but it is a bit suss presently regardless.

Yeah I was thinking this the other day as well


169 create get credentials

Solves #169
Created getCredentials method to get credentials from the metadata server.
The details from JSON file is left for the user to implement.

Coverage decreased (-0.4%) to 93.179% when pulling d04e52cf4a5f58134c67402c74d09622ec68e769 on bantini:169-create-get-credentials into b39c16f9e70b94789bc883bc1d0e5ab6d23bde2b on google:master.

@stephenplusplus Does it address your requirements?
@saicheems FYI.

Maybe it's possible to have this method be more generic, so that it would return the JSON key file contents, if they exist, otherwise the metadata from GCP?
Also, do you know what the response from GCP looks like that this new method would return?

@stephenplusplus The reason I have not added the functionality to return the contents from the JSON is that the user can retrieve the content from the JSON file directly if they are not on GCP.
The response from the server looks like this:
        {
        "default":
        {
          "aliases":["default"],
<EMAIL_ADDRESS>          "scopes":["https://www.googleapis.com/auth/cloud-platform"]
        },
<EMAIL_ADDRESS>        {
          "aliases":["test-creds"],
<EMAIL_ADDRESS>          "scopes":["https://www.googleapis.com/auth/cloud-platform"]}
        }

The point of the method that I'm looking for is basically: "Get the user's client_email and private_key". If the private key isn't available, just the email is fine. If they are both unavailable, that would be an error.
That way, instead of each application writing:
if GCP
  - call metadata server
  - detect email from response
else
  - read JSON file
  - parse out client_email and private_key

They only have to write:
authClient.getCredentials(function(err, credentials) {
  console.log(credentials.client_email)
})

@bantini Would that work for you?

@stephenplusplus Are we sure we want to make the PK accessible?  For use cases like generating a signed URL you don't actually need the keyfile as you can defer to the signBlob API.

Either way is fine with me. I would lean towards returning as much as possible, otherwise we are basically just creating a "getClientEmail()" method (which is also fine with me).

Test case for checking the environment variable and returning an error if it is not set is failing due to a time-out.

I just tried using this from google-auto-auth. We have a few conditionals for how it creates the auth client from this library: https://github.com/stephenplusplus/google-auto-auth/blob/0ce5ad9657c82c5264920a473544fca472b4fc74/index.js#L57
When the user gives a JSON key file, it uses the (new GoogleAuth).fromJSON() method to create an auth client. This new method (getCredentials()) isn't exposed on that client. Am I doing something wrong?

You can see how I'm trying to use this method in the PR to google-auto-auth: https://github.com/stephenplusplus/google-auto-auth/pull/26

The fromJSON method takes a JSON object as an input and returns the JWT/UserRefreshClient through a callback. The actual contents of the JSON are not stored anywhere. If you want the getCredentials method to handle this case then:

The JSON will have to be passed in the getCredential method so that it can parse the JSON, and return the client_email and private_key
The JSON object created from the fromJSON() method has to be stored as a private object.

Right now getCredentials handles GCE instances and getting data from the GOOGLE_APPLICATION_CREDENTIALS environmental variable because these 2 cases seem to be the ones where the user doesn't have direct access to the client_email and private_key.
However, fromStream(), fromJSON(), _getApplicationCredentialsFromFilePath() will involve changing the signature to accept a boolean flag to return the credentials or the authorization.
Alternately, we can have private methods specific to handling these cases which will be very similar to the public ones. (returning the credential object instead of the JWT client).
@ofrobots @lukesneeringer Should I change the signature of the public methods, or create specific private ones? Also, should I store the JSON contents in an object?

@stephenplusplus Can you try the latest commit?

I just tried with the latest commit, and the getCredentials method is still not available on the authClient returned from fromJSON(), or getApplicationDefault().

@ofrobots @lukesneeringer npm install gts is failing with the message 'Cannot read property '0' of undefined'

I now get TypeError: GoogleAuth is not a constructor from https://github.com/stephenplusplus/google-auto-auth/blob/0ce5ad9657c82c5264920a473544fca472b4fc74/index.js#L57.

@stephenplusplus Did you try it with the latest from master? There has been a lot of changes.

Oh, okay, that worked.
I am back to this error:
Uncaught TypeError: client.getCredentials is not a function
    at getAuthClient (C:\Users\sawch\dev\google-auto-auth\index.js:133:14)

client is a JWT instance.

The GoogleAuth instance returns the credentials, not the client. This code works.
var fs = require('fs');
var contents = fs.readFileSync("credential.json");
// Define to JSON type
 var jsonContent = JSON.parse(contents);
var googleauth = require("./fork-google-auth-library-nodejs");
var GoogleAuth = new googleauth.GoogleAuth();

GoogleAuth.fromJSON(jsonContent, function(err, authClient) {
  if (err) {
    console.log('Authentication failed because of ', err);
    return;
  }
  if (authClient.createScopedRequired && authClient.createScopedRequired()) {
    var scopes = ['https://www.googleapis.com/auth/cloud-platform'];
    authClient = authClient.createScoped(scopes);
  }
  GoogleAuth.getCredentials(function(err, result){
    if(err){
      console.log("First error");
      console.log(err);
    } else {
      console.log("First result");
      console.log(result);
    }
  });
});

Can you show how I get a JWT client?

In your example in google-auto-auth/index.js, the authclient in the callback by the addScope() method is a JWT client if the JSON that you give in the fromJSON() method is the application JSON. The fromJSON() method returns a JWT or a UserRefreshClient depending on whether the JSON is an application JSON or a client secret JSON.
If you want the JWT client to expose the getCredential method instead of the GoogleAuth client, that can be done. However, in that case, let me know what Error should be returned in case the auth client is a user-refresh client.

Got it working! https://github.com/stephenplusplus/google-auto-auth/pull/26
LGTM.

Nilayan, I experimented in another PR to fix the build:

From https://github.com/npm/npm/issues/17858#issuecomment-344523258

Run
npx npmc install
rm package-lock.json
npx npmc install

Recommit package-lock.json.

You'll have to do one of the following:

In .travis.yml delete the cache item.
-cache:
-  directories:
-  - node_modules/
-

^ the cached modules are corrupted and causing npm install to fail.
The alternative requires admin access to Travis. You'd have to clear the build cache for this PR. I think that's an option accessible from More options in the Travis interface.

@saicheems The changes worked. Thanks!

@bantini Can this be merged?

@lukesneeringer Yes. The tests are all passing. And it seems @stephenplusplus got it working in his code.

@bantini can you merge, or do we need someone else?

@stephenplusplus I don't have permission to merge.

@ofrobots ?

@bantini is this one good to merge?  And this fixes #169 still, correct?

@danoscarmike Yes. It fixes #169.

Thanks for this @bantini! Would you let me know when the release is out?

@stephenplusplus I have no information about the release process. But I will surely let you know if I hear something.

@ofrobots do you know of someone we could ping to send a release?

I can do it. Need a bit of time to move #173 off to a separate branch for 1.0.

Cool, thanks!

google-auth-library 0.12.0 published 🎉


Execute before PostProcess Stack

Hi Pjbomb2 testing the Pathtracer, I tried to stack it with PostProcess and it seems to be applied after the PostProcessing. Would it be possible to render it before PostProcess ?
There is an FSR 2.0 PostProcess which gives significant frame rate improvements by upscaling. I use it in most of my projects and it would probably be quite useful to make the Pathtracer even more performant.

Which render pipeline? and yeah I have tried with the github unity fsr 2.2, and its pretty good but janky(not a fault of it, more a fault with unity restrictions)

Hi Pjbomb2. I tried with BuiltIn.
I got decent results with FSR set to Quality with SSRT GI effects.
An example demo: https://file.io/sDJx2gnsL0T1
Toggle between NoFSR and different quality levels by pressing 0

mhm! It works, but I want to do more testing with it first


Add branches method for AST::CaseMatchNode

This PR adds branches method for AST::CaseMatchNode.
This makes a method similar to AST::CaseNode#branches available in AST::CaseMatchNode.
After this, I will be developing with this API on RuboCop.

@marcandre  I'd like to use this PR's feature in rubocop/rubocop#9930. Can you review, merge, and release?

Thanks @koic!
Merged and released as 1.8.0 🚀

@marcandre Thank you always!


[Feature request] Support  an extra user:password for http proxy

Thank for this great tool.
If I convert my sock5 proxy to http proxy, anyone can use  my http proxy if the IP:port is known.
So please add an extra user:password auth for http proxy to avoid any un-authority usage.
Thanks.

Ref:
https://github.com/KaranGauswami/socks-to-http-proxy/pull/21


[WIP] bug with node=None in schema_topology

In running some chemotaxis experiments, I came across a bug in which the environment attempts to update an agent that has already been deleted.  The schema_topology in Store uses node = self.get_path(path) with this agent's path, and retrieves a None. As a partial fix, I have it skip the update if the node is None. Though with this fix, the environment still continues to try updating the agent for the remained of the experiment. Maybe we should check if a node has been marked as deleted and update the environment's subschema?

Or perhaps the problem is this element is not being removed from the topology upon deletion? Do you have a test case for this I can use?

@prismofeverything -- I have a test case in vivarium-chemotaxis. Yeah, the element was not being removed.

Okay, I just made a bunch of improvements in this branch to track deletions to topology, as well as tracking added and deleted processes. This simplifies some logic in update (where it was walking the state tree to find processes and derivers), so should be a performance improvement as well.
I also solved the mystery of why this wasn't triggering before.... the transport_metabolism agent was in the DiffusionField schema!
        schema = {'agents': {}}
        for agent_id, states in self.initial_agents.items():
            location = states['boundary'].get('location', [])
            schema['agents'][agent_id] = {
                'boundary': {
                    'location': {
                        '_value': location},
                }
            }

This self.initial_agents thing to set the value is a bad practice, for just this reason. This process hangs on to a reference to transport_metabolism and resists all efforts to delete it! We can delete from processes and topology, but we can't delete references to things in a process's schema.
To fix this, please initialize the location of this agent using an initial state during experiment configuration! and maybe avoid the use of _value at all times in schema declaration.... it is a sign we should be using initial_state. This should maybe just be an error now that I think of it.


Create Second_tutorial_assignment

I have created a branch, created a file and made a commit, and opened a pull request


Fixes #20 - Disable use of the -noidme option on Catalina.

macOS Catalina deprecated the IDME-related options to hdiutil. If you use the -noidme option under Catalina (as dmgbuild currently does, you receive the warning:
hdiutil: attach: WARNING: ignoring IDME options (obsolete)

This patch checks the macOS version, and omits the IDME option if you're on Catalina (or later).

Same here. Thanks for your PR. Hope it'll be merged soon. Also thanks to the devs & contribs, I use dmgbuild a lot.


createBottomTabNavigator with indicator

is there any way to add line indicator to bottom navigation when "createBottomTabNavigator"

hi there! this is a question which is better suited for stackoverflow or discord. see https://reactnavigation.org/help.html for more information on how to get help.


Interpolate host IP address into environment variables

This will add a second string that is interpolated into env_vars calls. Anywhere that a value of an environment variable contains %DOCKER_HOST_IP% it will be replaced with the public address of the Docker server, as determined by a getaddrinfo call using the current server hostname as specified in the host directive of the config.
cc @jessedearing @didip

LGTM

:+1:
Tested this locally and it works.


proto3 support plans?

Let me start with a big thank you for everyone that contributes to this project, it's a wonderfull library that works like a charm.
Now the issue: Google is introducing a new version of protocol buffer language: Proto3. As far as I can see it's an update to the proto language itself. The binary encoding would not have to change (much) I presume.
Would this be something that is going to be included in nanopb? Would it have big implications?

https://groups.google.com/forum/#!topic/nanopb/1di-Ua50N3Q   Related discussion

Any clue on this? I'm wanting to use protobuf 3..

Once issue #182 has been fixed, I think protobuf 3 should be usable for many purposes with nanopb. But a lot remains to be done before one can claim full support.

i was thinking of writing the pb_json module for encoding from/to json. Encoding seems relatively straightforward in my mind, but decoding requires writing a json parser. Any objection to snapshotting something like jsmn? https://github.com/zserge/jsmn MIT license. Could either modify it to decode while parsing instead of tokenizing, or just use the tokenized outputs similar to the simple example.

Jsmn seems to be quite small, and indeed copying it could help for
customizing it to be more suitable for use in nanopb. Though using
it without modification is fine by me also, do whatever seems most
reasonable :)
License-wise I see no problem, atleast as long as jsmn is confined
to a module that can be selectively included. Maybe it should
go directly into pb_json.c, in order to keep the total numbers of
files small which IMO makes the structure of nanopb easier to
grasp.

so we could have:
a) pb_json.c/h which defines the encode/decode for json.
b) we could also have pb_json_encode.c/h and pb_json_decode.c/h with the parsing stuff being included/referenced from the decode module.
c) Or we could build the json encode/decode into pb_encode.c/h and pb_decode.c/h with preprocessor macros.
I like c but it might make those files pretty large. b introduces a bunch of new files. Maybe option a is best :)

I like a) or b), maybe a) more.

What does everyone think about gRPC and supporting services?

As long as its causes no additional code foot print for those not using it,
I'd say its fine.
On Tue, Nov 17, 2015 at 12:57 AM Benjamin Kamath<EMAIL_ADDRESS>wrote:

What does everyone think about gRPC and supporting services?
—
Reply to this email directly or view it on GitHub
https://github.com/nanopb/nanopb/issues/159#issuecomment-157312299.

Been thinking about this for a little bit before starting too far on it. One interesting thing I think we might need specifically for enums is a lookup table of like struct foo { enum_t value; char* name; } enum_t_string_table [] = { { ENUM_VAL_1, "EnumVal1"}, {...}, ... }; or something like that.
I can't think of a great way of encoding aliases as anything but the first/default one if the enum allows aliases. I think regular field names can be stored in the field entries without compiling into extra space for people not using them  (have some of that done).
Meanwhile, for decoding, if we use something like jsmn and want to decode into jsmntok_t's, we'd need some sort of pb_json_istream_from_buffer(s) type of function that takes a buffer for the stream and another chunk of memory provided for the array of tokens. The number of tokens could be deterministic if it is a deterministic size message (no callback_t fields). The tokens could come from some fixed pool but I like that less since it forces a threading model or would require a porting layer. Haven't played with it enough to know what the resource/performance trade-offs are of decoding the stream from json into the structs on the fly.

It's starting to seem that this work should go into some dev-branch
which will eventually become nanopb-0.4.0. There will be several
breaking changes necessary along the way.

One interesting thing I think we might need specifically for enums
is a lookup table

I agree. Doing #114 will free up the pointer field in the enum field
descriptors, this can then point to the enum lookup table.

I can't think of a great way of encoding aliases as anything but the
first/default one if the enum allows aliases.

Makes sense.

The number of tokens could be deterministic if it is a deterministic
size message (no callback_t fields).

Could nanopb call jsm_parse() several time to obtain a set of tokens?
This way there would not need to be a buffer for all the tokens.

I'm closing this off, basic proto3 support has been included since nanopb-0.3.7.
Support for Any fields, json etc. is still missing, but should be filed as separate feature requests once someone needs them.


#import "RCTBraintree.h" is not found

#import "RCTBraintree.h" is not found

Same issue for me if anyone faced the issue and solved please help.