TypeError: init() got an unexpected keyword argument 'port'

Stratux version: 1.4r1

Stratux config:
SDR

[ ] single
[X ] dual

GPS

[ X] yes
[ ] no
type: uBlox6, 7 and 8.  generic USB

AHRS

[ X] yes
[ ] no

power source:
USB, with volt/ammeter, so I know the power is good.
usb cable:
20AWG

EFB app and version: (e.g., WingX Pro7 8.6.2)
N/A
EFB platform: (e.g., iOS 9.2)
N/A
EFB hardware: (e.g., iPad Mini 2)
N/A

Description of your issue:

Brand new install, I get this on startup:
cat /var/log/stratux-screen.log
started with pid 600
Traceback (most recent call last):
File "/usr/bin/stratux-screen.py", line 92, in
daemon_runner.do_action()
File "/usr/lib/pymodules/python2.7/daemon/runner.py", line 186, in do_action
func(self)
File "/usr/lib/pymodules/python2.7/daemon/runner.py", line 131, in _start
self.app.run()
File "/usr/bin/stratux-screen.py", line 23, in run
oled = ssd1306(port=1, address=0x3C)
TypeError: init() got an unexpected keyword argument 'port'

Looks like another error with that ssd1306 Python package. It won't affect your build in any way unless you have an OLED screen that you want to display data on.

Raw metrics
-----------

+----------+-------+------+---------+-----------+
|type      |number |%     |previous |difference |
+==========+=======+======+=========+===========+
|code      |71     |87.65 |NC       |NC         |
+----------+-------+------+---------+-----------+
|docstring |0      |0.00  |NC       |NC         |
+----------+-------+------+---------+-----------+
|comment   |1      |1.23  |NC       |NC         |
+----------+-------+------+---------+-----------+
|empty     |9      |11.11 |NC       |NC         |
+----------+-------+------+---------+-----------+

External dependencies
---------------------
::

    PIL
      \-Image (stratux-screen)
      \-ImageDraw (stratux-screen)
      \-ImageFont (stratux-screen)
    daemon
      \-runner (stratux-screen)
    oled
      \-device (stratux-screen)
      \-render (stratux-screen)

Statistics by type
------------------

+---------+-------+-----------+-----------+------------+---------+
|type     |number |old number |difference |%documented |%badname |
+=========+=======+===========+===========+============+=========+
|module   |1      |NC         |NC         |0.00        |100.00   |
+---------+-------+-----------+-----------+------------+---------+
|class    |1      |NC         |NC         |0.00        |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|method   |2      |NC         |NC         |50.00       |0.00     |
+---------+-------+-----------+-----------+------------+---------+
|function |0      |NC         |NC         |0           |0        |
+---------+-------+-----------+-----------+------------+---------+

Messages by category
--------------------

+-----------+-------+---------+-----------+
|type       |number |previous |difference |
+===========+=======+=========+===========+
|convention |32     |NC       |NC         |
+-----------+-------+---------+-----------+
|refactor   |4      |NC       |NC         |
+-----------+-------+---------+-----------+
|warning    |2      |NC       |NC         |
+-----------+-------+---------+-----------+
|error      |2      |NC       |NC         |
+-----------+-------+---------+-----------+

Messages
--------

+-----------------------+------------+
|message id             |occurrences |
+=======================+============+
|invalid-name           |17          |
+-----------------------+------------+
|line-too-long          |9           |
+-----------------------+------------+
|missing-docstring      |3           |
+-----------------------+------------+
|unused-import          |2           |
+-----------------------+------------+
|unexpected-keyword-arg |2           |
+-----------------------+------------+
|trailing-whitespace    |1           |
+-----------------------+------------+
|too-many-statements    |1           |
+-----------------------+------------+
|too-many-locals        |1           |
+-----------------------+------------+
|too-few-public-methods |1           |
+-----------------------+------------+
|superfluous-parens     |1           |
+-----------------------+------------+
|old-style-class        |1           |
+-----------------------+------------+
|no-self-use            |1           |
+-----------------------+------------+

Global evaluation
-----------------
Your code has been rated at 3.33/10

Duplication
-----------

+-------------------------+------+---------+-----------+
|                         |now   |previous |difference |
+=========================+======+=========+===========+
|nb duplicated lines      |0     |NC       |NC         |
+-------------------------+------+---------+-----------+
|percent duplicated lines |0.000 |NC       |NC         |
+-------------------------+------+---------+-----------+


Minute tweak for breaking of theme assets.

Replace this:
https://github.com/bagisto/bagisto/blob/ef622b9e9bf9ca0bdcf63ae4f5c3443836b39285/packages/Webkul/Shop/src/Resources/views/layouts/master.blade.php#L14
With this:
@php
$storeCSSPath = "themes/". core()->getCurrentChannel()->theme ."/assets/css/shop.css";
@endphp

Replace this:
https://github.com/bagisto/bagisto/blob/ef622b9e9bf9ca0bdcf63ae4f5c3443836b39285/packages/Webkul/Shop/src/Resources/views/layouts/master.blade.php#L109
With this:
@php
$storeJSPath = "themes/". core()->getCurrentChannel()->theme ."/assets/js/shop.js";
@endphp

Hey @dejavuejs ,
I think, getting assets via bagisto_asset helper already provides us path according to the selected theme. Please have a look here

Try extending shop package's master.blade.php from any other package and use the route apart from shop package also.
Ex: Define route in Admin package that loads shop package's homepage view.

You need to add theme middleware to your route.


[v0.9.0-rc31] Error describing databases

After update to v0.9.0-rc31 the program stop working.
When I Run:
influxDB = InfluxDBFactory.connect("http://" + influxHost + ":"
+ influxPort, influxUser, influxPassword);
influxDB.describeDatabases();
I get this exception:
RuntimeException: 404 page not found
Im using JAVA API 1.5
Does anyone know how to solve this issue?

@MdSE What version were you running before upgrading to RC31?
This sounds more like an issue with the Java client than with the database. Can you verify that the following works:
curl -G 'http://<hostname>:8086/query' --data-urlencode 'q=CREATE DATBASE <db_name>'
If that works then the issue is not with InfluxDB but with the Java client. You can open an issue for that repo here: https://github.com/influxdb/influxdb-java/issues/new

Does the Java client actually already support the new API of InfluxDB 0.9? There is open issue regarding compatibilty (influxdb/influxdb-java/issues/22).

@beckettsean I was using 0.8.8.
When I run:
curl -G 'http://localhost:8086/query' --data-urlencode 'q=CREATE DATABASE multittest'
It returns:
{"results":[{}]}
and the database is created.
Thanks for your help

@MdSE thanks for the extra detail. The changes between 0.8 and 0.9 are substantial. Most of the query language has been updated, so very few things that work on 0.8 will work unaltered on 0.9, and most clients are waiting until 0.9 final to update their version support.
I'm glad the straight curl commands are working and stay tuned to the influxdb/influxdb-java repo for updates on that client.


Kindly help me to with proper config file for source and destination extraction

@akelad @RasaHQ
Rasa  NLU version: 0.15.0
Rasa core version : 0.14.3
Python version: 3.7
Operating system (windows, osx, ...): windows
Issue: I want to extract entities like Source and Destination from string like "Show me trains from Rajkot to Ahmedabad on 31st may" . Kindly help me with CONFIG.YML file. I am also attaching my training data.
I am annotating after "from" as source and after  "to" as destination.
digits similar to 3rd ,4th and 13th...etc   as "date" and "may", "june"....etc as month.
IT EXTRACTS MONTH AND DATE PROPERLY USING CRFENTITYEXTRACTOR BUT FOR SOURCE AND DESTINATION IT IS NOT WORKING.
Kindly help me with proper config.yml file
Content of configuration file (config.yml):

language: "en"

pipeline:
- name: "SpacyNLP"
- name: "SpacyTokenizer"
- name: "SpacyFeaturizer"
- name: "RegexFeaturizer"
- name: "CRFEntityExtractor"
- name: "EntitySynonymMapper"
- name: "SklearnIntentClassifier"

Content of domain file (domain.yml) (if used & relevant):

data.txt

@prashant334 please ask usage questions like this on the forum. Your problem likely has to do with the fact that your spaCy english model does not have word representations for the less populated cities in Asia.

@prashant334 please ask usage questions like this on the forum. Your problem likely has to do with the fact that your spaCy english model does not have word representations for the less populated cities in Asia.

@erohmensing Hi, There is no problem with spacy english model. There is no plroblem with even cities detection using spacy.
I think you did not get the question.
Here I want to extract source and destination from sentence like.
Show me bus from Delhi to Mumbai.
so Delhi as source city
Mumbai as destination city

As I said, this is not an appropriate place for these types of questions. Please ask on the forum with more information to show how it is not working, i.e. some print out of what it is parsing with your own description of what you would like it to be parsing.


Add support for Element Desktop

There is a repo for this already: https://github.com/catppuccin/element
It just needs nix integration. I was thinking about making a PR myself, but im unsure how to do this since, at least from what i can tell, for this to work there needs to be a home-manager option before a catppuccin option can be added.
Feel free to correct me or point me in the right direction on what to do to make this work. Im more than happy to do the PR myself, just need a little jumpstart with more experience in this.

for this to work there needs to be a home-manager option before a catppuccin option can be added

Its not a hard requirement for us, see glamour, but i assume upstream would appreciate it.


Dpocon and Dgecon mismatch

Our tests at Gonum for Dpocon compare to the result for Dgecon. For most matrices (randomly generated), the two outputs match to machine precision.
The matrix
[2.9995576045549965, -2.0898894566158663, 3.965560740124006,
-2.0898894566158663, 1.9634729526261008, -2.8681002706874104,
3.965560740124006, -2.8681002706874104, 5.502416670471008]
Gives a significantly different answer for the two routines. Dgecon gives 0.020928829230190116 while Dpocon gives 0.024054837369015203. The output is the same for both routines in OpenBLAS and gonum/lapack, which suggests a problem with the algorithm rather than the implementation.

Thank you for the feedback. Yes, it looks like an algorithm issue.

Should a bug with netlib be filed?

@btracey , I think so.

Filed http://icl.cs.utk.edu/lapack-forum/viewtopic.php?f=13&t=4817

As far as I can tell there has been no activity on the netlib bug since it was filed over two years ago. From the wording of the documentation of both functions "an estimate is obtained for norm(inv(A))..." it is not clear to me if there is any implied guarantee that both should always return the same result, even if it might be convenient for cross-checking between implementations. What is the current stance of Gonum on this issue - worthwile to pursue, or an oddity that was sufficiently worked around
by modifying the test ?

Finally I've had time to look into this more closely, motivated also by seeing similar differences recently in the condition number estimation for symmetric positive definite band matrices. The main computational part in the estimation is the DLACN2 routine which is an iterative algorithm to estimate the 1-norm of a matrix. The Fortran implementation has a reference to a paper where the algorithm, some theory and numerical experiments are presented. The first point in the summary of their experiments says:

Approximately three-fifths of the estimates were exact (i.e., the computed relative error was of the order of the unit roundoff). The proportion of exact estimates varies greatly with the type of matrix.

They also report that in all their tests the estimate was correct to within a factor of 10 which is what I use in Gonum in testlapack.DpbconTest and the test passes. Tests in the reference allow a more generous difference of factor 30.
In other words, a not-too-large difference is normal and I think you can close this issue.

Great, thanks for researching this.

One more remark: my comment above of course does not answer why there can be a difference between DGECON and DPOCON, only why there can be a difference between the estimated and exact condition number. The former comes most likely from DGETRF using pivoting which leads to DLACN2 producing different sequences of vectors that may converge to different results. If the above matrix is permuted so that DGETRF doesn't do it, both DPOCON and DGECON give the same (within eps) estimate for the condition number.


Frozen lake

hi!
i want to solve Frozen Lake problem with Pearl.But how can i do that?i do not know how to use Pearl for solving problems.You said Pearl can be used for real-world problems but you did not provide any example for that.
Please help me!

Should this already in our gymnasium benchmark? No?

Hi,
We are sorry that this created confusion to you. Currently we have two tutorials. One is about a DQN-based news recommender system while the other one is a contextual bandit tutorial.
In terms of the FrozenLake environment, I think the quick start code shown in the README file is pretty close to a solution.
One thing important to note here is that the you would like to convert each observation, which is an index of the current state, to some representation vector (like a "one-hot vector") for that state. This is because in our design, all states are assumed to be representation vectors instead of indices.
Then you would obtain some code like this, which works in this problem.
from pearl.action_representation_modules.one_hot_action_representation_module import (
    OneHotActionTensorRepresentationModule,
)
from pearl.policy_learners.sequential_decision_making.deep_q_learning import (
    DeepQLearning,
)
from pearl.replay_buffers.sequential_decision_making.fifo_off_policy_replay_buffer import (
    FIFOOffPolicyReplayBuffer,
)
from pearl.utils.instantiations.environments.gym_environment import GymEnvironment
import numpy as np
env = GymEnvironment("FrozenLake-v1", is_slippery=False)

num_actions = env.action_space.n
agent = PearlAgent(
    policy_learner=DeepQLearning(
        state_dim=env.observation_space.n,
        action_space=env.action_space,
        hidden_dims=[64],
        training_rounds=20,
        action_representation_module=OneHotActionTensorRepresentationModule(
            max_number_actions=num_actions
        ),
    ),
    replay_buffer=FIFOOffPolicyReplayBuffer(10_000),
)
for i in range(1000):
    observation, action_space = env.reset()
    # convert observation, which is an index, to an one-hot vector
    tmp = np.zeros(env.observation_space.n, dtype=float)
    tmp[observation] = 1
    observation = tmp.astype(np.float32)
    agent.reset(observation, action_space)
    done = False
    rtn = 0
    while not done:
        action = agent.act(exploit=False)
        action_result = env.step(action)
        # convert observation, which is an index, to an one-hot vector
        tmp = np.zeros(env.observation_space.n, dtype=float)
        tmp[action_result.observation] = 1
        action_result.observation = tmp.astype(np.float32)
        agent.observe(action_result)
        agent.learn()
        done = action_result.done
        rtn += action_result.reward
        if done:
            print(f"Episode: {i}, Return: {rtn} ")

So nice!thank you .
If i have any other question about solving problems like that,should comment here or open new issue?

You are welcome. We are glad that you play with the code and provide feedback.
If the new questions are not related to this issue, I suggest starting a new issue.

Please note we have revised the code example to avoid having to mess with numpy and numpy dtypes.
We also introduced a one_hot_vector function to make that operation more evident, and selected a very small environment (size 3) to make execution fast to check.

Please note we have added a FrozenLake Jupyter notebook tutorial that's an even simpler version (using the online_learning function which automates the learning loop for you).


Modifiers for variable substitution

One thing one may see in template engines is the ability to modify the resolved variable, for example by prepending it with : and the associated modifier - currently I cannot find any documentation regarding this regarding variable substitution in vscode (tasks), so hereby a feature request.
See below for some examples based on tasks.json inputs:
{
  "version": "2.0.0",
  "inputs":[
    {
      "id": "BuildConfiguration",
      "description": "The used Build Configuration: Debug or Release",
      "default": "Release",
      "options": [
        "Debug",
        "Release"
      ],
      "type": "pickString"
    }
  ],
}

Based on the above, one could expect the following modifiers (based on the default value) and their results:

${input:BuildConfiguration} => Release
${input:BuildConfiguration@upper} or ${input:BuildConfiguration@uppercase} => RELEASE
${input:BuildConfiguration@lower} or ${input:BuildConfiguration@lowercase} => release

These are some I can think of quickly, but others like case changes like change to snake_Case or camelCase could be helpful too

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

We closed this issue because we don't plan to address it in the foreseeable future. If you disagree and feel that this issue is crucial: we are happy to listen and to reconsider.
If you wonder what we are up to, please see our roadmap and issue reporting guidelines.
Thanks for your understanding, and happy coding!


Activate the kubernetes profile without API requests

Is your feature request related to a problem? Please describe.
Currently, when spring-cloud-kubernetes starts up, it decides to activate the kubernetes profile based on whether a get pod call succeeds. This requires that the API server be up and reachable, and it requires that the application have permissions to get pods.
Describe the solution you'd like
Activation of the kubernetes profile could be based, for example, on the presence of an environment variable
Describe alternatives you've considered
I guess the alternative could be to manually activate the profile, but then the issue is that each microservice that uses spring-cloud-kubernetes would have to be configured to activate the profile when running in kubernetes.

@ryanjbaxter can I help with this task?

@ThomasVitale for sure!  We would appreciate it!

why don't we add generateSimplePermissions: true to the base chart? from the wiki: https://autonomic-ai.atlassian.net/wiki/spaces/DnD/pages/1319764138/2020-10-15+Default+service+account+losing+kubernetes+permissions
all cases need generateSimplePermissions: true.

@ryanjbaxter I opened a PR here: https://github.com/spring-cloud/spring-cloud-kubernetes/pull/671
I added a check for the presence of the KUBERNETES_SERVICE_HOST env var. I did some research and couldn't find a final confirmation that the env var is always set (there have been a few issues registered about it), so I also kept the existing check as a fallback. What do you think?


feat(material/tabs/testing): polish harness API

Add label filter for MatTabHarness
Add ability to filter returned harnesses in MatTabGroupHarness#getTabs
Add a method for selecting a child tab to the MatTabGroupHarness
Remove getContentElement and replace it with getHarnessLoaderForContent

@jelbourn @devversion Curious about your opinions: My reason for removing getContentElement is that I want to avoid returning TestElement objects from our harnesses. I don't think they're particularly useful to test authors, and instead we probably want APIs that are more focused in purpose. I do think it makes sense, however, to allow users to get some handle on the content. I think the thing users are most likely to do with the content element is either query for more harnesses within it, or query outside of the harness system (e.g. document.querySelector). I settled on returning a selector because it allows the user to do either of those things. Another option would be to provide a method for loading a sub-harness directly (they pass the harness class and we return an instance). We either could provide this method instead of or in addition to the getSelectorForContent method.
Update: Changed getSelectorForContent to getHarnessLoaderForContent

As discussed on slack, switch to returning a HarnessLoader for the tab content, PTAL.


change the regitstry item name

https://github.com/MicrosoftDocs/windows-itpro-docs/issues/6641

Dear @JohanFreelancer9,
kindly do the copy editor review for this PR.
Dear @mypil, for your information
Thank you

We're retiring this topic, but OK.


Pod style with tailwind inside in-repo engine doesn't processed correctly

Hi, I'm having an issue when trying to use tailwindcss inside an in-repo engine. I have tailwindcss package installed in my main application - but when trying to write custom SCSS files inside my in-repo engine - using tailwindcss classes, for example:
.custom-class {
   @apply p-0;
}

As part of the build process, it takes that SCSS as is so it doesn't actually work. Is there a way to make it work without installing tailwindcss package in the in-repo engine as well?

@taltcher could you please provide a minimal reproduction in a public repo?

Any reason why you don't want to install tailwindcss as a dependency of the engine? If it uses tailwindcss, then it needs tailwindcss to be installed plus the corresponding setup I would think. The same would apply to a normal addon, so I don't think this issue is related to ember-engines as well.

@taltcher is this still an issue for you? Can you close if otherwise?


chore: moving events, interfaces and types to @openscd/core

closes #1476

not a small PR :D
Have you done it manually?  How big is the risk that something won't work?

I've done it manually, since it is not a change of the code itself but it's just some lines being moved from one file to the other, I expect the current tests with the updated imports to be enough. Fun fact: 176 files only changed their import statements.

Maybe interesting to discuss this in the review as well https://github.com/openscd/open-scd/issues/1476#issuecomment-2068696959


php72: 7.2.29 -> 7.2.31

Motivation for this change
Changelog: https://www.php.net/ChangeLog-7.php#7.2.30
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

This refs #85427

cc @NixOS/php
@ofborg build php72

@GrahamcOfBorg build php72

Hm, we get two test failures on aarch64.. Wonder if it's us or upstream; either way, we should probably disable the tests for php72 on aarch64.

@Izorkin @talyz did you find out why the build is broken on aarch64?

@GrahamcOfBorg build php72.unwrapped

@GrahamcOfBorg build php72

Error build plugin php72Extensions.intl.
I have no way to testing build on architecture aarch64-linux and x86_64-darwin.

I have no way to testing build on architecture aarch64-linux and x86_64-darwin.

Have you considered adding a PR here to gain access to some aarch64 infra: https://github.com/nix-community/aarch64-build-box/

I have no way to testing build on architecture aarch64-linux and x86_64-darwin.

Have you considered adding a PR here to gain access to some aarch64 infra: https://github.com/nix-community/aarch64-build-box/

Is there a similar variant for x86_64-darwin architecture?

Is there a similar variant for x86_64-darwin architecture?

AFAICS no. But fixing aarch64 would still be a step forward.

Superseded by #91495.


Vendorizing google-cloud-ruby only packages google-protobuf for current OS

If you vendorize a gemset that includes google-cloud-ruby (bundle package --all), the vendor folder will only include the google-protobuf .gem for the current platform.  This results in a missing gem error if you try to bundle install --local on another platform.
For example: if I bundle package --all on OS X, vendor/cache will include google-protobuf-3.3.0-universal-darwin.gem.  If I try to run bundle install --local inside a Docker container, the install fails because google-protobuf-3.3.0-x86_64-linux.gem does not exist.
As a workaround, it's possible to manually copy the set of platform variants into vendor/cache.  But it's a bit of an annoyance since this has to be done each time you update the version of the gem.

This looks like a Bundler issue to me. Can you suggest any changes we can make to support the behavior you are looking for?
My understanding is that for Bundler to support multiple platforms you need to add the platform using bundle lock, and then to package using the --all-platforms option. But in my tests this doesn't work right, as I'm seeing the same behavior described in bundler/bundler#5863.

Thanks for the lead!  Did some more tracing and there is a bug/undocumented quirk in bundler 1.15.1 where the --all-platforms flag doesn't work unless the specific_feature feature flag is tripped.
Here's what ended up working for me:
bundle config specific_platform true
bundle lock --add-platform=x86_64-linux
bundle package --all-platforms

This results in a vendor/cache with
google-protobuf-3.3.0-universal-darwin.gem
google-protobuf-3.3.0-x86_64-linux.gem
google-protobuf-3.3.0.gem

which works nicely because the platform-less version will compile at bundle install time on platforms without specific entries in vendor/cache.

Glad you figured it out. Thanks for sharing the solution!


Update main.py

Remove meow_gif.py due to links being saved in a separate module. Merged into one file and exploring alternative options such as using the Giphy API to avoid manual link saving.


Improve architecture

Currently, we have a couple of problems :

Controllers that handle business logic.
Plugins lack clear modularity, making it harder to distinguish between core infrastructure (like authentication) and domain-specific logic.
The structure is organized by technical concerns (e.g., routes, plugins) rather than by business domains, which can make it challenging to locate and manage related functionalities as the codebase grows.

It is ok for a very small demo, but it becomes inconvenient to maintain if we add more features.

I think controllers should be dedicated to HTTP handling (e.g., request/response handling, parameter validation...).

We can rename the plugins folder to something more representative, like infrastructure or shared, to reflect that these are reusable components providing core capabilities across the project.

Maybe we should rename routes to domains or modules to represent distinct business areas (e.g., auth, tasks, etc.).

src/
├── infrastructure/           # Core reusable logic
│   ├── authorization.ts
│   └── scrypt.ts
├── domains/                  # Business domains
│   ├── auth/
│   │   ├── controllers/
│   │   ├── services/
│   │   ├── schemas.ts
│   │   └── routes.ts
│   ├── tasks/
│   │   ├── controllers/
│   │   ├── services/
│   │   ├── schemas.ts
│   │   └── routes.ts
│   └── ...
└── app.ts

Wdyt? Do you want to work on it?

Ping for feedback: @climba03003 @melroy89 @Fdawgs

I'd keep it as routes and plugins as then the language is consistent with how the official Fastify documentation and plugin docs refer to these concepts, which should make it easier to understand.
For example, I have a database book that switches between using tuple, row, record, and entry, and it's a pain to follow along.

I fully support this initiative to reorganize the codebase, and i can heavily help on that.
Currently, controllers are handling too much, including business logic, which makes file navigation and code comprehension more challenging.
Also the reorganization by "buisness" domains can be very beneficial, so each component purpose is much clearer, intuitive...

I'd keep it as routes and plugins as then the language is consistent with how the official Fastify documentation and plugin docs refer to these concepts, which should make it easier to understand.

For routes, certainly easier to understand since it's standard/present on most demos.. Imo it is not really representative of the real purpose of the components inside.

You can take inspiration from this repo created by Matteo.
https://github.com/mcollina/modular_monolith/tree/main/modular
On Sat, Nov 2, 2024, 15:20 Mouad @.***> wrote:

I'd keep it as routes and plugins as then the language is consistent with
how the official Fastify documentation and plugin docs refer to these
concepts, which should make it easier to understand.
For routes, certainly easier to understand since it's standard/present on
most demos.. Imo it is not really representative of the real purpose of the
components inside.
—
Reply to this email directly, view it on GitHub
https://github.com/fastify/demo/issues/52#issuecomment-2453006061, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A2J2324KOIZYYSDCNDAX4QTZ6TNRZAVCNFSM6AAAAABRBXKBWGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJTGAYDMMBWGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

For the project structure that I currently using,
src/
├── plugins/                      # shared / global plugins (e.g. database)
│   ├── database.ts
│   └── config.ts
├── routes/                       # business logic with routes
│   ├── auth/
│   │   ├── index.controller.ts   # controller for database operation
│   │   ├── index.route.ts        # routes
│   │   ├── index.route.schema.ts # schema for routes - used to prevent cyclic import
│   │   └── index.schema.ts       # schema for database object
│   ├── tasks/
│   │   ├── index.controller.ts   # controller for database operation
│   │   ├── index.function.ts     # complicated logic that would reuse
│   │   ├── index.route.ts        # routes
│   │   ├── index.route.schema.ts # schema for routes - used to prevent cyclic import
│   │   └── index.schema.ts       # schema for database object
│   └── ...
├── app.ts                        # fastify application
└── index.ts                      # entrypoint

Basically, the domain you mention is scoped inside the route, and you can follow the routes name to find which files you need to edit.
Personally, I do not use @fastify/autoload and routes should be all encapsulated.

Personally, I do not use @fastify/autoload

Weird, I see you contributing to it, why?

I think if this PR is accepted, we should implement the changes here.
Fastify need to improve dependency injection I think.

My setup is currently looking like this (and yes I'm using autoload for sure):
tree -d src
src
├── handlers
├── interfaces
├── plugins
├── routes
│   └── api
│       └── v1
└── schemas

Autoroutes are used for plugins and routes.
I call it plugins because I use FastifyPlugin(Async), so I think it fit the purpose and meaning of the directory structure.
My handlers are the routes with the business logical, which I believe was also a good practice by Matteo himself, since decoupling the routes too far away from the logic can increase technical depth.
Then again, in my case Fastify is used as backend. The handlers are only doing database calls. If you have much more logical in the handles, I would advice to split that further. But again, in my use-case this level of abstraction is sufficient enough.
DB Pagination, advanced DB queries or payload validation is all done with plugins like preHandlerHookHandler or decoorate  or just addHook. So here is currently more advance logical.

Fastify community is too anarchic, we should create a fastify-opiniated package 😁

@Mathieuka IMO for a demo, I believe wrapping each 'action' (command or query) in a separate use case is excessive. This adds another layer (so + mental load and "complexity")—especially for something meant to be "straightforward". As @jean-michelet mentioned, Fastify lacks built-in dependency injection, but (correct me if i'm wrong) we can achieve a similar effect with the plugin mechanism.
routes -> controllers -> service -> output (e.g., database or file handling).
Using Fastify plugins, we can consider services like plugins, so we can injecte then into controllers via routes that have access to the app instance.
We could also instead of separating each route into its own file, consolidate all routes in one file per domain and handle separation at the controller layer or not at all.
However, I believe keeping a file per route still has its advantages. It gives us a clear, quick overview of each domain’s actions and makes it easy to locate and modify specific routes as needed.
Other thoughts on this approach?
src/
├── lib - infrastructure - shared .../
│   ├── authorization.ts # I think that also each domain should have his own authorization.
│   └── scrypt.ts
├── domains/
│   ├── auth/
│   │   ├── auth.controller.ts
│   │   ├── auth.service.ts
│   │   ├── routes/
│   │   │   ├── index.ts
│   │   │   ├── login.ts
│   │   │   ├── logout.ts
│   ├── user-account/
│   │   ├── user-account.controller.ts
│   │   ├── user-account.service.ts
│   │   ├── routes/
│   │   │   ├── index.ts
│   │   │   ├── register.ts
│   │   │   ├── update-account.ts
│   │   │   ├── update-password.ts
│   │   │   ├── delete-account.ts
│   │   │   ├── ...
│   ├── tasks/
│   │   ├── tasks.controller.ts
│   │   ├── tasks.service.ts
│   │   ├── routes/
│   │   │   ├── index.ts
│   │   │   ├── create-task.ts
│   │   │   ├── assign-task.ts
│   │   │   ├── upload-image.ts
│   │   │   ├── delete-image.ts
│   │   │   ├── ...
│   └── ...
└── app.ts

@Turn0xx
For a demo intended to be "simple," I agree that the complexity might indeed be unnecessarily high.
I don't yet know the plugin mechanism well enough.
Consolidating all the routes into a file by domain and managing the separation at the controller level seems like a good idea.
This approach mitigates complexity and offers interesting cohesion from a readability standpoint.

@Turn0xx
My question is whether controllers, routes, and services should be part of the domain.
Shouldn't they belong to the application layer instead ?
The domain should be self-sufficient through its modules and entities, if necessary ?
auth/
├── domain/
│   ├── entities/
│   ├── services/
│   └── authorization.ts
└── application/
    ├── auth.controller.ts
    ├── auth.service.ts
    └── routes/
        ├── index.ts
        ├── login.ts
        └── logout.ts

However, the trade-off you propose might be sufficient for the needs of the demo

As @jean-michelet mentioned, Fastify lacks built-in dependency injection

This is not what I said, I said it could be improved.
A lot of developers think about dependency injection trough constructor injection:
@Injectable()
class DateFormatter {
  format(date: Date): string {
    return date.toISOString();
  }
}

@Controller('some-prefix') // prefix routes path defined in the class
class AppController {
  constructor(
    private readonly dateFormatter: DateFormatter,
  ) {}

  @Get('/log-date')
  logCurrentDate(): { message: string } {
    const currentDate = new Date();
    const formattedDate = this.dateFormatter.format(currentDate);
    return { message: `${this.prefix} ${formattedDate}` };
  }
}

But the same is achievable via a functional paradigm:
function formatDate (date) {
  return date.toISOString()
}

fastify.register(function (instance, deps) {
  instance.get('/log-date', (request, reply) => {
    const currentDate = new Date()
    const formattedDate = deps.formatDate(currentDate)

    reply.send({ message: `Format date: ${formattedDate}` })
  })
}, {
  formatDate, // Function injection
  prefix: '/some-prefix' // prefix routes path defined in the plugin
})

@Mathieuka

Shouldn't they belong to the application layer instead ?
The domain should be self-sufficient through its modules and entities, if necessary ?

Totally agree with you,all actual components that i cited should reside in the application layer. In my example, I intended 'domains' simply to represent areas of work or functionality, rather than fully self-sufficient domains.

My point is that if we want to bring about the separation of responsibilities, it will be done through the entities, otherwise we will end up with the same problem of controllers who manage the state, apply the business rules, and persist the data.

Ideally, controllers won’t manage any of this. This should be handled by the service layer, and I don’t think the demo has enough business complexity to introduce entities as it add another another layer..
However, as you mentioned, I don’t think this level of separation is necessary for the demo, especially since it’s an "introductory" demo (I guess xD). So it should be accessible for all potential users, who may find to much layering / separation overwhelming or complex at this stage.

@Turn0xx fully agree about the demo has not enough business complexity to go in this direction
I took the time to read the plugin system documentation and I understand your arguments better.

Using Fastify plugins, we can consider services like plugins, so we can injecte then into controllers via routes that have access to the app instance.

I think a more modular approach could be beneficial, and it will be easily achievable when the PR I mentioned in this comment will be merged.
Example of what could be achieved:
export const deepPluginDependency = createPlugin((instance) =>
  instance
    .decorateReply('deep_dependency_reply', true))

export const pluginDependecy = createPlugin((instance) => instance
  .decorateRequest('dependency_request', true), { dependencies: [deepPluginDependency] })

export const dependantPlugin = createPlugin((instance) => {
  return instance.get('/', (req, res) => {
    expectType<boolean>(req.dependency_request)
    expectType<boolean>(res.deep_dependency_reply)
  })
}, { dependencies: [pluginDependecy] })

It would be plugin injection, and the plugin decorate the fastify instance, request and reply with features.


LDO THT Board BOM is expensive because of overkill parts

I tweaked the BOM to prevent people from having to order from Digikey, Mouser, and Amazon, and merged it all into one Digikey BOM to save on shipping costs. I also added prices to show that buying 10 of each piece would reduce the fully assembled PCB cost if they managed to get 9 other people interested in buying them (most PCB fab houses require ordering 5-10 boards minimum anyways). I changed C0G caps to X7R, switched form Nichicon to Panasonic with same approximate performance ratings, and also removed 100V caps in favor of some cheaper 50V caps since the highest voltage entering this board is approx 9V AC/DC. I would argue that you could even bump the 2200uF cap down to 16V instead of 25V, but I didn't want to bother with that one that much. The spreadsheet is down below. Feel free to steal this or any part of this for your next BOM version. Thanks! :)
https://docs.google.com/spreadsheets/d/1N5DvwrXRiYk291FSXcJ2-mtGektX3NYngcAIhR5h08c/edit#gid=0

Thank you for your work.  I am no longer developing these but I will leave this post up so that others can see it.


Getting 4.2.6 to PyPi

There seem to be some problems with the master branch. Unpushed commits anyone? @ibell ?

build/temp.macosx-10.5-x86_64-2.7/pyrex/CoolProp/phase_constants.cpp:739:66: note: forward declaration of 'phase'
static CYTHON_INLINE PyObject* __Pyx_PyInt_From_enum__phase(enum phase value);

I don't have time now, but I can have a look again later today or tomorrow.

I'll take a look.  Worst case we roll back to 4.2.5 plain, and build that,
which should work.  I'll see if I have missing commits on 4.2.6 on my
computer - I was able to get it to build on windows.
On Sat, Feb 13, 2016 at 6:52 AM, Jorrit Wronski<EMAIL_ADDRESS>wrote:

build/temp.macosx-10.5-x86_64-2.7/pyrex/CoolProp/phase_constants.cpp:739:66: note: forward declaration of 'phase'
static CYTHON_INLINE PyObject* __Pyx_PyInt_From_enum__phase(enum phase value);
I don't have time now, but I can have a look again later today or tomorrow.
—
Reply to this email directly or view it on GitHub
https://github.com/ibell/coolprop/issues/263#issuecomment-183669350.


Account for possible null/undefined during redaction

test failures are due to an unrelated change in Undici dependencies. will investigate in a separate branch.


Patch policy library for Azure China

Overview/Summary
This PR ensures policy definitions/policy set definitions along with policy assignments all work in Azure China. Internal SE backlog item here. This is because there are some policy definition differences in China especially wrt to the MS Defender for Cloud policies, hence there was a separate policy assignment definition  in the upstream enterprise-scale repo.
Just to be sure, re-tested the entire deployment flow as per https://github.com/Azure/ALZ-Bicep/wiki/DeploymentFlow and made some fixes.
This PR fixes/adds/changes/removes

Tested default policy assignments by introducing a new infra-as-code\bicep\modules\policy\assignments\alzDefaults\mc-alzDefaultPolicyAssignments.bicep file.
Added separate lib/china folder under infra-as-code\bicep\modules\policy\assignments\lib\china\policy_assignments. Also added default policy assignments which would work in Azure China.
Updated and tested Mooncake workflow file, and script to create separate lib/china folder under policy/assignments.
Added customer usage attribution module in customRoleDefinitions module.
Also fixes the deployment errors during MC testing found by @jfaurskov in #227

Breaking Changes

None

Testing Evidence

Tested workflow manual run successfully in my forked repo

As part of this Pull Request I have

[x] Read the Contribution Guide and ensured this PR is compliant with the guide
[x] Checked for duplicate Pull Requests
[x] Associated it with relevant GitHub Issues
[x] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[x] Ensured my code/branch is up-to-date with the latest changes in the main branch
[x] Performed testing and provided evidence.
[x] Updated tests (if required) Unit - Linting - E2E (End-To-End)
[x] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[ ] If relevant, created or updated Code Tours here

/azp run ValidateMcCloud

Nice work @faister :-) Added a couple of suggestions. Also If you could merge with main and put the validation of the mc-alzdefaultpolicyassignments into the /tests/pipelines/mc-base-unit-validate.yml pipeline in place of the current alzdefaultpolicyassignments (line 81 - 87) that would be ace!

Thanks @jfaurskov, done updated tests/pipelines/mc-base-unit-validate.yml and tested it in az cli successfully. see attached output.
Az CLI Validate Alz Default policy assignments-succeeded.txt

/azp run ValidateAzCloud

/azp run ValidateMcCloud

/azp run ValidateAzCloud


Change TruckMap universal link url to https

Looks good! Happy to merge if you would mind updating the corresponding test so that the unit tests continue passing.

Fixed in #221. Closing this one. Thanks for your contribution @naftalibeder!


[cxx-interop] Copy SwiftBridging modulemap into the build directory

The modulemap file is already copied to the installation directory, and shipped with the toolchain. This change makes sure it is also copied over to the build directory, which simplifies testing of the just-built compiler.

@swift-ci please smoke test


Update notebooks to point at titiler-pgstac.maap-project.org instead of titiler.maap-project.org

In pursuit of a single consolidated titiler deployment for MAAP:

[x] docs/source/technical_tutorials/visualization/stac_ipyleaflet.ipynb
[x] docs/source/technical_tutorials/user_data/create-datasets-for-dashboard.ipynb
[x] docs/source/technical_tutorials/visualization/visualize_mosaic_stac_ipyleaflet.ipynb
[x] docs/source/technical_tutorials/visualization/interval_color_mapping.ipynb

@hrodmn if we're merging the functionality, then don't we just need to deploy titiler-pgstac to the original titiler.maap-project.org and call it done?

@hrodmn if we're merging the functionality, then don't we just need to deploy titiler-pgstac to the original titiler.maap-project.org and call it done?

titiler-pgstac.maap-project.org has all* of the endpoints that we supported at the titiler.maap-project.org deployment, so in theory we could just flip a switch and point all titiler.maap-project.org to the titiler-pgstac Lambda.
* /cog/crop -> /cog/bbox
I suppose we can pick whatever url we want to stick with long-term and associate it with the titiler-pgstac deployment. I can see advantages/disadvantages to either url:
titiler.maap-project.org:

advantages

users do not need to update urls in notebooks

disadvantages

once we flip the switch it could break functionality for a few limited use-cases

titiler-pgstac.maap-project.org:

advantages

keeping the new domain would allow us to add some deprecation warnings to responses in titiler.maap-project.org endpoints

enables a "phaseout" period for titiler.maap-project.org

no changes required to domain name infrastructure, just need to tear down titiler.maap-project.org at some point

disadvantages

somewhat clunky domain name
adding a "phaseout period" where we maintain both APIs but add deprecation warnings might be more trouble than it is worth

Decided to not do this until we know what domain we want to use.


Allow builtin default override

Currently the BUILD_EXTENSTIONS_AS_BUILTIN applies the BUILTIN_ option to all extensions.  This branch instead uses it as a default value to the BUILTIN_FOO option, letting the user override for any particular extensions.

Thanks for your contribution :+1:
I am closing this PR. See #75 is a revised version.
Note that the command mark_as_advanced was incorrectly used.
See http://www.cmake.org/cmake/help/git-next/command/mark_as_advanced.html?highlight=mark_as_advanced


Missing Default Notification Channel metadata in AndroidManifest. Default value will be used

i have issue in notification when i open app and receiving notification from firebase its working
but when i out of app and receiving notification from firebase i get this issue
Missing Default Notification Channel metadata in AndroidManifest. Default value will be used
my code
Future _showNotificationWithDefaultSound(String t, String b,String id) async {

    var androidPlatformChannelSpecifics = new AndroidNotificationDetails(
        'your channel id', 'your channel name', 'your channel description',
        importance: Importance.Max, priority: Priority.High);
    var iOSPlatformChannelSpecifics = new IOSNotificationDetails();
    var platformChannelSpecifics = new NotificationDetails(
        androidPlatformChannelSpecifics, iOSPlatformChannelSpecifics);
    await flutterLocalNotificationsPlugin.show(
     int_notification,
      t,
      b,
      platformChannelSpecifics,
      payload: 'Default_Sound',
    );
  }

my Manifiest.xml
<meta-data
                android:name="com.google.firebase.messaging.default_notification_channel_id"
                android:value="default_notification_channel_id"/>

@Mostafa-SA-developer Is it still relevant? Please provide your flutter doctor -v output.

I'm seeing this too. My results are:
✓] Flutter (Channel master, v1.10.2-pre.78, on Linux, locale en_GB.UTF-8)
    • Flutter version 1.10.2-pre.78 at /home/ghenry/development/flutter
    • Framework revision 066cad21a6 (6 hours ago), 2019-09-13 11:05:00 -0700
    • Engine revision 7ea9884ab0
    • Dart version 2.5.0 (build 2.5.0-dev.4.0 be66176534)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /home/ghenry/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /home/ghenry/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5791312/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

[✓] Android Studio (version 3.5)
    • Android Studio at /home/ghenry/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/191.5791312
    • Flutter plugin version 37.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Android Studio (version 3.4)
    • Android Studio at /home/ghenry/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/183.5692245
    • Flutter plugin version 37.1.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b16-5323222)

[✓] VS Code (version 1.37.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.2.0

[✓] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

• No issues found!

@Mostafa-SA-developer
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/343. Any further collaboration will be done there.

I got it .. this is done by hacking firebase plugin
You need to add default notification channel id in main plugin in Android Manifest of plugin  as described by firebase guide  .. that worked for me

You need to add default notification channel id in main plugin in Android Manifest of plugin as described by firebase guide .. that worked for me

@ankii9600 Can you explain how to go about it? How do we change the AndroidManifest.xml of a plugin?

You need to add default notification channel id in main plugin in Android Manifest of plugin as described by firebase guide .. that worked for me

@ankii9600 Can you explain how to go about it? How do we change the AndroidManifest.xml of a plugin?

https://github.com/FirebaseExtended/flutterfire/issues/1478#issuecomment-562104143 check it over here


Move f1_keyword to a more appropriate place

@Youssef1313 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Youssef1313 Thanks for your PR; we appreciate you taking the time to create it. However, we decline to accept this particular PR. The F1 Help page is designed for internal-to-Microsoft processes (as are the F1 keywords) that might not be obvious to most. As well, we do have a link on the F1 Help page that points to information about the code editor, should someone land on this page who wants that info.

@TerryGLee Pressing F1 in Visual Studio with no open tabs redirected to that not-in-toc/details page based on the f1_keywords, I thought it's more appropriate to redirect just to the suitable page.


Disable fullscreen mode

Hi, it's possible to disable the click of the icon of full screen on the control bar to prevent the fullscreen mode?
Thank's

yes extends JZVideoPlayerStandard， overrite onClick function,  and intercept if(fullscreen button click).

or gone the fullscreen button.


Fix dialyzer issues

Thanks for the great project! I originally heard about it on your episode of Thinking Elixir.
I pulled Mint.WebSocket into my project, and found a bunch of dialyzer issues. This PR aims to address all of the dialyzer issues, as well as adding to CI.

Pull Request Test Coverage Report for Build af9b5676d8bec085b57fa93016ce833474eeb7b7-PR-23

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.952%

Totals

Change from base Build 69ed3cc832b49aa8b56658818847bd41ad874be3:
0.0%

Covered Lines:
314

Relevant Lines:
353

💛  - Coveralls

This has been taken care of in #27 so I will close this out, but thank you for your PR!

Sorry I couldn't get back to this, but that's for resolving! @the-mikedavis

Pull Request Test Coverage Report for Build af9b5676d8bec085b57fa93016ce833474eeb7b7-PR-23
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.952%

Totals

Change from base Build 69ed3cc832b49aa8b56658818847bd41ad874be3:
0.0%

Covered Lines:
314

Relevant Lines:
353

💛  - Coveralls

Pull Request Test Coverage Report for Build af9b5676d8bec085b57fa93016ce833474eeb7b7-PR-23
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.952%

Totals

Change from base Build 69ed3cc832b49aa8b56658818847bd41ad874be3:
0.0%

Covered Lines:
314

Relevant Lines:
353

💛  - Coveralls


Not including chevrotain in the bundle, Example 03 issues are fixed

I added npm run watch to build core with watch
chevrotain is not included for bundle, we need to load it separately before loading the IFC.js
Example 03 is fixed, it can be used as main example and we can remove other examples. Example 03 also have file reader.

Great! 🙂 I will merge this today or tomorrow.
What is the dist folder for? If this belongs to your example, wouln't it be better to have everything inside the 03 folder? Also, maybe the duplicate models folder is not necessary.

dist folder is trash. it was in gitignore, so it didnt get removed from my local and I pushed it by mistake.
It is removed now


Can ICMP be stateful or not?

Can you add stateful ACL rules (on a Hyper-V Virtual Switch) on the ICMP protocol?
If not, this leaves you to either open ICMP to everyone or close ICMP to everyone including the VM itself.
Neither is secure or practical for such an important and basic functionality (ping).

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

ID: 84e15063-e5c5-3bf1-e4e6-c80dcf234ccd
Version Independent ID: 46a2fe81-8a1b-a2f0-39f1-3a79d6cc0f1a
Content: Add-VMNetworkAdapterExtendedAcl (hyper-v)
Content Source: docset/windows/hyper-v/Add-VMNetworkAdapterExtendedAcl.md
Product: w10
Technology: powershell-windows
GitHub Login: @andreabarr
Microsoft Alias: v-anbarr

@andoryuu-cloud Looking at the article you mentioned, there is the posibility to choose ICMP for the "-Protocol" parameter. Is this not what you are looking for?

@andoryuu-cloud
The issue is being closed since your question seems to be answered. Feel free to re-open this issue if you feel that it hasn't been answered or that there are further suggestions to improve the documentation.
Thanks.

The the "-Protocol" parameter for ICMP in powershell does not work hence my question.

@andoryuu-cloud Are you getting an error, or what are you seeing? Could you provide the complete cmdlet syntax you are using?

Command:
Add-VMNetworkAdapterExtendedAcl -VMName "5f1521b8-66cb-401d-b23f-4f6e373f28ab" -Action Allow -Direction Outbound -Protocol "ICMP" -Weight 100 -Stateful $True
Error:
Add-VMNetworkAdapterExtendedAcl : The operation failed.
Failed while applying switch port settings 'Ethernet Switch Port Extended ACL Settings' on switch 'Ext_Team1': One or m
ore arguments are invalid (0x80070057).
At line:1 char:1

Add-VMNetworkAdapterExtendedAcl -VMName "5f1521b8-66cb-401d-b23f-4f6e ...

  + CategoryInfo          : InvalidArgument: (:) [Add-VMNetworkAdapterExtendedAcl], VirtualizationException
  + FullyQualifiedErrorId : InvalidParameter,Microsoft.HyperV.PowerShell.Commands.AddVMNetworkAdapterExtendedAcl

It won't even work without the stateful option, you have to type "1" instead of "ICMP" to work.
However, typing "1" instead of "ICMP" won't solve the stateful issue:
Command:
Add-VMNetworkAdapterExtendedAcl -VMName "5f1521b8-66cb-401d-b23f-4f6e373f28ab" -Action Allow -Direction Outbound -Protocol 1 -Weight 100 -Stateful $True
Error:
Add-VMNetworkAdapterExtendedAcl : The operation failed.
Failed while applying switch port settings 'Ethernet Switch Port Extended ACL Settings' on switch 'Ext_Team1': One or more arguments are invalid (0x80070057).
At line:1 char:1

Add-VMNetworkAdapterExtendedAcl -VMName "5f1521b8-66cb-401d-b23f-4f6e ...

  + CategoryInfo          : InvalidArgument: (:) [Add-VMNetworkAdapterExtendedAcl], VirtualizationException
  + FullyQualifiedErrorId : InvalidParameter,Microsoft.HyperV.PowerShell.Commands.AddVMNetworkAdapterExtendedAcl

@andoryuu-cloud Could you also try specifying a value for "-RemoteIPAddress"? If you are still getting the same error, i recommend opening a support ticket to have someone from support look into the error and syntax and see why this is failing.
Afterwards we will be able to conclude if anything needs to be changed in the article.

@VLG17 So you're saying I have to pay for a support ticket just so you can fix the article or the issue?
Also, -RemoteIPAddress is an invalid parameter with icmp.

@andoryuu-cloud I was not trying to imply that you need to pay to get this fixed. Under the link I provided, there are multiple options, some of which are free.
What i can help with is changing something in the documentation. But first we need to understand what needs to be changed. I cannot help with troubleshooting steps in order to isolate the issue, That needs to be handled separately by support. You can leverage the new Microsoft Q&A (in preview).
What i will do in the meantime is reach out to the authors of the article and try to get their insights into this issue.

@andoryuu-cloud
The independent contributors could not reproduce the problem in a standardized environment.
PowerShell Community would, therefore, be a better channel to discuss this issue because the focus of this repository is content fixes.
Feel free to reply if there is anything else that could be done to improve the document itself.
Thanks.

@e0i So the independent contributors got the powershell command in question to work?

@andoryuu-cloud I've tested with 1 instead of ICMP and it worked (replaced sensitive data with xxxxxxxx ):
PS C:\Windows\system32> Add-VMNetworkAdapterExtendedAcl -VMName "xxxxxxxx" -Action Allow -Direction Outbound -Protocol 1 -Weight 100
PS C:\Windows\system32> Get-VMNetworkAdapterExtendedAcl -VMName xxxxxxxx
Direction          : Outbound
Action             : Allow
LocalIPAddress     : ANY
RemoteIPAddress    : ANY
LocalPort          : ANY
RemotePort         : ANY
Protocol           : 1
Weight             : 100
Stateful           : False
IdleSessionTimeout : 0
IsolationID        : 0
ParentAdapter      : VMNetworkAdapter (Name = 'Network Adapter', VMName = xxxxxxxx) [VMId = '35f6a9d3-c293-4f06-92e1- xxxxxxxx ']
IsTemplate         : False
CimSession         : CimSession: .
ComputerName       : xxxxxxxx
IsDeleted          : False

@VLG17 Thanks for trying it out.
At least now we know that the article needs amending since the command does not take ICMP but instead a "1".
Back to my original query:
The article states that the extended ACLs can be stateful but even swapping "ICMP" with "1" the stateful parameter is not recognized. This either means that there is a bug with the powershell command or the article needs to specify that this is for TCP/UDP only. Could you kindly let us know which one is it?

@andoryuu-cloud I'll amend that.
Regarding using the -Stateful parameter with "1" (meaning ICMP), i could also reproduce the issue and error you are seeing I have also tested it for UDP, and it works with -Stateful $true, so the only problem is for 1 (ICMP).
Keeping this in mind, i think we need to ask @e0i to help with this part.

Closing via #1893

That's great, however, I'm disappointed that this issue has been masked and nothing else.
At least contact the Windows Server team to fix the issues here:
1.) Make "ICMP" valid in the powershell command instead of having "TCP", "UDP", and "1" ...
2.) Make ICMP stateful (it's 2020, cmon) ... or update the document stating that the stateful argument can only be done on TCP or UDP.

Any updates?

@andoryuu-cloud
The issue has been closed because the associated article has been updated according to your feedback.
The issues section of this repository is exclusively for documentation problems.
Please consider sending feedback to Microsoft with the Feedback Hub app, or checking with Microsoft Community for your feedback.
Thank you.

This repository is for PowerShell core documentation. You seem to be looking for support, which we can't provide here.
I suggest you try posting your issue with context in one of the available community support forums.

@scanum Can this be checked with the hyper-v team?


Made IntelliJ Inspection "Iteration over keySet() could be replaced with entrySet()" a error & Fixed the violations

Issue #6856
Make IntelliJ inspection "Iteration over keySet() could be replaced with entrySet()" a error
Old PR is messed up. Please consider this PR for review.

@leventov it's failing at Failed to handshake with host[https://<IP_ADDRESS>:8101]. Can you try running the tests again?


Asynchronous Restore Progress in Longhorn Manager

Earlier, the replica controller was doing restore when bringing up the replicas
but now it's executed by volume controller after all the replicas have come up.
Specifically, as soon as a restoration volume is created, it's automatically
attached and detached and in this window, the backup restore engine command is
executed to restore all the replicas.

We also need to expose the progress for each replica as a map[string]*RestoreStatus in the engine, for UI to get the restore progress.

Also, the logic in replica_controller regarding restoring replica before starting needs to be removed.
https://github.com/longhorn/longhorn-manager/blob/master/controller/replica_controller.go#L361

Close in favor of https://github.com/longhorn/longhorn-manager/pull/345


game crashes when i try to load

im trying to get my game to load but it crashes with the error java.lang.NoClassDefFoundError: exnihilocreatio/tiles/TileCrucibleStone

Same problem here is the  Crash Report
https://paste.dimdev.org/asazekekah.mccrash
I didn't have this bug before I add/update these mods :

Add mods :
LagGoggles
MineColonies
xXx_MoreToolMats_xXx (Plustic)
+Libs

-Update Mods :
Botanic Additions
CreativeCore
Galacticraft Add-on - More Planets
Hammer (Lib) Core
Ice and Fire: Dragons in a whole new light!
Immersive Posts
LittleTiles
RandomPatches
Redstone Arsenal
Refined Exchange
Refined Storage
Refined Storage Addons

I don't even have lag goggles I got rid of it I'm confused by that

LagGoggles does weird stuff to events, I would suggest reporting this issue to them.

Looked on LagGoggles/TickCentral, author claims to have fixed it.


Cherry-pick to earlgrey_1.0.0 [rom_ext_e2e] Add an ownership transfer test

This is a manual cherry-pick of #24419.

Create a library of helper functions for facilitating ownership transfer tests.
Create a basic ownership transfer test that transfers chip ownership from the default fake test owner to a dummy owner.

I manually triggered a CI run since the Azure Pipelines issue we are seeing was not triggering it.


crawler.response becomes null when connecting to specific chrome instance

What is the current behavior?
Okay, I dont know if this should be classified as a bug or not. But since I cant seem to find any trace of this in the docs, here it is.
When connecting to a specific chrome instance through the:
await HCCrawler.connect({     browserWSEndpoint: wsChromeEndpointurl,      ...more options goes here})  method, the call to await crawl() results in a null response.
The issue lies within:

\node_modules\headless-chrome-crawler\lib\crawler.js

async crawl(){
await this._prepare();
const response = await this._request(); //1. Results in Null
console.log(response); //2. ======> NULL INDEED
await this._waitFor();
const [
  result,
  screenshot,
  cookies,
  links,
] = await Promise.all([
  this._scrape(),
  this._screenshot(),
  this._getCookies(),
  this._collectLinks(response.url),  //3. ====> Exception gets thrown here
]);
return {
  options: this._options,
  depth: this._depth,
  previousUrl: this._previousUrl,
  response: this._reduceResponse(response),
  redirectChain: this._getRedirectChain(response),
  result,
  screenshot,
  cookies,
  links,
}; }

`
Steps to reproduce

Close all chrome instances.
Open CMD and run the following to start Chrome with remote debugging enabled:

start chrome.exe --remote-debugging-port=9222

Navigate to http://localhost:9222/json/version and copy the value of "webSocketDebuggerUrl"
Set the browserWSEndpoint to the value you just copied.

What is the expected behavior?
When running through a clean instance with the
await HCCrawler.launch({
    ...options
})

command, everything works fineand the result comes back as expected.
What am I missing here?
Please tell us about your environment:

Version: 1.8.0
Platform / OS version: Windows 10  / 1803
Node.js version: v10.16.3

I have the same error

Hi @carlkentor , I've made a fork of the project with updated dependencies and a fix that I found in Puppeteer's issues (https://github.com/GoogleChrome/puppeteer/issues/2479#issuecomment-408263504) that seems to fix the issue on my end.
I've also made a issue in Puppeteer because they seem to be the cause of the problem: https://github.com/GoogleChrome/puppeteer/issues/4970
Could you please tell me more about your setup ? Are you using docker ? And could you try my fork ?
To use the fork, change your package.json like this: "headless-chrome-crawler": "Tractr/headless-chrome-crawler-fork"
Have a nice day.

hey @simlevesque! Thanks for the suggestion! I've used it a while ago. Judging by the referenced thread, the issue has been resolved in puppeteer 1.8.0. I'm updating the puppeteer version to the latest v1 version: 1.20.0. So, should be resolved soon.


Release 1.4 on Maven Central

@rultor release tag=1.4

@rultor release tag=1.4

@uklimaschewski OK, I will release it now. Please check the progress here

@rultor release tag=1.4

@uklimaschewski Done! FYI, the full log is here (took me 2min)


git_repository_spec: group together shared examples

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

This PR creates a new shared example group in git_repository_spec named git_repository helper function that reduces duplication of tests. This will be especially useful for when additional helper functions are added to utils/git_repository

Review period will end on 2021-01-20 at 20:37:37 UTC.

Review period will end on 2021-01-20 at 20:37:37 UTC.

Review period ended.

Review period ended.


Push to Github Registry

Proposed changes
Push images to Github Registry
Builds on top of #377
Example run https://github.com/EnriqueL8/paladin/actions/runs/11685400404/job/32538740521

Please make sure to follow these points

[x] I have read the contributing guidelines.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have added tests that prove my fix is effective or that my feature works.

Screenshots
Tested on fork, you can see two Arch images published

I opened #383 instead


single-home the vanes

The Costello to #915's Abbott. This removes all the multi-homing indirection in the state, state-machines, and moves of all the vanes. The closes thing to an exception is %gall's %deal task, which takes a pair of ships, but uses them alternatively for bidirectional communication.
I've tested networking, syncing/merging, and cross-ship app pokes in a number of scenarios, but there's obviously still risk involved in making this many changes across this many places. Drive-by reviews of the area one knows best would be greatly appreciated.

Quality commit style! I drove by jael and parts of clay real quick, it all seems sane. I don't expect much weirdness here anyway, because of how the multi-homing code in vanes was used in practice.
So good to finally have this cleaned up, this used to be so ugly.

Much obliged, @Fang-!

Amazing.
Sent from my iPhone

On Dec 13, 2018, at 12:35 PM, Joe Bryan<EMAIL_ADDRESS>wrote:
Much obliged, @Fang-!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No excessive moving parts!  If you can split it, split it...
Sent from my iPhone

On Dec 13, 2018, at 2:10 PM, Joe Bryan<EMAIL_ADDRESS>wrote:
@joemfb commented on this pull request.
In sys/vane/gall.hoon:

@@ -1392,8 +1390,7 @@
^+  [*(list move) ..^$]
~|  [%gall-take tea]
?>  ?=({@ ?($sys $use) *} tea)

=+  our=(need (slaw %p i.tea))
Yeah, I believe it. I removed some parsing of wires (here and in %clay specifically), but I didn't change the serialization or shape of the wires. I was worried about too many moving parts in this PR. But maybe it'd be better to get all the changes in at once and then test thoroughly on a testnet? I'm feeling conflicted now ...

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Per Curtis, minimize moving parts.

Thanks all, I'll open an issue to track the removal of identity from these wires.


[FEATURE] asset deploy on same instance

right now one can override assets and also deploy across instances but not from 1 BU to another.

closed by mcdev-issue https://github.com/Accenture/sfmc-devtools/issues/341 when mcdev 4.0.0 was released: https://github.com/Accenture/sfmc-devtools/releases/tag/v4.0.0


Add states:DescribeStateMachine to datadogs perms

Adding states:DescribeStateMachine to Datadog permissions to alleviate errors

Closing this and moving the permission to the datadog_tf repo as an extra_policy_arns argument.


Sign Text data cannot be offered to a BlockSnapshot

Sign Text data applied to a BlockSnapshot using the with method has no effect on the BlockSnapshot.

@gabizou
Status update.

This is intentional, as the data is stored with the Sign TileEntity. Offering data to a BlockSnapshot goes through the blockstate, which doesn't support SignData.

A blocksnapshot stores TileEntity data too though. Shouldn't it fall back to the tileentity if the data is not accepted by the blockstate? (or other way around)

@simon816: You're right - I'll be implementing that.

This can be resolved once schematics necessary objects are added to the API, namely TileEntityArchetype and the relevant implemented processors dealing with raw data.

Gonna actually just close this as it can be overall joined with https://github.com/SpongePowered/SpongeCommon/issues/950


KYLIN-4206 Support Glue as Hive Metatdata; KYLIN-4298  Fix Wrong FS

Please check my test (AWS EMR 5.28) at https://github.com/hit-lacus/hit-lacus.github.io/issues/81 .

Pull Request Test Coverage Report for Build 5487

0 of 11 (0.0%)  changed or added relevant lines in 4 files are covered.
6 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.003%) to 27.457%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
0
1
0.0%

metrics-reporter-hive/src/main/java/org/apache/kylin/metrics/lib/impl/hive/HiveProducer.java
0
1
0.0%

source-hive/src/main/java/org/apache/kylin/source/hive/CLIHiveClient.java
0
2
0.0%

source-hive/src/main/java/org/apache/kylin/source/hive/HiveMetaStoreClientFactory.java
0
7
0.0%

Files with Coverage Reduction
New Missed Lines
%

core-dictionary/src/main/java/org/apache/kylin/dict/lookup/cache/RocksDBLookupTable.java
1
81.08%

core-cube/src/main/java/org/apache/kylin/cube/inmemcubing/MemDiskStore.java
5
77.81%

Totals

Change from base Build 5484:
-0.003%

Covered Lines:
24071

Relevant Lines:
87669

💛  - Coveralls

Codecov Report

Merging #1040 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #1040      +/-   ##
============================================
- Coverage     25.01%   24.99%   -0.02%
+ Complexity     6180     6179       -1
============================================
  Files          1437     1438       +1
  Lines         87639    87669      +30
  Branches      12250    12257       +7
============================================
- Hits          21919    21914       -5
- Misses        63568    63601      +33
- Partials       2152     2154       +2

Impacted Files
Coverage Δ
Complexity Δ

.../kylin/source/hive/HiveMetaStoreClientFactory.java
0% <0%> (ø)
0 <0> (?)

.../java/org/apache/kylin/common/KylinConfigBase.java
12.34% <0%> (-0.02%)
44 <0> (ø)

...va/org/apache/kylin/source/hive/CLIHiveClient.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ache/kylin/metrics/lib/impl/hive/HiveProducer.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...he/kylin/dict/lookup/cache/RocksDBLookupTable.java
72.97% <0%> (-5.41%)
6% <0%> (-1%)

...che/kylin/stream/core/storage/CheckPointStore.java
63.63% <0%> (-3.04%)
21% <0%> (-1%)

...n/java/org/apache/kylin/job/dao/ExecutableDao.java
27.51% <0%> (-1.8%)
24% <0%> (ø)

...c/main/java/org/apache/kylin/cube/CubeSegment.java
49.34% <0%> (-1.11%)
64% <0%> (ø)

...rg/apache/kylin/cube/inmemcubing/MemDiskStore.java
69.3% <0%> (-0.92%)
7% <0%> (ø)

.../org/apache/kylin/query/util/PushDownExecutor.java
0% <0%> (ø)
0% <0%> (ø)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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

LGTM


BAU - MoJ Unicorn was jumping, this makes Lion look bad

Remember we have several nodes that serve this content and there are going to be times when they’re all running different versions. So really we want three releases: one to add the SVG, one to make the page use it, and one to remove the PNG…

I used SVG because I wasn’t given a PNG originally, got one now, so switched it out, which negates a need for the other releases


Hi

Things:
-Ability to put multiple demos in and print the highest used fps, lowest used fps.
-Print everything you can possibly print for Goldsrc demos, very similar to this https://gyazo.com/b80350a4cc1b2f2d99c6d58cdeb8feba but if there is more you can print or retrieve from demos it would help. (waits, sv_cheats,sv_gravity,_sv_airaccel,etc. bxt_timer start/stop???, tas camera commands, basically anything you possibly can print pls)
-Most accurate time of demo for IL's (not sure if its already the most accurate), and maybe a total time of demos (sum of all times) when multiple demos are put in.
-checking for the use of these commands can help too, if possible. bxt_autojump, +bxt_tas_autotap
ps. change color of text in the drop down menu's :ytriggered:

I think I pretty much satisfied this. Tomorrow gonna do a release with this in the build.

Check the release if no good make a new issue.


Package depends on R >= 4.1

Here you declare that your package requires a minimum R version of 4.0
https://github.com/Executive-Communications-Dataset/ecdata/blob/2a8de2017d46b7dc169b9deff8492d7ca267edfd/DESCRIPTION#L22
However, you use an anonymous function which wasn't introduced until R 4.1
https://github.com/Executive-Communications-Dataset/ecdata/blob/2a8de2017d46b7dc169b9deff8492d7ca267edfd/R/zzz.R#L3

Great catch!  Bumped the minimal version in the description


Add a method to set possible maxidle params on the transport.

This method is not used anywhere outside pkg right now, so it's a safe change.
This will permit activator to set desired maxidle params and cache more connection for better performance
/assign @tcnghia mattmoor
/cc @julz @markusthoemmes

/approve
/lgtm

/lgtm


temporary fix for failing builds

This is a temporary fix for #1368, which lets make package go through on my computer. This needs a more thorough analysis though.
It is still producing error messages while unpacking:
[info] Unpacking package /Users/.../apalache/target/universal/apalache-0.20.4-SNAPSHOT.tgz to /Users/.../apalache/target/universal
[error] x apalache-0.20.4-SNAPSHOT/
[error] x apalache-0.20.4-SNAPSHOT/LICENSE
[error] x apalache-0.20.4-SNAPSHOT/bin/
[error] x apalache-0.20.4-SNAPSHOT/lib/
[error] x apalache-0.20.4-SNAPSHOT/lib/apalache.jar
[error] x apalache-0.20.4-SNAPSHOT/bin/apalache-mc
[info] Symlinking /Users/.../apalache/target/universal/current-pkg -> /Users/.../apalache/target/universal/apalache-0.20.4-SNAPSHOT
[success] Total time: 8 s, completed Feb 18, 2022 9:42:51 AM

Codecov Report

Merging #1369 (7ce30dc) into unstable (8c86a04) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           unstable    #1369   +/-   ##
=========================================
  Coverage     75.21%   75.21%
=========================================
  Files           334      334
  Lines         10851    10851
  Branches        518      518
=========================================
  Hits           8162     8162
  Misses         2689     2689

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 8c86a04...7ce30dc. Read the comment docs.

@konnov do you want to try to build with this branch?

Thanks! Let me try!


IS 6.2.0 - Installing as a Windows service not supporting JDK 17.

Describe the issue:
When trying to Install Product IS as a Windows Service with JAVA 17 the server won't be starting.
(P.S. -> Tested the flow with JDK 11 and working as expected.)
JDK Version - JDK 17
How to reproduce:

Download Yet Another Java Service Wrapper (YAJSW) version 13.10 (latest)

Copy and paste the configurations from wrapper.conf file which is located inside {wso2is-5.10.0_Home}/bin/yajsw to wrapper.conf file in {yajsw-Home} /conf.

Add a new system variable by providing
Variable name : carbon_home
Variable value: C:\wso2is-5.10.0 (path to the extracted Identity Server)

Use the Windows command prompt and go to the bat folder inside {yajsw-Home}

Start the wrapper in console mode
 runConsole.bat

IS Docs Link -> https://is.docs.wso2.com/en/latest/deploy/get-started/install/#install-as-a-windows-service
Expected behavior:
User should able to access the wso2 Identity Server by using the url "https://localhost:{port}/carbon"
Actual Result:
An error, when starting the wrapper in console mode using runConsole.bat (Installing as a Windows Service)
Error log -
C:\Program Files\yajsw-stable-13.10\bat>runConsole.bat

C:\Program Files\yajsw-stable-13.10\bat>pushd C:\Program Files\yajsw-stable-13.10\bat\

C:\Program Files\yajsw-stable-13.10\bat>call setenv.bat
"C:\Program Files\Java\jdk-17\bin\java" -Xmx30m -Dwrapper_home="C:\Program Files\yajsw-stable-13.10\bat\/.." -Djna_tmpdir="C:\Program Files\yajsw-stable-13.10\bat\/../tmp" -Djava.net.preferIPv4Stack=true -jar "C:\Program Files\yajsw-stable-13.10\bat\/../wrapper.jar" -c "C:\Program Files\yajsw-stable-13.10\bat\/../conf/wrapper.conf"
YAJSW: yajsw-stable-13.10
OS   : Windows 10/10.0/amd64
JVM  : Oracle Corporation/17/C:\Program Files\Java\jdk-17/64
Looking in classpath from org.rzo.yajsw.boot.WrapperClassLoader@3f99bd52 for /com/sun/jna/win32-x86-64/jnidispatch.dll
Found library resource at jar:file:/C:/Program%20Files/yajsw-stable-13.10/./lib/core/jna/jna-5.12.1.jar!/com/sun/jna/win32-x86-64/jnidispatch.dll
Extracting library to C:\Program Files\yajsw-stable-13.10\bat\..\tmp\jna7729326233399578443.dll
Trying C:\Program Files\yajsw-stable-13.10\bat\..\tmp\jna7729326233399578443.dll
Found jnidispatch at C:\Program Files\yajsw-stable-13.10\bat\..\tmp\jna7729326233399578443.dll
wrapper.logfile.umask -1
initOutputFiles: -1
open umask -1
WARNING|wrapper|WSO2 Carbon|23-07-06 13:07:39|YAJSW: yajsw-stable-13.10
WARNING|wrapper|WSO2 Carbon|23-07-06 13:07:39|OS   : Windows 10/10.0/amd64
WARNING|wrapper|WSO2 Carbon|23-07-06 13:07:39|JVM  : Oracle Corporation/17/C:\Program Files\Java\jdk-17/64
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:40|started process with pid 9872
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:41|restart internal RUNNING controller killed restart handler
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:41|stopping process with pid/timeout 9872 45000
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:41|process exit code: 1
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:47|started process with pid 13380
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:47|restart internal RUNNING controller killed restart handler
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:47|stopping process with pid/timeout 13380 45000
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:47|process exit code: 1
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:53|started process with pid 9144
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:53|restart internal RUNNING controller killed restart handler
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:53|stopping process with pid/timeout 9144 45000
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:54|process exit code: 1
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:59|started process with pid 13104
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:59|restart internal RUNNING controller killed restart handler
INFO|wrapper|WSO2 Carbon|23-07-06 13:07:59|stopping process with pid/timeout 13104 45000
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:00|process exit code: 1
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:05|started process with pid 18664
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:05|restart internal RUNNING controller killed restart handler
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:05|stopping process with pid/timeout 18664 45000
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:06|process exit code: 1
system.env 55
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:11|started process with pid 4404
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:11|too many restarts: 5
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:11|too many restarts: 5
INFO|wrapper|WSO2 Carbon|23-07-06 13:08:11|Shutting down Wrapper

Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 6.2.0 Alpha3
OS: Windows
Database: H2
Userstore: [e.g., LDAP, JDBC]

Optional Fields
Related issues:

Suggested labels:

Findings
This was tested using the following JDK versions for IS 6.2.0 Alpha 3 release.
JDK version setup procedure followed.

Set Java version for Identity Server :
Hard code relevant JAVA_HOME value to the wso2server.bat file.
Set Java version for YAJSW :
Set JAVA_HOME value in the console (this is used by YAJSW  setenv.bat file)

Results

Server starts up as expected on JDK 17.0.8 in Windows environment.
Results with the YAJSW

YAJSW Java Version
IS Java Version
Result

JDK 17.0.8
JDK 17.0.8
Fail

JDK 17.0.8
JDK 11.0.15
Fail

JDK 11.0.15
JDK 17.0.8
Pass

JDK 11.0.15
JDK 11.0.15
Pass

Notable observations

The failure is dependent on the Java version used by the YAJSW and not the IS.

Conclusion

Cause of failure is a compatibility issue on the YAJSW and not the IS.

Closing due to the issue not being related to implementation of the Identity Server.


gnome3.webkitgtk: 2.20.2 -> 2.20.3

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/webkitgtk/versions.
These checks were done:

built on NixOS
/nix/store/kg4axk1r8r3lw4hidrb639lvnhidkmg1-webkitgtk-2.20.3/bin/WebKitWebDriver passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 2.20.3 with grep in /nix/store/kg4axk1r8r3lw4hidrb639lvnhidkmg1-webkitgtk-2.20.3
directory tree listing: https://gist.github.com/9bcce0d9c55941e792a46ccc39ce7efb
du listing: https://gist.github.com/79054d0bb16c7cff8f636f5af11a2408

meta.description for gnome3.webkitgtk is: '"Web content rendering engine, GTK+ port"'.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 7ac71e987547aae0f45e794340923910c6682095 && nix-shell --pure -I nixpkgs=. -p gnome3.webkitgtk

CVE fixes https://webkitgtk.org/2018/06/11/webkitgtk2.20.3-released.html

@GrahamcOfBorg  build webkitgtk

Failure on x86_64-darwin (full log)
Attempted: webkitgtk
Partial log (click to expand)
  CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
  CMakeLists.txt:13 (project)

-- Configuring incomplete, errors occurred!
See also "/tmp/nix-build-webkitgtk-2.20.3.drv-0/webkitgtk-2.20.3/build/CMakeFiles/CMakeOutput.log".
See also "/tmp/nix-build-webkitgtk-2.20.3.drv-0/webkitgtk-2.20.3/build/CMakeFiles/CMakeError.log".
builder for '/nix/store/490j3zv9mzc81mb5znwhyyhcl8216qi5-webkitgtk-2.20.3.drv' failed with exit code 1
error: build of '/nix/store/490j3zv9mzc81mb5znwhyyhcl8216qi5-webkitgtk-2.20.3.drv' failed

Looks like this never built on Darwin according to Hydra history. Maybe we should mark it as broken.

That's why I suggested to mark it as broken on Darwin, not remove it from platforms.

Oh, sorry.


Some tests failed on Big Endian architecture

Test project /builddir/build/BUILD/scnlib-1.0/redhat-linux-build
      Start  1: test
      Start  2: empty
 1/30 Test  #1: test .............................   Passed    0.00 sec
      Start  3: fwd
 2/30 Test  #2: empty ............................   Passed    0.00 sec
      Start  4: util
 3/30 Test  #3: fwd ..............................   Passed    0.00 sec
      Start  5: small-vector
 4/30 Test  #4: util .............................   Passed    0.00 sec
      Start  6: string-view
 5/30 Test  #5: small-vector .....................   Passed    0.00 sec
      Start  7: reader
 6/30 Test  #6: string-view ......................   Passed    0.00 sec
      Start  8: range
 7/30 Test  #7: reader ...........................   Passed    0.00 sec
      Start  9: locale
 8/30 Test  #8: range ............................   Passed    0.00 sec
      Start 10: std-string-view
 9/30 Test  #9: locale ...........................   Passed    0.00 sec
      Start 11: pmr-string
10/30 Test #10: std-string-view ..................   Passed    0.00 sec
      Start 12: wrap
11/30 Test #11: pmr-string .......................   Passed    0.00 sec
      Start 13: utf8
12/30 Test #12: wrap .............................   Passed    0.00 sec
      Start 14: utf16
13/30 Test #13: utf8 .............................   Passed    0.00 sec
      Start 15: result
14/30 Test #14: utf16 ............................   Passed    0.00 sec
      Start 16: istream
15/30 Test #15: result ...........................   Passed    0.00 sec
      Start 17: format
16/30 Test #17: format ...........................   Passed    0.00 sec
      Start 18: tuple-return
17/30 Test #16: istream ..........................   Passed    0.01 sec
      Start 19: char
18/30 Test #18: tuple-return .....................   Passed    0.00 sec
      Start 20: integer
19/30 Test #20: integer ..........................***Failed    0.00 sec
[doctest] doctest version is "2.4.8"
[doctest] run with "--help" for options
===============================================================================
/builddir/build/BUILD/scnlib-1.0/test/integer.cpp:822:
TEST CASE:  consistency
  simple
/builddir/build/BUILD/scnlib-1.0/test/integer.cpp:822: ERROR: test case THREW exception: exception thrown in subcase - will translate later when the whole test case has been exited (cannot translate while there is an active exception)
===============================================================================
/builddir/build/BUILD/scnlib-1.0/test/integer.cpp:822:
TEST CASE:  consistency
DEEPEST SUBCASE STACK REACHED (DIFFERENT FROM THE CURRENT ONE):
  simple
/builddir/build/BUILD/scnlib-1.0/test/integer.cpp:822: ERROR: test case THREW exception: basic_string::substr: __pos (which is<PHONE_NUMBER>8) > this->size() (which is 7)
===============================================================================
[doctest] test cases:  38 |  37 passed | 1 failed | 0 skipped
[doctest] assertions: 941 | 941 passed | 0 failed |
[doctest] Status: FAILURE!
      Start 21: float
20/30 Test #19: char .............................   Passed    0.01 sec
      Start 22: string
21/30 Test #22: string ...........................   Passed    0.00 sec
      Start 23: string-set
22/30 Test #21: float ............................***Failed    0.01 sec
[doctest] doctest version is "2.4.8"
[doctest] run with "--help" for options
===============================================================================
/builddir/build/BUILD/scnlib-1.0/test/floating.cpp:285:
TEST CASE:  consistency
  simple
/builddir/build/BUILD/scnlib-1.0/test/floating.cpp:285: ERROR: test case THREW exception: exception thrown in subcase - will translate later when the whole test case has been exited (cannot translate while there is an active exception)
===============================================================================
/builddir/build/BUILD/scnlib-1.0/test/floating.cpp:285:
TEST CASE:  consistency
DEEPEST SUBCASE STACK REACHED (DIFFERENT FROM THE CURRENT ONE):
  simple
/builddir/build/BUILD/scnlib-1.0/test/floating.cpp:285: ERROR: test case THREW exception: basic_string::substr: __pos (which is<PHONE_NUMBER>4) > this->size() (which is 9)
===============================================================================
[doctest] test cases:  13 |  12 passed | 1 failed | 0 skipped
[doctest] assertions: 247 | 247 passed | 0 failed |
[doctest] Status: FAILURE!
      Start 24: buffer
23/30 Test #23: string-set .......................   Passed    0.00 sec
      Start 25: bool
24/30 Test #25: bool .............................   Passed    0.00 sec
      Start 26: usertype
25/30 Test #24: buffer ...........................   Passed    0.01 sec
      Start 27: list
26/30 Test #26: usertype .........................   Passed    0.00 sec
      Start 28: each-integer
27/30 Test #27: list .............................   Passed    0.00 sec
      Start 29: each-char
28/30 Test #29: each-char ........................   Passed    0.00 sec
      Start 30: file
29/30 Test #30: file .............................   Passed    0.00 sec
30/30 Test #28: each-integer .....................   Passed    0.06 sec
93% tests passed, 2 tests failed out of 30

Thanks for reporting, luckily this was just an issue with the tests -- fixed in 056e568c325a0cd464911146d1150c9abf4833b1


Unable to use Advanced Rest Client with endpoints that require form authentication

In the past when I needed to post to an endpoint behind a form login, I would make the request to the actual endpoint, the server would realize I am unauthenticated and would return a redirect, I would click on this redirect, a new tab would open and I would sign in.  Now, if I click on the link of the redirect location it changes the location of where the request points and does not open another tab.
Expected outcome
I should be able to authenticate to the from in a separate page that carries the authentication cookie over to Advanced Rest Client.
Actual outcome
It seems like the cookie is not transferred even if I login to the server on another chrome window.
Versions
App: <IP_ADDRESS>-stable
Chrome: 49.0.2623.110
Platform: Windows (7)
Steps to reproduce

Make request to endpoint that is behind login screen.
Open link to login screen in chrome and authenticate.
Make request again from step one, it will give you the authentication page again because the cookie has not transferred to arc.

The app do not share the same cookie storage as regular chrome tab. In this app it won't be possible.
However there should be some authentication settings in the app. Either for providing login and password (basic) and cookie data to send with all request sent to corresponding domain. It's core complicated in case of OAuth but it is planned to implement anyway.
Basically the app should store cookie data received from the server and restore it and the append to the headers list when making a request for the corresponding URL. It should be possible to turn off this behaviour to give more control over the request.

Right, I understand the new version does not share cookies with chrome, but the old version did, and that was a nice feature for the reason above.  Is there anyway I can transfer the cookie manually? I realize I can't do it with developer tools since they are not enabled in this standalone app.

You can just use 'cookie' header with the cookie value in the request. The app do not have access to Chrome cookie storage and there is no such API for apps available. So no, you can't transfer cookies.

This is duplicate of #435 so I'm closing this issue.


Include extents of TiledGraphicsProviders in ViewingSpace extents.

Fixes #6873.
Awaiting ImageTests results.

@Mergifyio backport release/4.7.x

I see that your ImageTest run from your PR also had some missing BG terrain on a few views. They must just be having some problems serving their data. Other than that, there are some diffs with the EWR views that I'm not sure about, so I'll let you be the judge on whether those were expected or not by your change.

Thanks for looking. I queued a run against master to try to determine which if any differences might be related to my changes.

Two different ImageTests runs produced two different sets of image differences, mostly related to map/terrain. Two different runs against master produced similar inconsistencies.
This PR does slightly alter the viewed extents for hypermodeling views, hence the slight differences in EWR. Hypermodeling graphics are provided by a TiledGraphicsProvider.


Master spdx

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

Only the License changed, right? - I am fine with that, it is just the title is misleading.


Export sys::kNumUGCResultsPerPage

Useful for chunking paginated results.

Thanks!

@Thinkofname Just noticed - steamworks::RESULTS_PER_PAGE is very ambiguous. steamworks::ugc is a private module, but is re-xported in main.rs so steamworks::ugc::RESULTS_PER_PAGE can be accessed via steamworks::RESULTS_PER_PAGE
I propose we change it to steamworks::UGC_RESULTS_PER_PAGE?