Add check for ComponentBitCount to checkerXmlBasicMetadataValidation.cpp

Add a check for ComponentBitCount in the Dimensions.Channels.Channel metadata.  If Channel metadata is present, the ChannelBitCount is used in ZEN applications and must be correct.
Description
CZI files with Dimension.Channels.Channel metadata must populate the ComponentBitCount field in this metadata.  If channels are present, this value must reflect the bit count of the pixels of the subblocks in this channel.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe the tests that you ran to verify your changes.
Provide instructions to reproduce.
Checklist:

[X] I followed the Contributing Guidelines.
[X] I did a self-review.
[ ] I commented my code, particularly in hard-to-understand areas.
[ ] I updated the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

Thanks for you contribution!
Here are some initial thoughts on this topic:

I'd not think that "ChannelBitCount" is a mandatory field. The schema marks it as optional, so at least from a "file-format perspective" this field is optional.
Notwithstanding - if some operations rely on the presence of this field, it might make sense to check for it. However - "in general" I'd guess it is a questionable requirement (to rely on the presence of this field, and it's correctness).

This would make me lean towards "reporting the absence of the field" as "Info" (rather than "Warning").

There are for sure complications with the semantic of this field - e.g. for float-pixeltypes it is undefined.
Then... besides checking for the mere presence, it would be nice to check for correctness. A basal check would be to check "PixelType" and "ChannelBitCount" together for consistency (e.g. if PixelType is Gray8, then ChannelBitCount must be <=8). A more thorough version would be to check the actual pixeldata.

I'd think - at least this "basal check" (check "PixelType" and "ChannelBitCount" together for consistency) I'd want to add before merging this PR.
Some more TODOs:

bump version (in CMakeLists.txt)
add note to version-history.md
adding a test-case would be desirable

Besides - may I kindly ask you to provide a signed CLA (c.f. here). Just print out the text, sign it and mail a scan/photo of it. Thanks!

I put it as a warning as there is evidence that histogram code in ZEN requires this value to be set.  I am also however not sure what it should be set to for various pixel types, or how to reliably validate that it is correct - I'd love to add checks for correctness if I knew this!  I'm sure I could get something together for the simple pixel types, but across the supported range?  I'll work on the TODO items you listed.  Thanks for the review.

I followed up on this PR in #19


GetAuthorizeUrl throws EpiOAuthException

Note, this is not identical to the other issue about GetAuthorizeUrl.
My problem is that we're getting an empty response from the request to https://api.twitter.com/oauth/request_token - or at least both messageText and code are returned empty.
I've changed the URL to https, tried API version 1 and 1.1, I tried with oauth_callback in the request and without. I tried checking curl for errors and nothing. Curl is working and supports https (we use it for amazon SES as well...) and besides, this has been working before without change. I've also tried the recent changes in other similar topics and none worked. I also checked whether we're rate-limited but this seems not to be the case.
What happens is that the curl request returns empty and with a code of 0, which causes EpiOAuth.php to raise an EpiOAuthException in class EpiOAuthResponse function __get, as the code is expected to equal 200 there.
Does anyone have any pointers as to what could be the cause? Do note that all this has been working perfectly in the past. We were running the latest twitter-async code with API 1 without problems.
I'd really appeciate any ideas and hints!

Wow thax a lot!!!!!

Thanks..
I spent one day debugging this. Finally got solution..
Thanks very much :+1:

This "bug" (?) persists to this day, except the code has changed slightly.  The solution is similar: remove/comment out the following two lines in EpiOAuth.php:
    if($this->serverAddr !== false)
      curl_setopt($ch, CURLOPT_INTERFACE, $this->serverAddr);

Also, for the sake of searchability, here's the output I was getting when I enabled verbose output in cURL:
* About to connect() to api.twitter.com port 443 (#0)
*   Trying <IP_ADDRESS>... * Name '<IP_ADDRESS>' family 2 resolved to '<IP_ADDRESS>' family 2
* Local port: 0
* Timeout
*   Trying <IP_ADDRESS>... * Name '<IP_ADDRESS>' family 2 resolved to '<IP_ADDRESS>' family 2
* Local port: 0
* Timeout
*   Trying <IP_ADDRESS>... * Name '<IP_ADDRESS>' family 2 resolved to '<IP_ADDRESS>' family 2
* Local port: 0
* Timeout
*   Trying <IP_ADDRESS>... * Name '<IP_ADDRESS>' family 2 resolved to '<IP_ADDRESS>' family 2
* Local port: 0
* Timeout
* connect() timed out!
* Closing connection #0


0.10.2 has broken import statements in quinn extensions

Importing anything in quinn.extensions is currently broken due to the __init__.py referencing the column_ext.py file which has been removed.
Running
from quinn.extensions import *
Results in:
ModuleNotFoundError: No module named 'quinn.extensions.column_ext'
As an example in Databricks:

@jamesfielder - should be fixed with quinn v0.10.3.


When pasting SVG images, Virgil font is not being used

My first issue :). I just found this project and it is wonderful. Thank you so much for your work :)
While experimenting with a bunch of hacky ways to get data from my app into excalidraw, I came across this:
I am trying to paste a SVG into excalidraw which contains a bunch of text. I've prepared the file to use the font-family "Virgil", but no matter how I reference that font in the SVG file, the text always shows up without a font. This is probably not intended.
This is the SVG I am pasting into excalidraw:
<svg version="1.1" id="Layer_1" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink" x="0px" y="0px"
	 viewBox="0 0 512 72" style="enable-background:new 0 0 512 72;" xml:space="preserve">
<style type="text/css">
	.st0{font-family:'Virgil';}
	.st1{font-size:36.0932px;}
</style>
<text transform="matrix(1 0 0 1 34.2061 45.1016)" class="st0 st1">Test SVG with Virgil Font</text>
</svg>

And it shows up like this:

The same happens when I select a text in excalidraw -> right-click -> "Copy to clipboard as SVG".
When I then remove the <!-- svg-source:excalidraw --> Tag and paste this SVG back into excalidraw, it doesn't show up with the font.

This is probably cause by https://github.com/excalidraw/virgil/issues/61

We've fixed this recently with (https://github.com/excalidraw/excalidraw/pull/8012).


Properly convert undefined/null header values to strings.

In sending undefined/null header values with the fetch polyfill, I was getting an error:
TypeError: 'undefined' is not an object (evaluating 'value.toString')

at normalizeValue (fetch.js:20:21)
    at Headers.append (fetch.js:42:14)
    at Headers.<anonymous> (fetch.js:30:15)
    at Array.forEach (native)
    at new Headers (fetch.js:29:45)
    at new Request (fetch.js:25:21)
    at self.fetch (fetch.js:273:18)

I tested this against the native window.fetch implementation in Chrome 43.0.2357.81 and observed that primitive values were handled fine when set as request header values. Using string concatenation for string conversion - over toString (as noted in Axel Rauschmeyer's article on the subject) - brings parity between the two implementations in this regard.

String concatenation is actually incorrect as well. You should be doing String(value).
String concatenation in particular gives the wrong result for { valueOf() { return 5; }, toString() { return "foo"; } }.

String concatenation in particular gives the wrong result for { valueOf() { return 5; }, toString() { return "foo"; } }.

Duly noted. Updated, thanks @domenic .

Thanks for looking into this! The PhantomJS build is alright but SauceLabs in CI seems to be acting up and failing the build because it takes too long. You should pay no attention to that, but it might be useful to manually test in Firefox and IE if SauceLabs is unable to do it.

You should pay no attention to that, but it might be useful to manually test in Firefox and IE if SauceLabs is unable to do it.

I ran the tests against my own Sauce Labs account. Here are the results:
⫸ SAUCE_PLATFORM="Windows 7" SAUCE_BROWSER="internet explorer" SAUCE_VERSION="11" make test
./node_modules/.bin/jshint *.js test/*.js
./script/test
Windows 7
internet explorer
11
{"suites"=>21, "tests"=>70, "end"=>nil, "passes"=>61, "reports"=>[], "start"=>nil, "duration"=>4394, "failures"=>0, "pending"=>9}
⫸ SAUCE_PLATFORM="Windows 7" SAUCE_BROWSER="internet explorer" SAUCE_VERSION="10" make test
./node_modules/.bin/jshint *.js test/*.js
./script/test
Windows 7
internet explorer
10
{"passes"=>61, "tests"=>70, "end"=>nil, "suites"=>21, "reports"=>[], "start"=>nil, "duration"=>2712, "failures"=>0, "pending"=>9}
⫸ SAUCE_PLATFORM="Windows 7" SAUCE_BROWSER="internet explorer" SAUCE_VERSION="9" make test
./node_modules/.bin/jshint *.js test/*.js
./script/test
Windows 7
internet explorer
9
{"suites"=>21, "tests"=>70, "end"=>nil, "passes"=>42, "reports"=>[], "start"=>nil, "duration"=>1364, "failures"=>0, "pending"=>28}
⫸ SAUCE_PLATFORM="Windows 7" SAUCE_BROWSER="firefox" SAUCE_VERSION="" make test
./node_modules/.bin/jshint *.js test/*.js
./script/test
Windows 7
firefox

"Test exceeded maximum duration after 180 seconds"
./script/saucelabs-result:13:in `<main>': undefined method `+' for nil:NilClass (NoMethodError)
make: *** [test] Error 1

I gathered that a test is failing in Firefox for a reason unrelated to this change. Namely, Mocha's error handler by way of window.onerror is reporting a failure due to this spec:
SyntaxError: JSON.parse: unexpected keyword at line 1 column 1 of the JSON data (:0)
process.on/global.onerror@http://localhost:3000/node_modules/mocha/mocha.js:6193:10

Global error handlers appear to handle errors thrown in promise fulfillment callbacks differently depending on vendor. Example:
// http://jsbin.com/boqilipecu/1/edit?js,console

window.onerror = function(e) {
  console.error(e);
};

new Promise(function(resolve, reject) {
  resolve('{');
}).then(JSON.parse).catch(function() {});

// Firefox (38.0.5): logs an error
// Chrome (43.0.2357.81): nothing logged;

Again, completely separate, but if you're seeing flaky test failures against Firefox, it's likely due to the above. Otherwise, tests look to be passing.

Thanks for the investigation. We'd welcome a separate PR that fixes Firefox on Sauce, if you know what's going on.
This looks ready to merge, although I'd like to figure out what's going on with Sauce first, as I don't want master to start failing again (although it's likely it would start failing on next build regardless of merging this). @josh do you have any idea why it's so abysmally slow that it eventually times out? Normal build times used to be under 2 minutes.

:+1:


Allow optional number in input-mask

Describe The Problem To Be Solved
Currently the user is forced to fill the input-mask, but if we are talking about phone numbers they have different amount of digits. This makes this package basically useless when it comes to this type of problem where there may be some optional part in the input.
Suggest A Solution
I've seen other libraries in other frameworks (Angular: ngx-mask) which have 0 as a mandatory number to fill while 9 is optional.
Example of ngx-mask:

I think I would prefer to have 0/O as optional, since 9/X are already used for numbers and letters and I would like to stay compatible. In the meantime, you could probably use the array form.

I'll have a solution ready soon.

Sorry, I was sick in the last few weeks. The solution was just merged, I expect it to be released soon.


Port Orders not available in Legacy Teams Portal

??

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

ID: e1829a37-a7a1-4dd1-fde8-82125c593ff1
Version Independent ID: dfedc4c5-eec4-2137-e975-399866aaafad
Content: Transfer phone numbers to Office 365
Content Source: Teams/transfer-phone-numbers-to-office-365.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@DKiefer770 Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

@DKiefer770 thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

@officedocsbot close

For anyone else who stumbles across this page and has the same problem like I did.
I reached out to Microsoft via Office 365 admin portal. I was told that once my account has an E5 licence then the option would show up. I am currently exploring options for soft phones and wanted to see the process but we are on E3 licences.


OnePlus3 needing to turn nearby of

@scottjenson a client of ours noticed that on the OnePlus3 you need to turn on "Nearby links available" and "Devices ready to set up" manually.
Is this the normal intended method? What we've learnt from most Android installations that the onboarding to Nearby is completely seamless.

I haven't tested a OnePlus3 so I can't comment directly. The only reason I can guess this is happening is if the BLE stack isn't good (the OnePlus1 had this issue) If that were the case, there would be only partial scanning and that wouldn't trip the initial alert.
The surefire test of this theory is to set a beacon to broadcast fairly rapidly (say 300ms) and the run nRF Connect and collect scan dataon that beacon for 15 seconds. If you see a large number of missing adv packets, there's your problem.

I'm so sorry, I clearly misread this issue. No that is not normal. You should never be forced to manually turn it on. I'm not sure what would trigger that.


Use GoJay for JSON marshalling

See francoispqt/gojay

This won't work, and actually makes no sense at all.


Add issue and pull request tracker to the dashboard

Closes #15
Closes #16

@jhamman, when you get a moment, can you take a look at the dashboard page and let me know if you have any feedback?

Merging this as-is for the time being :).
@jhamman, no worries about the feedback on this PR. I figured you might be swamped right now... If you get a moment, feel free to add more comments to the original issue: https://github.com/andersy005/xarray-website/issues/18


Git diff does not work over a Samba share

Type: Bug
I am accessing a network drive on a Linux server, shared using Samba, using the standard GNOME GVfs client. If I navigate to the file share on the terminal (/run/user/####/gvfs/smb-share:..../) and run git diff, the expected differences are displayed. However, in VS Code, it displays every file as if it is brand new; even worse, staged files won't show anything at all.
VS Code version: Code 1.95.2 (e8653663e8840adaf45af01eab5c627a5af81807, 2024-11-07T11:07:22.054Z)
OS version: Linux x64 5.4.0-200-generic snap
Modes:

System Info

Item
Value

CPUs
AMD Ryzen 7 2700X Eight-Core Processor (16 x 2063)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_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: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_offwebnn: disabled_off

Load (avg)
0, 0, 1

Memory (System)
15.61GB (11.27GB free)

Process Argv
--no-sandbox --force-user-env --crash-reporter-id d79ac254-0e9d-427a-a9ba-5adfd6c2b60d

Screen Reader
no

VM
0%

DESKTOP_SESSION
ubuntu

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
ubuntu

XDG_SESSION_TYPE
x11

Extensions (20)

Extension
Author (truncated)
Version

language-hugo-vscode
bud
1.3.1

githistory
don
0.6.20

xml
Dot
2.5.1

codespaces
Git
1.17.3

gc-excelviewer
Gra
4.2.62

csdevkit
ms-
1.11.14

csharp
ms-
2.50.27

vscode-dotnet-runtime
ms-
2.2.2

vscodeintellicode-csharp
ms-
2.1.11

debugpy
ms-
2024.12.0

isort
ms-
2023.10.1

python
ms-
2024.18.1

vscode-pylance
ms-
2024.11.1

hexeditor
ms-
1.11.1

wasm-wasi-core
ms-
1.0.2

vscode-yaml
red
1.15.0

ruby-lsp
Sho
0.8.13

shopify-liquid
sis
4.0.1

code-spell-checker
str
3.0.1

vscode-icons
vsc
12.9.0

A/B Experiments
vsliv368:30146709
vspor879:30202332
vspor708:30202333
vspor363:30204092
vscod805cf:30301675
binariesv615:30325510
vsaa593cf:30376535
py29gd2263:31024239
c4g48928:30535728
azure-dev_surveyone:30548225
vscrp:30673768
2i9eh265:30646982
962ge761:30959799
pythongtdpath:30769146
pythonnoceb:30805159
asynctok:30898717
pythonmypyd1:30879173
h48ei257:31000450
pythontbext0:30879054
cppperfnew:31000557
dsvsc020:30976470
pythonait:31006305
dsvsc021:30996838
0ee40948:31013168
dvdeprecation:31068756
dwnewjupyter:31046869
impr_priority:31102340
nativerepl2:31139839
refactort:31108082
pythonrstrctxt:31112756
cf971741:31144450
iacca1:31171482
notype1:31157159
5fd0e150:31155592
dwcopilot:31170013

git diff works fine:

whereas VS Code diff does not:

I have already tried changing the Git settings core.filemode and core.autocrlf that people have recommended elsewhere to fix some issues, but I don't see how those would actually make a difference here; the problem is that the "original" (left pane) is completely missing, not that the editor is flagging every line or every file unexpectedly.


[requires.io] dependency update on master branch

Looks like Sphinx 1.4.7 is just broken, oops. https://github.com/sphinx-doc/sphinx/issues/2996

Normally @Lukasa merges these, but I’ll do it since he’s on holiday.


Send error while publishing msg when queue is full

After merging this change let us update orbits to use this


correcting link to load-balancer-internet-overview

The page was linking back to itself.

@bennage Thanks for this contribution!


Not specifying optional steps results in an error in v3.0.0

Not specifying optional steps like ansistrano_before_setup_tasks_file will result in the following error:
ERROR! no action detected in task. This often indicates a misspelled module name, or incorrect module path.
Probably caused by the switch from include to include_tasks in https://github.com/ansistrano/deploy/commit/6a7a825cc03bf27ab59dd3804ba561dbedb46650.

It looks like you are using ansible 2.3, which is no longer supported.

In that case the error makes sense. Would it be possible to add that information to the release notes of version 3.0.0? This information might be helpful to others.

it is showing up in Ansible galaxy, in the meta.yml file, in our twitter
account and in the commit message
El jue., 11 oct. 2018 a las 11:39, Peter Ton<EMAIL_ADDRESS>escribió:

In that case the error makes sense. Would it be possible to add that
information to the release notes of version 3.0.0? This information might
be helpful to others.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ansistrano/deploy/issues/316#issuecomment-428889947,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAldXZjFa_yKEiY3SOnr03PkKHFgXz39ks5ujxHXgaJpZM4XXFzz
.


Run relevant ethereum/tests

https://github.com/ethereum/tests

We have covered the state tests, which is good enough for V1.
https://github.com/risechain/block-stm-revm/blob/main/tests/ethereum/main.rs#L321
If necessary we can cover more  (like the Blockchain tests with beneficiary handling) in V2.

Also: https://github.com/ethereum/execution-spec-tests


BaföG page blacklisted

The page for financial aid while studying in Germany is blacklisted, I doubt that this is a legitimate blacklist.

Hey, this is because you have the "Block all punycode domains" option enabled. See https://github.com/409H/EtherAddressLookup/issues/532#issuecomment-387731112


#2943 Do not fail aggregate code generation on parallel execution

@jeremydmiller I think it failed on a flaky test. So, maybe retry the build?


Netlink Bandwidth Monitor - Increase width of Host column

Is it possible to increase the width of the Host column as a number of devices that have long names on my network are being truncated with a "..."
Thanks

Hostnames are currently truncated after 12 characters. How long is your longest name?


workflow: Use per-tunnel keys for the IPsec upgrade test

[ upstream commit cb4144b672851d629bc848ad71c5e9cc30f0a6fc ]
All users should now be using per-tunnel keys (that is, with the + sign in the IPsec secret) instead of the insecure global key. Our up/downgrade test for IPsec should therefore reflect that.
Reported-by: Marcel Zieba<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Are you a user of Cilium? Please add yourself to the Users doc
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

/ci-ipsec-upgrade


Carry 11441: fix docker ps help message

Closes #11441

LGTM

Thanks for your contribution @jimmyxian !

@LK4D4 that is a possessive --- yay @jimmyxian thank you for being so precise! LGTM

OK, it's my first contribution. Thanks all the people who help me to known the contribution guidelines.I will continue to focus on the docker community.

@jimmyxian Congrats!


Error filing issue

https://github.com/sigstore/sigstore-probers/actions/runs/8026103515/job/21928139376 states the github_token is missing. We do pass it to the calling workflow: https://github.com/sigstore/sigstore-probers/blob/afcc606cc1ee48caa4a55a2f633d8467a800baa4/.github/workflows/prober-staging.yml#L24
I assume the token has expired? @cpanato @bobcallaway can you confirm?

This might be a GitHub outage also.

Looks intermittent.
On Fri, Feb 23, 2024 at 7:03 PM Hayden B @.***> wrote:

This might be a GitHub outage also.
—
Reply to this email directly, view it on GitHub
https://github.com/sigstore/sigstore-probers/issues/180#issuecomment-1962156500,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAVWTJPQNUVBNLI3GK6U2DLYVEU6NAVCNFSM6AAAAABDXPS3SWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRSGE2TMNJQGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

The secret is missing. Let's look at switching to a different issue filing workflow.


"show group" dialog shows all tasks, instead of only tasks of that group.

When viewing a group, the grid containing tasks shows all existing tasks (except for finished ones).
However, when deleting / updating a task inside the grid, the grid is fixed.

UpdateGrid() was not called after initiating the GroupFilter, fixed in f582679


Unable to get Facebook profile photo anymore

Describe the bug
Hi, I' having issue to get facebook user profile photo recently, it was working just fine previously.
Any idea what is the issue?
To Reproduce

Authenticate a new facebook user
And it return only the blank facebook user profile image.

Expected behavior
Expected to received user profile photo as previously.
Screenshots
If applicable, add screenshots to help explain your problem.
https://scontent.fkul17-1.fna.fbcdn.net/v/t1.30497-1/cp0/c<IP_ADDRESS>a/p50x50/84628273_176159830277856_972693363922829312_n.jpg?_nc_cat=1&_nc_sid=12b3be&_nc_ohc=avjs1eN81qIAX8LLQ-r&_nc_ht=scontent.fkul17-1.fna&_nc_tp=27&oh=7bc5a81b88f6a922b745b3c87f29808d&oe=5FA4D0B8
Additional context
Ionic:
Ionic CLI                     : 6.11.11 (/Users/.../.nvm/versions/node/v12.16.1/lib/node_modules/@ionic/cli)
Ionic Framework               : @ionic/angular 5.3.4
@angular-devkit/build-angular : 0.1001.4
@angular-devkit/schematics    : 10.1.4
@angular/cli                  : 10.1.4
@ionic/angular-toolkit        : 2.3.3
Capacitor:
Capacitor CLI   : 2.4.2
@capacitor/core : 2.4.2
Utility:
cordova-res : 0.15.1
native-run  : 1.2.1
System:
NodeJS : v12.16.1 (/Users/.../.nvm/versions/node/v12.16.1/bin/node)
npm    : 6.14.8
OS     : macOS Catalina

This is not plugin issue. It is a specification of Facebook.

This is not plugin issue. It is a specification of Facebook.


changed path to config.json in Chat.js handleWriteToConfig

The path is relative to the directory where node was started. This should fix #26
Tested on windows 10 and debian 10

Hey Jhn-k!
Thank you for the PR, but unfortunately this will not have the desired result.
I'm not sure if anyone actually runs NOALBS directly with node. But for example if someone
used node lib/app.js  in another directory it would create a copy of the config there.


support devtools

so you can see all the data in the stores in the devtools vuex tab

Hi Patrick,
Vue Class Store has a completely different paradigm than Vuex, on purpose.
The Vuex panel in DevTools is all set up to hook into dispatches, commits, etc to update. I don't think it would be the right direction for this lib to try to hook into that (though I'm not adverse to trying if it looked like an easy win)
You can add the store to a Vuex getter to see it in the Vuex DevTools, unfortunately the Vuex panel does not reflect any updates :)
A cheat I have used to show global stores, is to add them to the root component, something like this:
import Vue from 'vue'
import router from './router'
import App from './components/App.vue'
import products from './store/products'
import users from './store/users'

new Vue({
  router,

  // provide to all components in the app
  provide () {
    return {
      $store: this.$store
    }
  },

  // show on the root component, for debugging
  computed: {
    $store () {
      return {
        products,
        users,
      }
    }
  },

  render: h => h(App)
}).$mount('#app')

Does that help?


Added onClick for About section

Fixes: #223 .
Description:
Implemented an onClick for the Logo(s) present in the About Section of the app. When the user clicks on the logo, it will redirect to its relevant website. With the help of this feature, users would be able to know and gain more knowledge about the app and its investors and support.
Current About Section Screenshot:

onClick on About Section:
https://user-images.githubusercontent.com/54114888/160650317-e6ff3e4e-5c37-4dc5-9ce9-5d0a537c44b0.mp4

@sastry17 sir please review my PR.


typo in template

small typo for output lambda
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Hey! Thanks for catching this! We fixed some mixups on commits, if could you rebase and update this PR we'll get it merged in!

done


Use push.registry.devshift.net

With registry.devshift.net we need to secure push actions - this PR enables that

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@vpavlin snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1566-1

@hectorj2f That is a good question. It should fail push then, but I can add exit 1 in the else branch if you prefer

@vpavlin it will fail but it is your choice, not a blocker :)

I'd leave it as is - it prints a message and then fails..should be safe and clear enough:)

[test]

[test]

@vpavlin snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1566-2


[Python] Explicit delete of BLE scanner object

Problem
The Python module deletes scanner instance in the OnScanComplete() callback. We have to make sure that this will not be used after callback returns, which causes some issues in Linux platform.
Changes

add pychip_ble_scanner_delete which can be used for explicit memory management
in python, spawn daemon thread which deletes the scanner object after being triggered by onScanComplete callback
normalize (uppercase) BT address passed to DiscoverAsync()

Testing
Locally tested scanner functionality with chip-tool on Linux

This PR is obsole now.


Feature Enhancement: Ability to open files in pending view from the fuzzy-finder search bar

Current and suggested behavior
There is currently no way to open a file as a pending item from the fuzzy search. I suggest adding the ability for files that are being searched in the fuzzy finder search to open in pending mode. From my understanding, there are 3 possible logical approaches to implementing this without breaking the current configuration.
Options

Have a boolean option either in Core Settings or in the fuzzy-finder package settings that would cause all files opened from the fuzzy search to be in pending mode. This would be exactly like pre-1.6.0 behavior when the feature was in the tabs package.
Have a command option that allows keybindings to map to opening a fuzzy search that will always open files in pending mode. (Ex: I can map cmd-t to open a finder that will open files normally, and map cmd-p to open a finder that will open files as a pending item).
Implement a feature similar to Search All Panes in the fuzzy-finder package that allows some sort of on-the-fly choice of opening the file that is in focus on the fuzzy finder search to be opened as a pending item. (Ex: enter on the current highlighted file will open it normally, but something like cmd-enter might open it in pending mode).

If implemented carefully, I believe options 1 & 2, or 1 & 3 can be implemented in harmony. Options 2 & 3 probably won't work together.
Why would the enhancement be useful to most users?
I imagine that there are many users, such as myself, that do not use the tree view for most file opening operations, and prefer to use the command palette. I almost exclusively open files through cmd-p or cmd-t, and found the old Preview Tabs feature helpful since I would normally be browsing to check if the current file is the one I want to keep open. Option 1 above would return the behavior to this state, but I think options 2 or 3 are a better solution since they allow for more flexibility. I believe most people would love the ability to have the choice between opening their files as a pending or standard pane item.
@lee-dohm @kuychaco I would be willing to implement any of these if you guys think one in particular is the better direction.
Atom Version: 1.6.0+
OS and Version: All platforms & supported versions

Thanks for the great writeup @christopherciufo. I agree with your assessments and think it would be great to have this as an enhancement to the pending view feature.
I say go with Option 3. It gives users the option of making the decision of whether to open in pending/non-pending mode at the last second. Whereas with Option 2, users have to make the decision up front and if they change their mind it's an extra keystroke to switch to opening fuzzy-finder in the other mode.
And since you've mapped keybindings to different commands for opening files, there is less of a need for creating a config option (Option 1) as users can easily change the default behavior by re-mapping keybindings in their keymap.cson file by doing something like this:
'.fuzzy-finder atom-text-editor[mini]'
  'enter': 'fuzzy-finder:open-pending'  # where default is 'enter':'fuzzy-finder:open'

Hope this helps a bit! Thanks for taking this on :)

This is something I miss from Sublime, being able to ctrl+p and as you navigate up and down the list, the highlighted file is pseudo-opened into a tab so you can view it briefly, if the file is already open it just navigates to it.

Yes I miss the old cmd+p of sublime too :(


Hugo Doesn't Complete Generation With Large Number of Pages

Traviss-MacBook-Pro:Demo teamcoltra$ hugo
Started building sites ...
Built site for language en:
0 draft content
0 future content
0 expired content
124554 regular pages created
12 other pages created
0 non-page files copied
9967 paginator pages created
1 author created
0 tags created
0 series created
total in 4745107 ms

if I narrow the number down to 100 pages the site generates just perfectly...  124554 is the correct number of files, but ~70% of those have unique authors, tags, and maybe 50% have series (which, again, all this is working at small scale already so I know they work).
It's not kicking back errors or anything like that so it's hard to know exactly what is wrong. I realize I am really pushing the boundaries of a normal use case for hugo but I feel like it should still fail gracefully (and there shouldn't really be anything inherently holding it back, if it takes 3+ hours to generate... thats fine by me)

Add-on info: Counting the total number of files in public/ is 14,953 ... so I think it "lied" about how many pages it created as well.

This isn't an issue we can take concrete action on at the moment, so I suggest  https://discourse.gohugo.io/ for questions/troubleshooting. We may get better in this area in the future, but we have other issues to track that.

One thing to consider is the memory consumption. You're doing this on a laptop. It may work better on a beefier machine considering the significant workload.

Yeah this might be the case, there is also someone on the forums who mentioned this could be a Mac issue that has had a problem in particular with large numbers of files and hugo they gave me some numbers to change.
That said, this tells me Hugo still needs better error catching. If something is messing it up and not allowing it to complete it's process it shouldn't say/imply it completed successfully, preferably it should say whats up or at least say "expected x didn't complete it"

In an out of memory scenario, I'm pretty sure Hugo will crash.

I agree this isn't the best behavior. Unfortunately there isn't really a
lot we can do from within the application. This it really the OS'
responsibility.
On Fri, Aug 11, 2017 at 8:40 AM Cameron Moore<EMAIL_ADDRESS>wrote:

In an out of memory scenario, I'm pretty sure Hugo will crash.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/gohugoio/hugo/issues/3792#issuecomment-321802158, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAKlZFqVTbbpn7Ij7ooUYppkw20xmKrHks5sXEvXgaJpZM4Oz6nO
.


[EUWE] Set the checkable property based on the presence of @assign

Euwe equivalent of https://github.com/ManageIQ/manageiq-ui-classic/pull/1747
https://bugzilla.redhat.com/show_bug.cgi?id=1478368

@simaishi Note that the upstream code is quite different than Euwe, and hence this Euwe PR required only a small change. Thanks.

@h-kataria ping


#7932 view collection button too close

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[ ] Feature
[x] Refactoring

Needs QA check

@kodadot/qa-guild please review

Context

[x] Closes #7932
[ ] Requires deployment <snek/rubick/worker>

Did your issue had any of the "$" label on it?

[x] My DOT address: Payout

Screenshot 📸

[x] My fix has changed UI

Copilot Summary

😍 Perfect, I’ve sent the payout
💵 $20 @ 4.87 USD/DOT ~ 4.107 $DOT
🧗 13Qx65nLd6SwdtjrRyuoEtp9CKXhF651xdHBPaXcvhwKm4N1
🔗 0x26fc912e9d0368c872c16942ffc689383dab28a932a492d2004ecfa63a60e4c7
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


SharedMemory breaks if data contains null byte (\x00)

I am trying to share an image between several programs using the SharedMemory class. I noticed that only about half of my image was transfered and then the rest of it was completely black. When investigating closer I noticed that the break always happened at places with a blue color (0, 0, 255).
When I started digging into the source code I noticed this line.
https://github.com/MO-RISE/pycluon/blob/171b9696e48ebedf18811aefee6fe77ad9bed892/src/pycluon.cpp#L193
Here the data is converted to a NULL terminated string and I assume this is why my data is cut off.
I confirmed this by changing my blue color to (1, 1, 255) instead and now my image is complete on the receiving end.
I assume this is a bug and that it should be changed in future versions?
Code to replicate the error:
from pycluon import SharedMemory
import numpy as np

# Create image with one element equal to 0
img = np.ones((480, 640, 3), dtype=np.uint8)
img[240, 320, 0] = 0

sm = SharedMemory("img.rgb", img.size)

sm.lock()
sm.data = img.tobytes()
assert sm.data == img.tobytes() # Fails but should pass
sm.unlock()

Thanks for a nice and well-structured issue report. Yes, this definitely sounds like a bug.
Feel free to propose a PR that solves this!
I, unfortunately, will not have time to work on this for the foreseeable future.

@Paulsson99 Turned out to be a really quick fix. Version 0.2.3 is soon available.

Oh great! Thanks for the quick fix @freol35241!


Speaker images not getting displayed

None of the speaker images are showing up.
This is due to the sample json using an indirect link for the images instead of a direct link to them.
https://github.com/fossasia/open-event/blob/master/sample/FOSSASIA16/speakers#L7
Adding the direct link before the image url mentioned in the samples should fix this issue.

What do you mean by indirect link?
If you want links to photos on the net, let's talk to @aviaryan about implementing an alternative image location. Generally the whole zip should contain the image data already, so we can build apps with it without relying on external sources.

Therefore this is not a bug, but a request for an enhancement.

@mariobehling That's what i meant too.
But, the app is not updated to use this json and hence i marked it as a bug.

@the-dagger How is the app being created ? I remember reading something about having problems saving jpeg/media files in *.apk files so you planned to store direct links of image in app instead and use the links to download media. And that is why the sample json's are not working.
What if we create a separate samples folder with json's without media files (only direct links) .
I guess the json with media files is important too so that is why I am suggestion keeping a json without media files separate.
You can export the FOSSASIA event on open-event-dev without any media files to have a json with only direct links.

@aviaryan This would temporarily work, but it would not solve our use case, where 1. a user exports a file with picture, 2. the user generates an app with basic data, 3. the user starts the Android app and expects that all files are updated from external sources.
For 3. - we need the link to the external source in the JSON (as a second alternative source). Could we add something like "Alternative Photo" in the JSON that includes the URL of the photo?

Agreed with @mariobehling, having a separate field named alternative photo with the direct link to all images would be a good thing to have and will save us of all the hassle to convert and down scale the images on the server.

For 3. - we need the link to the external source in the JSON (as a second alternative source). Could we add something like "Alternative Photo" in the JSON that includes the URL of the photo?

@the-dagger @mariobehling How is the android app for the event updated once it has been created ? Doesn't it use the API endpoint to stay updated (from meta json). If that's the case, querying the API will always give the direct link to the image. So image can be downloaded thereafter if needed.

@aviaryan Can you link me to a live event which has enough content as in the samples?
So that I can check how the app responds to it and then get back to you on this?

@the-dagger Try event 179 . http://open-event-dev.herokuapp.com/api/v2/events/179/speakers
It seems to have speaker images. On the other hand, I think there is some bug in the orga server UI as http://open-event-dev.herokuapp.com/events/179/speakers/ shows no images.


devel: update local containerised development

Update multiple aspects of our docker development stack. The frontend now has a devel specific webpack that allows development against a local server/api. The docker-composer.yaml also includes both versioning and networking updates. Overall, the major change is to migrate our proxy access into the crc webpack proxy.

I'm trying to run this to verify, just having issues reaching the backend. I'm getting a proxy error

I don't know what I did to get this working haha. It was working on my mac but now on my fedora machine it does not work. It will run but won't connect to the api now. @lucasgarfield are you able to run it against the local backend?

I don't know what I did to get this working haha. It was working on my mac but now on my fedora machine it does not work. It will run but won't connect to the api now. @lucasgarfield are you able to run it against the local backend?

Hm, I seem to be experiencing the same thing. I can get the frontend to run, but when I try to do something like create an image, it is unable to contact the backend.

@lucasgarfield when you get a chance can you try verify this? I've added one or two small changes and added the networks back... it seems to be working on my end

I'm getting this error while following the devel readme:

docker-compose build
Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/urllib3/connectionpool.py", line 714, in urlopen
    httplib_response = self._make_request(
                       ^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/urllib3/connectionpool.py", line 415, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib64/python3.11/http/client.py", line 1286, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1332, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1281, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1041, in _send_output
    self.send(msg)
  File "/usr/lib64/python3.11/http/client.py", line 979, in send
    self.connect()
  File "/usr/lib/python3.11/site-packages/docker/transport/unixconn.py", line 30, in connect
    sock.connect(self.unix_socket)
FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/requests/adapters.py", line 489, in send
    resp = conn.urlopen(
           ^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/urllib3/connectionpool.py", line 798, in urlopen
    retries = retries.increment(
              ^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/urllib3/util/retry.py", line 550, in increment
    raise six.reraise(type(error), error, _stacktrace)
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/urllib3/packages/six.py", line 718, in reraise
    raise value.with_traceback(tb)
  File "/usr/lib/python3.11/site-packages/urllib3/connectionpool.py", line 714, in urlopen
    httplib_response = self._make_request(
                       ^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/urllib3/connectionpool.py", line 415, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib64/python3.11/http/client.py", line 1286, in request
    self._send_request(method, url, body, headers, encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1332, in _send_request
    self.endheaders(body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1281, in endheaders
    self._send_output(message_body, encode_chunked=encode_chunked)
  File "/usr/lib64/python3.11/http/client.py", line 1041, in _send_output
    self.send(msg)
  File "/usr/lib64/python3.11/http/client.py", line 979, in send
    self.connect()
  File "/usr/lib/python3.11/site-packages/docker/transport/unixconn.py", line 30, in connect
    sock.connect(self.unix_socket)
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.11/site-packages/docker/api/client.py", line 214, in _retrieve_server_version
    return self.version(api_version=False)["ApiVersion"]
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/docker/api/daemon.py", line 181, in version
    return self._result(self._get(url), json=True)
                        ^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/docker/utils/decorators.py", line 46, in inner
    return f(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/docker/api/client.py", line 237, in _get
    return self.get(url, **self._set_request_timeout(kwargs))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/requests/sessions.py", line 602, in get
    return self.request("GET", url, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/requests/sessions.py", line 589, in request
    resp = self.send(prep, **send_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/requests/sessions.py", line 703, in send
    r = adapter.send(request, **kwargs)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/requests/adapters.py", line 547, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/bin/docker-compose", line 33, in <module>
    sys.exit(load_entry_point('docker-compose==1.29.2', 'console_scripts', 'docker-compose')())
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/compose/cli/main.py", line 81, in main
    command_func()
  File "/usr/lib/python3.11/site-packages/compose/cli/main.py", line 200, in perform_command
    project = project_from_options('.', options)
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/compose/cli/command.py", line 60, in project_from_options
    return get_project(
           ^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/compose/cli/command.py", line 152, in get_project
    client = get_client(
             ^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/compose/cli/docker_client.py", line 41, in get_client
    client = docker_client(
             ^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/compose/cli/docker_client.py", line 170, in docker_client
    client = APIClient(use_ssh_client=not use_paramiko_ssh, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/docker/api/client.py", line 197, in __init__
    self._version = self._retrieve_server_version()
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.11/site-packages/docker/api/client.py", line 221, in _retrieve_server_version
    raise DockerException(
docker.errors.DockerException: Error while fetching server API version: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

That looks more like a docker error than an error with the stack

So I have a couple of thoughts on this PR and how to improve things. I think ultimately, chatting with @ochosi, we will probably move the docker stuff out of this repo and most likely into the osbuild/osbuild-getting-started repo. I just have to make some changes there first.
The second thing I suggest is maybe decoupling the frontend from the stack. I think we needed to have it in the docker stack because of the insights-proxy (I could be wrong though). With the webpack changes Jacob made, we could run the frontend separately and point it towards the backend stack.  Thoughts @lucasgarfield & @jkozol (or anyone else)?

https://github.com/RedHatInsights/image-builder-frontend/pull/1370 is a go, this PR can be closed. Thanks @kingsleyzissou!


Improve firmware flashing script with firmware download capability

Added functionality to download newer firmwarefiles

This one could be further approved by reading in the possible firmware names. #261

This one could be further approved by reading in the possible firmware names. #261

I agree, but let's go with the small step first of allowing a user to specify a known firmware file before expanding functionality.

This one could be further approved by reading in the possible firmware names. #261

I agree, but let's go with the small step first of allowing a user to specify a known firmware file before expanding functionality.

That's why I added this comment 😉

@sorryusernameisalreadytaken @Jo-Be-Co I tend to agree with @Jo-Be-Co's comments above - I am all in favour of improving the usability of this tool, but making it faster or making assumptions for the user is not necessarily the way we get there.
In fact, by making the tool "faster" we open ourselves (myself) up to a support headache when users wonder why the firmware file selected by the tool won't flash to their device, or why a listed firmware won't flash to their device, or why the tool is "recommending" certain firmwares.
I like @Jo-Be-Co's initial approach (asking for the filename and attempting to download if it is not local) because in most cases where the listed firmwares aren't what the user wants, they will have been recommended a firmware during some type of support experience (i.e. try firmware X).
Another issue with presenting a list of firmwares to the user is that this list is generated by a lot of sed/grepping which can easily break if the output changes even slightly. This has happened before and is one of the reasons I abandoned this approach previously.

You will also need to resolve the conflicts before we can merge.

You will also need to resolve the conflicts before we can merge.

@marthoc Actually I'm a little lost on this as I am new to the fork thing in github. I do work on my own fork.
But there I don't see any merge conflicts - do I need some kind of down stream merge?
Jogi

Thanks for this! Many will appreciate this feature.

Thanks for this! Many will appreciate this feature.

@marthoc You're welcome! This is just a tiny bit a can give back in reply for this well done docker package 👍
Do you think it's worth it to investigate more into the two features that where mentioned here?
I could put these into seperate pull-requests or for the beginning into one or two issues ...
List online Soruces
With the given code-snippets it would be straight forward to implement something like this:
Firmware available for flashing:
    deCONZ_ConBeeII_0x264a0700.bin.GCF
    deCONZ_ConBeeII_0x26580700.bin.GCF
    deCONZ_RaspBeeII_0x26610700.bin.GCF
    deCONZ_Rpi_0x26330500.bin.GCF
    deCONZ_Rpi_0x26350500.bin.GCF

Firmware available online for flashing (10 newest):
    deCONZ_ConBeeII_0x26420700.bin.GCF
    deCONZ_ConBeeII_0x26480700.bin.GCF
    deCONZ_ConBeeII_0x26490700.bin.GCF
    deCONZ_ConBeeII_0x26570700.bin.GCF
    deCONZ_ConBeeII_0x26650700.bin.GCF
    deCONZ_RaspBeeII_0x26650700.bin.GCF
    deCONZ_Rpi_0x26300500.bin.GCF
    deCONZ_Rpi_0x26320500.bin.GCF
    deCONZ_Rpi_0x26340500.bin.GCF
    deCONZ_Rpi_0x26370500.bin.GCF

Evaluate device list
By evaluating the device list you could

check for valid input - or are there situations where the device path you have to enter is not listed by GCFFlasher?
use the device (some sort of) automatically if it is the only listed
use the type of the device from the list (in my case ConBee II) to filter valid firmware files local and/or online

Bringing both together
The list of available firmware files could be starred or even filterd:
Firmware available for flashing:
  * deCONZ_ConBeeII_0x264a0700.bin.GCF
  * deCONZ_ConBeeII_0x26580700.bin.GCF
    deCONZ_RaspBeeII_0x26610700.bin.GCF
    deCONZ_Rpi_0x26330500.bin.GCF
    deCONZ_Rpi_0x26350500.bin.GCF

Firmware available online for flashing (10 newest):
  * deCONZ_ConBeeII_0x26420700.bin.GCF
  * deCONZ_ConBeeII_0x26480700.bin.GCF
  * deCONZ_ConBeeII_0x26490700.bin.GCF
  * deCONZ_ConBeeII_0x26570700.bin.GCF
 ** deCONZ_ConBeeII_0x26650700.bin.GCF
    deCONZ_RaspBeeII_0x26650700.bin.GCF
    deCONZ_Rpi_0x26300500.bin.GCF
    deCONZ_Rpi_0x26320500.bin.GCF
    deCONZ_Rpi_0x26340500.bin.GCF
    deCONZ_Rpi_0x26370500.bin.GCF

@Jo-Be-Co I like the look of your suggestions, I'm just slightly worried that this is going to open up a problem with support (presenting too many options to users).

@Jo-Be-Co Just as you like! We could tailor it to any "size". I agree with you that the update process should be streamlined and just a single workflow.
So its more about assistance with informations and additional checks.
Anyhow - should we put this on hold, track it with an issue or shall I commit some lines for it?


Started on Items UI

Need to wait for items endpoint.

Not sure if items will make it into MVP, because I want to change how the data is presented.  We can keep this open until we have an idea of what we want to do.

@nicholashh how can it work for you when the server side API isn't implemented? :P

LOL. @howardchung Can we have a simple server side API implemented which just lists 0 items for now. We can improve on that once this PR is merged.

thats probably equivalent to just making the action return [] instead of actually making the http call.

Hmm.. I think I will do that.

Items return [] for now.


Next button is active only after Period interaction in Simple Timer action

This is a...

[ ] Feature request
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Documentation issue or request

Description
Currently when I have Timer as start connection and I select 'Simple Timer' I have to interact with 'Period' otherwise 'Next' is inactive. I have to do this even if I want to keep the default value.

Yes we have an open issue for that.


Public assets not fully covered on Express template

What version of Remix are you using?
1.2.2
Steps to Reproduce

Create a new Remix project (npx create-remix@latest)
Choose the Express template, with or without typescript
Open the project and start it (npm run dev). The problem also exists in production mode
Open a new browser tab to see the result

Expected Behavior
We should see the Remix favicon on the browser tab
Actual Behavior
The favicon is not displayed.
I think the problem is on this line:
https://github.com/remix-run/remix/blob/c03d0fd0efa4aef57e79c6811eadee5383200e24/packages/create-remix/templates/express/server.js#L23
Indeed, we don't declare the rest of the /public folder, where the favicon is and where other assets could potentially be. Consequently, we get HTTP 404 errors when the browser tries to get these assets.

hey, I would like to work on this issue.

Fixed in https://github.com/remix-run/remix/commit/b1ce044f812a9bd47a1cffbc1e598bfb2a2e5868


Asyncio support

I forked this repo and added asyncio support. And  I added the duration parameter if the provider supports it.
I also added a new Tencent provider.
https://github.com/xizeyoupan/syncedlyrics_aio

Thanks for the contribution, I appreciate it.
Feel free to make PR so I can make some modifications if needed.
The problem with the async support is some packages like spotdl rely on the current not-async version and I think it would be incompatibility, therefore I think it is good to have it in another repo/package like yours.
Another solution is using asyncio.run to wrap all the async functions into a sync version of them, like https://stackoverflow.com/a/55365529/12696223

Yes , this is an alternative. I wish it would help if someone else needs async support.😛

thank you for your work, both @xizeyoupan and @0x7d4
the asyncio version is also great for easily timing out a search using asyncio's waiting syntax

thank you for your work, both @xizeyoupan and @0x7d4
the asyncio version is also great for easily timing out a search using asyncio's waiting syntax


Social media links not working on ios

Problem
If you use iPhone 13 either chrome or safari the medium/twitter URLs from the bottom of the website are not working on mobile version. I tried in chrome on desktop using devConsole to replicate the bug but is not working.  I think this bug is activated only on native env.
Expected Behavior
the links should be enabled so that we can open the social media accounts.
Steps to reproduce

i used an iphone 13 on chrome and safari
open any near.org page (i tried in wallet.near.org)
scroll till bottom of the page
try to click on any social media icons
the website is doing nothing.

Checked and I confirm the behavior of not showing these links, but I'm not 100% sure it is a feature and not a bug.
cc @Patrick1904 and @corwinharrell

Checked and I confirm the behavior of not showing these links, but I'm not 100% sure it is a feature and not a bug.
cc @Patrick1904 and @corwinharrell

@johny1690 thanks for reporting this. I am able to reproduce the issue where the links at the bottom of the page on near.org are unresponsive on iOS. @jakestutzman can you please check the links on near.org? I'm only able to reproduce this issue on a mobile device.
@stefanopepe we're not showing any social media links in the NEAR wallet but I can see there's an issue for adding Telegram and Twitter links: https://github.com/near/near-wallet/issues/2255.

This is fixed up


Filter Parameter Logging

hi,
i added a check for filter_parameter_logging and a test for this check.
Thx for your work,
TBAA

Done, thanks a lot man!


dynaconf_include definition precedence

When using dynaconf_include inside a configuration file, the imported file seems to get precedence over the file the inclusion is done from.  Is there a way to reverse this behavior?
Example:
base_config.yml:
default:
  val: 1

extension_config.yml:
default:
  dynaconf_include:
  - base_config.yml
  val: 2

If one now uses extension_config.yml, val will give 1 (the value defined in base_config.yml), while I would like it to give 2 instead (the value defined in extension_config.yml.

in the current implementation Dynaconf does:

Read the provided SETTINGS_FILE in your case extenson_config.yaml
Load all the variables inside it {'dynaconf_include': [paths], 'val': 2}
Check the dynaconf_include if provided and read it
Load the dynaconf_include path and load variables {'val': 1}

https://dynaconf.readthedocs.io/en/latest/guides/usage.html#loading-order
The include is happening post the file load not respecting the exact order, and I think you are right, semantically the include should respect the order where it is declared.
Right now you can solve your specific problem with:
from dynaconf import LazySettings

settings = LazySettings(
  PRELOAD_FOR_DYNACONF=["/path/*", "other/settings.toml"],    # <-- Loaded first
  SETTINGS_FILE_FOR_DYNACONF="/etc/foo/settings.py",          # <-- Loaded second (the main file)
  INCLUDES_FOR_DYNACONF=["other.module.settings", ... ]       # <-- Loaded at the end
)

https://dynaconf.readthedocs.io/en/latest/guides/advanced_usage.html#preloading-files
I'll mark this issue as a bug and I try to work on this soon, that will require some solution that can work for the variety of settings file formats we support.

Thank you for this detailed explanation!
I followed your suggestion to try PRELOAD_FOR_DYNACONF, but I am having trouble understanding it entirely.
My configurations now look like this:
base_config.yml:
default:
  val1: 1
  val2: 5

extension_config.yml:
default:
  val1: 2

and I import them like this:
settings = LazySettings(
  PRELOAD_FOR_DYNACONF=["base_config.yml"],    # <-- Loaded first
  SETTINGS_FILE_FOR_DYNACONF="extension_config.yml",          # <-- Loaded second (the main file)
  ENV_FOR_DYNACONF = "default",
  DYNACONF_WORKS="merge_configs",
  MERGE_ENABLED_FOR_DYNACONF=True
)

I now get the correct value of val1: 2. However, I cannot access val2 anymore:
AttributeError: 'Settings' object has no attribute 'VAL2'

With the merging settings I specified here, I was expecting to still be able to access val2. Is there a way to accomplish this?

But it is still not the ideal solution.
While DYNACONF_INCLUDE can be set from the main settings file itself, I have to ensure that PRELOAD_FOR_DYNACONF is set manually (by either settings the environment variable or manually configuring the settings object).

Lots of things changed on 3.0.0 and this issue is still required

This issue is added to 4.0.0 backlog, includes must be a loader feature part of #1150
loaders will handle include individually,
I am closing this issue and referencing there


[improve][API] support metrics interface in rest v2

Purpose of this pull request
current, metrics api is only support in rest v1, add this into v2

Does this PR introduce any user-facing change?

How was this patch tested?
add new test and with exist tests

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

cc @corgy-w as I can't add you into review list


Improve documentation for operations

More extensive API documentation as well as a tutorial would really be appreciated 👍

This is mostly addressed now. Assigning to 1.8.1 as I think we still want another getting started section about this...

@jlstevens  @philippjfr IMO this is resolved now, we could close this issue.


after merging of topics, the projector still shows part of the old topic

merged topic5 and topic1
at the bottom, there is a random topic, without any votes, and missing title, but just the facilitator Facilitator: user3, user12

also the moderation page has some divs still for the merged topic. they should disappear.


T stopped working after latest updates on Ubuntu/AWS

I'm getting the below error message from an Ubuntu / AWS instance (where T used to work just fine). I can run "sudo t authorize" but no other command works.
/usr/lib/ruby/1.9.1/psych.rb:203:in parse': (/home/ubuntu/.trc): could not find expected ':' while scanning a simple key at line 63 column 1 (Psych::SyntaxError) 	from /usr/lib/ruby/1.9.1/psych.rb:203:in parse_stream'
from /usr/lib/ruby/1.9.1/psych.rb:151:in parse' 	from /usr/lib/ruby/1.9.1/psych.rb:127:in load'
from /usr/lib/ruby/1.9.1/psych.rb:297:in block in load_file' 	from /usr/lib/ruby/1.9.1/psych.rb:297:in open'
from /usr/lib/ruby/1.9.1/psych.rb:297:in load_file' 	from /var/lib/gems/1.9.1/gems/t-2.9.0/lib/t/rcfile.rb:80:in load_file'
from /var/lib/gems/1.9.1/gems/t-2.9.0/lib/t/rcfile.rb:11:in initialize' 	from /usr/lib/ruby/1.9.1/singleton.rb:141:in new'
from /usr/lib/ruby/1.9.1/singleton.rb:141:in block in instance' 	from <internal:prelude>:10:in synchronize'
from /usr/lib/ruby/1.9.1/singleton.rb:139:in instance' 	from /var/lib/gems/1.9.1/gems/t-2.9.0/lib/t/cli.rb:37:in initialize'
from /var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor.rb:355:in new' 	from /var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor.rb:355:in dispatch'
from /var/lib/gems/1.9.1/gems/thor-0.19.1/lib/thor/base.rb:440:in start' 	from /var/lib/gems/1.9.1/gems/t-2.9.0/bin/t:20:in <top (required)>'
from /usr/local/bin/t:23:in load' 	from /usr/local/bin/t:23:in '

What Ruby version are you using?

ruby 1.9.3p484 (2013-11-22 revision 43786) [x86_64-linux]
With the same version T works fine on another AWS instance.
Thanks for quick response.

Did you copy your ~/.trc files onto those new instances? Is it possible those files were corrupted?
What is the output of the following command?
ruby -e 'require "yaml"; puts YAML.load_file("#{ENV.fetch(:HOME.to_s)}/.trc").to_yaml' | head -1

I get the output:
/usr/lib/ruby/1.9.1/psych.rb:203:in parse': (/home/ubuntu/.trc): could not find expected ':' while scanning a simple key at line 63 column 1 (Psych::SyntaxError) 	from /usr/lib/ruby/1.9.1/psych.rb:203:in parse_stream'
from /usr/lib/ruby/1.9.1/psych.rb:151:in parse' 	from /usr/lib/ruby/1.9.1/psych.rb:127:in load'
from /usr/lib/ruby/1.9.1/psych.rb:297:in block in load_file' 	from /usr/lib/ruby/1.9.1/psych.rb:297:in open'
from /usr/lib/ruby/1.9.1/psych.rb:297:in load_file' 	from -e:1:in '

re ~/.trc
I checked and none of the systems, 2 that are still working fine and 2 where t is not working have nothing in ~/.trc.
Also small clarification is that I've run t on the 2 instances that are not working now just fine for 6 months.

I don’t think it’s possible that it’s “working just fine” on a system without a .trc file. It might not be stored in your home directory but it’s gotta be somewhere.

Sorry my bad, I was looking for a folder with files in it. I do have ~/.trc and you were correct, the file is for some reason corrupted. It looks perfectly fine, but there is one newline and 'p' on the resulting new row. Once the 'p' is removed together with the newline, it works fine. In the other system the .trc file is completely empty. Will fix it later and report back. Thanks a lot, and sorry for the extra step in troubleshooting this.


Apply fixes from StyleCI

This pull request applies code style fixes from an analysis carried out by StyleCI.

For more information, click here.

Coverage remained the same at 69.009% when pulling 7dab5ab572f450849b166b3b04bddc34d205bd45 on analysis-qrw5Rg into facb32fc4844581d28ba3b7de54edcf2352ef2d5 on support-reindex.


feat: support passing multiple --package arguments

Description of changes:
Similar to https://github.com/rust-lang/cargo/issues/1528, support passing multiple package arguments.

Thanks!


Update readme

Request from @reyesrico . A tip or warning to user who are seeing test failures due to:

conflict tabs/windows/processes racing condition to the mock server on port 3000. It will result inconstant test results or test failures.

while running electron, minimize the window would cause browser process kick into pause state, hence, interrupt the test process, will always return false.

@photodesignch,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Stop rules on exit

This PR removes the firewall rules when the user exits the application (by clicking the X button, or closing it from the taskbar, or even when killing the process in Task Manager).
My reasoning here is that a user would probably expect the Solo Play functionality to cease when closing the app, and that they may become frustrated if they close the app and still experience solo play and/or longer-than-usual loading times when loading into an activity. Feel free to close this PR if you don't agree, but I think this would be a nice addition.
Cheers, and thank you for the nice work on this project.

You know, I had considered doing that when I first created the program, but decided not to. However, it seems to be something people want, so I'll merge the request.
Thank you for doing this, my man! 👍


Target class [JeroenNoten\LaravelAdminLte\Components\Widget\Card] does not exist.

Describe the bug
Just done composer u, now receiving Target class [JeroenNoten\LaravelAdminLte\Components\Widget\Card] does not exist.
Steps To Reproduce
Steps to reproduce the behavior:

composer u

Expected behavior
Not to break

Item
Version

Laravel
latest

Project
JeroenNoten\LaravelAdminLte

OS
various

@rightwayround components classes where moved to Views/Components to follow the Laravel way... Probably you need to clear your application caches...
php artisan cache:clear

Maybe view:clear too!

view:clear solved it, thanks. Will remember that!


Image Segmentation application

Create a demonstration of Dissolve in the context of Image Segmentation, beginning with the MSRC dataset.
The plan

Components

Loading images
Feature Function
Loss Function
Oracle Function using Factorie

This application is interesting. Is there a paper that describes this approach?

Hi Lukasz!
That’s in progress. :)
You could meanwhile checkout a mini-description at:
http://dalab.github.io/dissolve-struct/#image-segmentation-using-crfs
Or the code itself:
https://github.com/dalab/dissolve-struct/blob/tuning/dissolve-struct-examples/src/main/scala/ch/ethz/dalab/dissolve/examples/imageseg/ImageSegmentationDemo.scala
We’ll be glad to help you out in case you need more details!
Thanks,
Tribhu
—
Tribhuvanesh Orekondy
On Fri, Mar 6, 2015 at 11:43 AM, Lukasz Miroslaw<EMAIL_ADDRESS>wrote:

This application is interesting. Is there a paper that describes this approach?
Reply to this email directly or view it on GitHub:
https://github.com/dalab/dissolve-struct/issues/13#issuecomment-77539907


Allow running shipyard-cli non-interactively

Hello,
I would love to use Shipyard as part of my build process, but I need the ability to script the shipyard-cli. I've tried without luck to do this - here's an example:
$docker run -a stdin -t shipyard/shipyard-cli shipyard login <<EOF
> http://test:8080
> admin
> shipyard
> EOF
FATA[0000] cannot enable tty mode on non tty input

Is there a way to do this?

I haven't tried this but what happens if you add the "-i" flag to docker to
allocate the tty?
On Thu, Jan 29, 2015 at 6:10 PM, Branden Makana<EMAIL_ADDRESS>wrote:

Hello,
I would love to use Shipyard as part of my build process, but I need the
ability to script the shipyard-cli. I've tried without luck to do this -
here's an example:
$docker run -a stdin -t shipyard/shipyard-cli shipyard login < http://test:8080> admin> shipyard> EOFFATA[0000] cannot enable tty mode on non tty input
Is there a way to do this?
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/407.

It will still fail - I tried all different permutations of the command. I'll paste each result for completeness:
$docker run -a stdin -it shipyard/shipyard-cli shipyard login <<EOF
> http://test:8080
> a
> b
> EOF
FATA[0000] cannot enable tty mode on non tty input

$docker run -it shipyard/shipyard-cli shipyard login <<EOF
> http://test:8080
> a
> b
> EOF
FATA[0000] cannot enable tty mode on non tty input

$docker run -i shipyard/shipyard-cli shipyard login <<EOF
> a
> b
> c
> EOF
URL: Username: Password: panic: inappropriate ioctl for device

I think I have this working.  First, I ran the shipyard/shipyard-cli image and logged into it:
$ sudo docker run -it shipyard/shipyard-cli /bin/bash
shipyard cli> shipyard login
...
shipyard cli> exit

After exiting, that container stops, but its state is saved.  Run docker ps -a to find it, then commit it to a new image.  In this example, the random name of that container was trusting_torvalds:
$ sudo docker commit trusting_torvalds local/shipyard-cli

Now you have a saved local Docker image that is "pre-authenticated."  If you want to script some Shipyard CLI stuff, you can just use that image like so:
$ sudo docker run -it --rm local/shipyard-cli shipyard info
Controller Version: 2.0.8
Cpus: 4.00
Memory: 8096.00 MB
Containers: 3
Images: 29
Engines: 1
Reserved Cpus: 6.00% (0.24)
Reserved Memory: 9.49% (768.00 MB)
$

You could go even further and wrap that Docker command into a shell script that accepts input and pipes input right into the command:
#!/bin/bash

# use the local/shipyard-cli image (customized with pre-authentication)
sudo docker run --rm local/shipyard-cli "$@"

Nice, that works great. Thanks for the solution!


Installing the extension breaks Ctrl-r

Expected Behavior
Installing the extension should not disabled default keyboard shortcuts.
Actual Behavior
Installing the extension breaks the keyboard shortcut Ctrl-r.
Steps to Reproduce the Problem

Install the extension
Try opening the file/project history using Ctrl-r
It does not work (asks for a chord)

Specifications

Version: VS Code 1.41.1, vscode-yarn 1.7.0
Platform: Linux

+1


terminal.integrated.automationProfile.windows not working at all

Does this issue occur when all extensions are disabled? Yes

VS Code Version: 1.78.1 (user setup)
OS Version:  Windows 11

Steps to Reproduce:

configure terminal.integrated.automationProfile.windows setting as shown below
with no terminal open, start a debugging session
the newly spawned debugging terminal does not respect the  terminal.integrated.automationProfile.windows setting.

I want to pass command line arguments to the shell the debugger terminal is about to use (PowerShell). The documentation, Configuring the task/debug profile, states I can use the key, terminal.integrated.automationProfile.<platform> to do this.
This simply does not seem to work at all, I have tried several different set ups (fresh code, different machines etc), nothing works.
All extensions are disabled accept for The PowerShell-ms extension (version 2023.6.0), in order to demonstrate this issue.
If in my user settings I only have the following:
{
 "terminal.integrated.automationProfile.windows": {

	  "path": "C:\\Program Files\\PowerShell\\7\\pwsh.exe",
	  "args": [
			"-noexit",
		 "-command",
		 "Get-ChildItem",     //I know this makes no sense, its just for an example
	]
  },
}

and a single config in my launch.json file:
{
    "version": "0.2.0",
    "configurations": [
        {
            "name": "PowerShell: Launch Current File",
            "type": "PowerShell",
            "request": "launch",
            "script": "${file}",
            "args": []
        }
    ]
}

With no terminals open, starting a debugging session with the active file, shell.ps1 spawns a new terminal but it is run as if the setting terminal.integrated.automationProfile.windows is not in place:

In fact, entirely different command line arguments are being passed to the PowerShell executable, as shown here:

I have run into this issue with the Python-ms Debugger and PowerShell-ms Debugger.
With the standard Code terminal I can pass command line arguments to its PowerShell executable like this, and it works as shown HERE:
"terminal.integrated.profiles.windows": {
 "General": {
    "path": "C:\\Program Files\\PowerShell\\7\\pwsh.exe",
    "args": [
        "-noexit",
      "-command",
      "Get-ChildItem",
  ]
 }
}

VSCode information
Version: 1.78.1 (user setup)
Commit: 6a995c4f4cc2ced6e3237749973982e751cb0bf9
Date: 2023-05-04T09:48:08.683Z (4 mos ago)
Electron: 22.5.1
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Sandboxed: No

System information
CPUs	AMD Ryzen 9 5950X 16-Core Processor (32 x 3400)
GPU Status	2d_canvas: enabled canvas_oop_rasterization: disabled_off direct_rendering_display_compositor: disabled_off_ok gpu_compositing: enabled multiple_raster_threads: enabled_on opengl: enabled_on rasterization: enabled raw_draw: disabled_off_ok video_decode: enabled video_encode: enabled vulkan: disabled_off webgl: enabled webgl2: enabled webgpu: enabled
Load (avg)
Memory (System)	127.93GB (98.70GB free)
Screen Reader	no
VM	0%

A/B experiment info
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstes627:30244334
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vsdfh931cf:30280410
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vsclangdc:30486549
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30803844
3biah626:30602489
f6dab269:30613381
a9j8j154:30646983
showlangstatbar:30737416
a2ce3375:30757347
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
asynctok:30821568
dsvsc013:30795093
dsvsc014:30804076
diffeditorv1:30821571
dsvsc015:30829745

@roblourens @connor4312 does debugging respect the automation shell? I'm pretty sure it did in the past if not.
This is where tasks uses it for example:
https://github.com/microsoft/vscode/blob/5a031b7f040363a3039da6cd9e4b80326c43d015/src/vs/workbench/contrib/tasks/browser/terminalTaskSystem.ts#L1086-L1090

I think something is wrong, we ask for the automation profile here https://github.com/microsoft/vscode/blob/main/src/vs/workbench/api/node/extHostDebugService.ts#L85
and I set
"terminal.integrated.automationProfile.osx": {
  "path": "/bin/bash",
},

but that profile isn't returned, I get the default zsh profile. Am I missing something?

@roblourens do you have a simple launch.json that I could test this against that doesn't use the pwsh extension?
I see tasks is working fine:
  "terminal.integrated.automationProfile.windows": {
	  "path": "C:\\Users\\Daniel\\AppData\\Local\\Microsoft\\WindowsApps\\Microsoft.PowerShell_8wekyb3d8bbwe\\pwsh.exe",
	  "args": [
			"-noexit",
		  "-command",
		  "Get-ChildItem",     // I know this makes no sense, its just for an example
	  ]
  },

I just used something like this
{
"name": "Run",
"type": "node",
"request": "launch",
"program": "${file}",
"console": "integratedTerminal",
},

The automation profile on the ext host doesn't have the args in it:

Looks like we just don't react to changes when automation profile changes, updating the default profile after setting the automation profile will trigger the update and then work properly.

@RalfReddings  Hello, looks like this issue got completed. I am not very familiar with Github. Is it safe to assume that the fix will be present in the next VSCode update? Also thank you both for working on this, I really appreciate it.

@RalfReddings we're just about to release 1.83 and it won't be in that one unfortunately, it'll be shipped in a month to VS Code, or in a few days to VS Code insiders.

Okay, got it!


Add zuul job to run on OSP18

Depends-On: https://github.com/infrawatch/feature-verification-tests/pull/82

Build failed (check pipeline). Post recheck (without leading slash) to rerun all jobs. Make sure the failure cause has been resolved before you rerun jobs.
https://review.rdoproject.org/zuul/buildset/dd0d10057206429ca183b275c15ecea9
✔ openstack-k8s-operators-content-provider  SUCCESS in 1h 06m 50s ❌ functional-tests-on-osp18  FAILURE in 48m 40s

- import_role:
        name: '../name/telemetry_autoscaling'
              ^ here

stderr_lines:
That failure is my fault. I've pushed updates today.

recheck

recheck

recheck

recheck

I updated the branch in telemetry_autoscaling to pactch the openstack resources  and include the required versions of aodh and heat

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

recheck

I updated the efoley_dev branch in telemetry_autoscaling to remove the telemetry_autoscaling role and add it to https://github.com/infrawatch/feature-verification-tests/pull/82
This PR should still work, since the target playbook in telemetry_autoscaling/playbooks/autoscaling_osp18.yaml will clone FVT and run the tasks from there.

recheck


Opaleye is Postgres only

at least until someone adds support for more backends

@tomjaguarpaw Nice spot. Thank you for the contribution!


Add a loop to wait unitl Prereq Installer is ready

I have my Prereq Installer on a share and this share is not immediately after reboot ready. If I set DSC to reboot automatically, the Module fails.
I added a wait loop for this purpure for the first access to the InstallerPath.
I was unable to create a issue (Submit Button was grayed out), so I can discuss this front up.
The delay and timeout is currently not configurable. Should this changed?

[x] Change details added to Unreleased section of changelog.md?
[ ] Added/updated documentation and descriptions in .schema.mof files where appropriate?
[ ] Examples updated for both the single server and small farm templates in the examples folder?
[x] New/changed code adheres to Style Guidelines?
[x] Unit and Integration tests created/updated where possible?

This change is

Hi @StefanSchoof, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@StefanSchoof, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;

Just a thought:
Isn't this handled by DSC automatically? If after a reboot the share isn't available immediately, the resource fails but after 15 minutes DSC tries again and then is able to reach the share and continues.

I try to Set up a Dev Environment. Losing 15 minutes because a Share needs some seconds more is a long delay
Am 12.10.2016 17:09 schrieb Yorick Kuijs<EMAIL_ADDRESS>Just a thought:
Isn't this handled by DSC automatically? If after a reboot the share isn't available immediately, the resource fails but after 15 minutes DSC tries again and then is able to reach the share and continues.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/PowerShell/SharePointDsc/pull/425#issuecomment-253241639, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AEcjBxzqi2uSKotn8hiU1b56IHvl7k8Eks5qzPg4gaJpZM4KUofo.

Rechtliche Hinweise:
solutions direkt AG I Vorstand I Dipl.-Inform. Markus Breilmann I Dipl.-Kfm. Kai Lehmann I Aufsichtsrat (Vorsitz) I Dipl.-Inform. (FH) Nils Schultz I Sitz Hamburg I AG Hamburg HRB 140889 I USt-IdNr.  Die Nachricht ist durch das Briefgeheimnis geschuetzt und unterliegt gegebenenfalls den Regeln zum Schutz der Vertraulichkeit. Jede Benutzung, Versendung, Herstellung von Kopien oder Veroeffentlichung durch andere Personen ist ohne Zustimmung des Absenders untersagt. Wenn Sie diese Nachricht irrtuemlich erhalten haben, bitten wir Sie hoeflichst, sie auf Ihren Systemen zu loeschen und den Absender umgehend zu benachrichtigen.
This electronic mail transmission contains confidential information intended only for the person(s) named. It is subject to the laws of mail secrecy and may be protected by legal privileges. Any use, distribution, copying or disclosure by another person is strictly prohibited without the consent of the sender. If this transmission has been received in error, you are kindly requested to delete it from your system and to contact the sender immediately.
?

Got a point there, especially since this change is pretty small but has a high impact.
I do have two small comments on your code, see below.

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.

Modules/SharePointDsc/DSCResources/MSFT_SPInstallPrereqs/MSFT_SPInstallPrereqs.psm1, line 142 at r1 (raw file):

    )

    # This is the first access to the installer binary. If not yet exist (e.g. network share) give it a some time

Please move the code just below the Write-Verbose line, so the first thing the method does is displaying verbose information.

Modules/SharePointDsc/DSCResources/MSFT_SPInstallPrereqs/MSFT_SPInstallPrereqs.psm1, line 148 at r1 (raw file):

        sleep -Seconds 10
        $retryCount--
    }

Wouldn't it be nicer that if after three attempts the file still isn't found, an error is thrown with a good descriptive message?

Comments from Reviewable

Can I put another idea out there? I feel this is a bit of a niche scenario so I would rather not put it in to the main resource (especially since if the network share is not available for other reasons you are adding a wait for before an error is shown there).
I would suggest you use the out of the box script resource to detect if your share is available, and then make your PreReq installer wait for that one.
Script WaitForNetworkShare
{
    GetScript { return @{} }
    SetScript { }
    TestScript {
        [Your logic here to test if the share is up, including the wait]
    }
}

SPInstallPreReqs InstallPrereqs
{
    [Properties here]
    DependsOn = "[Script]WaitForNetworkShare"
}

@ykuijs what do you think?

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.

Comments from Reviewable

I just assumed that if a DSC Module requires a restart, after this Restart DSC starts at this Module And does not Test the modules before. I will test it. If it works it is cleaner sloution and keepts the SharePointDsc Module Clean from things that are Not needed.

Rechtliche Hinweise:
solutions direkt AG I Vorstand I Dipl.-Inform. Markus Breilmann I Dipl.-Kfm. Kai Lehmann I Aufsichtsrat (Vorsitz) I Dipl.-Inform. (FH) Nils Schultz I Sitz Hamburg I AG Hamburg HRB 140889 I USt-IdNr.  Die Nachricht ist durch das Briefgeheimnis geschuetzt und unterliegt gegebenenfalls den Regeln zum Schutz der Vertraulichkeit. Jede Benutzung, Versendung, Herstellung von Kopien oder Veroeffentlichung durch andere Personen ist ohne Zustimmung des Absenders untersagt. Wenn Sie diese Nachricht irrtuemlich erhalten haben, bitten wir Sie hoeflichst, sie auf Ihren Systemen zu loeschen und den Absender umgehend zu benachrichtigen.
This electronic mail transmission contains confidential information intended only for the person(s) named. It is subject to the laws of mail secrecy and may be protected by legal privileges. Any use, distribution, copying or disclosure by another person is strictly prohibited without the consent of the sender. If this transmission has been received in error, you are kindly requested to delete it from your system and to contact the sender immediately.
?

It does not work. I added
        Script WaitForNetworkShare
        {
            GetScript = { return @{} }
            SetScript = { }
            TestScript = {
                $retryCount = 3
                while (-not (Test-Path -Path "C:\SPInstall\prerequisiteinstaller.exe") -and $retryCount -gt 0)
                {
                    sleep -Seconds 10
                    $retryCount--
                }
                return $true
            }
        }

And get the error:
Error Message: The term 'C:\SPInstall\prerequisiteinstaller.exe' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.
I think since the reboot is requested by the SPInstallPrereqs module, does not test the Dependencies after the reboot again.

Every time DSC performs a compliancy check, it starts from top to bottom and follows the configured dependencies. So after a reboot, the compliancy check is kicked off and (if everything is ok) all resources before the one that led to the reboot should be compliant and the Set method not ran anymore for those resources. This means that the Script resource is always evaluated.
Which resource is throwing the above error in your last test? It looks like the script isn't doing very much. Can you add some logging to the script to see if it is evaluated?? For example an Add-Content to a file with the timestamp, so you know if it runs before and after the reboot.
Second: How do you access the file share? Via UNC notation (\server\share) or via a drive mapping (mapping \server\share to S: and then accessing that)??

I agree with @BrianFarnhill that this is a generic issue, which should be resolved in the resource. Especially since I suspect you might run into the same issue in the SPInstall resource, assuming the setup file is in the same location as the prerequisitesinstaller.exe.

Okey, I get It to work. (I do not know what I did wrong earlier, maybe not saved the file :( )
Thank you all. I think this is a good solution.


Expand round trip api to block bodies

Builds on https://github.com/ethereum/py-evm/pull/1159
What was wrong?
Fixes https://github.com/ethereum/py-evm/issues/1131
How was it fixed?
This finishes expanding the round trip API for the eth protocol to all of Headers, NodeData, Receipts, and BlockBodies request/response pairs.  The FastChainSyncer and RegularChainSyncer classes have been reworked to use this API.
Cute Animal Picture

Anyone who reviews should focus on the latest commit as this builds from another open pull request.

Something that jumped out at me was the final state of the FastChainSyncer and how much easier it is to read and understand with this pattern.  Makes me think that once we have this in place across the board that implementing the more complex syncing algorithms that we've been talking about is going to be a lot easier.


[SPARK-35035][PYTHON] Port Koalas internal implementation unit tests into PySpark

What changes were proposed in this pull request?
Now that we merged the Koalas main code into the PySpark code base (#32036), we should port the Koalas internal implementation unit tests to PySpark.
Why are the changes needed?
Currently, the pandas-on-Spark modules are not tested fully. We should enable the internal implementation unit tests.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Enable internal implementation unit tests.

Test build #137260 has started for PR 32137 at commit 0436734.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41839/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41839/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41839/

Test build #137260 has finished for PR 32137 at commit 0436734.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137260/

CC @ueshin @HyukjinKwon @itholic

Conflicts are about the order of test names in the test module.
I'm not sure which ongoing PR will be merged first.
I would prefer to resolve conflicts when other parts look good.

Test build #137297 has started for PR 32137 at commit 5860f8a.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41877/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41877/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41877/

Test build #137308 has started for PR 32137 at commit cf58ab2.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41888/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/41888/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/41888/

Test build #137308 has finished for PR 32137 at commit cf58ab2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/137308/

Merged to master.


Use cwd of Jupyter server when looking for files that could interfere with startup of kernels

Assume this is the kernel startup folder structure

-> Folder 1
    |-> sample.ipynb
    |-> Folder 2
        |-> random.py
        |-> sample2.ipynb

Disable zmq
Open file sample2.ipynb and run a cell
It will fail with the error message about random.py, (as expected)
Now open sample.ipynb, and this still fails without any reason apart from invalid kernel

The problem is jupyter server was started from Folder 2, hence random.py will always interfere with all notebooks in all directories for the same VS Code session.

This is a very edge case scenario, hence closing this.
After all, when the first notebook/IW is started user will be display a meaningful error message about the file random.py that could potentially cause issues.


Translation relationship fields should be refactored from selects to tabs

At the moment, relationships are being generated as fields, with the select field type, which is great.
However, the generator should be smart enough to detect translation relationships.
Those relationships shouldn't be defined as select fields. Instead we should add all translation fields to the translation tabs. See screenshots below.

should become more like...

fixed on development