CUDA/Windows instructions do not work; there is no setup.py

Prerequisites
Please answer the following questions for yourself before submitting an issue.

[X ] I am running the latest code. Development is very rapid so there are no tagged versions as of now.
[ X] I carefully followed the README.md.
[ X] I searched using keywords relevant to my issue to make sure that I am creating a new issue that is not already open (or closed).
[ X] I reviewed the Discussions, and have a new bug or useful enhancement to share.

Expected Behavior
I expect CUDA support to work, or not to be claimed to work.
Current Behavior
CUDA does not work.
Environment and Context
Win11 x64, followed all instructions. Still runs 100% CPU.

Physical (or virtual) hardware you are using, e.g. for Linux:

$ lscpu

Operating System, e.g. for Linux:

$ uname -a

SDK version, e.g. for Linux:

$ python3 --version
$ make --version
$ g++ --version

Failure Information (for bugs)
Your instructions are wrong.
Steps to Reproduce
There is no setup.py.
Try the following:

git clone https://github.com/abetlen/llama-cpp-python
cd llama-cpp-python
rm -rf _skbuild/ # delete any old builds
python setup.py develop
cd ./vendor/llama.cpp
Follow llama.cpp's instructions to cmake llama.cpp
Run llama.cpp's ./main with the same arguments you previously passed to llama-cpp-python and see if you can reproduce the issue. If you can, log an issue with llama.cpp

Failure Logs
There is no setup.py. Nothing else matters because your instructions don't even match the reality of which files exist.

ok first, have you actually tried building the project with commands suggested in llama-cpp-python readme? because it seems you are trying to build straight to vendor llama.cpp.
Try the following commands on windows (cmd):
git clone --recurse-submodules https://github.com/abetlen/llama-cpp-python.git llama-cpp-python
cd lama-cpp-python

set FORCE_CMAKE=1 && set CMAKE_ARGS=-DLLAMA_CUBLAS=on
python -m pip install -e . --force-reinstall --no-cache-dir

Note: this requires cuda installed and vs studio with cmake
Secondly, what does your script look like? you need to set gpu_layers param to use the gpu.


Recommendation: How do I implement it using RubixML ?

Hello, I'm looking to implement a recommendation engine using RubixML, but I see no documentation regarding this.
The idea is 3 files:

containing user id and it's dimensions
containing item id and it's dimentions
containing interaction that a user has with an item (user_id, item_id, score)

Any idea / example ?
Thanks

@andrewdalpino Any idea / example ? Thank you very much.

Right, we try to provide examples of common applications of Machine Learning but we cannot give examples for every problem. Recommendation systems are not unique to Rubix ML or PHP but rather a use case for Machine Learning in general. You can use a search engine such as Google to search for papers, software repos, and blog articles on how to implement a recommendation system using Machine Learning. Then you can reference the Rubix ML docs for how to apply those methods in PHP with the library.
Here's something to get you started https://www.kdnuggets.com/2019/09/machine-learning-recommender-systems.html


Bug: Tooltip in Tab

Info
We want to be able to apply a tooltip to a given Tab in our tab component, something like this:
<goa-tabs>
  <goa-tooltip content="This is a tooltip">
    <goa-tab heading="Profile">

        Profile Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut
  bore et dolore magna aliqua.

    </goa-tab>
  </goa-tooltip>
</goa-tabs>

So that we can incorporate a Tooltip on a Tab component.
Acceptance Criteria:

Have the ability to incorporate a Tooltip directly onto a Tab component

@Spark450 @twjeffery Is this a valid use case, including a tooltip inside a tab component?

@Spark450 follow up with the team that asked for this feature.

@ArakTaiRoth Find who requested this

@Spark450 Just documenting it here, requester was Yin Liu (https://goa-dio.slack.com/archives/C02PLLT9HQ9/p1706209145527069)

I spoke with Yin, and the team has already created a custom implementation for this feature.

@Spark450 Create something on the design board to actually investigate the implementation of this and how we want to do it.


Add start coordinates to level file

Add coordinates to file which tell where player will start out when level is loaded.

Implementation proposal: Consider a certain tile to be a starting tile.


Blur previous element when interacting with Cesium canvas.

Interacting with a canvas does not automatically blur the previously focused element. This leads to unexpected interaction if the last element was an input field. For example, clicking the mouse wheel could lead to the value in the field changing unexpectedly. The solution is to blur whatever has focus as soon as canvas interaction begins.
@emackey can you see any flaws/problems in this approach?

Thanks for the pull request @mramato!

:heavy_check_mark: Signed CLA found.
:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

This seems like a good idea.
One concern I might have is that there might not be a way to get the canvas to receive keyboard input, like hotkeys, if any apps are using that.  Clicking the canvas will cause it to blur itself.
Not sure the solution... maybe blur the active element and then attempt to focus the canvas?  Or just double-check that it's not the canvas that's being blurred?

Not sure the solution... maybe blur the active element and then attempt to focus the canvas? Or just double-check that it's not the canvas that's being blurred?

Good thinking.  However I'm having a hard time actually triggering what I agree would be a bug in the fix here.
I'm using this demo to test: http://localhost:8080/Apps/Sandcastle/index.html?src=HeadingPitchRoll.html
It listens to the document element, but with my code in this branch it still works.  I assume this is because the event bubbles up to the document when no element has focus.  This demo is also kind of sloppy by listening to document because if you active the geocoder and press up/down, it still affects the plane instead of the geocoder.
Working on that assumption, I then changed the Sandcastle to use viewer.scene.canvas.addEventListener('keydown', function(e) { so that it now only listens to the canvas.  I then select the geocoder input field and nothing happens with up/down arrows, as to be expected. But, when I click and interact with the Cesium window and then use the arrows, they still work!  As you pointed out, this doesn't seem to be possible because the canvas should no longer be the active element.
So it appears that nothing extra is actually needed here, but I don't fully understand what.
Thoughts?

Yeah that does seem to work.  The demo's click handler is registered after the viewer is constructed (of course, because prior to that there's no canvas).  So I guess the blur happens before the focus, and everything is OK.
I still feel like it would be good to check that the canvas isn't blurring itself.  But I don't quite know how an app would manage to get into that state exactly.

So yeah, if the canvas does focus itself but it has a tab index, then you can actually TAB your way into it if you try hard enough, and then a click will blur it.  Kind of an extreme edge case, probably no one has ever done that before.

yep I went there

Thanks for your diligence here @emackey, I pushed a fix to check if canvas is the element focused before blurring.


[BUG] After use vm template, Install guest agent checkbox does not work as exceptly

Describe the bug

To Reproduce
Steps to reproduce the behavior:

vm page
click Create
use vm template
click Install guest agent`

Expected behavior

Install guest agent  will work as expectly when users use vm template
Log

Environment:

Harvester ISO version:
Installation Mode: ISO/app-mode
Underlying Infrastructure (e.g. Baremetal with Dell PowerEdge R630):

Additional context
Add any other context about the problem here.

validated on master-0423.iso


Asked the instuctior about this, and this turns out to be a none issue and is due to the page being partially responsive.


Winget standard settings per package import and export?

Description of the new feature / enhancement
Hi,
I don't think this feature is in there but let me make a proposal or an idea.
(Or at least the way I want).
Lets say there is an standard way developers my expose there settings so that
not only what things are installed but settings.
Example: I install Visual Studio.
On my machine. Now not only are there lots of different workloads but all those
options as well.
Here my be the flow.

Install Visual Studio as I like to with options.
I use Winget to then do an export with the apps settings as well.
Export to package list.
Do an export of the settings as well.

Then come machine rebuild time - import package and settings.
I just great to add this to take to the next level - save time building and tuning
settings when we load our machines.
Feedback improvements welcome.
Thanks,
Ward.
Proposed technical implementation details
No response

It seems like this may be covered by winget configure and a DSC resource. @denelon - Do you have any example docs for installing visual studio and enabling workloads using DSC?
I know that it isn’t a perfect export/import, but the export case would be covered by

#3274

We do have some samples in Dev Home at https://aka.ms/dsc.yaml.
I'm going to go ahead and close this issue as a duplicate of #3274 since there is still work to be done. We're expecting to have WinGet configuration as a stable feature in WinGet 1.6 with:

#2845


WebSocket.java documentation comments update

Update documentation comments for methods getRemoteSocketAddress and getLocalSocketAddress in WebSocket.java.
Related Issue
Fixes #1108
Motivation and Context
Fixes documentation issue mentioned in the related issue.

@prashanttholia thank you very much for your contribution.
I quess I have to adjust SonarQube here..
Thx again

@prashanttholia thank you very much for your contribution.
I quess I have to adjust SonarQube here..
Thx again


Use Implicit Constructor Injection over Explicit

The @Autowired annotation for Spring components can be omitted for constructors in case there is only one constructor present on the class. Therefore, remove all the unnecessary annotation for those cases.

I guess it should be doable for me to get started


py>: operator supports python option arguments

Python version of https://github.com/treasure-data/digdag/pull/1022
FYI: It seems that sh> operator already supports List.
~I did not add tests because I could not find any existing tests for python operator, but I verified this works manually.~

Thank you for suggesting. I will take a look https://github.com/treasure-data/digdag/pull/1022 first.

https://github.com/treasure-data/digdag/pull/1027 is for rb> operator but, it . would be helpful for your py> operator testing.

https://github.com/treasure-data/digdag/pull/1022 looks good. Can I ask you to change this according to it?

Added tests.

Looks good. Thank you for adding tests.
@yoyama what do you think?

I commented on a very minor issue. After fix it I would like to merge it.

Where is the comment? (Maybe, you need to publish your review comments)


Experimental - Add Namespace

Hello @daneden and @eltonmesquita here's the feature request for the namespace.. I add it to the base sass file and let me know if necessary to do the same with individual animations. Thanks!

We probably want to add namespaces to all the animate.css classes, and find a way to automate it/allow users to specify their own namespace in the config file (with a default, of course).

@daneden yup! Will do that.. will update this one... it is only a base of our experimental feature. This one should be available on v4 of animate.css let's leave the 3.7.0 untouched except some dev-dependencies updates.

I think Postcss can do it easily. Maybe postcss-prefixer can take care of it. But first it's better to solve #836 first then solve this one. I might do it by the weekend if I get enough time.


Camera preview is oriented wrong on Nexus 6

Reported by waucka on 11 Feb 2015 04:03 UTC
The camera preview appears to be upside down (rotated 180 degrees?) on the Nexus 6.

Comment by w00t on 18 Nov 2015 19:02 UTC
This is also true on the Nexus 6P. I believe that a more accurate description is the the camera is mirrored on the long side. The short side seems to be correct.

Comment by abrody on 14 Dec 2015 22:56 UTC
Duplicate of #50

Fixed in c0dcf1c.


npm install cleanup

Please add .npmignore to exclude some of the dot files, webpack config files and docs folder. I don't see a use for them inside node_modules folder

Hi @vanthiyathevan , can you send us a PR?

Thank you.


Markdown in Jupyter Notebook not rendering properly in Sphinx Documentation

🐞 Markdown in Jupyter Notebook not rendering properly in Sphinx Documentation

See https://einsteinpy-project.readthedocs.io/en/latest/examples/Analysing Earth using EinsteinPy!.html

I am not sure about the complexity of the problem. It might be an upstream problem too!

Fixed in https://github.com/einsteinpy/einsteinpy/commit/d3ae5b7ad5fc4b37fe0eefd6cecaec5b72e51002


Attempt to solve issue #10

boost::current_exception uses C++11 std::current_exception when supported to transport exceptions without the need to wrap in boost::enable_current_exception.

gcc errors: some versions are complaining with
./boost/exception/detail/exception_ptr.hpp:436:90: error: cannot allocate an object of abstract type boost::exception_detail::clone_base
but the offending code does not in fact instantiate clone_base!
exception_detail::clone_base const& base {
     boost::enable_current_exception(std::current_exception())};
return exception_ptr(shared_ptr<exception_detail::clone_base const>(base.clone()));

I'm declaring base as a reference to the base class to bind the temporary of whatever class is returned from boost::enable_current_exception()
Maybe I should just be using auto?

clang errors: I don't see how the reported error is even remotely related to my changes...
bin.v2/libs/exception/test/cloning_test.test/clang-linux-3.6.2/debug/link-static/threading-multi/visibility-hidden/cloning_test.o: In function `void test_std_exception_what<std::ios_base::failure>()':
/home/travis/build/boostorg/boost-root/libs/exception/test/cloning_test.cpp:168: undefined reference to `std::ios_base::failure::failure(char const*, std::error_code const&)'

Clang errors are gone after adding dist: trusty to the Travis config.
However the gcc errors for older versions of gcc are still there!  :(


Runs compilation failure tests.

There are a couple negative compilation tests in compiler/front_end/module_ir_test.py named anonymous bits does not allow documentation and anonymous bits does not allow subtypes. These weren't being run because _make_negative_tests()wasn't being called.

@reventlov or @AaronWebster care to take a look?


Error when using css variables in scss files.

I think csscomb should support css variables by now, so this seems to be a plugin specific error?
Environment

VSCode Version: 1.27.2
OS Version: OSX 10.13.6

Actual behavior
[CSSComb]
Parsing error: /Users/tommymathiesen/Sites/Kirby/designsystem/src/styles.scss
Please check validity of the block starting from line #45
CSScomb Core version: 4.2.0
44 | :root{
45*|   --ion-font-family: "Effra";
46 |   font-family: var(--ion-font-family);
47 | }
Syntax: scss
Gonzales PE version: 3.4.7
...
Expected behavior
Validate and insert space after :root
...
Steps to reproduce
...
Config
{
  "remove-empty-rulesets": true,
  "always-semicolon": true,
  "color-case": "lower",
  "block-indent": "  ",
  "element-case": "lower",
  "eof-newline": true,
  "leading-zero": true,
  "quotes": "single",
  "sort-order-fallback": "abc",
  "space-before-colon": "",
  "space-after-colon": " ",
  "space-before-combinator": " ",
  "space-after-combinator": " ",
  "space-between-declarations": "\n",
  "space-before-opening-brace": " ",
  "space-after-opening-brace": "\n",
  "space-after-selector-delimiter": "\n",
  "space-before-selector-delimiter": "",
  "space-before-closing-brace": "\n",
  "strip-spaces": true,
  "tab-size": true,
  "vendor-prefix-align": true
}```

Hello, @jb-trm,
It's looks like a bug of CSSComb. We use latest version of CSSComb and most likely it can be reproduced with CSSComb as CLI. You can create issue in the CSSComb repository and after bump CSSComb version I'll update my plugin 👍

CSSComb did a fix. Is this plugin updated yet?

CSSComb did a fix. Is this plugin updated yet?

The plugin hasn't been updated. I had to update the plugin myself. Take advantage of it.
https://marketplace.visualstudio.com/items?itemName=naumstory.vscode-csscomb

Unfortunately, the new version of CSSComb has no effect for this problem.
const Comb = require('csscomb');

(async () => {
	try {
		const comb = new Comb('csscomb');

		const res = await comb.processPath('./smoke/test.css');

		console.dir(res, { colors: true });
	} catch (error) {
		console.dir(error, { colors: true });
	}
})();

This is work correctly if you drop the sort-order property:
const Comb = require('csscomb');

(async () => {
	try {
		const config = Comb.getConfig('csscomb');

		delete config['sort-order'];

		const comb = new Comb(config);

		const res = await comb.processPath('./smoke/test.css');

		console.dir(res, { colors: true });
	} catch (error) {
		console.dir(error, { colors: true });
	}
})();

mrmlnc, https://github.com/avdieiev/csscomb.js/commit/dfb352330b13218887f584bb2fdb9ec00cf67421

As you can see (https://github.com/csscomb/csscomb.js/commits/dev) this is unpublished version right now.

mrmlnc, I'm sorry, I sent the wrong link. https://github.com/csscomb/csscomb.js/pull/595
Everything works in this plugin (https://marketplace.visualstudio.com/items?itemName=naumstory.vscode-csscomb), because it simply updates the dependence


Fix docker builds

Problem
What is being fixed?  Examples:
Docker builds don't work right now
Change overview
Removing fixed dependancies that are broken
Testing
How was this tested? (at least one bullet point required)

CI

Note: More coming, trying to run through CI


See resultset of model

Hi Michiel
Quickly wanted to ask whether this functionality is still planned. From our perspective this would add great benefit to the extension as this would be similar to the "Run Query" in dbt cloud I guess. This way we don't have to copy/paste the compiled code and execute it separately in another tool or browser tab.
I thought how to best approach this one and probably it would be possible for the extension to take the compiled code and call a dbt operation dbt run-operation which uses the run_query macro Run_Query Jinja Macro to return the results. What do you think?

I also agree that this is the next priority for this plugin to become more productive with less context switching.
Ideally we would not rely on installing any user code into a dbt project itself.  I was thinking to create a custom task inheriting the internal dbt code. The disadvantage of this approach is that we don't rely on the public dbt api and experience might vary over different dbt versions.
The output should be displayed in a html table within vscode. Ideally the user would be able to modify the sql query to adapt for different test cases. Something like following plugin https://marketplace.visualstudio.com/items?itemName=koszti.snowflake-driver-for-sqltools
What do you think?

I have played around with a macro that could be called like this:
dbt run-operation get_results --args "sql: SELECT COUNT(*) FROM Table1""
and the macro definition would look like:
`{% macro get_results(sql) %}
{% set results = dbt_utils.get_query_results_as_dict(sql) %}
{{ log(results, info=True) }}
{% do return(results) %}
{% endmacro %}`
This would get the query results (as dict) in the console. But then again you made some very good points, as the extension should not rely on user defined macros. Additionally, having the output in a console window is just half the fun when having it as a html table would allow to filter/sort it 👍
Adapting/Changing the sql query on the fly would make perfect sense too. It would be cool to prevent another settings conf where I would have to enter the data warehouse credentials again. Do you think reusing them from the dbt project would be feasible?

I was thinking of following this approach:
Create a tasks similar to:
https://github.com/fishtown-analytics/dbt/blob/7ec5c122e17ab7e1a5dd1e1cab36527159564a0f/core/dbt/task/run_operation.py#L16
By doing this you can call all adapter methods (including running a query) and have access to the graph. In this task you could run whatever query and print it as a dict. so it can be picked up.
We can then call this task programmaticaly similar to how we run existing dbt commands.
https://github.com/innoverio/vscode-dbt-power-user/blob/5815cec7d4968dbc9b7ba7d07fef34cd3fffae85/src/dbt_client/dbtCommandFactory.ts#L105-L107

I see, you want to dig deep down the rabbit hole 🐇😀
I see the powerfulness of the approach, on the other side directly interacting with the library would result in more maintenance overhead going forward I would assume. For example the performance improvement they did in 19.1 could probably had an impact and could maybe lead to multiple version of the run_operation to maintain. But this are just simple thoughts from a person that did not dive too deep into their library yet. 😉
If you think its feasible why not trying and see how it works out 👍

I have seen that PopSQL did an integration with dbt: https://popsql.com/updates/dbt-integration which allows to execute the query during development. So this makes this issue even more important that we have that functionality as well in VSCode.

Were you able to test this already?

Well, the querying works, but haven't put the time in displaying the results.
I currenlty use following plugin in vscode to bridge this gap:
https://marketplace.visualstudio.com/items?itemName=mtxr.sqltools

Any updates on this one?

Just wanted to note that this is great idea and when this added we'll switch from dbt cloud to using this extension in vscode

related: #86

Implemented in latest version


Text is not well highlighted when using textprop in diff mode

Describe the bug
Text is well highlighted when using matchaddpos() but not textprop in diff mode.
To Reproduce
I am checking to highlight some text in diff mode.
For example, the following 2 windows are now in diff mode with the default color scheme

And execute below in the left window.
hi TestHL guibg=Blue guifg=Yellow gui=italic,nocombine

let hl = 'TestHL'
let col = 3
let len = 5

" Line 1: matchaddpos()
let ln = 1
call clearmatches()
call matchaddpos(hl, [[ln, col, len]])

" Line 2: prop_type_add() and prop_add()
let ln = 2
call prop_type_delete(hl)
call prop_type_add(hl, {'highlight': hl})
call prop_clear(ln)
call prop_add(ln, col, {'length': len, 'type': hl})

The, we can see as below.

Expected behavior
Line 1:
matchaddpos() can highlight text as expected if 'gui' includes nocombine.
Line 2:
When highlighting with prop_type_add() and prop_add(),
'guifg' is ok, but 'guibg' stays unchanged and 'gui' looks combined.
I tried to add {'combine': 0} in prop_type_add() but not yet well highlighted.
I would expect the same behavior between matchaddpos() and textprop.
Environment (please complete the following information):

vim 8.2.2067
Windows 10

Yes, combining match highlighting with diff highlighting, the match highlighting has priority.
Combining a text property with diff highlighting, the diff highlighting has priority.
We could perhaps use the existing priority field of a text property: when the value is 100 or more then it overrules diff highlighting (and perhaps some others).  Would that work?

I tried to add the priority as below, but the result is same. not well highlighted.
call prop_type_add(hl, {'highlight': hl, 'priority': 1000})
Does that what you suggested?

I didn't say adding a priority would work, it's only an idea for how we could change the code to make it work.

I find it not yet implemented.  It is nice to use the priority of textprop to overwrite tie diff highlighting. And also nice for matchadd because it can not overwrite the background color and attributes of the diff highlighting.


fix(grunt): Work around uglify to preserveComments as expected

fix(grunt): Work around uglify to preserveComments as expected until it's fixed in their repo
Root cause: gruntjs/grunt-contrib-uglify#366
Fixes #1461

@marclr : Thank you, picked as 32cdedb


Handle null input

providing handling to null input value , while doing aggregation

I'm confused - what change do you propose / request? Please, elaborate.

I've spent some time today experimenting with aggregation and NULL values, and I still can't say what this report is about. Baed on that, I'm going to close it - if you still think there's an issue (and there very well may be), please file a more detailed report. Ideally with an example that does not work the way you want it to work.


REDRIVER2.exe is being detected as a Trojan virus by Windows Defender.

Windows Defender is blocking the game executable, stating that it's a Trojan. This started happening when I updated the game files to the "Beta" update, before it was fine.
I know clearly that it's a false-positive, so I excluded REDRIVER2's folder from the anti-virus scan. I'll leave a screenshot of it below.

https://www.microsoft.com/en-us/wdsi/threats/malware-encyclopedia-description?name=Trojan%3AWin32%2FSabsik.FL.A!ml&threatid=2147780195

Sent to Microsoft.


Criterion C test framework does not support logging messages

According to the Criterion documentation it should be possible to generate logging output. The mentioned functions are however not supported on the Codewars platform. The only alternative is to use the stdio functions, but these interfere with test framework interpretation.

Closing as duplicate of https://github.com/Codewars/codewars-runner-cli/issues/596


Logos missing in published version

Describe the bug
It appears after installing the appilcation, the logos are missing: https://i.imgur.com/JcMvqNJ.png

fixed

Thanks, will merge once merge changes have been done.


[Suggestion] Any lock file for brew?

Is there any plan to add a config file and a lock file for homebrew like other package managers (e.g. RubyGem, npm, ) to ensure that multiple computers use the same versions of dev tools.

~/.brew: list the semver rule for installed tools
~/.brew.lock: list the locked versions of the  installed tools

See https://github.com/Homebrew/homebrew-bundle.

Someone could add lockfile support to Homebrew/bundle but we have no plans to do so at this time. We won't ever support full semver locking.


add support for java.time.Duration

This is the core changes needed to support java.time.Duration.
Because I am not familiar with the JDL tool (my fault, it looks great!), any feedback is very very very welcome.
Thank you!
see https://github.com/jhipster/generator-jhipster/issues/9095
Please make sure the below checklist is followed for Pull Requests.

[x] Travis tests are green
[x] Tests are added where necessary
[x] Documentation is added/updated where necessary
[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

@massimosiani Only one thing left to do is to update the JDL page here

Hi @MathieuAA, just to confirm, is it this one?

Perfect!

@MathieuAA : I'm ready to merge the PR on generator-jhipster. Could we merge this one?

Thanks & congrats @massimosiani for your first PR to jhipster-core !


fix: add missing args to lang('Auth.invalidEmail')

Description
Follow-up #1145
The argument has been added in #1145:
-    'invalidEmail'          => 'Unable to verify the email address matches the email on record.',
+    'invalidEmail'          => 'Unable to verify the email address matches the email "{0}".',

Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thanks!


upgrading go version to go 1.12

Hi @davecheney I forgot to add the signoff in the commit message. Is there any other way in which I can do the same after I have raised a PR

@Avni-Sharma you should be able to amend your commit locally to include the signature then force-push to your branch.

git commit -s —amend
git push -f

On 27 Feb 2019, at 01:37, Ross Kukulinski<EMAIL_ADDRESS>wrote:
@Avni-Sharma you should be able to amend your commit locally to include the signature then force-push to your branch.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you. I will review this when the 0.11 window opens.

@Avni-Sharma let's hold on this til Go 1.12.1 ships. There are some, uh, spicy bugs that are being backported from master rn.

Sure

I'm pretty sure 1.12 will be out soon, probably next week. Some of the reflection bugs are a bit scary.

@Avni-Sharma Go 1.12.1 is out. @davecheney shouldn't we pick 1.12.1 to complete this PR?

@Avni-Sharma yup, i'm just waiting for the 1.12.1 tag to show up in the docker hub, https://hub.docker.com/_/golang


How can I enable clangd index?

System information
Clangd version (clangd version 10.0.0 (https://github.com/llvm/llvm-project d32170dbd5b0d54436537b6b75beaf44324e0c28)):
clangd extension version (from :CocInfo):
## versions
vim version: NVIM v0.4.4
node version: v14.13.1
coc.nvim version: 0.0.79-80ae19dee6
coc.nvim directory: ~/.vim/.plugged/coc.nvim
term: xterm-256color
platform: linux

Operating system: debian10
When I use coc-clangd, it works fine, but there is not .clangd index directory generated in the directory.
I try set ycm's clang argument clangd --background-index=true -header-insertion-decorators=0 -resource-dir=~/clang/lib/clang/10.0.0 -limit-results=500(use ycm clang can gen index dir) to clangd.arguments, but still no directory generated.
How can I enable clangd index?

Try  "clangd.arguments": ["--background-index"]

I had try
"clangd.arguments": ["--background-index"]
"clangd.arguments": ["--background-index=true"]

but still no directory generated.

You need to have a compile_commands.json to tell clangd about your project files. otherwise it cannot index your project.

@kadircet can't use compile_flags.txt ?  when I use ycm, I also use .ycm_extra_conf.py, not use compile_commands.json

Be aware that using compile_flags.txt means background indexing won't work, as it doesn't provide the list of files to consider part of the project.

https://github.com/clangd/coc-clangd/issues/28#issuecomment-612569340

@fannheyward @kadircet Got it. Thanks.


Potential performance bottleneck when using reflection.

I'm reporting an issue I had with TCA. My app was lagging a lot and instruments showed that a bunch of the time was spent on the main thread computing debug description strings. This is caused by Mirror(reflecting:) which is used when trying to reassign local cases (EnumReflection.swift#L61).
I have huge data structs, with many containers and NSAttributedString, and generating the debug description doesn't come for free. The app is quite complex, but in the current state, ~80% of its work on the main thread is spent extracting CasePath's. I temporarily improved the situation by carped-bombing my data structs with a some protocol extending CustomReflectable and providing a very small default implementation, but this is far from optimal.
I think that this kind of call is not expected to happen on the main thread, furthermore at a high frequency rate like it happens in a reducer.
Maybe the result can be cached, or another approach to reflection can be used, with some kind of (optional) protocol or prescription over Action enums?

Hey @RabugenTom,
The performance of case path reflection is a known issue, and it's definitely a bummer. We're not sure there is anything we can do to speed it up. Ideally Swift would provide first class support for this (like they do for key paths), which would make it much much faster.
We have found that reflection case paths work just fine as long as actions aren't being sent at a rapid pace. We're working on a large application right now and the reflection hasn't been an issue yet.
However, if you do have some actions that are being sent in great volume, then you may need to write your case paths by hand rather than relying on the reflection. There's a discussion in this issue on TCA discussing this very topic:
https://github.com/pointfreeco/swift-composable-architecture/pull/327
Let us know if that helps at all.
Also, I do not believe your performance problems are coming from usage of .debug() (or at least it shouldn't be!). By default .debug() uses a synchronous background queue to do printing, and so hopefully it shouldn't slow down your application:
https://github.com/pointfreeco/swift-composable-architecture/blob/c1f88bd8608e8a26fd61ab46c9f11a63b0f4023d/Sources/ComposableArchitecture/Debugging/ReducerDebugging.swift#L158-L161
If your state is really large then I suppose there's a chance that printing the state is taxing your CPU and causing everything to slow down. However, none of that would show up as work on the main thread in instruments. Further, I don't think the code in EnumReflection.swift is executed at all for the .debug() operator. It's only in the case path reflection code.

Thanks Brandon, I'll look at the hand-crafted case paths. I wasn't aware of them. I'm already using Sourcery to generate a bunch of things in my app, so I guess can write a template to generate case paths automatically. I'll report if it solves the issue when restoring default Mirror's , description's, and  debugDescription's.
When I'm talking high rate, I'm considering a some kind of evolved macOS word processor using SwiftUI with two sidebars and many features (like between 5 and 10 "iOS screens" visible at the same time). There is not much I can do to lower actions traffic below the point I'm at, but as I said, the apps runs fine as soon as I'm able to mute my types' (debug)Description's, as ~80% of the time spent on the main thread was lost because of this during CasePath resolution.
Also, I'm not using .debug() at any place.

Upon closer inspection, it seems that the issue may not arise from Mirror instantiation at L61, but from String(describing:) at L63 instead, as _print_unlocked suggests  (see Swift's Mirror.swift). Maybe there is an alternative way to retrieve the path there, but I'm still very new to CasePath's codebase so I need to dig more into this.

I've written some Sourcery template to generate all my case paths. I confirm that using these explicit definitions resolves my performance issues.
I'm attaching the template here in any case someone wants to make use of it. You can also include it in swift-case-path if you want. You simply have to declare an AutoCasePath marker protocol and mark your Root enums with it. It generates case paths as static members of CasePath<Root, LocalAction> for actions of the kind .something(LocalAction) whenever LocalAction is AutoCasePath itself. It also works for indexed actions in the form .something(Key, LocalAction).
import CasePaths

// This template generates CasePath for enumerations implementing the `AutoCasePath` marker protocol.
// Declare `protocol AutoCasePath {}` in your code and mark enumerations that should participate to
// case path generation. For example:
//
//   protocol AutoCasePath {}
//
//   enum AppAction: AutoCasePath {
//     case profile(ProfileAction)
//   }
//   enum ProfileAction: AutoCasePath {
//     case buttonTapped
//   }

{% for type in types.enums where type.implements.AutoCasePath %}
{% for member in type.cases where member.hasAssociatedValue  %}
{% if member.associatedValues.count == 1 and member.associatedValues[0].type.implements.AutoCasePath %}
// CasePath for {{ type.name }}.{{ member.name }}({{ member.associatedValues[0].typeName }})
extension CasePath where Root == {{ type.name }}, Value == {{ member.associatedValues[0].typeName }} {
  {{ type.accessLevel }} static let {{ member.name }} = Self(
    embed: {{ type.name }}.{{ member.name }},
    extract: {
      guard case let .{{ member.name }}(action) = $0 else { return nil }
      return action
    }
  )
}

{% elif member.associatedValues.count == 2 and member.associatedValues[1].type.implements.AutoCasePath %}
// CasePath for {{ type.name }}.{{ member.name }}({{ member.associatedValues[0].typeName }}, {{ member.associatedValues[1].typeName }})
extension CasePath where Root == {{ type.name }}, Value == ({{ member.associatedValues[0].typeName }}, {{ member.associatedValues[1].typeName }}) {
  {{ type.accessLevel }} static let {{ member.name }} = Self(
    embed: {{ type.name }}.{{ member.name }},
    extract: {
      guard case let .{{ member.name }}(key, action) = $0 else { return nil }
      return (key, action)
    }
  )
}

{% endif %}
{% endfor %}
{% endfor %}

Hey @RabugenTom, thanks for the info! That is very interesting.
We'd love to figure out how to make the reflection API faster. If you figure anything out please let us know.
I'm pretty surprised that stack trace is so deep for you. Wonder what String(describing:) is doing. Is there any chance you could share your enums for us to check out?
Thanks!

Hey @mbrandonw, I've checked again and the issue is not where I thought it were (twice…). I've implemented some custom debug description in one of my data type and put a breakpoint to see where it was called. I should have done this from the start instead of making you loose your time.
The issue arises from L70, in ["\(root)"].
It is particularly visible when root is some collection. I don't know yet how to resolve this yet since there is nothing semantically wrong with the algorithm. One can maybe think about using "\(root.hashValue)" instead of "\(root)" whenever available. This doesn't seem to cause any issue and tests are still passing.
Or maybe some generalised high-order Parser can also be used to extract the Value's ;-).
About my stack trace being deep, this is not so surprising given the architecture of my app (and this doesn't take so long to dive into it when we are not in the special cases presenting the issue). It's a macOS app which allows to write songs with chords and tablatures with an enhanced text editor. It also separates content and presentation like LaTeX. What you see in the stack trace up to Array.description.getter is the [Root] -> [Editing] -> [SideBar -> SideBarModule -> SideBarModuleContent -> SideBarModuleContent.View] chain of features/actions. What happens after the Array.description.getter is the issue I was having, where a collection description was being generated. I'll extract the enums when the app is ready. Since they are now marked with my AutoCasePath protocol, I can extract them all in one place with another Sourcery template.

@RabugenTom Not sure if you ever revisited this, but I'm going to move to a discussion. Maybe more folks will have ideas they can share there!


Allow writing out a block to a file

This allows input data to be shown with syntax highlighing and then used in future steps. e.g.
<!-- $MDX write,file=data.json -->
```json
{ "value": 42 }
```

I tried using this for the capnp-rpc tutorial. But since discovering the dir label in #380 I found I could keep different versions of the file in different directories and achieve the same effect, so maybe this isn't needed.

This feature use to exist but was causing troubles and we removed it. Now that we have the dune stanza we might be able to eventually re-introduce it but I'd like to first have a good understanding of what you need it for!
If you found an alternative for now, I indeed think it's preferable! This otherwise can lead to quite complex promotion rules. Nothing dune can't handle but it can easily confuse users.


Modifying POST body has no effect

It seems modifying the POST body after sending a request with "Try it out" has no effect. Old body string is sent regardless. Is the body cached somewhere? My angular-swagger-ui version is 0.2.6.

Hi !
There is no cache when using API explorer.
I just made a test here and everything seems ok (version is 0.2.6 too).
Could you please give more details on how you use the component ?

My steps to reproduce it: I click on "model schema", which gets copied onto object which has type "body". I modify the copied JSON string a bit, click "try it out" and get a response. After that, I modify the JSON a bit more, and again click "try it out". This time however the same string that was sent the first time is POSTed.
I tried it with the test explorer you linked above and could not reproduce it there. I think I have to look more into it, to figure out what is going on.

+1


Investigate absolute vs relative path in infofile location vs cloudvolume directory for CVBackend

Currently, CVBackend uses abspath for the cloudvolume directory but not the infofile location. Turning the infofile into absolutes broke some tests.

@supersergiy I'm starting to look into this - I'm guessing our desired behaviour is that everything should work whether absolute or relative? Or do we want to require absolute in case it's running remotely?

I think it's OK to only allow absolute if it's less work, but the we have to throw on relative

So even though I want to do something like "everything will be converted to absolute at the time of initialisation", this is bad because then running something remotely with intermediaries in say ~/zetta_temp/ on my local machine means that the backends will get initialised to /home/dodam/zetta_temps and then be serialised. As such, the backends will still have to support relatives.
I think I did figure out what was likely breaking, though, which was that is_local was happy to declare local paths without file:// prefix to be remote.


Add ReleaseHowTo document

Add the document to record the release steps.

just a few minor changes rest looks good.


Sampling random contiguous windows of length N

I am trying to implement a data pipeline that samples random windows of length N in a contiguous sequence of samples in the WDS tarfile. For instance, if I have [0, 1, 2, 3, 4, 5, 6] in my dataset, and N=3, I want to sample sequences [0, 1, 2], [1, 2, 3], [2, 3, 4], ... [4, 5, 6].
The following code sort of does that by using two batched calls, but yields [0, 1, 2], [3, 4, 5] instead. How can I get the dataloader to sample overlapping windows?
keys_to_parse = ('obs_rgb.jpg', 'cls', 'log_idx.cls')
def get_ds(path, batch_size, train):
    ds = wds.WebDataset(path, shardshuffle=False)
    ds = ds.decode("torchrgb").to_tuple(*keys_to_parse)
    # construct sequences out of contiguous batches.
    if config.prefix_len > 1:
        ds = ds.batched(config.prefix_len, partial=False)
    if train:
        ds = ds.shuffle(100)
    preprocess_fn = functools.partial(preprocess, transform=train_transform if train else test_transform)
    ds = ds.map(preprocess_fn)
    return ds.batched(batch_size, partial=False).map(data_splitter_fn)

Figured it out. Instead of calling ds = ds.batched(config.prefix_len, partial=False), compose() function does exactly what I need:
def sample_windows(data):
		# concate in a deque, then yield.
		list_of_tuples = collections.deque(maxlen=config.prefix_len)
		for d in data:
			list_of_tuples.append(d)
			tuple_of_lists = tuple(zip(*list_of_tuples))
			if len(list_of_tuples) == config.prefix_len:
				yield tuple([torch.stack(l) for l in tuple_of_lists])

Usage:
ds = wds.WebDataset(path).decode().to_tuple('obs_rgb.jpg', 'cls').compose(sample_windows)


Add Flag to DbMap Enabling Clients to Toggle Off Errors on Extra Columns

The default behavior of gorp is to throw an error on select queries that return more columns than mapped destination struct fields. This behavior is undesirable in certain circumstances, specifically in situations when the development of database migrations and corresponding Go structure changes should be decoupled. This pull request gives client code the ability to configure gorp's behavior with respect to extra columns: if the toggle is set, gorp will not return errors when extra columns are selected. The default behavior is to throw an error when extra columns are selected, which is identical to the previous behavior of gorp.

Coverage remained the same at 66.635% when pulling 154ec5a891495dde526b6c0c88a507231a13534c on clyphub:master into 88f3ff2c0d5eb7b20af1b6b4d2051d7ddfffadb0 on go-gorp:master.

I'm going to close this PR.  We've had discussions about this already (most of which revolve around select *, which is a well-known bad practice), and we've decided that we may start to allow this in the future.  However, we want to make it difficult to use to discourage users from just using select * everywhere and setting a boolean flag on their main DbMap - it should be at least a little painful to use (and specifically selecting columns should be a lot easier to use than it currently is).


Create mechanism to record testbed success to workflows

Feature Request
Desired behaviour
An expansion of the verified workflow mechanism to accept a record of what platforms a workflow successfully ran on.
Will co-ordinate with @jaeddy

It works.


Improve domainmodel showcase

Added an interactive live preview for the domainmodel language using d3.js

@emilkrebs please squash your commits and rebase to the latest changes on the main branch: we've now switched deployment to GH Pages.


Bug:

Describe the bug
The live stream on dashboard has 6 seconds of delay but some actions like motion detection is done in real time in dashboard and calling services like setting presets are done in real time in app camera but delayed in dashboard.  Moreover, if I use MotionEye inside HA, there's no such delay...it works like in app camera. So it isn't a problem of the camera or HA or raspberry. it seems engine used to play live stream in this integration.
To Reproduce
Steps to reproduce the behavior:
Just check this two images that I uploaded. Same environment (Raspberry PI 4 + hassio updated) inside HA with MotionEye or outside with rtsp.

Expected behavior
Just normal delay 1-2 seconds like in MotionEye Addon or playing via rtsp.
Log
No errors
Camera (please complete the following information):

Device Model: C.200
FW:1.0.17

Using stream component
[yes/no]
Camera works through onvif component OR different cameras work in general
yes
Camera has all attributes filled out properly in developer tools
yes
HASS Environment
Docker
Additional context
I know there was a bug to this, https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/37, but it isn't a problem of the camera or environment because inside HA, with MotionEye or out with rtsp directly it is working nicely. It is a  bit frustrating to have to wait 6 seconds to see motion detection sensor images, for example. Any chance to be able to change this behaviour ? I don't know how internally works this component, but any possibility to be able to do same as in MotionEye ?
Thanks for all your nice job JurajNyiri!

Duplicate of https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/37 .

Duplicate of https://github.com/JurajNyiri/HomeAssistant-Tapo-Control/issues/37 .

Version 2.4 has now been released addressing this, might provide significantly shorter delay for some users.

Version 2.4 has now been released addressing this, might provide significantly shorter delay for some users.

Thank you JurajNyiri!
I just tried it and It works like a charm. That's exactly what I meant :-)
Now I can move my camera going to presets in real time and motion detection is synced with image.
Thank you very much for this nice integration !

Thank you JurajNyiri!
I just tried it and It works like a charm. That's exactly what I meant :-)
Now I can move my camera going to presets in real time and motion detection is synced with image.
Thank you very much for this nice integration !


Refactor pubsub - remove gossip-specific features

Attempting to make the pubsub layer as unopinionated as possible, for possible inclusion in libp2p-pubsub eventually.
The pubsub layer now only has minimal defaults to stream data, decode RPC objects, process SubOpt and Message objects, and maintain an updated set of local and peer subscriptions.
This shouldn't effect the pubic gossipsub interface (although a few subtle bugs were noticed/fixed in refactoring).
Major changes:

rename / refactor rpc handling methods

_processMessages (perhaps more aptly named _processRpcStream or similar) now calls _processRpc, which handles handles a single RPC object from a Peer
_processRpc calls _processRpcSubOpt for each included subscription and _processRpcMessage for each valid included message

many non-pubsub features moved from pubsub.js

creating gossip rpc objects
join/leave
gossipsub configuration options
gossipsub control messages
peer shuffling
seen message cache

Codecov Report

Merging #74 into master will increase coverage by 1.10%.
The diff coverage is 93.25%.

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   84.41%   85.51%   +1.10%
==========================================
  Files           8       10       +2
  Lines         494      497       +3
==========================================
+ Hits          417      425       +8
+ Misses         77       72       -5

Impacted Files
Coverage Δ

src/messageCache.js
100.00% <ø> (ø)

src/index.js
82.78% <89.58%> (+2.60%)
:arrow_up:

src/pubsub.js
85.00% <95.65%> (-1.17%)
:arrow_down:

src/createGossipRpc.js
100.00% <100.00%> (ø)

src/heartbeat.js
83.33% <100.00%> (+0.23%)
:arrow_up:

src/message/index.js
100.00% <100.00%> (ø)

src/shuffle.js
100.00% <100.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 acb5d28...795aad6. Read the comment docs.


Fix creation scripts

Fixes a few issues I ran into while running these scripts as-is:

Extra hyphen in -p arg
Needed to specify entities array in jq
Inconsistent file paths

I realize now that the jq filter was not correct because it was originally written while autopagination was enabled by default. I pushed another change that reverts the jq filter modification and explicitly enables autopagination because that is likely desirable for these scripts.


Closes #1285 - Agents regularly report status to config server

This change is

components/inspectit-ocelot-configurationserver/src/main/java/rocks/inspectit/ocelot/agentstatus/AgentStatusManager.java, line 63 at r1 (raw file):
Previously, Henning-Schulz (Henning Schulz) wrote…
That would mean we add UNKNOWN to the AgentHealth enum (which is used in the agent and the config server), which I honestly don't like. It does not make any sense in the agent, as it always knows is health. It only makes sense in the configuration server, in case an agent (because its version is too old) did not report a status. And it should be handled somehow special in that case. So I think it is better to have some special value that indicates no health is defined, a.k.a. null.
If we want to make the potential absence of a health value more explicit, we could use an Optional<AgentHealth>, but imho it does not really add any value here.

Okay, understood.
I wasn't aware that it is only version-dependent. In a scenario where for example an error would prevent the agent of setting its health, the UNKNOWN value would be appropriate.
I agree that Optional<...> is not necessary.


Python 3.6 baggage code removal

Description

Fixes # https://github.com/Azure/azure-functions-pyfx-planning/issues/139

PR information

[ ] The title of the PR is clear and informative.
[ ] There are a small number of commits, each of which has an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] If applicable, the PR references the bug/issue that it fixes in the description.
[ ] New Unit tests were added for the changes made and CI is passing.

Quality of Code and Contribution Guidelines

[ ] I have read the contribution guidelines.

We should be able to remove aio_compat.py as well as rethink if we need typing_inspect.py itself.


运行示例项目LinkHeaderExample提示：RefreshConfiguration.of(context) != null错误

运行时报错：
The following assertion was thrown building LinkHeaderExample(dirty, state: _LinkHeaderExampleState#8c2c0):
search RefreshConfiguration anscestor return null,please  Make sure that RefreshConfiguration is the ancestor of that element
'package:pull_to_refresh/src/smart_refresher.dart':
Failed assertion: line 970 pos 16: 'RefreshConfiguration.of(context) != null'。
pull_to_refresh: ^2.0.0
[✓] Flutter (Channel unknown, 2.5.0, on macOS 11.6.1 20G224 darwin-x64, locale zh-Hans-CN)
• Flutter version 2.5.0 at /Users/jckj/flutter/sdk/flutter
• Upstream repository unknown
• Framework revision 4cc385b4b8 (3 个月前), 2021-09-07 23:01:49 -0700
• Engine revision f0826da7ef
• Dart version 2.14.0
• Pub download mirror https://pub.flutter-io.cn
• Flutter download mirror https://storage.flutter-io.cn
运行的是示例中的源代码，运行时提示上面的错误！有碰到的吗？

请使用没有命名的构造函数,copyAnscestor要确保祖先结点存在RefreshConfiguration才能用


Unable to start td-agent after stopping it

This seems to be an ongoing issue, and it has occurred at least thrice.
OS : Centsos 7
On stopping td-agent and starting (not restarting) the service. The td-agent service doesn't start while throwing error message as
systemctl start td-agent
Job for td-agent.service failed because the control process exited with error code. See "systemctl status td-agent.service" and "journalctl -xe" for details.
On using the script to start the service
: td-agent.service: control process exited, code=ex...s=1
Failed to start td-agent: Fluentd based data coll...ta.
Unit td-agent.service entered failed state.
: td-agent.service failed.
td-agent.service holdoff time over, scheduling restart.
Stopped td-agent: Fluentd based data collector fo...ta.
start request repeated too quickly for td-agent.service
Failed to start td-agent: Fluentd based data coll...ta.
Unit td-agent.service entered failed state.
td-agent.service failed.
I have to delete all the td-agent folder and reinstall for it work again . I guess want to check if I am missing out something or ..is this a valid issue ?

The information is too little to grasp all of the error. Could you add all of the logs from journalctl and your situation more detail?

No response from an author. I'm closing. if you have additional info, feel free to notify me.


Please add ability to change bts client password in gui

Duplicate: https://github.com/BitShares/qt_wallet/issues/96


Bolt throws a 401 error when slack hits the events endpoint

Description
I have a simple slack bot built with slack bolt -- I have tested it locally with the bot + appToken and it works totally fine. However,  when the app/bot is in a production environment with the prod install options configuration it fails with an unAuth error.
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Reproducible in:
package version: 3.11.1
node version: v16.16
OS version(s): Macos 12.4
Steps to reproduce:
Not applicable
Expected result:
I expected it to authenticate and call the /slack/events endpoint with no issues
Actual result:
Bolt responds with a 401 (unauthorized) error
Attachments:
Error logs: (Slack makes an api request to https://myBotEndpoint.com/slack/events and it throws a 401 -- because it is unauthorized slack decides to not call any listeners and as a result the Bot will not function as expected)

Note the "[ERROR] bolt-app Error: No installation data found" and  "[WARN] bolt-app Authorization of incoming event did not succeed. No listeners will be called" logs.

App config code:

Additional Notes

My app config code is identical to the example(s) found here in the docs: https://slack.dev/bolt-js/concepts#authenticating-oauth
Slack seams to also retry timed out requests -- I understand there is a way to disable this using a header value however, I cant seam to be able to access response headers within the bolt framework
I am not on an outdated/deprecated version of slack bolt and I still have this issue
All keys, redirect Urls, and scopes are all configured correctly and the issue still continues to occur
The app runs fine in a debug/dev environment with the dev app config code seen bellow:

[UPDATE] - I was able to fix this issue but still having problems with the slack retries.
Info

I have a message listener that listens for incoming messages
Slack seams to retry events that take more than 3 seconds - this results in multiple duplicated events
I am aware that there is a way to turn of the "graceful retries" according to this doc: https://api.slack.com/apis/connections/events-api#the-events-api__responding-to-events:~:text=know why yet.-,Turning retries off,-If your server -- however, how is this done in bolt?

Hi @darren-pieces! BoltJS doesn't currently allow for setting additional headers, but we appreciate you bringing it up as it will help inform our work on allowing header changes going forward. In the meantime, to work around this limitation, we suggest you create your own receiver. You can find documentation on how to do that here and an example is provided here.
That said, setting the header to turn off retries is not recommended. Doing so means that your application will lose all event data should any temporary interruption occur with your application.
We hope this information is helpful. Let us know if this resolves your issue!


Update what's new message for 2.10

Summary
Fixes #12083
Occurred changes and/or fixed issues
Updated the what's new message for 2.10 and updated the version number in the link.
See screenshot.
Kept the message short - Learn more about the improvements and new capabilities in this version and read about the UI framework migration in 2.10
We will need to update the docs with more information.
To test, log in and view the updated banner on the home page. If it is hidden, click on the three dots icon in the top-right of the page and click on 'Restore hidden cards'.
I have updated the version number in the Chinese translation but not the message. This translation will be removed at a later date/time.
Screenshot/Video

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@kwwii FYI on the text used

@gaktive can you re-review - rebased now that Mo's PR has passed the gates and been merged


Update bayesmix dependencies

Summary

Update to boost v.1.84.0 (currently fetching stan-dev/math.git)
Update to matplotplusplus v.1.2.1 + improved patch to switch off openCV (otherwise I can not compile on my laptop)

Checks

[ ] Check if now we can compile with clang >= 15 on MacOS operating system
(this would close #155)

To Do

Sync bayesmix-dev/math with upstream repo
Fetching bayesmix-dev/math in cmake/math.cmake

It is working on my laptop
CMAKE_MODULE_PATH /Users/marioberaha/dev/bayesmix_origin
-- The C compiler identification is AppleClang <IP_ADDRESS>00309
-- The CXX compiler identification is AppleClang <IP_ADDRESS>00309
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/clang - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/clang++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Found PkgConfig: /opt/homebrew/bin/pkg-config (found version "0.29.2")
-- Found OpenMP_C: -Xclang -fopenmp (found version "5.0")
-- Found OpenMP_CXX: -Xclang -fopenmp (found version "5.0")
-- Found OpenMP: TRUE (found version "5.0")
.....

So the clang version is correct.
make run_mcmc produced no error as also ./examples/tutorial/run.sh
I would sync our fork of math and merge this PR


add my profile to snakemake-profiles

I created a profile mainly for slurm, but it acn be adapted to other cluster systems.
https://github.com/SilasK/adaptable_profile

Nice! Ideally, there would be a unit test as well, but other profiles in here don't have it either, so I think it should be fine to move it here. Do you prefer me to fork or do you want to move it here? In addition, I think it won't hurt to clean it up a bit, add some more comments and remove some commented out snippets. However, this can be done after moving whenever you or somebody else feels like.

Hey Johannes Köster
I cleaned the code a bit. For unit test I would need a docker implementation of a cloud system isn’t it?
How can I move it to the repo?

Unit testing should happen via travis ci. You should be able to  setup e.g. a slurm cluster within travis ci via https://github.com/giovtorres/slurm-docker-cluster.
For moving, you need to go to settings and transfer ownership to the snakemake-profiles organization.

Before moving, what do you think about renaming the repo to generic? Feels more in line with the other profile names, i.e., slurm, pbs-torque, generic.

Done
On Thu, Oct 11, 2018, 10:53 Johannes Köster<EMAIL_ADDRESS>wrote:

Before moving, what do you think about renaming the repo to generic?
Feels more in line with the other profile names, i.e., slurm, pbs-torque,
generic.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Snakemake-Profiles/doc/issues/10#issuecomment-428874931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHLK2scFJgTU06mH5MXiBwdV_A4zF9gtks5ujwcRgaJpZM4Wlq_5
.

I would agree to move my repro to the snakemake profile user, as described in #4 , If I can continue having access.

Yes, of course, once you did the move, I will give you write access.

What do I have to do exactly?
On Thu, Oct 25, 2018, 16:57 Johannes Köster<EMAIL_ADDRESS>wrote:

Yes, of course, once you did the move, I will give you write access.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Snakemake-Profiles/doc/issues/10#issuecomment-433083983,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHLK2lM_DWeXwXgAqJM4QrjP8RXGQuelks5uodFUgaJpZM4Wlq_5
.

In your repo, go to settings and there to transfer ownership. Then you enter snakemake-workflows as the new owner.

Unfortunately, I get errors that I don’t have the right to create repositories neither in snakemake-workflows nor in snakemake-profiles. What can I do?

I have invited you to both organizations. Now it should hopefully work?


chore: make the npm script portable in Maven example

Description

Make the npm script portable in Maven example. npm guideline is to not prefix with relative path, as executables are added to the $PATH. Tested on Windows and macOS.
Add missing "npm install" excution to Maven frontend plugin.

Checklist
I don't think it's applicable, as the examples are not run in the CI ?

@all-contributors please add @rquinio1A for code, examples


Fix #647 Add Ruby strftime link on the note.

I tested #647  in my environment, and I got the following error.
I use note section instead of the writing URL on the table.
I also write this note on the CSV parser section.
@toru-takahashi @dmikurube Please take a look.
built-in.rst:492: ERROR: Malformed table.

+----------------------+---------+-------------------------------------------------------------------------------------------------------+-----------------------------------------+
| name                 | type    | description                                                                                | required?                               |
+======================+=========+=======================================================================================================+=========================================+
| timezone             | string  | Time zone if type of this column is timestamp. If not set, ``default\_timezone`` is used.             | optional                                |
+----------------------+---------+-------------------------------------------------------------------------------------------------------+-----------------------------------------+
| format               | string  | Timestamp format if type of this column is timestamp.                                                 | ``%Y-%m-%d %H:%M:%S.%6N %z`` by default. The Timestamp format refers to https://docs.ruby-lang.org/en/2.4.0/Date.html#method-i-strftime |
+----------------------+---------+-------------------------------------------------------------------------------------------------------+-----------------------------------------+

@hiroyuki-sato IIUC, this #648 is entirely overwriting #647. Let's do this in one commit, and focus on the issue. (Meaning that let's split refactoring-like changes into separate PRs.)

@dmikurube
rebased. Please take a look again.

I'll split refactor change about preview executor into separate PR.

Ah, at last, can you update the PR description? To remove "#647" and to be more descriptive. For example: Add a note on the timestamp format in embulk-doc with a link to Ruby documents.

@dmikurube Thanks.
I changed the title.

Thanks! Will be merging.

Sorry to be late. Merging this.

Thank you!.


how to get element list?

I want to click this button

but I can't get a list of buttons.
how to get it?
I tried this but fail
`
String xpath = "/Pane[@ClassName="#32769"][@Name="데스크톱 1"]/Window[@ClassName="#32770"][@Name="네이버 MYBOX 탐색기"]";
List list = (List) winapp.findElementsByXPath(xpath);`

What is shown in inspect.exe?

What is shown in inspect.exe?

this is it

You can use automation id 30100.

You can use automation id 30100.

But the ID of all buttons is 30100. I want to press the button I want.

If the UI is static then you can click by index, if not then ask the application developer to implement automation properties.

If the UI is static then you can click by index, if not then ask the application developer to implement automation properties.

Thanks for your comment


Generating API with amplify codegen produces an empty (just imports) API.service.ts file.

Generated previously from a /backend schema and the API.service.ts was generated correctly.
I updated my schema in /backend/api//schema.graphql.  ran the following commands:
amplify api gql-compile
amplify codegen
Did not work.  Then did amplify push.
Still did not work. Then did amplify configure codegen followed by api gql-compile and codegen
Still did not work.
Lost as to what to try next without doing an amplify init.
Versions:
Macintosh 10.14.6
amplify cli 4.18.0
angular/cdk  8.2.3
angular/core 8.2.14
angular/compiler 8.2.14

node 12.15.0
Steps to reproduce:

Update schema under local dir /backend
amplify push
Verified in Appsync schema is correct
amplify api gql-compile
amplify codegen

Expectations:
The API.service.ts file was generated with all proper source for managing the data store through this API.

does your schema.graphql file contains directives models?

The following is one simple definition I have and the others follow this layout:
type Device @model{
id: ID!
cuuid: String
puuid: String
code: String
}

Hello,
i just start learning amplify and its tooling. I have hit this same issue.
Is there any feedback?
Greetings,
David

Apologize for the lack of response on this thread. We'll be looking into this issue.


Added a new Pub/Sub sample for publishing/receiving messages with custom attributes

@michaelawyu apologies for the delay. Could you add the new commands to the README.md?


In Android 5 chrome, the shaka player can not request a license.

hi there.
During the test in the mobile environment, i check that the request to issue a license does not work and leave this inquiry.
Shaka 2.2.1 does not work properly on some Android devices.
Shaka Player can not process license requests in Android 5 version of Chrome.
Android7 and version 6 of the same environment confirmed normal operation.
If you have any information about this issue, please share it.

This issue appears to be a duplicate of #1023. I will close this one, since the older issue has more discussion.


Prepare the base for integration tests

Prepare the base for integration tests for mlflow part. Integrate it into the github actions pipeline. Create few real tests as an example so we can use the same approach for aim api.
┆Issue is synchronized with this Jira Task by Unito

Since this issue is marked as done in the FastTrackML project, should this issue be closed?


FPS gets lower overtime

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.

What problem did you encounter?
during my time using bloxstrap, I encountered a problem. I set my fps to 120, it worked splendidly. But as time went by, the fps dropped lower and lower reaching about 50 fps averagely. I do not understand why this is happening. probably because of my potato pc.

Upload your configuration please: https://github.com/pizzaboxer/bloxstrap/wiki/Uploading-your-Bloxstrap-configuration

Issue has been awaiting clarification for over a month, closing as stale. If you have anything further to add, just respond and I'll reopen.


feature: Master Detail Support For Xamarin Forms

Is your feature request related to a problem? Please describe.
I'm confused how I do navigation using Sextant with Xamarin Forms. Master Detail usually have Navigation Page as root in detail but right now we can't (or I don't know how to) navigate between detail in Master Detail
Describe the solution you'd like
Maybe we can take a look how navigation used in default routing from ReactiveUI https://github.com/reactiveui/ReactiveUI.Samples/tree/master/xamarin-forms/MasterDetail/MasterDetail ?

@albilaga I'll put together a Sextant sample for Master Detail.  If that feels too cumbersome, I'll look into a true MasterDetailPage implementation.

Hi
Any news on this topic?
I would appreciate a small sample.
e.g. a master, 2 details inkl. going back.
Somehow I struggle a bit. ;-)

https://github.com/RLittlesII/ReactiveUI.Showroom/pull/5 has an example of how to implement Master-Detail using Sextant.


Add go module support.

Remove comment line in credentials file as rub…y ini cannot handle it.
Add aws session token support.

Pull Request Test Coverage Report for Build 20

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 94.595%

Totals

Change from base Build 18:
-0.07%

Covered Lines:
70

Relevant Lines:
74

💛  - Coveralls


GCC compilation

Hello,

Could I compile this project with GCC?
Do you have any examples of CMAKE or any open source tool?

Thank You in advance!
Kevin Patino

Hi @kdpatino,

It is possible to compile the project (specific to a particular dev board) using gcc-arm tools. This link has details on how to export an mbed-online project to compile locally using gcc tools: https://os.mbed.com/handbook/Exporting-to-GCC-ARM-Embedded
Unfortunately, we do not have examples of CMAKE to compile the examples.

Hi navsuda,
I have one same development as this site referred, but the mbed compile is so complex that I have cost one day for this problem.
david.wang

Added an example to compile without the need for mbed-cli here: https://github.com/ARM-software/ML-KWS-for-MCU/tree/master/Deployment/Examples/simple_test_k64f_gcc. Hope it helps.

@navsuda
Your example help me compile successfully.
What's more, I want to know the file folder which you added . Where did they come from ?

Hi @oneway3124,
You can get that by creating a simple hello world project on mbed online compiler with the specific board you are targeting and then export with Make-GNU-ARM toolchain as described here.

Closing the issue, please reopen it if you still face the issue.


Likes Feature

Issues related #12 #9 #6
Capstone Project

Added like button functionality.
Added new likes to the API and popped all the liked on the cards.

Thanks for the review!


Use client instead of virtctl in RunStrategy related VM tests and cleanup subresource_api_test.go further

What this PR does
Use the KubeVirt client instead of using virtctl through a
NewRepeatableVirtlCommand where applicable in tests/vm_test.go.
This restructures the RunStrategy related VM tests to work without
virtctl and also consolidates the tests where possible to avoid
duplication of test code.
The newVirtualMachineWithRunStrategy helper is dropped and instead the
tests create VMs with the help of libvmi now. This change allows to use
a guestless VM in most of the tests.
Drop duplicate tests in tests/subresource_api_test.go that are already
covered by tests in tests/vm_test.go and consolidate tests that assert
the behavior when trying to restart a stopped VM.
"[test_id:1304] should restart a VM" is already covered by
"[test_id:3164]with RunStrategyAlways".
"[test_id:3175]Should restart when VM is running" is already covered by
"[test_id:2035]with RunStrategyManual".
"[test_id:3176]Should restart the VM" is already covered by
"[test_id:2187]with RunStrategyRerunOnFailure".
"[test_id:1305]" is changed to assert behavior when using
RunStrategyHalted explicitly. This was already done implicitly before.
The goal of this and further PRs is to reduce the amount of unnecessary uses of
virtctl in the e2e tests to stabilize tests, to reduce the number of flakes and to
remove redudant tests.
Before this PR:
NewRepeatableVirtctlCommand is used unnecessarily and tests are duplicated.
After this PR:
The KubeVirt client is used instead and duplicate tests are dropped.

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer
Merge after https://github.com/kubevirt/kubevirt/pull/12717

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

None

/hold
Wait for https://github.com/kubevirt/kubevirt/pull/12717 to be merged

/sig code-quality

@dhiller @vladikr The label triage/duplicate does not apply to this PR.

/hold cancel

Looks great! Thanks once again for your efforts!
/lgtm

/lgtm

/lgtm

/approve
Thanks @0xFelix, the code looks great!

/retest-required

/lgtm

/retest-required


Fix certain paths in the docs containing spaces when the shouldn't

This addresses issue #3275

I'd love to see a fix in the autogen scripts as well but this is a good stopgap. Thank you!


Issue 58/azure serverless integrations

#53
I mistyped the issue number in the commit message, my bad

@dboulet01 It looks like merges from external forks don't work if the other developer doesn't have Azure CLI installed.

@dboulet01 It looks like this change broke the Terraform, but I can't see where I changed anything Terraform-

@dboulet01 It looks like merges from external forks don't work if the other developer doesn't have Azure CLI installed.

the pipeline can't access repo secrets from a fork PR. if you clone, new branch, PR it'll run

Closing so I can do this from a branch rather than a fork.

FYI, I get a 403 when I try to push to a new branch on the main repo.

FYI, I get a 403 when I try to push to a new branch on the main repo.

Can you try now? Thanks


[Issue]: Azure ML Studio deployed models and AutoGen

Describe the issue
Hey everyone! I want to use AutoGen together with Azure ML Studio deployed models. I assume that will be the same as using Azure OpenAI models, and such I would only need to set the config_list. I am trying to do so, but I always get APIStatusError: Error code: 424 - {'detail': 'Not Found'}, which I don't really understand.
I have verified that the deployed model is up and running using lite-llm and getting an answer.
My questions: Have I understand something wrong? Can I use Azure ML Studio deployed models with this setup? If yes, any help debugging would be appreciated. If no, a general direction on how to use those models would be helpful!
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
Also the config looks something like this:
{
      "model": <deployment_name>,
      "api_key": <api_key>,
      "base_url": "https://<deployment_name>-serverless.<region>.inference.ai.azure.com",
      "api_type": "azure",
      "api_version": "2024-05-01-preview"
}

python version: 3.10.11
openai==1.17.0
pyautogen==0.2.26

@mpalaourg did you get a solution

@gagb having the same issue. We don't have an api_version for Azure ML studio deployed models. This seems to work with langchain, how should this work in autogen?

Same problem. Azure Models other than the Azure OpenAI models don't work. Is there something like together.ai for all the Azure AI models.
I tried Llama deployed on Azure (Meta-Llama-3-70B-Instruct) with the configuration below but got errors.
{
"model": <deployment_name>,
"api_key": ,
"base_url": "https://<deployment_name>.eastus.models.ai.azure.com",
"api_type": "azure",
"api_version": "8"
}


Can't add auth triggers through CLI

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v18.16.0
Amplify CLI Version
12.1.1
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug
I'm trying to update my auth triggers with existing lambda functions but I get this error from CloudFormation:
Parameters: [functioncircleAuthPreSignupArn, functioncircleAuthPreSignupLambdaExecutionRole, functioncircleAuthPreSignupName, ...{same pattern for other functions}...] must have values

I had issues getting the cli to run perfectly so I manually changed some things in cli-inputs.json.
This is the error when trying to update auth config.
? Which triggers do you want to enable for Cognito Create Auth Challenge, Define Auth Challenge, Pre Sign-up, V
erify Auth Challenge Response
? What functionality do you want to use for Create Auth Challenge Create your own module
? What functionality do you want to use for Define Auth Challenge Create your own module
? What functionality do you want to use for Pre Sign-up Create your own module
? What functionality do you want to use for Verify Auth Challenge Response Create your own module
Error updating the Cognito trigger: The "path" argument must be of type string. Received undefined

Possible related to this (https://github.com/aws-amplify/amplify-cli/issues/12784)
Expected behavior
Stack deploys correctly with lambdas set as triggers
Reproduction steps
Create lambdas for auth triggers then add those lambdas through the cli or cli-inputs.json. Deploy and it will fail.
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @kevinxu3, thank you for reaching out. could you let us know the changes made on the cli-inputs.json file? as this would enable us in reproducing a identical environment.

additionally noticed that you are looking to add existing Amplify CLI created Lambda functions as a trigger on the Auth resource. unfortunately this is currently not supported by Amplify CLI.
Could you try reverting the changes made on the cli-inputs.json, then run amplify update auth to add the triggers?

Got it. I resolved this by creating new triggers and copying over code. Thanks!

Glad to hear you are back up and running. Closing the issue, please feel free in reaching out to us again.


scipy.sparse.linalg.lsqr problems

There is a problem with lsqr with certain matrices. A minimal example is given by the following:
scipy.sparse.linalg.lsqr(np.array([1,4]),np.array([1]))
which fails due to acond being 0 and therefore having a float division with 0 in test3 = 1/acond. Changing the matrix a bit to scipy.sparse.linalg.lsqr(np.array([1,5]),np.array([1])) works. Matlab lsqr can handle both.
Furthermore is the output print(msg[0]) in the case arnorm==0 annoying. It should only be printed to stdout if show is set.

This doesn't fail on my machine. It gives the correct answer.

Can confirm this on my machine with current scipy as well.


Swagger configuration

Add swagger configuration
Add docs to all endpoints

@goritm check the conflict

@goritm my only note will be if we can remove the file backend/src/config/swaggerSchemas.js and instead create each swagger schema on a folder in the same level as routes or controller. For example, inside the swagger folder you will have, swaggerActor.js, swaggerApplication.js. In each of those files you can export the swagger schemas individually. The whole purpose if to avoid the file swaggerSchemas.js growing too much and separate the files for each model.

LGTM


Fix typo in ImportMakeCommand.php

Requirements
Please take note of our contributing guidelines: https://laravel-excel-docs.dev/docs/3.0/getting-started/contributing
Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
Mark the following tasks as done:

[ ] Checked the codebase to ensure that your feature doesn't already exist.
[ ] Checked the pull requests to ensure that another person hasn't already submitted the feature or fix.
[ ] Adjusted the Documentation.
[ ] Added tests to ensure against regression.

Description of the Change
Minor typo fixed.
Why Should This Be Added?
To avoid confusion =)
Benefits
Everyone benefits from a proper language.
Possible Drawbacks
None
Verification Process
Nothing, it's a two line diff.
Applicable Issues
None

@ogg1980 can you make this PR towards 3.1 please. Thanks! :)

@ogg1980 thanks!

no, thank you =D


ROX-25266: Deploy compatibility test against supported versions

Description
Implements the logic for running non-groovy compatibility tests and extracts some common functionality between the gke_nongroovy_compatibility_tests target and the gke_version_compatibility_tests target.
Related openshift PR to add the new test target: https://github.com/openshift/release/pull/55872
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change
Added a main() function to get_compatibility_test_tuples.py to manually confirm that its output remains correct. Next is running gke_version_compatibility_tests to ensure that they still function as intended. For testing the new target itself I think this PR needs to be merged first, and then I will be able to run /pj-rehearse ... on the openshift PR and look at the test output.

/retest

/test gke-nongroovy-compatibility-tests

/test gke-version-compatibility-tests

/test gke-nongroovy-compatibility-tests
/test gke-version-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests

/test gke-nongroovy-compatibility-tests
/test gke-version-compatibility-tests

/test gke-nongroovy-compatibility-tests
/test gke-version-compatibility-tests

/test gke-nongroovy-compatibility-tests
/test gke-version-compatibility-tests

@jschnath I am getting a lot of emails with updates on this and other PRs. I use those e-mails to form a queue of things that I want to review. I was wondering if you could maybe mark this PR as draft and use /test all for as long as you need to work on it? You could then undraft it when it is ready for review and that would be a ping for the team to review this.

 After that I'll rebase https://github.com/stackrox/stackrox/pull/12556 to master and it'll be much easier to review.

/retest


Add comments support to sbt Launcher Configuration files

The current spec doesn't allow comments: https://www.scala-sbt.org/1.x/docs/Launcher-Configuration.html
I had a "legacy" ~/.sbt/repositories file that didn't have the allowInsecureProtocol and I wanted to add comments directly to the file.  Unfortunately, it doesn't look to be supported.
I think it would be reasonable to have # as a comment char and ignore that line in the file.

It does already support # as comment, but it must appear as the first character on the line:
$ cat repositories
[repositories]
#comment
  local
  local-preloaded-ivy: file:///${sbt.preloaded-${sbt.global.base-${user.home}/.sbt}/preloaded/}, [organization]/[module]/[revision]/[type]s/[artifact](-[classifier]).[ext]
  local-preloaded: file:///${sbt.preloaded-${sbt.global.base-${user.home}/.sbt}/preloaded/}
$ sbt -Dsbt.repository.config=/tmp/hello/repositories
[info] welcome to sbt 1.4.7 (AdoptOpenJDK Java 1.8.0_232)
[info] loading settings for project global-plugins from idea.sbt,pgp.sbt,plugins.sbt,vimquit.sbt ...
[info] loading global plugins from /Users/eed3si9n/.sbt/1.0/plugins
[info] loading project definition from /private/tmp/hello/project
[info] loading settings for project root from build.sbt ...
[info] set current project to hello (in build file:/private/tmp/hello/)
[info] sbt server started at local:///Users/eed3si9n/.sbt/1.0/server/478e6db75688771ddcf1/sock
[info] started sbt server
sbt:hello> compile
[warn]
[warn] 	Note: Unresolved dependencies path:
[error] stack trace is suppressed; run last update for the full output
[error] (update) sbt.librarymanagement.ResolveException: Error downloading org.scala-lang:scala-library:2.13.4
[error]   Not found
[error]   Not found
[error]   not found: /Users/eed3si9n/.ivy2/local/org.scala-lang/scala-library/2.13.4/ivys/ivy.xml
[error] Total time: 1 s, completed Feb 21, 2021 1:17:12 AM


Update harness-platform-faqs.md

Added 2 FAQ
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ X] Feature

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://65a5053ccede742745cbf87a--harness-developer.netlify.app


fix(jsonforms): overlapping buttons

Issue
Description
Checklist

Code is easy to understand
No obvious errors or bugs
CI/CD Pipelines did run successfully
Tests were implemented
Documentation was created

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 41.70%. Comparing base (c6a61fe) to head (f546648).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #583      +/-   ##
==========================================
- Coverage   42.39%   41.70%   -0.69%
==========================================
  Files          19       19
  Lines         585      585
  Branches       76       76
==========================================
- Hits          248      244       -4
- Misses        262      266       +4
  Partials       75       75

Flag
Coverage Δ

unittests
41.70% <ø> (-0.69%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Move tests from /tests/e2e/tests_on_networks to a proper file

Currently, those tests are run on integration network. After devnet-in-rs is released, move those to full_node_test.py.
Should be done with #1179

Should be done in #1179


With setup

I have already published it to pypi.

+1

This is to address #3.

I'm going to cherry-pick the commit that fixes the README.md, but not the setup.py commit since we already have one.

Closed by 23963fc


Handle no-fill shape outlines

This PR fixes 2 related issues.
First, shapes with no outline (including newly-created shapes) were erroneously reported as having outlines of color "000000", when they definitely were not solid black.
Second, attempting to set outline color to the valid value of null, led to presentation validation error. This is documented in the below-mentioned issue.
To fix the second part, I also implemented ability to set existing outline to no-fill.
Fixes #703

PR-Codex overview
This PR updates shape outlines in PowerPoint slides. It ensures outline color consistency and adds tests for setting and getting outline colors.
Detailed summary

Updated outline color handling in SlideShapeOutline.cs
Added tests for setting and getting outline colors in ShapeOutlineTests.cs

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Another approach, which is maybe a bit more akin to experience in PowerPoint client, is to make an SetNoOutline(). Still return null from color getter if no solid fill set. Throw an explicit exception on color setter, pointing user to SetNoOutline() if they want to remove the outline.

Another approach, which is maybe a bit more akin to experience in PowerPoint client, is to make an SetNoOutline(). Still return null from color getter if no solid fill set. Throw an explicit exception in color setter for null values, pointing user to SetNoOutline() if they want to remove the outline.

That was my first thought, too.
Maybe it's not relevant to this PR, but I think a redesign of something like the following for IShapeOutline would be beneficial:
string? HexColor { get; }
void SetHexColor(string hex);
void SetNoOutline();

Lately, I don't like the Setter in nullable property🙂

Maybe it's not relevant to this PR, but I think a redesign of something like the following for IShapeOutline would be beneficial

Yah, I think this is relevant here because it's all the area this PR is touching.

Lately, I don't like the Setter in nullable property🙂

I agree in this case, if it's only going to throw on null setting then, it's not so good. In a broad case where null is a perfectly reasonable value, is there a more comprehensive argument against a Setter in nullable property?

In a broad case where null is a perfectly reasonable value, is there a more comprehensive argument against a Setter in nullable property?

I'm not sure if this is another argument, but let me describe my understanding of nullable in this case: When the developer introduces a nullable property with a getter and setter, it says, "It can return null, and it can accept null". However, we throw an exception if the user sets it to null. This looks like we "lied", or at least it's not an exceptional situation.

I'm not sure if this is another argument, but let me describe my understanding of nullable in this case: When the developer introduces a nullable property with a getter and setter, it says, "It can return null, and it can accept null". However, we throw an exception if the user sets it to null. This looks like we "lied", or at least it's not an exceptional situation.

Ok, yes, absolutely agree in this case. I was wondering if you were thinking that null-accepting setters were always bad in all cases, for some reason. For sure here, the interface lies by accepting a null setter, as you say.
Anyway, this has been updated. Please do take a look when you have a free moment.


Backward slash is appearing in the file_name field of the exported COCO json file on Ubuntu

Description
When exporting annotations in the COCO format, a backward slash appears in the file_name field of the exported COCO JSON file.
To Reproduce
Steps to reproduce the behavior:

I had a pre-annotated COCO json file (generated by the model), and I wanted to refine the bounding boxes.
I followed the steps mentioned over here, and I was able to import the annotations.
I have set 'Local files' as my Source storage type, and my absolute path is: /home/roshan/images
Now, when I export my tasks in the COCO file format, the project gets downloaded and I get the images and result.json in my Downloads folder.
[ISSUE: ] In the result.json file, the file_name field is in the following format: /home/roshan/images/image_name.jpg

Expected behavior
The file_name field should be in this format: /home/roshan/images/image_name.jpg
Screenshots

Environment:

OS: Ubuntu 22.04.2 LTS
Label Studio Version: 1.8.0

JSON requires that special characters be escaped to render properly. The \/ is an escape sequence that represents /.


Message template is not updated while property renaming

Which version of Serilog.Sinks.Grafana.Loki are you using?
v8.0.0-beta.0
Which version of .NET are you using?
net6.0
Describe the bug
New feature, that adds a level as a property fixes the collision with naming, but the MessageTemplate is not updated and rendered output is broken
To Reproduce
_logger.LogInformation("Hero's {level}", 42);

Expected behavior
Templated should be consistent with renamed properties names
Log/SelfLog output or exception with stacktrace
No response
Application or code sample, which could be used to reproduce a bug
No response
Additional context

I have read the documentation

[X] I have read Common Problems section in project wiki

Many thanks for this fix, we are now using this lib again and it works perfectly! <3

@rogeralsing seems like you are talking about another issue, aren't you?

After research I've found that it is not possible to fix it easily. So I think we will keep it for the future releases