New helper functions

Added two helper functions: path_relative_from_include() which is the counter part of path_relative_to_include() and get_aws_account_id() which returns the current AWS account number.
Updated the documentation accordingly (Read documentation to view use cases for the new interpolation functions).
Added state to terraform remote state configuration.
Note that there are a lot of changes in the README.md file. This is because my editor automatically removed trailing blanks. But only the documentation relative to the two new functions has really been modified.

Hi @brikis98, I pushed a new version of the documentation (without the trailing blanks removal to help reading).
Regarding get_aws_account_id(), I understand your concern. Our problem is that we use the account id to make sure that our remote state bucket is unique. We have several account and it is the way we use to ensure that the bucket name is unique. We are facing an egg and chicken problem because we cannot access the terraform data source that gives us the accound id while we are configuring terragrunt to use that bucket name.
It can also help to determine the environment automatically based on AWS credentials. This could avoid an error of pushing the wrong environment in the wrong account (which could happen by relying solely on environment variables).
I also realized after implementing path_relative_from include() that what we really needed was a function like get_tfvars_dir() but returning the absolute path of the root configuration instead of the leaf. I added it (get_parent_dir(), but I wonder if path_relative_from_include() is still required).
Finally, I have a question for you. Is the limitation of not being able to have cascading find_in_parent_folders() is by design or you have any other reason. We structured our terraform folders with a subdirectory named private when we have sensitive element like secrets and don't want to make them visible to everybody. We can then control the access to the remote state file through IAM rules and ensure that secrets are not exposed in tfstate that have to be referred by other terraform folders.

Our problem is that we use the account id to make sure that our remote state bucket is unique. We have several account and it is the way we use to ensure that the bucket name is unique.

Don't the AWS creds you use determine which account gets used? Therefore, if you accidentally plug in the wrong AWS creds, you'd end up with the wrong bucket name by accident...

Is the limitation of not being able to have cascading find_in_parent_folders() is by design or you have any other reason.

It's mainly to keep things simpler. If you know there is only one level of includes, it's easier to reason about the code, and the implementation is simpler too. I wasn't aware of any real-world use cases for multiple levels of includes, so this seemed like a reasonable limitation for now.

We structured our terraform folders with a subdirectory named private when we have sensitive element like secrets and don't want to make them visible to everybody. We can then control the access to the remote state file through IAM rules and ensure that secrets are not exposed in tfstate that have to be referred by other terraform folders.

How does having multiple levels of includes help with this? Perhaps a concrete example of the folder structure and various .tfvars files would help me understand.

Thanks!

Just created a new release here: https://github.com/gruntwork-io/terragrunt/releases/tag/v0.12.7


Update and unify maven-shade-plugin version

Versions below 3.5.1 may have issues with classes compiled in Java 20 and 21, which we may find in multi-release jar dependencies

We should go with #12712


Expand Promise example

Sorry, I'm not following at all how promise ends up available to catchError() and then().  If all of that example code exists under the aegis of a larger object implementation, it would be clearer to the reader if he or she could see that.
It's also not completely clear to me how this is better than just capturing the exception via an ordinary Promise#catch and vetting the exception type inside the function.  If it isn't what you want, just re-throw from there.  No extension to the Promise prototype would be required.  What am I missing here?

promise ends up to catchError() via the operator. Following the rule x |> f is equivalent to f(x), we can deduce the following:
fetchPlayers().then( processGames )
  |> catchError( ProcessError, err => [] )

// Equivalent to:

catchError( ProcessError, err => [] )(
  fetchPlayers().then( processGames )
)

This works because catchError() return an arrow function that expects a promise. Its primary purpose is to remove the boilerplate of catching specific errors – something Bluebird.js has built in, and something I use quite often.

Yeah, I thought I was reading that incorrectly.  Thanks!


Relogin flag isn't working - [IOS- React-Native]

Version using. -  "@web3auth/react-native-sdk": "^3.0.1",

it's deprecated now


Add hts precompile approval/allowance tests

Signed-off-by: nikolay<EMAIL_ADDRESS>Description:

Related issue(s):
Fixes #399
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #443 (6279eee) into main (a66f3df) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6279eee differs from pull request most recent head f453e94. Consider uploading reports for the commit f453e94 to get more accurate results

@@           Coverage Diff           @@
##             main     #443   +/-   ##
=======================================
  Coverage   76.12%   76.12%
=======================================
  Files          11       11
  Lines         909      909
  Branches      142      142
=======================================
  Hits          692      692
  Misses        165      165
  Partials       52       52

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Minor text fixes, updated screenshot, eclipse plugin instead of maven

This change is

Review status: 0 of 3 files reviewed at latest revision, 5 unresolved discussions.

index.asciidoc, line 27 at r1 (raw file):

The tutorials will build off one another, each starting where the previous left off. Various steps of the tutorial and
alternative approaches will in the future be available as tags/branches at https://github.com/vaadin/spring-tutorial.
Some code samples could be found at https://github.com/vaadin/spring-demo7 as well.

could say "Additional" or "More" rather than "Some"

index.asciidoc, line 379 at r1 (raw file):

=== Creating a Vaadin project
If you've created Vaadin projects before, there's nothing new here. Usually Vaadin projects are created with

I'd drop "Usually" and replace "are" with "can be"

index.asciidoc, line 386 at r1 (raw file):

image::eclipse-plugin-new-project.png[Project Creation]

Then select the "Simgle Module Application Project". Set your group and

Single

index.asciidoc, line 391 at r1 (raw file):

You can also create the project without the plug-in, just create a project from maven archetype
`com.vaadin:vaadin-archetype-application:7.7.3` using Eclipse Maven wizard, or with your preferable IDE,

... with your IDE, or on/with the command line.

img/eclipse-plugin-new-project.png, line 0 at r1 (raw file):
it looks like this picture has a green frame? or just a GitHub artifact?

Comments from Reviewable

Review status: 0 of 3 files reviewed at latest revision, 5 unresolved discussions.

index.asciidoc, line 27 at r1 (raw file):
Previously, hesara (Henri Sara) wrote…
> could say "Additional" or "More" rather than "Some"

Done.

index.asciidoc, line 379 at r1 (raw file):
Previously, hesara (Henri Sara) wrote…
> I'd drop "Usually" and replace "are" with "can be"

Done.

index.asciidoc, line 386 at r1 (raw file):
Previously, hesara (Henri Sara) wrote…
> Single

Done.

index.asciidoc, line 391 at r1 (raw file):
Previously, hesara (Henri Sara) wrote…
> ... with your IDE, or on/with the command line.

Done.

img/eclipse-plugin-new-project.png, line  at r1 (raw file):
Previously, hesara (Henri Sara) wrote…
> it looks like this picture has a green frame? or just a GitHub artifact?

Picture fixed

Comments from Reviewable

Reviewed 1 of 3 files at r1, 3 of 3 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Annotation-based plugin architecure

Since Pentaho 3.0.0.RC1 it is possible/recommended (?) to write Plugins using annotations instead of the old plugin.xml. However, there are some conventions to be honored - most notably the step has to be defined below org.pentaho.di.trans. The full documentation about annotation-based plugin development is available from the Pentaho wiki.
As I could not get the plugin up and running using the old style I have refactored the step to use the annotation-based approach. Not additional functional changes are included in this branch - the plugin works great. Thanks!

Looks great. Does this mean we stop supporting older Pentaho versions?

According to the documentation the annotation-based plugins are developed from 3.0.0-RC1.
The oldest version I have lying around here is 4.1.0 (released in 2010) and the plugin works fine there. So I assume it is save to say that older versions are supported.

@fhossfel Thank you for the contribution!


Expose new CRF classes and function.

#12056 introduced a new tensor-based CRF decoding. This PR exposes the new names at the module level.

Can one of the admins verify this patch?

Jenkins, test this please


Libfunc completion tracking list.

Total: 177 libfuncs

91 implemented (in the current refactor, there are more we haven't ported from the pre-refactor branch yet).

of which 14 of them are either incomplete, placeholders or we aren't sure they are implemented correctly.

[ ] alloc_local

[x] array_append

[x] array_get

[x] array_len

[x] array_new

[x] array_pop_front_consume

[x] array_pop_front

[ ] array_slice

[x] array_snapshot_pop_back

[x] array_snapshot_pop_front

[ ] bitwise

[x] bool_and_impl

[x] bool_not_impl

[x] bool_or_impl

[ ] bool_to_felt252

[x] bool_xor_impl

[x] branch_align

[ ] call_contract_syscall (StarkNet)

[ ] class_hash_const (StarkNet)

[ ] class_hash_to_felt252 (StarkNet)

[ ] class_hash_try_from_felt252 (StarkNet)

[ ] contract_address_const (StarkNet)

[ ] contract_address_to_felt252 (StarkNet)

[ ] contract_address_try_from_felt252 (StarkNet)

[ ] deploy_syscall (StarkNet)

[x] disable_ap_tracking

[ ] downcast

[3] drop

[3] dup

[ ] ec_neg

[ ] ec_point_from_x_nz

[ ] ec_point_is_zero

[ ] ec_point_try_new_nz

[ ] ec_point_unwrap

[ ] ec_point_zero

[ ] ec_state_add_mul

[ ] ec_state_add

[ ] ec_state_init

[ ] ec_state_try_finalize_nz

[ ] emit_event_syscall (StarkNet)

[x] enable_ap_tracking

[x] enum_init

[x] enum_match

[ ] enum_snapshot_match

[4] felt252_add_const

[x] felt252_add

[x] felt252_const

[ ] felt252_dict_entry_finalize

[ ] felt252_dict_entry_get

[ ] felt252_dict_new

[ ] felt252_dict_squash

[4] felt252_div_const

[4] felt252_div

[x] felt252_is_zero

[4] felt252_mul_const

[x] felt252_mul

[4] felt252_sub_const

[x] felt252_sub

[ ] finalize_locals

[x] function_call

[ ] get_available_gas

[ ] get_block_hash_syscall (StarkNet)

[5] get_builtin_costs

[ ] get_execution_info_syscall (StarkNet)

[2] into_box

[x] jump

[ ] keccak_syscall (StarkNet)

[ ] library_call_syscall (StarkNet)

[ ] match_nullable

[ ] null

[ ] nullable_from_box

[x] pedersen

[ ] pop_log (StarkNet, testing)

[ ] poseidon

[x] print

[ ] redeposit_gas

[x] rename

[ ] replace_class_syscall (StarkNet)

[1] revoke_ap_tracking

[ ] secp256k1_add_syscall (StarkNet)

[ ] secp256k1_get_point_from_x_syscall (StarkNet)

[ ] secp256k1_get_xy_syscall (StarkNet)

[ ] secp256k1_mul_syscall (StarkNet)

[ ] secp256k1_new_syscall (StarkNet)

[ ] secp256r1_add_syscall (StarkNet)

[ ] secp256r1_get_point_from_x_syscall (StarkNet)

[ ] secp256r1_get_xy_syscall (StarkNet)

[ ] secp256r1_mul_syscall (StarkNet)

[ ] secp256r1_new_syscall (StarkNet)

[ ] send_message_to_l1_syscall (StarkNet)

[ ] set_account_contract_address (StarkNet, testing)

[ ] set_block_number (StarkNet, testing)

[ ] set_block_timestamp (StarkNet, testing)

[ ] set_caller_address (StarkNet, testing)

[ ] set_chain_id (StarkNet, testing)

[ ] set_contract_address (StarkNet, testing)

[ ] set_max_fee (StarkNet, testing)

[ ] set_nonce (StarkNet, testing)

[ ] set_sequencer_address (StarkNet, testing)

[ ] set_signature (StarkNet, testing)

[ ] set_transaction_hash (StarkNet, testing)

[ ] set_version (StarkNet, testing)

[6] snapshot_take

[x] storage_address_from_base_and_offset (StarkNet)

[x] storage_address_from_base (StarkNet)

[x] storage_address_to_felt252 (StarkNet)

[x] storage_address_try_from_felt252 (StarkNet)

[x] storage_base_address_const (StarkNet)

[x] storage_base_address_from_felt252 (StarkNet)

[ ] storage_read_syscall (StarkNet)

[ ] storage_write_syscall (StarkNet)

[ ] store_local

[x] store_temp

[x] struct_construct

[x] struct_deconstruct

[ ] struct_snapshot_deconstruct

[x] u128_byte_reverse

[x] u128_const

[x] u128_eq

[ ] u128_guarantee_mul

[x] u128_is_zero

[ ] u128_mul_guarantee_verify

[x] u128_overflowing_add

[x] u128_overflowing_sub

[x] u128_safe_divmod

[ ] u128_sqrt

[x] u128_to_felt252

[x] u128s_from_felt252

[x] u16_const

[x] u16_eq

[x] u16_is_zero

[x] u16_overflowing_add

[x] u16_overflowing_sub

[x] u16_safe_divmod

[ ] u16_sqrt

[x] u16_to_felt252

[x] u16_try_from_felt252

[ ] u16_wide_mul

[ ] u256_is_zero

[ ] u256_safe_divmod

[ ] u256_sqrt

[x] u32_const

[x] u32_eq

[x] u32_is_zero

[x] u32_overflowing_add

[x] u32_overflowing_sub

[x] u32_safe_divmod

[ ] u32_sqrt

[x] u32_to_felt252

[x] u32_try_from_felt252

[ ] u32_wide_mul

[ ] u512_safe_divmod_by_u256

[x] u64_const

[x] u64_eq

[x] u64_is_zero

[x] u64_overflowing_add

[x] u64_overflowing_sub

[x] u64_safe_divmod

[ ] u64_sqrt

[x] u64_to_felt252

[x] u64_try_from_felt252

[ ] u64_wide_mul

[x] u8_const

[x] u8_eq

[x] u8_is_zero

[x] u8_overflowing_add

[x] u8_overflowing_sub

[x] u8_safe_divmod

[ ] u8_sqrt

[x] u8_to_felt252

[x] u8_try_from_felt252

[ ] u8_wide_mul

[2] unbox

[ ] unwrap_non_zero

[ ] upcast

[5] withdraw_gas_all

[5] withdraw_gas

[1]: It is implemented but we're not sure if it has some stuff we don't know of.
[2]: It is implemented but we're still debating whether it should be a Rust-like Box<T> or if it's
fine treating it like another variable.
[3]: It is implemented but side-effects are not yet handled (ex. array cloning/dropping).
[4]: Not supported by the Cairo to Sierra compiler.
[5]: Implemented with a dummy. It doesn't do anything yet.
[6]: It is implemented but we're not handling potential issues like lifetimes yet.

Closing this tracking issue as it has become stale and progress has been tracked elsewhere.


Cloud master add command can now register and configure current master

$ kontena cloud master add --current
Proceeding will:
 * Register the Kontena Master jormaxxxz to Kontena Cloud
 * Configure the Kontena Master to use Kontena Cloud as the
   authentication provider

After this:
 * Users will not be able to reauthenticate without authorizing the
   Master to access their Kontena Cloud user information
 * Users that have registered a different email address to Kontena
   Cloud than the one they currently have as their username in the
   master will not be able to authenticate before an administrator
   of the Kontena Master creates an invitation code for them
   (kontena master users invite<EMAIL_ADDRESS>> Proceed? Yes
 [done] Registering current Kontena Master 'jormaxxxz' to Kontena Cloud
 [done] Loading Kontena Cloud auth provider base configuration to Kontena Master
 [done] Updating OAuth2 client-id and client-secret to Kontena Master

Also the URL's to production cloud have been updated.

Docs need update

Done


the commands are not working

is the bot running?

You must type it with bot name. For example:  @PossumBot !servernotgood


Throw correct exceptions on non existing files and folders

The same exceptions are thrown in LocalDriver
This prevents issues like "call to a member function getProperties()
on boolean."
It also contains little performance optimization (one call to getContainerProperties in "getFileInfobyIdentifier" less) due too removal of not needed $this->fileExists($fileIdentifier) call.

@benjaminhirsch would be great if you review it. Thanks


Reviewer is both a blocker and an approver according to the table.

For 3427 uilianries is both a blocker and an approver according to the table. I guess you do not correctly overwrite the old reviews if a new one exists from the same user
Originally posted by @Croydon in https://github.com/prince-chrismc/conan-center-index-pending-review/issues/1#issuecomment-751474283

Moving bug

I guess you do not correctly overwrite the old reviews if a new one exists from the same user

That is exactly correct,

still learning golang and I have not found an easy way to reduce the array/slice.
There is pagination so not all the reviews come up on the first page, making it a little cumbersome

This issue how now been fixed with https://github.com/prince-chrismc/conan-center-index-pending-review/commit/884003b1d1be5609b538eb4ae62dc0932eb8e92c


att2all

In AttModel.py
You write an annotation that "Note this is my attempt to replicate att2all model in self-critical paper. However, this is not a correct replication actually". Will fix it." Could you tell me where is the error of the model ? I will try to fix it and use the model. Thanks.

I don't remember clearly, but I think it is the implementation of the LSTM cell. You can check out their paper, and use it.


runtime(kconfig): fix broken syntax highlighting due to help section

fixes: #15306

Thanks, but that should go through the upstream repo: https://github.com/chrisbra/vim-kconfig

New pull request as requested: https://github.com/chrisbra/vim-kconfig/pull/2


AttributeError: 'odict_values' object has no attribute 'reverse'

Python version: 3.5.2
Error appears in two lines:

https://github.com/coderfly/fixtures-mongoengine/blob/master/fixtures_mongoengine/mixin.py#L161
https://github.com/coderfly/fixtures-mongoengine/blob/master/fixtures_mongoengine/mixin.py#L171

    ...

    fixtures = self.get_fixtures().values()
    fixtures.reverse()

    ...

Reason:

self.get_fixtures() returns an OrderedDict object,
self.get_fixtures().values() returns an odict_values object,

But, at least in Python 3.5.2 it's not possible to call reverse() on odict_values.
Easiest workaround:
fixtures = list(self.get_fixtures().values())

I haven't investigated it deeper.

@danie1k the same problem, thank you for solution, made pull request

Thanks for fix


Make filter value input to TemperatureResponseFundamental consistent with Sunpy

Currently the value returned by an XRTMap object created via
import sunpy.map
xrtmap = sunpy.map.Map(filepath)
gives the filter combination as an attribute of xrtmap:
xrtmap.measurement = 'Be thin-Open'
However the format is not compatible with that accepted by XRTpy.response.temperature_response.TemperatureResponseFundamental
e.g. 'Be-thin' or 'Be_thin'
For the sake of interoperability these should be made compatible (not saying which should change).

Hi Joy,
I created a new branch to work on this:
improve-filter-value-input-handling
Jon
On Tue, Jul 18, 2023 at 4:12 PM Joy @.***> wrote:

Assigned #117 https://github.com/HinodeXRT/xrtpy/issues/117 to @jslavin
https://github.com/jslavin.
—
Reply to this email directly, view it on GitHub
https://github.com/HinodeXRT/xrtpy/issues/117#event-9855060361, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE5OY7GYUQEYCRYSAGLC43XQ3U3NANCNFSM6AAAAAAUHUYP3E
.
You are receiving this because you were assigned.Message ID:
@.***>

One tricky part about this is the question of what we'll accept as inputs. I would think we should accept ones in the Sunpy style, e.g. Open-Ti poly (with a space), and in the fits header style, e.g. Ti_poly or Open/Ti_poly. Any others? What if someone gives 'Ti poly' for example, or 'Open/Ti poly'? The parsing could get complicated.

It appears that currently it is okay to supply "open" as the filter name to Channel. Why would that be? It seems that that should not be allowed since, as far as I know, that's not an allowed observation mode.


ThemeProvider is not exported from root.js

<EMAIL_ADDRESS>Attempted import error: 'ThemeProvider' is not exported from '@material-ui/core/styles'.
^C

Can you provide more detail?

version of mui-treasury/layout
may be repo | codesandbox that I can reproduce

@drtobbyas I think I know why you got error. Please try upgrade @material-ui/core to latest, I think that will solve.

I have resolved it.  But notably the documentation is a bit confusing.
There are some packages that were not natively exported from the module but
used as if they were exported.  Many people would prefer to copy and paste
the doc example to test a package or import using repl.it while testing a
new package.
All the same great work  and thanks for the reply.
Regards
On Fri, Jan 3, 2020, 02:13 Siriwat Kunaporn<EMAIL_ADDRESS>wrote:

Can you provide more detail?

version of mui-treasury/layout
may be repo | codesandbox that I can reproduce

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/siriwatknp/mui-treasury/issues/367?email_source=notifications&email_token=AJYHCVPEDXJNQ2NJ675QS2LQ32GMDA5CNFSM4KBXWLKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIAABMY#issuecomment-570425523,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJYHCVKQYZBUU7VCPQG7OI3Q32GMDANCNFSM4KBXWLKA
.

@drtobbyas any suggestion on how to make it better for other people?


feat: ini arbiters

📚 $\text{Stack Overview}$
Pulls submitted in this stack:

#5 👈
main

This comment was automatically generated by st.


Not found error for endpoints

I cloned the repo, changed to public port of nginx to 8001 and now I am getting this error for every page I try to open:
➜  ✗ curl http://localhost:8001/demo-service
{"detail":"Not Found"}%                                                                                ➜ ✗ curl http://localhost:8001/demo-service/hello
{"detail":"Not Found"}%                                                                                ➜ ✗ curl http://localhost:8001/

<title>404 Not Found</title>

404 Not Found

nginx/1.17.3

Could you maybe guide me what I am doing wrong here?

Which branch did you clone? If you simply checked out master then as you can see from the README there are a number of modifications necessary to make it work. Also, I hope it's clear that this whole repo was meant to demonstrate a particular problem and not really to be used for anything else :wink:

I was using the master branch which of course did not contain the behavior I wanted to see. I know it's for demo purposes and I wanted to reproduce what is described in the README.

If you check out the https://github.com/Midnighter/fastapi-mount/tree/root-path branch it should be ready for you to modify as you intended.

Exactly. It works. Thanks for hinting me into the branch issue.

I was wondering if I switch the branches around, is that more intuitive?


Add an encryption option

Currently traffic is encrypted, but data is not

Thanks for atom-pair @jpatel531 - could you expand on the difference between traffic vs. data?

Perhaps this?
Traffic is over HTTPS, but data's over cleartext?
Maybe using something like OTR to start off (it's not the most secure) and upgrading to use NaCl to use a PSK to encrypt the data.
Of course, speed + performance considerations come into play.

So currently you may have some of or all of the code your sharing at rest on Pusher server in plain text?


create schema with union type

I am having issues when trying to validate a schema with a schema union type. To simplify the description I have created a small example. My test endpoint here can receive two types of query params. So the querystring can either receive Name:
export const Name = z.object({
  name: z.string(),
  lastName: z.string()
});

or It can receive Address:
export const Address = z.object({
  street: z.string(),
  postcode: z.string()
});

So the schema type for my endpoint would be a union of Name and Address:
const UserDetails = z.union([Name, Address]);
const { schemas, $ref } = buildJsonSchemas({ UserDetails });
const test: FastifyPluginAsync = async (fastify) => {
  for (const schema of schemas) {
    fastify.addSchema(schema);
  }

  fastify.route({
    method: 'GET',
    url: '/',
    schema: {
      operationId: `userDetails`,
      querystring: $ref(`UserDetails`)
    },
    handler: async function authTokenHandler(request: FastifyRequest, reply: FastifyReply) {
      return 'Schema Validated, All Good!';
    }
  });
};
export default test;

The issue I am having is that the validation is not recognizing which type to enforce when a param is missing. So if test calling:
/test?name=someName // "querystring should have required property 'lastName', querystring should have required property 'street', querystring should match some schema in anyOf"

And here I expected the validation to have identified the type as Name, and only required lastName. But no, it is requiring street as well. Although is a different type from the one I am sending.
Is this expected ? I would have expected Name Schema to have been identified by sending name and only enforced that type on the schema instead of both types from the union. Is there a way around this?

Hello,
Hello,
I'm sorry, as mentioned in https://github.com/elierotenberg/fastify-zod/issues/15 for some reason GitHub notifications got lost in my mailbox.
There were 2 issues with the example you provide.
The first is lack of support for querystring, which is now available in 1.0.0.
The second is, I think, due to how discriminated unions work. Without a discriminator property (in OpenAPI as well as in TypeScript), it is not possible to know for sure which branch of the union you are referring to. With a discriminator property however, the correct error is raised. I added a test to illustrate.
Does this answer cover your needs?


No output from PHP

I updated my server and suddenly websocketd does not provide any output from PHP-scripts (tried the example ones as well), it connects but no output at all. The bash-script works just fine.
PHP 7.4.6 (cli) (built: May 12 2020 15:48:23) ( NTS )
Is there an issue with websocketd and this PHP version?
I know that it works just fine with:
PHP 7.4.2 (cli) (built: Jan 21 2020 18:16:58) ( NTS )

if you run the demo on windows , you should use PHP_ EOL replaces the "\n" in demo


SCSSLint::Linter::Comment - Length must be more than 0, passed -38

linting this:
@mixin make-grid($class) {
  @if ($class == "lg")
    { $grid-columns-lpo: $grid-columns + 3;  } /*sev */
}

results in this:
SCSSLint::Linter::Comment raised unexpected error linting file repro.scss: 'Length must be more than 0, passed -38'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/location.rb:14:in `initialize'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:78:in `new'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:78:in `location_from_range'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:173:in `extract_location'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:61:in `add_lint'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter/comment.rb:7:in `visit_comment'
/home/gpoul/.gem/ruby/2.1.0/gems/sass-3.4.23/lib/sass/tree/visitors/base.rb:36:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:139:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:150:in `block in visit_children'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:148:in `each'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:148:in `visit_children'
/home/gpoul/.gem/ruby/2.1.0/gems/sass-3.4.23/lib/sass/tree/visitors/base.rb:38:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:139:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:150:in `block in visit_children'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:148:in `each'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:148:in `visit_children'
/home/gpoul/.gem/ruby/2.1.0/gems/sass-3.4.23/lib/sass/tree/visitors/base.rb:38:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:139:in `visit'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/linter.rb:41:in `run'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:53:in `run_linter'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:35:in `block in find_lints'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:33:in `each'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:33:in `find_lints'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:19:in `block in run'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:18:in `each'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/runner.rb:18:in `run'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/cli.rb:67:in `scan_for_lints'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/cli.rb:59:in `act_on_options'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/lib/scss_lint/cli.rb:33:in `run'
/home/gpoul/.gem/ruby/2.1.0/gems/scss_lint-0.52.0/bin/scss-lint:7:in `<top (required)>'
/home/gpoul/.gem/ruby/2.1.0/bin/scss-lint:23:in `load'
/home/gpoul/.gem/ruby/2.1.0/bin/scss-lint:23:in `<main>'
Report this bug at https://github.com/brigade/scss-lint/issues

To help fix this issue, please include:
- The above stack trace
- SCSS-Lint version: 0.52.0
- Sass version: 3.4.23
- Ruby version: 2.1.5

Ouch. Can you help narrow this down by perhaps reformatting that if-statement a little and see if there is any change?

@trotzig what I found was that this goes away as soon as you add a new line after the /* but simply inserting a blank doesn't fix it.

Okay, good to know.
I don't have much time to focus on this now, so if you want to take a stab at it, I'd be happy for a PR.

This is caused by https://github.com/sass/sass/issues/2284. I wouldn't wait on a fix, but we can work around it in scss-lint.


Add OORT Devnet (Testnet)

Description

Add OORT DevNet (Testnet)
Backward compatibility

Testing

@molecula451 I see this PR is still in Draft mode. Was that intentional or would you like me to take another look?

hey @jmrossy this was intentinal, re-opening this ASAP, im checking somethings on the metadata, re-tagging you

@molecula451 no rush, take your time :)
I'll keep an eye out for updates here

OORT is already on mainnet, i think this PR could be re-do for testnet


Project Card: Tooltip when hovering over contributor circle

BEFORE STARTING: make sure to coordinate with the developer working on #126 about API response for contributors.
User story: As a user, I would like to see the profile when clicking into it.
Deliverable
Hovering over on the contributor avatar will pop up a tooltip with the following information from github:

Github avatar
Full name
Github user bio
Clickable profile link

Mockup:

Higher definition PDF file:
She's_Coding_Projects.pdf

Hi there


SSL support

Motivation
To have SSL support and be able to use HTTPS. The use cases of swifter library will be much broader.
Our company use swifter to run mock API server in iOS UI tests (https://github.com/treatwell/twuitests), but currently arbitrary loads have to be enabled in iOS app because iOS enforces using SSL connections.
What's in this PR
This PR introduces SSL support using TLS session on Darwin OS (if everything will turn out fine I'll try adding support for Linux OS as well).
Demo optional localhost (if appears at ~/.swifter/localhost.p12) certificate loading is added to SwifterSampleOSX target.
Closes #111 #395
How did I test it?
Using mkcert tool:
mkcert -install
mkcert -pkcs12 localhost <IP_ADDRESS> ::1

and then imported generated .p12 file to System keychain.

Hey, @viktorasl Thanks for taking the time to create this. I'll try to check your PR during this week, can you please in the meantime fix the issues with Danger? Thanks.

I'll be reviewing more soon and I'll try to test it locally with the certificates also.

I'll probably fix everything after more thorough reviews.

Sure, please let me know to continue with the review as it's a little long.

@viktorasl Are you still interested in this PR? Please continue with the rest of the changes to be able to continue reviewing it 🙂.

Hm, miscommunication happened. I was about to apply changes after your full review. But as I understand that I should apply changes now. I'll do that.

Ups yes it was sorry about that ☹. Sounds good please apply the changes in order to continue. Thanks again for your PR!

Any news around this? It would be really nice when SSL support is added soon.

@MartyCatawiki @viktorasl I would be reviewing the latest changes ASAP. Sorry for the delay 😞.

Sorry it took me so long. Fixed according to comments, please check it out

@viktorasl I'll during the week! Thanks for the heads up.

@viktorasl Can you please compile your PR with Xcode 11.1. I'm getting errors trying to compile it in order to continue the review. Thanks.

@viktorasl Can you please compile your PR with Xcode 11.1. I'm getting errors trying to compile it in order to continue the review. Thanks.

Found the issue regarding force cast and added swiftlint rule ignore for that one particular line. Hope it solves the problem.

@viktorasl Hi! Could we use this functionality on iOS? Maybe you can add some examples and instructions to the SwifterSampleiOS?
Hope to see this pull request merged :)

@viktorasl Hi! Could we use this functionality on iOS? Maybe you can add some examples and instructions to the SwifterSampleiOS?
Hope to see this pull request merged :)

I can look into that 👍  not sure if all APIs are supported on iOS.

Hi guys, any update on this feature?

hei, it's that any update for this?

@viktorasl when send 2k data use ssl,it would give a err "errSSLWouldBlock = -9803, / I/O would block (not fatal) /" , and disconnect this session.

@viktorasl this fork commit use ssl can't send data over 16384 bytes,


SVR with timeline series

Is there a way to have my sample set as a date time time series, to say predict stock prices...

You could try to convert time series to numeric values, but you must do it yourself. For now there is no helpers for timeseries (like for example in Pandas: http://pandas.pydata.org/pandas-docs/stable/timeseries.html). PR are welcome :D


Oxa/devfic autodeploy kakh

What does this PR do? Please provide some context
Minor fixes for oxa autodeployment
Where should the reviewer start?
templates/stamp/stamp-v2.json
How can this be manually tested? (brief repro steps)
Deploy stamp specifying a new parameter -DeploymentType as one of
"bootstrap", "upgrade", "swap" ("upgrade" is the default)
What are the relevant TFS items? (list id numbers)
User Story: 100753
Definition of done:

[ ] Title of the pull request is clear and informative
[ ] Add pull request hyperlink to relevant TFS items
[ ] For large or complex change: schedule an in-person review session
[ ] This change has appropriate test coverage

Reminders BEFORE merging

Get at least two approvals
If you're merging from a feature branch into the development branch then "flatten" or "squash" commits
If merging from the development branch into master (or porting changes from upstream) then use github's UI to get review feedback, but use the git command line interface to complete the actual merge.

Reminders AFTER merging

Delete the remote feature branch
Resolve relevant TFS items
(reverse merge) If you merged from the development branch into master then check to see if there are any changes in master that can be merged down to the development branch (like hotfixes, etc). In this case, use github's UI for feedback and the git command line interface for the actual merge.

@kabirkhan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


IndexError causes internal error

If student code has an index error, you get the hint that "this is most likely a bug in wypp".

Fixed with #136


Comments feature fixes

Description
Minor fixes related to the comments feature for the viewer (published) mode
We can ignore the profile image related changes, since those are a part of #4783  as well
Type of change

Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

Manually

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: comments-feature-fixes-1

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
51.75 (0.06)
33.03 (0.1)
29.67 (0.03)
52.22 (0.05)

:green_circle:
app/client/src/comments/utils.ts
92.31 (0.88)
25 (12.5)
66.67 (0)
90.63 (0.97)

:green_circle:
app/client/src/comments/CommentCard/CommentCard.tsx
75.31 (0.15)
69.16 (0.23)
43.9 (0)
76 (0.16)

:green_circle:
app/client/src/comments/inlineComments/AddCommentInput.tsx
72.12 (0.28)
42.5 (0.83)
63.16 (0)
72.22 (-0.51)

:red_circle:
app/client/src/comments/inlineComments/useOrgUsers.tsx
100 (0)
75 (-25)
100 (0)
100 (0)

:green_circle:
app/client/src/components/ads/MentionsInput.tsx
65.79 (0)
45.24 (8.7)
27.27 (0)
63.77 (0)

:red_circle:
app/client/src/constants/messages.ts
71.73 (-0.02)
100 (0)
14.49 (-0.07)
75.78 (0.09)

:green_circle:
app/client/src/constants/routes.ts
82.79 (1.8)
39.02 (4.87)
64.71 (5.89)
80.95 (3.07)

:green_circle:
app/client/src/pages/Editor/ToggleModeButton.tsx
77.01 (2.72)
50 (4.55)
47.37 (0.7)
74.36 (2.05)

:red_circle:
app/client/src/pages/common/ProfileDropdown.tsx
69.23 (1.23)
60 (-15)
0 (0)
69.23 (1.23)

:green_circle:
app/client/src/pages/organization/OrgInviteUsersForm.tsx
40.19 (0.57)
24 (-0.49)
0 (0)
41.75 (0.57)

:green_circle:
app/client/src/sagas/CommentSagas/index.ts
49.14 (1.35)
26.09 (0)
53.33 (3.33)
48.35 (0.57)

:green_circle:
app/client/src/selectors/commentsSelectors.ts
42.42 (0.84)
11.67 (1.06)
40 (0)
34.78 (0.49)

:green_circle:
app/client/src/selectors/themeSelectors.tsx
94.44 (0.32)
100 (0)
80 (0)
92.31 (0.64)

:green_circle:
app/client/src/utils/AppsmithUtils.tsx
54.19 (0)
23.26 (1.31)
40.54 (0)
49.68 (0)

/ok-to-test sha=dea7be0

/ok-to-test sha=89f8598


Set up ElasticSearch indexes for character, analysis, fix proposals, fix

Work plan:

Define whether we need an index "mapping"
Write code to create indexes
Manage access control:

write with access token
otherwise: read-only

Write ElasticSearch handlers in the JsonStore class.
Populate with some content
Create some minimal documentation for querying

Names for indexes:

roocs-char-YYYYMMDD  (Alias: roocs-char)
roocs-???

Aliases easy to create using python put_alias() method


Socket hangs up before handshake response when using Node's http server

Include the following information (where applicable):

What operating system are you running?

Windows 10

What platform/language are you using?

Node.js 7.3.0
uws ^0.12.0

Are you using SSL, if so, how?

SSL is not part of the equation

Can you show me some short code that triggers your issue?

server.js
'use strict';

const WebSocket = require('uws');
const http = require('http');

const wss = new WebSocket.Server({ noServer: true });
const server = http.createServer();

server.on('upgrade', (request, socket, head) => {
  setTimeout(() => {
    wss.handleUpgrade(request, socket, head, () => console.log('connection'));
  }, 100);
});

server.listen(3000, () => console.log('listening on *:3000'));

client-side
Either open a browser to http://localhost:3000 and run the following in the console or use wscat to connect.
const ws = new WebSocket('ws://localhost:3000')

You should get an error about connection closing before handshake response (browser), or socket hanging up (wscat)

I've confirmed that

This only occurs on Windows, could not reproduce on OS X
When uws is replaced with ws, it works.
Starting a server directly with uws also works
e.g.

const wss = new WebSocket.Server({ port: 3000 })

The problem seems to be in attaching to Node's http server on Windows.

Why not use...
const http = require('http');
const WebSocketServer = require('uws').Server;
const server = http.createServer();
const wss = new WebSocketServer({ server: server });

wss.on('connection', function(ws) {
    ...
});

Hmm, that works.
The problem is that I'm facing this while using Primus, and this seems to be how it handles attaching to Node's http server.
https://github.com/primus/primus/blob/0458cfdeaae0e80df310bb97d0a63e8fddd52c3f/transformers/websockets/server.js#L34

#338

If you call handleUpgrade directly without any timeout then I assume it works?

Yep, that's correct

Does the connection close before handleUpgrade is called? What if you increase the timeout to 5 seconds - does the browser get hung up before the timer fires? You can add text logging to track what happens.

The connection gets closed at the same time as handleUpgrade, regardless of the timeout.
Browser remains responsive, error shows up when handleUpgrade is called.
Here's my modified snippet
'use strict';

const WebSocket = require('uws');
const http = require('http');

const wss = new WebSocket.Server({ noServer: true });
const server = http.createServer();

server.on('upgrade', (request, socket, head) => {
  console.log('Upgrade request')
  setTimeout(() => {
    console.log('Handling upgrade')
    wss.handleUpgrade(request, socket, head, () => console.log('connection'));
  }, 5000);
});

server.listen(3000, () => console.log('listening on *:3000'));

If you also print socket._handle.fd right before handleUpgrade that would be nice, last question :wink:

It says -1

Unfortunately this is an issue with nodejs. I have reduced the bug down to this:
'use strict';

const uws = require('./dist/uws');
const http = require('http');
const server = http.createServer();

server.on('upgrade', (request, socket, head) => {
  console.log('Got upgrade request');

  // this duplicates the socket
  uws.native.transfer(socket._handle, null);

  // destroying here works fine, leaves duped socket alive
  // socket.destroy();

  setTimeout(() => {
    // destroying here closes the duped socket
    socket.destroy();
  }, 2000);
});

server.listen(3000);

Calling socket.destroy directly from the upgrade callback works as it should, it leaves the duped socket as is but calling socket.destroy from within the timer 2 seconds later causes the duped socket to close. They have varying behavior in their destroy function depending on where it is called from which is strange.
I don't know if socket.destroy sends FIN in some cases and thus causing the client to close or if it is something else but the main issue is within nodejs itself. They need to either fix this (which they probably will refuse to do) or provide a documented way of transferring sockets (which they probably will refuse to do).
Nodejs camp is not very easy to work with you know.

Hmm, I found your issues on the topic
https://github.com/nodejs/node/issues/7627
https://github.com/nodejs/node/issues/8974
Anyways, thanks for the help

Yes exactly. 7627 is already worked around and works but 8974 is what is needed here (to officially support what I do). They don't like what I do, though.

Ah, splendid, I'm blocked from commenting or filing issues in nodejs camp. Well that's perfect then.

I did some further digging and this is the flow:
socket.destroy() calls socket._handle.close() which calls uv_close on the uv_tcp_t which calls (at least) close() on the native SOCKET handle. I can trigger the problem by calling uv_close directly, but not by calling close() directly, which is good news because then the problem is somewhere between uv_close and close.

According to @lpinca, your example is slightly off because the 2nd socket.destroy() apparently happens on the same tick as uws.native.transfer()

Fixed it. Apparently libuv has its own logic for duping sockets that misbehaves if the uv_tcp_t is closed (uv_close) without having a specific "duplicated socket" flag set. Setting it, handle->flags |= UV_HANDLE_SHARED_TCP_SOCKET; fixes the problem.
Should probably swap over to using the (undocumented) uv_tcp_duplicate_socket function instead.

Fixed in 24ad90c689fef123675f7a8cf9145a6e9f851371 should be in v0.13.0


Azure: Fix : BVT-HOT-ADD-DISK-SERIAL.  #57

Logs:
[LISAv2 Test Results Summary]
Test Run On           : 12/28/2018 09:41:53
ARM Image Under Test  : OpenLogic : CentOS : 7.5 : 7.5.20180815
Total Test Cases      : 1 (1 Pass, 0 Fail, 0 Abort)
Total Time (dd:hh:mm) : 0:0:16
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-HOT-ADD-DISK-SERIAL                                            PASS                11.41

Logs:
[LISAv2 Test Results Summary]
Test Run On           : 12/31/2018 04:40:21
ARM Image Under Test  : OpenLogic : CentOS : 7.5 : 7.5.20180815
Total Test Cases      : 1 (1 Pass, 0 Fail, 0 Abort)
Total Time (dd:hh:mm) : 0:0:16
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-HOT-ADD-DISK-SERIAL                                            PASS                11.84

is it better to put this case to FunctionalTests-Storage.xml? it should not be a BVT case

Logs:
[LISAv2 Test Results Summary]
Test Run On           : 01/02/2019 08:01:17
ARM Image Under Test  : OpenLogic : CentOS : 7.5 : 7.5.20180815
Total Test Cases      : 1 (1 Pass, 0 Fail, 0 Abort)
Total Time (dd:hh:mm) : 0:0:15
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-HOT-ADD-DISK-SERIAL                                            PASS                11.63

is it better to put this case to FunctionalTests-Storage.xml? it should not be a BVT case

Moved to FunctionalTests-Storage.xml

[LISAv2 Test Results Summary]
Test Run On           : 01/04/2019 11:03:19
ARM Image Under Test  : OpenLogic : CentOS : 7.5 : 7.5.20180815
Total Test Cases      : 1 (1 Pass, 0 Fail, 0 Abort)
Total Time (dd:hh:mm) : 0:0:21
XML File              : TestConfiguration
ID TestCaseName                                                 TestResult TestDuration(in minutes)
1 BVT-HOT-ADD-DISK-SERIAL                                            PASS                15.27

Should not we wait for the lisav2 refactor before the merge?


Fixed header resize observer

I was often having issues with the fixed header overlapping my scrollbar. I did two things to fix this. First, I removed the width: 100% rule from the CSS. Secondly, I set the function that gets the fixed header columns to recalculate their size to trigger not on a window resize, but on a resize observer of the parent element instead. This way if the page layout changes, the header columns will resize even if the page doesn't. As a demo, wrap the vue-good-table element in a div. In your Dev Tools, resize that div. Before this change, the fixed header columns don't resize.
In addition to the above, I fixed the Vetur warnings that a v-if should not be used with a v-for, that instead the v-for should be used on a computed value with the filtered input. I either did the above suggestion, or in one case, I moved a v-if to a template wrapper element.

I'm actually going to split this into two different tickets, with an improvement on the resize observer part.


Chapter02 Sanic Generates Error When Trying to Run src.server:app

sanic is installed in a virutalenv. Within the Chapter02/booktracker folder, I issue this command at the Windows Command prompt:
sanic src.server:app -p 7777 --debug --workers=2
I get this traceback:
Traceback (most recent call last):
File "C:\Program Files\Python310\lib\runpy.py", line 196, in _run_module_as_main
return run_code(code, main_globals, None,
File "C:\Program Files\Python310\lib\runpy.py", line 86, in run_code
exec(code, run_globals)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\Scripts\sanic.exe_main.py", line 7, in
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic_main.py", line 12, in main
cli.run()
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\cli\app.py", line 82, in run
app = self._get_app()
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\cli\app.py", line 135, in get_app
module = import_module(module_name)
File "C:\Program Files\Python310\lib\importlib_init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\Chapter02\booktracker\src\server.py", line 4, in
app = create_app()
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\Chapter02\booktracker\src\utilities\app_factory.py", line 22, in create_app
autodiscover(app, *init_blueprints)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\Chapter02\booktracker\src\utilities\autodiscovery.py", line 28, in autodiscover
app.blueprint(bp)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\app.py", line 527, in blueprint
blueprint.register(self, options)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\blueprints.py", line 357, in register
route = app._apply_route(apply_route)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\app.py", line 393, in _apply_route
routes = self.router.add(**params)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic\router.py", line 137, in add
route = super().add(**params)  # type: ignore
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic_routing\router.py", line 243, in add
group.merge(existing_group, overwrite, append)
File "D:\Development\PyCharmProjects\Python-Web-Development-with-Sanic-main\venv\lib\site-packages\sanic_routing\group.py", line 160, in merge
raise RouteExists(
sanic_routing.exceptions.RouteExists: Route already registered: book [GET]

Thanks for bringing this up! This looks like it is the result of a change between v21.3 and v21.6 (when the chapter was written). Basically what is happening is the autodiscover is trying to load both the Blueprint and the BlueprintGroup.
I will get a correction for this up probably later today.

Still working on a fix for this?

I just pushed a fix. In short, before we add the BlueprintGroup, we make sure to filter out any blueprints attached to it.

Please keep me informed of any more challenges you see. :heart:


[APIM-3.1.0] Gateway Worker Profile doesn't start

Description:
When the latest WUM updated APIM 3.1.0 server starts with the Gateway Worker profile, it keeps lagging due to an OSGI issue occurred due to a missing org.wso2.carbon.identity.configuration.mgt.core-5.17.5.jar.
Steps to reproduce:

Start the server - sh wso2server.sh --optimize -Dprofile=gateway-worker

Affected Product Version:
APIM 3.1.0 - WUM updated

This has been fixed now in the WUM updated APIM 3.1.0 pack with the timestamp<PHONE_NUMBER>746.


CA1707 is firing for discard _ parameter

CA1707 is triggered when the underscore indicates a discard parameter.
https://docs.microsoft.com/en-us/dotnet/csharp/discards
The following causes CA1707 to fire:
public void DoNothing(string _) {  }

@alandraper  Thank you for the feedback.
Unfortunately this a bug and needs to be submitted under Report a Problem in the Visual Studio community.  Please move the issue there to start a dialog with the product team.
@gewarren FYI on CA1707

This looks like it's already been reported on the roslyn-analyzers repo: https://github.com/dotnet/roslyn-analyzers/issues/2440.


Enhancement/Idea : std::ops::Len trait

Much of the types have a len method.
The signature keeps the same: len(&self) -> usize.
Why not make it generic and put into a universal trait?
Maybe this can be a new operator like the # operator of lua.
I've tried to implement it but there is much documentation and compatibility macros and I really don't want to break nothing, then I will send with this a little of what can be made. Basically I created the trait in one place and changed the len method's impl block  and put it inside the impl std::op::Len for $type in every object that has the method.
Here is the patch to demonstrate the idea:
From 8e39d16212d71804e9d3f7657bbb5bb5cca8438c Mon Sep 17 00:00:00 2001
From: lucasew<EMAIL_ADDRESS>Date: Sun, 3 Nov 2019 22:29:11 -0300
Subject: [PATCH] Trying to implement std::ops::Len trait

---
 src/libcore/ops/mod.rs   |  1 +
 src/libcore/ops/vec.rs   |  3 +++
 src/libcore/slice/mod.rs |  9 +++++----
 src/libcore/str/mod.rs   | 10 ++++++----
 4 files changed, 15 insertions(+), 8 deletions(-)
 create mode 100644 src/libcore/ops/vec.rs

diff --git a/src/libcore/ops/mod.rs b/src/libcore/ops/mod.rs
index 5d1d3efd412..480ab7284f3 100644
--- a/src/libcore/ops/mod.rs
+++ b/src/libcore/ops/mod.rs
@@ -154,6 +154,7 @@ mod index;
 mod range;
 mod r#try;
 mod unsize;
+mod vec;

 #[stable(feature = "rust1", since = "1.0.0")]
 pub use self::arith::{Add, Sub, Mul, Div, Rem, Neg};
diff --git a/src/libcore/ops/vec.rs b/src/libcore/ops/vec.rs
new file mode 100644
index<PHONE_NUMBER>0..5cfaa4edffd
--- /dev/null
+++ b/src/libcore/ops/vec.rs
@@ -0,0 +1,3 @@
+pub trait Len {
+    fn len(&self) -> usize;
+}
diff --git a/src/libcore/slice/mod.rs b/src/libcore/slice/mod.rs
index cdada1252d2..ae3e182fd8f 100644
--- a/src/libcore/slice/mod.rs
+++ b/src/libcore/slice/mod.rs
@@ -48,10 +48,7 @@ mod sort;
 //
 // Extension traits
 //
-
-#[lang = "slice"]
-#[cfg(not(test))]
-impl<T> [T] {
+impl std::ops::Len<T> for [T] {
     /// Returns the number of elements in the slice.
     ///
     /// # Examples
@@ -70,7 +67,11 @@ impl<T> [T] {
             crate::ptr::Repr { rust: self }.raw.len
         }
     }
+}

+#[lang = "slice"]
+#[cfg(not(test))]
+impl<T> [T] {
     /// Returns `true` if the slice has a length of 0.
     ///
     /// # Examples
diff --git a/src/libcore/str/mod.rs b/src/libcore/str/mod.rs
index f67012d8f2f..ea439e84079 100644
--- a/src/libcore/str/mod.rs
+++ b/src/libcore/str/mod.rs
@@ -2069,10 +2069,8 @@ fn slice_error_fail(s: &str, begin: usize, end: usize) -> ! {
            index, ch, char_range, s_trunc, ellipsis);
 }

-#[lang = "str"]
-#[cfg(not(test))]
-impl str {
-    /// Returns the length of `self`.
+impl std::ops::Len for str {
+   /// Returns the length of `self`.
     ///
     /// This length is in bytes, not [`char`]s or graphemes. In other words,
     /// it may not be what a human considers the length of the string.
@@ -2093,7 +2091,11 @@ impl str {
     pub const fn len(&self) -> usize {
         self.as_bytes().len()
     }
+}

+#[lang = "str"]
+#[cfg(not(test))]
+impl str {
     /// Returns `true` if `self` has a length of zero bytes.
      E.g., could any function take advantage of an impl Len argument?

I was about to ask exactly what @kennytm ask. Imagine a function such as:
fn do_something<T>(x: T) where T: Len {
  // …
}

What do you think you’re going to do in that function?

This trait can even turn into a new operator, like #x in lua to get the length of the x variable.
Why not?

@lucasew seems unnecessary to have an operator for such a short functino

This trait can even turn into a new operator, like #x in lua to get the length of the x variable.
Why not?

When adding a feature, the question should never be "why not?".
The question should be "why yes?".
There are countless (valid or not so valid) reasons and arguments for adding a feature, e.g. an operator, to the language. Everyone programs in a slightly different style, and there are things and patterns that people like and use often or at least sometimes.
Turning every pattern that has some justification into a new language feature would lead to complexity and bloat of a disastrous extent. Features must add significant value and be coherent with the rest of the language in order to be worth implementing.
A (subjectively) valid reason for a length operator might be that coming from Lua, you find it familiar. However, it provides no such advantage for all the other people coming from one of the literally thousands of other programming languages that are not Lua. Whereas the current alternative – calling .len() – is perfectly clear and usable, without any downsides compared to an operator.

I was about to ask exactly what @kennytm ask. Imagine a function such as:
fn do_something<T>(x: T) where T: Len {
  // …
}

What do you think you’re going to do in that function?

For example, allocate some space according to T and length. It can be useful in some very specific cases (like VM).

But you can't use that to allocate space, because you don't know what T contains. It could be Vec<i32> or HashMap<String, SomeLargeStruct>. Len would only tell you how many elements it contained, and that isn't enough information to allocate memory.

Yes it is, because "allocate the space for T" needs size_of::<T>() bytes no matter the value. And allocating the space for N values of T is size_of * N.

No it wouldn't. For example, let's say we passed in a Vec<i32>.
// this
fn do_something<T: Len>(x: T) {
  // …
}
// would become
fn do_something(x: Vec<i32>) {
  // …
}

So you would be allocating x.len() count of Vec<i32>, when you wanted to allocate x.len() count of i32. This api doesn't give you enough information to be useful.

Oh, I forgot that the container is T, and rust doesn't have a way to talk about T. Haskell moment, sorry.

We could extend the Len trait a bit,
trait Len {
    type Value;

    fn len(&self) -> usize;
}

impl<T> Len for Vec<T> {
    type Value = T;

    fn len(&self) -> usize {
        self.len()
    }
}

This way you could use it for allocations,
fn do_something<T: Len>(x: T) {
  let ptr = malloc(std::mem::size_of::<T::Value>() * x.len());
}

But I don't think this is a good enough motivation, because you can't trust Len::len in unsafe code, so you can't use it to do anything useful with the allocation. On that note, what would you do with the allocation? @tema3210

In terms of allocation it is wrong outside of Vec<T>, e.g. HashSet<T> needs to allocate at least size_of::<Option<T>>() * hash_set.capacity() bytes.
If the purpose is to collect a HashSet<T> into an array of T, there is already Iterator::size_hint().

The std::ops::Len trait should also implement an is_empty function like clippy suggests.
pub trait Len {
    fn len(&self) -> usize;
    #[inline]
    fn is_empty(&self) -> bool {
        self.len() == 0
    }
}

I'm going to close this because std currently has the convention of not making traits for things.  There are a great many things in the library that are common between multiple types that don't have traits.  For example, there's no trait for .clear() or .wrapping_add(x) either.  I see no strong reason that .len() would break that trend -- and you can always make your own trait and implement it for the std things if you so wish.  (Like the num crate does for numerics, for example.)


MD024/no-duplicate-heading to flag errors only when there are duplicate titles under the same top level

After upgrading to markdownlint-cli2 to v0.11.0, I got error:
README.md:594 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Simulation"]
README.md:614 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Embedded, IoT, Hardware"]
README.md:642 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Computer Vision"]
README.md:702 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Cloud Native"]
README.md:756 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Testing"]
README.md:762 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Ops"]
README.md:788 MD024/no-duplicate-heading Multiple headings with the same content [Context: "## Web"]
README.md:802 MD024/no-duplicate-heading Multiple headings with the same content [Context: "### Security"]

However, take ## Simulation for example, it is under multiple first level header.
# Tech Stack
## Simulation
xxx

# Highlights
## Simulation
yyy

Also, they have different contents.
I'm wondering if the "MD024/no-duplicate-heading" linter should only throw an error when there's a duplicate title under the same top level. Would considering that scenario as the standard for duplication be better? Thanks! ☺

This scenario is supported today, please have a look at the documentation: https://github.com/DavidAnson/markdownlint/blob/main/doc/md024.md

Thanks @DavidAnson ! ☀
I should have checked the document first.


Not showing any articles in WSL

Describe the bug
The VS Code extension works great in Windows environment, but when I set VS Code to remote to WSL, I can't get any article.

I didn't set up Hexo: Hexo Project Root, and I have tried reinstall the extension.
Plugin version(please complete the following information):
v0.1.21

I do

I do

Do you want to contribute this feature ? If yes, that should be very thankful. because recently, I don't have enough time to do this.

@cwxyz007 我同时也想试试，我估计是路径问题。不过我一直写python的，希望自己能帮上点点吧


Length is flakey and incorrect sometimes after flattening:

When I run
from nested_ragged_tensors.ragged_numpy import JointNestedRaggedTensorDict
tensors = {
    "code": [[1,2],[3,4],[5,6],[7,8,9,10],[11,12]],
    "time": [0,1,2,3,4],
}
data = JointNestedRaggedTensorDict(raw_tensors=tensors)

print(len(data.flatten()))

When I run this 10 times in a jupyter notebook (restarting the session after each run) I get
$12$ (the expected length)  6/10 runs and $1$ 4/10 runs.

I believe this is because the 0-dim bounds are being retained when they should be dropped, and that is a one element list that is sometimes the first key in the unconstrained order of dictionary key iteration. That's why it is roughly 2/3 of the time it is right (if the code or time key are first) and 1/3 of the time wrong (if the bounds key is first)


Error in renderPlot({ : could not find function "renderPlot"

Hi there, can someone explain to me if:
Error in renderPlot({ : could not find function "renderPlot"
is possible. Even after:
install.packages( "shiny" )
install.packages( "flexdashboard" )
No matter what I do I have these errors which all resources suggest is because I don't have Shiny installed but I believe I do.

Add "library(shiny)" to the first code chunk in the lab, along with "library(Lahman)" and "library(dplyr)"~


remove current day background color

is there any way to remove current day background color? i dont want to change it to transparent because it will overwrite the event indicator.

You mean the currently selected day when you tap and select a day you don't want to highlight it? OR the today highlight that is inly higlighlted for today's date?
Depending on what you what, you can take a fork of the code. Import the library to your project and simply delete some lines of code, like the below:
https://github.com/SundeepK/CompactCalendarView/blob/master/library/src/main/java/com/github/sundeepk/compactcalendarview/CompactCalendarController.java#L870

thankyou for the answer, it works.

@Override
public void onMonthScroll(Date date) {
String monthDate = getMonthyear(date);
tvMonth.setText(monthDate + "");
if (date.getMonth() != new Date().getMonth()) {
monthCalendar.setCurrentSelectedDayBackgroundColor(getResources().getColor(R.color.transparent));
} else {
monthCalendar.setCurrentDate(new Date());
monthCalendar.setCurrentSelectedDayBackgroundColor(getResources().getColor(R.color.colorPrimary));
}
        }


Integration tests for minimina

Problem
MiniMina is starting to get pretty complex and needs some integration tests to ensure command changes/additions don't introduce any unexpected behavior.
Acceptance Criteria
Integration testing that spins up a network with and without archive nodes and covers the basic commands we implement.
Tasks

[ ] Documentation/README to reflect the changes
[ ] implement

There were some basic integration tests added in https://github.com/MinaFoundation/minimina/pull/83.


Can't scroll down on desktop after closing slideout menu

When I toggle the slideout menu closed on desktop (Mac OSX 10.10.4, Chrome Browser) I can no longer scroll down. Works fine on touch devices.

Hi @InTheScript,
Could you share an example? I tested on https://mango.github.io/slideout/ and it works fine.
Thanks.


HamburgerButtonInfo.Visibility doesn't work if ButtonType=Command

When setting ButtonType to Command visibility isn't changed depending on Binding.
When removing ButtonType -- everything works. Unfortunately, it's essential for Command to be removed when needed.

Not good. Not at all.

Added repro, just tap primary buttons to see that Secondary toggle appears, and secondary command doesn't.
HumburgerButtonInfoTest.zip

Just an untested idea, maybe there is a Mode=OneWay missing in HamburgerMenu.xaml.cs, line 218, at the binding for Visibility of the Button in NavCommandButtonTemplate.
Visibility="{x:Bind Visibility}">

should be
Visibility="{x:Bind Visibility, Mode=OneWay}">

Thanks, @carstenschuette !
It looks like that was the reason. I changed it to test and it worked.

Resolved in NuGet preview package. Can you test?

@JerryNixon it seams to be working correctly


vcf corrupt

I'm using the gatk joint calling pipeline. Looks like some corrupt vcf's are sneaking through. Similar issue to #771
# first error
[2015-06-28T06:36Z] scg3-0-1.local: [E::hts_idx_push] unsorted positions
[2015-06-28T06:36Z] scg3-0-1.local: Uncaught exception occurred
Traceback (most recent call last):
  File "/srv/gsfs0/projects/ashley/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 21, in run
    _do_run(cmd, checks, log_stdout)
  File "/srv/gsfs0/projects/ashley/apps/bcbio/anaconda/lib/python2.7/site-packages/bcbio/provenance/do.py", line 95, in _do_run
    raise subprocess.CalledProcessError(exitcode, error_msg)
CalledProcessError: Command 'set -o pipefail; /srv/gsfs0/projects/ashley/apps/bin/tabix -f -p vcf /home/dwaggott/scratch/bcbiotx/f
[E::hts_idx_push] unsorted positions
tbx_index_build failed: /home/dwaggott/scratch/bcbiotx/f8f1c431-ede2-40e8-83ff-9a4fa36f0f05/tmpF8B95F/aaa_12888610
' returned non-zero exit status 1

# looks to be a bad vcf
$ bcftools view bcbio/tk_gatk_joint/work/gatk-haplotype/5/aaa-5_128886102_144562715.vcf.gz | grep 136836378
[vcf.c:1723 _vcf_parse_format] Number of columns at 5:136836378 does not match the number of samples (0 vs 1).

# deleted the vcf and restarted
  File "/srv/gsfs0/projects/ashley/apps/bcbio/anaconda/lib/python2.7/site-packages/IPython/parallel/client/asyncresult.py", line 104, in get
    raise self._exception
IPython.parallel.error.CompositeError: one or more exceptions from call to method: concat_variant_files
[119:apply]: CalledProcessError: Command '/srv/gsfs0/projects/ashley/apps/bin/gatk-framework org.broadinstitute.gatk.tools.CatVariants -R /srv/gsfs0/projects/ashley/apps/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa -V /srv/gsfs0/pr
INFO  13:56:15,787 HelpFormatter - -------------------------------------------------------
INFO  13:56:15,790 HelpFormatter - Program Name: org.broadinstitute.gatk.tools.CatVariants
INFO  13:56:15,795 HelpFormatter - Program Args: -R /srv/gsfs0/projects/ashley/apps/bcbio/genomes/Hsapiens/GRCh37/seq/GRCh37.fa -V /srv/gsfs0/projects/ashley/dwaggott/projects/tk/results/bcbio/tk_gatk_joint/work/gatk-haplotype/LP6
INFO  13:56:15,798 HelpFormatter - Executing as<EMAIL_ADDRESS>on Linux 2.6.32-504.16.2.el6.x86_64 amd64; Java HotSpot(TM) 64-Bit Server VM 1.7.0_03-b04.
INFO  13:56:15,798 HelpFormatter - Date/Time: 2015/06/28 13:56:15
INFO  13:56:15,798 HelpFormatter - -------------------------------------------------------
INFO  13:56:15,799 HelpFormatter - -------------------------------------------------------
.........10.........20.........30.........40.........50.........60.........70.........80.........90.........100.........110.........120.........130.........140.........150.........160.........170.........180.......##### ERROR ----
##### ERROR stack trace
htsjdk.samtools.FileTruncatedException: Premature end of file
        at htsjdk.samtools.util.BlockCompressedInputStream.readBlock(BlockCompressedInputStream.java:382)
        at htsjdk.samtools.util.BlockCompressedInputStream.available(BlockCompressedInputStream.java:127)

# looping over all vcfs I find some additional files
for i in `cat results/bcbio/tk_gatk_joint/work/gatk-haplotype/aaa-files.list`; do echo $i; bcftools view $i | wc -l; done;

[vcf.c:1166 bcf_write] Broken VCF record, the number of columns at X:363136 does not match the number of samples (0 vs 1).

Daryl;
Sorry about the issues. This should be separate from #771 since you're using GATK, so this uses a totally different set of programs for creating and merging the VCFs. It appears like some of the variant call files from GATK are truncated/invalid but it's hard to tell why from this debugging info. A couple of thoughts/suggestions:

Are you using the latest GATK?
When you remove the problem files and re-run, do the same files have issues or do they build cleanly?
If you look at the problem files manually, can you spot the issues?

If we can get more of a sense of what exactly is going wrong we can try to look for it or work around the issues. Hope this helps some.

Yup, it should be a separate issue. I wasn't sure if there was a common vcf validation step.

GATK is up to date and the version is being recognized by the the yaml / logs
I removed the problem files and so far it seems to run fine. Stochastic not deterministic.
Yes, bcftools gets all prissy on generally one line where the sample column is missing. Could be a cluster i/o thing but I've got no clue.

How's the vcf verification code? For fault tolerant-ness, this seems like a worthwhile effort.

Daryl;
Thanks for the additional details. That's a tough one, it sounds like there is some kind of intermittent write error on the filesystem since you're getting truncated lines with no error messages. Short of running bcftools on every output to double verify it, I'm not sure of a good way to detect and avoid this. I don't want to add too much overhead on what appears to be a filesystem issue. If you can come up with any way to reproduce/detect this during runs I'd be happy to add that in. Glad it's running cleanly for you now and hope it finished okay.

Not sure what a reasonable solution is. However, I just ran into the same problem on a completely different cluster using freebayes instead of gatk. A single line in a single vcf is truncated and kills the whole pipeline at the catvariants step.
My vote is for some vcf validating, maybe a fault tolerant mode.
What about only running the vcf validation if catvariants errors out?

Daryl;
Sorry about the problems on other systems as well. I'm not exactly sure what is going on and will try to reproduce and see what we can come up with for checking. Unfortunately by the time we get to the catvariants step it's going to be difficult to unwind and go back to the previous processes. Would you be able to pass along one of the problem files (off list is fine) and I can try to see if I can come up with any ideas to validate them cheaply? Thanks much.

Thanks for re-opening, I'll rerun everything to see what pans out. I deleted all the problem vcfs so can't send much for diagnostics.
What do you think... if I delete the vcf that was problematic in *-joint-files.list and restart the pipeline will the pipeline understand? Or do I need to do more to roll back the pipeline to one of the transactional points.
Thanks!!!

Daryl;
If you delete the files and re-start the pipeline will pick up right where it left off. You do want to remove checkpoints_parallel/full.done so it'll re-parallelize the variant calling but otherwise you shouldn't need to change anything. If you get any additional details/examples I'm happy to explore more.

Thanks Daryl, closing this out for now since it seems like it might have been a one off issue and the actual files that were failing were deleted so we can't do much forensics. Feel free to reopen it if I'm wrong or there is still a problem going on.


[ceres] build failure when using cxsparse option

Hi,
I just got an error when trying to build ceres with cxsparse option. Log file attached.
Host Environment

OS: Windows
Compiler: Microsoft (R) C/C++ Optimizing Compiler Version 19.29.30146 for x64

To Reproduce
Steps to reproduce the behavior:
./vcpkg install ceres[cxsparse]
Failure logs
D:\Dev\sight-vcpkg\vcpkg\buildtrees\ceres\src\1e64f72f66-a013de02a8.clean\internal\ceres/cxsparse.h(46): fatal error C1083: Cannot open include file: 'cs.h': No such file or directory

Additional context
install-x64-windows-dbg-out.log

BTW, the support for CXSparse has been removed from upstream Ceres, so anyway, the support should be dropped for the next Ceres release


Does Xplat work under Microsoft Release Manager

I would like to know if Xplat works under Microsoft Release Manager.

Yes.  Version 0.3.13 and greater

And it's the new RM vnext

Thx any examples?

I'll send to RM folks and ask for docs/videos etc. but it should work just like build vnext.  You create a def, add tasks and hook up an agent ( or use hosted )
Sent from Outlookhttp://taps.io/outlookmobile
On Sat, Jan 16, 2016 at 11:32 PM -0800, "BuddyWork" <notifications<EMAIL_ADDRESS>wrote:
Thx any examples?

Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/vso-agent/issues/173#issuecomment-172298402.

Here is a youtube video that can help you get started.
https://www.youtube.com/watch?v=82i78IHpl2A
I ll dig in for some docs in the mean time.

Thx for the video link do you have an example how to use xplat just for releases with Microsoft Resource Management 2015 with xNext

Please refer to http://blogs.msdn.com/b/visualstudioalm/archive/2015/11/28/deploy-artifacts-from-onprem-tfs-server-with-release-management-service.aspx for a step by step guide.


refactor: remove old v2 code

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign rawlingsj
You can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

let's wait and build a few more downstream plugins to verify we can remove these functions
/hold

/hold cancel

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: rawlingsj
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Setting the TextField widget's 'textAlign' property to 'TextAlign.center' creates an issue using the space button on the keyboard.

Hi all,
I'm running the Flutter app I'm building on an iOS simulator and I have a build on my phone as well. When I set the TextField's textAlign property to TextAlign.center, pressing space on the keyboard does not result in a space until after I press another key. Any insight into what's going on? My code is below:
              TextField(
                buildCounter: (BuildContext context,
                    {int currentLength, int maxLength, bool isFocused}) => null,
                decoration: InputDecoration(
                  border: InputBorder.none,
                ),
                cursorColor: fontColor,
                cursorWidth: 2,
                maxLines: null,
                style: TextStyle(color: fontColor, fontSize: 24, fontWeight: FontWeight.w700),
                maxLength: 220,
                textAlign: TextAlign.center,
                textInputAction: TextInputAction.done
              )

Hi @sunyatax
Testing this issue doesn't seem to reproduce on Flutter 1.12.13+hotfix.8 • channel stable
If you're still experiencing this issue, please create a new issue and provide the information required in the template.
Thank you


Array[File] doesn't validate correctly.

If you have a parameter of type Array[File], validation always fails because the primitive type coercer is trying to do @coercer[val], and @coercer is ::File.
optional(:nested, type: Hash, default: {}) do
  optional :files, type: Array[File]
end

Version 1.2.5 doesn't have this issue, so 1.3.0 broke parameter type coercion.

Appreciate a spec + fix in a PR. Thanks.


Cannot install on mac

throw err;
^
Error: Cannot find module '/Users/myname/index.js'
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Function.Module.runMain (module.js:441:10)
at startup (node.js:139:18)
at node.js:968:3
logout
Saving session...
...copying shared history...
...saving history...truncating history files...
...completed.

I get this error when starting Discord on OS X 10.11.

This happened on the latest versions of Discord and BD available to me, as far as I can tell.

Getting a similar error when I attempt to install on 10.9.5 Mavericks. Latest Discord app and BetterDiscord versions.

Oh god dammit. It hasn't even been up for a day and people are already hijacking it. V_V Now they'll probably fix your errors and forget all about mine. >_> Why can't people post their own issues?

Dharengo, your issue is that you're not in the right place when executing install.sh
etrastew, Discord is supported for versions 10.10 or plus, you can remove discord and install only discord and try running it, if it works then you can install betterdiscord. a pkg version will be made available soon.
michaelagazzola, try the same thing as etrastew, try a fresh install after removing Discord.app from Applications

@iVEnoVaLue
Hi, I'm getting the same problem as @etrastew and @michaelagazzola.
Removing and reinstalling Discord didn't work (yeah, I've removed ALL files and directories).
https://gfycat.com/DisgustingAmbitiousFoal

https://github.com/iVEnoVaLue/BetterPlugins/raw/master/BetterDiscord.pkg
Version 0.2.7 won't work with the new update. Try using the pkg installer, it would become official one of these days, maybe.

That worked, thanks!

Yup, thanks @iVEnoVaLue!

Works perfectly--thanks! :)

What Do i do when i install it?

What do i do when I install it do I restart (I did.) @iVEnoVaLue


Allow null queue and long_queue

Is your feature request related to a problem? Please describe.
SLURM/scripts/juicer.sh provides "#SBATCH -p $queue" or "#SBATCH -p $long_queue" with every job submission.  At some supercomputing sites requesting an explicit partition is not recommended.  To support such sites, juicer.sh should allow a null $queue and $long_queue.
Describe the solution you'd like
If the user provides no -q or -l arguments, and any site customization does not give values to $queue or $long_queue, then the "#SBATCH -p" directives should not be included in the job submission scripts.
Additional context
Compute Canada clusters (https://docs.computecanada.ca) have a complex partition structure.  Jobs are sorted into appropriate partitions by a submission filter script and by Slurm itself based on run times, memory requests, etc., and recommended practice is that the user does not try to determine the correct partition.
A "queue" in PBS corresponds to a "partition" in Slurm.
I hope to provide a code patch presently.

Sorry, a code patch won't be forthcoming after all.  I was able to patch this up in 'juicer.sh', but then discovered that 'split_rmdups.awk' would also have to be patched.  I've got a fork that works on our systems, but it's a hack.  Sending the changes back upstream would probably cause it to break on Slurm clusters where one should explicitly specify a partition.

Thanks for the update.
On Mon, Nov 16, 2020 at 2:00 PM Ross Dickson<EMAIL_ADDRESS>wrote:

Sorry, a code patch won't be forthcoming after all. I was able to patch
this up in 'juicer.sh', but then discovered that 'split_rmdups.awk' would
also have to be patched. I've got a fork that works on our systems, but
it's a hack. Sending the changes back upstream would probably cause it to
break on Slurm clusters where one should explicitly specify a partition.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/190#issuecomment-728260223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EW2HDWRR26CG5MAPD5TSQFZETANCNFSM4TLUPZSQ
.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Assistant Professor |  Molecular and Human Genetics
Aiden Lab | Baylor College of Medicine
www.aidenlab.org

Dear Neva,
it seems I've ran into the same issue as Ross - certain research HPC at Canadian universities and the CCDB platforms do not allow selecting a partition for the standard user (except for researchers that have contributed hardware). Is there a way you would consider to provide a patch for this issue allowing not to specify partitions? I guess three files would have to be patched: 'juicer.sh', ‘split_rmdups.awk’ and 'mega.sh'.
Many thanks for your consideration.
Martin


created food items

found that after creating any food items  they are not in the store to buy

Yes the infinite HP exploit was removed, for healing items at the shop buy potions


Want only LED notifications, but get also sound and vibration

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
I configured K-9 so it should only notify me via LED. I don't want sound or vibration. (see my screenshot).
Actual behavior
I get all of this notofications. LED and sound and vibration.
Steps to reproduce

Configure like in my screenshot.
Receive an email.

Environment
K-9 Mail version: 5.403
Android version: 7.1.2
Account type (IMAP, POP3, WebDAV/Exchange): IMAP (Posteo.de)
Phone: OnePlus One with LineageOS

We set up the notification as configured. If it's not working as expected it's most likely an issue of the Android version you're using.


Testing CDK version to 1.107.0

Codecov Report

Merging #248 (e78d155) into master (5659a9b) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head e78d155 differs from pull request most recent head 7c933af. Consider uploading reports for the commit 7c933af to get more accurate results

@@           Coverage Diff           @@
##           master     #248   +/-   ##
=======================================
  Coverage   97.29%   97.29%
=======================================
  Files           3        3
  Lines          74       74
  Branches       15       15
=======================================
  Hits           72       72
  Misses          2        2

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 fe3283c...7c933af. Read the comment docs.


storage.buckets.create permission requirement is redundant

Description
The storage.buckets.create permission is required if the selected method is gcs_stage, even if the bucket already exists.
In a real-life scenario, I would like to separate infrastructure concerns from the loading logic. I will create the bucket and assign the service account only the necessary permissions. Since the bucket is already taken care of, I don't feel the need for the service account to be able to create another one.
I believe that this method should check for the existence of the bucket instead of trying to create it and assuming that it already exists only if it fails on a Conflict exception. I understand the logic and in many cases, it wouldn't be a problem, but this approach makes it difficult for me to maintain multiple environments in a single GCP project securely.
Suggested Solution
I presume something like this could work:
if not hasattr(self, "_gcs_bucket"):
    self._gcs_bucket = self.client.get_bucket(self.as_bucket())
    if self._gcs_bucket is None:
        self._gcs_bucket = self.client.create_bucket(self.as_bucket(), location=location)

This way we only require the caller to have storage.buckets.get permission if the bucket already exists.
What do you think?

Yeah this is a great point, I am fine with this. Can do it later or someone else can contrib.

Thanks, I've prepared a PR #67. Let me know if I should change anything, happy to contribute.

LGTM, thanks for the contribution!


Trivy scan started failing recently for html format

Hello,
Recently trivy scan started failing for format template, specifically for html.tpl.
Few days back it was working fine.
Here is the relevant details I'm using in the workflow
     uses: aquasecurity/trivy-action@master
      with:
        scan-type: image
        format: 'template'
        template<EMAIL_ADDRESS>        output: 'trivy-results.html

Getting error: Fatal error	report error: unable to write results: failed to initialize template writer: error retrieving template from path: open /contrib/html.tpl: no such file or directory
Error: Process completed with exit code 1.

same here

@DmitriyLewen I would assume this is because the templates used to exist in the Trivy docker image but not by default in the composite action?

@simar7 you are right.
install script doesn't copy contrib dir...
I can copy contrib dir in setup-trivy.
But I can't select better best to do this.
By default we install Trivy into $Home/.local/bin/trivy-bindirectory. I can keepcontrib` in this dir, but i am not sure that this is user friendly.
@simar7 wdyt?

@simar7 you are right. install script doesn't copy contrib dir... I can copy contrib dir in setup-trivy. But I can't select better best to do this. By default we install Trivy into $HOME/.local/bin/trivy-bin directory. I can keep contrib in this dir, but i am not sure that this is user friendly.
@simar7 wdyt?

Yes I understand. I think it's the best option for now.


Add exchange message dispatch support for TCP if needed

Problem
The current ApplicationExchangeDispatch is assuming unreliable transport, as CHIP is using UDP for application messages. The code can be refactored to support TCP when such a need arises.

Will confirm, if spec requires all app messaging run over only on UDP, then we should update all examples to disable TCP option.


Use go routines for containers list

This is just a showcase to show  go routines give nothing on that.

Marking as blocked since we don't want this merged :)

Closing this as I believe we've got the point and obviously won't be merging something that doesn't actually improve performance :)
Sounds like we'll have to do some proper benchmarking to figure out where the bottleneck is.


Stopping a thread

Is there any way to stop a running thread, like thread:stop()?

No. As I  know it is possible do for pthreads on *nix. But there not safe way to do this on Windows.


[11.x] Fix: Prevent invalid AWS credentials options being created

Improves on 909ea24597b86cfc7a49ce41074280d0347bf6a3 which could create credentials arrays with only a token

An upstream bug in https://github.com/brefphp/laravel-bridge was causing credentials to be generated for AWS services with just a token and no key/secret. It's fixed here: https://github.com/brefphp/laravel-bridge/pull/165, however I thought it would be good to prevent it in the Laravel source as well.
Only the code on src/Illuminate/Filesystem/FilesystemManager.php and src/Illuminate/Cache/CacheManager.php is incorrect in their current behaviour, the others are updated for consistency.
If preferred I can remove the changes to:

src/Illuminate/Bus/BusServiceProvider.php
src/Illuminate/Mail/MailManager.php
src/Illuminate/Queue/Connectors/SqsConnector.php
src/Illuminate/Queue/QueueServiceProvider.php


python37Packages.bugwarrior: 1.7.0 -> 1.8.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python37Packages.bugwarrior is: "Sync github, bitbucket, bugzilla, and trac issues with taskwarrior"
meta.homepage for python37Packages.bugwarrior is: "https://github.com/ralphbean/bugwarrior"
meta.changelog for python37Packages.bugwarrior is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.8.0 with grep in /nix/store/pm66nkqszj8smsvv8i519xbg7q3j3qns-python3.7-bugwarrior-1.8.0

directory tree listing: https://gist.github.com/a3a744894ff694345aee3d00c18cd0e4

du listing: https://gist.github.com/7243ad25f37d0842c86878eb83b383a2

Rebuild report (if merged into master) (click to expand)

9 total rebuild path(s)

3 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
3 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python37Packages.bugwarrior
python38Packages.bugwarrior
python39Packages.bugwarrior

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pm66nkqszj8smsvv8i519xbg7q3j3qns-python3.7-bugwarrior-1.8.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python37Packages.bugwarrior https://github.com/r-ryantm/nixpkgs/archive/9d142b4a5edf71f60d88516fd8e94b8616628388.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pm66nkqszj8smsvv8i519xbg7q3j3qns-python3.7-bugwarrior-1.8.0
ls -la /nix/store/pm66nkqszj8smsvv8i519xbg7q3j3qns-python3.7-bugwarrior-1.8.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python37Packages.bugwarrior
python38Packages.bugwarrior

Maintainer pings
cc @nbp @yurrriq for testing.

I can test Tuesday.

I can test Tuesday.

err.. tomorrow, I swear :angel:

it's not a major version bump, but it would hlep


purple screen of death

A few minutes after long loading ferdi became purple screen and stay like that even hours.
using on win 10, cant check ferdi version but I keep it update.
version might be last beta, last actions was to try the nextcloud recipes

version might be last beta, last actions was to try the nextcloud recipes

@schapira - did you use Ferdi with a login - if so, was it franz server or ferdi server?

Delete your ferdi profile on your local machine - if you were using Ferdi with a server, then all your profile data will get resurrected.
If it is with a server, I would suggest that you try to login and edit your profile/saved services - to remove the ones that you think might be the cause of the issue.

Please report back as to which step worked - and after that, we can try to check if there's any other debugging / help that is needed.

possible duplicate of #873 ?

For me this worked.
Move the entire Ferdi folder in appdata to a different location.
Relaunch Ferdi.
Replace the new Ferdi folder with the old one.

@schapira - do you have any updates on the workarounds suggested above?

Closing due to inactivity.


black/white repertoire

at the moment the tool works mainly to create a white reportoire. Handle properly the black side

no it's not. There is the Color field in the configuration to specify for which color the repertoire has to be obtimized.


should make csv filename customizable instead of only route name

should allow to customize csv file name
example:
return $ssp->response()->csv("my-file-20230923");

released in 3.6.0


Addedd possibility to authenticate with client certificate

Added functionality to authenticate via client certificate stored in personal store specified by its thumprint.

Updated packages

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: nicolonsky sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@nsoy @rohitramu or anyone at MSFT, please process this request and update the module - it's been 1.5y since it was made and is a really wonderful added feature to further support Azure Automation.

@nsoy @rohitramu or anyone at MSFT, please process this request and update the module - it's been 1.5y since it was made and is a really wonderful added feature to further support Azure Automation.

Can I second that request please? Trying to use in Azure Automation currently and this would be really useful.

@nsoy @rohitramu or anyone at MSFT, please process this request and update the module - it's been 1.5y since it was made and is a really wonderful added feature to further support Azure Automation.

Ping :)

FYI since people are obviously interested - I had to open a support case for my issue (https://github.com/microsoft/Intune-PowerShell-SDK/issues/115) and I queried them on the lifecycle of this module while I was at it. The response was not overly promising, simply indicating they 'can't comment on it currently'.
So....don't get your hopes up?


[C#/GDScript] Addition of _PreDelete() method in Node API for de-init code

I think it would be beneficial to add support into API for a function override where you would be able to put all the de-init code of the script/autoloaded script. The current implementation looks rather ugly:
public override void _Notification(int what) { if(what == NotificationPredelete) { ...de-init code... } }
and relies on notification usage that I never had to touch until this very moment.

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Remove bolding from "beginner's guide..."

Fixes https://github.com/ambuda-org/ambuda/issues/157

Is this the template you mean to fix? It's not the one you mention in #157.

And to take a step back, I agree that this is confusing UX, but I want to make sure the root problem still remains solved -- that is, we should draw a new proofer's attention to this guide so that they at least skim it and familiarize themselves with what it contains.

Yeah, you're right, I edited the wrong page.
I think from a UX standpoint, what if we did this:

always show the list of proofing pages ("beginner's guide to proofing", etc.), so it basically becomes like a menu / navigation section -- the contents of each help guide / proofing-related page will be below it. The current page the user is on should be bolded in the nav menu.
add a link called "Ongoing projects" that shows the ongoing proofing projects
By default, show the "beginner's guide to proofing" page. (maybe after the user has opened "ongoing projects", then this page becomes the default one shown at first?)

that sounds great to me!


[Protection] Unreal Engine

https://store.steampowered.com/app/1901330/Reflex_King_of_Elements/
Incorporates 3rd-party DRM: Unreal engine
1 PC (concurrent use) machine activation limit
I think you can just put anything in that field on steam but worth looking into at least?

It seems likely that this would be referring to an Epic Games protection, and not one tied to Unreal Engine itself. Though that being said, I'm not about to spend $5 on this game to find out for sure.