We can running the test without using ssh

Thank you for new version release the other day.
We had better not use ssh on Docker container.
See also http://jpetazzo.github.io/2014/06/23/docker-ssh-considered-evil/
And we have some issues like #40 #81
So, I have an idea to solve them.
And the idea has been implemented on this driver plugin that made from scratch.
https://github.com/marcy-terui/kitchen-docker_cli
I started to make the plugin because it was thought to have stopped development.
But, the mainstream of kitchen-docker has resumed development.
And I have welcome it.
@portertech
If you want, I 'm going to make and send some PRs.
But, they become big changes.
Therefore, there are some possibilities that become unstable.
So, I want to hear what you think.

Kitchen is using Docker as an API to/for Linux containers, I do not see this as an anti-pattern. Many cookbooks are installing/configuring a collection of services, managing machines. Using SSH supports the common TK experience, kitchen login, etc. I'll check out kitchen-docker_cli to see how you're using docker exec :-)

Using SSH on Kitchen that is not anti-pattern. I think so too.
But, I think that is important to keep the state of the base image.
Please say me if you need cooperation when you see my usage :smile:

@portertech where I would see the main benefit for this, is when I'm testing cookbooks/manifests that start upstart services. SSH does not work as a valid replacement for Upstart, and using docker images that enable upstart won't work via the current SSH experience.
I think we are operating at a rare use-case for docker images, and attempting to use them as lightweight VM's inside test-kitchen. Some of our test roles require starting multiple upstart jobs. Like, Sensu for instance. It is currently impossible to test our convergence of our sensu role, as the sensu service depends on Upstart running in order to start.
Unless we are doing something wrong in our CI pipeline, or we want to run our CI Test Slave on a bare metal server, we will need to run TK tests from a docker or lxc driver in our cloud provider.

Offering only SSH access can be a problem.  As an example, consider test-kitchen suites running with docker on CircleCI, which offers its docker interface through an encrypted URL that changes with each environment setup.  In this environment kitchen fails everytime because it can't connect to the SSH socket of the test container.  Using the native docker exec and docker cp would circumvent this issue.

This is planned for the future, though for context docker exec didn't exist when this ticket was filed :)

Any update on this?

@hurricanehrndz Nope, other than that you should check out kitchen-dokken if this is a priority.

Going to close this since while it is on my long-term roadmap, it's not in any specific planning.

Hey, just wanted to know if this is on the roadmap nowadays ? Using SSH testing over docker feels unclean, and it's enforcing some weird stuff. For example I had to remove systemd ssh service file to rely on sysV init script otherwise kitchen couldn't login, whenever I've to test systemd services
---
driver:
  name: docker
  use_sudo: false
  provision_command:
    - rm /lib/systemd/system/ssh.service
  run_command: /bin/systemd
  privileged: true
  volume:
    - "/sys/fs/cgroup:/sys/fs/cgroup:ro"

Meanwhile on verifier side, I just don't use SSH at all because docker backend is native for testinfra
verifier:
  name: shell
  remote_exec: false
  command: pytest --junitxml=test/${KITCHEN_INSTANCE}_test_report.xml --html=test/${KITCHEN_INSTANCE}_test_report.html --self-contained-html --color=yes --host="docker://root@${KITCHEN_CONTAINER_ID}" "test/integration/"

On my user side, I feel like I just miss a transport: name: docker to be good to go

@rgarrigue kitchen-docker no longer has a roadmap, no further development is planned.

Okay. So what's the recommandation? Dokken ?


cartographer cannot run after setting "publish_tracked_pose=true" in eloquent

I am using ros2 eloquent version, as well as cartographer, I want to let the cartograher node publish 'pose' data to compare with ground truth data, so I add "publish_tracked_pose" in file "/$HOME/src/turtlebot3/turtlebot3_cartographer/config/turtlebot3_lds_2d.lua".
BTW, I start cartographer using "ros2 launch turtlebot3_cartographer cartographer.launch.py use_sim_time:=True", which use "turtlebot3_lds_2d.lua" configuration.
Accoring to the official documentation here."configure tracked_pose"
In "turtlebot3_lds_2d.lua", add "publish_tracked_pose_msg = true," in "options".
After adding, my lua file is as below:
-- Copyright 2016 The Cartographer Authors
--
-- Licensed under the Apache License, Version 2.0 (the "License");
-- you may not use this file except in compliance with the License.
-- You may obtain a copy of the License at
--
--      http://www.apache.org/licenses/LICENSE-2.0
--
-- Unless required by applicable law or agreed to in writing, software
-- distributed under the License is distributed on an "AS IS" BASIS,
-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-- See the License for the specific language governing permissions and
-- limitations under the License.

-- /* Author: Darby Lim */

include "map_builder.lua"
include "trajectory_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  trajectory_builder = TRAJECTORY_BUILDER,
  map_frame = "map",
  tracking_frame = "imu_link",
  published_frame = "odom",
  odom_frame = "odom",
  provide_odom_frame = false,
  publish_frame_projected_to_2d = true,
  use_odometry = true,
  use_nav_sat = false,
  use_landmarks = false,
  num_laser_scans = 1,
  num_multi_echo_laser_scans = 0,
  num_subdivisions_per_laser_scan = 1,
  num_point_clouds = 0,
  lookup_transform_timeout_sec = 0.2,
  submap_publish_period_sec = 0.3,
  pose_publish_period_sec = 5e-3,
  trajectory_publish_period_sec = 30e-3,
  rangefinder_sampling_ratio = 1.,
  odometry_sampling_ratio = 1.,
  fixed_frame_pose_sampling_ratio = 1.,
  imu_sampling_ratio = 1.,
  landmarks_sampling_ratio = 1.,
  publish_tracked_pose = true,
}

MAP_BUILDER.use_trajectory_builder_2d = true

TRAJECTORY_BUILDER_2D.min_range = 0.12
TRAJECTORY_BUILDER_2D.max_range = 3.5
TRAJECTORY_BUILDER_2D.missing_data_ray_length = 3.
TRAJECTORY_BUILDER_2D.use_imu_data = true
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.motion_filter.max_angle_radians = math.rad(0.1)

POSE_GRAPH.constraint_builder.min_score = 0.65
POSE_GRAPH.constraint_builder.global_localization_min_score = 0.7

-- POSE_GRAPH.optimize_every_n_nodes = 0

return options

then RUN "ros2 launch turtlebot3_cartographer cartographer.launch.py use_sim_time:=True", the cartographer node is not started, cannot find the node in rqt(before change the setting in options, the cartographer node can be started successfully), and I got this error in terminal:
[cartographer_node-1] F0503 09:50:53.577739  6078 lua_parameter_dictionary.cc:410] Check failed: 1 == reference_counts_.count(key) (1 vs. 0) Key 'publish_tracked_pose' was used the wrong number of times. [cartographer_node-1] [FATAL] [cartographer_ros]: F0503 09:50:53.000000  6078 lua_parameter_dictionary.cc:410] Check failed: 1 == reference_counts_.count(key) (1 vs. 0) Key 'publish_tracked_pose' was used the wrong number of times.
Can anyone help me on this question?
BTW, in another page of official documentation here, to publish "tracked_pose" topic, we are required to add "publish_tracked_pose_msg=true" instead of change "publish_tracked_pose=true" in previous documentation, which one should we use?

Thank you for reporting the bug in the documentation. The correct one to use is publish_tracked_pose, I will fix this on the referenced page.
The error above indicates though, that either you did not add the parameter publish_tracked_pose = true or that you are not loading this file. Did you verify that you are loading the correct parameter file and that you are using the correct parameter name (without _msg) ?

Thank you for reporting the bug in the documentation. The correct one to use is publish_tracked_pose, I will fix this on the referenced page.
The error above indicates though, that either you did not add the parameter publish_tracked_pose = true or that you are not loading this file. Did you verify that you are loading the correct parameter file and that you are using the correct parameter name (without _msg) ?

Thank you for replying. About the 2 questions you asked.

If I did not added "publish_tracked_pose" in the file, there will not be an error which indicates "Key 'publish_tracked_pose' was used the wrong number of times. [cartographer_node-1] [FATAL]", am I right?
If I edit the wrong file, as I have described in the question, I think there will be no error that I cannot find "cartographer node" in rqt graph, because before I add this line, the cartographer node runs well, haha.

This is a bit odd, because the parameter is optional and I don't fully understand how it can come to this exception yet.
can you try if you have similar behavior if you set another optional parameter (e.g. publish_to_tf = true) ?

Do you know what version of cartographer is released in eloquent?
I'm hitting the same error in melodic, and I think it is because the released version is v 1.0, which was tagged 3 years ago,  before that parameter was introduced.
I guess in my case a source-build is required, perhaps it is the same for you.
Looking at cartographer_ros branches, I see there is a ros2-dashing branch which looks quite close to master, and a ros2-dashing-1.0.0 branch, which is only one commit ahead of release-1.0. If you're using the later branch that might explain why the parameter is unavilable.
I'm getting my cartographer version by:
apt-cache show ros-melodic-cartographer-ros | grep Version
Version: 1.0.0-1bionic.20210414.231737


CASSANDRA-18195 Feature Flag for Accord Transactions

patch by Caleb Rackliffe; reviewed by ? for CASSANDRA-18195

https://app.circleci.com/pipelines/github/maedhroz/cassandra?branch=CASSANDRA-18195

This patch adds a cassandra.yaml flag that, by default, prevents the following things from happening:
1.) At startup, prevents...
a.) ...Accord virtual tables from being created
b.) ...the Accord system keyspace/tables from being created
c.) ...the AccordService from being initialized, along w/ the threads it owns. (The implementation involving a no-op IAccordService might be up for discussion, but the idea seems to have been agreed to on the Jira already.)
2.) In general, prevents any CQL Accord transaction from executing.

Fixed a couple test failures related to the simulator and LOCAL_SYSTEM_KEYSPACE_NAMES...tests running again...


Add missing mock injection arguments

The mock injections for axAssets and axConfiguration need access to the artifacts and configuration options of createSetupForWidget. Currently, their are instantiated without arguments.

Fixed on master (v2.0.0)


Expose the ResponseStub type

I would like to use the ResponseStub to be able to extract a function that redirects.

We're still considering some changes to the ResponseStub approach (https://github.com/remix-run/react-router/issues/11707, https://www.youtube.com/watch?v=f5z_axCofW0) so for now I would just duplicate the type locally
export type ResponseStub = {
  status: number | undefined;
  headers: Headers;
};

This is no longer relevant as the response stub API has been removed


JSX support

Ability to provide "jsx" property for compiler options. Right now I cannot use gulp-typescript with ntypescript and jsx property.

Can you test it with the tsx branch from GitHub?

@ivogabe it works with tsx branch and also tried again with version on npm and realized that I forgot to add '.tsx' flies to gulp.src. However, neither old version (npm) nor new (tsx branch) does not correctly emits .jsx files with option "jsx": "preserve", they emits .js files.
"jsx": "react" works fine with both old and new versions.

Might it be some npm cache?

I haven't released it on NPM yet. I have fixed the extension problem, I'll try to release a new version on NPM later this week.

Thanks for the update!

I think there is an issue with type casting
ex:
var tvId: string = inputEl.value.trim();
How can we use both typescripts type casting and the jsx syntax?

@cstefan In short, you must use the .tsx extension instead of .ts and typecasts must use the new as operator: foo as Bar instead of <Bar> foo. (You can still use the old cast syntax in .ts files). See the TypeScript repo for more details.

+1, @ivogabe any update on when this is available on npm?

Just released it as 2.8.0

Can anyone provide me direction on how to compile tsx file with v 2.8.0? I'm getting error "File {filename} must have extesion .ts or .d.ts"

@nripendra Are you using TS 1.5.3 or typescript@next (development version of ts 1.6)? 1.5.3 doesn't support jsx

Got it thank you. I downloaded latest from the github repo, and assigned to typescript option.


Add function to return an array of available songs

Just use os.listdir on the folder and return the array

Fixed by d2f9809db2a7650098f0fb5d25720215a9274b81


Exporter brakes after "unsuccessful" VM migration to another pool

I've faced the following just after start (added some debug output):
['vm', 'd5de7e57-b1a6-4e14-c334-5729395dade3', 'vbd_xvda_latency'] vm d5de7e57-b1a6-4e14-c334-5729395dade3 vbd_xvda_latency
['vm', 'd5de7e57-b1a6-4e14-c334-5729395dade3', 'vbd_xvda_read'] vm d5de7e57-b1a6-4e14-c334-5729395dade3 vbd_xvda_read
['vm', 'd5de7e57-b1a6-4e14-c334-5729395dade3', 'vbd_xvda_read_latency'] vm d5de7e57-b1a6-4e14-c334-5729395dade3 vbd_xvda_read_latency
['vm', 'd5de7e57-b1a6-4e14-c334-5729395dade3', 'vbd_xvda_write'] vm d5de7e57-b1a6-4e14-c334-5729395dade3 vbd_xvda_write
['vm', 'd5de7e57-b1a6-4e14-c334-5729395dade3', 'vbd_xvda_write_latency'] vm d5de7e57-b1a6-4e14-c334-5729395dade3 vbd_xvda_write_latency
['vm', '7ab09e90-de79-190e-1109-d1fae4690189', 'cpu0'] vm 7ab09e90-de79-190e-1109-d1fae4690189 cpu0
Traceback (most recent call last):
  File "/app/xen-exporter.py", line 265, in do_GET
    metric_output = collect_metrics().encode("utf-8")
  File "/app/xen-exporter.py", line 191, in collect_metrics
    vm = get_or_set(vms, collector, lookup_vm_name, xen)
  File "/app/xen-exporter.py", line 72, in get_or_set
    d[key] = func(key, *args)
  File "/app/xen-exporter.py", line 29, in lookup_vm_name
    return session.xenapi.VM.get_name_label(session.xenapi.VM.get_by_uuid(vm_uuid))
  File "/usr/local/lib/python3.10/site-packages/XenAPI/XenAPI.py", line 306, in __call__
    return self.__send(self.__name, args)
  File "/usr/local/lib/python3.10/site-packages/XenAPI/XenAPI.py", line 187, in xenapi_request
    result = _parse_result(getattr(self, methodname)(*full_params))
  File "/usr/local/lib/python3.10/site-packages/XenAPI/XenAPI.py", line 281, in _parse_result
    raise Failure(result['ErrorDescription'])
XenAPI.XenAPI.Failure: ['UUID_INVALID', 'VM', '7ab09e90-de79-190e-1109-d1fae4690189']

I've diagnosed the problem: exactly the same VM (with this UUID) exists in another Xen pool where I moved it several days ago. I noticed that this VM was migrated with and error in XCP, but didn't realise that after migration it exists in both pools simultiniously.
XCP-ng still stores metrics for this machine I suppose, but from the point of view of XCP-ng it's not a real VM (and it actually does nothing). Although, I was able to catch its metrics when I disabled exception.
That's not the problem of your exporter, I suppose. But maybe it would be better to catch that situation for generating some alerts.
And the solution of the problem is to reboot pool-master of the poll which was the source of migration by the way. After that this false VM has disappeared

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."


Support for legacy JavaScript versions (e.g. WSH+JScript & Co)

See discussion here: https://github.com/beautify-web/js-beautify/commit/1b4be9b13cf53dfcbc4bf917a97d4bb0d96fccf0#commitcomment-11757796
First of all is needed for my port for AkelPad editor:
https://github.com/Infocatcher/AkelPad_scripts/blob/master/jsBeautifier.js
http://akelpad.sourceforge.net/forum/viewtopic.php?p=11246#11246
And finally yay!

Cool.


MINIFICPP-2178 - Move extension tests to their directories

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically main)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI results for build issues and submit an update to your PR as soon as possible.

Due to merge conflicts the CI didnt run on this, could you rebase the PR?

The bugprone-unchecked-optional-access issues can be ignored, because they will be removed, but the other clang-tidy issues should be fixed before merging:
2023-09-26T13:33:07.1338215Z /home/runner/work/nifi-minifi-cpp/nifi-minifi-cpp/extensions/rocksdb-repos/tests/DBProvenanceRepositoryTests.cpp:28:1: error: replace macro with enum [modernize-macro-to-enum,-warnings-as-errors]
2023-09-26T13:33:07.1338918Z #define TEST_PROVENANCE_STORAGE_SIZE (1024*100)  // 100 KB
2023-09-26T13:33:07.1349887Z ^~~~~~~~
2023-09-26T13:33:07.1350121Z                                      =         ,
2023-09-26T13:33:07.1350987Z /home/runner/work/nifi-minifi-cpp/nifi-minifi-cpp/extensions/rocksdb-repos/tests/DBProvenanceRepositoryTests.cpp:28:9: error: macro 'TEST_PROVENANCE_STORAGE_SIZE' defines an integral constant; prefer an enum instead [modernize-macro-to-enum,-warnings-as-errors]
2023-09-26T13:33:07.1352068Z #define TEST_PROVENANCE_STORAGE_SIZE (1024*100)  // 100 KB
2023-09-26T13:33:07.1352312Z         ^
2023-09-26T13:33:07.1353068Z /home/runner/work/nifi-minifi-cpp/nifi-minifi-cpp/extensions/rocksdb-repos/tests/DBProvenanceRepositoryTests.cpp:29:9: error: macro 'TEST_MAX_PROVENANCE_STORAGE_SIZE' defines an integral constant; prefer an enum instead [modernize-macro-to-enum,-warnings-as-errors]
2023-09-26T13:33:07.1390007Z #define TEST_MAX_PROVENANCE_STORAGE_SIZE (100*1024*1024)  // 100 MB
2023-09-26T13:33:07.1390384Z         ^

@lordgamez could you confirm that it's good now?

@lordgamez could you confirm that it's good now?

After the rebase some new clang tidy checks were introduced that should be addressed as well. There are only a few and they are simple enough that they can be fixed here IMO.


Removing SonarCloud context (closes #695)

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] If this PR comes from a fork, please Allow edits from maintainers
[x] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes # 41)
[x] Link your Pull Request to an issue (if applicable)
[x] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[x] Make sure that your PR is not introducing unnecessary reformatting (e.g., introduced by on-save hooks in your IDE)
[x] Make sure that your PR does not contain changes in text files, therefore the PR must not contain changes in values/strings/* and / or assets/* (see issue #332 for further information)
[x] Make sure that your PR does not contain compiled sources (already set by the default .gitignore) and / or binary files

Description

Could you try to remove this line in the config.yml of your fork? And also the colon in the line before. Might be that the context is not accessible. If the build starts then, we'll have to remove the context and add the SONAR_TÒKEN as a separate instance variable in our builds...

Closes #695

Does anyone care to merge this pull request? 😄

Sure, we just needed to double check that the environment variable is also set w/o the context.


I have only one slide and  its replicate the inner content three times

here is settings:
const settings = {     dots: true,     infinite: true,     speed: 500,     slidesToShow: 1,     slidesToScroll: 1,   };
and slider code:
<Slider {...settings}>

1

     </Slider>
Output:

I think it's the same issue as https://github.com/akiran/react-slick/issues/2337

any update aboyt this issue??

I've switched to using another library.
Please use this: https://github.com/davidjerleke/embla-carousel

me too. have the problem. X 3

adaptiveHeight: true is alternative bug


Ignoring Tensorflow.bundle that is produced on OSX

On OSX the compilation of the extension is producing a bundle artifact. I'd like to ignore it since we better not check this in.

@sebastiandeutsch Thanks for this.


Ci/add actions

Description of changes: Add github actions. Based mostly on Jobs version.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

An example of the run: https://github.com/nateglims/device-defender-for-aws-iot-embedded-sdk/actions/runs/307003664


Use AI_ADDRCONFIG hint with getaddrinfo if available.

This prevents failure of openssl s_server socket binding to wildcard
address on hosts with disabled IPv6.
Checklist

Ping?

Merged into master.
b8472b4e67ec7ad49254821f2da578ce588df4e6 Use AI_ADDRCONFIG hint with getaddrinfo if available.

I would think this should be cherry-picked to 1.1.1 as well.  Do you agree, @kroeckx?

I would think this should be cherry-picked to 1.1.1 as well.  Do you agree, @kroeckx?

Yes

Cool.  Thanks.  Merged into 1.1.1 as well:
2423b5f07f01a7318fb48a277c1c4f372958d664 Use AI_ADDRCONFIG hint with getaddrinfo if available.


Integrate custom .targets file in a new csproj files with .Net Core 3.0

Hi.
I faced with an issue that I cannot find info about how to integrate my .targets file from .Net Framework to a new .Net Core project file.
Can someone point me in the right direction please?
Update
Currently I am struggling with my .targets file trying to make it work, but in one case it just dont run at all, in other - there is compilation error saying:
task could not be loaded from the assembly...
Could not load file or assembly 'System.Runtime, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a' or one of its dependencies. The system cannot find the file specified. Confirm that the <UsingTask> declaration is correct, that the assembly and all its dependencies are available, and that the task contains a public class that implements Microsoft.Build.Framework.ITask.
Which is I think related to some inconsistency between .Net Core and .Net standard (which uses MSBuild.Uitilities nuget package

Ok, I actually found out what is going on. MSBuild for some reason in trying to start wrong environment (seems .Net framework instead of .Net Core) In Rider there is an option to select MSBuild version and it working quite well, but in such configuration it fails to run under VS 2019.
Is there any way to make it work or I need to switch using Exec task instead?
I cannot use netstandard2.0 in my projects, so this variant is inappropriate for me.

Can you provide some more details, please? Does your .targets file use a custom task? If so, how is that built?

@rainersigwald I already found out that this happenes because MSBUILD cold not run .NetCore 3.0 or higher tasks, but only netstandrad2.0 or lower, which war very strange and unexpected to me.
For ex., Rider is building and running such task without issues, because it uses MSBUILD for .Net core instead of some custom, which Visual Studio is using.
And my question is:
Do you to ship fix for this anytime soon? Maybe at least for .Net 5 release?

That's tracked by #4834, which isn't currently scheduled.
Closing this since you figured out the issue and we have tracking for the underlying problem.


Don't attempt to make timeseries plots if required objects are missing from the report

in template._get_render_kwargs

closed by #585?


docs(readme): edit docs

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Storybook is broken in child projects

We need to simplify the config that we install. See: https://github.com/UKHomeOffice/hods-poc/commit/9a3ebd4609e6d26dfe06b89455ba8778072ee1a0

It should also be updated to a stable version.


GridPro: onItemPropertyChanged always returns last cell state, logic issue in validating error

1- onItemPropertyChanged={itemPropertyListener} should return latest state but now it's returning last state, so having issue while doing validation
2- here ! (not) in if statement should not be used...  regex.test(str) returns true/false if regex matches, so making it ! not works properly

Thanks. The logic seems correct here: if the regex does not match, then the user input is invalid, discard the edit and show the error.
However I think calling preventDefault(); like so is incorrect, because it is not passing this reference correctly for the event. Let's try to reproduce this use case and fix the example.

Current issues:
1- Cell's previous state while validating,
Here I have updated email value to ahsin.siddique@     but cell is trying to validate old value which was<EMAIL_ADDRESS>looks like onItemPropertyChanged  is returning one last previous value

2-  Here preventDefault() is breaking the application, so we should remove


Remove the util.RPCTest hack

Signed-off-by: rao yunkun<EMAIL_ADDRESS>This PR is related with issue #33

Codecov Report

Merging #38 (68c07bb) into master (f3df416) will decrease coverage by 0.21%.
The diff coverage is 30.00%.

:exclamation: Current head 68c07bb differs from pull request most recent head 22213a9. Consider uploading reports for the commit 22213a9 to get more accurate results

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   78.74%   78.52%   -0.22%
==========================================
  Files          10       10
  Lines         668      666       -2
==========================================
- Hits          526      523       -3
- Misses        115      116       +1
  Partials       27       27

Impacted Files
Coverage Δ

internal/server/server.go
51.26% <30.00%> (-1.64%)
:arrow_down:

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 f3df416...22213a9. Read the comment docs.


Update README.md

add release shield

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 96.043%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
898

Relevant Lines:
935

💛  - Coveralls


add location type aggregation

I had to invent a new piece of model specifically for the aggregations. It does seem a little odd that we:

decode into the internal model from elastic specific JSON
convert those to display models
encode the display models as JSON

It feels like we should be able to just encode elastic JSON => display model as that's the only place it is used. Thoughts?

How come you can't use the same type as for the filter? LocationTypeQuery

How come you can't use the same type as for the filter? LocationTypeQuery

Good shout. I suppose they are different things, and I will hopefully do some refactoring on the Location types when removing the new types that we don't need any more and add more descriptive data to the locations.


datepicker formatting year issue on horizontal style

I am using the datepicker control in a different style format than the examples.  I'm using it in a horizontal style rather than a stacked style.  I'm running into an issue where the year is truncated.  Here is my html:

Contract Start Date

Here is the end result:

Is there any minor adjustments that can be made to prevent the year from cutting off when using a horizontal style over a stacked or vertical?
Thanks,

Hi @camerori, thank you for logging this! We'll look into the matter in the coming days and hopefully provide a fix for you. Stay tuned!

Hi @camerori, we've tested this plugin in the context of a slds-form--horizontal like your example, but we could not reproduce the issue. It also seems like your example html was not included in your original post, although the intention was there.
Could you please post your markup when possible? Also, which browsers are producing this issue?


difference between Portrait and face Plusv2? training details of Portrait?

https://github.com/tencent-ailab/IP-Adapter/issues/266


Add SOC2PSD Reduction

Description
Added a reduction SOC2PSD that converts the SOC constraints to PSD constraints, for solvers that support the PSD cone but not the second order cone.
This enables the SDPA solver to be used for solving QPs and SOCPs.
Type of change

[x] New feature (backwards compatible)
[ ] New feature (breaking API changes)
[ ] Bug fix
[ ] Other (Documentation, CI, ...)

Contribution checklist

[x] Add our license to new files.
[x] Check that your code adheres to our coding style.
[x] Write unittests.
[x] Run the unittests and check that they’re passing.
[x] Run the benchmarks to make sure your change doesn’t introduce a regression.

This is quite a nice contribution, thanks! We definitely should have an SOC2PSD reduction. The solving_chain code is very critical and already pretty messy so may require more organization.

scipy.sparse.eye uses the DIAgonal format by default and cvxpy.bmat accepts it.
I believe conversion from diagonal format to either CSR or CSC should have the same complexity, so we should perhaps let CVXPY decide whichever is more appropriate when it needs to performs matrix operations internally.

Thanks for making the change!

scipy.sparse.eye uses the DIAgonal format by default and cvxpy.bmat accepts it.
I believe conversion from diagonal format to either CSR or CSC should have the same complexity, so we should perhaps let CVXPY decide whichever is more appropriate when it needs to performs matrix operations internally.

I agree that we should let cvxpy decide on any conversions if needed. I just didn't know if we accepted diagonal format SciPy sparse matrices.

Note: the test failures are unrelated. (They're due to a test where we expect an ARPack convergence failure in is_psd but ARPack actually converges.)

@SteveDiamond can you take another look and see if it's good from your perspective?


[rel/3.6] Fix running cleanup after first test method

When the same method is requested to run multiple times (e.g. in test cases) make sure we run cleanup after the last method, and not after the last unique method.
Fix #3018

pls recall to cherry pick on main after


Updates README.md with svg badge

Coverage increased (+0.04%) to 99.583% when pulling 67b4c87d273e3bb3c67450608ce8f80c30cc72ca on sobolevn:patch-1 into c2f04561e3d8b5847459dcc2ca2675269113678d on timothycrosley:develop.

Looks good! Thanks!


About Training on 2D images

Hi ~ thank you for your excellent work in the medical imaging analysis community.
Well, I wanted to train the dataset in 2D, so I converted the dataset to 3D (like nnUnet trained the dataset in 2D), but I did the training as Task120_Massachusetts_roadSegm.py suggested, I find that it seems to be similar as the previous nnUnet training.So,What command should I use if I want to use nnFormer on a 2D dataset? Or what parts of the code do I need to change?
Thank you very much

ACDC和synapse本身就是2d图像吧，论文里面作了实验了已经

Hi ~ thank you for your excellent work in the medical imaging analysis community.
Well, I wanted to train the dataset in 2D, so I converted the dataset to 3D (like nnUnet trained the dataset in 2D), but I did the training as Task120_Massachusetts_roadSegm.py suggested, I find that it seems to be similar as the previous nnUnet training.So,What command should I use if I want to use nnFormer on a 2D dataset? Or what parts of the code do I need to change?
Thank you very much
ACDC和synapse本身就是2d图像吧，论文里面作了实验了已经


[Snyk] Upgrade com.networknt:json-schema-validator from 1.4.3 to 1.5.2

Snyk has created this PR to upgrade com.networknt:json-schema-validator from 1.4.3 to 1.5.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 3 versions ahead of your current version.

The recommended version was released on 23 days ago.

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Fails

:no_entry_sign:

PR Title '[Snyk] Upgrade com.networknt:json-schema-validator from 1.4.3 to 1.5.2': subject may not be empty, type may not be empty

Warnings

:warning:

Commit Message 'fix: upgrade com.networknt:json-schema-validator from 1.4.3 to 1.5.2
Snyk has created this PR to upgrade com.networknt:json-schema-validator from 1.4.3 to 1.5.2.
See this package in maven:
com.networknt:json-schema-validator
See this project in Snyk:
https://app.snyk.io/org/connectors-0mk/project/c5f5cc4c-0f85-4577-aedf-aa71c686e963?utm_source=github&utm_medium=referral&page=upgrade-pr': body's lines must not be longer than 100 characters

Pull request title should follow Conventional commits.

  Generated by :no_entry_sign: dangerJS against 9c9f17d7f6b05f45855742c70e4877902f63d503


refactor: refactor envbuilder to use coder/serpent as CLI engine

Related to https://github.com/coder/envbuilder/issues/130

@mafredri @johnstcn @dannykopping I think this PR is in a good state to get merged. Here are some followups from the reviews:

Test CLI output against a golden file
. Reference
Test to validate that envs given previously will still produce the same options now in Serpent
. Reference
Keep the comments on the struct and instead go generate the serpent descriptions from those. Reference

I'm planning to create an issue + PR for each one of them after merging https://github.com/coder/envbuilder/pull/147.


No split screen/screen rect support?

I am making a split screen game using cinemachine, and all seems well, except that the HDRP UI Camera doesnt seem to account for the camera that its rendering Screen rect (as this has been halfed for each of my 2 cameras).
I've looked at the code, I assume I need to change the internal texture size (as Im using the automatic option) to something other then camera.pixelWidth and camera.pixelHeight? or is there another way?

Hello, I pushed a change to simplify the split-screen setup: https://github.com/alelievr/HDRP-UI-Camera-Stacking/tree/1.3.0
You can take a look at the SplitScreen scene if you want an example. Also don't forget to have matching camera rects between the UI camera and main camera otherwise you'll experience streching

Thanks alot, will definitely give a it go as the clipping of my HUD right now when I'm close to objects just won't cut it for a release! Appreciate the effort!


fix: -check-up-to-date should not imply -check-deactivated-plans

The interaction of flags became complex so we made a strategic decision that they should all be orthogonal. But it looks like we missed one instance of that where -check-up-to-date also triggers the -check-deactivated-plans logic, and this is now resolved.

:eyes:


[ENHANCEMENT] Implement harvester-system-agent

Is your enhancement related to a problem? Please describe.
Followup: https://github.com/harvester/harvester/issues/5278
In before, we rely on rancher-system-agent to upgrade nodes or set registries. The rancher-system-agent is not in Kubernetes cluster, so it can restart kubelet / RKE2 services without error. To simply harvester dependencies, we will not have rancher-system-agent. We need a similar service in each node to do upgrading node and setting registries, proxy, etc.

Close this one since we can run rancher-system-agent without embedded rancher. However, we need some enhancement in https://github.com/harvester/harvester/issues/7039.


update workflows to run on chef-18 branch

Description
Update github actions workflows to run on chef-18 branch when pushed to chef-18 branch
Related Issue

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] I have run the pre-merge tests locally and they pass.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] If Gemfile.lock has changed, I have used --conservative to do it and included the full output in the Description above.
[x] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Duplicates https://github.com/chef/chef/pull/14686

<3


fix: use tsconfig.spec.json instead of tsconfig.e2e.json

Why?
Because if you have libraries in your nx workspace, you can't use them in your e2e project.
Following these steps:

Generate a new nx workspace
Add the Jest Playwright Preset
Add a simple library that exports a function, a string, whatever, doesn't matter
Try to call that function in your e2e Project

You get errors similar to this:

Where @demo/foo is the library I just added.
Checked everything, and everything seemed to be just fine configured, then I realized the hint, I didn't notice before:

Cannot locate a tsconfig.spec.json. Please create one at /home/paul.heising/dev/e2e-demo/apps/demo-e2e-test/tsconfig.spec.json

Seems to be that Jest looks up a tsconfig.spec.ts specifically, which does not exist, therefore jest does not understand the @demo/foo path referencing the library in the workspaces root tsconfig.base.json
If we rename the tsconfig.e2e.json to tsconfig.spec.json and adjust all references to the file, it works fine, jest understands the library pathes and the tests work again

@Bielik20
Sorry, I thought I created the PR in my personal fork to merge my branch into my own master first.
Just updated the description with some information, why this fix is necessary imo

Yeah, that is weird. I would probably need to inspect Nx source for Jest to know more. For me it is fine 👍 Let me know if you are ready for me to merge it and I will do it. Thank you for contributing.


Modify defaults for parent from empty strings to null.

Due to, it seems some changes to the aws_route53_zone data source in provider version 5.51.0, it seems using empty strings here causes an issue where the following error is received.
│ Error: Invalid combination of arguments
│
│   with
module.cloudfront.module.cloudfront_distribution.module.route53_alias.data.aws_route53_zone.default[0],
│   on
.terraform/modules/cloudfront.cloudfront_distribution.route53_alias/main.tf
line 4, in data "aws_route53_zone" "default":
│    4:   name         = var.parent_zone_name
│
│ "name": only one of `name,zone_id` can be specified, but
`name,zone_id`
│ were specified.

By using null in place of empty strings here, the validation seems to pass without issue. This can be validated by trying to use this module with 5.51.0, and finding that no matter which is provided both are interpreted as having been provided, triggering the error.
history of data source
file

Sounds good, @yuekui. Glad to hear it. I still think providing empty strings in lieu of not providing a value when the desired intent is no value, is an odd choice in languages with nulls; evil as they are. 😅 If this is merged, cool; if closed directly, that's sensible too.

/terratest

/terratest


Support Laravel 7.*?

Are you going to support laravel 7.* ?

bump

https://github.com/czechbox/laravelplans/pull/5


Asset conversion script - max call stack size exceeded

Hi guys, I'm using 2.0.0-rc11 assetConversion.js script to migrate my assets. I've successfully migrated all asset I have, but there's one, large one. which fails and throws maximum call stack size exceeded error on libSetup method. The file is 10372K, 386061 lines of code. I know, it's very big resource file, but is there any to chance to get this fixed, to have higher limit of source assets file size? I'll try to split it manually, or reexport from Animate, but I'd expect the script should convert assets of any size? Maybe there's something that can be optimized to allow this large conversion, what do you think about this, have you got any reports, or is there someone, who had the same problem? Thanks a lot, good luck in upcoming releases!:)

I've successfully exported the mentioned assets of a big project from the source fla from Animate.

Hmm, it's been a while since I was in that script, but I didn't think there was any recursion going on that would hit a max call stack. I will take a look when I get a chance.


Add a general non-geospatial layer-browser

We need an example similar to layer-browser for non-geospatial layers so that they won't be accidentally broken.

PR #1210 adds the capability to show the plot layer. Remaining work:

event handling still doesn't work. Two ways of fixing this could either be support for multiple controllers or an alternate "dual tab" approach.

Also we should add more layers in COORDINATE_SYSTEM.IDENTITY mode. At least a PointCloudLayer showing in the OrbitViewport.


Support for templating?

We run a multi tenant system. In general environments are the same except for the name of the tenant. To keep hosting overhead down we'd like to have all our tenants share one Kafka cluster. Where we keep the tenants apart using ACLs.
To do this now with Kafka-Gitops we would need to have a lot of repetition in our state file, and thus a lot of messy copy- paste-adapting every time we need to connect a new tentnat to our system. Or when we want to introduce a new application service for all our tenants.
In Terraform you can have variables and such, and generate resources for each item in a list. We've set it up so that we can just add the Tenant to a list and from there on Terraform creates all the resources needed for that tenant.
Are there any plans to add this kind of templating to Kafka-Gitops as well?
Maybe even just a way to share a custom ACL between multiple services or users without having to repeat the whole ACL for each and every one. That would already make the state file a whole lot more concise for us.

Hi @erikhh, apologies for the late response. It's a busy time of year for me. :)
I think this is a great idea, and we've discussed some similar features in some of the other open feature requests. I think some sort of templating would be helpful. Do you have specific examples of the types of templating that would benefit you the most?

Maybe even just a way to share a custom ACL between multiple services or users without having to repeat the whole ACL for each and every one. That would already make the state file a whole lot more concise for us.

I like this idea -- agreed. I'll have to think about what the format of this would look like, but I think this is something that could be added in the near future.

Hi @devshawn, I know right?! Same here, sorry for the late response.
For now I just had to get 'my thing' here going. So I went ahead and added a bit of Mustashe to at least get my principal based repetition out of the way. It ain't exactly pretty but it's functional. I made our CI pipeline take a Mustasche template of the state.yaml and apply the correct variables for the environment it's building for to produce a full fledged state.yaml file that's published to a S3 bucket to be picked up.
Based on my experience with kafka-gitops so far it would already be really good (for me anyway) to DRY up the way you can add and assign custom service acls. There's a lot of repetition in using those. For example this is one of my entries now.
  {{.}}_journal-extractor:
    create-tenant-journal:
      name: journal-{{.}}
      type: TOPIC
      pattern: LITERAL
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: CREATE
      permission: ALLOW
    describe-tenant-journal:
      name: journal-{{.}}
      type: TOPIC
      pattern: LITERAL
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: DESCRIBE
      permission: ALLOW
    write-tenant-journal:
      name: journal-{{.}}
      type: TOPIC
      pattern: LITERAL
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: WRITE
      permission: ALLOW
    create-ship-journal:
      name: journal-{{.}}-ship-incoming-
      type: TOPIC
      pattern: PREFIXED
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: CREATE
      permission: ALLOW
    describe-ship-journal:
      name: journal-{{.}}-ship-incoming-
      type: TOPIC
      pattern: PREFIXED
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: DESCRIBE
      permission: ALLOW
    write-ship-journal:
      name: journal-{{.}}-ship-incoming-
      type: TOPIC
      pattern: PREFIXED
      host: '*'
      principal: User:{{.}}_journal-extractor
      operation: WRITE
      permission: ALLOW

First of all for some reason I had to repeat the principal over and over again, even tough it's also specified with the service already. But this might also be, just a bug.
It would already save a lot of repetition if you could pass more than one operation, maybe by accepting a list there in stead of a single value.
That could already reduce the above to something like:
{{.}}_journal-extractor:
  tenant-journal:
    name: journal-{{.}}
    type: TOPIC
    pattern: LITERAL
    host: '*'
    principal: User:{{.}}_journal-extractor
    operation: CREATE, DESCRIBE, WRITE
    permission: ALLOW
  ship-journal:
    name: journal-{{.}}-ship-incoming-
    type: TOPIC
    pattern: PREFIXED
    host: '*'
    principal: User:{{.}}_journal-extractor
    operation: CREATE, DESCRIBE, WRITE
    permission: ALLOW

And something similar would also clean up the custom user acls by pretty much the same amount.

For what it's worth, I've been using jsonnet and been happy with it (… at least, as happy as one can be when they need to template JSON).
They even have a tool to convert YAML -> JSON to make the import easier: https://jsonnet.org/articles/kubernetes.html
Here's a bit of the config I use:
local envswitch(vals) = vals[std.extVar('env')];

local config = {
  env: std.extVar('env'),
  admin_user: envswitch({
    'local': '',
    dev: 'User:160482',
  }),
  rw_user: envswitch({
    'local': '',
    dev: 'User:160484',
  }),
  ro_user: envswitch({
    'local': '',
    dev: 'User:160485',
  }),
  partitions: 3,
  replicas: 3,
};

local ACL(base) = {
  pattern: 'LITERAL',
  host: '*',
  permission: 'ALLOW',
} + base;

local ifNotLocal(obj) = if config.env != 'local' then obj else {};

[{
  // Topic config reference:
  //   https://docs.confluent.io/platform/current/installation/configuration/topic-configs.html
  topics: {
    'example': {
      partitions: config.partitions,
      replication: config.replicas,
      configs: {
        'cleanup.policy': 'delete',
        'retention.ms': 15 * 24 * 60 * 60 * 1000,
      },
    },
  },

  users: ifNotLocal({
    'admin-user': {
      principal: config.admin_user,
    },
    'read-write-user': {
      principal: config.rw_user,
    },
    'read-only-user': {
      principal: config.ro_user,
    },
  }),

  customUserAcls: ifNotLocal({
    'admin-user': (
      {
        'cluster-alter': ACL({
          type: 'CLUSTER',
          name: 'kafka-cluster',
          operation: 'ALTER',
        }),
      } +
      {
        ['topic-' + op]: ACL({
          type: 'TOPIC',
          name: '*',
          operation: op,
        })
        for op in ['CREATE', 'ALTER', 'DELETE', 'DESCRIBE', 'DESCRIBE_CONFIGS', 'ALTER_CONFIGS', 'READ', 'WRITE']
      }
    ),
    'read-write-user': (
      {
        ['topic-' + op]: ACL({
          type: 'TOPIC',
          name: '*',
          operation: op,
        })
        for op in ['DESCRIBE', 'DESCRIBE_CONFIGS', 'READ', 'WRITE']
      } +
      {
        ['group-' + group + '-' + op]: ACL({
          type: 'GROUP',
          name: group + '.',
          pattern: 'PREFIXED',
          operation: op,
        })
        for op in ['READ', 'WRITE', 'CREATE']
        for group in ['ephemeral', 'app']
      }
    ),
    'read-only-user': (
      {
        ['topic-' + op]: ACL({
          type: 'TOPIC',
          name: '*',
          operation: op,
        })
        for op in ['DESCRIBE', 'DESCRIBE_CONFIGS', 'READ']
      } +
      {
        ['group-' + group + '-' + op]: ACL({
          type: 'GROUP',
          name: group + '.',
          pattern: 'PREFIXED',
          operation: op,
        })
        for op in ['READ', 'WRITE', 'CREATE']
        for group in ['ephemeral', 'ro']
      }
    ),
  }),
}]


unsupport aggregation specific dynamic column

When I defined value column as dynamic column, I use select sum(value)  group by labels.method that is ok, but I use select sum(value.request)  group by labels.method that return:
Details

        invalid aggregation
        Aggregation [sum(value.request)] Group: [labels.method]
          TableScan Table: 3469660e-d808-44bf-baf3-006abef8751d Projection: [] Filter:  Distinct: []
          -> invalid expression: column not found: value.request: sum(value.request)

I found logicalplan/validate.go no support dynamic clolumn:
		column, found := schema.ColumnByName(named.Name())
		if !found {
			return &ExprValidationError{
				message: fmt.Sprintf("column not found: %s", named.Name()),
				expr:    expr,
			}
		}

The above code may need to be modified to support specific dynamic columns

Thank you for the report!


Make Druid DimensionSpec work in time series slices

Druid DimensionSpec doesn't work in time series slices when there're
more than 1 groupBy fields, because in this case Superset would issue
two queries and DimensionSpec fields will break the value filters in the
2nd query.
This commit address this by always using the real dimension names in
DimensionSpec in the first query, as well as in the value filters of the
2nd query. DimensionSpec fields are still used in the 2nd query's
groupBy fields so value lookups eventually work in final output.
See #2246 for more details about the issue and this fix.

@rlei: can you also add a test case for this case ?

@nishantmonu51 sure. Will try to find some time for that in this week.


Priya1289/issue956

fix for app crash

the app crashed because it couldn't handle very large data. It crashes for any group fields selected that returns a very large dataset. Intent putextra and getextra couldn't handle this load, and app crashed. Fixed using another suggested way to pass data between different activities -with a public static field.

Thanks @TADraeseke !


feature request

Add web integration for deepseek ai.
ITS Just Phenomenal.

https://github.com/deepseek-ai/awesome-deepseek-integration/blob/main/docs/chatgpt_box/README.md


Update Collossus to work with google provider v2.0

The following errors haunt us with google provider v2.0:
Error: module.network.module.nat.google_compute_instance_group_manager.primary: "auto_healing_policies": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_group_manager.primary: "auto_healing_policies.0.health_check": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_group_manager.primary: "auto_healing_policies.0.initial_delay_sec": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_group_manager.secondary: "auto_healing_policies": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_group_manager.secondary: "auto_healing_policies.0.health_check": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_group_manager.secondary: "auto_healing_policies.0.initial_delay_sec": [REMOVED] This field is in beta. Use it in the the google-beta provider instead. See https://terraform.io/docs/providers/google/provider_versions.html for more details.

Error: module.network.module.nat.google_compute_instance_template.primary: "network_interface.0.address": [REMOVED] Please use network_ip

Error: module.network.module.nat.google_compute_instance_template.secondary: "network_interface.0.address": [REMOVED] Please use network_ip

We can either remove auto_healing_policies blocks or require google-beta provider. Fun.
Ideally we'd just replace this with the managed nat service.

:clap:
Thanks for submitting this. Since this would presumably break backwards compatibility with previous providers, it might make sense to push a new release to modules/gcp/v0/2 with your patches.
Also, I'm still hoping to move Colossus to naftulikay/colossus.

@smaslennikov Excellent. Have you tested that this does in fact build with the new provider? You might be able to use the examples to test building a network.

@naftulikay more cleanup is to be done here; hold tight.

@naftulikay this is now ready to go. I tested it with google-beta v2.2 provider just now. Unfortunately, to keep the auto_healing_policies, we do have to require google-beta provider. Again, the proper alternative is to replace the whole module with the managed NAT. Sounds like work.

@naftulikay oh, and I don't have write on the new Colossus repo - you do the migration when ready, or give me write access.

Generated diff between GCP 0.1 and 0.2:

git diff --no-index modules/gcp/v0/{1,2}
diff --git a/modules/gcp/v0/1/colossus_nat/instance_groups.tf b/modules/gcp/v0/2/colossus_nat/instance_groups.tf
index 32a6106..a3ebdc9 100644
--- a/modules/gcp/v0/1/colossus_nat/instance_groups.tf
+++ b/modules/gcp/v0/2/colossus_nat/instance_groups.tf
@@ -8,12 +8,14 @@ resource "google_compute_instance_group_manager" "primary" {
   description = "Colossus Primary NAT Gateway Instance Group in ${local.availability_zones[count.index]}."
   zone = "${local.availability_zones[count.index]}"

-  update_strategy = "RESTART"
   target_size = 1

   wait_for_instances = true

-  instance_template = "${element(google_compute_instance_template.primary.*.self_link, count.index)}"
+  version {
+    name = "nat-1"
+    instance_template = "${element(google_compute_instance_template.primary.*.self_link, count.index)}"
+  }

   named_port {
     name = "health-check"
@@ -34,12 +36,14 @@ resource "google_compute_instance_group_manager" "secondary" {
   description = "Colossus Secondary NAT Gateway Instance Group in ${local.availability_zones[count.index]}."
   zone = "${local.availability_zones[count.index]}"

-  update_strategy = "RESTART"
   target_size = 1

   wait_for_instances = true

-  instance_template = "${element(google_compute_instance_template.secondary.*.self_link, count.index)}"
+  version {
+    name = "nat-2"
+    instance_template = "${element(google_compute_instance_template.secondary.*.self_link, count.index)}"
+  }

   named_port {
     name = "health-check"
diff --git a/modules/gcp/v0/1/colossus_nat/instance_templates.tf b/modules/gcp/v0/2/colossus_nat/instance_templates.tf
index e1673a6..fbd2f33 100644
--- a/modules/gcp/v0/1/colossus_nat/instance_templates.tf
+++ b/modules/gcp/v0/2/colossus_nat/instance_templates.tf
@@ -27,7 +27,7 @@ resource "google_compute_instance_template" "primary" {

   network_interface {
     subnetwork = "${var.dmz_subnet_id}"
-    address = "${element(google_compute_address.private_primary.*.address, count.index)}"
+    network_ip = "${element(google_compute_address.private_primary.*.address, count.index)}"
     access_config {
       nat_ip = "${element(google_compute_address.public_primary.*.address, count.index)}"
     }
@@ -69,7 +69,7 @@ resource "google_compute_instance_template" "secondary" {

   network_interface {
     subnetwork = "${var.dmz_subnet_id}"
-    address = "${element(google_compute_address.private_secondary.*.address, count.index)}"
+    network_ip = "${element(google_compute_address.private_secondary.*.address, count.index)}"
     access_config {
       nat_ip = "${element(google_compute_address.public_secondary.*.address, count.index)}"
     }
diff --git a/modules/gcp/v0/1/colossus_network/main.tf b/modules/gcp/v0/2/colossus_network/main.tf
index c315a9e..67e0860 100644
--- a/modules/gcp/v0/1/colossus_network/main.tf
+++ b/modules/gcp/v0/2/colossus_network/main.tf
@@ -55,6 +55,7 @@ resource "google_dns_managed_zone" "public" {
   name = "${local.public_zone_name}"
   dns_name = "${local.zone}."
   description = "${local.public_zone_description}"
+  visibility = ""
 }

 resource "google_compute_route" "public" {

@smaslennikov can you give each google_compute_instance_group_manager for the NATs the same version name? Docs here.
Ideally, name them something like nat-v0 or something.
Also, for the google_dns_managed_zone (the public one) can you change visibility = "" to visibility = "public" to make things explicit?
So:

Give each NAT google_compute_instance_group_manager the same version.name
Update the public google_dns_managed_zone's visibility value to be public.

@naftulikay the visibility part requires recreation, oddly:
-/+ module.network.google_dns_managed_zone.public (new resource required)
      id:             "public" => <computed> (forces new resource)
      description:    "Public Hosted Zone for the Colossus Stage Network." => "Public Hosted Zone for the Colossus Stage Network."
      dns_name:       "stage.pizza.com." => "stage.pizza.com."
      name:           "public" => "public"
      name_servers.#: "4" => <computed>
      project:        "pizza-stage" => <computed>
      visibility:     "" => "public" (forces new resource)

these diffs are impossible


Server crashes

The server will run for a while and everything will work fine and then all of a sudden it will crash and tell me to send you guys this file. I don't think it's unturned because I've tried reinstalling unturned, rocket and all the plugins. please help.
http://download1504.mediafire.com/viimkh0hsw7g/ja83azi3tl1ewb9/Rocket+Crash.zip

If it was the crash next to the exe in a folder named with a timestamp that is an unturned crash, not rocket. In which you should be sending it to Nelson.

ok, thanks and how do i send it to him?

Click the discussions tab on the unturned steam page, then post a new topic in the unturned 3.x bugs section!

ok thank you very much!

You can also report it here: https://github.com/SmartlyDressedGames/Unturned/issues


make sure to cleanup tmp output_dir for e2e tests

fixes #830

It looks to me like the file there should be a file tests/e2e/__init__.py in order to discover the helper function. See pytest - Good Integration Practices


pip install . doesnt work well

still prompts ModuleNotFoundError: No module named 'dialognlu'

@elvinaqa Please, provide your system details and steps you did to reproduce the issue.

Duplicate #28


Bot Not Starting

i wil start the bot and the bot wil not stating...

Did you have discord.js v13 and node.js 16+ ?

Did you have discord.js v13 and node.js 16+ ?

I Have now node.js 16+ how i can get the discord.js v13?

npm i<EMAIL_ADDRESS>

![image](https://user-images.githubusercontent.com/92209198/141368804-bd130f60-d59c-4a50-9438-960c98b476d7.png)

Not working...

npm i<EMAIL_ADDRESS>
node_modules\discord.js\src\client\websocket\WebSocketManager.js:249
throw new Error(WSCodes[error.code]);
^
Error [DISALLOWED_INTENTS]: Privileged intent provided is not enabled or whitelisted.
at WebSocketManager.createShards (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketManager.js:249:15)
at async Client.login (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\Client.js:245:7) {
[Symbol(code)]: 'DISALLOWED_INTENTS'
}
Node.js v17.1.0
this is the error

You need to have this box checked

You need to have this box checked

error..
events\ready.js:31
oniChan.bulkDelete(100).then(() => {
^
TypeError: Cannot read properties of undefined (reading 'bulkDelete')
at Object.execute (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\events\ready.js:31:13)
at Client. (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\index.js:29:46)
at Client.emit (node:events:390:28)
at WebSocketManager.triggerClientReady (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketManager.js:383:17)
at WebSocketManager.checkShardsReady (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketManager.js:366:10)
at WebSocketShard. (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketManager.js:188:14)
at WebSocketShard.emit (node:events:390:28)
at WebSocketShard.checkReady (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketShard.js:474:12)
at WebSocketShard.onPacket (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketShard.js:446:16)
at WebSocketShard.onMessage (C:\Users\Kristof\Desktop\DISCORD BOTS\Team Release NL Bot\node_modules\discord.js\src\client\websocket\WebSocketShard.js:300:10)
Node.js v17.1.0

That's a configuration problem on your part

That's a configuration problem on your part

how to do i can fixed?

By configuring well the config.json

By configuring well the config.json

what should i check?

All the config

All the config

hij toont de error bij dit:
oniChan.bulkDelete(100).then(() => {
  sendTicketMSG()
})

},
};

All the config

it shows the error with this
oniChan.bulkDelete(100).then(() => {
  sendTicketMSG()
})

},
};

C:\Users\user\Desktop\ticket-bot-main\node_modules\discord.js\src\client\websocket\WebSocketManager.js:256
throw new Error(WSCodes[error.code]);
^
Error [DISALLOWED_INTENTS]: Privileged intent provided is not enabled or whitelisted.
at WebSocketManager.createShards (C:\Users\user\Desktop\ticket-bot-main\node_modules\discord.js\src\client\websocket\WebSocketManager.js:256:15)
at async Client.login (C:\Users\user\Desktop\ticket-bot-main\node_modules\discord.js\src\client\Client.js:245:7) {
[Symbol(code)]: 'DISALLOWED_INTENTS'
}
Any idea why ?

Did you join bot on server?


Remove-statements action in ECR resource

Remove-statements action in ECR resource. Currently the below are supported for ecr
ecr:
actions: [invoke-lambda, notify, put-metric]
filters: [and, cross-account, event, not, or, value]

merged


Clarification regarding "non-Windows and mobile devices"

Does Defender for Identity monitor only domain-joined devices?
No. Defender for Identity monitors all devices in the network performing authentication and authorization requests against Active Directory, including non-Windows and mobile devices.

I would like to understand better how MDI monitors "non-Windows and mobile devices"... is it meaning NNR and SAM-R, or is it something else altogether?

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

ID: 83b234d2-4a23-99df-d67f-2891645145ea
Version Independent ID: ded9e856-0cd7-79d0-9f9c-e66adca87863
Content: Microsoft Defender for Identity frequently asked questions - Microsoft Defender for Identity
Content Source: ATPDocs/technical-faq.yml
Service: microsoft-defender-for-identity
GitHub Login: @batamig
Microsoft Alias: bagol

Thank you for your comment. We'll investigate and get back to you.

Hi @LijuV-MSFT, this doesn't use NNR/SAM-R. MDI monitors user / identity authentications coming from all devices in the network.
I suggest we continue internally if you need more clarifications, and I can loop in the relevant focal points as needed.
I'm going to close this issue for now, but again - please be in touch internally if you have more questions. Thanks!
#please-close


Any advice, please.

Hi. I read your paper very well.
I want to do two experiments.

Multivariate regression
You said "The NBEATSx model offers a solution to the multivariate regression problem" in your paper.
I will try with a multivariate datasets.

A probabilistic forecast, not a point  forecasting
I'm going to apply 'Quntile regression'.
I wonder if it can be implemented without difficulty in the current architecture.
For example, simple 'loss function, ..' modifications, etc.

Please let me know if you have any advice.
Thank you.

Hi @TaeniKim,

Multivariate regression problem:
We are tackling the problem P(y_[t+1:t+H] | y_[t-L:t], X_[t+1:t+H], S), where H is the forecast horizon and L are the lags, y is the target variable and X are temporal exogenous variables available at the prediction time.

We have the Multi-Quantile Loss  available in the neuralforecast library.

You can use it to model marginal distribution, with a non parametric approach.
In the past we adapted the outputs of the MLPs in the N-BEATS architecture to track the number of quantiles that you want to predict.


feat: add option to hide attributes in attribute selector plugin

What does this pull request change?
add option to hide attributes in the attribute selector plugin.  This can be controlled by the config (see packages/dm-core-plugins/blueprints/tabs/AttributeSelectorPluginConfig.json)
Why is this pull request needed?
Issues related to this change
closes #119

@soofstad @ingeridhellen feel free to take over this PR since I will not be able to fix review suggestions this week :)


[Monitoring] Harvester connection message wrong

I unplugged the harvester physically which lead correctly to a disconnect event. A second afterwards a connection message came wrongly. Checking with $monitors indeed revealed that the harvester is not under supervision anymore. After the DELETE event from the farmer there is oddly an ADD event following. This might be related to the logfile checks but further investigation is needed.

After about 10 to 15 mins another disconnect message came. This very much looks like being a problem with the logfile since in the logs it takes much longer to not recognize a harvester from the view of the farmer.

The add event seems to be caused by an increment of nDisconnects. The harvester from the view of the farmer is removed with a "DELETE" event and then again added since an attribute changed.

did not find a nDisconnects variable in the develop branch. Is this bug still there?

@tnt-manu the deployed version is still before your PR changing nDisconnects to nTimeouts. There are just a few things I don't understand:

According to here there may not be any message about a harvester from the farmer if either API or logfile say that it is diconnected. So how can this previous message even get through?
If this happens with nDisconnects it might but must not also happen with nTimeouts.

last_message_time reported by the farmer is last time a message was sent to the harvester and not received!!!


Issue NST if incoming ST is of unexpected size

Issue # (if available):
Description of changes:
Previously if s2n couldn't recognize ticket size, it would ignore it and not issue a new session ticket, causing client to hold to the bad ticket without ability to recover.
With this change we would issue a new session ticket in such cases.
Also fixed buffer overrun in s2nd.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

:exclamation: No coverage uploaded for pull request head (fix-unrecognized-session-tickets@886c5ed). Click here to learn what that means.
The diff coverage is n/a.

Travis build failed due to the broken python on build machine (same issue in master).


[firebase_messaging]onLaunch is called multiple times

Hi,
My app uses firebase cloud messaging for push notifications. Unfortunately, onLaunch is called too often.
You’ll find an example app to reproduce the issue below (unfortunately, you have to follow all steps to configure the messaging plugin like state here (https://pub.dev/packages/firebase_messaging#-readme-tab-) before you can reproduce the issue).
Following scenario:
The app has two pages. If a push notification is received the app should open page 2. On Page two is a button to go back to page 1. After the user clicks on this button, the app is expected to be on page 1. This works fine, if the app wasn’t terminated when the notification was received. Alas, if it was terminated, the app opens page 1 just for a few minutes and jumps forward to page 2 again.
Steps to reproduce:

Start the app and note the
Terminate the app
Send a cloud push message to the noted token
--> A notification should appear
Click on the notification
--> Apps opens page 2
Click Button “Back to page 1”
--> <Expected:> App is on page 1
--> <Actual:> App is on page 1 for a few milliseconds and opens page 2 again

Logs
Unfortunately, logs aren’t possible because is only happens, if the app was terminated previously…
C:\Users\x\Projekte\flutter_test\notification\flutter_app>flutter doctor -v
[√] Flutter (Channel beta, v1.5.4-hotfix.2, on Microsoft Windows [Version 10.0.17763.437], locale de-DE)
    • Flutter version 1.5.4-hotfix.2 at C:\Dev\flutter
    • Framework revision 7a4c33425d (2 weeks ago), 2019-04-29 11:05:24 -0700
    • Engine revision 52c7a1e849
    • Dart version 2.3.0 (build 2.3.0-dev.0.5 a1668566e5)

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\x\AppData\Local\Android\sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    • All Android licenses accepted.

[√] Android Studio (version 3.4)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[√] Connected device (1 available)
    • Pixel 2 • xxxxx • android-arm64 • Android 9 (API 28)

Example App:

import 'package:flutter/material.dart';
import 'package:firebase_messaging/firebase_messaging.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);
  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  final FirebaseMessaging _firebaseMessaging = FirebaseMessaging();

  @override
  void initState() {
    _firebaseMessaging.getToken().then((token) {
      print(token);
    });
    _firebaseMessaging.configure(
      onMessage: (Map<dynamic, dynamic> message) => _redirectToPageTwo(),
      onResume: (Map<dynamic, dynamic> message) => _redirectToPageTwo(),
      onLaunch: (Map<dynamic, dynamic> message) => _redirectToPageTwo(),
    );
    super.initState();
  }

  _redirectToPageTwo(){
    Navigator.pushReplacement(context, MaterialPageRoute(builder: (context) => PageTwo()));
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(child: Text('Page1'),),
    );
  }
}

class PageTwo extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Column(
        children: <Widget>[
          Center(child: Text('Page2'),),
          RaisedButton(child: Text('Go back to page 1'), onPressed: () => Navigator.pushReplacement(context, MaterialPageRoute(builder: (context) => MyHomePage())),)
        ],
      ),
    );
  }

}

name: flutter_app
description: A new Flutter application.

version: 1.0.0+1

environment:
  sdk: ">=2.1.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

dev_dependencies:
  flutter_test:
    sdk: flutter
  firebase_core: ^0.4.0
  firebase_messaging: 5.0.0

flutter:

  uses-material-design: true

@ehhc
The issue at https://github.com/flutter/flutter/issues/32698 has been closed and moved here. Future collaboration on this issue will be done here.

Any news from the devs?

I also have this problem. Is there any information available on when it will be fixed? And for now, is there any work around solution?
Thank you!

Hello,
This issue is always present even with the last version of the plugin.

I have been battling with this issue for some time now.
From main.dart my app uses routes: and onGenerateRoute: to navigate to the 12 other pages in the app.
From 'routes:' there are two pages that are navigated to using pushReplacementNamed and these pages navigate to the other pages from their side drawers using push and pop (simplistically). As others have said, sometime you just HAVE to use pushReplacementNamed.
I am using FCM to notify the user of several different events such as other people wanting to contact them. These notifications need to work when the app is active, in background, or closed. All good so far.
When the notification is received I use onMessage, onLaunch, or onResume to navigate to certain pages depending on the type of message received. I also show a dialogue, giving them a choice, in the case of onMessage and onResume.
The app also supports multiple languages so the text in the dialogues needs to be translated.
The app also uses scoped model to separate database logic (Firebase) from UI, some of the functions in scoped model are needed in the processing of the received notifications.
I would consider this a relatively complex, but normal, real-world app.
Because of the above and as far as my understanding of Flutter/Dart goes, the FCM handler (that has the call to .configure) needs to access context and scoped model in order to show the correctly translated dialogues and to navigate to the correct page. I cannot call .configure in main.dart initState because it doesn't have scoped model yet. Equally, it doesn't have a context that supports navigation or the data needed from the pages.
After many attempts at different 'solutions' and  much Googling, StackOverflowing, Mediuming, etc. I settled on calling the FCM handler in the initStates of the two 'root' pages,. All notifications are then processed successfully whether the app is active, in background or closed. The only downside is the 'onLaunch' loop which shows itself whenever I switch between the two 'root' pages, FCM handler is fired again in the initState of each page and I get a repeat navigation of what was just shown in the other 'root' page. However, I HAVE to call the FCM handler at the opening of each of these pages in order to ensure the correct context for the text translation and dialogues. Hope that I am making sense. :-)
Anyway, I seem to have a simple workaround for now (hopefully, the behaviour gets fixed properly) so thought I would mention it here. I have set up two 'global' bools to ensure that onLaunch and onResume only fire once, irrespective of how many times I switch between the two 'root' pages. The bools are checked and set so:
onLaunch: (Map<String, dynamic> message) async {
  if (!globals.hasResumed && !globals.hasLaunched) {
    globals.hasLaunched = true;
...}
onResume: (Map<String, dynamic> message) async {
  if (!globals.hasResumed && !globals.hasLaunched) {
    globals.hasResumed = true;
...}

Hope this helps.

Hi @GrahamDi ,
thank you for your workaround solution. Unfortunately it doesn´t work for me, because the first click on a push works but the click on the second push notification doesn´t work. Or do I missunderstand something?
Thank you in advance.

@julian247 I would need to understand more about what you mean by 'first click on a push' and 'click on second push'. How are the notifications being sent? What is intended to happen when they are received?  What is the state of the app when they are received, etc.? Maybe if you shared some code I could help.
Fyi, my FCM notifications are all sent from the Firebase backend using node.js cloud functions.

@GrahamDi Thank you for your fast reply.
I try to explain it in detail. The FCM notifications are sent by a PHP Script and a CURL call. This works well. The problem is that if the user clicks on the notification I want to push him to a special screen. When the user click on it, onLaunch fires multiple times. I have implemented that the onLaunch can only be called once. Now it works for the first click. But when an other notification arrives the onlaunch is not called because the Bool is already set to True from the other push notfication.
At my app, in every Page is at the bottom the Messaging_widget called with the configure... in it.
Do you understand the problem?

@julian247 I don't know much about PHP or curl but as you say the notifications are working fine on being sent. Some questions:

To confirm, it is a Flutter app that the users are receiving the message with?
If so, what is the 'first click' ie. what are they clicking on?
Is the app opening from closed, in background, or are they receiving the message when it is active?
Assuming that the app is closed, then I am puzzled why onLaunch then has the bool set to true. If it was closed, then when it restarts the bool should be reinitialised (I set it initially to false in the global config code file).
When you say every page has the .configure call, do you navigate between these pages with push and pop or do you use pushReplacementNamed?

Holen Sie sich Outlook für iOS
On Sat, Dec 21, 2019 at 6:25 PM +0100, "GrahamDi"<EMAIL_ADDRESS>wrote:
@julian247 I don't know much about PHP or curl but as you say the notifications are working fine on being sent. Some questions:
To confirm, it is a Flutter app that the users are receiving the message with?
If so, what is the 'first click' ie. what are they clicking on?
Is the app opening from closed, in background, or are they receiving the message when it is active?
Assuming that the app is closed, then I am puzzled why onLaunch then has the bool set to true. If it was closed, then when it restarts the bool should be reinitialised (I set it initially to false in the global config code file).
When you say every page has the .configure call, do you navigate between these pages with push and pop or do you use pushReplacementNamed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

@GrahamDi

Yes, it is.
They click on the push notification they get on their phone. The app is in the background at this moment.
The problem occurs when the app is in the background.
When the app is closed, it works fine.
I navigate between the pages mostly with push.

Thank you! :)

@julian247 I find my simple fix is not working perfectly, so I guess it is not a fix :-(
I am going to spend more time on it today.
One thing I would suggest for your situation is only to call .configure once, in the initstate of the root (first) page in your app. As long as that root page is never closed (it isn't when you 'push' to another page and then 'pop' back), FCM should work fine. It does in my case. I only get the problem when I have to pushreplace the root page with an alternate one, which calls another FCM .configure ie. the problem I experience is when configure is called more than once.

@GrahamDi Thank you for the input. Now only the main screen calls it. But now I have the problem, that when i click on the main screen it navigates me automatically to the other screen. I think you solved this problem with the bools. Can you please describe this in more detail?
Thank you!

That almost sounds like the onLaunch loop is still there. Are you absolutely sure that FCM.configure is only being called once? If you haven't already, perhaps put print statements at appropriate places in the .configure method to double check. My bools were simply my (bad) attempt to handle the multiple calling of .configure.
I have appear to have solved my problem by replacing my Navigator.pushReplacementNamed calls with Navigator.pushNamed and so making sure that all my pages are children of my home page. I then call FCM.configure only once in the home page initState. I have removed the bools as I no longer need them. Everything seems to be working fine, although I am still doing testing.

@GrahamDi Yes, unfortunately it is. :-/
And yes, i have removed all the other calls.
I try to Test the the pushNamed idea. Thank you:)

@julian247
I fear I am out of ideas then, without seeing your code. :-(
Sorry and all that.
Just to say that pushReplacementNamed closes the page being replaced, so if the page you call configure from is that page then maybe that also impacts FCM.configure or it is called again.
My code seems to work now that I never close the page that called FCM.configure. Good luck.

I have a similar issue, what would help me it is a clear function for FCM configuration. For now I'm just using a simple validation to avoid my problem.

I have a similar issue, what would help me it is a clear function for FCM configuration. For now I'm just using a simple validation to avoid my problem.

@kroikie This issue still appears in the latest plugin. Please take a look.

This is occurring for me on iOS too

https://github.com/FirebaseExtended/flutterfire/issues/1669#issuecomment-625187936

Hello everyone ! Its a serious problem but i have done a work around that every notification has a unique id found in message['data']['google.message_id'] where message is the Map<dynamic, dynamic> that u pass while configuring the fcm , if you can store that id and can perform operations.
Like if the id is same as the previous notification then just just do nothing and if its not then perform your operation u want to perform.

@Soumyadeep21  Thanks, it works!

News on this?

Hey all 👋
As part of our roadmap (#2582) we've just shipped a complete rework of the firebase_messaging plugin that aims to solve this and many other issues.
If you can, please try out the dev release (see the migration guide for upgrading and for changes) and if you have any feedback then join in the discussion here.
Given the scope of the rework I'm going to go ahead and close this issue in favor of trying out the latest plugin.
Thanks everyone.


Implement page.$eval

This patch:

implements page.$eval and frame.$eval
drops elementHandle.attribute() method in favor of the page.$eval

References #625

Can this take ...args like the old page.$?

Could use an  example.. maybe one of these?
const searchValue = await page.$eval('#search', el => el.value);

const preloadHref = await page.$eval('link[rel=preload]', el => el.href);

const html = await page.$eval('.main-container', e => e.outerHTML);

@JoelEinbinder sure. Do you have an example where arguments would be useful?


Scrim doesn't contain statusbar

Please complete the following information:

Library Version 0.1.0
Affected Device(s) Samsung Galaxy s23 with Android 13]

Describe the Bug:
With only Horizontal sides' insets scrim doesn't contain status bar 🧐
windowInsets = BottomSheetDefaults.windowInsets.only(WindowInsetsSides.Horizontal)

Expected Behavior:
A clear description of what you expected to happen.

Hey @chandroidx, sorry for the late response. This is a very expected result since it should be done by the user side.
You can simply resolve this issue by changing the statusbar color like the method below:

Add the accompanist dependency below:

```kotlin

Hey @chandroidx, my apologies for the delayed response. It's expected that this task falls under the responsibility of the user.
To address this issue, you can easily change the status bar color using the following method:

Add the accompanist dependency below:

implementation("com.google.accompanist:accompanist-systemuicontroller:0.32.0")

Observe the flexible bottom sheet's target value changes and change the statusbar color when the state value is not hidden like the example below:

  var currentSheetTarget by remember { mutableStateOf(FlexibleSheetValue.IntermediatelyExpanded) }
  val systemUiController = rememberSystemUiController()

  val primaryColor = MaterialTheme.colors.primary
  val scrimColor = Color.Black.copy(alpha = 0.65f)

  LaunchedEffect(currentSheetTarget) {
    if (currentSheetTarget == FlexibleSheetValue.Hidden) {
      systemUiController.setStatusBarColor(
        color = primaryColor
      )
    } else {
      systemUiController.setStatusBarColor(
        color = scrimColor
      )
    }
  }

  FlexibleBottomSheet(
    onDismissRequest = onDismissRequest,
    sheetState = rememberFlexibleBottomSheetState(
      flexibleSheetSize = FlexibleSheetSize(),
      isModal = true,
      skipSlightlyExpanded = false,
    ),
    onTargetChanges = { sheetValue ->
      currentSheetTarget = sheetValue
    },
    containerColor = Color.Black,
    scrimColor = scrimColor
  ) {
     ..
    }

This is the result as you've expected:

Hey @chandroidx, I'm closing this issue for now. If you still face the same problem while addressing it with my suggestion, please reopen and discuss this anytime!


[Backend] POST API For Questions under Q&A

Deliverables-

[ ] Make the POST API for question addition containing below fields-

title[string]
description[string]
tags[array of strings]
isApproved [boolean default-value=false]
upvotes[number]

Can i work on this, issue ?

Sure @salil-naik please go ahead with this issue.

Do I need to add auth middleware in posting question ?

No @arinmodi this api should be open.

Hey @Kajol-Kumari, when running test using npm run test command, all the tests are failing, do i need to do some setup for testing ?

Okay

Hey @Kajol-Kumari, i made a PR

close via #703


CPreprocessOptions should include /E

If you are specifying a non-empty CPreprocessOptions, it should include /E to actually pre-process the idl file

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

ID: 1c9801fb-3278-8e3d-e11a-19cfc07b903c
Version Independent ID: e5900f57-a15f-19be-eb93-51da01b3ad1b
Content: MIDL Task - Visual Studio
Content Source: docs/msbuild/midl-task.md
Product: visual-studio-windows
Technology: msbuild
GitHub Login: @ghogen
Microsoft Alias: ghogen

It appears to be fixed already - the article says to use /E option.


Fix thread sanitizer issue

Signed-off-by: Dmitriy Khaustov aka xDimon<EMAIL_ADDRESS>

Referenced issues

Description of the Change

Benefits

Possible Drawbacks

Usage Examples or Tests

Alternate Designs

It will applied as part of PR #981


Accidentally dropped support for Solaris and Illumos?

Describe the bug
Prior to 35c31f21, bubbletea supported GOOS=solaris and GOOS=illumos.
35c31f21 bumped the version of github.com/containerd/console from v1.0.3 to the unreleased v1.0.4-0.20230313162750-1ae8d489ac81. This version of containerd/console includes https://github.com/containerd/console/commit/fa15abf58f2a4dcfbac2767caf17f57c3e029006, which drops support for Solaris and Illumos.
Is there any reason to drop support for Solaris and Illumos?
To Reproduce
$ git clone https://github.com/charmbracelet/bubbletea.git
...
$ cd bubbletea
$ git checkout v0.24.0
...
$ GOOS=solaris GOARCH=amd64 go build .
# github.com/containerd/console
/Users/twp/pkg/mod/github.com/containerd/console@v1.0.4-0.20230313162750-1ae8d489ac81/console.go:83:12: undefined: checkConsole
/Users/twp/pkg/mod/github.com/containerd/console@v1.0.4-0.20230313162750-1ae8d489ac81/console.go:86:9: undefined: newMaster

Source Code
n/a
Expected behavior
bubbletea builds with GOOS=solaris and GOOS=illumos.
Screenshots
n/a
Additional context
n/a

I opened https://github.com/containerd/console/pull/76 to restore Solaris support to containerd/console.

That was indeed unintentional. I bumped to that particular console revision for unrelated Windows fixes. Thanks for opening a PR over there, @twpayne!

So, there's not been much interest from the containerd/console developers in restoring Solaris support. See https://github.com/containerd/console/issues/75 and https://github.com/containerd/console/pull/76.
As I want to support these OSes, I am currently blacklisting charmbracelet repos that require the versions of containerd/console that drop Solaris and Illumos support. I also have a user request to run on AIX (https://github.com/twpayne/chezmoi/issues/2787), and if containerd/console is dropping support for Solaris and Illumos then it is extremely unlikely that they would want to support AIX.
Where do we go from here?
As the current versions I could continue to blacklist charmbracelet repos. I don't need the new charmbracelet functionality yet, but this creates extra maintenance effort for me as I have to blacklist each new version and is a poor long term strategy as at some point I will want to use some feature or bug fix that is only available in a more recent charmbracelet repo.
I would be happy to co-maintain a friendly fork of containerd/console that supports a full range of OSes. As I argue in https://github.com/containerd/console/issues/75#issuecomment-1567605232, I believe that the underlying APIs are stable and so maintaining such a repo would not be a significant burden. Given that charmbracelet is focused on console-based tools, to me it makes more sense to fork containerd/console to charmbracelet/console rather than twpayne/console.
Thoughts?

I agree, a friendly fork is probably the easiest way forward here. We actually require only very little functionality from containerd/console, which could also eventually be extracted.

OK. If you want to fork containerd/console to charmbracelet/console then I'll open a PR to restore Solaris support and optionally strip out the unused functionality.

@twpayne There we go: https://github.com/charmbracelet/console

Next step: switch the dependency in Bubble Tea + other Charm apps that use console.

Closing in favor of https://github.com/twpayne/chezmoi/pull/3060.


Discrimination by model value?

Hey, let's say I have the following model:
model Vehicle {
  type VehicleType
  settings Json @default({})
}

enum VehicleType {
  CAR
  BIKE
  BICYCLE
  QUAD
}

And my settings column might have different JSON data depending on type. Is there any way to have type as the discriminator?

I don't think this is possible because it would require us to make Vehicle type have a generic parameter in the dts file:
export type Vehicle<T extends VehicleType> {
  type: T;
  settings: T extends VehicleType.CAR ? ...
}

closing this until further solution is available


Add summary card component and styles

The Access your teaching profile prototype uses the summary card component. This PR provides the summary components and styles.

Does https://github.com/DFE-Digital/govuk-components provide some of this?

https://github.com/DFE-Digital/govuk-components/pull/351

Closing as govuk-components provides this now.


Could u change compile 'com.android.support:appcompat-v7:25.3.1' to privided?

Could u change compile 'com.android.support:appcompat-v7:25.3.1' to provided?

The next release will have it changed to api. Closing this issue as it's very old.


Updated logic for self sparkles

Moved the logic to check if the sparkler is the sparklee to the top of the if block.

superseded by #20


tab: init at 7.1

Motivation for this change
Things done

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

[x] 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 nix-review --run "nix-review wip"
[x] 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)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Version 7.2 is available, so I am closing this PR, and submitting a new one: #71466


⚠ BeyondHD has degraded performance

In 58f840e, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 3076 ms

Resolved: BeyondHD performance has improved in 9c156ca after 9 minutes.


so is it crashing for a lot of people or no?

i need info if i should use it. also whats that chance of me getting vac banned. i already got 3 vac bans for using alagman or whatever it was. i dont wanna waste another 40$ just to get my taunts back

use vac bypass by daniel and at that point just get a paid cheat to avoid bans and wasted money

i would get lmaobox if i would get paid cheats, but thats why im here

im broke

also @Viceroyy does TFGhost work too? thats what i used the last few times

i would get lmaobox if i would get paid cheats, but thats why im here

its 2024 ideally you should either be getting rijin or moneybot anyway

also @Viceroyy does TFGhost work too? thats what i used the last few times

idk i never used it, i recommend using xenos instead

alr thanks. ill try xenos out

yeah it is kinda crashing... do you have a fix?

idk, haven't tried it yet lmao
On Mon, Aug 19, 2024, 06:52 gamergod69299 @.***> wrote:

yeah it is kinda crashing... do you have a fix?
—
Reply to this email directly, view it on GitHub
https://github.com/Viceroyy/Fusion/issues/16#issuecomment-2295665467,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BJKHQRHKQ4IRL3OEF4H42PTZSF22RAVCNFSM6AAAAABMUBNR7OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJVGY3DKNBWG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


WIP - Fix: nested attributes {}

#WIP

adds a test for checking of attributes:{} existance for relationship's relationship when both are array of objects
will fix https://github.com/bornfight/aardvark/issues/72
e.g.

relationships: {
                    owner: {
                        data: [
                            {
                                id: "2",
                                type: "owner",
                                attributes: undefined,
                                relationships: {
                                    diploma: {
                                        data: [
                                            {
                                                id: "4",
                                                type: "diploma",
                                                attributes: undefined,
                                            },
                                        ],
                                    },
                                },
                            },
                        ],
                    },
                },

We will further discuss this.

@JuraJuki What is the status of this PR?

@JuraJuki What is the status of this PR?

The current status of it, is that I need to discuss with you about it. Testing it is a bit of a problem because of nested data..the console would not write the differences. Other than that, I'm not sure why is it happening...Why does attributes:{}  show up again

We agreed upon the frontend team to do the work for this in a short debate with Matteo and Renato. The attributes: {} is unnecessary here, it is not breaking but it would be better off without it

We could probably remove attributes somewhere here https://github.com/bornfight/aardvark/blob/master/src/json-api-client/JsonaDataFormatter.ts#L102
@JuraJuki

@renatoruk I think this should be the proper fix, mind taking a look? I'm not sure how to test this since tests do not print this deep. Other than that it is kinda odd that tests (when logged in a console) show only id and type keys on the object. But clearly on the example it shows that it is not true.

Cool! Awesome that you've updated the docs. Mind taking a look why the coverage fell? It should be available by going to codecov detail report


[Adaptive] Can't return an Array type in the ResultProperty of EndDialog

Version
What package version of the SDK are you using.
4.6.0-Daily-2019-08-13-01
Describe the bug
When trying to return an Array property as the ResultProperty of the EndDialog action, I get the following error "Can not convert Array to String".
To Reproduce
Steps to reproduce the behavior:

Set the ResultProperty of EndDialog to an array property.
Try to capture the result in the BeginDialog action.
Get error

Expected behavior
Give a clear and concise description of what you expected to happen.
Should be able to return property of any type as the result of EndDialog.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
[bug]

We should support string value, array as well as object in what can be passed in to a dialog via options as well as what can be returned from a dialog via EndDialog -> property.

Try with the latest in 4.Future. This is probably fixed but if not, reactivate.