User is not navigated last active conversation after loading Home URL - Reported by @K4tsuki

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

Action Performed:

Go to staging new expensify
Visit chat using LHN
Close expensify tab
Open staging expensify in new tab by home url [staging new expensify com]
Verify active chat is not last visited chat

Expected Result:
Active chat must be last visited chat before closing the expensify tab
Actual Result:
Active chat is not last visited chat
Workaround:
unknown
Platform:

Where is this issue occurring?

Web

Version Number: 1.1.46-3
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: Any additional supporting documentation
https://user-images.githubusercontent.com/43995119/160735028-bcc545af-5b9a-4069-8812-b7dd20f77590.mp4
Expensify/Expensify Issue URL:
Issue reported by: @K4tsuki
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1647325938362069
View all open jobs on GitHub

Hmmm I can't reproduce this. When I close and open a new tab on staging, I'm taken to my last active chat.

Is this still happening for you @K4tsuki?

I couldn't reproduce this. Adding the triage label again.

@michaelhaxhiu - I'd say we can close if you also don't see this happening.

@lschurr thanks for testing this forreal - not everyone does that consistently!
Also, I agree with you - this isn't reproducible for me either. Closing.


[Feature]: Replace the reachable with an injected function

Description about the feature
Replace the reachable with an injected function. Currently, we use an atomic bool to control whether the server is reachable. We can extend the logic to a function that could help us test in other ways.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can we close this issue when we merge pr #288? @bsbds

Can we close this issue when we merge pr #288? @bsbds

This will be closed in #316


Synthesis failed for Google.Cloud.RecaptchaEnterprise.V1

Hello! Autosynth couldn't regenerate Google.Cloud.RecaptchaEnterprise.V1. :broken_heart:
Please investigate and fix this issue within 5 business days.  While it remains broken,
this library cannot be updated with changes to the Google.Cloud.RecaptchaEnterprise.V1 API, and the library grows
stale.
See https://github.com/googleapis/synthtool/blob/master/autosynth/TroubleShooting.md
for trouble shooting tips.
Here's the output from running synth.py:
2021-03-19 22:45:14,011 autosynth [INFO] > logs will be written to: /tmpfs/src/logs/Google.Cloud.RecaptchaEnterprise.V1
2021-03-19 22:45:14,498 autosynth [DEBUG] > Running: git config --global core.excludesfile /home/kbuilder/.autosynth-gitignore
2021-03-19 22:45:14,502 autosynth [DEBUG] > Running: git config user.name yoshi-automation
2021-03-19 22:45:14,505 autosynth [DEBUG] > Running: git config user.email<EMAIL_ADDRESS>2021-03-19 22:45:14,509 autosynth [DEBUG] > Running: git config push.default simple
2021-03-19 22:45:14,513 autosynth [DEBUG] > Running: git branch -f autosynth-Google.Cloud.RecaptchaEnterprise.V1
2021-03-19 22:45:14,517 autosynth [DEBUG] > Running: git checkout autosynth-Google.Cloud.RecaptchaEnterprise.V1
Switched to branch 'autosynth-Google.Cloud.RecaptchaEnterprise.V1'
2021-03-19 22:45:14,730 autosynth [DEBUG] > Running: git rev-parse --show-toplevel
2021-03-19 22:45:14,737 autosynth [DEBUG] > Running: git log -1 --pretty=%H
2021-03-19 22:45:14,741 autosynth [DEBUG] > Running: git remote get-url origin
2021-03-19 22:45:14,774 synthtool [ERROR] > Failed executing git checkout master:

error: Your local changes to the following files would be overwritten by checkout:
	google/cloud/dialogflow/v2/answer_record.proto
	google/cloud/dialogflow/v2/conversation.proto
	google/cloud/dialogflow/v2/conversation_profile.proto
	google/cloud/dialogflow/v2/participant.proto
Please commit your changes or stash them before you switch branches.
Aborting

Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 354, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 189, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 312, in _inner_main
    git_source.enumerate_versions(sources, pathlib.Path(temp_dir))
  File "/tmpfs/src/github/synthtool/autosynth/git_source.py", line 202, in enumerate_versions
    source_versions = enumerate_versions_for_source(git_source, temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/git_source.py", line 166, in enumerate_versions_for_source
    local_repo_dir = str(synthtool_git.clone(remote))
  File "/tmpfs/src/github/synthtool/synthtool/sources/git.py", line 85, in clone
    shell.run(["git", "checkout", "master"], cwd=str(dest), check=True)
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 39, in run
    raise exc
  File "/tmpfs/src/github/synthtool/synthtool/shell.py", line 33, in run
    encoding="utf-8",
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['git', 'checkout', 'master']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


test__diff fails with git if coloring is enabled (now with patch)

Apologies for the duplicate report, but I don't seem to be able to change the ticket I just created. (I had expected to be presented with some sort of upload button to attach a patch once the ticket is created, but this didn't appear, nor am I able to delete or amend the ticket in any way. Is this only possible for registered users? If so, how can I create an account? To be honest, Trac's user interface seems rather obscure and not very user-friendly.)
Anyway, here is the bug report again, together with the promised patch:
The test {{{BaseTestWorkingCopy.test__diff}}} in test_versioncontrol.py fails if I have the following setting in my ~/.gitconfig file:
{{{
[color]
diff = always
}}}
(As an aside, when I change the setting either to 'diff = true' or 'diff = false' then the test runs fine.) So, just to be on the safe side, it seems like a good idea to always turn coloring off when producing a git diff. The attached patch does this.
I was going to add a test which exposes the faulty behaviour, however I wasn't sure how to do this without breaking the other {{{WorkingCopy}}} classes (mercurial, subversion and bazaar), so I haven't done it yet.
Patch:
{{{
HG changeset patch
User Maximilian Albert
Date<PHONE_NUMBER> 0
Node ID 7eec389040e6f5ff7cfdf4d3cd08dc44eeefdd76
Parent  db6bcdbedeaec7d1c542f74c13ba850752f6198b
Make sure that git-diff doesn't accidentally colorize the output (e.g. when this option is set in .gitconfig), otherwise comparing the diff may fail.
diff --git a/src/versioncontrol/_git.py b/src/versioncontrol/_git.py
--- a/src/versioncontrol/_git.py
+++ b/src/versioncontrol/_git.py
@@ -102,7 +102,7 @@
def diff(self):
"""Difference between working copy and repository."""
g = git.Git(self.path)

   return g.diff('HEAD')

   return g.diff('HEAD', color='never')

def content(self, digest):
repo = git.Repo(self.path)
}}}

Imported from Bitbucket issue 114,  created by cilix on 10-30-2012 at 15:53, last modified: 01-21-2014 at 12:00

patch applied in r577
Trac comment by apdavison on 12-10-2012 at 12:47

Hi Maximilian,
I'm sorry for the problems you had with the issue tracker. I've created ticket:118 to remind me to do something about it. Send me your preferred username (e.g. via http://www.andrewdavison.info/contact/) and I'll create an account for you.
Andrew
Trac comment by apdavison on 12-10-2012 at 13:16


Support for canvas/video/audio captureStream()

I would like to add an image overlay to the created video. I am using other libs to add overlay for my images and gifs. I was wondering if this can support something like that?

@marloeleven there's this: https://github.com/brightcove/videojs-overlay Can you give that a try and let me know how it goes?

@marloeleven also take a look at this recordrtc example: https://webrtc-experiment.com/RecordRTC/simple-demos/show-logo-on-recorded-video.html

Support for canvas.captureStream() is basically needed for this.

@thijstriemstra Safari does support capturestream for canvas. It would be extremely useful to support canvas recording with an option to record with the microphone as well.


Refactor Redis Connector

Refactoring the Redis connector so that it does not require an implicit redis client to be passed on each call (impure!), but instead it will handle the creation and the release of the client with Resouce.
Additionally, the lettuce api will be completely obscured to the user so it will avoid confusions and all will be under monix.connect.redis.
Finally, the way in which currently underlying lettuce api is called is not ideal, since if the mono is empty a failed Task is returned. It is preferably to return a Task[Option[T]] which in some cases we could safely replace it with Task[T]] by using default values.

WIP

https://github.com/monix/monix-connect/pull/590


Use Function("return this") to get global?

Underscore currently breaks in 'strict' mode while trying to access properties on the global object. The global object is being set using the (function(root) { ... })(this) pattern. However, in this case, root remains undefined and hence the issue.
Using the Function("return this")() pattern instead alleviates the issue, though arguably ugly. Thoughts?
Steps to reproduce: Import underscore in a babel-jest test harness. (uses strict mode)

Would be better to just create a root object on L12.
  var root = this || {};

Underscore doesn't use strict mode.

It's not an issue of Underscore using strict mode, but the new ES6 module loaders forcing strict mode.

Seriously? Strict mode is enforced now in "real" ES6? Or is this just "polyfill" module loaders being zealous?

http://people.mozilla.org/~jorendorff/es6-draft.html#sec-strict-mode-code
Ugh.
Is this going to play nice with _.template?

It's not strict mode exactly. The ES6 spec says that this is undefined for module contexts.

http://people.mozilla.org/~jorendorff/es6-draft.html#sec-strict-mode-code

I stand corrected, it's always strict mode.

Can we replace this with self? I'm pretty sure that should work with node >= 0.8 (haven't tested earlier other versions) and all browsers I've ever used

^ That would also add support for loading underscore in webworkers

@jashkenas jest doesn't play well with loading underscore either.

Also, this issue affects Backbone as well. Relevant issue: jashkenas/backbone#3566

@megawac self won't work in node or iojs.

See pull
On Apr 16, 2015 8:13 AM, "Diwank Singh Tomer"<EMAIL_ADDRESS>wrote:

@megawac https://github.com/megawac self won't work in node or iojs.
—
Reply to this email directly or view it on GitHub
https://github.com/jashkenas/underscore/issues/2152#issuecomment-93719147
.

ran into this when using babel-jest as scriptPreprocessor. Any idea?
  "dependencies": {
	"react-native": "^0.12.0",
	"react-native-animatable": "^0.1.0",
	"react-native-webintent": "^1.0.2"
  },
  "devDependencies": {
	"babel-jest": "*",
	"jest-cli": "^0.7.0"
  },
  "jest": {
	"collectCoverage": true,
	"rootDir": "./",        (I also used "", but same thing)
	"scriptPreprocessor": "<rootDir>/node_modules/babel-jest",
	"testFileExtensions": ["es6", "js"],
	"moduleFileExtensions": ["js", "json", "es6"]
  }

● Runtime Error
TypeError: /Users/yourname/Proj/ReactNativeExperiment/__tests__/test2-test.js: /Proj/src/components/assets/multiSelect.js:Proj/lib/underscore.js: Cannot read property '_' of undefined
    at Proj/lib/underscore.js:15:32
    at Object.<anonymous> (Proj/src/lib/underscore.js:1548:1)
    at Object.runContentWithLocalBindings (/Proj/node_modules/jest-cli/src/lib/utils.js:394:17)
    at Loader._execModule (/Proj/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:234:11)
    at Loader.requireModule (/Prou/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:845:14)
    at Loader.requireModuleOrMock /Prou/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:875:19)
    at Object.<anonymous> (Proj/src/components/assets/multiSelect.js:9:45)
    at Object.runContentWithLocalBindings (/Proj/node_modules/jest-cli/src/lib/utils.js:394:17)
    at Loader._execModule (/Proj/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:234:11)
    at Loader.requireModule (Proj/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:845:14)
    at Loader.requireModuleOrMock (Proj/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:875:19)
    at Object.<anonymous> (//Proj/__tests__/test2-test.js:4:19)
    at Object.runContentWithLocalBindings (/Proj/node_modules/jest-cli/src/lib/utils.js:394:17)
    at Loader._execModule /Proj/node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:234:11)
    at Loader.requireModule (/Proj//node_modules/jest-cli/src/HasteModuleLoader/HasteModuleLoader.js:845:14)
    at jasmineTestRunner (/Proj/node_modules/jest-cli/src/jasmineTestRunner/jasmineTestRunner.js:264:16)
    at /Proj/node_modules/jest-cli/src/TestRunner.js:379:14


[core] feat: improve date rendering in html formatter

This change will cause feed item dates to be
rendered as
2022-03-20 19:03:30
instead of
2022-03-20T19:03:30-00:00
This applies only for the html output format.

If this works every, yes please.
Is there a reason for the current stamp? Compatibility to some rss readers or so?

As I understand the code, this would only affect rendering of the bridge html form. It won't affect existing bridges (bridge urls)


feat: Add pipeline evaluation module

This PR introduce a dedicated module for streamlining evaluation of our scoring pipelines.
The new functions will allow to persist model performance :

Add performance diagnosis plots
Add classification reports (csv,  json)

Here is how it looks for now:

Let me know what you think!

@dataJSA not sure if it was on purpose, but you didn't delete the remaining branch ;)


ceph/daemon totally broken

for example, if run MDS:
sudo docker run -d --net=host \
-v /var/lib/ceph/:/var/lib/ceph/ \
-v /etc/ceph:/etc/ceph \
-e CEPHFS_CREATE=1 \
ceph/daemon mds

in entrypoint.sh kviator runs with --kvstore=none argument, what causes error:  Backend storage not supported yet, please choose another one

That's not good.  I'm guessing these KV_TYPE checks are failing... looking into it

Nope; leaky KV_TYPE walling... line 502 is the problem.  I'm going to hack-fix this for now, but this script needs to be refactored.

Sorry, I was wrong not all broken, but only MDS and RESTAPI.

line 572?

Yeah, I see.  I'm working on a cleanup right now that will get that, too.

Feel free to PR a quick-fix for that one, too, though.  This cleanup will take me a while, but I'll be happy to merge in a hack-fix

I believe that should do it; reopen, if necessary


Does it work on Private Account? or it only scraps public account?

or it only scraps public account?

Please refer to error #615. It outlines that twint only searches public tweets.
I checked using my personal private account and found that if a user has "Protect your Tweets" enabled in their privacy settings (rendering the account private) then that account's data is not searchable through twint.
In regards to obtaining private tweets please refer to pielco11's comment over in #615.

the same issue. have not working on Private Account


Add badge label for buttons

I've added an ability to set up badge text for every button and customize this label.
I've also added an ability to get current view controllers stack

@GigabyteTheOne This PR is good, I really apologise for the delay!
If you can have a look at the comments I wrote and fix those things, that would be awesome. I'll merge it afterwards. Thanks!


Add training API

Description
This PR implements changes described in #212
Affected Dependencies
n/a
How has this been tested?
Added unit tests
Checklist

[x] I have followed the Contribution Guidelines and Code of Conduct
[x] I have commented my code following the OpenMined Styleguide
[x] I have labeled this PR with the relevant Type labels
[x] My changes are covered by tests

Updated docs and added additional input spec type to easier reference values in PyGrid's client_config.


Selecting query-key patches

@lyndonzheng Hi, thanks for your great work. After reading your paper and source code, I have a question about selecting query-key patches. In your source codes, the selecting codes are as below.

In line 246, left, top = pos_x - int(pw/2), pos_y - int(ph/2).  What's the reason to subtracts int(pw/2) from pos_x ?
Watching codes from line 244 to line 259, It seems like a strategy to select query-key patches. Is there some reason to use this strategy or we could use other strategy to get same results ?

Hi @YangGangZhiQi, thanks for your interesting. This strategy is applied to ensure the valid sample patches. Without this processing, the patch may be out of the original feature scale.

Get it. Thanks for your quick answer.
These days a question often troubles me when working in Image-to-Image transfer field.  How to  explain the correctness of the generated results? Is there some other way except just observation to declare there is no error in the generated images?
I would appreciate it if someone could help me.


Research benefits of our design decisions for prototype pitch

In order to effectively pitch our prototype we should understand and be able to clearly explain the reasons for and benefits of our design decisions.

Research completed, information compiled from sources such as stack overflow, official webpages (https://www.postgresql.org/about/advantages/ http://camel.apache.org/jpa.html https://material.angularjs.org/latest/), and other well-known programming resources.
Compiled information to be discussed in the pitch can be found here: https://docs.google.com/document/d/11t-IkGQB93z_mcB4BcgBt6xACNyGceWQOas0EZauGhM/edit?usp=sharing


Update tox.ini envlist to match supported Pythons

Pythons 3.1, 3.2 are no longer supported, but Python 3.5, Pypy, and
Pypy3 are.

I think it might be the case that Python 3.2 is still supported (and I just prematurely removed the trove classifier when 3.2 support was considered deprecated). No matter, tox.ini isn't officially supported, so whatever the community agrees on is fine.


Compiler doesn't like obj[Symbol.iterator] on a @struct

See this example:
Code:
var map = new Map();
var iter = map[Symbol.iterator];

Result:
input0:2: WARNING - Cannot do '[]' access on a struct
var iter = map[Symbol.iterator];
               ^

Long term, the type system should understand that "symbol" is a special type which is allowed in this way. Shorter term, I'm not sure what we want to do.

This has come up in a few times now: #1720 #1507

This is also a problem for anyone hoping to compile RxJS with Closure. RxJS marks Observables with such a symbol.


Fixed missing builtin arguments on "cms" command

Description
When ./manage.py cms is run, it currently crashes. This is because Django expects the settings attribute to be on the parsed arguments object: https://github.com/django/django/blob/42fea5d5b8e69ae2d72d1cd81fcb31a9b532c29d/django/core/management/base.py#L76
This PR adds the missing built-in arguments to the main "cms" command same as the other subcommands.
Checklist

[x] I have opened this pull request against develop
[x] I have updated the CHANGELOG.rst
[x] I have added or modified the tests when changing logic
[x] I have read the contribution guidelines  and I have joined #workgroup-pr-review on Slack to find a “pr review buddy” who is going to review my pull request.

@sin-ack Please merge develop into this branch and push.


pymkup throws 'column_data' error

All of a sudden every time I try to use this I get a:
Exception has occurred: ModuleNotFoundError
No module named 'column_data'
File "\pymkupTest.py", line 1, in
from pymkup import *
The module is for sure in the packages folder...
pymkup\column_data.py
I'm not quite fancy enough to figure out what's going on since the column_data package is there.

I also ran into this error. I moved the files into my local directory and then commented out line 15 of pymkup.py, see below. Then it appears to work. `class Pymkup:
def init(self, file):
try:
self.file = file
self.inpfn = dirname(realpath(file)) +"\" + self.file
self.fd = open(self.inpfn, "rb")
self.template_pdf = PDFDocument(self.fd)
# Checking if the PDF was authored by BB
# "Bluebeam" in self.template_pdf.metadata['Creator'] # comment out this line
        self.all_pages = [p for p in self.template_pdf.pages()]
        self.spaces_path = {}
    except Exception:
        print(self.inpfn, "doesnt exist.")`

What do you mean by Local directory?

I also ran into this error. I moved the files into my local directory and then commented out line 15 of pymkup.py, see below. Then it appears to work.

I also ran into this error. I moved the files into my local directory and then commented out line 15 of pymkup.py, see below. Then it appears to work.
    def __init__(self, file):
        try:
            self.file = file
            self.inpfn = dirname(realpath(__file__)) +"\\" + self.file # added '+ "\\"' to allow for document name to be entered
            self.fd = open(self.inpfn, "rb")
            self.template_pdf = PDFDocument(self.fd)
            # Checking if the PDF was authored by BB
            # "Bluebeam" in self.template_pdf.metadata['Creator'] # comment out this line

            self.all_pages = [p for p in self.template_pdf.pages()]
            self.spaces_path = {}
        except Exception:
            print(self.inpfn, "doesnt exist.")```

I've followed exactly this but I'm still getting the same error. I've tried everything at this point even manually importing only the 3 python files. For some reason I can't get column_data,py used within the other scripts.


Can support for persistance ?

Thanks for such good library , helps me a lot .
Is there a plan for support save the data to disk and read from disk function ?

Sorry for the slow response. There are currently are no plans for persisting data with this library.


Change timeouts to account for sidekiq queue

This has changed the amount of time it takes for an email to get through
the system, resulting in smoke test timeouts.  This change should fix
that.

This seems reasonable.


main branch no more using rocksdb v9 ?

Since the merge of this commit
https://github.com/Congyuwang/RocksDict/commit/128ef0f1fd504aa3f01241a7d6c2b6b947537dab#diff-2e9d962a08321605940b5a657135052fbcef87b5e360662bb527c96d9a615542R14
the main branch is no more based on https://github.com/Congyuwang/rust-rocksdb but on rust-rocksdb
does it mean RocksDict is no more using rocksdb v9 ?
maybe a compatibility matrix in the readme of RocksDict and rocksdb versions would be great , wdyt ? thanks

No it is using rocksdb v9. The main branch has added some C-API to support Columns api, the PR of which has not yet been accepted by RocksDB.

thanks for your answer @Congyuwang , but I'm note sure to understand
this issue is still open https://github.com/rust-rocksdb/rust-rocksdb/issues/884


Add docs about alternatives and inspiration

:memo: Add docs about alternatives and inspiration

Codecov Report

Merging #62 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #62      +/-   ##
==========================================
- Coverage   99.96%   99.96%   -0.01%
==========================================
  Files         166      166
  Lines        2925     2901      -24
==========================================
- Hits         2924     2900      -24
  Misses          1        1

Impacted Files
Coverage Δ

tests/test_others.py
100.00% <0.00%> (ø)

docs/src/commands/help/tutorial001.py
100.00% <0.00%> (ø)

docs/src/commands/index/tutorial002.py
100.00% <0.00%> (ø)

docs/src/commands/options/tutorial001.py
100.00% <0.00%> (ø)

docs/src/commands/callback/tutorial003.py
100.00% <0.00%> (ø)

docs/src/commands/callback/tutorial004.py
100.00% <0.00%> (ø)

docs/src/subcommands/name_help/tutorial002.py
100.00% <0.00%> (ø)

docs/src/subcommands/name_help/tutorial004.py
100.00% <0.00%> (ø)

docs/src/subcommands/name_help/tutorial005.py
100.00% <0.00%> (ø)

docs/src/subcommands/name_help/tutorial006.py
100.00% <0.00%> (ø)

... and 9 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 dd72411...52f214c. Read the comment docs.


Sample viewer: use new API key

Sample

[x] Be able to use both old API key and new API keys
[x] Avoid double checking the same logic in two different places

Checklist

[x] PR title follows convention - keyword: Short description of change
[x] PR targets the correct branch
[x] Self-review of changes
[x] There are no warnings related to changes
[x] A build was made and tested on all relevant platforms
[x] No unrelated changes have been made to any other code or project files
[x] No unnecessary includes or namespaces added
[x] Code follows plugin coding style
[x] New code and changed code has proper formatting
[x] No unintentional formatting changes
[x] Commits have descriptive titles

ArcGIS Maps SDK Version
Tested with the newest daily.

Can this go into the Release Branch instead of main? We are trying to avoid having to force update the Release branch with main. Conflicts can get nasty.

Can this go into the Release Branch instead of main? We are trying to avoid having to force update the Release branch with main. Conflicts can get nasty.

I think @mbranscomb wants to prioritize this. This is an issue that users are potentially having problem right now. So better to go into main?

Okay, that's fine then.

Thanks!


Added NetCDF

Please include new package NetCDF in UppHub.

I'm glad, it's my first pull request :relaxed:


Fix useLayoutEffect warnings

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR updates the usage of useLayoutEffect to useIsomorphicLayoutEffect for consistent server-side rendering compatibility.
Detailed summary

Replaces useLayoutEffect with useIsomorphicLayoutEffect in multiple files
Ensures consistent rendering behavior across client and server side

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#4079  👈
main

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

Join @MananTank and the rest of your teammates on  Graphite


Homebridge crashes

Hi,
please look at this crashlog:
SyntaxError: Unexpected token < in JSON at position 0
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/usr/lib/node_modules/homebridge-jablotron/lib/jablotron-client.js:56:38)
    at IncomingMessage.emit (events.js:333:22)
    at endReadableNT (_stream_readable.js:1201:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

Same here. And no sign in MyJablotron-App possible. Maybe a server problem?

Yes, server problem

if server make problems, plugin cannot crash the homebridge

@grzegorz914 you are right, it is not programmed defensively enough. But I'm not sure what we should give back if the system is down, I'm not aware of a state of error.
And I would prefer to see an error than to have it return some value and not knowing if that value is the actual value or an error value.
Any ideas?

Now we get this without Homebridge crash:
[3/5/2020, 7:37:58 PM] [Alarm] {
  status: false,
  error_status: 'service_unavailable',
  error_message: 'The service is not available.',
  checksum: '01b7cd8d40b14b97cbe8364440d285c1',
  server_id: '3523069-1583433477',
  client_id: null
}
[3/5/2020, 7:37:58 PM] [Alarm] Error: service_unavailable
    at /usr/lib/node_modules/homebridge-jablotron/lib/jablotron-client.js:25:31
    at IncomingMessage.<anonymous> (/usr/lib/node_modules/homebridge-jablotron/lib/jablotron-client.js:56:17)
    at IncomingMessage.emit (events.js:333:22)
    at endReadableNT (_stream_readable.js:1201:12)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)

If server id down return error value and prevent homebridge crash.

Ok gotcha.
The server seems to be working again, however for me Home app still does not update.

New error:
[3/5/2020, 7:16:23 PM] [Jablotron] { Error: getaddrinfo EAI_AGAIN api.jablonet.net api.jablonet.net:443
    at GetAddrInfoReqWrap.onlookup [as oncomplete] (dns.js:56:26)
  errno: 'EAI_AGAIN',
  code: 'EAI_AGAIN',
  syscall: 'getaddrinfo',
  hostname: 'api.jablonet.net',
  host: 'api.jablonet.net',
  port: 443 }```


pipenv cannot resolve dependencies properly

Describe the problem
when installing jupyter-book and jupyterlab with pipenv it was impssible to find a set of versions and packages to fulfill all the requirements.
by changing setup.cfg of the jupyter-book package and using the 6be37cd66d1b7668b16a4bab385222b58f38930d commit of myst-nb this problem was solveable.
Please provide / point me to a way to install jupyter-book + jupyterlab without using a git hash as a version...
Link to your repository or website
No response
Steps to reproduce
pyhton -m ensurepip
pip install pipenv
pipenv install jupyter-book jupyterlab
The version of Python you're using
3.9.6 (conda)
Your operating system
rhel
Versions of your packages
jupyter Book      : 0.11.3
External ToC      : 0.2.3
MyST-Parser       : 0.14.0
MyST-NB           : 0.13.0
Sphinx Book Theme : 0.1.3
Jupyter-Cache     : 0.4.3
NbClient          : 0.5.4
Additional context
the issue becomes visible by using pipenv
pipenv install jupyter-book jupyterlab
Resolution
This problem is not something that MyST-NB or Jupyter Book has control over. Instead, it is an issue with pipenv itself (since it does not have a proper dependency resolver. See https://github.com/jazzband/pip-tools/issues/1372 for some discussion of this. However, the next chain of releases in the stack will make this problem go away in the short term. In addition, see https://github.com/executablebooks/MyST-NB/issues/355#issuecomment-907656056 for a workaround that uses pips resolver.

Hey,
i understand, this is not a issue of jupyter-book.
i leave here a link to the solution which in the end really helped me.
https://github.com/franzhaas/pipenv_by_pip
essentially using pip to resolve the dependencies.
I hope this is helpfull in the future

I've added a "resolution" section to the top comment and linked to @franzhaas's helpful post - thanks very much for following up!


homework 7

Topics Covered

Annotation
Bean scope
dependency injection all three types
@Autowire
@Qualifire

Screenshots

Drift Video
Watch the Drift Video

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
1

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

The commit message must be a bit descriptive

Needed to Identify the highest and lowest prices vehicles that are generated and print their details.


Failure when temperature is a multiple of 16.0 deg C

My application contains the following (pseudocode) loop which is called once per second:
if (sensors.isConversionAvailable(chSensor)) {
ch.temp = sensors.getTempC(chSensor);
if (ch.temp != DEVICE_DISCONNECTED) {
. . .
}
sensors.requestTemperatures();
}
This code fails if the temperature reading is exactly 16.0 degrees C - isConversionAvailable() always returns FALSE.
Looking at the library code, isConversionAvailable() reads all of the scratchpad registers and returns TRUE if register 0 (temperature LSB) is non-zero. However a temperature of 16.0C is stored as 0x0100 in the scratchpad so the LSB will be zero and FALSE will be returned.
I have replaced isConversionAvailable() by the following, which follows the device datasheet by issuing a single read slot to test the data line, and my code now works correctly.
bool DallasTemperature::isConversionAvailable(uint8_t* deviceAddress)
{
// Check if the clock has been raised indicating the conversion is complete
_wire->reset();
uint8_t b = _wire->read_bit();
_wire->reset();
return (b == 1);
}
EDIT: Re-reading the data sheet I don't think my fix is correct. Certainly the reset() calls are not needed. However all my function does is to return 0 if any sensor on the bus has a conversion in progress - it is not possible to address a particular device. In fact I can't see from the data sheet any way of checking the status of one sensor, including the approach taken by the original function!

current code does select the address first, so it is per sensor.
would "return scratchPad[0] + scratchPad[1];" work?  I'm about to try it, was just doing research first and saw this.  basically, if the first 2 bytes are both zero it's zero, otherwise it's truthy

Adding (or ORing) scratchPad[0] and scratchPad[1] would still fail at 0
degrees C; whether that was a problem would depend on the application.
I've been running more tests. It seems that isConversionAvailable()
returns TRUE if there has ever been a conversion completed on the
device, it says nothing about whether the most recent conversion request
has completed.
The library header file used to define another function
isConversionComplete() but there was no code for this function in the
library and the definition was removed by commit 45db7e7 on 2014-04-10.
I've added the following:
bool DallasTemperature::isConversionComplete()
{
uint8_t b = _wire->read_bit();
return (b == 1);
}
and this works as I would expect, it returns FALSE for as long as there
is a conversion in progress on the bus. I don't have enough spare
hardware to test it with multiple sensors on the bus but I would expect
to only return TRUE when all conversions were complete.

I'd just like to confirm the problems with isConversionAvailable() and the success of using the isConversionComplete() code above with a single device.
Thanks so much

I've never been entirely certain why the body of that method was removed. It must have been a mistake. Thanks for your comments

I've restored isConversionComplete for the time being. However, it sounds like isCompleteAvailable may has a bug that needs investigating

I came here checking if anybody else is having problems with the temperature blocking code, and it seems I am not alone.
Here is what I can show (non-parasite power, one sensor, leaving all optional behavior at its default):

Set resolution to 12 bits
Call getTempC() which reads scratch and maps into float.  You get the most recent temp (Temp1) reading (sitting in scratch).
Use micros() to time a call to requestTemperaturesByAddress.  It returns too soon.
Call getTempC() which reads the old temp (Temp1).  The temperature calculation is still in progress.
Repeatedly call getTempC() until it changes.  If it does change, it takes about 70 ms (datasheet says max is 750ms).

I believe the proper way to poll for conversion on a self-powered bus is:
_wire->reset();
_wire->select(target);
_wire->write(STARTCONVO, parasite);
if (parasite) {
    delay(maxConversionTime(resolution)+1); // +1 for slop
    _wire->depower(); // write(*, true) left bus power on
} else {
    while (_wire->read_bit() == 0) {
        // Somebody is still busy
        // yield() here?
        // timeout at 1.1 * 750ms (recover from errant sensor)
    }
}

I think isConversionAvailable is completely bogus.  The comment leads me to believe it is a misunderstanding of the read time slots:
bool DallasTemperature::isConversionAvailable(const uint8_t* deviceAddress){

    // Check if the clock has been raised indicating the conversion is complete
    ScratchPad scratchPad;
    readScratchPad(deviceAddress, scratchPad);
    return scratchPad[0];

}

The comment sounds like it means to implement "_wire -> read_bit() != 0", but is reading the scratchpad instead.
BTW, this is also why I get an 85C at startup (powerup).  The scratchpad init temp is 85.0C.  The getTempC() call is run before the first temperature conversion is complete, and the init temp is returned.
The following client code correctly blocks (not for parasite power, though):
    dalTemp.setWaitForConversion(false);
    dalTemp.requestTemperaturesByAddress(addr); // takes 19ms
    while (!dalTemp.isConversionComplete()) // takes 80ms
        yield();
    float temp = dalTemp.getTempC(addr);

I can/will probably clean up the code now that I see that other people are having problems/concerns with the temperature blocking.
Also, while I only have one temperature sensor, I believe this would also apply to multiple sensors.  If reset / skip / write(STARTCONV0) is run, each (every) sensor will pull the bus low in response to the read time slots until they each have completed their conversion.  Once the last (slowest) has completed, it will no longer pull the bus low, and isConversionComplete() will return true.
Comments?

I went crazy and did a lot of cleanup.  I forked this repo and pushed back my changes into my github account (first fork on github!).  There are many commits but each should be easily digestible.
A few thoughts:

Is CheckForConversion feature (true/false) useful?  My 12-bit sample takes about 70ms when polling, but worst-case time is 750ms.  Seems like there would be no reason to disable CheckForConversion, and removing it would eliminate/simplify some code.

There are a few places where _wire->reset() is called after the command is complete.  It seems to only waste time since every command always starts with a reset() anyway, and I don't think devices are any "better off" after a reset.

If bus power is used, and check for conversion is always true, then since parasite == false, the maxBitResolution is unnecessary.  Thus, begin() doesn't need to poll bitResolution if !parasite.

The crc check (crc(buf, count-1) == buf[count-1]) can be alternately written (crc(buf, count) == 0) which makes for more readable code.  I didn't touch any of those - there are a few instances.

@milesburton - Let me know if you want to see any of this split out into separate independent commits, create a separate issue, or whatever.

@vallon Sorry the epic delay. Do you want to create a PR and I'll go through it?

Some good changes on your fork. I've raised a PR https://github.com/milesburton/Arduino-Temperature-Control-Library/pull/103/files
Will need to review the changes given the time that's elapsed but it's useful to see the comparison.


Incorrect project tab names

Type: Bug

open several vscode with different projects (3-5)
merge all windows to have possibility view it in tabs
select middle tab
then select "window->new tab"

Expected result:
New tab, which appears near the tab selected in step (3) marked as new tab and other tab names has relevant naming
Actual result:
The naming of tabs are incorrect and system will recognize that the tab was added in the end of tabs
VS Code version: Code 1.86.0 (05047486b6df5eb8d44b2ecd70ea3bdf775fd937, 2024-01-31T10:29:15.765Z)
OS version: Darwin x64 22.6.0
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz (12 x 2600)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
4, 3, 2

Memory (System)
16.00GB (2.70GB free)

Process Argv
--crash-reporter-id c263b599-21a2-4dd5-bb82-d029c928e195

Screen Reader
no

VM
0%

Extensions (58)

Extension
Author (truncated)
Version

Bookmarks
ale
13.4.2

vscode-sqlite
ale
0.14.1

vscode-django
bat
1.15.0

vscode-raml
blz
3.0.1

doxdocgen
csc
1.4.0

vscode-notes
dio
1.2.1

githistory
don
0.6.20

python-environment-manager
don
1.2.4

python-extension-pack
don
1.7.0

xml
Dot
2.5.1

gitlens
eam
14.8.0

copy-relative-path-and-line-numbers
ezf
0.3.1

vscode-diff
fab
1.4.2

copilot
Git
1.160.0

copilot-chat
Git
0.12.2

vscode-pull-request-github
Git
0.80.1

go
gol
0.41.0

gc-excelviewer
Gra
4.2.58

CppSnippets
har
0.0.15

jbockle-format-files
jbo
3.4.0

better-cpp-syntax
jef
1.17.2

vsc-python-indent
Kev
1.18.0

lldb-vscode
lan
0.2.3

rainbow-csv
mec
3.11.0

git-graph
mhu
1.30.0

theme-monokai-pro-vscode
mon
1.2.2

compare-folders
mos
0.24.2

vscode-docker
ms-
1.29.0

debugpy
ms-
2024.0.0

isort
ms-
2023.10.1

python
ms-
2024.0.1

vscode-pylance
ms-
2024.2.2

jupyter
ms-
2024.1.1

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

cmake-tools
ms-
1.17.15

cpptools
ms-
1.18.5

cpptools-extension-pack
ms-
1.3.0

makefile-tools
ms-
0.8.22

sqltools
mtx
0.28.1

autodocstring
njp
0.6.1

indent-rainbow
ode
8.3.1

sqlite-viewer
qwt
0.3.13

vscode-thunder-client
ran
2.17.8

vscode-yaml
red
1.14.0

synthwave-vscode
Rob
0.1.15

code-spell-checker
str
3.0.1

vscode-make
tec
1.0.2

cmake
twx
0.0.17

vscode-lldb
vad
1.10.0

intellicode-api-usage-examples
Vis
0.2.8

vscodeintellicode
Vis
1.2.30

gitblame
wad
10.9.0

vscode-todo-highlight
way
1.0.5

jinja
who
0.0.8

markdown-all-in-one
yzh
3.6.2

(5 theme extensions excluded)

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

With 1.86.1 when I'm trying to execute step (4) it even crash vscode entirely

The same result with 1.86.1


fix: S3 file listing if the prefix resolves to slash

:loudspeaker: Type of change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring

:scroll: Description

See Motivation and Context
:bulb: Motivation and Context

fixes: https://github.com/awspring/spring-cloud-aws/issues/900
:green_heart: How did you test it?
By using the implementation against a S3 bucket. see Motivation and Context
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

Thanks @klopfdreh


Database Information: split into usage and info tables (like server information / usage)

Alternative ?
UPDATE custom_environment_usage
SET entity_id = du.id, entity_kind = 'DATABASE_USAGE'
FROM custom_environment_usage as foo
 JOIN database_information_1_33 old_db ON (old_db.id = foo.entity_id)
 JOIN database_information new_db ON (old_db.external_id = new_db.external_id)
 JOIN application a ON old_db.asset_code = a.asset_code
 JOIN database_usage du ON new_db.id = du.database_id
WHERE foo.entity_kind = 'DATABASE';


[FEATURE] Supporting Aggregate Observations by Geometry

Issue Description
Difficulty: Intermediate
Time Estimate: 8-12 hours
Description: The goal is to write functions that can take individual or household level observations and aggregate them by country-level or state-level geometries. This requires identifying each observation with a specific geometry, then applying statistical methods to summarize the data. The challenge lies in handling missing values, outliers, and ensuring accurate aggregations.
Requirements
Expected Outcomes:

Accurate aggregated statistics (mean, median, k-means) for each country or state.
Improved data summarization capabilities for future analysis.
A robust and scalable solution that can handle large datasets.

Hey @00krishna , could you help me break this out further based on our discussions over Slack lately? Feel free to edit the original issue. :)


Added grpc ssl config

Added ssl conf for grpc service with port and ssl credentials
Added end point for grpc services to be able to run different services on different ports

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/


GBIF check at genus-level doesn't work

The GBIF check at genus-level doesn't seem to work
genus <- as.vector(tax_table(physeq_insekp_annelida)[, "genus"])
genus <- genus[!grepl("_otu|_XX_unclassified|_unclassified|_sp\\.|\\bsp\\..*|swarm\\d+", genus)]
genus <- gsub("_", " ", genus)
genus <- unique(genus)
source(here::here("source/r/check_presence.R"))
gbif_check <- check_presence(genus)
gbif_check %>%
filter(!present) %>%
  kable(caption = "Not present according to GBIF in Europe")

because it returns genera that have been detected in Europe before:
Not present according to GBIF in Europe
present	scientific_name
FALSE	Achaeta
FALSE	Lumbricus
FALSE	Fridericia
FALSE	Cognettia
FALSE	Buchholzia
FALSE	Henlea

The below code at species level does however seem to work:
species <- as.vector(tax_table(physeq_insekp_annelida)[, "species"])
species <- species[!grepl("_otu|_XX_unclassified|_unclassified|_sp\\.|\\bsp\\..*|swarm\\d+", species)]
species <- gsub("_", " ", species)
species <- unique(species)
source(here::here("source/r/check_presence.R"))
gbif_check <- check_presence(species)

@silkedebacker kan je hier de meest recente versie van het source/r/check_presence.R script posten?

check_presence.zip


Update sys-dm-database-backups-azure-sql-database.md

Friendlier resultset that is easy to consume

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@SudhirRaparla - Could you review this proposed update to your article and enter #sign-off in a comment if it's approved for merge?
Thanks!
#label:"aq-pr-triaged"

@sqlworldwide Thank you for your contribution! This is fantastic. I've made suggestions for formatting.

Thank you @rwestMSFT for suggesting the changes. @sqlworldwide sorry for the delayed response. Can you please include Randolph's changes and we should be good to go.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rwestMSFT thank you for your suggestions. I did not get Microsoft's formatting right (especially with space counts). Feel free to change with the standard.

@SudhirRaparla I reached out to @sqlworldwide privately and we’re going to do a PR on his local fork and then push here again. Once that’s done then we can sign it off.

@sqlworldwide : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@rwestMSFT: I'm sorry - only the author of this article, @SudhirRaparla, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@SudhirRaparla, you can sign off at any time now. @sqlworldwide, we'll merge this in when the content freeze is lifted.

#Sign-off

Thank you @rwestMSFT and @sqlworldwide for your contribution.


Remove unmaintained docs/getting-started-guides/coreos/coreos_multinode_cluster/

A user linked to this guide from an issue: http://kubernetes.io/docs/getting-started-guides/coreos/coreos_multinode_cluster/
I believe that guide is unmaintained.  It still specifies k8s 1.1.2.
@philips can we delete?

Thank you. Yes, this guide was unmaintained and community contributed. I am
OK with this change.
On Mon, Nov 21, 2016 at 2:50 PM devin-donnelly<EMAIL_ADDRESS>wrote:

Closed #1688
https://github.com/kubernetes/kubernetes.github.io/issues/1688 via
c4914ef
https://github.com/kubernetes/kubernetes.github.io/commit/c4914efbf5f032f5b46357141051d35e2a70b9a9
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/issues/1688#event-867368149,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACDCBZlQW0rQMsH3780hfd8UmjMXcdsks5rAiAfgaJpZM4KysYa
.


feat(logs): add tail flag

BREAKING CHANGE

changed -t for time to be -T so -t can be for tail. I tried to do -f, but it's already taken globally by --file.

Features

ads the ability to tail logs
works with all previous options

Todos (can be merged without these features)

if stream is broken, request from helmit for new containers, because the container we were listening too most likely died. This will cause the stream to break. However all the user has to do is rerun the command and all will be well again, as helmit, will find the new containers.

I believe there is potentially a way to achieve this functionality without adding any code to harbor-compose. We could put this combined streaming logic into helmit, so that any UI can benefit from the ability to stream logs from all containers at once.

closes #11

Awesome!  Can you get logs by container id? harbor-compose logs 1234

No. figured that's a separate feature that should be implemented in a different PR.

Ok. LGTM


Questions On Separate Page 4

As a user
I want to have all the existing questions on a second page
So that i have a better overview of the questions

Description:

AC:

[ ] /questions Page
[ ] All Questions get displayed on the questions page

Tasks:

[ ] Create new branch
[ ] Create new page
[ ] Create route for page
[ ] Create link to the route
[ ] Append link to the Navbar with an icon
[ ] Import the QuestionComponent into the Question-Page
[ ] Move the dummy array into the Question-Page
[ ] Render when loading the page with useEffect

Complexity:
M
Previous Story
#3 #10

Hey Laurenz, this looks very good :)
One little suggestion: In the AC maybe change the first point to i.e. "routing to questions Page". I think that is more clear. But that is up to you ;)

Hey Laurenz, this looks very good :) One little suggestion: In the AC maybe change the first point to i.e. "routing to questions Page". I think that is more clear. But that is up to you ;)

Thanks for the feedback. I made some adjustments :)


Switch to only supporting Rails 4.

Also updates the Travis CI configuration to the new, faster version and changes Ruby 2.1.1 to 2.1.5.

Note this isn't the final solution. Rolling back is still an issue.

No way for me to easily update this branch with a fix now, so I'm closing it. I'm up for fixing it in a separate pull request if there's a desire for it.

Not being able to delete my branches doesn't bother me.
Not being able to delete my branches doesn't bother me.
Not being able to delete my branches doesn't bother me.


Statistics Reset

add feature to reset (delete) the statistics table.
To achieve this, we've added a new method to db class: truncateTable($table) --> see https://github.com/YaWK/yawk.io/issues/297

done.


Initiated system management support

This patch adds systemd support for running OpenBazaar as a daemon. Has bug that config file is not found (because systemd does not run ob from within the repo directory). This bug is unrelated to systemd support however.

I'm not that familiar here if someone else wants to take a look at this. Also @tcharding I don't think you want your home folder hard coded in there.

I think I like the concept. I think many people will still want/need custom configurations for these things, but we could perhaps provide baselines that works for most people.
Would the idea be to provide scripts/support for a bunch of common systems, or just one? A lot of people still prefer to use upstart. I use personally am using Docker on CoreOS. How do we decide which things to include support for?
I think the main issue I see is that to run OB well requires quite a bit more setup/configuration than I think we might be able to reasonably provide here. E.g. it shouldn't run as root, files should have tight permissions, etc.
TL;DR Overall I like the idea, but think we should think more about how we want to execute on it.


Keep original case of non-numbers

I don't think it was intended to change the case of the words so I fixed it in this PR. If someone wants the string to be lowercase, then I think it should be done by the user itself :-)

Looks good to me! Thanks for catching this :)


fix key in chain

Merged another PR that should fix this issue. Please reopen if that's not the case. Thanks!


Option to skip authentication if no credentials provided

Kind of specific to my development experience case but sometimes it's useful to just skip authentication if no credentials are provided instead of raising an error. Not sure if someone else needs this :/
If just one of credentials is set I intend to raise an error, because, well, it's strange to skip authentication with just missing password or username.
As i'm not experienced in elixir/plug/phoenix any suggestions are more than welcome =)
(didn't bump version in mix.exs)

@vemperor hey, isn't it easier to just have something like
if System.get_env('BASIC_AUTH_USERNAME') do
  plug BasicAuth, ...
end

@nashby probably, but

You have to duplicate ENV variable names in case of plain System.get_env.
Оr use trickery if application config uses {:system, "BASIC_AUTH_USERNAME"}.

Sorry, we haven't commented on this yet. Thanks for the PR @vemperor.
I'm interested to know what you circumstances are that you want this? (My guess is that it is something along the lines of you want to secure a staging deployment but not the production deployment.)
@nashby I think the main issue with conditionally including the plug is that it gets compiled in so reflects the compilation environment rather than the deployment environment.

@paulanthonywilson 2 cases:

We (me and my colleagues at work) are used to configure application via ENV as much as possible, because we don't even know who and where will setup the application (exploitating guys don't know a thing about elixir - they are not programmers).
Every opensource project should be easy to configure to use basic auth via non-programming-language specific means (like in the first case), e.g. ENV variables or plain config. I prefer ENV variables =)


📃 Property dictionary support

Refer to the contributor docs to learn how to generally add new properties to Pandora!

Similar to https://github.com/bitbrain/pandora/issues/54 users may want to define a list of data, but in a dictionary fashion, e.g. have a key/value style data type Dictionary.
Workflow

User can add new property of type 'dictionary'
upon clicking, a popup appears to specify the types for both key and value (can be anything except array or dictionary)
the Variant will always be of type Dictionary (and stored Pandora type is "dictionary")
the types can be changed inside the settings (which may clear existing values if incompatible)

Example use cases

recipe ingredients (e.g. quantity + reference)
material composition (e.g. percentage + reference)

Good

Awesome! I came here to request exactly this feature.
I ran into this issue when trying to create loot drop / recipe quantities.
By the way, this is an amazing system. I really like the integration of CastleDB into Heaps.io, however the lack of information made it difficult to continue to develop in that framework. This was the missing puzzle piece for the switch to Godot.


Add new thread variable __jmv_THREAD_START_TIME_ITERATION contains the currentTimeMillis when the thread iteration start.

Corresponding to the issue (new feature): https://github.com/apache/jmeter/issues/6319
Description
Add a new unmodifiable JMeter Thread variable named "__jmv_THREAD_START_TIME_ITERATION"
Motivation and Context
It is interesting to know the start timestamp of a thread iteration to calculate cadences or pacing.
This variable can also be used to help other Thread Groups calculate cadences (E.g : Concurrency Thread Group or may be Open Model Thread Group)
How Has This Been Tested?
Yes, add a groovy sampler first sampler of the thread group and compute the delta between the value in the variable __jmv_THREAD_START_TIME_ITERATION and the current time System.currentTimeMillis() in the groovy sampler.
Groovy code
+++++++++++++++++++++++++++
long currentTime = System.currentTimeMillis();
long startThreadIter = Long.parseLong(vars.get("__jmv_THREAD_START_TIME_ITERATION"));
long delta = currentTime - startThreadIter;
return delta;
+++++++++++++++++++++++++++
Results 0 or 1 ms
Screenshots (if appropriate):
Types of changes

New feature

Checklist:

[x ] My code follows the code style of this project.
[ x] I have updated the documentation accordingly.
Add new variable __jmv_THREAD_START_TIME_ITERATION in the documentation :
functions.xml, section 6 Pre-defined Variables

I don't think, that this can be used by OMTG, as the thread is started by OMTG itself and not initialised, when that information is needed. But that problem is hopefully addressed by #6353 .
Apart from that, the information might be helpful for listeners or users, so LGTM.


fix: get_call_graph TypeError

modalbetatesters.slack.com/archives/C031Z7H15DG/p1678322711034919
Looks like there's an unhandled state, TERMINATED, which is triggering _missing_, and that doesn't have the @classmethod annotation so it's not passing cls as first arg.
I checked some relevant inputs and they do indeed have state = 3. I think that this state might be new for inputs, by way of input cancellation? cc @freider.

Yeah inputs that are cancelled by the user are marked as TERMINATED. Actually realizing now that we should probably use the call graph to automatically cancel any subcalls made by cancelled function calls as well!


Re-enable TF GPU device

This change causes TF CPU to stall. Will look into it.

This is now working. PTAL

Thanks for the explanation.
For 1, we would prefer to have accuracy verified for each benchmark run so decoupling is not ideal.
For 2, did you look into having the process return results? Something like this: https://stackoverflow.com/questions/10415028/how-to-get-the-return-value-of-a-function-passed-to-multiprocessing-process


Wrong age shown for message

Describe the bug
When clicking on the lower right chat button on https://papercups.io/ so that the widget pops up, I get the following message:

Hi there! Send us a message and we'll get back to you as soon as we can. In the meantime, check out our demo!
Papercups · Sent 60 minutes ago

Expected behavior

Papercups · Sent 60 minutes ago

should be

Papercups · Sent 1 second ago

Screenshots

Additional context
My TZ is Europe/Berlin and we've moved from DST (CEST = UTC+2) to normal time (CET = UTC+1) on 25. October 2020.

thanks for flagging this @danielfaust! this is a known issue (https://github.com/papercups-io/chat-widget/issues/62)

hey @danielfaust! i think this should be fixed now -- can you double check for me and let me know if I can close this?

Hi @reichert621 it's fixed. Great job!

glad to hear it, thanks for flagging this!


finish btrfs persistance

https://mt-caret.github.io/blog/posts/2020-06-29-optin-state.html

Cannot open /etc/machine-id no such file or directory

gkr-pam: unable to locate daemon control file??


Fix uploading to HF proxy

Fix bug reported in https://github.com/huggingface/huggingface_hub/issues/2114 (cc @padeoe).
Can't/don't want to add a proper test for it but it works fine when testing it locally with the snippet shared in #2114.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Thanks for the quick review @LysandreJik :)


Should be no instance resolution if server, instance and port are specified

When specifying server, instance and port (both URL and ADO connection string format) the connector tries to list the server instances while the information should be sufficient to connect directly to the instance. In the situation where it is impossible to enumerate the list of servers this component doesn't work.
Connection string formats are as follows:
sqlserver://uid:pwd@<IP_ADDRESS>:50000/instance?database=db
server=<IP_ADDRESS>\instance;port=50000;database=db;user id=uid;password=pwd
I can see from the code that in the method "connect" in "tds.go" it is checked that if the instance is not empty then the connector tries to list all instances.

To Reproduce

import (
	"database/sql"
	"log"
	"time"

	_ "github.com/denisenkom/go-mssqldb"
)

func main() {
	cstr := "server=srv\\inst;port=50000;database=db;user id=uid;password=pwd"

	c, err := sql.Open("sqlserver", cstr)

	if err != nil {
		log.Fatalf("unable to open connection to database: %q", err)
	}

	defer c.Close()

	r := c.QueryRow("select getdate() as dt")

	var dt time.Time

	err = r.Scan(&dt)

	if err != nil {
		log.Fatalf("unable to get data from db: %q", err)
	}
}

Expected behavior
If server, instance and port are specified then the connector should directly try to connect the database (at least Management Studio is able to do that).
Further technical details
SQL Server version: Microsoft SQL Server 2016 (RTM) - 13.0.1601.5 (X64)
Operating system: Windows Server 2016 Datacenter 6.3  (Build 14393: ) (Hypervisor)

Hi @raufismayilov, I'll look further into this.

I have just fixed this in PR https://github.com/denisenkom/go-mssqldb/pull/543#issue-344604995


fix: filter report before applying changes

Description

Filter report when the report is generated instead of filtering it at the end.

/rtm


Merge incoming po files, instead of overwriting

changed i18n.smartling_download() logic - now, when we get PO files from Smartling, instead of overwriting the existing files, we merge any changes from the new files into the existing ones

important to merge the changes, because we'll have PO files for the same language coming from (at least) two different projects, each with their own subset of unique strings; just overwriting the files means deleting those unique string translations

added polib library to support PO file handling
fixed db string extraction to properly escape double-quotes (double backslashes become a single backslash when uploaded to Smartling), to avoid issues with polib during file parsing
minor fixes and cleanup with general Smartling upload/download process
updated POT, fr_CA PO, and fr_CA MO files

Hello @OptimusRhine! Thanks for updating the PR.

In the file portal/models/i18n.py, following are the PEP8 issues :

Line 147:41: E231 missing whitespace after ','
Line 149:29: E231 missing whitespace after ','
Line 149:33: E231 missing whitespace after ','
Line 156:55: E502 the backslash is redundant between brackets
Line 157:29: E128 continuation line under-indented for visual indent
Line 176:22: E231 missing whitespace after ','

Looks great @OptimusRhine.  Would it be worth adding some tests?

@pbugni looks like Ivan's actually got some TOKENENV=translations tests in place - in fact, you can see where they failed (as they should have) in the breaking into smaller functions commit, above

good point - my grep was obviously lacking!

@OptimusRhine I started a review after you requested but you fixed some of my comments as I was writing them!


[ERROR] Microk8s for Kubeflow

https://www.kubeflow.org/docs/other-guides/virtual-dev/getting-started-multipass/
Throws below error on step: kubeflow-tools/install-kubeflow.sh
+ pushd /home/multipass/mykubeflow/ks_app
~/mykubeflow/ks_app ~/mykubeflow
+ ks param set application name mykubeflow
+ [[ false == true ]]
+ ks param set application components '["ambassador","jupyter","notebook-controller","jupyter-web-app","profiles","centraldashboard","tf-job-operator","pytorch-operator","spartakus","argo","pipeline","katib"]'
+ ks show default -c metacontroller -c application
ERROR find objects: lib/ksonnet-lib/v1.15.4/k8s.libsonnet:17681:93-94 Unknown variable: x

          "withX-Kubernetes-Embedded-Resource":: self + { "x-kubernetes-embedded-resource": x-kubernetes-embedded-resource },

Related:

https://github.com/kubeflow/website/issues/1113
https://github.com/kubeflow/website/issues/1163

Let's track the work on the first-raised issue:
https://github.com/kubeflow/website/issues/1113
/close


Broken link

"Social C Sample AsyncIntegration.cpp" is broken.

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

ID: b74a8977-3a86-719c-01a9-0d8ef98d1504
Version Independent ID: 2147a655-135c-5c30-1be6-97aa6b7aa678
Content: Asynchronous C API calling patterns - Windows UWP applications
Content Source: windows-apps-src/xbox-live/flatc-async-patterns.md
Product: uwp
GitHub Login: @KevinAsgari
Microsoft Alias: kevinasg

Hi, @jonaserikssonnet, and thank you for filing this issue (I just noticed it, so I did some investigation).
I've hunted for the "Asynchronous C API calling patterns - Windows UWP applications" topic, and for the original source file, and I can't find them. It looks like the topic has been removed or relocated somewhere beyond our reach.
This issue is over a year old, so that might account for the topic being missing. I'll go ahead and close this now in light of all that.
If you find a problem with an extant topic then feel free to leave a comment on that one.
Thanks again!
Steve


Add metrics to file upload process

I've thrown the kitchen sink at this for now so that every part of the file upload process it measured (and with the same buckets and same lack of labels).
When we see how it's actually performing, we may decide to customise the labels and buckets for some of the parts (or even remove them)


[5.1] [Runtime] Short-circuit search for context descriptor for protocol extensions

When looking for a context descriptor for a protocol extension, we
search based on the mangled name--"x", a generic type parameter with
depth and index 0--and are guaranteed to fail because there is no such
concrete type. However, _findContextDescriptor will fail very slowly,
spinning through all of the types and conformances in all of the
loaded images. Moreover, negative results aren't cached, so this can
happen repeatedly.
Short-circuit _findContextDescriptor when it receives a dependent
generic type parameter type, avoiding the expensive search when it
will find nothing.
Potential fix for rdar://problem/53560010.

@swift-ci please test

This is a good fix, but we also expect that we need the caching. Closing out this pull request so we can address the whole problem in one PR.


Rules "false points" should be all painted in grey

(Skope) rules should be painted in blue / grey far all points in space, not only for points in selection

solved


r-ryantm: first aid kit

Motivation for this change
https://discourse.nixos.org/t/improve-a-pkgs-tests-for-nixpkgs-update/9530
TODO
Add actionable test examples.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/improve-a-pkgs-tests-for-nixpkgs-update/9530/7

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/improve-a-pkgs-tests-for-nixpkgs-update/9530/1

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/improve-a-pkgs-tests-for-nixpkgs-update/9530/15


Relative path for for assumeFilename not working

Hi all,
I am trying to set a relative path for assumeFilename, but it does not work.
For example the .clang-format file is in the .vscode folder.
My current settings are:
"clang-format.style": "file",
"clang-format.assumeFilename": "${workspaceFolder}/.vscode/.clang-format"

Also tried using ${workspaceRoot} to no avail.
An absolute path to the same file works!
My current vscode version is 1.85.1
Is a relative path for assumeFilename supported?
Thanks in advance

this is a feature that im going to add hopefully in the next few hours

https://github.com/Mr-Bossman/vscode-clang-format/commit/9e8f068605cc992e241d7f9ee2fc37fa5894f0c3
Oh for assumeFilename? I don't know how that option works there arent good docs. is it the same as stye?

Thank you for working on this..
It is an old comment but I think what I was trying to do at the time was have a relative path to my .clang-format file.
The file resides in "${workspaceFolder}/.vscode/.clang-format"
I think the way to do this is through assumeFilename ? Is there an alternative way?

you are suposed to put ur clang-format file in the "clang-format.style"  option like so "clang-format.style": "file:${workspaceFolder}/.clang-format", with my changes or "clang-format.style": "file:/home/jesse/code/vscode-clang-format/tests/.clang-format", for the extension without my changes

Ok then, your changes should solve my issue.. Thank you!

@filip-os  https://marketplace.visualstudio.com/items?itemName=Something-Software.clang-format-2


Fix the time of the cron job.

Currently the cron job is scheduled to run twice a day via {rand_minute} */2 * * *.  This is somewhat random depending on when the VM is started up or reset, and impacts user processes and troubleshooting because the server will restart.  So, it would be extremely nice to fix this to a static time.

The server restart is being fixed in issue #1396.  So, perhaps that mitigates the need for the cron job to be static.


Is q ever used?

Hi, I am very interested in using this method with my own sentence encoder, so I have been going through the code/paper line-by-line to understand it. I noticed that this variable is computed but never used:
https://github.com/BinWang28/SBERT-WK-Sentence-Embedding/blob/ca65d43cbed637f2a7129669110782e9a6f7fe3b/utils.py#L118
Any reason to keep it around or is this just a typo?

Hi, John,
It is never been used. Feel free to remove it! :)


Missing Languages

I do not see a languages section

I have added a languages section to our wiki page

I would clarify the issue a little bit more!


Add changelog for the v0.11.2 release

What this PR does / why we need it:
Add changelog for the v0.11.2 release.
Does this PR introduce a user-facing change?:
NONE

/assign @kron4eg

/retest


Drop appveyor in favor of travis and increase minimum python version to 3.6

This drops appveyor for obvious reasons and raises the minimum python version in prep for the first release since python 3.8 is almost out.
And I really like f-strings....

Merging; the appveyor failure is because there is no setup file 😀


Missing EndpointUrl in telemetry format

Describe the bug
With opcuaPublsher:2.8.0, the telemetry output (json) does not match the online documentation.
Moreover, changes in the command line option --mm has no effect on the telemetry messages.
To Reproduce

Run a plc simulator with docker run --rm -it -p 50000:50000 -p 8086:8086 --name opcplc mcr.microsoft.com/iotedge/opc-plc:latest --pn=50000 --autoaccept --sph --sn=5 --sr=10 --st=uint --fn=5 --fr=1 --ft=uint --ctb --scn --lid --lsn --ref--gn=5
Deploy the opcuaPublisher as explained in this tutorial
Don't use the option --mm!
Add routing to $upstream
Configure the pn.json file with some nodes:

[
  {
    "EndpointUrl": "opc.tcp://<IP_ADDRESS>:50000",
    "UseSecurity": false,
    "OpcNodes": [
      { "Id": "ns=2;s=StepUp" },
      { "Id": "ns=2;s=AlternatingBoolean" },
      { "Id": "ns=2;s=RandomSignedInt32" },
      { "Id": "ns=2;s=RandomUnsignedInt32" }
    ]
  }
]

Adapt the ip address according to the location of the plc simulator.

The telemetry messages will show up as follow
[
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=StepUp",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"Value": {
			"Value": 560934,
			"SourceTimestamp": "2021-09-24T05:56:12.202727Z"
		}
	},
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=RandomSignedInt32",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"Value": {
			"Value": -1001090625,
			"SourceTimestamp": "2021-09-24T05:56:12.2029103Z"
		}
	},
        ...
]

Expected behavior
The messages should look like those in the documentation page "https://docs.microsoft.com/en-us/azure/industrial-iot/reference-opc-publisher-telemetry-format".
The documentation page does not report which messaging model is used as default. It looks like "Samples" format but

the EndpointUrl is missing
the MonitoredItem is not there neither
the node id has an extra prefix and does not match the one in pn.json

Moreover, if you run the publisher with --mm=Samples or -mm=PubSub the output does not change.
Desktop (please complete the following information):

OS: Ubuntu Server 18
Version iotedge 1.1.4 and opcuaPublisher 2.8.0

Hello @cristipogacean
thank you for the reply and for pointing out the good documentation. Yes, we gave it a second try.
With the option (extracted from the deployment manifest"
\"Cmd\": [\"--autoaccept\", \"--logfile=logs/opcuaPublisher.log\", \"--publishfile=workspace/nodes.json\"]

we get messages like these
[
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Value": {
			"Value": true,
			"SourceTimestamp": "2021-09-28T15:16:58.9037588Z"
		}
	},
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Value": {
			"Value": false,
			"SourceTimestamp": "2021-09-28T15:17:03.9025818Z"
		}
	},
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Value": {
			"Value": true,
			"SourceTimestamp": "2021-09-28T15:17:08.9023353Z"
		}
	}
]

and we deduce that the defaults are "Samples" messages without the "full featured payload". They are both the default behaviour.
With the options
\"Cmd\": [\"--autoaccept\", \"--logfile=logs/opcuaPublisher.log\", \"--publishfile=workspace/nodes.json\", \"--fm=true\"]

we see
[
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"EndpointUrl": "opc.tcp://<IP_ADDRESS>:50000/",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Timestamp": "2021-09-28T15:21:19.4990399Z",
		"Value": {
			"Value": true,
			"SourceTimestamp": "2021-09-28T15:21:18.903539Z",
			"ServerTimestamp": "2021-09-28T15:21:18.9035915Z"
		},
		"SequenceNumber": 12,
		"ExtensionFields": {
			"PublisherId": "opc.tcp://<IP_ADDRESS>:50000_e873ab7d8040cb3b79e7b4a38296093f97c32da3",
			"DataSetWriterId": "opc.tcp://<IP_ADDRESS>:50000_c70079e5431fea7bfa4f9fbba7f2aad440d55efb"
		}
	},
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"EndpointUrl": "opc.tcp://<IP_ADDRESS>:50000/",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Timestamp": "2021-09-28T15:21:24.5207237Z",
		"Value": {
			"Value": false,
			"SourceTimestamp": "2021-09-28T15:21:23.9036481Z",
			"ServerTimestamp": "2021-09-28T15:21:23.9039577Z"
		},
		"SequenceNumber": 13,
		"ExtensionFields": {
			"PublisherId": "opc.tcp://<IP_ADDRESS>:50000_e873ab7d8040cb3b79e7b4a38296093f97c32da3",
			"DataSetWriterId": "opc.tcp://<IP_ADDRESS>:50000_c70079e5431fea7bfa4f9fbba7f2aad440d55efb"
		}
	}
]

which means that the full featured messages is on. So far so good.
Last, with this option
\"Cmd\": [\"--autoaccept\", \"--logfile=logs/opcuaPublisher.log\", \"--publishfile=workspace/nodes.json\", \"--fullfeaturedmessage=true\"]

we get back to messages without full payload
[
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Value": {
			"Value": false,
			"SourceTimestamp": "2021-09-28T15:24:03.903729Z"
		}
	},
	{
		"NodeId": "http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean",
		"ApplicationUri": "urn:OpcPlc:b0803e843ad2",
		"DisplayName": "AlternatingBoolean",
		"Value": {
			"Value": true,
			"SourceTimestamp": "2021-09-28T15:24:08.9040022Z"
		}
	}
]

Is there a problem in the "long version" of the command line?
This is visible also if we use the long version of the --mm option, as I mentioned above. When I opened this issue I used the short for simplicity without knowing that there was a problem behind.
Moreover, i had a look to this source code and the default is told to be "True".

Is there a problem in the description too?
Thanks

Hello @alevincenzi,
Indeed I can replicate the issue with --fullfeaturedmessage=true. This seem to be a bug related to string lowercase comparison of the arguments name since the --FullFeaturedMessage=true works fine. As you said, the same issue might be the issue with other arguments you have been using.
We will fix this long-names issue as well as the default mismatch in the documentation of the  --fullfeaturedmessage.
Thank you for taking the time to go to the bottom of this issue!

Hello!
thank you for he update.
We'll be waiting for the next release. As for today, we are gong to use the command line options in short format.
Regarding the other open points about the payload,
The missing MonitoredItem is not a problem. I reported it just to list the differences with the documentation.
The expansion of the NodeId can be problematic for us. We use the DisplayName whenever possible and we put it in the configuration of the publisher (--pn=node.json). this way the publisher puts it in the telemetry sent as output. If the display name is not set by "our users", we use the nodeId to identify the opc node but with the 2.8.0 the nodeId does not match the one in the configuraton. It canges from ns=2;s=AlternatingBoolean to http://microsoft.com/Opc/OpcPlc/#s=AlternatingBoolean and the match fails. It would be nice to keep the one in the nodes file.
Thanks.

fixed in main branch


Several guidebook changes

Description

Fix all spelling mistakes (in edited guidebooks)
Improve Standard Operating Procedure
Add a ton of cool formatting to things
Fix inconsistencies in word casing

Changelog


Dynamic cluster createUiDefinition.json throwing error during testing at sandbox

Issue noticed:
arm-oraclelinux-wls-dynamic-cluster createUiDefinition.json is throwing following error when it is tested at Azure SandboxBlade.
"The element for [basics('basicsOptional').useSystemAssignedManagedIdentity] could not be found in the createuidefinition"
I think recent change "Add system assigned managed identity and ELK index cleanup (#91)" has caused the issue.
Probably following is causing the error.
"useSystemAssignedManagedIdentity" section is not under "basicsOptional" but it is referred as "useSystemAssignedManagedIdentity": "[basics('basicsOptional').useSystemAssignedManagedIdentity]"

at outputs section.
Currently live OHS base images skuUrnVersion update and testing is waiting for this fix.

Currently live OHS base images skuUrnVersion update and testing is waiting for this fix.

Hello @zhengchang907 , could you have a check, thank you!

Thank you @galiacheng!
Hi @sanjaymantoor , thanks for creating the issue. I've located the problem and fired a PR for it: https://github.com/wls-eng/arm-oraclelinux-wls-dynamic-cluster/pull/94

@sanjaymantoor Hi Sanjay, the PR is merged, can you please verify & close this issue?

Changes are merged and tested.


Add a signal handler

Q
A

Issue Type
Feature Request

Deployer Version
5.0.1

Local Machine OS
Ubuntu 16.04

Remote Machine OS
Ubuntu 16.04

Description
When I want to abort a deploy and press CTRL-C it stays locked and I have to manually unlock it. It would be great to handle this by adding a signal handler to remove the lock and maybe cleanup before abort.

I do not agree with auto unlocking, if you want that to happen why not just add unlock as your first step in the build ?

I don't think deployer can presume that unlocking is the best option in the case of an aborted deploy. Is there some sort of finally mechanism which might be used if users want to add auto-unlock?

// [Optional] if deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');

I think that's not run when you ctrl+c it, right?

Right

That requires that PHP is build with pcntl - Which is not enabled by default.
Otherwise if you @1ed have it in your PHP installation, you can create do it in your configuration file.
pcntl_signal(SIGTERM, function() {
   `dep deploy:unlock`
});

So this feature will be available only for those who have pcntl installed.

Thats the only way you can catch CTRL+C commandline command

And will never be available on windows

What about Bash on Windows?

Will be cool if someone can implement this feature.

Good question, if you can install php with pcntl extension, then maybe :)

Maybe this could be a inspiration
https://github.com/chEbba/ConsoleSignals


Merge python EnvironmentError, IOError and WindowsError into OSError.

Since Python 3.3 (PEP3151), EnvironmentError, IOError and WindowsError have been merged into OSError. This patch converts all instances of EnvironmentError, IOError and WindowsError into OSError.
Should fix the (unrelated) "Syntax Error" found by LGTM.

Thanks!


Draw Screen UI

@am-MongoDB, I left some TODO notes for you in DrawViewController.swift. I'll leave this PR open, and let you merge if you want to. UI is done, the TODO tasks are pretty simple.
Before:

After:

Thanks @rajohns08 – working on it now


refactor(ids)!: replacing incremental ids with auto-id

Overview

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: [ABC-123]
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.278-alpha.99a1f1.0@codecademy/gamut-labs@35.5.4-alpha.99a1f1.0@codecademy/gamut-patterns@0.9.2-alpha.99a1f1.0@codecademy/gamut@52.0.3-alpha.99a1f1.0@codecademy/monaco@0.7.3-alpha.99a1f1.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63e3dccdd33dd40ce1b944b1--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.278-alpha.f8ddfc.0@codecademy/gamut-labs@35.5.4-alpha.f8ddfc.0@codecademy/gamut-patterns@0.9.2-alpha.f8ddfc.0@codecademy/gamut@52.0.3-alpha.f8ddfc.0@codecademy/monaco@0.7.3-alpha.f8ddfc.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63e4023f721e051bf0e0f96a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.278-alpha.b825ee.0@codecademy/gamut-labs@35.5.4-alpha.b825ee.0@codecademy/gamut-patterns@0.9.2-alpha.b825ee.0@codecademy/gamut@52.0.3-alpha.b825ee.0@codecademy/monaco@0.7.3-alpha.b825ee.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.278-alpha.9ef07d.0@codecademy/gamut-labs@35.5.4-alpha.9ef07d.0@codecademy/gamut-patterns@0.9.2-alpha.9ef07d.0@codecademy/gamut@52.0.3-alpha.9ef07d.0@codecademy/monaco@0.7.3-alpha.9ef07d.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.279-alpha.a0c9fc.0@codecademy/gamut-labs@35.5.5-alpha.a0c9fc.0@codecademy/gamut-patterns@0.9.2-alpha.a0c9fc.0@codecademy/gamut@52.0.3-alpha.a0c9fc.0@codecademy/monaco@0.7.3-alpha.a0c9fc.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.280-alpha.a1604d.0@codecademy/gamut-labs@35.5.6-alpha.a1604d.0@codecademy/gamut-patterns@0.9.2-alpha.a1604d.0@codecademy/gamut@52.0.4-alpha.a1604d.0@codecademy/monaco@0.7.4-alpha.a1604d.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://63e534c5da2ab5102cfc3297--gamut-preview.netlify.appDeploy Logs


Sample for SSO between different applications using @azure/msal-angular

Please follow the issue template below. Failure to do so will result in a delay in answering your question.
Library

[ ]<EMAIL_ADDRESS>or<EMAIL_ADDRESS>[ ]<EMAIL_ADDRESS>
Documentation location

[ ] docs.microsoft.com
[ ] MSAL.js Github Wiki
[ ] README file
[ ] Other (please fill in)
[ ] Documentation does not exist

Description

Closing as duplicate of #2177.


Conflict with cms ajax

Hello! Help me please.
If i set form button id

in my site does't work ajax after i click Submit button

if i comment button id

ajax work fine!)

may be it's some trigger or event, but, how do I get around this?
Thank you for the plugin, it's great work!!

Hey sorry if took long to reply, are you still having this issue with the latest version?

Helllo!!!
This was the case for me, i comment code in plugin:
//node.on("submit", false);
//e.preventDefault();
//options.submit.callback.onSubmit ? _executeCallback(options.submit.callback.onSubmit, [node]) : submitForm();
//return false
ну и еще кое-что переделал под себя, чувтсвую себя как в танке!)

http://tuning-soft.ru/1c-bitrix/modules/demo/api.formdesigner/

Oh!.. Yea so the form validation removes any binding you have previously done on the form submission or the input submit click. You need to bind your for submit inside the callback:
typeof $.typeahead === 'function' && $.typeahead({
    ...
    callback: {
        onSubmit: function (node) {
           // Do your ajax call here
        }
    }
})

let me know if that helps


Only emit planned transform operation in project context

It is possible to register transforms in the context of a script, and those transforms might run on substituted included build dependencies.
Even though these transforms will never be part of any execution plan, calculating the node identity of them will currently fail, because they don't have an owning project.
This causes the Build Scan plugin to fail.
Therefore, this PR stops emitting the build operation in this case.

I split out some tests from #26131 and changed them so they document the current behavior. There are more dragons when not emitting any of the unplanned planned transform steps, so I'll keep the changeset smaller like in this PR.

I've triggered a build for you. Click here to see all failures if there's any.


Add jest as a peer dependency

Jest should be a peer dependency of the plugin. I'm not sure which range of versions should be supported though, but any version included should have the same interface in the part of the API that is being used.
Probably the easiest thing to do is to just use ^29.0.0.

Thank you so much for your help again @fvictorio 🙏🏻
I just added Jest as a peer dependency.


ChefDK 1.x using Ruby version with known Seg Fault issue

Description
After upgrading to ChefDK 1.5 we started noticing a significant number of segfaults when running our testing/automation scripts. After a bit of debugging it was determined that the version of ruby shipping with ChefDK 1.5 has a known seg fault issue in the GC.
https://bugs.ruby-lang.org/issues/12614
Unfortunately we cannot upgrade to ChefDK 2+ yet, as we cant use chef client 13. Would it be possible to release a patched version of ChefDK 1.5 that includes a fixed/later version of ruby?
EDIT: I just tried installing ChefDK 1.4.3, 1.3.43 & 1.0.3, all of which also had the same Ruby version, with the same error.
Thanks!
ChefDK Version
# chef --version
Chef Development Kit Version: 1.5.0
chef-client version: 12.21.2
delivery version: master (17c1b0fed9be4c70f69091a6d21a4cbf0df60a23)
berks version: 5.6.4
kitchen version: 1.16.0
inspec version: 1.25.1

# ruby --version
ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]

Platform Version
CentOS 6.8
Replication Case
Not sure this is necessary, because but the linked ruby-lang ticket has a test script that can replicate it. https://bugs.ruby-lang.org/issues/12614
class Foo
  GC.start
  n = 64 + 32 - 2
  z = 5
  h = {}

  c = 0
  begin
    name = "foo#{c+=1}"
    i = name.object_id % 128
    h[i] ||= []
    h[i].push name
    names = h[i] if !names || names.size < h[i].size
  end while names.size < n * z
  names_list = names.each_slice(n).to_a

  [1, n - 8].each do |q|
    names_list.each do |names|
      names = names[0, q]
      names.each do |name|
        define_method(name) {}
      end
      names.each do |name|
        remove_method name
      end
    end
  end
end

Stacktrace
Not sure if this is relevant, but:
[BUG] Segmentation fault at 0x00000000000000
ruby 2.3.1p112 (2016-04-26 revision 54768) [x86_64-linux]

-- Control frame information -----------------------------------------------
c:0003 p:---- s:0008 e:000007 CFUNC  :gets
c:0002 p:0059 s:0005 e:000004 BLOCK  script.rb:61 [FINISH]
c:0001 p:---- s:0002 e:000001 (none) [FINISH]

-- Ruby level backtrace information ----------------------------------------
script.rb:61:in `block (3 levels) in _request_from_json'
script.rb:61:in `gets'

-- Machine register context ------------------------------------------------
 RIP: 0x0000003699c79252 RBP: 0x00007faf380008b0 RSP: 0x00007faf3f402210
 RAX: 0x00000000000000d0 RBX: 0x76527951664d5a52 RCX: 0x0000000000000077
 RDX: 0x0000000000002010 RDI: 0x00007faf38000020 RSI: 0x00000000000000a0
  R8: 0x00007faf38000088  R9: 0x0000000000000001 R10: 0x00000000000000a0
 R11: 0x0000000000000206 R12: 0x00007faf38000078 R13: 0x00007faf38000020
 R14: 0x0000000000000000 R15: 0x00000000000000b0 EFL: 0x0000000000010287

-- C level backtrace information -------------------------------------------

NOTE: CHEFDK BUGS ONLY
This issue tracker is for the code contained within this repo -- chefdk.

Server issues
Chef-client issues
Cookbook Issues (see the https://github.com/chef-cookbooks repos or search Supermarket or GitHub/Google)

👍

I can't speak for the other maintainers, but I think we are unlikely to do this. Changing Ruby versions is a major project, and this would de-sync Chef 12.x and ChefDK 1.x which would be subpar. I think you should plan to move to 2.x as soon as possible.

Hey @coderanger I get where you're coming from, but at the same time, chef-dk is packaged with a known broken version of ruby. And there's not really a work around for seg-faults in the GC, not that I could find at least.
We've been using 0.16.x and earlier versions of ChefDK without any issues for a while, but I noticed it was getting very dated, and that the 1.x.x versions were using Chef 12, which didn't seem to require significant changes in our cookbooks/infra. It made sense to update to v1.5 which is latest before Chef 13
I'd love to update to Chef 13, but we don't have the bandwidth to migrate 100's of cookbooks and upgrade all our infrastructure scripts right now. Not to mention that I work at an enterprise that has a bunch of major compliance agreements, so core infrastructure changes require a bit more time & vetting.
Getting a patched version of ChefDK would be my best option, but I'm open to ways to update the chefdk ruby version myself, or workarounds for the GC issue.

I agree the doing the upgrade would be the better or more correct solution, just trying to be up-front about the amount of time we can put in to maintaining what is now a bug-fix-only branch. If one of the other devs rolls up and says they've been allocated a big block of time to make this happen because of a customer support agreement, I'll be happy too, but otherwise it just doesn't seem realistic given the scope.

chefdk 1.6.1 is on 2.3.4p301 now:
# /opt/chefdk/embedded/bin/ruby --version
ruby 2.3.4p301 (2017-03-30 revision 58214) [x86_64-darwin14]`

generally, though, no we will not be bugfixing anything in chef-dk 1.x going forwards -- we had to upgrade the ruby in order to ship because of a gcc7 on windows rathole.

Awesome. I know this fix wasn't just for us, but I really appreciate it nonetheless. 🎉


Could not find launch information for the current OS: Mac OS X (amd64)

I'm not sure if this is the same problem as #1246 or not. I thought that was fixed, but I may have not tested it properly. I'm using 1.13.1-SNAPSHOT.
When I try to launch IntelliJ in the debugger with the Flutter plugin sources, I get this error:
Execution failed for task ':flutter-idea:runIde'.
> Could not find launch information for the current OS: Mac OS X (amd64)

I get the same error when I try to run unit tests.
I think you can reproduce the problem without too much trouble.

Install the Flutter plugin source from GitHub
Patch in this PR: https://github.com/flutter/flutter-intellij/pull/6651
From the command line, cd to that directory
bin/plugin setup
./gradlew test

The IDE you use for running doesn't contain the proper product-info.json file relevant to your OS.
Could you please provide the content of that file? An example path is:
/Users/hsz/.gradle/caches/modules-2/files-2.1/com.jetbrains.intellij.idea/ideaIC/2022.3.1/2a2986db4e11c3d378b4d1c8b9fd040cd6d99802/ideaIC-2022.3.1/product-info.json

I see what the problem is. Thanks! I know how to fix it.


feat: add section for local or quasi-local companies

@MandyMeindersma has posted (and edited) a list of companies based in or having a presence in Edmonton in Slack. Since the Slack history is not permanent, this information should be displayed on the website.

Slack convo below for reference:
Mandy Meindersma
:des-slack:  Jun 26th at 10:04 AM
Master List of local Edmonton companies or companies with an Edmonton Presence:
https://getjobber.com/about/careers/
https://www.scopear.com/jobs/
https://www.resolver.com/careers/
https://www.dotdashmeredith.com/careers
https://careers.intuit.com/country/canada
https://jobs.lever.co/altaml/
https://lanternstudios.com/careers/ (previously DevFacto)
https://www.bazaarvoice.com/company/careers/ (previously Granify)
https://yellowpencil.com/careers/
https://punchcard.io/careers/
https://gskinner.com/careers
https://drivewyze.com/careers/
https://www.drugbank.com/about/careers/
https://dealcloser.bamboohr.com/careers
https://cybera.io/careers/
https://onlea.org/careers/
https://levven.com/job-openings/
https://www.showbie.com/jobs/
https://www.exo.inc/careers (previously MedoAI)
https://www.amii.ca/your-career/career-opportunities/
https://www.bioware.com/careers/
https://careers.hootsuite.com/jobs/
https://quotetome.applytojobs.ca/
https://cgi.njoyn.com/corp/xweb/xweb.asp?NTKN=c&clid=21001&Page=joblisting
https://careers.ama.ab.ca/
https://www.haemonetics.com/careers
https://coparenter.com/
https://www.lawdepot.ca/career/?loc=CA
https://muckrack.com/careers
https://place.com/careers/
https://www.darkhorseanalytics.com/careers
https://www.ea.com/careers
https://www.inflexion.io/work-with-us
https://www.apple.com/careers/ca/
https://www.digitalocean.com/careers
https://careers.atb.com/careers
https://www.invidi.com/contact-us/careers/
https://rithmik.com/work-with-us/
https://www.computronix.com/
https://paper-leaf.com/careers/

@bottbott I think this is awesome, I also endorse this!

For context, original list: https://docs.google.com/document/d/11LBGg4bZp8l8bYnHJxi6tY9CQxlXix7jIiuWuHOVitc/edit


WIP: Add test vectors

Issue credential DIDComm v2 messages with PRC use case
Present proof DIDComm v2 messages with PRC use case

part of #1

Added WIP prefix as I work through the review comments.

@OR13

Its its possible to make all of them did:example and provide the did documents for each, thats probably best.

Would it be possible to share a reference for did:example document ? I always thought it was made up DID for sample data and no DID Document as such :smiley:

@OR13

Its its possible to make all of them did:example and provide the did documents for each, thats probably best.

Would it be possible to share a reference for did:example document ? I always thought it was made up DID for sample data and no DID Method/Document as such 😃

Added sample did documents.


compatibility between turicreate and swift greyscale images

I have used - tc.image_analysis.resize to resize my training images to fit the size requirement for the model and to convert them to greyscale using the following command.
turicreate.image_analysis.resize(image, width, height,channels=1)
Then I tried to deploy it as an iOS app. But the results I get on my unbuntu and iphone are close but different. I think this has to do with how I am converting the image to greyscale in my app.
I have tried CIPhotoEffectMono  and CIPhotoEffectNoir to convert to greyscale. Both of them did not yield the result  I was getting from my ubuntu runs on the same images.
Is there an equivalent greyscale converter in swift  to match the one used in tc.image_analysis.resize(channel=1)?

There is no equivalent grayscale converter in Swift. There are certainly ways to convert images to grayscale in Swift, but these will likely give slightly different answers than the Turi create methods.

How does Turicreate convert an image to greyscale? I could create a function based on this.

I think this call to resize boils down to a usage of the Boost Generic Image Library. Take a look at their sample code, under "Using Image Views", for an example of converting a JPEG from RGB to grayscale (and then transforming it some more).
FWIW, our code (that calls into Boost GIL) is under src/image/image_util_impl.cpp, but most of that stuff handles non-color-related transformations: resampling, etc.

@nickjong thanks for pointing me in the right direction.


chore: drop paginator and use gapic to return streams for paginated calls instead

[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

@AVaksman I know it's still a draft, so sorry if I'm jumping in too soon. But, I was wondering if you could describe the intentions behind this when you get the chance? Thanks!

gapic now supports paginated calls and its streamified versions directly.

Does it also retry requests?

Yes, gax supports retries for paged calls.
RetryOptions{retryCodes; backoffSettings;} can be passed via gaxOptions.retry argument.
Streamified version of paged call is essentially same paged call where results pushed one by one via an object mode stream.

Thanks!

@stephenplusplus I added another unit test after the approval

Nice, lgtm!