[No QA] Upgrade Onyx to 1.0.24

Details
A number of PRs have been recently merged in Onyx, but are not yet being used in this repo. The most significant of those are three performance-related PRs:

https://github.com/Expensify/react-native-onyx/pull/186
https://github.com/Expensify/react-native-onyx/pull/188
https://github.com/Expensify/react-native-onyx/pull/191

This PR upgrades Onyx to take advantage of those fixes.
Fixed Issues
$ n/a (but related to https://github.com/Expensify/App/issues/10962 and WA Quality)
Tests
These improvements have already been benchmarked separately, so the main goal of testing here is just to make sure everything functions as it did before. Therefore, I'm going to recommend some simple regression tests for local testing:

Run the app and make sure it builds normally
Make sure you can sign out and back in without issues
Make sure you can create new chats

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

PR Review Checklist

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 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 included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] iOS / native
[x] Android / native
[x] iOS / Safari
[x] Android / Chrome
[x] MacOS / Chrome
[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 was added in all src/languages/* files
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team 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] 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 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] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

PR Reviewer Checklist
The reviewer will copy/paste it into a new comment and complete it after the author checklist is completed

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] 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)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] 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).
[ ] I verified that comments were added to code that is not self explanatory
[ ] 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.
[ ] I verified any copy / text shown in the product was added in all src/languages/* files
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by tagging the marketing team on the original GH to get the correct copy.
[ ] 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.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] 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
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] 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)
[ ] 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)
[ ] All JSX used for rendering exists in the render method
[ ] 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

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

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

[ ] 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)
[ ] 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.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

QA Steps
None, regression tests only.

[ ] Verify that no errors appear in the JS console

Screenshots

Web
https://user-images.githubusercontent.com/47436092/196308189-fbce7d21-3e03-4c31-aa62-4711db373898.mov
Mobile Web - Chrome
https://user-images.githubusercontent.com/47436092/196309301-93c9bb90-1390-4427-a571-d75683f61e2a.mp4
Mobile Web - Safari
https://user-images.githubusercontent.com/47436092/196309037-8a2b3059-bd26-4942-9389-be0c77cc0141.MP4
Desktop
https://user-images.githubusercontent.com/47436092/196308410-4c3f1d83-70b9-45bf-8d5e-c25168f3614a.mov
iOS
https://user-images.githubusercontent.com/47436092/196309500-2fbb90a4-7dac-41b9-b2e7-2947890aa389.MP4
Android
https://user-images.githubusercontent.com/47436092/196309462-6a389dee-b1c7-4319-bb47-a03393513dc3.mp4

All yours @Gonals

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

The checks seemed fine when I merged

🚀 Deployed to staging by @Gonals in version: 1.2.18-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by @Gonals in version: 1.2.18-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @chiragsalian in version: 1.2.18-10 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Compact Labels in Recommendations Table

Added props isCompact to the labels in Recommendations List Table
To update the label total risk in the expanded rule we need to create a separate patch for the rhel advisor
Asked Martin about  props and he told me that there is a bug so currently, we need to pass rest={{ isCompact: true }} instead isCompact="true" to get the result we need.

@gkarat After discussion with Marie Doruskova it became clear that RHEL Advisor team will fix styling on their end. She advised us to merge this patch as it is and wait for them to change labels for https://github.com/RedHatInsights/insights-advisor-frontend/blob/master/src/PresentationalComponents/RuleDetails/RuleDetails.js

@gkarat PTAL


Remove function count stats

#1591
Decided to rip out the function count section from the header of the OU page.  It doesn't make too much sense in the new 'measurable' world.

@kamransaleem pls review

ta


Servertime Chat Command

Added servertime chat command that shows the current server time.

Besides that though, I am in favor of this command being added. 👍

What probably went wrong in this rebase: you rebased whatever branch you used to create this PR with our master production branch, which then merged all commits from the time you made that clean working branch to  when you updated, and it rebased them all into your commit.
How to probably prevent this from happening again: if you update a pull request, and you want to rebase/squash it into one commit (which, by the way, is optional now), then you should do so by adding a new commit to change/alter whatever you want the second time, and then rebasing them.
Again, if you need help, please do not hesitate to PM me directly on PS with any questions.


Support Sass 3.5

This is not the PR we deserve, but it's the one we need right now.
Sass 3.5 is supported, custom properties aren't: all specs are green (included the new one which deals with custom properties), and it does not fail against Bootstrap v4
Also:

Update development gems
Test against latest Ruby versions
Fix all new (0.50.0) RuboCop offences

Fix #877

👏 🙇

@sds
I want to submit a new PR that:

Move development dependencies to the proper place (gemspec). Take a look here: having them in the Gemfile lists them as runtime dependencies
Drop Ruby 2.0 support. This doesn't mean that Ruby 2.0 will not work anymore, but Ruby 2.1 will be required to develop the gem and we will exclude tests on Ruby 2.0, so we can also use RuboCop 0.51.0
Use coveralls_reborn, my (temporary I hope) fork of coveralls, which seems unmaintained.

What do you think?

Hey @tagliala, thanks for offering to move forward on some of these issues.

The reason we explicitly declared the RuboCop gem in the Gemfile was to avoid a warning about duplicate dependencies. See the commit message in 98655a735 for details. At its core, if another project includes SCSS-Lint as a development dependency, we don't want to force them to use the RuboCop version specified in the gemspec, since it's pinned to a specific version. If this isn't a practical issue, I'm open to suggestions here (is this behavior no longer the case?)

I'm fine with dropping Ruby 2.0 support, as it's been EOLed since February 2016. I know your suggestion is to simply remove it from the Travis build matrix, but at that point we're not ensuring we support it, so I'd rather just drop support.

Is there a summary of what exactly has changed with coveralls_reborn?

Thanks!

Hi,

Got it, 👍
Perfect, will do a separate PR just for this
It's not a big deal. It comes with updated development dependencies (mainly SimpleCov). I've submitted a couple of PRs in their repo three months ago, but it seems kinda unmaintained.
Ref: diff, comment

Thanks for linking to the diff for coveralls_reborn. The update to SimpleCov doesn't seem like it would benefit the SCSS-Lint project just yet (but if I'm missing something let me know) as none of the changes seemed to be something that would affect this project.
If you find yourself down the road having an issue with the coverage information reported by SimpleCov, then I would be open to merging coveralls_reborn. My hope is that your PR eventually gets merged into the main project soon.

Perfect, will do a separate PR just for this (after RuboCop releases a new version, because it has a lot of issues right now)

RuboCop released a new version, but it has a lot of false positives, so I will keep waiting.
Moreover, Sass 3.5.4 has an issue (fixed in the stable branch) and a lot of tests with this gem fail  sass/sass#2423, so I will wait for a new release of Sass, too


Any JPG returns "It is a dog !"

In passing test_dog.jpg or test_cat.jpg, both return "Its a dog!".   Maybe a problem on my end?  Im running train.py first and then predict.py
thanks

Can you share training outputs?

This is the output I'm getting.
(Sandbox) C:\WatsonWorkspace\Dog-Cat-Classifier-master>python train.py
Using TensorFlow backend.
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\tensorflow\python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\keras\backend\tensorflow_backend.py:3445: calling dropout (from tensorflow.python.ops.nn_ops) with keep_prob is deprecated and will be removed in a future version.
Instructions for updating:
Please use rate instead of keep_prob. Rate should be set to rate = 1 - keep_prob.
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\tensorflow\python\ops\math_ops.py:3066: to_int32 (from tensorflow.python.ops.math_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Use tf.cast instead.
2019-07-23 07:05:46.717814: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2019-07-23 07:05:46.726499: I tensorflow/core/common_runtime/process_util.cc:71] Creating new thread pool with default inter op setting: 8. Tune using inter_op_parallelism_threads for best performance.
Epoch 1/25
157/157 [==============================] - 5s 31ms/step - loss: 0.7005 - acc: 0.4936 - val_loss: 0.6928 - val_acc: 0.5143
Epoch 2/25
157/157 [==============================] - 4s 27ms/step - loss: 0.6993 - acc: 0.5082 - val_loss: 0.6931 - val_acc: 0.4571
Epoch 3/25
157/157 [==============================] - 4s 25ms/step - loss: 0.6910 - acc: 0.5478 - val_loss: 0.6816 - val_acc: 0.6143
Epoch 4/25
157/157 [==============================] - 4s 27ms/step - loss: 0.6846 - acc: 0.5690 - val_loss: 0.6977 - val_acc: 0.4929
Epoch 5/25
157/157 [==============================] - 4s 25ms/step - loss: 0.6568 - acc: 0.6277 - val_loss: 0.6237 - val_acc: 0.6071
Epoch 6/25
157/157 [==============================] - 4s 24ms/step - loss: 0.6655 - acc: 0.6064 - val_loss: 0.6444 - val_acc: 0.6000
Epoch 7/25
157/157 [==============================] - 4s 23ms/step - loss: 0.6530 - acc: 0.6473 - val_loss: 0.6379 - val_acc: 0.5714
Epoch 8/25
157/157 [==============================] - 3s 22ms/step - loss: 0.6123 - acc: 0.6608 - val_loss: 0.5783 - val_acc: 0.7071
Epoch 9/25
157/157 [==============================] - 3s 20ms/step - loss: 0.6155 - acc: 0.6646 - val_loss: 0.7093 - val_acc: 0.5286
Epoch 10/25
157/157 [==============================] - 3s 21ms/step - loss: 0.6033 - acc: 0.6738 - val_loss: 0.5198 - val_acc: 0.7429
Epoch 11/25
157/157 [==============================] - 3s 19ms/step - loss: 0.5811 - acc: 0.7017 - val_loss: 0.5091 - val_acc: 0.8000
Epoch 12/25
157/157 [==============================] - 4s 24ms/step - loss: 0.5637 - acc: 0.7033 - val_loss: 0.5144 - val_acc: 0.7500
Epoch 13/25
157/157 [==============================] - 3s 20ms/step - loss: 0.5667 - acc: 0.6921 - val_loss: 0.5312 - val_acc: 0.7143
Epoch 14/25
157/157 [==============================] - 3s 19ms/step - loss: 0.5703 - acc: 0.7025 - val_loss: 0.5330 - val_acc: 0.7357
Epoch 15/25
157/157 [==============================] - 4s 24ms/step - loss: 0.5250 - acc: 0.7418 - val_loss: 0.4624 - val_acc: 0.7643
Epoch 16/25
157/157 [==============================] - 3s 21ms/step - loss: 0.5406 - acc: 0.7322 - val_loss: 0.4992 - val_acc: 0.7643
Epoch 17/25
157/157 [==============================] - 3s 20ms/step - loss: 0.5238 - acc: 0.7264 - val_loss: 0.5255 - val_acc: 0.7643
Epoch 18/25
157/157 [==============================] - 3s 19ms/step - loss: 0.5182 - acc: 0.7338 - val_loss: 0.4222 - val_acc: 0.7857
Epoch 19/25
157/157 [==============================] - 3s 20ms/step - loss: 0.5308 - acc: 0.7290 - val_loss: 0.4632 - val_acc: 0.7857
Epoch 20/25
157/157 [==============================] - 3s 18ms/step - loss: 0.4965 - acc: 0.7460 - val_loss: 0.4493 - val_acc: 0.7929
Epoch 21/25
157/157 [==============================] - 3s 19ms/step - loss: 0.5071 - acc: 0.7479 - val_loss: 0.4314 - val_acc: 0.8214
Epoch 22/25
157/157 [==============================] - 4s 23ms/step - loss: 0.4826 - acc: 0.7763 - val_loss: 0.4818 - val_acc: 0.8000
Epoch 23/25
157/157 [==============================] - 3s 21ms/step - loss: 0.4858 - acc: 0.7556 - val_loss: 0.4664 - val_acc: 0.7929
Epoch 24/25
157/157 [==============================] - 3s 17ms/step - loss: 0.4902 - acc: 0.7548 - val_loss: 0.4056 - val_acc: 0.8214
Epoch 25/25
157/157 [==============================] - 3s 19ms/step - loss: 0.4848 - acc: 0.7641 - val_loss: 0.4354 - val_acc: 0.8357
Model and weights saved
(Sandbox) C:\WatsonWorkspace\Dog-Cat-Classifier-master>python predict.py test_cat.jpg
Using TensorFlow backend.
image= test_cat.jpg
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\tensorflow\python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\keras\backend\tensorflow_backend.py:3445: calling dropout (from tensorflow.python.ops.nn_ops) with keep_prob is deprecated and will be removed in a future version.
Instructions for updating:
Please use rate instead of keep_prob. Rate should be set to rate = 1 - keep_prob.
2019-07-23 07:13:41.817051: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2019-07-23 07:13:41.827181: I tensorflow/core/common_runtime/process_util.cc:71] Creating new thread pool with default inter op setting: 8. Tune using inter_op_parallelism_threads for best performance.

(Sandbox) C:\WatsonWorkspace\Dog-Cat-Classifier-master>python predict.py test_dog.jpg
Using TensorFlow backend.
image= test_dog.jpg
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\tensorflow\python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
WARNING:tensorflow:From C:\ProgramData\Anaconda3\envs\Sandbox\lib\site-packages\keras\backend\tensorflow_backend.py:3445: calling dropout (from tensorflow.python.ops.nn_ops) with keep_prob is deprecated and will be removed in a future version.
Instructions for updating:
Please use rate instead of keep_prob. Rate should be set to rate = 1 - keep_prob.
2019-07-23 07:13:54.682680: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX AVX2
2019-07-23 07:13:54.692240: I tensorflow/core/common_runtime/process_util.cc:71] Creating new thread pool with default inter op setting: 8. Tune using inter_op_parallelism_threads for best performance.

(Sandbox) C:\WatsonWorkspace\Dog-Cat-Classifier-master>

You have got 83% val acc in last epoch. That is not bad.
You can try again with other cat photos from Google Images.
I will be so happy, If you share your next experiences with us.

Maybe I'm misunderstanding.  So the model is 83% accurate, it just so happens the pictures I'm supplying are predicting dogs?
I supplied these cats at http://expertofdreams.com/data_images/cat/
they predict dogs.  Does your system correctly predict them?
Thanks in advance,

Can you share more information? I don't know what is the problem now but we can fix it together.

After training the model, any picture of a cat (python predict.py cat.jpg) returns "It is a dog !".  I have tried numerous pictures without success.  I'm not sure if the predictions are wrong by coincidence or if I have some other problem.
Thanks in advance.

What is the output possibilities of classes?

I am also having this problem

@michaelyan-coupa Can you give me detailed information about the problem so that I can find a solution?

Basically the loss and val loss are both very low (so very good) however when test.py is run, it only returns cat.

@kggraham just gets "Dog" results.
@michaelyan-coupa Can you help me for debugging your problem ?

Sure. I am unsure where the root of the problem is. When I try to train on a different set, it works a little better.

Would you like to share your experiences with us?

@michaelyan-coupa  I have the same problem,
Every outputs show "It is a dog" after running train.py
Would you find the method to solve it?


Update to 1.12 (almost fully working)

Couple bugs that I've unfortunately not had the time to work out yet, but I'm going to investigate a little more soon. Figured I would start the pull request anyway since some of the updates are pretty straightforward.

80408db541145702c5f7bdf937620067967e4f62


Added function to set Time with Foundation.Date

Can you explain what the second commit fixes?

I'm trying to add my component through code but the separator are visible even if in setupTableView there is tableView.separatorStyle = .none . allora ho cercato la motivazione su internet and I found a post on stackOverflow (https://stackoverflow.com/questions/31429147/uitableview-separator-not-hiding-for-ios9) that says to set separator in layoutSubviews. I tried and now work in both cases, if you add the component by storyboard and if you add it by code.

Great! Thanks for this PR


[AMP] AMP markup missing or incorrect (Critical issue)

Fix missing or invalid AMP markup on your page. AMP pages with missing required code or invalid AMP syntax may appear in Google search results without AMP-specific display features.
List of pages: https://drive.google.com/file/d/1O7b3GyuONyZqwgojKharjHpaQqRF2-DH/view?usp=sharing
Please mention me once the issue is resolved so I can get back to the customer

@Poliuk This issue has been solved but the costumer will have to update our plugin. @luisherranz should let you know when it's released.

@orballo We should check if the other issues (#60 #61 #62) are related. So we just release one version that fixes all the problems :) Thanks!

:tada: This issue has been resolved in version 0.7.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 1.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improve highlighting of repainted areas

Is your feature request related to a problem? Please describe.
https://github.com/lc-soft/LCUI/blob/4a9516e8eb3bc23b681575653ac31468093d8a2c/src/display.c#L173-L184
Describe the solution you'd like
Refer to the paint flashing of Google Chrome:

Describe alternatives you've considered
None.
Additional context
None.

IssueHunt Summary

Backers (Total: $5.00)

 lc-soft ($5.00)

Become a backer now!
Or submit a pull request to get the deposits!
Tips

Checkout the Issuehunt explorer to discover more funded issues.
Need some help from other developers? Add your repositories on IssueHunt to raise funds.

IssueHunt has been backed by the following sponsors. Become a sponsor

hi, @lc-soft can you describe further what you mean by "improve" the highlighting, can you show some examples?

@lc-soft do you have any design/style in mind?

@anshulxyz This is a low priority task and I will update the description as soon as possible.

Alright :pray:

@anshulxyz The issue description has been updated. Maybe the implementation of this feature is a bit complicated.


Question: Is putResolve typed correctly?

Firstly, massive thanks for this library. As I start my journey into the saga wildnerness, being able to trust the types I get back is a HUGE win. Really appreciate this library.
I'm also using redux-saga-promise-actions so that I can await sagas from within other sagas. In my adventures there, I've started to suspect that the type definitions for put and putResolve should be different, but they appear to be the same.
https://github.com/agiledigital/typed-redux-saga/blob/bc524857403989fc518e00e5d369f899fd58594e/types/index.d.ts#L175-L185
In VSCode when I try const result = yield* putResolve(action()) I get the type of result as the return type of action. The same for put() instead of putResolve(). But putResolve() should be the return type of store.dispatch(action()) I think.
I've tried playing around with the types to see if I can figure out the fix, but unfortunately I'm stumped, my TypeScript fu is insufficient to the task.

It seems like in my tests putResolve() from typed-redux-saga doesn't behave the same as putResolve() from redux-saga/effects when used like:
const result = yield* putResolve(action())

When I switch to the redux-saga putResolve() then result is populated with the correct value, otherwise its undefined. Maybe it's something else in my config that's causing the issue. I can't figure it out, figured I'd mention just in case.


bug: cordova camera plugin gallery image upload duplicate

Bug Report
Ionic version:

[x] 4.x
Current behavior:

Expected behavior:

Steps to reproduce:

Related code:

insert short code snippets here

Other information:

Ionic info:

insert the output from ionic info here

Ionic version:
4.12.0
Current behavior:
My app allows users to select 2 images from gallery / camera using navigator.camera.getPicture, then retrieves the Camera.DestinationType.FILE_URI, which is later used for uploading via the FileTransfer plugin, sending the file to server in sequence. Now, the weird problem is the 2 photos uploaded by our Android app users ended up with BOTH being the second photo. That means, after user selected photo 1 and 2, and uploaded both, the server end received the images as both being photo 2. Please advise, thanks!
Expected behavior:
multiple images uploaded by cordova camera plugin and file transfer plugin should not override with one another. It should separate.
Related code:
async upload(sourceType: PictureSourceType) {
let options: CameraOptions = {
  quality: 50,
  sourceType: sourceType,
  saveToPhotoAlbum: true,
  correctOrientation: true,
  destinationType: this.camera.DestinationType.FILE_URI,
  encodingType: this.camera.EncodingType.JPEG,
  mediaType: this.camera.MediaType.PICTURE,
  allowEdit: true
};

//clicking image or getting image from gallery.
await this.camera.getPicture(options).then((imageData) => {

  this.album.push(imageData);
  this.dataService.setData(this.album);

  if (this.plt.is('android') && sourceType === this.camera.PictureSourceType.PHOTOLIBRARY) {
    this.filePath.resolveNativePath(imageData)
      .then(filePath => {
        let correctPath = filePath.substr(0, filePath.lastIndexOf('/') + 1);
        let currentName = imageData.substring(imageData.lastIndexOf('/') + 1, imageData.lastIndexOf('?'));
        this.copyFileToLocalDir(correctPath, currentName, this.createFileName());
      });
  } else {
    var currentName = imageData.substr(imageData.lastIndexOf('/') + 1);
    var correctPath = imageData.substr(0, imageData.lastIndexOf('/') + 1);
    this.copyFileToLocalDir(correctPath, currentName, this.createFileName());
  }
});

}
Other information:
Ionic info:
Ionic:
ionic (Ionic CLI)             : 4.12.0 (/usr/local/lib/node_modules/ionic)
Ionic Framework               : @ionic/angular 4.4.2
@angular-devkit/build-angular : 0.13.9
@angular-devkit/schematics    : 7.2.4
@angular/cli                  : 7.3.9
@ionic/angular-toolkit        : 1.4.1
Cordova:
cordova (Cordova CLI) : 9.0.0<EMAIL_ADDRESS>Cordova Platforms     : android 8.0.0
Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.3, cordova-plugin-ionic-webview 3.1.2, (and 14 other plugins)
System:
Android SDK Tools : 26.1.1 (/home/ttpl34/Documents/Android-sdk/)
NodeJS            : v10.15.3 (/usr/bin/node)
npm               : 6.9.0
OS                : Linux 4.15
insert the output from ionic info here

Thanks for the issue. Ionic Framework does not manage the camera plugin. I would recommend filing a bug on the cordova-plugin-camera GitHub instead.
I am going to close this as this is not an Ionic issue. Thanks!


Update Gradle to 3.4

Codecov Report

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

@@            Coverage Diff             @@
##             master      #87    +/-   ##
==========================================
  Coverage     95.11%   95.11%
- Complexity        0      203   +203
==========================================
  Files            15       15
  Lines           676      676
  Branches        151      134    -17
==========================================
  Hits            643      643
  Misses           14       14
  Partials         19       19

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 89797bb...872e229. Read the comment docs.


Verify on server side

public bool Verify([FromBody] List<string> datas)
        {
            var value = JsonConvert.DeserializeObject<List<int>>(datas[0]);
            var sum = value.Sum();
            var avg = sum * 1.0 / value.Count;
            var stddev = value.Select(v => Math.Pow(v - avg, 2)).Sum() / value.Count;
            return stddev != 0;
        }

When slide image successful or unsuccessful, always return the number greater than 0

I confirm, same with javascript inside  _proto.verify = function ()
        var sum = function (x, y) { return x + y; };
        var square = function (x) { return x * x; };
        var average = arr.reduce(sum) / arr.length;
        var deviations = arr.map(function (x) { return x - average; });
        var stddev = Math.sqrt(deviations.map(square).reduce(sum) / arr.length);
        verified = stddev !== 0

"verisfied" is always TRUE.
Only javascript "spliced: Math.abs(left - this.x) < this.options.offset" is usefull to confirm valid captcha.
That's a nice captcha but need to be secure with real server side feature.


Ability to expand all collapsed regions

In the past it was nice to just scroll and read about all possible settings.
Right now they are all collapsed and one need to click a lot of boxes to open them.
Right click to see the option to expand all would be very useful

Use Unfold all (Ctrl+K Ctrl+J) to unfold all.

@wallverb Reason to collapse all is to show all available sections of configurations. Given the number of configurations, one can seldom browse through all the configurations. I think Find is the only option to look for something in the default settings. Given everything is collapse, it might let user to navigate sections easily than before.
Any other ideas are much welcome.
Thanks

@sandy081 That's a great reason/idea.
What I was only saying that it would be great if there was a right click here:

(on + itself or collapsed region) to show the Unfold All / Fold All option - this would be also helpful in other files as well (not only vscode settings).
I will be ok now knowing there is 'Unfold All / Fold All` command / keyboard shortcut. Thanks

FYI @aeschli


"join google group" not working?

The "Join our Google Group" link on the READNE.md file leads me to a page that says "content unavailable"

The link is currently redirected to this URL:
https://groups.google.com/g/free-our-knowledge-community/
Is there a new join link that would work?

I accidentally changed the google group name, should be fixed now


run_tests.py --auto_set_flakes should be disabled by default

Currently the default behavior of run_tests.py is to fetch flaky tests by default. We should change it to opt-in because:

flake detection requires extra dependencies (that are not well documented). External users are struggling with this (e.g. see https://github.com/grpc/grpc/issues/15100) and we should make first-time compilation of grpc as easy as possible for external developers.
it makes more sense for add-on options to be opt-in in general, otherwise this can lead to confusion (e.g. currently things are a little backward: we explicitly disable auto_set_flakes on master tests and we don't set any flag for PR tests).
our flakiness level should be in a good shape to not require this feature on pull requests. If that turns out not to be true, we can explicitly enable --auto_set_flakes on pull request jobs.
the option adds extra lines run_test.py output.

Fixes https://github.com/grpc/grpc/issues/15100

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

[microbenchmarks] No significant performance differences

@ncteisen  to answer your questions:

currently we are only using --auto_set_flakes (by not disabling it) on PR tests and because our tests have recently become much less flaky, we might get away with not setting this flag anymore.
we can reintroduce --auto_set_flakes on PRs if needed.
monitoring flakiness is an unrelated topic, we will continue uploading results to BQ on kokoro.


Add script probe type

Monika Pull Request (PR)
What feature/issue does this PR add

Feature: Adds a script probe type that will execute a local script for slightly more complex probes. The probe status will be the return value of the script (0 == "success").
Fix: The prometheus feature was breaking for non-request probes. (cannot destructure request in collector.ts)

How did you implement / how did you fix it

Using the postgres probe as an example, I created a new script probe.
Wrote unit tests for the probe.
Added documentation.

How to test
monika.yml
probes:
  - id: '1'
    name: 'Script tests'
    script:
      - cmd: 'echo Hello World'
      - cmd: exit 123
      - cmd: sh probe.sh

probe.sh
#!/bin/sh
echo "Probing things..."
exit 0

Hello @jgillick sorry for responding late for this PR, we're still discussing and trying this internally. I tried this PR but I have a question:
When a probe has multiple cmds, say 3 commands, and the second command's exit is not 0, any reasons why the third command is still executed? The behaviour of monika when a HTTP probe has multiple requests, the requests are chained, and if one fails, the subsequent requests won't be executed.

Hi @nicnocquee. That's a great point. I can't see any reason they would continue to execute if one of them fails. I can update the PR sometime in the next week.

@nicnocquee I've updated the PR to stop executing scripts in a probe when one fails.

@sapiderman I have updated the PR for the JSON schema changes and synced upstream.

Hey @jgillick apologies for the delayed response. We've been swamped with fixing implementations and integrating Monika with our SaaS. Unfortunately, we're closing this PR because we Monika is designed as a monitoring tool, and executing arbitrary scripts falls outside of its intended use. Thanks for your contribution, though. If the script execution feature is crucial for you, feel free to fork the project.


FreeBSD XPath problem

I have a problem with using find/get methods even in simplest cases. It just returns "undefined" results. Even example test case doesn't work. I'm using FreeBSD 8.1, libxmljs 0.4.2, node 0.4.8 and libxml 2.7.8.

Can you please try against the latest 'develop' branch and see if there are problems? I believe all the issues discussed here have been resolved.


Choose specific VideoCaptureFormat

I have a camera which supports two almost similar VideoCaptureFormats - one is raw (Y16) and one is YUYV. I'm using the API and I can't find a method to specifically choose the raw source on this device. It gets listed with GetCaptureFormatsAsync(source.id), but I can't seem to set it when I later call DeviceVideoTrackSource.CreateAsync.
I've tried various ways of specifying the VideoCaptureFormat. Currently, my code looks like this:
var formats = await DeviceVideoTrackSource.GetCaptureFormatsAsync(source.id);
foreach (var format in formats)
{
    Debug.Log($"Supported format: {format.fourcc} with framerate {format.framerate} and {format.width} x {format.height}");
}
videoTrackSource = await DeviceVideoTrackSource.CreateAsync(new LocalVideoDeviceInitConfig()
{
    videoDevice = source,
    enableMrc = false
});
Debug.Log($"Chosen frame encoding: {videoTrackSource.FrameEncoding}");

with outputs:
Supported format:<PHONE_NUMBER> with framerate 9 and 160 x 120
Supported format:<PHONE_NUMBER> with framerate 9 and 160 x 120
Chosen frame encoding: I420A

So, how do I create the videoTrackSource with the raw format (Y16)?
(As you can see, the VideoCaptureConstraints are insufficient to distinguish between the two. And GetCaptureProfilesAsync is empty for this device.)
Thanks,
Roman

I am not 100% sure, but go to WebcamSource component, select manual CaptureFormat tick constraint width, height and framerate, write in your values and afterwards write the number of your format into the videoprofileid-constraint. Let me know if it works, otherways, maybe try all possible videoProfileKinds after each other and eventually one of them is yours. Notice that you should only constraint videoProfileKind or VideoProfileID.

Thanks for your reply @SilverLive
Unfortunately, GetCaptureProfilesAsync is empty for this device. I interpret this as "this webcam does not support video profiles".
Also, what I'm trying to set is the VideoCaptureFormat, not the VideoProfile. Those are two different things.
Also, since both streams have the same width, height and framerate I can't use those as constraints to select a specific one.


Collection support

Hello!
I consider this library very useful, thanks for that ❤
After review example, I implemented a loop in my code for sort an input array of colors: https://github.com/microlinkhq/splashy/blob/master/test/index.js#L10
Maybe could be useful to leverage this functionality as part of the library?

Hi,
Glad you liked it!
I have been meaning to update it with some better functionality - but have
too much to do irl. Will see if I can implement something this xmas ^^
Thanks for your support!

Dag

Den tir. 11. des. 2018 kl. 11:54 skrev Kiko Beats<EMAIL_ADDRESS>
:

Hello!
I consider this library very useful, thanks for that ❤
After review example, I implemented a loop in my code for sort an input
array of colors:
https://github.com/microlinkhq/splashy/blob/master/test/index.js#L10
Maybe could be useful to leverage this functionality as part of the
library?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dagthomas/hexSorter/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/AEHKeM4XVx0DXE8XYIap-VWUTmNKOcw7ks5u347egaJpZM4ZNJaF
.


vgg_16.ckpt

Hi, could you provide vgg_16.ckpt, it seems need be downloaded
thank you

Hi, I downloaded you VGG-16 and ran you code, but python reported error that the weight and bias of inverse net missiing in your VGG-16, could you help me?
thank you

@yanx27 You need to train the inverse net first before you transfer the style. python main.py -c config/example.json

can you expand on training the inverse net? do i need these?
"record_file_name": "e:\Github\fast-style-transfer\datasets\image_train_.tfrecord",
"Image_files": "e:\Github\style-swap\images\style\.jpg",

Where *.tfrecord comes from?
@ benbenlijie


Update overview.md

Closing this in favor of https://github.com/MicrosoftDocs/powerapps-docs-pr/pull/7912


Experiment with nix as a repeatable cross-platform building tool

Again and again and again, we have problems with project building on a different platform. One of the perspectives and popular ways to do this in the Haskell community is nix (https://nixos.org). We need to try.

The main idea of this experiment: provide to developers with a simple command for preparing the development environment.
In theory:

A developer install nixos.org
Type: nix-shell
Work.

In practice the first attempt on os x finish with the following result:
{
  pkgs ? import (fetchTarball https://nixos.org/channels/nixos-unstable/nixexprs.tar.xz) {}
}:

pkgs.mkShell {
#  CPATH = "${darwin.Libsystem}/include";

  buildInputs = [

    (pkgs.python39.withPackages (ps: with ps; with pkgs.python39Packages; [
      tensorflow
    ]))
  ];

  # Automatically run jupyter when entering the shell.
  shellHook = "jupyter notebook";
}

$ nix-shell
unpacking 'https://nixos.org/channels/nixos-unstable/nixexprs.tar.xz'...
these derivations will be built:
  /nix/store/6lxdi3gll0v34njnk1jwm9kjjznsga55-tensorflow-deps-merged.drv
  /nix/store/mr6bp3vwsfjal2ckmap0inavqw9axznm-builder.pl.drv
  /nix/store/yxb6ddd3zkbhf7spisn7g9kzb2acqzzz-python3-3.9.6-env.drv
  /nix/store/xfjxvg0p6xp8zrww0f7pj9m0lpiab4jl-tensorflow-2.4.2.drv
  /nix/store/wr77862fhqx2c16yp20hvk0jdq2818ca-python3.9-tensorflow-2.4.2.drv
  /nix/store/wm86a2l7m6h929wqd999aqlbljb5jjwh-python3-3.9.6-env.drv
these paths will be fetched (1124.70 MiB download, 3115.23 MiB unpacked):
  /nix/store/00ngdc8ay9srb7ckm99kx6ims2siwqgq-apple-framework-CFNetwork
  /nix/store/00vcq1i9x1dwwn21dqkagybcxa1lcnwi-xz-5.2.5-bin
  /nix/store/00zpf7n02qndqbvqvnqfighy3q4yn5wd-protobuf-3.17.3
  /nix/store/03zk2c1zh3jfi56m407mgyfpdm8s4r30-bazel-3.7.2
  /nix/store/05zmnmy39zp5j3prwfhiy84zv98c7v1d-python3.9-chardet-4.0.0
  /nix/store/07z7bvmzfzkcjww57n5rgz2c949lncjk-python3.9-opt_einsum-3.3.0
  /nix/store/0b99wnpwzhdhdrin7y1jfnpfhygxq91x-hook
  /nix/store/0gycsq6xf46d77bafk16vj6qkawj5gbv-libjpeg-turbo-2.1.0-bin
  /nix/store/0idd3d3y4vbiilwzhjq1my4npac6bp3r-libxml2-2.9.12
  /nix/store/0j6i4hjvs3c2yr9ymn42409q155zbvkj-hook
  /nix/store/0lglqsh5z3vv9r4hi1fgdqw7nngi8zpb-python3.9-pbr-5.6.0
  /nix/store/0m3krl0s796idzgn68rpbrwlg1wq9cl3-brotli-1.0.9-dev
  /nix/store/0yrrpxkc6l1kvdf491qa1m7scamsmqb7-libcxxabi-7.1.0
  /nix/store/11gpk9yrv6az1zy4p6wzx21bd21cn5lw-bazel-rc
  /nix/store/156rmc1d7m0w0awhdn8jky03i1nyf7rh-perl5.34.0-Test-Needs-0.002006
  /nix/store/1899zapiyri6fihqgw6vj6pms2s660zp-lapack-3-dev
  /nix/store/1ahb54by7x5h7yn8703nq25fiwdzg4ix-nghttp2-1.43.0-lib
  /nix/store/1g105mph26hggq6xpcnygdxg2b4rrv77-nghttp2-1.43.0-bin
  /nix/store/1h653vqxkd69ylzgxbdrj4sx7idki7g5-apple-framework-CoreVideo
  /nix/store/1hh0llvgrp1kfnvq0fxmv75i2pmv7x59-python3.9-idna-3.2
  /nix/store/1phiyhyg1b4sk6glc0a2j8wwlg9bm5ca-python3.9-pyopenssl-20.0.1
  /nix/store/1vnni9z5wc1s7h58vavs93qm53clpfc8-libxcb-1.14
  /nix/store/1xjwb0bd02n1y621mjqani68kh6n6gig-perl-5.34.0
  /nix/store/21h26pzvm6rj6g0ia65lvh2m7dd02qkm-python3.9-absl-py-0.13.0
  /nix/store/23gw3vdxi9r0ivnlj9q6ml18gra994qg-python3.9-urllib3-1.26.6
  /nix/store/23slacyrhblpgyfhaqk3nbrzvh3hx2x1-python3.9-packaging-20.9
....
[0 / 348] [Prepa] BazelWorkspaceStatusAction stable-status.txt
[31 / 506] 8 actions, 7 running
    @com_google_protobuf//:protoc; 0s local
    @com_google_protobuf//:protoc_lib; 0s local
    @com_google_protobuf//:protoc_lib; 0s local
    @com_google_protobuf//:protoc; 0s local
    @com_google_protobuf//:protoc_lib; 0s local
    @com_google_protobuf//:protoc_lib; 0s local
    @com_google_protobuf//:protoc_lib; 0s local
    [Prepa] //tensorflow/python/tools:freeze_graph
ERROR: /private/tmp/nix-build-tensorflow-2.4.2.drv-0/output/external/com_google_protobuf/BUILD:301:11: C++ compilation of rule '@com_google_protobuf//:protoc_lib' failed (Exit 1): cc_wrapper.sh failed: error executing command external/local_config_cc/cc_wrapper.sh -U_FORTIFY_SOURCE -fstack-protector -Wall -Wthread-safety -Wself-assign -fcolor-diagnostics -fno-omit-frame-pointer -g0 -O2 '-D_FORTIFY_SOURCE=1' -DNDEBUG ... (remaining 38 argument(s) skipped)
[38 / 506] 2 actions running
    @com_google_protobuf//:protoc_lib; 0s local
    //tensorflow/python/tools:freeze_graph; 0s local
warning: unknown warning option '-Wno-stringop-truncation'; did you mean '-Wno-string-conversion'? [-Wunknown-warning-option]
In file included from external/com_google_protobuf/src/google/protobuf/compiler/code_generator.cc:35:
external/com_google_protobuf/src/google/protobuf/compiler/code_generator.h:41:10: fatal error: 'string' file not found
#include <string>
         ^~~~~~~~
1 warning and 1 error generated.
[40 / 506] checking cached actions
INFO: Elapsed time: 39.316s, Critical Path: 0.42s
[43 / 506] checking cached actions
INFO: 43 processes: 42 internal, 1 local.
[43 / 506] checking cached actions
FAILED: Build did NOT complete successfully
FAILED: Build did NOT complete successfully

builder for '/nix/store/xfjxvg0p6xp8zrww0f7pj9m0lpiab4jl-tensorflow-2.4.2.drv' failed with exit code 1
cannot build derivation '/nix/store/wr77862fhqx2c16yp20hvk0jdq2818ca-python3.9-tensorflow-2.4.2.drv': 1 dependencies couldn't be built
cannot build derivation '/nix/store/wm86a2l7m6h929wqd999aqlbljb5jjwh-python3-3.9.6-env.drv': 1 dependencies couldn't be built
error: build of '/nix/store/wm86a2l7m6h929wqd999aqlbljb5jjwh-python3-3.9.6-env.drv' failed

But it works on the Linux installation.
Conclusion: nix is a very nice idea and I think works well in some cases, but not in ours. Maybe, it can be simply fixed (like problems with npm or yarn), but that just moves the problem from one place to another and adds new not common technology in the project. So I decide to reject nix at the current stage.
But the problem stays and the solution should be solved. Any ideas except naive docker application and virtual machines?


To add a calculator for calculating circumradius of a triangle

Is your feature request related to a problem? Please describe.
To add a calculator for calculating circumradius of a triangle

Go ahead
Deadline 22 May,2021


User is not logged in

I always get that error after login on my site: User is not logged in
My site: http://www.beautyjackpot.byethost33.com/

https://github.com/ztizzlegaming/CSGOWinBig/issues/58


Getting Started: 🤔 Start thinking about project ideas & the technical stack

Task Description

[x] Check out apps that Voyage teams have created in prior Voyage by reviewing the Project Showcase
[x] Come up with 1 to 3 ideas for projects you'd like to build so you can discuss them with the rest of the team when the Voyage starts.
[x] Think about what technical stack you'd like to use to build that app. Similarly, you'll discuss this as a team at the start of the Voyage.

Technical Considerations
N/a
Additional Considerations
N/a
Resources
Choose a project & create a Vision Statement (Chingu Handbook)
Member of Epic: #15

Completed


Optimized Windows builds take an abnormally high time (6h+)

It seems TFRT has hit a MSVC optimizer bug:
https://developercommunity.visualstudio.com/t/Abnormally-high-compilation-time-4h-sp/1448863

Waiting for a response from the MSVC team

An workaround was implemented in #70, closing


 It would be very interesting to see how it scales compared to other algs, and how increasing the maxLevel constant respectively affects scaling.
https://github.com/MauriceGit/skiplist/blob/77f5c8d3e14534b3c6e56997a7e164363c7bff00/skiplist.go#L40


Options API

I started to try to transition from AMS 0.8, however I am running into issues where I want to pass in options to the serializer. If you are not familiar, options is just metadata that you can use when you are serializing the object.
I was thinking something like:
class ItemSerializer < SimpleSerializer
  attribute :foo do |object, options|
    object.foo if options[:include_foo]
  end
end

> item = Struct.new(:foo).new('foo')
> ItemSerializer.new(item, options: { include_foo: true }).serialize
{ foo: 'foo' }
> ItemSerializer.new(item, options: { include_foo: false }).serialize
{ foo: nil }

Thoughts? I am open to working on this if this is something you would consider.

I never used that feature, but you are welcome to submit a PR if you have time to work on it! I probably won't for a week or two. Just add an options parameter to the constructor, and pass it to instance_exec.


Defining animation sequence

Is it possible to define a sequence in which a group of elements will be animated (i.e., animate this element first, this one second, etc.)?

Hi, you can use the animationend event to achieve it.

Best Regards, Dariel

Hi I recently came across this same basic concept (changing css classes based on events from within the html) in the form of the toggles and switches library formerly a part of the gumby framework. I am comparing them, and trying to see if there is any functionality of that framework that is not also present in ani.js.
How would you add one class while removing another from an element? How would you toggle between two classes (add one, remove another) using ani.js?

Hi, maybe this example can help:
data-anijs="if:click, on: #switch, do: $toggleClass active, to: #switch;
if:click, on: #switch, do: $toggleClass red, to: #switch"

I know that could be to verbose, but to solve it, we need to improve the toggleClass helper. Which is really easy, but unfurtunately, I don't have time at this moment to improve it. I'm so sorry. If you want to make some contribution, you're welcome.
Best Regards, Dariel

@darielnoel  First, Thank you so much for this amazing resource!
Question: Similar to the original question stated by Appwerks. Is there a way to have animations in sequence (using the animationend function) where each element in sequence would not be visible until until the animationend is called ?
So if i had 3 elements and i was using the fadeIn animation, I would like all three elements to be invisible then element 1 would fadeIn, followed by element 2 then 3. However, I also want all three elements to be invisible again when the user scrolls off the page and then re-animate the sequence when the user scrolls back to view the elements.

So if i had 3 elements and i was using the fadeIn animation, I would like all three elements to be invisible to start then element 1 would fadeIn, followed by element 2 then 3.

http://codepen.io/darielnoel/pen/ONvxmq?editors=1100

However, I also wlant all three elements to be invisible again when the user scrolls off the page and then re-animate the sequence when the user scrolls back to view the elements.

That is little bit tricky, I recommend you play a bit with the before and after functions
Best Regards, Dariel


Replace SE or ME time in logbook by checked or not

indication if the aircraft is SE or ME, if applicable.
Page 31
https://www.easa.europa.eu/sites/default/files/dfu/Part-FCL.pdf
I attach an example

Was thinking about that, I probably will add it as an option in the settings page. I personally prefer the full time field instead of the check box


Failed to install dependencies: Command failed: yarn

Can not understand, when running command
graph init --from-example <GITHUB_USERNAME>/<SUBGRAPH_NAME>
I get the result as in the log file below, what is the possible problem and solutions?(used Hosted Service)
Generate subgraph from ABI
Write subgraph to directory
✔ Create subgraph scaffold
✔ Initialize subgraph repository
✖ Failed to install dependencies: Command failed: yarn
ERROR: There are no scenarios; must have at least one.
/usr/local/lib/node_modules/@graphprotocol/graph-cli/node_modules/binaryen/index.js:7
if(n){p=__dirname+"/";var ea,fa;a.read=function(c,e){var b=q(c);b||(ea||(ea=require("fs")),fa||(fa=require("path")),c=fa.normalize(c),b=ea.readFileSync(c));return e?b:b.toString()};a.readBinary=function(c){c=a.read(c,!0);c.buffer||(c=new Uint8Array(c));assert(c.buffer);return c};1<process.argv.length&&(a.thisProgram=process.argv[1].replace(/\/g,"/"));a.arguments=process.argv.slice(2);process.on("uncaughtException",function(c){if(!(c instanceof ha))throw c;});process.on("unhandledRejection",u);a.quit=
Error: Command failed: yarn
ERROR: There are no scenarios; must have at least one.
at ChildProcess.exithandler (child_process.js:294:12)
at ChildProcess.emit (events.js:198:13)
at maybeClose (internal/child_process.js:982:16)
at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)

In this case, when trying to overcome the problem, I tried to install in different ways, including installing several packages (yarn and npm), the problem was solved after adding settings to the github global settings (name and email)


sap.m.Select : selectedKey Attribute not working with dynamic Content in a view.xml

I have a view.xml with a sap.m.List wrapped around multiple sap.m.HBox-es. Inside such an HBox is a sap.m.Select element with the attribute selectedKey set. If the content of the List is loaded by binding to a model (in this case to a json object) then the Select does not preselect the key according to selectedKey.
The model is rendered correctly if set. If no json object is set, the selectedKey of the Select works. In both cases a static string is used as a key (but with the intention to use an attribute of the json object later on).
(I also tried two ways to provide the json object. Unfortunatly only the one with dynamic loading with sap.ui.model.json.JSONModel.loadData() works but not .setData(). - But that is another issue.)

For further investigation we need a sample to debug.
Could you please provide jsbin sample http://jsbin.com/fowiciciho/1/edit?html,js,output

I tried to reproduce the code as closely as possible without too much overhead.
http://jsbin.com/didekuzoja/edit?html,output
Unfortunatly in the jsbin-application I have to use a string to represent the xml-view-code. This somehow breaks even more because now not even the selects items are complete (just one is shown). This has to be some other issue. However if I run the same code on a local test-application with a Test.view.xml and a Test.controller.js (renaming the controller in the code as well) then everything works fine regarding showing all items from the select. However, still the defined item of the select is not automatically preselected because it is dynamically created as part of the List. If it is not part of the List then everything works as expected.

Thanks for the jsbin internal incident<PHONE_NUMBER> has been created for this issue.

Unfortunatly in the jsbin-application I have to use a string to represent the xml-view-code. This somehow breaks even more because now not even the selects items are complete (just one is shown). This has to be some other issue.

@v-schulz Thanks for your ticket. The issue was not reproducible, because in the bootstrap the sap.m library was missing, but instead the  sap.ui.commons library was included. However, after fixing this error, I could reproduce the selection issue.


Grey Tile Upgrades not applied correctly on Remote Clients — Examine the Action and Effects created.

Grey Tile Upgrades not applied correctly on Remote Clients — Examine the Action and Effects created.

The Functionality here tested with 1856 Tiles, and works correctly.
Will test with 1835 when the functionality improvements allow


feat: add plugin demo gif

Adding to the README a short gif that demonstrates what the plugin does and the built-in colorschemes in it would be a nice small addition.

you are right, i was thinking of doing something like this. maybe in a day or two.


Should we add a red line for 2020 cohort in Figure 9, or wait another year?

I think definitely not; let's wait.  We need to see this cohort a bit more before we truly classify it as 'known large'.


sidecarInjectorWebhook.enabled=false no longer generates sidecar injector configmap

Describe the bug
Looks like as of istio 1.0 using helm to generate the istio config with sidecarInjectorWebhook.enabled=false no longer includes the sidecar injector configmap. This was not the case in 0.8.0. Looks like it happened as a part of https://github.com/istio/istio/pull/7408/files#diff-de5983fcb0f040db64cf0281f3b152c0R1
Expected behavior
sidecarInjectorWebhook.enabled=false should still generate the configmap so that it's possible to do something like the following to get the appropriate configs to inject manually:
$ kubectl -n istio-system get configmap istio-sidecar-injector -o=jsonpath='{.data.config}' > inject-config.yaml
$ kubectl -n istio-system get configmap istio -o=jsonpath='{.data.mesh}' > mesh-config.yaml

From https://istio.io/docs/setup/kubernetes/sidecar-injection/#manual-sidecar-injection
Steps to reproduce the bug
 ~/istio-0.8.0 $ helm template install/kubernetes/helm/istio --name istio --namespace istio-system --set sidecarInjectorWebhook.enabled=false > ./istio.yaml
 ~/istio-0.8.0 $ grep istio-sidecar-injector istio.yaml
  name: istio-sidecar-injector

vs
~/istio-1.0.0 $ helm template install/kubernetes/helm/istio --name istio --namespace istio-system --set sidecarInjectorWebhook.enabled=false > ./istio.yaml
 ~/istio-1.0.0 $ grep istio-sidecar-injector istio.yaml

Version
istio 0.8.0 and/or 1.0.0
kube 1.10.2
Is Istio Auth enabled or not?
yes
Environment
bare metal, debian jessie
Cluster state
n/a

I am happy to submit a PR to fix this but would appreciate a 👍 before I do.

Duplication of #7567

Thanks!


ESP32 does not connect to WIFI after settings confidential

I programmed the ESP32 using all suggested methods and it does not connect to WIFI after setting and saving/ encrypting the WIFI/MQTT-Data.
Check for credentials.dat
Fill in your credentials:
SSID: poessl.net
Wifi-password: PassworT@ZUhause
MQTT-Server - IP or hostname: <IP_ADDRESS>
Username: snipah
User-Password: anho1572
Your inputs are:
SSID       : poessl.net
Wifi-PW    : edited
MQTT-Server: <IP_ADDRESS>
Username   : snipah
User-PW    : anho1572
ok for you (yes/no): yes
Write credentials encrypted to disk
Found ESP32 Board, using UART2 on GPIO 16 and 1�����������������������������������������������������������������������������������������������������������������

Sorry, I think you fetched a buggy version. I try to set a new version - at the moment the spiritlevel-part isn't running. I apologize for the situation.

Hi,
the error could be clarified. Unfortunately, the I2C pins for the SpiritLevel support were chosen in an unsuitable way and therefore no monitor was possible after the initialisation of the I2C bus. Please use the new version that has just been posted. New in this version is that the desired addons are queried in the start menu. Sadly it is unavoidable to connect the MPU6050 to the changed pins. Sorry again for this inconvenience.

Thanks a lot for the quick fix! Tested and confirmed to be working!

Really good news - thanks for your feedback!

New version boots up nicely.


[FEATURE REQUEST] Input text remains visible beyond the parameters of the box

Is your feature request related to a problem? Please describe.
My captures are sometimes longer than the input bar. After typing more than the width of the input bar , the text is no longer visible.
Describe the solution you'd like
I would like for the text to be visible in the input bar no matter how much I type.
Additional context
I use this a lot for typing quick journal entries in other languages that I'm learning. Super helpful without having to open a note.

Hey! Thanks for the feature request.
I added a setting for multi-line inputs, which features a much larger box. Does this resolve the request for you, or are you looking for something else?

Almost! Thanks for the multi-line input. Works really well for LTR languages like French, German Greek, or English.
Unfortunately still facing the same issue with RTL languages like Arabic where the letters get stuck on the left side. Would it be possible to have this scroll left as well as right?
I've attached a screenshot of RTL and LTR. The latter works well, but unfortunately the former still gets stuck.
Thanks!

t

Issue with RTL text was raised again in #739 - fixed now :)


Training inexplicably interrupted

Dear author,I've encountered a bug while running blip4cir/train.py. The training process terminates unexpectedly during the encoding of bank features without any error messages.
Could you please help me with this issue?
Thank you:)

其他的模型您可以跑吗

CLIP4CIR可正常跑，但BLIP模型异常中断

CLIP4CIR可正常跑，但BLIP模型异常中断

可以发您的微信到我的邮箱<EMAIL_ADDRESS>

If you need to run the blip4cir and blip24cir training, try to prepare a machine with sufficient cuda memory (at least A100 40G) and RAM (300G)
By the way, since the size of the bank is too large, there is no plan to put the bank directly on the huggingface, sorry about this


[Instructions of Inference] Could you provide complete instructions for reproducing the results on the OGB-LSC leaderboard?

Hi authors,
Thank you for open-sourcing this cool work! I want to try your models with the provided checkpoints. Could you further provide complete instructions for reproducing the results (e.g., 0.0671 eV/MAE on a valid set) on the OGB-LSC leaderboard with the available checkpoints?
Looking forward to your reply!
Best

Hi,
We will be adding a notebook soon explaining this and data preparation. But for now, here is a set of steps you can follow.

The preprocessed data is available at https://huggingface.co/datasets/shamim-hussain/pcqm. The parquet and npz files must be put in the data/PCQM directory. You can also download them by running the following command:

bash download_data.sh

The model weights are available at https://huggingface.co/shamim-hussain/tgt. You may directly copy the models directory from the huggingface repository. The raw weights are contained in the model_state.pt files in the checkpoint directories.
Make distance predictions (on the training and validation sets by default)

python make_predictions.py configs/pcqm/tgt_at_200m/pcqm_dist_pred/tgt_at_100m_rdkit.yaml

This will create a 'predictions' directory (e.g. bins50) in the model directory, containing the predictions for the training and validation sets. To reduce the number of distance samples (and thus save time and disk space) add the following argument 'prediction_samples: 10' (we used 50 samples, you can increase it during the final inference to get better results).
4. Final evaluation:
python do_evaluations.py configs/pcqm/tgt_at_200m/pcqm_gap_pred/tgt_at_100m_rdkit.yaml

the results will be printed to the console and also saved in the predictions directory

We have now added instructions for data preparation and inference. Please refer to the README for details.


Browser setup documentation

Hi
First of all, thanks a lot for sharing your code. Made all process a lot easier. I'm trying to setup the web interface or at least call the control.php file but can't find the documentation pertaining to this setup. Do I need to create a symlink in /var/www/html or something like this ?
thanks

Glad you like the project!
I am actually in the process of redoing the web interface using Flask/Python instead of PHP. I thought that calling a python script from PHP to be pretty inefficient. Setting up a quick Flask app would be preferred.
So I removed how to implement control.php (that file will also be removed soon from the repo). I backed up the master branch before I made that change so you can still find some documentation on that here if you still want to set up the PHP script: https://github.com/naztronaut/dancyPi-audio-reactive-led/tree/master_bak#browser-setup
No need or a Symlink, just make sure you have Apache and PHP installed on the pi, move control.php into the /var/www/html directory, change ownership to Apache, and then you can run it. If you run into a permissions issue, there is one troubleshooting step right below that explains how to get past that.
Hope this helps! Let me know if you have questions. Good luck!


[Backport release-23.05] {ungoogled-,}chromium: 118.0.5993.70 -> 118.0.5993.88, fix update.py

Description of changes
Manual backport of

#262147

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262316 run on x86_64-linux 1

17 packages built:

chromium
chromium.sandbox
fast-cli
google-chrome
netflix
playwright-test
puppeteer-cli
python310Packages.pytest-playwright
python310Packages.pytest-playwright.dist
python311Packages.pytest-playwright
python311Packages.pytest-playwright.dist
reveal-md
sharedown
ungoogled-chromium
ungoogled-chromium.sandbox
vhs
wayback

nix-build -A stable -A ungoogled nixos/tests/chromium.nix
/nix/store/f01v8vz7s63wqgpbzdclhrzwp61jja6v-vm-test-run-chromium-stable
/nix/store/fd2141zxfa6mb40qkjcr3py23klgm8xv-vm-test-run-chromium-ungoogled


Request: AFNumberField should open a numeric keyboard

Would be nice if the AFNumberField opened up a numeric keyboard rather than the default alphanumeric keyboard (Android API 30)

Hello @tomstgeorge ,
This is now the default behaviour for the new version (currently on the dev channel on pub.dev):
Use 1.0.0-dev.1 to select it.


Redundant private registry in AWS not working as expected

Versions:
Docker 1.8.2
Docker registry 2.1.1
Setup:
S3 bucket
two EC2 instances
one Route53 A record that resolves to IPs of the two EC2 instances
Problem:
When both EC2 instances are up and running everything works fine. As soon as I take one of them down docker pulls stop working and I keep getting "not found" messages. As soon as I bring up the second node the registry is working again.
Docker run command:
docker run -d -p 5000:5000 --restart=always -v /certs:/certs -e "REGISTRY_HTTP_SECRET=<secret-same_for_both_nodes>"  -e "REGISTRY_STORAGE=s3" -e "REGISTRY_STORAGE_S3_REGION=us-east-1" -e "REGISTRY_STORAGE_S3_BUCKET=" -e "REGISTRY_STORAGE_S3_ACCESSKEY=" -e "REGISTRY_STORAGE_S3_SECRETKEY=" registry:2.1.1

I don't think the problem here is with the registry.  If it is not being contacted when you remove an EC2 instance there the problem lies with routing or the docker daemon.

@RichardScothern
Is is being contacted, and I see the registry returning 200 OK, but there's no communication past that.

So it's being pinged correctly and returning OK.  What do the daemon logs say?

Here's what I see in the docker container log output:
Client_IP - - [24/Sep/2015:19:12:15 +0000] "GET /v2/ HTTP/1.1" 200 2 "" "docker/1.8.1 go/go1.4.2 git-commit/d12ea79 kernel/3.10.0-229.11.1.el7.x86_64 os/linux arch/amd64"
Client_IP - - [24/Sep/2015:19:12:45 +0000] "GET /v1/repositories/test-image/images HTTP/1.1" 404 19 "" "docker/1.8.1 go/go1.4.2 git-commit/d12ea79 kernel/3.10.0-229.11.1.el7.x86_64 os/linux arch/amd64"
is daemon log different from that? Where is is located?

That is the daemon log.  You can see that it's attempting a call to a v1 registry for some reason (I presume because it's falling back due to not being able to contact the v2 registry)
If you run your daemon in debug mode (-d flag) you will get more context.

added -D to the daemon, not seeing much useful info
time="2015-09-24T20:34:43Z" level=info msg="response completed" http.request.host="<docker_reg_dns_name>:5000" http.request.id=5c2f9d01-06bf-4990-9995-973477e276b8 http.request.method=GET http.request.remoteaddr="Client_IP:41563" http.request.uri="/v2/" http.request.useragent="docker/1.8.1 go/go1.4.2 git-commit/d12ea79 kernel/3.10.0-229.11.1.el7.x86_64 os/linux arch/amd64" http.response.contenttype="application/json; charset=utf-8" http.response.duration=1.487644ms http.response.status=200 http.response.written=2 instance.id=ebb6be95-4bc3-44c9-a047-f30ae66b85dd version=v2.1.1
Client_IP - - [24/Sep/2015:20:34:43 +0000] "GET /v2/ HTTP/1.1" 200 2 "" "docker/1.8.1 go/go1.4.2 git-commit/d12ea79 kernel/3.10.0-229.11.1.el7.x86_64 os/linux arch/amd64"
Client_IP - - [24/Sep/2015:20:35:13 +0000] "GET /v1/repositories/test-image/images HTTP/1.1" 404 19 "" "docker/1.8.1 go/go1.4.2 git-commit/d12ea79 kernel/3.10.0-229.11.1.el7.x86_64 os/linux arch/amd64"

I brought up the second node and I see the same get request on both nodes. My expectation was that Route53 will resolve the name to one of the addresses. Need to better understand what's going on in Route53.

@RichardScothern I found the problem in the Route53 config. Thank you for your help.

glad to hear it.


Updated environment versions and instructions

Hi,
I'm trying to train my own dataset but failed to deploy after many different CUDA, CUDNN, CuPy, Python version trials on Ubuntu 16.04 on NVIDIA GTX-950M.
Is there any updated environment versions for CUDA, CUDNN, CuPy, Python, Chainer, NumPy?
Cheers,

To be honest, currently, I don't have enough time to update the versions of packages used in this repo. How about using Nvidia docker to construct the environment?

Here's a Dockerfile created by @Skylion007: https://github.com/mitmul/ssai-cnn/pull/5

Thanks for the code and info, I'll give it a try.
Cheers!


feat: TableReport dark theme

This PR introduces a dark mode for the TableReport.
I tried to use as much as possible css variables all centralized in a _variables.css file.
UI preview
https://github.com/user-attachments/assets/7201da44-306b-4d8a-a816-1c94aabf52cf

Hi @rouk1 ! thanks so much for doing this :rocket:  the TableReport looks really nice in the proposed dark mod:star_struck:
also having more systematic naming for all css variables will definitely help with maintainability
I noticed that some aspects that are not related to colors are affected by the PR as well; here are a first few comments

I noticed standalone-report.css has been removed; this is where we set the font when we ask the report for a full html page (when we ask for a fragment it uses the font of the parent element). as a result the report now uses the default browser's font but I would like to keep a sans-serif font which I think tends to be a bit more compact and more readable for short text

that's really a detail but a line has appeared that separates the selected tab and its panel, I think it is a bit clearer that it is the selected tab when they are connected

the selected column's card is not very well separated from the background anymore (I think due to removing the shadow)

the "Copied!" message from the copy to clipboard button used to be in reverse video, I think that helps notice it and separate it from the background

the copy buttons on the sample from the dataframe used to be bigger and in the top-left corner, now they are centered

also that sample text box used to have a max width and scroll horizontally which I think can be useful if the text is very long

in the summary statistics column the buttons are now smaller and have more padding which makes the icons hard to read IMO

In the summary statistics there used to be a thick gray line separating the first column when it overlaps the rest, I think it helps realize that this column is sticky and some content is hidden behind it due to horizontal scrolling, that has been removed

in the toggle tip shown below, there used to be a transition delay so that if we hover the ( i ) button and then move the mouse to hover over the text it doesn't disappear; now as soon as the mouse leaves the button the text disappears. also there used to be a ring around it when we click it

not sure if it is intentional but the background color of the header in the associations panel is gone, I don't have a strong opinion as to whether it is better or worse

most of these changes don't seem to be related to dark mode but it is true we had discussed that this is an opportunity to refactor the css a bit. so I'm not sure which of these are intentional or not, maybe it would help to have another quick call also with @glemaitre  to discuss this? it might be  easier interactively while looking at an open report.
now regarding the colors and the dark mode itself, here are a couple of comments:

the card in the first panel was green to show its association with the selected column, but that does not apply to the "distributions" panel so using a neutral gray color there was intentional
I think the orange in dark mode looks good but the reason we had used a different color is that we show histograms of numeric columns in blue and bar plots for categorical columns in orange so it might be good to use a third color for the card header to not give the impression there is a connection with the plot?
the axis lines and ticks in dark mode are not visible; I think we can find the previous pr one way to make sure they are

I think the white text on orange background does not have a very high contrast and is a bit hard to read

thanks again for all this work and LMK if you would like to schedule a call or I can also come to montparnasse sometime!

Thanks for the review @jeromedockes, I'll try to work on your feedbacks on friday.

cool, thanks!!

Hello @jeromedockes  !
I think I've addressed your feedback. Please let me know if you want something else or if I miss something.
tackled feedbacks

standalone-report.css has been re-added (using new css vars nomenclature)
selected tab border bottom color is now equal to background color
fix column summaries shadow
copy button tooltip is now inverted and bold
copy text box as a max width of 100% hence it will never be larger than it's container (a horizontal scroller will appear at need)
summary statistics is now bigger and icons fits it
grey line that indicates that first two columns are sticky is visible to me
toggled tip content is now part oh the hover, hence it does not disappear when hovering it
tip button outline is now blue
association table header now uses default tertiary background color
card header are now in highlight color only whena ssociated with a table column
primary highlight color is now raisin instead of orange, it provides high contrast and is not related to plot colors
use 4 spaces in CSS to avoid large diff

the axis lines and ticks in dark mode are not visible; I think we can find the previous pr one way to make sure they are

Not sure on how I may address this.
UI preview
https://github.com/user-attachments/assets/dbf7a7de-abc9-474e-a2a4-8bb443abaa60


Icon Request: Incognition / Secret

Icon Request

Icon name: secret
Tags of this icon: secret
Use case: Many, i.e. Browsers
Screenshots of similar icons:

https://remixicon.com/#icon-spy-line
https://remixicon.com/#icon-spy-fill
You can find them in the user&faces category.
Thank you for your advice.


[Custom]

How can I display all device images on page without folders?

Please run through the example before ask.

Actually I did but  I couldn't figure out. Im new at coding.With getAssetListPaged I get images but need size,couldn't get all.  I need get all images and display. I can be glad if you can help me @AlexV525 .Thank you

I'm using images in bottom sheet. Just dont want with display with folders.I want to correctly display all images


Fix misspell

Common issue-line3: in Activity on Fragment ->in Activity or Fragment

The change is too minor


Stetson j

Added function StetsonJ that compute StetsonJ variability index given a measurement of flux and a measurement of flux errors. It is possible to specify on which band to compute the index; if band is not specified the measurement is done on all avaliable bands.

This looks good overall, two things:

Is it worth putting special handling for the case where you have 0 or 1 data points? In particular, should they return a NaN? I tried it with a dummy example and ran into that for the N=1 case, because it attempts an N/N-1 calculation. Just thinking about the use case where we have O(10^6+) running and don't want these to be erroring out necessarily
stetson_j_single has required parameters of fluxes and errors, meaning that a user would need to filter down to those band specific values manually for ingest into the function, would a single argument for band be better? This aligns it with stetson_J, but also would potentially make it redundant as it looks like stetson_J can handle a single input band string

I have placed a restriction that the flux array passed to _stetson_J_single has to have length larger than 1 or it returns np.nan.
I have renamed stetson_j_single to _stetson_j_single - I think this is really only a private function that user should never see, and all of the handling is via stetson_j functioning almost like a wrapper for a workhorse _stetson_j_single function


Implementing CooldownData

Implementation of SpongeCooldownData.

[x] Registration of field getters and setters
[x] Accurately used the appropriate AbstractData implementation

Implementation of ImmutableSpongeCooldownData

[x] Accurately extended the appropriate AbstractImmutableData implementation
[x] Accurately instantiating final instance fields with an ImmutableValue counter part for any value getters
[x] If necessary, creating a new ImmutableValue for a value that should not be cached, or, using the existing caching utils.

Implementation of the CooldownValueProcessor

[x] Appropriately extended AbstractSpongeValueProcessor
[x] Individual processors for each source type possible (one for ItemStack/TileEntity/Entity as necessary).

Registration:

[x] Registered the Key correctly by COOLDOWN in the KeyRegistry
[x] Registered the DataProcessors and DataManipulatorBuilder in SpongeSerializationRegistry
[x] Registered the ValueProcessors in the SpongeSerializationRegistry

Test Plugin code:
@Plugin(id = "CodeTester", name = "CodeTester", version = "1.0.0")
public class CodeTester {

    @Listener
    public void onPlayerInteractBlock(InteractBlockEvent event) {
        Optional<Player> optional = event.getCause().first(Player.class);
        if (optional.isPresent()) {
            Player player = optional.get();
            if (player.getItemInHand().isPresent()) {
                ItemStack item = player.getItemInHand().get();
                Hopper hopper = (Hopper) event.getTargetBlock().getLocation().get().getTileEntity().get();
                if (item.getItem() == ItemTypes.STICK) {
                    player.sendMessage(Texts.of("HopperTransferCooldown: ", hopper.get(Keys.COOLDOWN).get()));
                }
                if (item.getItem() == ItemTypes.BLAZE_ROD) {
                    hopper.offer(Keys.COOLDOWN, 100);
                }
            }
        }
    }
}

So, a couple of things about your test plugins:

Your test plugin is very buggy and easily throws exceptions in many places unless you know exactly what is going on.
You don't check any optionals, resulting in assumptions and exceptions
You aren't testing everything as asked

Manipulators aren't being retrieved, you're just interacting with values directly
Serialization and deserialization isn't being tested.

Aside from that, I've tested it enough to know that it works, but that's only because I know the underlying code well enough, but seriously, you should consider your test code design as all corner cases should be considered.


Cloud code functions not getting called

I have set up parse-server on Heroku, but I am having trouble with the cloud code. I used the parse-server-example, and replaced the contents of main.js with my own:
`console.log('##### TEST');
Parse.Cloud.afterSave('SomeClass', function(request) {
console.log('##### INSIDE AFTER SAVE');
});`
And this is my parse-server setup:
var api = new ParseServer({   databaseURI: 'mongodb://...',   cloud: __dirname + '/cloud/main.js',   appId: '...',   masterKey: '...',   dotNetKey: '...',   clientKey: '...',   fileKey: '...',   serverURL: 'https://<appname>.herokuapp.com/parse',   push: {     ios: [       {         pfx: '....p12',         bundleId: '<bundle-id>',         production: false       },        {         pfx: '....p12',         bundleId: '<bundle-id>',         production: true       }     ]   } });
When i run heroku logs in the terminal i can see the "### TEST" getting printed after start up. But when I save an object of the specified type, nothing is printed in the logs, so it seems that the beforeSave and afterSave isn`t getting called at all.

I am facing the same issue. It looks like that I can get objects from cloud code, but can't save an object. For example, here is my bar cloud code function.
`Parse.Cloud.define('bar', function(req, res) {
var GameScore = Parse.Object.extend("GameScore");
var gameScore = new GameScore();
gameScore.set("score", 2012);
gameScore.set("playerName", "Jerry Petric");
gameScore.set("cheatMode", false);
gameScore.save(null, {
  success: function(gameScore) {
    // Execute any logic that should take place after the object is saved.
    alert('New object created with objectId: ' + gameScore.id);
  },
  error: function(gameScore, error) {
    // Execute any logic that should take place if the save fails.
    // error is a Parse.Error with an error code and message.
    alert('Failed to create new object, with error code: ' + error.message);
  }
});

});`
While I use cUrl to trigger this function,
curl -X POST \   -H "X-Parse-Application-Id: myAppId" \   -H "X-Parse-Master-Key: myMasterKey" \   -H "Content-Type: application/json" \   http://localhost:1337/parse/functions/bar
And execute this cUrl in terminal. After long waiting, I received:
curl: (52) Empty reply from server

what's your cloud function looks like?  Did you really save a 'SomeClass' object in your cloud function?

The SomeClass object is saved from a client. The code I posted is my entire main.js.

did you have
Parse.serverURL="https://fast-badlands-XXXX.herokuapp.com/parse/";
in your main.js?

No, the serverURL is set in index.js, in my ParseServer initialization. Is that wrong?

Are you sure your parse server instance is receiving that object save and processing it?..  for troubleshooting purposes, it's important to know.. are you using an app Id which is also a parse.com app id?  Is the parse app also connected to the same database, etc..   I'd run the server with the environment variable VERBOSE=1 and make sure the save was received.

No, I am not sure of that, and don`t quite know how to check it. This is the situation:

I clicked the Heroku button on the parse-server-example, and set up a Heroku app
I started the database migration from parse to my new MongoLab base
I copied keys from parse, into my own Parse Server initialisation (appId, masterKey, fileKey, clientKey)
I use a client (.NET-client) to make changes to an existing "SomeClass" object (this client connects to the parse database)
I verify that the change was made in the parse database, and it was synced to the MongoLab database
I thought that at the point the data was synced to the MongoLab instance, the afterSave() would get called
I`ve also tried adding records directly to the MongoLab database

The .NET SDK would make a call with the dotNetKey, which if you haven't configured would cause a problem.  Here's an easy test... Change the appId for your ParseServer instance and also in your .Net app.  I recommend removing any clientKey/javascriptKey/dotNetKey/restAPIKey options.

When going through my .NET client, and not just saving objects directly in the MongoLab database, the cloud functions were called as they should.


Add exceptions for lineup package

Codecov Report

Merging #72 (db44a8f) into master (e7da674) will decrease coverage by 0.17%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master      #72      +/-   ##
============================================
- Coverage     64.54%   64.36%   -0.18%
  Complexity      419      419
============================================
  Files            85       87       +2
  Lines          1475     1479       +4
  Branches        148      148
============================================
  Hits            952      952
- Misses          472      476       +4
  Partials         51       51

Impacted Files
Coverage Δ

...eup/lineupExceptions/DuplicateLineupException.java
0.00% <0.00%> (ø)

...neup/lineupExceptions/LineupNotFoundException.java
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7da674...db44a8f. Read the comment docs.


fold transitions into renderpass

https://github.com/SaschaWillems/Vulkan/blob/master/base/vulkanexamplebase.cpp#L1571
If you set the initialLayout to UNDEFINED and the finalLayout to Present (while keeping the subpass reference to COLOR_ATTACHMENT_OPTIMAL). And add dependencies from EXTERNAL to 0 and 0 to EXTERNAL with the correct masks. Then you can remove the pre and post present vkCmdPipelineBarrier calls as the renderpass will transition the attached images as required.

Sorry for the long delay, this kinda got lost until I was asked about the present barriers today.
So thanks for hinting me at the directions :+1:
I already have this up and working in the triangle example and will check if it works on all the implementations I have here to test. If it works as expected I'll add this to the other samples too, push the changes and maybe do a small write up on how this actually works as it seems that most of the samples also use dedicated barriers instead of the sub pass dependencies along with the layouts.

Is the explicit render pass dependency necessary? The specification seems to implicitly create this dependency if no dependencies are specified and the alias flag is not passed.

I just read over the current specs (1.0.20) and did not see anything that hints at an implicit dependency, so I'd say yes it's actually necessary (unless you do manual barriers).

@SaschaWillems actually:

If the attachment does not include the VK_ATTACHMENT_DESCRIPTION_MAY_ALIAS_BIT bit and there is no subpass dependency from VK_SUBPASS_EXTERNAL to the first subpass that uses the attachment, then it is as if there were such a dependency with srcStageMask = srcAccessMask = 0 and dstStageMask and dstAccessMask including all relevant bits (all graphics pipeline stages and all access types that use image resources), with the transition executing as part of that dependency. In other words, it may overlap work before the render pass instance and is complete before the subpass begins.

The explicit EXTERNAL to 0 dependency is necessary because the implicit pre-renderpass transition without the dependency is implicitly with a barrier from top-of-the-pipe (it says 0 but logically it's top) while you need it to be from whatever waitStage you set the acquire semaphore to.
In the optimal case that stage will be COLOR_ATTACHMENT_OUTPUT because that is the only other thing that needs to wait until the image is free to be used.
Though the 0 to EXTERNAL isn't necessary because the implicit post-renderpass transition is to bottom of the pipe and the semaphore that gets signaled from that command buffer will only get signaled after all work is done (including those transitions) aka bottom-of-the-pipe.

@ratchetfreak : Thx :+1:  I'm at work right now so I only had time for a quick look and missed that part.

Would the dependency look something like this?
VkSubpassDependency dependency = {};
dependency.srcSubpass = VK_SUBPASS_EXTERNAL;
dependency.dstSubpass = 0;
dependency.srcStageMask = VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT;
dependency.dstStageMask = VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT;
dependency.srcAccessMask = 0;
dependency.dstAccessMask = VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;
dependency.dependencyFlags = 0;

I'm not entirely sure about the source stage and access mask because presentation is not part of the graphics pipeline.

I'm at work, so I can't check but out of my head these one should do it (and pass validation). I use them in the updated triangle example (sorry, haven't had the time yet to upload that one).
Initial (which is necessary):
dependency.srcSubpass = VK_SUBPASS_EXTERNAL;
dependency.dstSubpass = 0;
dependency.srcStageMask = VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT;
dependency.dstStageMask = VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT;
dependency.srcAccessMask = VK_ACCESS_MEMORY_READ_BIT;
dependency.dstAccessMask = VK_ACCESS_COLOR_ATTACHMENT_READ_BIT | VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;

Final (not necessary due to implicit dependency):
dependency.srcSubpass = 0;
dependency.dstSubpass = VK_SUBPASS_EXTERNAL;
dependency.srcStageMask = VK_PIPELINE_STAGE_COLOR_ATTACHMENT_OUTPUT_BIT;
dependency.dstStageMask = VK_PIPELINE_STAGE_BOTTOM_OF_PIPE_BIT;
dependency.srcAccessMask = VK_ACCESS_COLOR_ATTACHMENT_READ_BIT | VK_ACCESS_COLOR_ATTACHMENT_WRITE_BIT;
dependency.dstAccessMask = VK_ACCESS_MEMORY_READ_BIT;

I wouldn't blindly trust validation on this quite yet, the synchronization validation is still quite incomplete.

I wouldn't blindly trust validation on this quite yet, the synchronization validation is still quite incomplete.

That's right and why I did test these dependencies on NVIDIA (Windows and Android), AMD and Intel and they worked on all of them. So I guess they're okay :wink:

Finally got around replacing the explicit image layout transitions for the attachments in my triangle example.
They're now folded into the render pass using subpass dependencies. I tried to comment as much as possible (see the setupFrameBuffer function. If some of the comments don't make sense then it's probably because it's late at night over here ;)
I'll now start on replacing the explicit transitions for the base class and the other examples too.
Thanks again @ratchetfreak, folding them into the renderpasses makes the code look cleaner and once you get how this works you start to realize how cool that feature is :wink:

Update base classes and all examples to use sub passes for layout transitions instead of explicit barriers.


minor bug fix

If there is any new update, should I just run the code and update it?
What is the process to keep up to date with your toolbox?


[1.30] bump kubernetes release information page

PR updates the https://www.kubernetes.dev/resources/release/ page to point to the ongoing 1.30 Release cycle timeline.
cc: @kubernetes/sig-release-leads, @gracenng @katcosgrove

/lgtm
/approve


PR on HIP7 - Managing HIP Process

Initial Pull Request for HIP7

@anthonyra can you change the filename back so we can see the diff between what's currently in place and this proposal?

@anthonyra can you change the filename back so we can see the diff between what's currently in place and this proposal?

Ahhh yes, that would make sense... new to the whole "teamwork" Github commits, fetches, merges, and Pull Requests. I'll be fixing that in a jiffy.

@anthonyra separating PRs from discussion is why we adopted Tracking Issues. For this one it is #26
This branch/PR now has your HIP0014 proposal in it and I can't merge (or push to your branch to fix). Happy to ping you on Discord to help walk through how to excise, I'm @jamiedubs there

Closing this Pull Request due to re-submission


Dependencies not included in package

Originally reported by @sjdilkes
The wheels being built are not including dependencies. This is probably because the dependencies are only under the poetry tool section of pyproject.toml. They may need to be duplicated. Is there a way that avoids duplication?

Short resume of various issues:

PEP621 specifies that dependencies should go under [project]
Maturin does not support poetry's custom pyproject fields. It expects it to follow PEP621.
maturin#632
Poetry has had an open issue about it for almost 4 years:
https://github.com/python-poetry/poetry/issues/3332
There's a recent draft PR in poetry that implements that. It will be included in "poetry 2.0.0", once that comes out.
poetry#9135

So in short; we'll have to duplicate things for now :/


Request : swap Error and Success type params + make Effect.request dual

Type

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
As discussed on this message https://discord.com/channels/795981131316985866/795983589644304396/1212891296944103424

Should the type param order for Request.TaggedClass be swapped? It's currently <Error, Success, A> instead of <Success, Error, A>.

Since the message author did not respond to @mikearnaldi, I created the PR.

impl seems broken, can you add a test pls

My bad @mikearnaldi this is fixed now.