[SPARK-44770][WEBUI] Add a displayOrder variable to WebUITab to specify the order in which tabs appear

What changes were proposed in this pull request?
Add a displayOrder variable to WebUITab to specify the order in which tabs appear. Currently, the tabs are ordered by when they get attached, which isn't always desired. The default is MIN_VALUE, meaning if it's not specified, it will appear in the order added before any tabs with a non-default displayOrder.
Why are the changes needed?
These changes allow control over the order which tabs appear in the UI. Currently, the tabs are ordered by when they get attached, but this doesn't always match the desired order. For example, we would like to have the SQL Tab appear before the Connect tab; however, based on the code flow, the Connect tab will be attached first and with the current logic, that tab would also appear first.
Does this PR introduce any user-facing change?
Yes, minor user-facing change affecting the order in which tabs appear in the Spark UI
How was this patch tested?
Manually tested
Before:

After:

Thanks, merging to branch-3.5


B 21000 main - Allow customer to edit and view Counseling Office

Agility ticket
Summary
We want to give the customer the following capabilities:
Allow the customer to edit the counseling office.
Allow the customer to review the counseling office.

[!IMPORTANT]
Contains 20465 Code
Referencing PR
B-20465-INT - Add Counseling Office Selection to Customer Application

How to test
Create move

Login as Customer
Create a move with Origin duty location as Scott AFB
Select a counseling office for the move and submit as normal

Once move is created

Select to view current move
Verify the counseling office is displayed in the orders summary
select edit on the orders summary
change the counseling office and submit
verify the counseling office is displayed in the orders summary
selected edit on the order summary
change the origin duty location to NS Mayport
verify the counseling office is blank in the orders summary

508 Screenshots
B-21000-508Screenshots.pdf
Integration PR
Integration PR INT-B-21000

Test coverage on server failed.
Last run: Thu 17 Oct 2024 04:21:28 PM UTC
Refer to this report to see coverage details for the server app.
Documentation for test coverage can be found at this link.

Test coverage on client failed.
Last run: Thu 17 Oct 2024 06:38:56 PM UTC
Refer to this report to see coverage details for the client app.
Documentation for test coverage can be found at this link.


how to send through email generated pdf document

Thank you for submitting an issue to jsPDF. Please read carefully.
Are you using the latest version of jsPDF?
Have you tried using jspdf.debug.js?
Steps to reproduce
Ideally a link too. Try fork this http://jsbin.com/rilace/edit?html,js,output
What I saw
What I expected

pdf is generating that is not a problem. i want to take that pdf file and need to send to back end and attach with mail. i cannot take pdf file to back end .

pdf is generating that is not a problem. i want to take that pdf file and need to send to back end and attach with mail. i cannot take pdf file to back end .

i've the same problem.

Solution,,
You have to send file as imageUrl from font end canvas.toDataURL('image/png')..
In back end   buf = new Buffer(req.body.imageBinary.replace(/^data:image/\w+;base64,/, ""), 'base64'); create buffer of base 64, as image or pdf format..
var data = {
Key: fileName,
Body: buf,
ContentEncoding: 'base64',
ContentType: 'pdf',
ACL: "public-read-write"
};
the data objec will be pdf file. you can store or can attache in email

Solution,,
You have to send file as imageUrl from font end canvas.toDataURL('image/png')..
In back end buf = new Buffer(req.body.imageBinary.replace(/^data:image/\w+;base64,/, ""), 'base64'); create buffer of base 64, as image or pdf format..
var data = {
Key: fileName,
Body: buf,
ContentEncoding: 'base64',
ContentType: 'pdf',
ACL: "public-read-write"
};
the data objec will be pdf file. you can store or can attache in email

Frontend (Angular):
var data = pdf.output('blob');
var formData = new FormData();
formData.append("pdf", data, "myfile.pdf");
  var request = new XMLHttpRequest();
  request.open("POST", "http://localhost:3000/caseforms", true);
  request.responseType = "blob";
  request.send(formData);

backend(Nodejs) i've got gibberish raw file:
%PDF-1.3
%�߬�
3 0 obj
<</Type /Page
/Parent 1 0 R
/Resources 2 0 R
/MediaBox [0 0 595.28 841.89]
/Contents 4 0 R

endobj
4 0 obj
<<
/Length 10
How to send a file or how to convert pdf file in backend.
Thanks for quick response..


Simplify forSome statements into wildcard existentials (fix #471)

Also fixes #353
This will prevent unreadable existential types from being ascribed, as well as circumventing a bug where an existential type might be qualified: Box[MyScope._1] forSome { type _1 ... }.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Win Wang seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Update docker_installation.md

Tags are outdated:

Thanks for your contribution @mkania0 and for discovering this!
We will have to change that section of the doc to not have any reference to a hard-coded version as it will be too much of hassle to change it every time we release a new version. See #43.

Also, I think we should switch to ghcr.io in our documentation too, since that is our new canonical container registry, even if we still continue pushing to Docker Hub


Make SettingsVersion parameter mandatory in Failover-GcSqlInstance cmdlet

Make SettingsVersion parameter mandatory in Failover-GcSqlInstance cmdlet. For consistency with API and other cmdlets.

LGTM


Fix comment around safari fingerprint check

Fixes #2442

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Didn't apply cleanly to 1.1.0 so I only applied it to master.


mailbox: port friendly take and destroy message functions

This change is a simpler attempt to fix the same issue that #459 PR tried to handle.
Rather than making generic for any message, let's just focus on taking the first message (and removing it later).
These changes are made under both the "Apache 2.0" and the "GNU Lesser General
Public License 2.1 or later" license terms (dual license).
SPDX-License-Identifier: Apache-2.0 OR LGPL-2.1-or-later

Tested and it works. Ready for review.

I also renamed mailbox_take to mailbox_take_first.

Superseded from #424


Loading LDES Configuration File won't start Docker Container for harvesting LDES feed

I am experiencing a problem with the vocabserver-app project involving the loading of a  configuration file with a LDES source specified. A configuration file that uses an LDES feed with mapping provided is located at: bodc_p02.vocab-config.ttl.
Expected Behavior:
After loading the above configuration file, it is expected that a Docker container will start which will harvest the bodc_p02 LDES feed. Additionally, the sources should be displayed for the created vocabulary. After the LDES feed is harvested the mapping and unification should be done automatically.
Actual Behavior:
Upon loading the configuration file, no extra Docker container starts for the LDES harvesting and no sources are displayed for the created vocabulary.
Steps to Reproduce:

Load the configuration file from the provided link.
Observe that no extra Docker container starts.
Check the sources for the created vocabulary and note that none are shown.

Could you confirm this has been fixed @cedricdcc ? (make sure to pull the latest vocab-configs image)
ref: https://github.com/vlizBE/vocabserver-app/commit/7681eab2695debb6d44e17ff60f28e907d710441

When uploading a config .ttl file the vocab-config image initiates a container that harvest an LDES feed but it initiates 2 of them at the same time.

The config service works with intermediate processing graphs. Insertions of the same dataset in multiple graphs mislead the delta processing in the ldes-consumer-manager to think there were multiple ldes datasets inserted. See https://github.com/redpencilio/ldes-consumer-manager/commit/f43445eb4847691cf3c3c9dc6ac97452da15dbe3 for more background.


Fix for Issue #9

Found this while refactoring for variables/feedback.
You will need to re-drag the preset onto the button to update

Does this mean that these presets doesn’t work in earlier versions?
Have this been included in 1.3 or earlier? If so, this would be a good time to use addUpdateScript

This is a fix for  #9 , an issue since the presets were first added to the QLab module.
A button created from the "Autoload enable" preset won't fire due to a case mismatch.
Any buttons created earlier have the wrong action stored in the config.
This patch fixes the preset so new (or re-created) buttons from the preset will work.
I haven't looked at the addUpdateScript to see if this can be fixed automatically.

I've included 'addUpgradeScript' which should take care of button actions created by the earlier presets.


AttributeError: 'str' object has no attribute 'copy'

Describe the bug
some errors related to file operations
Setup and configuration
Current version:

commit e0c7492609bf8ebdbd7270c9e9261bf2944f3bc6 (HEAD -> main, origin/main, origin/HEAD)
Author: Boxuan Li<EMAIL_ADDRESS>Date:   Fri Apr 12 12:01:52 2024 -0700

    Traffic Control: Add new config MAX_CHARS (#1015)

    * Add new config MAX_CHARS

    * Fix mypy linting issues

My config.toml and environment vars (be sure to redact API keys):
LLM_API_KEY="ollama"
LLM_MODEL="ollama/atlas:002"
LLM_EMBEDDING_MODEL="local"
LLM_BASE_URL="http://localhost:11434"
WORKSPACE_DIR="./workspace/"
SANDBOX_TYPE="exec"

My model and agent (you can see these settings in the UI):

Model:
Agent:

Commands I ran to install and run OpenDevin:

Steps to Reproduce:

run opendevin

Logs, error messages, and screenshots:
==============
STEP 13

PLAN:
write a bash script that prints hi

ERROR:
'str' object has no attribute 'copy'
Traceback (most recent call last):
  File "/home/atlas/OpenDevin/opendevin/controller/agent_controller.py", line 145, in step
    action = self.agent.step(self.state)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/atlas/OpenDevin/agenthub/monologue_agent/agent.py", line 219, in step
    action = prompts.parse_action_response(action_resp)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/atlas/OpenDevin/agenthub/monologue_agent/utils/prompts.py", line 176, in parse_action_response
    return action_from_dict(action_dict)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/atlas/OpenDevin/opendevin/action/__init__.py", line 31, in action_from_dict
    action = action.copy()
             ^^^^^^^^^^^
AttributeError: 'str' object has no attribute 'copy'

OBSERVATION:
'str' object has no attribute 'copy'

Additional Context

Is this still happening? I thought it's fixed as I had the same issue.

Is this still happening? I thought it's fixed as I had the same issue.

as of 20 minutes ago, yes.

deplicated with #718

Close it. Prefer only keep one issue tracking it.


Use BCH unit on email & push notifications

Coverage decreased (-0.004%) to 83.277% when pulling f0a48e121718cf10389d8279bbcb071984b57aa1 on isocolsky:bch-unit into 03a61f68f2b8922d0443302cf0bc3c5cf9bb44a6 on bitpay:master.


current master macOS jobs with OpenMPI fail

-- The C compiler identification is GNU 9.1.0
-- The CXX compiler identification is GNU 9.1.0
-- Checking whether C compiler has -isysroot
-- Checking whether C compiler has -isysroot - yes
-- Checking whether C compiler supports OSX deployment target flag
-- Checking whether C compiler supports OSX deployment target flag - yes
-- Check for working C compiler: /usr/local/bin/gcc-9
-- Check for working C compiler: /usr/local/bin/gcc-9 -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Checking whether CXX compiler has -isysroot
-- Checking whether CXX compiler has -isysroot - yes
-- Checking whether CXX compiler supports OSX deployment target flag
-- Checking whether CXX compiler supports OSX deployment target flag - yes
-- Check for working CXX compiler: /usr/local/bin/g++-9
-- Check for working CXX compiler: /usr/local/bin/g++-9 -- works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Could NOT find MPI_C (missing: MPI_C_LIB_NAMES MPI_C_WORKS)
-- Could NOT find MPI_CXX (missing: MPI_CXX_LIB_NAMES MPI_CXX_WORKS)
CMake Error at /usr/local/Cellar/cmake/3.14.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find MPI (missing: MPI_C_FOUND MPI_CXX_FOUND)
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.14.2/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:378 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/Cellar/cmake/3.14.2/share/cmake/Modules/FindMPI.cmake:1672 (find_package_handle_standard_args)
  CMakeLists.txt:72 (find_package)
-- Configuring incomplete, errors occurred!
See also "/Users/travis/build/microsoft/LightGBM/build/CMakeFiles/CMakeOutput.log".
make: *** No rule to make target `_lightgbm'.  Stop.

https://travis-ci.org/microsoft/LightGBM/jobs/568163839
Possible causes:

https://github.com/Homebrew/homebrew-core/commit/617b2026ac132850bb5637183a645ae9d171fca7
https://github.com/Homebrew/homebrew-core/commit/a94e3b3cd5b5a92944d9c809debae6b8934f3dda

Ping @henry0312, @jameslamb as macOS users.

Seems to be resolved. Maybe in this commit https://github.com/Homebrew/homebrew-core/commit/3b365ff570bad99d5ce4814e53ae809ac66a9f16, idk.


Introducing the Catalyst Community Partner Highlights series

This issue tracks the drafting, review and approval for adding a new blog post to the Catalyst Project blog.
This issue is complete when: the copy of this page is approved and published on the website.
This blog post describes the initial collection of  seven community partner highlights, and will be cross-posted to CSCCE, 2i2c, and OLS blogs as part of a coordinated comms plan (available here: https://docs.google.com/document/d/1NbWQ8Xf3aRXOyHPBxhMuw-2K5SPLB7xIkuoROD-9zec/edit )
Please edit and insert information enclosed by the  below
Create a Google Doc in Catalyst Project Team Drive > External Documentation > Blog posts > your_filename to draft the copy for your new page, with a title of the form "[VERSION #] <your_title>" and append this title with "(ES)" if the draft is in Spanish.
Please consult the Style Guide for more information, including which metadata tags to choose from for your blog post.
Google doc for drafting copy: https://docs.google.com/document/d/1S0nYUa_igMINAd54bmPBn49ZYU8ISg779SIws3bEhbw/edit
Roles

[x] Drafting content: @katiepratt
[x] Sub-team review: @LouWoodley
[x] Sub-team review: @jnywong
[x] Sub-team review: @Tajuddeen1
[x] Hand off to @katiepratt for final review @katiepratt
[x] Hand off to MD for translation @SLLDeC / @4iro
[x] Hand off to @jnywong  to draft website previews - 2 separate posts
[ ] Final approval of English post by Katie @katiepratt
[ ] Final approval of Spanish post by Sabrina @SLLDeC / @4iro

I'm adding Laura to this issue so she can manage the translation step in case it happens while I'm out of the office.

Hey @katiepratt ! I've left suggestions for the draft in the doc. I really like how concise this overview is in introducing the series!
One thing to bear in mind is that if we want to publish by the end of this month before the IOI community call, then we will need to get a move on with reviews and translation. @lauracion let us know if you need more notice for the translations.

Thanks, @jnywong !! I've accepted all of these changes. @LouWoodley we're just waiting on your feedback on this post before it can be translated, if you have time when you're back in the office tomorrow (Tuesday).

@jnywong, Irene Vazano, @4iro,  will answer your question. Can you please add her to this project so she can be added and tagged on these issues? Thank you!

The English version of this blog post is all signed off and ready for translation, thank you @4iro ! (cc @jnywong )

Hello @jnywong and @katiepratt, we can deliver the translation on October 28th. Is that time convenient for publishing and reviewing?

@4iro October 28th sounds great, thank you for coordinating 🙌

Hi @jnywong, the translated document is available here.
Please let me know if there is something else that needs my attention. Thank you!

Website previews are available to preview.
Spanish preview 👉 https://catalyst-website--106.org.readthedocs.build/en/106/blog/community-highlight-intro-es.html
English preview 👉 https://catalyst-website--106.org.readthedocs.build/en/106/blog/community-highlight-intro-en.html
You can add comments directly to the pull request here: https://github.com/czi-catalystproject/czi-catalystproject.github.io/pull/106/files (hover over the line you want to make a suggestion, press the small '+' button that pops up, and leave a comment)

@4iro let me know if https://catalyst-website--106.org.readthedocs.build/en/106/blog/community-highlight-intro-es.html looks good to you

@4iro let me know if https://catalyst-website--106.org.readthedocs.build/en/106/blog/community-highlight-intro-es.html looks good to you

@jnywong done through #110


Management API layer refactoring and CollectorService

Goals:

Extracted management registry api to terracotta-management so that other products could re-use the same mechanism to expose their stats and capabilities
Added Ehcache service that uses the management registry and its configuration to frequently send stats and also sends ehcache notifications to a management message consumer.

Some details:

Frequency to send stats is by default based on the TimeToDisable (75% of this value). Default value for ttd is 30 seconds, so by default, stats would be sent each 22 seconds.
Depends on new version 2.1.0 of terracotta-management (not yet released)
New module dependency added: terracotta-management-registry (besides current existing terracotta-management-model)
Test / example added: DefaultCollectorServiceTest

test this please

@chrisdennis @lorban @ljacomet @anthonydahanne : terracotta-management 2.1.0 has been release, so this PR is complete now.
Also, after discussing with Anthony, we removed the cluster topology classes from terracotta-management, to only keep monitoring and management metadata inside this lib.

@chrisdennis : PR updated. Could you please check again (specifically for the racy code, if it is fixed correctly). Thank you!

@ljacomet @chrisdennis @lorban : just rebased.
Please tell us (@anthonydahanne and I) if anything else is required for this to be merge :-)
Thank you!

test this please

From my point of view this looks okay now... might want to get @anthonydahanne to review the management aspects of it, but other than that it's fine.

test this please

@chrisdennis  could you please merge ? LGTM


⚠ Jenkins Plugins Artifactory has degraded performance

In 1aa4c94, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1158 ms

Resolved: Jenkins Plugins Artifactory performance has improved in b8ca46a after 12 minutes.


Q: What do you think about implement layer depth logic?

Tx for good ideas 👍🏻 I opened new(old) world today when reading yours articles🤓
What do you think about implement layer depth logic with colour differentiation?
I’m experimented with other scss framework spectre.css (maybe you know)… & it’s haven’t darkened & depths color logic… & I trying add this something like:
<div class="deps deps-background">

If I understand your question correctly, you could achieve this by using rgba or hsla values when defining colors in the $feo-colors array. This can be achieved by hand, you you can write a function that generates the array for based on some settings? You could have a black background with light colors with a small opacity, to create the effect you have in the image.
If you don't want to work with opacity hex values, you could go with hsl where you keep the h and s the same, but generate an array of values where the l changes. Below is for instance the grayscale color palette I use on my personal website. Although its just 4 colors, it shows how you can be in the same tone.
$h: 218;
$s: 25%;

$feo-colors: (
  'gray-100': hsl($h, $s, 98%),
  'gray-200': hsl($h, $s, 70%),
  'gray-300': hsl($h, $s, 18%),
  'gray-400': hsl($h, $s, 9%),
);

Is that you are looking for?

Yep, it’m understood. I’m using now same logic.
And how I can generate classes by tokens in theme & this approach need mixin in class:
.deps {
color: var(—color);
background-color: var(—back);
}
P.s. transparency logic not useful with overlaying layers…

Just for example color-layer-depth logic...
In dark mode :hover, :focus element color lighter then background color

In light mode :hover, :focus element color darker then background color

But for true logic mast be lighter then background color


Autosave in cache unsaved files

Add an autosave feature in localstorage
The document is saved every ~2 minutes and cache is delete when the editor is closed or the file is save

Maybe a quick save button in a corner of the editor ?

Done 👍
in 1759acdba16da5e436b79cdefd9ce547de5d1c1a

Interesting feature!


automate post-release validation

homebrew test
prod artifacts version check

This can be a Prowjob that runs brew install against the latest release of EKS-A.


add Dockerfile.

based on:
https://github.com/fybrik/fybrik/blob/master/connectors/katalog/Dockerfile
Signed-off-by: Doron Chen<EMAIL_ADDRESS>

addresses https://github.com/fybrik/openmetadata-connector/issues/43


Batch Analysis mode: File with sampling frequencies >48 kHz not recognized

Describe the bug
It is not possible to load files into input field. When trying to add folders, or drag-and-drop multiple files in the input field, the files are not processed and recognized. Therefore, no batch analysis is possible. Files with more than 48kHz sampling frequency are not processed or even recognized by the input field.
To Reproduce
Steps to reproduce the behavior:
For the GUI:

Go to Batch analyis tab
Click on Select input directory
search for folder with files of interest and select
click ok
Nothing happens, no files appear where they are normally listed.
For the CLI provide us with the arguments used.

Expected behavior
List of files/Paths would appear after selecting them
Desktop (please complete the following information):

OS: Win10

I created some test files above 48kHz but I can't reproduce your problem. Could you provide more information?

I created some test files above 48kHz but I can't reproduce your problem. Could you provide more information?

Thank you for looking into it. I Am trying to analyze files (preferably around 4000 per run/folder). However, once I select the folder with the files in it to analyze it takes just really long for the GUI to "recognize" or list the files. I tried it just now again with a folder with fewer files in it, which worked. With folders with a lot of files in it (~4000), it takes some minutes. Therefore, it seemed I was just not patient enough. Very sorry to open this issue gratuitous.

Interesting, that was my guess as well, I'll change the GUI to show that something is being done in the background.


added support for browser verison and the browser-dgram messagechannel…

… implementation
See https://github.com/hundredrabbits/Pilot/pull/37 for reference.

I will have a look. I have not seen this code nor Pilot nor Orca for three years :-)

It's been a while since I've looked at the code too haha, I just started getting back into this stuff myself. Let me know if you run into any issues, I'd be happy to help out.

Hmm, the console is filled/killed with thousands of warning messages from tone.js. And few real JS errors as well.

I moved the index.html to browser/ and pegged the Tone.js dependency to major version 13 (latest is 14 and is probably Pilot-incompatible). I managed to make some sounds this way :-)

Excellent! I'll take a look sometime today. I'll have to look into the tone.js stuff. Yesterday I updated the version of electron which should have taken care of all those vulnerability alerts. Tone.js can be next.

 Looks like you removed index.html but I don't see it in browser/ 🤔

Yeah, sorry for that. Included & pushed right now.

Great, I see it now! How are you serving this on your local machine? Serving index.html directly doesn't load the JS, so I'm guessing you have something bootstrapped to load it; that might be useful to include in this PR too so that it could be developed easily. Or if you want the user to use their own web server, maybe add a README to the browser directory with some instructions/recommendations on serving index.html. Loading it directly in the browser throws CORS errors (expected).

Great, I see it now! How are you serving this on your local machine?

Using any web server that is currently available. Typically via Apache or php -S.

. Loading it directly in the browser throws CORS errors (expected

Yeah, because ES modules need to be CORS-enabled.

Or if you want the user to use their own web server, maybe add a README to the browser directory with some instructions/recommendations on serving index.html.

Hmm, I would say that having your infrastructure set up is somewhat out of the scope for the project itself. But I can add that, if you wish.

I would say that having your infrastructure set up is somewhat out of the scope for the project itself.

Fair enough and I'm good with saying "here's a browser version but you have to figure out how to host it yourself."


Orcas Landing Inc Water System

Received from Henry Parsons

I was looking at our Orcas Landing Inc. water system (WA Dept of Health Drinking Water #63990V) on the San Juan County GIS site (http://sjcgis.org/water-systems/) and noticed that two of the houses served by us are not within the colored area.

Those two properties are 430 Orcas Hill Rd (Waterman) and 456 Orcas Hill Rd (Peterson).  456 Orcas Hill Rd is shown in the Bay Head Water Users Association, which isn't correct.

Parcel<PHONE_NUMBER>00 is not a part of the OLI water system.

Updated maps.


Options -> Controls crash

Hi,
this is a somewhat annoying bug.
It happens when I try to enter the "Options" -> "Controls" GUI both from the main menu or within a loaded world. This crash is repeatable and happens only when HxCEnchants 2.2.4 is added to the mods folder.
HxCCore 1.10.0 alone works fine.
http://pastebin.com/gU1UD1XW

That's an odd crash... BTW I wonder try it without super enchants, also did you delete your HxCEnchants config before updating?

Reason was the HxCEnchants config file.
I had disabled many of the enchantments and changed the settings for some other for testing purposes. Also I didn't replace it on updating.
But there aren't any changes between the old and the new config file at all that should cause that crash normally.
Here is my modified config file:
HxCEnchants.cfg.txt

What's your new file look like? I have a feeling I did change something, what version did you use previously, and what version now?

You have offered it to me for testing purposes. Do you remember?
See
https://github.com/HxCKDMS/HxCEnchants/issues/42

Oh, xD


allows cluster to setup port forwarding for testing e.g. kafka

(2) print exec result in log (3) renamed packages

+1


Add support for Unix Domain Socket Connection to telegram-cli

The problem
I would like to expose telegram-cli api with finer granularity access control.
Using network socket the interface to user personal data are accessible system wide,
It may be useful to have the possibility to restrict the communication with telegram-cli at user level.
telegram-cli already support unix domain socket, that allow per user privileges on the connection.
There is any plan to support unix domain socket along with network socket also in pytg ?
Your environment
python 3.5
archlinux

That would be a awesome feature.
Sadly, I won't be able to add that anytime soon.
However, I appreciate all contributions.
If you like to add that functionally, I'd be looking forward to a pull request of you.


Bluetooth: Audio: bap_broadcast_assistant API inaccesible from recv_state callback

Describe the bug
BAP Broadcast Assistant uses a callback API to indicate remote state changes. An application would expect to use these callbacks to make API calls itself. However those API calls then fail with -EBUSY.
The problem is related to the use of Write Request in one direction and Notification in the other. The peer notification "overtakes" the Write Response so BAP Broadcast Assistant is unable to make further Write Requests.
To Reproduce

Zephyr is running at both ends (BAP Broadcast Assistant and BAP Scan Delegator)
Nordic nRF 5340 Audio DK boards - though the problem appears to be in high-level BAP support code so platform may be irrelevant
Boards Advertise/Scan, Connect, Bond
Client discovers Broadcast Audio Scan Service on Server
Client sends Add Source write request (BASS v1.0 <IP_ADDRESS>)
Server responds with Broadcast Receive State notification (BASS v1.0 3.2) containing Source ID
Client responds with Set Broadcast Code write request using Source ID (BASS v1.0 <IP_ADDRESS>)
bt_bap_broadcast_assistant_set_broadcast_code() fails -EBUSY

Expected behavior
bt_bap_broadcast_assistant_set_broadcast_code() API (and others...) should be available within bt_bap_broadcast_assistant_cb::recv_state callback
Impact
An application using the current behaviour would be required to duplicate much of the work of the BAP Broadcast Assistant by storing the Source ID from recv_state callback then waiting for add_src callback before progressing to Set Broadcast Code.
I modified the BAP Broadcast Assistant to use Write No Response rather than Write Request and this works fine. However this prevents most of the callbacks firing since they handle the Write Response which will no longer come. In order to proceed in this direction an architectural review would be required to establish whether these callback are actually useful.
Logs and console output
Note that these logs demonstrate the proposed solution of Write No Response, but they give an overview of the problem.
Server:
HL [00:00:00.313,629] <inf> bt_hci_core: Identity: E9:95:8A:50:69:39 (random)
HL [00:00:00.313,659] <inf> bt_hci_core: HCI: version 5.2 (0x0b) revision 0x0d15, manufacturer 0x07e8
HL [00:00:00.313,690] <inf> bt_hci_core: LMP: version 5.2 (0x0b) subver 0x0d15
HL [00:00:00.314,544] <inf> ble: Controller version: 3349
HL [00:00:00.435,272] <inf> bis_headset: Advertising successfully started
HL [00:00:00.645,507] <inf> bis_headset: Connected: 14:D4:F9:A0:45:33 (public)
HL [00:00:01.060,546] <inf> ble: Passkey accepted for 14:D4:F9:A0:45:33 (public): 942432
HL [00:00:01.599,639] <inf> ble: Pairing completed: 14:D4:F9:A0:45:33 (public), bonded: 1
HL [00:00:04.839,752] <dbg> bt_bap_scan_delegator: recv_state_cfg_changed: value 0x0001
HL [00:00:05.019,744] <dbg> bt_bap_scan_delegator: read_recv_state: Index 0: Not active
HL [00:00:05.199,829] <dbg> bt_bap_scan_delegator: write_control_point: Data
                                                   02 00 00 00 00 00 00 00  00 56 34 12 00 ff ff 00 |........ .V4.....
HL [00:00:05.199,859] <dbg> bt_bap_scan_delegator: write_control_point: Assistant adding source
HL [00:00:05.199,859] <dbg> bt_bap_scan_delegator: scan_delegator_add_source: Index 0: New source added: ID 0x00
HL [00:00:05.200,012] <dbg> bt_bap_scan_delegator: bt_debug_dump_recv_state: Receive State[0]: src ID 0, addr 00:00:00:00:00:00 (public), adv_sid 0, broadcast_id 1193046, pa_sync_state 0, encrypt state 0, num_subgroups 0
HL [00:00:05.379,852] <dbg> bt_bap_scan_delegator: write_control_point: Data
                                                   04 00 30 31 32 33 34 35  36 37 38 39 41 42 43 44 |..012345 6789ABCD
                                                   45 46                                            |EF
HL [00:00:05.379,852] <dbg> bt_bap_scan_delegator: write_control_point: Assistant setting broadcast code
HL [00:00:05.379,913] <dbg> bt_bap_scan_delegator: scan_delegator_broadcast_code: Index 0: broadcast code added:<PHONE_NUMBER>3536373839414243444546
HL [00:00:05.379,913] <inf> bis_headset: BASS recv 0 ID 0x123456 broadcast code
HL [00:00:05.380,187] <dbg> bt_bap_scan_delegator: write_control_point: Data
                                                   03 00 01 ff ff 00                                |......
HL [00:00:05.380,218] <dbg> bt_bap_scan_delegator: write_control_point: Assistant modifying source
HL [00:00:05.380,249] <dbg> bt_bap_scan_delegator: scan_delegator_mod_src: src_id 0
HL [00:00:05.380,249] <inf> bis_headset: BASS recv 0 ID 0x123456 pa sync
HL [00:00:05.559,936] <dbg> bt_bap_scan_delegator: recv_state_cfg_changed: value 0x0000
HL [00:00:05.560,119] <dbg> bt_bap_scan_delegator: scan_delegator_disconnected: Instance 0 with addr 14:D4:F9:A0:45:33 (public) disconnected
HL [00:00:05.560,241] <inf> bis_headset: Disconnected: 14:D4:F9:A0:45:33 (public) (reason 0x16)
HL [00:00:05.560,882] <inf> bis_headset: Scanning successfully started

Client:
GW [00:00:15.312,072] <inf> cis_gateway: Device found
GW [00:00:15.414,154] <inf> cis_gateway: Connected: E9:95:8A:50:69:39 (random)
GW [00:00:15.920,379] <inf> ble: Passkey accepted for E9:95:8A:50:69:39 (random): 942432
GW [00:00:16.460,449] <inf> ble: Pairing completed: E9:95:8A:50:69:39 (random), bonded: 1
GW [00:00:16.461,395] <inf> cis_gateway: Updated to 2M PHY
GW [00:00:18.798,614] <inf> cis_gateway: SIRKs discovered: 1
GW [00:00:18.978,637] <dbg> bt_bap_broadcast_assistant: service_discover_func: [ATTRIBUTE] handle 0x000E
GW [00:00:19.158,691] <dbg> bt_bap_broadcast_assistant: char_discover_func: [ATTRIBUTE] handle 0x000F
GW [00:00:19.158,721] <dbg> bt_bap_broadcast_assistant: char_discover_func: Control Point
GW [00:00:19.158,721] <dbg> bt_bap_broadcast_assistant: char_discover_func: [ATTRIBUTE] handle 0x0011
GW [00:00:19.158,752] <dbg> bt_bap_broadcast_assistant: char_discover_func: Receive State 1
GW [00:00:19.518,676] <dbg> bt_bap_broadcast_assistant: char_discover_func: Found 1 BASS receive states
GW [00:00:19.878,692] <inf> cis_gateway: BASS recv discovered: 1
GW [00:00:20.058,746] <dbg> bt_bap_broadcast_assistant: notify_handler: Receive state notification:
                                                        00 00 00 00 00 00 00 00  00 56 34 12 00 00 00    |........ .V4....
GW [00:00:20.058,776] <inf> cis_gateway: BASS recv state: src_id 0 sid 0 state 0/0
GW [00:00:20.328,857] <inf> cis_gateway: Disconnected: E9:95:8A:50:69:39 (random) (reason 0x13)
GW [00:00:20.329,589] <inf> cis_gateway: Scanning successfully started

Environment (please complete the following information):

OS: MacOS build, Nordic nRF5340 targets
Toolchain: Zephyr SDK
Commit SHA: 28a3fca

Here's the patch I'm successfully using, but as I said in the description this simply disables the (most of) the callbacks so we need to decide if those callbacks should be removed.
diff --git a/subsys/bluetooth/audio/bap_broadcast_assistant.c b/subsys/bluetooth/audio/bap_broadcast_assistant.c
index 7e586d2b8e..3949025b8d 100644
--- a/subsys/bluetooth/audio/bap_broadcast_assistant.c
+++ b/subsys/bluetooth/audio/bap_broadcast_assistant.c
@@ -500,6 +500,11 @@ static int bt_bap_broadcast_assistant_common_cp(struct bt_conn *conn,
                return -EINVAL;
        }

+#if 1
+       err = bt_gatt_write_without_response(conn,
+                                            broadcast_assistant.cp_handle,
+                                            buf->data, buf->len, false);
+#else
        broadcast_assistant.write_params.offset = 0;
        broadcast_assistant.write_params.data = buf->data;
        broadcast_assistant.write_params.length = buf->len;
@@ -510,7 +515,7 @@ static int bt_bap_broadcast_assistant_common_cp(struct bt_conn *conn,
        if (err == 0) {
                broadcast_assistant.busy = true;
        }
-
+#endif
        return err;
 }

Here's the patch I'm successfully using, but as I said in the description this simply disables (most of) the callbacks so we need to decide if those callbacks should be removed.
diff --git a/subsys/bluetooth/audio/bap_broadcast_assistant.c b/subsys/bluetooth/audio/bap_broadcast_assistant.c
index 7e586d2b8e..3949025b8d 100644
--- a/subsys/bluetooth/audio/bap_broadcast_assistant.c
+++ b/subsys/bluetooth/audio/bap_broadcast_assistant.c
@@ -500,6 +500,11 @@ static int bt_bap_broadcast_assistant_common_cp(struct bt_conn *conn,
                return -EINVAL;
        }

+#if 1
+       err = bt_gatt_write_without_response(conn,
+                                            broadcast_assistant.cp_handle,
+                                            buf->data, buf->len, false);
+#else
        broadcast_assistant.write_params.offset = 0;
        broadcast_assistant.write_params.data = buf->data;
        broadcast_assistant.write_params.length = buf->len;
@@ -510,7 +515,7 @@ static int bt_bap_broadcast_assistant_common_cp(struct bt_conn *conn,
        if (err == 0) {
                broadcast_assistant.busy = true;
        }
-
+#endif
        return err;
 }

@kruithofa is looking into this issue. The fix may be as simple as moving the busy = false to before the callbacks, but since the recv_state callback is called both during discovery and when we receive a notification, it may not be that simple.

@Chris-StJohn-Bose
Although it is possible to use a ‘Write No Response’ this has 2 drawbacks.

There is no notification in case the write fails
‘Write No Response’ can not be used to write data larger than the MTU size
The relevant part of the spec is in BT Core spec 5.4, vol 3, part G, 4.9.4

We looked into moving the busy=false statement, but that didn’t give the desired result.
A better solution is to wait for the write response before doing the ‘Set Broadcast Code’ instead of doing this in the callback.
I suggest to close the issue as a ‘will not do’

@kruithofa

There is no notification in case the write fails

Write failure is a pretty unexpected event really and suggests an implementation or connection problem rather than a state problem? One could equally argue that Notification is the wrong choice.
I'm thinking about similarities with ASCS, for instance.

‘Write No Response’ can not be used to write data larger than the MTU size
The relevant part of the spec is in BT Core spec 5.4, vol 3, part G, 4.9.4

Fair point. I think BAP requires an MTU of at least 64 octets presumably for the same reason.

We looked into moving the busy=false statement, but that didn’t give the desired result. A better solution is to wait for the write response before doing the ‘Set Broadcast Code’ instead of doing this in the callback.

I don't think it's reasonable to expect the application to sort out these things. One approach might be to queue callbacks whilst busy==true which would probably be the least surprising behaviour for the application implementer. We still have the problem of potentially two threads working the API at the same time (the UI thread and presumably BT_RX) but hey that's callbacks for you!

Write failure is a pretty unexpected event really and suggests an implementation or connection problem rather than a state problem? One could equally argue that Notification is the wrong choice.

Unless the server rejects your request :)

I'm thinking about similarities with ASCS, for instance.

In ASCS all writes are "responded" to by notifications. BASS does not have feature. BASS even defines it own ATT error messages, so arguably it has been designed with write (with response) in mind.

Fair point. I think BAP requires an MTU of at least 64 octets presumably for the same reason.

Indeed, but it doesn't take much metadata to get past the 64 octets.

I don't think it's reasonable to expect the application to sort out these things. One approach might be to queue callbacks whilst busy==true which would probably be the least surprising behaviour for the application implementer.

The problem is that in order to support long writes, we'll need to store a buffer of up to 512 octets per pending write request. The challenge is essentially a memory issue.
In order to get past those memory issue, we'd have to rework the API so that the write buffer is stored in the application, which would move the issue from the stack, to the application (which arguably is better handled for it).

Ok @Thalley and @kruithofa I guess closing this ticket is probably the right way to go unless a common solution emerges that could be integrated into the API.
I'll probably rework my Broadcast Assistant application to queue recv() callbacks whilst busy==true as the data for these callbacks will be relatively short (they're derived from a notification so can't be that much data in there!)
So we're going with WILL NOT DO -- feel free to re-open if appropriate :-)
Thanks for both of your time looking at this.

(they're derived from a notification so can't be that much data in there!)

The notifications won't even be > GATT MTU, but they may be truncated, in which case you'd need to re-read the receive state. That is a feature still missing from the Broadcast Assistant implementation (will probably use the same implementation of supporting this as we have in the BAP Unicast Client).


I got an error.

So, I ran the command, and I got this:
Traceback (most recent call last):
  File "/home/sealington33/.local/bin/textual-paint", line 5, in <module>
    from textual_paint.paint import main
  File "/home/sealington33/.local/lib/python3.9/site-packages/textual_paint/paint.py", line 35, in <module>
    from textual_paint.action import Action
  File "/home/sealington33/.local/lib/python3.9/site-packages/textual_paint/action.py", line 5, in <module>
    from textual_paint.ansi_art_document import AnsiArtDocument
  File "/home/sealington33/.local/lib/python3.9/site-packages/textual_paint/ansi_art_document.py", line 583
    match instruction.region:
          ^
SyntaxError: invalid syntax

You'll need Python 3.10 or later. You can check what version you have with python --version


python310Packages.types-dateutil: <IP_ADDRESS> -> <IP_ADDRESS>

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.types-dateutil is: Typing stubs for python-dateutil
meta.homepage for python310Packages.types-dateutil is: https://github.com/python/typeshed
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS> with grep in /nix/store/ji32fnaqbga6hb3x9ypylb2mqlnz8a8s-python3.10-types-dateutil-<IP_ADDRESS>
found <IP_ADDRESS> in filename of file in /nix/store/ji32fnaqbga6hb3x9ypylb2mqlnz8a8s-python3.10-types-dateutil-<IP_ADDRESS>

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.types-dateutil
python311Packages.types-dateutil

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ji32fnaqbga6hb3x9ypylb2mqlnz8a8s-python3.10-types-dateutil-<IP_ADDRESS> \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.types-dateutil https://github.com/r-ryantm/nixpkgs/archive/cfc46fb39a33e59942b62f7538298cc937d4215f.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ji32fnaqbga6hb3x9ypylb2mqlnz8a8s-python3.10-types-dateutil-<IP_ADDRESS>
ls -la /nix/store/ji32fnaqbga6hb3x9ypylb2mqlnz8a8s-python3.10-types-dateutil-<IP_ADDRESS>/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.types-dateutil
python311Packages.types-dateutil

Maintainer pings
cc @milibopp for testing.

Result of nixpkgs-review pr 218833 run on aarch64-darwin 1

2 packages built:

python310Packages.types-dateutil
python311Packages.types-dateutil


Cant run web version

Describe the bug
Cant make it run on quasar SPA with vite

Expected behavior
Web version works as ios/android

OS: macOS
Browser chorme
Version latest 4.2.2

Quasar latest

@Nickolas- can you share the main.ts and the app.ts files

@jepiqueau thanks , for quick response! Quasar framework do not have a main.ts it uses a boot files.
App.vue
`

`
database.ts
`import { boot } from 'quasar/wrappers'
import AppDataSource  from 'src/services/DatabaseService';
import connection from 'src/services/SqliteConnection';
import {Capacitor} from '@capacitor/core';
import {CapacitorSQLite} from "@capacitor-community/sqlite";
// "async" is optional;
// more info on params: https://v2.quasar.dev/quasar-cli/boot-files
export default boot(async (/* { app, router, ... } */) => {
try {
console.log('Start to setup database');
const pSqliteConsistent = await CapacitorSQLite.checkConnectionsConsistency({
  dbNames: ['local'], openModes: ['open'] });

const platform = Capacitor.getPlatform();
console.log(platform);
if (platform === 'web') {
  // Create the 'jeep-sqlite' Stencil component
  const jeepSqlite = document.createElement('jeep-sqlite');
  document.body.appendChild(jeepSqlite);
  await customElements.whenDefined('jeep-sqlite');
  // Initialize the Web store
  await connection.initWebStore();
}

await AppDataSource.initialize();

const result = await AppDataSource.runMigrations();

if (platform === 'web') {
  // save the database from memory to store
  await connection.saveToStore('local');
}

console.log('Database inited');

} catch (e) {
console.log(e.message);
}
})
`

@Nickolas- can you share appDataSource

@Nickolas- the openModes must be RW or RO or both read-write RW, read-only RO. Open does not exist

@jepiqueau  FYI

I do no think the issue , in wrong open modes, becuase as I said everything works on ios/android
I read something about stencil components to experimental import

I did add that too

@Nickolas- the checkConsistency is mainly used for live reload . Any how the 'open' mode clearly does not exist.
I do not know if you have a vue.config.js file look at what i put in to have typeOrm working in https://github.com/jepiqueau/vue-typeorm-app.  That could help you. I am away from my computer for the next 3 weeks so i cannot run your app. Tell me if s'éteint the vue.config.js help in solving your issue

@jepiqueau vue.config.js just store some info for webpack and im using vite. I do no think they are related ?

@Nickolas- there is nothing more i can do for the next 3 week-end so you are on your own

Hi @jepiqueau , I found an issue at stencil github about dynamic loads and Vite.
https://github.com/ionic-team/stencil/issues/2827
What do you think , could be it related ? I will try today add this flag to stencil config and rebuild a package. Let's see

@Nickolas- i remember for vite app the way to import the Stencil component is different. Look at https://github.com/gepiqueau/vuevite-app this should solve your issue

@Nickolas- Did that help you to solve the issue ?

@jepiqueau I didnt test actually , do not have a time right now to play around. Will do maybe on weekeneds.
For right now I did a workaround to make it work

@jepiqueau btw your link does not work for me. I got 404.

@Nickolas- you get a 404 as you certainly did not copy the sql-wasm.wasm file from sql.js installed into the public/assets folder of the app as mentioned in the doc

@jepiqueau I got 404 on your link , not in app :) https://github.com/gepiqueau/vuevite-app this one not works :)

@jepiqueau about sql-wasm.wasm I try in to different ways, one is load from static files , other to load from  sqljs.org at runtime

@Nickolas- sorry https://github.com/jepiqueau/vuevite-app

@Nickolas- to have it working with capacitor 4 you have to follow the instructions from ionic capacitor 3 to 4

@Nickolas- have you had time to try?

Not yet , maybe on weekends.

On 3 Nov 2022, at 10:15, QUEAU Jean Pierre @.***> wrote:
@Nickolas- https://github.com/Nickolas- have you had time to try?
—
Reply to this email directly, view it on GitHub https://github.com/capacitor-community/sqlite/issues/340#issuecomment-1301766736, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB6K7IDYAEGMZWTGTDE64UDWGNYDPANCNFSM6AAAAAARSFJN7E.
You are receiving this because you were mentioned.

Best Regards,
Mykola Shpotenko
P: +380679907582
E: @.***

@Nickolas- OK good luck

@jepiqueau so after of tone wasted time , i would not recommend anyone to work with typeorm and vite. Its just not ready , to many things fail in runtime.
With webpack everything work as a charm. No extra coded need.
Have a good day :)

@Nickolas- thanks for your painfull work sorry that you cannot succeed but at least it is a good feedback for other developers

@Nickolas- i close the issue

@jepiqueau so after of ton wasted time , i would not recommend anyone to work with typeorm and vite. Its just not ready , to many things fail in runtime.

I also tried to develop project vite + vue3 + capacitor + @capacitor-community/sqlite and ran into a project build problem when connecting typeorm. I will leave a link to the issue for those who will look for information in the future (unfortunately this method did not help me, but maybe this method will work for you or a solution will be found that works for everyone).

I also had problems when generating typeorm migrations with dataSource type 'capacitor', but maybe I missed something in this case.

Hi @jepiqueau, coming back with a good news.
To run sqlite typeorm and Vite was a challenge but I did it successfully.
First of all do not use

Vite do not know how to resolve all required dynamic dependencies. Use:

This will solve the issue in production mode related to

Second is to solve all typeorm issues with Vite in production.
Here a github issue from typeorm repository.
Maybe later I can create a demo with Vite sqlite typeorm quasar.
Have a good day

Same issue here (vite+quasar+vue3+capacitor+sqlite).
I can confirm the workaround calling customElements.define('jeep-sqlite',JeepSqlite); before jeepSqlite(window) (not after)

@mmouterde look at https://jepiqueau.github.io/2023/09/05/Ionic7Vue-SQLite-CRUD-App.html  you will see how it is done for vite vue app


Fixed typo in enc.c warning

"warning: iv not use by this cipher" -> "warning: iv not used by this cipher"
CLA: trivial

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

@t-j-h a formality, is this trivial?
I think it is.

Labeled for cherry picking to 1.1.1

A trivial confirmation is still required...

A trivial confirmation is still required...

It is not. The raw commit message shows that a CLA:trivial is present. Only the empty line between title and body is missing, the usual problem why it is not recognized. IMHO this could simply be fixed by the merger.

Sorry. My last comment was nonsense.

Should the word iv be uppercased?

It is completely trivial - and my approval should have made that clear.

Merged to master and 1.1.1.  Thanks.

my approval should have made that clear.

For the record I don't think approval alone is sufficient. There should be explicit acknowledgement of accepting the triviality.


Terminal logging

JANUS now logs information to the terminal (stdout, stderr) via the logging module, rather than with the print function. This integrates better with PROTEUS and offers more flexibility going forward.
This means that PROTEUS will no longer need to redirect stdout to its own logger when JANUS is called. Closes #46.
Replaced "SOCRATES" env variable with "RAD_DIR" for consistency with other modules (although the name is probably less intuitive). Closes #51.

Updated this PR to resolve the issues raised above. The loggers are now assigned names in the format of "fwl."+__name__ as recommended by the Python documentation for libraries.
https://docs.python.org/3/howto/logging.html#configuring-logging-for-a-library
This means that it's still clear where in the code a logging call is made, but only a single handler is required at a time (or two if you are also logging to a file). There will be a corresponding update to PROTEUS.
Some other minor changes including refactoring of some functions and documentation.


Misaligned AppBar Widgets

What
The similar products screen has an AppBar with:

Misaligned back arrow
Misaligned Filters button

Steps to reproduce the behavior

Open a product
Click on 'Compare to category'

Can i take this one?

Sure @mse-moises

My fix for #1863 will also fix this issue

@g123k Sorry, I was working on this the last friday, but i actually was without my computer in this weekend so i could'nt unassign myself from this. Did you fixed the issue?

No pb, my PR should fix it.
The repo doesn’t miss some issues 😝


Excessive Row Padding in Tables

Describe the Bug
I have a large table -- two columns but many rows -- and no matter what I do BookStack adds enormous amounts of padding below the cell contents. There must be a way to make it minimize the row height to just accommodate the contents of the cells.
Steps to Reproduce

Create a table (mine is 2 x many)
Add a little text to each cell
Save
Edit and add more rows, with a little text in each cell
Keep repeating steps 3 and 4.
Rows will start getting excessively padded.

Expected Behaviour
Just make a table (mine is 2 x many) and add a little text in each cell. Then save it and edit it again and add more rows with a little text in them, and keep doing that. The cells will grow way beyond the text within them.
Screenshots or Additional Context

Browser Details
Firefox 102.3.0esr (64-bit)
Exact BookStack Version
v22.09.1
PHP Version
No response
Hosting Environment
Linux Mint 21 Cinnamon

From some quick testing, I think this is maybe due to variance in width when in and out of the editor, after making adjustments to row heights which would start fixing table sizes. The fixed row heights will then present empty spacing if the table becomes wider. It is possible to reset heights back to be automatically adjusting to content by selecting all cells in the table then using the "table > cell properties", "table > row properties" and "table > table properties" menu options (accessed via the dropdown icon on the table toolbar item) and then removing any height values or by setting a value of "auto".
TBH though, if you need content to be flexible, I'd advise against tables, and instead recommend maybe using using additional header levels for each of your shown left-hand cell items, with right-hand cell items as normal paragraphs below.

Thank you, but that didn't change anything. The table cell width does not vary. I expect the row height to conform to the contents. That does not seem unreasonable.

Thank you, but that didn't change anything.

Did you reset it for all three? (Table, row and cell?)

I expect the row height to conform to the contents. That does not seem unreasonable.

It does by default, but upon any manual tweak to table dimensions sizes will start to become fixed which is what I believe is causing the additional spacing here.

Ok, yes, that works. I thought I had selected all cells, but I had not scrolled to the end of the table. Thank you.

Happy to help, Will therefore close this off.


Install TagBot as a GitHub Action

You're receiving this pull request because the now-deprecated Julia TagBot GitHub App is installed for this repository. This pull request installs TagBot as a GitHub Action.
If this PR does not belong in this repository for any reason, feel free to close it.
See this Discourse thread for more details and to ask questions.
Please do not comment on this pull request, because it is unlikely that I (@christopher-dG) will see it.

Codecov Report

Merging #88 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #88   +/-   ##
=======================================
  Coverage   91.59%   91.59%
=======================================
  Files          17       17
  Lines         488      488
=======================================
  Hits          447      447
  Misses         41       41


Submodule diffs are misaligned

What?

Primary: Prevent center-aligned filenames in submodule diff.
Secondary: If submodule diffs were previously ignored during styling, then reconsider the styling.

Why?
A center-aligned file path listing is not as easy to scan as is a left-aligned file path listing, because similarities in paths are not aligned as they are in a left-aligned file path listing.
Where?
https://github.com/TACC/Core-CMS/pull/216/commits/c8b7cbe270ad083dd11d3f81da35cb281fe3d56f
Screenshot while using Github Refined

Screenshot while using Github

How?
Offending Style:
https://github.com/sindresorhus/refined-github/blob/8c1ab3125fcdb5edf4d78e86bca2da30da97ca45/source/refined-github.css#L99-L103
Suggestions: None

Can I take this up?

Sure

I'm able to shift the table by overwriting text-align: center; to text-align: left;
:root .file .data table {
	padding: 54px;
	text-align: left !important;
}

But the Submodule taccsite_custom updated 9 files isn't overwriteable as the class of TACC/Core-CMS@c8b7cbe#diff-00db495ec685cbdb37dc9dfede576323eb49fd5d4156be453590c5b7650bb9e6 and this clashes.
Is this good to go as this is the best which we can get without much complexity I feel? Any suggestions?


Sprint Review Document

Sprint Review 4 (/5)

Should follow the template.
For any missing item from the template, deduct 2 points.
Should be written for this past sprint / week (i.e. for Deliverable 1, Deliverable 1 and Revisions for Deliverable 0) but reflect about the whole project experience so far
Use the template as an agenda for the sprint review meeting. Step through each point as a team, have a discussion and have someone take notes. The team should review the notes during the meeting in a shared Google Doc to make sure everyone is heard, and anyone can edit the notes.
For any missing item from the template, deduct 2 points.
Document should contain all prior Sprint Reviews as well, with the most recent at the top, otherwise -5.

https://docs.google.com/document/d/1-uYBb9TfM7NNzkcgMSotAvynuoYKNZia/edit?usp=sharing&ouid=108948216072188806493&rtpof=true&sd=true

This looks solid


Confusing error messages when Docker machine initialization fails

When attempting to use testcontainers in OS X, I get the following exception.  Note the cause right at the bottom, which has the message "Could not find a suitable docker instance - is DOCKER_HOST defined and pointing to a running Docker daemon?":
java.lang.IllegalStateException: Failed to prepare Docker client
	at org.testcontainers.DockerClientFactory.<init>(DockerClientFactory.java:49)
	at org.testcontainers.DockerClientFactory.instance(DockerClientFactory.java:60)
	at org.testcontainers.containers.GenericContainer.<init>(GenericContainer.java:72)
	at org.testcontainers.containers.JdbcDatabaseContainer.<init>(JdbcDatabaseContainer.java:27)
	at org.testcontainers.containers.PostgreSQLContainer.<init>(PostgreSQLContainer.java:17)
	at org.testcontainers.containers.PostgreSQLContainerProvider.newInstance(PostgreSQLContainerProvider.java:14)
	at org.testcontainers.jdbc.ContainerDatabaseDriver.connect(ContainerDatabaseDriver.java:111)
	at com.mchange.v2.c3p0.DriverManagerDataSource.getConnection(DriverManagerDataSource.java:161)
	at com.progressfin.dataplatform.DataPlatformETLTest.runInitializationScripts(DataPlatformETLTest.java:121)
	at com.progressfin.dataplatform.DataPlatformETLTest.<init>(DataPlatformETLTest.java:79)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:408)
	at org.junit.internal.builders.AnnotatedBuilder.buildRunner(AnnotatedBuilder.java:29)
	at org.junit.internal.builders.AnnotatedBuilder.runnerForClass(AnnotatedBuilder.java:21)
	at org.junit.runners.model.RunnerBuilder.safeRunnerForClass(RunnerBuilder.java:59)
	at org.junit.internal.builders.AllDefaultPossibilitiesBuilder.runnerForClass(AllDefaultPossibilitiesBuilder.java:26)
	at org.junit.runners.model.RunnerBuilder.safeRunnerForClass(RunnerBuilder.java:59)
	at org.junit.internal.requests.ClassRequest.getRunner(ClassRequest.java:26)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.runTestClass(JUnitTestClassExecuter.java:88)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassExecuter.execute(JUnitTestClassExecuter.java:56)
	at org.gradle.api.internal.tasks.testing.junit.JUnitTestClassProcessor.processTestClass(JUnitTestClassProcessor.java:64)
	at org.gradle.api.internal.tasks.testing.SuiteTestClassProcessor.processTestClass(SuiteTestClassProcessor.java:50)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:483)
	at org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
	at org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
	at org.gradle.messaging.dispatch.ContextClassLoaderDispatch.dispatch(ContextClassLoaderDispatch.java:32)
	at org.gradle.messaging.dispatch.ProxyDispatchAdapter$DispatchingInvocationHandler.invoke(ProxyDispatchAdapter.java:93)
	at com.sun.proxy.$Proxy2.processTestClass(Unknown Source)
	at org.gradle.api.internal.tasks.testing.worker.TestWorker.processTestClass(TestWorker.java:106)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:483)
	at org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:35)
	at org.gradle.messaging.dispatch.ReflectionDispatch.dispatch(ReflectionDispatch.java:24)
	at org.gradle.messaging.remote.internal.hub.MessageHub$Handler.run(MessageHub.java:360)
	at org.gradle.internal.concurrent.ExecutorPolicy$CatchAndRecordFailures.onExecute(ExecutorPolicy.java:54)
	at org.gradle.internal.concurrent.StoppableExecutorImpl$1.run(StoppableExecutorImpl.java:40)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:744)
Caused by: java.lang.IllegalStateException: Could not find a suitable docker instance - is DOCKER_HOST defined and pointing to a running Docker daemon?
	at org.testcontainers.DockerClientFactory.determineConfiguration(DockerClientFactory.java:145)
	at org.testcontainers.DockerClientFactory.client(DockerClientFactory.java:72)
	at org.testcontainers.DockerClientFactory.<init>(DockerClientFactory.java:44)
	... 45 more

The problem is that this exception's descriptive message has, in my case, nothing to do with the actual cause of the failure.  The logs before the point this exception was thrown show this:
[Test worker] INFO org.testcontainers.DockerClientFactory - Checking environment for docker settings
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - I/O exception (org.apache.http.conn.UnsupportedSchemeException) caught when processing request to {s}->https://localhost:2376: https protocol is not supported
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - Retrying request to {s}->https://localhost:2376
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - I/O exception (org.apache.http.conn.UnsupportedSchemeException) caught when processing request to {s}->https://localhost:2376: https protocol is not supported
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - Retrying request to {s}->https://localhost:2376
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - I/O exception (org.apache.http.conn.UnsupportedSchemeException) caught when processing request to {s}->https://localhost:2376: https protocol is not supported
[Test worker] INFO org.apache.http.impl.execchain.RetryExec - Retrying request to {s}->https://localhost:2376
[Test worker] INFO org.testcontainers.DockerClientFactory - Could not initialize docker settings using environment variables: org.apache.http.conn.UnsupportedSchemeException: https protocol is not supported
[Test worker] INFO org.testcontainers.DockerClientFactory - Checking for presence of docker-machine
[Test worker] INFO org.testcontainers.DockerClientFactory - Could not initialize docker settings using docker machine: docker-machine must be installed for use on OS X

That last message is apparently the real cause of the failure, but it's being reported as an info-level log message, one which several of my unit test reporting tools don't even report on.

I should note that I have docker-machine installed.  In at least one instance, the cause of the error seems to be that IntelliJ is not including /usr/local/bin in the PATH.  So a much more useful failure mode would be something like this:
Could not find docker-machine executable; PATH = [/usr/bin, /bin/]

Luca,
Thank you for trying testcontainers out, and thanks raising this. I'd agree that these logs are confusing and need to be improved.
DockerClientFactory tries a couple of ways of accesing docker (via environment vars, then falling back to docker-machine) but at the moment I think it's too generic about the final (fatal) failure reporting. It should clearly state what's been tried and failed in the exception message. (Being in a 'caused by' is probably not all that great for clarity either).
I'll try and take a look at this in the next few days and prioritise for the next release, as I think not much work could make a worthwhile improvement.
Thanks!
Richard

Your PATH reporting suggestion looks like a very good thing to do as well - nice and clear!

I have a proposed improvement in a branch.
Log output in a case of failure is something like:
21:00:21.746 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Looking for Docker environment. Trying Environment variables, system properties and defaults: DockerClientConfig{uri=https://localhost:2376, version='null', username='root', password='null', email='null', serverAddress='https://index.docker.io/v1/', dockerCfgPath='/root/.dockercfg', sslConfig='LocalDirectorySSLConfig{dockerCertPath=/root/.docker}'}
21:00:24.377 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Looking for Docker environment. Trying docker-machine
21:00:24.433 [main] ERROR org.testcontainers.dockerclient.DockerConfigurationStrategy - Could not find a valid Docker environment. Please check configuration. Attempted configurations were:
21:00:24.435 [main] ERROR org.testcontainers.dockerclient.DockerConfigurationStrategy -     Environment variables, system properties and defaults: DockerClientConfig{uri=https://localhost:2376, version='null', username='root', password='null', email='null', serverAddress='https://index.docker.io/v1/', dockerCfgPath='/root/.dockercfg', sslConfig='LocalDirectorySSLConfig{dockerCertPath=/root/.docker}'}: failed with exception message: ping failed
21:00:24.437 [main] ERROR org.testcontainers.dockerclient.DockerConfigurationStrategy -     docker-machine: failed with exception message: docker-machine executable was not found on PATH ([/usr/local/sbin, /usr/local/bin, /usr/sbin, /usr/bin, /sbin, /bin, /usr/games, /usr/local/games])
21:00:24.442 [main] ERROR org.testcontainers.dockerclient.DockerConfigurationStrategy - As no valid configuration was found, execution cannot continue
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 4.4 sec <<< FAILURE!

If, however, environment-based settings fail but docker-machine succeeds, the logs are a slightly more succinct:
20:36:05.439 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Looking for Docker environment. Trying Environment variables, system properties and defaults: DockerClientConfig{uri=foobar, version='null', username='rnorth', password='null', email='null', serverAddress='https://index.docker.io/v1/', dockerCfgPath='/Users/rnorth/.dockercfg', sslConfig='LocalDirectorySSLConfig{dockerCertPath=/Users/rnorth/.docker}'}
20:36:40.615 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Looking for Docker environment. Trying docker-machine
20:36:42.344 [main] DEBUG org.testcontainers.utility.CommandLine - Executing shell command: `docker-machine ls -q`
20:36:45.269 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Found docker-machine, and will use machine named dev
20:36:45.275 [main] DEBUG org.testcontainers.utility.CommandLine - Executing shell command: `docker-machine status dev`
20:36:45.824 [main] DEBUG org.testcontainers.utility.CommandLine - Executing shell command: `docker-machine ip dev`
20:36:46.834 [main] INFO  org.testcontainers.dockerclient.DockerConfigurationStrategy - Docker daemon IP address for docker machine dev is <IP_ADDRESS>

I'd appreciate feedback on any ways this could be made better!

I'm trying to use library to do integration test of my project.
The error message really confused me. Currently I'm using docker-machine on mac osx which was installed by docker toolkit.
I also met the error message,

Could not find a suitable docker instance - is DOCKER_HOST defined and pointing to a running Docker daemon?

After debugging the source of testcontainers a bit, the original error was caused by the docker-machine was not in PATH. Then looks like the configuration of docker, this lib was trying to access the https://<ip of docker machine>:2736, then I got the error of certification.

22:32:51.811 [main] INFO  o.testcontainers.DockerClientFactory - Docker daemon IP address for docker machine default is <IP_ADDRESS>
22:32:51.814 [main] INFO  o.testcontainers.DockerClientFactory - Could not initialize docker settings using docker machine: Error in SSL Configuration

Could you add the prerequisites of docker's configuration in documentation or how to configure the test suites to use this library?

@zxkane thanks for pointing that out. It seems like the library is having trouble picking up the right configuration settings for docker-machine. I've tried to make it work with docker-machine out of the box, so this is a bit odd. I assume you haven't made any unusual tweaks to your docker-machine configuration?
Please could you help me track this down by giving me the output of:

docker-machine -v
docker version (when you're using docker from the command line)
docker-machine env default

These might yield some clues...

I don't think I made tweaks on docker-machine.
Below is output of above commands,

➜  testcontainers-java-examples git:(master) docker-machine -v
docker-machine version 0.5.2 ( 0456b9f )
➜  testcontainers-java-examples git:(master) docker version
Client:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.3
Git commit:   a34a1d5
Built:        Fri Nov 20 17:56:04 UTC 2015
OS/Arch:      darwin/amd64
Server:
Version:      1.9.1
API version:  1.21
Go version:   go1.4.3
Git commit:   a34a1d5
Built:        Fri Nov 20 17:56:04 UTC 2015
OS/Arch:      linux/amd64
➜  testcontainers-java-examples git:(master) docker-machine env default
export DOCKER_TLS_VERIFY="1"
export DOCKER_HOST="tcp://<IP_ADDRESS>:2376"
export DOCKER_CERT_PATH="/Users/kane/.docker/machine/machines/default"
export DOCKER_MACHINE_NAME="default"
Run this command to configure your shell:
eval "$(docker-machine env default)"

A bit difference of running the test, I used IDE(eclipse 4.5) to run the test case.

Thanks - that output looks correct, so this is a bit strange. I'll try and
release a milestone version (1.0.0.M1) this weekend which should address
the original issue described in this ticket and may (hopefully) address
what you're seeing to. It should at least provide a bit more debugging
information to help track it down.
Sorry you've been having trouble with this - I hope the update will help.
On Fri, Feb 5, 2016 at 2:57 AM, Meng Xin Zhu<EMAIL_ADDRESS>wrote:

I don't think I made tweaks on docker-machine.
Below is output of above commands,
➜ testcontainers-java-examples git:(master) docker-machine -v
docker-machine version 0.5.2 ( 0456b9f )
➜ testcontainers-java-examples git:(master) docker version
Client:
Version: 1.9.1
API version: 1.21
Go version: go1.4.3
Git commit: a34a1d5
Built: Fri Nov 20 17:56:04 UTC 2015
OS/Arch: darwin/amd64
Server:
Version: 1.9.1
API version: 1.21
Go version: go1.4.3
Git commit: a34a1d5
Built: Fri Nov 20 17:56:04 UTC 2015
OS/Arch: linux/amd64
➜ testcontainers-java-examples git:(master) docker-machine env default
export DOCKER_TLS_VERIFY="1"
export DOCKER_HOST="tcp://<IP_ADDRESS>:2376"
export DOCKER_CERT_PATH="/Users/kane/.docker/machine/machines/default"
export DOCKER_MACHINE_NAME="default"
Run this command to configure your shell: eval "$(docker-machine env
default)"
A bit difference of running the test, I used IDE(eclipse 4.5) to run the
test case.
—
Reply to this email directly or view it on GitHub
https://github.com/testcontainers/testcontainers-java/issues/52#issuecomment-180169656
.

Hi @ldcasillas-progreso, @zxkane
I released a new version last night, which I hope goes some way towards improving the usefulness of the logging and documentation around docker daemon discovery. Please let me know if you have a chance to try again, and if things look better for you now.
Many thanks
Richard


onSlidingComplete not firing in Android

I know that in the description it says that the react-native-snap-slider was developed based in SliderIOS but I was trying to use it on both platforms. In Android, I'm able to create the Slider and load its values, but the onSlidingComplete event is not firing at all.
Did someone manage to use the slider in Android? Could it be something that I'm doing wrong? The exact code is working in iOS, that's why I'm asking here.

@maxhalty  please see this
https://github.com/callstack/react-native-slider/issues/118#issuecomment-711875760


feat(export): allow GMP collectors to startup without metadata

The way our e2e tests are written is usually we check for the GMP collector to start-up correctly.
The problem is that if we defer the metadata labeling to the configuration as opposed to flags, then the collector will never start-up correctly (and also goes against our goal which is to prevent the collector from restarting at initialization).
The simple solution here is to defer error handling until we have a configuration with scape or rule configurations.
See also: https://github.com/GoogleCloudPlatform/prometheus-engine/pull/965

It also looks like there's some build and test failures as well - maybe a rebase from main will fix things?


Fixing restart flag

Part of the uri, not the options

yep, verified by https://mesosphere.github.io/marathon/docs/rest-api.html#post-v2-apps-appid-restart ...
LGTM


Add steps to link for local testing

It took me a while to figure out how to setup a local version of the VTA with a local version of the CLI for testing. This adds the steps I used to save others some time!

I didn't know you could specify a path. Usually you'd run yarn link in the source repo (VTA) and then yarn link @chromatic-com/storybook in the target repo. Did you try that?

I did but they changed it in Yarn 4. Should I add steps for both?

:rocket: PR was released in v3.2.3 :rocket:


Upgrade C# testing images to debian stretch

stretch is now the stable debian distro
.NET core 2+ (now the stable version) doesn't support jessie (it supports stretch and newer) and we should switch to using .NET core 2.x SDK soon as it is desirable to use an up-to-date SDK.
this PR has no influence on platforms that gRPC C# actually supports, only allows us to use newer  SDK for build.
.NET core 2.x is necessary to build netstandard2.0 and netcoreapp2.1 targets.

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 2,021,324      Total (=)      2,021,324

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,177,662      Total (>)     11,177,657

 No significant differences in binary sizes

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences

Known failures:
https://github.com/grpc/grpc/issues/15599
https://github.com/grpc/grpc/issues/15308

oauth2_auth_token on CoreCLR seems to be failing: https://source.cloud.google.com/results/invocations/a816bca0-272b-4fbb-835e-6562f48b08e7/targets/github%2Fgrpc%2Finterop_test/tests;group=tests;test=cloud_to_prod_auth:csharpcoreclr:default:oauth2_auth_token:tls;row=1

the interop test failure (https://source.cloud.google.com/results/invocations/a816bca0-272b-4fbb-835e-6562f48b08e7) seems to be due to a broken dotnet 1.0.x installation on debian stretch, which is hard to fix.
superseded by https://github.com/grpc/grpc/pull/17309.


Installation doesn't work, broke Windows Explorer

Ever since installing File Metadata for x64 on Windows 7, I am still unable to edit meta for files like txt, and Explorer is having some major issues: File deletions and folder creation are not reflected until I hit refresh, and other shell extensions (like dragging and dropping from FileZilla) do not work anymore.
Attached is a list of all of my non-MS shell extensions, as reported by ShellExView.
I'm going to uninstall Tabbles and your product, reboot, then reinstall File Metadata and see if that fixes things.
Attachments
Shell Extensions List.htm
This work item was migrated from CodePlex
CodePlex work item ID: '9387'
Vote count: '1'

[Dijji@22/07/2014]
I don't see anything obviously scary in that list of extensions.  When you say that you can't edit metadata for .txt, what problems do you see?
I wouldn't expect File Metadata to have any impact on drag-and-drop or refresh, as it goes nowhere near the hooks for either of these. The only extension installed by File Metadata is a context menu, which by the way is an optional component that you can deselect during installation.
Also, remember that explorer.exe keeps state, so when you uninstall or install extensions, it is often necessary to kill all copies of  explorer.exe and restart it in order to see changes correctly. Of course, reboot does that job too!

[Dijji@22/07/2014]
Of course, File Metadata does also install a property handler. However, this extends the property system rather than Explorer: it has no impact on Explorer unless and until you use the File Association Manager to register extensions for which it should be used.
So, installing File Metadata without the optional context menu and without using File Association Manager to associate the handler with any extensions, should have no impact on Explorer whatsoever. Note, though, that you would have to previously have used the File Association Manager to remove any associations that you have previously set up, as these are not removed by uninstalling File Metadata.

[M_Pixel@22/07/2014]
Dijji,
I was able to fix the Explorer problems by uninstalling a few things, then rebooting.
I wasn't initially aware that each extension needed to be manually activated through the Property Handler. The tidbit on the main description doesn't make it clear that this is a necessary step toward achieving functionality. I found out eventually when I read through the documentation.
For ease of use, I would recommend having some bulleted install instructions on the main description that point out that using the Property Handler is a necessary step.
It's good to know that the program doesn't actually interface with Explorer. I was curious why I didn't see it in my extensions list (I suspected that was why I wasn't seeing it working).
Thanks for taking the time to reply! This is a great project. It's too bad SkyDrive doesn't support searching by tags (it preserves them on Office documents, so it must save those streams!). We'll see what happens in Windows 9 then...

[Dijji@23/07/2014]
I'm very pleased to hear that you've now got things working properly - thanks for the update.
Also, thanks for the documentation tip: I am always grateful for feedback that lets me know where I am not being clear. I will tighten this up. It's so hard to think yourself into in the shoes of your readers!
Dijji

[UnknownUser@23/07/2014]

Issue closed by Dijji with comment
Documentation improved to try and make things clearer
Reason closed
Fixed


FIXES Ericsson/codechecker#90

Added delete runs functionality to the web-client.
This involved some code restructuring, and also involved transforming the DiffWidget to the ListOfRunsWidget, which now encompasses the functionality of both the late DiffWidget and the new deleting of runs.
Also made a few minor formatting and renaming changes here and there for better clarity of the code. The biggest one is the merging of checkedRunIds and checkedRunNames arrays into the new diffRuns array in CodeCheckerViewer.js.

Please move refactoring and feature implementation modifications into separate merge request.


Create an example that shows all AMP video types in both an amp-carousel and an amp-according

Use the list of AMP video types in #41 to build one example that shows both the carousel and the accordion on the same page. This will allow people to easily compare and contrast the UX of both components.

@adewale still trying to develop this example.
I am having problems displaying correctly amp-facebook and amp-instagram inside amp-carousel, bugs below:
https://github.com/ampproject/amphtml/issues/2661
https://github.com/ampproject/amphtml/issues/2662

Thanks for the update and for filing the bugs.
On Monday, 21 March 2016, Chiara Chiappini<EMAIL_ADDRESS>wrote:

@adewale https://github.com/adewale still trying to develop this
example.
I am having problems displaying correctly amp-facebook and amp-instagram
inside amp-carousel, bugs below:
ampproject/amphtml#2661
https://github.com/ampproject/amphtml/issues/2661
ampproject/amphtml#2662
https://github.com/ampproject/amphtml/issues/2662
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ampproject/amp-by-example/issues/42#issuecomment-199504413


I don't see any documentation or description about Inference Clusters and how they are different from Compute Clusters

[Enter feedback here]

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

ID: ec8b4be5-1978-ad95-4c40-30e8b52e8cb7
Version Independent ID: eb8b1d57-70c6-7cf0-2ebd-a43dede30bb9
Content: What are compute targets - Azure Machine Learning
Content Source: articles/machine-learning/concept-compute-target.md
Service: machine-learning
Sub-service: core
GitHub Login: @sdgilley
Microsoft Alias: sgilley

Thanks for the feedback! We are currently investigating and will update you shortly.

@sdgilley Hello Sheri, I have seen some external blog about difference between inference cluster and compute clusters like this: https://naadispeaks.wordpress.com/2020/11/30/different-computation-options-on-azure-machine-learning/
Do you think we could add more information about the differences? Thanks.

I'm not sure I understand the ask.  Is it just the terminology?
There's a whole section about different types of compute resources available for training and another for inference.
You can use different types of compute resources for each, including the Azure Machine compute cluster, as shown in the tables in those sections, along with many other choices.


Convert all forms to django forms

Description
Use the following format to name and describe your user stories
As a user"when I try to submit a form without all the fields",
So that " A pop up appears in the field to tell me",
I want "to have this instead of the error message at the tip"
Acceptance Criteria
What are some implicit tasks/criteria to be achieved within this story?

[ ] Criteria/Task 1 Convert forms to django forms
[ ] Criteria/Task 2 remove old error message "invalid form submission"
[ ] Criteria/Task 3 Make sure all fields upload properly

Changed the title from Django Forms to model forms, since that's what it's being changed to.
Change the priority from low to high since it works now, and with the UI overhaul, it might just be a time sink. If this isn't the case then we can change the priority back.