Exception when using tension while data[ ] is empty

Expected Behavior
for no exception to be thrown at all.
Current Behavior
can't access property "x", point is undefined gets thrown if any of the dataset's data arrays is empty.
Possible Solution
no idea really, but for me setting the data array to [null] made the exception vanish without any noticeable side effects.
Steps to Reproduce
my apologies, but i won't be submitting an example via codepen since my brain has melted enough for now.
alternative step(s) to reproduce:

visit https://www.chartjs.org/samples/master/charts/line/interpolation-modes.html
run myLine.config.data.datasets[0].data = [] (aka: set the data of any dataset with a tension of like 0.4 to empty)
cause it to redraw (e.g. by resizing the browser window slightly)
notice the can't access property "x", t is undefined exception

Context

for me the datasets get initialized with an empty data[] array while a background ajax request fetches the actual data and updates this afterwards, typing this i realize i might as well just not initialize that dataset until there is actual data present, but i spent like an hour trying to debug this so i'm just sharing my find here hoping it will help anyone else with their project ❤
Environment

Chart.js version: ^3.0.0-rc
Browser name and version: firefox developer edition 87.0b9
Link to your project: compiled through laravel mix

the bezier control points update procedure is to blame, I suppose.

yes, its named in the stack trace ^


Fix Line breaks are removed after special copy/paste scenario

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/15925
PROPOSAL: https://github.com/Expensify/App/issues/15925#issuecomment-1468436105
Tests

Use the following text for the test:

Case 1 normal text: (message with few line breaks)
a
b
c

Case 2 markdown text: (markdown message with few line breaks)

a
b
c

Steps:

On web or desktop app, go to any Report page
Copy the "normal text" to chat composer and send the message
Highlight the message we just sent then right click on the highlighted text
Click on the "Copy to clipboard" icon at the Context menu
Paste result normally to the chat composer
Verify that it still contains line breaks same as "normal text" above in the chat composer
Paste result without formatting into the chat composer user ctrl+shift+v
Verify that it still contains line breaks same as "normal text" above in the chat composer
Repeat 2->9 for case "markdown text"

[x] Verify that no errors appear in the JS console

Offline tests

n/a this feature doesn't change or is impacted by offline mode
QA Steps

Same as above

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

https://user-images.githubusercontent.com/9639873/227381838-7d1eca93-b387-40e4-ba71-c2c5b082d07c.mp4

Mobile Web - Chrome

Mobile Web - Safari

Desktop

https://user-images.githubusercontent.com/9639873/227381854-0ad57dab-ddd3-4aa0-aa9b-edee858f6d60.mp4

iOS

Android

@grgia @sobitneupane One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@hoangzinh Can you please update the Tests with all the steps from https://github.com/Expensify/App/pull/13739.

@sobitneupane sure, I just updated the test steps. Could you review again? If we're okay with it, I will start to screenshot/record again on those steps

@hoangzinh Yes please do update recordings.

🚀 Deployed to staging by https://github.com/grgia in version: 1.2.91-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.2.91-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Feature Request: Support adding scenes to the custom folder

I see that Moulinette creates a scenes/custom folder but I could not find any information on what I would have to add to it to have it import custom scenes. This Discord conversation suggests that Moulinette does not (yet) support that, so I thought I would put in an official request for that use case.
This would be particularly useful to allow me (and others) to offload a lot of large scenes to S3. I am aware that Moulinette will import images as a scene, but that of course is not the same as importing a scene with walls and other data. Thanks!

Moulinette (the module) won't support this because scenes have dependencies (tiles, sounds, notes, etc.) that cannot be easily be handled by the module itself
Moulinette Cloud support it for creators but not yet for users (private storage). I'm planning to work on this in September/October.
Other modules like Scene Packer could help, too. I'm planning to support it on Moulinette Cloud.


Implement intent filter for hacker news links

So that when the user click a HN link, it will be opened directly in Pumpkin Reader
http://developer.android.com/guide/components/intents-filters.html

Hahaha, I've been meaning to add this to the backlog.
Roy has asked me personally about this feature.
He usually reads from Feedly. When he needs to read the comments, he has to open it in browser. Would be great if it were to open on Pumpkin Reader.


 Get npm start changed so when we run that script it runs the express server instead of a webpack server.

[x] npm install express
[x] Add a basic express app
[x] Make it so that the server serves the dist folder to the browser
[x] Make it so that npm start will run the server
[x] Make a separate script for running webpack (This was actually already made: npm run build-dev)

Trello Ticket

 Setting Up Express Server

I also installed nodemon for this change.

Thanks Mitch!


README

README
User Story
AS A marketing agency,
I WANT a codebase that follows accessibility standards,
SO THAT our own site is optimized for search engines.
Acceptance Criteria
GIVEN the bootcamp-week-1-challenge project repository,
WHEN I view the README for the repository,
THEN I can see a professional README that provides information on the project and how to use it.

I updated the README file adding information about the project, how the solution was implemented and instructions on how to use it.


Deployment should optionally configure sentry

If you specify an API key or something like that.

Relates to #168


Day of week is invalid

Hi,
I used $.format.date(new Date('2014-10-25'), "E") it display "Fri" from US and "Sat" from Sri Lanka
Actual day is "Sat", why this show "Fri" for US ?
/Premila

Perhaps it is reading the browser's timezone by default and formatting the day according to that?
Sri Lanka is ahead of the US, so it makes sense that it has passed midnight in Sri Lanka at the time that date was generated.

hi @premilafernando
$.format.date converts what the browser returns:
var shortDaysInWeek = ['Sun', 'Mon', 'Tue', 'Wed', 'Thu', 'Fri', 'Sat'];
console.log(shortDaysInWeek[new Date('2014-10-25').getDay()]);
// =>  Fri
// More https://github.com/phstc/jquery-dateFormat/blob/master/src/dateFormat.js#L5

@ako977 Perhaps it is reading the browser's timezone by default and formatting the day according to that?
Sri Lanka is ahead of the US, so it makes sense that it has passed midnight in Sri Lanka at the time that date was generated.

:+1:


[WIP] testing Device Attestation

This is a bit exploratory, probably not something we want to support.  But maybe something to allow supporting outside OkHttp by extending okhttp-tls classes?

Will revisit after we can land the support @swankjesse added cleanly


Stillmaintained.com icon

Funnily enough, stillmaintaned.com is not maintained anymore, we should probably remove the project status icon from Readme.md.

Thank you.


ES6 spread operator

I just installed mern-starter without any modifications.
When I run npm start I get an error:
SyntaxError: mern-starter/client/modules/Intl/IntlReducer.js: Unexpected token (9:2)
   7 |   locale: initLocale,
   8 |   enabledLanguages,
>  9 |   ...(localizationData[initLocale] || {}),
     |   ^
  10 | };
  11 |
  12 | const IntlReducer = (state = initialState, action) => {
    at Parser.pp.raise (mern-starter/node_modules/babylon/lib/parser/location.js:22:13)
    at Parser.pp.unexpected (mern-starter/node_modules/babylon/lib/parser/util.js:89:8)
    at Parser.pp.parseIdentifier (mern-starter[nodemon] app crashed - waiting for file changes before starting...

It seems like it do not like es6 spread operator. Strange, isn't it?
Where I got wrong?

Ok, nvm. I did something wrong when upgrading.


feat: use ReadOnlySpan over strings for string-accepting methods

This PR aims to implement #31 by swapping out string parameters for the listed string-accepting methods with ReadOnlySpan<char>.
Members identified across both constant & non-constant generators:

FromName(string)
TryFromName(string, T)
IsNameDefined(string)

I should note that the implementation of the non-constant TryFromName method as it was using Dictionary's TryGetValue which does not accept a ReadOnlySpan<char>. Instead, I've used a foreach loop (for now, see TODO) to check each element in the key list. Not ideal, but it works for now and removes a secondary string allocation.
TODO:

[ ] Benchmark the foreach loop (I'd like to potentially use Memorymarshall and Unsafe but KeyCollection is weird)
[ ] Benchmark this change against mainline (after above item)

Since there's no implicit cast from string to ReadOnlySpan<char> in .NET Framework 4.6.x (and potentially 4.7.x), I'm considering adding a pre-processor conditional just for those TFMs to also generate the original string-based methods, then pass on the .ToSpan()ed string to the ReadOnlySpan<char> methods.
I'll revisit this later once I do some Windows testing with .NET Framework support.

Still haven't gotten around to benchmarking this. It's on my list for this week 👍

Benchmarks done and didn't yield anything special. The loop is barely noticeable and I don't believe it's worth investing time in speeding it up as you'd need to allocate another array/List just for the faster key searches.

Thanks again for your contribution! Apologies for the slow response.


op2 doesn't support &mut [u8] or &mut [u32]

Required for FFI and op_fs_stat_sync among others.

Fixed by #197


chore: bump kclvm version to 0.4.3

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: bump kclvm version to 0.4.3

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.005%) to 59.897%

Files with Coverage Reduction
New Missed Lines
%

kclvm/compiler/src/codegen/llvm/context.rs
2
74.15%

Totals

Change from base Build<PHONE_NUMBER>:
-0.005%

Covered Lines:
22974

Relevant Lines:
38356

💛  - Coveralls


Support S3 GetObjectLockConfiguration command

We return NOT_FOUND for S3 GetObjectLockConfiguration command.

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

Project coverage is 52.55%. Comparing base (087dfea) to head (26cff6a).
Report is 2 commits behind head on master.

Files
Patch %
Lines

.../java/com/github/ambry/frontend/NamedBlobPath.java
0.00%
22 Missing and 1 partial :warning:

...va/com/github/ambry/frontend/s3/S3ListHandler.java
12.50%
21 Missing :warning:

...va/com/github/ambry/frontend/s3/S3BaseHandler.java
0.00%
2 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2756       +/-   ##
=============================================
- Coverage     70.50%   52.55%   -17.96%
+ Complexity    11608     7965     -3643
=============================================
  Files           837      837
  Lines         71245    71295       +50
  Branches       8552     8561        +9
=============================================
- Hits          50232    37468    -12764
- Misses        18393    30814    +12421
- Partials       2620     3013      +393

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


Isolating and binding key points to results in training dataset

Completed with the latest commit. Check point 1.


RFM - Star Wars: The Old Republic

Do not do enough Windows development to justify being the main maintainer of this package any longer.
Source for the existing package exists here: https://github.com/brianmego/Chocolatey

@brianmego I would love to take over the maintaining of this package, if you wouldn't mind adding me as an maintainer (same nickname on chocolatey as here on github)

scratch that, I was already looking into it and couldn't get it to work.
So leaving it to someone else if they want

I will look at picking this up.

I've got this working so will submit the package to the feed shortly.

@pauby the Status: Published tag is for when the package have been approved.

I do not mind remaining as a maintainer. I may contribute from time to time as I still have love for the project. I simply do not live my life in the Windows world much anymore, so the daily "Oh, there's a new version! I should update that in chocolatey" call to action hardly ever comes up.
Looks like you were able to get your name on the chocolatey package, but let me know if I need to do anything else to help you, and thanks for picking it up!

The updated package have been approved.


Tabs to right sidebar,

Tabs on the right sidebar,
Is it possible to make tabs for the right sidebar and adjust the order in which the right sidebar is placed by adjusting the order of the tabs?

I think this tab plugin is meant to work for the mean view only and the ux for side bar is questionable.


Enhance TextureButton and TextureRect with resize

Add STRETCH_KEEP_ASPECT_COVERED to TextureRect StretchMode
Make TextureButton to the same resize behavior with TextureRect

Looks like I merged a bit fast (travis being regularly broken by other changes in the master branch doesn't help with CI), build now fails on X11 and Windows:
scene/gui/texture_button.cpp: In member function 'void TextureButton::_notification(int)':
scene/gui/texture_button.cpp:123:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:126:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:127:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:130:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:134:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:138:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:139:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:149:26: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:156:12: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_SCALE_ON_EXPAND' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_SCALE' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_TILE' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_KEEP' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_KEEP_CENTERED' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_KEEP_ASPECT' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_KEEP_ASPECT_CENTERED' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp:122:13: warning: enumeration value 'STRETCH_KEEP_ASPECT_COVERED' not handled in switch [-Wswitch]
scene/gui/texture_button.cpp: In static member function 'static void TextureButton::_bind_methods()':
scene/gui/texture_button.cpp:212:38: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:213:28: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:214:27: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:215:27: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:216:36: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:217:34: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:218:43: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp:219:42: error: 'StretchMode' is not a class or namespace
scene/gui/texture_button.cpp: In constructor 'TextureButton::TextureButton()':
scene/gui/texture_button.cpp:316:17: error: 'StretchMode' is not a class or namespace

I guess it's simpler/cleaner if you just reimplement the StretchMode enum in TextureButton instead of trying to proxy it from TextureRect :)

I have compile and tested the code on X11.
I have to make another PR to fix this?

I have compile and tested the code on X11.

Maybe you're using clang which would be more forgiving in this case?

I have to make another PR to fix this?

Yes, since this one is merged already.

OK I will check and make a new PR.


docs: Add JSON schema for IDE validation

Checklist:

[x] My organization is added to USERS.md - I'm a consultant

See #4477

Adds JSON Schema validation as the first IDE support method, with instructions for IDEA and VSCode
Deletes existing CRD validation approach

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

Still WIP?

@brabster can you do the DCO please?

Sorted sorry hadn't seen DCO before! Also sorted WIP label etc.

LGTM!

Back-ported to v2.12.0-RC5


Make a charset conversion depending on the language

due to subdb inability to set right set-encoding header

Thanks. Looks great.

Thanks! Just to know, are you going to release a new version?

Yes. I'll tag a new release.

@IvanAlegre I just published 1.0.3 to npm.

🎉 🎉 🎉 🎉 🎉 🎉
Thank you @arshad !


Integrate with gproc

Integrate with gproc_pool:

emqttd_pubsub_sup to start 'pubsub' pool to control pubsub mnesia transactions.
emqttd_pooler, emqttd_pooler_sup to provide a general pooler.


Staff can download a file by clicking on the filename in the Pet > show > Files tab table

Acceptance criteria

Staff can download a file by clicking on the filename in the Pet > show > Files tab table

Can I give this try?

Sure @DreamVenerable all yours


Enhance TRY support in the Fuzzer

Add logic to re-try the original expression without TRY for rows that didn't
generate errors when running with TRY (e.g. rows with non-NULL results of TRY
evaluation).
Fix handling of 'canThrow' parameter in ExpressionVerifier::verify.
When 'canThrow' is false, expression evaluation failure in the common path
should be propagated even if simplified path fails with a compatible exception.
Ran the fuzzer successfully for 10 minutes with the following flags and a number
of pending fixes: #3809, #3806, #3805, #3804, #3801, #3692, #3773 + fix for
least and greatest.
velox/expression/tests/velox_expression_fuzzer_test -lazy_vector_generation_ratio 0.2 --enable_variadic_signatures --velox_fuzzer_enable_complex_types --velox_fuzzer_enable_column_reuse --velox_fuzzer_enable_expression_reuse --logtostderr=1 --retry_with_try

The code looks good, but I'm confused about what we are trying to achieve.
I remember that the original intent of the retry-with-try flag was to wrap an expression that failed in a try() node, to ensure it won't throw in that case. If we're now only re-executing on the rows which won't throw exceptions, aren't we losing coverage on checking that the expression won't actually throw when using try()?

@pedroerp We are doing both: (1) re-run expression with TRY -> verify no errors and results match; (2) re-run original expression on a subset of rows that didn't produce an error and verify no errors and results match.


Communication error: Transferring files from Samsung to Mac by using Quick Share

Transferring from Samsung Galaxy S24 Ultra to MacBook Pro M3 using Quick Share worked fine until this week. Now, anytime I try, the Mac shows this error. However, transferring from Samsung Galaxy Tab S9+ to Mac still works.

Thanks RokGis,
I am experiencing the same issue as you are with my S24U and MacBook Pro M1. They both stopped working and show the same error message. I appreciate you for raising this issue on GitHub, and I want to thank Grishka in advance for fixing it :)

Can confirm that this is happening. My S24 Ultra and S24+ are having the same issue, but my other phones do not have this problem.
No idea what caused the issue though.

My Mac doesn't even show up in the list.

same thing for me, always worked till recently (S21 ultra)

Just downloaded the app and same thing with me (S20 FE)

+1 Seems like something is broken in between after the rebrand :(

Same problem, same notification here

Same on Galaxy 23 and M2 Pro Max Sonoma 14.4.1

Recently get this error with Pixel 8 and Google Play Services 24.15.58.
Another phone with Google Play Service 24.15.18 sees no issue.

it works fine for me on my galaxy s23 & m1 macbook air. when you share, you have to press the quick share icon, it won't just pop up like other devices do, it's available under the "share to devices nearby" section.

In the meantime, it has started to work. I was able to successfully send files from my phone to my Mac. I think Google fixed something.

this is what shows up for me. it shows up as a regular device now too once you share once

Guys please note that discovery issue is off-topic. This thread is about communication error after sharing.

Same for me, get the notification but it disappears instantly on macOS Sonoma 14.4.1 sending from my S24 Ultra

In the meantime, it has started to work. I was able to successfully send files from my phone to my Mac. I think Google fixed something.

Unfortunately it still doesn't work for me

⚠ ⚠ ⚠
PLEASE READ
⚠ ⚠ ⚠
If you

have the same issue
no new information to contribute,
please just leave an emoji reaction on this or the initial comment (or just wait patiently).

There is no point in flooding this thread & repository with comments -- it will just annoy the maintainer.

⚠ ⚠ ⚠
PLEASE READ
⚠ ⚠ ⚠
If you

have the same issue
no new information to contribute,
please just leave an emoji reaction on this or the initial comment (or just wait patiently).

There is no point in flooding this thread & repository with comments -- it will just annoy the maintainer.

I did a little bit of digging and was able to find out that the protobuf file wire_format.proto seems to be out of date.
Location_Nearby_Connections_V1Frame receives a frame type with the rawValue of 12 which is not defined in the definition file I was able to find.
My (really bad not a proper solution) fix is to handle rawValue 12 as unknownFrameType
After that, everything worked correctly and I received the file on my Mac without any other issues.
If someone can find the updated protobuf files we should be able to fix this issue properly

I did a little bit of digging and was able to find out that the protobuf file wire_format.proto seems to be out of date.
Location_Nearby_Connections_V1Frame receives a frame type with the rawValue of 12 which is not defined in the definition file I was able to find.
My (really bad not a proper solution) fix is to handle rawValue 12 as unknownFrameType After that, everything worked correctly and I received the file on my Mac without any other issues. If someone can find the updated protobuf files we should be able to fix this issue properly

Any chance you could rephrase this in words that a lay person could understand....please.

I did a little bit of digging and was able to find out that the protobuf file wire_format.proto seems to be out of date.
Location_Nearby_Connections_V1Frame receives a frame type with the rawValue of 12 which is not defined in the definition file I was able to find.
My (really bad not a proper solution) fix is to handle rawValue 12 as unknownFrameType After that, everything worked correctly and I received the file on my Mac without any other issues. If someone can find the updated protobuf files we should be able to fix this issue properly

Any chance you could rephrase this in words that a lay person could understand....please.

@DayDreamingTime He is just saying that there is a data transfer format that was changed by google. Which is out of date for this code and needs to be updated for fixing this connection issue.

Which I guess means that that's something the App developer (Grishka) needs to do and not something that I can do.  I am so frustrated about how difficult it is to transfer images from my Samsung phone to my Macbook Pro....I really was enjoying how easy NearDrop was after making myself half crazy with Dex for years.

@MobileSam
Hey Sam thank you for your input. I did some digging and got this protobuf file from google's repo https://github.com/google/nearby/blob/44725985088d0afa3ae6ac0246cce5bf553e5c5c/sharing/proto/wire_format.proto
I hope this is the correct file and would really appreciate if you could fix the protocol buffer issue.
I am not entirely familiar with swift and we are unsure when the library maintainer will get some time to address the issue. I am hoping if someone opens a proper fix PR the maintainer will approve them.

Hey @bartick
I'm sorry but this file is also out of date. You can see on line 195 that the FrameType enum stops at 7. We would need the one where it goes up to 12 or at least reference 12.

Now i am getting same "communication error" problem while transferring from my S21 FE to my MacBook Air. I think it needs an update i guess.

Thank you!!!!!  It's back and working again.  I am so grateful for all the work that you do.


Styling error and reset password pages

Maybe it's time we style these:

I am creating a new issue in the dark corners repo and closing this as well as issue #78 which are the same more or less.


user_guide: modernizr.min.js is taken from remote server

Currently the user guide references this js file from a remote server:

https://cdnjs.cloudflare.com/ajax/libs/modernizr/2.6.2/modernizr.min.js

Is this intended? Could it be made local, or rather not?
On the debian lintian checks, this is reported as a security breach:
https://lintian.debian.org/tags/privacy-breach-uses-embedded-file

BTW, is there a release plan/ETD for 3.1.12 ?

No, this isn't really intentional. It seems to be a part of the theme chosen for our docs a long, long time ago. I've never even touched that. I don't think it can be made local, because the reference is included at many different path levels, but I can add an integrity check for sure.
As for 3.1.12 ... Historically, we've never committed to estimates; things change, and all of our development happens on someone's spare time, so I can't answer that in a more straightforward way than "soon".


util: convert unicode ability names into --sync--

I am a bit skeptical of this, but my Emerald Weapon normal has untranslated autos that make make_timeline complain when trying to print them.  I could change print to use .encode("utf-8") but then I can't print to the command line without the newlines being escaped.
Maybe there's some better way to do this? It does seem a bit risky to just assume no unicode chars.

Huh. I didn't even notice this, it looks like it's terminal-specific? At least on my Windows 10 machine, Powershell seems to display it correctly, cmd can't display the characters but doesn't throw errors, and Git Bash just stops and complains.
I'm slightly concerned about silently swallowing anything non-ASCII, but given that to date it hasn't been an issue, it's probably okay?

I think this seems like the wrong approach.  I'll try a different shell if I see this again.


FIX: Use Guardian.basic_user instead of new (anon)

c.f. discourse/discourse@de983796e1b66aa2ab039a4fb6e32cec8a65a098
There will soon be additional login_required checks
for Guardian, and the intent of many checks by automated
systems is better fulfilled by using BasicUser, which
simulates a logged in TL0 forum user, rather than an
anon user.

maybe keep this cross compatible? If not then we need to now flag that AI no longer works with stable via the plugin compatibility thing.

 I am not sure how to keep it compatible in this case -- Guardian.new will no longer work for login required sites, so I don't think we have a choice in this case, unless we backport that BasicUser class to core stable.
We are due for a version bump soon, and also I wouldn't think AI is on many sites running stable (I could be wrong though).

@SamSaffron there is already a .discourse-compatibility entry for X introduced by Roman here:
https://github.com/discourse/discourse-ai/commit/e0691e70e87d512517f68fa323682db5f027259d
So this change will have no impact on stable. Just waiting on a core fix for guardian which is causing the test failure here then I need to merge this (and other plugin changes) so I can do redeployments.


License Missing

I noticed this was missing license, is it possible to add one?

Heya @archey - sure. I've added Apache License 2.0 (https://tldrlegal.com/license/apache-license-2.0-(apache-2.0)). Please let me know if this license doesn't suit your needs, however I feel like it should allow people to openly use this without much of an issue.


No such file or directory: '../../environment/frontend_server/storage/test-simulation/movement/0.json'

Klaus Mueller persona.scratch.importance_trigger_curr:: 145
150
aldhfoaf/????
the Ville:Dorm for Oak Hill College:Klaus Mueller's room:bed
Traceback (most recent call last):
File "/Users/zhlei/generative_agents/reverie/backend_server/reverie.py", line 468, in open_server
rs.start_server(int_count)
File "/Users/zhlei/generative_agents/reverie/backend_server/reverie.py", line 401, in start_server
with open(curr_move_file, "w") as outfile:
FileNotFoundError: [Errno 2] No such file or directory: '../../environment/frontend_server/storage/test-simulation/movement/0.json'

mkdir ./generative_agents/environment/frontend_server/storage/base_the_ville_isabella_maria_klaus/movement then start the simulation

add code before File "/Users/zhlei/generative_agents/reverie/backend_server/reverie.py", line 400
         curr_move_path = f"{sim_folder}/movement"           # If the folder doesn't exist, we create it.           if not os.path.exists(curr_move_path):             os.makedirs(curr_move_path)


Issues with spatial initial mapping

In supplementary section 1.3, you have noted that you found that a heuristic for initializing the OT solver based on an alignment found using only the spatial coordinates improves the performance of PASTE for some datasets. In this heuristic, you solve an optimization problem that seeks to find a rotation of the spatial coordinates in one slice that minimizes the Wasserstein OT between the spots of the two slices where the distance is measured according to the spatial distances between the spots.
But your implementation of match_spots_using_spatial_heuristic function takes the spatial coordinates of the two slices, centers them, calculates the pairwise distances between the spots of the two slices, and then solves the Wasserstein OT using ot.emd. Sorry if I am missing any detail here but I was not able to follow how your implementation finds an appropriate rotation. It just solves the OT based on the given spatial coordinates of the two slices. I've run Paste by rotating one of the given two slices (say 151673, and 151674 samples of the DLPFC) by multiple angles, and the initial mapping based on only the spatial coordinates was not finding the appropriate rotation.

Here, I've rotated the 151673 sample of DLPFC (left side) by 90 degrees. You can see that the final mapping of Paste is not what is expected (as Paste is considered to be rotation invariant) when the initialization of the OT solver is based on the spatial alignment heuristic and the value of alpha is 0.1.
However, for this case, I was able to make the final alignment better by changing the value of alpha to 0.001 giving higher priority to gene expression difference of the spots than the spatial coordinates. The alignment looked like the following:

But for the case of mapping between DLPFC samples 151670 and 151671, similar problems remain at alpha = 0.1.

The mapping is not properly fixed at alpha = 0.001, it seems to be going towards a somewhat better solution at alpha = 0.0001.
Here is the mapping when alpha = 0.0001

The link to the analyses: https://colab.research.google.com/drive/1nTsbY5nxgeYLXXS9E0xTGS5jpUeTjwTl?usp=sharing
From the above discussion, it seems to me that, the current implementation of PASTE doesn't really remain rotation invariant of any of the slices when the OT solver is initialized based on an alignment found using only the spatial coordinates, as the implementation of the mentioned heuristic doesn't seem to find any rotation angle, even though in the supplementary section 1.3 it is said that the heuristic seeks to find a rotation of the spatial coordinates in one slice that minimizes the Wasserstein OT distance.

Hi,
I'm so sorry for some reason I am just now seeing this-- I thought this was resolved.
I agree with your points and I just wanted to clarify something. When we say PASTE is rotation invariant, it is in the sense that PASTE utilizes pairwise distances between spots, which remains constant even upon rotation. Thus, you should expect the same result upon rotation.
Now, as you've shown above, the reason we used the match_spots_using_spatial_heuristic() function to find an initial mapping, is that sometimes our OT solver can get stuck a local minima during convergence depending on the initialization.  The function was intended to be more of an example on how to find an initialization. However, it doesn't always work as you've shown.
Intuitively, the more symmetric the shape of the tissue the harder it is to find the optimal rotation based solely spatial information. For example, if the spots between layers formed a perfect square and you were only looking at spatial coordinates, you'd have no idea which 90 degree rotation would give the optimal rotation. This is also why in the case above you see better results at lower alpha (giving more weight to gene expression).
In situations like these, if you know the proper orientation (to some degree), I would recommend rotating the spatial coordinates yourself before running PASTE.
Let me know if that answers your question.
Max


[docs]Add old version api document

Motivation
<EMAIL_ADDRESS>Currently, there is a lack of api documents for the old version, and the old version documents need to be added.
http://pulsar.apache.org/api/python/2.5.0-SNAPSHOT/
http://pulsar.apache.org/api/client/2.5.0-SNAPSHOT/
http://pulsar.apache.org/api/pulsar-functions/2.5.0-SNAPSHOT/
http://pulsar.apache.org/api/admin/2.5.0-SNAPSHOT/
http://pulsar.apache.org/api/pulsar-broker/2.5.0-SNAPSHOT/
http://pulsar.apache.org/api/cpp/2.5.0-SNAPSHOT/
Modifications

Add API document for the old version
support 2.2.0, 2.2.1, 2.3.0, 2.3.1, 2.3.2, 2.4.0, 2.4.1

@tuteng It seems that the generated javadoc contains a lot of chinese characters. Can you change the locale of your laptop to english and regenerate the documentation?

@tuteng updates?

@tuteng can you regenerate the api documents?

It seems to be my local language environment problem. I will regenerate these documents.

@sijie PTAL


Allow for left aligned labels for form fields

Currently, our form fields (input, select, combobox) have the labels appearing above the field and the fields display full width.  While this works well with longer forms, it often doesn't work in smaller spaces such as table pagination, small toolbars, and so on.  We need to add the flexibility to allow input, select, combobox and other form field components to display their labels to the left, as in the example shown in this screenshot.

Additionally, we also need to allow consumers to use this component without having the empty space reserved for an error or helper message.

Additionally, we also need to allow consumers to use this component without having the empty space reserved for an error or helper message.


Support Slack Ephemeral Messages

Need to add support for Slack's ephemeral messages for more informational -- not super important for everyone else type of messages.
This could also prove useful for dealing with Duo auth messages.

I wish to work on this issue. Pls let me know the details

Hello @tezansahu !
Thank you for looking into this :)
The long and the short of this is that it should be implemented as a dictionary parameter in the say_* methods that I have. Something like ephemeral=True.  That should follow the Slack API so that ephemeral messages are sent.
This way, anyone who implements a plugin can write out ephemeral messages.

is anybody working on this one ?

Hello @viveksyngh . If @tezansahu isn't working on it, you are more than welcome to.
It's not on my timeline for Q4 (maybe Q1).

Okay .. I will work on this. to send an ephemeral slack api also required user id ? How that user id will be passed ?

We currently make use of Slack's python-rtmbot which itself makes use of the python-slackclient.
The slackclient is set up here.  Once's that is setup, it is then placed into the bot_components package and referenced in slack_comm.py.  Here is an example.
It looks like https://github.com/slackapi/python-slackclient/blob/2a9c54601b98ce4559f4fd8dd30635ec248c101c/docs-src/basic_usage.rst#sending-a-message documents how to send an ephemeral message via the slackclient.

Thanks you explanation and references. As per python-slackclient documentation
Sending an ephemeral message, which is only visible to an assigned user in a specified channel, is nearly the same as sending a regular message, but with an additional user parameter.
f```
rom slackclient import SlackClient
slack_token = os.environ["SLACK_API_TOKEN"]
sc = SlackClient(slack_token)
sc.api_call(
"chat.postEphemeral",
channel="#python",
text="Hello from Python! :tada:",
user="U0XXXXXXX"
)
so will this user will also be an argument of **say** which will none for other case but some value in case of empheral message ?

I think it makes sense to have a keyword arg for ephemeral_user=THEUSER to all of the say functions.
The user details are a dict that is mostly passed into each command via the get_user_data function in slack_comm.py: https://github.com/Netflix/hubcommander/blob/develop/bot_components/slack_comm.py#L110
This shows the command being executed and getting user_data passed in:
https://github.com/Netflix/hubcommander/blob/develop/hubcommander.py#L74-L75

Should not functions like send_error, send_info, send_success should be modified to send ephemeral message and ephemeral_user values ?

Implemented in #63


Feature request: make it work in Google Docs and Google Keep

Would be great if Markdown Here worked with Google Docs and Google Keep

So it looks to me like there are two problems we need to solve to make this work:

Find a way to get the raw markdown from the open Google Doc

I did a little research into how we would get the raw markdown from google docs–it seems like the raw text is stored in the kix-wordhtmlgenerator-word-node class. So we could just loop over all the

var gDocTextElements = document.getElementsByClassName("kix-wordhtmlgenerator-word-node");
var i;
for (i = 0; i < gDocTextElements.length; i++) {
    console.log(gDocTextElements[i].innerText);
}

Trying this script on a example document, I get the exact markdown contents of the document output to console.
# This is a test
## Test2
* Hello World

Process raw text and write markdown format to Google docs

This one is a little trickier. We probably need to reverse engineer how Google does its HTML formatting. If it matches up with an existing formatter we have, then it should be straightforward. Otherwise we will need a new type of formatter.

Hope this helps someone get a start on this!


Intro example 1

Why are these changes needed?
This example showcases some of the useful transforms of DPK.
PDFs ---> text ---> chunks --->   exact dedupe ---> fuzzy dedupe ---> embeddings

code runs on colab (single click, no setup required) and local python dev env
have both python and Ray notebooks
works on simple PDF input, so user can track the transformations along the way
I created these, so I can understand the transformations.  I also used them during workshop.  they were well received

For reviewers

current URLs (colab links, image links ..etc) are pointing to my fork of data prep kit. This way the code will work on colab and can be reviewed.  Once review is concluded, i will update the URLs to point to main repo.
current location is examples/notebooks/intro as of now
using synthetic (generated) data as input,  they are checked into input/solar-system.  I hope this is ok

Related issue number (if any).

@sujee This is a nice introductory example. I was able to run the python version of this on colab, but because fuzzy dedup is only available with ray, I cannot see whether fuzzy dedup has a positive effect on reducing the number of chunks or not. On the other hand, testing the ray version gives a "ray job failed' error in pdf2parquet (before getting to doc id error in issue #719 ), so let's wait and see if PR #721 fixes the Ray issues.

Thanks for reviewing @shahrokhDaijavad
1 - Ray version erroring on pdf2pq step is due to downloaded model cleanup, i believe : #667
Is there fix in the works for this?
2 - are we good on location of this example : examples/notebooks/intro ?

@sujee

For the ray version, I see 3 errors. One is #667: ERROR - Exception creating transform  [Errno 2] No such file or directory: '/root/.EasyOCR//model/temp.zip' The second one is this: ERROR - Exception during execution out of 2 created actors only 1 alive and the third one is: ERROR - Exception during execution 'processing_time'
The location of this intro example is good.
Good to see the effectiveness of Fuzzy dedup. Does the ray version run successfully on the local machine?

For the ray version, I see 3 errors. One is [Bug] pdf2parquet ray version erroring out when downloading models for the very first time #667: ERROR - Exception creating transform  [Errno

confirming:
1A.  [Bug] pdf2parquet ray version erroring out when downloading models for the very first time #667
Possible fix : add MultiLock class #693
1B. [Bug] one of the created Ray actors die during docid transform #722
1C. Related to #722 above.
[Bug] docid ray transformation errors when running on colab (release 0.2.2dev1) #719
possible fix: Fix metadata logging even when actors crash #721

Good to see the effectiveness of Fuzzy dedup. Does the ray version run successfully on the local machine?

Yes, completes on local dev env  :white_check_mark:

Updated using DPK release 0.2.1


Fixed list of top-level package name is flawed for uncommon TLD's

https://github.com/bazel-contrib/rules_jvm/blob/0bef82e8d7038a6628faad06b9a57d10e536c2c5/java/private/package.bzl#L2C1-L2C72
The implementation is fundamentally flawed.
For example, I've just spent a few hours trying to figure out why my test classes were not being properly named for their package (TLD of kiwi), in my specific case, tests in src/test/java/kiwi/**/*. All country-code TLD based packages would fail too.
Bazel itself appears to use a heuristic based on the maven-style layout of a prefix of src/test/java/ which is even noted in a comment!
The simplest mitigation is probably to allow the prefix to be specified in the rule. E.g.
java_test_suite(
    name = "test",
    size = "small",
    srcs = glob(["src/test/java/**/*"]),

    src_prefix = "src/test/java",

    resources = glob(["src/test/resources/**/*"]),
    runner = "junit5",
    runtime_deps = JUNIT5_DEPS,
    deps = [
        ":lib",
        artifact("org.junit.jupiter:junit-jupiter-api"),
    ],
)

Just found #104

So, the work-around is to specify `package_prefixes:
package_prefixes = [".nz.", ".kiwi.", ".xyz."],

This is just missing from the documentation.

Closing as it's just a duplicate.


Calcite tests remove expected exception

update testcases using expectedException to utilize assertThrows instead
remove BaseCalciteQueryTest#expectedException
fixes cannotVectorize so it doesn't anymore stops further processing
msqIncompatible is not anymore toggles a boolean - its an Assume instead

Fixes #15423
This PR has:

[x] been self-reviewed.

failure of jdk11 static checks seem unrelated

The rule ordering cleanup changes can come in a later patch


Possible spelling mistakes

Bug report
Describe the bug
In the readme.md, at 'Routing | Intercepting popstate' section
There is a describe 'If you return a falsy value from beforePopState'
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:

Go to 'https://github.com/zeit/next.js#intercepting-popstate'
Search 'falsy'
See error

Expected behavior
I suppose it should be changed from
If you return a falsy value from beforePopState
to
If you return a false value from beforePopState
Screenshots

Feel free to send a pull-request 😄

Yes @EdenLiang falsy is a valid term not a typo.

IMO this issues should be closed. The sentence 'If you return a falsy value from beforePopState' is perfectly valid.
Falsy is the opposite of truthy. These are proper terms in JS. They are distinct from false and true. Falsy values act like false in if statement, and truthy values act like true in if statements.

It seems a change has been merged that changes this https://github.com/zeit/next.js/pull/5513/files
IMO that phrasing
If you return a `false` value from `beforePopState`, `Router` will not handle `popstate`

is overly verbose, and could be simplified to
If you return `false` from `beforePopState`, `Router` will not handle `popstate`


Integration test enhancements

For all provides, we'll need integration tests for:

[ ] GPU
[ ] Custom VPC
[ ] Spot instances (preemptible)
[ ] Upgrade from previous version
[ ] Redeploy via GitHub-Actions
[ ] Redeploy via GitLab-CI

xref: https://github.com/nebari-dev/nebari/issues/1951 & similar

For handling the items below, just adding as ref., we used to have a test that handled that in a private repo, it might be worth checking that:
 Redeploy via GitHub-Actions
 Redeploy via GitLab-CI


Test the programs in array and datastructures/linkedlists

All source files in the said directories need to have tests.
All components (methods) must be tested responsibly with respective edge-cases.

i am working on this,  can you assign me

Sure, @Kesav890.

Are you still working on this, @Kesav890 ?
I shall reassign the issue in case you reply in negative or do not reply before 12 hours.

Sorry but can you assign it to someone else?

No problem. This issue is up for the taking now.

Can you assign me because I am working on this .

Here you go, @akashkeshriofficial.

If you aren't interested in this (or any other) Issue(s) anymore, do let me know @akashkeshriofficial. I will remove you as an assignee.


version 1.92 can't select configure checkout box for new or editing existing profile  but 1.91 can in Macbook M1 pro with Mac 15.1.

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.92.1
OS Version: Mac 15.1 Beta版(24B5024e)

Steps to Reproduce:

Create a new profile not edit a existing profile, when you click the checkout-box, all of them will be disable, and can't enable any more. Just like this.

/duplicate of #224788 which offers a workaround.


Add conv_utils_test

This PR adds conv_utils_test to cover the following:
keras/utils/conv_utils.py                    79     24    70%   34-35, 38, 43-44, 56, 66, 68, 89, 111, 120-121, 137-146, 151, 155

The results are:
keras/utils/conv_utils.py                    79      1    99%   66


Full check with checksums?

Output of restic version
restic 0.9.6 compiled with go1.13.4 on windows/amd64

What should restic do differently? Which functionality do you think we should add?
restic check should not only check if all files exist, but also do a checksum / crc check of files content.

What are you trying to do?

Trying to verify if all the files are really intact or if they've been changed because of RAM errors, HDD errors or stuff like that.
Did restic help you today? Did it make you happy in any way?

Yeah, restic does a very good job in backup data. Its a good feeling to know your data is safely backuped

restic check --read-data does what you want.


Feature request: fonts.

Probably sufficient to have an option that chooses a front from powerline-web-fonts's stylesheet, and another option to override the stylesheet and allow other fonts (there's dozens of fonts available between nerdfonts.com and programmingfonts.org).
Examples:

--font=inconsolata adds @import<EMAIL_ADDRESS>and sets font-family:inconsolata,monospace;
--stylesheet=https://www.example.com/shitty-fonts.css --font="Comic Sans" adds @import url('https://www.example.com/shitty-fonts.css') and sets font-family: "Comic Sans", monospace; and maybe an option for text-transform: full-width?

What's the use case? Can't you add these styles yourself?
If you're using the --doc flag, I think a better solution would be an option to add custom CSS, perhaps even custom HTML:
to-html -d --css ".terminal { font-family: Inconsolate, monospace }" 'cargo run'

I can absolutely add the styles myself, however the primary use case would be automating it. This way you can set up an alias that re-runs the previous command, captures the output, uploads it to a gist or private server and copies a link to the clipboard for pasting elsewhere.
A --css option would work just as well and it's probably a better idea, though it would make the command a bit longer.


style(blink): use block vs. icon for colors

Description
Replace use of icon for Color kind with block characters for blink.cmp autocomplete items to make swatches more usable. This also makes the blink.cmp built-in Tailwind integration better match nvim-cmp.
Note: I didn't want to replace the default LazyVim.config.icons.kinds.Color in case it's used by other plugins (e.g. context), but didn't actually find any real uses, so perhaps that could be done instead? Figured best to leave those untouched as they can be more generically used as actual icons, and are not always guaranteed to have custom highlights applied.
Related Issue(s)
None.
Screenshots
Old

New

Checklist

[x] I've read the CONTRIBUTING guidelines.

like it. ty!


Nuxt v3 with Nitro worker threads - Module did not self-register

Possible bug
Is this a possible bug in a feature of sharp, unrelated to installation?

[x] Running npm install sharp completes without error.
[x] Running node -e "require('sharp')" completes without error.

If you cannot confirm both of these, please open an installation issue instead.
Are you using the latest version of sharp?

[x] I am using the latest version of sharp as reported by npm view sharp dist-tags.latest.

If you cannot confirm this, please upgrade to the latest version and try again before opening an issue.
If you are using another package which depends on a version of sharp that is not the latest, please open an issue against that package instead.
What is the output of running npx envinfo --binaries --system --npmPackages=sharp --npmGlobalPackages=sharp?
System:
    OS: macOS 12.3.1
    CPU: (10) x64 Apple M1 Pro
    Memory: 21.15 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.19.0 - ~/.nvm/versions/node/v14.19.0/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.14.0 - ~/.nvm/versions/node/v14.19.0/bin/npm

What are the steps to reproduce?
Install sharp package from Nuxt 3 RC5 and import * as sharp from "sharp";
I have even tried to install using --platform and --arch but still not working.
What is the expected behaviour?
not available
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this problem
not available
Please provide sample image(s) that help explain this problem

Please see https://sharp.pixelplumbing.com/install#worker-threads

Hi @lovell thanks for the prompt response.
Btw, how do I tell my Nuxt 3 on this Worker Thread ?

Perhaps you could create a minimal repo that allows someone else to reproduce.

Found this thread https://github.com/lovell/sharp/issues/2981
It seems not working on Nuxt 3?

#2981 relates to the creation of an ESM wrapper for sharp; mention of Nuxt is coincidental.

#2981 relates to the creation of an ESM wrapper for sharp; mention of Nuxt is coincidental.

Sorry, I'm still don't get it 🥹
Is it really Nuxt 3 can't use Sharp yet?

Please create a minimal repo that allows someone else to reproduce, then we can start to answer your questions.

Here's the link https://stackblitz.com/edit/nuxt-starter-hw78au?file=server%2Fapi%2Fsharp.ts
run npm run dev and you will encounter the error.

Thank you. Although Stackblitz doesn't work, I was able to download the project and run it locally to reproduce.
Remembering that https://sharp.pixelplumbing.com/install#worker-threads says:

the main thread must call require('sharp') before worker threads are created.

I can see that Nuxt uses Nitro for its local dev server, which uses worker threads.
https://github.com/unjs/nitro/blob/main/src/dev/server.ts
The Nitro docs state that it provides hooks that allow you to inject custom lifecycle logic.
https://nitro.unjs.io/config/#hooks
Putting all this together, the following appears to work:
export default defineNuxtConfig({
    nitro: {
        hooks: {
            'dev:reload': () => require('sharp')
        }
    }
})

As an aside, the Cannot call a namespace ('sharp') warnings can be silenced with the following change:
- import * as sharp from 'sharp';
+ import sharp from 'sharp';

@juno-w Were you able to make progress with this?

@juno-w Were you able to make progress with this?

Hi @lovell , the buffer() is not working though. I will close this issue and reopen it once I get more details on this.


defaultValue or defaultChecked stop columns sorting

Hi man!,
I don't know why but when I use cellRenderer with input and defaultValue or defaultChecked the sorting stop to works... some columns works but not where is present a cellRender with this setup. If I remove they works properly an I can sort again.
My goal is to have custom checkbox, input date and input text ( all customized directly using html )... My way is more simple and not require tons of call back. It's works with data, onclicks events in general and are really very easy to manage... but I incurred into this issue. Can you help me to figure out a way?
It's an example:
  id: 3,
  visible: true,
  resizable: true,
  field: 'active',
  label: 'Active',
  width: "90px",
  //sortable: false,
  className: styles.centerText,
  cellRenderer: ({ value, data, column }) => {
    return (
    <input
      id={`product[${column.field}]`}
      name={`product[${column.field}]`}
      type="checkbox"
      row-id={ data.product_id }
      defaultChecked={ value == -1 ? true : false }
      onInput={ (e)=> onInputProd( 2, [ data.product_id, props ], e ) }
      onChange={ onChange }
    />
  )}

Hi, I'm not sure what you're trying to achieve here, are you trying to sort a column made out of checkbox only?

Also, if you can recreate a basic live example of the issue you're facing in a codesandbox or something it will be much easier and faster to find the problem... Thanks

Since you're not responding can i assume that you got it fixed? let me know if I can close this issue...


Windows wheels

Please provide Windows wheels. I can send a PR if you'd like?

PR would be super helpful, yes.


[silgen] Refactor WritebackScope functionality into a new FormalEvaluationScope.

[silgen] Refactor WritebackScope functionality into a new FormalEvaluationScope.
As per John, WritebackScope was always an unfortunate name. Generally these
scopes are meant for formal evaluations of inout parameters. The cases that I am
interested in generalizing them to be used for are borrows of the base of a
class that will then be used as an lvalue.
This also eliminates the out of line vector of lvalue writebacks.
rdar://29791263

@swift-ci Please smoke test and merge


Mullvad 2022.5 error

Reporting a bug 🐛
Expected behavior
Install new version of MullvadVPN when deb-get upgrade
Actual behavior
dpkg: warning: version '2022.5
2022.5' has bad syntax: invalid character in version number
  [!] ERROR! Failed to download https://github.com/mullvad/mullvadvpn-app/releases/download/2022.5/MullvadVPN-2022.5_amd64.deb
https://github.com/mullvad/mullvadvpn-app/releases/download/2022.5/MullvadVPN-2022.5_arm64.deb. Deleting /var/cache/deb-get/MullvadVPN-2022.5_arm64.deb...
E: Unsupported file /var/cache/deb-get/MullvadVPN-2022.5_arm64.deb given on commandline

Steps to reproduce
deb-get update
deb-get upgrade
System information
/etc/os-release
PRETTY_NAME="Debian GNU/Linux 11 (bullseye)"
NAME="Debian GNU/Linux"
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye
ID=debian
HOME_URL="https://www.debian.org/"
SUPPORT_URL="https://www.debian.org/support"
BUG_REPORT_URL="https://bugs.debian.org/"
deb-get version
0.3.6

Manually checking the indicated URL seems to work: https://github.com/mullvad/mullvadvpn-app/releases/download/2022.5/MullvadVPN-2022.5_amd64.deb

Thanks for the report.  They added arm support and found a hole in our logic. Sorted by :point_up:


Update path casing for Android Java files to align with package name

Java package names expect to follow the path name of files, including casing. Currently the Java code lives at src/main/com/xbox/httpClient, but the package name is com.xbox.httpclient (note the lowercase).
This PR changes the path of the Java files to be src/main/com/xbox/httpclient, following lowercasing. Prefer to change the path name than the package name, since it affects fewer things.

I'm slightly worried about the change in case for the folders: on windows the new path and the old path are considered equal, so it will not change the folder names in the clone and then when more files are added they will pick up the old case for the folder and things will get messy

Per offline convo b/w me and @lubeltra, we were able to confirm that Windows will do the "good/correct" thing here and re-case the paths. See:
On main currently:

On sasha/javaPathCasing:


Providing Device-ID

Error while letting the musicbot list all aviable device ID's.
Traceback:
No GMusic device ID found. Do you want to use this devices ID (1) or select an existing device ID (2, recommended)? 2
Enter a device ID (leave empty to show all registered devices):
/usr/local/lib/python3.5/dist-packages/Cryptodome/Math/_Numbers_gmp.py:230: UserWarning: implicit cast to 'char *' from a different pointer type: will be forbidden in the future (check that the types are as you expect; use an explicit ffi.cast() if they are correct)
  _gmp.gmp_snprintf(buf, c_size_t(buf_len), b("%Zd"), self._mpz_p)
14:20:53 (ERROR, music_apis.py:664): No device ID provided, printing registered devices:
14:20:53 (ERROR, music_apis.py:665): [
    {
       List of all aviable Device ID's - working and with correct information
    }
]
14:20:53 (CRITICAL, main.py:99): Error accessing GMusic. (Missing device ID in secrets)
14:20:53 (INFO, main.py:100): Maybe try offline mode (--offline argument)
14:20:53 (INFO, async_handler.py:45): Shutting down...
Traceback (most recent call last):
  File "main.py", line 96, in <module>
    gmusic_api = music_apis.GMusicAPI()
  File "/media/***/music_apis.py", line 332, in __init__
    self._connect()
  File "/media/***/music_apis.py", line 666, in _connect
    raise ValueError("Missing device ID in secrets")
ValueError: Missing device ID in secrets

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 101, in <module>
    async_handler.shutdown()
  File "/media/***/async_handler.py", line 48, in shutdown
    os.kill(os.getpid(), signal.SIGINT)
KeyboardInterrupt

Seems like you interrupted the shutdown process by hitting Ctrl+C.
Apart from this, shutting down after printing devices is expected behavior right now.

I did not, but if the process should shutdown anyway this works for me.
The device-ids are listed as error, though. Wouldn't it be better to list them and call sys.exit() manually instead of letting the program run into an critical error?

Probably...
That doesn't have a high priority right now though.

Followup issue: #100


Cannot deploy from "set up > data" in Amplify Studio

Before opening, please confirm:

[X] I have installed the latest version of the Amplify CLI (see above), and confirmed that the issue still persists.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

How did you install the Amplify CLI?
curl
If applicable, what version of Node.js are you using?
14.18.2
Amplify CLI Version
7.6.2
What operating system are you using?
Mac
Amplify Categories
auth, storage, function, api
Amplify Commands
push
Describe the bug
I have a bunch of functions which I've deployed, along with an API:

In amplify studio I get this error when I try to run Save and Deploy from the Amplify Studio UI (appid d26seotyr25w3q):
Command failed with exit code 1: /root/.amplify/bin/amplify push --yes --allow-destructive-graphql-schema-updates ⚠ Amplify command hooks are disabled in the current execution environment. ⚠ See https://docs.amplify.aws/cli/usage/command-hooks/ for more information. An error occurred pushing an env "dev" due to missing environment variable values.
Expected behavior
All I want to do is get the Content editor running. Any ideas?
Reproduction steps

Create a heap of functions and an api with auth
Cognito user pool set as default auth method
Conflict resolution recommended mode

GraphQL schema(s)
I've got a big graphql schema but I don't want to put it here.
Log output

# Put your logs below this line

Additional information
No response

Hi @isaacrowntree
we currently dont support functions with environment variables inn amplify studio.
this link might be able to help you better : https://github.com/aws-amplify/amplify-adminui/issues/332#issuecomment-964380483


[WIP] Cloud migration tutorial

A tutorial showing users how to use a local cluster to simulate cross-cloud deployment and then migration from one cloud to another.
I'm planning to add a simple screencast (perhaps with audio or text) demoing this across real cloud platforms, and offering that video as something to watch before you begin. But I'm opening this to get initial feedback on the flow and basic messaging.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a457025035dfaa8eb2056186993f715bf99a99b6/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/33577695c91c0934e1125ec765253f4da0d31077/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9de181bb50387f6b60fde752e27f51cfb5f5de5a/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f8645eb6b95b234363c8ca97c85ab28b6092f424/

LGTM

Review status: 0 of 10 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

demo-cloud-migration.md, line 7 at r4 (raw file):

---

CockroachDB's flexible [replication controls](configure-replication-zones.html) make it trivially easy to run a single CockroachDB cluster across cloud platforms or to migrate a cluster from one cloud to another without any service interruption. This page walks you through a local simulation of the process.

Cross-cloud migration is just a special case of cross-DC migration. Since this is just a simulation, it seems weird to say it's a simulation of the special case instead of the general case. But I guess that's what gets the clicks...

demo-cloud-migration.md, line 55 at r4 (raw file):

~~~ shell
$ brew install haproxy

This is mac-specific. I'm OK with not having instructions for other platforms at this point, but we should say something like "In a new terminal, install HAProxy. If you're on a Mac and using Homebrew, use brew install haproxy"

demo-cloud-migration.md, line 64 at r4 (raw file):

~~~

This command generates an `haproxy.cfg` file automatically configured to work with the 3 nodes of your running cluster. In the file, change `bind :26257` to `bind :26270`. This changes the port on which HAProxy accepts requests to a port that is not already in use by a node and that won't be used by the nodes you'll add later.

These port numbers all start to run together. How about a round number like 26000 instead of 26270?

demo-cloud-migration.md, line 66 at r4 (raw file):

This command generates an `haproxy.cfg` file automatically configured to work with the 3 nodes of your running cluster. In the file, change `bind :26257` to `bind :26270`. This changes the port on which HAProxy accepts requests to a port that is not already in use by a node and that won't be used by the nodes you'll add later.

~~~ shell

shell mode doesn't highlight this correctly (it colors bind but not any other keywords). Unless there's a more precise option, better to just remove the shell tag completely.

demo-cloud-migration.md, line 98 at r4 (raw file):

~~~ shell
$ go get github.com/cockroachdb/loadgen/ycsb

There should be an introductory section that tells people they'll need to install Go (and we could put the haproxy installation instructions there too)

demo-cloud-migration.md, line 183 at r4 (raw file):

## Step 9. Stop the cluster

Stop HAProxy and YCSB by switching into their terminals and pressing **CTRL + C**. Do the same for each CockroachDB node.

Stop YCSB first, or it will start spewing errors as soon as you stop haproxy.

Comments from Reviewable

Review status: 0 of 10 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

demo-cloud-migration.md, line 64 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
These port numbers all start to run together. How about a round number like 26000 instead of 26270?

Done.

demo-cloud-migration.md, line 66 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
shell mode doesn't highlight this correctly (it colors bind but not any other keywords). Unless there's a more precise option, better to just remove the shell tag completely.

Done.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/be3014696c690cb648503a8092024eb017585ca4/

Minor feedback (we discussed in person too)

it would be useful to have people start in some sort of ./cloud-migration-test directory so they don't get a bunch of data files in their home
as written, the ycsb command doesn't work after the go-install. need to include the command to build
We shouldn't pre-split the ycsb ranges. I think this breaks (or at least doesn't reinforce) a beginners mental model for how CRDB works. Taking off the split shows new ranges emerging as new load comes in, vs just having it be flat the whole time.
In the final 'aha' graph where the load switches we ask users to hover over the graph to see the replica counts, but in my instance all but 1 were zero, which clearly wasn't correct. I think I would just say look at the graph, it's clear from the shape and color change that the load has moved to the other cloud.  For the record, I just realized my crdb local install is beta, but according to Jesse this is still a bug.

Review status: 0 of 10 files reviewed at latest revision, 9 unresolved discussions.

demo-cloud-migration.md, line 55 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
This is mac-specific. I'm OK with not having instructions for other platforms at this point, but we should say something like "In a new terminal, install HAProxy. If you're on a Mac and using Homebrew, use brew install haproxy"

Done.

demo-cloud-migration.md, line 98 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
There should be an introductory section that tells people they'll need to install Go (and we could put the haproxy installation instructions there too)

Done.

demo-cloud-migration.md, line 183 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
Stop YCSB first, or it will start spewing errors as soon as you stop haproxy.

Done.

Comments from Reviewable

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/ff00ea8aaf016751d72212494e6dd71e2b5a99be/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/2d750409e4ae2c1db9f0532ae914d02774f8b2ea/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/80c4bbc227863b57de4a1ab51cfb40ae72e44007/

@nstewart:

Added a "before you begin" suggestion to create a new directory for running nodes.
Moved all installations to "before you begin". And since go get does in fact build ycsb in $HOME/go/bin by default, I've added that path to the command to start ycsb.
No longer pre-splitting ranges with ycsb.
Called out that the replica count on hover will be off for a few minutes but that the graph lines are accurate. Also pointed to the node list view for another accurate view of per-node replica counts.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/02bec3497a9df3647a2347c46e32ae0f850c4359/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9a770304ed498e4eda92ada7511890ad79edfb18/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/9a770304ed498e4eda92ada7511890ad79edfb18/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/838cb2fb203b365a4b0216d2764931d238704cc4/

Merging. Will open a follow-up PR for the screencast part.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/5ae031f0b04d0c645a4c1be7d3752d1723ad5d7e/


"configuration" breaks Syntax Highlighting

"configuration" breaks syntax highlighting after it.

This is what it should look like.

This is fixed in dev already by https://github.com/SublimeText/PowerShell/pull/137


Push to NPM

Can you please publish this project as an NPM package without the docker folder?
This will be useful for projects which want to use xliffmerge tool and provide a default editor.

Hello,
I' m not sure if I understand your request.
tiny-translator is not a library or a command line tool, but a webapp that runs in the browser.
So what should I publish to npm?
There is a demo installation on github pages, so you just have to click the link to start it.
Maybe you can explain your needs a little bit further.

I can think of publishing tiny-translator as Angular component (just like Material components) that users can install and insert into their app. When it will be possible to configure it so this component loads xlf automatically. It can create a great workflow: users generate a new xlf file, run their app, and immediately see and translate all new words!

You are right. In general it should become much more modular to allow internal usage scenarios.
But at the moment the whole i18n workflow of angular is not really modular and open.
Let us see what the future will bring...

In 5.0 NG team promises to simplify library publishing and in 5.x I hope they introduce text translations in source files


[UI] Tab Headings of Troubleshoot Guide in Dark Mode

Current Behavior
The tab headings have a light background  when their content is being displayed.
Screenshots/Logs

https://user-images.githubusercontent.com/103871275/222959961-407a4340-9c2c-4bfa-b479-846a62aa3cd2.mp4
Environment

Browser: Firefox
Host OS: Linux
Meshery Server Version: stable-v
Meshery Client Version: stable-v
Platform: Docker

Contributor Guides and Handbook

🎨 Wireframes and designs for Meshery UI in Figma
🖥 Contributing to Meshery UI
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I want to work on this can you assign this to me?

Sounds Great !! @aryans1319

Hey, @hs05june can you provide me access to the Figma file?
Email -<EMAIL_ADDRESS>

Hey, @hs05june can you provide me access to the Figma file? Email -<EMAIL_ADDRESS>
@leecalcote

@aryans1319 Figma access is with community members only. Really Sorry, I will be assigning you another issue.


Allow setting an expiration date for the pinning policy

Would this mean that pinning can be disabled by moving the system date up a bit? I'm not entirely sure but it should be noted in the docs.

Correct, but pinning is not something meant to prevent someone with full access to the phone from being able to decrypt the device's network traffic (although it may, as a side effect, make it harder).

Oh yeah. I was thinking more along the lines of compromising iOS NTP than manually changing the system time 😉 Not a big deal.

@nabla-c0d3 @adamkaplan Might this be better if the expiration date results on a "drop matching connections if pins expired" behaviour, rather than allow all valid certs if pins expired?
Or is this going to be more an optional feature (if exp date provided, check the date etc)?

It will be optional. If an expiration date is set and we're past the date, TrustKit will not perform pinning validation (any valid cert will be accepted).

Hey @nabla-c0d3 this actually came up in a review discussion today. The use case we’re thinking about are the small minority of users who do not update their app, their OS, or both, sometimes for years. There are many active users who become unsupportable after some time, and ideally their apps don’t just brick.
Any idea when you’ll have something ready? Or, do you want some help on this one?

Yes, that's exactly the use case for an expiration date. I have started looking into this and it should be a quick change - probably a couple weeks at most. Also, if you'd like to use our dashboard for the SSL failure reports, feel free to email me (email is on my GH profile).


HHH-5927 Performance risk: Suboptimal synchronization in org.hibernate.en

I would like someone review this fix:
this commit using google guava replaced commons collection

See also comments at https://hibernate.onjira.com/browse/HHH-5927

closing this one, will create a new one


Mac App Store not working after persists BCLM on reboot

macOS version: 10.14.6
MacBook Pro 13-inch, 2018

@goodjack, are you able to verify that unpersisting bclm fixes this and then persisting breaks it again?
On its own, all persist is doing is running bclm once and exiting on startup. It is the same as just running it from the terminal. The only thing I could think to check is to make sure that sudo launchctl list com.zackelia.bclm.plist shows an exit status of 0 but even then that doesn't really make sense in affecting the App Store. Let me know what you find!

{
	"LimitLoadToSessionType" = "System";
	"Label" = "com.zackelia.bclm.plist";
	"TimeOut" = 30;
	"OnDemand" = true;
	"LastExitStatus" = 0;
	"Program" = "/usr/local/bin/bclm";
	"ProgramArguments" = (
		"/usr/local/bin/bclm";
		"write";
		"77";
	);
};

That output looks right, bclm didn't run into any errors on startup. Are you able to verify that this App Store behavior is consistent with persisting and unpersisting bclm?

Sorry, I can't reproduce the error, but I've had this issue twice.
Once I unpersist, everything in the Mac App Store will go back to normal.
The abnormal condition here means that I cannot update or download the software.

This sounds like it might just be a coincidence, please do let me know if you can get this to be repeatable. The persist command is just running bclm once on startup. Because of this, I would think that any time you ran bclm in the terminal, your App Store wouldn't work which I don't think is the case. Going to close for now.

Thank you for your time


Update the quickstart code to fix compilation error

The quickstart code in README causes compilation error as follows.
error: incompatible types: Map<String,String> cannot be converted to Map<String,Object>
        LOG.log("follow", data);
                          ^

I've replaced String with Object.

Thanks!


upgrade IDEA target version to latest and briefly check things work

otherwise we can only test with the highest target version listed for plugin ( 2019.3.3 )

clean -> assemble -> runIde gives the "A+.Menu" error

For me, it seems to happen when one tries to open the menu too soon after opening a project (could it be: before the indexing is finished?). It may be an issue with this DumbAware thing (which I am too dumb to be fully aware of).

but is it at all related to the version upgrade... ? :confused:

Perhaps our configuration was always "wrong" but the old IDEA version was more forgiving in that sense. Or it can be a bug of new IDEA.


Reverse dataset order

Reverse the order datasets are displayed on the regions page, so that they appear newest-first instead of oldest-first.

I could also make the order toggle-able, similarly to the dataset-counts page.

:books: Documentation preview :books:: https://datacube-explorer--518.org.readthedocs.build/en/518/

@robbibt Unfortunately I haven't had any luck making DataTables responsive, so I don't think I can make it toggleable at this time.

No worries, thanks so much @Ariana-B!


Nesting controllers

I'm wondering because i don't see it in the documentation in the README, does the controller system all for nesting controllers.  Express router supports this, just trying to find out if controllers do in routing-controllers.
Use case:
/api
/api/v1
/api/v1/user
i'd like to setup a controller for api that would point at a folder for its controllers.  In that folder, i could setup multiple folders for v1, v2, etc.  In those folders would be the controllers for the respective versions.

You cannot do it the way you describe it, and you should not do it. however, you can do it. routing-controllers expects all of its routes to be preloaded at startup time like:
import 'reflect-metadata';
import { useExpressServer } from 'routing-controllers';
// other imports

import './routes';

class App {
  public express: express.Application = express();

  constructor() {
    useExpressServer(this.express, {
      // your config
    });
  }
}

export default new App().express;

And you can prefix your routes via:
@JsonController('/v1/categories')
export class CategoriesRoute {
  // your route handlers here
}

So if you structure your routes into the routes/v1, routes/v2 folders and re-export them from an index.ts (a barrel) file, and then re-export it from routes then it will work.
// in routes/index.ts
export * from './v1';
export * from './v2';

However, I strongly advise you to not do this. Until you don't make a breaking change you don't have to update the version number and as soon as you need to you should copy-paste your current app, and create a separate app from it for v2.
If you don't do it sooner or later your app will become a mess as you need to keep all the old logic and a) monkey patch the new logic into it (I mean in services), b) start copying only services both way your codebase will start to be filled with hacks and duplicated code which will make maintenance superhard.
In the future, we may have some support for versioning though, a decorator like @Version('1.2') which will make it possible to manage the versioning in the same app.


[v3] Firebase Remote Configuration

It's listed in the README as being available, however I noticed in Firestack.m that the entire code block for remote config is commented out. What was the reason for removing / when do you think it will be made available via Firestack ?

We removed it a bit for the refactoring we did a few weeks ago... it's reasonable to add it back in. Have experience with it and a few cycles to implement it back in?

ok no worries. ive not used Remote Config api yet this was first foray. am on a RN client project at mom (using Firestack!) not sure if ive got the bandwidth just now but if i can i'm happy to help (ill ping u if so)

We worked on this today @Jeanmsilva89 and I, that is. I'm expecting a PR with this feature this evening.

@auser Submited the PR for both repos. Thanks a lot for your time today! Have a great one!

when is it going to be ready to use remote config for both android and ios? I can help on making that feature available if there is any roadblock with the pull request.

+1 it is a blocker for my app, If you need help with javascript side, i can certainly help.

Is this ever coming back?

Would love to have this :)
Awesome library guys!


Update to TF 1.0

I've updated most of this to the latest version of Tensorflow. I just notice though that it doesn't work with Tensorboard. Any ideas on where to start making those changes?

I have incorporated the SummaryWriter for tensorboard. please try with fresh checkout for latest updates compatible with tensorflow v1.2.1


[Snyk] Upgrade @typescript-eslint/eslint-plugin from 7.13.0 to 7.18.0

Snyk has created this PR to upgrade @typescript-eslint/eslint-plugin from 7.13.0 to 7.18.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 100 versions ahead of your current version.

The recommended version was released on 23 days ago.

Release notes

Package name: @typescript-eslint/eslint-plugin

7.18.0 - 2024-07-297.18.0 (2024-07-29)
🚀 Features

types: update ECMA versions (#9634)

🩹 Fixes

eslint-plugin: [no-unnecessary-type-assertion] prevent runtime error when asserting a variable declared in default TS lib (#9660)
eslint-plugin: [unbound-method] report on destructuring in function parameters (#8952)
eslint-plugin: [no-duplicate-type-constituents] shouldn't report on error types (#9600)
eslint-plugin: [strict-boolean-expressions] support branded booleans (#9297)

❤  Thank You

auvred @ auvred
Oliver Salzburg
Vinccool96
Yukihiro Hasegawa @ y-hsgw

You can read about our versioning strategy and releases on our website.

7.17.1-alpha.9 - 2024-07-29

7.17.1-alpha.8 - 2024-07-29

7.17.1-alpha.7 - 2024-07-28

7.17.1-alpha.6 - 2024-07-28

7.17.1-alpha.5 - 2024-07-28

7.17.1-alpha.4 - 2024-07-25

7.17.1-alpha.3 - 2024-07-23

7.17.1-alpha.2 - 2024-07-23

7.17.1-alpha.1 - 2024-07-23

7.17.1-alpha.0 - 2024-07-23

7.17.0 - 2024-07-227.17.0 (2024-07-22)
🚀 Features

eslint-plugin: backport no-unsafe-function type, no-wrapper-object-types from v8 to v7 (#9507)
eslint-plugin: [return-await] add option to report in error-handling scenarios only, and deprecate "never" (#9364)

🩹 Fixes

eslint-plugin: [no-floating-promises] check top-level type assertions (and more) (#9043)
eslint-plugin: [strict-boolean-expressions] consider assertion function argument a boolean context (#9074)
eslint-plugin: [no-unnecessary-condition] false positive on optional private field (#9602)
typescript-estree: don't infer single-run when --fix is in proces.argv (#9577)
typescript-estree: disable single-run inference with extraFileExtensions (#9580)
website: expose ATA types to eslint instance (#9598)

❤  Thank You

Armano @ armano2
Josh Goldberg ✨
Kirk Waiblinger @ kirkwaiblinger
StyleShit @ StyleShit

You can read about our versioning strategy and releases on our website.

7.16.2-alpha.22 - 2024-07-22

7.16.2-alpha.21 - 2024-07-21

7.16.2-alpha.20 - 2024-07-21

7.16.2-alpha.19 - 2024-07-21

7.16.2-alpha.18 - 2024-07-20

7.16.2-alpha.17 - 2024-07-20

7.16.2-alpha.16 - 2024-07-20

7.16.2-alpha.15 - 2024-07-20

7.16.2-alpha.14 - 2024-07-20

7.16.2-alpha.13 - 2024-07-20

7.16.2-alpha.12 - 2024-07-20

7.16.2-alpha.11 - 2024-07-19

7.16.2-alpha.10 - 2024-07-18

7.16.2-alpha.9 - 2024-07-18

7.16.2-alpha.8 - 2024-07-18

7.16.2-alpha.7 - 2024-07-18

7.16.2-alpha.6 - 2024-07-17

7.16.2-alpha.5 - 2024-07-17

7.16.2-alpha.4 - 2024-07-16

7.16.2-alpha.3 - 2024-07-16

7.16.2-alpha.2 - 2024-07-16

7.16.2-alpha.1 - 2024-07-16

7.16.2-alpha.0 - 2024-07-15

7.16.1 - 2024-07-157.16.1 (2024-07-15)
🩹 Fixes

eslint-plugin: [no-unnecessary-type-parameters] descend into all parts of mapped types in no-unnecessary-type-parameters (#9530)

❤  Thank You

Dan Vanderkam @ danvk

You can read about our versioning strategy and releases on our website.

7.16.1-alpha.7 - 2024-07-14

7.16.1-alpha.6 - 2024-07-12

7.16.1-alpha.5 - 2024-07-12

7.16.1-alpha.4 - 2024-07-10

7.16.1-alpha.3 - 2024-07-10

7.16.1-alpha.2 - 2024-07-10

7.16.1-alpha.1 - 2024-07-09

7.16.1-alpha.0 - 2024-07-08

7.16.0 - 2024-07-087.16.0 (2024-07-08)
🚀 Features

ast-spec: add parent property to AccessorProperty node types (#9487)
eslint-plugin: [no-unnecessary-type-parameters] port from v8 to v7 (#9473)
eslint-plugin: [no-unnecessary-parameter-property-assignment] add new rule (#8903)
eslint-plugin: add support for nested namespaces to unsafe-member-access (#9478)
eslint-plugin: [no-floating-promises] add checkThenables option (#9263)
rule-tester: stricter rule test validations (#9086)

🩹 Fixes

deps: update dependency @ eslint-community/regexpp to v4.11.0 (#9497)
eslint-plugin: [no-floating-promises] add suggestions to tests from #9263 checkThenables (#9515, #9263)
eslint-plugin: [restrict-template-expressions] don't report tuples if allowArray option is enabled (#9492)
utils: context.parserPath may be undefined (#9486)
website: react key error on internal pages of website (#9506)

❤  Thank You

Abraham Guo
auvred @ auvred
Josh Goldberg ✨
Juan Sanchez @ JSanchezIO
Vinccool96
YeonJuan @ yeonjuan
Yukihiro Hasegawa @ y-hsgw

You can read about our versioning strategy and releases on our website.

7.15.1-alpha.13 - 2024-07-08

7.15.1-alpha.12 - 2024-07-08

7.15.1-alpha.11 - 2024-07-08

7.15.1-alpha.10 - 2024-07-06

7.15.1-alpha.9 - 2024-07-06

7.15.1-alpha.8 - 2024-07-05

7.15.1-alpha.7 - 2024-07-05

7.15.1-alpha.6 - 2024-07-05

7.15.1-alpha.5 - 2024-07-04

7.15.1-alpha.4 - 2024-07-04

7.15.1-alpha.3 - 2024-07-03

7.15.1-alpha.2 - 2024-07-03

7.15.1-alpha.1 - 2024-07-02

7.15.1-alpha.0 - 2024-07-01

7.15.0 - 2024-07-017.15.0 (2024-07-01)
🚀 Features

eslint-plugin: [array-type] detect Readonly<string[]> case (#8752)
eslint-plugin: back-port new rules around empty object types from v8 (#9443)

🩹 Fixes

disable EXPERIMENTAL_useProjectService in disabled-type-checked shared config (#9459)
eslint-plugin: [no-unsafe-return] differentiate a types-error any from a true any (#9254)
eslint-plugin: [no-unsafe-call] differentiate a types-error any from a true any (#9304)
utils: clean outdated RuleTester export (#9322)

❤  Thank You

auvred @ auvred
Kim Sang Du @ developer-bandi
rgehbt @ Gehbt
Vinccool96

You can read about our versioning strategy and releases on our website.

7.14.2-alpha.16 - 2024-07-01

7.14.2-alpha.15 - 2024-07-01

7.14.2-alpha.14 - 2024-07-01

7.14.2-alpha.13 - 2024-07-01

7.14.2-alpha.12 - 2024-06-30

7.14.2-alpha.11 - 2024-06-30

7.14.2-alpha.10 - 2024-06-30

7.14.2-alpha.9 - 2024-06-29

7.14.2-alpha.8 - 2024-06-29

7.14.2-alpha.7 - 2024-06-29

7.14.2-alpha.6 - 2024-06-28

7.14.2-alpha.5 - 2024-06-27

7.14.2-alpha.4 - 2024-06-26

7.14.2-alpha.3 - 2024-06-26

7.14.2-alpha.2 - 2024-06-25

7.14.2-alpha.1 - 2024-06-24

7.14.2-alpha.0 - 2024-06-24

7.14.1 - 2024-06-247.14.1 (2024-06-24)
🚀 Features

support TypeScript 5.5 (#9397)
ast-spec: tighter types and documentation for declaration/* (#9211)

🩹 Fixes

keep warnAbountTSVersion in sync with package.json (#9400)
eslint-plugin: [no-extraneous-class] handle abstract members (#9367)
eslint-plugin: [prefer-nullish-coalescing] handle intersected primitive types (#9378)
eslint-plugin: [no-invalid-this] support AccessorProperty (#9411)
eslint-plugin: [prefer-nullish-coalescing] treat enums and literals as their underlying primitive types (#9376)
eslint-plugin: [prefer-nullish-coalescing] ensure ternary fix does not remove parens (#9380)

❤  Thank You

Brad Zacher @ bradzacher
cm-ayf @ cm-ayf
Jake Bailey @ jakebailey
James Zhan
Joshua Chen
yoshi2no @ yoshi2no

You can read about our versioning strategy and releases on our website.

7.14.1-alpha.0 - 2024-06-24

7.14.0 - 2024-06-24

7.13.2-alpha.11 - 2024-06-24

7.13.2-alpha.10 - 2024-06-24

7.13.2-alpha.9 - 2024-06-23

7.13.2-alpha.8 - 2024-06-22

7.13.2-alpha.7 - 2024-06-22

7.13.2-alpha.6 - 2024-06-22

7.13.2-alpha.5 - 2024-06-22

7.13.2-alpha.4 - 2024-06-22

7.13.2-alpha.3 - 2024-06-22

7.13.2-alpha.2 - 2024-06-21

7.13.2-alpha.1 - 2024-06-17

7.13.2-alpha.0 - 2024-06-17

7.13.1 - 2024-06-17

7.13.1-alpha.6 - 2024-06-17

7.13.1-alpha.5 - 2024-06-17

7.13.1-alpha.4 - 2024-06-17

7.13.1-alpha.3 - 2024-06-17

7.13.1-alpha.2 - 2024-06-13

7.13.1-alpha.1 - 2024-06-11

7.13.1-alpha.0 - 2024-06-10

7.13.0 - 2024-06-10

    from @typescript-eslint/eslint-plugin GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

This update was applied in R88608 - MSCA-Dashboard v1.2.0


Fixed implicit conversion of email addresses

The implicit conversion of strings to instances of InternetAddress would not work as expected.

Sorry for the delay @AuspeXeu. I don't use courier in production so my opinion does not matters much but I usually prefer non-breaking changes. That's why I'm waiting on @dmurvihill feedback.

Somehow I missed this till today, sorry guys. I’m on vacation till next week, okay if this sits until then?

Thanks for submitting this change @AuspeXeu! I don't personally use Courier's implicit conversions, so I didn't realize they were broken.
Since the library was designed to support implicit conversion from String to InternetAddress I think we ought to stick to that choice.
One immediate change: I definitely don't think we should remove the addr.addr method in any 1.x release as that would be a breaking change; maybe we could deprecate it for removal in 2.0.

Thanks again for the contribution. @dmurvihill your call to merge it or not. If you don't can you close that.

I'm going to close this pull request because on closer inspection it looks like this pattern was never supported, and I agree with @daddykotex that adding supporting it would add more magic than it's worth. Moreover, the patch as written breaks code that I already have in production (you can see this if you merge master and try to run sbt test).
This is the recommended idiom:
mailer(Envelope()
<EMAIL_ADDRESS><EMAIL_ADDRESS>  // etc.
)

Thanks @AuspeXeu for the suggestion and @daddykotex for the nice test code!