:bug: TypeError: cannot pickle sqlite3.Connection object

This problem caused by type:
torch.utils.data.dataloader.DataLoader
here:
https://github.com/nicolay-r/ARElight/blob/e18e7441f2964c4cac3e1feaf4f3d12e3b0520ab/arelight/pipelines/items/inference_bert_opennre.py#L125
Raised #149 related issue.

[x] #149

The most similar issue mentioned there:
https://discuss.pytorch.org/t/dataloader-multiprocessing-error-cant-pickle-odict-keys-objects-when-num-workers-0/43951
which navigates us to:
https://github.com/pytorch/vision/issues/689
from where as we can see, we got no control over the related parameter (its value binded to 8):
https://github.com/nicolay-r/ARElight/blob/e18e7441f2964c4cac3e1feaf4f3d12e3b0520ab/arelight/third_party/torch.py#L89-L97
Traceback (most recent call last):
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/Users/kmax/PycharmProjects/ARElight-main/arelight/run/infer.py", line 328, in <module>
    BasePipelineLauncher.run(pipeline=pipeline, pipeline_ctx=PipelineResult(merge_dictionaries(settings)),
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/arekit/common/pipeline/base.py", line 18, in run
    item_result = item.apply(input_data=input_data, pipeline_ctx=pipeline_ctx)
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/arekit/common/pipeline/items/base.py", line 48, in apply
    output_data = self.apply_core(input_data=input_data, pipeline_ctx=pipeline_ctx)
  File "/Users/kmax/PycharmProjects/ARElight-main/arelight/pipelines/items/inference_writer.py", line 31, in apply_core
    self.__writer.write(header=header, contents_it=contents_it,
  File "/Users/kmax/PycharmProjects/ARElight-main/arelight/predict/writer_sqlite3.py", line 21, in write
    SQLiteProvider.write(
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/arekit/common/service/sqlite.py", line 26, in write
    for uid, data in data_it:
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/tqdm/std.py", line 1195, in iter
    for obj in iterable:
  File "/Users/kmax/PycharmProjects/ARElight-main/arelight/predict/provider.py", line 16, in __iter_contents
    for sample_id, uint_label in sample_id_with_uint_labels_iter:
  File "/Users/kmax/PycharmProjects/ARElight-main/arelight/pipelines/items/inference_bert_opennre.py", line 125, in iter_results
    for iter, data in enumerate(eval_loader):
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 441, in iter
    @property
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 388, in _get_iterator
    def multiprocessing_context(self, multiprocessing_context):
  File "/Users/kmax/PycharmProjects/ARElight-main/venv/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1042, in init
    self._data_queue = queue.Queue()  # type: ignore[var-annotated]
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/process.py", line 121, in start
    self._popen = self._Popen(self)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/context.py", line 224, in _Popen
    return _default_context.get_context().Process._Popen(process_obj)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 32, in init
    super().__init__(process_obj)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_fork.py", line 19, in init
    self._launch(process_obj)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/popen_spawn_posix.py", line 47, in _launch
    reduction.dump(process_obj, fp)
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/multiprocessing/reduction.py", line 60, in dump
    ForkingPickler(file, protocol).dump(obj)
TypeError: cannot pickle 'sqlite3.Connection' object

Fixed: could be reproduced with no issues on Mac


Hansteen @ Norske Korrespondanser?

Hei -
How about joining the Norwegian Correspondences-project with the metadata from Hansteen letters?!
https://github.com/arockenberger/NorKorr

Very interesting, we'd be happy to! Didn't know about the Correspondence Metadata Interchange format, but will look into it, we can probably produce a CMIF version of the Hansteen's correspondence.

That would be awesome. However, at this point, CMIF only works for digital scholarly editions or printed editions, not for digital facsimiles. I am working on an adaptation of the CMIF-standard for digitized materials that have a persistent identifier and are "long-term archived", i.e. a library or archive takes care of curation of the digital object. We could talk about that at some point @danmichaelo @superLine ?


Official banner / logo

It would be nice to have a good looking banner for the README/Wiki, and a square logo/icon for the executable, that could also be repurposed as a badge for system notifications.
Ideas for what would be suitable graphics for these purposes are welcome.
If anyone has experience with this kind of work and want to contribute, that would be great!
To stick with the spirit of the rest of this project, I would like any contributed graphics / design to be made available under a permissive license (CC0?).
Thanks for taking an interest in this software.

@koekeishiya  Hi, my name is Richard. I'm a graphic designer and I love making contributions for open source projects. I see that they want a new logo for your application. If you wish, I can do it for you without any problem ... I can present some modern ideas that would improve the appearance and image of your project.
My Portfolio

@richardbmx
That would be great, I'd like that a lot. Looking forward to see what ideas you can come up with

@koekeishiya  Perfect I will work on it

@koekeishiya Happy first day I want to apologize for the delay I had some problems with the internet connection.
Here I want to present these two ideas that I developed for your project in both I express the management of windows for IOs, in the second idea I created a figure that works a window with the letter Y to give a more unique personality for your project, I hope you please if you want some change you just have to request it and I will gladly do it

@richardbmx
No need to apologize for the delay. Thanks again for doing this!
I think both are pretty great. I think the first one would fit best as a banner used in the README/wiki, where as the Y in the second one would be a great fit as the executable logo.
What do other people think?

I particularly like the idea of the top right icon, so I tried to re-create it using the 10px border radius of macOS windows and the traffic light buttons original size, position and colours and gave the front window a 2px shadow. I think this fits a bit better.
I'm not sure about using a Y on the icon. I think logo + "yabai" to the right (bottom left) is the best layout for the banner. I like the the font you used very much.
I experimented with using fontawesome icons that signal automation or power user tools instead of the Y on the icon, but I'm not too sure I like it.

Logo

Banner (ignore the font)

I really like the multiple windows, but shouldn't the windows be somewhat tiled? The second design could be visually confusing since it can resemble the Kodi icon.

My idea:

The icon represents tiling while rotated to present a "Y." And of course, the colors in the lettering signify window controls. Simple.

My idea:

The icon represents tiling while rotated to present a "Y." And of course, the colors in the lettering signify window controls. Simple.

Maybe a different font? Either way I'm loving it as is.

Hi i just created a logo/iocn for Yabai, tell me where you need improvement. When the logo will be finalized then i will move to the banner. feel free to criticize.

The stacked windows don't really reflect a tiling window manager.
I think @r3volution11 has the right idea. Either the logo should be literal (representing tiling) or absolutely not literal (text with another illustration.)
Like r3v's, I think it should be more along lines of something like this -- but not this because it's not good.

The idea of @r3volution11  is very clever and I want to contribute my version to it. So I tried to make the logo a bit more playful.
I hope you like it.

@fools-mate I like this one a lot actually.
There are a lot of great ideas present here, and I'm not sure how to go about deciding on which one to go with.

@fools-mate Thanks for the comment. I like what you've done as well. I'm not so sure about overlaying the font when showing the full name but I see how it makes sense in keeping it together. I wonder if removing the window control colors would be it look less cluttered (just a white fill).

@koekeishiya & @r3volution11 - Thank you for the feedback!
@r3volution11 - I fully understand your arguments and I think the new variant could be a good approach to make the logo cleaner.
I also fixed the optical illusion that the Y looked thinner than the rest.

@fools-mate I mostly like the last variant, it's great!
Can you tilt the 'y' a little more? I feel that the slope of the right line of 'y' is too steep.

I really, really like that last one by @fools-mate. That‘s the one for me, I think.

@fools-mate I like that last variant one as well. Could you upload in this issue or send me a 512x512 version of the icon and I'll add it to the repository. As @dominiklohmann is responsible for the wiki he would need the banner image. I suppose that could also be stored in the repo if necessary, but it is probably sufficient to simply upload it in an issue and refer to it by url like I do with screenshots.

I‘d prefer Sketch/PSD if possible, then we can always adjust the size later. Also icons are 1024x1024 per Apple Human Interface Guidelines for the 2x version.

Also icons are 1024x1024 per Apple Human Interface Guidelines for the 2x version.

Hmm weird, the following docs say 512: https://developer.apple.com/documentation/appkit/nsworkspace/1529882-seticon?language=objc

https://developer.apple.com/design/human-interface-guidelines/macos/icons-and-images/app-icon/

@koekeishiya  & @dominiklohmann - I'm really glad you like it!
In the archive you will find everything you need.
(banner & icons as svg and png (16px up to 1024px))
This is a version without drop shadows. If you want one, just tell me.
yabai-logo.zip

Thank you so much! 👍

@fools-mate in the banner, there is a white circle around the red dot in the a, while there's no white circle in the yellow and green dots. Can you make it transparent for the a as well so the banner doesn't look off on backgrounds other than white? That'd be very much appreciated!

@dominiklohmann - Oh, I missed that. Here you get the corrected version.
yabai-banner-fix.zip

Thanks to everyone who participated with ideas, and designs. I've decided to go with the version made by @fools-mate. Closing this issue.

Out of curiosity, I've always been wondering what the name "yabai" actually means. All I know is that in Japanese it means "very bad"/"dangerous" 😅. Did you have that in mind when you chose that name @koekeishiya (also your handle sounds Japanese but I don't think it's an actual Japanese word, or is it?)


Renaming binary

Issue #, if available: N/A
Description of changes:

Removing amazon- prefix from binary name
Updated documentation, config files, and tests to reflect updated name
Updated finalCfgDir to align with updated name

Testing:

make clean && make build && make test were successful
make build-binaries resulted in binaries with the updated name

Notes:

I will update README installation instructions after v0.9.1 release is created and its assets populated correctly by travis

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #16 into master will not change coverage.
The diff coverage is 62.50%.

@@           Coverage Diff           @@
##           master      #16   +/-   ##
=======================================
  Coverage   51.65%   51.65%
=======================================
  Files           5        5
  Lines         211      211
=======================================
  Hits          109      109
  Misses        100      100
  Partials        2        2

Impacted Files
Coverage Δ

pkg/cmd/scheduledevents/scheduledevents.go
35.18% <50.00%> (ø)

pkg/cmd/spotitn/spotitn.go
38.09% <50.00%> (ø)

pkg/cmd/root/root.go
43.54% <75.00%> (ø)

Continue to review full report at Codecov.

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


Change dependency on Formtastic from <2.0.0 do >=1.1.0

When i've tried to use active_admin and client_side_validations in my app, i've got an error when trying to run server:
uninitialized constant Formtastic::FormBuilder

I've found some info about this issue here: http://stackoverflow.com/questions/6438449/rails-3-uninitialized-constant-formtasticformbuilder-error-after-deploying-a and it turns out that it is beouse client_side_validations needs Formtastic 2, but activeadmin has dependency on formtastic < 2.0.0, changing it to >= 1.1.0 fixed it.

Take a look at this pull request: https://github.com/gregbell/active_admin/pull/527
It does not work to change the dependencies only ;).

well, for me it worked - now i have no error on my app using activeadmin and client_side_validations, but I see that your request will fix it too :) so i'm closing this one and i hope that this request will be pulled soon!


Identify members

Add github handles to member list in README.md we know who is whom
ie: Dan (dldf), John (johntpaxton), Carson (1cf)

complete


[test](copy into) add copy into regression test case (#41529)

pick (#41529)
Add copy into regression test case.

run buildall


I can't submit comments when i use es-es language

I can't submit an error using es-es language nor see the existing ones. There is an error message. Thanks Microsoft Team.

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

ID: ca396f1f-9063-203c-5663-bc1750ac0d7f
Version Independent ID: 639bd566-335e-b107-d60d-ba158b29c908
Content: Filtering - PowerApps
Content Source: powerapps-docs/developer/component-framework/reference/filtering.md
Service: powerapps
GitHub Login: @Nkrb
Microsoft Alias: nabuthuk

@danielftk Thanks for the feedback. Are you trying to raise a GitHub issue on the doc or do you have issue with localization?

yes, first I found a typo using the page on es-es lang (green highlighted). Then I tried to raise an issue using the same es-es page but it returns me an error on the GitHub/issues section (Aqua highlighted

). Finally, I post the issue using the en-us page. The issue: https://github.com/MicrosoftDocs/powerapps-docs/issues/1011#issue-561416052

@danielftk Thanks for the details. This is a localization issue and will be fixed in the next update.

I will investigate and fix.

Assigning to @jowilco

Hello @danielftk - I believe that you should now be able to create issues in the future. Please let us know if you are still having issues.
I will address #1011 shortly.


upgraded apiman from 1.0.x to 1.1.x

Upgraded the console image to use apiman 1.1.0.CR1 to match the upgrade in fabric8's http gateway.

Can one of the admins verify this patch?

ok to test

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/272/
Test PASSed.

Thanks its merged now


Any possibility of an option to just download ROMs/CHDs and leave them as is?

This is amazing software.  I want to thank you for this.  I've never seen a more convenient way to just get and run games in MAME.
However, I want to be able to launch my games from a different frontend (specifically Playnite) and use the ROMs in alternative emulators, examples...

Running NAOMI, NAOMI2 and Atomiswave ROMs in Flycast
Hikaru ROMs in DEmul (at least until a different emulator adds support for Hikaru)
Lindbergh ROMs in Teknoparrot

Basically ROMs for other systems that run better in emulators other than MAME.
Would you consider adding an option to just download the ROMs or export the ROMs to their original .zip format?

Hello @geoneil I have added a .export command in 1.55.
It only does machine ROM at the moment, CHD & SL coming soon.
Cheers
Sam

Export is now fully operational in 1.56.
Cheers
Sam

Closing, import & export implemented.


Unexpected ##vso[task.setvariable]value behavior with Task Groups

There's a behavior difference between a build containing two PowerShell tasks and a build containing a Task Group with the two same PowerShell tasks.
A build variable, Pass1 is set to build variable.
Here are the two PowerShell inline scripts:
Script 1
Write-Host "Here's $(Pass1)"
Write-Host "##vso[task.setvariable variable=Pass1;]HelloWorld"

Script 2 (Arguments: $(Pass1))
Write-Host "here's args: $args"
Write-Host "here's $(Pass1)"
Write-Host "here's $Env:Pass1"

Expected output for a build with two tasks (debug is true):
2016-11-18T15:43:08.9556405Z Here's build variable
2016-11-18T15:43:08.9836430Z ##[debug]Processed: ##vso[task.setvariable variable=Pass1;]HelloWorld

and
2016-11-18T15:43:09.8146428Z here's args: HelloWorld
2016-11-18T15:43:09.8486420Z here's HelloWorld
2016-11-18T15:43:09.8486420Z here's HelloWorld

Unexpected output for a build with a task group (debug = false):
2016-11-18T15:44:24.1943543Z Here's build variable

and
2016-11-18T15:44:25.2743560Z here's args: build variable
2016-11-18T15:44:25.2753553Z here's build variable
2016-11-18T15:44:25.2793550Z here's HelloWorld

Unexpected output for a build with a task group (debug = true):
Here's build variable
Processed: ##vso[task.setvariable variable=Pass1;]HelloWorld

and
here's args: build variable
here's build variable
here's HelloWorld

This was discovered when I upgraded my build to use Task Groups.

Temporary workaround: creating build variables with the same name as the task group parameters and passing the build variable value to the parameter using the $(variableName) syntax resolved the issue.

@GeorgeTsiokos When you create a task group containing these two tasks, it will show Pass1 as a input to the task group. Are you setting that build variable ?
The inputs to the task group are resolved before the build/release is created, so if you are setting Pass1 to build variable and create a build, wherever you have used $(Pass1) inside your task group will be replaced with build variable. Then when the tasks run, the variable Pass1 is set and since $env:Pass1 is the only one which isn't replaced, it echoes out the right value.

Earlier thread on the same thing.
https://github.com/Microsoft/vsts-tasks/issues/2439

@harshil93 yes. I believe the variable substitutions prior to the execution of the steps, especially where ##vso[task.setvariable is called, is causing the usage confusion.

Wrote a blog post explaining this.
https://blogs.msdn.microsoft.com/harshillodhi/2016/11/29/vststfs-understanding-task-groups-and-its-various-use-cases-with-setvariable-logging-command/

That definitely clarifies whats going on, thank you!

Sorry, solution is not good "to set a variable inside Task Group and use it inside Task Group" !
It's a workaround, not a good solution ...
•This is hacky scenario since any variable used inside of Task Group is exposed as an input which has to be filled with some value. Fill the input with the same variable name as the input7
•You may not need to define this variable in your Variables section, you will just need to make sure that you are setting it in one of your tasks in Task Group before consuming it in the same Task Group.
•When the Task Group is expanded $(filePath) in tasks will be replaced by $(filePath) and you will be setting filePath in one of your tasks before consuming it.

Wrote a blog post explaining this.
https://blogs.msdn.microsoft.com/harshillodhi/2016/11/29/vststfs-understanding-task-groups-and-its-various-use-cases-with-setvariable-logging-command/

Updated link: https://mitra.computa.asia/articles/msdn-vststfs-understanding-task-groups-and-its-various-use-cases-setvariable-logging

Is there any hope for a good solution to the "set variable inside Task Group" problem? The solution presented in the article does not seem to work anymore with DevOps Server (the tasks do not resolve the replaced $(foo) for some reason).

Oh my God! You've saved my ass:))) I have spent four days looking how to set pipeline variable in powershell wondering why "##vso[task.setvariable variable...." doesn't work!
THANK YOU SO MUCH!
Great workaround solution!!!

@Arithmomaniac Do you have an updated link? @kirillkanyukov can you share your solution?


Some metadata validation is missing from flit_core?

First of all, thanks a ton for flit! We are very, very happy with it on pip-audit 🙂
Someone raised an issue with us recently (https://github.com/trailofbits/pip-audit/pull/353), noting a discrepancy between these two commands:
python -m flit build

# using flit_core's build backend internally
python -m build

In particular, the former (correctly) warns about invalid classifiers, while the latter doesn't:
$ python -m flit build --format wheel
Fetching list of valid trove classifiers                                   I-flit.validate
Unrecognised classifier: 'Programming Language :: Python :: 3 :: 3.10'     E-flit.validate
Unrecognised classifier: 'Programming Language :: Python :: 3 :: 3.7'      E-flit.validate
Unrecognised classifier: 'Programming Language :: Python :: 3 :: 3.8'      E-flit.validate
Unrecognised classifier: 'Programming Language :: Python :: 3 :: 3.9'      E-flit.validate
Config error: Invalid config values (see log)

and:
$ python -m build
* Creating venv isolated environment...
* Installing packages in isolated environment... (flit_core >=3.2,<4)
* Getting dependencies for sdist...
* Building sdist...
* Building wheel from sdist
* Creating venv isolated environment...
* Installing packages in isolated environment... (flit_core >=3.2,<4)
* Getting dependencies for wheel...
* Building wheel...
Successfully built pip_audit-2.4.3.tar.gz and pip_audit-2.4.3-py3-none-any.whl

In particular, this seems to be happening since flit.validate is not part of flit_core.
Is this intended behavior and, if so, is it documented anywhere? It makes sense to me that the flit package would contain more functionality than flit_core, but the discrepancy in validation was slightly surprising as an end user 🙂

Thanks! It's sort of intended behaviour - the net result obviously isn't desirable, but the pieces that go into it are deliberate.
My expectation when I designed this was that package producers - the people who choose to use Flit to publish their package - would use the flit command line directly, and package consumers would use tools like pip that may call Flit as a backend interface (using the APIs from PEP 517). So the command-line interface does a bunch of checks, aiming to ensure that your package is A-OK before you publish it, whereas the backend API (which python -m build calls) assumes that you want to use a package, and therefore if it can build a wheel, it will.
build (the project providing python -m build) also hides the backend's (Flit's) output unless it fails. So we can't warn and carry on with building: the only way to tell you something is wrong is to error out completely. And the backend doesn't know who is using it or what they want to do. :thinking:

Thanks @takluyver!
To rephrase from the perspective of a package repository maintainer:

The up-front validation that flit performs is mainly intended for upstream project maintainers so they have a chance to learn about issues before they proceed to cut a release.

As a maintainer of a (binary) package repository, I start with an already-released upstream source package in hand, and my goal is to just get the thing to build.
In that context, I’m basically in the role of a package consumer, because I roughly have the same goal as a regular user who would run pip install.

So my take-away for future work as a package repository maintainer is:
I’m going to accept the (already-released) upstream source package as is, and just run build instead of invoking flit directly.
Makes sense?

As a maintainer of a (binary) package repository, I start with an already-released upstream source package in hand, and my goal is to just get the thing to build. In that context, I’m basically in the role of a package consumer...

That sounds right. It depends a bit on the nature of your repository, but if there are problems with the package, you probably can't fix them as easily as the package authors (even if you have a patching mechanism, the downstream packaging ecosystems I've seen generally prefer not to use this unless necessary).

I’m going to accept the (already-released) upstream source package as is, and just run build instead of invoking flit directly.
Makes sense?

Makes sense. It also has the advantage that the same build command should work for any Python package, regardless of whether they use Flit, Setuptools, Poetry, Hatch, or other packaging tools.
Depending on how tightly your repository likes to control things, this can be a big simplification or a small one. python -m build will make a temporary build environment by default and install build dependencies into it. If your repository is relaxed enough to allow that, then you can build and install most Python packages with a simple formula. If your repository is stricter, then you'll need to work out which of your own packages build dependencies live in, and run build with the --no-isolation option to stop it using a temporary build environment.


Added the logic to refresh histogram metrics as part of diagnostics thread

Changes look mostly good. We discussed offline about controlling the 'reset' thorough a sysvar on the plugin side. Could you merge all your commits?

Created new pull request https://github.com/anirbanr-fb/kuduraft/pull/105 that merge all the commits.


Because flutter_tools depends on test any which doesn't exist (could not find package test at http://storage.flutter-io.cn), version solving failed. pub get failed (server unavailable) -- attempting retry 1 in 1 second... Running "flutter pub get" in flutter_tools...

C:\Users\mccar>flutter doctor
Flutter assets will be downloaded from http://storage.flutter-io.cn. Make sure you trust this source!
Because flutter_tools depends on test any which doesn't exist (could not find package test at http://storage.flutter-io.cn), version solving failed.
pub get failed (server unavailable) -- attempting retry 1 in 1 second...
Running "flutter pub get" in flutter_tools...

Hi @hoodyicon, Thanks for filing the issue. It looks like you this issue is caused due to the Chinese firewall. could you please try the solution mentioned here https://github.com/flutter/flutter/issues/20809#issuecomment-414550239


Limit exception. Found string instead of an integer

Mango DB: v3.3.1
Laravel 5.5
Other packages: Datatables
I'm trying to use the query builder with datatables that makes the query based on params and etc, but Mango freaks out with the limit feature. It throws the error:
MongoDB\Exception\InvalidArgumentException: Expected "limit" option to have type "integer" but found "string" in I:\xampp\htdocs\ngis\vendor\mongodb\mongodb\src\Exception\InvalidArgumentException.php:32
Stack trace:
#0 I:\xampp\htdocs\ngis\vendor\mongodb\mongodb\src\Operation\Find.php(183):

Which I know if you type cast, it's fine, but is there a reason why Mongo isn't able to type cast? I couldn't figure out where Datatables is trying to build the query. It's similar to this issue: https://github.com/yajra/laravel-datatables/issues/1442
THANKS!

You have to use "intval" both in take && skip
$trans = DB::collection("collection_name")
         ->take( intval($limit) )
         ->skip( intval($skip) )->get();

You have to use "intval" in both function of take && skip
$trans = DB::collection("collection_name")
         ->take( intval($limit) )
         ->skip( intval($skip) )->get();

I have a similar problem but after print this: ErrorException: Illegal offset type in /var/www/html/vendor/jenssegers/mongodb/src/Jenssegers/Mongodb/Query/Builder.php:383


Move Types dependencies to devDependencies

This is causing Types conflicts as @types/fs-extra is including @types/node as well (see https://github.com/EddyVerbruggen/nativescript-plugin-firebase/issues/332 for example)

@kevinbeaty @dcharbonnier I'm not familiar with types/TypeScript.

@RyanZim this is correct, should be merged IMO. Thx @SBD580

@dcharbonnier Is this semver-patch?
@kevinbeaty Can you merge and publish a new version?

Published 2.0.1
Thanks!

Thanks @kevinbeaty and @SBD580


is it possible ?

Hi,
is it possible to get this fork run server _ client like the old cigar ?
thank you

Do you still have the IOS edition of cigar??

yes i'm just updating it due to copyright issues that's all

Could I please have the source code if it is open source?? I would appreciate it much

sorry at this time it's not possible add me in skype matrix.real

Yes, you can use this server with Cigar.

@Megabyte918
i mean is it possible to run it server + client like old cigar ?
because as i can see there is no client side here

@matrixreal You have to run the server, then connect to it through a client. As you said before, there is no client side here. So I cannot help you with client-sided issues.


Add additional debug information for intermittently failing ContainerProxyTests

Add additional debug information for intermittently failing ContainerProxyTests
The ContainerProxyTests are failing intermittently in the pipeline. All of these intermittent test failures have either a wrong initialisation-time or it is missing completely.
Our theory currently is, that the Buffer in acker is called in the wrong order, as these calls are made asynchronously. If that theory is correct, this can be proven with this additional output.
Unfortunatly I was not able to reproduce the error locally.
Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[ ] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[x] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[x] Bug fix (generally a non-breaking change which closes an issue).
[ ] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4278 into master will increase coverage by 21.17%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #4278       +/-   ##
===========================================
+ Coverage   59.36%   80.54%   +21.17%
===========================================
  Files         162      162
  Lines        7585     7585
  Branches      500      500
===========================================
+ Hits         4503     6109     +1606
+ Misses       3082     1476     -1606

Impacted Files
Coverage Δ

...pache/openwhisk/core/invoker/InvokerReactive.scala
82.24% <0%> (+0.93%)
:arrow_up:

.../org/apache/openwhisk/core/entity/EntityPath.scala
100% <0%> (+1.88%)
:arrow_up:

...a/org/apache/openwhisk/core/entity/Parameter.scala
95.45% <0%> (+2.27%)
:arrow_up:

.../org/apache/openwhisk/http/PoolingRestClient.scala
90% <0%> (+3.33%)
:arrow_up:

...hisk/core/controller/actions/SequenceActions.scala
92.1% <0%> (+3.5%)
:arrow_up:

...pache/openwhisk/core/containerpool/Container.scala
88.6% <0%> (+3.79%)
:arrow_up:

...abase/cosmosdb/CosmosDBArtifactStoreProvider.scala
4% <0%> (+4%)
:arrow_up:

...tainerpool/docker/DockerClientWithFileAccess.scala
96% <0%> (+4%)
:arrow_up:

...rg/apache/openwhisk/core/entity/ExecManifest.scala
97.87% <0%> (+4.25%)
:arrow_up:

...he/openwhisk/core/database/CouchDbRestClient.scala
87.23% <0%> (+4.25%)
:arrow_up:

... and 92 more

Continue to review full report at Codecov.

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


CASMINST-4663

Description
Adding process for utilizing an rbd device for a landing space for the csm-release tarball
Checklist Before Merging

[x ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[x ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x ] My commits or Pull-Request Title contain my JIRA information, or I don't have a JIRA.

backport release/1.2

I committed my linting suggestions that were not content changes. I had one suggested change that will require Craig's input, as I wasn't 100% sure if my rewording captured what he was trying to say.

backport release/1.2

You have to use "/backport" to do this.
But I'd also recommend not doing it before all of the suggested changes are done, otherwise you have to go and make sure they get applied to the backport too (rather than just having them all included when you backport)

/backport release/1.2

/backport release/1.2

You should probably add in the other links to this procedure before doing the backport, whether you do it manually or automatically.

I'll manually create the 1.2 backport PR

1.2 backport PR is here: https://github.com/Cray-HPE/docs-csm/pull/1835

@mharding-hpe these commits should have been squashed before merging.  This PR has 37 commits, which really adds a lot of useless cruft to the git history, and makes it much harder for anyone looking at the git history to really understand the changes.


Support accessing group_member_generator yielded member_props by strings

Suggestion made by @taldcroft

@taldcroft - do you know of a way I can support this without converting each yielded sequence into a structured array or table? Doing that will result in a significant performance hit.

@taldcroft - I see your point here, but I think there is now enough usage of this generator that I'm going to opt for keeping the API stable rather than having a moderately more aesthetically appealing calling signature.


SIM800L TCP communication support

Is there any current implementation for supporting TCP communication using SIM800 (GPRS modems)?
If not, what are the files/fiunctions/configurations that could be wrapped to be able to use my current implementation?
Thank you!

I just had a quick look at the SIM800 AT commands. That allows to open a TCP connection, meaning that the device already has a built-in TCP/IP{ stack.
Do you know if it has a possibility to pass a raw Ethernet II packets?

@htibosch I don't know much about the SIM800 functionality, I use a driver written by a colleague, which use the TCP layer of the modem.

See my earlier comment on #22 - basically you need to bring your own aws_secure_sockets.c functionality, and it should be a fairly clean swap.

@nickfaughey  yeah, I see that comment and will start implement this layer.
Thanks a lot!

@nickfaughey  another question before starting the implementation, could we use different broker other than AWS IOT? because we are developing our back-end environment.

I don't see why not, but that's independent of the communication layer. I'm not sure how much of the broker logic in amazon-freertos is AWS-flavored, but theoretically it should all be swappable.


chore: grammar & spelling

Minor grammatical fixes, nothing big
P.S. I will try to make more significant contribs, but these basic fixes are usually good to get a bit more acquainted with the code structure

@dtolnay there is one more -- https://github.com/dtolnay/cxx/compare/master...nyurik:cxx:grammar?expand=1

Oh wow, I was not even aware that panick is a valid spelling,  nvm


Sometimes stopping resource hangs

Happens on empty resource as well.
Repro: start and ctrl + C the server few times.

That has been a core bug for ages


Update migrate-from-express.md

Information

Type
Breaking change

Feature/Fix/Doc/Chore
Yes/No

Description
A few sentences describing the overall goals of the pull request's commits.
Usage example
Example to use your feature and to improve the documentation after merging your PR:
import {} from "@tsed/common";

Todos

[ ] Tests
[ ] Coverage
[ ] Example
[ ] Documentation

:tada: This PR is included in version 6.26.0 :tada:
The release is available on:

v6.26.0
GitHub release

Your semantic-release bot :package::rocket:


pass string value to piedata

Hello,
I am having problem in calling values from a label to piedata for chart.
var PieData = [document.getElementById("LblValue").innerHTML]
value of LblValue is - { value: 2, color: '#F074CB', highlight: '#F074CB', label: 'DIRECTOR - Male' }, { value: 1, color: '#5174B5', highlight: '#5174B5', label: 'DIRECTOR - Female' }
when I put value itself in piedata, it works great.
var PieData = [{ value: 2, color: '#F074CB', highlight: '#F074CB', label: 'DIRECTOR - Male' }, { value: 1, color: '#5174B5', highlight: '#5174B5', label: 'DIRECTOR - Female' } ]
I don't know what I am missing.

Hi,
You can post your question directly on the Chartjs repo https://github.com/chartjs/Chart.js/issues and read the doc at http://www.chartjs.org/
As your question it is not related to the template itsefl.
You can also download the latest Chartjs version as the one that come with the template may be old.


Manually add point to drawing instance(ol.interaction.Draw polygon)

I'm working on a webpage based on android browser, and I want to add a point who's coordinate is from html5 Geolocation API to a polygon drawing by the ol.interaction.Draw instance(the polygon is unfinished).
First I want to get the draw interaction's sketchCoords_(private) ,and push a new coordinate. But I can't get sketchCoords_,I remember previous version allow that, but now I use the latest version, and failed!
Then I noticed draw interaction has a method 'handleEvent(mapBrowserEvent), so I want to initialize a ol.MapBrowserEvent instance with my custom coordinate, but an error said that: 'ol.MapBrowserEvent()' is not a constructor.
Well, what I want to ask is:

Is there any possible I can get the drawing sketch?
How can I initialize a custom ol.MapBrowserEvent?
Or
My solutions are not right, any other ideas is welcomed. Can someone help me out? Thanks in advance!

You have several options, including

Create a polygon geometry without Draw interaction, and add point to it using a point Draw interaction for manual input and Geolocation for GPS input.
Use a polygon Draw interaction with a geometryFunction, which gives you access to the sketch coordinates.

Thank you @ahocevar !  It works!


Webpage down

The url is returning {"error":"Server error."}

Thanks for the heads-up. Sorry about that! Fixed.


ImportError: DLL load failed: The specified procedure could not be found.

Hi. I am trying to use EUReCA in PyCharm with the Conda Eureca mod environment. However, the code is throwing an error message as given in the title. I am attaching the snapshot of the entire output here for your reference. Requesting to check whether this can be resolved

Hi @OmprakashRR
Seems something linked pvlib.
What version of pvlib is installed in your env? 0.7.2 should be the one. Eventually, try to uninstall it and install it again.
Are you running EUReCA on windows?

Thanks @enrico-prataviera. 0.7.2 is installed and yes, I am running it on windows


:

Problem:
Command 'vimppm' already exists, Not replacing existing command. Use ! to override.

Fix:
Change:
commands.addUserCommand(..., ..., ..., ...)

to
commands.addUserCommand(..., ..., ..., ..., true)
`

I guess that your vimperator load vimppm.js twice.
Normally, vimppm.js is not only once at startup of vimperator.


fix(Cluster): show loader if capabilities not loaded

CI Results
Test Status: ✅ PASSED
📊 Full Report

Total
Passed
Failed
Flaky
Skipped

222
222
0
0
0

😟 No changes in tests. 😕
Bundle Size: ✅
Current: 66.16 MB | Main: 66.16 MB
Diff: +1.24 KB (0.00%)
✅ Bundle size unchanged.

ℹ CI Information

Test recordings for failed tests are available in the full report.
Bundle size is measured for the entire 'dist' directory.
📊 indicates links to detailed reports.
🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

Maybe it would be better to merge this condition to get user LoaderWrapper


fix(package_info_plus): restore backward compatibility

Description
This PR restores backward compatibility of package_info_plus.
package_info_plus violates semantic versioning spec in 1.0.4 where #340 added a required parameter to PackageInfo's public constructor, this is a breaking change to lines declaring PackageInfo as a non-nullable type with dummy values.
The change removes the required keyword and sets a default value(empty string) to allow the variable to be non-nullable.
Related Issues
Closes #392
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (flutter analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[x] No, this is not a breaking change.

@mhadaily
Thanks for the review.
The label is package_info_plus :)


send nodeId when capturing screenshot of an element

too many conflicts- will try to clean and raise a new one.


Restrict duplicate deps warning only to published packages

Fixes #10752

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

:pushpin: Commit 08d10184aa19ea902f809a81bd29555b74709e39 has been approved by epage

:hourglass: Testing commit 08d10184aa19ea902f809a81bd29555b74709e39 with merge c9d8c28cba959c347271eaf31c9abfbb74c690bd...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing c9d8c28cba959c347271eaf31c9abfbb74c690bd to master...


Source specs from OpenAPI repo

#407  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @RajatPawar and the rest of your teammates on  Graphite


Fe clean up

a lot of reorganisation of sass for reuse

Looks good, though CI is failing and it may be worth fixing up your commits while you're at it?


Streamline UI

We should:

[x] Remove the type dropdown in Package search. Instead we should be able to search by purl prefixes
[x] Enable searching by packages using all purl fields and be case insensitive
[ ] Streamline the UI code before the release

Closed with the merge #894


OpenCL error -11 'failed to build program' in xgemm.cc, on Windows

Using NVIDIA Corporation , OpenCL platform: NVIDIA CUDA
Using OpenCL device: GRID K520
initializing clblas
OpenCL error -11 on line 252 of ..\..\..\..\..\clMathLibraries\clBLAS\src\librar
y\blas\xgemm.cc
Assertion failed: false, file ..\..\..\..\..\clMathLibraries\clBLAS\src\library\
blas\xgemm.cc, line 252

Line 252 is basically this:
#ifdef AUTOGEMM_PRINT_DEBUG
      printf("makeGemmKernel: Creating program from source\n");
#endif
      clProgram = clCreateProgramWithSource(
        clContext,
        1, &kernelSource,
        NULL, &err );
      CL_CHECK(err)
      err = clBuildProgram(
        clProgram,
        1, &clDevice,
        sourceBuildOptions, NULL, NULL );
      CL_CHECK(err)
    }

Associated test scenario:
TEST(testClBlas, basic) {
    EasyCL *cl = EasyCL::createForFirstGpuOtherwiseCpu();

    float A[] = {1, 3,
                 2, 7,
                 9, 5};
    float B[] = {3,
                 -1};

    float C[3];
    ClBlasInstance clblasInstance;
    CLWrapper *AWrap = cl->wrap(6, A);
    CLWrapper *BWrap = cl->wrap(2, B);
    CLWrapper *CWrap = cl->wrap(3, C);
    AWrap->copyToDevice();
    BWrap->copyToDevice();
    CWrap->createOnDevice();
    ClBlasHelper::Gemm(
        cl,
        clblasRowMajor,
        clblasNoTrans, clblasNoTrans,
        3, 2, 1,
        1,
        AWrap, 0,
        BWrap, 0,
        0,
        CWrap, 0
    );
    cl->finish();
    CWrap->copyToHost();
    EXPECT_EQ(0, C[0]);
    EXPECT_EQ(-1, C[1]);
    EXPECT_EQ(22, C[2]);

    cl->finish();

    delete CWrap;
    delete BWrap;
    delete AWrap;

    cl->finish();

    delete cl;
//    clblasTeardown();
}

Can you test with the nvidia driver on linux and another vendor's driver on windows?  The -11 error codes maps to  CL_BUILD_PROGRAM_FAILURE.  I've been running into this on mesa, but it is probably unrelated.

Also, it's be useful to know if your driver relies on a LLVM backend for it's openCL compilation or something custom.  I'm not sure if that information is available.

On linux, using nvidia 352.55 driver, works ok.
For Windows, I'm using Amazon EC2, on a g2 box.  Tricky to use a different vendor's driver I would think :-)


About how to load subgame of Libratus

Hi, I am currently working on experiments for my algorithm and intend to test it on the subgame of Libratus mentioned in [^1]. I have noticed that the universal_poker has some interfaces related to the subgame.txt definition, but I am having trouble figuring out how to load it using the pyspiel module.  Could you provide some instructions on how to use the pyspiel module, perhaps something like pyspiel.load_game("universal_poker(...)")?
I apologize for not thoroughly reading the entire C++ source code; your assistance would be greatly appreciated!
[^1]: Solving Imperfect-Information Games via Discounted Regret Minimization

Hi @Atan03,
Ok so there are some examples of how to use those subgames here: https://github.com/google-deepmind/open_spiel/blob/d8a3babc2703d58dfd7a86974c91a28b9593b90a/open_spiel/games/universal_poker/universal_poker_test.cc#L662
My guess is this was never exposed to Python.. yeah doesn't look like it. You'd just need to expose those functions under python/pybind11. There are docs explaining how to do that here: https://github.com/google-deepmind/open_spiel/blob/master/docs/developer_guide.md#adding-game-specific-functionality
I've not used this much and hasn't been used in a while so maybe @michalsustr can follow up if you have more specific questions about that functionality.

Your response has been really helpful! This is something I haven't tried before, and I will follow your guidance to give it a shot. I'll provide feedback here once I have results.

Closing due to inactivity, but feel free to use this thread if you have any more questions.


Openseeds logo

@bebatut I closed the previous pr and made this.

Great
Few comments:

Can you rename the folder openseeds without logo?
Can you rename READme.md to README.md?

Thanks a lot

Done. @bebatut


XCode 5.NEXT / Linker Errors

Hello. I'm not sure this is the best place to raise this issue. I've installed Expecta through Cocoapods — and when I run a unit test containing expecta statements, I get several linker errors. I can whack-a-mole quiet them by manually adding source code, but that doesn't seem right. About 1/2 a day into trying to figure this out, so I thought I'd ask rather than pull my own teeth.
I'm on XCode Version 5.1 (5B90f)
Any ideas? Also happy to take this on a different wire: julian at nearfuturelaboratory dot com
My Podfile contains:
platform :ios, '7.0'
#pod 'AFNetworking', '~> 1.3.3'
pod 'SSToolkit'
pod 'ConciseKit'
pod 'SBJson'
pod 'RestKit'
pod 'SSKeychain'
pod 'SAMCache'
pod 'iCarousel'
pod 'GMGridView'
pod 'NSLogger'
pod 'MGBoxKit'
target :humansTests, :exclusive => true do
        pod 'OCHamcrest', '~> 3.0.1'
         pod 'OCMockito', '~> 1.0'
  pod 'Specta',      '~> 0.2.1'
   pod 'Expecta',     '~> 0.2.3'   # expecta matchers
   pod 'OCMock',      '~> 2.2.1'   # OCMock
   pod 'LRMocky',     '~> 0.9.1'   # LRMocky
end
post_install do |installer_representation|
    installer_representation.project.targets.each do |target|
        if target.name == 'Pods-AFNetworking'
            target.build_configurations.each do |config|
                config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= ['$(inherited)']
                config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] << '_AFNETWORKING_ALLOW_INVALID_SSL_CERTIFICATES_=1'
            end
        end
    end
end

The linker errors look like this, for two statements:
- (void)test_foogetStatusCountForHuman
{
    [Expecta setAsynchronousTestTimeout:10];
    expect(@"foo").to.equal(@"foo");
}

Undefined symbols for architecture i386:
  "_OBJC_CLASS_$_Expecta", referenced from:
      objc-class-ref in HuUserHandlerTest.o
  "__EXPObjectify", referenced from:
      -[HuUserHandlerTest test_foogetStatusCountForHuman] in HuUserHandlerTest.o
      ___51-[HuUserHandlerTest test_foogetStatusCountForHuman]_block_invoke in HuUserHandlerTest.o
  "__EXP_expect", referenced from:
      -[HuUserHandlerTest test_foogetStatusCountForHuman] in HuUserHandlerTest.o
ld: symbol(s) not found for architecture i386
clang: error: linker command failed with exit code 1 (use -v to see invocation)

OK, so I've tried replicating this in a demo project and failed. http://cl.ly/3b2S052y3529/demo.zip
FWIW, it's a pretty crazy Podfile having 3 mocking libraries + 2 matcher libs in there.
Can anyone offer a re-producable version? Happy to re-open if not.


unable to access koken

I have it deployed on a server and serve it using a subdomain (nginx) on koken.domain.com
I get an api error on accessing the normal page "check loopback connections".
If I put in a manual url I get redirected to localhost which of course doesn't work.
I'm not sure what the issue is...

I was able to fix this problem by modifying user_setup.php and adding following line:
define('LOOPBACK_HOST_HEADER', true);

For new build change file deploy/php/user_setup.php
For already deployed app file is under storage/configuration/

When you start the container try adding the hostname option -h koken.domain.com

I have the same issue and tried adding the hostname to the docker run command. This did not solve the problem. Any more advice @MikeCostello ?


[video_player] plugin should expose a Stream of change events instead of callbacks

From a comment on flutter/plugins#1531
Exposing a Stream of notification events on the VideoPlayerController would support StreamBuilder based widget hierarchies and that might make it easier for the developer to register and unregister safely.
This might be a breaking change to the plugin API.

I think if we did anything like that it should probably just be a Listenable. Streams have all kinds of problems.
Is this still something we think we should do?

It doesn't seem like a priority to me, based on low activity on this issue.

I'm assuming this is resolved since the controller is a value notifier now.


PowerShell Pager Feature roadmap plan

Motivation
Oftentimes, the output of a command is too large to fit on a single screen.  Customers need the ability to scroll forward and back through the content for readability. The PowerShell Pager is a virtual terminal pager utility implementation similar to linux equivalents more and less.  The PowerShell pager is not intended to replace less, instead be an enhancement to more.
Scenerios
The PowerShell pager should support the ability to display cmdlet help similar to the parameter -ShowWindow
Help Start-Process -page

The PowerShell pager should have a cmdlet Format-Page that supports pipeline input of content.
Get-Service | Where-Object {$_.status -eq “Stopped”} | Format-Page

Version 1.0 pager features
Planned features of the initial 1.0 release of the PowerShell Pager to support page navigation.
Feature | KeyBinding

Top of File | HOME
Bottom of file | END
Next page | SPACEBAR
Previous page | b
Exit | q
Down one line | UP ARROW
Up one line | DOWN ARROW

Planned features of Version.Next release
To support the customer in identifying specific data, search with highlight will be added to the pager.  Within the paging utility, a user will be able to specify a  argument to be highlighted.
Feature | KeyBinding

Search forward | /
Search backward | \
Next forward match | f
Next backward match | b

Alternative Proposals

The addition of Half page navigation similar to less.  Keybindings move the content 1/2 page forward or backward.

supporting mouse VT mouse scrolling would be a good one too

Also, please don't forget to support page up and page down keys to scroll up / down one screen at a time.
With only the arrow keys currently supported and the constant fullscreen refresh (issue #20), scrolling long documents (integrated help) is painful at the moment.

I'm thinking in implementing it myself, it doesn't seem too hard.  but +1 for this feature. Page Up and Page Down are essential features !
I was thinking in writing a pager myself to page the history when I press F7 in PSReadLine.
But I'm ironically using less.exe from Pscx for that now for that.


fix: upgrading terraform version to 1.3.0

Description

Optional attributes are removed in order to support latest terraform version i.e.
experiments = [module_variable_optional_attrs]
Terraform version is updated i.e.required_version = ">= 1.3.0"

Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[x] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[x] If relevant, a test for the change has been added or updated as part of this PR.
[ ] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

Covered by https://github.com/terraform-ibm-modules/terraform-ibm-landing-zone/pull/264


Export for Sonarqube

Hello,
Is it possible to create an export format for sonarqube?
thank you

@axil76 I know nothing about SonarQube, but this Test Coverage & Execution page seems to indicate that Cobertura XML files are accepted?  The "coverage xml" command writes a .xml file in Cobertura format.
Can you try it and report back?

Hello,
I have the impression that the output XML format is not the same
https://docs3.sonarqube.org/latest/analysis/generic-test/
below the error message
13:50:17.979 ERROR: Error during SonarScanner execution
Error during parsing of the generic coverage report '/tests/coverage.xml'. Look at SonarQube documentation to know the expected XML format.
Caused by: java.lang.IllegalStateException: Unknown report version: 5.3. This parser only handles version 1.

Hmm, coverage uses the version attribute as the version of coverage.py.  Looks like SonarQube uses it as the version of the format, though I thought Cobertura was up to version 3 or 4....
Do you mind editing the xml file to set the version to 1, and see what SonarQube does with it?

Currently facing this issue. Can someone help me pls?

this answer is still valid today (ref), I believe this issue is fine to be closed.

this answer is still valid today (SQ's doc), I believe this issue is fine to be closed.

@mattwang44 that answer features incorrect use of pytest args. It suggests that --junitxml generates coverage, but that's not true — it generates test report data, which is a different thing that also happens to use XML-based format.


Wip/mehrad/updates

"Fixing requote.js bugs"
Fixing a couple of bugs when requoting consecutive tokens or parameters with lexical overlap.
It will now consume almost anything as long as the sentence and the program have matching parameters!

What's the bug in requote.js specifically? Do you have a test case that fails with master? I've run requote.js extensively, and also fixed it a bit recently (hence the merge conflict). I did not need the complete rewrite you did.

Belive me you do! There are a lot of cases when the script fails. Just run it once on the restaurants trains set: s3://almond-research/mehrad/dataset/multilanguage/restaurants/final1-original-en/train.tsv
Example:
RPS5000099412-1	what are restaurant with their summary containing " and may not " but not " and " and " italian " in the served cuisine ?	now => ( @org.schema.Restaurant.Restaurant ) filter not param:description:String =~ " and " and param:description:String =~ " and may not " and param:servesCuisine:String =~ " italian " => notify

But yeah we need more comprehensive unit tests.


Migrate WinRTComponent to C#

It looks like this is supported and straightforward, so it would make it much easier to define WinRT API's to interop with:
https://learn.microsoft.com/en-us/windows/apps/develop/platform/csharp-winrt/create-windows-runtime-component-cswinrt

Better stick to C++ since that will be able to represent every WinRT features and do low-level interop better.


tape.jacobian with chain rule cannot compute the gradient correctly

Click to expand!
Issue Type
Bug
Source
binary
Tensorflow Version
tf2.7
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 21.10
Mobile device
No response
Python version
3.9.7
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
11.3.1/<IP_ADDRESS>
GPU model and memory
No response
Current Behaviour?
I want to use the jacobian matrix and chain-rule to compute the gradient. When the composed functions are basic arithmetics, the gradient is right. But when the composed function is a convolution, the gradient  calculated by the chain-rule is not consistent with the one outputed by 'tape.gradient'.

Standalone code to reproduce the issue
import tensorflow as tf

layer = tf.keras.layers.Conv2D(1, 3, activation='relu', padding="same")
x = tf.random.normal([2,7, 7,1])
xx = tf.random.normal([2,7, 7,1])

with tf.GradientTape(persistent=True) as tape1:
    tape1.watch(x)
    y1 = layer(x)
    y2 = layer(y1)
    tape1.watch(y2)
    loss=tf.keras.losses.MeanSquaredError()(y2,xx)

j1=tape1.jacobian(y2,x)
[g_x,g_y2]=tape1.gradient(loss,[x,y2])
j1_vec=tf.reshape(j1,[x.shape[0]*x.shape[1]*x.shape[2]*x.shape[3],
                     y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3]])
g_y2_vec=tf.reshape(g_y2,[y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3],1])
G=tf.matmul(j1_vec,g_y2_vec)
G_x=tf.reshape(G,[x.shape[0],x.shape[1],x.shape[2],x.shape[3]])
e1=G_x-g_x
print('error value of e1:',tf.reduce_max(abs(e1)).numpy())

with tf.GradientTape(persistent=True) as tape2:
    tape2.watch(x)
    y1 = x**3+100
    y2 = tf.math.cos(y1)+10
    tape2.watch(y2)
    loss=tf.keras.losses.MeanSquaredError()(y2,xx)

j2=tape2.jacobian(y2,x)
[g_x,g_y2]=tape2.gradient(loss,[x,y2])
j2_vec=tf.reshape(j2,[x.shape[0]*x.shape[1]*x.shape[2]*x.shape[3],
                     y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3]])
g_y2_vec=tf.reshape(g_y2,[y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3],1])
G=tf.matmul(j2_vec,g_y2_vec)
G_x=tf.reshape(G,[x.shape[0],x.shape[1],x.shape[2],x.shape[3]])
e2=G_x-g_x
print('error value of e2:',tf.reduce_max(abs(e2)).numpy())

Relevant log output
error value of e1: 0.10533138
error value of e2: 2.3841858e-07

@sachinprasadhs  ,
I was able to reproduce the issue in v2.8, v2.9 and nightly.Please find the gist of it here.

Hi @wangwei-cmd  ,
The issue is from the wrong order of matmul in the chain rule. Please change these lines:
From
j1_vec=tf.reshape(j1,[x.shape[0]*x.shape[1]*x.shape[2]*x.shape[3],
                     y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3]])
g_y2_vec=tf.reshape(g_y2,[y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3],1])
G=tf.matmul(j1_vec,g_y2_vec)

To
# the Jacobian matrix is first ordered by f and then by x
j1_vec=tf.reshape(j1,[y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3],
                     x.shape[0]*x.shape[1]*x.shape[2]*x.shape[3]])

# let gradient to be a "row" vector
g_y2_vec=tf.reshape(g_y2,[1,y2.shape[0]*y2.shape[1]*y2.shape[2]*y2.shape[3]])

# apply the chain rule of  (dg / dy2)  *  (dy2 / dx) , not the other direction
G=tf.matmul(g_y2_vec,j1_vec)

In the other example, when the functions are only self-correlated, the Jacobian matrix is diagonal, so the wrong matrix multiplication order didn't affect the results.
Let me know if you have further questions. Thanks.


bundle solid-start/entry-client

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Refactoring (no functional changes, no api changes)
[x] Build related changes

What is the current behavior?
Solid-Start is distributed as Typescript, leading to issues like https://github.com/solidjs/solid-start/issues/1008 and https://github.com/solidjs/solid-start/issues/255 This is not recommended.
What is the new behavior?
This PR creates a new solid-start/entry-client "package", similar to web, store, h, etc. in SolidJS. (Really I just looked at SolidJS and was like ctrl-c ctrl-v.) I only did entry-client for now to solicit feedback - I imagine the other folders of start to be more or less the same type of change. If the maintainers approve, I'll do the other dirs too.
This PR introduces breaking changes. Users on "moduleResolution": "node", will need to make changes like
-      import mount from "solid-start/entry-client/mount";
+      import mount from "solid-start/entry-client/dist/mount";

I'd argue this is a good thing as it forces Solid (and users) to think about the API, buuuuut I'd get it if that means we need to wait for 0.4 before this PR gets any traction. Also, since I'll be moving nearly every file in /start (to a nested /src folder), it may introduce conflicts with other PRs/branches. IMO this is a reason to get this done sooner rather than later, but choosing the right time may be important. Sorry if this ruins your timing, but I'm tired of getting ts errors in my node_modules 😭

Truthfully.. we're sort of completely changing all of this in 0.4.. like completely changing the entry imports etc.. so I will take this under consideration but no action at the moment.

Coolbeans, more than happy to make PRs if/when you get to that point!


Bugs #22588 and #22591 fix

bug #22588: missing preset save to file after removing a preset.
bug #22591: shader failed to compile when shadow enabled because of incompatible assignation  of vec3 to a vec4.

Thanks for the contribution. For future PRs, please keep unrelated changes in their own PRs so that they can be reviewed separately (as such, create branches on your fork for PRs instead of using your master branch).
The two fixes seem good, but could you rebase the branch to remove the "Merge branch 'master'" unnecessary commit? You can do it with git rebase upstream master (provided you defined an upstream remote pointing to this repository). See PR workflow for more details.

You need to untangle this a little :)
Don't ever do your changes on your own master branch, now a lot of changes from other people are mangled into your PR.  Then you can rebase your branch whenever other peoples changes are merged into the system.
Its probably easiest if you join us on #godotengine-devel on IRC and get some help but in theory you could right now:

create a branch based on your current work just to save guard it,
do a git log to get the commit ID of your changes,
then restore master back to its original glory so it is properly aligned again,
create a new branch for your changes based on the restored master and do a checkout of that branch
cherry pick the changes you made into that new branch, commit and push that branch
raise a new PR with just those changes

Good luck!


UI Bug: IP assignment in deploying a VM

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

Main, 4.17, 4.17.2

SUMMARY

When you set a name for a VM in deployment tab, the custom IP address will be removed. For more info checkout the following gif.
https://user-images.githubusercontent.com/9499410/207056253-702ddb63-c1f5-485b-9cdf-b57fb298ec54.mov
STEPS TO REPRODUCE

1. Try to deploy a virtual machine.
2. Add a custom IP in the box.
3. Assign a name to the VM.
4. The IP is removed.
5. Input the IP again and deploy the VM
6. VM deployed with a different IP

EXPECTED RESULTS

Appropriate IP assigned to the VM

ACTUAL RESULTS

Cloudstack pick the first free IP in the range and assign it to the VM.

@soreana are you working on a fix for this?

@DaanHoogland Not actively, I'm not good at VueJS, I appreciate any help :)


jsonc/sort-key not worked as expected ?

Reproduce:
Move any key in package.json to random order
checkout https://github.com/davidNHK/busybox
yarn
./node_modules/.bin/eslint ./package.json

expected :
eslint report error and --fix can auto resort
actual:
no error
My config here:
https://github.com/davidNHK/busybox/blob/development/packages/eslint-config/preset/base.js#L84

More info
https://github.com/ota-meshi/eslint-plugin-jsonc/blob/master/lib/rules/sort-keys.ts#L297
I try add logging here, the parserServices always undefined

You need to change the presets you use.
-      extends: ['plugin:jsonc/prettier'],
+      extends: ['plugin:jsonc/base'],

@ota-meshi i tried what you suggested , but no luck

turn out i find it has conflict between my eslint config , thanks @ota-meshi for support
You did a great job to make our life more efficient !

for fork who hit exact problem to me .
Here is some situation on my eslint config
i have shared eslint config which aim to support react, typescript, js in one js config
and my ts eslint config look like that
const { overrides } = require('./base');

const jsConfig = overrides.find(override => override.files.includes('*.js'));

module.exports = {
  overrides: [
    {
      extends: [
        ...jsConfig.extends,
        'plugin:@typescript-eslint/recommended',
        'plugin:import/typescript',
        'plugin:typescript-sort-keys/recommended',
      ],
      files: ['*.ts', '*.tsx'],
      parser: '@typescript-eslint/parser',
      rules: {
       // ....some ts rule here
      },
    },
  ],
  parser: '@typescript-eslint/parser',
  plugins: ['@typescript-eslint/eslint-plugin', 'typescript-sort-keys'],
  settings: {
    'import/resolver': {
      typescript: {},
    },
  },
};

i have put parser in top level of my json.
and seem it cause conflict between json parser used here.
i have attempt add parser options on .json but no luck.
and turn out i change my ts eslint like that
const { overrides } = require('./base');

const jsConfig = overrides.find(override => override.files.includes('*.js'));

module.exports = {
  overrides: [
    {
      extends: [
        ...jsConfig.extends,
        'plugin:@typescript-eslint/recommended',
        'plugin:import/typescript',
        'plugin:typescript-sort-keys/recommended',
      ],
      parser: '@typescript-eslint/parser',
      plugins: ['@typescript-eslint/eslint-plugin', 'typescript-sort-keys'],
      files: ['*.ts', '*.tsx'],
      parser: '@typescript-eslint/parser',
      rules: {
       // ....some ts rule here
      },
    },
  ],
  settings: {
    'import/resolver': {
      typescript: {},
    },
  },
};


motoDEX Grant

Project Abstract

Please provide a brief description of your project here summarizing key points (1-2 paragraphs).
If your application is a follow-up to a previous grant, please mention which one in the first line of the abstract and include a link to previous pull requests if applicable.

For which grant level are you applying?

[ ] Level 1:  Up to $10,000
[x] Level 2:  Up to $50,000
[ ] Level 3:  Up to $250,000, OAK Network Council approval required

Application Checklist

[x] The application template has been copied, renamed ( project_name.md) and updated.
[x ] A Ethereum Mainnet wallet address for the payment of ERC20 USDC is provided inside the application.
[x ] The software delivered for this grant will be released under an open-source license specified in the application.
[x] The initial PR contains only one commit (squash and force-push if needed).
[x] The grant will only be announced once the first milestone has been accepted.

Hi @oleksiivinogradov, thank you for submitting the grant proposal. From the description provided, is it correct that your game is implemented with an EVM smart contract?

Hi @oleksiivinogradov, thank you for submitting the grant proposal. From the description provided, is it correct that your game is implemented with an EVM smart contract?

Hi, yes, we are. Core and NFT smart contracts

@oleksiivinogradov, got it. Unfortunately our chain does not currenlty support EVM smart contract, making it unfeasible to deploy the contracts for your solution at this time. We’ll definitely reach out to you once the EVM capability is added!

@oleksiivinogradov, got it. Unfortunately our chain does not currenlty support EVM smart contract, making it unfeasible to deploy the contracts for your solution at this time. We’ll definitely reach out to you once the EVM capability is added!

No worries, we can create non-evm verson too. We already done rust near, rust concordium, Ink pokadot e.t.c.


User address asked twice?

From @CaroRob:

Explore housing conditions
No heat / No lawsuit
Tenant's address showed twice

@CaroRob I could not recreate this issue. Do you recall what you answered on id: triage? Here are either of the options (note we plan to make the options the same regardless of selection)


Added cloud models and appropriate JSON fields to fix #252

Fixes bug where json fields would not be collected from the response for new cloud models.  Closes #252.
Issue arises from this block of code.  If class does not have the _json_field attribute set to True, then it's skipped.
https://github.com/nautobot/pynautobot/blob/84f28a53dd9d581a088f1d4115a7e9f7e76ad135/pynautobot/core/response.py#L253-L260

Any tests you can add that would have failed before the change (TDD)?

@joewesch - Added test and confirmed they failed prior to these changes.


Task not loaded when coming from another tab

Describe the bug
If opening a tab on the parameter tab https://dev.algorea.org/en/a/5948378700861088397;p=39530140456452546;a=0/parameters, then switching to the "content" tab makes an empty tab
Bug was introduced with the design change I think.

I found the problem:
private readonly itemInfo$ = new Subject<{ route: FullItemRoute, isTask: boolean|undefined }>();
I guess it need to be ReplaySubject, otherwise when we switch to content tab - there are no events and state.

Ok I'll check


Induced subgraphs

Description
# Assuming that Graph building is complete,

graph[node_1, node_2, node_3, hop_size: 2]
#=> Subgraph involving neighbors that are close to given set of nodes, within given hop_size

Relevant Issues : (optional)
Relevant PRs : (optional)
Type of issue :

User mode :

[ ] Installation
[ ] Feature request
[ ] Bug in existing feature

Developer mode :

[ ] Code quality
[ ] Tests
[ ] Documentation

No one seems to want to implement it


fix(cloudevents-server): fix the issue of the EventConsumer close calls

wrong added the defer close call, it will caused the consumer reading failure.
I tested the change manually, it's passed.
Signed-off-by: wuhuizuo<EMAIL_ADDRESS>

/review default


Adding missing error label to Package

Critical Changes
Changes
Issues Closed
New Metadata
Deleted Metadata

I'll merge this at a less busy time since it looks like the broken beta builds are already fixed.


tree treasures: typo correction, added the birds

bird up

birds probably warrant a second quote?
 How about you take care of it instead?


Clonegenesis

About the PR
A new experimental discipline technology, "Clonogenesis", has been added to the game. This tech costs 50,000 points and is a Tier 3 technology. Unlocking it provides access to everything needed for human cloning, including the medical scanner, cloning pod, and cloning console. The process of constructing the cloning pod has also been made slightly more challenging, now requiring 5 plasma and 5 glass.
Why / Balance
The cloning machine represents an interesting mechanic, and I believe it’s unfair to deny players the opportunity to interact with it. Currently, seeing a functioning cloning machine even once in 20 rounds is quite rare. It doesn’t seem right that the only ones who can bring the cloning machine to the station are the disposals. Adding this machine to the experimental discipline would significantly increase its popularity. Furthermore, a tier 3 technology priced at 50,000 points seems quite justified for a cloning machine.
Media

Requirements

[X] I have read and am following the Pull Request and Changelog Guidelines.
[X] I have added media to this PR or it does not require an ingame showcase.

Breaking changes
Changelog
:cl:

add: A new experimental discipline technology, "Clonogenesis", has been added to the game!
tweak: Now, in addition to everything else, you'll need 5 glass and 5 plasma to create the cloning pod!

this feels like it should be a civilian tech, since thats where all the other med stuff is, and means we might actually see that T3d at a reasonable timeframe. Although i do agree with Blackern that cloning should probably not be a tech at all

we will get back on square one : Security taking science hostage and threatening to gib each and every scientist if they don't research cloning

Cloning was made inaccessible through normal means because it just isn't particularly interesting mechanically. Adding a means to get cloning reliably, no matter how difficult, doesn't fix the flaws with cloning that are still extremely present.

okay hear me out, what if it was a T3 but not present in the research nodes? it is a hidden tech only available from the tech disk printing machine?

Cloning isn't an interesting mechanic, and this was done originally when cloning was removed from roundstart medical.

Round removal is apart of the game, rotting corpses shouldn't be revived (at all, in my opinion, but oppo is a whole different conversation.) so easily.
Science will get harassed for not researching cloning because people are just like that.
Cloning isn't engaging, you just scan the guy, duplicate 'em, and now they're alive with little to no effort on behalf of medical.
This was already removed from research in the past, it wasn't engaging then, it won't be now.

1: I would agree if it werent for the fact that every single murder HAS to be round removal if done by an antag, not just a normal heal-able kill.
2: I just disagree
3: WYCI i guess? there is certainly opportunity to make it far more interesting
4: Refer to point 3
I do think it should be civ tech though

Cloning isn't an interesting mechanic, and this was done originally when cloning was removed from roundstart medical.

Round removal is apart of the game, rotting corpses shouldn't be revived (at all, in my opinion, but oppo is a whole different conversation.) so easily.
Science will get harassed for not researching cloning because people are just like that.
Cloning isn't engaging, you just scan the guy, duplicate 'em, and now they're alive with little to no effort on behalf of medical.
This was already removed from research in the past, it wasn't engaging then, it won't be now.

The game already has a substance called Opporozidon, which is significantly easier to obtain than a cloning machine, yet it has almost the same properties. Furthermore, as I understand it, cellular damage is needed in the game only because of the cloning machine.

we will get back on square one : Security taking science hostage and threatening to gib each and every scientist if they don't research cloning

Has this actually happened or is it just an old anecdote that gets brought up every time cloning is mentioned (sec rulebreaks not withstanding)

it happened twice on salamander last year

okay hear me out, what if it was a T3 but not present in the research nodes? it is a hidden tech only available from the tech disk printing machine?

The issue with this proposal is that the cloning machine could be researched almost at the start of the round. While I understand that it would depend on chance, such an opportunity would still exist

or the year prior, i believe it was the last straw that caused cloning to be ultimately removed fro mT3

okay hear me out, what if it was a T3 but not present in the research nodes? it is a hidden tech only available from the tech disk printing machine?

The issue with this proposal is that the cloning machine could be researched almost at the start of the round. While I understand that it would depend on chance, such an opportunity would still exist

the tech disk printing machine print a single crafting research(and not an entire research node) and is completely random, with T2 and T3 techs being less likely to roll

Cloning requires biomass, which is procured through a Biomass Reclaimer machine, or salvage to find chunks of biomass on expedition/VGroid.
If cloning were to come back, you'd need to add the biomass reclaimer to this research as well, as I don't think it makes sense for salvage to be the last piece of the puzzle. (albeit, salvage can proclaim a bunch of biomass compared to grinding a ton of monkeys)

Cloning requires biomass, which is procured through a Biomass Reclaimer machine, or salvage to find chunks of biomass on expedition/VGroid. If cloning were to come back, you'd need to add the biomass reclaimer to this research as well, as I don't think it makes sense for salvage to be the last piece of the puzzle. (albeit, salvage can proclaim a bunch of biomass compared to grinding a ton of monkeys)

As I remember, there is already a research in the game that unlocks the biomass reclaimer

the tech disk printing machine print a single crafting research(and not an entire research node) and is completely random, with T2 and T3 techs being less likely to roll

I still think this is a bad idea, sorry. Scientists should have a clear goal, not a vague task

1: I would agree if it werent for the fact that every single murder HAS to be round removal if done by an antag, not just a normal heal-able kill. 2: I just disagree 3: WYCI i guess? there is certainly opportunity to make it far more interesting 4: Refer to point 3

Did you just tell me to make the mechanic this pr is implementing interesting instead of telling the person making the pr to do it...?

Anyone who cares enough to fix the problem

Cloning requires biomass, which is procured through a Biomass Reclaimer machine, or salvage to find chunks of biomass on expedition/VGroid. If cloning were to come back, you'd need to add the biomass reclaimer to this research as well, as I don't think it makes sense for salvage to be the last piece of the puzzle. (albeit, salvage can proclaim a bunch of biomass compared to grinding a ton of monkeys)

As I remember, there is already a research in the game that unlocks the biomass reclaimer

Ah, I see it now.
It's a T1 service research, meat manipulation (wow that name)
Thank you for reminding me.

the tech disk printing machine print a single crafting research(and not an entire research node) and is completely random, with T2 and T3 techs being less likely to roll

I still think this is a bad idea, sorry. Scientists should have a clear goal, not a vague task

and i am adamant that such mechanic should not be available behind and limiting research tier period as it will lead to player animosity toward science.

There was also a large culture of people saying getting any T3 that wasn’t cloning was just throwing the round / helping antags. I imagine having more options can help this but it’s still a valid concern as cloning solely helps the crew vs things like speed boots or bags of holding which can also help antags if used correctly

There was also a large culture of people saying getting any T3 that wasn’t cloning was just throwing the round / helping antags. I imagine having more options can help this but it’s still a valid concern as cloning solely helps the crew vs things like speed boots or bags of holding which can also help antags if used correctly

and if we try to think like them, allowing antagonist to work in department allow them to get more themed gears, cargo traitors can easily get guns, engineering traitors can break wherever using their tools, medbay traitor get a large panel of medicine/topical/poison.
So is cloning really and issue or was the players behaviour was the real problem back then? I for myself do understand that death should be impactful, but it death REALLY impactful when every methods of RR cannot be dodged?
Unless you play as the whinny careful crew with a broom stuck up your backside, denying antags was not really liked last time i played but again, every 5th PR are salt PR made to make Greentext easier without any meaningful or solid arguments(looking at you PR that made escape pods that bring you to CC count you as marooned on station but also count traitor as escaped alive to CC)

died 2023 born 2024 welcome back CMO killing RD because he decided to research something fun (speedboots) instead of cloning

This will become an "essential" unlock, meaning everybody will yell at science if it is not picked as the tier 3, killing veriaty with the others. Since this is likely too powerfull to be a tier 2, even if it costs 50k points, I feel like this PR should not be merged. Maybe it could be an ultra rare drop from the tech disk terminal but even then thats iffy.

cloning is the opposite of an interesting mechanic

Cloning is not gonna help anyone who is an antag target, they usually gib you. This would just ensure they would do it in a way where you can't be cloned, which would satisfy only those who really enjoy being killed the same way every round. You are only punishing the few antags who are trying to make the round more interesting instead of toolbox bombing a random guy and then not doing anything for another 94 minutes.
Cloning also completely invalidates the fear of workplace accidents, if your depth 6 artifact burned you to crisp or whatever the paramedic will just put you in the magic respawn machine. The "RP environment" falls apart if people are not afraid to die.

y'know, maybe people re-introducing removed content should learn from why the content was removed in the first place. This PR doesn't seem to address any of the issues that t3 cloning had in the past besides the fact that it's not in medical research so it's not an objectively bad tier 3 option

Pretty much everything that's been said before on why cloning is bad and was removed has been said, and the reasoning in why this PR should happen doesn't change any of these issues for an inherently bad mechanic.

cloning

Cloning was removed from medbay (barring jackpot loot from salv) for good reasons. Reintroducing it with the "balance" that the cloning pod now requires some more (trivially obtainable) materials doesn't address any of said reasons. This PR should be closed.

Cloning was removed from medbay (barring jackpot loot from salv) for good reasons. Reintroducing it with the "balance" that the cloning pod now requires some more (trivially obtainable) materials doesn't address any of said reasons. This PR should be closed.

where i do agree it could be a PH piece of content while waiting for surgery to come in and allow brain transplantation, ANY undodgeable RR methods should ALSO be removed

where i do agree it could be a PH piece of content while waiting for surgery to come in and allow brain transplantation, ANY undodgeable RR methods should ALSO be removed

Round removal is part of the game.  That's why rounds are only 1-2 hours long, and ghost roles exist. I genuinely feel we need MORE round removal in the game so people get used to it. People are too accustomed to being revived no matter what and always mald in OOC that they didn't survive the things meant to kill them. Syndies round removing is part of the game. Dragons round removing is part of the game. Immovable rods exist and are part of the game. Oftentimes, the only thing you can do is try again next time. Losing is fun, after all.

i NEVER said RR was bad, i only said that UNDODGEABLE RR should not be part of the game, yeah it should happen but it SHOULD NOT be a sweaty pro-gamer move neither an unskilled manipulation like moving the fucking recycler on top of disposal chute, you wanna RR someone put in some effort!

also ghost role exist? are you talking about the 7+ mouses that instantly get murdered on sight by anyone?
or the botany trays or vending machine? i sure love having nothing to do than yelling on passerby or having the botanist growing weed in my tray
or the minibots? that NEVER get picked during conscious event
or positronic brains? to have the crew yell at you and treat you worse than a door opener.
the game barely have any content because contributors dont care about what the job should be and continue mergin SS13 slop again and again, thought they were supposed to be a focus on job content? the only final-vision job content i saw since i started playing was the few addition and changes to to science.
botany? barely anything but additional mutation when the system as a whole need a reactor
Chef and cooking? same it need a complete overall
Medical and Chemistry? someone is working on it, someone singular so it's bound to take a while
Meanwhile the contributors? keep pumping antagonists and other content that doesn't bring any department closer to their 1.0 version

also ghost role exist? are you talking about the 7+ mouses that instantly get murdered on sight by anyone? or the botany trays or vending machine? i sure love having nothing to do than yelling on passerby or having the botanist growing weed in my tray or the minibots? that NEVER get picked during conscious event or positronic brains? to have the crew yell at you and treat you worse than a door opener. the game barely have any content because contributors dont care about what the job should be and continue mergin SS13 slop again and again, thought they were supposed to be a focus on job content? the only final-vision job content i saw since i started playing was the few addition and changes to to science. botany? barely anything but additional mutation when the system as a whole need a reactor Chef and cooking? same it need a complete overall Medical and Chemistry? someone is working on it, someone singular so it's bound to take a while Meanwhile the contributors? keep pumping antagonists and other content that doesn't bring any department closer to their 1.0 version

these damn contributors (which are basically just random people implementing stuff for the game for free) coding what they want to see in the game! So evil!
about cloning: I am personally all for cloning, as long as it's not just a T3 research and rather a rare tech disk salvage can find; cloning shouldn't count as the same person for the sake of antagonist objectives (so if you killed your target and it gets cloned, you still succeeded).
using ghost roles as an argument against cloning isn't sufficient: ghost roles are very underwhelming at the moment, most of them have practically 0 roleplay/gameplay opportunities, spectating a round might be unironically more engaging and fun.

@Aisu9 please be respectful, ranting about contributors is not appropriate.

Thank you all for your opinions and suggestions!
I've reconsidered my proposal and identified two significant issues:

The limited and unengaging gameplay of the cloning machine (yes, I’ve changed my mind)
The cloning machine overshadows the other research disciplines

Once again, thank you all!


fix TFOpLambda name for KerasOpDisparcher

Try to fix https://github.com/tensorflow/tensorflow/issues/43840

@scd75 The test are failing. I see that they name was specifically removed with this claim https://github.com/tensorflow/tensorflow/blob/f718b7fd197836e335a69844dfec4e2c2ef2a615/tensorflow/python/keras/layers/core.py#L1340-L1344 /cc @tomerk

Well I guess that is a big change triggering many issues. Does that mean that the "name" parameter in all tf-op layers constructors become useless??
I guess the solution to the original issue would rather be to fix the way tf-ops are automatically named (to make sure they are uniquely identified), rather than totally getting rid of the "name" attribute which is very useful in many cases (and which I guess is widely used)

I am trying to compile this locally to run tests. But as always it is a quite heavy a long process without a fresh cache

I've replied on issue #43840 and you can find more info about the functional API refactoring in the release notes:
https://github.com/tensorflow/tensorflow/blob/master/RELEASE.md
The names of individual ops in the graph generated by a Keras model should not be considered load-bearing, and we have never made guarantees about them. If we did it would prevent us from making any change to Keras layers that alters the actual ops in the TF graph that we generate. That would prevent us from making all sorts of performance improvements, bug fixes, or code cleanups.
If your use case is that you need to identify the generated TFOpLambda layer by its name and the auto-incrementing names aren't sufficient for you then we can explore alternative ways of allowing users to explicitly override the name of the auto-generated layer. But we'd need to understand why you treat the op names generated by tf op layers in a load-bearing way to see what changes/features would even make sense.

@tomerk What what it will be the role of of name param in the ticket?
tf.identity(test, name='abcd')

In the linked issue, because test is a keras input (which mimics a tensor) rather than actual tf.Tensor, it will create an op layer in the functional model that calls tf.identity.
Generally when called on standard tensors, tf.identity uses the name argument to set the names of any ops in the graphs it creates.
But, when we create op layers we have to drop the name argument because if the op layer is reused and it passes the same name arg to tf.identity each time it tends to create multiple ops in the same graph that have the exact same name. This is unsupported and causes an error to be raised.
So, in the specific tf.identity(test, name='abcd') call from the ticket name will remain completely unused.
The more important question here is why is name being set to abcd, and what is the user (in this case @scd75 ) hoping to get by having the name arg reflected in the ops generated by the Keras model? E.g. is it a slightly more interpretable tensorboard graph? Is there code that is trying to directly manipulating nodes in the graph that ends up treating names as load-bearing? etc.

E.g, there are things we could to that set name as the name of the layer we generate (even though we would still drop name before calling the actual tf.identity when running the layer). This would help if the user just cares about identifying the layer w/o using the auto-incremented names, but it wouldn't help if the user is looking to maintain exact compatibility for pre-existing code that treats graph node names in a load-bearing way.

@tomerk I understand the logic but by an API usability point of view it is not so good to have a parameter that it is more or less silently ignored or at least we could raise a Warning.
I don't think that the proposed abcd/Identity:0 it is more consistent but could give a role to the name param.

Thanks for the reply and detailed explanation on #43840 @tomerk .
To clarify my use case: I treat some input features differently in my model. Each of those (named) features in the input dataset in mapped to a different keras Input layer whose is named after the feature name.
I then perform several preprocessing operations on those input layers (which differ depending on the feature), and I end up naming the resulting tensor through the use of a tf.identity layer, which I name "preprocessed_{feature_name".
in the rest of the model, I am then able to apply different subsequent operations for each preprocessed feature, depending on their name, by relying on the "name" attribute of the preprocessed input tensors.
But maybe there is a better way to do it.. I just found it easy and quite clean to carry over the feature name all along the process, from the very first Input layer, to the entry of the rest of the model.

@tomerk Can you please take a look on above comments from @bhack, @scd75. Thanks!


Support for "aggregate checks"

Hi there,
I'm not sure the title really conveys what I'm trying to say here, let me explain.
One of the thing I want to monitor is a Mesos cluster, which runs Docker. One of my checks would be "I need to have at least three instances of this Docker container across my whole cluster".
The same paradigm could be applied to some other datacenter-oriented OS's, with a check that would be triggered if the total available RAM across the cluster falls beneath a certain point.
At this moment, to my knowledge, checks only apply to a single node's context.
Thank you for your ideas and suggestions.
Regards,
A.

Hello @apognu, this functionality currently available Sensu core, and is known as check aggregates :+1: You can add the check definition attribute "aggregate": true (https://sensuapp.org/docs/latest/checks#definition-attributes) to any check definition and an aggregate will be created for each execution. Sensu check aggregates are "lazy", int that there is technically no end to one, you can query the check aggregates via the Sensu API https://sensuapp.org/docs/latest/api-aggregates

Just to be clear, there is or is not support for standalone aggregates yet?  I see a few different open issues/PRs about this.  Unless I'm missing something, my only option for standalone aggregates at this point is to make a check querying/aggregating redis data directly like @solarkennedy has done with his cluster check script. @portertech , is this true?

Tagged this for renewed discussing in #1218.

A proposal for "aggregates 2.0" has been published as a comment on issue #1218. Barring any objections, this proposal will be implemented in Sensu 0.24 and will resolve this issue.

Aggregates 2.0 has been merged into the release/0.24 branch https://github.com/sensu/sensu/pull/1273 Sensu 0.24 will be available sometime next week.


[analyzer] The analyzer fails to report "Type variables can't be used as constants."

Consider:
void main() {}

void foo<T>() {
  switch (List<int>) {
    case const (List<T>): print("bar");
  }
}

As I expected, this program can't be compiled. The compiler emits the following error message:
Error compiling to JavaScript:
lib/main.dart:5:17:
Error: Type variables can't be used as constants.
    case const (List<T>): print("bar");
                ^
Error: Compilation failed.

However, the analyzer does not report any errors. I expected that the analyzer would warn about the error here.
Reproducible on DartPad:
master channel
Use Flutter version 3.11.0-15.0.pre and Dart version 3.1.0-155.0.dev
+ Dart experiments: --enable-experiment=inline-class

@scheglov Just FYI

https://dart-review.googlesource.com/c/sdk/+/307663


Create new configuration for builds to Firebase

Note: XCConfig’s set from the fastlane scripts are applied to all the targets in the project (including the dependencies). Creating the new configuration and applying the XCConfig specific to that configuration is the easiest way to limit the XCConfig supplied settings only to the app target

A test run with the new changes -> https://github.com/dolbyio-samples/rts-app-ios-viewer/actions/runs/5250031074
I have verified the app works fine now.


New sample: App showing chunking

#1019 adds a test with an App showing manually chunking. I don't think this is a common scenario that warrants a new sample. #176 was about layering transfer encodings, but we can cross that bridge when we get there.


Set user sessions to DB in config.php

@hosungsmsft - sorry for the confusion, this change request is to set user session handler to database when the template is not using redis for user sessions. I have updated the code to reflect this change.

We can close this one as this does not have any significance proof that it will improve the performance

Closing this per above testing and comments.


Feature: Added support for HTTP/2 to improve perf

Type

[ ] Bug Fix
[x] New Feature
[ ] Sample

Related Issues?
NA
What is in this Pull Request ?
Added HTTP/2 support for all HTTP requests to improve performance.
It will by default fallback to 1.1 if the server doesn't support.

Tested this PR. When I execute i.e. Get-PnPMicrosoft365Group, I see it has HTTP 2.0 in the requestmessage. Fiddler however shows a normal HTTP 1.1 connection is being made:

Is that as expected? What would make the code ask the server for HTTP 2.0 I'm wondering?

No issues from my current machine. Must have been a connection issue. Going to proceed with merging.