multiple operationId's to same function

Hello, I'm trying to find a way to have multiple API endpoints reuse the same operationId / method (ie like a generic handler), however get an error back from Flask:
AssertionError: View function mapping is overwriting an existing endpoint function
It looks like its possible in blueprint add_url_rule:  http://stackoverflow.com/questions/17256602/assertionerror-view-function-mapping-is-overwriting-an-existing-endpoint-functi
perhaps via a custom resolver - or is there an easier way?
Sort of coupled to this, is there anyway to add additional configuration into the swagger yaml that could get passed to the generic function / handler?
thanks
roger

It is not allowed by swagger specification thus Connexion does not support that. From the specification:
operationId (string)

Unique string used to identify the operation. The id MUST be unique among all operations described in the API. Tools and libraries MAY use the operationId to uniquely identify an operation, therefore, it is recommended to follow common programming naming conventions.

You are probably looking for vendor extensions. But Connexion currently does not pass any metadata of vendor extension to the handlers. Maybe that something that could change, but currently I do not see how it could work. If you are interested in that, you can suggest something for discussion.

Thanks for confirming, maybe if something like x-connexion-operation is optionally supplied, then this is used instead of operationId, which is left as whatever it is defined as (and as you highlight, has to be unique)
Eg:
paths:
/greeting/{name}:
post:
summary: Generate greeting
description: Generates a greeting message.
operationId: something-unique
x-connexion-operation: hello.post_greeting
x-connexion-operation-params:
-name: someKey1
value: someValue1
type: string
-name: someKey2
value: someValue2
type: string
}
Maybe also, x-connexion-operation-params could be used to additional pass params/configuration to the operation as a dict, additionally available somehow to the operation (worse case, written into flask.request perhaps...)
thanks

@rog555 With Connexion we try to stay as close as possible with Swagger specification and handle creation of standard REST API's. What you are proposing here goes far beyond what Connexion suppose to support. In that case I would recommend you to maybe fork Connexion and modify it to behave the way you are trying to describe here.


pretty print JSON quote and fix response header

Pretty print the JSON response from the  /quote?type=json API, so it appears formatted.
{
    "quote": "I arise full of eagerness and energy, knowing well what achievement lies ahead of me.",
    "author": "Zane Grey"
}

instead of
{"quote": "I arise full of eagerness and energy, knowing well what achievement lies ahead of me.","author": "Zane Grey"}

There is a reason to send non-pretty printed response , it's  for micro optimization of api and performance


Backport derive-where to v0.2

This is a non-breaking change. It replaces the macro with derive-where, which also fixes the Debug implementations and adds Ord and PartialOrd implementations.

Just realized that we should definitely not remove the base64 encoding for human-readable serializers in the backport, that's actually a breaking change.


Add Giant Country Horns Guest tags to 1991 shows

https://phish.in/1991-07-11 for example

These tags already got added at some point


Add delay instruction to assemble schedules and disassemble

Summary
This PR adds Delay instruction to be converted to Qobj and vice-vera.
Details and comments

@taalexander Good point - let's just support incoming delays and keep the previous assemble behavior. @SooluThomas do you have any questions?


Update package.yaml

for some reason the package upgrade was not being picked up by renovate.
this has been outdated since the beginning of feb, haha.

Seems like there's a missing configuration for renovate, thanks!


perception_pcl: 2.5.1-1 in 'iron/distribution.yaml' [bloom]

Increasing version of package(s) in repository perception_pcl to 2.5.1-1:

upstream repository: https://github.com/ros-perception/perception_pcl.git
release repository: https://github.com/ros2-gbp/perception_pcl-release.git
distro file: iron/distribution.yaml
bloom version: 0.11.2
previous version for package: 2.4.0-5

Hi @SteveMacenski,
I believe the changes here, specifically from https://github.com/ros-perception/perception_pcl/pull/384, is causing a regression with bosch_locator_bridge which is blocking the Iron sync.
I've commented in the PR on what should be fixed.
Do you think you'll have time to bloom a fix in the next day or two? Else would it be okay if I revert this bump?

On second thought, I think it's best to go ahead and revert the PR so that the sync is not blocked any further. https://github.com/ros/rosdistro/pull/40773

Agreed - leave this for next time to not block!


Linux Hardened Kernels 2024-05-03

Description of changes
I simply ran update.py (after fixing a bug in it) and it did its thing.
This obsoletes https://github.com/NixOS/nixpkgs/pull/301858.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

I wasn't able to run nixpkgs-review to completion here yet due to hardware limitations. But I built 6.6.30 and 6.1.90 successfully and ran the kernel-generic NixOS tests for them.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3889

@fabianhjr this is ready for another look

Oof, I should have waited for that before pinging, sorry. I fixed that now and eval passes.
Also, now that I've touched the correct files it looks like ofborg managed to add the reviewers :D

Will give some time to let other comment. LGTM
Thanks for contributing to nixpkgs! uwu


New version: ajeetdsouza.zoxide version 0.9.6

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

ADO Build not found.

(Automated response - build 895.)

Automatic Validation ended with:

Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Packages\ajeetdsouza.zoxide__DefaultSource\zoxide.exe returned exit code: 2
Executable C:\Users\validator\AppData\Local\Microsoft\WinGet\Links\zoxide.exe returned exit code: 2

(Automated response - build 896.)


Added requirements to setup.py

I just made a little change to let the users of dnstwist version on PyPi to use it without installing the dependencies manually.

The tool is designed to work fine and provide basic functionality without any additional packages. For this reason extra packages need to be installed manually.

Ok nevermind so I will add the required dependencies manually. Thank you for the answer

It does packages are extra then it would be nice if they are mentioned in extras_require in the setup.py file.


Fix apache/wicket#WICKET-7007

Links to issue https://issues.apache.org/jira/browse/WICKET-7007

Thank you, @brbog !


Enter buffer with

Is it possible to open a buffer from the quick menu using the  key instead of ?
I hadn't been able to create a keybind that only affects the harpoon buffer.
Thanks in advance for the answers!

I think you could do something like autocmd FileType harpoon nnoremap <buffer><silent> <space> :lua require('harpoon.ui').select_menu_item()<CR>

Worked like a charm, thanks!


scope $window and $container to plugin closure

I might be missing something. Not sure why $window and $container are global. If you want to lazyload two different sets of elements with different containers this is a problem.

No reason. I just always used it myself with one set of elements.


/platforms/android/README.md is depricatedwarning when using plugin

Here is the full warning I get when I try to run an android app which uses your plugin.

WARNING: The file: /Users/georgeedwards/Desktop/Personal-App/node_modules/nativescript-bluetooth/platforms/android/README.md is depricated, you can read more about what will be the expected plugin structure here: https://www.nativescript.org/blog/migrating-n-android-plugins-from-version-1.7-to-2.0

I'm using v1.1.1.

Hello Eddy
Any luck with this one?
We are evaluating your very useful plugin, however cannot build with it on Android... Getting the same error as mentioned above.  Same thing running your demo app (nativescript-bluetooth-demo).  Runs well in iOS but breaks on Android.
Any advise would be greatly appreciated.
I am using the latest native script CLI / download.
Thanks

Is it a warning or error?
You can simply remove the file to avoid it..

On 28 Jul 2016, at 19:20, vogatsiko<EMAIL_ADDRESS>wrote:
Hello Eddy
Any luck with this one?
We are evaluating your very useful plugin, however cannot build with it on Android... Getting the same error as mentioned above.  Same thing running your demo app (nativescript-bluetooth-demo). Runs well in iOS but breaks on Android.
Any advise would be greatly appreciated.
I am using the latest native script CLI / download.
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Closing since NS 4x has been released and plugins have went through many changes since then.


New integrator, and add some metadata to integrators.py

Addresses #679

[x] We should be able to understand what the PR does from its title only;
[x] There is a high-level description of the changes;
[x] There are links to all the relevant issues, discussions and PRs;
[x] The branch is rebased on the latest main commit;
[x] Commit messages follow these guidelines;
[x] The code respects the current naming conventions;
[x] Docstrings follow the numpy style guide
[x] pre-commit is installed and configured on your machine, and you ran it before opening the PR;
[x] There are tests covering the changes;
[x] The doc is up-to-date;

I see that code coverage is complaining about integrator_order, name_integrator and calls_per_integrator_step. These seem useful, but depending on your view, I could remove them

I see that code coverage is complaining about integrator_order, name_integrator and calls_per_integrator_step. These seem useful, but depending on your view, I could remove them

Yeah please remove them, depending on what you want to check there should be better ways to return these properties.


ERROR noalbs::chat::twitch: Twitch authentication failed

When I open noalbs.exe, I get error message that says ERROR noalbs::chat::twitch: Twitch authentication failed.
Where in the config file do I add the correct things to make the authentication to work?
I've watched this video if that gives any info: https://www.youtube.com/watch?v=FT53kTTi4kM

I don't see anything that tells me how to connect to twitch?


2 implement adc interface

ADC is functional on continuous mode. Python file added for quick and dirty calibration.

Fix the conflicts and ping me to review the code

@stevegtaylor any update on the conflicts?

I think I need your approval @jmadden173 to finish merging.
I am getting some build errors that I think will be easier to fix once the merge is complete and I can diagnose the problems in vscode.

I'll see if I can get around to test the version today. @stevegtaylor was build errors are you getting? Can you paste the platformio logs here so I know what to expect when I test?

Waiting for implementation of reading both voltage and current channels to approve PR


🐛 BUG: npx<EMAIL_ADDRESS>init in a directory without a package.json it crashes

Quick checklist

[X] I am using the latest version of Snowpack and all plugins.

What package manager are you using?
npm
What operating system are you using?
macOS
Describe the bug
running $ npx<EMAIL_ADDRESS>init in a directory without a package.json it crashes. 𐄂
Steps to reproduce

npx snowpack#3.8.0 init

node:internal/validators:119
    throw new ERR_INVALID_ARG_TYPE(name, 'string', value);
    ^

TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received undefined
    at new NodeError (node:internal/errors:363:5)
    at validateString (node:internal/validators:119:11)
    at Object.join (node:path:1172:7)
    at Object.<anonymous> (/Users/gedalos/.npm/_npx/b2e9e046e0de12f7/node_modules/snowpack/lib/cjs/config.js:25:20)
    at Module._compile (node:internal/modules/cjs/loader:1095:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1124:10)
    at Module.load (node:internal/modules/cjs/loader:975:32)
    at Function.Module._load (node:internal/modules/cjs/loader:816:12)
    at Module.require (node:internal/modules/cjs/loader:999:19)
    at require (node:internal/modules/cjs/helpers:93:18) {
  code: 'ERR_INVALID_ARG_TYPE'
}

Link to minimal reproducible example (optional)
No response

Thanks for filing. Would someone be willing to create a PR for this? It should be as simple as throwing a friendly error on Snowpack startup if no package.json is present.

IMHO I think that the existence of the package.json file should not be necessary to initialize snowpack. With version 3.6.2 it was not necessary.

Still not working on versions 3.8.1 and 3.8.2
:-(

Getting the same error as well with 3.8.1.


#2524 Improved immutability of Trace

#2524

remove setter of DefaultTrace

Current coverage is 40.16% (diff: 55.00%)

Merging #2540 into master will decrease coverage by <.01%

@@             master      #2540   diff @@
==========================================
  Files          1701       1701
  Lines         59681      59675     -6
  Methods       11578      11577     -1
  Messages          0          0
  Branches       8318       8317     -1
==========================================
- Hits          23975      23967     -8
- Misses        35706      35708     +2
  Partials          0          0

Powered by Codecov. Last update 261d962...29e5149


bugfix: avoid returning stale data in useDocument and useHandle

useDocument and useHandle had a footgun where after the input url changed, they would continue returning the old doc/handle for the previous input for 1 or more renders.
This behavior could cause problems in downstream computations or useEffect hooks which did not account for this brief inconsistency.
This patch fixes the problem by checking whether the input and output match before returning out of the hook. In useHandle we can just check the handle URL directly; in useDocument we need to store some additional state to remember the URL associated with a given doc.
tests

added unit tests which fail on old code and pass on new
using these updated hooks in https://github.com/inkandswitch/tiny-essay-editor/pull/42 and it instantly solved a ton of weird bugs throughout the app

I took a stab at simplifying this patch a bit @geoffreylitt -- useHandle it turns out could be mostly deleted and I wanted to at least take a stab at simplifying useDocument. This is where I wound up. On the whole I'm increasingly convinced that we should be using some kind of state management library instead of this but hey, here we are.
Tests pass, but note that enabling or disabling the setDoc(v) on line 53 has no effect on the success of the tests.

@pvh I simplified the code further based on your idea, and I also improved the tests to ensure that URL changes lead to immediately updated data on first render.

Merged & shipped in 1.1.10.


メッセージの数によって何らかの報酬を得るシステム

貢献して健康

How about just creating one RDB record when received one message?
Record should be like (guild_id, channel_id, message_id, author_id).
This enables you to get various statistics easily afterwards.

Creating データベース sounds めっちゃ awesome だけど it だろう require 大きめ storage なので I すべき think について the サイズ of テーブル

It's hard to estimate the amount of data in the table.
Instead I'll make a bot for testing which just creates a record on a received message and see how much storage is consumed after some weeks.
I want to ping Bacon for introducing the bot for testing, however, he is absent here. I'll ping him on discord.

ベーコンをべこーん

Bacon is baconed

Just beckoned the back of bacon

Beckondo (in a pronunciation of Taekwondo)

I didn't know that


Issue with guff Conversion After Finetuning with Unsloth

🦥 Unsloth: Will patch your computer to enable 2x faster free finetuning.
==((====))==  Unsloth: Fast Llama patching release 2024.6
\   /|    GPU: NVIDIA A100 80GB PCIe MIG 7g.80gb. Max memory: 79.151 GB. Platform = Linux.
O^O/ _/ \    Pytorch: 2.3.0+cu121. CUDA = 8.0. CUDA Toolkit = 12.1.
\        /    Bfloat16 = TRUE. Xformers = 0.0.26.post1. FA = True.
"-____-"     Free Apache license: http://github.com/unslothai/unsloth
Unsloth: unsloth/llama-3-8b-Instruct-bnb-4bit can only handle sequence lengths of at most 8192.
But with kaiokendev's RoPE scaling of 4.0, it can be magically extended to 32768!
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Unsloth 2024.6 patched 32 layers with 32 QKV layers, 32 O layers and 32 MLP layers.
I have the above development environment. Using the Unsloth library, finetuning Llama works well, and the trained model performs correctly when loaded and used for inference with the transformers TextStreamer.
However, during the conversion process with the embedded llama.cpp in Unsloth, no error messages appear, but running the converted model in ollama fails to work properly.

Notably, loading and converting a pretrained model without any training works fine. The issue arises only after performing even a single step of training with Unsloth.
Is there a solution for this problem?
Thank you.

Apologies a lot on the delay! My bro and I just relocated to SF, so it took a while to get back to you!
Interesting - its most likely the chat template. You could try our Ollama chat templates notebook here: https://colab.research.google.com/drive/1WZDi7APtQ9VsvOrQSSC5DDtxq159j8iZ?usp=sharing

Thank you for the update and the suggestion regarding the chat template. I appreciate your effort in providing a solution despite your recent relocation.
However, I've discovered that the issue persists even when not using Ollama. I've tested the converted GGUF file directly with llama.cpp using the following command:
./llama-simple -m ../ollama_model/llama-38-Q8_0.gguf -p "What is Your Name?"

The result shows the model only repeating exclamation marks after the input prompt, instead of generating a meaningful response. This behavior suggests that the problem might be in the GGUF file generation process itself, rather than being related to Ollama's chat templates.
I'm wondering if there's a way to check the version information of all dependent libraries in an environment where the model is working correctly. This could help identify any potential version mismatches or compatibility issues between my setup and a working configuration.
I'm happy to provide any additional information or logs that might be helpful in resolving this issue. Thank you for your continued support.


[PyTorch] Let Fused RoPE support CP with THD format

Description
Let the Fused RoPE kernels support CP with THD format.
Type of change

[ ] Documentation change (change only to the documentation, either a fix or a new content)
[ ] 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 not work as expected)
[ ] Infra/Build change
[ ] Code refractor

Checklist:

[ ] I have read and followed the contributing guidelines
[ ] The functionality is complete
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

/te-ci pytorch

LGTM.

/te-ci pytorch

Noticed that Xin started a new CI: [19245373]. Let's wait until it finishes.


client cursor moving on entering session

Maybe already reported these days.
In some cases the clients player cursor starts moving when the client user enters the session. The Player is not giving sound, but cursor is moving automatically without being able to stop it, because the player . The owners activated „Bearbeiten“ and „In Bearbeitung“ but nor „Player“.
We observed that this behaviour changed after the owner entered the session. Maybe there are some parameters not set, before the owner is online?

@mxordn we couldn't reproduce it locally. So if this situation happens again, please reopen the issue with potentially a sample session (id or name) for us to look into it.


Replicants documentation update

Development branch set up.
Touched up some documentation.
Sending you this to remind you to have a look at it when you get the chance

I'm a bit puzzled insofar as the pull request seems to include partially functioning b-metadata stuff? Is that right?

I'm a bit puzzled insofar as the pull request seems to include partially functioning b-metadata stuff? Is that right?
The commits before the request were intentional. The commits after (the partially functioning b-metadata stuff) were me pushing to a branch with an active pull branch (i.e. me doing something silly).
I'll split the intended changes off into a branch. That makes more sense - single issue branches with a pull request is best practice for a reason, I suppose.


docker.RegistryImage is used in example, but not found in package

File: docs/reference/pkg/nodejs/pulumi/docker/_index.md
In dynamic image example (https://www.pulumi.com/docs/reference/pkg/nodejs/pulumi/docker/#dynamic-image) there is a use of docker.RegistryImage, but there is no such resource in the package

I think that this will be fixed with the next update we do to the documentation. The content that is checked in to pulumi-docker correctly references getRegistryImage in place of RegistryImage: https://github.com/pulumi/pulumi-docker/blob/master/sdk/nodejs/remoteImage.ts#L28-L41


[CL Incentives] Re-add JoinTime and FreezeDuration to higher level position object

Closes: #4578
What is the purpose of the change
Removing JoinTime and FreezeDuration broke a number of open PRs, as even though they were technically being double-stored in state as key & value, they still need to be retrievable.

Change Position proto to PositionContent proto (doesn't include jointime and freezeduration)
Make new Position struct in model folder that has the full backwards-compatible position interface (including jointime and freezeduration)
Make GetPosition populate jointime and freeze duration using the parameters that are passed in (should return Position

Brief Changelog

added position struct that contains joinTime, freezeDuration

Testing and Verifying
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Just noticed this PR which overlaps with #4573.
That change is needed to correctly implement genesis: https://github.com/osmosis-labs/osmosis/pull/4546
Could you please wait for #4573 to get in? It is the final state we would need for genesis and is going to be broken by this change.
What #4573 does is the following:

proto-generates a Position struct to be added to genesis only
Only stores liquidity as sdk.Dec for position's state value

It seems to me that the ideas of using another Position struct are similar but this would just require resolving more conflicts

Just noticed this PR which overlaps with #4573.
That change is needed to correctly implement genesis: #4546
Could you please wait for #4573 to get in? It is the final state we would need for genesis and is going to be broken by this change.
What #4573 does is the following:

proto-generates a Position struct to be added to genesis only
Only stores liquidity as sdk.Dec for position's state value

It seems to me that the ideas of using another Position struct are similar but this would just require resolving more conflicts

I'm only going to get to #4573 on Monday so if this change is needed for further incentives work, feel free to merge

Just noticed this PR which overlaps with #4573.
That change is needed to correctly implement genesis: #4546
Could you please wait for #4573 to get in? It is the final state we would need for genesis and is going to be broken by this change.
What #4573 does is the following:

proto-generates a Position struct to be added to genesis only
Only stores liquidity as sdk.Dec for position's state value

It seems to me that the ideas of using another Position struct are similar but this would just require resolving more conflicts

sorry about that @p0mvn. I didn't see you PR or else i would've branched off of it. This PR will make lots of other incentives PR backwards compatibility  and fix conflicts therefore @AlpinYukseloglu  and I are pushing to merge this asap. Moving forward will communicate extra just in case


Better way to get results of a function

I was thinking something like this...
function define x
{
    determine 3;
    wait 1;
    determine 4;
}
y ^= x;
echo <{[x]}>; // Echoes "3|" (a list). The other queue is left to do whatever.
y &= x;
echo <{[x]}>; // After a second passes, will echo "3|4|".

Not sure about ^= notation, but the concept is there.
Should still be able to pass parameters via -a 1 -b 2 on the line after x

Can also use this for general commands.
y ^= do a with b;

if do (or whichever) command supports variable storage (as marked by a default-to-false AbstractCommand field), it will retranslate that command into do a with b -\0VARNAME y to be parsed by the command. If the named parameter is not present at run time, it will simply discard the result.


Refactor stylus themes to separate theme variable definitions

This allows users to easily extend themes and gives them an easier upgrade path as well. Inspired by our mistakes with #1090
Example:
@import 'ag-grid/src/styles/vars/fresh';

ag-background-1 = pink

@import 'ag-grid/src/styles/theme-common';

I'm not super familiar with Stylus, but SCSS has a !default property (https://robots.thoughtbot.com/sass-default) that would make this PR unnecessary.
After a quick glance, I couldn't find the equivalent in Stylus.

interesting. i'll need to revisit this when i have more time, as need to make double sure no side effects.

I'd love to have this feature for customizing a theme


Table delta not evaluating differences for USER-DEFINED type columns correctly

I have configured a postgis geometry as a duplicate column (property is of type Npgsql.PostgisPoint) like so:

(..).Duplicate(x => x.Geolocation, "geometry")

Running AssertDatabaseMatchesConfiguration reports a difference in that duplicated column, because data_type returned by Postgres is user-defined and not geometry.
I propose changing the SQL in Table.cs ConfigureQueryCommand to make it properly handle user-defined data types.
From:

select column_name, data_type, character_maximum_length

to:

select column_name, CASE WHEN data_type='USER-DEFINED' THEN udt_name ELSE data_type END AS data_type, character_maximum_length

Would that kind of change break some other functionality (all tests seem to pass)?
I can create a pull request with the proposed change if you think the change is safe.

If all the tests pass, that's fine by me -- and probably 2/3 of the tests would hit that query, so it's probably covered well;)
What are you doing with the Postgis functionality? We've had other folks ask about it from time to time, but we've never added any direct support because we didn't have concrete use cases to develop against.

Nothing too fancy with Postgis, I just have a point that I want to be able to apply geometric SQL queries easily to and have Marten read/write it properly. That works ok.
There seems to be another issue with schema diff when using CreateOrUpdate - hilo table always gets recreated because Marten generates the column type as biging default(0) and compares with bigint from the query referenced above. Will try to include a fix for that in the pull request as well.

Aah. Fixing the hilo comparison issue just lead me to issue with function comparison, which is not that trivial. The generated and function returned from Postgres don't seem to be that different, though (only the delimiters, language directive position etc).

@VladanStojanovic Hey, do you have this PR you could submit? Even if the function comparison is failing. I can address that one when I pull it in.

Duplicate issue with #959


Fix the "must check that a field has the correct value when a choice is changed" scripting integration test

We should not wait for an arbitrary amount of time, which can easily cause intermittent failures, but wait for a value change instead. Note that this patch mirrors the approach we already use in other scripting integration tests that also check for a value change; see e.g. the "must check that a field has the correct formatted value" test.
Fixes #17944.
Fixes a part of #17656.

/botio integrationtest


Responsive version: When clicking +, menu disappears

Hi,
In my cell, when I click (+) to expand a section, menu closes. I have to click on the menu again, and now, section is expanded.

Hello, have you solved it? here I'm having the same problem.

nop

Latest version has been redesigned which fixes this issue and many others v2.0.0


Question: Does the bam file needs indexing ?

Hello, I am taking the out sam file from my mapper and I am trying to convert it in bam format.
The command that I am using is this:
#/mnt/big/Tools/samtools-1.9/samtools view -S(b)u -@20 Aligned.out.sam  > alg.bam
#/mnt/big/Tools/samtools-1.9/samtools sort alg.bam -@20 -o ./alg.sorted.bam
#/mnt/big/Tools/samtools-1.9/samtools index alg.sorted.bam
is it correct? or indexing step is unnecessary?
Thank you

StringTie does not need or use a BAM index.


RuboCop 3ls.rb

Create a branch with minor changes in order to test
switch from travis-ci.org to .com

Fixes all RuboCop offenses in script/s3ls.rb
Updates RuboCop todo list

Closing. Will re-open in a few minutes under another name "Travis switch", with an additional commit.

Closing. Will re-open in a few minutes under another name "Travis switch", with an additional commit.


Is exists a built-in function that makes sys.now() into timestamp format?

I want to create a document of Cloud Firestore, and containing the timestamp value. but, there isn't such a function that converts a floating value(sys.now) to timestamp format. Doesn't the function that plays the same role as below datetime.timestamp() exist?
Thank you.
Sample
main:
  params: [args]
  steps:
  - init:
      assign:
        - project_id: ${sys.get_env("GOOGLE_CLOUD_PROJECT_ID")}

  - test_firestore:
      call: googleapis.firestore.v1.projects.databases.documents.createDocument
      args:
        parent: ${"projects/" + project_id + "/databases/(default)/documents/xxx"}
        collectionId: yyy
        documentId: xxx-yyy
        body:
          fields:
            name:
              stringValue: kim
            created_at:
              timestampValue: ${datetime.timestamp(sys.now, 'JST')}
      result: resp

  - check:
      return: ${resp}

This problem has been resolved as time.format function.
- timestampValue: ${datetime.timestamp(sys.now, 'JST')}
+ timestampValue: ${time.format(sys.now(), "Asia/Tokyo")}

So, I'm going to the closing this issue.


Fix memory leak in TCP

Signed-off-by: gargnupur<EMAIL_ADDRESS>For an explanation of how to fill out the fields, please see the relevant section
in PULL_REQUESTS.md
Description: Fix memory leak in TCP
Ref: https://github.com/istio/istio/issues/22007
Risk Level: Low
Testing: Tested e2e in istio/proxy
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

/cc @howardjohn , @mandarjog , @kyessenov


LocationServicesDialogBoxModule  is not abstract and does not override abstract method onActivityResult(int,int,Intent) in ActivityEventListener

I'm using RN 0.26.0 and after installing "react-native-android-location-services-dialog-box" I got this error
...\showlocationservicesdialogbox\LocationServicesDialogBoxModule.java:12: error: LocationServicesDialogBoxModule is not abstract and does not override abstract method onActivityResult(int,int,Intent) in ActivityEventListener
class LocationServicesDialogBoxModule extends ReactContextBaseJavaModule implements ActivityEventListener {
^
C:\RN\1\node_modules\react-native-android-location-services-dialog-box\android\src\main\java\com\showlocationservicesdialogbox\LocationServicesDialogBoxModule.java:22: error: method does not override or implement a method from a supertype
@Override
^
C:\RN\1\node_modules\react-native-android-location-services-dialog-box\android\src\main\java\com\showlocationservicesdialogbox\LocationServicesDialogBoxModule.java:77: error: method does not override or implement a method from a supertype
@Override
^
3 errors
FAILED
FAILURE: Build failed with an exception.

Please Use rn 30 and older versions v1.1.7
Use 1.1.4 for other lower versions.

Please see rn30 and the change of distortion of the above.

I am using the below config and still the above error.
"react-native": "0.44.3",
"react-native-android-location-services-dialog-box": "2.3.2",

@roots-ai please read releases note

We get this compilation error since we upgraded to RN 0.50, running latest version "react-native-android-location-services-dialog-box": "2.4.3"
Any ideas?

Seems that with latest RN some compatibility breaks happened for android that needs to be applied here.

I've figured out that these compilation errors happens when "react-native-image-crop-picker" is installed in an application.
Without this library everything gets compiled.
Any ideas?

this bug may be a react-native link issue.
try to manual installation please.


Add Nong Song: c u l8r on feat. soulja boy

Song Name
c u l8r on feat. soulja boy
Artist Name
fur:trash vs. coltrane
Source
Youtube
Youtube Link
https://www.youtube.com/watch?v=9oNWikZvcJ4
Direct File Link
No response
Song ID
1001
Start Offset [ms]
0
Extra
No response

ok github actions


db add columns price & maxseats in tbl TicketEventDates

Added new columns to db

all good in the hood


Add clang option for specifying OS version to application/library build documents

There is a clang option for specifying the OS version that the output is built for. If you do not specify it, it defaults to the current OS version.
If you try to link two files that were built for diffent versions you will see a warning. The Pony runtime is built for OS X 10.8, so when you build c++ apps or Machida you will see warnings about how the versions for different files are not the same.

@SeanTAllen hey, remember this issue? This still comes up when we build Machida on the Mac. We might want to think about addressing this before hard launch.

As far as I can tell so far, it has no impact other than a warning. Have you seen any indication of an issue beyond that @aturley ?


fix: address false-positive matches for autorelease branch naming

See https://github.com/googleapis/release-please/pull/1311

@bcoe, could you please take a look when you have time?

@joeyparrish apologies for being slow on this one, looks like we already landed a newer release please in the last release:
https://github.com/google-github-actions/release-please-action/blob/main/package.json#L29

No worries!  Thanks for letting me know.


Error installing Sensu on Windows Server 2012R2

When trying to install Sensu fresh on a Win2012R2 server, I get the following error:
Invalid relationship: File[C:/opt/sensu/ssl] {require => Package[sensu]}, because Package[sensu] doesn't seem to be in the catalog
I have a feeling it's because the module is expecting to see a package called 'sensu' whereas on Windows, the package name gets changed to 'Sensu' (uppercase S) and this is throwing the module right off.
Is this a known issue and is there a fix somewhere?

@ghoneycutt I've resolved the Package[Sensu] error by rebasing the work in #635.  I'm running into this issue now:
vagrant provision win2012r2-client
==> win2012r2-client: Running provisioner: shell...
    win2012r2-client: Running: tests/provision_client_win.ps1 as c:\tmp\vagrant-shell.ps1
==> win2012r2-client: Info: Loading facts
==> win2012r2-client: Info: Loading facts
...
==> win2012r2-client: Warning: Scope(Class[Sensu::Package]): Managing a local windows user is not supported
==> win2012r2-client: Notice: Compiled catalog for win2012r2-client.home in environment production in 1.59 seconds
==> win2012r2-client: Error: Parameter ensure failed on Package[sensu]: Provider windows must have features 'upgradeable' to set 'ensure' to 'latest' at C:/ProgramData/PuppetLabs/code/environments/production/modules/sensu/manifests/package.pp:64

It seems we need to specify an explicit version for windows.  I notice sensu::version defaults to latest though.
Do you have any guidance on how to handle this for windows?  Maybe specify an explicit default version via hiera data in the module with an entry in the hierarchy for osfamily?

Fixed in v2.8.0

@jeffmccune this seemed to work from my Mac -> Windows doing acceptance testing which includes the sensu module in my profile for windows but had someone run from Windows -> Windows and there is a newline/CR between 2012r2 and sensu-version...
https://github.com/sensu/sensu-puppet/blame/master/manifests/package.pp#L72
Guessing possibly something with how the template is rendered from Windows vs a *nix platform? Template should be working without carriage return -%> but I figured I'd pass along that there might be issue here that I'm missing with $pkg_url_dir

What do you mean running acceptance tests windows to windows? I'm not following you.

@ghoneycutt I have a profile_base::windows class that includes sensu module and configurations.
So when I run these beaker tests from Mac to windows 2012r2 nodeset it works fine. When run from Windows 10 to same windows 2012r2 nodeset it has issues with carriage return.
To get around conditional here I hardcoded value for windows_pkg_url and it still has another carriage return issue:
  Notice: /Stage[main]/Sensu::Package/Package[sensu]/ensure: ensure changed '<IP_ADDRESS>' to '<IP_ADDRESS>
  '
  Info: Class[Sensu::Package]: Scheduling refresh of Service[sensu-client]

@ghoneycutt just to give full code:
class profile_base::windows {
  include ::profile_base::sensu
}

class profile_base::sensu (
  Boolean $server,
  String $rabbitmq_host,
  String $rabbitmq_password,
  String $rabbitmq_vhost,
  Array[String] $subscriptions,
  Array[String] $sensu_additional_plugins,
  String $client_bind,
  String $client_port,
  Hash $client_plugins_script,
  Hash $client_keepalive,
  Hash $client_custom,
  String $default_client_name,
  Array[String] $sensu_plugins,
  String $sensu_client_prefix,
  String $version,
  Optional[Boolean] $enterprise,
  Optional[Boolean] $enterprise_dashboard,
  Optional[String] $enterprise_user,
  Optional[String] $enterprise_pass,
  Optional[Boolean] $manage_user = false,
  Optional[String] $package_checksum = undef,
  Optional[String] $windows_repo_prefix = undef,
  Optional[Boolean] $windows_logrotate = false,
  Optional[String] $windows_log_number = undef,
  Optional[String] $windows_log_size = undef,
  Optional[String] $windows_pkg_url = undef,
)
{
  assert_private()
  #notify { 'Applying class profile_base::sensu': }

  if $facts['ec2_metadata'] {
    $client_name = $facts['ec2_metadata']['instance-id']
  } elsif $::hostname {
    $client_name = $::hostname
  } else {
    $client_name = $default_client_name
  }

  $client_name_with_prefix="${sensu_client_prefix}${client_name}"

  if $::ipaddress {
    $client_address = $::ipaddress
  } else {
    $client_address   = '<IP_ADDRESS>'
  }

  class { '::sensu':
    version              => $version,
    install_repo         => true,
    server               => $server,
    api                  => $server,
    client               => true,
    use_embedded_ruby    => true,
    rabbitmq_host        => $rabbitmq_host,
    rabbitmq_password    => $rabbitmq_password,
    rabbitmq_vhost       => $rabbitmq_vhost,
    subscriptions        => $subscriptions,
    client_name          => $client_name_with_prefix,
    client_address       => $client_address,
    client_keepalive     => $client_keepalive,
    client_custom        => $client_custom,
    client_port          => $client_port,
    client_bind          => $client_bind,
    enterprise           => $enterprise,
    enterprise_dashboard => $enterprise_dashboard,
    enterprise_user      => $enterprise_user,
    enterprise_pass      => $enterprise_pass,
    package_checksum     => $package_checksum,
    windows_repo_prefix  => $windows_repo_prefix,
    windows_logrotate    => $windows_logrotate,
    windows_log_size     => $windows_log_size,
    windows_log_number   => $windows_log_number,
    manage_user          => $manage_user,
    windows_pkg_url      => $windows_pkg_url,
  }

# Install plugins and scripts
...
}

profile_base::sensu::version: '0.28.5-2'
profile_base::sensu::package_checksum: 'ae321125091331771aa6f9edbcbacb0a'
profile_base::sensu::windows_repo_prefix: 'http://artifactory.domain.local/sensu/msi'
profile_base::sensu::windows_logrotate: true
profile_base::sensu::manage_user: false
profile_base::sensu::windows_pkg_url: 'http://artifactory.domain.local/sensu/msi/2012r2/sensu-0.28.5-2-x64.msi'

So without windows_pkg_url it seems to have a carriage return after pkg_url_dir but when I use the windows_pkg_url it has issue with carriage return with sensu::version.
Checked and not sure if this is correct but his git config core.autocrlf = true (on my mac it's false) is set which is normally necessary for windows users checking out code and spinning up linux vagrant boxes to run puppet code on... but maybe when it's windows it needs to be different? Sorry for so much info just wanted to provide as much detail as possible.

@cdenneen I appreciate all the effort on this.
We aren't planning to support running tests from a windows host and we don't have the platform to test with. If there's an issue with the module running on Windows, we certainly do want to get that fixed. Is that the case or is the issue only with running acceptance tests?
If you can figure out how to get the acceptance tests to work with windows, we are more than happy to accept a pull request.


Feat/java sdk

PR-Codex overview
Detailed summary

Added new classes Collection, Database, Document, ResultSet, TypedMessage, IndexProvider, EIP712TypedMessage, StorageProvider
Added a new method main in the Main class
Added a new test method testSendMutation in the StorageProviderTest class
Added dependencies in the pom.xml file
Updated the MutationUtil class with a new test method test_java_sdk_verfiy_ut
Updated the parse_mutation macro in the mutation_utils.rs file

The following files were skipped due to too many changes: java/src/main/java/network/db3/client/Client.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report

Merging #538 (fb3af69) into main (2a6b48c) will increase coverage by 0.20%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #538      +/-   ##
==========================================
+ Coverage   55.39%   55.59%   +0.20%
==========================================
  Files          72       72
  Lines       13994    14006      +12
  Branches       61       61
==========================================
+ Hits         7752     7787      +35
+ Misses       6242     6219      -23

Flag
Coverage Δ

rust
55.59% <100.00%> (+0.20%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/node/src/storage_node_light_impl.rs
27.23% <ø> (ø)

src/node/src/mutation_utils.rs
59.06% <100.00%> (+17.96%)
:arrow_up:

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


Remove deprecated underscore parameters

This PR removes (previously deprecated) underscore parameters that make mget queries fail on elastic search >=7.0.
See https://www.elastic.co/guide/en/elasticsearch/reference/current/breaking-changes-7.0.html#_camel_case_and_underscore_parameters_deprecated_in_6_x_have_been_removed.

I wonder if we should do a PR upstream as well to backport https://github.com/olivere/elastic/commit/5d20c799557a7637d81468010fc1246cef0b8ec6 to the v6 client

We could, it looks like it always been accepting parameters both with and without the underscores for a while (and until 7.0), so it should not break anything, https://github.com/elastic/elasticsearch/blob/15a62448343fd24f8e63f43b1e4b16f50005e4a5/core/src/main/java/org/elasticsearch/action/bulk/BulkRequest.java#L312.


Type hinting fixed for riot endpoints

Acceptance Criteria

Type hinting errors are resolved for riot endpoints

Resolved in #325


Adding FormData to WebWorker

I'm trying to add FormData interface to webworker.d.ts (https://github.com/Microsoft/TypeScript/issues/14085). IIUC, I need to add to inputfiles/knownWorkerInterfaces.json. It generates:
interface FormData {
    append(name: any, value: any, blobName?: string): void;
}

declare var FormData: {
    prototype: FormData;
    new (form?: HTMLFormElement): FormData;
}

The problem is the FormData's constructor. Is there a way to add different constructors for DOM and Workers?

This is fixed by #231.

Cool!


Build failed

Hi, I am trying to create the GriefDefender jar file but an error always occurs.
I have already downloaded the API project but that does not solve it.

It cannot find the spigot jar. GD issue tracker is not meant for helping you build spigot.


[Wait for #2247][Application] add RMS normalization for LLaMa

I added rms normalization layer for llama app.
now it only supports inference(not training).
Signed-off-by: Seungbaek Hong<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #2250. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2250-202307201131040.18005800247192-fa46da3b762359328a6c880588864467ff51a31d/.

:octocat: cibot: @baek2sm, Applications/LLaMA/jni/rms_norm.cpp includes bug(s). Please fix incorrect coding constructs in your commit before entering a review process.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2250-202307252046040.57666707038879-8f87d0859f5c75224edcea0dc245b06460fd33c8/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2250-202307252054170.54860901832581-b072d92ddb01d772ee8116b41555f856106d1dcf/.

:octocat: cibot: @baek2sm, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/2250-202308011823590.09294605255127-908df4a4b7645dcec822b09c1988d9c31ad79508/.


pijul: 1.0.0-beta.6 → 1.0.0-beta.7

Description of changes
No changelog is available, basic compilation testing under NixOS Linux x86_64
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

pijul

Thanks!


Check Upper Case

Description
Check if the given character is upper case (A-Z).
Input  : A
Output : Upper Case

Program added successfully :tada:
Thanks for your contribution :hugs:


[translation]: Update translations for contributing documentations

We need your help in translating the contributing documentations for Arabic, Chinese, Portuguese, Russian and Spanish.
There are the individual issues that you can help on, if you speak any of these:

[x] Arabic: #18311
[ ] Chinese: #18310
[ ] Portuguese: #18309
[ ] Russian: #18308
[ ] Spanish: #18307

Please reach out to us in Forum or Contributors Chat room for any queries.
Happy contributing.
There is still work pending on this. When you make a PR for this particular issue, do not tag it with Closes #XXXXX instead just reference it with #XXXXX

i want to help you to translate in spanish, but i cannot see the issue

Chinese: #18310
I want to help you translate the contributing documentations for Chinese.

@Alhern, you did a great job translating the Guide README to French here. Maybe you could help us translate the Contributing Doc to French if you have some time?

Hey @scissorsneedfoodtoo ! I'd love to help with translating the Contributing Doc to French, I'm going to start working on it.

I'd love to help with translating the FreeCodeCamp Curriculum in Italian and Romanian, Do you know when we can start working on it and when it will be open to work on?

Hi,
@Alhern and @lindakovacs thanks a lot for taking the time to contribute to freeCodeCamp.org
At the minute we are focused to have translations only in the 5 additional languages as you see on the contributing doc. This is primarily because of the operational logistics involved in maintaining the separate platforms. We want these languages to be ready and stable before going firth with other languages.
We will definitely consider additional languages in future, however we do not have any timeline on these.
We will make an announcement when that happens.
Thanks.

Hi Mrugesh,Thank you for getting back to me.I didn't know all of this and I just sent 2 pull requests for Creating the Romanian and Italian translation for the Contribution Guidelines.Mostly I did it after I read Quincy's article "I just got my free Hacktoberfest shirt. Here’s a quick way you can get yours.".At this point, I don't know if I should continue up to 5 pull requests as it says in the article and you just put it on the side until you are ready to go with the Romanian and Italian versions or I should try other modifications for the pull requests.I'm new myself to all of this stuff and I'm trying to learn as fast as possible.Please let me know how you like me to proceed or what works best in your interest.Thank you,Linda Kovacs
I just got my free Hacktoberfest shirt.  Here’s a quick way you can get y...
Quincy Larson
Note: this article is now up-to-date for getting a 2018 Hacktoberfest shirt.
|
|
|
--
Linda KovacsRemote Freelancer Web Developer, Web Designer, and Graphic Designer
<EMAIL_ADDRESS>http://www.lindakovacs.x10host.com/
On Friday, October 12, 2018, 6:27:35 PM EDT, mrugesh mohapatra<EMAIL_ADDRESS>wrote:

Hi,
@Alhern and @lindakovacs thanks a lot for taking the time to contribute to freeCodeCamp.org
At the minute we are focused to have translations only in the 5 additional languages as you see on the contributing doc. This is primarily because of the operational logistics involved in maintaining the separate platforms. We want these languages to be ready and stable before going firth with other languages.
We will definitely consider additional languages in future, however we do not have any timeline on these.
We will make an announcement when that happens.
Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@raisedadead I just closed my PR #18661 and #18632.
Thank you for your help.

I can help with Arabic #18311 translation but I have a problem with Document Direction as Arabic is RTL Language. Is there a solution for this issue ?

I don't have any translation skills, but I fixed the links in the READMEs to go to the correct language documents that everyone is doing such an awesome job translating. #28207

Thanks for working on the translations for the "Contributing to freeCodeCamp". Here on we are making contributing to translations for contribution docs ad hoc.
You are free to make pull-requests for improving these translations. We are closing the issue as these do not need tracking per se.
You can raise a fresh issue if you find something incorrect in the existing docs.


z-index layer conflict between mobile menu and pagination button

This issue does not seem to be present on develop since PaginationComponent has gone through major refactor.  Please retest.

was not able to reproduce it on tablet and mobile view, on desktop, pointing to dev10.
Will leave it to @developpeurweb to retry on a real device, pointing to dev10.
But I believe this was fixed in #6060.

@developpeurweb confirmed it looks good now, pointing to dev10.


Fix hardcoded raycast distance in viewport object picking

fix hardcoded raycast distance in viewport object picking
having the raycast distance hardcoded to 10000 caused input events to not be registered in very large 3D scenes.
This resolves the issue by using the cameras far distance instead. Creating the more predictable behavior of "if an object is visible, it will be picked by the viewport."
More info about the issue can be found in #49735
Closes #49735

Thanks! And congrats for your first merged Godot contribution :tada:

Thanks! And congrats for your first merged Godot contribution tada

happy to help!

Cherry-picked for 3.5.


bug: Clear fill setting does not show icons in toolbar buttons.

Bug Report
Ionic version:

[x] 4.4.1-dev.201905212016.fb63a6f
Current behavior:

The icon is not visible. Only the button text is shown.
Expected behavior:

The icon and the text of the button are shown.
Steps to reproduce:

Look at code below with same version of ionic as stated above.
Related code:

<ion-toolbar>
  <ion-buttons slot="secondary">
    <ion-button fill="clear">
      <ion-icon slot="start" name="contact"></ion-icon>
      Contact
    </ion-button>
  </ion-buttons>
</ion-toolbar>

Other information:

I noticed this issue as a side-effect of upgrading to the new nightly build to fix #16834
After some inspection I figured out that in this new version the icon-only buttons got a fixed height and width. This seems to have to do with this issue.
Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.12.0 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.4.1-dev.201905212016.fb63a6f
   @angular-devkit/build-angular : 0.13.8
   @angular-devkit/schematics    : 7.3.8
   @angular/cli                  : 7.3.9
   @ionic/angular-toolkit        : 1.5.1

System:

   NodeJS : v11.0.0 (/usr/local/Cellar/node/11.0.0/bin/node)
   npm    : 6.9.0
   OS     : macOS Mojave

Thanks for the issue! I am not able to reproduce this with the code provided. Could you please provide some code I can use to reproduce? https://codepen.io/brandyscarney/pen/YbYVMo

Nevermind I was able to reproduce this using a setTimeout on the button text, thanks! Will have a fix in soon.

Awesome @brandyscarney. I confirm everything working and looking pretty too: Sweeeet!
Thank's for digging into it and fixing it so quickly.


Textarea: Add modifiers for resize properties

This issue is related to an issue in React: https://github.com/patternfly/patternfly-react/issues/2839
There are two parts:

Make a modifier for the text area component that sets resize to just be on the vertical axis. Also add a modifier for the horizontal axis. We could also have a modifier for resize none, but that would be up to design.

Currently in Core, we are able to horizontally resize the TextArea component outside of its parent container. Resizing should only be limited to within its parent constraints. See screenshot here:

We story-pointed on the enhancement - step 1 of this issue

:tada: This issue has been resolved in version 2.34.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


wrong encoding

When i run python make_data_splits.py i get the following output.
validated that titles in scraped pk files are the same as in ./splits.json
90 books total, train: 72, val: 10, test: 8
4695 chapters total, train: 3746, val: 440, test: 509
9942 summary-chapter pairs total, train: 7733, val: 1028, test: 1181

composing multi-chapter summaries from single-chapter summaries...
4695 chapters total, train: 3746, val: 440, test: 509
12412 summary-chapter pairs total, train: 9585, val: 1377, test: 1450
wrote to raw_splits/train.pk
wrote to raw_splits/val.pk
wrote to raw_splits/test.pk
wrote to raw_splits/pair_ids.json

WARNING: 63 pair ids are missing from collected summaries, but exist in expected summaries
You should consider rescraping
My Ántonia.Book 1: Chapter 1-4.barrons
My Ántonia.Book 1: Chapter 1-4.cliffsnotes
My Ántonia.Book 1: Chapter 1-4.gradesaver
My Ántonia.Book 1: Chapter 1-4.monkeynotes
My Ántonia.Book 1: Chapter 1-4.novelguide
My Ántonia.Book 1: Chapter 13.cliffsnotes
My Ántonia.Book 1: Chapter 14-16.barrons
My Ántonia.Book 1: Chapter 14-16.cliffsnotes
My Ántonia.Book 1: Chapter 14-16.gradesaver
My Ántonia.Book 1: Chapter 14-16.monkeynotes
My Ántonia.Book 1: Chapter 14-16.novelguide
My Ántonia.Book 1: Chapter 17-18.barrons
My Ántonia.Book 1: Chapter 17-18.cliffsnotes
My Ántonia.Book 1: Chapter 17-18.gradesaver
My Ántonia.Book 1: Chapter 17-18.monkeynotes
My Ántonia.Book 1: Chapter 17-18.novelguide
My Ántonia.Book 1: Chapter 19.cliffsnotes
My Ántonia.Book 1: Chapter 5-8.barrons
My Ántonia.Book 1: Chapter 5-8.cliffsnotes
My Ántonia.Book 1: Chapter 5-8.gradesaver
My Ántonia.Book 1: Chapter 5-8.monkeynotes
My Ántonia.Book 1: Chapter 5-8.novelguide
My Ántonia.Book 1: Chapter 9-12.barrons
My Ántonia.Book 1: Chapter 9-12.cliffsnotes
My Ántonia.Book 1: Chapter 9-12.gradesaver
My Ántonia.Book 1: Chapter 9-12.monkeynotes
My Ántonia.Book 1: Chapter 9-12.novelguide
My Ántonia.Book 2: Chapter 1-4.barrons
My Ántonia.Book 2: Chapter 1-4.cliffsnotes
My Ántonia.Book 2: Chapter 1-4.gradesaver
My Ántonia.Book 2: Chapter 1-4.monkeynotes
My Ántonia.Book 2: Chapter 1-4.novelguide
My Ántonia.Book 2: Chapter 11-15.barrons
My Ántonia.Book 2: Chapter 11-15.cliffsnotes
My Ántonia.Book 2: Chapter 11-15.gradesaver
My Ántonia.Book 2: Chapter 11-15.monkeynotes
My Ántonia.Book 2: Chapter 11-15.novelguide
My Ántonia.Book 2: Chapter 5-7.barrons
My Ántonia.Book 2: Chapter 5-7.cliffsnotes
My Ántonia.Book 2: Chapter 5-7.gradesaver
My Ántonia.Book 2: Chapter 5-7.monkeynotes
My Ántonia.Book 2: Chapter 5-7.novelguide
My Ántonia.Book 2: Chapter 8-10.barrons
My Ántonia.Book 2: Chapter 8-10.cliffsnotes
My Ántonia.Book 2: Chapter 8-10.gradesaver
My Ántonia.Book 2: Chapter 8-10.monkeynotes
My Ántonia.Book 2: Chapter 8-10.novelguide
My Ántonia.Book 3: Chapter 1-4.barrons
My Ántonia.Book 3: Chapter 1-4.cliffsnotes
My Ántonia.Book 3: Chapter 1-4.gradesaver
My Ántonia.Book 3: Chapter 1-4.monkeynotes
My Ántonia.Book 3: Chapter 1-4.novelguide
My Ántonia.Book 4: Chapter 1-4.barrons
My Ántonia.Book 4: Chapter 1-4.cliffsnotes
My Ántonia.Book 4: Chapter 1-4.gradesaver
My Ántonia.Book 4: Chapter 1-4.monkeynotes
My Ántonia.Book 4: Chapter 1-4.novelguide
My Ántonia.Book 5: Chapter 1-3.barrons
My Ántonia.Book 5: Chapter 1-3.cliffsnotes
My Ántonia.Book 5: Chapter 1-3.gradesaver
My Ántonia.Book 5: Chapter 1-3.monkeynotes
My Ántonia.Book 5: Chapter 1-3.novelguide
My Ántonia.Introduction.cliffsnotes
to fix, try deleting and rescraping the books with issues (see FAQ.md)

The summaries for My Ántonia  are not scraped, because it contains the letter 'Á'. I looked into the code and the following code seams to be the problem:
https://github.com/manestay/novel-chapter-dataset/blob/4c4e3a8dd0c3ef08e14e251ead5e82f5ea62d9a7/scraping/scrape_lib.py#L31-L40
The requests library sometimes applies ISO-8859-1 encoding when calling page.text. That way My Ántonia becomes My Ã\x81ntonia The html however specifies the encoding mostly as utf-8 (<meta charset="utf-8">). This is ignored by requests. I recommend to rewrite the code like this:
def get_soup(url, encoding=None, sleep=0):
    s = requests.Session()
    retries = Retry(total=4, backoff_factor=.3)
    s.mount('http://', HTTPAdapter(max_retries=retries))
    page = s.get(url)
    if sleep:
        time.sleep(sleep)
    return BeautifulSoup(page.content, 'html5lib', from_encoding=encoding)

page.content gives the html as binary. That way BeautifulSoup is able to apply its own methods to determine the encoding which also consider the content of the html document.
I already tested it and it fixed the issue for me.

hi @dominik-schwabe , another great catch from you. Much appreciated!


Gradle plugin: better detection of logback config and improved logback.xml generation

This is refactoring caused by #3946
Currenly, logback.xml is generated linked to processGatlingResources task.
Current issues

Another issue is that existing logback detection is bound to hardcoded path (src/gatling/resources) and neglects files other than logback.xml

Scope of changes

Detect if gatling resources contains either logback.xml, logback.groovy etc. Currently the file location is hardcoded, this should be changed to analyze source directories and only limit sources to gatling resources.
Logback files should be searched using Grade sourceSet API, not using hardcoded pathes.
If gatling resources has logback configuration - do nothing.
If gatling resources doesn't include any logback config files - then

generate logback.xml somewhere in build directory. A separate directory should be used, the file must not be put into build/resources as now. The generated file is better to place into build/generated to prevent interference with real resources processed by Gradle.
As now generated logback.xml should use exiting gatling.logLevel parameter to configure root logger level

Generated logback.xml should be passed to every forked JVM that executes Gatling via additional -Dlogback.configurationFile system property as logback docs explains.
Make sure current tests work.
Add test cases for other files, like logback.groovy etc.

I agree we should use the SourceSet API, but we shouldn't search in the resources directory by in output one. Ultimately, what we want is to detect files that are in the classpath, not files that will be/have been copied somewhere in the classpath.

Another issue is that existing logback detection is bound to hardcoded path (src/gatling/resources) and neglects files other than logback.xml

This is a bug that should have its own ticket, not part a this refactoring. Agree?

I extracted the "other issue" part as a dedicated ticket as it's not strictly related to this refactoring, see #3952

oh my god, why :((
what you had extracted is exaclty this issue, they can not be separated.
please close #3952, it's excessive

Why would missing to detect logback-test.xml and logback.groovy files be related to the large refactoring with is the topic of this issue?
#3952 could perfectly be implemented on top of the existing implementation without this refactoring.
PS: please refrain from using disrespectful comments such as "oh my god, why :((".


[Addition]: Deeper farm again in France ?

Hotspot b58 Addresses
112mwhJoFgDTEDuuyhcGtLUhNPTrgGzc59WpxRTq4wexZYK85ZH4
11qnKxCn6AXRT1jdhftSHgLFktqFdtLDnyTWHkoL59UWekCjCVD
11U5TG5FLMJ3ZjqEDucQCeXybNZJTgKdbSvnt9ccUod6k6gAaaA
11TKiFPit5jyEDRtKJwHhMVGc9XhxomaJ4n7UnUhdBgw4b6gTFX
112amfqRW4R2Sy4L5JGxEBu5HtPE5kBy4cE9jhDVk6UQiACrLCMU
11LZPH71phXzVL4QshcpBKy9W1DnHmCYoCvpjUUxCMxj5iWPyv1
11ztfiHXqiBEyC4aWbJaynSpJ71Reb71nANXKFr9A2ysTAjPu9f
1124J8YqkWepAspVugqDs8xxVpqNDXNFnau5ruF5ngs9SWid8p6W
17f4Dx96cWCXLiGh3vyHWQvNP4674DxQAJUBEwDi8MZNnZRNe7
112iHW9T6nKP9cDdnWsZQiDsgyjf4hjzFwJwWZ3AuVQ6DQz2RVmu
1122HkK1tM9c2n3pbBRapS4cUpqBV1eDBbmbkmsmbhHX4Xqidc17
11Ssify7mqddj77YtP1iLDtmkjA2Z6xEbCvBET56oYBH65zkoRh
112r7M27LjSEvLuB6S9NBWh8eVa4KKtAB6dVpxid4nM84YF192J3
Discord Handle
No response
Reason(s)
Hello,
Sorry for my bad english, it's been a long time since I wrote in English (but Google is an amazing friend).
Like the Request Addition to Denylist from Stipus (https://github.com/helium/denylist/issues/2300), we have the same problem near Bordeaux (also a French city) with Deeper Hotspot Brand :
-They didn't witness any of the legitimate existing hotspots,
-No legitimate hotspot ever witnessed their beacons (even if they are located right between 2 witnesses).
It's a bit annoying for the people network.
Those 13 miners are distributed on 7 accounts :
Account : 14JQ4auasCCTqVD7YA1C9raXejubd2faLF3HR3tGVCfmE5MfFx9
Clean Foggy Trout : 112mwhJoFgDTEDuuyhcGtLUhNPTrgGzc59WpxRTq4wexZYK85ZH4, BORDEAUX, adding 10 mar 2022, 7:30 am
Abundant Gauze Raven : 11qnKxCn6AXRT1jdhftSHgLFktqFdtLDnyTWHkoL59UWekCjCVD, MERIGNAC, adding 10 mar 2022, 2:34 am
Account : 14XeupQX4BeBHbuqVDzucmemPwceArQRXVMs6fGE87fBFYD87JU
Fresh Seaweed Spider : 11U5TG5FLMJ3ZjqEDucQCeXybNZJTgKdbSvnt9ccUod6k6gAaaA, MERIGNAC, 13 mar 2022, 4:31 pm
Account : 14QQNixSLDL8HX1NFm2pGLyxEgXVLAX29SXcdNjordVup9BigoK
Old Pine Nightingale : 11TKiFPit5jyEDRtKJwHhMVGc9XhxomaJ4n7UnUhdBgw4b6gTFX, TALENCE, adding 11 mar 2022, 9:05 am
Tame Chartreuse Robin : 112amfqRW4R2Sy4L5JGxEBu5HtPE5kBy4cE9jhDVk6UQiACrLCMU, MERIGNAC, adding 11 mar 2022, 7:14 am
Account : 14ZS1iTKwdNRjwBZ3T7RrBhHkTEak1Um2NGEDCkjU8DHPXRVmDb
Broad Topaz Lynx : 11LZPH71phXzVL4QshcpBKy9W1DnHmCYoCvpjUUxCMxj5iWPyv1, BORDEAUX, adding 13 mar 2022, 12:35 pm
Cheery Concrete Sealion : 11ztfiHXqiBEyC4aWbJaynSpJ71Reb71nANXKFr9A2ysTAjPu9f, BRUGES, adding 13 mar 2022, 1:56 pm
Account : 14D79CxaEW5DfRiGh7FvjRs2we68HbZafZr256KgmVYGG7aSY5W
Slow Tangerine Wallaby :	 1124J8YqkWepAspVugqDs8xxVpqNDXNFnau5ruF5ngs9SWid8p6W, MERIGNAC, adding 10 mar 2022, 2:58 pm
Jovial Macaroon Vulture :	 17f4Dx96cWCXLiGh3vyHWQvNP4674DxQAJUBEwDi8MZNnZRNe7, BORDEAUX, adding 10 mar 2022, 2:08 am
Account : 14H2iYM1U7ZJdC9A8rNTSMxrBvDFzaNxgJ16WJTs3Kti8dNmxfh
Round Hotpink Squid : 112iHW9T6nKP9cDdnWsZQiDsgyjf4hjzFwJwWZ3AuVQ6DQz2RVmu, LE BOUSCAT, adding 11 mar 2022, 3:26 pm
Stale Carbon Falcon : 1122HkK1tM9c2n3pbBRapS4cUpqBV1eDBbmbkmsmbhHX4Xqidc17, BORDEAUX, adding 10 mar 2022, 5:02 am
Account : 14CFrQAmtCLwSYgxLbR7VqQQhVA4uukaNpztgaMhtujnYtedQQ8
Faint Sable Hedgehog : 11Ssify7mqddj77YtP1iLDtmkjA2Z6xEbCvBET56oYBH65zkoRh, EYSINES, adding 10 mar 2022, 3:26 pm
Brave Topaz Terrier : 112r7M27LjSEvLuB6S9NBWh8eVa4KKtAB6dVpxid4nM84YF192J3, LE BOUSCAT, adding 10 mar 2022, 2:57 pm
For example : in the same Hexagon :
Clean Foggy Trout, deeper brand : https://explorer.helium.com/hotspots/112mwhJoFgDTEDuuyhcGtLUhNPTrgGzc59WpxRTq4wexZYK85ZH4
Witnessed : 11, all Deeper Brand. Can witness a Hotspot at 6.2 km (Tame Chartreuse Robin) with a 3 DBi antenna at 0 meter high, while it is placed in the historical city center with thick and tall buildings

Activity : with very good RSSI for a 3 DBi (85 to 95 dbm RSSI).

Agreeable Plum Shell, nebra brand : https://explorer.helium.com/hotspots/112mZbYj1RkqBxXe9kWxhkoM1VuPntUCaXean3rpBWxnkcF9yBjk
Witnessed : 20, especially Nebra, Call-Chip, Bobcat, Heltec.....But not a Deeper miner. Can witness a Hotspot at 5.5 km with a very good antenna and 24 m high.

Activity : more logical RSSI for the antenna, 89 to 120 dbm RSSI

Is it normal ? Since when can we open a private helium network?
Same problem for Dijon (https://explorer.helium.com/hotspots/112owkbD6zTqHon4eY9jKGo5GYZhBJKDZr9Lfa7nJojfKtoyuqbV), or Les Sables-d'Olonne (https://explorer.helium.com/hotspots/1127e6peUkEk5x7XFZjCnpUoY1k4EuUTq3xYFkjzh6vxFexrUwPH).
Have a nice day.

All hotspots have been included in
https://github.com/helium/denylist/pull/2471
You can close.


Add hCaptcha on Subscribe

Closing this for now, will revisit this again if we decide to implement a captcha solution


application with agent startup and report illegal field modifiers in class

jdk: 1.7
skywalking: 6.6
spring: 4.0.4
Other same versions projects with agent are startup normally except this，the error in agent log:

my code :
public interface TaskCallback<T> extends ListenableFutureCallback<T> { public void onTimeout(); }
public interface ListenableTask<T> extends Callable<T>, TaskCallback<T> { }

@ascrutae Please take a look.

@ascrutae Please take a look.

i rewrite my interface to class and reboot success now, thank you

@ascrutae Please take a look.

i rewrite my interface to class and reboot success now, thank you

how do you fix it？I met the same error


Search stops working

Describe the bug
When I start searching for cities there comes a point where the search stops working and no longer searches for locations.
Reproduction - (required - issue will be closed without this)
Steps to reproduce the behavior - a minimal reproducible code example, link to a snack or a repository.
Please provide a FULLY REPRODUCIBLE example.

Click to expand!
import React, {useContext} from 'react';
import {StyleSheet, View} from 'react-native';
import AuthContext from './navigation/AuthProvider';
import {GooglePlacesAutocomplete} from 'react-native-google-places-autocomplete';
import Ionicons from 'react-native-vector-icons/Ionicons';

const Prueba = ({navigation}) => {
// Variables
const {user} = useContext(AuthContext);
const {logout} = useContext(AuthContext);

return (
  <GooglePlacesAutocomplete
    placeholder="Search"
    onPress={(data, details = null) => {
      // 'details' is provided when fetchDetails = true
      console.log(data, details);
    }}
    query={{
      key: 'PRIVATE KEY',
      language: 'es',
    }}
  />
);
};

const styles = StyleSheet.create({
container: {
  flex: 1,
  flexDirection: 'column',
  backgroundColor: '#ffffff',
},
});

export default Prueba;

Additional context

Library Version: 2.4.1

React Native Version: 0.64.3

[X] iOS

[X] Android

[ ] Web

If you are using expo please indicate here:

[ ] I am using expo

Add any other context about the problem here, screenshots etc

Same Issue any solution ?

Same issue here what is going on ?


Clarify changes to CLIC memory-mapped registers

I've got some questions about the recent changes to the CLIC memory-mapped register definition. Can you help?
In section 4.1.1 (CLIC Memory Map), talking about M-mode, it still says this:

The intent is that only the necessary address regions are made accessible to each privilege mode using the system’s standard memory protection mechanisms. This can be done either using PMPs in microcontroller systems, or page tables (and/or PMPs) in harts with virtual memory support.
The CLIC specification does not dictate how CLIC memory-mapped registers are split between privilege regions as well as the layout of multiple harts as this is generally a platform issue and each platform needs to define a discovery mechanism to determine the memory map locations. Some considerations for platforms to consider are selecting regions that allow for efficient PMP and virtual memory configuration. For example, it may desired that the bases of each privilege mode CLIC region is VM page (4k) aligned so they can be mapped through the TLBs.

I think it would be worth reworking these two paragraphs in the light of the other changes here. The first paragraph above is talking about "each privilege mode", but now all memory-mapped registers here are relevant to M-mode only, aren't they? In the same way, the second paragraph is talking about memory-mapped registers split between privilege regions. It feels like these two paragraphs are a bit stale and reflecting an earlier implementation.
Moving on to section 5.1.1. (ssclic CLIC Memory Map), it says:

It is not intended that the interconnect to the CLIC memory-mapped interrupt regions be required to carry the privilege mode of the initiator. A possible implementation of the CLIC memory map would be to alias the same physical CLIC memory-mapped registers to different address ranges, with each address range given different permissions for each privilege mode. Interrupts configured as M-mode interrupts appear as hard-wired zeros in the S-mode address range.

Given that each privilege level now has its own cliccfg register, is the second statement about aliasing still true? If this was done, the cliccfg registers would not be independent.
There are also inconsistencies between M-mode, S-mode and U-mode definitions. For S-mode and U-mode, it states the blocks must be aligned to 4k boundaries. For M-mode, this is an option but not mandated. Is this inconsistency intentional?
Thanks.

Additionally, can you please clarify the behavior of the xcliccfg.nlbits fields; specifically:

Are xcliccfg.nlbits fields independent of each other, or are they aliases of the same thing? and:
If xcliccfg.nlbits fields are independent, is it the case that mcliccfg.nlbits is used to determine the level of interrupts targeting M-mode, scliccfg.nlbits is used to determine the level of interrupts targeting S-mode and ucliccfg.nlbits is used to determine the level of interrupts targeting U-mode?

Thanks.

notes from TG meeting 2/28/23:  will split into 2 issues to separate cliccfg.nlbits question (issue #311).  will try to re-write how mem-mapped registers .   platform can mandate to 4k boundaries if it is virtual memory but spec can leave it open.  use same language as in m-mode.   for cliccfg could put fields in different places in the word.   put in one word with the byte for each priv mode.  high byte is m mode, low byte is u-mode.  byte 1 is s-mode.

closed with pull #312 and text updates


how can i  set this "public async Task Consume(ConsumeContext context)" as synchronized func

how can i  set this "public async Task Consume(ConsumeContext context)" func as synchronized func

This isn't Java, there are no synchronized methods.
I'd recommend looking at articles on C# thread pool use to learn more about how .NET manages threads.
A simple answer to the likely problem you're seeing is to add a concurrency limit to your receive endpoint.
    cfg.ReceiveEndpoint(host, "my-queue", x =>
    {
        x.UseConcurrencyLimit(1);
    });

i want to consume next message after  this message had been processed completely.
sometimes the process will cost a long time, i want the message to be processed in order and i only have one consumer.

You just process the one message....and your function ends.  The next message will then call your function again - you do not have to do anything.

@waynebrantley  the consume function returns a Task.     the first message function not end ,the second message has  already been token.it is not synchronized. so  perhaps the second function ends firstly.

If you use the x.UseConcurrencyLimit(1) as shown above, it will not start the 2nd function until the first one finishes it.

ok,thanks a lot 😋


v7 ideas 💡

I am already thinking about version 7 for a moment and wanted a place to talk about it.

Split the lib into multiple packages to have a core and a package by template/client. Like that we can have multiple advantages. Versioning for the template and be able to use an older version of a template if your project is on an older version of react-query for example. Also having a core will help people that want to create their own template and then use it with other features of orval.

Rethink the config file since I started this lib a lot of stuff has been added and I think we can improve some points. For example, the generation could be more custom. I think for example that having a mix of this and some of Orval features could be great

move to esm?

if you have any other ideas throw them here so that we can discuss it

I definitely like the idea of breaking the lib up so its easier to find and make changes to the different libs.  Changing the config file though might be a big and painful change.  However it may be worth it but I am not sure.

I was also thinking about having a migrate command to do it easily for users

https://github.com/anymaniax/orval/issues/347 This one should be great to have :)

What do you think of this for query keys

What do you think of this for query keys

I LOVE this idea. I find myself doing this a lot in my code...
const QUERY_LIST = "list-cars";
 const queryList = useGetEntityCars(
        { request: JSON.stringify(lazyParams) },
        {
            query: {
                queryKey: [QUERY_LIST, lazyParams],
                refetchOnWindowFocus: false,
                retry: false,
                cacheTime: 0
            }
        }
    );

So then later I can do this...
queryClient.invalidateQueries([QUERY_LIST]);
So that library looks like what I wanted to do all along!

Although I do hate tying myself to another runtime library written by just one guy.  it would be cool if you could just cope his code and include it in the generated code?

I've been playing with the codebase for a bit and have been experimenting with using the Typescript AST factory to generate code as opposed to plain string manipulation. I've used it to encapsulate mock generation logic into separate type classes, which I could then compose together to build more complex representations.
It could also solve your idea for additional customization by allowing users to hook into the AST generation process to change the code at generation time.
I've committed the code to my fork if it's something that might interest you, I'd be happy to put more work into this.

have been experimenting with using the Typescript AST factory to generate code

I also think that this is the way going forward as the string manipulation approach is quite unmaintainable. A cleaner approach can be seen for example here however it's quite opinionated and orval has much more configuratoin options. I think orval could attract a lot more contributors in the long run if it were rewritten with TS generators.

Now with react-query v5 and breaking contracts it would be very handy to choose which major your client is using. Maybe support up to 1 major back?

Maybe have zod be an includable separate layer (i.e. request/response validation on top of axios on top of react-query). Or just splitting things into layers, like having the request engine layer be between axios or native fetch, then an optional validation layer with zod, then the caching control/hooks/query library layer for tanstack query or swr.

@Will-Mann-16 yes, this would be perfect, I'm searching for this solution.

I've done extensive work with code generation, specially in typescript, I'll write here some of my opinions and I'd love (as directly spoken with @anymaniax before) to help with v7.

Split the lib into multiple packages to have a core and a package by template/client.

Awesome idea, 100% agree with you. Initial work may be a little bit harder,  here's some tips:

The greater problem with splitting into multiple packages, is having to force the user to install them separately, if I understood right, it were supposed to be something like npm i -D @orval/core @orval/react-query. Keeping these versions pined when releasing is the best we can do, pnpm's workspace protocol helps here. Here's how I did it with @kitajs/* repositories.

In code generation tools like this one, a good architecture is to provide a AST-like abstraction layer, firstly a package reads the swagger and convert it into orval type nodes with all needed information, then another package reads this proprietary AST and generates the code with it, ts-json-schema-generator did it right, this allows someone to code a tool and generate the orval's AST in their own way to be used with a @orval/react-query, for example, or someone can use the AST generated by orval to write their own generator without having to hardcode swagger/openapi related things. kitajs/parser also did something similar with its AST nodes. Something like (Swagger JSON -> [swagger parsers] -> Orval AST -> [AST formatters] -> Generated code) should work.

Think for example that having a mix of this and some of Orval features could be great.

Splitting the generation inside multiple files will only give headache and bump into all horribleness that happens within the bundle scenario in JS in general, trust me, this will be a never ending source of bugs and features to be implemented. As long we emit TS code in ESM, tree shaking (currently supported by all major bundlers) will resolve all of our performance/frontend problems. Although it should provide a way to generate types separately (to gain speed benefits from the import type typescript's feature). There's no real benefit into supporting what we currently know as mode option.

Move to esm?

ESM in general is still a great pain-in-the-ass for the current state of JS, if ESM is inside a package users really need, they may consider changing their current build/bundle process, however, orval will mostly (if not always) be consumed as a CLI, which providing a ESM only bundle will hardly affect user adoption. If a ESM bundle is REALLY needed, we could provide a non default esm version, like orval and orval-esm binaries.

changing the config file.

JS in general is a bit too much unopinionated in my opinion, as long as we have a typesafe way to writing our configuration file, i'm happy with it. Not too permissive as https://github.com/cosmiconfig/cosmiconfig but nothing too strict as a json file, current orval.config.js is awesome but probably will have its properties changed upon refactor. Custom config lookup/parsing/resolution usually takes a good portion of a CLI's runtime, so this should be taken care of.

Code->string generation tools.

I've tried a lot in the past, generating TS, generating dts+js, generating inside node_modules (like what prisma does), generating inside the user's src dir, using handlebars-like templates, using raw ` or using an ast builder like typescript compiler api.
Somehow I keep finding myself going back to writing raw template strings with some helpers, I'm experimenting ts-writer (usage example), pairing it with vscode-sql-tagged-template-literals gives a nice DX with instantaneous performance. AST generators may be more type safe, but comes with a super slow performance compared to template literals and a worse DX, which WILL scare devs away from implementing custom generators. Handlebars could be an option, but I could not find a nice way to get typescript syntax highlighting to work. Simple top level functions used everywhere in combination with ts-writer is the best I could achieve until now, but would love new ideas.

Sorry if the above text has spelling errors, this is everything I could come up to at 1am.

@arthurfiorette would love any help.  Your plan sounds very ambitious.

@arthurfiorette can you contact me on discord to discuss it?

@anymaniax Do you think it would be difficult to implement support for https://github.com/ngneat/query and then for the official package for Angular Query from TanStack once it is out of experimental stage? https://tanstack.com/query/v5/docs/angular/overview

Bro just fix the #671 in the next release. it's a big issue.

Splitting the generation inside multiple files will only give headache and bump into all horribleness that happens within the JS bundling scenario currently, This will become a never ending source of bugs and features to be implemented. As long we emit TS code in ESM, tree shaking (currently supported by all major bundlers) will resolve all of our performance/frontend problems. Although it should provide a way to generate types separately (to gain speed benefits from the import type typescript's feature). There's no real benefit into supporting what we currently know as mode option. Also generated code shouldn't be edited/checked manually so readability does not applies here.

I would not trust any code generated by any code generator. Split mode allows us to easily manage the generated code with git, improve the review experience, and give us less conflicts

Future Orval idea: TypeSpec support? To be honest the worst part of using Orval is writing YAML. We can use TypeSpec with Orval today by compiling to OpenAPI then generating with Orval, but a YAML free workflow would be awesome. Given Microsoft support I think this standard will stay around, would be interesting to think about what features we could get by using TypeSpec as a generation source.
https://github.com/Microsoft/typespec
https://typespec.io/x

I recommend referring to kubb

Kubb was just deprecated by its owner.  You can read his reasons on his github page

I appreciate the move towards ESM, but I have some reservations. In my experience, particularly with the projects I’ve been working on, supporting ESM correctly has been quite challenging. While I understand the push for outputting ESM, the "type": "module" setting has been a significant headache in several cases. There always seems to be something that doesn’t fully transform into ESM, leading to unexpected issues. A recent example of this is the vitest-chrome project, which illustrates some of the difficulties I've encountered.

changing the config file.

JS in general is a bit too much unopinionated in my opinion, as long as we have a typesafe way to writing our configuration file, i'm happy with it. Not too permissive as cosmiconfig/cosmiconfig but nothing too strict as a json file, current orval.config.js is awesome but probably will have its current properties changed upon refactor in favor of v7. Custom config lookup/parsing/resolution usually takes a good portion of a CLI's runtime, so this should be taken care of.

One thing I wanted to mention, the current config file, being in JS, allows to have within a monorepo local codegen based on Orval, without the need to publish them on npm. It also allows to have presets within a monorepo to share configuration in a easier way.
If the config is in yaml / json / whatnot, we may loose this kind of features and would need to happen on orval land instead of user land. This is a big part of why eslint moved back to js only (RFC from 2019 about it)

Simple one: instead of having a custom axios instance involved in code generation, just pass in an optional axios instance to the generated getFastAPI function ? It's a lot simpler, it's just normal dependency injection at that point. e.g:
export const getFastAPI = (axios?: AxiosInstance = defaultAxios) => { ... }

I recommend referring to kubb

and

Kubb was just deprecated by its owner. You can read his reasons on his github page

Cross-linking to the GitHub issue that mentioned the deprecation, and then the un-deprecation: https://github.com/kubb-labs/kubb/issues/1124. That post did mention forwarding users to Orval.
Context: I’m looking to find the right library to generate TypeScript types/clients from (.NET) Open API documents. Sorry if this interrupted the discussion.


Adding support for checksum & signature validation.

This might not be universal for all hashicorp products, but I thought it might be useful to check the checksums, also validating the signature. Might need to add checks to fail quietly if the target machine doesn't have gpg installed.

Closing this as it was handled by #29 .


Api route fallback issue

Laravel Version: 8.40
PHP Version: 7.4.3
Database Driver & Version: Mysql  15.1

Description:
i'm trying to add route fallback to api routes for handle the 404 routes error, but my Route::falback() is not fired when i navigate to the 404 route
this is my api.php
Route::fallback(function () {
    return response()->json([
        'message' => 'Page Not Found. If error persists, contact<EMAIL_ADDRESS>    ], 404);
});

Hi there,
Thanks for reporting but it looks like this is a question which can be asked on a support channel. Please only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repo you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.


change version=latest to version=master for afni and ants

This is only when building the current master branch from source.
fixes #281

Codecov Report

Merging #297 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #297   +/-   ##
=======================================
  Coverage   80.93%   80.93%
=======================================
  Files          33       33
  Lines        1568     1568
  Branches      170      170
=======================================
  Hits         1269     1269
  Misses        218      218
  Partials       81       81

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 ad31d24...ed37334. Read the comment docs.


Feature Request: Expose client IP variable for runOnRead

Describe the feature
Expose a client_IP variable to runOnRead when a clients starts reading.
Could be useful for debugging, logging, or security.

Ideally, for HLS, this could also inspect X-Forwarded-For or X-Real-IP (though knowing how far along a proxy chain to trust is not a trivial problem)

moved to #1414


dashbls: replace libsodium with minimum implementation needed for secure (de)allocation

Additional Notes

The minimum implementation is dubbed "libminialloc" and is a wrapper around LockedPoolManager, which is taken from Bitcoin 23.x (cpp, h), alongside cleanse (cpp, h) and is supposed to mimic the behavior of their secure allocators
The wrapper is heavily influenced by Botan's libsodium compatibility support (source). loadstor.h is a barebones version of loadstor.h from 2.19.2 (source).
Guidance for libsodium behavior was taken from utils.c from 1.0.18 (source) and support for dynamic linking was taken from export.h (source)

I triggered 2 failed jobs and they are ✅ now https://github.com/dashpay/bls-signatures/actions/runs/3229176267. Not sure why they were failing initially - could be simply some github glitch but let's keep an eye on it 👀
post-merge utACK

not good https://github.com/dashpay/bls-signatures/actions/runs/3237002352/jobs/5303527720 :/


Sharing shows no effect

If I want to share from another app (for example play store) to a contact this does not show any effect. I choose Chatsecure from "share" to options, then I get to the point where I can choose account, then to contacts overview where I choose one but it will only open the conversation and no sharing takes place at all (OTR is active for this conversation).

I think we've fixed this, or at least it works for content/apps that we support sharing.


TabError: inconsistent use of tabs and spaces in indentation

Come on man. why does every repo on github have to be broken in some way
  File "G:\CodeTemp\Audio-auto-tagging-master\4_CNN_main.py", line 264
    sess.run(init)
                 ^
TabError: inconsistent use of tabs and spaces in indentation

Hello @fractaldna22. This project started as a personal project and is not maintained for almost 4 years now. I am trying to find time to refactor many parts of it so if you are interested and would like to contribute I would be more than happy to merge your PR.

i cant even get one single repo to work lol. sucks i really was looking forward to this for my psytrance folders. im a dj not a programmer T_T


System.InvalidOperationException on C# UWP client

In a blank UWP app:
namespace App1
{
    public sealed partial class MainPage : Page
    {
        public MainPage()
        {
            this.InitializeComponent();

            Init();
        }

        private async void Init()
        {
            await Task.Delay(4000);
            // Wait for server to listen

            var hub = new HubConnectionBuilder()
                .WithUrl("http://localhost:5000/api/v1/myhub")
                .Build();

            await hub.StartAsync();
        }
    }
}

I get the exception System.InvalidOperationException: 'Unable to connect to the server with any of the available transports.'
I reproduced it at https://github.com/Tristyn/SignalrUwpRepro, by starting multiple projects in VS it will work on the console app, however throws on UWP.
I used Kestrel with https disabled for simplicity.

Please follow the diagnostics guide and gather the logs for the UWP app, https://github.com/aspnet/SignalR/wiki/Diagnostics-Guide#net-client-logging

Here is the debug log.
debug log.txt
The log states System.IO.FileLoadException: Could not load file or assembly 'System.Threading.Tasks.Extensions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040) Adding the nuget package System.Threading.Tasks.Extensions v4.5.1 fixed the issue for me, I think you guys should be pulling that package?

We have https://github.com/aspnet/SignalR/issues/2474 tracking that

Thanks again. The internal FileLoadException (or an AggregateException) should be an inner exception to the InvalidOperationException to help users self diagnose. My coworker recalled a similar missing package on Signalr/Xamarin but the FileLoadException was wrapped which made it an quick temp fix

I also ran into this problem. And spent about 3 hours looking for the solution until I found this. Yes, using SignalR in UPW apps seems to require manual adding System.Threading.Tasks.Extensions nuget package. Or you get the misleading "Unable to connect to the server with any of the available transports" Exception.
As Tristyn has already written: Shouldn't this be a dependency of SignalR.Client when installig in an UWP app?

See our announcement regarding this topic

Shouldn't this be a dependency of SignalR.Client when installig in an UWP app?

The problem isn't that we don't have the dependency, it's that we have a dependency on an earlier version that fails to load in UWP. This was an issue in the RTM version of .NET Core 2.1 which was resolved in a patch. Our practice is to depend upon the lowest patch version corresponding to the minor release we depend on and that version has this issue in UWP. Users are able to upgrade the package manually to resolve the issue but our policy is not to force this upgrade on all users in a patch release, so we haven't updated that reference yet.
https://github.com/aspnet/SignalR/issues/3209 is tracking fixing this in 2.2, when we have a chance to update to the next minor version of .NET Core's packages.

This is resolved in 2.2. The latest preview build (1.1.0-preview3-35497) of Microsoft.AspNetCore.SignalR.Client should work fine on UWP with no additional dependencies.