Errors in Stylus/CSS/... builds leave dev environment stuck

If a .css/.styl file has an error – in my case a problem resolving a relative path I hadn't changed while refitting an old site to Next.js – it'll cause the dev server to get stuck on "wait - compiling" and the browser will never navigate to the site (as the server never responds), like:
$ ./node_modules/.bin/next dev
[ wait ]  starting the development server ...
[ info ]  waiting on http://localhost:3000 ...
[ ready ] compiled successfully - ready on http://localhost:3000
[ wait ]  compiling ...
[ ready ] compiled successfully - ready on http://localhost:3000
[ event ] build page: /
[ wait ]  compiling ...

Running next build will expose the error though:
$ ./node_modules/.bin/next build
Failed to compile.

./styles/style.styl
ModuleNotFoundError: Module not found: Error: Can't resolve '../images/jumbo.jpg' in 'styles'

> Build error occurred
Error: > Build failed because of webpack errors
    at build (node_modules/next/dist/build/index.js:7:847)

This smells like a promise error being dropped on the floor or similar. :)
Relevant package versions:
    "@zeit/next-css": "^1.0.1",
    "@zeit/next-stylus": "^1.0.1",
    "next": "^9.0.6",

Hi, thanks for creating an issue. We currently recommend using https://nextjs.org/docs/basic-features/built-in-css-support as the plugins have been deprecated in favor of the built-in support.


GATConv aggregates by default

GATConv seems to aggregate the features by default.
So if I give a FxN matrix to a GATConv, the output is a F' vector (where N is the number of nodes in the graph, and F, F' are the dimensions of the features.
Why is that? Is it on purpose?
Because it makes it impossible to have a chain of GATConv for example.

I had not really understood what the aggregation was, my bad.


Deep Q-value network evaluation in SAC algorithm

I am implementing Soft-Actor Critic (SAC) agent and need to evaluate q-value network inside my custom environment (for the implementation of a special algorithm, called Wolpertinger's algorithm, to handle large discrete action spaces). I have tried to get the q-values from SAC class object, but failed. Any method or function like the one with stable baselines' PPO algorithm's implementation (namely, .value) will be very helpful.

I would first suggest moving using stable-baselines3: it is more refined and still mantained. This version is no longer mantained.
To answer your question: there is no convenience function for this, but you can check how SAC does the value prediction in SB3 here, and try to replicate it yourself.

Unfortunately I have implemented rest of RL algorithms, layers and optimizers in tensorflow and stable-baselines2 ecosystem. I cannot switch right now but I will consider using stable-baselines3 and specially Rllib in the future.
Also it will be a great coding exercise for me to implement this q-value evaluation method.
Cheers..


why tfdbg dumps so much things out

I was running a very small network(only 1 million parameters), everything is right with the training process, then I tried tfdbg, when I get a tfdbg -> run, it dumps so many things out, then root directory is filled, before run, there is 11G left, with a run, quickly fills all the space and prompts no space left:
2018-02-07 10:23:07.433343: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/add_grad/Shape:0:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/add_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3;
2018-02-07 10:23:07.443635: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/conv1/Conv2D_2_grad/ShapeN:0:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1/Conv2D_2_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1;
2018-02-07 10:23:07.443804: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/conv1/Conv2D_2_grad/ShapeN:1:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1/Conv2D_2_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1;
2018-02-07 10:23:07.445778: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/conv1/Conv2D_1_grad/ShapeN:0:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1/Conv2D_1_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1;
2018-02-07 10:23:07.445894: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/conv1/Conv2D_1_grad/ShapeN:1:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1/Conv2D_1_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1;
2018-02-07 10:23:07.446192: E ./tensorflow/core/kernels/debug_ops.h:186] Debug node of watch key tower1/gradients/tower1/group2/block3/conv1/Conv2D_grad/ShapeN:0:DebugIdentityfailed to publish debug tensor data to all URLs file:///var/tmp/tfdbg_2DnmZj, due to: Publishing to 1 of 1 debug target URLs failed, due to the following errors: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1/Conv2D_grad, due to: Failed to create directory  /var/tmp/tfdbg_2DnmZj/_tfdbg_device_,job_localhost,replica_0,task_0,device_GPU_1/tower1/gradients/tower1/group2/block3/conv1;

How to make it work?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

Nagging Awaiting Response: It has been 14 days with no activityand the awaiting response label was assigned. Is this still an issue?

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

This question is better asked on  StackOverflow since it is not a  bug or feature request. There is also a larger community that reads questions there. Thanks!


Updated the README to simplify steps for new users

Addresses some of the issues discussed in #44

Overall it looks really good! Thanks for doing this :)

Everything you mentioned makes sense, especially the quickstart section. Added everything in the latest commit

Fab! Thank you so much again!


Doesn't work on themes that have more than two variants

It only changes the theme, not the shell theme. So, the panel, right-click on desktop menu, password box..etc stay at whatever previous theme it was on.

It gets confused and doesn't work when a theme has several variants. For example, it worked flawlessly on Yaru and Pop themes, because they only have (Yaru and Yaru-dark.. and Pop and Pop dark), but when I try it on the zorin theme (which has zorin blue, zorin blue dark, zorin red, zorin red dark etc of other colors), Qogir (which has Qogir, Qogir win, Qogir dark, Qogir win dark, Qogir rounded, Qogir rounded dark....You get what I am saying), it doesn't do anything.

I'd love for this to apply to icons, too.

Automatic switching (sunset to sunrise, for example) would be superb.

It only changes the theme, not the shell theme. So, the panel, right-click on desktop menu, password box..etc stay at whatever previous theme it was on.

Working on it. Thanks for reporting!

[...] when I try it on the zorin theme (which has zorin blue, zorin blue dark, zorin red, zorin red dark etc of other colors), Qogir (which has Qogir, Qogir win, Qogir dark, Qogir win dark, Qogir rounded, Qogir rounded dark....

Can you try the latest master against the Zorin themes? I've just pushed a patch. For the Qogir ones, can you please provide the exact names of the themes?  In which case, giving me the output of ls /usr/share/themes/Qogir* or ls ~/.themes/Qogir* (depending on how you installed the theme) should be enough.

I'd love for this to apply to icons, too.

Thanks. On it.

Automatic switching (sunset to sunrise, for example) would be superb.

 In that case, I would suggest using Night Theme Switcher. Or if you are using a DE, you can try adding a hook for tools like Gammastep, or Red Shift. I have a gist for a gammastep hook that tries to do that: https://gist.github.com/rifazn/50b4fa6b297137756a0f97b655026967

It has worked on the zorin themes.
here is the output for the Qogir (these are only the ones I use, it originally has more than these)

/home/kalzi/.themes/Qogir:
AUTHORS   COPYING      gtk-2.0  gtk-4.0      metacity-1  unity
cinnamon  gnome-shell  gtk-3.0  index.theme  plank       xfwm4

/home/kalzi/.themes/Qogir-dark:
AUTHORS   COPYING      gtk-2.0  gtk-4.0      metacity-1  unity
cinnamon  gnome-shell  gtk-3.0  index.theme  plank       xfwm4

/home/kalzi/.themes/Qogir-light:
AUTHORS   COPYING      gtk-2.0  gtk-4.0      metacity-1  unity
cinnamon  gnome-shell  gtk-3.0  index.theme  plank       xfwm4

/home/kalzi/.themes/Qogir-win:
AUTHORS   COPYING      gtk-2.0  gtk-4.0      metacity-1  unity
cinnamon  gnome-shell  gtk-3.0  index.theme  plank       xfwm4

I am using Fedora workstation (gnome 40). That extension is actually very good and works just fine, but how is it going to work with your script? Wouldn't it eliminate the need for this script? I still think that you should implement an automatic switching in this script to make it complete.

Hi @KarkanAlzwayed!
Can you see that it works for your themes now, please?
And for themes that don't work, can you try adding a theme mapping in the config file?
For using the config file, just copy the config file as ~/.config/dark-toggle/config (of course after cloning the repo or after pulling the latest changes).
You can add a theme mapping as:
theme_mapping="theme1-variantA: theme2-variantB"

This way, if your current theme is theme1-variantA, calling dark-toggle will change it to theme1-variantB, and vice-versa.
I have the thought that adding a custom user config gives us much more flexibility (as a user can have any theme combination they like), and also the code maintains a better quality (as data (theme names, etc.) is not getting hard-coded into the source).

Hi there, apologize for the late response. I don't run gnome anymore, unfortunately. :/

It's alright. I hope you'll let me know if anything breaks when you ever use it again!


Fix some test fixture-handling nits.

dir_fixture() did not concatenate a source list into a path as the testing doc says it should. file_fixture() did not either, and the doc didn't say so; now it does (docstring and external testing doc).
There was a way to slip through both dir_fixture and file_fixture with the path invalid - if you have fixture dirs defined, and the dir/file is not found in them, you'll come out of the loop set to the last fixture dir and not try the current dir. This doesn't break anything further (the lookup would fail anyway later), just leads to a somewhat misleading message if the fixture really isn't found - the traceback indicates the fixture was not found in whatever the last passed-in fixture dir was. Now it looks like it was not found in the source testing dir. The message can still be improved to be more descriptive.
A couple of minor Py2 removals.
Testing doc didn't mention FIXTURE_DIRS, so this was added.
Signed-off-by: Mats Wichmann<EMAIL_ADDRESS>Contributor Checklist:

[X] I have created a new test or updated the unit tests to cover the new/changed functionality.
[ ] I have updated master/src/CHANGES.txt directory (and read the README.txt in that directory)
[X] I have updated the appropriate documentation

ah, sider spotted an actual mistake there...


Add @toMap Annotation ?

Dear Lombok team, is can add  @ToMap Annotation ,
something like this:
@ToMap
public DemoBean {
   String code;
   String name;
}

to:
public DemoBean{
   String code;
   String name;

  public Map<String,Object> parseMap(){
    Map<String,Object> map = new HashMap<String,Object>();
    map.put("code",this.code);
    map.put("name",this.name);
    return map;
  }
}

Although it can be created through beanUtil or BeanCopier, this performance is better?

@ToMap is T,not t

Although I can imagine some use cases, it is for now below our threshold regarding the effort versus benefits.


Content Format use openlink-api

Now with APIM supporting OpenApi v3 would it be appropriate to change contentFormat: swagger-json to contentFormat: openlink-api
I did a quick smoke test and the latter support importing both v2 and v3 artifacts
Thoughts?

Hello,
Indeed, that makes sense. I'll add it to the list of things to do for the next release.
Thanks
Best Regards

Hi,
Tried it out.  FYI, it is not openlink-api but openapi-link and it only works with v3. While you might think it works with v2, you'll notice that your web service URL will be screwed up. The doc states that it only works with v3 anyway. I'm now waiting for MS feedback as I can't get it work with a native YAML definition. To be more explicit, the openapi-link works with both but trying to send the YAML content using "openapi" doesn't and I'd like to have all the possibilities.
Best Regards

Hello,
It is working now. I'll publish the new version next week-end as I'll be out for a few days so I don't want to take the risk to publish a new release (there are other changes too) while not being available.
Best Regards

Will it also support openapi+json?

Yep it will. I'll finish that next Sunday

Hello,
The last release supports this scenario.
Best Regards


Rename release branch to release-2.XX instead of release-2.XX.0

This makes more sense because it allows us to cut patch releases off the same branch. This is also convenient because it allows us to cherry-pick low risk, moderate value fixes to the release branch in case a patch happens, and it is just all around a more accurate naming.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

This is ready to go, but I will hold it until the current release completes to avoid breaking things for that release.


Update to support Ecto 2.2.

Coverage remained the same at 97.756% when pulling b1065867190e4df810b34f10e7189d8fa3555c41 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 5bfb4b493de2793884aa1a2d9b6371c0fd69c326 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 9c0adbee4b4d9e4b1572c07198d65cf5e240694a on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 9c0adbee4b4d9e4b1572c07198d65cf5e240694a on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 9c0adbee4b4d9e4b1572c07198d65cf5e240694a on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling b1181e46dcdd33f23c6803f660c5b3c02280eeac on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 5b5c031bbb706a4741b400f01f14abf5ed903ae2 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage remained the same at 97.756% when pulling 5b5c031bbb706a4741b400f01f14abf5ed903ae2 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 874b8efb4168b854e7421a93e6c938bb17313319 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling fb1d191828065bb0c12e865740a6c0e86598272a on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling fb1d191828065bb0c12e865740a6c0e86598272a on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 1b38c81eb0f3dc608a0e8240aa6b83a858ffc965 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 8b2b000ab7720bd0d5429a9dab63a03bb10f80c8 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 8b2b000ab7720bd0d5429a9dab63a03bb10f80c8 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 8b2b000ab7720bd0d5429a9dab63a03bb10f80c8 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 8b2b000ab7720bd0d5429a9dab63a03bb10f80c8 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage increased (+0.3%) to 98.071% when pulling f4eab9c482014c2404409eba603df9ab3fbeeb44 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage increased (+0.3%) to 98.071% when pulling f4eab9c482014c2404409eba603df9ab3fbeeb44 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 6d2e06001618e06ff74aa3cdf848756b96272a6b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 6d2e06001618e06ff74aa3cdf848756b96272a6b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 6d2e06001618e06ff74aa3cdf848756b96272a6b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.007%) to 97.749% when pulling 6d2e06001618e06ff74aa3cdf848756b96272a6b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.01%) to 97.742% when pulling d118fb98c0294d834a35a0e6407c01014a01bb39 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.01%) to 97.742% when pulling d118fb98c0294d834a35a0e6407c01014a01bb39 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.01%) to 97.742% when pulling d118fb98c0294d834a35a0e6407c01014a01bb39 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 004215da81e14e71b5a9dc33949697662dee7f08 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling 23a7520fefdf842fd77eb81aef797f687ab84230 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling ff4b3d359820a46b984706646416b8a62c059c0d on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-0.05%) to 97.705% when pulling ff4b3d359820a46b984706646416b8a62c059c0d on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling bb965d1656febce690b2ef6379ff923364630cd3 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling bb965d1656febce690b2ef6379ff923364630cd3 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling bb965d1656febce690b2ef6379ff923364630cd3 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling bb965d1656febce690b2ef6379ff923364630cd3 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 220b3e8a1211645e98ef108b45d5f7194af2461b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 220b3e8a1211645e98ef108b45d5f7194af2461b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 220b3e8a1211645e98ef108b45d5f7194af2461b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 220b3e8a1211645e98ef108b45d5f7194af2461b on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 3d0e6ef48baa222667f89d968c13504697063f44 on ecto-2.2 into f2780eb904304c5d0d902e3c81b9b9be2eaa6e1a on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12e49bdf0df117e4459f8c1c94f98bc4d006a315 on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12e49bdf0df117e4459f8c1c94f98bc4d006a315 on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12e49bdf0df117e4459f8c1c94f98bc4d006a315 on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12e49bdf0df117e4459f8c1c94f98bc4d006a315 on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12bb7a6275e9e3648f348a4943d68ca6ca1eaf0b on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.

Coverage decreased (-1.4%) to 96.393% when pulling 12bb7a6275e9e3648f348a4943d68ca6ca1eaf0b on ecto-2.2 into 33b9859116e2609f9624e0cd92b54c4d52604058 on master.


New endpoint for weak subjectivity data

Resolves https://github.com/ethereum/beacon-APIs/issues/198
This adds an endpoint for retrieving weak subjectivity data. Please read #198 for rationale and uses. Thank you!

It seems that there is no consensus with implementation teams to implement this. We can either not include or include it as optional with disclaimer

I'm going to withdraw this proposal. I think the inclusion of state_root is pretty specific to the use case of using this endpoint for checkpoint sync, which prysm is no longer doing.


Master

Sorry, I shouldn't be merging this PR here as it would override the template defaults.

Sorry about confusion!


circuit-breaker always closes immediately on success() call

BreakerTest.txt
The problem we're seeing is that the circuit breaker opens immediately when calling success() on a future from the breaker handler. Even if the breaker was just closed moments before. This means it does not respect the closing timeout, nor does it pass trough the half-open state.
Our usecase:
In the circuit-breaker, we're doing some async work. This means a bunch of events can be inside the breaker handler queued.
At a certain moment, most of the async events start failing, resulting in the resultbreaker to open.
From this moment, no new work is scheduled on the breaker anymore but passed to the fallback.
However, since there are async tasks inside the breaker handler, some of them might still complete after the breaker opened. If one of them succeeds, it will call success() on the breaker future.
-> I would expect in this case that the breaker still stays opened, as the timeout period did not end yet. However the breaker will close immediately.
With this logic, I don't see a way how you can handle async events inside the circuit breaker.
I believe the problem is in CircuitBreakerImpl.java on line 203 where the reset() is called:
https://github.com/vert-x3/vertx-circuit-breaker/blob/master/src/main/java/io/vertx/circuitbreaker/impl/CircuitBreakerImpl.java#L203
I've added a testcase to illustrate the behavior.

BreakerTest.txt

Thanks for reporting this issue (and for the reproducer). I've fixed it in the circuit breaker master branch.


Feature: customizable builder name

The builder name is currently hardcoded in https://github.com/slsa-framework/example-package/blob/main/.github/workflows/scripts/e2e-verify.sh#L143
We should make it parameterizable, e.g. via a BUILDER env variables declared in workflows.
This will allow us to support other builders

Looks like we can close this, since it's handled via different scripts instead.
@ianlewis agreed?

Yeah, I'm not sure this is relevant anymore.


Initialize Static Routes  before session initalization

There is a very good explanation for this here
Avoid Sessions for Static Resources
I learned that if you add the session middleware before your static directory, Express will generate     sessions for requests on static files like stylesheets, images, and JavaScript.
If a new visitor without a session loads a page with 10 static files, the client's browser will not yet have     a cookie and will send 10 cookieless requests all triggering Express to create sessions. Ouch! So that's     what was happening... If you haven't done something smart to detect bots and scrapers, things can blow     out pretty quickly!
Simply put your static files first, or better yet on a CDN that has nothing to do with your Node.js app and your session collection should stay much healthier

I believe this was already addressed by #1093.

My bad I had older version of meanjs


[DOC] Add more field types editorial comments

What do you want to do?

[ x] Request a change to existing documentation
[ ] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request.
Add more editorial comments for the new field types content
Do you have any related resources to help us get started?
For example, a link to a related fixed issue, a design doc, or any particular POCs (Eng or PM) who have more information?
Is there anything else you'd like to add?

Closed as already implemented.


⚠ Bofrost.fr has degraded performance

In 4a36300, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1050 ms

Resolved: Bofrost.fr performance has improved in e1492f9 after 6 minutes.


[BUG] Cannot read properties of undefined (reading 'toUpperCase')

What happened?
GUI stuck while plotting Bladebit. Temp directory -342 GB on disk, 3080 files. Free space on disk only 2 GB left. No changes after 10 hours.
Error occured when switched several times between different GUI tabs (Pols-Wallet-Full Node)
Version
1.6.1b3
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output or stacktrace
Cannot read properties of undefined (reading 'toUpperCase')

URL
#/dashboard/wallets/7

Stacktrace
at webpack://@chia/wallets/dist/esm.js:800:67 labels
at webpack://@chia/gui/node_modules/victory-core/es/victory-util/helpers.js:169:29 evaluateProp
at webpack://@chia/gui/node_modules/victory-tooltip/es/victory-tooltip.js:136:17 value
at webpack://@chia/gui/node_modules/victory-tooltip/es/victory-tooltip.js:492:32 Helpers.evaluateProp
at webpack://@chia/gui/node_modules/victory-tooltip/es/victory-tooltip.js:508:18 value
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:187:187 qi
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:186:172 pi
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:269:426 a=b.pendingProps;f=e._init;e=f(e._payload);b.type=e;f=b.tag=hk(e);a=lg(e,a);switch
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:346 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:277 Pj

Fix has been included in 1.6.1


Channel.cs tries to make Slack use suggested actions

Version
master branch
Describe the bug
Slack does not support suggested actions. Suggested actions are rendered as text, as can be seen in this customer's screenshot:

However, Channel.cs acts as though Slack does support suggested actions:
                case Connector.Channels.Slack:
                case Connector.Channels.Telegram:
                case Connector.Channels.Emulator:
                case Connector.Channels.Directline:
                case Connector.Channels.Webchat:
                    return buttonCnt <= 100;

This problem is presumably in the other SDK languages as well.
To Reproduce
Steps to reproduce the behavior:

Connect a bot to the Slack channel
Have the bot use a choice prompt with the list style set to "auto"
Observe that the choices are text instead of buttons

Expected behavior
The SDK should acknowledge that Slack doesn't support suggested actions, or the Slack connector should convert suggested actions into Slack buttons
Additional context
I don't think there's any need to have the Slack connector support suggested actions since buttons can already be easily implemented with hero cards
[bug]

@cleemullins - Has anyone looked at this?

@cleemullins - Who is in charge of triaging these issues?

I ran into this recently too.

The SDK should acknowledge that Slack doesn't support suggested actions, or the Slack connector should convert suggested actions into Slack buttons.

I followed the documentation on how to implement sequential conversation flow.
My expectation was that if I follow the docs, I'd get a nice interactive choice rendering in Slack. That's sort of the reason I'm choosing Microsoft Bot Framework and ABS in the first place. So I don't have to think about every platform I target in most mainline cases.
So my $0.02 would be to update the Slack connector to convert suggested actions into Slack buttons.
Or if the idea is people should not use suggested actions, add an obsolete warning on them and update the existing docs to describe how people should implement interactive choice options with the bot framework.

A third option is to simply update the docs I mentioned to use ListStyle.HeroCard. Though not sure if that causes issues for other platforms.
Might be nice if there's a way to have a fallback style. So I could have a choice that renders as SuggestedActions for platforms that target it, but fallback to HeroCard if the target platform doesn't support it.

Might be nice if there's a way to have a fallback style. So I could have a choice that renders as SuggestedActions for platforms that target it, but fallback to HeroCard if the target platform doesn't support it.

That's what ListStyle.Auto already does

Ah, thanks! I looked at the docs for ListStyle.Auto and it says,

Automatically select the appropriate style for the current channel.

In looking at the code, ListStyle.Auto is the default, which as my original issue points out, renders choices as text in Slack. Just to be sure, I explicitly set the ListStyle to ListStyle.Auto and I still got a text response.
So it seems to me the bug is that when you set the ListStyle of a ChoicePrompt to ListStyle.Auto, it should choose ListStyle.HeroCard for Slack. That seems to me to be a more "appropriate style" than a text response.

@haacked - Yes, that is why this bug was raised that you're commenting on now


Copyright notices from the original codebases are missing

You have marked every file with (C) 2024 Willy Fitra Hendria, also the LICENSE file only has your name in it. This is a break with the licence of the projects that you have based yours on.
For example, visualkeras (which has substantial code similarities, and which you acknowledge as a basis), has an MIT licence with "Copyright (c) 2020 Paul Gavrikov"
The MIT licence states, as its one requirement that "The above copyright notice [...] shall be included in all copies or substantial portions of the Software.
"
Action point: the LICENSE file and any code file that has substantial pieces of code from other sources, need to include the copyright line from those original sources in addition to yours.
An example for graph.py:
# Copyright (c) 2020 Paul Gavrikov
# Copyright (C) 2024 Willy Fitra Hendria
# SPDX-License-Identifier: MIT

I have not checked the other two packages you mention as inspiration, the same should be done for those authors if there is code overlap.
https://github.com/openjournals/joss-reviews/issues/6678

Hi @dgrell, thanks for your feedback. I'll thoroughly check the source code and add the appropriate copyright lines

Hi @dgrell, thank you for your feedback. I will thoroughly check the source code and provide the appropriate copyright lines.


Problem with deletion of objects

Hey!
I really like this library and so far it works like a charm. However, I'm experiencing some unwanted behaviour with iOS 8 / Swift.
Deletion of objects in the view works fine, except for the case when I use the removeContact function as it does not seem to remove the object. The bubble stays there.
Do you know how to work around this issue? Thank you!

@Luke47  Even i m facing the same issue. Were you able to solve it?

@hardikamal
Yes, I remember that I've made it work. You can check out my fork. I think it was this specific commit:
https://github.com/Luke47/THContactPicker/commit/30e1a3f84ba4ec494eb6701b1a4a06ea6a804ac5

@Luke47 cool thanks


Source Spotify, song title missing (RX-V475)

I have a Yamaha RX-V475, selecting Spotify as Source the song title is missing (should be Sabotage):
Dev Tools State Attributes:
source_list:
  - ''
  - AirPlay
  - Apple TV
  - Blu-ray
  - Fire TV
  - iPod
  - iPod (USB)
  - NET RADIO
  - SERVER
  - Spotify
  - TUNER
  - TV
  - V-AUX
sound_mode_list:
  - 2ch Stereo
  - 5ch Stereo
  - Action Game
  - Adventure
  - Cellar Club
  - Chamber
  - Drama
  - Mono Movie
  - Music Video
  - Roleplaying Game
  - Sci-Fi
  - Spectacle
  - Sports
  - Standard
  - Straight
  - Surround Decoder
volume_level: 0.4690721649484536
is_volume_muted: true
source: Spotify
sound_mode: 2ch Stereo
device_class: receiver
icon: mdi:audio-video
friendly_name: Receiver
supported_features: 89533
media_artist: Gang Starr
media_album_name: The Ownerz

Is it not provided by the Receiver or is it just missing from in the integration?
diagnostics file
(Tried to make a template sensor providing the played media depending on Source)

The title shows in the diagnostics. It just seems that the Spotify source sends the title as TRACK while all other sources seem to send title as SONG. My receiver does not support Spotify so never noticed it.
I will update the integration to look at the TRACK also.

This should be fixed in the v7.1.0 release

Yes, fixed. Thanks!


Add basic command line interface

@garezana - I felt like doing a little coding this afternoon so I added a basic command line interface. Can you see if it works for you and give the code a read and let me know anything you see to improve?

Merging as part of the move over to CarpenPi


BUG: Sign up page not functional

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Description
The login/register button leads to login page. It seems like there are two separate pages for login and sign up but only one link in the nav bar that redirects to login page. The user is unable to login before signing up and sign up page is not accessible.
Proposal
I suggest to create an overlay signin/signup page where both the options are available. After the user registers , he can simply login on the same page
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Please assign this issue to me @Its-Aman-Yadav

Sure work on it


Tutorial or Document Needed!

Hi,
thanks for your contribution and open-sourcing! I have watched some demo videos from yours and quite impressive!
I am with the background of SLAM / 3D point cloud perception, and would like to have some trials for the combination of dynamics like MJPC with other stuff. However, I found that there is almost no docs for this project,
Can you provide any docs or information wrt the usage. Basically, I am trying to control a quadruped's pose with python API and it's pose could be online solved with MJPC. This task is similar with your demo HERE but with python API.
Bests

Documentation is available here.


/etc/openvpn/server.conf issue

Hi,
Thank you for all the latest updates, however, I was not aware that the server also got release versions and used the latest with ui <IP_ADDRESS> and now I'm in a bit of a mess :)
When using "- ./server.conf:/etc/openvpn/server.conf" as the example in docker-compose.yml shows, it creates an empty directory
Touching an empty file and mounting it as a volume doesn't have the configuration written to it
The only thing that worked was commenting out the "server.conf" line from the docker-compose.yml, let the container start and create the server.conf, down everything, and uncomment out the "server.conf" line, which means, I'll have to save the server.conf and copy it over to all my future setups
Eventually, what I did was this
`services:
openvpn:
container_name: openvpn
image: d3vilh/openvpn-server:0.5.1
privileged: true
ports:
- 1194:1194/udp
- 2080:2080
environment:
TRUST_SUB: <IP_ADDRESS>/24
GUEST_SUB: <IP_ADDRESS>/24
HOME_SUB: <IP_ADDRESS>/24
volumes:
- ./:/etc/openvpn
- ./config:/etc/openvpn/config - only holds easy-rsa.vars file
- ./log:/var/log/openvpn
- ./fw-rules.sh:/opt/app/fw-rules.sh
cap_add:
- NET_ADMIN
restart: unless-stopped
openvpn-ui:
container_name: openvpn-ui
image: d3vilh/openvpn-ui:<IP_ADDRESS>
environment:
- OPENVPN_ADMIN_USERNAME=admin
- OPENVPN_ADMIN_PASSWORD=gagaZush
privileged: true
ports:
- 8080:8080/tcp
volumes:
- ./:/etc/openvpn
- ./db:/opt/openvpn-ui/db
- ./pki:/usr/share/easy-rsa/pki
- /var/run/docker.sock:/var/run/docker.sock:ro
restart: unless-stopped`
So, all is working again and thank yo uagain for all your hard work on this

Hi @reamasesa
Agree, such kind of core changes always brings mess without proper documentation and  upgrade procedure.
We have Important changes in Server documents, but for UI I just add release note.
So, I'll push upgrade procedure for UI on this weekend.
Basically, you have to move server.conf from openvpn-server/conf catalog to root of openvpn-server.
cp ~/openvpn-server/conf/server.conf ../server.conf

and re-create container from the updated compose file (- ./server.conf:/etc/openvpn/server.conf as volume option):
---
version: "3.5"

services:
    openvpn:
       container_name: openvpn
       image: d3vilh/openvpn-server:latest
       privileged: true
       ports:
          - "1194:1194/udp"
       environment:
           TRUST_SUB: <IP_ADDRESS>/24
           GUEST_SUB: <IP_ADDRESS>/24
           HOME_SUB: <IP_ADDRESS>/24
       volumes:
           - ./pki:/etc/openvpn/pki
           - ./clients:/etc/openvpn/clients
           - ./config:/etc/openvpn/config
           - ./staticclients:/etc/openvpn/staticclients
           - ./log:/var/log/openvpn
           - ./fw-rules.sh:/opt/app/fw-rules.sh
           - ./server.conf:/etc/openvpn/server.conf
       cap_add:
           - NET_ADMIN
       restart: always

    openvpn-ui:
       container_name: openvpn-ui
       image: d3vilh/openvpn-ui:latest
       environment:
           - OPENVPN_ADMIN_USERNAME=admin
           - OPENVPN_ADMIN_PASSWORD=gagaZush
       privileged: true
       ports:
           - "8080:8080/tcp"
       volumes:
           - ./:/etc/openvpn
           - ./db:/opt/openvpn-ui/db
           - ./pki:/usr/share/easy-rsa/pki
           - /var/run/docker.sock:/var/run/docker.sock:ro
       restart: always

I think it is a good idea to add Important changes with reference to updated upgrade procedure as I did for the version 0.4.
Glad you manage it and thank you for the reporting!

Thanks :)


Cosmos v4 encryption

Please read following before reviewing:
This PR adds the backbone for encryption to java v4 sdk:

This PR adds the backbone for Encryption code implementation in a standalone lib (azure-cosmos-encryption) outside of the SDK jar.
This PR has e2e test for decrypting data which is encrypted by dot net SDK (inline with dotnet encryption code)
This PR has e2e test for encrypting data using java v4 sdk and decrypting.
This PR keeps the encryption feature an internal feature (public api aren't exposed yet.)

I need to merge this PR to have a snapshot of my work to avoid dealing with more upcoming time-consuming merge conflicts.
I am addressing the following  out of the scope of this PR once this PR merge I will work on these:
TODO:

public surface area massaging incluging adding Encryptor interface.
DataEncryptionKeyContainer has basic APIs, the support for query/read-feed, etc needs to be added.
async vs sync. massaging the API to choose betwen Flux/Mono async or sync.
decryption on query

For sample on how to use
sample code:
CosmosClientBuilder builder = new CosmosClientBuilder();

CosmosClient client = builder.key("key")
    .endpoint("endpoint")
    .dataEncryptionKeyProvider(cxDataEncryptionKeyProvider)
    .buildClient();

CosmosContainer container = client.getDatabase("myDb").getContainer("myCol");
Pojo pojo = new Pojo();
pojo.id = UUID.randomUUID().toString();
pojo.mypk = UUID.randomUUID().toString();
pojo.nonSensitive = UUID.randomUUID().toString();
pojo.sensitive1 = "this is a secret to be encrypted";
pojo.sensitive1 = "this is a another secret to be encrypted";

CosmosItemRequestOptions options = new CosmosItemRequestOptions();
EncryptionOptions encryptionOptions = new EncryptionOptions();
encryptionOptions.setPathsToEncrypt(ImmutableList.of("/sensitive1", "/sensitive2"));
ModelBridgeInternal.setEncryptionOptions(options, encryptionOptions);

CosmosItemResponse<Pojo> response = container.createItem(pojo, options);
assert response.getItem().nonSensitive != null;
assert response.getItem().sensitive1 == null;
assert response.getItem().sensitive2 == null;

CosmosItemResponse<Pojo> readResponse = container.readItem(pojo.id, new PartitionKey(pojo.mypk), Pojo.class);
assert response.getItem().nonSensitive != null;
assert response.getItem().sensitive1 != null;
assert response.getItem().sensitive2 != null;

/azp run java - cosmos - tests

/azp run java - cosmos - tests

/azp run java - cosmos - tests (Cosmos_CI Session_Single_Region_Tcp_Direct)

/check-enforcer override


Do not trigger GHAs for changes to irrelevant Go tests

This PR keeps Go test GHAs from being triggered by irrelevant tests. For example, in #7460, even though only Traffic Ops unit tests were modified, Grove unit tests were triggered (which #7464 fixes).

Which Traffic Control components are affected by this PR?

Automation - Go tests

What is the best way to verify this PR?

Check the paths
PR submission checklist

[x] This PR has tests
[ ] This PR has documentation
[ ] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

TP tests are failing but are not related to this PR.


Implement ToSql/FromSql for u64

As discussed in #802 it is pretty awkward that ToSql and FromSql are not implemented for u64. The current justification is that SQLite cannot actually store integers between i64::MAX and u64::MAX, but it would be simpler just to return an error in that case (FromSqlError::OutOfRange, or is it Error::IntegralValueOutOfRange, or possibly Error::FromSqlConversionFailure?).

Whatever is done here should be done for usize too. Patches welcome, I don't think the specific error message matters that much for ToSql, but however we handle out of range for FromSql on other integers should be respected for that one.


update to py3.6

The issue with nightly is letting setuptools auto-install dependancies which have wheels on pypi.  Explicitly installing them with pip will fix the problem (I see this locally too....really need to sort out who to report it to....)

I'll go ahead and merge. I don't think there's disagreement. Thanks for the info @tacaswell .
About that, so that means we need to pip install the dependencies of pytest explicitly? Have you figured out where to report it?


store examples failing with Cannot redefine property

e.g. the cart example here https://jsfiddle.net/kennyfrc12/qjs8c2gb/15/
fails with
82:22: Uncaught TypeError: Cannot redefine property: cartItems

in current Chrome (123.0.6312.94)

I think I'm running in to two separate issues here:

The "Cannot redefine property" error comes from the fact that ReactiveElement.connect now calls Object.defineProperty, which means one should not attempt to define the property with normal field declaration syntax in the class body.

(I think it's also worth mentioning that the key used to retrieve the thing from the store is always used to define the name of the property on the Element.)

The store is somehow not updating? debug mode does send a cami:state:change message to the console with the correct newValue, but the store.state retains the old value.

Hi @keturn! Thanks for calling this out.
My mistake on that one -- you essentially looked at outdated documentation (for some reason, I updated everything except the store docs).
I've updated the relevant sections in the site, and sharing the updated jsfiddle link below:

JS Fiddle Link: jsfiddle cart example
API: store

One thing that you may have noticed is that if you're familiar with other store implementations, it's now less like zustand and more similar to a mix of express/redux. I just found that in the production application that I'm working on, it's a lot more manageable with this new API.
In general, my folder structure looks like this:
elements/
  ├── elementName1
  ├── elementName2
  └── etc
index.js (this contains all the actions or .register calls)
store.js

I plan to write more about the architecture about non-trivial applications once I launch the production app that uses Cami (by June 2024)


Entering the wrong app ID causes the app to crash and fails to load unless the app cache is cleared or the app reinstalled

Describe the bug
Entering the wrong app ID causes the app to crash and fails to load unless the app cache is cleared or the app reinstalled
To Reproduce
Steps to reproduce the behaviour:

Launch app
Enter wrong app ID
Press Load configurations
See an error

Expected behaviour
The app should prompt the user that the App ID does not exist
Screenshots/Video

Video Link  - here
Smartphone (please complete the following information):

Device: Emulator or Phycical device

Additional context
N/A

Was able to reproduce and resolve the issue.

closed by https://github.com/opensrp/fhircore/pull/925


The target table of the DML statement cannot have any enabled triggers if the statement contains an OUTPUT clause without INTO clause.

Bug description

When executing the prisma.create() statement on a SQL Server table that contains a trigger, the following error message is presented and no rows are created.
(node:37320) UnhandledPromiseRejectionWarning: Error:
Invalid `prisma.formSets.create()` invocation:

  Error occurred during query execution:
ConnectorError(ConnectorError { user_facing_error: None, kind: QueryError(TokenError { code: 334, state: 1, class: 16, message: "The target table \'dbo.FormSetSequence\' of the DML statement cannot have any enabled triggers if the statement contains an OUTPUT clause without INTO clause.", server: "dev-dfleming", procedure: "", line: 1 }) })
    at PrismaClientFetcher.request (C:\dev\graph-ql-prisma-ta\node_modules\@prisma\client\runtime\index.js:78786:15)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:37320) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 3)
(node:37320) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

How to reproduce

Data setup. Create a valid Prisma / TypeScript project. Using SQL Server create a table with a trigger. An example trigger used for this test is below. Configure schema.prisma and prisma.yml to connect to the SQL Server instance.

CREATE OR ALTER TRIGGER [dbo].[FormSetSequence_InsertUpdateDelete]
				ON [dbo].[FormSetSequence]
				FOR INSERT, UPDATE, DELETE
				AS
				SET NOCOUNT ON

				UPDATE LastChanges
				SET LastChanged = GetUtcDate()
				WHERE TrackedTable = 'LastDynamicFormChange';

				Update LastChanges
				SET LastChanged = GetUtcDate()
				WHERE TrackedTable = 'LastDynamicFormRuleChange';

Create a schema using prisma introspect and prisma generate.
Create a file index.ts and use the prisma client to call prisma.{tableWithTrigger}.create(data).
Run npx ts-node index.ts.
The error message should be presented to the user.

Expected behavior

The row should be inserted without error.
Prisma information

Environment & setup

OS:  Windows 10
Database: SQL Server 15.0.4063.15
Node.js version: v14.8.0
Prisma version:

@prisma/cli          : 2.11.0
@prisma/client       : 2.11.0
Current platform     : windows
Query Engine         : query-engine<PHONE_NUMBER>2e47bdcec77a2f1e7c1fb83a463918 (at ..\..\..\Users\dfleming-local\AppData\Roaming\npm\node_modules\<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli<PHONE_NUMBER>2e47bdcec77a2f1e7c1fb83a463918 (at ..\..\..\Users\dfleming-local\AppData\Roaming\npm\node_modules\<EMAIL_ADDRESS>Introspection Engine : introspection-core<PHONE_NUMBER>2e47bdcec77a2f1e7c1fb83a463918 (at ..\..\..\Users\dfleming-local\AppData\Roaming\npm\node_modules\<EMAIL_ADDRESS>Format Binary        : prisma-fmt<PHONE_NUMBER>2e47bdcec77a2f1e7c1fb83a463918 (at ..\..\..\Users\dfleming-local\AppData\Roaming\npm\node_modules\<EMAIL_ADDRESS>Studio               : 0.311.0
Preview Features     : microsoftSqlServer

I confirmed we cannot use OUTPUT with tables using triggers.
There's a good SO post describing workarounds, and looks like we should not rely on OUTPUT (it being buggy and harmful). To fix this might take a bit longer due to the temp tables needed per query, so meanwhile I'd just try the solution described in the issue using Prisma's queryRaw until we have a proper fix (modifying it of course to your needs):
SET NOCOUNT ON;

DECLARE @generated_keys table([CustomerID] int);

INSERT Customers (FirstName, LastName)
OUTPUT inserted.[CustomerID] INTO @generated_keys
VALUES ('Steve', 'Brown');

SELECT t.[CustomerID], t.[CustomerGuid], t.[RowVersion], t.[CreatedDate]
FROM @generated_keys AS g
   INNER JOIN Customers AS t
   ON g.[CustomerGUID] = t.[CustomerGUID]
WHERE @@ROWCOUNT > 0;

Due to SQL Server supporting multiple active result sets, these queries should be run in the same queryRaw call for it to work properly.

Is this going to be fixed in Entity Framework for SQL Server, or do we need to start writing builder.UpdateUsingStoredProcedure()?

I think you are in the wrong repository @hovek - this is about Prisma ORM.


Works good, but too simple

you dont really earn much from this but it still works good none the less
some suggestions:

more farm tasks to do (I find the ones that pay a lot are more skill based, but that might be an issue since a self bot probably cant do those)
visual upgrades, like making the terminal title have stats for how many times you've done a task
stop command

Yea i kinda rushed to update the bot to use node.js instead of Python, but yes i will implement those, Thanks!


Migration still try to run even when auto-create-tenant-database is false

Description
I try to disable auto-create-tenant-database so i could create it later manually (and as a part of my testing), but when I try to seed it, it gave me Unknown database error.

Actual behavior
The Hyn\Tenancy\Listeners\Database\MigratesTenants is subscribing to Events\Websites\Created
Expected behavior
Because I didnt create the database, it also shouldn't try to migrate. MigratesTenants should subsribe to Events\Database\Created

Information

hyn/multi-tenant version: 5.6
laravel version: 7.1.0
database driver and version: MySQL (Using mariadb 10.4.8)
webserver software and version: Nginx 1.17.4
php version: 7.3.13

tenancy.php config
<?php

/*
 * This file is part of the hyn/multi-tenant package.
 *
 * (c) Daniël Klabbers<EMAIL_ADDRESS> *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @see https://tenancy.dev
 * @see https://github.com/hyn/multi-tenant
 */

use Hyn\Tenancy\Database\Connection;

return [
    /**
     * Random key used for tenant database user password
     */
    'key' => env('TENANCY_KEY', env('APP_KEY')),

    'models' => [
        /**
         * Specify different models to be used for the global, system database
         * connection. These are also used in their relationships. Models
         * used have to implement their respective contracts and
         * either extend the SystemModel or use the trait
         * UsesSystemConnection.
         */

        // Must implement \Hyn\Tenancy\Contracts\Hostname
        'hostname' => \Hyn\Tenancy\Models\Hostname::class,

        // Must implement \Hyn\Tenancy\Contracts\Website
        'website' => \Hyn\Tenancy\Models\Website::class
    ],
    /**
     * The package middleware. Removing a middleware here will disable it.
     * You can of course extend/replace them or add your own.
     */
    'middleware' => [
        // The eager identification middleware.
        \Hyn\Tenancy\Middleware\EagerIdentification::class,

        // The hostname actions middleware (redirects, https, maintenance).
        \Hyn\Tenancy\Middleware\HostnameActions::class,
    ],
    'website' => [
        /**
         * Each website has a short random hash that identifies this entity
         * to the application. By default this id is randomized and fully
         * auto-generated. In case you want to force your own logic for
         * when you need to have a better overview of the complete
         * tenant folder structure, disable this and implement
         * your own id generation logic.
         */
        'disable-random-id' => false,

        /**
         * The random Id generator is responsible for creating the hash as mentioned
         * above. You can override what generator to use by modifying this value
         * in the configuration.
         *
         * @warn This won't work if disable-random-id is true.
         */
        'random-id-generator' => Hyn\Tenancy\Generators\Uuid\ShaGenerator::class,

        /**
         * Enable this flag in case you're using a driver that does not support
         * database username or database name with a length of more than 32 characters.
         *
         * This should be enabled for MySQL, but not for MariaDB and PostgreSQL.
         */
        'uuid-limit-length-to-32' => env('LIMIT_UUID_LENGTH_32', false),

        /**
         * Specify the disk you configured in the filesystems.php file where to store
         * the tenant specific files, including media, packages, routes and other
         * files for this particular website.
         *
         * @info If not set, will revert to the default filesystem.
         * @info If set to false will disable all tenant specific filesystem auto magic
         *       like the config, vendor overrides.
         */
        'disk' => null,

        /**
         * Automatically generate a tenant directory based on the random id of the
         * website. Uses the above disk to store files to override system-wide
         * files.
         *
         * @info set to false to disable.
         */
        'auto-create-tenant-directory' => false,

        /**
         * Automatically rename the tenant directory when the random id of the
         * website changes. This should not be too common, but in case it happens
         * we automatically want to move files accordingly.
         *
         * @info set to false to disable.
         */
        'auto-rename-tenant-directory' => true,

        /**
         * Automatically deletes the tenant specific directory and all files
         * contained within.
         *
         * @see
         * @info set to true to enable.
         */
        'auto-delete-tenant-directory' => false,

        /**
         * Time to cache websites in seconds. Set to false to disable.
         */
        'cache' => 600,
    ],
    'hostname' => [
        /**
         * If you want the multi tenant application to fall back to a default
         * hostname/website in case the requested hostname was not found
         * in the database, complete in detail the default hostname.
         *
         * @warn this must be a FQDN, these have no protocol or path!
         */
        'default' => env('TENANCY_DEFAULT_HOSTNAME'),
        /**
         * The package is able to identify the requested hostname by itself,
         * disable to get full control (and responsibility) over hostname
         * identification. The hostname identification is needed to
         * set a specific website as currently active.
         *
         * @see src/Jobs/HostnameIdentification.php
         */
        'auto-identification' => env('TENANCY_AUTO_HOSTNAME_IDENTIFICATION', true),

        /**
         * In case you want to have the tenancy environment set up early,
         * enable this flag. This will run the tenant identification
         * inside a middleware. This will eager load tenancy.
         *
         * A good use case is when you have set "tenant" as the default
         * database connection.
         */
        'early-identification' => env('TENANCY_EARLY_IDENTIFICATION', true),

        /**
         * Abort application execution in case no hostname was identified. This will throw a
         * 404 not found in case the tenant hostname was not resolved.
         */
        'abort-without-identified-hostname' => env('TENANCY_ABORT_WITHOUT_HOSTNAME', false),

        /**
         * Time to cache hostnames in seconds. Set to false to disable.
         */
        'cache' => 600,

        /**
         * Automatically update the app.url configured inside Laravel to match
         * the tenant FQDN whenever a hostname/tenant was identified.
         *
         * This will resolve issues with password reset mails etc using the
         * correct domain.
         */
        'update-app-url' => true,
    ],
    'db' => [
        /**
         * The default connection to use; this overrules the Laravel database.default
         * configuration setting. In Laravel this is normally configured to 'mysql'.
         * You can set a environment variable to override the default database
         * connection to - for instance - the tenant connection 'tenant'.
         */
        'default' => env('TENANCY_DEFAULT_CONNECTION'),
        /**
         * Used to give names to the system and tenant database connections. By
         * default we configure 'system' and 'tenant'. The tenant connection
         * is set up automatically by this package.
         *
         * @see src/Database/Connection.php
         * @var system-connection-name The database connection name to use for the global/system database.
         * @var tenant-connection-name The database connection name to use for the tenant database.
         */
        'system-connection-name' => env('TENANCY_SYSTEM_CONNECTION_NAME', Connection::DEFAULT_SYSTEM_NAME),
        'tenant-connection-name' => env('TENANCY_TENANT_CONNECTION_NAME', Connection::DEFAULT_TENANT_NAME),

        /**
         * The tenant division mode specifies to what database websites will be
         * connecting. The default setup is to use a new database per tenant.
         * If using PostgreSQL, a new schema per tenant in the same database can
         * be setup, by optionally setting division mode to 'schema'.
         * In case you prefer to use the same database with a table prefix,
         * set the mode to 'prefix'.
         * To implement a custom division mode, set this to 'bypass'.
         *
         * @see src/Database/Connection.php
         */
        'tenant-division-mode' => env('TENANCY_DATABASE_DIVISION_MODE', 'database'),

        /**
         * The database password generator takes care of creating a valid hashed
         * string used for tenants to connect to the specific database. Do
         * note that this will only work in 'division modes' that set up
         * a connection to a separate database.
         */
        'password-generator' => Hyn\Tenancy\Generators\Database\DefaultPasswordGenerator::class,

        /**
         * The tenant migrations to be run during creation of a tenant. Specify a directory
         * to run the migrations from. If specified these migrations will be executed
         * whenever a new tenant is created.
         *
         * @info set to false to disable auto migrating.
         *
         * @warn this has to be an absolute path, feel free to use helper methods like
         * base_path() or database_path() to set this up.
         */
        'tenant-migrations-path' => database_path('migrations/tenant'),

        /**
         * The default Seeder class used on newly created databases and while
         * running artisan commands that fire seeding.
         *
         * @info requires tenant-migrations-path in order to seed newly created websites.
         * @info seeds stored in `database/seeds/tenants` need to be configured in your composer.json classmap.
         *
         * @warn specify a valid fully qualified class name.
         */
        'tenant-seed-class' => false,
//      eg an admin seeder under `app/Seeders/AdminSeeder.php`:
//        'tenant-seed-class' => App\Seeders\AdminSeeder::class,

        /**
         * Automatically generate a tenant database based on the random id of the
         * website.
         *
         * @info set to false to disable.
         */
        'auto-create-tenant-database' => false,

        /**
         * Automatically generate the user needed to access the database.
         *
         * @info Useful in case you use root or another predefined user to access the
         *       tenant database.
         * @info Only creates in case tenant databases are set to be created.
         *
         * @info set to false to disable.
         */
        'auto-create-tenant-database-user' => false,

        /**
         * Set of database privileges to give to the tenant database user.
         *
         * @info Useful in case your database restricts the privileges you
         *       can set (for example AWS RDS).
         * @info These privileges are only used in case tenant database users
         *       are set to be created.
         *
         * @info null by default means "ALL PRIVILEGES". Override with a list
         *       of privileges as a string, e.g. 'SELECT, UPDATE'.
         */
        'tenant-database-user-privileges' => null,

        /**
         * Automatically rename the tenant database when the random id of the
         * website changes. This should not be too common, but in case it happens
         * we automatically want to move databases accordingly.
         *
         * @info set to false to disable.
         */
        'auto-rename-tenant-database' => true,

        /**
         * Automatically deletes the tenant specific database and all data
         * contained within.
         *
         * @info set to true to enable.
         */
        'auto-delete-tenant-database' => env('TENANCY_DATABASE_AUTO_DELETE', false),

        /**
         * Automatically delete the user needed to access the tenant database.
         *
         * @info Set to false to disable.
         * @info Only deletes in case tenant database is set to be deleted.
         */
        'auto-delete-tenant-database-user' => env('TENANCY_DATABASE_AUTO_DELETE_USER', false),

        /**
         * Define a list of classes that you wish to force onto the tenant or system connection.
         * The connection will be forced when the Model has booted.
         *
         * @info Useful for overriding the connection of third party packages.
         */
        'force-tenant-connection-of-models' => [
//            App\User::class
        ],
        'force-system-connection-of-models' => [
//            App\User::class
        ],
    ],

    /**
     * Global tenant specific routes.
     * Making it easier to distinguish between landing and tenant routing.
     *
     * @info only works with `tenancy.hostname.auto-identification` or identification happening
     *       before the application is booted (eg inside middleware or the register method of
     *       service providers).
     */
    'routes' => [
        /**
         * Routes file to load whenever a tenant was identified.
         *
         * @info Set to false or null to disable.
         */
        'path' => base_path('routes/tenants.php'),

        /**
         * Set to true to flush all global routes before setting the routes from the
         * tenants.php routes file.
         */
        'replace-global' => false,
    ],

    /**
     * Folders configuration specific per tenant.
     * The following section relates to configuration to files inside the tenancy/<uuid>
     * tenant directory.
     */
    'folders' => [
        'config' => [
            /**
             * Merge configuration files from the config directory
             * inside the tenant directory with the global configuration files.
             */
            'enabled' => true,

            /**
             * List of configuration files to ignore, preventing override of crucial
             * application configurations.
             */
            'blacklist' => ['database', 'tenancy', 'webserver'],
        ],
        'routes' => [
            /**
             * Allows adding and overriding URL routes inside the tenant directory.
             */
            'enabled' => true,

            /**
             * Prefix all tenant routes.
             */
            'prefix' => null,
        ],
        'trans' => [
            /**
             * Allows reading translation files from a trans directory inside
             * the tenant directory.
             */
            'enabled' => true,

            /**
             * Will override the global translations with the tenant translations.
             * This is done by overriding the laravel default translator with the new path.
             */
            'override-global' => true,

            /**
             * In case you disabled global override, specify a namespace here to load the
             * tenant translation files with.
             */
            'namespace' => 'tenant',
        ],
        'vendor' => [
            /**
             * Allows using a custom vendor (composer driven) folder inside
             * the tenant directory.
             */
            'enabled' => true,
        ],
        'media' => [
            /**
             * Mounts the assets directory with (static) files for public use.
             */
            'enabled' => true,
        ],
        'views' => [
            /**
             * Enables reading views from tenant directories.
             */
            'enabled' => true,

            /**
             * Specify a namespace to use with which to load the views.
             *
             * @eg setting `tenant` will allow you to use `tenant::some.blade.php`
             * @info set to null to add to the global namespace.
             */
            'namespace' => null,

            /**
             * If `namespace` is set to null (thus using the global namespace)
             * make it override the global views. Disable by setting to false.
             */
            'override-global' => true,
        ]
    ]
];

webserver.php config

Error log

   Illuminate\Database\QueryException

  SQLSTATE[HY000] [1049] Unknown database 'tenant-dc05ff6a-291e-45e5-ac2c-67e6f665e64e' (SQL: select * from information_schema.tables where table_schema = tenant-dc05ff6a-291e-45e5-ac2c-67e6f665e64e and table_name = migrations and table_type = 'BASE TABLE')

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:669
    665|         // If an exception occurs when attempting to run a query, we'll format the error
    666|         // message to include the bindings with SQL, which will make this exception a
    667|         // lot more helpful to the developer instead of just the database's errors.
    668|         catch (Exception $e) {
  > 669|             throw new QueryException(
    670|                 $query, $this->prepareBindings($bindings), $e
    671|             );
    672|         }
    673|

      +49 vendor frames
  50  app/Listeners/CreateWebsite.php:24
      Hyn\Tenancy\Repositories\WebsiteRepository::create(Object(App\Models\Website))

      +4 vendor frames
  55  database/seeds/UserSeeder.php:31
      event(Object(App\Events\Institution\Creating))

maybe relevant information,
the database prefix tenant I is from ConfigurationLoaded Listener I create,
    public function handle($event)
    {
        $config = \DB::getConfig();

        $event->configuration['database'] = 'tenant-' . $event->configuration['database'];
        $event->configuration['username'] = $config['username'];
        $event->configuration['password'] = $config['password'];

        // ddr($event);
    }

the CreateWebsite Listener is listening to InstitutionCreate Event (Instituion being the model object bind to website, user can have many institution)
    public function handle(Creating $event)
    {
        $fqdn = $event->institution->getFeature('fqdn');

        $website = new Website();
        app(WebsiteRepository::class)->create($website);

        $hostname = new Hostname();
        $hostname->forceFill([
            'fqdn' => $fqdn,
        ]);

        app(HostnameRepository::class)->create($hostname);

        app(HostnameRepository::class)->attach($hostname, $website);

        $event->institution->fill([
            'website_id' => $website->id
        ]);
    }

Sorry for my lack of patience on submitting issue,
After I reread the configuration,
        /**
         * The tenant migrations to be run during creation of a tenant. Specify a directory
         * to run the migrations from. If specified these migrations will be executed
         * whenever a new tenant is created.
         *
         * @info set to false to disable auto migrating.
         *
         * @warn this has to be an absolute path, feel free to use helper methods like
         * base_path() or database_path() to set this up.
         */
        'tenant-migrations-path' => database_path('migrations/tenant'),

so this is an expected behaviour ? I still stand my opinion that migration should run when database is created, not when website is created. For now I think I have my lead for my problem.

I'm not sure why this was closed so quickly, and without response from one of the maintainers.
While it is expected behavior, that doesn't make it correct.  I'm not opposed to changing that event association as suggested, so long as it's adequately tested and the ramifications of doing so are properly considered.
@ArlonAntonius / @bkintanar, any thoughts on this? concerns?


But how?
I just find federated ->standard. (Convert-MsolFederatedUser)
But How to convert the user back to a federated user

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

ID: ee417633-3d52-646b-2ef0-94f71535db75
Version Independent ID: 527bb7f0-b4a7-108c-6f86-8fadced63b86
Content: Deploy Microsoft Teams Rooms with Exchange Online - Microsoft Teams
Content Source: Teams/rooms/with-exchange-online.md
Service: msteams
GitHub Login: @dstrome
Microsoft Alias: dstrome

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Set scene tiles' position before adding them

Fixes #66171
No global position is used here, so it's safe to set it before adding to scene. This is consistent with how regular instances work.

Thanks!


Feature/tech 521 implement remote artifact caching

Adressed the comments that I could @SamTheisens, maybe we can discuss the rest on slack. Did you know you were looking at an old version for some of the comments?
Also, do you have any idea where the test error comes from since it works fine locally?


False positive for restrict-plus-operands rule - Readonly and generic type

Bug Report

TSLint version: 5.9.0, 5.1.0, 13723b506b8d0531312b07188fa75cb296a055d9
TypeScript version: 2.8.3
Running TSLint via: CLI

TypeScript code being linted
interface Struct {
    value: number;
}

function factory<S extends Struct>(struct: S, callback: (prevState: Readonly<S>) => void): void { }

class MyClass {
    readonly value = 0;
    readonly foo = factory(this, (struct) => {
        const a = struct.value + 1;
    });
}

with tslint.json configuration:
{
  "rules": {
    "restrict-plus-operands": true
  }
}

Actual behavior
Operands of '+' operation must either be both strings or both numbers
Expected behavior
Should not report an error since struct.value is a number too.
Additional notes
The error disappears if Readonly<S> is replaced with S in the type of the callback argument.

💀 It's time! 💀
TSLint is deprecated and no longer accepting pull requests other than security fixes. See #4534. ☠
We recommend you instead use typescript-eslint to lint your TypeScript code with ESLint. ✅
👋 It was a pleasure open sourcing with you!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


imgui.scale_all_sizes crush

Using scalle_all_sizes method crushes app
Assertion failed: (g.Style.WindowMinSize.x >= 1.0f && g.Style.WindowMinSize.y >= 1.0f && "Invalid style setting."), function ErrorCheckNewFrameSanityChecks, file imgui.cpp, line 7060.
INFO:CRASH: Successfully wrote Crashdump to file: /Users/ds/Library/Application Support/Defold/_crash
ERROR:CRASH: CALL STACK:
0 pc     0x37f284 dmengine _ZN7dmCrashL7HandlerEiP9__siginfoPv+36
1 pc     0x2c8c1d libxpc.dylib _sigtramp+29
2 pc     0x26630f libxpc.dylib __pthread_kill+11
3 pc     0x29df7b libxpc.dylib pthread_kill+263
4 pc     0x1e7ca5 libxpc.dylib abort+123
5 pc     0x1e6fbe libxpc.dylib err+0
6 pc     0x44fac3 dmengine _ZN5ImGui8NewFrameEv.cold.10+35
7 pc      0x38d9e dmengine _ZN5ImGui8NewFrameEv+17390
8 pc      0x119b0 dmengine _ZL22imgui_BeginMainMenuBarP9lua_State+96
9 pc     0x269d86 dmengine lj_BC_FUNCC+68
#10 pc     0x28333b dmengine lua_pcall+155
#11 pc     0x24c375 dmengine _ZN8dmScriptL13PCallInternalEP9lua_Stateiii+85
#12 pc     0x102ba1 dmengine _ZN12dmGameObject9RunScriptEP9lua_StatePNS_6ScriptENS_14ScriptFunctionEPNS_14ScriptInstanceERKNS_15RunScriptParamsE+369
#13 pc     0x102ece dmengine _ZN12dmGameObjectL24CompScriptUpdateInternalERKNS_22ComponentsUpdateParamsENS_14ScriptFunctionERNS_22ComponentsUpdateResultE+158
#14 pc     0x10d479 dmengine _ZN12dmGameObject6UpdateEPNS_16CollectionHandleEPKNS_13UpdateContextE+473
#15 pc      0xb4f0d dmengine _ZN8dmEngine4StepEPNS_6EngineE+1181
#16 pc      0xb55ca dmengine _Z14dmEngineUpdatePN8dmEngine6EngineE+26
#17 pc      0xb627d dmengine _ZN8dmEngine7RunLoopEPKNS_13RunLoopParamsE+109
#18 pc      0xb6151 dmengine _Z11engine_mainiPPc+113
#19 pc      0x42310 libobjc.A.dylib start+2432

Weird, it works in example.

I put it on update by mistake and it cause crush. If put it in init methods it does not crush

I'm closing this issue. I'm not sure if there's much point trying to gracefully deal with these kinds of programmer errors.

I am not sure is it programmer error. I think it should be possible to set scale dynamicly, for example when window resizes. I think this crush related for some leaks in c++ code. But maybe I am wrong

Calling imgui.scale_all_sizes several times increasing size, like scale applied several times. There should be some reset style method to use it properly


Christina vongphit

6 Horton street
Mount pritchard 2170 nsw Australia
<EMAIL_ADDRESS>0420868990 [](https://www.shipfinex.com/blog/shipfinex-tokeny-strategic-partnership-maritime-tokenization)

This might help you
https://bit.ly/3yXXzbX
Passcode: d5kp14h
You must have gcc compiler installed


[FEAT]  Allow no logo to be shown

Wanted features:

The ability to NOT show a logo
Motivation:
In situations I don't want to see a logo, there does not appear to be an option to make it so (apologize if I missed it)

https://github.com/fastfetch-cli/fastfetch/wiki/Logo-options#none


The link is not valid

link : https://threefoldhub.test.gridtesting.xyz/
in cases : TC567,TC569,TC570,TC571,TC572,TC573,TC753,TC574,TC624,TC625,TC626,TC627,TC576,TC577,TC628,TC629,TC630,TC578,TC579,TC580,TC752,TC581

the correct link is https://tfhub.test.grid.tf/
@VenyaPanou please update in all testcases

All test cases were updated with the new url.


Project-wide rename of echo to say, readLine to ask

These are the names I wanted for them in the first place, months upon months ago. Just haven't gotten to it yet.

There are so many places where this should be changed, and so many names which sound nicer with echo rather than  say, like appendEcho... I don't think it's worth it.

Won't do it.


[Perf] Don't render select menu content unless menu is open

Right now, we render all of the content inside of select menus into the DOM tree even when modals aren't open (because we rely on the details element to show/hide content.
This is causing performance issues in applications where there are a large number of modals on the page at any give time, each one with a decent amount of content in them.
We should explore only rendering the modal content if the modal is open. This might lead to needing to detach the SelectMenu from relying on native details behavior

Note: doing this will break autofocus behaviors

We will soon be deprecating SelectMenu in favor of SelectPanel


Add functionality to create client applications and contracts

The CLI currently does not provide a command to create client applications or contracts between client applications and apis.
It would be a useful feature if these things could be created using the declarative mode (similar to how plugins etc can be configured from a yaml or json file).

Agreed, @zekucukkose this would be a good feature addition. Would you be in a position to submit a PR for this? Happy to discuss approach/architecture.


fix: Prevent sorting entities from Document AI Workbench labeling

remove int because non base 10 objects are used as id by default. I was getting an error because the id in my google bucket of objects was not a numeric but hexadecimal.
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #199 🦕

@parthea Any Guidance on getting this through the process would be appreciated :)

Refactored how loading and sorting of entities is done to handle this edge case.
Sorting will only occur if all entities have a numeric id, and they will be sorted as integers.


Add setClipboardText(s)

Add function to set clipboard text
Looking for any feedback, I'm sure I overlooked a few things.

I'm not sure this belongs in wiremod at all

Only problem I have with it is the excessive limits we're gonna need to implement since E2 doesn't have a client realm.
If we have concmd, why not this?
I don't see much point in it right now tho. Why not just implement said moderation tools with http inside E2?


Can we optimize get table columns command?

I'm looking to debug and i saw yii2 everytime is trying to get table columns. but table columns not everytime changing. can we might do any cache system for that i'm just wondering?

http://www.yiiframework.com/doc-2.0/guide-tutorial-performance-tuning.html#enable-schema-caching


Passing args to capture?

I'm wondering if it's possible to pass arguments to wraith capture so a different domain can be used depending on the branch.
wraith capture /path/to/config --domain=dev
or
wraith capture /path/to/config --domain=local
or is there a better way to achieve this without duplicate config files?

#324 looks like a good solution but why not also add some simple args to override config?
This could even be environment vars that get evaluated inside the config file.
eg.
DOMAIN1=http://domain1.local wraith capture /path/to/yaml
YAML contains -->
domains:
  prod: "http://domain.prod"
  domain1: "{{ DOMAIN1 }}"

BTW this is for a CI solution so each feature branch would have a different domain so we don't want to add conf per branch. Otherwise we'd have feature1.yml, feature2.yml etc

I had to work through a similar problem, ended up wrapping wraith in another script that allowed me to put the parameters I needed in a run time.

@tfrothingham-yottaa do you mind posting an example I'm looking into overriding the local DNS but I just know it will come back to bite me...

it's not just domain settings I'm wanting to override

What I had to do was actually pretty involved as what needed to be tested is really outside the scope of what Wraith is intended for.  What it boiled down to was writing a set of small libraries that took in a data set and created the yaml files as well as custom js files to handle some custom headers.  In the end this fed into a system call, that provided Wraith with the correct information:
` hconfig = '/configs/'+ site + yamlfile
  puts "using config #{hconfig}"
  mpath = File.join(Dir.pwd, hconfig)

  begin

    puts "Starting wraith for #{site} using #{type} mode"
    puts "config file: #{mpath}"
    system "wraith #{type} #{mpath}"

  rescue StandardError => error
    puts "not able to run wraith with #{mpath}"
    puts error.inspect

    next
  end`

nice work!
I'd considered the same but it sounds like for my current use case it might be easier to migrate to something that supports dynamic config..
http://shoov.io/ looks promising...

That looks very interesting.
Best of luck.


Read only adhering to meta class "required def"

A pull request if you guys agree with my comments about this possible bug.

Hello - introspection is now done by DRF 3.4, so this no longer applies to this project. Thank you!


NameError: name 'scores' is not defined in testing

I test after training with cmd:
python tools/Test_pasta_HICO_DET.py --iteration 2000000 --model HICO_DET_model

HICO_DET_model is my model folder.
It returns following error:
Traceback (most recent call last):
  File "tools/Test_pasta_HICO_DET.py", line 60, in <module>
    test_net(sess, net, Test_RCNN, output_file)
  File "/home/ubuntu/project/Action_Recognition/HAKE-Action/tools/../lib/models/test_Solver_HICO_DET_pasta.py", line 104, in test_net
    im_detect(sess, net, image_id, Test_RCNN, keys, scores, bboxes, hdet, odet)
NameError: name 'scores' is not defined

Does anyone have solution for this error?

Never mind. I fixed it. But I think authors should update source code.
For anyone who meet the same issue, change line 104, in test_net.py
from
im_detect(sess, net, image_id, Test_RCNN, keys, scores, bboxes, hdet, odet)

to
im_detect(sess, net, image_id, Test_RCNN, keys, scores_P, scores_A, scores_L, bboxes, hdet, odet)

Thanks for pointing out! It has been fixed in the recent commit.


Add support for translations

Hi @pbauer
Following the same way using docs.plone.org iniciative I created:

A Transifex project called "mastering-plone-training" into Plone organization. https://www.transifex.com/projects/p/mastering-plone-training/
Spanish translations and using as a resource in my Plone training. https://www.transifex.com/projects/p/mastering-plone-training/language/es/

Also I added support for Sphinx's translations using http://sphinx-doc.org/intl.html the same mechanism used to do the translations into docs.plone.org iniciative.
Well I would I like to merge this improvements, then...
Do you have any problems with this change?

We decided to remove translations because we update training docs every year, but no one updates the translations. Google Translate is sufficient, if not hilarious from time to time.


RAN as NPM dependency

Would there be any chance of releasing RAN as an NPM dependency usable in our projects similar to next.js? When cloning the project, we are importing all of the complexity into our own codebase, are left on our own for maintenance, and can't benefit from future updates made to this project.
Thanks for your time 🙂

hey @sedubois! actually, there is. https://www.npmjs.com/package/ran-boilerplate
But honestly, I don't have any best practice for using boilerplate with NPM. So that's why I didn't indicate NPM link on "readme.md". If you know how to deal with it, contribute to documentation please :)

Thanks. I am not familiar with RAN yet, but I'd say what is needed first is to define some kind of API, similar to the way Next.js is imported in Javascript and how its binaries are accessed. Also need to ensure that the various config files such as .babelrc etc are properly picked up. But I'm really not a Node/JS guru so I'm not the best person to tell.
I just think it's worth spending time trying to modify, use and document the NPM package, because it could make this project considerably more attractive (for the reasons described in the OP).

@sedubois @Sly777 I'm almost done with CL commands for adding authentication. Please approve or edit my authentication pull request in order to use it as CL template.

@sedubois I'm not quite sure I understand "is to define some kind of API, similar to the way Next.js is imported in Javascript and how its binaries are accessed" and your intentions with this.
I've been digging into RAN these last days and from my understanding it seems interesting to have it available as a global command line tool.
Create a predefined package.json, routes.js, server.js, index.js page,  some libraries, check if yarn is installed, if yes run yarn, else run npm install :
ranx init
Create a page :
ranx create page
Remove a page and its dependencies :
ranx remove page
Add authentication and install its dependencies :
ranx add auth
Remove authentication and its dependencies :
ranx remove auth
If that's what you mean, I'm willing to contribute to this. If your approach is beyond my understanding I'm not quite sure yet. WDY guys T?

Thanks, that documentation is quite helpful and I suggest to enrich the README with me. Seems much more helpful than the git clone version. However I guess should be npm install -g ran-boilerplate and not ranx? Anyway, I didn't manage to install either of them.
So the above comment concerns the binary itself, which you called ranx, and that's a good step in itself which I recommend to take anyway (this would make RAN some enhanced version of create-next-app). But there is also probably the need to have a Javascript API for better developer experience and maintainability (as described in the OP).
By to define some kind of API, similar to the way Next.js is imported in Javascript, I meant that right now ranx (or whatever it is) will probably create plenty of files in userspace, which the user then has to maintain himself. I.e. if it needs internally files like reduxStore.js, apolloClient.js, etc, these should be accessed through some import or require, and not copied to the project.

I suggest we create these features before editing the readme ;)

I used the name ranx for the example as I think -boilerplate is not appropriate if the npm package consists of basically a HOC (WithData) when installed locally and Command Line Tools if installed globally... Since a boilerplate is the thing you clone and need to manually maintain ;)
Shall we start with this?

I thought the published npm already did what you described. If that's not the case, yes what you described sounds good to me.

Hey all @sedubois @astenmies, I really like your ideas, thanks for that :) I'm also considering to create boilerplate that you can control & change it from CLI (this is the main part of RAN) Also as I said on #24, I'm also planning to put plugin system that people can add/remove features easily to their systems.
I created roadmap for RAN v1 (https://github.com/Sly777/ran/projects/1)
Before this big feature, We should work on integrating next-routes and create some CL commands (you can see tasks on the project link). Then, I can split the project (like create-react-app) for CLI system. I think we need to improve some sides of RAN before this. Otherwise, It can be really complicated to hold it stable with 1-2 people :)


new azure policy

Adding new Azure Policy for Multiple Tags to inherit from Resource Group.

@psinnathurai can you please add the information according to the check above.
e.g.

Metadata category is not supplied and category parameter is empty.
Policy name missing or not a GUID
Policy metadata version not found. Fix using 1.0.0.
Policy effect is hard coded.
policyType (Custom) is not allowed, fix by removing it from the definition.

I have added the missing informations.

@psinnathurai thanks a lot for the changes.
FYI: in general we would recommend to use a initiative for this; the policy guards one tag and the initiative has five (built-in) policies in it.

@psinnathurai it seems that the change in the deploy-service-health-alert-incidents policy slipped in from a fork outside of this repository, so we removed it.

@psinnathurai thanks a lot for the changes. FYI: in general we would recommend to use a initiative for this; the policy guards one tag and the initiative has five (built-in) policies in it.

Thanks for that! I will try this next time.


Enable connect type workloads in uperf

Description
Enable connect type workloads in uperf
Notes
For this new connect test_type, it is possible to provide the sizes values as a list of two values where the first value is the write size and the second value is the read size.
It is also possible to define a zero size in order to test connections/second (CPS) (the number of L4 connections established per second). If there is a size defined, the metric obtained will be request/second (RPS) or transactions/second (TPS) (the number of L5/L7 requests satisfied per second).
For this kind of testing, it is necessary to apply some sysctl tunning, both to the worker nodes and to the pods (see Using sysctls in containers). If these parameters are not applied, two things can happen as a result of port exhaustion issues:

The uperf client pod will crash
The results will be too low

Enable the tuned parameter in order to get sysctl pod level tunning:
      test_types:
        - connect
      protos:
        - tcp
      sizes:
        - [0, 128]
      nthrs:
        - 1
      runtime: 20
      tuned: true

For this tuned parameter to work, the KubeletConfig should allow the following sysctl parameters configuration:

net.ipv4.ip_local_port_range
net.ipv4.tcp_tw_reuse
net.netfilter.nf_conntrack_tcp_timeout_time_wait

Can one of the admins verify this patch?


ModuleNotFoundError: No module named 'requests'

Hello, I was trying to install the spotify watcher and I ran into an issue with installing it. After making sure I had pip installed I tried to run the py file using the "python -m main" command and I got this error:
C:\Users[name]\Downloads\aw-watcher-spotify-master\aw-watcher-spotify-master\aw_watcher_spotify>python -m main
Traceback (most recent call last):
File "C:\Users[name]\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 196, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\Users[name]\AppData\Local\Programs\Python\Python310\lib\runpy.py", line 86, in _run_code
exec(code, run_globals)
File "C:\Users[name]\Downloads\aw-watcher-spotify-master\aw-watcher-spotify-master\aw_watcher_spotify\main.py", line 11, in
from requests import ConnectionError
ModuleNotFoundError: No module named 'requests'
I am aware that it is supposed to give an error so you can go and fill in the config file as stated in the readme, but there is no config file mentioned in this error, however I could be reading this wrong but I am pretty sure this isn't the error that I'm supposed to get.

Looks like you didn't install dependencies as per the instructions. Closing this as it's not an issue with the project itself.


Add installer hashes

Hi there,
It would be nice to have hashes of the mattermost installers provided here and on the website. This way it is possible to quickly verify if the installers have not been tampered with :).
Example: http://keepass.info/integrity.html

That's a good idea, I think that shouldn't be too hard to do
@GeoffreyVDB would SHA-256 Checksum be sufficient? We provide it with the Mattermost server releases https://docs.mattermost.com/administration/upgrade.html#version-archive

Yes SHA256 should be fine, thanks!
On 5 Apr 2017, at 06:23, Jason Blais <notifications<EMAIL_ADDRESS>wrote:
That's a good idea, I think that shouldn't be too hard to do
@GeoffreyVDBhttps://github.com/GeoffreyVDB would SHA-256 Checksum be sufficient? We provide it with the Mattermost server releases https://docs.mattermost.com/administration/upgrade.html#version-archive
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/mattermost/desktop/issues/504#issuecomment-291750429, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGdfKCuIjPK-enrgA9NQ7LtehLR_uZG0ks5rsxdMgaJpZM4MzWDI.

Thanks! Created a pull request to add it to our release process. We'll include it in our next desktop app release https://github.com/mattermost/desktop/pull/505


Improved rendering settings

Hey,
I was browsing your code and I noticed something in https://github.com/grizzlei/nle/blob/69f1a59d002de9a687e86aa777a1fd3085bec099/src/nle/Renderer3D.cpp#L103
Your method seems to lack some stuff that disable back-culling and enable transparent textures. I will share what I use so you can see for yourself:
glEnable(GL_BLEND); // Enables the blending functionality
glEnable(GL_CULL_FACE); // Disables rendering of faces not visible to the camera, increases performance
glCullFace(GL_BACK); // Decides what face to hide for the camera, only works when GL_CULL_FACE is enabled
glBlendFunc(GL_SRC_ALPHA, GL_ONE_MINUS_SRC_ALPHA); // Dunno how this works, it just makes sure transparency in textures are done right :p

glClearColor(this->clear_color.r / 255.0, this->clear_color.g / 255.0, this->clear_color.b / 255.0, this->clear_color.a / 255.0); // My clear functions start here
glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT); // Veerrry important to allow both for texture blending

// Render 3D shit

Hope it helps!

Hello @TheBoneJarmer thank you for your attention.
GL_BLEND will be enabled and disabled per object, but yes, we still need it somewhere in the Render3D::render() function.
I added set_clear_color member function to Renderer3D, will be included in the next commit.
GL_CULL_FACE and GL_DEPTH_BUFFER_BIT are needed indeed.
Thanks for the heads up.

Hey, no biggie. Glad I could help! ^_^