Hosted service Damage_Assessment_in_Daraa_Syria/FeatureServer does not have a service definition item

The hosted service "Damage Assessment in Dar'a Syria" item exists id:abafa1c91d5349548b7fb65c10ffd194 but does not have an associated service definition portal item. Without the service definition item the service won't be published to any user conference machines.
NOTE: Only hosted services published from ArcGIS Desktop or ArcGIS Pro, which upload a service definition item to portal, will be published to the UC servers.
Service owned by Sarah S.

@sschwoppe

@jrweakland Thanks Jim. We couldn't assign to proper person until her account was added to the Esri org. I'm reassigning now.

This was fixed as part of the release process I assisted @nfeuerstein with.


[WIP] disable cdef and lrf for parallelism testing

Coverage decreased (-8.3%) to 67.292% when pulling cff21887622040458ad12e3a2142f85078336261 on tdaede:no_cdef_lrf into 513ca08d15c43823f2397332926ff3eeb57b8da0 on xiph:master.

I'd land the speed feature, it is useful to craft specific bitstream in itself.


Cleanup and harmonize markdown with focus on code

This PR does a cleanup and harmonizes the use of markdown in this article, with focus on the markdown. In summary it:

Adds space to the headings as recommended in the contribution guidelines
Uses code syntax for all samples
Adds correct language hints to all code samples (including VB and Kusto)
Fixes syntax errors in code samples
Removes unnecessary default indentations for code samples
Harmonizes the order of languages in code samples
Harmonizes the way of writing Kusto queries

 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.:x: PeterOrneholm sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@PeterOrneholm : Thanks for your contribution! The author, @mrbullwinkle, has been notified to review your proposed change.

@PeterOrneholm this is awesome, thank you so much for your contribution to our docs! This particular doc is the most popular from a page view perspective in the Application Insights doc set and your changes will make the experience better for everyone.

#sign-off

@mrbullwinkle Thanks for accepting it so quickly! Glad to be able to help as Application Insights is a product I use often and therefore a docs I read and refer to as often :)


Error: undefined method relationship for "":String .../lib/jsonapi/path_segment.rb:33:in eql?

This issue is a (choose one):

[X] Problem/bug report.
[ ] Feature request.
[ ] Request for support. Note: Please try to avoid submitting issues for support requests. Use Gitter instead.

Checklist before submitting:

[x] I've searched for an existing issue.
[x] I've asked my question on Gitter and have not received a satisfactory answer.
[ ] I've included a complete bug report template. This step helps us and allows us to see the bug without trying to reproduce the problem from your description. It helps you because you will frequently detect if it's a problem specific to your project.
[ ] The feature I'm asking for is compliant with the JSON:API spec.

Description
Bug reports:
Occasionally, the server starts giving errors like this:
rails_1      | E, [2020-05-12T14:34:02.613861 #1] ERROR -- : [cbd6c33b-0846-4e3c-8477-1831b935119b] Internal Server Error: undefined method `relationship' for "":String /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/path_segment.rb:33:in `eql?'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:123:in `has_key?'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:123:in `add_join_details'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:179:in `block (2 levels) in perform_joins'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:144:in `each'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:144:in `block in perform_joins'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:143:in `each'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:143:in `perform_joins'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation/join_manager.rb:41:in `join'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation_resource.rb:720:in `apply_joins'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation_resource.rb:695:in `apply_request_settings_to_records'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation_resource.rb:400:in `find_related_monomorphic_fragments'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/active_relation_resource.rb:223:in `find_included_fragments'
rails_1      | /usr/local/bundle/bundler/gems/jsonapi-resources-25911bc19871/lib/jsonapi/processor.rb:441:in `block in load_included'

https://sentry.io/share/issue/b7f7c312a14e43aabd7e8c982613a4c6/
When it starts happening, it seems to continue to happen, but restarting the server clears it up (until it happens again radomly some time later). This has the feel of a caching bug? or perhaps something related to loading?
This started happening after upgrading Ruby from 2.6 to 2.7.  Another user reported seeing the same issue on Gitter, and for them it cleared up after downgrading Ruby. I would like to avoid doing that as there are features in 2.7 I'm already using.

I see now this is a dupe of  #1333 , now fixed.


Does it work on Linux?

This tool should work on Linux?

It has some windows specific calls. It would be relatively simple to configure for linux. I will look into it.


Fix logging

Description
Fix logging by updating related packages for SLF4J 2.0.x
Issues Resolved
Fixes #2097
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Looks good. Thanks!
I wonder if we can have a simple integration test to ensure that logging continues to work so that we don't hit this issue again.


拼接屏幕

Description of the new feature / enhancement
将数个物理屏幕模拟成一个逻辑屏幕
Scenario when this would be used?
使用多个屏幕时，方便显示
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/972


Crash with view for vertex table

What happens?
Hi, I'm trying the extension for the first time and ran into a crash with:
CREATE TABLE vdata(id TEXT PRIMARY KEY);
CREATE VIEW v AS SELECT * FROM vdata;
CREATE TABLE w(id TEXT PRIMARY KEY);
CREATE TABLE v_w(v_id TEXT, w_id TEXT);
LOAD duckpgq;
CREATE PROPERTY GRAPH g1
  VERTEX TABLES (v, w)

  EDGE TABLES (
    v_w SOURCE KEY (v_id) REFERENCES v (id)
        DESTINATION KEY (w_id) REFERENCES w (id)
    LABEL hasW
  );

it crashes at
thread #1, queue = 'com.apple.main-thread', stop reason = EXC_BAD_ACCESS (code=1, address=0x0)
  * frame #0: 0x0000000000000000
    frame #1: 0x00000001038af5e0 duckpgq.duckdb_extension`duckdb::Binder::BindCreateViewInfo(duckdb::CreateViewInfo&) + 224
    frame #2: 0x00000001038b3bb8 duckpgq.duckdb_extension`duckdb::Binder::Bind(duckdb::CreateStatement&) + 1676
    frame #3: 0x00000001037f4078 duckpgq.duckdb_extension`duckpgq::core::duckpgq_bind(duckdb::ClientContext&, duckdb::Binder&, duckdb::OperatorExtensionInfo*, duckdb::SQLStatement&) + 216
    frame #4: 0x000000010017b930 duckdb`duckdb::Planner::CreatePlan(duckdb::SQLStatement&) + 1188
    frame #5: 0x0000000100d0d6c4 duckdb`duckdb::ClientContext::CreatePreparedStatementInternal(duckdb::ClientContextLock&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>, duckdb::optional_ptr<std::__1::unordered_map<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>, duckdb::BoundParameterData, duckdb::CaseInsensitiveStringHashFunction, duckdb::CaseInsensitiveStringEquality, std::__1::allocator<std::__1::pair<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const, duckdb::BoundParameterData>>>, true>) + 512
    frame #6: 0x0000000100d0e0c4 duckdb`duckdb::ClientContext::CreatePreparedStatement(duckdb::ClientContextLock&, std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const&, duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>, duckdb::optional_ptr<std::__1::unordered_map<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>>, duckdb::BoundParameterData, duckdb::CaseInsensitiveStringHashFunction, duckdb::CaseInsensitiveStringEquality, std::__1::allocator<std::__1::pair<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char>> const, duckdb::BoundParameterData>>>, true>, duckdb::PreparedStatementMode) + 480
    frame #7: 0x0000000100d6574c duckdb`std::__1::__function::__func<duckdb::ClientContext::PrepareInternal(duckdb::ClientContextLock&, duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>)::$_2, std::__1::allocator<duckdb::ClientContext::PrepareInternal(duckdb::ClientContextLock&, duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>)::$_2>, void ()>::operator()() + 60
    frame #8: 0x0000000100d0ffb8 duckdb`duckdb::ClientContext::RunFunctionInTransactionInternal(duckdb::ClientContextLock&, std::__1::function<void ()> const&, bool) + 124
    frame #9: 0x0000000100d102ac duckdb`duckdb::ClientContext::PrepareInternal(duckdb::ClientContextLock&, duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>) + 216
    frame #10: 0x0000000100d10658 duckdb`duckdb::ClientContext::Prepare(duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>) + 168
    frame #11: 0x0000000100d1a35c duckdb`duckdb::Connection::Prepare(duckdb::unique_ptr<duckdb::SQLStatement, std::__1::default_delete<duckdb::SQLStatement>, true>) + 52
    frame #12: 0x0000000100028830 duckdb`duckdb_shell_sqlite3_prepare_v2 + 788
    frame #13: 0x000000010000eb54 duckdb`shell_exec + 184
    frame #14: 0x000000010001a258 duckdb`runOneSqlLine + 156
    frame #15: 0x000000010000f9c4 duckdb`process_input + 964
    frame #16: 0x00000001000082b8 duckdb`main + 1788
    frame #17: 0x0000000181063154 dyld`start + 2476

In my real application I use a lot of views for what would be both the edge tables and the vertex tables. Would pgq still be useful without all the foreign keys?
To Reproduce
CREATE TABLE vdata(id TEXT PRIMARY KEY);
CREATE VIEW v AS SELECT * FROM vdata;
CREATE TABLE w(id TEXT PRIMARY KEY);
CREATE TABLE v_w(v_id TEXT, w_id TEXT);
LOAD duckpgq;
CREATE PROPERTY GRAPH g1
  VERTEX TABLES (v, w)

  EDGE TABLES (
    v_w SOURCE KEY (v_id) REFERENCES v (id)
        DESTINATION KEY (w_id) REFERENCES w (id)
    LABEL hasW
  );

OS:
osx, arm64
DuckDB Version:
1.1.2
DuckDB Client:
unix cmd line
Full Name:
eaubin
Affiliation:
n/a
How did you load the extension?
LOAD duckpgq;
Did you include all relevant data sets for reproducing the issue?
Yes
Did you include all code required to reproduce the issue?

[X] Yes, I have

Did you include all relevant configuration (e.g., CPU architecture, Python version, Linux distribution) to reproduce the issue?

[X] Yes, I have

Thanks for the report! Unfortunately, I don't have an immediate solution to support property graph creation on views. Internally they are different than standard tables. I just pushed b29e8ab27d6aae8e61eaee991ba41b5a269cb6fc and ac29e245d2172eb8f2e696e2f1ae0708c3a34bd6 that throws a better error message. In your case:
Invalid Error: Found a view with name v. Creating property graph tables over views is currently not supported.

Would pgq still be useful without all the foreign keys?
I am not exactly sure what you mean with foreign keys in this case, sorry. To support views for property graphs I need more time to figure that out, but I wanted to at least throw a proper error message instead of crashing.

In the mean time I have created this issue to track the progress: https://github.com/cwida/duckpgq-extension/issues/159

Nevermind the foreign keys question, trying to minimize my example I had foreign key errors at some point and now can't reproduce it.


How to compile to generate the linux64 package

Hello, I am currently using the version v0.5-Epsilon exactly the package reskit-linux64.zip
and I have found that I cannot assign the products.
In v0.6 there is no linux64 package and I don't know how to compile.
Thank you very much for the help

I think I have managed to compile using the Dockerfile command
dotnet publish apimtemplate.sln -c Release -o out


Is there an image width limit of 640 pixels?

Before you create this issue
This library takes compressed PNG data and outputs pixels to your callback functions. If your problem is related to LCD displays, SD cards, or any other issue that has nothing to do with this library, please stop here. If you need help with your private or commercial project, please email me<EMAIL_ADDRESS>to ask about paid consulting. Github issues are for submitting actual bugs.
Describe the bug
A clear and concise description of what the bug is.
An image of width 641 pixels is not decoded, but it is decoded if the width is 640.
To Reproduce
Steps to reproduce the behavior:

Go to: https://notisrac.github.io/FileToCArray/ and convert an 641x800 image

Compile de code: https://github.com/Bodmer/TFT_eSPI/blob/master/examples/PNG Images/Flash_PNG/Flash_PNG.ino

Change the MAX_IMAGE_WIDTH to 900

See error

When the problem occurs (above 640 width) the message "Successfully opened png file" does not appear on the serial monitor.

Expected behavior
A clear and concise description of what you expected to happen.

A 480x800 image is displayed. While an 800x480 image is not.
I don't know if it's a problem with the image converter website or this PNG library.
Do you have another converter to test?

Image you're having trouble with
If applicable, add original image files to help explain your problem.
Additional context
Add any other context about the problem here.

Hardware: ESP32 DEVKIT V1

https://github.com/Bodmer/TFT_eSPI/issues/3185

Yes, this is to reduce RAM needed on MCUs. You can change it - see PNGdec.h:
/* Defines and variables /
#define PNG_FILE_BUF_SIZE 2048
// Number of bytes to reserve for current and previous lines
// Defaults to 640 32-bit pixels max width
#define PNG_MAX_BUFFERED_PIXELS (6404 + 1)

How is PNG_FILE_BUF_SIZE related to PNG_MAX_BUFFERED_PIXELS?
I increased the value of PNG_MAX_BUFFERED_PIXELS, but it decreased the value of PNG_FILE_BUF_SIZE, and even so it is now working with an 800x480 image.
/* Defines and variables */
#define PNG_FILE_BUF_SIZE 1024
// Number of bytes to reserve for current and previous lines
// Defaults to 640 32-bit pixels max width
#define PNG_MAX_BUFFERED_PIXELS (800*4 + 1)

Sorry, the max buffered pixels needs to be large enough to hold 2 lines (current and previous). If you're decoding 32-bit pixels, then that's 800 x 2 x 4

Sorry, the max buffered pixels needs to be large enough to hold 2 lines (current and previous). If you're decoding 32-bit pixels, then that's 800 x 2 x 4

So if '800 x 2 x 4' = '6400', it should look like this, as shown below?
#define PNG_FILE_BUF_SIZE 6400 // 800 x 2 x 4
#define PNG_MAX_BUFFERED_PIXELS (800*4 + 1)

Or should it look like this, as shown below?
#define PNG_FILE_BUF_SIZE 2048
#define PNG_MAX_BUFFERED_PIXELS (800*2*4 + 1)

Does the value of PNG_FILE_BUF_SIZE not matter?

MAX_BUFFERED_PIXELS is for the pixels, FILE_BUF is for uncompressed data. A small file buffer size will slow it down due to more calls to read more data.

There is TRUECOLOR (24-bpp) and TRUECOLOR with ALPHA (32-bpp). You can use my imageinfo tool to show the difference: https://github.com/bitbank2/imageinfo

There is TRUECOLOR (24-bpp) and TRUECOLOR with ALPHA (32-bpp). You can use my imageinfo tool to show the difference: https://github.com/bitbank2/imageinfo

Thank you,
I found a website: Get All Metadata Info Of Your Files https://www.metadata2go.com/view-metadata
Apparently the image is 8 bits, which is why it is displayed with PNG_MAX_BUFFERED_PIXELS of '800x4+1' instead of '800x2x4+1':

bit depth = 8 can mean 24 or 32-bit (R=8, G=8, B=8, A=8)
color_type = indexed = palette = 8 bits or less total
your image has color_type RGB with Alpha = 32 bits per pixel

bit depth = 8 can mean 24 or 32-bit (R=8, G=8, B=8, A=8) color_type = indexed = palette = 8 bits or less total your image has color_type RGB with Alpha = 32 bits per pixel.

Yes, thank you. I found this other website that explains about this (Interesting how the small images do not show such significant visible differences)
Can I consider the '800x4+1' setting as appropriate since the converter website is generating 8-bit data?
When the "Treat as binary" option is not active and 32 bits is selected, then the data is 8 bits x 4 in length:

"Treat as binary" data: 0x89
The 32 bit RGBA data: 0x6e97d0ff

Use my image_to_c tool to convert files for Arduino. No options needed, the data comes out the way it should.


Mass conversion insert API version 2

For Include 'Add Branding Campaign' to Mass Adjustment Tool in BO we need to create approved conversions in Cake.
The API to create manual conversions in Cake is MassConversionInsert. The problem is that conversions created by the API are not approved. To approve them we need to use the UpdateConversion API. But MassConversionInsert doesn't give us conversion_ids that were created.
The UpdateConversion API allows the use of transaction_id as alternative conversion identifier, but only version 2 of MassConversionInsert can set the conversion transaction_id.
The MassConversionInsert was downgraded to version 1 because of an "invalid action" error in version 2.
So I figure out a way to fix the error in version 2, so we can use it to set conversion transaction_id and then use UpdateConversion to approve it.

 (8) Include 'Add Branding Campaign' to Mass Adjustment Tool in BO


Disable setting a profile.carbratio_adjustmentratio < 1.

This could have surprising or dangerous effects when used with autotune or smb.

What's the use case for keeping carbratio_adjustmentratio at all?  In what situations would you want to set it > 1?

If no one is successfully using it on an ongoing basis, I'm inclined to deprecate/remove carbratio_adjustmentratio entirely.  It seems to be a knob people try, unsuccessfully, to use before moving on to doing things another way.

I thought it might be good to keep it for safe measure, i just wanted to make it difficult to set it lower than 1 as it's now dangerous, specially since there are old documentation suggesting to do so and that it might only apply to the pump CR which is not true.
This also adjusts the autotuned CR.
i don't personally needed right now, but it might possibly be a useful advanced tweaking point once i re-enable autotune in case it suggest CR that are still too low for any reason.

thought about this a little more, I guess you're right that there are too many parameters, so it if you prefer i'll remove it, and if needed again it could easily be re-added, maybe with a different name somethnig like autotune_carbratio_multiplier to make it clear.

i removed the parameter completely have the code on my rig, but didn't test a re-install

oops, didn't mean to close it, was supposed to just be a comment.

I use this, I have it set to 1.2 making it more aggressive with meals. tends to do a great job for me.

That sounds backwards. 1.2 makes what more aggressive? Also, are you running autotune?

All:  If you've used carbratio_adjustmentratio in the past, or are using it currently, I'd like to know how you used / are using it, and what your experience has been with it. If anyone is using it successfully I'd like to understand exactly how you're using it so we can decide whether to keep it, and if so how to make it safer for new users and those using autotune. And if you used it but gave up on it, we'd also like to understand why you tried it originally and why you stopped using it.

I started using this when I was having issues with SMB, this helped me at least during that transition. But I've for the past week set it as 1 so yeah it's no longer needed for me, so I'm OK with this.

Ok.  If no one else thinks this needs to be kept, I'm planning to merge this PR as-is.


Can't install  @bearclaw/cel, @buf/cel_spec.bufbuild_es not found

Describe the bug
% npm install @bearclaw/cel

npm error code E404
npm error 404 Not Found - GET<EMAIL_ADDRESS>- Not found
npm error 404
npm error 404  '<EMAIL_ADDRESS>is not in this registry.
npm error 404
npm error 404 Note that you can also install from a
npm error 404 tarball, folder, http url, or git url.
npm error A complete log of this run can be found in: /Users/robertyokota/.npm/_logs/2024-12-19T04_18_34_361Z-debug-0.log

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Also [@buf/google_cel-spec.bufbuild_es](https://www.npmjs.com/package/%40buf%2Fgoogle_cel-spec.bufbuild_es) is not public as well

You’ll need to configure your package manager to use the buf registry. This is documented here but I should have added that to the readme.
For npm, the command is npm config set @buf:registry https://buf.build/gen/npm/v1/
After that, install should work


Fixes segfault and openssl issues as well as various environment setup issues

Before contributing, please read our contributing guidelines and code of conduct.
Overview

Fixes segfault with mysql2 with crossing openssl linkage
Fixes nokogiri compiling issues
ensures all native extensions compiling against older openssl for EOL Rubies
shorter job names
better organized flows in the build action
improvements to multiverse runner

@jjarmain. This is the PR with the javascript action.  Would love your input/review of this PR


... showing up in the title of code block

What version of starlight are you using?
0.13.0
What version of astro are you using?
3.2.3
What package manager are you using?
npm
What operating system are you using?
Mac
What browser are you using?
Firefox
Describe the Bug
I am not expecting ... to show up as the title of a code block.
Here's the source code:
## Run loop

Our application runs in a continuous loop, constantly checking for user input and updating the
state, which in turn updates the display on the next loop.

```rust
  // Main application loop
  loop {
    // draw UI based on state
    // ...
    // Update state based on user input
    // ...
    // Break from loop based on user input and/or state
  }
```

Every TUI with `ratatui` is bound to have (at least) one main application run loop like this.

https://github.com/ratatui-org/website/blob/28d94f28861b882c3855b33d2ba76d9d54f9d327/src/content/docs/tutorials/counter-app/single-function.md#L110-L126
And this is what it renders as:

I've never used stackblitz before and I'm not able to get starlight to even render any of my changes there, so wasn't able to create a reproducible build
Link to Minimal Reproducible Example
https://stackblitz.com/edit/github-8w69sw-ubyxl7?file=src%2Fcontent%2Fdocs%2Findex.mdx
Participation

[ ] I am willing to submit a pull request for this issue.

Thanks for your feedback.
I feels like this may be an issue with Expressive Code and the frame plugin feature to add a frame title using comments. Maybe the first comment in the file with a dot and not whitespace in the text is interpreted as frame title but I am not sure without looking at the code.
(ping @hippotastic )
There are two potential workarounds in the meantime:

Hiding the frame:
```rust frame="none"
// Main application loop
loop {
  // draw UI based on state
  // ...
  // Update state based on user input
  // ...
  // Break from loop based on user input and/or state
}
```

Giving explicitely a frame title:
```rust title="main.rs"
// Main application loop
loop {
  // draw UI based on state
  // ...
  // Update state based on user input
  // ...
  // Break from loop based on user input and/or state
}
```

Here is a link to a repro and the 2 workarounds.

The workaround is good enough for us. Feel free to close this issue if you think it is not relevant for this repo!

Thank you for reporting this! I'll fix the issue upstream in Expressive Code and let you know here.

Thanks for the quick fix @hippotastic! Will close this issue as resolves upstream for now.


withdraw rewards confirmation popup text update

Describe the bug
when you try to withdraw your rewards, the transaction confirmation popup is using the text "You are staking and protecting:"

To Reproduce

go to protection tab
click "withdraw" rewards
proceed with "withdraw"
see a confirmation popup

current text: You are staking and protecting:
updated text: You are withdrawing rewards:

Fixed here

Fixed here


Wrong converter name in links

Docs for Toolkit PR #
What changes to the docs does this PR provide?
IndexToArrayItemConverter name was incorrectly used on EnumToBoolConverter docs page.
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] For new pages, used the provided template.
[ ] For new features, added an entry in the Table of Contents.
[x] Ran against a spell and grammar checker.
[x] Contains NO breaking changes.

Other information

Learn Build status updates of commit 0ace278:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/converters/enumtoboolconverter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Thanks @jeff-fichtner! We do actually have an EnumToBoolConverter example in CommunityToolkit.Maui.Sample: https://github.com/CommunityToolkit/Maui/blob/main/samples/CommunityToolkit.Maui.Sample/Pages/Converters/EnumToBoolConverterPage.xaml
Perhaps we could update this PR to point to the sample? We must've written the docs before we created the sample 😅

@brminnick added!

Learn Build status updates of commit 9fe0200:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/converters/enumtoboolconverter.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin Community Toolkit. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI Community Toolkit is the replacement for Xamarin Community Toolkit and its documentation can be found here.


A few minor changes

There are a couple of reversions here (formatting and boxing-related), plus handling for negative zero floats, plus a couple of tweaks to the tests. Apart from these things https://github.com/circe/circe/pull/603 looks good to me.

Just one small comment. Outside of that, looks good to merge.

Also, want me to squash this all into one commit?

Ugh, yes, you're right about the Arbitrary. I'm not at a computer for the next hour but can fix it then, or if you don't mind you could do it.
I could go either way on squashing—your call.

Merged and squashed, thanks!


Feature's label is not displayed in result print

I am using Geoserver 2.10.0 for printing map. Geoserver uses mapfish-print for printing.
I wanted to print a vector layer with a label in Russian. And in the result of print this label is not displayed, but when I set label in English label is displayed.
What should I do to solve this problem?
Code example:
"type": 'Vector',
     "styles": highlighFeaturesStyle,
           "geoJson": {
                "type": "FeatureCollection",
                "features":  [ {
                     "type": "Feature",
                     "geometry": JSON.parse(_geojson),
                     "properties": {
                          "_style": {
                              "strokeColor": "#336791",
                              "strokeOpacity": 0.8,
                              "strokeWidth": 3,
                              "fillColor": "#336791",
                              "fillOpacity": 0.8,
                              "label": "Текст пример",
                              "labelYOffset": "14.0",
                              "fontColor": "#000000",
                              "fontFamily": "sans-serif",
                              "fontSize": "10px",
                              "fontStyle": "normal",
                              "fontWeight": "bold"
                             }
                        }
                   } ]
              }

Looks like someone haven't been using Unicode sometime. Its interesting if they still support version 2 i.e. fixing bugs.

No, we don't work on version 2 anymore. But, usually PRs are welcome and we can do releases for those.

Well, here you are :)
https://github.com/mapfish/mapfish-print/pull/986
Though I don't sure if branch 2.1.x is stable, there are some checks and tests errors.

Yeah, cicrcleci doesn't work with this branch. But I don't understand why travis didn't trigger...

Or @sbrunner, can you have a look?

@pvalsecc We don't seem to have any CI configured on this branch ...


Build with Scala 2.13

A few projects are broken because of missing dependencies:

play-java-grpc-example

com.lightbend.akka.grpc#akka-grpc-runtime_2.13;0.6.2: not found
com.lightbend.play#play-grpc-testkit_2.13;0.7.0: not found

play-scala-grpc-example

[warn] 	:: com.thesamet.scalapb#scalapb-runtime_2.13;0.8.4: not found
[warn] 	:: com.lightbend.akka.grpc#akka-grpc-runtime_2.13;0.6.2: not found
[warn] 	:: com.lightbend.play#play-grpc-scalatest_2.13;0.7.0: not found
[warn] 	:: com.lightbend.play#play-grpc-specs2_2.13;0.7.0: not found

play-scala-rest-api-example

[warn] 	:: com.netaporter#scala-uri_2.13;0.4.16: not found
  // scala-uri also needs
  [warn] 	:: org.parboiled#parboiled_2.13;2.1.5: not found
  [warn] 	:: io.spray#spray-json_2.13;1.3.4: not found
[warn] 	:: net.codingwell#scala-guice_2.13;4.2.1: not found
  send PR https://github.com/codingwell/scala-guice/pull/81

play-scala-isolated-slick-example

// they updated the repo, but no release so far. I pinged them
[warn] 	:: com.github.tototoshi#slick-joda-mapper_2.13;2.3.0: not found

play-java-rest-api-example

This is due to Gatling (Java lib) using some Scala 2.12 lib underneath
[error] java.lang.RuntimeException: Conflicting cross-version suffixes in: com.typesafe.akka:akka-actor, org.scala-lang.modules:scala-xml, org.scala-lang.modules:scala-java8-compat, org.scala-lang.modules:scala-parser-combinators, com.typesafe.akka:akka-slf4j

play-java-dagger2-exmaple

I think this is a regression either in Play 2.7.3 or Twirl 4.0.2.(see https://github.com/playframework/play-samples/blob/2.7.x/play-java-dagger2-example/app/views/index.scala.html#L2)
The timeZones should is a java.util.List[String]. I think this is expected since it's a java app. Users in the controller, call the template and pass a java collection. However, the helper.options is expecting a Scala collection.
[error] /home/travis/build/playframework/play-samples/play-java-dagger2-example/app/views/index.scala.html:12:67: overloaded method value apply with alternatives:
[error]   (options: List[String])List[(String, String)] <and>
[error]   (options: java.util.Map[String,String])Seq[(String, String)] <and>
[error]   (options: Map[String,String])Seq[(String, String)] <and>
[error]   (options: (String, String)*)Seq[(String, String)]
[error]  cannot be applied to (java.util.List[String])
[error]         @helper.select(field = form("timeZone"), options = helper.options(timeZones))

Hmm, the dagger example puzzles me.
There is a overloaded method for java.util.List[String]
https://github.com/playframework/playframework/blob/2.7.3/core/play/src/main/scala/views/helper/Helpers.scala#L140

Hmm, the dagger example puzzles me.
There is a overloaded method for java.util.List[String]
<EMAIL_ADDRESS>
I think the problem is, unlike other options.apply() overloads, the apply(java.util.List[String]) returns a mutable Buffer not an immutable.List or Seqand helper.select expects a Seq.
If that's correct it's a regression introduced by changes in the scala collections API and we'll need to fix in Helpers.options so the returned type is Seq.

@ignasi35, tested locally, and this seems to do the trick
https://github.com/playframework/play-samples/pull/32/commits/a4e78665f4295fd8261776443acd2ad571c8f812#diff-766d86b2881bb24ed2680edfb82dd524R12
Thanks for catching this up. That was exactly as you explained.

I did some updates here:

projects that are blocked are now on Scala 2.12.8, we don't want to block all the others because of this
fixed some dependencies that are now providing binaries for 2.13.
migrated one sample from 'com.pretaporter' to 'io.lemonlabs' (project moved in GitHub)

@ignasi35

@renatocaval let's create separate issues for each project that still won't bump to 2.13 and needs revisiting

This is already done.
See #33 and #34


Modal dialog closes when dragging and releasing mouse button outside dialog

When clicking and holding the left mouse button inside a modal dialog and then dragging the mouse outside the dialog and releasing the mouse button results in the modal dialog closing.
I have a use case where you can crop an image inside a dialog and when the users drags his mouse to far and releases the mouse button the dialog closes, causing the user to restart the process.
I know I can make the backdrop static so it doesn't close when clicking on it, but this would be a last resort.
I think that the backdrop should only close if the mousedown event happened on the backdrop and not on the dialog.

This is still an issue when cropping images inside non-static modals. Doing something with stopPropagation is not an effective workaround when using a third-party cropping library like Jcrop. Temporarily making the modal static while cropping is also not workable in my use case. Are there any plans to re-fix this issue?

try
.modal{   pointer-events: none;   .modal-dialog{     pointer-events: none;     .modal-content{       pointer-events: auto;     }   } }

try
.modal{ pointer-events: none; .modal-dialog{ pointer-events: none; .modal-content{ pointer-events: auto; } } }

Just FYI, this fix breaks scrolling on chrome when modal height is more than screen height.
It's also odd, that it's usually happens on wider screen, but on smaller screen it's seems to be working just fine. Changing screen size randomly make this work, and randomly breaking.
I believe this is chrome bug, as Firefox handles pointer-events and scrolling differently, and this fix works with Firefox.

Just FYI, this fix breaks scrolling on chrome when modal height is more than screen height.

This may be relevant to finding a fix, but in my case, scrolling only breaks if the page itself lacks a scroll bar. Ie, if my page doesn't scroll, the modal won't scroll. Resizing the window by 1px to give the page a scroll bar causes scrolling to work again with the modal even though the modal still exceeds the page and window height, so it doesn't seem to actually depend on the modal height, but rather the presence of a scroll bar on the page.

Just FYI, this fix breaks scrolling on chrome when modal height is more than screen height.

This may be relevant to finding a fix, but in my case, scrolling only breaks if the page itself lacks a scroll bar. Ie, if my page doesn't scroll, the modal won't scroll. Resizing the window by 1px to give the page a scroll bar causes scrolling to work again with the modal even though the modal still exceeds the page and window height, so it doesn't seem to actually depend on the modal height, but rather the presence of a scroll bar on the page.

Can anyone suggest where I can set a breakpoint that will break after the triggering event has occurred, but before the modal has closed?


Optimization: Add support for Poly IC

There are different forks of quickjs that implement bits and pieces of really awesome things around QuickJS. One such effort by @openwebf team implements support for Poly IC in quickjs which results in some significant improvement in performance by up to 40%.
https://github.com/openwebf/quickjs
And even a patch shared on freelists devel: https://www.freelists.org/post/quickjs-devel/Support-Poly-Inline-Cache-in-Get-and-Set-Property
Any thoughts about whether this could be something that is added here in future too?

Aha coincidence! I was just trying out that patch with quickjs-ng

You might want to try the code in the repo rather than the freelist one since it was updated with some fixes. The perf difference is day and night btw.

I did face some issues with the fork: more details in this issue: https://github.com/openwebf/quickjs/issues/53

I see they de-amalgamated QuickJS, which is something I wanted to do, but it complicates integrating other patches...

Yeah you are right about that. Each file corresponds to a section of code in the quickjs.c file so it's still possible to patch it where needed.
However I actually prefer a single quickjs.c file, it's easier to compile and setup anywhere with simple build steps. Breaking down the code kind of complicates the build process. There's probably a good reason Bellard put everything in a single file haha.

Managed to condese most of the early patches into https://github.com/quickjs-ng/quickjs/pull/120
It does crash when running test262 (probably a silly copy pasta error though)

Managed to condese most of the early patches into https://github.com/quickjs-ng/quickjs/pull/120
It does crash when running test262 (probably a silly copy pasta error though)

That was quick! 🙏


Linked titles don't wrap on mobile

On the Guides page on mobile, the titles (which are links) don't wrap onto another line and, therefore, extend off the page / device. https://donejs.com/Guides.html

Hi, Just came across this.
For anyone bothering fixing this.
In the I assumed compiled static.css theres the following css:
body:not(.donejs):not(.community)>.container-fluid .content.docs .comment a, body:not(.donejs):not(.community)>.container-fluid .content.docs .description a {     white-space: nowrap;     font-weight: 900;     color: #3B83A3;
I have no knowledge of why this has been done and what changes it may affect or if this is still included because of older code.
But the white-space: nowrap should be removed or overwritten with white-space: normal.
This should fix it for this page.

Fixed by #820


New shared ride type shared-start-and-target

Scenario:

two or more passengers share a ride from the same start AND target addresses
this scenario has not been considered yet, the middle route address (B) can't be deleted

Screenshot:

This has been fixed in commit https://github.com/TIXI24/sfitixi/commit/bd0b59ea736e0c0ee5e8c60a73bccc73ee7d208d


Pie chart tool tip is blinking

Move the cursor to pie chart tool tip is blinking. its not showing properly

Mow to remove the legend board on the right?

Solution add css:
svg > g > g:last-child { pointer-events: none }


Any support or workarounds to be compatible with Player Revive mod

I know it's too much to ask for a support but maybe someone knows about a workaround or something like that: when I use the Player Revive Mod and a player dies, the bot will show death message for when the player is down (and waiting for revival) and when it dies for real
This way it's impossible to know on Discord if someone is only waiting to be revived or dead dead.
Thanks for the help, if possible!
I'm using version forge 47.2.2 and mod version 3.0.1

I'll take a look at how we can support it. Hopefully they have an api or some way for us to add compatibility


ORDER doesn't work for me

Prolog: First, thank you so much for writing way-displays. It is exactly the tool I have been wanting - so much so that I started writing one myself in zig awhile back -  but better than I would have had time to implement. I greatly appreciate your work on this.
Issue: I can't seem to get the ORDER directive to have an effect.
I have a laptop with eDP-1 and DP-1 (HDMI via thunderbolt 3).
eDP-1 is always on top of the column, regardless of the ORDER in the config.
ARRANGE: COL
ALIGN: MIDDLE
ORDER:
  - 'DP-1'
  - 'eDP-1'
AUTO_SCALE: FALSE
SCALE:
  - NAME_DESC: 'eDP-1'
    SCALE: 2.0

The log reports the correct order:
I [15:53:06] Reloading configuration file: /home/edrex/.config/way-displays/cfg.yaml
I [15:53:06]
I [15:53:06] New configuration:
I [15:53:06]   Arrange in a COLUMN aligned at the MIDDLE
I [15:53:06]   Order:
I [15:53:06]     DP-1
I [15:53:06]     eDP-1
I [15:53:06]   Auto scale: OFF
I [15:53:06]   Scale:
I [15:53:06]     eDP-1: 2.000
I [15:53:06]
I [15:53:06] eDP-1 Changing:
I [15:53:06]   from:
I [15:53:06]     scale:    1.000
I [15:53:06]     position: 0,0
I [15:53:06]     mode:     3200x1800@60Hz (59,982mHz) (preferred)
I [15:53:06]   to:
I [15:53:06]     scale:    2.000
I [15:53:06]     position: 160,0
I [15:53:06]
I [15:53:06] DP-1 Changing:
I [15:53:06]   from:
I [15:53:06]     scale:    1.000
I [15:53:06]     position: 640,1800
I [15:53:06]     mode:     1920x1080@60Hz (60,000mHz) (preferred)
I [15:53:06]   to:
I [15:53:06]     position: 0,900
I [15:53:06]
I [15:53:06] Changes successful

related: #44

It seem that we might be having some issues matching the name of the displays: eDP-1 contains DP-1.
Please try specifying the description instead: https://github.com/alex-courtis/way-displays/blob/master/doc/CONFIGURATION.md#on-names-and-descriptions
The names (DP-1, eDP-1 etc) are ephemeral and change depending on what hardware is plugged in etc. Description never changes.

Hey, good find, that that was it.
The DP-\d device identifiers indeed aren't stable, if multiple adapters are used, but eDP-1 is.
Semantically, the order rule I want is "external first (regardless of device name), then eDP-1 (name or desc doesn't matter, as both are stable identifiers barring a graphics card swap)". It seems like https://github.com/alex-courtis/way-displays/issues/46 would allow that.
I don't want to use desc for the external display, since I want a generic rule that works for whatever monitor I hook up to.
Mixing desc and name works for me for now:
ORDER:
  - DP-1
  - "Sharp Corporation 0x144A 0x00000000 (eDP-1)"

I'd be happy to submit a fix for the unintended substring match if it's helpful.

I don't want to use desc for the external display, since I want a generic rule that works for whatever monitor I hook up to.

Yes. That's a good use case.

I'd be happy to submit a fix for the unintended substring match if it's helpful.

That would be most gratefully appreciated!

Mixing desc and name works for me for now

Hmm, this morning I'm finding that sometimes this workaround isn't doing the job (maybe after device unplug/replug or suspend, not sure. Restarting way-displays seems to fix it.

I'd be happy to submit a fix for the unintended substring match if it's helpful.

Don't hold it for me, if you find yourself in that section of the code :sweat_smile:. I was feeling optimistic about my availability yesterday, which is a character flaw, but realistically I'm not sure when I'll have my plate clear enough to prioritize it.

Updated workaround: List out the names of all displays used under ORDER.

Hmm, this morning I'm finding that sometimes this workaround isn't doing the job (maybe after device unplug/replug or suspend, not sure. Restarting way-displays seems to fix it.

Perhaps that's an isue with unpredictable enumeration order of the outputs.
Don't feel any pressure to put in a fix; we will get to it eventually.

Fixed in 1.7.0
I'd be most grateful if you tested this.

Thanks!
Preliminary testing looks good. Because it's nondeterministic I'll have to run it for awhile to feel confident it's fixed.


Problem with sweetalert

sweetalert2 seems not compatible with your theme, is it right?
In my application I need to use this type of alert, is there a fix to make it works?
At the moment when the alert appears I don't see nothing in the background

not familiar w/ sweet alerts but would material dialogs work for your use case? https://teradata.github.io/covalent/#/components/dialogs

Not entirely sure how sweetalert2 works.. can you provide a stackblitz with the issue and explain what the problem is with bigger detail?

Closing issue, reopen if still a problem


heavy(full) stemcell cannot be used with create-env

it seems that trying to use a full stemcell with bosh create-env it tries to connect to the metadata server from the localhost.
which ofcourse cannot be reached.

more info on this @friegger

We've seen various errors, even when running create-env from EC2 instances, like:
CPI 'create_stemcell' method responded with error: CmdError{"type":"Bosh::Clouds::CloudError","message":"Cannot find EBS volume on current instance","ok_to_retry":false}

or
  CPI 'create_stemcell' method responded with error: CmdError{"type":"Bosh::Clouds::CloudError","message":"Timed out reading instance metadata, please make sure CPI is running on EC2 instance","ok_to_retry":false}

We will instead create a light stemcell before hand, as this does not seem trivial to solve.

Heavy stemcell require access to mounted disks to be able to copy over the stemcell contents using: https://github.com/cloudfoundry/bosh-aws-cpi-release/blob/master/src/bosh_aws_cpi/bin/stemcell-copy
As discussed during working group sync meeting, in this case bosh create-env was run inside a container without access to mounted disks.


Ponder portrait video recording

According to a comment on issue #130, portrait video recording does work on some devices, at least for some cameras. That means the block imposed by that issue is too draconian. Figure out what to do to re-enable portrait video recording, at least for supported environments.

@GNima Did you have any luck using TextureMovieEncoder?


Hotkey to cycle ILs in reverse order

Since pressing I cycles ILs forward, it would be nice if pressing Shift I cycled the ILs in the opposite order.

implemented before stable 2.2


THANKFUL

Hey bro, I just want to demonstrate my thakful for this module. I'm brazilianboy and this solver all my problems.  THANKS SO MUCH!

Heya, thank you very much for the kind words 😄, good to see you are enjoying the module


Needs an option to stop prompting for permissions

:warning: Is your feature request related to a problem? Please describe
OMGSoundboard constantly requests permissions to modify system settings. I will never grant these permissions and never intend to use the features that require these permissions. The constant nagging to enable these permissions makes this app unusable for me, it needs a setting to disable this nagging.
:bulb: Describe the solution you'd like
an options panel allowing me to tell it "stop nagging me for permissions"
Alternatively a "never" option on the nag screen so that it never asks me again.
:bar_chart: Describe alternatives you've considered
Uninstall / use other Apps. This nag screen makes omgsoundboard unusable for me.
:page_facing_up: Additional context
n/a
:raising_hand: Do you want to develop this feature yourself?

[ ] Yes
[X] No

Are you interested to work on this, @Rashxz?

Sure!

thanks!

Hey @AntiSol, this issue was addresed before (#51 ), are you using the latest version of the app?

version 11.0, the one in f-droid
On 20/8/24 04:56, Andrés Velásquez wrote:

Hey @AntiSol, this issue was addresed before (#51 ), are you using the latest version of the app?

Just test that one, the permission prompt it's only appearing once, as intended. Apart from that it only appears if you try to set a tune as ringtone/alarm/notification.

well that's weird. I just tried it again and it hasn't prompted me once.
It was still installed though, so I'm not trying from a fresh install. It definitely nagged me a bunch of times when I first installed it.
just a note that I'm using a de-googled phone, maybe that has something to do with it.
but since I can no longer replicate the problem feel free to close this if you want
thanks for your time looking into it.

No problem.


[Doc]  bug conflicting docs after switching solver to libmamba-solver

📚 Documentation
Bug
Latest doc suggests config libmamba-solver instead of classic for speed -up  https://llm.mlc.ai/docs/install/conda.html
BUT once that's switched,  the mlc_chat_cli from prebuilt no longer works.   It keeps insisting on reverting to classic.
https://llm.mlc.ai/docs/deploy/cli.html

Well, this is a bummer...Which OS are you using btw? I'd love to test if I could reproduce this issue

I am on Windows.   But cannot reproduce the problem today 😞    Closing the issue.  Thanks.


Fix example for windows

It was added derive macros for Android request_permission() function to fix macroquad-permissions example building for Windows.

Able to merge


When processing docx, nothing was returned and no errors were reported

Bug
example.docx
This is my docx. And i run docling example.docx on linux.
but nothing return.
I can open the file on python(using python-docx).
Docling version
Docling version: 2.5.2
Docling Core version: 2.5.1
Docling IBM Models version: 2.0.6
Docling Parse version: 2.1.2
Python version
Python 3.10.8

@nanzai1 there have been some important fixes for docx in the recent releases. Can you please confirm your issues with the latest Docling version?
cc: @maxmnemonic


add spinning wheel for loading wds tables

See AJ-761
For an azure workspace, if WDS is not ready, show a spinning wheel and a message that the database is being created.  Once it exists it will show the same "Select a data type from the navigation panel on the left" message as GCP workspaces.

@calypsomatic for the scope of this ticket, let's go with David's #1 example, "The database that powers your data tables is unavailable. It may take a few minutes after initial workspace creation to be ready."
I agree with following up to provide better messaging for different possible states of WDS after SU-270 is complete.


ROS2 Linting: pointcloud_preprocessor

Summary
This looks like a lot of changes but really this is the result of a file being renamed and git not displaying the diff properly. There were a file which had a .h extension which I converted to a .hpp though another file already had that name so that file name needed to be changed as well.

passthrough_uint16.h -> passthrough_uint16.hpp
passthrough_uint16.hpp -> passthrough_uint16_impl.hpp

Let me know if you think the names don't fit as I don't think its the correct use of "impl" but the #ifndef tag seems to suggest that is the case.

When I batch renamed files from *.h to *.hpp, this was skipped because the hpp file existed already. Now I've taken a second look, and I don't see why this needs to be in two separate files. It is one class template, but two methods are defined in the *_impl.hpp. Maybe it would be best to just merge them into passthrough_uint16.hpp?

@nnmm Thanks for the comments. So I removed the _impl file and merged the two files together. Now the file changes should look like this:

passthrough_uint16.h -> merged into passthrough_uint16.hpp
passthrough_uint16.hpp -> no change

I also made some CMake changes to get rid of compiler warnings, see #129

@fred-apex-ai @nnmm I have updated the CMake as suggested, see 2877403


Dynamic Aurelia dependencies

This is a meta-issue to track progress across all Aurelia repos.
In #17 it was decided that using PLATFORM.moduleName(), exported from aurelia-pal was the way to document dynamic dependencies consumed by the aurelia loader.
The function has been added with da92298.
To follow up on that, it must now be used in every aurelia module that has dynamic imports.
Here's a checklist of all modules to track progress, most probably don't need any change at all:

[x] animator-css
[x] animator-velocity
[x] binding
[x] bootstrapper
[x] cache
[x] dependency-injection
[x] dialog
[x] event-aggregator
[x] fetch-client
[x] framework
[x] history
[x] history-browser
[x] hot-module-reload
[ ] html-import-template-loader
[x] http-client
[x] i18n
[x] loader
[x] loader-default
[x] loader-nodejs
[x] loader-webpack
[x] logging
[x] logging-console
[x] metadata
[x] pal
[x] pal-browser
[x] pal-nodejs
[x] pal-worker
[x] protractor-plugin
[x] route-recognizer
[x] router
[ ] skeleton-plugin
[x] task-queue
[x] templating
[x] templating-binding
[x] templating-resources
[x] templating-router
[x] testing
[ ] ui-virtualization
[ ] ux
[x] validation
[ ] web-components

Question: was the work with aurelia.build.resources complete? I.e. if there is no such key in package.json, can I assume the library doesn't need to be modified?
For now I think it's better to keep the aurelia.build.resources in place, in order to not break existing tools. But once this is in place a deprecation is probably in order, with removal at a later stage.
CC: @niieani @EisenbergEffect

The main location is going to be the bootstrapper, I think.

Regarding your question, I think we took a shortcut in that we put most of the aurelia.build.resources in the bootstrapper-webpack package, instead of the respective packages. There aren't many dynamic requires throughout Aurelia, so this should be a pretty simple task.

I managed to get my demos running without using aurelia.build.resources. I did not modify aurelia modules yet, instead I used a plugin of mine to introduce the dependencies I wanted in my build, like so:
new ModuleDependenciesPlugin({
      "aurelia-bootstrapper": [
        // "aurelia-loader-webpack",    // detected by webpack itself
        "aurelia-pal-browser",
        "aurelia-framework",
      ],
      "aurelia-framework": [
        "aurelia-history-browser",
        "aurelia-logging-console",
        "aurelia-templating-binding",
        "aurelia-templating-resources",
        "aurelia-templating-router",
        "aurelia-event-aggregator",
      ],
      "aurelia-templating-resources": [
        "./compose",
        "./if",
        "./with",
        // [...] some more skipped for brievity
      ],
      "aurelia-templating-router": [
        "./router-view",
        "./route-href",
      ],
    })

Most of it will be trivial enough, e.g. templating-resources or templating-router.
But I think we need to have chat about bootstrapper and framework. The culprit is that those dynamic loads may not occur, depending on your configuration or your environment. And we most certainly don't want to embed those dependencies in the build if they are not used. For instance, I only listed one of the three PALs.

I don't think there's any use case for having more than one PAL bundled. In some cases you might want to have a separate bundle for pal-worker, but it would still only contain that one PAL.

@niieani Yes, that's what I'm saying. But the problem is that choosing which one is included must be done at build time and can't be declared inside aurelia-bootstrapper.
My thinking for aurelia-bootstrapper is:

the convoluted syntax used to import aurelia-loader-webpack is automatically picked up -> no change;
the PAL must be decided by the build tool and should not be wrapped with moduleName;
aurelia-framework is normal, should be wrapped with moduleName.

For aurelia-framework the problem is that all six modules are standard but optional configs. If I wrap them with moduleName everything will work but someone using basicConfiguration() instead of standardConfiguration will not want the router in the bundle.
Thoughts?

What is bootstrapper-webpack?
Despite its name I don't use it in my webpack builds...

@jods4 bootstrapper-webpack was the webpack-specific bootstrapper code, which we will be deprecating soon, because we managed to get together a universal aurelia-bootstrapper with build-system detection for all our use cases (hence the "convoluted syntax" - its the only one that does not trip up SystemJS, RequireJS and works with Webpack when needed by not adding extraneous dependencies).
Your thinking is correct.
Not sure how to solve the basicConfiguration() vs standardConfiguration(). I've been thinking about this too some time ago, but dropped the matter as most users use standardConfiguration anyway. But it would be good to have this configurable.

Not sure how to solve the basicConfiguration() vs standardConfiguration(). I've been thinking about this too some time ago, but dropped the matter as most users use standardConfiguration anyway. But it would be good to have this configurable.

From the discussion with @EisenbergEffect, I won't add moduleNames for basic configurations. Rather, the build tooling will propose equivalent default configs that you can pick from.
The default will include everything so that it "just works", and then people can remove pieces that they don't use if they want a minimal size.
A "custom" configuration happens in user code, which means moduleName.

Sounds good.

OK, I have 4 PR open referencing this request.
As you can see in the checklist this covers all "core" aurelia modules and is enough to be able to build an app that doesn't use optional modules without any extra manual dependencies.
(well, considering that the default extra dependencies are added by the build tool, as described above)

Reviewed all. Great comment work, looking good to me!

Thanks @niieani, can someone merge them? They're relatively trivial.
@EisenbergEffect I've looked at almost all modules (see the checklist). Would be nice to get help for the remaining ones.
I am pretty sure the following modules need changes:

dialog
validation
html-import-template-loader (it fetches webcomponentsjs/HTMLImport.min through the loader).

For the rest I don't know.

@EisenbergEffect I see that you are merging all my PR above and about to publish the new webpack tools.
It would be awsome if some Aurelia members could have a quick look and test with the items in this list that are not ticked yet.
Some might require no change at all, some might not work out of the box.

There's a problem with dialog (and possibly with other external plugins) that is a bit harder to solve. See this part of the code: https://github.com/aurelia/dialog/blob/22022bd8d25ae488d45566f509378a8813aeef7a/src/dialog-service.js#L111.
This is currently causing errors such as https://github.com/aurelia/dialog/issues/203.
In short, everything works as far as you provide a string path to the ViewModel and let Aurelia load it instead of an actual class. The problem is that the Origin is set by Aurelia whenever a module is being loaded by the loader-webpack, unless it was in Webpack cache before eachModule is executed. However if you import a class directly, Aurelia will have no knowledge of the file that it came from. For reference, the Origin is the reverse-map of exports to moduleIds, used to "guess" things like the path to the related View using a convention, so this problem is not specific to Dialog, but to all the times an Origin is expected but the underlying module has not yet been touched by the Loader.
This is also the reason my v2 Webpack solution was naming all the Webpack modules, as opposed to the @jods4 re-written solution which only names those specifically referenced by PLATFORM.moduleName. This is what used to happen in v1 and leads to problems like https://github.com/aurelia/dialog/issues/241.
One semi-solution that comes to my mind would be to hook Webpack's chunking mechanism to run the Origin setter right after loading a given chunk. This would only work if all the modules had actual names, since it actually concatenates moduleId + .html, hence the above problem of Cannot find module './76.html'. The big downside of this is that all the modules in the whole chunk would get evaluated immediately, as if they're all imported, just so we can set their Origin.
Another two solutions would be breaking changes, but overall reduce complexity in some ways:

instruct users to user PLATFORM.moduleName even for ViewModels imported with import {} syntax if they plan on using them this way, while at the same time add logic to PLATFORM.moduleName which runs the ensureOriginOnExports logic on all the referenced modules. Not great.
disallow passing imported ViewModel classes directly, ask users to always specify paths as strings so they go through the loader. Simplest, but breaking users and plugins.

Any better ideas?

@niieani So passing VM instance is also ok since it was already loaded?

@StrahilKazlachev if it was loaded by Aurelia, and not via an import, it is ok currently, and will be okay with v2 of Webpack stack.

@niieani I think we'll need to talk about this.
To find the view, using @useView should be a work-around, right?
But the whole Origin thing is surely going to be broken for all classes that are not loaded through aurelia-loader.

Yeah, explicitly decorating the ViewModel with @useView does work; not depending on the conventionally named views solves this. But currently the experience is sub-par, since it's that "sometimes" it works automatically, and sometimes it does not, and the user has no idea why.
Still, one of the nice things about Aurelia is that you don't need to use useView explicitly for every ViewModel.
I'm open to chatting about this on Gitter.
Thoughts about this one, @EisenbergEffect? This is a problem especially with code loaded on demand, since even if we consider re-running PAL.everyModule with each new file, we would also evaluate all the modules inside it, as it loads. Not sure how is this solved in RJS or SystemJS (do we even support loading modules on demand in the latter?), unless its not, and we just missed a bug?
This is also one of the blockers that keeps us from implementing logic that replaces moduleNames with random names for security, since it simply tries to append .html to the end.

The situation is even worse with SystemJS since there's no logic at all there.
Reference logic for RJS and SystemJS here.
Having just looked at the implementation of Origin, it seems that when origin is undefined, it is to be retrieved and set on demand, as the call occurs. Having said that, there might be a less-then-perfect way to do this with Webpack, since we know for sure a module should be in the Webpack cache once any imports have occured, we could just iterate over the whole cache and all of the keys of each element until we have found a matching function/class. The performance of this operation would be pretty bad though, however, if it's any consolation, this operation would only have to happen once (per each undefined Origin). With @jods4 approach of only naming modules which have been explicitly added via PLATFORM.moduleName, the user would still need to re-declare the imported file with a call to that function. If we renamed all modules as in my approach, the imports should work automatically.
Another thing is that our NodeJS loader will behave the same way as Webpack currently does, since as far as I know there is no way to find out the file path from an already loaded reference to a function/class, and since there is no bundling involved here, running eachModule equivalent to set Origin for all js files under the root directory is probably out of the question (startup performance would be abysmal). Perhaps we could use some hacky node APIs to list all cached modules, but these APIs are explicitly defined as unsupported/deprecated in the NodeJS documentation, so even if we solve the Webpack problem somehow, I'm not sure how we'd do this for Node, and we'll need to support it if we want server-side-loading, going forward.

@niieani
Trying to make some progress here:

I think you made protractor work with the new build, can I check it?
Now that I added support to preserve the name of modules that are vm of views by convention, is aurelia-dialog ok?

@jods4 protractor is good since the updates that made it execute scripts synchronously. Don't remember if we published that to NPM yet or not.
Haven't retested aurelia-dialog  yet.

Checking protractor. Doesn't matter if it's published yet, just want to make sure we have everything covered.

Just spent some time with user mike-suggit who got his tests with Karma + Jasmine + aurelia-testing + Protractor working.
Was a bit rough because he had quite a few bad configs from existing build. But once everything was referencing the correct packages, the config properly updated, moduleName added where required, it all worked.
So although I did not look at it, I'm going to check that aurelia-testing checkbox.

@jods4 - can you check validation off please- I've made the platform changes but I don't have privileges to update issues in this repo. Thanks!

Thanks, I checked validation off the list.

@jods4 could you share how you and mike-suggit configured karma to work with aurelia-testing?

I don't remember, better ask @mike-suggitt directly!

It doesn't load aurelia-ux due to the ${$design} interpolation inside .css files.

@khuongduybui I don't think this thread is the right place, you should file an issue in aurelia-ux repo.
This meta-issue is to keep track of where PLATFORM.moduleName were added in core aurelia libraries.
What you have sounds like a loader issue and should be tracked separately.

Ah okay, thanks. I'll file the issue in ux repo instead.

@jods4 the html-import-template-loader module doesn't seem to need it. I think it can be ticked.

@bigopon Thanks for that! I checked it.

Et alors?
(-:
something about sprinkling moduleName all over the place
just a ping. seems almost baked.

Closing since I think all libs that require this have been instrumented.


⚠ RPC server - Optimism has degraded performance

In 18aca64, RPC server - Optimism ($OPTIMISM_RPC_URL) experienced degraded performance:

HTTP code: 429
Response time: 303 ms

Resolved: RPC server - Optimism performance has improved in 1776e5e after 6 minutes.


variadic rest in the tuple with min & max capability

I have a scenario, where I need to define the schema in the form of a tuple with certain restrictions in length of min, and max on the variadic rest.
const TypeOneSchema = z.object({
  name: z.string(),
  value: z.number(),
});

const TypeTwoSchema = z.object({
  title: z.string(),
  amount: z.number(),
});

const TupleSchema = z.tuple([
  TypeOneSchema, **// First element: TypeOneSchema**
  z.array(TypeTwoSchema).min(1).max(3) **// Rest elements: 1 to 3 TypeTwoSchema**
]);
Note: The above syntax of the tuple isn't supported, adding it here for a better understanding of the use case.

Say something like this:
For example: the scenario is that a tuple can be a minimum of 2 items & at max, it can have 4 items in the array.
// Example usage
Valid
const validTuple1 = [
{ name: 'John', value: 20 },
{ title: 'Title 1', amount: 100 }
];
const validTuple2 = [
{ name: 'John', value: 20 },
{ title: 'Title 1', amount: 100 },
{ title: 'Title 2', amount: 150 }
];
Invalid
const invalidTuple1 = [
{name: 'John', value: 20 }
];
const invalidTuple2 = [
{ name: 'John', value: 20 },
{ title: 'Title 1', amount: 100 },
{ title: 'Title 2', amount: 150 },
{ title: 'Title 3', amount: 200 },
];
Please suggest how to achieve this using tuple, because it also gives the benefit of auto-suggestion in the code-editor.

I'll provide an idea: first, define a tuple(typeA, ...typeB). This way, you can handle the first type and the remaining types. As for the length restriction, you can use an array, but the array only deals with the count, so the values handled by min() and max() need to be increased by one.
const TupleSchema = z
  .tuple([TypeOneSchema])
  .rest(TypeTwoSchema)
  .and(z.array(z.any()).min(2).max(4)); // not 1 and 3


remove templating from strings when building database

Cleaned up effect text strings.
Example:
[]{move:dragon-tail} will effect the opponents [HP]{mechanic:hp}.
Becomes:
dragon tail will effect the opponents HP.

Nice, will I have to run the db build script again after this @zaneadix ?

Yeah at the very least the build methods for abilities, items and moves but it'd probably just safer to run build_all()

@zaneadix Okay, I can merge this and then run it again this week :)


Where is SteamVR_LaserPointer.cs

I can't find SteamVR_LaserPointer.cs in SteamVR v2.0.1

This script was not upgraded to the new version and so it was removed. I'll readd it in the next version though since people seem to want this functionality.

@zite It makes sense to me that anything that is 'default' in the SteamVR Home scene would also be default with the SteamVR Unity Plugin.

I've readded this script in the beta version of the plugin. If you'd like you can test it before proper release here: https://github.com/ValveSoftware/steamvr_unity_plugin/releases/tag/2.1b2

@zite 2.0.1 tag and 2.1b2 tag point to the same commit.
Do not you publish the 2.1 source yet?
https://github.com/ValveSoftware/steamvr_unity_plugin/tree/2.0.1
https://github.com/ValveSoftware/steamvr_unity_plugin/tree/2.1b2

No, haven't published the source yet. All of it is in the linked unitypackage though


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/opstest2/docs-archive-test-source.git

Docs Build status updates of commit f9bb7a3:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Fix WebApp.requestContact typing

Wrong typing for WebApp.requestContact method. Provided type requires callback of type
(access: boolean) => unknown
meanwhile
(access: boolean, result: ???) => unknown
works just fine with result of method call in result field.

Hello. Thanks for MR. Please update your branch to fix conflicts

@wishyoudie Please update your branch to fix conflicts


Improvements & Feature ..

proposing a re factory which includes some improvements
and pointing out a BUG which may be a feature but the have to be handled properly ...

The commit is huge. For people who are lazy to read the whole changes, The PR adds use-cases for
+  assert.strictEqual(leftPad('feature'  ,  8, 'bug or '), 'bug or feature' );
+  assert.strictEqual(leftPad('feature'  , 10, 'buggy ') , 'buggy buggy buggy feature' );

I don't know if anyone is using it.

well , actually these two use case were supposed to highlight a BUG ..
which I didn't fix either ... because maybe can be converted in feature ..
the fact is that as it is right now is just a BUG
because anyone going to what a function that pads with another expect it to count differently
say ...
leftPad('feature'  ,  8, 'bug or ')  ..should be ..
leftPad('feature'  ,  1, 'bug or ') instead ...

If you just look at the whole file instead of to diff changes isn't so huge ...
while yes it changes a lot from original ...

[whole file] (https://github.com/cloned2k16/left-pad/edit/master/index.js?pr=/stevemao/left-pad/pull/28)


Add reference to node.js library to README.md

Proposal to add a Libraries section to the readme with a link to a javascript npm module.

This makes sense to me. I didn't actually see the Swift Module before I added my Swift example to this doc. Should that be the model we take for code examples in general, separate modules for code examples? That's the way Stripe breaks things up today, seems like a pretty good model to follow: https://github.com/stripe


Typechecker gets confused and cannot recover

Often only the first error from the typechecker is relevant as the typechecker gets confused and prints errors that only arise from the first one.
I don't know how to handle that best, cause we probably would need to introduce a new intermediate type called Undetermined or Confused so that the parser ignores all expressions where one type is undertermined.

Yes, we would have to catch errors in the assignment and would have to store r with an error type or something and every time we use r we would have to check if it is of the error type and throw an error type exception which does not get printed to the user. I think that's the way how swift handles this (you can try it your self if you assign something wrong to an variable and then inspect its type in the ide. It will say <>.
I think it wouldn't be too hard to implement, since it does only require changes in the assignment of identifier and in the typecker visit of an identifier.

Yeah sounds good.

The typechecker now has internal errortype which it assigns to experssions it cannot reason about.
If such an experssion is assigned to an variable it will also get an error type. At this point further usage of this variable will get an free pass and will not get typechecked again.
Therefore only the original sinn will be highlighted and not errors that also originate from the original mistake.
This makes the reported errors of much higher quality.


[test] Add test case for convenience init inheritance

This test case tripped up an attempt I made to simplify addImplicitInheritedConstructorsToClass for clang decls. Add it to the test suite to make sure we don't regress it.

@swift-ci please smoke test

That is a wacky test case and I would hope it never comes up in real life. Does it, though?

@jrose-apple Yeah, I managed to reduce this down after hitting a similar case in CloudKit.
CKSubscription has a (deprecated) designated init initWithRecordType:predicate:subscriptionID:options:, and its subclass CKQuerySubscription also defines it as a designed init, but SwiftPrivate.

@swift-ci please smoke test OS X platform

:cry: Okay, thanks for identifying it, then.


Add warning message for username/password

Hello,
The username and group defined in default authenticator are not exported by WDT.
This is not a real blocking point but will it be possible to add warning message in generated model file

The username/password are not exported so please check if they exist and add them in model file if needed

or to make this warning message customizable from command line property in case others customers don't want to see it ?
Regards.

Thanks a lot for this

@gregoan Having thought a little about this, the concern that I have is that this warning would always appear for every single run of the Discover Domain tool.  It's not like there is some domain characteristic we could use to determine whether to do this warning since every domain has at least one authentication provider.
Given that a warning will cause the discoverDomain shell script to exit with an exit code of 1 (denoting that there was one or more warning).  We could make this a notification level log message and this will still cause it to show up in the tool summary without forcing a non-zero exit code.
If the message always appears, it's not clear that it is better to create the message or just document the limitation (assuming it isn't already).  In my experience, if something happens all the time, people tend to ignore it.
Thoughts?

Hi,
Today, I execute :
${HOME}/weblogic-deploy/bin/discoverDomain.sh -oracle_home ${oracleHome} -model_file ./DomainModel.yaml -variable_file ./DomainVariable.properties -admin_url ${oracleServerUrl} -remote -skip_archive
Tomorrow, I would like to know if it's possible to execute something like this (-print-warning is added - it's an example):
${HOME}/weblogic-deploy/bin/discoverDomain.sh -oracle_home ${oracleHome} -model_file ./DomainModel.yaml -variable_file ./DomainVariable.properties -admin_url ${oracleServerUrl} -remote -skip_archive -print-warning

The default mode for print-warning is false
If it's not set, we are using the current behavior.
If it is set then a header message could be added inside generated file (or an INFO message on STODUT) informing the user none username/group will be generated meaning an additional and manual check must be executed.
The warning/info message is not interfering with shell script execution (return code)

What do you think ?

@gregoan Sorry, I must not have communicated my concern clearly.  Let me try again.
Since the message will appear on every invocation (when you enable it), I am concerned that we are just adding information that the user will learn to ignore (since it happens every time).  Why do we need to log a message about this with every Discover Domain execution rather than just documenting the limitation; for example, on https://oracle.github.io/weblogic-deploy-tooling/userguide/limitations/limitations/?

@robertpatrick I see and you are totally right :)
I knew the limitation because I read the doc but it's not the case of everybody :)
I agree with your proposal ; let's try for it

@gregoan How is this?
https://oracle.github.io/weblogic-deploy-tooling/userguide/limitations/limitations/#discover-domain-tool-does-not-discover-users-or-groups

It's good from my point of view


"screen resolution" -> "viewport size"

What the page is calling screen resolution here is actually the size of the viewport.
Readers might incorrectly think that this is referring to the resolution of the screen (eg: 1920x1080) or that this has something to do with window.devicePixelRatio.
Viewport size is the dimensions of the browser tab without the chrome. (address bar, dev tools, etc)
Screen resolution is the resolution of the display. (note: this might not map 1:1 to the number of pixels on the display)
EDIT: I'm not sure if terminology elsewhere needs to be updated - I'm new to Grommet.

Hi @matthewoates can you try updating you branch with the master branch? Looks like some of the checks are failing on this PR

@jcfilben updated!


.ForEach() enumeration fails on PowerShell 4 with single instances

Introduction
Starting PowerShell 4, you can efficiently enumerate collections by using the automatic .ForEach() method, which dbachecks uses.
HOWEVER

In PS4, this only exists for collections, not single items
Starting PS5, it automatically turns a single object into a collection, causing this to work

Consequence
dbachecks is currently not compatible with PS4, since it uses the notation on single items (notably isntances) also.
Proof of issue:
Run this on both PS4 & PS5:
$item = 42
$item.ForEach({ $_ })

Workaround / Solution
Run this on both PS4 & PS5:
$item = 42
@($item).ForEach({ $_ })

$item2 = 23,42
@($item2).ForEach({ $_ })

A little research using my favorite PSModuleDevelopment command (find):
PS C:\> (find -Path F:\Code\Github\dbachecks -Pattern "\.ForEach" | measure).Count
144
PS C:\> (find -Path F:\Code\Github\dbachecks -Pattern " \(Get-SqlInstance\)\.ForEach" | measure).Count
59
PS C:\> (find -Path F:\Code\Github\dbachecks -Pattern " \(Get-ComputerName\)\.ForEach" | measure).Count
7

Meaning we have:

144 Instances of use
66 Instances that can be bulk updated
78 Instances that need to be verified and potentially fixed manually

We addressed this by always returning an array - have you encountered issues that you can see?

Haven't looked at the life version - my pre-release repo was enough to dig to the root cause based on the error message users were reporting.
Basically, at one point it's having a hiccup on an DbaInstance object

I was about to ask the same.

C:\Program Files\WindowsPowerShell\Modules\dbachecks\checks\Agent.Tests.ps1: line 63

This was the fix (that Chrissy did) when I first spot it https://github.com/sqlcollaborative/dbachecks/blob/master/internal/functions/Get-SqlInstance.ps1#L11

Yes - a case of going through and having a look - I too thought we had caught all of these


Redshift data sharing - Cluster encryption guardrails and information

Feature or Bugfix

Feature
TO BE MERGED AFTER https://github.com/data-dot-all/dataall/pull/1424

Detail
This PR is part of the design explained in detail in #955

Add check that verifies the Redshift cluster imported is encrypted before creating a redshift connection
Store the Redshift encryption type in the redshift-connection table. This is needed because Redshift datashares only work between namespaces that use the same encryption type
In the migration script and in the db model of the redshift-connection table we add the columns encryptionType and connectionType. ConnectionType is not needed in this particular PR, but it will be needed in the next PR. It is added to simplify the migration scripts

Relates

#955 - Sharing part

Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.

Does this PR introduce or modify any input fields or queries - this includes
fetching data from storage outside the application (e.g. a database, an S3 bucket)?

Is the input sanitized?
What precautions are you taking before deserializing the data you consume?
Is injection prevented by parametrizing queries?
Have you ensured no eval or similar functions are used?

Does this PR introduce any functionality or component that requires authorization?

How have you ensured it respects the existing AuthN/AuthZ mechanisms?
Are you logging failed auth attempts?

Are you using or adding any cryptographic features?

Do you use a standard proven implementations?
Are the used keys controlled by the customer? Where are they stored?

Are you introducing any new policies/roles/users?

Have you used the least-privilege principle? How?

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

Testing:

(no testing needed for this scenario) - Serverless clusters are encrypted by default, so it is not possible for them to not be encrypted. At a minimum they are encrypted with AWS-Managed keys
[ ]  Provisioned clusters not encrypted cannot be onboarded with data.all Redshift connections - creation of a connection throws an error
[X] Serverless clusters encrypted with customer managed keys are correctly classified as RedshiftEncryptionType.CUSTOMER_MANAGED_KMS_KEY
[ ] Serverless clusters encrypted with AWS managed keys are correctly classified as RedshiftEncryptionType.AWS_OWNED_KMS_KEY
[ ] Provisioned clusters encrypted with customer managed keys are correctly classified as RedshiftEncryptionType.CUSTOMER_MANAGED_KMS_KEY
[ ] Provisioned clusters encrypted with AWS managed keys are correctly classified as RedshiftEncryptionType.AWS_OWNED_KMS_KEY
[ ] Provisioned clusters encrypted with HSM are correctly classified as RedshiftEncryptionType.HSM


Bullet 12 is still incorrect.

shows using https:// but the PKI server setup did not include anything about a certificate and port 443 is not assigned.  I think this should be just http://

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

ID: 2929b93e-fe36-d3e4-ee28-2a4aaf977da6
Version Independent ID: 689f7606-45b0-3444-65dc-85c2ba217e9b
Content: Configure the CDP and AIA Extensions on CA1
Content Source: WindowsServerDocs/networking/core-network-guide/cncg/server-certs/Configure-the-CDP-and-AIA-Extensions-on-CA1.md
Product: windows-server-threshold
GitHub Login: @shortpatti
Microsoft Alias: pashort

#please-close


git: 2.17.1 -> 2.18.0, round 2

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@layus is the hero behind this! See: https://github.com/NixOS/nixpkgs/pull/42376#issuecomment-399989290
Anyway I haven't tested this with the breakage it caused on various PR's, will do so shortly.

LGTM -- tried nix-build -A lollypop.src --check (might need to run without --check first time) with/without the fix from @layus, seems to work with and fails without. \o/

LGTM but I'd prefer someone else to merge / confirm since I/we got it "wrong" the first time around :).

I tested it too and it seems to work :hand:

lgtm

the chicken says: staging, maybe?

Sounds like a plan, no rush here AFAIK anyway.

@GrahamcOfBorg build git gitFull

What was causing the failures with the last major bump causing it to be reverted 3 times? Is it a reasonable risk here as well? Is it perhaps an idea to test it by building a smaller job?

Failure on x86_64-darwin (full log)
Attempted: git, gitFull
Partial log (click to expand)
cannot build derivation '/nix/store/zhdncxbqspnak1i3kn6sfl6bzn9w0f91-opensp-1.5.2.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/kck48jwbwip7b4b9xsz9g6ixj029ymb4-serf-1.3.9.drv': 10 dependencies couldn't be built
cannot build derivation '/nix/store/92cll8baa995mfympkqkzizxq65xr38p-docbook2X-0.8.8.drv': 16 dependencies couldn't be built
cannot build derivation '/nix/store/g8axfs3rwdlj2f1nhsdcqs2knfgk8464-libXrender-0.9.10.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/17p7x39ql012knmhf16ndkqbbby4ha7b-subversion-1.9.7.drv': 12 dependencies couldn't be built
cannot build derivation '/nix/store/ilc1khy1pld25c3sv5n5ck4flrfyhn3i-git-2.18.0.drv': 29 dependencies couldn't be built
cannot build derivation '/nix/store/85z1jahrsd2r6w5fj10hv9kpv5j1h3s9-libXft-2.3.2.drv': 8 dependencies couldn't be built
cannot build derivation '/nix/store/4bsihm57ns8rga954dz88ygcz52w1mr0-tk-8.6.6.drv': 6 dependencies couldn't be built
cannot build derivation '/nix/store/8d53fx7mihp3mzc5hhg4vlni1wrdipvq-git-2.18.0.drv': 32 dependencies couldn't be built
error: build of '/nix/store/8d53fx7mihp3mzc5hhg4vlni1wrdipvq-git-2.18.0.drv', '/nix/store/ilc1khy1pld25c3sv5n5ck4flrfyhn3i-git-2.18.0.drv' failed

Success on x86_64-linux (full log)
Attempted: git, gitFull
Partial log (click to expand)
Files=816, Tests=19219, 216 wallclock secs ( 5.06 usr  1.05 sys + 287.42 cusr 275.58 csys = 569.11 CPU)
Result: PASS
make clean-except-prove-cache
make[2]: Entering directory '/build/git-2.18.0/t'
rm -f -r 'trash directory'.* 'test-results'
rm -f -r valgrind/bin
make[2]: Leaving directory '/build/git-2.18.0/t'
make[1]: Leaving directory '/build/git-2.18.0/t'
/nix/store/7zqshcw39jn3cv8dk5a08rjgl4hl4zxg-git-2.18.0
/nix/store/9c88lp6mwvakckpsf1alh5zp5qs8n7z1-git-2.18.0

Success on aarch64-linux (full log)
Attempted: git, gitFull
Partial log (click to expand)
Files=816, Tests=19219, 285 wallclock secs (81.41 usr 17.18 sys + 2735.20 cusr 6858.77 csys = 9692.56 CPU)
Result: PASS
make clean-except-prove-cache
make[2]: Entering directory '/build/git-2.18.0/t'
rm -f -r 'trash directory'.* 'test-results'
rm -f -r valgrind/bin
make[2]: Leaving directory '/build/git-2.18.0/t'
make[1]: Leaving directory '/build/git-2.18.0/t'
/nix/store/lp7rciszywn3jmwmmfprd0g5rjx1bhyi-git-2.18.0
/nix/store/7bs2p7dlq9vdnydsmwwy83hbgbrnj1r5-git-2.18.0


Update bundled python to 3.11.3

For CVE-2023-24329.
Also, run dockerfile commands quietly to reduce noisy console output.

See https://github.com/signalfx/signalfx-agent/pull/2797 for expanded test coverage with these changes.


added align-items: center and increased line-height on the span

Summary
vertically aligns text in the channel header trigger correctly. It is not cutting the lower parts of text when they are reaching down (as with the letters: g, j, p, q and y)
Ticket Link
Fixes MM-35446
Screenshots

before
after

Release Note
NONE

Nice one! Thanks @michelengelen! :tada:


CI: Cassandra Tests failed because image was not able to be found

Build link: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/7891/testReport/junit/(root)/runtime/RuntimeCassandra_Tests_policy_allowing_all_actions/
Output:
/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/ginkgo-ext/scopes.go:340
failed to create container cass-server
Expected command: sudo docker run -d --name cass-server --net cilium-net -l id.cass-server cassandra
To succeed, but it failed:
Exitcode: 125
Stdout:

Stderr:
 	 Unable to find image 'cassandra:latest' locally
	 docker: Error response from daemon: Get https://registry-1.docker.io/v2/library/cassandra/manifests/latest: Get https://auth.docker.io/token?scope=repository%3Alibrary%2Fcassandra%3Apull&service=registry.docker.io: dial tcp <IP_ADDRESS>:443: connect: network is unreachable.
	 See 'docker run --help'.

/home/jenkins/workspace/Cilium-PR-Ginkgo-Tests-Validated/src/github.com/cilium/cilium/test/runtime/cassandra.go:52

We should not use a 'latest' image in our tests, as this is fragile.

@ianvernon AFAICT this was hit once a few months ago and not since, so either we've fixed it or it's no longer a problem. Closing, feel free to reopen if you disagree.


Impossible to import module (even if path is set correctly) and "run in terminal" not present

In my file main.py I am importing a module utilities.py located in another folder.
The directory structure is

Project_folder
        |-- execution_folder
        |          |-- main.py
        |-- util_folder
                   |-- utilities.py

I have
import sys
sys.path.insert(0, '../../project_folder/util_folder/')
import utilities as util

but the last import statement is underlined in red and raises "unable to import utilities" error.
This resembles the question posted here: https://github.com/Microsoft/vscode/issues/10391
Note that I have no problem running the script as-is in Spyder (!)
Additionally, when I open my main.py script, the "run Python file in terminal" option is missing from the context menu. If I navigate and open another script, then return back to main.py then the option reappears. This is exactly the same as the issue here https://github.com/Microsoft/vscode-python/issues/3694

"run Python file in terminal" is a command from the vscode-python extension, please go through the extension team first as they might have an idea on what the underlying cause is if it is indeed a vscode issue.

"run Python file in terminal" is a command from the vscode-python extension, please go through the extension team first as they might have an idea on what the underlying cause is if it is indeed a vscode issue.

Sure, but what about the other - and way more important - problem?

Wow, you closed it... amazing. Love this cherry-picking style to "fix" issues.

@filipporemonato all python support is provided by vscode-python, you should report it there as I said https://github.com/Microsoft/vscode-python. https://github.com/Microsoft/vscode/issues/10391 is about typescript support which is built in so this is the right repo for that.

Oh, I apologise, I understood wrong. My wrong. I will report the Python issue there then, and at least now you know about the typescript thing.
From: Daniel Imms<EMAIL_ADDRESS>Sent: onsdag 13. februar 2019 19:07
To: Microsoft/vscode<EMAIL_ADDRESS>Cc: Filippo Remonato<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Microsoft/vscode] Impossible to import module (even if path is set correctly) and "run in terminal" not present (#68614)
@filipporemonatohttps://github.com/filipporemonato all python support is provided by vscode-python, you should report it there as I said https://github.com/Microsoft/vscode-python. #10391https://github.com/Microsoft/vscode/issues/10391 is about typescript support which is built in so this is the right repo for that.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/vscode/issues/68614#issuecomment-463303720, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AF8p5vPq-BNf4DUwzqrT_yfuBvDm5Szuks5vNFQsgaJpZM4a5OQy.


Add TypeScript types

Adds TS type definitions
Cleans up tests for<EMAIL_ADDRESS>Upgrades dev dependencies in yarn.lock

Fixes #15

Coverage remained the same at 100.0% when pulling 385c1f5580fe77743614984b18fd41adb5043f20 on upgrade-deps-types into b4f8cc9487fdb1d1991e3a5d9bf4a536eb76213f on master.


Add Button to Hide/Show Chat Window

Currently, there is no functionality for participants to hide or show the chat window in the chat application. This can be problematic if participants need to view other elements of the application or want to reduce distractions during a call. Therefore, we need to add a button to hide and show the chat window.
Acceptance Criteria:
A button should be added to the chat window.
Clicking the button should hide or show the chat window.

Wait for @yulianag-qualabs to implement this ticket


[BUG] It doesn't work on MacOS sonoma m3 pro max.

The binary that you provided does not work on MacOS sonoma m3 pro max.
url: https://github.com/Anjok07/ultimatevocalremovergui/releases/download/v5.6/Ultimate_Vocal_Remover_v5_6_MacOS_arm64.dmg
Can you check out this problem?

For me, grant app permission can solve this problem
SOC：M3MAX 40GPU
sudo "/Applications/Ultimate Vocal Remover.app/Contents/MacOS/UVR"

@baka-gourd Hi, thank you for sharing the solutions. They work, but it's a bit troublesome to do this every time I open the app. Is there a permanent solution?

@baka-gourd Hi, thank you for sharing the solutions. They work, but it's a bit troublesome to do this every time I open the app. Is there a permanent solution?

For me, I wrote a direct sh to start it, not looking for any other solution at the moment


test: misc ut

What this PR does:
Which issue(s) this PR fixes:

Fixes #784
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 40.20%. Comparing base (a27c47b) to head (b05d470).

Files
Patch %
Lines

pkg/config/misc.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #824      +/-   ##
==========================================
- Coverage   40.20%   40.20%   -0.01%
==========================================
  Files         265      265
  Lines       26708    26708
==========================================
- Hits        10738    10737       -1
- Misses      14823    14825       +2
+ Partials     1147     1146       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Clicking date in simple bar open different calendaring app

I happen to use Fantastical, is there a way to switch what is opened when clicking on the date in the simple-bar?

This is hardcoded for now, I'll add a field in settings in which any other app could be set.

This is hardcoded for now, I'll add a field in settings in which any other app could be set.

Cool thank you.

Cool thank you.

Okay, is it full path?
Just putting Fantastical doesn't seem to work.
Would it be
I ended up having to do
/Applications/Fantastical\ 2.app/Contents/MacOS/Fantastical

Okay, is it full path?
Just putting Fantastical doesn't seem to work.
Would it be
I ended up having to do
/Applications/Fantastical\ 2.app/Contents/MacOS/Fantastical

simple-bar was simply doing open -a ${appName} when user clicks on the date widget.
I added some double quotes around ${appName}:
const openCalendarApp = (calendarApp) => {
  const appName = calendarApp ? calendarApp : 'Calendar'
  run(`open -a "${appName}"`)
}

This may solve some issues where the app name is composed of several words.
Can you try running open -a "Fantastical 2" in your terminal?

simple-bar was simply doing open -a ${appName} when user clicks on the date widget.
I added some double quotes around ${appName}:
const openCalendarApp = (calendarApp) => {
  const appName = calendarApp ? calendarApp : 'Calendar'
  run(`open -a "${appName}"`)
}

This may solve some issues where the app name is composed of several words.
Can you try running open -a "Fantastical 2" in your terminal?

yep, that 'open -a' works
so just replace it with
Fantastical 2 no quotes in the settings?

yep, that 'open -a' works
so just replace it with
Fantastical 2 no quotes in the settings?

yep, that works! thanks!

yep, that works! thanks!

Yes, this should normally do the trick! :)

Yes, this should normally do the trick! :)