Seconds/ minutes not functioning as expected

Every 60 seconds doesn't work and silently errors, same with every 60 minutes - however every 61 seconds does work.

Later doesn't work the way you think it does.  It's a constraint based system where you specify the constraints a time must meet in order to be considered valid.  A constraint of 'every 1 second' means that any time where the seconds value is 1 will be considered valid - it does not mean return the next value every 1 second.  Since no minute or hour every has a seconds value of 60, no times will ever be considered valid.


feat: add collapseSamePrefixes option to prevent duplication inside namespaced component name

When using directoryAsNamespace: true a component path like src/components/collapse/collapseFolderAnd/CollapseFolderAndComponentPrefixes.vue would be converted into a component named CollapseCollapseFolderAndCollapseFolderAndComponentPrefixes.
I implemented an option to collapse same prefixes in folder names and file name, so that it becomes CollapseFolderAndComponentPrefixes.
Closes #307

This does not seem to work.
File path: ~/components/app/sidebar/Sidebar.vue
Generated type: AppSidebarSidebar: typeof import('./../components/app/sidebar/Sidebar.vue')['default']
The collapseSamePrefixes option is also missing from the README.
@DrJume @antfu

Did you set directoryAsNamespace: true?
If it still doesn't work, maybe create an issue with reproduction and reference this PR.
I will create a PR for adding it to the README. Somehow I forgot to add it there. 😅


apache conf

Hi there, i unable to load you web apps
here my site available conf
<VirtualHost ci.my:80>
	# The ServerName directive sets the request scheme, hostname and port that
	# the server uses to identify itself. This is used when creating
	# redirection URLs. In the context of virtual hosts, the ServerName
	# specifies what hostname must appear in the request's Host: header to
	# match this virtual host. For the default virtual host (this file) this
	# value is not decisive as it is used as a last resort host regardless.
	# However, you must set it for any further virtual host explicitly.
	#ServerName www.example.com

	ServerAdmin<EMAIL_ADDRESS>	DocumentRoot /var/www/html/ci/public

	ServerName ci.my
	ServerAlias ci.my

	<Directory /var/www/html/ci/public>
                Options Indexes FollowSymLinks MultiViews
                AllowOverride All
                Order allow,deny
                allow from all
	</Directory>

	# Available loglevels: trace8, ..., trace1, debug, info, notice, warn,
	# error, crit, alert, emerg.
	# It is also possible to configure the loglevel for particular
	# modules, e.g.
	#LogLevel info ssl:warn

	ErrorLog ${APACHE_LOG_DIR}/error.log
	CustomLog ${APACHE_LOG_DIR}/access.log combined
	#ProxyPassMatch ^/(.*\.php(/.*)?)$ unix:/run/php/php7.0-fpm.sock|fcgi://localhost/var/www/html/pos/

	# For most configuration files from conf-available/, which are
	# enabled or disabled at a global level, it is possible to
	# include a line for only one particular virtual host. For example the
	# following line enables the CGI configuration for this host only
	# after it has been globally disabled with "a2disconf".
	#Include conf-available/serve-cgi-bin.conf
</VirtualHost>

# vim: syntax=apache ts=4 sw=4 sts=4 sr noet

any reason why it doesnt load? I already put ci.my into my host file

Can you tell me if you are using php-fpm or cgi mode? This will be in your main http.conf file.  Also would be helpful if you can post (or email) error you get in logs file.  Thanks, L

restarted apache solve my issue

Glad to hear you figured it out. Let me know if you have more questions. Good luck.


Code Cleanup

This commit cleans up the intDisableAxis functions so that there is only a single primary function and the remaining four overloaded functions simply call the main 5-parameter function with "BLANK" in the empty parameter spots. It also enforces the wu_axis_t datatype for intDisableAxis rather than casting to uint8_t, and fixes a minor typo in README.md.

Apologies for the late response, but yes, this PR is the build I've been using in my own project.


support nixos command-not-found, closes #912

This PR adds a rule to suggest package installations in response to NixOS command-not-found errors for uninstalled packages available in the repo, see @anka-213's #912 for the original request.
My implementation was partly inspired by the apt-get rule, and is quite simple, simply judging applicability by the availability of nix, and scanning for the nix-env -iA line, only to suggest that same command in return.
Feedback welcome!

Looks good, thanks!


Fix enum usage after refactor

Fixes reference to LibVips::GConnectFlags::GConnectAfter to LibVips::GConnectFlags::After.
This ought to have been caught by the Github workflow, but apparently not, the commit history shows that only the other workflow has been run.
The workflow log tells us that the scheduled running has been disabled due to inactivity, but that shouldn't disable the workflow entirely on main.
Running the specs locally, I get a segfault, but I'll create a separate issue for that (it was what caused me to upgrade in the first place).

I've added an issue on install-crystal explaining the problem, and filed a bug with GitHub on the schedule disabling.


previewHtml iframe does not use entire space

VSCode Version: 1.7.1
OS Version: Windows 10

Steps to Reproduce:

Create a markdown file and open the preview window side by side

Actual behaviour:

The webview developer tools window shows the following when selecting the top iframe element:

Expected behaviour:
The space on the right and on the bottom of the preview tab should be used as well. The blue selection should cover the entire white space.
This happens whenever the previewHtml command is used, not only when previewing markdown.

👍

Since the html preview uses a WebView element, I believe this is another manifestation of #5745 (which is focused on zooming, but I believe the core problem is the same). We're tracking a bug on the electron side related to this, and also investigating some potential workarounds for the issue.
Closing this issue in favor of that one. Thanks.


Start-cli argument parsing off 1 index for port when running an LV exe

Above is the error shown in the front panel of the top-level vi of the exe. The exe is invoked via PowerShell using the command:
g-cli "<path of exe>"
The exe has one vi with cli-start and cli-stop subvi calls, The cli-start vi errors and is unable to open a TCP connection.

App argument 2 is the port whereas the below VI is looking for the port in arg 1.

Thanks for the report - I'm taking a look into it but seems like it should be straightforward to replicate.

I faced the same issue in LabVIEW 2016 32-bit, exe invoked from cmd. On comparing the difference between the Build Specification of Echo Test with the spec I had, the following boolean was enabled in mine.

Disabling this fixed the issue.
P.S. The issue can be reproduced by enabling the above mention boolean in Echo Test Build Specification.

I was using 2021 64-bit. Unchecking Pass all command line arguments to application fixed the issue for me as well. thanks


std::transform fails to apply to all elements

The following minimal reproducer fails randomly when using the par and par_unseq execution policies, but passes when using the seq or just the default transform.
#include <algorithm>
#include <execution>

#include <assert.h>

int main(int argc, char **argv) {
  constexpr int size = 1024;
  std::vector<double> v(size);
  for (int i = 0; i < size; i++) {
    const double value = static_cast<double>(i);
    v[i] = value * value + 5.0 * value;
  }

  constexpr double min_v = 1000000.0;
  std::vector<bool> ge_v(size);
  for (int pass = 0; pass < 10; pass++) {
    std::transform(std::execution::par, v.begin(), v.end(), ge_v.begin(),
                   [](const double v) { return v >= min_v; });
    for (int i = 0; i < v.size(); i++) {
      assert((v[i] >= min_v) == ge_v[i]);
    }
  }
  return 0;
}

This is with the version of the parallel included with GCC 9.1. My GCC version is 9.1.1 20190503 (Red Hat 9.1.1-1). My version of tbb is tbb-devel-2019.5-1.fc30.x86_64, which I believe is just the latest release, 2019 update 5.

@mfdeakin, the problem is that you use vector<bool> which is not thread-safe. See e.g. https://en.cppreference.com/w/cpp/container/vector_bool:

std::vector<bool> behaves similarly to std::vector, but in order to be space efficient, it:
...
- Does not guarantee that different elements in the same container can be modified concurrently by different threads.

Ah, I didn't realize they had a thread unsafe specialization


New update to addTarget(selector) via Swift 2.2 update

Per Pull Request #18, I have updated the addTarget methods to integrate the new #selector update Swift 2.2 just gave us.
For your consideration.

Okay someone is getting selector happy.


I am unable to find an equivalent off-chain library for building a SMT that is compatible with the on-chain version

Describe the feature
I’ve been using this library: https://github.com/privacy-scaling-explorations/zk-kit/tree/main/packages/smt, but it generates a different root hash, so I assume it is not compatible with your optimizations. Is there an off-chain library that’s compatible with your on-chain SMT implementation?

Hey @andrea-rizzini, thanks for opening this issue. This JS library from iden3 is compatible with our implementation. However, their delete operation may still have some discrepancies as their test coverage is very poor.
Regardless, we'll take a look at the zk-kit approach and get back to you.

Are you sure that one is compatible? Looking at the test; line 116 I found:
const inputs = [BigInt('100'), BigInt('200'), BigInt('1')];
But on your documentation  you states this: H(1 || k || v)
Is the domain separator inverted?

It is definitely compatible. This must be a typo in the documentation, will fix it, thanks for noticing.

Ook thanks, just asked before trying.

There was a mismatch in hashers I was using, you can close the issue. Thanks for the support


Hi，when using the inference.py and loading the ckpt file I get by the training, I faced some problem. Can you help me？Thanks！

The specific error message is：
Traceback (most recent call last):
File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 76, in
cli_main()
File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 72, in cli_main
infer(args.path_to_ckpt, preprocessed_ct_scan, args.path_to_result_dir, name)
File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 42, in infer
model_t = LitLungTumorSegModel.load_from_checkpoint(path_to_ckpt)
File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 156, in load_from_checkpoint
model = cls._load_model_state(checkpoint, strict=strict, kwargs)
File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 198, in _load_model_state
model = cls(_cls_kwargs)
TypeError: init() missing 2 required positional arguments: 'model' and 'loss_fn'
But when I use the ckpt file offered by you, it can run without problem.
After reviewing the logs in Wandb, I discovered the cause of the problem，but I don't know how to solve it, can you help me? The specific content of the log is as follows：
D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'model' removed from hparams because it cannot be pickled
rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled")
D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'loss_fn' removed from hparams because it cannot be pickled
rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled")

The specific error message is： Traceback (most recent call last): File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 76, in cli_main() File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 72, in cli_main infer(args.path_to_ckpt, preprocessed_ct_scan, args.path_to_result_dir, name) File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 42, in infer model_t = LitLungTumorSegModel.load_from_checkpoint(path_to_ckpt) File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 156, in load_from_checkpoint model = cls._load_model_state(checkpoint, strict=strict, kwargs) File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 198, in _load_model_state model = cls(_cls_kwargs) TypeError: init() missing 2 required positional arguments: 'model' and 'loss_fn' But when I use the ckpt file offered by you, it can run without problem. After reviewing the logs in Wandb, I discovered the cause of the problem，but I don't know how to solve it, can you help me? The specific content of the log is as follows： D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'model' removed from hparams because it cannot be pickled rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled") D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'loss_fn' removed from hparams because it cannot be pickled rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled")

try to install this version of pytorch lightning which let you to consider model and loss_fn too.
pip3 install pytorch-lightning==1.4.9

Thank you for your reply！

? ? ?
@.***

------------------ 原始邮件 ------------------
发件人: @.>;
发送时间: 2024年8月17日(星期六) 凌晨2:03
收件人: @.>;
抄送: "🌸 🌸 @.>; @.>;
主题: Re: [Ola-Vish/lung-tumor-segmentation] Hi，when using the inference.py and loading the ckpt file I get by the training, I faced some problem. Can you help me？Thanks！ (Issue #4)
The specific error message is： Traceback (most recent call last): File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 76, in cli_main() File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 72, in cli_main infer(args.path_to_ckpt, preprocessed_ct_scan, args.path_to_result_dir, name) File "E:/VH/Experiment/lung-tumor-segmentation-main/project/inference.py", line 42, in infer model_t = LitLungTumorSegModel.load_from_checkpoint(path_to_ckpt) File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 156, in load_from_checkpoint model = cls._load_model_state(checkpoint, strict=strict, kwargs) File "D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\core\saving.py", line 198, in _load_model_state model = cls(_cls_kwargs) TypeError: init() missing 2 required positional arguments: 'model' and 'loss_fn' But when I use the ckpt file offered by you, it can run without problem. After reviewing the logs in Wandb, I discovered the cause of the problem，but I don't know how to solve it, can you help me? The specific content of the log is as follows： D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'model' removed from hparams because it cannot be pickled rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled") D:\Anaconda\envs\VH\lib\site-packages\pytorch_lightning\utilities\parsing.py:104: UserWarning: attribute 'loss_fn' removed from hparams because it cannot be pickled rank_zero_warn(f"attribute '{k}' removed from hparams because it cannot be pickled")
try to install this version of pytorch lightning which let you to consider model and loss_fn too.
pip3 install pytorch-lightning==1.4.9
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Fix a dead link on website/index.html

Can one of the admins verify this patch?

Thanks!
Changes will be available on the website soon.

@alexey-milovidov Thanks for your checking and the great DBMS. I watched your presentation at Data@Scale 2017, got interested in ClickHouse and I'm trying to use it now.


Can't add multiplied value to new value...

$new = $product->price()
            ->multiply($request->amount);

And then convert it to human readable format, after log it
$currencies = new ISOCurrencies();

$numberFormatter = new \NumberFormatter('nl_NL', \NumberFormatter::CURRENCY);
$moneyFormatter = new IntlMoneyFormatter($numberFormatter, $currencies);

Log::info('new: ' . $moneyFormatter->format($new));

It says: [2017-01-31 21:52:38] local.INFO: new: € 1.088,00
When i want to add $new to $total
$total = Money::EUR(0);

$total->add($new);

Log::info('after add: ' . $moneyFormatter->format($total));

The log says: [2017-01-31 21:52:38] local.INFO: after add: € 0,00
Installed with composer require moneyphp/money

That solves it, many thanks!


Simplify matrix calculations

Describe your PR, what does it fix/add?
Removes unnecessary matrix calculations.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
No
Is it ready for merging, or does it need work?

Are you mega sure this is correct? Have you also tested transformed displays?

It's not changing anything really, just taking away the two minuses in matrixProjection infront of t[3] and t[4] ,which are the values that effect the y value of the vectors multiplicated with that matrix. look at the transforms[],
in every transform the value pairs for 3,4 and 0,1 are :
one is zero and the other one is 1 or -1;
that basicly switches x and y for the resulting vector and multiplicates -1 for some of those transforms.
If you thing about e.g. switching x and y of a vector and multiplying x with -1 would be a rotation of 90 degrees counter clockwise. this is how rotations are done here(it's basicly the mechanism of imaginary numbers).
by multiplying t[3] and t[4] with -1 like you did, all it does is it changes the sign of the resulting y coordinate of the vectors.
With that you would end up with a image that is flipped180, to deal with that you had to multiply that matrix with your flipped180 matrix (which if you look at it does nothing but flipping the sign of the y value of the resulting vector.
I took away those sign flips in your matrixProjection, so the flipped180 is not needed anymore.
The second thing i did was just taking away the wlr_matrix_transpose(matrix,matrix) which "mirrors the matrix across the diagonal, that is because the matrices in the code are ordered the other way around as inside opengl's mechanism.
This is so common, that opengl's glUniformMatrix functions have an boolean value that controls if opengl should to that to the matrix itself (in this case it's basicly for free because it just effects how opengl copies those values).
I tested it with scaling and all 8 available transpositions.

If your unsure,
@fufexan would you be so kind and try to break this!

I've been running it for half an hour, been playing games. No issues found.

@fufexan thanks.


add __version__ attribute to top level

When reporting #1 I was looking to print out the version of daschlab I have installed, in the same way I woudl get e.g. the astropy or numpy version.
import daschlab
import numpy
import astropy
print('numpy:' ,numpy.__version__)
print('astropy:', astropy.__version__)

numpy: 1.26.4
astropy: 6.1.1

However, daschlab does not have that attribute:
daschlab.__version__
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[14], line 1
----> 1 daschlab.__version__

AttributeError: module 'daschlab' has no attribute '__version__'

The documentation for daschlab includes installing directly from github, so some people may pull a newer development version etc., I recommend to define a version attribute to help find out what's going on when you receive bug reports and error messages.
Personally, I recommend setuptools-scm which will generate the version tag automatically when building a distribution or installing directly fomr a git checkout and what is used by e.g. astropy and other major projects in our ecosystem.

Solved today!


build-script-impl: For cmake configuration use distcc

@swift-ci Please smoke test

@swift-ci Please smoke test

@swift-ci Please smoke test

If the assumption is that the cmake configure step distributes to other machines if the C compiler path is something like -DCMAKE_C_COMPILER:PATH=/opt/distcc-r796/bin/distcc, then configuring through distcc but restricting the machines to localhost should stop clang from distributing.
If that's not what's happening, then I still don't know what to do.

I guess I don't know enough to make a judgement here.  If you test it and everything works as it should with and without distcc, I can't argue.

@erg what is the status of this PR?

@erg isn't working on Swift anymore, so we can probably close it. (He can always reopen if he picks it up again.)

@jrose-apple Sounds good!


Expose collision resolution bias in editor for rigid bodies/collision shapes

It would be very useful to extend the PhysicsServer interface and rigid bodies to expose a custom bias for collision resolution and add a global property with the default bias factor for the project. In fact I had to change it in the PhysicsServerSW implementation to have small bodies not to sink inside polygonal collision shapes.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Is it possible to use models of StyleGan2-ada?

Thanks for sharing! And do you know how can I use Stylegan2-ada to replace Stylgan2's model in your code? Does the code support it?

The codebase only supports the stylegan2 repo by rosinality


How to update config an runtime?

hystrix.ConfigureCommand(key, apiConfig.HystrixConfig) set config at init, how to update at runtime?

Timeout、ErrorPercentThreshold、RequestVolumeThreshold and SleepWindow can update at runtime


Fix use local auth replace session

Related issue: https://github.com/pedroslopez/whatsapp-web.js/pull/1257

If you remove the MemoryCard support from a puppet, then you will not be able to restore sessions from the Wechaty framework in the future.

If you remove the MemoryCard support from a puppet, then you will not be able to restore sessions from the Wechaty framework in the future.

I will think about how to use MemoryCard to store LocalAuth session.
For example, use MemoryCard store the LocalAuth clientId from Wechaty framework.
this.whatsAppClient = new WhatsApp({
  authStrategy: new LocalAuth({ clientId: 'default-client' }),
  ...
})

Using MemoryCard with Wechaty/Puppet is quite easy and straightforward:
this.memory.set('clientId', clientId)
await this.memory.save()

Then it will be enough.

Thank you for your suggestion, and MemoryCard will be back later in this pr.

Great to know MemoryCard will be back later in this pr, and you are welcome!

@huan MemoryCard back now, and do you have some other suggestions for this pr?

Great to see that, I have no more suggestions, thank you very much.


How to: PCIe Bifurcation on X99

Document the process of making bifurcation work on my new storage server (Asus Strix X99)
Useful links:

https://winraid.level1techs.com/t/guide-how-to-bifurcate-a-pci-e-slot/32279

Category: How To
Tags:

PCIe Bifurcation
TrueNAS


docs(storybook): refactor fileinput story template

This PR is about minor changes of the story of the File input component

Codecov Report

Merging #382 (9c1d581) into main (3410753) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #382   +/-   ##
=======================================
  Coverage   96.70%   96.70%
=======================================
  Files          61       61
  Lines         667      667
  Branches      106      106
=======================================
  Hits          645      645
  Misses         21       21
  Partials        1        1

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

:tada: This PR is included in version 1.10.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to convert webpack.config.js to accept .scss instead of .sass?

Hey everyone,
I really enjoyed Ben's blog post https://www.bensmithett.com/smarter-css-builds-with-webpack/ it's definitely an eye opening change for me and I'm really trying to figure out how to use web pack
I want to change the example to use .scss and basically use this as a scaffolding. I have a lot to learn but any guidance on how to at least get web pack to succeed in building this would be very much appreciated.
So far I've tried changing webpack.config.js, by adjusting the loaders. I've also changed the file extensions from .sass to .scss but the build continually failed. I'm getting this error:
ERROR in ./src/javascripts/footer.js Module not found: Error: Cannot resolve module 'stylesheets/modules/footer' in /Users/mb/srv/www/webpack-css-example/src/javascripts  @ ./src/javascripts/footer.js 11:0-37
I'd have one of these errors for all the respective modules, header, container, and clearfix.
Any advice?
Thanks Ben for the great tutorial!
Thanks everyone

{
  test: /\.scss$/,
  loader: ExtractTextPlugin.extract('style-loader', sassLoaders.join('!'))
}

This fixed it so I'm closing this issue

Hey @her (and anyone else who stumbles onto this issue!)
You probably also want to

remove indentedSyntax=sass from this line
add scss to the resolved extensions on this line

hey thanks @bensmithett
you're amazing, this helped a lot.
also wanted to say it's so cool how you have react in this repository. it's really helpful for someone like me who is just learning. thanks again!


Korean Translated README.md file Added

I've translated the README.md file so that more and more Koreans can use this library.
Thank you.

KOREAN Translated README


Calculator::Process() for node "facedetectionfrontgpu__ImageToTensorCalculator" failed: Only BGRA/RGBA textures are supported, passed format: 24

Having an issue with all mediapipe builds on my system. Everything compiles fine then when I run it, I get the BRGBA error
GLOG_logtostderr=1 bazel-bin/mediapipe/examples/desktop/face_detection/face_detection_gpu   --calculator_graph_config_file=mediapipe/graphs/face_detection/face_detection_mobile_gpu.pbtxt
Result:
I20201118 09:54:38.296219  8095 demo_run_graph_main_gpu.cc:51] Get calculator graph config contents: # MediaPipe graph that performs face mesh with TensorFlow Lite on GPU.
GPU buffer. (GpuBuffer)
input_stream: "input_video"
Output image with rendered results.  It passes through
the very first incoming image unaltered, and waits for downstream nodes
(calculators and subgraphs) in the graph to finish their tasks before it
passes through another image. All images that come in while waiting are
dropped, limiting the number of in-flight images in most part of the graph to
1. This prevents the downstream nodes from queuing up incoming images and data
excessively, which leads to increased latency and memory usage, unwanted in
real-time mobile applications. It also eliminates unnecessarily computation,
e.g., the output produced by a node may get dropped downstream if the
subsequent nodes are still busy processing previous inputs.
node {
calculator: "FlowLimiterCalculator"
input_stream: "input_video"
input_stream: "FINISHED:output_video"
input_stream_info: {
tag_index: "FINISHED"
back_edge: true
}
output_stream: "throttled_input_video"
}
Subgraph that detects faces.
node {
calculator: "FaceDetectionFrontGpu"
input_stream: "IMAGE:throttled_input_video"
output_stream: "DETECTIONS:face_detections"
}
Converts the detections to drawing primitives for annotation overlay.
node {
calculator: "DetectionsToRenderDataCalculator"
input_stream: "DETECTIONS:face_detections"
output_stream: "RENDER_DATA:render_data"
node_options: {
[type.googleapis.com/mediapipe.DetectionsToRenderDataCalculatorOptions] {
thickness: 4.0
color { r: 255 g: 0 b: 0 }
}
}
}
Draws annotations and overlays them on top of the input images.
node {
calculator: "AnnotationOverlayCalculator"
input_stream: "IMAGE_GPU:throttled_input_video"
input_stream: "render_data"
output_stream: "IMAGE_GPU:output_video"
}
I20201118 09:54:38.296731  8095 demo_run_graph_main_gpu.cc:57] Initialize the calculator graph.
I20201118 09:54:38.297314  8095 demo_run_graph_main_gpu.cc:61] Initialize the GPU.
I20201118 09:54:38.305438  8095 gl_context_egl.cc:158] Successfully initialized EGL. Major : 1 Minor: 5
I20201118 09:54:38.363840  8106 gl_context.cc:324] GL version: 3.2 (OpenGL ES 3.2 NVIDIA 455.45.01)
I20201118 09:54:38.363948  8095 demo_run_graph_main_gpu.cc:67] Initialize the camera or load the video.
I20201118 09:54:38.848876  8095 demo_run_graph_main_gpu.cc:88] Start running the calculator graph.
I20201118 09:54:38.849045  8095 demo_run_graph_main_gpu.cc:93] Start grabbing and processing frames.
INFO: Created TensorFlow Lite delegate for GPU.
ERROR: Following operations are not supported by GPU delegate:
DEQUANTIZE:
164 operations will run on the GPU, and the remaining 0 operations will run on the CPU.
I20201118 09:54:39.205772  8095 demo_run_graph_main_gpu.cc:175] Shutting down.
E20201118 09:54:39.238298  8095 demo_run_graph_main_gpu.cc:186] Failed to run the graph: CalculatorGraph::Run() failed in Run:
Calculator::Process() for node "facedetectionfrontgpu__ImageToTensorCalculator" failed: Only BGRA/RGBA textures are supported, passed format: 24
I am running ubuntu 18.04 / with a 2080ti. Nvidia drivers 455..
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Nov__3_21:07:56_CDT_2017
Cuda compilation tools, release 9.1, V9.1.85
Any help is greatly appreciated..

Hello,
I faced the same problem while testing the new palm_detection_gpu and hand_landmark_tracking_gpu graphs on Ubuntu desktop (mediapipe tag 0.8.0).
I tried severeal things:
1, Modified the demo_run_graph_main_gpu.cc source file to convert the camera_frame_raw with cv::COLOR_BGR2RGBA flag instead of cv::COLOR_BGR2RGB and then used mediapipe::ImageFormat::SRGBA to create ImageFrame
or
Added a SetAlphaCalculator to the entry level graph descriptor file.
These both solved the "Only BGRA/RGBA textures are supported, passed format: 24" but then with both solution I got an error stack like this:
I20201119 13:44:13.844982  3912 demo_run_graph_main_gpu.cc:88] Start running the calculator graph.
I20201119 13:44:13.848717  3912 demo_run_graph_main_gpu.cc:93] Start grabbing and processing frames.
INFO: Initialized TensorFlow Lite runtime.
INFO: Created TensorFlow Lite delegate for GPU.
 The graph could run and detect hands but it performed far worse than with CPU in the same version (tag 0.8.0) or the earlier versions with GPU.
While I was trying to find other solutions, I found a notification in the new TensorConverterCalculator:
// IMPORTANT Notes:
//  GPU tensors are currently only supported on mobile platforms.
So, I assume this is a known issue in the latest version.
I would also appriciate any help on this.

Just a quick update, related to the error stack mentioned in my last comment at point 1.:
As the error suggests, there is a mutex attempted to locked twice without releasing it.
It can be fixed by closing the scope in tensors_to_detections.cc (TensorsToDetectionsCalculator::ProcessGPU), row:435 and opening the new scope in the next row. So separate decoded_boxes_view and scored_boxes_view variables.
After this fix, the graph can run with GPU support, but sadly it still performs bad (in sense of frequency of detecting hands). The CPU version somehow performs way better on the same video sample. I didn't experience such difference at earlier versions.

Sorry, I made some more tests and eventually with the two fixes I mentioned before, the GPU and CPU version performs similarly.
In summary, these are the steps to fix the issue:
1,
a, Modify the demo_run_graph_main_gpu.cc source file to convert the camera_frame_raw with cv::COLOR_BGR2RGBA flag instead of cv::COLOR_BGR2RGB and then used mediapipe::ImageFormat::SRGBA to create ImageFrame
row101.: cv::cvtColor(camera_frame_raw, camera_frame, cv::COLOR_BGR2RGBA);
row108.: mediapipe::ImageFormat::SRGBA, camera_frame.cols, camera_frame.rows,
OR
b,
Add a SetAlphaCalculator to the entry level graph descriptor file. In your case: face_detection_mobile_gpu.pbtxt
change input_stream: "input_video" to input_stream: "input_videoraw" in row 4.
and add:
node {
calculator: "SetAlphaCalculator"
input_stream: "IMAGE_GPU:input_videoraw"
output_stream: "IMAGE_GPU:input_video"
node_options: {
[type.googleapis.com/mediapipe.SetAlphaCalculatorOptions] {
alpha_value: 255
}
}
}
In this case you need to change demo_run_graph_main_gpu.cc row 33 to constexpr char kInputStream[] = "input_videoraw";
Also do not forget to add the calculator in the build file: mediapipe/graphs/face_detection/BUILD to mobile_calculators for example to row 22 like this:
"//mediapipe/calculators/image:set_alpha_calculator",
2,
Fix the mutex issue by closing the scope in tensors_to_detections.cc (TensorsToDetectionsCalculator::ProcessGPU), row:435 and opening the new scope in the next row. So separate decoded_boxes_view and scored_boxes_view variables.

Wow this is great ! Thank you so much for your help.  Just a quick question, I am working on adding a feature to the desktop version of multi hands that labels left versus right.. I thought I saw this out there as a modification to a calculator at some point , but I don't se it anymore.. Any chance you have this around as well?

If I understand, you are looking for the mediapipe/graphs/hand_tracking/subgraphs/hand_renderer_gpu.pbtxt file.
There is a GateCalculator in line 156. which disallows rendering the handedness label when both hands are detected.
So the demo app renders handedness data to a fix position of the picture when only one hand is detected on the frame.
If you want to render the handedness around the related hands, you need to use the multi_hand_rects to position the labels.
So you need to edit this subgraph a little and also you need to extend LabelsToRenderDataCalculator functionality to be able to receive starting location (for example the rect of the hand) through an input stream and to offset the label starting position according to that. Currently it is a fixed parameter option in the graph descriptor file horizontal_offset_px and vertical_offset_px.

Yes sir, and I have both of those functions working, but the current hand landmarks only show 0 and 1 , where 0 is the first hand it detects and 1 is the second hand (regardless of whether it is left or right) . I have seen some code that uses the palm vector in relation to the thumb position to tell left from right.. I cannot find it now.. I was able to do it in python but for speed sake I would prefer to do it in the calculator as you mentioned.  Just struggling in C++ a bit with out things like Numpy.py etc.. was hoping it was lying around and I missed it. Again thank you for your prompt response and help! you have been great.

Confirmed that your solution solved my original issue, I did number 1 and I changed row 442 in tensors_to_detections_calculator.cc  from : glDispatchCompute(num_boxes_, 1, 1);
to :
glDispatchCompute(num_boxes_ + 1, 1, 1);

Follow up on the handness, I was compiling  the wrong version (too many adjustments trying to fix my original version) . I see exacty what you are talking about and I am going to attempt the modifications to label the hands above the hand images.

Great! I'm happy it helped!


Blocks Release: CSS for faceted search results needs work

There are a couple of things going on here. I can break out separate tickets if preferred, but the screenshot below shows:

green color for search results in faceting. Im not exactly against it but it is a bit unexpected and jarring. @sdellis any suggestions on what would/could work here?
Button for what was searched for (to the right of the blue "start over" button)

Take the green color off the header, but use prod green on text.


2.2.4 doesn't exists in npm package https://www.npmjs.com/package/material-ui-pickers

we are currently pulling in the branch by tag name yet the tag 2.2.4 is missing from this npm package: https://www.npmjs.com/package/material-ui-pickers.  Please fix if possible.  This is breaking our production app

https://codesandbox.io/s/cocky-greider-38fmi
Check it out. Everything works with material-ui-pickers v2.2.4. NOT @material-ui/pickers

@dmtrKovalenko  there is no tag it can't be pulled in.  you closed this too quickly....it's the fact that it's working, it's the fact that it can't be pulled in in the first place.


CoordinateTuple should be a trait, rather than a type

Problem
Currently, Rust Geodesy forces input data to be in the form of a slice of CoordinateTuples, which really is forcing the coordinate representation conventions of the library on to the application code calling it.
In PROJ, I eliminated this unfortunate characteristic in 2017, by introducing the proj_trans_generic(...) function in the 4D API (cf https://github.com/OSGeo/PROJ/pull/530).
With Rust having much better support for generic programming than C, one would think that it would be possible to do something better in the case of Rust Geodesy. I believe the demo code below shows that this is actually the case.
Solution
Essentially, we do four things:

Introduce the internal coordinate representation Coord, formerly known as CoordinateTuple:pub struct Coord(pub [f64; 4]);

Introduce the minimal CoordinateTuple trait, describing anything that can be both constructed from a Coord, turned into a Coord, and copied:pub trait CoordinateTuple: Copy + From<Coord> + Into<Coord> {}
impl<T: Copy + From<Coord> + Into<Coord>> CoordinateTuple for T {}

Introduce the CoordinateSet trait, describing a set of any kind of CoordinateTuples. Here CoordinateSet refers to the ISO19100 concept of coordinatesets, i.e. a collection of CoordinateTuples, all of the same base type.pub trait CoordinateSet {
    fn len(&self) -> usize;
    fn into_coord(&self, index: usize) -> Coord;
    fn from_coord(&mut self, index: usize, coord: Coord);
}

Switch the internals to work on generic CoordinateSets, rather than slices of CoordinateTuples

Conclusion
With this done, users of Rust geodesy may implement the two tiny traits above, for their particular data type, and RG will automagically support that type.
The code below includes two examples:

A generic plain coordinate set type, demoed as container for a 2-D single precision coordinate type.
A generic "Point Feature" coordinate set type, combining coordinates and attributes.

I believe the demo suggests that this will make Rust Geodesy much more generally useful.
Demo
/// Demo implementation of `CoordinateTuple` as trait. Makes it possible to
/// support transformations *directly on generic user-defined data types*,
/// as in the PROJ function `proj_trans_generic(...)`.
///
/// Compared to `proj_trans_generic` (introduced by yours truly on 2017-07-07
/// in https://github.com/OSGeo/PROJ/pull/530), `CoordinateTuple` as trait
/// is cleaner, clearer, and much safer.
///
/// Thomas Knudsen, 2022-01-02

// ----------------------------------------------------------------------------

/// `Coord` is the generic 4-D, RG-internal, double precision, coordinate
/// representation formerly known as `CoordinateTuple`
#[derive(Clone, Copy, Debug, Default)]
pub struct Coord(pub [f64; 4]);

/// `coord[0]` is a bit more readable than `coord.0[0]`, so we implement
/// the indexing traits for `Coord`
impl std::ops::Index<usize> for Coord {
    type Output = f64;
    fn index(&self, i: usize) -> &Self::Output {
        &self.0[i]
    }
}

impl std::ops::IndexMut<usize> for Coord {
    fn index_mut(&mut self, i: usize) -> &mut Self::Output {
        &mut self.0[i]
    }
}

// ----------------------------------------------------------------------------

/// A `CoordinateTuple` in the new trait sense, is anything that can be both
/// 1: copied, 2: constructed from a `Coord`, and 3: turned into a `Coord`
pub trait CoordinateTuple: Copy + From<Coord> + Into<Coord> {}
impl<T: Copy + From<Coord> + Into<Coord>> CoordinateTuple for T {}

// ----------------------------------------------------------------------------

/// `Xy` is an example of a user-defined coordinate representation.
/// `Xy` implements the CoordinateTuple trait by deriving `Copy`, and
/// providing `Into<Coord>` and `From<Coord>`.
/// `Xy` is 2-D, single precision, as may make sense for e.g. small
/// scale map data, where it will provide ample resolution, while
/// weighing only 1/4 of a `Coord`
#[derive(Clone, Copy, Debug, Default)]
pub struct Xy(pub [f32; 2]);

impl Into<Coord> for Xy {
    fn into(self) -> Coord {
        Coord([self.0[0] as f64, self.0[1] as f64, 0., f64::NAN])
    }
}

impl From<Coord> for Xy {
    fn from(coord: Coord) -> Xy {
        Xy([coord.0[0] as f32, coord.0[1] as f32])
    }
}

// ----------------------------------------------------------------------------

/// In the ISO-19100 standards series sense, a `CoordinateSet` is a set of
/// `CoordinateTuple`s, all of the same type.
/// The trait implementation is surprisingly simple: We just need to be able
/// to iterate over the contents, to extract each element as a `Coord`, and
/// to re-introduce the contents of the transformed `Coord` into the original
/// data element
pub trait CoordinateSet {
    fn len(&self) -> usize;
    fn into_coord(&self, index: usize) -> Coord;
    fn from_coord(&mut self, index: usize, coord: Coord);
}

// ----------------------------------------------------------------------------

/// In the simplest case, our CoordinateSet consists of a slice of raw
/// `CoordinateTuple`s
#[derive(Debug, Default)]
struct PlainCoordinateSet<'a, C: CoordinateTuple> (
    &'a mut [C]
);

/// Since the data type `T` implements the `CoordinateTuple` trait,
/// it is extremely simple to implement the `CoordinateSet` trait,
/// as we already have the necessary `From<Coord>` and `Into<Coord>`
/// implementations.
///
/// Note that due to the from/into duality, `From<Coord>` is not
/// explicitly used: The compiler auto-derives `Into<T> for Coord`
/// when it sees `From<Coord> for T`
impl<'a, T: CoordinateTuple> CoordinateSet for PlainCoordinateSet<'a, T> {
    fn len(&self) -> usize {
        self.0.len()
    }

    fn into_coord(&self, index: usize) -> Coord {
        self.0[index].into()
    }

    fn from_coord(&mut self, index: usize, coord: Coord) {
        self.0[index] = coord.into()
    }
}

// ----------------------------------------------------------------------------

/// This is a more realistic case of implementing `CoordinateSet` for a rather
/// generic class of potential user defined types, composed from a payload and
/// a coordinate tuple.

/// The element `PointFeature`
pub struct PointFeature<T: CoordinateTuple, P> {
    pub coordinatetuple: T,
    pub payload: P
}

/// ...a set of `PointFeature`s
struct PointFeatureSet<'a, T: CoordinateTuple, P> (
    &'a mut [PointFeature<T, P>]
);

/// And the implementation of the `CoordinateSet` trait for generic `PointFeatureSet`s
impl<'a, T: CoordinateTuple, P> CoordinateSet for PointFeatureSet<'a, T, P> {
    fn len(&self) -> usize {
        self.0.len()
    }

    fn into_coord(&self, index: usize) -> Coord {
        self.0[index].coordinatetuple.into()
    }

    fn from_coord(&mut self, index: usize, coord: Coord) {
        self.0[index].coordinatetuple = coord.into()
    }
}

// ----------------------------------------------------------------------------

/// Code taking a generic `CoordinateTuple` as arg
fn print_coordinatetuple<T: CoordinateTuple>(ct: T) {
    let coord: Coord = ct.into();
    println!("{:#?}", coord);
}

/// Dummy transformation, here using a vtable
fn transformation_of_a_coordinate_set(cs: &mut dyn CoordinateSet) {
    for i in 0..cs.len() {
        let mut co = cs.into_coord(i);
        co[0] += 2.;
        cs.from_coord(i, co);
    }
}

fn main() {
    // A vector of `CoordinateTuple`s
    let mut xxxx = vec![Xy([1.,2.]), Xy([3.,4.])];
    print_coordinatetuple(xxxx[1]);

    // Now turn the vector into a `PlainCoordinateSet` (or perhaps any other
    // type, T, implementing the `CoordinateSet` trait), in order to be able
    // to transform it.
    //
    // Note that the life time of the `PlainCoordinateSet` struct need not be
    // any longer than the run time of the transformation, so we generate it
    // inline with the function call.
    transformation_of_a_coordinate_set(&mut PlainCoordinateSet(&mut xxxx));
    dbg!(&xxxx);

    // But of course we may also generate it prior to the call
    let mut xyxy = PlainCoordinateSet(&mut xxxx);
    transformation_of_a_coordinate_set(&mut xyxy);
    dbg!(&xyxy);
}

I would expect a basic API like
for coord in mygeom {
    let (x, y) = transformer.transform((coord.0, coord.1));
    // ...
}

which returns transformed coordinates as a copy. Would the trait API be more efficient in transforming a list of coordinates, or is the goal to support transforming geometry coordinates in-place?
The latter case would probably look like
transformer.transform_all_coords(mygeom);

Fore me personally, the single coord API would be good enough.

Time to reconsider this, given https://github.com/georust/geo/issues/901 and https://github.com/georust/geo/issues/898

Time to reconsider this, given https://github.com/georust/geo/issues/901 and https://github.com/georust/geo/issues/898

Closed with the merging of CoordinateSet #40


make book demo applications in one namesapce

I have run osm demo and found that osm create one namespace for each application,like that bookbuyer,bookstore,bookief,bookwarehouse.
I think deploy the demo in one namespace can make it more simplicity and more suitable for the use of namespace.

\assign

I think i can try to resolve it. someone can assign it to me ?

The reason the demo uses multiple namespaces is because there are scenarios we want to showcase communication between traffic in different namespaces.
That being said, the app namespaces are driven via environment variables, so by default we could set the app namespaces to be the same: https://github.com/openservicemesh/osm/blob/main/demo/run-osm-demo.sh#L20-L23


Display issue with SSD1306 and SPI?

With my  SPI SSD1306 128x64 OLED display, the examples don't render correctly. Here's a shot of maze.py.
Has SPI with SSD1306 been tested with recent releases? Looks like it could be trying render RGB?

Pasting your image here for easier reference

So I dont think it is an RGB issue: it is more likely that the init sequence (see https://github.com/rm-hull/luma.oled/blob/master/luma/oled/device.py#L128-L135) is not quite right, the lower part of the image looks like it is not being written to, hence all the random dots.
I dont have an SPI version (only the I2C versions) - however I have a feeling that the SPI versions are mostly SH1106 devices, and if you look here you'll see it has a different init sequence, and its memory is mapped differently.
Can you try and run the demo with python examples/maze.py -d sh1106 --spi-port=... etc instead to see if that solves it?

Thanks for the fast reply!
It works perfectly with -d sh1106 :) I was focusing on that the display worked back when the library was called "ssd1306" but forgot that SH1106 support had already been added.
Would you like me to add this info and the screenshot to a troubleshooting section in the docs?

Yes please, that would be great !

Done. I did a quick check on eBay but didn't find a good correlation between SPI / I2C  and SH1106 / SSD1306, so I just left that out of the troubleshooting note.

Thanks.
I'll leave this here if anyone's google-fu brings them here.
http://www.waveshare.com/wiki/1.3inch_OLED_(B) is sh1106
while it's 0.96 inch counterparts is ssd1306


cannot handle import map in other deno package

e.g.
$ deno run -A main.ts ../with-help/mod.ts
error: Uncaught (in promise) Error: Unable to output during bundling: load_transformed failed: failed to analyze module: failed to resolve @std/cli/parse-args from file:///home/po/ghq/github.com/podhmo/with-help/parse-args.ts: Cannot resolve "@std/cli/parse-args" from "file:///home/po/ghq/github.com/podhmo/with-help/parse-args.ts".
      const ret = new Error(getStringFromWasm0(arg0, arg1));
                  ^
    at __wbg_new_28c511d9baebfa89 (https://jsr.io/@deno/emit/0.46.0/emit.generated.js:557:19)
    at <anonymous> (wasm://wasm/010a1b62:1:3287273)
    at <anonymous> (wasm://wasm/010a1b62:1:429097)
    at <anonymous> (wasm://wasm/010a1b62:1:1820413)
    at <anonymous> (wasm://wasm/010a1b62:1:2928941)
    at __wbg_adapter_46 (https://jsr.io/@deno/emit/0.46.0/emit.generated.js:247:6)
    at real (https://jsr.io/@deno/emit/0.46.0/emit.generated.js:231:14)
    at ext:core/01_core.js:291:9
    at eventLoopTick (ext:core/01_core.js:175:7)

fixed by #7


Background service

Create a background service, its job is processing logs, aggregate statistics .etc, something like the dirty job robot.
It will be implemented as Console application.

The application name would be Jarvis.

The fundamental has been created, so close this issue now. Will open additional ones for individual  job.


First pass of a helm chart for kubernetes tentacle

Background
Part of #project-k8s-agent
We are adding a new helm chart to be used by customers wanting to install the new Kubernetes Tentacle. This helm chart will be used by an install script which will be generated by a wizard in octopus when users request to install a k8s target. This will make the script more succinct and make it easier for customers to manage.
Results
This is the first pass for the helm chart. It currently works for Polling tentacles only with an input close to this:

(It also works with serverApiKey in place of bearer token)
After this is in, we can iterate on it and add functionality. The next thing to add will be to create the appropriate rules for the service account so that it can issue jobs.

@APErebus As discussed, I've added and updated the READMEs. Does this draw enough attention?

This is the main README (Should I put the Caution there too?)


Is there a way to set MaxBytes for tendermint through burrow config?

I want to limit the block size for tendermint. Can we do that through some burrow config?
For example setting MaxBytes.
https://github.com/tendermint/tendermint/blob/master/types/params.go#L43

We are intentionally opinionated about what we expose from Tendermint, however if you wish to add this please see the configuration setup here.

Could you tell us a bit about your motivation for doing this, what are you trying to achieve? This can help us to understand it it should be part of a broader feature.

Closing for lack of response


WIP: Do not reject entire batch of updates, if items get past validation.

Missing tests at the moment.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ozevren
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ymesika
If they are not already assigned, you can assign the PR to them by writing /assign @ymesika in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: costinm, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [costinm]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
82e51cbca154069b367901538ce38d60ee138262
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
82e51cbca154069b367901538ce38d60ee138262
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
82e51cbca154069b367901538ce38d60ee138262
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Fixes: https://github.com/istio/istio/issues/12474

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
82e51cbca154069b367901538ce38d60ee138262
link
/test istio-pilot-multicluster-e2e

prow/istio-integ-k8s-tests.sh
004100fcadeaa13baf676f6eac36de74dc6a43f8
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
004100fcadeaa13baf676f6eac36de74dc6a43f8
link
/test istio-integ-k8s-tests

prow/istio-pilot-multicluster-e2e.sh
004100fcadeaa13baf676f6eac36de74dc6a43f8
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

The e2e-pilot-cloudfoundry-v1alpha3-v2 problem seems unrelated.
--- FAIL: TestWildcardHostSidecarRouterWithMockCopilot (0.00s)
	copilot_test.go:223: internal backend is running on port 6868
panic: discovery service: listen tcp <IP_ADDRESS>:15010: bind: address already in use [recovered]
	panic: discovery service: listen tcp <IP_ADDRESS>:15010: bind: address already in use

The codecov failure seems unrelated:
  main_test.go:216: Coverage dropped:
        istio.io/istio/pilot/pkg/proxy/envoy/v2/rds.go:-12.200000% (73.200000% to 61.000000%)
=== RUN   TestGoPath

Will be in 1.1rc5


Sync editor issues

stolostron/backlog#21945
stolostron/backlog#21995
stolostron/backlog#22010
https://github.com/stolostron/backlog/issues/21611

/lgtm

/lgtm


Docs.MS Release Updates for azure-spring-boot-starter-active-directory

Update docs metadata and targeting for release of azure-spring-boot-starter-active-directory

Docs Build status updates of commit 296ee74:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-services/latest/spring-boot-starter-active-directory-readme.md
:warning:Warning
View (azure-java-stable)
Details

docs-ref-services/latest/spring-boot-starter-active-directory-readme.md

Line 2, Column 1: [Warning-ms-prod-and-service] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 9, Column 16: [Warning-ms-prod-technology-invalid] Invalid value for 'ms.technology': 'azure' is not valid with 'ms.prod' value 'azure'.
Line 11, Column 13: [Warning-ms-service-subservice-invalid] Invalid value for 'ms.service': 'springbootspringboot'.
Line 51, Column 1: [Warning-file-not-found] Invalid file link: 'resource/web-application.png'.
Line 57, Column 1: [Warning-alt-text-bad-value] Alt text 'web-application-set-redirect-uri-1.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 57, Column 1: [Warning-file-not-found] Invalid file link: 'resource/web-application-set-redirect-uri-1.png'.
Line 58, Column 1: [Warning-alt-text-bad-value] Alt text 'web-application-set-redirect-uri-2.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 58, Column 1: [Warning-file-not-found] Invalid file link: 'resource/web-application-set-redirect-uri-2.png'.
Line 113, Column 1: [Warning-alt-text-bad-value] Alt text 'web-application-visiting-resource-servers.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 113, Column 1: [Warning-file-not-found] Invalid file link: 'resource/web-application-visiting-resource-servers.png'.
Line 164, Column 1: [Warning-file-not-found] Invalid file link: 'resource/resource-server.png'.
Line 193, Column 1: [Warning-alt-text-bad-value] Alt text 'get-app-id-uri-1.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 193, Column 1: [Warning-file-not-found] Invalid file link: 'resource/get-app-id-uri-1.png'.
Line 194, Column 1: [Warning-alt-text-bad-value] Alt text 'get-app-id-uri-2.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 194, Column 1: [Warning-file-not-found] Invalid file link: 'resource/get-app-id-uri-2.png'.
Line 221, Column 1: [Warning-alt-text-bad-value] Alt text 'resource-server-visiting-other-resource-servers.png' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 221, Column 1: [Warning-file-not-found] Invalid file link: 'resource/resource-server-visiting-other-resource-servers.png'.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.
Line 2, Column 8: [Suggestion-duplicate-titles] Title 'Azure AD Spring Boot Starter client library for Java' is duplicated in: 'docs-ref-services/latest/active-directory-spring-boot-starter-readme.md(2,8), docs-ref-services/latest/spring-boot-starter-active-directory-readme.md(2,8)'. Titles must be unique within a docset.
Line 30, Column 3: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: https://docs.microsoft.com/java/azure/jdk/?view=azure-java-stable. Otherwise, remove the view parameter.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


[Feature] Server Integration

Idea
There’s currently an autoresponse that allows users to post a log and Payload will render a preview of that. In this case, running a plugin on a server will allow users to automatically post logs into discord.
Integration
A new command (pls server -> change pls server to pls rcon)
This would send the user a unique ID (probably from uuid- or some other library) that would be used instead of a discord channel. This would link the channel to the unique ID, so no one could steal and abuse that (like making their own server, and grabbing a channel and sending bad stuff through the api).
A command to remove the ID would be needed, and probably limiting the bot to 1 ID per discord server would be ideal. An edit command as well would be useful, so you can edit the channel (maybe this would keep the same ID?)
Api Changes
POST /api/logs
Returns status (401, 500, 200 ect).
Potentially include the channel it was sent to, or maybe the id?
Use Authorization header with their included key to send over the network.

Seeing how powerful JWT's are, could create a JWT service on the backend to parse any JWT coming in.
JWTs should be considered safe on the Client side, and creating one is very simple.
New plan: Create a JWT that encodes the Guild ID and Channel ID. Next, give that to the user. On the server plugin, you would paste in that token (would be a long one perhaps?).
When sending a request to the API, send that token along with it inside the body. Server would then parse that token, have the guild and channel id to send it, and it can send it directly without any database interaction to compare keys with or any of that nasty thing.
One caveat is to remove a token or "revoke" access of a token. There could be an expiration date, but I see some people not liking the idea of setting their token every month or so, it would get annoying. I could store this key in the database, and use that to see if the key matches the one in the database. When deleted or not present, it would nullify the request. Only DB interaction is 1 to see if that guild has a token. Probably would want to compare both tokens.
An advantage gained by this approach is that we could encode a token to encapsulate a user instead of a guild. This means that you could deliver logs to your DMs every time you play. Same problems arise stated above. I can see this being used if people want to have logs of every game played on their match server, or if people enjoy logs that much. Lots of potential with JWT instead of a UUID.

Moved to ideas on idea board


pytest==5.1.2 breaks test_diff_with_editable on py35+ (windows)

Latest pytest==5.1.2 breaks test_diff_with_editable. I've tested with pytest==5.1.1 and it works fine. According to the changelog most likely it caused by https://github.com/pytest-dev/pytest/issues/5792.
Environment Versions

OS Type: Windows
Python version: 3.5+
pip version:  any
pip-tools version: pip-compile, version 4.1.0

Steps to replicate

Run tox and make sure it installs pytest==5.1.2.

Expected result
Test test_diff_with_editable passes.
Actual result
Test test_diff_with_editable fails.
Possible solutions
We need to fix the test, as well as pin test requirements.

@blueyed
I see you have reviewed https://github.com/pytest-dev/pytest/issues/5792 recently, do you have any insights how it affects the test?

FTR, it's fixed in https://github.com/pytest-dev/pytest/pull/5840


IPv6 Support

Please describe which feature you have a question about?
AWS Amplify DNS Records
Provide additional details
Are AAAA records for IPv6 supported in AWS Amplify?
What AWS Services are you utilizing?
AWS Amplify Hosting with CI/DI
Provide additional details e.g. code snippets
Is IPv6 turned on by Amplify Hosting in CloudFront distribution? Can I just create an AAAA record in DNS and point it to the same CloudFront distribution alias as A records?

Commenting because I'm also interested in this.

I opened a support case to AWS for this. Below was their reply. I will have to test this myself and see.

The team informed that you can host a static site on Amplify and have that site be available via an IPv4 and IPv6 address. Amplify’s hosting solution leverages Amazon CloudFront and Amazon S3 to deliver your site assets via the AWS content delivery network (CDN). Amazon Simple Storage Service (Amazon S3) supports the ability to access S3 buckets using the Internet Protocol version 6 (IPv6), in addition to the IPv4 protocol. Amazon S3 dual-stack endpoints support requests to S3 buckets over IPv6 and IPv4.

Getting back around to this. I don't think Amplify hosting is IPv6-ready. I uploaded a test site and the auto-generated URL is failing IPv6 tests. Checking DNS, I only see A records, no AAAA records. I also don't see a way to enable IPv6 in the Amplify web interface.

I just landed here after setting up my first AWS Amplify site. I can confirm that IPv6 is still not natively supported, but can be manually set up.
Here's my workaround:

Go to Route 53 dashboard and navigate to your Hosted Zone.
Find an A record that's an Alias for a CloudFront distribution. Click it open to see the "something.cloudfront.net" URL it's pointing to.
Create a new AAAA record and make it an Alias for a CloudFront distribution. Enter the same URL you just saw. It won't be autocompleted because the distribution is managed by Amplify.


fixes bug 1290445 - Ability to upload and transcode closed captions

Coverage decreased (-0.2%) to 84.805% when pulling f7ecb5c61437f97632e6ccd4657acc9939a7a259 on peterbe:bug-1290445-ability-to-upload-and-transcode-closed-captions into ec9bb8616aeab21213f6a14554c4585f4710611b on mozilla:master.

Coverage decreased (-0.2%) to 84.805% when pulling 504528ce9b4b7e8af210ad385087c901307be8c2 on peterbe:bug-1290445-ability-to-upload-and-transcode-closed-captions into ec9bb8616aeab21213f6a14554c4585f4710611b on mozilla:master.

Coverage increased (+1.0%) to 85.941% when pulling 4a81ca67858888b920bac29803914124f6911ee0 on peterbe:bug-1290445-ability-to-upload-and-transcode-closed-captions into ec9bb8616aeab21213f6a14554c4585f4710611b on mozilla:master.


[TestServer] Isolated and Integration Testing

Hello,
In order to do integration testing with our Azure Function we use some code like this as we are in  not isolated mode :
 var host = new HostBuilder()
            .ConfigureWebHost(web => web.UseTestServer().Configure(_ => { }))
            .ConfigureWebJobs(c =>
            {
              // some specific configuration
            }).Build();
await host.StartAsync();
var jobs = host.Services.GetRequiredService<IJobHost>();

and from jobs we use CallAsync to execute a function.
It would be great to be able to do some thing close to this or even simpler for Isolated function.
I m not sure where the change could be done but from what I see it could be done with a custom IWorker Implementation or by using testserver for grpc part.

I'm confused, @bhugot. Isn't Isolated and Out of Process the same SDK?

Yes sorry forgot to change it. I fixed original message

Some of this work was started about a year ago but was put on hold for various reasons -- we'll discuss bringing this back into active development.
https://github.com/Azure/azure-functions-dotnet-worker/issues/281

We've just re-introduced the original issue into our planning for next Sprint. I'm going to close this one in favor of the older one. Thank you for bringing this back up.


improve identity deletion

disable all current login tokens
ignore new mention
ignore new follow
delete all data
fanout delete posts, interactions
fanout unfollow both directions
fanout nullified identity, then fanout delete identity
keep nullified identity for 7 days then delete forever

Great - get the lint failures passing (just run pip install pre-commit; pre-commit run -a locally if you've not installed the hooks yet) and I think we can merge this in.

Thanks!


Optimize hooks a little

Overview
This is a tricky one because I don't know if it 100% works.
Basically the premise is to do two things:

Remove the setTimeout logic out of useCoordinates and move it to the apply wrapper. Thanks to that, way less time is spent just waiting for the timeout to happen.
Introduce a new useAreCoordinatesSelected hook to mitigate the need to run heavy comparison for every cell click in useCellIsSelected. Now it's going to return a boolean and should make things quicker for larger tables.

Risks
Point 1. is the risk-prone one. I've experimented with a couple of possible solutions, including using lodash.debounce instead of setTimeout, but then I ended up at what's in the PR and it still worked, so I stayed with that. I have not seen it do anything weird but then again, testing npm packages locally isn't an easy task.
If we vote against Point 1, Point 2 definitely seems like an easy win, and I'd be happy to cherry-pick just that change.
Testing
See if coordinates stay updated. Monitor the speed of useCellIsSelected.

This looks really promising. I'm going to push a couple stylistic changes and do some testing, and maybe we can get this through!


Amplify repeatedly throwing -10878

Describe the bug
Upon launch of my application, I fetch the current user from DynamoDB by using a simple get request by fetching the user by id.
I get back a successful response, and my app is still fully functional however this log just concerns me and I want to make sure I am not doing anything wrong.
None of the error blocks are hit in the code snippet below, a successful response is always returned when this is logged out.
 func get<T: Model>(id: String, completion: @escaping (Result<T?, AppError>) -> Void) {
        Amplify.API.query(request: .get(T.self, byId: id)) { event in
            do {
                let response = try event.get()
                switch response {
                case .success(let entity): completion(.success(entity))
                case .failure(let error):
                    print(error)
                    completion(.failure(.network(type: .custom(errorDescription: error.debugDescription))))
                }
            } catch {
                print(error)
                completion(.failure(.network(type: .custom(errorDescription: error.localizedDescription))))
            }
        }
    }

Steps To Reproduce
Steps to reproduce the behavior:
1. Launch application
2. Fetch object by ID
3. Error -10878 is logged out

Expected behavior
Expected behavior is to successfully retrieve the object without this alarming error log.
Amplify Framework Version
1.15.3
Amplify Categories
API
Dependency manager
Cocoapods
Swift version
5.0
CLI version
4.41.2
Xcode version
13
Relevant log output
`2021-11-02 09:00:56.301160-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.303492-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.303639-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.303775-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.303928-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.304039-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.304190-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.304315-0400 AppName-Staging[3287:884891] throwing -10878
2021-11-02 09:00:56.304434-0400 AppName-Staging[3287:884891] throwing -10878`

Is this a regression?
No
Regression additional context
No response
Device
iPhone 12 Pro Max
iOS Version
iOS 15
Specific to simulators
no
Additional context
No response

Hi @michaeljajou, where is the "throwing -10878" coming from? do you have a stack trace?

@michaeljajou Could you set a couple of breakpoints to identify where this is happening? You can see a few options. I suggest setting the Exception Breakpoint and Runtime Issue Breakpoint. When it reaches the point where it throws this error code please open the Debug Navigator in the left panel in Xcode and expand each of the threads then copy all of them and paste it into a text file which you can attach to a comment here. That would give us a lot of the details we need.

Closing due to inactivity.


Searching for katas of multiple difficulties results in a wrong number of katas listed

Filtering katas by difficulty you can see that there're 39 1 kyu katas, and 64 2 kyu katas. If you search for katas in a 2-1 kyu difficulty range, there will be only 39 katas listed - the same amount as the number of katas of the highest difficulty in the selected range. Using other combinatations has the same result: 39 katas with any minimum difficulty and 1 kyu maximum difficulty, 64 katas with any minimum difficulty and 2 kyu maximum difficulty etc.

Is this #936?

Yes. GitHub did not suggest that issue as a possible duplicate :(


this dataset  for point cloud unsupervied domain adaptation semantic segmentation?

hello author, thanks to your great work. I want to know that if the dataset can be used for point cloud unsupervised domain adaption semantic segmentation research. it's important for me, thanks again.@mattiasegu @fyu@suniique

Hey @shuchao97, thanks for your interest in SHIFT!
If I understand your question correctly, you are looking for point cloud data with domain changes. If so, we are sorry to tell you that the LiDAR data in our dataset are not sensitive to domain shift (e.g., rain, fog, …), as the CARLA LiDAR simulation is not affected by different weather conditions. However, one of our plans is to include physics-based models for LiDAR simulation under different weather. Let's keep in touch!

@suniique Thank you for the great dataset, I am working on a similar weather shift task (both with Carla simulator and without Carla) and would like to know whether there's some progress from your side ? Would be happy to collaborate on this as well!
Also, as a side question, is there any possibility to get my hands on the code for generating SHIFT dataset ?


SimpleProperties, SD save function will not create the file specified;

Describe the Issue
If file does not exist on the SD card, save function does not appear to be able to create it.
Library Name and Version:
Name: SimpleProperties
Version: V1.0.4
Environment:

Platform/OS: [e.g., Windows, macOS, Linux] Windows
Arduino IDE version (if applicable): NA
PlatformIO version (if applicable): NA
Development board (if applicable): Arduino Uno, Simulated Arduino Uno

Library Source:

[x] GitHub
[x] PlatformIO Registry

Expected Behavior
File should be created if it does not exist, then the items of the properties should be added to the file.
Actual Behavior
An error occurs but system continues running anyway.
To Reproduce
Steps to reproduce the behavior:

Include library '...'
Initialize the library '...'
See error

Error Message & Code Snippet
If applicable, add the error message and the piece of code that causes the issue.
// Your code that triggers the error
Screenshots
If applicable, add screenshots to help explain your problem.

Additional Context
Add any other context about the problem here.

<!-- End of the issue report template. -->

Issue seems to be resolved. Issue patched in Github Release. Issue will be resolved in next Platformio release.


Craft 5

This makes the few changes needed for Craft 5.
Not sure how you want to handle branching between versions so I made the PR against master for now.

Thanks @janhenckens we'll take a look and thank you for the help on this one 🎉 We should be able to get to this later this or next week

Any update on this? Fantastic plugin. Would love it for Craft 5 too.

The scrutinizer error is due to the settings of your test environment not matching Craft 5 requirements.
I’m currently building a site that would greatly benefit from this visual solution instead of a boring menu list, please do the merge :)

Fantastic thanks for this. :-)


Fix command for resetting user profile

Fix command for resetting user profile

@pascalberger,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/ide/synchronized-settings-in-visual-studio.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks!


nonCoherentAtomSize and allocation size

Does the library guarantee that for non-coherent host visible memory allocation size will be multiple of nonCoherentAtomSize?
From implementation I can see that allocation offset is aligned according to nonCoherentAtomSize but I can't figure out if size is also adjusted. I've checked documentation for vkGetXXXMemoryRequirement functions and it doesn't mention that returned size takes into account potential usage of non-coherent memory and any guarantee that size will be multiple of nonCoherentAtomSize.

You are right. Allocation size is not aligned to nonCoherentAtomSize and neither is the size of entire VkDeviceMemory block, but I think that's OK. All allocations in a non-coherent memory type are aligned to nonCoherentAtomSize, so there is never a situation where two allocations share same "atom size". Flush/Invalidate calls align their offset and size to nonCoherentAtomSize.
If you find a proof that this behavior is wrong according to Vulkan spec and should be changed, please let me know.


Fix several lexer bugs involving special comments

What problem does this PR solve?

Interior of /*! … */ should use the same SQL mode as outside
Fixed #330.
Fixed #336.

What is changed and how it works?

Makes the specialCommentScanner inherits the SQL mode of the parent Scanner
Makes the parent Scanner use the specialCommentScanner's stmtTexter if exists
Force advance the lexer when parsing optimizer hints, even when an invalid token is found.

Check List
Tests

Unit test

Code changes
Side effects
Related changes

PTAL @lysu

LGTM
Good job! @kennytm

👍


Automatic Deployment Steps with Vercel, Staging

Automatic Deployment Steps with Vercel:

Login to Vercel:

Go to Vercel Dashboard and log in to your Vercel account.

Create a New Project:

Click on the "Add" dropdown and select "Project."

Import Repository:

If your repository is not listed, click on "Missing Git repository?" and adjust GitHub App Permissions to add the repository to Vercel.

Enter Project Name:

Enter a name for your project.

Framework Detection:

Vercel will automatically detect the framework. If not, select "Next.js" as the framework.

Deploy:

Click the "Deploy" button.

Automatic Framework Detection:

If the framework is automatically detected, proceed to deployment.
If not, you may need to specify the build command using Yarn. Ensure your project is configured to build correctly with the following command:yarn build

Monitor Deployment:

Vercel will initiate the deployment process. Monitor the progress on the Vercel dashboard.

Verify Deployment:

Once the deployment is complete, visit the provided URL to verify that the application is live.

Not relevant now


happo dev command not supported with some plugins

Currently, none of the happo-plugin-storybook and happo-plugin-gatsby plugins work when running happo dev. The dev command mostly makes sense when you have your own -happo.js files, so it might not make sense to fix support -- we might just have to bail with some useful error message.

I think the dev command seems like it'd be useful for storybook still. Aren't stories in storybook essentially analogous to -happo.js files in the end?

I think the dev command seems like it'd be useful for storybook still. Aren't stories in storybook essentially analogous to -happo.js files in the end?

@monokrome They definitely are. Getting happo dev to work for storybook integrations is a little more tricky though because of how the integration is set up. Basically happo delegates preparing a static package to storybook (i.e. build-storybook) and then that package is sent to happo.io for screenshots. In a regular Happo examples integration we are in control of the build process, which means we can single out -happo files and make smaller packages, which makes the dev experience better as we're getting screenshots back quicker.
I've been thinking about a slightly different dev experience for Storybook projects with happo where we would have a button in the Storybook UI to quickly check what this components look like in different browsers, and then you can incrementally make changes to the component and get diffs along the way. Do you think this would be interesting? If so, maybe we can outline/sketch out the feature here?

@monokrome They definitely are. Getting happo dev to work for storybook integrations is a little more tricky though because of how the integration is set up. Basically happo delegates preparing a static package to storybook (i.e. build-storybook) and then that package is sent to happo.io for screenshots. In a regular Happo examples integration we are in control of the build process, which means we can single out -happo files and make smaller packages, which makes the dev experience better as we're getting screenshots back quicker.
I've been thinking about a slightly different dev experience for Storybook projects with happo where we would have a button in the Storybook UI to quickly check what this components look like in different browsers, and then you can incrementally make changes to the component and get diffs along the way. Do you think this would be interesting? If so, maybe we can outline/sketch out the feature here?

I've had an initial look at the outlined dev experience (button in the Storybook UI) and these are some thoughts and challenges:

In Storybook dev mode (start-storybook), there's no ready-made package we can send to happo.io. I'm not 100% sure how the dev storybook server works, but I believe it might be generating bundles on the fly -- I couldn't find the files referenced from the storybook web app in the filesystem.
Instead of sending a package (which is what the plugin normally does) we could send html+css payloads to happo.io. But that would make it behave different from what a full CI run.
Perhaps we could use something like ngrok to set up a tunnel to the local development server 🤔? Then we would be able to talk to the local dev server from happo.

I've had an initial look at the outlined dev experience (button in the Storybook UI) and these are some thoughts and challenges:

In Storybook dev mode (start-storybook), there's no ready-made package we can send to happo.io. I'm not 100% sure how the dev storybook server works, but I believe it might be generating bundles on the fly -- I couldn't find the files referenced from the storybook web app in the filesystem.
Instead of sending a package (which is what the plugin normally does) we could send html+css payloads to happo.io. But that would make it behave different from what a full CI run.
Perhaps we could use something like ngrok to set up a tunnel to the local development server 🤔? Then we would be able to talk to the local dev server from happo.

By the way, this is what I had in mind for the Storybook UI (in its simplest form):

If you hit that button, we would show one or more screenshots in that box. If you hit it again, we would compare the new screenshots with the previous ones.
There are of course other features we could consider down the road. Here are some ideas:

Directly comparing the screenshot(s) to what's already on the main branch.
Allowing you to edit the browser targets.

By the way, this is what I had in mind for the Storybook UI (in its simplest form):

If you hit that button, we would show one or more screenshots in that box. If you hit it again, we would compare the new screenshots with the previous ones.
There are of course other features we could consider down the road. Here are some ideas:

Directly comparing the screenshot(s) to what's already on the main branch.
Allowing you to edit the browser targets.


hero-animation-added

Hero animation on Page transition

Thanks a lot for this PR but this sample cannot be added to the list as it's quite basic and doesn't add value. Feel free to work on it and come up with something new and awesome.


Docker image missing cuda libs

When using a docker image created from the repo, libs like cusolver and libcudnn are not installed.  I get the following set of warnings, ending with "Skipping registering GPU devices".
Was the docker image meant to include these libs?  I see libcusolver.so.10 (a different version than listed in the warning), in the 11.0-runtime image (the Dockerfile is based on 11.0-base).  And cudnn would require an additional install step.
I0718 14:20:35.488188<PHONE_NUMBER>18464 run_docker.py:180] 2021-07-18 14:20:35.487292: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcusolver.so.11'; dlerror: libcusolver.so.11: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
I0718 14:20:35.632766<PHONE_NUMBER>18464 run_docker.py:180] 2021-07-18 14:20:35.632116: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libcudnn.so.8'; dlerror: libcudnn.so.8: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
I0718 14:20:35.633013<PHONE_NUMBER>18464 run_docker.py:180] 2021-07-18 14:20:35.632167: W tensorflow/core/common_runtime/gpu/gpu_device.cc:1766] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
I0718 14:20:35.633130<PHONE_NUMBER>18464 run_docker.py:180] Skipping registering GPU devices...

I am also encountering the same warning message. For additional context, I am attempting to run this on a GCP machine with the nvidia-gpu-cloud-image-20200629 boot disk image, with two nvidia-tesla-v100 GPUs attached.

I see libcusolver.so.10 (a different version than listed in the warning), in the 11.0-runtime image (the Dockerfile is based on 11.0-base). And cudnn would require an additional install step.

See https://github.com/deepmind/alphafold/issues/11. Installation of libcudnn and libcusolver is not included in the Dockerfile.

Thanks for pointing me to the duplicate.  It looks like #11 is an error/crash and using a different cuda version then provided in the repo (11.1 vs 11.0).
In our setup, with the provided 11.0 image, I get the above reported logging errors, but the process still completes.
I hope DeepMind wants the world to be able to reproduce what they did, so I'm curious what their recommended solution is.

I am also curious as to whether people have had success getting the released Dockerfile to work with GPUs within a CUDA 11.0-based environment.
So far, I have been able to get this Dockerfile working without GPU support (at least, with the same "Skipping registering GPU devices" warnings noted above). I have also gotten this to work without GPU warnings by upgrading the Dockerfile to CUDA 11.1. But ideally I would like to run this with GPU support in a CUDA 11.0-based environment, which seems like it should be possible given the description of this release.

Given the requirement of tensorflow-2.5.0 and the TF compatibility matrix, it seems it should be using cuda-11.2.
The solution I am now trying is #36 , which updates the base image to use nvidia's cuda 11.2 runtime, which brings in the missing libraries and uses versions in agreement with the compatibility matrix.

Hi,
Thank you for the report and discussion. Please see if acf25fc87964cab56b79cc5f5a4929c3805ecbeb fixes this issue and reopen if not. This change moves to using the cudnn base image and works in testing (though we also didn't encounter this issue in testing, so I'm not sure exactly what's causing it).


Dashboard usability study

Overview
The objective of this usability test is to learn if the new data dashboard helps users see the facilities that are reporting to them and how often. The focus will be on successfully completing tasks with a sub-goal of learning more about the dashboard use broadly.
Low current use of tools like this means that metrics are not a helpful gauge of success and moderated usability tests will provide more information to iterate on quicker. The test will also support growing in our key task completion metrics and future iterations for UP.
Participants
Beth Hopkins - MT
Ann Kayser - MN
Others pending
To do

[x] Email/schedule session with STLT/partner
[x] Tailor the facilitation guide
[x] Read background materials and previous emails
[x] Confirm schedule and set up team for notetaking and recording
[ ] Conduct call (facilitation, notes, recording)
[ ] Upload any documentation notes and recording
[ ] Identify high-level findings, put in shared document
[ ] Share in Slack
[ ]

Links
Research Plan

At this point we have 2 STLTs participating. Will make another attempt today to recruit an additional ~3 via email. Also considering doing some cold usability studies with internal staff to simply identify general usability needs.


use Backup CR labels as tags for snapshots

This allows users to define custom tags to be added to snapshots, by
specifying custom labels on the Backup CR with the velero backup create --labels flag.
closes #1132
Signed-off-by: Adnan Abdulhussein<EMAIL_ADDRESS>

LGTM - let's include a changelog for this one.


Webpage for UCP?

I have proposed a simple webpage design for UCP here. Complete with installation instructions, information on aic values, contributing info, and all the current wiki information.
It can be put under version control in this repo by using gh-pages branch (will be separate from the rest of the code base)

Definitely yes!
Due to the technical limitations, the wiki sadly sucks. Additionally I see new people getting lost if they do not directly land on the github releases page with the way too small download buttons. Maybe put the mod additionally on moddb.com for more people to notice this project?

Glad you like the idea 😃
If help is needed with initial setup, let me know here or in discord.
If yourself or anyone wishes to contribute to a section, or suggests feedback or adding of a new section -> feel free to let me know and I can add it in :)
Regarding moddb.com, I am not familiar with it, but at a glance sounds like a good idea, perhaps around the timing of the next release.

I have proposed a simple webpage design for UCP here. Complete with installation instructions, information on aic values, contributing info, and all the current wiki information.
It can be put under version control in this repo by using gh-pages branch (will be separate from the rest of the code base)

Wow!
Thank you !!!

It's not responsive...
On Thu, 19 Mar. 2020, 6:55 pm Mark Spiridonof<EMAIL_ADDRESS>wrote:

I have proposed a simple webpage design for UCP here
https://patel-nikhil.github.io/UnofficialCrusaderPatch/index.html.
Complete with installation instructions
https://patel-nikhil.github.io/UnofficialCrusaderPatch/getting-started.html,
information on aic values, contributing info, and all the current wiki
information.
It can be put under version control in this repo by using gh-pages branch
(will be separate from the rest of the code base)
Wow!
Thank you !!!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Sh0wdown/UnofficialCrusaderPatch/issues/555#issuecomment-601037932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGGWTWOFVAEOVJ6VLRDKDDRIHFXJANCNFSM4LO2BGVA
.

@GRhin added sizing breakpoints and fixed word wrap on buttons - should look fine on mobile now except for the AI Personality Values page (/aic/detailed.html). Not sure why the table breaks it so badly. Have to look into how to fix that.
On desktop should be fine though.

Updated the features page to include descriptions of all the different selectable features in the UCP.
I'm no expert at localization unfortunately. I could add say, a dropdown to allow language selection but not sure of a good way to choose and populate text (especially if formatted into a list) based on language. Unless someone comes with an idea it'll stay English-only for time being.
Pages left to add content to:

Add in the values for aic enums such as FarmBuildings, etc.
The entire aiv section as I have very little experience with aiv editor

If someone could write a few paragraphs would be awesome

FAQ section

I can think of one common question to put in here (increasing map size past 400x400 to which answer is currently no).

More content is better so if you have some to add, then great

Faq should be more along the lines of

"does this work with multiplayer?" A: yes, as long as everyone playing
has the same options selected
"Can you include such-and-such a feature" A: search in the issue section,
if it hasn't been asked, ask away. If a developer likes the idea they may
look into it, no promises though.
"Im a programmer how can I get involved?" A: look through the issues and
find ideas you want to fix or implement. Follow this link to a getting
started article for our codebase.
"Am I allowed to modify this for personal use?" A: Of course!
"Where can I get community ais?" A: there are a few bundled with the
installer, or you can check out the content creation channel on our discord
"I'm not a programmer, is there any other way I can help?" A: testers are
always welcome. Let us know in discord if you want to test. If you follow
this github repository you will get an alert when someone submits new code
to be used. See what the code is supposed to do, ask for an installer, and
test the changes in game to ensure it works as intended.
"is there a tutorial on how to make my own AIs?" A: there are a few,
check this one out by krarilotus (if there are others link them here too)
"I'm having trouble installing, can anyone help?" A: of course, pop into
discord and ask on the help-and-issues channel.

 Of I think of more I'll message you
on discord.
On Fri, 20 Mar. 2020, 2:32 pm Nikhil<EMAIL_ADDRESS>wrote:

Updated the features page to include descriptions of all the different
selectable features in the UCP.
I'm no expert at localization unfortunately. I could add say, a dropdown
to allow language selection but not sure of a good way to choose and
populate text (especially if formatted into a list) based on language.
Unless someone comes with an idea it'll stay English-only for time being.
Pages left to add content to:

Add in the values for aic enums such as FarmBuildings, etc.
The entire aiv section as I have very little experience with aiv
editor

If someone could write a few paragraphs would be awesome

FAQ section

I can think of one common question to put in here (increasing map
size past 400x400 to which answer is currently no).

More content is better so if you have some to add, then great

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Sh0wdown/UnofficialCrusaderPatch/issues/555#issuecomment-601510216,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGGWTR4PCYZS7DAQOAYTQLRILPUDANCNFSM4LO2BGVA
.

Good ideas, I've updated it.
Yes, @Krarilotus has a fairly in-depth video tutorial for aic and aiv creation that's objective (ie. no advertising, etc.) and therefore suitable - I hope you don't mind me embedding it in the page (I can remove or link it if you'd rather not)
I'm not aware myself of any other videos/tutorials for aic/aiv creation. The page written by Kimberly in the wiki is good, however would like to update the images to reflect the json changes before putting it in

Sorry to interfere. Tell me please what is the latest information on UCP? Will there be continued work to improve UCP. I just need information.

we are currently working on it, we are planing to release an update soon (TM)

Dear friend! Tell me please, what exactly will you add and correct in the new version of UCP? Where can I read the new?

You can see from the tickets that many new features are being worked on for next release. When it is released you can get it from the same place you got the current release. If you have questions please ask on a new issue instead of taking an existing unrelated issue on a tangent.

Where is? where i can read and learn more. Show me. I asked Specifically. What exactly is planned and when will the release be? What is not clear?
And something that is off topic - so any message can be deleted if you still do not know about it

This issue you are commenting on is specifically about the webpage. You are
asking about a new release. If there is not an issue for that, create one.
You know how to do that, you have created issue before, you dont need me to
give you instructions on how to do that.
On Mon, 23 Mar. 2020, 4:45 pm Russianstory<EMAIL_ADDRESS>wrote:

Where is? where i can read and learn more. Show me. I asked Specifically.
What exactly is planned and when will the release be? What is not clear?
And something that is off topic - so any message can be deleted if you
still do not know about it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Sh0wdown/UnofficialCrusaderPatch/issues/555#issuecomment-602398026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALGGWTRXSH4OLPEJXALZ2RDRI3ZODANCNFSM4LO2BGVA
.

Where is? where i can read and learn more. Show me. I asked Specifically. What exactly is planned and when will the release be? What is not clear?
And something that is off topic - so any message can be deleted if you still do not know about it
@Russianstory

You can access the webpage on the link that patel-nikhil postet.
If you want to know more prior to release of the next update of the UCP, join the official discord server. Link is on the Wiki page.

we are currently working on it, we are planing to release an update soon (TM)

I'll be waiting !
Thank you !

Website has been added to UCP repo in #573 - closing this issue as resolved now.