Suggestions for improvement

I'm currently reviewing the JOSS submission, and I have some suggestions to improve code quality and usability. I did not install/run any code yet, I'll get to that soon.
Software Engineering (Python)
Important:

[ ] Add a .gitignore file to remove all contents from the repository that should not be there, for example pycache folder, dist/build folders. You can go to https://www.toptal.com/developers/gitignore/ and enter Python, PyCharm, Windows, etc. to get a good default .gitignore file.
[ ] Add tests, absolutely essential to ensure your software functions as intended, and keeps doing so as you add/change functionality over time. I recommend using pytest

[ ] Consider automatically running tests using Github actions CI/CD

[ ] Use one way to declare dependencies: better to integrate this in the setup.py file (even better to use setup.cfg)
[ ] Add a pyproject.toml file: this file tells Python dependency management tools what tool should be used to install the library (in PDOPT's case: setuptools). Some info: https://godatadriven.com/blog/a-practical-guide-to-setuptools-and-pyproject-toml/
[ ] Add community guidelines for: contributing, reporting bugs, getting support
[ ] Improve code styling, e.g. by adhereing to PEP8 or using a formatter like black
[ ] Remove unused code (cpu_test.py; a big chunk of data.py)

Suggestions:

[ ] Consider managing stable versions by creating tags and releases
[ ] Consider publishing your code on PyPI
[ ] In your current documentation file, Figure 1.2 seems to be missing some refs
[ ] Consider hosting the documentation as a website, for example on readthedocs.io

[ ] To create the documentation, consider using modern tools like mkdocs and Jupyter Notebooks instead of a PDF file (I know LaTeX is quite common in research circles, but for software it's usually much nicer to navigate a website with search, and Jupyter Notebooks are great for explaining code as it allows mixing code, execution output and explanatory text in one interactive document)
[ ] You can use mkdocstrings to automatically generate your API reference automatically (this also makes it easier to keep in sync with your actual code)

Paper

[ ] Add a comparison to other approaches/software ("state of the field")

[ ] All the paper references are own references of previous publications, I would also have expected some references to theory (e.g. the probabilistic model or set-based design)

@kyleniemeyer I'd like to know your opinion about the paper authors: I can only find code contributions by Andrea Spinelli, however Timoleon Kipouros is also listed as an author

@jbussemaker regarding the author list: JOSS does not require that all paper authors have contributed source code, since people can contribute in other ways (direction, etc.)

I'm currently reviewing the JOSS submission, and I have some suggestions to improve code quality and usability. I did not install/run any code yet, I'll get to that soon.
Software Engineering (Python)
Important:

[x]  Add a .gitignore file to remove all contents from the repository that should not be there, for example pycache folder, dist/build folders. You can go to https://www.toptal.com/developers/gitignore/ and enter Python, PyCharm, Windows, etc. to get a good default .gitignore file.

[ ]  Add tests, absolutely essential to ensure your software functions as intended, and keeps doing so as you add/change functionality over time. I recommend using pytest

[ ]  Consider automatically running tests using Github actions CI/CD

[ ]  Use one way to declare dependencies: better to integrate this in the setup.py file (even better to use setup.cfg)

[ ]  Add a pyproject.toml file: this file tells Python dependency management tools what tool should be used to install the library (in PDOPT's case: setuptools). Some info: https://godatadriven.com/blog/a-practical-guide-to-setuptools-and-pyproject-toml/

[ ]  Add community guidelines for: contributing, reporting bugs, getting support

[ ]  Improve code styling, e.g. by adhereing to PEP8 or using a formatter like black

[ ]  Remove unused code (cpu_test.py; a big chunk of data.py)

Suggestions:

[ ]  Consider managing stable versions by creating tags and releases

[ ]  Consider publishing your code on PyPI

[ ]  In your current documentation file, Figure 1.2 seems to be missing some refs

[ ]  Consider hosting the documentation as a website, for example on readthedocs.io

[ ]  To create the documentation, consider using modern tools like mkdocs and Jupyter Notebooks instead of a PDF file (I know LaTeX is quite common in research circles, but for software it's usually much nicer to navigate a website with search, and Jupyter Notebooks are great for explaining code as it allows mixing code, execution output and explanatory text in one interactive document)
[ ]  You can use mkdocstrings to automatically generate your API reference automatically (this also makes it easier to keep in sync with your actual code)

Paper

[ ]  Add a comparison to other approaches/software ("state of the field")

[ ]  All the paper references are own references of previous publications, I would also have expected some references to theory (e.g. the probabilistic model or set-based design)

@kyleniemeyer I'd like to know your opinion about the paper authors: I can only find code contributions by Andrea Spinelli, however Timoleon Kipouros is also listed as an author

Dear @jbussemaker, I've started working on these and I've already introduced the .gitignore file.
Could you please elaborate on the dependencies...? I am a bit inexperienced on creating python packages, this is my first time.
I was using an external text file to load the dependencies, so it was easier while I was developing the software. Should I have them hardcoded in the setup.py file, or external? Do you perhaps have a tutorial or guideline link I could look into?

@spinjet yes sure! The link I provided for the point after the "dependencies" actually contains a bit about specifying dependencies in a setup.cfg file (https://godatadriven.com/blog/a-practical-guide-to-setuptools-and-pyproject-toml/).
Some other links I found through a quick google query:

https://dev.to/2320sharon/making-sense-of-pyprojecttoml-setupcfg-and-setuppy-2o6m
https://setuptools.pypa.io/en/latest/userguide/quickstart.html

In particular: https://setuptools.pypa.io/en/latest/userguide/declarative_config.html

Here is another good guide to making an installable package: https://packaging.python.org/en/latest/tutorials/packaging-projects/

Progress so far:
Software Engineering (Python)
Important:

[x] Add a .gitignore file to remove all contents from the repository that should not be there, for example pycache folder, dist/build folders. You can go to https://www.toptal.com/developers/gitignore/ and enter Python, PyCharm, Windows, etc. to get a good default .gitignore file.
[ ] Add tests, absolutely essential to ensure your software functions as intended, and keeps doing so as you add/change functionality over time. I recommend using pytest

[ ] Consider automatically running tests using Github actions CI/CD

[x] Use one way to declare dependencies: better to integrate this in the setup.py file (even better to use setup.cfg)
[x] Add a pyproject.toml file: this file tells Python dependency management tools what tool should be used to install the library (in PDOPT's case: setuptools). Some info: https://godatadriven.com/blog/a-practical-guide-to-setuptools-and-pyproject-toml/
[x] Add community guidelines for: contributing, reporting bugs, getting support
[x] Improve code styling, e.g. by adhereing to PEP8 or using a formatter like black
[x] Remove unused code (cpu_test.py; a big chunk of data.py)

Suggestions:

[ ] Consider managing stable versions by creating tags and releases
[ ] Consider publishing your code on PyPI
[ ] In your current documentation file, Figure 1.2 seems to be missing some refs
[ ] Consider hosting the documentation as a website, for example on readthedocs.io

[ ] To create the documentation, consider using modern tools like mkdocs and Jupyter Notebooks instead of a PDF file (I know LaTeX is quite common in research circles, but for software it's usually much nicer to navigate a website with search, and Jupyter Notebooks are great for explaining code as it allows mixing code, execution output and explanatory text in one interactive document)
[ ] You can use mkdocstrings to automatically generate your API reference automatically (this also makes it easier to keep in sync with your actual code)

Paper

[ ] Add a comparison to other approaches/software ("state of the field")

[ ] All the paper references are own references of previous publications, I would also have expected some references to theory (e.g. the probabilistic model or set-based design)

@kyleniemeyer, @jbussemaker
Thank you so far for your help, I've adopted the new .toml approach with a setuptools .cfg file. I'll try to have it uploaded on PyPI for easy install through pip, albeit I may not be able to maintain the code 24/7.
I'm going to be addressing the tests soon, I'll try to add a few using pytest.
I'll also address the paper and extend it with some background referencing directly some of the building blocks that aren't directly developed by me (Set-based design being one). I'm not sure for other approaches, I reckon I could mention some design space exploration approaches or general use of optimisation as a discovery tool.

Hi @spinjet - good progress! I might suggest that for simplicity, you just include your package's dependencies directly in the pyproject.toml file, rather than a separate file: https://packaging.python.org/en/latest/guides/writing-pyproject-toml/#dependencies-optional-dependencies

And I just wanted to note that publishing on PyPI is also possible by CI/CD workflow, to automate most of it:
https://packaging.python.org/en/latest/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/

Dear @kyleniemeyer @jbussemaker
Thank you again for your help.
I've been writing tests and integrating them with pytest and GitHub actions, I am close to completion to this task.
I need to fix the testing of the optimisation module as it seems there's some randomness I cannot control (the module relies on stochastic optimisation approahces).
Hopefully this should be done by the week along with the CI for publishing!

Great, thanks for the update @spinjet!

As mentioned in https://github.com/openjournals/joss-reviews/issues/6110#issuecomment-1906266871
Automated testing with CI is now fully implemented.

@spinjet great progress here
One comment about the .gitignore file: please also remove all content from the repository that would be "ignored". For example, build and dist should not be present.

Added a almost completed section for State of the Field with relevant references.

Dear @jbussemaker.
I've completed the paper improvements.
Added a State of the Field section providing more context on the code and comparison with other frameworks/approaches. Introduced more references, including to the methods used in PDOPT. These also address the points @e-dub made in https://github.com/spinjet/pdopt-code/issues/4.
Hope this is satisfactory from the paper side, I'll focus on publishing in PyPI and having an online version of the documentation (especially for the API).

Online documentation is complete and online.
https://pdopt-code.readthedocs.io/en/latest/

Code has been published on PyPI: https://pypi.org/project/pdopt/
And I've made an official release as version 0.5.0.
@jbussemaker  Let me know if you are satisfied with the completion of the tasks.

@spinjet after these small updates, I am satisfied:

[ ] Link to the documentation website in the repository (click on the settings cog next to "About" and add the URL)
[ ] Fix small typos in the main readme

PyPI is styled exactly like that: "PyPI"
"Alternatevly it is possible to donwload the release..."

[ ] Also update installation instructions on the documentation website
[ ] On the documentation main page there is a typo in the acknowledgements: "cotnributed"

@jbussemaker Thank you again for the feedback.
I've made the corrections you asked for.

Thank you! Then I will go ahead and close this issue, and recommend publication :)
Very nice work!


Forgetting to prefix local path with ./ leads to unexpected installation

I wrote a simple module, that is located injs/admin. Without thinking about it much, I wrote
import colourizeColourselects from 'js/admin/colourizeColourselects';

and was a bit shocked to see "Installing js..." appear in my terminal, as I certainly didn't want to install anything.
Since anyone can upload basically anything to npm, and npm provides hooks such as postinstall, this is potentially dangerous. Could there be an additional check, just to see if the user actually meant to import a local module instead of blindly? Is js/admin/colourizeColourselects even a valid name for a package?

Better if provided an option in the command prompt before install is this the package that you want to install 'example-package' if yes from the user then go with installation if not comple without installing that package.

 I'm also pretty sure that Webpack doesn't allow prompting while running a watcher.
I stopped using this package shortly after writing this issue since it's just too big of a security risk. It should check for local paths before installing anything.


Restructuring dependencies

Resolves #24 Reevaluate production vs. peer/dev dependencies
Resolves #34 Constrain typing of randomValue function to object | array

[x] Related GitHub issue(s) linked in PR description
[x] Destination branch merged, built and tested with your changes
[x] Code formatted and follows best practices and patterns
[x] Code builds cleanly (no additional warnings or errors)
[-] Manually tested
[x] Automated tests are passing
[x] No decreases in automated test coverage
[-] Documentation updated (readme, docs, comments, etc.)
[-] Localization: No hard-coded error messages in code files (minimally in string constants)

@all-contributors add @jhugs for review

@all-contributors add @jstevenson for review


gitea,nixos/gitea: add SuperSandro2000 as maintainer, remove inactive…

… maintainers, unify maintainers
Description of changes
For the last months @ma27 and me basically already maintained gitea, so we might as well write that into code.
fyi @srhb @pyrox0 @disassembler @kolaente
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Sure thing :)


Feature Flaw: Find

When searching for tags, duplicated prefix are connected using AND
Perhaps tags can use OR relationship, so that users can see everything related to one tag, but seach for multiple tags at once
Understand that the implementation will take alot of effort since all existing prefixes are connected using AND,
could list this feature as a potential implementation in the future
Could also explain in DG and UG why AND is used between prefixes instead of OR

This issue has been resolved


Feature/deposit bot

Added automatic deposit bot functionality

Pull Request Test Coverage Report for Build 975

86 of 105 (81.9%)  changed or added relevant lines in 4 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.2%) to 50.913%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/helpers/dateUtil.js
0
2
0.0%

src/runBots.js
0
6
0.0%

src/bots/DepositBot.js
81
92
88.04%

Files with Coverage Reduction
New Missed Lines
%

src/runBots.js
1
0.0%

Totals

Change from base Build 958:
1.2%

Covered Lines:
1732

Relevant Lines:
3180

💛  - Coveralls


Install OSX

I can't for the life of me get OpenMP working on the CCPi-Framework on OSX. I tried a Conda install as per the README.md:
conda create -y --name cil python=3 cvxpy ccpi-framework=19.10 ccpi-astra=19.10 tomophantom ccpi-plugins=19.10 -c oxfordcontrol -c conda-forge -c astra-toolbox -c ccpi

but got the following error:
Collecting package metadata (current_repodata.json): done
Solving environment: failed with repodata from current_repodata.json, will retry with next repodata source.
Collecting package metadata (repodata.json): done
Solving environment: failed

PackagesNotFoundError: The following packages are not available from current channels:

  - tomophantom
  - ccpi-plugins=19.10

Removing extra packages:
conda create -y --name cil python=3 cvxpy ccpi-framework=19.10 -c oxfordcontrol -c conda-forge -c astra-toolbox -c ccpi

then activate and use:
conda activate cil
python
>>> from ccpi.optimisation.algorithms import PDHG
[1]    21786 segmentation fault  python

@rijobro have you fixed this? tomophantom and ccpi-plugins are not available on the conda channel.

tomophantom and ccpi-plugins are not available on the conda channel.

Ok, you should probably update your README accordingly then.
Yes, closing as with a few PRs we can now compile/install with CMake.


Clarify stateful #dockter comment

The README says:
Dockter does this by looking for a special # dockter comment in a Dockerfile. Instead of throwing away layers, it executes all instructions after this comment in the same layer - thus reusing packages that were previously installed.

This explanation was a bit difficult to understand, as I thought it was an alternate implementation of the --squash option. That would create a new single layer in which to execute all images, but a new one for each build. This instead will reuse an existing layer, overwriting the contents.
Some thoughts to polish this feature:

# dockter does not convey the meaning of this, a more specific annotation like # dockter: stateful could make it easier to search for documentation about this option
clarify in README this is going to overwrite an existing, single layer
clarify where this state is kept, in the Docker daemon's state files, I guess
clarify rolling back to a previous version of the dependencies will be slower as the trade-off, having to modify the existing, single layer

@giorgiosironi : thanks for the great suggestions! I've added this issue to the 1.0.0 milestone.


Anonymous classes trigger errors with version 0.12.11

Since the release of version 0.12.11 we're now seeing errors that look like:
  ------ ------------------------------------------------------------------------------
  Line   modules/contrib/localgov_directories/src/Plugin/Block/ChannelSearchBlock.php
 ------ ------------------------------------------------------------------------------
  87     Class
         Drupal\localgov_directories\Plugin\Block\AnonymousClass01a859945ded72
         cee689e1589045065e was not found while trying to analyse it -
         discovering symbols is probably not configured properly.
         💡 Learn more at https://phpstan.org/user-guide/discovering-symbols
 ------ ------------------------------------------------------------------------------

An example of the code that triggers this error can be seen here: https://github.com/localgovdrupal/localgov_directories/blob/7877f4c15e164b8524df30a0db343865c3621d5e/src/Plugin/Block/ChannelSearchBlock.php#L87
We don't get these errors when pinning phpstan-drupal to version 0.12.10. We're using phpstan 0.12.92.
Are there changes in Drupal coding standards for this or is it an error?

😱  well, that's odd. I don't know why anonymous classes would be causing autoload errors. PHPStan supports anonymous classes.

I'm trying to discern what could have changed in 0.12.11 to cause this: https://github.com/mglaman/phpstan-drupal/compare/0.12.10...0.12.11
Is it because we bumped the minimum phpstan?
-        "phpstan/phpstan": "^0.12.64",
+        "phpstan/phpstan": "^0.12.65",

Or maybe the new PluginAnnotationContextDefinitionsRule and ConfigEntityConfigExportRule break anonymous classes.

Shoot, yep.
        $className = $node->name->name;
        $namespace = $scope->getNamespace();
        $reflection = $this->reflectionProvider->getClass($namespace . '\\' . $className);
        $implementsExpectedInterface = $reflection->implementsInterface($this->getExpectedInterface());
        if (!$implementsExpectedInterface) {
            return [];
        }

It tries to perform a reflection look up without making sure it isn't anonymous.  I thought the following check would be enough
        if ($node->name === null) {
            return [];
        }

While reading this issue, https://github.com/nikic/PHP-Parser/issues/554, I found out why the name === null didn't work: PHPStan gives it a name.

A new release will be rolling out shortly! Thanks for letting me know about the bug

Release is out! Thanks for your patience: https://github.com/mglaman/phpstan-drupal/releases/tag/0.12.12

@mglaman Many thanks for the quick fix on this.


Test for approximate equality, instead of exact.

Closes #335

Pull Request Description
What issue does this change request address?
#335
What are the significant changes in functionality due to this change request?
Categorical works if the total probability is 1.0000000000000002

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[x] 1. Review all computer code.
[x] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[x] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[x] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[x] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[x] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[x] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[x] 8. If an analytic test is changed/added is the the analytic documentation updated/added?

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code. ok
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts). n/a
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details. ok
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass. ok
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True. ok
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync. n/a
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done. ok
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added? n/a

PR can be merged....


Intermittent TCWorkerCommManagerTest>TCTestCase.runBare:196->testWorkerCommDistributionAfterReconnect:377 Expected <2> but got <3>

This is seen very rarely:
-------------------------------------------------------------------------------
Test set: com.tc.net.core.TCWorkerCommManagerTest
-------------------------------------------------------------------------------
Tests run: 4, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 5.256 sec <<< FAILURE! - in com.tc.net.core.TCWorkerCommManagerTest
testWorkerCommDistributionAfterReconnect(com.tc.net.core.TCWorkerCommManagerTest)  Time elapsed: 3.436 sec  <<< FAILURE!
com.tc.util.TCAssertionError: Expected <2> but got <3>
        at com.tc.net.core.TCWorkerCommManagerTest.testWorkerCommDistributionAfterReconnect(TCWorkerCommManagerTest.java:377)

My initial suspicion is that this is either a race between the connection threads and the test thread (since the test is multi-threaded) or that some environmental issue caused a connection to not close when the test expected.
Note that there is a timeout in the connection which must be disabled in order to debug this:  ClientMessageTransport.java:277

I think that the problem is a race to assign connections to worker threads.  If this is true, it probably isn't a problem (we could fix it but that would add some overhead to the connection path - probably not worth the benefit of a more perfect thread balance) and we likely just need to change the test to verify that all the connections are accounted for, not necessarily perfectly balanced across threads (or change the test to connect them serially).
Since this is intermittent, I will augment the test with more information and proceed with other matters.  That way, we can pick this up with more information, when it is next observed.

While preparing this change for PR, I hit the bug again.  The weights were 3, 1, 2 instead of the expected 2, 2, 2.  This does mean that we have the correct total number of connections but that they are distributed incorrectly.  This would be what we see in response to the race condition I suggested, above.
I will look into proving whether or not that can happen, as an explanation.

Timebombed the test until Feb 1.  This code needs to be reworked.

Covered by #425.


online vs offline

Hi, I'm testing the script on a single sensor, your default listed sensors in sensors {} are still in place, with one changed to my actual sensor.
I'm using MQTT explorer (on my PC) to monitor my mosquito queue which is running on Home Assistant.
The sensor and scripts are running on a raspberrypi.
On the first run, the online/offline status is correct in that your default sensors are showing as offline. My sensor is showing as online.
However, when I end the script - using Ctrl+c on the command line, I would expect my sensor to show the state as offline.
Is this what you expect also?

That's not quite how it works unfortunately. It's the script that reports the sensors as available. or unavailable, based on the IDs that respond to get_available_sensors().
So if you kill the script, it no longer reports the availability. If you want it to report as unavailable if the script is killed; you can utilize the $CONNECTED topic as availability topic instead. It doesn't have to be named $CONNECTED, that is just what I use.

thanks - i'll have a play, need to figure out a way to change status for only those devices on this pi - eventually i aim to have this running on 2-3 pi's so don't want the ones that do not have a particular sensor to mark others as offline when they are in the sensor array!


config shell & bt shell same time cause fail

Hi,pals:
I find eable the config same time, will fail:
CONFIG_SHELL=y

CONFIG_NET_SHELL=y
CONFIG_BT_SHELL=y
CONFIG_NET_L2_BT_SHELL=y

the log:
-- Generating zephyr/include/generated/generated_dts_board.h
-- Cache files will be written to: /home/frank/.cache/zephyr
CMake Error at /home/frank/work/project/zephyr/cmake/extensions.cmake:796 (add_subdirectory):
  The binary directory

    /home/frank/work/project/nrf52_moderate/apps/ipsptest/build/zephyr/subsys/shell

  is already used to build a source directory.  It cannot be used to build
  source directory

    /home/frank/work/project/zephyr/subsys/shell

  Specify a unique binary directory name.
Call Stack (most recent call first):
  /home/frank/work/project/zephyr/subsys/CMakeLists.txt:7 (add_subdirectory_ifdef)

if disable CONFIG_SHELL or other config will pass.
The means bt shell cmd and shell history function can't co-exist. This is a issue or sepc？Can support bt shell cmd using history function? Thanks !

I digged in a bit more.
New shell has different design compering to the old one. We are in the middle of transition so it may not still not work.
Current concept is that you can enable shell commands via Kconfig for NET, BT and so on. But shell instance itself needs to be created separately.

@jarz-nordic Yes, I tested look like you。 BT_SHELL is old shell style , but CONFIG_SHELL is new shell style. Is there a plan covert BT&Net shell to new shell style?

@lgl88911 : yes of course. It is ongoing activit. I hope that in 2 weeks we will be able to do it.
Once PR: #10511 is merged it will be very easy task.

@jarz-nordic  Thanks, I‘ll wait your news

@lgl88911 : Please track PR #9825. Once it will be merged you shall be able to use new shell.

@lgl88911 : Please check now if it is working as you expect it.

@jarz-nordic Update code & test OK, Thanks a lot！
uart:~$    bt       clear    gatt     help     history  kernel   l2cap    log   net      net_bt   resize   shell    ticker


Updated Vagrantfile to work with the supported Mesos and Hadoop versi…

…ons.
Broke provisioning into separate scripts in the ./vagrant directory.
Updated vagrant documentation
Added .npmrc that eliminates node symbolic links so build will work under Vagrant.

Thanks @klucar . Would you please add the Apache headers to the new files?
I'll review some more in the mean time.

Ship it! :rocket:


Issue when using AddApiExplorer and JsonTranscoding

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug

Hey guys, i'm having a problem with a project i'm working on and i wanted to see if i'm missing anything or maybe i've stumbled on a bug.
I have created a minimal project where i'm able to reproduce the behaviour, I'm goint to try to guide you pasting snippets of code here.
When i run the program, and try to hit the endpoint v2 with postman everything goes as expected.

But, instead, if i call v1, which should be responded by the jsontranscode, i get 404.

I've tracked this down to the call of AddApiExplorer. If i remove that call i'm able to get the v1 as i should.
What do you think about this? Is this the expected behaviour? Am i doing something wrong? Is this a bug? If it is, is a bug in the jsontranscoding or asp.versioning library?
Thank you so much for your time and sorry the lenght of this post. Let me know if you need anything else that could help you answering this.
Expected Behavior
Endpoint is found and executed.
Steps To Reproduce
Program.cs
static void Main(string[] args)
 {
     var app = CreateHostBuilder(args).Build();

     app.UseRouting();

     app.MapControllers();

     app.MapGrpcService<GreeterService>();

     app.Run();
 }

 private static WebApplicationBuilder CreateHostBuilder(string[] args)
 {
     var builder = WebApplication.CreateBuilder(args);

     builder
         .WebHost.ConfigureKestrel((context, options) =>
         {
             options.ListenLocalhost(8080);
         });

     builder.Services.AddControllers();

     builder.Services.AddApiVersioning(options =>
          {
              options.ReportApiVersions = true;
          })
          .AddApiExplorer(options =>
          {
              options.GroupNameFormat = "'v'VVV";
              options.SubstituteApiVersionInUrl = true;
          });

     builder.Services
         .AddGrpc()
         .AddJsonTranscoding();

     return builder;
 }

V2Controller.cs
[ApiController]
[ApiVersion("2.0")]
[Produces("application/json")]
[Route("api/v{version:apiVersion}/controller/{param}")]
public class V2Controller : Controller
{
    [HttpGet]
    [Route("route")]
    public async Task<IActionResult> GetEndpoint()
    {
        return Ok("ok get v2");
    }
}

And greet.proto (This is the wiki example as you can see, just added the route i need to demostrate my point)
syntax = "proto3";

option csharp_namespace = "TestVersioningGRPC.Proto";
import "google/api/annotations.proto";

package greet;

service Greeter {
  rpc SayHello (HelloRequest) returns (HelloReply) {
    option (google.api.http) = {
      get: "/api/v1/controller/{name}/route"
    };
  }
}

// The request message containing the user's name.
message HelloRequest {
  string name = 1;
}

message HelloReply {
  string message = 1;
}

Exceptions (if any)
No response
.NET Version
8.0.304
Anything else?
No response

I would be very surprised that anything about AddApiExplorer caused this behavior. It doesn't change any routing behavior whatsoever. It's purely discovery. I can't say I have much firsthand experience with JSON transcoding with gRPC, but there are some incongruencies I see that don't make a lot of sense.

Why use Controller (usually full MVC with views) instead of ControllerBase
Why are there two different routes to GetEndpoint?
Why does V2 have a route parameters with no parameter?
a. I don't even see how this can work since the parameter is required

I would have expected 2.0 to look something like:
[ApiController]
[ApiVersion(2.0)]
[Produces("application/json")]
[Route("api/v{version:apiVersion}/controller")]
public class V2Controller : Controller
{
    [HttpGet("{param}/route""]
    public IActionResult GetEndpoint(string param) => Ok("ok get v2");
}

You can include ApiVersion as parameter if you don't want to hardcode v2

I'm not sure I understand exactly how ProtoBuf matches up to a controller. Is that code-generated?
I suspect that the next problem is a mismatch in names. API Version collation is performed on logic name, not route template. Route template may seem intuitive, but there are many nuances. order/{id} and order/{id:int} are not identical, but are semantically equivalent. Should order/{id}/items be part of the Orders or LIne Items (if there is one) API?
By default, ASP.NET Core uses the suffix convention Controller. If this present in the type name, it is removed. A type named Controller will end up as "" and a type named V2Controller will end up as "V2". Unless you otherwise configure it (and there are a few ways), this will be the logical name of the controller and API. These must match for collation to work. If the types are in different namespaces, then it's easy to retain the same name. If they aren't, then they need distinct names because it is enforced by the type system. In this case, API Versioning adds a second (but configurable) convention that supports the format <Name>[V]#Controller. As such, OrderController and OrderV2Controller will both have the logical name "Order" and be collated. There are some edge cases where you might not want that; for example S3Controller.
"" and "V2" are not the same so they will not be collated. You are also versioning by URL segment. By doing so, you have violated the Uniform Interface REST constraint, which makes the API version part of the resource identifier. Since 1.0 is not collated into the set of API versions, the identifier cannot exist, therefore 404 is returned in the response. If you had AddProblemDetails enabled, you would probably see additional details indicating that the API version doesn't match/exist.
My recommendation for your next steps is to:

Use a more specific name; Controller is too generic
a. If this is not possible due to backward compatibility, then apply [ControllerName("controller")] to V2Controller
Make sure your controller implementations correct and complete
If you're not using OpenAPI, don't both with AddApiExplorer; just use AddMvc

Hi, thanks for taking your time to answer me. This is just an example i've put together just to show that is reproducible. I created the same issue in the grpc repo but i havent heard from them. (See: https://github.com/grpc/grpc-dotnet/issues/2500)
I can upload a zip with the code i posted above so you can check that commenting out that line shows the described behaviour. I appreciate your feedback though.

I followed your suggestions and updated the project i wrote. I got the same behaviour. It would be great if we, at least, can rule out the versioning library as the problem.
TestVersioningGRPC.zip


removed incompatible cap from readme

Removed timezone cap from README since its not supported.

@googlebot I signed it!


Dead link in docs

https://orm.drizzle.team/kit-docs/commands#prototype--push

Extended article on Turso and drizzle-kit push — read here.

The link to this extended article - https://driz.li/sqlite-push - appears to be dead.

Hello @sbbu. Thank you, it was fixed


Add electron app and build environment [WIP]

Add electon packages
Add electron-main.js as hook
Add electron-main.ts config file

will automatically be compiled to .js

Add electron-builder to create packages
Add electron-builder.json for a maintainable build config
Heavily optimize builder for angular projects
Add build instructions to development.rst
converts OpenSlides as optional electron application
server switcher on the start page

enter some URL, makes it more pretty automatically
if the URL changed, reboot and OpenSlides and recall login info
so the theme should directly apprear on the startpage

always open links in new tabs
resolve some django cookie problems to send valid requests as third
party application
Custom electron menu elements

Most importantly, "force reload" will wipe the local storrage

Safe the last used server in indexDB

this was necessary to have reloading work
re-opening the application will enter the last used server for us
(but no automatic login)

"help" menu will forward to openslides.com and create issues
use OpenSlides default theme if no other was found durgin login

no more white login page ever

map file lookup to URL to call mediafiles and projector
communicate between angular and electron using IPC

BUGS:

custom logo does not yet show

TODO:

dynamic updates of OpenSlides in Electron
dynamic updates of Electron itself

Don't expect to build for mac. Mac Apps can only be signed on a macOS machine

same goes for windows store appliactions, but regular portable windows works using wine

The OpenSlides Client needs to be configured to change the server to send and get requests to (and from) in runtime ( any hints @FinnStutzenstein ? )
Client updates might work using this:

https://www.electronjs.org/docs/api/auto-updater
https://www.electron.build/auto-update

There is an extra repo for electron: https://github.com/OpenSlides/openslides-electron
So, we can close this PR.


cors does not work on localhost

I use the demo code as shown below. Then I open the website of "http://localhost:3000/" in my Chrome. it should not get a message of "This is CORS-enabled for only example.com." since it's not within the origin list. Am I wrong?
var express = require('express')
  , cors = require('cors')
  , app = express();

var corsOptions = {
  origin: 'http://example.com'
};

app.get('/', cors(corsOptions), function(req, res, next){
  res.json({msg: 'This is CORS-enabled for only example.com.'});
});

app.listen(3000, function(){
  console.log('CORS-enabled web server listening on port 80');
});

Are you making the request from another domain. CORS is for sharing resources across domains, it doesn't block access to the page your currently on or block people from looking at it. It blocks HTTP requests from another page to your own resources. You can read more about CORS on https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS.

Thanks, i made mistakes on basic concept


thread 'main' panicked at 'no entry found for key', src\tools\cargo\src/cargo\core\compiler\unit_dependencies.rs

As you submitted the issue earlier，I don't know how to solve this problem.

Could you clarify what you're trying to do and how this is related to the bootloader crate?

I am now migrating from bootloader v0.9.X to bootloader v0.11. I follow the guide and  use the build script to build kernel.But ran into the problem I mentioned above.

In the root package, I created a config.toml
[unstable]
bindeps = true
build-std-features = ["compiler-builtins-mem"]
build-std = ["core", "compiler_builtins", "alloc"]

[build]
target = "x86_64-simple_os.json"

In the root package, the Cargo.toml is
[package]
name = "simple_os"
version = "0.1.0"
edition = "2021"

[build-dependencies]
bootloader = "0.11"
kernel = { path = "kernel", artifact = "bin", target = "x86_64-unknown-none"}

[workspace]
members = ["kernel"]

In the kernel package, I created a config.toml as well
[unstable]
build-std-features = ["compiler-builtins-mem"]
build-std = ["core", "compiler_builtins", "alloc"]

[build]
target = "x86_64-simple_os.json"

The problem is that when I execute command Cargo build, the error occurs.

By the way, I've tried leaving only bindeps option in the root package's config.toml.Then the error would be

Thanks for the information! You don't need the custom target file and build-std config keep keys anymore.

By the way, I've tried leaving only bindeps option in the root package's config.toml.Then the error would be

That's the right approach. To fix that error, run rustup target add x86_64-unknown-none.

I've added target x86_64-unknown-none. But the error is still not resolved

Try removing the build.target key from your Cargo.toml. And make sure that your kernel is declared as no_std.

Thanks, now I can successfully build and run the os. But I have another problem that with BIOSbooting, the println macro doesn't seem to work while with UEFI booting, it does work

But I have another problem that with BIOSbooting, the println macro doesn't seem to work while with UEFI booting, it does work.

What does your println macro do? Be aware that the v0.11 version of the bootloader sets up a pixel-based framebuffer, so the VGA text mode won't work anymore (it's not compatible with UEFI anyway). See https://github.com/rust-osdev/bootloader/blob/main/docs/migration/v0.9.md for details.

We also set up a framebuffer for BIOS booting so that you don't need to special-case your output code depending on the boot mode.
(The screenshot you posted above just shows the output from the bootloader, no kernel output yet.)

My output is based on framebuffer and doesn't change anything else. Completely consistent code applied to UEFI booting is correct, while BIOS booting is wrong


[WIP] [Swift 2.2] Enable ios host tests

This change allows to easily run iOS tests that require only running the compiler, and don't require running the generated code.

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test


Blueprint does not work for watermeter

I am using the water-only version of S0 Tool, installed through the button here. In generates this sensor: sensor.s0tool_watermeter_versie
The Blueprint referenced on this  page compares another sensor:
- condition: template
    value_template: |-
      {{
                states('sensor.s0tool_versie_github')
                !=
                states('sensor.s0tool_versie')
              }}

Are you able to create a separate blueprint for the watermeter?

when update v22.11.24 is getting online.
i think with a couple off days.
u can use the old blueprint again.
i changed te name , so i don't need more blueprints.


[Snyk] Upgrade @types/react from 18.2.21 to 18.2.33

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/react from 18.2.21 to 18.2.33.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 12 versions ahead of your current version.
The recommended version was released 21 days ago, on 2023-10-26.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

Comparison is base (623ea38) 99.74% compared to head (665cc1b) 99.74%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #272   +/-   ##
=======================================
  Coverage   99.74%   99.74%
=======================================
  Files         573      573
  Lines        9504     9504
  Branches     2076     2076
=======================================
  Hits         9480     9480
  Misses         22       22
  Partials        2        2

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


input, textarea, contenteditable double tap on text doesn't select it on ios safari

When I activate fastclick, input, textarea and contenteditable double tap on text doesn't select it on ios safari. The behavior is quite weird on contenteditable, it focus and unfocus on double tap. On input and textarea, it does nothing.
If I disable fastclick, it works fine, text is selected and it's possible to copy, paste, change style...
Thanks for your help.

Was this issue fixed? I see that there was a successfully merged pull request.
I have a contentEditable that doesn't work on iPhone 6 Safari (can't click and activate it), but when I remove fastclick library, it works. Anyone know how I can fix this?

I came across this thread after experiencing the issue with my contenteditable editor and I'm a little confused — are the needsclick and/or needsfocus classes described in the README supposed to remedy this? I've tried them in every combination and they don't seem to be doing anything for me, using 1.0.6. Are pull requests like #211 and #381 still open because they're still needed?

One quick work around is to modify the FastClick.prototype.needsFocus function to check for div elements that have "contentEditable" set to true.

There are a couple of problems with Donovan's patch above:

If not contenteditable, div will fall-through to the input case!
you may have trouble if you use clicks on divs in other contexts (I did)

I am using this with bootstrap-expandable-input.js. So, YMMV if using your own code or some other plugin.
I tried making-up my own tag (expandableinput) but had trouble with that. The tag seems to have to be a "natural" block element. (At least for bootstrap-expandable-input)
I settled on using <blockquote> which is at least somewhat semantic.
    case 'blockquote':   // Note HTML5 rules permit only alphanumeric characters in tag names - no - or _ etc.
      if (target.contentEditable) {  // Really only needed if we don't use our own unique tag
        return !target.disabled && !target.readOnly;
      } else {
        return (/\bneedsfocus\b/).test(target.className);  // really only needed if we don't use our own unique tag
      }


Add stylelint config file

ℹ Related to this PR: https://github.com/dcos/dcos-ui/pull/1122

This PR introduces the and stylelint configuration package that will be consumed by other DC/OS frontend projects.

The style changes was reviewed here: https://github.com/dcos/dcos-ui/pull/1092


Unable to use Boruta-Server through a reverse proxy due to X-Frame-Options header

Dear Maintainer(s),
I've tried to run boruta-server using the system-wide setup, behind a reverse proxy (because non-standard ports are most often blocked on corporate firewalls/proxies), and I've run with an issue when trying to authenticate on the admin interface.
For reference, here is my .env.production file:
MIX_ENV=prod
SECRET_KEY_BASE=<stripped sensitive data>
POSTGRES_USER=boruta
POSTGRES_PASSWORD=<stripped sensitive data>
POSTGRES_DATABASE=boruta_release
POSTGRES_HOST=localhost
POOL_SIZE=5
MAX_LOG_RETENTION_DAYS=60
BORUTA_ADMIN_OAUTH_CLIENT_ID=5905075e-8c62-4b53-9f19-154b0ef2b0e0
BORUTA_ADMIN_OAUTH_CLIENT_SECRET=<stripped sensitive data>
BORUTA_ADMIN_OAUTH_BASE_URL=https://login.example.com
<EMAIL_ADDRESS>BORUTA_ADMIN_PASSWORD=<stripped sensitive data>
BORUTA_ADMIN_HOST=localhost
BORUTA_ADMIN_PORT=8181
BORUTA_ADMIN_BASE_URL=https://adm.login.example.com
BORUTA_OAUTH_HOST=localhost
BORUTA_OAUTH_PORT=8180
BORUTA_OAUTH_BASE_URL=https://login.example.com
BORUTA_GATEWAY_PORT=8182
BORUTA_GATEWAY_SIDECAR_PORT=8183
BORUTA_GATEWAY_CONFIGURATION_PATH=config/example-gateway-configuration.yml

The reverse proxy is configured to forward any requests on https://adm.login.example.com to https://localhost:8181, and on https://login.example.com to https://localhost:8180.
When trying to access the web page in this configuration, the returned web page is blank, and the developper console reports the web browser is refusing to load the iframe content (pointing to https://login.example.com, AKA the OAuth port), because of the X-Frame-Options: SAMEORIGIN header (cross-tested on Mozilla Firefox and on Google Chrome).
The running version of boruta-server is 0.3.0.
Shouldn't we rely on a CORS header for this?
Kind regards,

Hi,
Thank you for your interest in boruta.
Can you have a look to the systemd logs? The database may have not been seeded properly.
sudo journalctl -u boruta.service

Hi,
Here is the application log:
Jun 06 11:33:13 arwen systemd[1]: Starting boruta.service - Boruta server...
Jun 06 11:33:13 arwen boruta[2396038]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Jun 06 11:33:15 arwen boruta[2396038]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Jun 06 11:33:15 arwen boruta[2396038]: 2024-06-06T11:33:15.824Z [info] Migrations already up
Jun 06 11:33:15 arwen boruta[2396038]: 2024-06-06T11:33:15.861Z [info] Migrations already up
Jun 06 11:33:15 arwen boruta[2396038]: 2024-06-06T11:33:15.880Z [info] Migrations already up
Jun 06 11:33:16 arwen boruta[2396038]: [lib/boruta_auth/application.ex:20: BorutaAuth.Application.setup_database/0]
Jun 06 11:33:16 arwen boruta[2396038]: repo.__adapter__.storage_up(repo.config) #=> {:error, :already_up}
Jun 06 11:33:16 arwen boruta[2396038]: 2024-06-06T11:33:16.642Z [info] Migrations already up
Jun 06 11:33:16 arwen boruta[2396038]: 2024-06-06T11:33:16.644Z [info] Migrations already up
Jun 06 11:33:16 arwen boruta[2396038]: 2024-06-06T11:33:16.669Z [info] Running BorutaWeb.Endpoint with cowboy 2.10.0 at <IP_ADDRESS>:8180 (http)
Jun 06 11:33:16 arwen boruta[2396038]: 2024-06-06T11:33:16.682Z [info] Access BorutaWeb.Endpoint at https://localhost:8180
Jun 06 11:33:17 arwen boruta[2396135]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Jun 06 11:33:18 arwen boruta[2396135]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Jun 06 11:33:18 arwen boruta[2396135]: 2024-06-06T11:33:18.611Z [info] Migrations already up
Jun 06 11:33:18 arwen boruta[2396135]: 2024-06-06T11:33:18.622Z [info] Migrations already up
Jun 06 11:33:18 arwen systemd[1]: Started boruta.service - Boruta server.
Jun 06 11:33:20 arwen boruta[2396212]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8. Please ensure your locale is set to UTF-8 (which can be verified by running "locale" in your shell)
Jun 06 11:33:27 arwen boruta[2396212]: warning: the VM is running with native name encoding of latin1 which may cause Elixir to malfunction as it expects utf8.
Jun 06 11:33:42 arwen boruta[2396212]: 2024-06-06T11:33:42.822Z request_id=F9ZnDy_a87s3jDkAAAEC [info] boruta_web GET /oauth/authorize - sent 401 in 9ms

Kind regards,

The administration interface performs an implicit grant.
Jun 06 11:33:42 arwen boruta[2396212]: 2024-06-06T11:33:42.819Z request_id=F9ZnDy_a87s3jDkAAAEC [info] boruta_web authorization authorize - failure client_id=5905075e-8c62-4b53-9f19-154b0ef2b0e0 status=unauthorized error=invalid_client error_description="Invalid client_id or redirect_uri."

It looks like the seeded client is not the same as the one you provided in the environment variables for the administration interface:
BORUTA_ADMIN_OAUTH_CLIENT_ID=5905075e-8c62-4b53-9f19-154b0ef2b0e0
BORUTA_ADMIN_OAUTH_CLIENT_SECRET=<stripped sensitive data>

Having a look to oauth_clients table into database, you should see the env variable configured client inserted, if not I think the best is to drop the database and restart the application.


Failed to access file (C:/ or D:/)

Whenever I am scanning C:\ or D:\ this error pops up. So, I ran a scan in documents it worked. Then, I tried running the scan in Program Files but then also it said "Failed to access file".  I also tried running with command prompt but then this error showed up - Unable to access jarfile scanner.jar

if your scanning the entire disk, you should open it as administrator, when running from command prompt, you will want to pass in the full path to scanner.jar eg: c:/users/me/downloads/scanner.ja

Thank you so much, it is working.

I don't know why but this morning I tried running the scanner again with the same instructions, but it doesn't seem to work (it worked yesterday when I followed those instructions).


fix oci\Schema::findConstraints() and pgsql\Schema::findTableNames()

Also added missing view to oci tests.
@samdark I left some bugs when rebasing my recent PRs, sorry about that. Should have run the tests.

Thanks :)


add gradle 8.7 compatibility via org.ysb33r.gradle:grolifant-herd:4.0.0

will resolve:

https://github.com/erdi/webdriver-binaries-gradle-plugin/issues/43
https://github.com/grails/grails-testing-support/issues/443

replaces:

https://github.com/jamesfredley/webdriver-binaries-gradle-plugin/pull/1

Changes were made to enable tests to run on Windows 11.
Fully tested against:

https://github.com/grails/grails-functional-tests
https://github.com/grails/grails-testing-support - also requires org.asciidoctor:asciidoctor-gradle-jvm from 4.0.1 to 4.0.3

@erdi @jonnybot0

Hey, @jamesfredley! Neither I nor the Geb project maintain this repository. For our part, we're actively working to move from this to Selenium Manager, which solves the same core problem. You can see the in-progress pull request for this at https://github.com/geb/geb/pull/200.
Even without that update to Geb, you can start using Selenium Manager by upgrading to Selenium 4.11 or later. It works just fine with the latest Geb release. I would recommend upgrading to the latest version of Selenium you can in your environment, as Selenium Manager did have a few bugs in the earlier iterations that are now resolved. That may entail Java updates, though, depending on your project.
Obviously, you can always use an artifact built from your fork here in the interim, and if @erdi wants to merge this here, more power to you both. :)

Thanks for the reply.
For Grails, we are moving the default Geb setup to use org.testcontainers:selenium via https://github.com/grails/geb/blob/7a50d0aa78671f41ce0237b98b501cf844208081/src/testFixtures/groovy/grails/plugin/geb/ContainerGebSpec.groovy#L49 which is working great.
This PR would help users that want to use Gradle 8.6+ with webdriver-binaries-gradle-plugin.


The current Dart SDK version is 3.3.4.

flutter pub add pay
exit code 65
Note: intl is pinned to version 0.18.1 by flutter_localizations from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.
The current Dart SDK version is 3.3.4.
Because pay <1.0.0 doesn't support null safety and pay >=1.0.0 depends on flutter_localizations from sdk, every version of pay requires flutter_localizations from sdk.
And because every version of flutter_localizations from sdk depends on intl 0.18.1, every version of pay requires intl 0.18.1.
So, because app depends on both intl ^0.19.0 and pay any, version solving failed.
The lower bound of "sdk: '>=2.10.0 <3.0.0'" must be 2.12.0 or higher to enable null safety.
For details, see https://dart.dev/null-safety

I have same problem

Hi @Pawankant87 and @macuser79, I just created a new project and added pay using the console and can't reproduce the error. Also the error logs don't seem to apply to the current version of the package, which uses sdk > 3.0.0 on all its subpackages.
Can you share more details about your environment, and steps to reproduce the issue?

Hi @Pawankant87 and @macuser79, I just created a new project and added pay using the console and can't reproduce the error. Also the error logs don't seem to apply to the current version of the package, which uses sdk > 3.0.0 on all its subpackages.
Can you share more details about your environment, and steps to reproduce the issue?

Hello.
I have this error:
Because pay >=1.0.0 depends on flutter_localizations from sdk which depends on intl 0.18.1, pay >=1.0.0 requires intl 0.18.1.
So, because myapp depends on both intl ^0.19.0 and pay ^2.0.0, version solving failed.
[✓] Flutter (Channel stable, 3.19.3, on macOS 14.4.1 23E224 darwin-arm64, locale en-IT) [✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0) [✓] Xcode - develop for iOS and macOS (Xcode 15.3) [✓] Chrome - develop for the web [✓] Android Studio (version 2023.3) [✓] IntelliJ IDEA Ultimate Edition (version 2024.1.1) [✓] VS Code (version 1.90.0-insider) [✓] Connected device (1 available)     ! Error: Browsing on the local area network for iPhone di Vincenzo. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.       The device must be opted into Developer Mode to connect wirelessly. (code -27)     ! Error: Browsing on the local area network for iPhone di Vincenzo. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.       The device must be opted into Developer Mode to connect wirelessly. (code -27) [✓] Network resources

facing same issue

facing same issue


Bot aren't reading messages? There's only one tick on them.

Since those last updates, I'm not able to get it to work... someone could make a wiki page about how to compile and get the whatsapi test exemple to work?
I really loved that API since I saw it, I actually did some tests code and I was able to receive and do commands, but since axolotl changes I can't even build it.
Can anyone help me to the right direction? How to compile, the dependences and get it to work?
Thanks in advance, sorry for my english and my questions... I'm not a experienced programmer, neither a english native speaker.

Actually, after a bit trickly, I can build it, but I'm not getting the messages, what's going on? I'm on an Android phone... I can see the presence and typing but onMessage event seems not triggering. Also, the double tick message doesn't appear on WhatsApp, so the bot isn't reading the message actually...

Hi, I'm having the same issue. OnGetMessage is not triggering.

Same problem for me...

I was having the same issues until i debugged deep:#178


[jest-each] Refactor logic

Summary
This PR refactors the jest-each library to use the shared bind logic integral to Jest's core.
This way we have only one implementation and the changes propagate to all instances of the each logic
Test plan
Tests should still pass

Nice cleanup!

Codecov Report

Merging #6345 into master will decrease coverage by 0.26%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #6345      +/-   ##
==========================================
- Coverage   63.89%   63.63%   -0.27%
==========================================
  Files         228      226       -2
  Lines        8705     8648      -57
  Branches        4        4
==========================================
- Hits         5562     5503      -59
- Misses       3142     3144       +2
  Partials        1        1

Impacted Files
Coverage Δ

packages/jest-each/src/index.js
77.77% <100%> (+7.18%)
:arrow_up:

packages/jest-each/src/bind.js
100% <0%> (+2.12%)
:arrow_up:

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 c13dd2f...689378b. Read the comment docs.


[file_selector] Migrate FileSelectorApi to Dart

This PR includes the changes in #6518 and in #6539.
This exposes a new FileSelectorApi that will replace the one generated by pigeon.
This will be integrated with the next PR.
Things we have done:

Add FileSelectorApi
Add SelectionOptions class
Add DialogWrapperFactory for testability

Issue: [file_selector_windows] Migrate CPP implementation to Dart. #112212
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[ ] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[ ] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Closing this because it is opened in the main repository
[file_selector] Migrate FileSelectorApi to Dart #6546


Aggregate data automaticly

Feature Request
Proposal:
Aggregate data automaticly like zabbix.
Desired behavior:
I write some metrics to influxdb in every 1s. I hope 1h later influxdb can  delete some metrics and save metrics that interval by 1m.
Use case:
System monitor

I think that's exactly what continuous query and retention policy does, you can take a look

@marchtea  I had solved my problem by continuous query and retention policy, thank you


Why wandb can only show the first step log after second step?

Description
I log the loss, metrics and video every epoch step.
I found that wandb will not show up the first step log immediately, it always show up after the second step. This makes the debug process longer, cause I always need to wait two epochs.

Hi Yumin, thanks for letting me know about this. How have you instrumented your code? Could you share a sample script so we can reproduce this behavior?

Hi Yumin, thanks for letting me know about this. How have you instrumented your code? Could you share a sample script so we can reproduce this behavior?

def train(self):
        """
        Full training logic
        """
        for epoch in range(self.start_epoch, self.epochs + 1):
            val_log = self._valid_epoch()
            train_log = self._train_epoch()
            result = {**train_log, **val_log}

            # Log loss and metrics to wandb
            for key, value in result.items():
                if key == 'train_metrics':
                    for m in self.metrics:
                        wandb.log({'train_' +m: result['train_metrics'][m]}, step=epoch)
                elif key == 'val_metrics':
                    for m in self.metrics:
                        wandb.log({'val_' + m: result['val_metrics'][m]}, step=epoch)
                else:
                    wandb.log({key:value}, step=epoch)

            # Log video to wandb
            with torch.no_grad():
               # some procedure to generate movie
                wandb.log({"train_movie": wandb.Video(train_movie, fps=0.5, format="gif")},step= epoch)
                wandb.log({"valid_movie": wandb.Video(valid_movie, fps=0.5, format="gif")},step= epoch)

            # save model

Hey @Yumin-Sun-00, you can add commit=True to your last call to wandb.log.  By default we only commit the data when the step changes.  By specifying commit, you're saying you're not going to add anymore keys to that step.

Hey @Yumin-Sun-00
In the past year we've majorly reworked the CLI and UI for Weights & Biases. We're closing issues older than 6 months. Please comment to reopen.


Can't verify provider when the consumer uses git sha in the application number version

Following the best practices in the Participant version numbers section, our consumer is specifying it's version using the semver+sha notation, such as 1.0.36+f8d4be53.
However, after following the workshop, our provider test with @PactBroker annotation fails with No HAL document found at path 'http://my-broker.com/pacts/provider/my-provider/consumer/my-consumer/version/1.0.36 f8d4be53'
I believe the problem is the missing + sign in the URL (there's a blank space instead), because by copying the URL into the browser and adding the plus sign, it resolves with no problem.
Seems like this is a library problem unless I'm missing something. Thanks in advance

Is this not an issue anymore?


Fix redirect support for pipe method.

Fixed redirect support for pipe method, I've stumbled on this issue for a while.
Max redirects can be set with .redirects(n), just be the same with 'end' method.
Hope this helps.  :smile:

Fixes #178.


Feature/ssm: Add gov cloud region us-gov-east-1 and us-gov-west-1

This PR adds the SSM AWS Linux resources available in two not publicly available gov cloud regions - us-gov-east-1 and us-gov-west-1.
The data was provided by a customer who preferred not to share the original AMI values; hence, these values are randomly generated and differ from the official AWS values.

Hi Bert,
That's a very valid point, I think for now I also don't have a suggestion on how to automate this to be able to guarantee updates. I will merge it into our LocalStack fork of moto for now move forward, and keep you update if I get access to gov-cloud so we can have a more automated way.
Let's close this for now until we have a more satisfactory approach.


DPC_WATCHDOG_VIOLATION (133)  SINGLE_DPC_TIMEOUT_EXCEEDED

Hi there,
I  am hoping some can either help or educate me on how to diagnose or troubleshooting a constant BSOD that keeps happening and im not sure why.
I have updated all the relevant drivers for device ID shown in the .dmp file,but it keeps happening,
Ant advise would be appreciated
I have uploaded the .dmp files in a zip file below.
Microsoft (R) Windows Debugger Version 10.0.19041.1 AMD64
Copyright (c) Microsoft Corporation. All rights reserved.
Loading Dump File [\corp.well.co.uk\Data\UserData\ben.hirst\Desktop\Minidumps\200425\070720-5812-01.dmp]
Mini Kernel Dump File: Only registers and stack trace are available
************* Path validation summary **************
Response                         Time (ms)     Location
Deferred                                       SRVC:\SymCachehttp://msdl.microsoft.com/download/symbols
Symbol search path is: SRVC:\SymCachehttp://msdl.microsoft.com/download/symbols
Executable search path is:
Windows 10 Kernel Version 17134 MP (4 procs) Free x64
Product: WinNt, suite: TerminalServer SingleUserTS
Built by: 17134.1.amd64fre.rs4_release.180410-1804
Machine Name:
Kernel base = 0xfffff80034497000 PsLoadedModuleList = 0xfffff80034843ce0
Debug session time: Tue Jul  7 15:04:04.606 2020 (UTC + 1:00)
System Uptime: 0 days 3:00:29.829
Loading Kernel Symbols
...............................................................
................................................................
.....................................................
Loading User Symbols
Loading unloaded module list
...........
For analysis of this file, run !analyze -v
0: kd> !analyze -v
ERROR: FindPlugIns 8007007b

                                                                        *

                   Bugcheck Analysis                                    *

                                                                        *

DPC_WATCHDOG_VIOLATION (133)
The DPC watchdog detected a prolonged run time at an IRQL of DISPATCH_LEVEL
or above.
Arguments:
Arg1:<PHONE_NUMBER>000000, A single DPC or ISR exceeded its time allotment. The offending
component can usually be identified with a stack trace.
Arg2:<PHONE_NUMBER>000501, The DPC time count (in ticks).
Arg3:<PHONE_NUMBER>000500, The DPC time allotment (in ticks).
Arg4: fffff800348e2378, cast to nt!DPC_WATCHDOG_GLOBAL_TRIAGE_BLOCK, which contains
additional information regarding this single DPC timeout
Debugging Details:
*** WARNING: Unable to verify timestamp for cdcacmpo_x64.sys

***    Either you specified an unqualified symbol, or your debugger   ***
***    doesn't have full symbol information.  Unqualified symbol      ***
***    resolution is turned off by default. Please either specify a   ***
***    fully qualified symbol module!symbolname, or enable resolution ***
***    of unqualified symbols by typing ".symopt- 100". Note that     ***
***    enabling unqualified symbol resolution with network symbol     ***
***    server shares in the symbol path may cause the debugger to     ***
***    appear to hang for long periods of time when an incorrect      ***
***    symbol name is typed or the network symbol server is down.     ***

***    For some commands to work properly, your symbol path           ***
***    must point to .pdb files that have full type information.      ***

***    Certain .pdb files (such as the public OS symbols) do not      ***
***    contain the required information.  Contact the group that      ***
***    provided you with these symbols if you need this command to    ***
***    work.                                                          ***

***    Type referenced: TickPeriods                                   ***

*** WARNING: Unable to verify timestamp for win32k.sys
KEY_VALUES_STRING: 1
Key  : Analysis.CPU.Sec
Value: 7

Key  : Analysis.DebugAnalysisProvider.CPP
Value: Create: 8007007e on HSCL02793

Key  : Analysis.DebugData
Value: CreateObject

Key  : Analysis.DebugModel
Value: CreateObject

Key  : Analysis.Elapsed.Sec
Value: 42

Key  : Analysis.Memory.CommitPeak.Mb
Value: 79

Key  : Analysis.System
Value: CreateObject

BUGCHECK_CODE:  133
BUGCHECK_P1: 0
BUGCHECK_P2: 501
BUGCHECK_P3: 500
BUGCHECK_P4: fffff800348e2378
DPC_TIMEOUT_TYPE:  SINGLE_DPC_TIMEOUT_EXCEEDED
BLACKBOXBSD: 1 (!blackboxbsd)
BLACKBOXPNP: 1 (!blackboxpnp)
CUSTOMER_CRASH_COUNT:  1
PROCESS_NAME:  System
STACK_TEXT:
fffff80036fe4b78 fffff800346a76e1 :<PHONE_NUMBER>000133<PHONE_NUMBER>000000<PHONE_NUMBER>000501<PHONE_NUMBER>000500 : nt!KeBugCheckEx
fffff80036fe4b80 fffff800345a064d : 00001c89d39e1aa7 fffff8003337f180<PHONE_NUMBER>000246<PHONE_NUMBER>0a9375 : nt!KeAccumulateTicks+0x103571
fffff80036fe4be0 fffff800345a1733 : fffff78000000340 fffff8003337f180<PHONE_NUMBER>0a9375<PHONE_NUMBER>000000 : nt!KiUpdateRunTime+0x5d
fffff80036fe4c30 fffff80034406883 : 00001c89d39e0acb<PHONE_NUMBER>000000<PHONE_NUMBER>000000 fffff80036fd66f0 : nt!KeClockInterruptNotify+0x8f3
fffff80036fe4f40 fffff80034515aa5 : fffff80034469020 ffffde02db6a8c30 ffffde02db6a8c30 fffff800346430db : hal!HalpTimerClockInterrupt+0x63
fffff80036fe4f70 fffff8003464331a : fffff80036fd6770 fffff80034469020<PHONE_NUMBER>000000 fffff80036fd6b30 : nt!KiCallInterruptServiceRoutine+0xa5
fffff80036fe4fb0 fffff80034643807 :<PHONE_NUMBER>000000 ffffde02db6003c0<PHONE_NUMBER>6000a4<PHONE_NUMBER>000002 : nt!KiInterruptSubDispatchNoLockNoEtw+0xea
fffff80036fd66f0 fffff8003450046c :<PHONE_NUMBER>001000 ffffde02df2cfeb0<PHONE_NUMBER>000ea0 fffff80e00000020 : nt!KiInterruptDispatchNoLockNoEtw+0x37
fffff80036fd6880 fffff80034500425 : ffffde02db6a8c30 ffffde02db6a91d0<PHONE_NUMBER>000010 fffff80ea51f1fad : nt!KxWaitForSpinLockAndAcquire+0x2c
fffff80036fd68b0 fffff80eaa2f5d04 : ffffde02db6a8000 fffff8003337f180<PHONE_NUMBER>000000 fffff80033381f80 : nt!KeAcquireSpinLockRaiseToDpc+0x65
fffff80036fd68e0 ffffde02db6a8000 : fffff8003337f180<PHONE_NUMBER>000000 fffff80033381f80 fffff8003337f180 : cdcacmpo_x64+0x5d04
fffff80036fd68e8 fffff8003337f180 :<PHONE_NUMBER>000000 fffff80033381f80 fffff8003337f180 fffff8003459eb27 : 0xffffde02db6a8000 fffff80036fd68f0<PHONE_NUMBER>000000 : fffff80033381f80 fffff8003337f180 fffff8003459eb27 fffff80033381f80 : 0xfffff8003337f180
SYMBOL_NAME:  cdcacmpo_x64+5d04
MODULE_NAME: cdcacmpo_x64
IMAGE_NAME:  cdcacmpo_x64.sys
STACK_COMMAND:  .thread ; .cxr ; kb
BUCKET_ID_FUNC_OFFSET:  5d04
FAILURE_BUCKET_ID:  0x133_DPC_cdcacmpo_x64!unknown_function
OS_VERSION:  10.0.17134.1
BUILDLAB_STR:  rs4_release
OSPLATFORM_TYPE:  x64
OSNAME:  Windows 10
FAILURE_ID_HASH:  {f5551ce5-8989-8021-8ac7-7a7fd5ceed41}
Followup:     MachineOwner
0: kd> !analyze -v
ERROR: FindPlugIns 8007007b

                                                                        *

                   Bugcheck Analysis                                    *

                                                                        *

DPC_WATCHDOG_VIOLATION (133)
The DPC watchdog detected a prolonged run time at an IRQL of DISPATCH_LEVEL
or above.
Arguments:
Arg1:<PHONE_NUMBER>000000, A single DPC or ISR exceeded its time allotment. The offending
component can usually be identified with a stack trace.
Arg2:<PHONE_NUMBER>000501, The DPC time count (in ticks).
Arg3:<PHONE_NUMBER>000500, The DPC time allotment (in ticks).
Arg4: fffff800348e2378, cast to nt!DPC_WATCHDOG_GLOBAL_TRIAGE_BLOCK, which contains
additional information regarding this single DPC timeout
Debugging Details:

***    Either you specified an unqualified symbol, or your debugger   ***
***    doesn't have full symbol information.  Unqualified symbol      ***
***    resolution is turned off by default. Please either specify a   ***
***    fully qualified symbol module!symbolname, or enable resolution ***
***    of unqualified symbols by typing ".symopt- 100". Note that     ***
***    enabling unqualified symbol resolution with network symbol     ***
***    server shares in the symbol path may cause the debugger to     ***
***    appear to hang for long periods of time when an incorrect      ***
***    symbol name is typed or the network symbol server is down.     ***

***    For some commands to work properly, your symbol path           ***
***    must point to .pdb files that have full type information.      ***

***    Certain .pdb files (such as the public OS symbols) do not      ***
***    contain the required information.  Contact the group that      ***
***    provided you with these symbols if you need this command to    ***
***    work.                                                           This looks like something our technical support team should be able to help you with. Did you contact them?

Hi there,
Unfortunately not,how would I go about doing that?
Regards
<EMAIL_ADDRESS>Benjamin Hirst
IT Support Analyst
HSC, Enterprise Way, Whittle Road
Meir Park, Stoke-on-Trent ST3 7UN
Did You know you can now Raise, Update or View an IT ticket via the following link:
http://Well.Topdesk.nethttp://well.topdesk.net/
From: Denise Vangel-MSFT<EMAIL_ADDRESS>Sent: 09 July 2020 03:23
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Ben Hirst<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] DPC_WATCHDOG_VIOLATION (133) SINGLE_DPC_TIMEOUT_EXCEEDED (#7109)
Hello @ben121091https://github.com/ben121091. This looks like something our technical support team should be able to help you with. Did you contact them?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/7109#issuecomment-655857197, or unsubscribehttps://github.com/notifications/unsubscribe-auth/APQCY7EBWKF3J6TVKGDSMALR2USX5ANCNFSM4OUOLCVQ.
This email is from Bestway Panacea Holdings Ltd (Reg No. 09225479) or one of its subsidiaries or affiliates (collectively "Bestway Panacea"). Details of the subsidiaries and affiliates of Bestway Panacea Holdings Ltd and further information can be found at www.pharmacy.co.uk. Bestway Panacea is part of the Bestway Group which is owned by Bestway (Holdings) Ltd (Reg No. 01392861).
This email (including any attachments) contains information that is private and confidential and may be legally privileged and is for the exclusive use of the individual or entity to which it is addressed. If you have received this e-mail in error, you are on notice of its status. Please preserve the confidentiality of this email and advise the sender immediately by return email of any error in transmission and then delete this message from your system. Any disclosure, distribution, copying or alteration of this information, in whole or in part, is strictly prohibited. Bestway Panacea cannot accept liability for any damage which you may sustain as a result of virus infection so please carry out your own virus checks. In replying to this e-mail you are granting the right for that reply to be forwarded to any other individual within Bestway Panacea and also to be read by others. Any views or opinions presented are solely those of the author and do not necessarily represent those of Bestway Panacea unless otherwise specifically stated. Bestway Panacea is not responsible for any reliance placed on the information contained herein and excludes all liability. Bestway Panacea reserves the right to intercept, monitor and record communications for lawful business purposes. Well is the trading name for Bestway Panacea.

@ben121091,  issue caused due to  cdcacmpo_x64.sys, so delete this file from C:\windows\system32\drivers
before deleting, take the ownership of that file in normal mode, But deleting file in safe mode.
If you installed any recent device driver means then completely uninstall that driver with related software.
.

Hi Thank you @RAJU2529 ,I do have another .Dump file which I am very interested in know the RCA for?
Is it possible you can inform me on how you are able to troubleshoot and find the RCA for this.
the below is the dumpfile
I can understand some of this but not all of it
ie
USB\VID_05F9&PID_4204\S/N_E16I37565 is the device
However the bit that interests me is,the bit below is an executeable we use,and would like to know in what way this could be causing the system to crash.:
CUSTOMER_CRASH_COUNT:  1
PROCESS_NAME:  analyst.exe
DPC_STACK_BASE:  FFFFD5057E830FB0
Microsoft (R) Windows Debugger Version 10.0.19041.1 AMD64
Copyright (c) Microsoft Corporation. All rights reserved.
Loading Dump File [\corp.well.co.uk\Data\UserData\ben.hirst\Desktop\Minidumps\200247\061520-6984-01.dmp]
Mini Kernel Dump File: Only registers and stack trace are available
************* Path validation summary **************
Response                         Time (ms)     Location
Deferred                                       SRVC:\SymCachehttp://msdl.microsoft.com/download/symbols
Symbol search path is: SRVC:\SymCachehttp://msdl.microsoft.com/download/symbols
Executable search path is:
Windows 10 Kernel Version 17134 MP (4 procs) Free x64
Product: WinNt, suite: TerminalServer SingleUserTS
Built by: 17134.1.amd64fre.rs4_release.180410-1804
Machine Name:
Kernel base = 0xfffff8009021a000 PsLoadedModuleList = 0xfffff800905c6ce0
Debug session time: Mon Jun 15 12:19:00.574 2020 (UTC + 1:00)
System Uptime: 3 days 23:02:31.633
Loading Kernel Symbols
...............................................................
................................................................
........................................................
Loading User Symbols
Loading unloaded module list
.............................
For analysis of this file, run !analyze -v
1: kd> !analyze -v
ERROR: FindPlugIns 8007007b

                                                                        *

                   Bugcheck Analysis                                    *

                                                                        *

IRQL_NOT_LESS_OR_EQUAL (a)
An attempt was made to access a pageable (or completely invalid) address at an
interrupt request level (IRQL) that is too high.  This is usually
caused by drivers using improper addresses.
If a kernel debugger is available get the stack backtrace.
Arguments:
Arg1:<PHONE_NUMBER>000047, memory referenced
Arg2:<PHONE_NUMBER>000002, IRQL
Arg3:<PHONE_NUMBER>000000, bitfield :
bit 0 : value 0 = read operation, 1 = write operation
bit 3 : value 0 = not an execute operation, 1 = execute operation (only on chips which support this level of status)
Arg4: fffff800902bfdbe, address which referenced memory
Debugging Details:
*** WARNING: Unable to verify timestamp for cdcacmpo_x64.sys
KEY_VALUES_STRING: 1
Key  : Analysis.CPU.Sec
Value: 4

Key  : Analysis.DebugAnalysisProvider.CPP
Value: Create: 8007007e on HSCL02793

Key  : Analysis.DebugData
Value: CreateObject

Key  : Analysis.DebugModel
Value: CreateObject

Key  : Analysis.Elapsed.Sec
Value: 21

Key  : Analysis.Memory.CommitPeak.Mb
Value: 78

Key  : Analysis.System
Value: CreateObject

BUGCHECK_CODE:  a
BUGCHECK_P1: 47
BUGCHECK_P2: 2
BUGCHECK_P3: 0
BUGCHECK_P4: fffff800902bfdbe
READ_ADDRESS: fffff80090665388: Unable to get MiVisibleState
Unable to get NonPagedPoolStart
Unable to get NonPagedPoolEnd
Unable to get PagedPoolStart
Unable to get PagedPoolEnd
fffff800905b3d90: Unable to get Flags value from nt!KdVersionBlock
0000000000000047
BLACKBOXBSD: 1 (!blackboxbsd)
BLACKBOXPNP: 1 (!blackboxpnp)
CUSTOMER_CRASH_COUNT:  1
PROCESS_NAME:  analyst.exe
DPC_STACK_BASE:  FFFFD5057E830FB0
TRAP_FRAME:  ffffd5057e830980 -- (.trap 0xffffd5057e830980)
NOTE: The trap frame does not contain all registers.
Some register values may be zeroed or incorrect.
rax=0000000000000000 rbx=0000000000000000 rcx=0000000000000000
rdx=00000000ffffffff rsi=0000000000000000 rdi=0000000000000000
rip=fffff800902bfdbe rsp=ffffd5057e830b10 rbp=ffffd5057e830d02
r8=0000000000000000  r9=0000000000000000 r10=ffff950014e20180
r11=0000000000000000 r12=0000000000000000 r13=0000000000000000
r14=0000000000000000 r15=0000000000000000
iopl=0         nv up ei pl zr na po nc
nt!IoReuseIrp+0x4e:
fffff800902bfdbe 0fb67347        movzx   esi,byte ptr [rbx+47h] ds:0000000000000047=??
  This is usually
caused by drivers using improper addresses.
If a kernel debugger is available get the stack backtrace.
Arguments:
Arg1:<PHONE_NUMBER>000047, memory referenced
Arg2:<PHONE_NUMBER>000002, IRQL
Arg3:<PHONE_NUMBER>000000, bitfield :
bit 0 : value 0 = read operation, 1 = write operation
bit 3 : value 0 = not an execute operation, 1 = execute operation (only on chips which support this level of status)
Arg4: fffff800902bfdbe, address which referenced memory
Debugging Details:
KEY_VALUES_STRING: 1
Key  : Analysis.CPU.Sec
Value: 5

Key  : Analysis.DebugAnalysisProvider.CPP
Value: Create: 8007007e on HSCL02793

Key  : Analysis.DebugData
Value: CreateObject

Key  : Analysis.DebugModel
Value: CreateObject

Key  : Analysis.Elapsed.Sec
Value: 19

Key  : Analysis.Memory.CommitPeak.Mb
Value: 80

Key  : Analysis.System
Value: CreateObject

BUGCHECK_CODE:  a
BUGCHECK_P1: 47
BUGCHECK_P2: 2
BUGCHECK_P3: 0
BUGCHECK_P4: fffff800902bfdbe
READ_ADDRESS: Unable to get NonPagedPoolStart
Unable to get NonPagedPoolEnd
Unable to get PagedPoolStart
Unable to get PagedPoolEnd
fffff800905b3d90: Unable to get Flags value from nt!KdVersionBlock
0000000000000047
BLACKBOXBSD: 1 (!blackboxbsd)
BLACKBOXPNP: 1 (!blackboxpnp)
CUSTOMER_CRASH_COUNT:  1
PROCESS_NAME:  analyst.exe
DPC_STACK_BASE:  FFFFD5057E830FB0
TRAP_FRAME:  ffffd5057e830980 -- (.trap 0xffffd5057e830980)
NOTE: The trap frame does not contain all registers.
Some register values may be zeroed or incorrect.
rax=0000000000000000 rbx=0000000000000000 rcx=0000000000000000
rdx=00000000ffffffff rsi=0000000000000000 rdi=0000000000000000
rip=fffff800902bfdbe rsp=ffffd5057e830b10 rbp=ffffd5057e830d02
r8=0000000000000000  r9=0000000000000000 r10=ffff950014e20180
r11=0000000000000000 r12=0000000000000000 r13=0000000000000000
r14=0000000000000000 r15=0000000000000000
iopl=0         nv up ei pl zr na po nc
nt!IoReuseIrp+0x4e:
fffff800902bfdbe 0fb67347        movzx   esi,byte ptr [rbx+47h] ds:0000000000000047=??
  Moreover, DPC watchdog violation caused by incompatible unsigned device driver   .sys   .dll  .exe  files ,   so  reinstall  the culprit device  driver , I could not able to found out the exact device name and  brand too.

Hello, @ben121091
I hope the comments provided by @RAJU2529 serve helpful in clarifying your question.
However, Looks like this question can best be answered via relevant Windows 10 Support channels.
Please consider checking with Microsoft Community or contacting support for your problem; The issues section of this repository is for content fixes in documentation.
If you feel that the associated article also needs to be updated for any reason feel free to re-open this issue or contribute with a pull request.
Thank you.


[Broker] Change create topic return error to Status.BAD_REQUEST

Motivation
When create transaction system topic, broker will return Status.CONFLICT, It does not conform to the wrong return semantics of the method. So change it to Status.BAD_REQUEST, when client receive Status.BAD_REQUEST, it will generic PulsarAdminException and error code is Status.BAD_REQUEST
Modifications
change the error code to Status.BAD_REQUEST when create transaction system topic.
Verifying this change
change test verify

@congbobo184 Could you please check the comments?


removed assert from evaluation of dict, func: utils.is_dict.

Removing assert from utils.is_dict. Removing the risk of executing the passed down variable when assert  evaluates the variable. In addition set a default None return, since Nothing is already expected upstream.

Tests failed and would it not be better if you changed it to return isinstance(d, dict) ?

Also, you would have to correct the unit tests so they would pass with this new logic becuas they are currently failing.

ok I will do so

sorry about that ..

@gmolight I will go through this and review it sometime during the week

@gmolight I implemented this fix in a different way then this but with the same result. The commit is 4738b6df9dd802a24ada8ff9920a3285306a33f3
I felt that raising a more proper exception on any error was better. Credit is till in your name even if i did the commit :]
Thanks for the idea and initial work.


Question Recreating Dataset

Hello,
currently I am trying to recreate your dataset, regarding that I have some questions:

The routes and scenarios are provided for everything except "right_dataset" and "left_dataset". Are there reasons for this or how are they structured?
Some folder's names contain "clipped", "10m", "30m" etc. Do the provided scenarios and routes result in these data or do I have to clip or cut them manually? In other words: If I run the generation on every route and scenario you provided, will I get the same dataset  you provided, except the folders in 1.? (With "the same" I mean with respect to scenario/route and length. Of course they will slightly vary because of non-determinism)
(Not regarding dataset) Is there an overlap in the routes of the Longest6 Benchmark and the training data?

Thank you!

right_dataset and left_dataset are generated from the route xml files for Scenarios 8 and 9. However, we combine these routes with an empty scenario json file. This gives data at left and right turns, but without the added complexity of background traffic running red lights and entering the intersection. In our follow-up projects, we stopped generating these additional folders, which does not significantly impact the driving performance.
The provided routes will result in the same data distribution as our released dataset. The folder names are a result of some naming inconsistencies in preliminary versions of the route and scenario files, and the names shouldn't be relevant for reproducing the dataset.
The training routes are sampled densely (e.g. from all intersections) in a town, so the static environment encountered in the Longest6 evaluation routes is indeed overlapping with the training set at several locations. That said, the model does not see the exact same weather, traffic patterns etc. during training and evaluation on Longest6. For testing generalization to completely unseen routes, we would recommend the benchmark proposed in LAV, which holds out Town02 and Town05 during training.


🌱 udpate to CAPI 1.6.0

this commit updates to cluster API 1.6.0

[x] Merge it after the new builder PR is merged. (https://github.com/SovereignCloudStack/cluster-stack-operator/pull/78)
[x] Merge is after we have updated to go 1.21 series (https://github.com/SovereignCloudStack/cluster-stack-operator/pull/63)

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes https://github.com/SovereignCloudStack/cluster-stack-operator/issues/77
Special notes for your reviewer:
Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.
TODOs:

[ ] squash commits
[ ] include documentation
[ ] add unit tests

@janiskemper tests are failing because of caching, you can verify this on your local system as well.
On my system, it's passing.
$ make test-unit
go install<EMAIL_ADDRESS>go install<EMAIL_ADDRESS>curl -sSL https://get.helm.sh/helm-v3.12.2-linux-amd64.tar.gz | tar xz -C hack/tools/bin --strip-components=1 linux-amd64/helm
chmod a+rx /home/k7/work/scsoss/cluster-stack-operator/hack/tools/bin/helm
CREATE_KIND_CLUSTER=true KUBEBUILDER_ASSETS="/home/k7/work/scsoss/cluster-stack-operator/hack/tools/bin/k8s/1.25.0-linux-amd64" /home/k7/work/scsoss/cluster-stack-operator/hack/tools/bin/gotestsum --junitfile=.coverage/junit.xml --format testname -- -mod=vendor \
-covermode=atomic -coverprofile=.coverage/cover.out -p=4 ./internal/controller/...
PASS internal/controller.TestMakeDiff (0.00s)
PASS internal/controller.TestGetLatestReadyClusterStackRelease/a_ready,_latest_cluster_stack_release_present (0.00s)
PASS internal/controller.TestGetLatestReadyClusterStackRelease/No_ready_cluster_stack_releases (0.00s)
PASS internal/controller.TestGetLatestReadyClusterStackRelease (0.00s)
PASS internal/controller.TestGetUsableClusterStackReleaseVersions/two_ready_cluster_stack_releases (0.00s)
PASS internal/controller.TestGetUsableClusterStackReleaseVersions/no_ready_cluster_stack_releases (0.00s)
PASS internal/controller.TestGetUsableClusterStackReleaseVersions (0.00s)
PASS internal/controller.TestGetClusterStackReleasesInSpec (0.00s)
PASS internal/controller.TestControllers (307.10s)
coverage: 51.1% of statements
PASS internal/controller (coverage: 51.1% of statements)

DONE 9 tests in 333.114s

I think it makes sense to use 1.6.2 instead? Then the version would be directly up to date.

the goal would be to let this be done by renovate bot automatically ;) Otherwise you are completely right that we should make it up to date even in this PR


How to access other routes by creating a folder in /routes ?

I don't wish to initiate a new project for each topic and thus, I though of creating separate routes for those.
I created a route called /closures and inside, I created a +page.svelte file.
However, If I enter the route http://localhost:5137/#/closures , it would take me to homepage instead.
Is this how it will be or is there something I can do about it?

Instead of http://localhost:5173/#/closures the URL should be http://localhost:5173/closures and it should work.


Please could someone provide a Blazor template that has a normal menu bar

The Blazor template has a nav menu that is a side bar. Despite many multiple attempts I have not been able to get a normal top menu bar working satisfactorily (may be my unfamiliarity with Blazor). Can you provide an option to create a template with a top menu bar?

Hi @robalexclark. We don't plan to provide an alternative version of the template with a top menu bar, but you should be able to update the template code to add your own menu bar with with normal HTML and CSS.  Can you share with us what issues you've been having getting a top menu bar to work in a Blazor app?

@robalexclark If you mean template like in ASP.NET Core. You can take a look in MenuHorizontal.
It used the CSS and layout from ASP.NET Core template.

I understand that the menu is just driven by bootstrap but being new to blazor I really struggled to unpick the sidebar and get it across the top. I probably spent 5-6 hours trying/crying to get it to work but hit issues (I know it shouldn't be this hard - right!?).
I think that there will be many folks that hit the same issue and struggle to get this working. Thank you @soeleman for the demo, it looks good and I've managed to copy it into a new app.
There is a bounty available at https://stackoverflow.com/questions/58235005/blazor-template-with-menu-across-the-top for you to answer with your demo if you want.

Thanks again to @soelman for his work which I built on and produced a repo with a top menu bar:
https://github.com/robalexclark/BlazorTopMenu


Development -- suggested defs and a few questions

Hello,
This is an offshoot of Doug's SF request for the paraxial mesoderm
terms (SF:889881).
First off, some suggested defs - def refs are Dorlands, aka ISBN:
0721601464
ectoderm development, GO:0007398
The formation and development of the ectoderm, the outermost
layer of cells of the three primary germ layers of the embryo.
From it are developed the epidermis and the epidermal tissues,
such as the nails, hair, enamel of teeth, and glands of the skin, the
nervous system, the external sense organs such as the ear and
eye, and the mucous membrane of the mouth and anus.
endoderm development, GO:0007492
The formation and development of the endoderm, the innermost of
the three primary germ layers of the embryo; from it are derived
the epithelium of the pharynx, respiratory tract (except the nose),
digestive tract, bladder, and urethra.
mesoderm development, GO:0007498
The formation and development of the mesoderm, the middle
layer of the three primary germ layers of the embryo, lying
between the ectoderm and the endoderm. From it are derived the
connective tissue, bone and cartilage, muscle, blood and blood
vessels, lymphatics and lymphoid organs, notochord, pleura,
pericardium, peritoneum, kidney, and gonads.
gut mesoderm development, GO:0007502
The formation and development of gut mesoderm cells, the middle
layer of the three primary germ layers of the embryo which will go
on to form the gut of the organism.
gonadal mesoderm development, GO:0007506
The formation and development of gonadal mesoderm cells, the
middle layer of the three primary germ layers of the embryo which
will go on to form the gonads of the organism.
Next up, some of those "what's the difference" questions. What's
the difference between:
- 'epidermal differentiation ; GO:0008544' and 'epidermal cell
differentiation ; GO:0009913'
- 'mesoderm cell migration ; GO:0008078' and 'mesoderm
migration ; GO:0007509'
- 'xxx formation' and 'xxx development' - we could do with
standard defs for both if they are different concepts.
Lastly, under histogenesis, things don't always adhere to the
principles laid down in the St. Croix meeting relating to the
relationships between cell development terms. Are these principles
applicable to tissue development? If not, can we formulate some
rules for tissue development?
Cell differentiation terms should be arranged thus:
[i] cell differentiation
---[p] cell fate commitment (exact synonym: cell fate specification)
---[p] cell fate determination
---[p] cell development (exact synonyms: cell morphogenesis, cell
maturation)
Doing a search for 'ectoderm', there are loads of terms scattered
all over the place.
eg. in the ectoderm section, we have:
histogenesis
[i]ectoderm development
---[p]ectoderm formation
------[p]ectoderm cell fate determination
Elsewhere, under 'cell fate commitment', we have:
ectoderm cell fate commitment
[p]ectoderm cell fate determination
[p]ectoderm cell fate specification
and elsewhere still, we have:
ectodermal gut morphogenesis
[p]anterior midgut (ectodermal) morphogenesis
How do these tissue terms relate to each other and to the tissue
cell terms (eg. ectoderm cell differentiation vs. ectoderm
differentiation)?
Reported by: girlwithglasses
Original Ticket: "geneontology/ontology-requests/1628":https://sourceforge.net/p/geneontology/ontology-requests/1628

thanks www.movied.org