pip install cashu in python venv failing on mac

pip install cashu runs for a while and fails with the following traceback
Building wheels for collected packages: coincurve
  Building wheel for coincurve (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for coincurve (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [72 lines of output]
      running bdist_wheel
      running build
      running build_py
      creating build
      creating build/lib.macosx-14.0-x86_64-cpython-312
      creating build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/_windows_libsecp256k1.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/flags.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/__init__.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/keys.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/types.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/context.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/utils.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/ecdsa.py -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      copying coincurve/py.typed -> build/lib.macosx-14.0-x86_64-cpython-312/coincurve
      running build_clib
      Can't exec "aclocal": No such file or directory at /usr/local/Cellar/autoconf/2.72/share/autoconf/Autom4te/FileUtils.pm line 299.
      autoreconf: error: aclocal failed with exit status: 2
      Traceback (most recent call last):
        File "/Users/mac/Documents/Work/bitcoin/cashuvenv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 353, in <module>
          main()
        File "/Users/mac/Documents/Work/bitcoin/cashuvenv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 335, in main
          json_out['return_val'] = hook(**hook_input['kwargs'])
                                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/Users/mac/Documents/Work/bitcoin/cashuvenv/lib/python3.12/site-packages/pip/_vendor/pyproject_hooks/_in_process/_in_process.py", line 251, in build_wheel
          return _build_backend().build_wheel(wheel_directory, config_settings,
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 410, in build_wheel
          return self._build_with_temp_dir(
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 395, in _build_with_temp_dir
          self.run_setup()
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 487, in run_setup
          super().run_setup(setup_script=setup_script)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/build_meta.py", line 311, in run_setup
          exec(code, locals())
        File "<string>", line 266, in <module>
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/__init__.py", line 103, in setup
          return distutils.core.setup(**attrs)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/core.py", line 184, in setup
          return run_commands(dist)
                 ^^^^^^^^^^^^^^^^^^
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/core.py", line 200, in run_commands
          dist.run_commands()
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
          self.run_command(cmd)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/dist.py", line 968, in run_command
          super().run_command(command)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "<string>", line 97, in run
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/normal/lib/python3.12/site-packages/wheel/bdist_wheel.py", line 368, in run
          self.run_command("build")
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/cmd.py", line 316, in run_command
          self.distribution.run_command(command)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/dist.py", line 968, in run_command
          super().run_command(command)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/command/build.py", line 132, in run
          self.run_command(cmd_name)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/cmd.py", line 316, in run_command
          self.distribution.run_command(command)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/dist.py", line 968, in run_command
          super().run_command(command)
        File "/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-build-env-prws6a_l/overlay/lib/python3.12/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
          cmd_obj.run()
        File "<string>", line 154, in run
        File<EMAIL_ADDRESS>line 413, in check_call
          raise CalledProcessError(retcode, cmd)
      subprocess.CalledProcessError: Command '['/private/var/folders/yw/f_ms07y17h9bcdq5g7swp2x80000gn/T/pip-install-dsc4dzn3/coincurve_0df94ae4af4543b3bc0227d024a8e379/libsecp256k1/autogen.sh']' returned non-zero exit status 2.
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for coincurve
Failed to build coincurve
ERROR: Could not build wheels for coincurve, which is required to install pyproject.toml-based projects

Steps I carried out

created a python virtual environment i.e python3 -m venv "venvname"
pip install cashu

Environment:
Device: Macbook Pro 2018 2.9GHZ 6 Core intel-core 19
Operating system: MacOS Sonoma 14.5

Can you successfully install from the repo using poetry?


Status of the project?

I'm new to this project.
So can someone explain the status of the project, because I have not seen any activity recently.
Is it not maintained anymore?
Because I have noticed some issues in the right-click menu & other panels & new home page
To solve these issues I would appreciate some documentation If there is any ?

It is still maintained.
If you're running into problems the best thing to do is look through the open issues.  If you see the same problem you are having you are welcome to contribute to the conversation there.  Otherwise you can open a new one with the requested info in the issue template.


Sending move packet with join corrupts userdb file

This
void Bot::join()
    {
        // send join packet
        do_write("join|" + username + "|" + password + "\n");
        // send first move packet
        do_write("move|" + firstMove + "\n");
        // start game loop
        gameLoop();
    }

without specifying a firstMove
makes the user file come out as
name$passwd_hash
move

Why?
No clue.


#107: Add config input parameters for user pool client generate secret

Types of changes

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

And

[x] This Pull Request DOES NOT target the main branch on this repository (if so it must point to
a feature branch or to the release branch).
[ ] I have read the CONTRIBUTING
document.
[ ] I have added/updated tests to cover my changes.
[ ] All new and existing tests passed.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] There are no other open Pull Requests
for the same update/change.

Proposed Changes

closes #107

Codecov Report

Merging #108 (5633c48) into release (1810084) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             release     #108   +/-   ##
==========================================
  Coverage      98.78%   98.78%
  Complexity       173      173
==========================================
  Files              9        9
  Lines            986      987    +1
  Branches          37       37
==========================================
+ Hits             974      975    +1
  Partials          12       12

Flag
Coverage Δ

unittests
98.78% <100.00%> (+<0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...rc/main/java/org/wcdevs/blog/cdk/CognitoStack.java
98.13% <100.00%> (+<0.01%)
:arrow_up:

Continue to review full report at Codecov.

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


.Net 8 and Minimal APIs

When using this in a minimal API setup, under .NET 8, I'm getting a circular reference exception when serialising the JsonPatchDocument, and looking at the Options value it's JsonSerializerOptions with a couple of TypeInfoResolvers (ProblemDetailsJsonContext and DefaultJsonTypeInfoResolver) and the ProblemDetails one (ironically enough) is causing the problem.
Workaround is currently to change that value before serialising, wondering if there is a better solution?

Are you able to provide an example program where the failure occurs, as a zip file or github repository. That should speed up fixing that issue

Hi @Havunen I believe I've found the issue, and whilst it's not with this library directly, it is a case others might hit.
Issue is when using AddProblemDetails() and MassTransit's UseNewtonsoftJsonSerializer() together, the latter being left over from when I was using MS' JsonPatch library.
AddProblemDetails looks to insert a TypeInfoResolver for ProblemDetails, which the JsonPatchDocument now has in its Options property, and UseNewtonsoftJsonSerializer presumably picks the first one, which is now the one injected by AddProblemDetails, which then explodes with the circular reference error.
I imagine there are other combinations like this which might cause the same issue.
I've created an example repo at if you want to take a look. I should have an easy fix by removing UseNewtonsoftJsonSerializer(), but if/when this happens with other libraries, I'm not sure there is a solution to this other than forcing the Options value to the desired TypeInfoResolver.
I'll close this though.


MIM tools - Pull request from Craig's master branch to PnP/Dev branch

Hey Vesa, I hope I am doing this right! I think this is creating a pull request from Craig's master branch, which has all our changes to PnP/Dev branch. Once the code makes it to PnP/Dev branch, you'd have to move it to PnP/Master branch before Monday morning (PST) since that's when we plan to talk to customers.
I know I am asking a favor.. I'd really appreciate if you could help out!

Hi @gdoshi, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@gdoshi, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, MSBOT;


Throw an error if an invalid rootElement is passed to assert.dom()

Resolves #633

Should we change the label to bug instead, based on the issue identifying this as a bug?

based on the issue identifying this as a bug?

I would consider the dom('foo', 'bar') usage itself a bug, but I don't think it's necessarily a bug that we don't catch this issue early 😉

Fair. Seems fine.


Initial liquidity causes market preview to throw

create market with liquidity and go to market preview. UI throws

can't reproduce @bthaile

Still an issue,
In market creation, created scalar market with price range of -10 to -100 with a bid and an ask order then nav to market preview and it exploded.


Update dev comment for periodicity for translation problems in graphical mode

Fixes internal bug [MachineTranslated] Translation problems in graphical mode. - Update dev comment
Description of the changes:

Added definition of period in en-US comment.

How changes were validated:

No need for validation as this is a non-technical issue.

/azp run

/azp run

/azp run


Display Ship Group Sequence ID on Order Detail Page

What is the motivation for adding/enhancing this feature?
On the order detail page, the ship group sequence ID is not displayed, causing a lack of crucial information for order tracking and management. It is essential for users to have access to the ship group sequence ID to accurately identify and manage orders
What are the acceptance criteria?
The ship group sequence ID should be displayed prominently on the order detail page, providing users with the necessary information to track and manage orders effectively.

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information

https://www.figma.com/file/wK5FMs4z2zXQR9VfHV9ycE/Dhiraj's-Ionic-6-Material-UI-Kit-(Community)?type=design&node-id=29258-164395&mode=design&t=chIMtrSgKa5wcmta-4

@Dhiraj1405 Here is the UI link
https://www.figma.com/file/bVPRRw282CqGKMdbz7dciH/HC-Ionic-design-system?type=design&node-id=29707%3A14605&mode=design&t=juWvhZn4LOFHF3ii-1


Visual shaders dont have flow control operators

For now it misses if/else, while and for loops

Indeed, this makes them simpler to make. Most visual shader editors I know work this way.

Do you mean that they are not required and may not be implemented ?

Probably not going to be implemented. Adding flow control ports like Visual Script does would make them  unnecessarily more complex for 99% of the tasks visual shader editing is aimed at.

Ah, true that even UE4 does not have loops inside material editor, but it have if/else (for simple floats). Not a bad idea to add this..


Diagnostic mixing libraries/binaries with incompatible runtimes

A library or binary that requires panic=abort but depends on a library that requires panic=unwind should produce a diganostic.
I don't know whether the reverse case should be fine. That is, a library or binary that requires panic=unwind but depends on a library that requires panic=abort might work just fine. We might still want to prevent mixing different run-times though.

This is actually more of a rustc concern than a Cargo concern (e.g. Cargo has no knowledge if something requires the panic or abort runtime). Perhaps this should be opened against rust-lang/rust?

I think it concerns rustc, cargo, and crates.io. We probably want to catch this at the highest level possible (e.g. if a crate has this in its Cargo.toml file or crates.io meta-data there is probably no need to involve rustc), but we want to be able to catch this at the lowest level anyways.

I don't know what the appropiate wording for such an issue at the rustc level would be. Maybe something "emit a diagnostic when linking/compiling crates with incompatible run-times" ?

Not having dealt with this too much, how does a library declare that it needs panic=unwind?  As far as I'm aware, cargo's only knowledge is through profile which is only respected by the workspace building the root crates.


Finders don't work correctly for ManyToMany bidirectional relationship

This is a know issue in Kundera-2.0.5 release.
When inbuilt secondary indexes are enabled, finder doesn't populate parent entity into child entity.

fixed and uni tested.


New Rule: Single Line Per Property

Add rule to enforce each property (/ @include / @extends) within a selector exists on a single line. As per the existing scss-lint SingleLinePerProperty rule.
Allowed:
p {
  margin: 0;
  padding: 0;
}

Disallowed:
p {
  margin: 0; padding: 0;
}

There should be a allow_single_line_rule_sets option that defaults to true to allow the usage of single-line rule sets, to make the following allowed:
p { margin: 0; padding: 0; }

As an extension to the exisiting functionality I think it could be useful to add an additional configuration option maxPropertiesForSingleLineRuleSet to allow users to limit the amount of properties within a single-line ruleset, defaulting to Infinity. For instance if maxPropertiesForSingleLineRuleSet was set to 1 then p { margin: 0; padding: 0; } would be invalid, but p { margin: 0; } would be acceptable. This would allow people to limit these single-line rulesets to only be used in cases where the number of properties won't affect readability.

Surprised we don't have this one yet!

Ah, dangit, after looking through the existing rules docs, we've already got one-declaration-per-line which does this. Sorry.
Though it looks like it's missing the allow_single_line_rule_sets option, so that'd still be useful to add IMO. Shall I rewrite the title/description or close as invalid and fire up a new ticket? (related: looking back on this it seems my option naming convention is all over the bloody shop, sometimes snake_case, sometimes camelCase, when sass-lint's actual convention is dashed-case)
Is it worth renaming rule to match scss-lint? Or are you not that worried about compatibility.

I think this actually raises a valid point.. I'm wondering why we went with one-declaration-per-line and then single-line-per-selector naming convention, should probably rename the former. It confused me too!
let's close this ticket and fire up a new one if you don't mind.
We should hopefully be getting 1.3.0 out this weekend and there are a LOT of fixes and changes included.

Closing this ticket in favour of:

#288 for discussion of renaming one-declaration-per-line to single-line-per-property
#287 for adding the allow-single-line-rule-sets option to one-declaration-per-line


Add a section about how git works

Closes #1
This PR adds a section that includes links for readers to learn how git works.  It includes a link to a free book and a YouTube video from Scott Chacon, who I believe should be regarded as a git expert.

Perhaps we can an image linking to the video as well so it the README just isn't a wall of text?

go ahead!

Will add image referencing video in a future PR


Add NER PII Masking module

close #314
Use a Hugging Face NER model for PII Masking.

Plus, are you sure NER PII Masking is 'passage filter' node? I think it is not. Passage filter means that 'decreasing the context' while not modifying each contexts content itself. I think we have to make new node... like privacy filter or something...?
Plus, it is tricky to setup metric for privacy filter isn't it?

I agree with you ... I'll give it a little more time and think about it ,,,,


Update IosProject.kt (Removing .iosApp from bundleId)

When creating compose app project, in ios for bundleId it adds .iosApp to the projectID. Is there any special reason for that? This PR removes that .iosApp part from bundleID and keeps it same as given project id.

the reason is to have explicit differences between different app's logs, crash reports etc.


ENYO-3184: Remove srcElement property for generated media events.

Issue
We had previously been providing a srcElement property for our generated media events. This is more of an optional change, and we are not utilizing this property anywhere internally, but in the spirit of being more standards-compliant and consistent, we should move away from this.
Fix
We have removed the srcElement property when generating media events.
Enyo-DCO-1.1-Signed-off-by: Aaron Tam<EMAIL_ADDRESS>

That's not deprecation, that's deletion! :)

Check again!


Added a few links

Issue Number : #3
.
.
.
Modified File Path : ./Readme.md
.
.
.
Description : Added a few links to the readme, Removed the canva pro lifetime as it is not available anymore
.
.
.
@ShreyamMaity

you need to change files on the database.md


Rule for case sensitive imports

Some of our builds fail because import *  from './module' is lowercase while the file on the disk is uppercase (Module.ts). This is OK on Windows and macOS but not on Linux.
Maybe it would be a good candidate for a new rule?

Sounds expensive since you'd have to read file names from disk. I wouldn't want that in the core rules. Instead, you might consider writing a rule that validates module names of relative imports (perhaps with a regex).

eslint has a couple of plugins that do this, but it would be nice to have it integrated with tslint: https://www.npmjs.com/package/eslint-plugin-dependencies

I just found out typescript has this feature built in to the compiler:
https://www.typescriptlang.org/docs/handbook/compiler-options.html
--forceConsistentCasingInFileNames

Thanks, I didn't know about this, adding it to all our tsconfig.jsons now :)

Closing this out as compiler options are generally preferred to lint rules, especially for something related to I/O. I can reopen if there's widespread interest in this.

--forceConsistentCasingInFileNames does not work. even when my file name is in incorrect case, compilation with this option as true, does not fail.

@roopaprabhun you may want to report this bug in the TypeScript repo to get this fixed.

--forceConsistentCasingInFileNames works in different way. If there will be multiple imports imports to the same file with inconsistent casing, e.g. import b from 'bluebird' and import b from 'Bluebird' it will throw an error. So it doesn't prevent from wrong spelling when you're writing this import in the first place.

so according to the last comment, the feature request is still be relevant, right?

Yes, it's relevant, and I just got bitten by this.  I was updating a DT definition, imported from "GeoJSON" (which is the class name, not the correctly-cased module name), and it worked on my Windows dev box but failed Travis (which is presumably on Linux).

Also bitten.

Just realized nobody has brought this to the attention of @adidahiya , so: hello! Could we get this re-opened please?

Reopening to encourage more discussion - I agree with @adidahiya that I don't think this should be a core rule due to perf implications. That said, I think given the interest that it could still be worthwhile to implement.

I don't have a horse in the race as far as where it gets checked, I just want to make sure that it fails some time before I commit my code.  For DT updates, it could be specific to dtslint; for my own code I'd be fine if the linter didn't catch it but the AOT compiler did -- just something that takes a look at my import and says "hey dummy, your case is wrong"... other than a user from another platform inexplicably not finding a dep.

Note: per #4534, this issue will be closed in less than a month if no PR is sent to add the rule. If you really need the rule, custom rules are always an option and can be maintained outside this repo!

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for new rules. See #4534. 😱
If you'd like to see this rule implemented, you have two choices:

Recommended: Use typescript-eslint to lint your TypeScript code with ESLint ✅
Legacy: Write it as a custom TSLint rule 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!


acquire_unique_packet_id / acquire_publish -> "packet_id exhausted error"

I publish messages using the following (partial) code:
auto pid = client_->acquire_unique_packet_id();
client_->acquired_subscribe(pid, topic, MQTT_NS::qos::exactly_once);

After sending 65k messages exception packet_id_exhausted_error is thrown for each message that I try to send. I publish ~2k messages per minute (using MQTT_NS::qos::exactly_once as you can see above).
My understanding was that if I call acquired_publish() with packet id that I got from acquire_unique_packet_id then it is automatically passed to release_packet_id, since library takes ownership of this packet_id and I don't have to do it. But it doesn't seem to be the case, release_packet_id is mentioned in source code only three times, one of them is method declaration and other two are in comments.
I tried simply publishing messages without using acquired_*** methods, but I still get packet_id_exhausted_error.
So what should I do to get "proper" behaviour, that is reuse of ids?  But if answer is yes, then what does it mean that library takes ownership of packet id? And if I should use release_packet_id then how do I know if I should call it in pubrec_handler (for messages with qos less than exactly_once) or in pubcomp_handler (that is function/method/lambda registered via set_pubcomp_handler`)?
I use synchronous client, but I suppose that the same thing happens with asynchronous one.

Hello,
my colleague had a similar problem publishing with qos::exactly_once. I also have this problem. I just wanted to open an issue but saw that you have the same problem.
It looks like packet ids are not released when receiving pubcomp. Executingconst auto success = client->release_packet_id(id); within the client's pubcomp_handler seems to solve this (as long as you don't have over 65k messages in flight).
I searched the source code a bit and found this suspicious snippet in v7.0.0:
https://github.com/redboltz/mqtt_cpp/blob/9a63149965556bc5c16da3a974b9544e04acff5a/include/mqtt/endpoint.hpp#L10255-L10279
Seems like this is a small copy paste error from pubrec_phase::finish? Cause the packet_id is erased within the puback handler and this issue happened only with qos 2 but not with qos 1.
https://github.com/redboltz/mqtt_cpp/blob/9a63149965556bc5c16da3a974b9544e04acff5a/include/mqtt/endpoint.hpp#L9777-L9800
But I have to admit that I only read small parts of the code.

I modified pubcomp_handler so that now it releases packet_id:
client_->release_packet_id(packet_id);

and everything works find. I wonder why it doesn't work automatically when I use automatic packet_id manager.
Still it isn't obvious how to determine whether release_packet_id can be safely called - are all handlers always called and are called in predetermined order and only once? And also how to determine that packet_id was acquired using acquire_unique_packet_id.

@ineffective , @andybayer , thank you for reporting the issue.
On puback,  packet_id_.erase(info.packet_id); is called. It is the same effect as release_packet_id(). It is right code.
https://github.com/redboltz/mqtt_cpp/blob/7d4ac58462112558e349b5b1d5e7f796f76550c5/include/mqtt/endpoint.hpp#L9404-L9413
On pubrec, the packet_id in the container packet_id_ is not erased. It is expected.
https://github.com/redboltz/mqtt_cpp/blob/7d4ac58462112558e349b5b1d5e7f796f76550c5/include/mqtt/endpoint.hpp#L9549-L9557
However, on pubcomp,  packet_id_.erase(info.packet_id);  should be called but not called.
https://github.com/redboltz/mqtt_cpp/blob/7d4ac58462112558e349b5b1d5e7f796f76550c5/include/mqtt/endpoint.hpp#L9882-L9892
I will fix it.

@ineffective , user doesn't need to call release_packet_id() manually if you call acqruired_*() APIs. If you don't call acqired_*() APIs some reason, then yo need to call release_packet_id() to cancel.

Just created PRs.

#541 is fix for master
#542 is fix for v7.x.

Could you try this? No manual relased_packet_id() is required.

Seems to work for me. Thank you for your very fast response.

Great, it seems to work without problems, both when I call acquire_unique_packet_id and when I just send messages without it.
One question though: could you describe scenario when using acquire_unique_packet_id would be required?

The acquire_ version of the api is used when the end user guarantees that the packet Id provided is acceptable (e.g. unique).
The acquire_ version of the api is going away in the master branch as soon as I finish writing the pull request. See https://github.com/redboltz/mqtt_cpp/issues/534

Yes, I am aware of that, this is one of the reasons why I asked if it is
used at all by the clients. I understand from discussion under #534 that
there is no need for that whatsoever in the client and only server would
use that?

Ah, I misread your original question.
You would use acquire_unique_packet_id when you want to know the packet id that's going to be used prior to the creation / sending of the mqtt message.
For example, imagine you have a clustered mqtt broker, that has multiple instances of the mqtt-broker process running, and coordinating with itself using some unspecified cluster technology (e.g. custom network protocol, or corosync, or a database backend, so on).
You would want to ensure that your cluster doesn't use the same packet id twice at the same time,
when generating an outgoing mqtt message. So you would "acquire" the packet id, and the acquire_unique_packet_id function would need to use your application specific coordination mechanism to lock that packet id for all instances, before you send.
You should not need to use acquire_unique_packet_id, or even really register_packet_id in client code.
That being said, if you want to implement the more advanced publish behavior, such as journaling the packets you've received into a local database, before sending the ACK, so that your message can survive a crash, you can use those two functions (and the various customization points for them) to do so.
In my code, I've structured my MQTT protocol usage such that I don't care if my application crashes and restarts. But everyone's application is different :-)

@jonesmz Thank you, this is what I wanted to know.

It was a serious bug so I've merged the fix not only master but also v6.x and v7.x branches, then released them. See https://github.com/redboltz/mqtt_cpp/wiki/News

I saw that this bug affects QoS2 messages only. As I've been faced to the same issue but I use only QoS0 and QoS1, I am wondering whether this bug could also affect QoS1 messages.


Identity ignoring RequiredLength

I used ASP.NET 5 MVC 6 (1.0.0-rc1-update1) "Web Application" Template with "Individual User Accounts" and changed default password requirements to:
options.Password.RequireDigit = false; options.Password.RequireLowercase = false; options.Password.RequireUppercase = false; options.Password.RequireNonLetterOrDigit = false; options.Password.RequiredLength = 12;
Everything works fine, except RequiredLength.

When RequiredLength is lower than 6, the register form always shows "The Password must be at least 6 characters long.".
Is RequiredLength higher than 6, and the input lenght between 6 and the defined number (12), I get "Passwords must be at least 12 characters.", is the input lenght lower than 6 the form shows the error from above.

This is because the validation of the form is not tied to the underlying identity code... The validator length is controlled by the View model, see
https://github.com/aspnet/Templates/blob/dev/src/Rules/StarterWeb/IndividualAuth/Models/AccountViewModels/RegisterViewModel.cs#L17

@HaoK doesn't this mean that there is a bug in the template?

The data validation attributes in templates cannot easily share identity configuration. Since the template hardcodes a required length of 6 by default. It needs to be manually changed to match whatever the new required length is to match identity.

What I mean is: should we propose that we remove this attribute from the template?

Then there would be no client side validation for the for the password length which seems weird.


Setting background-color in custom ComponentBuilder prevents editing

Hi,
Great job with super_editor!
I was playing around with the example code for AnimatedTaskComponentBuilder. After making the following minimal change I can select the text only when the color is set to null.
So the problem is that the text is no longer selectable if it has color set to anything else than null

(If I wrap everything with an IgnorePointer it works again, but that workaround prevents me from e.g. showing an icon only when hovering that component.)
https://github.com/superlistapp/super_editor/assets/18383533/59af4d25-06a8-4c77-ba51-eb94fa895665
Here is an example, branched out from the stable-branch: https://github.com/proformance/super_editor/tree/background-color-problem

Thanks for your comment, I tried to clarify the issue

@matthew-carroll The issue seems to be that when using the color property, the Container adds a ColoredBox in the widget tree. The ColoredBox has an opaque hit test behavior, and it seems to be preventing the document layout to receive the gestures:
class _RenderColoredBox extends RenderProxyBoxWithHitTestBehavior {
  _RenderColoredBox({required Color color})
      : _color = color,
        super(behavior: HitTestBehavior.opaque);

It's possible to use a custom implementation of a ColoredBox which would use HitTestBehavior.translucent. That way the selection would work as expected.
I'm not sure if there's anything we could do on our side.

I'm going to close this due to inactivity. If the issue persists, please check the above questions/recommendations. If they don't help, please file an updated ticket.


Touch selection is not working

Steps to reproduce

Using a touch device or chrome debugger
tap and hold to bring up menu
release to select but it does not select

This works in 2.9.2 but not in 2.9.4. This behavior occurred in the chrome debugger with touch events and on my Android phone.

This is because of PR https://github.com/cytoscape/cytoscape.js-cxtmenu/pull/52
See also https://github.com/cytoscape/cytoscape.js-cxtmenu/issues/53
I've rolled back #52 and put new commits for fixes that followed thereafter.  2.9.5 should work like 2.9.2 for cxt swipes, and it should work like 2.9.4 for taphold swipes.


Where is the "PoseLandmarkCPU" subgraph defined?

Hello,
I am trying to use a modified version of the pose_landmark_cpu graph. I have been using the sample PoseTrackingGraph, which uses the graph config pose_tracking_cpu.txt, which refers to the calculator "PoseLandmarksCpu." It is not clear to me where this calculator is defined in the source for the Mediapipe Unity plugin or how I could modify it to suit my purpose (outputting some extra data from intermediate computations). I searched the text of all the files in the repo and could not find it.
Overall, I'm not sure I understand the difference between the regular mediapipe build process and the one used here (.pb files become .txt, not all solution-specific calculators appear to be part of the source, models use .bytes instead of .tflite). Would you mind providing a brief overview?
I am using the source for version 0.10.0

Never mind, I was misunderstanding how the build worked. I understand now it downloads the mediapipe source.

Hello, I am facing the same problem. Can you tell me how I can modify the subgraph? @megawattfs


Separate virtual environment for each name

Currently, at least for Pip, the name is not taken into account when creating/activating the virtual environment. This has the advantage that venv can be manually specified and is a fixed path.
In the case where we want to build groups of versions (eg. v1..and v2..) differently, and have a different environment for each sub-version, this is a bit cumbersome, as we have to spell out all versions to get different environments.
Changing this behavior is simple, by just changing this line from
super().__init__(path, name, venv, creator=creator)

to
super().__init__(path, name, Path(venv) / name, creator=creator)

(maybe with some additional escaping, depending on what name is exactly).
The harder part is to communicate well what is going on. As far as I understand it, the Poetry class also does not strictly respect the venv path, so maybe it would not be much of a problem?
If this is something worth implementing, what would be a good way to do it? Just a flag venv_use_name to activate the behavior?

Similar to what I wrote in https://github.com/real-yfprojects/sphinx-polyversion/issues/17#issuecomment-2556935662, for each revision a different directory is used to put source files and venv into. Thus there is no need to use the revision name when creating a venv. However the Environment interface defines and ensures that the name parameter is available to those Environment that need it. Imo this makes the code more general and extensible.

I agree. Both issues come up for me because Pip currently does not create the venv in the temporary directory, but in the local one (as far as I can tell). I have made a longer comment in #17, maybe it's best to discuss it there.


Request On Training Data

Hello, I have been watching the project for some time now and would like to help in any way possible!
My main goals pertain to the game engine NPC capabilities, video understanding, robot control and chatbot!
If I were going to turn these into usable traning data with the current tools I have or can buy, what would you suggest?
Also have there been any tests with any training datasets yet?

Since I am not familiar with reinforcement learning, hence I can't recommend any of tools for RL dataset generation for the NPC tasks.
And also, Gato in this repo haven't yet been tested with any of datasets.
It's just model architecture now.

Issue has been closed due to there are no more conversations.


Apply default deny-all network policies to giantswarm and kube-system namespaces

Issue: https://github.com/giantswarm/roadmap/issues/2815
If this is omited, the default (false) is used and it will install this.

/run cluster-test-suites

/run cluster-test-suites

@AverageMarcus The failed tests don't belong to CAPV. May I merge this PR?


Exit code 127 with yarn

yarn license-compatibility-checker emits
env: node\r: No such file or directory
error Command failed with exit code 127.

while npx license-compatibility-checker successfully runs.

node 11.3.0
yarn 1.12.3
license-compatibility-checker 0.3.3

https://github.com/yarnpkg/yarn/issues/5480


poetry update alternative?

It has the following two commands:
rye sync --update <UPDATE>             Update a specific package
rye sync --update-all                  Update all packages to the latest

see with rye sync --help

Closing this for now. I believe the commands provided already do the right thing.

Ok


Annihilate legacy re_components

All the grunt work necessary to get rid of re_components entirely.
Mostly surprise-free.

A good chunk of #3243
Very related to #3436

What
Checklist

[ ] I have read and agree to Contributor Guide and the Code of Conduct
[ ] I've included a screenshot or gif (if applicable)
[ ] I have tested [demo.rerun.io](https://demo.rerun.io/pr/{{ pr.number }}) (if applicable)

[PR Build Summary](https://build.rerun.io/pr/{{ pr.number }})
[Docs preview](https://rerun.io/preview/{{ pr.commit }}/docs)
[Examples preview](https://rerun.io/preview/{{ pr.commit }}/examples)
Recent benchmark results
Wasm size tracking

Waiting to see how the attempted removal of the legacy python Transform3D goes.


Default Identity schema breaches SQL Server rule for clustered index limit of 900 bytes

Description
The issue is exactly as described here: https://github.com/aspnet/Identity/issues/1451.
When creating the Identity schema, two of the tables (AspNetUserRoles and AspNetUserTokens) created have an index that exceeds limits.
If we attempt to re-create the automatically generated schema by using "Script as CREATE" in SSMS or Azure Data Studio, then run these scripts on an empty database, we see the following warnings:
Warning! The maximum key length for a clustered index is 900 bytes. The index 'PK_AspNetUserRoles' has maximum length of 1800 bytes. For some combination of large values, the insert/update operation will fail.
Warning! The maximum key length for a clustered index is 900 bytes. The index 'PK_AspNetUserTokens' has maximum length of 1412 bytes. For some combination of large values, the insert/update operation will fail.
Reproduction Steps

Create Web App using dotnet new webapp --auth Individual
Run dotnet ef database update
Observe that the AspNetUserRoles table has an index of maximum length 1800 bytes.

Expected behavior
I would ideally like:

The size of the indexes to be appropriate to avoid the warning - or
Some reassurance that this exceeding this maximum length will never be an issue.

Argh, sorry, user error. Having investigated it appears I was inadvertently using a preview version of EF Core (3.0.0-preview3.19153.1), and it looks from some documentation as if the intention is that these ID columns should be user configured. Bit of a strange default for the template in my view but I'll close this.


feat: default slug from title or name of page + small refactor

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


go version updated

related to https://github.com/jenkins-x/jx/issues/8283

There was another PR that updated the go version, can you close this PR?
Thank you.


8/27/2019 Publish

Docs Build status updates of commit 21aca48:
:clock10: Full build: the average full build time is 138 min(s) 45 sec(s), this is based on the last 93 build(s) for this repository.

Docs Build status updates of commit 21aca48:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-autogen/azure-functions/azure.yml
:warning:Warning
View (azure-python)
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

docs-ref-services/virtualmachines.md
:white_check_mark:Succeeded
View (azure-python)

docs-ref-mapping/reference.yml
:white_check_mark:Succeeded

docs-ref-autogen/azure-functions/azure.yml

[Warning] Uid(azure) has already been defined in docs-ref-autogen/azure-functions/azure.yml.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


HTML in examples badly formatted

For example Radios. See the 'other' example

 Issue #745 HTML in examples badly formatted


Test Flake: TestProcessor_EventAccumulation_WithFullSync

https://k8s-gubernator.appspot.com/build/istio-prow/pr-logs/pull/istio_istio/12626/istio-unit-tests/22329

@nmittler I believe you made some changes in this area recently. Could PTAL?

Still flakes https://k8s-gubernator.appspot.com/build/istio-prow/pr-logs/pull/istio_istio/12822/istio-unit-tests/22736


I can't use the init function which from my wire.go in my main.go

here, I used four struct, and I define a init function in my wire.go , but the question is I can't use it in my main.go

Hey @tudoutiaoya, could you show me your wire_gen.go? It seems like your IDE is using that for its analysis.

remove wire plugin from ide, it's useless and cause IDE error

Firstly, add wireinject to your custom tags. Next, run go get github.com/google/wire. Once this is done, Jetbrains Goland will analyze your code success.


Additional network adapter in VMs

I have a question rather than an issue, and I'm not sure if this belongs in corectl and kube-cluster-osx. I was wondering if its possible to add an additional network adapter in the VMs spun up to run kube-cluster?
The context is I'm trying to run Openstack on the Kubernetes cluster and for the networking and hypervisor components I need a second, unconfigured adapter on the host.
Its seems from this that its possible but I'm sure where or how I can influence this in the higher level components.

@abattye, glad you ask... I looked at it in the past but never went to the end for lack of a true use case... will put that on my radar again... and plz open this as a feature request in corectl repo plz...
Thanks for using us!

Thanks for the quick reply - I've added an issue in corectl


How do you upgrade an existing project to 1.1.1?

I uninstalled and then reinstalled Template10 in Visual Studio yesterday before creating a project based on Minimal and yet I was left with this version:
nuget\packages\Template10\1.0.8
Is there a way to upgrade that template in place to 1.1.1?
Other than the build output is there a place within the code to see what version of Template10 is being used?
Why wouldn't the install through Visual Studio use 1.1.1 in the first place?
Thanks,
Rick

Hello Rick,
Right click on your project in Visual Studio and choose Manage NuGet packages. Under the Installed tab, you will see the list of installed packages and their version number. Moreover, if you move to the Updates tab, you will see all the packages which have an update, with the option to go on and perform it.

OK great I found that and upgraded.  I really appreciate the help.  I'm left with a few compile errors like:
Severity	Code	Description	Project	File	Line	Suppression State
Error	CS0115	'DetailPageViewModel.OnNavigatedTo(object, NavigationMode, IDictionary<string, object>)': no suitable method found to override	i3RemoteWP_Universal2
I'm assuming these relate to the breaking news issues on 1.1.1 that Jerry posted and require a manual fix up?
Thanks,
Rick

Correct. The new signature of the method is now:
Task OnNavigatedToAsync(object parameter, NavigationMode mode, IDictionary<string, object> state)

I think I have it, does this change look right?  It did fix a compile error:
Change this:
public override void OnNavigatingFrom(NavigatingEventArgs args)
{
args.Cancel = false;
}
to this:
public override Task OnNavigatingFromAsync(NavigatingEventArgs args)
{
args.Cancel = false;
        return Task.CompletedTask;
    }

Just want to make sure I got it right.
Thanks,
Rick

Yes, it's correct. Just remember that the code:
return Task.CompletedTask;
is required only if, like your case, the method doesn't perform any asynchronous operation (so you're not using the async / await keyword). Otherwise, you can omit it.

That cleared up the existing compile errors but left me one last one I don't see instructions for.  The error is:
Error	CS1503	Argument 1: cannot convert from 'Template10.Services.NavigationService.INavigationService' to 'Template10.Services.NavigationService.NavigationService'	i3RemoteWP_Universal2
In this code:
// runs even if restored from state
public override async Task OnInitializeAsync(IActivatedEventArgs args)
{
// setup hamburger shell
var nav = NavigationServiceFactory(BackButton.Attach, ExistingContent.Include);
Window.Current.Content = new Views.Shell(nav);
await Task.Yield();
}
Any idea what the fix is for that?
Rick

See here https://github.com/Windows-XAML/Template10/issues/556
// Shell.xaml.cs
public Shell(INavigationService navigationService)

I'm getting this error
void DetailPageViewModel.OnNavigatedTo(object parameter, NavigationMode mode, IDictionary<string, object> state)
'DetailPageViewModel.OnNavigatedTo(object parameter, NavigationMode mode, IDictionary<string, object> state)': no suitable method found to override
since I update to 1.1.1

I couldn't find that reference easily in #556 but I did find that line and replaced it.   I then compiled again and got 22 compile errors very similar to this one:
Error	CS0433	The type 'Interaction' exists in both 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' and 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'	i3RemoteWP_Universal2
This is so hard, its like a scavenger hunt of pain.  I wish there was just a fixed, ready to go version of a template 10 minimal project ready to go to use as a baseline.
Rick

You need to remove the Behaviors SDK reference you find in the references of your project and keep just the Behaviors NuGet package

By the way, a full list of the changes to do when you upgrade your project is listed in the wiki https://github.com/Windows-XAML/Template10/wiki/Upgrading-from-Template-10-Nuget-Nuget-1.0-to-1.1

Thanks Matteo, I ran through all of the changed and think I did them but was left with 1 last error:
Error		Assembly 'Template10Library' with identity 'Template10Library, Version=1.0.5866.5306, Culture=neutral, PublicKeyToken=d74e748b43bf937b' uses 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35' which has a higher version than referenced assembly 'Microsoft.Xaml.Interactivity' with identity 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35'	i3RemoteWP_Universal2
I'll comb through the links you sent to make sure I didn't miss anything.  I did remove the reference to the Behaviors SDK.
Rick

It looks like the project has a dependency on 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>' but the NuGet package is dependent on 'Microsoft.Xaml.Interactivity, Version=<IP_ADDRESS>,

I never added that dependency though, I installed a fresh copy of Template 10 through VS 2015, I think it was 1.0.8, upgraded that in place to 1.1.1 and did the fix ups.   I didn't make any modifications the to project yet, its pretty much as is based on the Minimal template.  I don't see a reference to 'Microsoft.Xaml.Interactivity so its not clear how to fix it to make it depend on .Xaml.Interactivity, Version=<IP_ADDRESS>

I got my solution building again by deleting all my bin and obj folders to get rid of any caches.  Then I changed "Microsoft.Xaml.Behaviors.Uwp.Managed" version="1.0.2"  to "Microsoft.Xaml.Behaviors.Uwp.Managed" version="1.0.3" in the Template10.nuspec file in my .nuget folder.  Now it all builds again.
I know its a bit hacky but the 1.1.1 NuGet package is broken, this just makes it less broken.  Any fix is going to mean a new version of the NuGet package, so I don't mind hacking it as a temporary measure.

Actually the 1.1.1 NuGet Package is not broken. I updated to it recently and you just need to follow the upgrade guide here. Here's what's in my project.json.
"dependencies": {
    "Microsoft.NETCore.UniversalWindowsPlatform": "5.0.0",
    "Microsoft.Xaml.Behaviors.Uwp.Managed": "1.0.3",
    "MvvmLight": "5.2.0",
    "Rx-WinRT": "2.3.0-beta2",
    "Rx-XAML": "2.3.0-beta2",
    "Template10": "1.1.1",
    "Newtonsoft.Json": "8.0.2"
  },

@JustinXinLiu that is a better fix - still getting used to the new project.json files. I'm pretty sure those instructions weren't there when I originally went to that page, I never thought to go back, d'oh!.  However, the Nuget package is still wrong; it shouldn't specify a version of a dependency that is not compatible with the contents of the package.

OK its working.   I edited Template10.nuspec (I had to figure out where that was) and changed the behavior dependency to 1.0.3.  I thought it was odd that in that file it showed dependency for Newtonsoft.Jason for 7.0.1 even though I upgrade that to 8:

In my project.json file I don't even show the dependency for Microsoft.Xaml.Behaviors.Uwp.Managed:
"dependencies": {
"Microsoft.ApplicationInsights": "1.0.0",
"Microsoft.ApplicationInsights.PersistenceChannel": "1.0.0",
"Microsoft.ApplicationInsights.WindowsApps": "1.0.0",
"Microsoft.NETCore.UniversalWindowsPlatform": "5.0.0",
"Newtonsoft.Json": "8.0.2",
"Template10": "1.1.1"
},
I am thrilled that my project is finally alive and running for the first time!
Rick

Oh no, after my post I was about to load the Minimal project in Blend and see how it looked but literally after simply compiling fine, exiting the app in Local Machine and then terminating the app, I was left with an invalid markup error on the designer surface and the error in here of MainPageViewModel not existing in the Viewmodels namespace.  I also noticed that I now have 37 compile errors.   I realized I didn't do the 1.1.1 fixup about the bootstrapper code but I can't find where that code exists to modify it.  I think the release notes need to give more guidance as to precisely where all these change nuggets originate from.
Its amazing that I can get a perfect compile, run the app and then upon completion have a completely new set of failures.   Its a linked list of complications.
Rick

Version 1.1.0 of Microsoft.Xaml.Behaviors.Uwp.Managed has been released and it seems to be breaking. I got this error using version 1.1.0

the assembly version has a public key token that does not match that of the request

So, I need to go back to 1.0.3. And now it is working perfectly.


Plugin does not work for several Directories with dependent Protos

Hello everyone,
I have two proto files in two different directories whereas one is dependent on the other and I don't get it to work, it seems to me that there might be a bug how imports are handled - either by the protoc compiler or the maven plugins (I have the same issue with the protobuf maven plugin. My directory structure looks approximately like this:
.
├── pom.xml
├── src
│   └── main
│       ├── java
│       │   └── ...
│       ├── proto
│       │   └── service.proto
│       └── resources
│           └── submodule
│               ├── protobuf
│               │   └── framework.proto

I need to preserve this directory structure and I also want to use the same framework.proto in other maven modules by accessing the resource path ideally.
The service.proto is using definitions of the framework by importing: import "framework.proto";
Now I tried to build the protobuf classes by copying the examples on the github site:
<plugin>
                <groupId>com.github.os72</groupId>
                <artifactId>protoc-jar-maven-plugin</artifactId>
                <version><IP_ADDRESS></version>
                <executions>
                    <execution>
                        <phase>generate-sources</phase>
                        <goals>
                            <goal>run</goal>
                        </goals>
                        <configuration>
                            <addProtoSources>all</addProtoSources>
                            <includeStdTypes>true</includeStdTypes>
                            <includeDirectories>
                                <include>${basedir}/src/main/resources/submodule/protobuf</include>
                            </includeDirectories>
                            <inputDirectories>
                                <include>${basedir}/src/main/proto</include>
                            </inputDirectories>
                            <outputTargets>
                                <outputTarget>
                                    <type>java</type>
                                </outputTarget>
                                <outputTarget>
                                    <type>grpc-java</type>
                                    <pluginArtifact>io.grpc:protoc-gen-grpc-java:1.0.1</pluginArtifact>
                                </outputTarget>
                            </outputTargets>
                        </configuration>
                    </execution>
                </executions>
</plugin>

Now the compilation is failing because the ServiceOuterClass : cannot find symbol [ERROR] symbol: variable Framework. Inspecting the generated sources on target/generated-sources, I realised that Framework.java was not generated, so I added the path to the <inputDirectories> - That will indeed generate the Java class, but the Java imports on the ServiceOuterClass will still be wrong unfortunatelly.
I have seen many issues on Stackoverflow and Github concerning the imports of other proto-files but unfortunatelly no good solution - any tipps are appreciated!

Note: It does work if both the service.proto and framework.proto are defined in the same package AND both directories are defined as inputDirectories, which IMHO shows that the Java dependencies are not generated correctly (might be a problem of protoc and not of the plugin itself though ...)

Could you provide the log? We should be able to see whether the plugin prepares the protoc invocation correctly or not

@os72 Please excuse the belated response, I just had a look into this again. So I did the mistake of not defining a packagein the framework.proto which leads to wrongly generated imports, as all dependent protos need to have a proper namespace.
Additionally, the protoc compiler relies on the protos that are included to be built "somewhere" in the class path, the log is approximately like:
[INFO]     Processing (java): service.proto
protoc-jar: executing: [/var/folders/q0/8cy4jxy13ml92frk1jdpmrxw0000gn/T/protoc1700105763840564952.exe, -I/src/main/resources/submodule/protobuf, -I/src/main/proto, --java_out=target/generated-sources, src/main/proto/service.proto]

Which is fair enough I guess, the developer just has to be aware of this, so I would propose to close the issue :-)

Ok great :)


Skills & Grimoire Table - custom rules text box

Skills & Grimoire Table - custom rules text box
expand button - to show/hide house rules

Already completed except show/hide. Will wait on that.


Updated ASP.NET ES2016 skeleton

When using Gulp with Visual Studio, projects will sometimes not run if there is an issue with one of the dependencies. This PR updates the gulp-eslint and run-sequence dependencies so that the project compiles.
I also added the gulp-jspm task so that jspm install can be run by Visual Studio without having to open a command prompt. Finally, I updated gulpfile.js so Visual Studio automatically runs the clean, gulp-system, gulp-html, gulp-css, and gulp-jspm tasks when the project is built.

@PWKad Can you review this?

Hi @RandomlyKnighted , and thanks for the contributions!
Can you amend the commit messages to match our guidelines
Can you also squash the commits down so that the history will have less but more focused commits?  If you need any help just let me know.  I'll review the substance of the commit now.

@PWKad how's that?

That looks good to me. @PWKad Any final reivew you need to do? or can we merge this?

lgtm!

Thanks!


add first class support for Java runtime #390

This change provides out of the box support for the Java runtime. It includes being able to build the Java project via Gradle or Maven as well as their respective wrappers (gradlew or mvnw). If a file named pom.xml is found in the project root directory it's assumed the project can be built via Maven. If a file named build.gradle is found it's assumed the project is built via Gradle.
There are a couple of assumptions:

The Java project will produce a shaded jar with the name server.jar.
The artifact produced can be run (on port 3000 by default) via java -jar server.jar

I’ve also reworked all the Java* up-examples and will open a PR for that once this is merged

During testing for this change I'm seeing the proxy command get set to the default ./server instead of the expected java -jar server.jar although I've followed the same pattern for the Python runtime for overriding. Is this a bug or an I missing something?

hmmm what you have looks like it should be fine to me, make sure that up version is 'master' just in case you're accidentally using the official release still

Verified I'm using master and everything works fine when I use up start to run things locally. It's only when I actually deploy via up it seems to use the default ./server when I go to test via up url --open.

Finally got to the bottom of the issue of the proxy command getting reset. Apparently, having *.gradle entry in the .upignore file for my test project was the cause. I'm baffled why this.

I got a little further in my investigation about why once I deployed Java runtime projects the proxy command would get reset. If I don’t include the build.gradle or pom.xml for Gradle or Maven projects respectively, the proxy command will get reset. I believe this is because of the check in config.go to determine the runtime.
My assumption was that this check only happens at build/package time before the deploy but it seems my assumption was wrong and this runtime check code path gets executed again after deploy. So what this means for now is that Gradle and Maven projects need to ensure that build.gradle or pom.xml get included with the deploy. Another less straight forward workaround is to make the final artifacts executable and match the default ./server path instead of the more intuitive server.jar artifact name and proxy command of java -jar server.jar.
My feeling is that Up should have a mechanism so neither of these workarounds are necessary or at the very least document the above scenarios.
Thoughts?

I agree with that. I think it would be reasonable for up to embed the full
config at deploy time.  So take the current config, and freeze it into
up.json while building the zip.
On Thu, Oct 26, 2017, 8:37 AM Shane Witbeck<EMAIL_ADDRESS>wrote:

I got a little further in my investigation about why once I deployed Java
runtime projects the proxy command would get reset. If I don’t include the
build.gradle or pom.xml for Gradle or Maven projects respectively, the
proxy command will get reset. I believe this is because of the check in
config.go to determine the runtime.
My assumption was that this check only happens at build/package time
before the deploy but it seems my assumption was wrong and this runtime
check code path gets executed again after deploy. So what this means for
now is that Gradle and Maven projects need to ensure that build.gradle or
pom.xml get included with the deploy. Another less straight forward
workaround is to make the final artifacts executable and match the default
./server path instead of the more intuitive server.jar artifact name and
proxy command of java -jar server.jar.
My feeling is that Up should have a mechanism so neither of these
workarounds are necessary or at the very least document the above scenarios.
Thoughts?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/apex/up/pull/391#issuecomment-339707088, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAkJMgafdAnpyiwjhsi8l9_wr_K8qkGBks5swKcsgaJpZM4QCBa4
.

Can this be merged now?
Embedding the config at deploy time seems like it should be a separate issue.

Ahh shit sorry I wasn't thinking of that, yeah the files it uses for inference should be available within the Lambda as well (currently). Not quite ideal!


Attach views back in series instead of async

When using rendr with requirejs in AMD mode the child views are attached with null as parent on the first render. Because we are waiting for the module to be loaded . This change is to forcefully attach the views in series.

:+1:

@rendrjs/maintainers Can you take a look please? AMD gods will appreciate it :)

:+1:

Seems okay, @jmerrifield might want to try it out with browserify to verify it doesn't break that one too :) yay amd vs commonjs.

@saponifi3d Sorry, I got carried away with work things and lagging behind on testing app. (Looking forward for the holidays). :)

No worries. I haven't had time to get to it yet either. :'(

@jmerrifield Any news on browserify front? :) Thanks.

Haven't had a chance to check this yet, and won't be back at work until Monday to do so. I think this change will be totally fine though, probably no need to wait on me :)


Bug: Hack button for physical car

Sometimes the hack button is displayed in the driver ui of a physical car.
This seems to happens after switching cars.

I got this bug when testing with a simulated physical car (a virtual car in the physical car list). As soon as I join the game with this car, the "Hack" Button is visible, although without effect. Sometimes as soon as a second player joins, sometimes as late as after hacking, the button disappears.


Syntax error: redirection unexpected on Database::PostgreSQL step

Using 4.1.10 I'm getting this in my log
[2015/04/20 20:44:15][info] Database::PostgreSQL Started...
[2015/04/20 20:44:15][warn]   Pipeline STDERR Messages:
[2015/04/20 20:44:15][warn]   (Note: may be interleaved if multiple commands returned error messages)
[2015/04/20 20:44:15][warn]
[2015/04/20 20:44:15][warn]   sh: 1: Syntax error: redirection unexpected
[2015/04/20 20:44:15][info] Database::PostgreSQL Finished!

which is preventing the database from being backed up. How do I trace this problem?

Hi @axsuul can you start by providing your model ~/backup/models/my_model.rb en global config ~/backup/config.rb?

# encoding: utf-8

##
# Backup v4.x Configuration
#
# Documentation: http://meskyanichi.github.io/backup
# Issue Tracker: https://github.com/meskyanichi/backup/issues
# encoding: utf-8

require 'dotenv'
require 'yaml'

# Load environment variables
Dotenv.load

# Define Rails constants
RAILS_ROOT = File.expand_path(File.dirname(File.dirname(__FILE__)))
RAILS_ENV  = ENV['RAILS_ENV'] || "development"

# Defaults
Storage::S3.defaults do |s3|
  s3.access_key_id     = ENV['S3_ACCESS_KEY_ID']
  s3.secret_access_key = ENV['S3_SECRET_ACCESS_KEY']
end

# Load database config
database_config = YAML.load_file("#{RAILS_ROOT}/config/database.yml")[RAILS_ENV]

# Backup model
Backup::Model.new(:app, "Backup") do
  split_into_chunks_of 250

  database PostgreSQL do |db|
    db.name               = database_config['database']
    db.username           = database_config['username']
    db.password           = database_config['password']
    db.host               = database_config['host']
    db.port               = database_config['port']
    db.skip_tables        = ['schema_migrations']
    db.additional_options = ['--data-only', '--no-privileges']
  end

  archive :files do |archive|
    archive.add "/etc/environment"
    archive.add "/srv/shared/images/"
    archive.add "/srv/shared/uploads/"
    archive.exclude "/srv/shared/uploads/tmp"
  end

  store_with S3 do |s3|
    s3.region             = 'us-west-1'
    s3.bucket             = 'xxxx'
    s3.path               = '/backups'
    s3.keep               = 10
  end

  compress_with Gzip do |compression|
    compression.level = 6
  end
end

@tombruijn Any other suggestions?

@axsuul what OS and version are you running on? Ubuntu, Debian or something?
Try running it through /bin/bash -l -c "cd $HOME/backup && bundle exec backup perform -t issue_657"
Do make sure that your Ruby is added to your PATH env variable if bash is not your default shell.
Is it only going wrong with postgres? Can you post the entire logs.
Try removing/disabling/commenting out all the other things like archiving, s3 and compression, splitting and try again.

Thanks for your help, it turned out my database_config had unevaluated ERB. So it was passing <%= and %> characters.
Nothing wrong on backup's end :)

@axsuul great to hear it's resolved. So yeah, just parse it through ERB and then yaml. Even so, the ERB codes should be escaped, but I think that only applies to the password flag. I assume that you also use ERB for options like host and username then? Then indeed random < and > can throw a redirection error.


Give ws stringer a way to break cycles.

Testing https://github.com/helloeave/worksheets/pull/115 I found that printing out worksheets with cycles was not working.
Open to suggestions for the format for refs already seen. Currently it prints out <ref_to_seen_worksheet>.

And LGTM after an additional test for slices.


Loop freezes and nested prompts are not reachable

Hello.
I use plop for generation some stuff in project and I need recursive question in prompts array, but loop freezes after answering on the first question and then nothing happens.
Any thoughts?
"plop": "^2.5.3",
"inquirer-recursive": "0.0.6",

Fixed by updating of the inquirer version to the latest for inquirer-recursive package.

I have the last version and I have the same problem....


Map virtual key before triggering keybd_event

I'm using RobotJS to simulate keyboard input into an RDP session. However, because the Windows RDP client listens for scan codes instead of the key codes (for security reasons?), simulating key events with RobotJS doesn't work. See this thread on SO.
This PR adds the correct scancode using MapVirtualKey. With this change, RobotJS can send key events locally and through an RDP session. Tested on Windows Server 2012 and Windows 10.
Cheers,
Mike

Thanks Mike! Does this cause any issues for regular Windows users?
@Deltatiger do you see anything wrong with using scan codes instead of key codes?

As far as I can tell, there are no issues for regular Windows users. The key code is also still being used, as well as the scan code - this PR only adds a value for the optional second parameter to keybd_event, instead of leaving it 0.
I haven't found any documentation or issues that would suggest that setting the scan code could cause issues. I have only tested on Win10 and Windows Server 2012 though (both locally & through RDP).

@octalmage I am not familiar with this concept so I did some googling and think this is good. It doesn't look like it will be of any difference to the normal (non RDP) user. So we can go along with this.
@mgmartel Could you please check your editor to make sure that it does not mess with the whitespace. More lines are tagged with whitespace removal than normal code change.
Good thing is even if we shift to SendInput this feature can be supported without any big modification to the logic.

@mgmartel It would be better if the function was moved to CamelCase. The function still follows Macro naming convention. Since this is a normal function now proper changes should be made I guess.

@Deltatiger I updated the PR to not include the whitespace changes, and I made the function camelCase. It's been a while since I formatted any code in C, so let me know if you think anything else needs updating.

@mgmartel awesome thanks! I'll test this and get it merged.


Debugging with HMR

Environment

CLI: next (v)
Cross-platform modules: 5.0.0
iOS Runtime: 5.0.0

Describe the bug
When a file is updated with HMR, the dev tools are not updated and the breakpoints are not working anymore.
To Reproduce

tns create newApp --js
cd newApp
tns debug ios --hmr
open chrome dev tools and set a breakpoint
change a js file
the change will not be applied in the dev tools and the breakpoint will not be hit anymore

Expected behavior
The dev tools code should be updated and the breakpoint should be refreshed and hit after livesync. (try the tns debug android --hmr behavior)

Issue moved to NativeScript/nativescript-cli #4230 via ZenHub


example lookup data

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#944  👈 (View in Graphite)
#943
#942
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Codecov Report
Attention: Patch coverage is 73.79310% with 38 lines in your changes missing coverage. Please review.

Project coverage is 92.06%. Comparing base (a416c87) to head (278d03e).

Files with missing lines
Patch %
Lines

crates/air_utils/src/trace/examle_lookup_data.rs
73.79%
38 Missing :warning:

Additional details and impacted files
@@                   Coverage Diff                   @@
##           ohad/par_trace_iter     #944      +/-   ##
=======================================================
- Coverage                92.26%   92.06%   -0.20%
=======================================================
  Files                       98       99       +1
  Lines                    13516    13661     +145
  Branches                 13516    13661     +145
=======================================================
+ Hits                     12470    12577     +107
- Misses                     976     1014      +38
  Partials                    70       70

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


Display current release version in the UI

In the publish Github action we could probably set an environment variable APP_VERSION after semantic-release runs.
We could display the version in the Settings page (or maybe in a MOTD if #50 is implemented).

This feature is only partially implemented, there was an attempt to fix it in 0.7.0 .. 0.7.3 but I couldn't get the package.json to be updated with the correct version when running the publish.yml GH action.
We would also need to make sure the Cloudflare worker release also has the correct version. Might need to move that pipeline to the GH action.

This is finally working 🎉


Added text shadow to improve readability

Helps improve readability on text content.  Not sure if the other would benefit from a shadow, but this helps a lot.

Sorry, just saw your PR! Thanks for the contribution.

No worries, it was a tiny one.  We really like the environment styler!  Thanks for it!


Fixed sort bug

Fixed a bug that caused the sort to not behave how I expected and very infrequently makes searches less relevant.

Codecov Report

Merging #85 into master will increase coverage by 11.86%.
The diff coverage is 87.25%.

@@              Coverage Diff              @@
##             master      #85       +/-   ##
=============================================
+ Coverage     75.22%   87.08%   +11.86%
- Complexity       76      219      +143
=============================================
  Files             9       18        +9
  Lines           452     1285      +833
  Branches         46      135       +89
=============================================
+ Hits            340     1119      +779
- Misses          100      128       +28
- Partials         12       38       +26

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/google/sps/Firebase.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

...main/java/com/google/sps/servlets/TaskHandler.java
0.00% <0.00%> (ø)
0.00 <0.00> (?)

src/main/java/com/google/sps/SecretHandler.java
20.68% <11.76%> (+20.68%)
2.00 <1.00> (+2.00)

src/main/java/com/google/sps/EventRating.java
76.00% <76.00%> (ø)
6.00 <6.00> (?)

...main/java/com/google/sps/servlets/UserServlet.java
89.93% <84.69%> (+7.93%)
25.00 <10.00> (+5.00)

src/main/java/com/google/sps/Utils.java
68.13% <84.78%> (ø)
17.00 <15.00> (?)

...java/com/google/sps/servlets/RecommendServlet.java
87.70% <87.70%> (ø)
17.00 <17.00> (?)

src/main/java/com/google/sps/Interactions.java
86.17% <88.60%> (+6.17%)
24.00 <20.00> (+15.00)

src/main/java/com/google/sps/Recommend.java
89.65% <89.65%> (ø)
26.00 <26.00> (?)

...in/java/com/google/sps/servlets/SearchServlet.java
95.74% <94.63%> (+1.12%)
46.00 <35.00> (+28.00)

... and 21 more

Continue to review full report at Codecov.

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


Resolve dependent call through simple smart pointer type

During the review of #141, @sam-mccall suggested getting the following go-to-definition testcase to work:
template <typename> struct S { void [[foo]](); };
template <typename T> struct unique_ptr {
  T* operator->();
};
template <typename T>
void test(unique_ptr<S<T>>& V) {
  V->fo^o();
}

I pointed out that getting this testcase to work will not imply getting real-world code using std::unique_ptr to work, because the actual return type of std::unique_ptr::operator-> in standard library implementations involves a bunch of metaprogramming.
However, it would still be useful to get this case to work, as people may work with other smart pointer types whose operator-> is simply declared to return T*, as in the testcase.

I tried implementing this using Sam's suggestion from the review:

So it might be worth trying to get the pointee type (if the base type is a TemplateSpecializationType, look up operator-> in the primary template, and replace the base with its return type).

However, even for this simple case, there is a complication. To explain it, I will repeat the testcase, but with unique_ptr's template parameter's name changed from T to U:
template <typename> struct S { void [[foo]](); };
template <typename U> struct unique_ptr {
  U* operator->();
};
template <typename T>
void test(unique_ptr<S<T>>& V) {
  V->fo^o();
}

We start by computing the type of V, which is unique_ptr<S<T>>. This is a TemplateSpecializationType, with the primary template unique_ptr.
To look up operator->, we basically discard the template parameters S<T>, and look up operator-> in the Decl of the primary template. The resulting method decl is U* operator->, whose return type is U*, where U is a TemplateParmType.
Notice how this type is useless for the subsequent operation we were hoping to do, which is to look up the name foo inside this type.
It seems we are missing a step: to save the template parameter mapping U --> S<T> that we discarded when we went from unique_ptr<S<T>> to the primary template unique_ptr, and then substitute this mapping back into the return type U* to get S<T>*, after which we can proceed as desired.
This should certainly be doable in theory, but I'm not familiar enough with clang'd AST to say how doable it is in practice. Would appreciate some guidance: does this seem like a reasonable thing to try to do?

Would appreciate some guidance: does this seem like a reasonable thing to try to do?

What gives me pause, in particular, is having seen comments like this one:

That substitution is exposed by sema (SubstDefaultTemplateArgumentIfAvailable) but we don't have access to sema (and if we did, it's probably slow and has side effects like diagnostics).

Would we run into similar limitations if we tried to do the what I described?

Thanks a lot for looking into this, and sorry for the "drive-by" nature of some of my comments!
This indeed gets hairy quickly. On the other hand, almost every templated smart pointer is of the form Tmpl.
What do you think about this hack: if we have a TemplateSpecializationType whose first argument is T, assume its operator-> returns T*? And then do the member lookup in T.
This should address the common case, I think.

What do you think about this hack: if we have a TemplateSpecializationType whose first argument is a type T, assume its operator-> returns T*? And then do the member lookup in T.

That's a good idea, thanks for suggesting it!
Proposed fix


handle install failures better

Right now, if a toolchain install fails (for example, if the msi server was not running, or one was running multirust in an account without the needed permissions to make installations), one has to start the process all over again. This isn't too bad, except for the case where one has a slow network connection (take hours to download) and the installer was downloaded by multirust (not obvious where to find the downloaded installer file, assuming it still exists somewhere on the the machine). Would be nice to have either some way of retrying the install or a way to download and save the installers somewhere, waiting for the user to install manually (some might like the latter better as it allows them to run multirust with less privileges when downloading the installers).

I've been meaning to get around to implementing something like this for a while, it's just a little tricky to get right without leaving unnecessary files lying around. Thanks for creating an issue.

Yes ! :+1: to this issue !

it's just a little tricky to get right without leaving unnecessary files lying around.
What about a journal ? A simple flat text with the last events since the last successful execution, so failures don't reset the journal. This shouldn't be more than a few kilobytes.

When using multirust update stable --continue, the journal keep trace of the useful files created by the precedent failed executions (could be a crash, a SIGINT, whatever). If no --continue, simply trim the journal and log the execution.


[ServiceBus] Change name to _name naming for remaining "name" parameters

For AdministrationClient to normalize with existing naming schemes.

/azp run python - servicebus - tests


fix(core): Add custom 400 and 404 error messages

fix(core): Add custom 400 and 404 error messages
Fixes #1504

One remaining problem is that ideally, the bad URL should remain the same (i.e. /articles/noarticlehere rather than /not-found) and still display an error message. I was able to get halfway there by deleting the url property from the error routes, but then you have another issue where the valid page before getting the error cannot be reached in the browser history. The 404 article page is intercepted before it is displayed, so then the browser would think that the page before that and the error page were the same entity, rather than the bad article page and the error. But that's for another PR.

Coverage decreased (-0.1%) to 72.929% when pulling 50879f090d670641268f5492be80729930f85c5a on hyperreality:upstream into 8b5466961430627356dd6967b296c8528c044e23 on meanjs:master.

A good point being thrown up by the E2E testing there. We already have loads of bad requests that don't expect to be redirected to another page, such as in the user sign up:
return res.status(400).send({
    message: errorHandler.getErrorMessage(err)
});

This could be avoided by removing the intercepting of 400 bad requests I just added and change the 'Invalid Article ID' error to another 404, what do you think @mleanos?

I think the concern of some 400 responses not requiring a redirect, was what someone brought up a while back. I couldn't find that conversation. Either way you're right.
In the case of a failed sign up, 400 is not appropriate. What about 409 or 422?
Another option might be to use 400, but also pass a flag (something like: bypassInterceptorRedirect) that we can check in our interceptors to decide if we should redirect the user. I think this option should only be used if we can't come up with a better HTTP response code than 400.
Side Note: We seem to be using 400 as an error response in some cases where it's probably not the best code to return. I can't recall where these are. However, I just want to point out that we should try to use more appropriate codes in certain situations.

Overall, this looks pretty good to me. We just need to figure out how to handle the Sign Up error response, and keep the user on that view. Thus, solving the E2E tests that are failing.

ping @hyperreality

Grepping through the project finds loads of 400 errors. I think we should ignore that for now and look at improving the error status codes at some point in the future? I actually think that 403 is the most appropriate code for a failed login, but that's also intercepted by this code already.
I wonder if the bypassInterceptorRedirect type solution is a commonly used option in projects.

A failed login should be 403. However, a failed Sign Up might be a 409 or 422.
http://www.bennadel.com/blog/2434-http-status-codes-for-invalid-data-400-vs-422.htm
I feel like 422 is more appropriate in this case.

As for the bypassInterceptorRedirect, it shouldn't be a common solution. It's probably not a good solution, but a last resort option; then again maybe we shouldn't consider it.

There's just one last issue we need to resolve; we need to decide on how to handle the current 400 HTTP response codes in the Sign-Up & Sign-In views. See this comment for more context. My opinion is that they should be returning 422; this is described here http://www.bennadel.com/blog/2434-http-status-codes-for-invalid-data-400-vs-422.htm
@meanjs/contributors These changes are pretty important. I'd really like to see this in for 0.5.0. Let's make a decision on how to handle the 400 error codes discussed above, and finalize this PR.
@hyperreality If you don't have time to finish this, I can take it over & create a new PR based off of the work done here. I'll create a branch & merge these changes in, to keep these commits in the history.

Hey @mleanos, I agree with all your decisions. Yes, I would appreciate if you could take over this PR, I'm extremely busy atm.

Closing in favor of #1547


Add sensible default for gfak ids --start-ids

There is no check that the user has supplied gfak ids --start-ids, but without it gfak crashes with an uncaught exception (invalid input to strtoul()). I'm not intimately familiar with the GFA spec, but I assume that start IDs of zero are appropriate for all five fields?
Otherwise, we would need to a "you didn't give --start-ids/-s" warning to ids_main() after arg parsing.
Best,
kevin

Hi - thanks for the PR! I think this is probably valid, though I vaguely remember it being useful to have non-colliding seq/edge ID spaces (e.g., the first edge is numbered n+1 if the last seq is n). For now, this is good enough.


chore(dependencies): fix vulnerabilities

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


version bumped to 13.1 only in REST.pm. Refer https://www.ebi.ac.uk/p…

…anda/jira/browse/ENSCORESW-3488
Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion;
Review the contributing guidelines for this repository; remember in particular:

do not modify code without testing for regression
provide simple unit tests to test the changes
the PR must not fail unit testing
if you're adding/updating documentation of an endpoint, make sure you add/update the necessary parameters to the (template) configuration files in the ensembl-rest_private repo

Description
Version bumped to 13.1 for 102 release branching.
Use case
Release branching. Only a minor update in documentation.
Benefits
Release activity.
Possible Drawbacks
NA
Testing
NA
If so, do the tests pass/fail?
NA
Changelog
NA

Coverage remained the same at 44.44% when pulling 805cba6dc85127639862e96bbba4317f41e684d7 on 102_versionbump into 92327b3f7d906ab0dbb1c20333fe90ea4e046bc8 on master.


chore(e2e): change-passcode-spec-fix

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #
Additional comments?:
Developer Checklist:

[ ] Read your code changes at least once
[ ] Tested on iOS/Android device (e.g, No crashes, library supported etc.)
[ ] No console errors on web
[ ] Tested on Light mode and Dark mode*
[ ] Your UI implementation visually matched the rendered design*
[ ] Unit tests*
[ ] Added e2e tests*
[ ] Added translations*

Codecov Report

Merging #4000 (e491690) into main (b5077ca) will decrease coverage by 41.46%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #4000       +/-   ##
===========================================
- Coverage   57.78%   16.33%   -41.46%
===========================================
  Files         459      448       -11
  Lines       12347    12172      -175
  Branches     4082     4060       -22
===========================================
- Hits         7135     1988     -5147
- Misses       5124    10144     +5020
+ Partials       88       40       -48

see 241 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Cannot build latest version from master

Before I was using cocoapods so maybe there is something I don't understand re Swift Packages. I get the following errors after trying to compile the project:
Cookbook project Group
Validate Project Settings Group
/Users/patryk/Developer/workspace/ios/playground/Cookbook/Cookbook/Cookbook.xcodeproj Update to recommended settings
Cookbook Project Group
Package Loading Group
terminated(128): XPC_FLAGS=0x0 LOGNAME=patryk PATH=/Applications/Xcode.app/Contents/Developer/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin MallocNanoZone=0 HOME=/Users/patryk XPC_SERVICE_NAME=com.apple.xpc.launchd.oneshot.0x10000003.Xcode LD_LIBRARY_PATH=/Applications/Xcode.app/Contents/Developer/../SharedFrameworks/ CA_DEBUG_TRANSACTIONS=1 SHELL=/bin/zsh SECURITYSESSIONID=186a6 LaunchInstanceID=DEC86BD3-5BF1-4D93-9C92-45C669F6C961 CA_ASSERT_MAIN_THREAD_TRANSACTIONS=1 TMPDIR=/var/folders/p9/qr_6zy4x0pz4_q2kdt15tl5c0000gn/T/ COMMAND_MODE=unix2003 __CF_USER_TEXT_ENCODING=0x1F5:0x0:0x0 USER=patryk SSH_AUTH_SOCK=/private/tmp/com.apple.launchd.UWKGEX5Jhw/Listeners /Applications/Xcode.app/Contents/Developer/usr/bin/git -C /Users/patryk/Library/Developer/Xcode/DerivedData/Cookbook-cqoqdubjrdtiisghouzsouljfkfr/SourcePackages/repositories/AudioKit-ba28193d rev-parse --verify '9e9f1ab8312c71c02288f73034b0393faa5e0bd3^{commit}' output:
fatal: Needed a single revision
Cookbook Group
Package Loading Group
/Users/patryk/Developer/workspace/ios/playground/Cookbook/Cookbook/Cookbook.xcodeproj Missing package product 'AudioKit'
x-xcode-log://E7637D31-3A0F-4101-9E9F-C78FF1D105F1 Package resolution errors must be fixed before building
/Users/patryk/Developer/workspace/ios/playground/Cookbook/Cookbook/Cookbook.xcodeproj Missing package product 'Sliders'
x-xcode-log://E7637D31-3A0F-4101-9E9F-C78FF1D105F1 Package resolution errors must be fixed before building

after openning the project Swift Package Manager seems to correctly fetch both packages but still not able to build project.
using Macos Catalina 10.15.6 and XCode Version 12.1 (12A7403)

On xcode, go to File > Swift Packages > Update to latest package versions, that should fix it.

thanks! That solved the problem.


Error Unexpected token '<' in Nuxt

Error Unexpected token '<' in Nuxt

I also had the same problem. Have you found a solution yet? @geofany

I'm using pdf viewer from html embed tag
<embed         src="https://media.neliti.com/media/publications/52819-ID-kekerasan-seksual-terhadap-anak-dampak-d.pdf"         type="application/pdf"         height="500px"       >

to solve this issue, you need to create plugins on plugins folder ex :
- plugins/vue-pdf.js

import Vue from 'vue'
import pdf from 'vue-pdf'

Vue.component('vue-pdf', pdf)

add the plugin to nuxt.config.js on client mode
  ...
  plugins: [
    ...
    { src: '~/plugins/vue-pdf', mode: 'client' },
    ...
  ],
  ...

after you do the above and encounter error like this Typeerror: cannot read properties of undefined (reading'catch')
change your package.json for vue-pdf version to 4.2.0 version like this "vue-pdf": "4.2.0", this will temporarily solve the the issue until the owner decide to update this package again


upload the file on other server please

it is very difficult to download from baidu
please at least provide monthly update link on release section of github

New server cluster are planned to be online these weeks, hope this will help a lot. It will have a dedicated 100Mbps connection.
发自我的 iPhone

在 2018年8月25日，上午12:16，msh2050<EMAIL_ADDRESS>写道：
it is very difficult to download from baidu
please at least provide monthly update link on release section of github
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi there,
Before our server cluster get online this week, I just upload a preview
build of openthos 2.0 at microsoft onedrive:
https://1drv.ms/f/s!AlYwncDPcD8SgaoNXKFSw5ieqCk9oQ
please use ultraiso(commercial software with free trial) to burn this image
file to a USB disk (4GB or larger) then boot with that USB disk and follow
the instructions.
Cheers.
msh2050<EMAIL_ADDRESS>于2018年8月25日周六 上午12:16写道：

it is very difficult to download from baidu
please at least provide monthly update link on release section of github
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openthos/openthos/issues/13, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANb3gFzEzOFaBt43cpAAlN-pyfuPrM18ks5uUCbAgaJpZM4WLn-b
.

Why not use official release spaces like Fosshub?
https://www.fosshub.com/OPENTHOS.html


Responsive logo

References
This PR depends on:

#34

Objectives

Make the logo fit on all screens
Reduce overlap between logo and background image

Visual changes
Before

After

Great this PR 👏 !!!


tests(ticdc): fix broken storage tests

What problem does this PR solve?

Issue Number: ref #8772
What is changed and how it works?
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
Please refer to [Release Notes Language Style Guide](https://pingcap.github.io/tidb-dev-guide/contribute-to-tidb/release-notes-style-guide.html) to write a quality release note.

If you don't think this PR needs a release note then fill it with `None`.

/test all

Thanks!
/merge
/test all

/retest

/test cdc-integration-kafka-test