[BUG] File_url parameter is optional but it says mandatory field.

When Up Uploading a file from local storage it shows file_url is mandatory.
The Error logged :-
{"_server_messages": "["{\"message\": \"Fields file_name or file_url must be set for File\", \"title\": \"Message\", \"indicator\": \"red\", \"raise_exception\": 1, \"__frappe_exc_id\": \"edfbf39cb7c6aa07fd6c67fcbaadbc703635505b7b87ff1571247742\"}"]", "exc_type": "MandatoryError", "exception": "", "httpStatus": 417, "httpStatusText": undefined, "message": "There was an error while uploading the file."}
fileName>>>>> 820b6003-d080-4e81-ac44-afcc909aaf96.png
Payload :-  const fileArgs = {
isPrivate: true,
folder: "Home",
file_name: fileName,
doctype: "User",
docname: "sub doc type",
fieldname: "user_image"
};
file
.uploadFile(myFile, fileArgs, (completedBytes, totalBytes) =>
console.log(Math.round((completedBytes / totalBytes) * 100), " completed")
)
.then((r) => console.log(r))
.catch((e) => console.error(e));

@Abhishekranjan56 Which version are you using?
Also ,you do not need to pass in the "file_name" either if you are uploading a File object.

I am using     "frappe-js-sdk": "^1.4.3", and if I am not passing the file_name it doesnot matter it still shows same error.
{"_server_messages": "["{\"message\": \"Fields file_name or file_url must be set for File\", \"title\": \"Message\", \"indicator\": \"red\", \"raise_exception\": 1, \"__frappe_exc_id\": \"9939bee13b0e544363299709426c69a0af0b249b360be674854890bc\"}"]", "exc_type": "MandatoryError", "exception": "", "httpStatus": 417, "httpStatusText": undefined, "message": "There was an error while uploading the file."}
still shows this error with
const fileArgs = {
isPrivate: true,
folder: "Home",
doctype: "User",
docname: "sub_doc_name",
fieldname: "user_image"
};
with this payload

Can you clear your node modules and .lock file and install all the dependencies again? I am not able to reproduce it on our end. We were facing some issues earlier, but that was fixed in v1.4.3

Can you clear your node modules and .lock file and install all the dependencies again? I am not able to reproduce it on our end. We were facing some issues earlier, but that was fixed in v1.4.3

Sure
Will test and Update.

Done that but same error still persist.

This is the sample of file object I am sending
file:///data/user/0/com.abhishekapp.dev/cache/ImagePicker/f321fbf2-ddec-4bda-bffe-8b57bd1e8289.png
it is the local path of image I want to attach to frappe dashboard using expo image picker.
@nikkothari22

Oh okay. I didn't know you were using Expo (RN). I haven't tried it on Expo myself, but I know Hussain has an example of a file upload function in doppio_mobile. You won't be able to use the inbuilt function here for raw file uploads but we plan on adding support for that in frappe-JS-SDK and frappe-react-SDK

In my case, I use this code in my React Native and this error resolved
const pictureData = {       name: photo.fileName,       type: photo.type,       uri: photo.uri,     };
Make a data like this in your project


WIP: Pydantic schema generation

I've been experimenting with Pydantic and Django models generally, and realised that it might not be too much effort to include support for Pydantic schema generation directly in this project.
This PR is a work in progress to demonstrate how we might use Pydantic models here. There are a few things that could be improved, like error handling and overriding the default schema, maybe a base model or dynamic model - there's a lot of features in Pydantic to consider.

Thanks for your Pull request @jordaneremieff. I definitely see a lot of value in using Pydantic for API validation (faster to validate, better readability schemas in comparison to JSON schema, autocomplete). The goal of this project is slightly different though. The goals is to provide dynamic UI for structured JSON data. As it's rely heavily on JSON Schema, I'm afraid using Pydantic here would be more harmful as you need to translate Pydantic configs to JSON schemas. During this translation you might loose some important JSON schema details which will break frontend lib.
I suggest you can reuse this code and create custom seraliser field (for use with DRF) and form fields (for use with Django) based on Pydantic. A project like his might already exist though.
After I did a quick research I found this library which does a more generic integration of DRF and Pydantic:
https://github.com/rsinger86/drf-typed-views
It seems it's a different library though as it replaces the Seriliazers with Pydantic. The goal of your project could be to extend the standard JSON field in Django with Pydantic based validation.
Happy to discuss this more offline if you wish.

As it's heavily rely on JSON Schema, I'm afraid using Pydantic here would be more harmful as you need to translate Pydantic configs to JSON schemas. During this translation you might loose some important JSON schema details which will break frontend lib.

Pydantic provides fairly robust JSON schema generation features that are compliant with JSON Schema Core, JSON Schema Validation and OpenAPI, and it also allows customisation and extension, https://pydantic-docs.helpmanual.io/usage/schema/#json-schema-types, I think its JSON support is generally stable and reliable.
That said, I don't know how well it might work with this particular React library. I did run into a few differences in how it understands generated field descriptions, so it may not serve well as a drop-in replacement for the current field.
Thanks for taking a look at the PR! I learned a bit about how to implement custom Django field widgets and Pydantic features by exploring this, and I can continue my efforts in a separate package.


Add other values of user-select property

From the spec...user-select: none is in there but there is no data for user-select: all or user-select: contain
Thanks!

Yeah, this was mostly 'cause user-select only recently started being spec'd, originally only support for "none" seemed to be consistent.

👍

+1
Saw https://css-tricks.com/force-selection-text-block which has a demo for user-select: all: https://codepen.io/chriscoyier/pen/dXKxvv (doesn't work in Chrome 52, does work in 54/canary)

+1

These are now available via MDN BCD data.


Synx staking - no rewards coming in

I have been staking 1000 coins (Synx) and in 36 hours havent recieved a single reward.
Appreciate some assistance

I got the same issue.  I went in the debug console and i saw this error... not sure what it means.
ThreadStakeMinter() error

I am having this same issue as well. Not only is my wallet not staking even though it is unlocked for staking, but I also lost 450 SYNX that I obtained from staking on the previous version of the wallet. when I loaded the new version I was told that the wallet had an error reading and that it may be missing some address book information or transactions. I am missing ALL transactions outside of the original deposit of 428 that I made way back when I first started. There should be at least 2 more transactions from cryptopia where I converted some of my BTC to SYNX and several transactions from staking rewards.
When I try my wallet backups, all the recent ones are corrupt, and the previous ones are loaded in before I even created the staking wallet and have no addresses to send SYNX. 3 of the "backups" say my password is wrong when I try to unlock for staking and have the wrong receiving address.
This is a very bad update. Very broken, there is no staking, and your previous transactions will probably be deleted...


Fixes for Image.table and regular sampling

There's a bug in Image dimension_values, which processes the meshgrid incorrectly and which means that the .table output is broken. Additionally regular sampling on Images now sorts the data correctly. I've also included some fixes/improvements for the Sampling Data Tutorial since this PR changes almost all the output there.

Tests are passing, this PR is now ready for merge.

Ok. Merging!


Mock: Time Controller

Requirements:

The controller use the following path /v1/time,
The controller must handle an HTTP GET request that returns the current time of the server.
The controller must handle an HTTP POST request that contains the following JSON structure

{
   requesteesTime: *someDateFormateHere*
}

The server will return the difference in time between the sent DateTime as json.
For example:
{
   timeDifference: *difference in time here*
}

Let me know if you have any questions :)

@wesbyte29 any update on this controller?


update wording for genomics community

I didn't mean to add the first two of these commits sorry @neoformit

I think that should be ok Anna... still only one file changed. If you haven't already, check out the files tab to see the diffs for your PR.


[DM local testing] Docker image and tutorial for local testing

Additional scope implemented.
Fixed dm/CI/cft_base_container - wasn't able to build it initially.
Added make command to build cft-base-image to build it.
Added cft-test-bats command to run test locally, example command:
cd ./dm
make cft-test-bats TEST=templates/instance/tests/integration/instance.bats
Fixes #291

One things to change, can you please remove the cft_base_contianer folder and make it a symbolic link to cft_base_container?

@ocsig

One things to change, can you please remove the cft_base_contianer folder and make it a symbolic link to cft_base_container?

It's already there but github for some reason don't show simbolic link file here, checkout branch to check it.


pypy 4.0.1

We have released PyPy 4.0.1, three weeks after PyPy 4.0.0. We have fixed a few critical bugs in the JIT compiled code, reported by users. We therefore encourage all users of PyPy to update to this version. There are a few minor enhancements in this version as well.  -- http://morepypy.blogspot.com

:+1: this would be nice! A few fixes in 4.0.1 we'd love to have.

Deployed!


fix(snapshot): support invariant entry parsing

Motivation
parse invariant test entries in snapshots correctly
cc @transmissions11

Solution

y'all simply dont miss


Email for reporting tool

Fixes  https://github.com/CircuitVerse/CircuitVerse/issues/1465
Describe the changes you have made in this PR -
I have added a email text field in the reporting tool using html and fetch the value of email text field using jquery and concatenate it with message variable
Screenshots of the changes (If any) -
Reporting tool

Slack:

Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.379% when pulling 7b867bf26889072bc034b263ac4ec53e461568a5 on ryuk156:Mail-reporting-tool into 3553368dc7e37c6ceaf64c55c576079f6e81c484 on CircuitVerse:master.

@nitin10s  @satu0king @tachyons  review and merge please

@nitin10s  @tachyons @satu0king  review and merge please

This is what I see when I tried it ...

I made changes can you check it again please it will work now


SNOW-683056 Add unit test to cover DF_PANDAS_GENERAL_EXCEPTION

Description
as title
Testing
unit test
Please answer these questions before submitting your pull requests. Thanks!

What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-683056

Fill out the following pre-review checklist:

[ ] I am adding a new automated test(s) to verify correctness of my new code
[ ] I am adding new logging messages
[ ] I am adding a new telemetry message
[ ] I am adding new credentials
[ ] I am adding a new dependency

Please describe how your code solves the related issue.
Improve test coverage for _internal/error_message.py

__init__ is deleted because pytest cannot find modules when this is present.


Addition to clarify SQS batch size limit on Lambda trigger

Description of changes:
Added clarity on SQS trigger batch size limit to 'Using Lambda with SQS' docs. It's not clear on the Lambda GUI or the Lambda docs what this limit is, or what imposes it.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks, I've added the limit to the procedure. It's enforced by the console and the Lambda API, to prevent creation of a mapping that would fail later when it tries to read a batch.


conftest: 0.23.0 -> 0.24.0

Motivation for this change
Bump conftest to 0.24.0
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-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)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119655 run on x86_64-linux 1

1 package built:

conftest


Accessing 'Submit a Listing' from any UI in OZP

When a user clicks 'Submit a Listing,' the toolbar indicates they are taken to Center.  When you're done submitting an app for listing, is the user taken back to the UI they were using (for example, clicking 'Submit a Listing' from Webtop) or do they have to manually navigate themselves back?

When the user is done they are taken to the My Listings page not to the where they navigated from originally.

Not an issue or to low priority


select box does not work on pressing enter

if we are focused on any option and press enter., It doesnt work. It works by pressing space bar. I know behind he scene we are using checkboxes, thats why selection works by pressing spacebar. But plz make it selected even by pressing enter key.

Hi,
You'll need to modify the code yourself. Just find the ASCII 32 and replace it with.. 13 (I think enter is 13)

Thanks for quick response. I am not so good in this. Can you help me.
I am using the following js file.
http://isteven.github.io/angular-multi-select/js/libs/isteven-multi-select.js

Sorry, after re-looking at the problem, I think you would need create an extra keypress listener instead. Try to detect key 13 (Enter) in this listener and trigger the "syncItems" function.
Again, you will need to modify the code yourself.

I have event on click of enter. In "syncItems" we are passing three arguments, that are item, e, ng_repeat_index. I think e is for event which I have, what about rest two??

Item is the currently highlighted object in the input-model.
ng_repeat_index is the index of the currently highlighted object in the input-model
Yes, e is the event.
You can see the way it is called from the template at the bottom of the code.

Thanks for support.
Finally I have done it.
Thanks

:-)

I have used the following line, when we press enter
element.bind("keydown keypress", function (event) {
if(event.which === 13) {
event.preventDefault();
var check = angular.element(event.target);
check.trigger('click');
$scope.$apply();
}
});


Error building application emqx:No project builder is configured for type mix

===> Compiling c_src/async_queue.c
===> Compiling c_src/bcrypt.c
===> Compiling c_src/bcrypt_nif.c
===> Compiling c_src/blowfish.c
===> Running neotoma...
===> Compiling emqx
===> Error building application emqx:
No project builder is configured for type mix
what's wrong?how to resolve Question??

any update for this issue ? I faced same problem

use hook
发自我的iPhone
------------------ Original ------------------
From: manhvu1997 @.>
Date: Thu,Jun 20,2024 8:07 PM
To: emqx/emqx-auth-redis @.>
Cc: macren @.>, Author @.>
Subject: Re: [emqx/emqx-auth-redis] Error building application emqx:No projectbuilder is configured for type mix (Issue #207)
any update for this issue ? I faced same problem
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

use web hook
发自我的iPhone
------------------ Original ------------------
From: manhvu1997 @.>
Date: Thu,Jun 20,2024 8:07 PM
To: emqx/emqx-auth-redis @.>
Cc: macren @.>, Author @.>
Subject: Re: [emqx/emqx-auth-redis] Error building application emqx:No projectbuilder is configured for type mix (Issue #207)
any update for this issue ? I faced same problem
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Use breadcrumbfiles associated with each asset, versus shared

See title. This addresses the issues that were temporarily mitigated by this .NET PR.

Is this going to have any conflicts when people upgrade locally? i.e. if the new proxy ignores the existing old breadcrumb file, will it try to overwrite stuff or use different paths? I don't fully understand the "pre-breadcrumb file" behavior.

Is this going to have any conflicts when people upgrade locally? i.e. if the new proxy ignores the existing old breadcrumb file, will it try to overwrite stuff or use different paths? I don't fully understand the "pre-breadcrumb file" behavior.

It'll write new ones now. Should be seamless upgrade.

/azp run tools - test-proxy - tests

/apz run tools - test-proxy - tests

Is this going to have any conflicts when people upgrade locally? i.e. if the new proxy ignores the existing old breadcrumb file, will it try to overwrite stuff or use different paths? I don't fully understand the "pre-breadcrumb file" behavior.

It'll write new ones now. Should be seamless upgrade.

However, thinking about this over the afternoon, this upgrade will require an update of the transition script + documentation. My current plan is to

Get this tested and entirely green and merged.
In the same PR that I bump the version of the proxy in, I'll also update the migration script resources under eng/common/testproxy/transition-script/. We were also asking that I move it under onboarding as well. I may as well knock out all of that at the same time while I touch the script.

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

No commit pushedDate could be found for PR 6604 in repo Azure/azure-sdk-tools

What does this even mean devops bot!?

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests

/azp run tools - test-proxy - tests


Fix OSN leader election failure

Type of change

Bug fix

Description
Currently in fabric 1.4/2.x with raft consensus, when one OSN is deleted
from channel and rejoin later, it will be assigned a new raft id.
However, in some cases including that catchup is triggered or the OSN
is stopped during the re-configuration, the OSN still uses the old raft
id.
Other OSNs including the leader are using the latest view and believe
that the rejoined OSN will use a new raft id.  This will never be
recoved and may fail the leader election due to qurum lost. No new
transaction will be accepted, and no new configuration can be changed.
This patchset fixes the issue by halting the chain proactively and let
it obtain the correct raft id after tracking the latest config block.
Change-Id: I6e352b59e9feba85f2bbc0119b9eb47bce746adc
Signed-off-by: Baohua Yang<EMAIL_ADDRESS>Signed-off-by: Baohua Yang<EMAIL_ADDRESS>Signed-off-by: Jay Guo<EMAIL_ADDRESS>Additional details
Since this is a critical issue that will block the service, need to backport to v1.4 branch too.
Related issues
https://jira.hyperledger.org/browse/FAB-17875

Where is the integration test that reproduces the bug? My guess is that you reproduced this locally without giving the removed node the chance to detect its removal. Am I right?

yacov, currently, there is no integration testing covering such remove and rejoin scenario.
It can be reproduced easily. I do not know how to achieve what you said, because there is no API to let a node detect its removal. There is eviction suspicion, however, cannot block the issue's happening.

The code is taking what I think is the wrong approach. I don't think the chain should mutate its identifier during its runtime. When Raft detects it left the channel, it shuts its chain down, and begins monitoring for re-addition. Until that moment you should not remove it from the channel, as this would cause what you describe. Therefore, if you wish to re-introduce that orderer to the channel you need to wait until it recognizes it's not in the channel and only afterwards add it. This will ensure it will create the chain instance properly with the right ID, and the previous chain instance with the old ID would terminate prior to that.

That's only for very ideal cases.
However, fabric's code does not guarantee a "Raft detects it left the channel, it shuts its chain down, and begins monitoring for re-addition.". And users will not know how long they should "wait until it recognizes it's not in the channel and only afterwards add it.".
In this case, the problem will happen. If a problem can be triggered by users repeatedly,  it makes sense to consider to improve the software?

yacov, currently, there is no integration testing covering such remove and rejoin scenario.
It can be reproduced easily. I do not know how to achieve what you said, because there is no API to let a node detect its removal. There is eviction suspicion, however, cannot block the issue's happening.

We actually have tests that test that nodes detect their eviction, take a look here for an example. We also have tests that remove a node and then later on re-add it.
You claim there is a bug that can be reproduced easily, therefore we should make an integration test to ensure:

There is a way to actually make the bug manifest
The production code changes fix the bug.
Further production code changes will not re-introduce the bug.

However, fabric's code does not guarantee a "Raft detects it left the channel, it shuts its chain down, and begins monitoring for re-addition."

It does guarantee it, we have tests that test it.

And users will not know how long they should "wait until it recognizes it's not in the channel and only afterwards add it.".

If you want to improve the documentation, by all means please do so. We always welcome documentation improvements.

In this case, the problem will happen. If a problem can be triggered by users repeatedly, it makes sense to consider to improve the software?

Improving software is a constant trade-off between risk and benefit.
The Raft chain has a ton of state (fields) and maintaining their lifecycle is a delicate thing.
By making the chain reconfigure sensitive fields you introduce risk, which I think outweighs the benefit.

There is eviction suspicion, however, cannot block the issue's happening.

Eviction suspicion eventually confirms eviction or confirms non-eviction.

tests that test it

Hi Yacov
If I understand it correctly, your suggestion here is to add some integration testing?  If so, we can think about adding the testing case later.

tests that test it

Hi Yacov
If I understand it correctly, your suggestion here is to add some integration testing? If so, we can think about adding the testing case later.

Sure, you may do it at your leisure, no rush.
However, should this pull request be merged, it will have the integration test in it.

tests that test it

Hi Yacov
If I understand it correctly, your suggestion here is to add some integration testing? If so, we can think about adding the testing case later.

Sure, you may do it at your leisure, no rush.
However, should this pull request be merged, it will have the integration test in it.

OK, will do it later.

If I can summarize my impression of this bug and fix.
Bug: If an OSN is removed from the consenter set, and, because he is not the leader he does not get the block which triggers his eviction.  Then, the network progresses some amount, and, the same OSN is added back.  This OSN now begins receiving Raft messages again, and detects that the network has moved beyond the point that simply playing the log forward is possible, and instead of committing the blocks serially (and realizing its been evicted), instead does a state transfer, looks at the latest block and says "great, we were reconfigured, but I'm still in the membership set".  Now, the node is operating with the wrong Raft ID and therefore isn't really participating in consensus.
Question:  Does the OSN continue to commit blocks at this point? Does the OSN recompute its ID properly if restarted?
Fix: I do think that if this is a bug, and I think that it is, that it should be fixed.  But, I think I agree with @yacovm, it is dangerous to be tweaking internal state like node IDs dynamically.  Instead, we should ensure that the Raft component shuts down entirely, and is restarted with the correct ID.  @guoger what do you think?

If I can summarize my impression of this bug and fix.
Bug: If an OSN is removed from the consenter set, and, because he is not the leader he does not get the block which triggers his eviction. Then, the network progresses some amount, and, the same OSN is added back. This OSN now begins receiving Raft messages again, and detects that the network has moved beyond the point that simply playing the log forward is possible, and instead of committing the blocks serially (and realizing its been evicted), instead does a state transfer, looks at the latest block and says "great, we were reconfigured, but I'm still in the membership set". Now, the node is operating with the wrong Raft ID and therefore isn't really participating in consensus.
Question: Does the OSN continue to commit blocks at this point? Does the OSN recompute its ID properly if restarted?
Fix: I do think that if this is a bug, and I think that it is, that it should be fixed. But, I think I agree with @yacovm, it is dangerous to be tweaking internal state like node IDs dynamically. Instead, we should ensure that the Raft component shuts down entirely, and is restarted with the correct ID. @guoger what do you think?

@jyellick
Yes, that is a bug, and we've repeated it many times locally. I'm working on creating a ginkgo based testing case, but seems it lacks some API to trigger the catchup(). It will be helpful if you know how to do that.
I'm fine with the solution that we restart the chain, but as i said, it will take more time comparing with just updating the related configuration items. It is not that dangerous during this catchup period, because at that time, the OSN cannot be a leader, and will not receive new msg. Make sense?

I do think this is a bug (thanks for catching it!), however simply recomputing raft id on-the-fly does not solve the problem.
What happens here is that when re-added node starts with stale ID, it does not catch up with leader via normal raft replication, because its ID is not among active nodes tracked by leader. On the other hand, since it still recognizes leader by receiving heartbeats, suspicion thread and forced catchup is not triggered. Therefore, it gets stuck in limbo.
We could:
* make suspicion more aggressive (even if a node is not leaderless, we still perform `confirmSuspicion` periodically, AND

* halts the chain if it finds itself evicted by a config block during `catchUp` [here](https://github.com/hyperledger/fabric/blob/c799dcc188d76c5d3f3e2cdab931f30a00273d17/orderer/consensus/etcdraft/chain.go#L900)

I have a working prototype locally and l'll demo it to you @yeasy

If the leader knows the new ID of the node and it sends it heartbeats, then we can look at the ID of the node that leader tells us and if it's not our current ID then we can halt the chain and then forcely synchronize ourselves and restart the chain again somehow... @guoger  wdyt?
	if stepMsg.Type == raftpb.MsgHeartbeat {
		if c.raftID != stepMsg.To {
			c.HandleExoticBug()
		}
	}

I do think this is a bug (thanks for catching it!), however simply recomputing raft id on-the-fly does not solve the problem.
What happens here is that when re-added node starts with stale ID, it does not catch up with leader via normal raft replication, because its ID is not among active nodes tracked by leader. On the other hand, since it still recognizes leader by receiving heartbeats, suspicion thread and forced catchup is not triggered. Therefore, it gets stuck in limbo.
We could:
* make suspicion more aggressive (even if a node is not leaderless, we still perform `confirmSuspicion` periodically, AND

* halts the chain if it finds itself evicted by a config block during `catchUp` [here](https://github.com/hyperledger/fabric/blob/c799dcc188d76c5d3f3e2cdab931f30a00273d17/orderer/consensus/etcdraft/chain.go#L900)

I have a working prototype locally and l'll demo it to you @yeasy

If the leader knows the new ID of the node and it sends it heartbeats, then we can look at the ID of the node that leader tells us and if it's not our current ID then we can halt the chain and then forcely synchronize ourselves and restart the chain again somehow... @guoger wdyt?
	if stepMsg.Type == raftpb.MsgHeartbeat {
		if c.raftID != stepMsg.To {
			c.HandleExoticBug()
		}
	}

a slightly different variation i tried was adding this to chain.Consensus:
	if stepMsg.To != c.raftID {
		c.logger.Errorf("Received msg to %d, my ID is probably wrong, cowardly halting", stepMsg.To)
		c.Halt()
		return nil
	}

however, i was concerned that halting the chain is too aggressive a reaction to such a message. Admittedly the code is much simpler, we will need a lengthy comment to explain the reason.
Anyway, i'm ok with either

however, i was concerned that halting the chain is too aggressive a reaction to such a message

In Raft, we assume the leader is honest and correct. If this assumption breaks, Raft breaks apart.

make suspicion more aggressive (even if a node is not leaderless, we still perform confirmSuspicion periodically, AND halts the chain if it finds itself evicted by a config block during catchUp here

I'm not OK with doing eviction suspicion periodically, when everything is fine, just to fix a scenario that occurs due to incorrect use of Raft.
This scenario is avoided if the user doesn't do something against his best interest.
We should not change the normal common case just to cater to a rare failure case.
I suggest we proceed by halting the chain if we got a heartbeat with a To that is different than our identifier, and make an integration test that reproduces this as part of the change set.

Sounds good to me. @yeasy here's a quick draft of what i had in mind. If you'd like, you could start from there and work out a proper PR.

here

Sure, let me have a look and work with you on an appropriate patchset.

@yeasy per our offline discussion, we've agreed that rejecting step message with incorrect node id is a better choice. Also, your concern around repeatedly adding/removing a node while the node itself is behind, is not a problem. It would pull the latest config and extract correct node id to start with. It can be easily proved by adding such operations to the IT i demonstrated (which should be added i think).
looking forward to your PR update

you'll need to add id to the raft message at this line

And you need to add To section to this line too, for the same reason above

/ci-run

I got you

/ci-run

/azp run

there's a bug in current validation logic. I will submit a fix and let you know @yeasy
https://jira.hyperledger.org/browse/FAB-17951

@yeasy #1367 is now merged, i believe you could rebase this


Bar Chart Goals Not working in 1.0.10

Creating a new bar chart with a data array containing {x: 'Name', y: Value, goals: [{...}]} appears to have stopped working in 1.0.10.  You will just see a chart with an x label of "[object Object]".  While troubleshooting, I noticed that when rendering the chart, the alphabetical order of the objects in the data array seems to matter (i.e. - it will try to interpret "goals" as "x").  If I rename "x" to "a" and "y" to "b", the bar will show up correctly again, but no matter what I do, I cannot get the goal to appear.  Reverting to 1.0.9 fixed the issue.  Of course I didn't export a backup of the view before reverting to 1.0.9...that would have been smart!  If you need more info to diagnose, let me know.  Thanks!
P.S. - Love the module!

That did the trick!  Thanks for the quick fix.


False positive suggestions

I read through the entire list from start to finish, to look for false positives.

Thanks! Fixed in d859e9918482017dbc4932a9f89cf125c492a20e. I'm sure there are more false positives — it's a long list, and mostly based on other lists from around the internet — so please feel free to open a new issue or a pull request if any are causing issues.

Something I recently found was "dp" and "wn". WN apparently means white nationalists, DP I'm unsure of

@Lilja see the second page. But I removed that one just now, because it's pretty vague. Also removed WN, because it's also ambiguous.


[BugFix] Fix does not supporting s3 path with spaces

Fix not supporting s3 path with spaces (BRPC's URI does not allow path contains spaces, so we change to use AWS CRT URI)
Rename the ut name, so that we can run it by default. otherwise, we have to run the below command ./run-be-ut.sh --with-aws;

What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5


Update terragrunt.hcl

Ran Plan for dir: . workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/atlantis-data/repos/italorafaeltavares/Atlantis/20/default/default.tfplan\"" in "/atlantis-data/repos/italorafaeltavares/Atlantis/20/default": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

atlantis plan

Ran Plan for dir: . workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/atlantis-data/repos/italorafaeltavares/Atlantis/20/default/default.tfplan\"" in "/atlantis-data/repos/italorafaeltavares/Atlantis/20/default": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

atlantis plan

Ran Plan for dir: . workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/atlantis-data/repos/italorafaeltavares/Atlantis/20/default/default.tfplan\"" in "/atlantis-data/repos/italorafaeltavares/Atlantis/20/default": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

Ran Plan for dir: . workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/atlantis-data/repos/italorafaeltavares/Atlantis/20/default/default.tfplan\"" in "/atlantis-data/repos/italorafaeltavares/Atlantis/20/default": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵

atlantis plan

Ran Plan for dir: . workspace: default
Plan Error
running "/usr/local/bin/terraform plan -input=false -refresh -out \"/atlantis-data/repos/italorafaeltavares/Atlantis/20/default/default.tfplan\"" in "/atlantis-data/repos/italorafaeltavares/Atlantis/20/default": exit status 1
╷
│ Error: No configuration files
│
│ Plan requires configuration to be present. Planning without a configuration
│ would mark everything for destruction, which is normally not what is
│ desired. If you would like to destroy everything, run plan with the
│ -destroy option. Otherwise, create a Terraform configuration file (.tf
│ file) and try again.
╵


Invalid types in 0.20.1

Environment
Knex version: 0.20.1
Database + version: Postgress
OS: Macos
Bug
Migrating to 0.20.1 causing type error:
Type '"then" | "catch" | "finally"' does not satisfy the constraint '"then" | "catch"'.   Type '"finally"' is not assignable to type '"then" | "catch"'.
https://app.circleci.com/jobs/github/aerogear/graphback/1295

Upon investigation. Types from 0.20.1 will clash if the project uses @types/knex.
Removing them will resolve problem.

Still happening

That page is not found... sounds like you have invalid typings somewhere? For example if you are passing QueryBuilder's around pretending them to be promises.
Can be opened if reproduction provided.

@wtrocki Is this happening only in Node 8 ?

@lorefnon Using node 10

This is also likely due to the TypeScript target or lib array in your tsconfig. You'll need to target es2018 or newer, or include es2018.promise as a lib. Promise.prototype.finally is an ES2018 feature that only Node 10+ supports.

This should get fixed (by https://github.com/knex/knex/commit/af129408ca7173bfa0f4a777fb5d3206339cd634) once master is released.

@citycide Thank you so much! This was the issue. I have cleaned up types but only when I have added promise lib it started working.


Can't build with zig <IP_ADDRESS>85

There are couple of errors
First is easy, at https://github.com/thimenesup/GodotZigBindings/blob/master/binding_generator.zig#L282
changed to something like
const class_is_instanciable = if (class.get("instanciable").?.Bool) "true" else "false";
const class_is_singleton = if (class.get("singleton").?.Bool) "true" else "false";
Second, in https://github.com/thimenesup/GodotZigBindings/blob/master/src/core/basis.zig#L270
There are two places where compiler complains about space between c -s
The third one is hard to me to understand
src\core\gdnative_api_types.zig:173:5: error: extern structs cannot contain fields of type '?fn(?*const anyopaque, [*c]const u8, core.gdnative_api_types.godot_gdnative_api_version, core.gdnative_api_types.godot_gdnative_api_version) callconv(.C) void'
report_version_mismatch: ?fn (?*const godot_object, [*c]const u8, godot_gdnative_api_version, godot_gdnative_api_version) callconv(.C) void,
src\core\gdnative_api_types.zig:173:5: note: only pointer like optionals are extern compatible
src\core\gdnative_api_types.zig:277:5: error: extern structs cannot contain fields of type '?fn(?*anyopaque, ?*anyopaque) callconv(.C) ?*anyopaque'
create_func: ?fn (?*godot_object, ?*anyopaque) callconv(.C) ?*anyopaque,
src\core\gdnative_api_types.zig:277:5: note: only pointer like optionals are extern compatible
src\core\gdnative_api.zig:861:5: error: extern structs cannot contain fields of type '?fn(?*anyopaque, [*c]const u8, [*c]const u8, c_int, [*c]const core.gdnative_api_types.godot_method_arg) callconv(.C) void'
godot_nativescript_set_method_argument_information: ?fn (?*anyopaque, [*c]const u8, [*c]const u8, c_int, [*c]const gd.godot_method_arg) callconv(.C) void,
src\core\gdnative_api.zig:861:5: note: only pointer like optionals are extern compatible
Note that I can build it with older zig (<IP_ADDRESS>94 for example), but can't make it work on Win64. DLL loads in Godot, prints debug messages from godot_nativescript_init(), but no actual TestNode2D methods are registered, all calls fails and even nativeScript.get_script_method_list() not listing them.

I would wait until it 0.10 gets a stable release before actually trying to fix anything.

I see, it wasn't mentioned in readme so I assumed it was close to master branch. With 0.9.1 it indeed working on windows. However it seems Zig (due to LLVM bug) unable to build .so for android yet, so I'm exploring totally another way now, with zig build -ofmt=c emitting C-code which later will be fed into usual GDNative or Godot module pipeline. Thanks for your project for some insights, though.


Cleanup: added note about oc version

addressed comment in #175

@SlavoKrupa mind taking a look?

👍 Great! Thanks

I have just realized that there is also interaction with OpenShift Online via client. ( AFAIK this covers only oc from minishift ) But users do not have to use whole minishift if they want just try cloud integration. They would only need oc binary which is also available at:
https://github.com/openshift/origin/releases/tag/v1.4.1

I have just realized that there is also interaction with OpenShift Online via client. ( AFAIK this covers only oc from minishift ) But users do not have to use whole minishift if they want just try cloud integration. They would only need oc binary which is also available at:
https://github.com/openshift/origin/releases/tag/v1.4.1

@SlavoKrupa do you want me to hold this PR? or can i merge it?

@SlavoKrupa please file a separate issue for the oc client outside of minishift. merging.

merged: https://github.com/openshiftio/appdev-documentation/commit/9c4139d9b57e414debbf52194a745dfa8f79cc72


[BUG] Popup errors out if opened the second time with the same declaration

In regards to this discussion
Code used
M.pop = function()
    local Popup = require("nui.popup")

    local popup = Popup({
        border = {
            style = "rounded",
            highlight = "FloatBorder",
        },
        position = "50%",
        size = {
            width = "80%",
            height = "60%",
        },
    })

    popup:mount()
end

Result:

Same happens to me. The styles table gets messed up after the first use. I was able to fix it by passing a copy of the border style table instead of the original table in lua/nui/popup/border.lua:L251 like so:
-- put this anywhere in the file
function table.shallow_copy(t)
  local t2 = {}
  for k,v in pairs(t) do
    t2[k] = v
  end
  return t2
end

-- change L251 to
    props.char = table.shallow_copy(styles[style])

Hey @srafi1, yes, you're absolutely right! We're messing up the styles table after first usage.
You can use the vim.deepcopy instead of manually creating table.shallow_copy by the way.
Do you want to open a PR to fix this? or should I go ahead and do it? 😃

Didn't know about vim.deepcopy. Just submitted a PR. 🚀


Slack Bot Not Updating Message

The Slack Bot I have configured, it is not updating (chat.update) the original message. It is posting a new message for each change in my pipeline. Your gif shows that it should be updating the original message as the steps are being executed in the pipeline.
Please advise.

Hey Connor, Did you add the permission scope to your bot?

It is required to add the permission scope 'Access user’s public channels' (channels:history).

Yes, and it posting to the channel. Just not updating the messages. I should mention this is a private Slack channel (if that makes a difference).

So that's what it was... it works if I use a PUBLIC slack channel.

However, when I click on Build Logs it takes me to a null logGroup and null logStream.

Well, I do not know if this suits to this issue, but the final result is far away from the result shown at build.gif on readme.md 🙁
Result:

Gif:

@rayjanoka or @connor-tyndall
I can't get the bot to post anything at all. I have the cloudformation stack deployed with the master branch of repo instead of the outdated deployment link. I also added in:
  SLACK_CHANNEL: !Ref SlackChannel
I have both tokens set but no messaging ever appears. Curious if you could point me in the right direction to test the lambda


Packer v2 http.FileServer(box) failed to serve index.html with go v1.12.4 and packr Version 2.2.0?

Hello,
index.html create a redirect loop then fail with url like "http://localhost:3000//"
(All other static file work as expected);
OS MacOs v10.14.4
Go version 1.12.4
Packr2 V2.2.0
test program :
package main

import (
  "fmt"
  "net/http"

  packr "github.com/gobuffalo/packr/v2"
)

func main() {
  box := packr.New("my template", "../mysite/templates")

  http.Handle("/", http.FileServer(box))
  fmt.Println("listening on :3000")
  http.ListenAndServe(":3000", nil)
}

Hello, same thing happened with me.
Using:
go.12
packr2 v2.2.0
Linux Mint 18.2 Sonya

Also observing this problem
Windows 10
go 1.12.4
packr2 v2.2.0
Was using statik and look forward to switching to this solution

Hi @gpkfr @marcotuna @StevenWeathers
I also ran into this issue, I tried to make a fix if you want to have a look : #201

Hi @gpkfr @marcotuna @StevenWeathers
I also ran into this issue, I tried to make a fix if you want to have a look : #201

yes, it seem to resolve this issue. Thanks for your time ;)

I have the same issue, inside a Buffalo app.
When I put my "index.html" file inside the "public" folder, I get redirected to an invalid location. At the same time, requesting other files (or skipping packr and loading from the filesystem directly) works.
curl -v http://localhost:3001/
*   Trying ::1...
* TCP_NODELAY set
* Connected to localhost (::1) port 3001 (#0)
> GET / HTTP/1.1
> Host: localhost:3001
> User-Agent: curl/7.54.0
> Accept: */*
>
< HTTP/1.1 301 Moved Permanently
< Cache-Control: max-age=31536000
< Etag: 159eb8d6c587bfa0
< Location: ..//
< Vary: Origin
< Date: Wed, 15 May 2019 02:02:33 GMT
< Content-Length: 0
<
* Connection #0 to host localhost left intact

See #173

@sagikazarmark I looked at the issue but not sure what to make out of that?

As far as I can tell it's the same issue. I posted my findings and a possible workaround there, but it turned out later there is already a PR for the issue.

Has there been any updates to this issue? I am also running into this same issue using Go v1.12.4 and Packr v2.3.2 and run into the same redirect loop with a URL like http://localhost:3000//.
I am using the same script as is directed in the v2 README for serving static web files from a binary:
package main

import (
	"net/http"

	"github.com/gobuffalo/packr/v2"
)

func main() {
	box := packr.New("someBoxName", "./templates")

	http.Handle("/", http.FileServer(box))
	http.ListenAndServe(":3000", nil)
}

I also noticed the PR from @nlepage above, I will test this to see if it fixes my issues - has there been any updates to its status?

Hello there,
@markbates : Is there any reason why did you revert #217 ? Because #201 fix the issue

Yes. As I said the tests keep failing. Another PR needs to be opened to resolve the failing tests.

I have this little workaround: https://gist.github.com/mhewedy/65895bced03e75167f2c78cc08994cb8


Regex Support Fixes

Did

Fixes to code style
Change readdir to readdirSync, as readdir is asynchronous.
Uncommented single param RexExp test.
Added test for mixed content.

Thanks :D


A way to read inside content groups?

Is there a way to get this tool to dig a little bit deeper into content groups?
Compared to my own manual VoiceOver testing, this tool seems to give me different more "high level" output. It seems like it skips over the page content and just reads out the higher level "Content - group". Is there a way to get it to read into groups?
Here's this site I'm testing: https://kamadojoe.com
And this is the output I'm seeing:
Welcome to macOS. VoiceOver is on. Playwright 🎭 Playwright: about:blank window web content has keyboard focus
Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill
Links menu
Headings menu
Form Controls menu
Window Spots menu
Loading more items…
No items in Window Spots menu
Content - group
Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill
In Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill KamadoJoe's Instagram feed
Session ended

Maybe this is how the tool is expected to behave. Was trying to see if this cool could speed up my workflow of checking what VoiceOver says for content in the page.

@Christopher-Hayes thanks for the message, you can use the VoiceOver.js standalone JS library for this kind of thing. You'd instantiate a new VoiceOver object and use the startInteracting command. let me know if there's anything missing in the docs that I can help with.

Thanks @ckundo, that does work pretty nice.
Having a little trouble with startInteracting, it seems to just stop after a few items. But, the repo test did help me get to where I was going. The somewhat hacky node code below does the job of iterating through the page printing everything out. I have it advancing in search of text that appears at the very bottom of the page.
/usr/bin/open -a /Applications/Google\ Chrome.app https://kamadojoe.com && node index.js

import { VoiceOver } from '@accesslint/voiceover';

const voiceOver = new VoiceOver({ log: true })
await voiceOver.launch()
await new Promise(resolve => setTimeout(resolve, 3000))

try {
  await voiceOver.advance({
    target: {
      text: 'Cookies'
    },
    steps: 100,
  })
} catch (e) {
  console.log(e)
} finally {
  await voiceOver.quit()
}

Btw, in terms of the docs. It might help to show code using the commands. It took me a a couple minutes to figure out I had to do the following to use the command.
import { voiceOver, startInteracting } from 'voiceOver';

voiceOver.execute(startInteracting)

@Christopher-Hayes thanks for your thoughts on this.

@Christopher-Hayes oh, I also just realized auto-vo is working on your page, but you need to specific both the until and the limit flags.
% npx @accesslint/auto-vo --url https://www.kamadojoe.com/ --until Cookies --limit 100
Welcome to macOS. VoiceOver is on. Playwright 🎭 Playwright: about:blank window toolbar
Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill web content
90% loaded.
and 1 more item web dialog button
Web Spots menu
No items in Web Spots menu
Form Controls menu
Window Spots menu
Content - group
Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill web content
In Kamado Joe® | Ceramic Grills | Ceramic Smoker | Kamado Grill web content KamadoJoe's Instagram feed frame
button
SHOP OUR FATHER'S DAY DEALS
6/3 - 6/11
FREE Cover, Charcoal & Fire Starter with a Series II or III Grill Purchase button
30% Off Charcoal Baskets button
FREE Probe Pack with the Purchase of an iKamand button
*Offers valid only for in-stock items while supplies last for U.S. Contiguous Customers only.

Thanks @ckundo that looks perfect. I'm having trouble running npx @accesslint/auto-vo, NPM is saying the package is deprecated. https://www.npmjs.com/package/auto-vo

make sure you include the @accesslint/ scope.

I did, NPM is erroring that it was not found: https://registry.npmjs.org/@accesslint/auto-vo

ah, i published to the GitHub registry thinking it'd propagate... will look into it! in the meantime, what happens if you npm install -g @accesslint/auto-vo and then run auto-vo?

The CLI command seems to work fine with just auto-vo, so I can use that, thanks.


Remove package-lock.json

Pre-Flight checklist

[x] Testing

[x] Code is tested
[x] Changes have been manually tested

What are the relevant tickets?
N/A
What's this PR do?
Removes package-lock.json which is breaking
How should this be manually tested?
Tests should pass

👍

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@bc50d98). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #10   +/-   ##
=======================================
  Coverage        ?   97.58%
=======================================
  Files           ?      115
  Lines           ?     2738
  Branches        ?        0
=======================================
  Hits            ?     2672
  Misses          ?       66
  Partials        ?        0

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 bc50d98...2206fd0. Read the comment docs.


Merging dev_doc into main

Codecov Report

Merging #87 (e751a24) into master (c5840cd) will not change coverage.
The diff coverage is n/a.


汇总一下有显示问题的页面

https://www.dqsj.cc/index.php/vod/play/id/223/sid/1/nid/1.html

https://linuxhint.com/install-go-ubuntu-2/#1

shadow-root is used to fix this issue


New Puzzle

New Puzzle

[x] If there's not puzzle in progress, generate new puzzle and display it.

[ ] If there's a puzzle in progress, warn user they will lose progress if the current puzzle isn't saved.

Primary: Austin

Secondary: Tess

Size: M

Actual Size: (e.g. t-shirt size after completion)

Actual Time: (in minutes or hours -- after completion)

Need to check for if theres a puzzle in progress

Need to check if a puzzle is in progress, ask about saving
Need to wipe user guessed words (in current session) whether or not game is saved.


Stop react-player to load HLS video on unmounted without killing the player instance.

Current Behavior
I am using HLS video stream for the react-player, when the player is unmounted it still keeps loading the stream ( .m3u8). I am testing the loading state in BrowserStack with iOS 15 and 16 beta. However, the player media does not load when unmounted on Android 11 and 12.
Expected Behavior
No loading of video in the network tab of DevTools
Environment

URL attempting to play: https://cph-p2p-msl.akamaized.net/hls/live/2000341/test/master.m3u8

Device: iPhone 12-13 and Samsung Galaxy S22 (BrowserStack)

Operating system: iOS (15, 15.4, 15.5, 15.6, 16(beta)) and Android (10, 11, 12)

code example: https://codesandbox.io/s/stoic-meadow-ekh824?file=/src/App.js

Other Information
PS: I want to avoid killing the player instance or use useRef hook. I am looking for more elegant way with HLS configurations

Unless you are passing config={{ forceHLS: true }}, HLS streams on iOS use the native HLS support of the device and do not use hls.js. I’m not sure what we can do about this as it’s entirely up to iOS to continue loading data after the <video> element has been removed.


mermaid does not work with images

"diagram of chatcraft.org(include favicon https://chatcraft.org/favicon-32x32.png) being hosted on cloudflare pages and talking to gpt api via langchain"

I don't think this is a bug we can fix.  Re-open if something new happens with upstream Mermaid support.


Added dataview to explanations.md

Contributor Checklist

[ ] Did you add Scaladoc to every public function/method?
[ ] Did you add at least one test demonstrating the PR?
[ ] Did you delete any extraneous printlns/debugging code?
[x] Did you specify the type of improvement?
[x] Did you add appropriate documentation in docs/src?
[x] Did you state the API impact?
[x] Did you specify the code generation impact?
[x] Did you request a desired merge strategy?
[x] Did you add text to be included in the Release Notes for this change?

Type of Improvement
Documentation

API Impact
No impact

Backend Code Generation Impact

No
Desired Merge Strategy
Squash

Release Notes
Added dataview to explanation.md

Reviewer Checklist (only modified by reviewer)

[x] Did you add the appropriate labels?
[x] Did you mark the proper milestone (Bug fix: 3.3.x, [small] API extension: 3.4.x, API modification or big change: 3.5.0)?
[x] Did you review?
[x] Did you check whether all relevant Contributor checkboxes have been checked?
[x] Did you mark as Please Merge?

Hello @mwachs5 this issue is still not solved, dataview explanation is still absent.

Oh okay I understand you.

It'd be great if we could host multiple versions at a time on the website (and I've filed tracking issue: https://github.com/freechipsproject/www.chisel-lang.org/issues/203), but this is quite a bit of work to support.


Is an update possible?

Hi,
I just got a stove working on easy Connect. Your repository is not accessible for version 2024.5.x. Are you considering an update?
Best regards,

Hi @fabienlmbt. There should be no issues using the integration with HA 2024.5 (also using it myself). Why do you state it is not accessible?

When I search for "micronova" on HACS, I get no results. and when I try to add the deposit, I also get an error.

It would help if you also post the error here.

When I look for it, there's nothing

@fabienlmbt You don't have to manually install the repo into HACS because it is already part of the default set. In HACS go to Integrations, click the + button and search for Micronova.


Disposables, url fragment and tweaks

Changes:

Disposables
Added the IDisposable interface to PageRequester, Scheduler, WebContentExtractor and WebCrawler. This is useful when you have custom modules that needs to be disposed correctly!

Url fragment
Added  new parameter to the HyperlinkParser to remove or not the url fragment on each link ie everything after the "#" in the url. This is useful when a website keeps relevant data here, for instance single-page apps do this. I've kept the default behaviour of removing it!

WebContentExtractor
I've changed some methods in the WebContentExtractor to make them protected and overridable to be able to make a custom content extractor while still reusing the same GetCharsetFromBody and GetEncoding methods. GetCharsetFromBody now takes an input string because our custom WebContentExtractor downloads a string directly (no stream), and it does not affect the behaviour.

Root Uri validation
If you try to crawl with a root uri that gets redirected on the first call, nothing will be crawled. For instance, "http://dotnetperls.com/" redirects to "http://www.dotnetperls.com/". The redirection works but the redirected page is now considered external since it is on another authority than the starting address. To fix this we validate the root page, checking if the page was redirected and changing the RootUri if it was. Then, we consider links to "dotnetperls.com" and "www.dotnetperls.com" to be internal (since the website can have mixed links too).

All of this was thoroughly tested in our environment. If you have any questions just let me know!

Thanks for your contribution. Very clean changes and I appreciate the added unit tests.

See my last push https://github.com/sjdirect/abot/commit/eb719d8476d8c66edab78779bc1deffd86747b29. A few changes to what you had. Added the removeUrlFragment (now called isRespectUrlNamedAnchorOrHashbangEnabled) to the config file. Also added a few more unit tests.

@sjdirect I post a note on Abot/Core/HyperLinkParser.cs. with an another way  to remove Url fragment.

Thanks LeMoussel. I replied inline to that note.


Issue with Pug template v2.0.0 - never read warning

Seem likes the new version has not been compatible with template using pug yet?
I'm using the components and methods on template but it's still showing unused warnings.

I thought I was going crazy from running into the same issue after having set up my typescript/linting within our pug templates last week and having no issue. Unfortunate timing I guess.
Yeah, seems like it's a language tools v2.0 issue on composition API components, whether lang="ts" is on or not

Option clicking on references in the template also no longer works, could be related.

Option clicking on references in the template also no longer works, could be related.

Yeah likely the same issue, same for hovering over script variables in the template which would usually give type info

Same issue

2.0.5 does not resolve the issue with the never read warning for me, even after re-installing the extension and rebooting vscode.
Has anyone managed to get it to work for them?
I have this in my tsconfig.json:
"vueCompilerOptions": {
    "plugins": [
      "@vue/language-plugin-pug"
    ]
  }

Have you made sure to be on the latest version of @vue/language-plugin-pug? That fixed the problem for me.
npm i -D @vue/language-plugin-pug@latest

Yep that worked, thank you so much :tada:

Updated to @vue/language-plugin-pug 2.0.21  but still occur..


Current model size is too big. Discontinuing training this model to search for other models.

Hi. I'm trying to apply autoKeras on own datasets. The dataset size is (1380, 299, 299, 3), but it's not working with "Current model size is too big. Discontinuing training this model to search for other models." error.
So, I resized images almost half, (1380, 128, 128, 3) and then tried again.
However, it was working initially, but the moment of training 9th model, the same error again.
What's the problem? What's the limitation of model size?

I had the same problem.
This error is "out of memory problem."
The searcher creates a very complex model, and don´t get to put to execute on GPU.
I didn't find a solution. If you find, share with us.

@dhlee-jubilee @rafaelmarconiramos
This is a expected behavior. It should not affect the search.
Thanks.

@jhfjhfj1 Could you elaborate?
Your statement implies "this error is generated by autokeras", which is obvious. Your statement does not imply that this error is good or bad.
Does this mean that the model is not finding a good search criterion and that we should stop the algorithm / modify our data?
Or does it mean that the algorithm is working as expected and we should continue.
thank you

@jhfjhfj1  got the same problem as @exowanderer !

Could someone suggest a way to implement a network size constraint in the tree search procedure ?
autokeras/search.py
Additional condition before adding to training_queue may suffice ?
  if not self.training_queue:
                searched = True

                while new_father_id is None:
                    remaining_time = timeout - (time.time() - start_time)
                    new_graph, new_father_id = self.bo.optimize_acq(self.search_tree.adj_list.keys(),
                                                                    self.descriptors,
                                                                    remaining_time)
                new_model_id = self.model_count
                self.model_count += 1
                self.training_queue.append((new_graph, new_father_id, new_model_id))
                self.descriptors.append(new_graph.extract_descriptor())

I am facing the same problem and it seems to be an infinite loop. In 'search.py', we have the following:
        except RuntimeError as e:
            if not re.search('out of memory', str(e)):
                raise e
            if self.verbose:
                print('\nCurrent model size is too big. Discontinuing training this model to search for other models.')
            Constant.MAX_MODEL_SIZE = graph.size() - 1
            print(Constant.MAX_MODEL_SIZE) # Not in original file.
            return

As shown above, I have added a print(Constant.MAX_MODEL_SIZE), which showed me that Constant.MAX_MODEL_SIZE stays constant (because graph.size() does not change) after each iteration of Current model size is too big. Discontinuing training this model to search for other models..
I'm unsure whether it's a mistake in the code or on my part, but it seems that Constant.MAX_MODEL_SIZE should be reduced everytime there was not enough memory to load a model, for example by having Constant.MAX_MODEL_SIZE = max(1, round(Constant.MAX_MODEL_SIZE/2)) instead of Constant.MAX_MODEL_SIZE = graph.size() - 1.

Constant.MAX_MODEL_SIZE = max(1, round(Constant.MAX_MODEL_SIZE/2))

I had this same problem, it seems graph.size() should be reduced under Constant.MAX_MODEL_SIZE, but it didn't.

very interesting

https://www.simonwenkel.com/2018/09/05/autokeras-german-traffic-sign-recognition-benchmark.html
maybe this can help


Image worker fails on Tizen TV

We cannot use the image worker on Tizen because Vite cannot transpile this piece of code, as it is a string.
https://github.com/lightning-js/renderer/blob/28b65e2a4eb1f2f5998e59374b7c44764bad86d0/src/core/lib/ImageWorker.ts#L61-L90

Great catch! Should be fairly easy to update to something that is ES5 friendly.

Careful with this one, if we select chrome >49 we'd only have fetch API. We can either move this to a seperate file and have babel deal with it. Or rewrite this in an super small inline ES5 based image worker.
async --> chrome >52
fetch --> chrome >40
?? operator --> chrome >80
arrow functions --> chrome >40
I wouldnt run this through Babel, because then you get 100s of lines of code to provide polyfills for everything and anything you write.
Might just be better to rewrite this in a plain & simple ES5 using simple functions, vars, callbacks and XMLHttpRequest.


chore(Topology.Instances.EReal): simplify proofs

And remove a couple of unnecessary hypotheses along the way.

bors r+
Thanks!


Release KubeOne 1.6.3

Scheduled to happen: Thursday, 2023-09-07
Before Release

[x] Write and publish the changelog (#2887, #2888, #2892, #2893)

Release

[x] Create and push a new tag
[x] Ensure that the release job has succeeded
(watch https://public-prow.loodse.com/?job=post-kubeone-release)

After Release

[x] Update the release's description on the GitHub Releases page to replace
the automatically generated description with the changelog
[x] Download the binaries from GitHub and make sure that:

[x] Checksums are matching
[x] kubeone version returns the expected version

Action Items

/assign

The release is out! 🎉
/close


Update title to distinguish traditional Azure CNI from other Azure CNIs

Proposal:
"Azure CNI" -> "Traditional Azure CNI"/ "Azure CNI Legacy", etc.
Supporting points:
It is confusing for users that there are so many different CNIs. As they all have "Azure CNI" title, it will make users believe that other "Azure CNIs" (e.g Podsubnet, CIDR static subnet, overlay) as same as "Azure CNI", even they work differently.
Hence, I believe it will be better to append the word like "legacy", "traditional" to the "Azure CNI".

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

ID: 37ba0343-7e43-6e2e-e564-5d3a0f523a75
Version Independent ID: 0ba581a1-3e4a-27b4-c667-4a9b969a3ade
Content: Configure Azure CNI networking in Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/configure-azure-cni.md
Service: azure-kubernetes-service
Sub-service: aks-networking
GitHub Login: @asudbring
Microsoft Alias: allensu

@JoeyC-Dev
Thanks for your feedback! We will investigate and update as appropriate.

@JoeyC-Dev
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

Thanks for the input! This will be part of a larger effort to clarify the usage of Azure CNI in the docs but will have additional considerations than simply adding "traditional" or "legacy".

#please-close


Specifying RestrictChildren and empty HtmlTargetElement results in an error

[RestrictChildren("div")]
[HtmlTargetElement]
public class MyTagHelper : TagHelper
{

}

    <my>

    </my>

Expected: This should render MyTagHelper correctly
Actual: The <my> tag is not allowed by parent
 tag helper. Only child tags with name(s) 'div' are allowed.

By design. By specifying [HtmlTargetElement] you're saying that the TagHelper applies to every tag.

We need to update the docs on that attribute to indicate it's a catch all tag helper.

The doc comments for HtmlTargetElementAttribute could be improved. The default constructor summary should specify what * means - you shouldn't have to read the remarks.
Also, the error message doesn't specify clearly which tag helper is causing the issue. What if you have multiple tag helpers targeting
?

1c7af0d


(IDEA) Show both speakers in the details of the speaking slot

description
So far we have both speakers shown on the agenda but only one in the "info"/details. It would be great if we could standardize it.

reason
We have more companies willing to take a partner/customer with them to present together. It is a great opportunity for us (more practical knowledge + more recognizable brads to show) it would be great if we could show it.
list of benefits
It is a great opportunity for us - more practical knowledge + more recognizable brands to show. It would be great if we could show it.
priority
6
team members who discussed the idea
MT & JP

good news @jprzybylski4
agenda widget has been totally rewritten (final version is visible on teh website)

but i cannot tell the exact date when the updated library will be pushed to ebe website (there are some major differences that must me hidden first...like "recent bookings")

@jprzybylski4 live on ebe


Add install/uninstall buttons to clusters in the cluster list

Summary
Fixes #367
Occurred changes and/or fixed issues

Enhance the initial experience of the ui-extension by guiding the user to install epinio on clusters that we can't find it on

for clusters which we've found epinio on... and they can... show uninstall action
for cluster with no epinio on... and they can... show install button

Technical notes summary

In order to do this we now show all clusters in the list, instead of just those that have epinio installed
Blocked on #364

Areas or cases that should be tested

install action leads user to the cluster's epinio chart detail page
uninstall action leads users to cluster's helm app page with the query string "epinio"
when there are no epinio instances detected we show a nice intro guiding to install action

Screenshot/Video

The functionality looks good overall, It will require another round of review after #364 is merged, rebase and code clean-up.


AppB is able to modify store in AppA that doesn't have registered global actions

Describe the bug
Hi everyone,
I'm playing arround with this library and I found out that I'm able to update state from AppA -> AppB even if AppB doesn't have registered global actions.
AppA:
            this.globalStore = GlobalStore.Get(false);
            this.globalStore.CreateStore('MFAppStore', MfReducer, []);
            this.globalStore.RegisterGlobalActions('MFAppStore', null);

AppB:
                this.globalStore = GlobalStore.Get(false);
                this.globalStore.CreateStore('GlobalAppStore', GlobalReducer, []);
                this.globalStore.RegisterGlobalActions('GlobalAppStore', [INCREMENT_GLOBAL, DECREMENT_GLOBAL]);

I'm able to dispatch action from AppB to AppA and modify the 'MFAppStore' even if there are not registered global actions.
In documentation ( readme ) it says:
this.globalStore.DispatchAction("App1", action); // This will dispatch the action to current app's store as well other stores who might have registered 'Action-1' as a global action
but it seems that it doesn't work as it is intended or I'm missing something?
Or it was always allowed?
Thanks

Ideally this shouldn't happen. Can you please give me an example of which action is modifying MFAppStore, and how are you dispatching it.

AppB is able to modify store in AppA that doesn't have registered global actions. you just have to do the dispatchAction('APPA',{type:'INCREMENT'}) in appb then it will update the store of AppA.


please translate everything in english

Describe the problem related to the feature request
because we dont know chinese
Describe the solution you'd like
and google translator cant translate
Describe alternatives you've considered
idk why
Additional context
No response

There is no plan to support this feature.
Just because requires effort to maintain i18n

@Harry-zklcdc this is better with your space check speed of bing https://niansuh-bingo.hf.space with bing api
Thanks to @wegaic

@Harry-zklcdc this is better with your space check speed of bing https://niansuh-bingo.hf.space with bing api Thanks to @wegaic

Because I control the speed of output. If you use BingAPI, it will faster than web view.
https://github.com/Harry-zklcdc/go-proxy-bingai/assets/21104213/b6e1b705-8442-4090-a18b-6b3624dc1532
And next time, if you ask such meaningless questions again or do not follow the feedback rules, I will block you again.

sir , can you host a api site for replacing openai api , where i will use my key and give access to others . so they can use api inderectly using my api?(i dont wanna share original key)

sir , can you host a api site for replacing openai api , where i will use my key and give access to others . so they can use api inderectly using my api?(i dont wanna share original key)

Use one-api to manage
https://github.com/songquanpeng/one-api


96 create new drop

This PR (draft) implements a page where an admin can create a new drop. As I do not have admin access to any of our new contracts, I only tested it up until the point where I sent MetaMask the data which then displays that the transaction will fail (as it can pre-compute that I am not an admin). Once I am admin of a contract this will turn into a real PR. Other than that single component, everything was tested by me.
Remember that this story is not about perfect design but about the functionality of the page.
Some screenshots:

Just tested it properly with our most recent contract and me being an admin. Should be reviewable and mergable.


updated links and added temporary place holder to share link

These commits reflect an update to the links after the DKMS work was moved to the doc/design/005-dkms repo. It also adds a placeholder file for a link that was placed in a document that was shared in a congressional hearing. It can be deprecated in 6 months (11/8/18) as stated in the place holder. The link provided in that place holder doc will remain persistent because it links to commit: 677a0439487a1b7ce64c2e62671ed3e0079cc11f

Why we don't use relative links here?

I didn't add them at the beginning because I didn't have a relative link at the beginning. I'll submit a PR that switches them to relative links.


Translation refactor

Works on https://github.com/HospitalRun/hospitalrun-frontend/issues/547
This updates

pricing
imaging
diagnosis
labs

A few different modules ported over.
Text stays relatively the same, there is a pull request/s in the works for:

medication.
invoices.
inventory.
procedure.

the last remaining module is

patients.

The main goal here it easy to know where translations belong, i will be typing out some documentation upon completion of the port.
This is a breaking change for locales i am not sure if they are deployed somewhere if you would like to make a locale refactor branch to merge into i will be happy to do that.
i will implement transfix as soon as the refactor is completion.
thanks to @gabrielle-ong and @RenchChua

@billybonks I have been waiting to merge this in until the placeholder issue I brought up is resolved.  I'm fine waiting on it, but just thought I would ping you to make sure we are on the same page on this one.

@billybonks bump. :)

@jkleinsc I think this is ready to go, but I'll leave that to you.

@tangollama @billybonks I had asked that the "placeholder" label be changed.  That I can see, it has not been changed.

@billybonks closing this PR for now. Feel free to re-open when/if it makes sense.


support xml body in http steps (message validator)

When "testing" http request with xml content I get the error:
com.consol.citrus.exceptions.TestCaseFailedException: Failed to find proper message validator for message
Doing the same request with json body works.
Can be reproduced with this feature file (the api supports xml and json, depending on the header):
`Feature: http request with xml-body
Scenario: xml
    Given URL: http://api.nbp.pl
    Given HTTP request headers
        | Accept | application/xml |
        | Content-Type | text/xml |

    When send GET /api/exchangerates/tables/A/
    Then receive HTTP 200

Scenario: json
    Given URL: http://api.nbp.pl
    Given HTTP request headers
        | Accept | application/json |
        | Content-Type | text/json |

    When send GET /api/exchangerates/tables/A/
    Then receive HTTP 200`

First scenario fails with the above error, second is successful

the XML message validator is not part of the default YAKS runtime. You will have to add it as an explicit dependency to the test runtime.
You can do this via yaks-config.yaml or using the require Cucumber tag in your feature file.
yaks-config.yaml
config:
  runtime:
    settings:
      dependencies:
        - groupId: com.consol.citrus
          artifactId: citrus-validation-xml
          version: "@citrus.version@"

require tag_
@require('com.consol.citrus:citrus-validation-xml:@citrus.version@')
Feature: Foo

  Scenario: xml
  ...

Dear Christoph,
thank you very much and sorry, that I didn't find out myself.
Works perfectly


[feat] Portable app release?

Describe the problem
Run application without installing
Describe the solution you'd like
Is there a way create an app that doesn't require installation and can be run from a removable storage on any computer (assuming the same OS). On top of that any settings stored in the same directory as the executable?
For now I'm only interested in Windows OS.
Alternatives considered
No response
Additional context
No response

Same question here, is it possible to use cargo build to release and it will be great if theres a guide to do so

You could just ship the .exe file if you don't use resources or sidecars. But the user will need to have the Webview runtime installed, since our installer is responsible for managing it.

And the settings?

The only settings stored are the Webview ones, which can be customized with the data directory.


When I use nni Compressor Pruner, a 'NOT compressed'  error is occurred.

The error is ：

My code is：
import math
import torch
import torch.nn as nn
import torch.nn.functional as F
from torchvision import datasets, transforms
from nni.compression.torch import L1FilterPruner
from vgg import VGG
def main():
torch.manual_seed(0)
model = VGG(depth=16)

torch.save(model.state_dict(), 'vgg16_cifar10.pth')

# Pruning Configuration, in paper 'PRUNING FILTERS FOR EFFICIENT CONVNETS',
# Conv_1, Conv_8, Conv_9, Conv_10, Conv_11, Conv_12 are pruned with 50% sparsity, as 'VGG-16-pruned-A'
configure_list = [{
    'sparsity': 0.5,
    'op_types': ['default'],
    'op_names': ['feature.0', 'feature.24', 'feature.27', 'feature.30', 'feature.34', 'feature.37']
}]

# Prune model and test accuracy without fine tuning.
print('=' * 10 + 'Test on the pruned model before fine tune' + '=' * 10)
pruner = L1FilterPruner(model, configure_list)
pruned_model = pruner.compress()
pruner.export_model(model_path='test_pruned_model.pth', mask_path='test_mask_model.pth')
mask_state_dict = torch.load('./test_mask_model.pth')
print(mask_state_dict)

if name == 'main':
main()
The code is from the nni project example. I tried multiple pruning algorithms, but all got the same error.

Hi @yeliang2258, could you provide the nni version you are using, also could you try update nni to the latest version v1.4 and see if error still exists?

Hi @yeliang2258, could you provide the nni version you are using, also could you try update nni to the latest version v1.4 and see if error still exists?

At first I tried version v1.3 and found the above problem, then I installed the latest version using source code and found that it didn't work either

You have to train to calculate mask, default mask is None, which is why you get not compressed message.

You have to train to calculate mask, default mask is None, which is why you get not compressed message.

I get it, thanks!


Proposal to Add Blog Component for Website

This issue aims to propose the addition of a blog component to the website to enhance content creation, engage our audience, and improve overall website functionality. Introducing a blog will allow us to regularly publish articles, news updates, and other valuable content regarding ways to get rid of their habits.
I am eager for my proposal to be accepted and for the opportunity to be assigned to this particular issue under GSSoC'23.

Heyy..I loved the idea..you can work on this!
Thanks for the issue :-)


Implement From<OneOfVariant> for message_with_oneof::OneOfName

I saw the related item https://github.com/danburkert/prost/issues/42 -- I agree changing the representation or having two is unneeded.
However, I think ergonomics could be dramatically improved by generating implementations like the one in the title.
Does that make sense to you? If I can find the time, it'd be something I'd be happy to contribute if so.
For now, I've just been writing these by hand.
For more clarity on what I mean:
message ContainerMsg {
    oneof container_field {
        A a = 10;
        B b = 11;
    }
}

message A { ... }
message B { ... }

Would include these generated impls:
impl From<A> for container_msg::ContainerField {
    fn from(x: A) -> Self {
        container_msg::ContainerField::A(x)
    }
}

impl From<B> for transaction_update::Status {
    fn from(x: B) -> Self {
        container_msg::ContainerField::B(x)
    }
}

This isn't possible because oneofs can and often do have multiple variants with the same type.

@danburkert Ah, yeah that makes perfect sense. Would you be open to adding the From impl in the event that they don't? I have no such cases in my proto defs, and being able to call Variant.into() rather than fieldname::Variant(Variant) would be pretty nice.
I'd be happy to contribute it if you're open to it.

@danburkert Ah, yeah that makes perfect sense. Would you be open to adding the From impl in the event that they don't? I have no such cases in my proto defs, and being able to call Variant.into() rather than fieldname::Variant(Variant) would be pretty nice.
I'd be happy to contribute it if you're open to it.

Nevermind, for anyone who comes across this while searching, this is easily accomplished now by using https://docs.rs/prost-build/0.7.0/prost_build/struct.Config.html#method.type_attribute to add derive(::derive_more::From)

Nevermind, for anyone who comes across this while searching, this is easily accomplished now by using https://docs.rs/prost-build/0.7.0/prost_build/struct.Config.html#method.type_attribute to add derive(::derive_more::From)


Possible removal of .htaccess from document root

I think I have ran into a bit of an edge case.
We had an issue where the .htaccess from two of our domains (out of 60~) where removed yesterday. After much searching and hunting I believe the issue was caused by this module updating the certificates. I checked the crontab and it contained two entries to run the renew-certificates.php at exactly the same time [1]. The sites stopped working shortly after due to the missing .htaccess files.
I didn't think it was possible, until I saw in the module code that this script will create and remove a .htaccess file as part of the challenge. I believe that there is some race condition / edge case (eg .well-known exists or the whole challenge html file exists already or the .well-known exists but is removed by first instance before second instance can write its challenge, etc) then some function returns false in which case the self.verify_path is empty so instead of /.well-known/.htaccess it was just /.htaccess.
The crons ran at the same time due to the duplicates in the crontab. Given that, is this possible?

[1] We had the MAILTO in the crontab set. For some reason this module (maybe it's a plesk thing) added an entry under an empty MAILTO and then again under the defined MAILTO. This caused the renew-certificates.php to run twice.

Thank you for the reply and very useful module.
Since it's a new major version will Plesk automatically pickup and install it? Any timeframe?

Yes, Plesk updates extensions on weekly basis.
On Fri, 31 Mar 2017 at 02:01, Nick<EMAIL_ADDRESS>wrote:

Thank you for the reply and very useful module.
Since it's a new major version will Plesk automatically pickup and install
it? Any timeframe?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/plesk/letsencrypt-plesk/issues/119#issuecomment-290512005,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFIQLQimMic2Z1wCjB-Jv5F1P3E_wi27ks5rq_whgaJpZM4J5-cX
.

--
Eugene Kazakov

Is there a known timeframe work v2 release?


Can't import Bestiary - Storm Giant

Can't import statblock:
Storm Giant      CR 13
XP 25,600
Male storm giant (Pathfinder RPG Bestiary 152)
CG Huge humanoid (giant)
Init +2; Senses low-light vision; Perception +27
Defense
AC 28, touch 10, flat-footed 26 (+6 armor, +2 Dex, +12 natural, -2 size)
hp 199 (19d8+114)
Fort +17, Ref +8, Will +13
Defensive Abilities rock catching; Immune electricity
Offense
Speed 50 ft. (35 ft. in armor), swim 30 ft.
Melee mwk greatsword +27/+22/+17 (4d6+21/17-20) or
2 slams +21 (2d6+7)
Ranged mwk composite longbow +15/+10/+5 (3d6+14/×3)
Space 15 ft.; Reach 15 ft.
Spell-Like Abilities (CL 15th; concentration +17)
Constant—freedom of movement
2/day—control weather, levitate
1/day—call lightning (DC 15), chain lightning (DC 18)
Statistics
Str 39, Dex 14, Con 23, Int 16, Wis 20, Cha 15
Base Atk +14; CMB +30 (+32 bull rush, +32 sunder); CMD 42 (44 vs. bull rush, 44 vs. sunder)
Feats Awesome Blow, Cleave, Combat Reflexes, Improved Bull Rush, Improved Critical (greatsword), Improved Sunder, Improved Vital Strike, Iron Will, Power Attack, Vital Strike
Skills Acrobatics +18, Climb +17, Craft (Enter Choice) +11, Intimidate +20, Perception +27, Perform (sing) +12, Sense Motive +24, Swim +22
Languages Auran, Common, Draconic, Giant
SQ water breathing
Other Gear mwk breastplate, mwk composite longbow (+14 Str), mwk greatsword
Ecology
Environment any warm
Organization solitary or family (2-5 plus 1 sorcerer or cleric of 7th-10th level, 1-2 rocs, 2-6 griffons, and 2-8 sharks)
Treasure standard (mwk breastplate,  mwk composite longbow [+14  str bonus] with 20 arrows, mwk  greatsword, other treasure)
Special Abilities
Awesome Blow As standard action, damage and move smaller foe 10 ft. +1d6 dam if collide with something.
Cleave If you hit a foe, attack an adjacent target at the same attack bonus but take -2 AC.
Combat Reflexes (3 AoO/round) Can make extra attacks of opportunity/rd, and even when flat-footed.
Immunity to Electricity You are immune to electricity damage.
Improved Bull Rush You don't provoke attacks of opportunity when bull rushing.
Improved Sunder You don't provoke attacks of opportunity when sundering.
Low-Light Vision See twice as far as a human in dim light, distinguishing color and detail.
Power Attack -4/+8 You can subtract from your attack roll to add to your damage.
Rock Catching (Ex) You can catch rocks that are thrown at you with a successful Reflex save.
Swim (30 feet) You have a Swim speed.
Vital Strike Standard action: x2 weapon damage dice.
Water Breathing You can breathe water as well as air.

I got this statblock from Hero Lab.  You might benefit from retesting with aonprd.com.  Sometimes Hero Lab truncates some of the descriptions.
I used the latest Pathfinder system and your latest module to test this.

Identified Issue: Context Notes for Base Speed.

Thanks for all the Input @adventuremagic123 ! This one is the only one thats not yet fixed. The other ones should work in 0.3.5

You are absolutely welcome.  I thought I'd help out by trying to import all of the statblocks for the Bestiary.
If you provide the feature where I can import multiple statblocks at once that would help me a lot.
It took about 6 hours to do them one at a time.
If you provide that feature, I can very quickly test Bestiary 2 through 6 -- after I get the fixes you've made for Bestiary.
Your import module is the critical feature that enables me to use Foundry VTT -- and I very much appreciate it!
--
Mark Young (aka Adventure Magic 1, 2, 3 ...)
On Jun 28, 2020, 12:52 PM, at 12:52 PM, Lavaeolous<EMAIL_ADDRESS>wrote:

Thanks for all the Input @adventuremagic123 ! This one is the only one
thats not yet fixed. The other ones should work in 0.3.5
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Lavaeolous/PF1-StatBlock-Converter-Module/issues/78#issuecomment-650800046

BTW, PCGen can also be used as a source for statblocks, although their statblocks require simple changes to work because they broke the standard in minor ways.

Export with template "csheet_fantasy_statblock_pathfinder.htm.ftl".

Copy-and-paste from a web browser.

Look at a standard Pathfinder statblock and fix theirs.

--
Mark
On Jun 28, 2020, 3:12 PM, at 3:12 PM, Mark Young<EMAIL_ADDRESS>wrote:

You are absolutely welcome.  I thought I'd help out by trying to import
all of the statblocks for the Bestiary.
If you provide the feature where I can import multiple statblocks at
once that would help me a lot.
It took about 6 hours to do them one at a time.
If you provide that feature, I can very quickly test Bestiary 2 through
6 -- after I get the fixes you've made for Bestiary.
Your import module is the critical feature that enables me to use
Foundry VTT -- and I very much appreciate it!
--
Mark Young (aka Adventure Magic 1, 2, 3 ...)
On Jun 28, 2020, 12:52 PM, at 12:52 PM, Lavaeolous
<EMAIL_ADDRESS>wrote:

Thanks for all the Input @adventuremagic123 ! This one is the only one
thats not yet fixed. The other ones should work in 0.3.5
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Lavaeolous/PF1-StatBlock-Converter-Module/issues/78#issuecomment-650800046

Fixed in the next update, but there is no context field in the sheet for land/swim/burrow/climb speeds.

I don't have steps-to-reproduce.  I've lost any useful data.
Yesterday, all I did was import the NPC for Bestiary 2 to help test your importer module -- and later that day I downloaded and installed about 40 modules at once.
I started the world to configure the modules.  I found that most of the views on the right side weren't rendering.  A message would appear after initially starting the world view that the connection to the server had failed, and a reconnection was being attempted.
I'm technical.  I've developed a scheme for backing up and do so regularly.  I found that my world was responsible for causing all the problems.  Using the prior world from my backup fixed the problems.
Just something to watch for.  I don't know exactly what did it.  The module installer got stuck at "complete".  I closed FoundryVTT with the X button and restarted.  Maybe that did it?
--
Mark
On Jun 28, 2020, 4:07 PM, at 4:07 PM, Mark Young<EMAIL_ADDRESS>wrote:

BTW, PCGen can also be used as a source for statblocks, although their
statblocks require simple changes to work because they broke the
standard in minor ways.

Export with template "csheet_fantasy_statblock_pathfinder.htm.ftl".

Copy-and-paste from a web browser.

Look at a standard Pathfinder statblock and fix theirs.

--
Mark
On Jun 28, 2020, 3:12 PM, at 3:12 PM, Mark Young<EMAIL_ADDRESS>wrote:

You are absolutely welcome.  I thought I'd help out by trying to
import
all of the statblocks for the Bestiary.
If you provide the feature where I can import multiple statblocks at
once that would help me a lot.
It took about 6 hours to do them one at a time.
If you provide that feature, I can very quickly test Bestiary 2
through
6 -- after I get the fixes you've made for Bestiary.
Your import module is the critical feature that enables me to use
Foundry VTT -- and I very much appreciate it!
--
Mark Young (aka Adventure Magic 1, 2, 3 ...)
On Jun 28, 2020, 12:52 PM, at 12:52 PM, Lavaeolous
<EMAIL_ADDRESS>wrote:

Thanks for all the Input @adventuremagic123 ! This one is the only
one
thats not yet fixed. The other ones should work in 0.3.5
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Lavaeolous/PF1-StatBlock-Converter-Module/issues/78#issuecomment-650800046

I found what caused that.  Installing the "Permission Viewer" module creates this problem.  Don't even have to enable it.  Foundry VTT server fails.  UI doesn't populate (mostly).  Remove this module and the problem goes away.  All your data is ok!
--
Mark
On Jul 3, 2020, 11:35 AM, at 11:35 AM, Mark Young<EMAIL_ADDRESS>wrote:

I don't have steps-to-reproduce.  I've lost any useful data.
Yesterday, all I did was import the NPC for Bestiary 2 to help test
your importer module -- and later that day I downloaded and installed
about 40 modules at once.
I started the world to configure the modules.  I found that most of the
views on the right side weren't rendering.  A message would appear
after initially starting the world view that the connection to the
server had failed, and a reconnection was being attempted.
I'm technical.  I've developed a scheme for backing up and do so
regularly.  I found that my world was responsible for causing all the
problems.  Using the prior world from my backup fixed the problems.
Just something to watch for.  I don't know exactly what did it.  The
module installer got stuck at "complete".  I closed FoundryVTT with the
X button and restarted.  Maybe that did it?
--
Mark
On Jun 28, 2020, 4:07 PM, at 4:07 PM, Mark Young<EMAIL_ADDRESS>wrote:

BTW, PCGen can also be used as a source for statblocks, although their
statblocks require simple changes to work because they broke the
standard in minor ways.

Export with template "csheet_fantasy_statblock_pathfinder.htm.ftl".

Copy-and-paste from a web browser.

Look at a standard Pathfinder statblock and fix theirs.

--
Mark
On Jun 28, 2020, 3:12 PM, at 3:12 PM, Mark Young
<EMAIL_ADDRESS>wrote:

You are absolutely welcome.  I thought I'd help out by trying to
import
all of the statblocks for the Bestiary.
If you provide the feature where I can import multiple statblocks at
once that would help me a lot.
It took about 6 hours to do them one at a time.
If you provide that feature, I can very quickly test Bestiary 2
through
6 -- after I get the fixes you've made for Bestiary.
Your import module is the critical feature that enables me to use
Foundry VTT -- and I very much appreciate it!
--
Mark Young (aka Adventure Magic 1, 2, 3 ...)
On Jun 28, 2020, 12:52 PM, at 12:52 PM, Lavaeolous
<EMAIL_ADDRESS>wrote:

Thanks for all the Input @adventuremagic123 ! This one is the only
one
thats not yet fixed. The other ones should work in 0.3.5
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/Lavaeolous/PF1-StatBlock-Converter-Module/issues/78#issuecomment-650800046


remove max-line-length for doc8

Description:
Our best practice for docs is to not use
hard line breaks, and to let editors wrap.
See https://open-edx-proposals.readthedocs.io/en/latest/oep-0019-bp-developer-documentation.html#best-practices
Merge checklist:

[ ] All reviewers approved
[x] CI build is green
[x] Changelog record added
[ ] Documentation updated (not only docstrings)
[ ] Commits are squashed

Post merge:

[ ] Delete working branch (if not needed anymore)

I was actually under the impression we were limiting docstrings to 79 characters based on : https://edx.readthedocs.io/projects/edx-developer-guide/en/latest/style_guides/python-guidelines.html#syntax-and-organization

thanks for links to best practices

@jinder1s: FYI: I am changing the rules for .rst docs.
I think  pydocstyle is what deals with docstrings.  I think people still generally try to keep those to 80 characters, but 120 is also legal.


Use our standard placeholder variable names

I noticed that the .env-example used API_KEY rather than NEXMO_API_KEY which the code seemed to expect, so I changed it to match.

Got carried away and added a repo as well.  Could you take another look please?

This commit changes the .env-example to use FROM, as our building blocks do.  I couldn't find any other examples that used it, they mostly hardcode the from.  Hopefully good to go now.


Spinner not hiding when using with alert

Hi, Having an issues if I display the spinner and then call my api, when I get back the result from the api I change my state to hide the spinner. The problem is that if I show an alert to display the error the spinner doesnt hid. It just keeps spinning and cant cancel or anything.
Here is my code
ValidatePromoCode() {
    if (this.state.email != '') {
      this.setState({
        visible: true,
        spinnerText: 'Validating email address'
      });

      //Make the API call
      StudentApi.SetupBuyOnline(this.state.email, DeviceInfo.getUniqueID(), DeviceInfo.getSystemName())
        .then((responseData) => {
          this.setState({
            visible: false,
            spinnerText: ''
          });

          alert(data);
    }
    else {
      this.props.navigation.navigate('GetRegisteredScreen')
    }
  }

Can some one please let me know what I am doing wrong? if i dont have the alert the spinner seems to behave correctly. Thanks in advance

I will suggest you try to wrap "alert" with setTimeout().
setTimeout(() => {
Alert.alert(I18n.t('MESSAGE'),I18n.t('ACCOUNT_OR_PASSWORD_ERROR'))
}, 100);

@Daky I have put the setTimeout and its still not working

the problem isn't the alert box. The spinner does not stop when I show the alert box. I have tried the above but its not working.