Fix examples/json_lines.cpp

The extra newline throws an exception.

Pull request checklist
Read the Contribution Guidelines for detailed information.

[x]  Changes are described in the pull request, or an existing issue is referenced.
[x]  The test suite compiles and runs without error.
[x]  Code coverage is 100%. Test cases can be added by editing the test suite.
[x]  The source code is amalgamated; that is, after making changes to the sources in the include/nlohmann directory, run make amalgamate to create the single-header files single_include/nlohmann/json.hpp and single_include/nlohmann/json_fwd.hpp. The whole process is described here.

Please don't

The C++11 support varies between different compilers and versions. Please note the list of supported compilers. Some compilers like GCC 4.7 (and earlier), Clang 3.3 (and earlier), or Microsoft Visual Studio 13.0 and earlier are known not to work due to missing or incomplete C++11 support. Please refrain from proposing changes that work around these compiler's limitations with #ifdefs or other means.
Specifically, I am aware of compilation problems with Microsoft Visual Studio (there even is an issue label for this kind of bug). I understand that even in 2016, complete C++11 support isn't there yet. But please also understand that I do not want to drop features or uglify the code just to make Microsoft's sub-standard compiler happy. The past has shown that there are ways to express the functionality such that the code compiles with the most recent MSVC - unfortunately, this is not the main objective of the project.
Please refrain from proposing changes that would break JSON conformance. If you propose a conformant extension of JSON to be supported by the library, please motivate this extension.
Please do not open pull requests that address multiple issues.

What is your exact error message? The code of docs/examples/json_lines.cpp compiles fine as-is.

The original report says it throws an exception, so I assume this is a runtime error, and the error is on the json::parse of an empty line after the last valid line.

As gregmarr said, this is a runtime error.
Sorry, there is no problem with this example.
As you can see on the below snippet, I put 4 extra whitespaces before )"; on 6th line.
I think this exception is related to the <nlohmann/json.hpp>'s core.
Should I have to open a Bug Report issue?
int main() {
    input << R"({"name": "Gilbert", "wins": [["straight", "7♣"], ["one pair", "10♥"]]}
             {"name": "Alexa", "wins": [["two pair", "4♠"], ["two pair", "9♠"]]}
             {"name": "May", "wins": []}
             {"name": "Deloise", "wins": [["three of a kind", "5♣"]]}
    )";

    try
    {
        while (std::getline(input, line))
        {
            std::cout << json::parse(line) << std::endl;
        }
    }
    catch (json::parse_error& ex)
    {
        std::cerr << "parse error at byte " << ex.byte << std::endl;
    }
}

{"name":"Gilbert","wins":[["straight","7ΓÖú"],["one pair","10ΓÖÑ"]]}
{"name":"Alexa","wins":[["two pair","4ΓÖá"],["two pair","9ΓÖá"]]}
{"name":"May","wins":[]}
{"name":"Deloise","wins":[["three of a kind","5ΓÖú"]]}
parse error at byte 5

Sorry: I meant I can compile and run the example as-is.

Sorry: I meant I can compile and run the example as-is.

I can confirm the error with the blank line at the end: https://www.godbolt.org/z/3hzGz75WE

As you can see on the below snippet, I put 4 extra whitespaces before )"; on 6th line.

Yeah, that's the source of the issue.  You changed from having a newline at the end of the data, which is handled cleanly, to having whitespace after the final newline, which does not get handled cleanly.  You'd need to strip leading whitespace from line before passing it to json::parse() if you wanted to support that.


Upgrade tested AGP version

From 7.3.0-beta05 to 7.3.0-rc01

@bot-gradle test AST

OK, I've already triggered the following builds for you:

AllSmokeTestsPullRequestFeedback build

@bot-gradle test and merge

No milestone for this PR. Please set a milestone and retry.

@bot-gradle test and merge

OK, I've already triggered a build for you.

Pre-tested commit build failed.

@bot-gradle test and merge

OK, I've already triggered a build for you.


float min, max and step props for numeric input

Description
Allow using decimals for min, max and step properties for numeric input
Related Issues

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


IndexError: piece id is out of range.

Hi there,  does anybody meet the same issue? After lora tarining with self_cogition.json, I try to inffer model and raise:

请问怎么解决的

请问怎么解决的

好像说是和transformer的版本有关

我也遇到了这种情况，用NPU推理的时候出现的


(REMASTERED/MORE UNDERSTANDABLE) "Multi-copy section: option in chart editor"

What feature do you want to get added on the base engine?
Just like "Copy next to.." but this option will have an extra checkbox on the bottom left but upper on the lil' buddies, and it shows a text like "sections checked: (example:2)", so when you check it the current section turns green, you could like press the checkbox as many section you want, then it will have another option right under the "sections checked" text that says "Copy the checked sections" hope this helps for you to understand more!!
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

i don't understand this

Added in https://github.com/JordanSantiagoYT/FNF-PsychEngine-NoBotplayLag/commit/4a95aac1a2d3b93d32e61846ee67b3deb7c48f9c!


Updates build.gradle

Makes it so only mcmod.info has the version required to update :D

ModID must be all lowercase

Answering 1. I'm not...
Answering 2. You're correct my bad set the version number in mcmod.info to the project ID

Embed does not work with FG5
Yeah but that won't fix it you need to find a way to replace the String in ModReference each time we compile

I'll go do that and I saw embedded doesn't work I just left that there


Upgrade MDX

Align version numbers with the latest included in gatsby-plugin-mdx, to match https://github.com/Lullabot/architecture.
This is a stab at resolving https://github.com/Lullabot/architecture/issues/494

Tugboat has finished building the preview for this pull request!
Link:

https://pr95-tn9nuf4vmm2ekyxmzjnf5fqmc010wffc.tugboatqa.com

Dashboard:

https://dashboard.tugboatqa.com/655d0c13382826646cd2a009

Visual Diffs:

https://dashboard.tugboatqa.com/655d0c13382826646cd2a009/visualdiffs


Making it work with dual?

No issue just wondering if we could make it work on dual-agar.online ,
I play all time dual, and i was wondering if we could do an copy of that? Maybe if someone had the exact settings in their Gameserver.js and gameserver.ini
That could be an idea.. But if you have a copy that looks like duals, send me copy/paste id love to have it

sorry raphy, but this is client sided.
Dual doesn't support private server connections in anyway.

@AlexHGaming I think so, but protocol that nobody knows about it lol @raphyjb

Nope


Add 'Filter Ratio' to tuning parameters

Adds 'Filter Ratio' as a new per-profile tuning parameter.
If the Profiles table in the current database does not have a filter-ratio column then the table is updated and the tuning profiles are reset to defaults.
--ET

Do you use this parameter? I added this as an experimental value and found the default to work the best.
If you do use it, what values are you typically using and what situations does it help with?
If there is not a lot of value to the parameter, my preference would be to remove it rather than add to the already confusing set of tuning parameters.

Yeah, I think increasing the divider in the 'filterRatioFloat' setups (in the node .ino) from 1000.0 to 10000.0 is worth doing.  Even with that and "Filter Ratio" set to 1 the RSSI readings still seem fairly responsive.  A "Filter Ratio" setting of 100 would match the current default, and 10 seems fine.


Explanation of taint analysis/tracking and ideas about extensions

Taint Analysis Setup (Introduction)
The original purpose of the taint tracking was to be able to follow all instructions which used information from a certain source (e.g. a sensor), which could be marked by the user with a certain taint colour. A basic form of taint tracking has been implemented for sunflower. This can be enabled/disabled before compiling by setting the SF_TAINTANALYSIS flag in config.h to 1/0. The functions used for taint analysis are defined in taint.c, and taint.h and used in op-riscv.c . Other files which include any amount of implementation of taint analysis (to find these use "rg -i taint") are: main.c main.h lex-riscv.c sf-riscv.y sf.h (which includes taint.h) machine-riscv.h machine-hitachi-sh.c pipeline-riscv.c and mfns.h.
Taint is handled thus: the user can use the taintreg and taintmem instructions to mark a register or piece of memory as tainted, assigning a PC value at which this acts as an origin of taint, and a taintcolour that should then be propagated. Taintcolours are uint64_t types, allowing there to be up to 64 different sources of taint in a program. For each register and each byte of memory there is a ShadowMem struct (which are contained in the arrays taintR (taintfR for float registers), and TAINTMEM which are defined just below the arrays for normal registers and normal memory) which holds information on the taint of that register/memory address. The ShadowMem struct contains a taint colour and memType (an enumeration type defined in taint.h), and for every RISCV assembly instruction in the op-riscv.c file, the taint of the inputs is put into a bitwise OR function, and the resulting taint is written to the output. This is achieved by the taintretreg, ftaintretreg and taintretmems functions which return the taint colour of a register/floating point register/memory address, and the taintprop function, which takes two taint colours, performs a bitwise OR, and writes the result to the shadow output register. Each time taintprop is called, the taint of the inputs is also passed to the array of ShadowMem values for each instruction (instruction_taintDistribution), so that that instruction is marked as having handled a certain taint.
Extension ideas
Liberal/Conservative tainting
This idea came out of a discussion with Samuel Wong.
The idea is that taint propagation falls into two different categories: direct propagation, which involves a tainted register being copied to another register, or being the input to a logic operation whose output is written to another register. Indirect propagation is when a tainted register is involved in control flow; for example if register A is tainted, and B, C and D are all untainted then the code below would be an example of indirect propagation:
if ( read(A) > read(B))
{
       C++;
}
else
{
       D++;
}

The proposition from the discussion was to have two different modes of tainting. Liberal tainting would cause taint to be passed on in both direct and indirect propagation, whereas conservative tainting involves taint only being passed on if there is direct propagation.
To implement this, there should be a flag in the State structure which could be set or cleared by the user to determine whether tainting should be liberal/conservative. Currently (27/08/2019) this is not being implemented, with all tainting being fully liberal.
Taint History
The implementation of taint analysis which is currently (27/08/2019) being worked on has the following rule on taint and overwriting: if a register is overwritten, its taint is changed to the taint of the things overwriting it. Therefore if register A has no taint, and register B is tainted, but subsequently register B is overwritten to have the same value as A, after the overwrite operation, B should also have no taint. Potentially an interesting extension is to look at taint history, that is to be able to see all of the different taint values that a particular register or memory address had at any point in its history.
Immediate Taints
Currently immediate values, such as the imm0 input for the addi instruction in op-riscv.c are treated as entirely free of taint always. In certain cases, perhaps these immediate values are taken originally from a tainted source, and so should lead to taint propagation. Assigning and propagating taint for immediates presents two problems: the taint must be propagated when the addi instruction is called, as some immediates may be free of taint, and other won't be. In addition, there is no Shadow data structure for immediate values, so a new one would have to be created. One idea is to have an array of taint of immediates, which might be 10 items long. Then when an immediate value is loaded from somewhere, it can be assigned a place in that array, and given the corresponding taint, which can be propagated later when the immediate is used. However, the immediate array would have to be cleared straight after this to prevent writing taints incorrectly at later calls of instructions with immediate values.
Taint Granularity
Currently each register has one taint colour, and each byte of memory has its own taint. One extension that might be useful would be the idea of varying the size of each unit of taint. For example, if you assigned one taint for the upper 16 bits of a register and a different taint for the lower 16 bits of the register, then you could be more precise in taint propagation in cases such as the LH instruction, where really only half the register being written to should be tainted.
Implementing for other processors
Currently taint analysis is only implemented for the RISC-V processor, whereas Sunflower can also emulate the Hitachi SH as well as the TI MSP430. Implementing taint analysis for these other processors would make sense and should be relatively easy as all the structures from the implementation for RISC-V are already in place (note the person saying this has no knowledge of the Hitachi SH or TI MSP430, so they may in fact have significant differences which might make the task more difficult).

Additional Extension ideas
Safety checking
Currently there is no checking that occurs to see if memory addresses and register addresses passed to taint propagation functions are within an acceptable range. Hence if memory indexing finishes at address 50 and a memory address of 75 is fed into the taint analysis functions this would lead to undefined behaviour. By putting the taint propagation code at the bottom of each RISC-V assembly instruction in op-riscv.c, I assumed that any out of bounds values would be caught earlier and lead to an appropriate error message. Obviously, this is inherently risky and if someone has time to improve the taint analysis implementation, I would prioritise this.
Automate taint marking
This idea was independently conceived by Phillip Stanley-Marbell.
Currently the marking of a tainted register/fregister/memory address is done by hand using a taintreg or equivalent instruction in a '.m' file, as exemplified for a bubblesort program below:
newnode 	riscv
sizemem		96000000
srecl		bubble.sr
taintmem	        0x0000000008009888 0x50 1 4
run
on

This requires people to know the memory address at which data is stored, and the PC value at which it is accessed, which requires looking at debugger information. Ideally, this process would be automated so that a user could mark a variable in an input program as an origin of taint, and then the correct taint addresses and PC values would automatically be marked as tainted.
Unimplemented RISC-V instructions
There are currently 10 RISC-V instructions which have no implementation whatsoever. These can be found in the op-riscv.c file. Look for the fence, or csrrci instruction. If and when these instructions are implemented, taint analysis should be implemented where relevant.
Similarly the ecall instruction, as it appears to have no relation to any tainted register, has no taint analysis implemented.
More Usage Statistics
Currently the only statistics available to the user of a program are the taints that each RISC-V assembly instruction has come into contact with at some point. These can be accessed by the dumptaintdistr Sunflower command. Other statistics that might be interesting to implement include:
-Dumping every register (/ floating point register / memory address) that has a specific taint, or any non-zero taint.
-Dumping every register which, at a certain point in program execution, has a specific taint.
Endianness
Endianness is not an issue which has been considered or worried about at all in the implementation of taint analysis. If any memory or register addresses are ever referred to in by the little endian version of their address, this will cause problems with the current taint analysis implementation. This was assumed to not be likely, hence no solution was implemented.


First cut at a Rust Eventhubs client.

I would appreciate any feedback on this PR - there are a couple of places I moved outside the amqp/eventhubs codebase, mostly to fix some wasm32 build issues related to tokio.


file/ftw: use ftw::FileType and impl Fromlibc::mode_t for FileType

I support these changes.  However, ftw is under active development by Joseph, and I do not wish to create merge conflicts with his work.
It is ok in the future (T+3 months).


[Experimental] Support for prisma

TODO:

Testing
Proper type conversion (I've removed a lot that will need to be handpicked)
Add support for wire protocol via websockets

👋 Heyho from Prisma.
If you want, you can do a temporary PR similar to https://github.com/prisma/prisma-engines/pull/4342, which should make it simpler to run the Prisma test suites against the driver adapter. We are unfortunately not ready yet with a more general approach for that, but I am happy to work with getting that run in this temporary way.

Actually, the driver living in your repo is the right spot - but temporarily (via a PR) having a copy in our test setup can be useful for you I think. But we are currently also moving things around (soon the driver adapters live in prisma/prisma...), so maybe we wait a few days until we tackle that.
How do you think about transactions? As far I know your client does not support that yet. Many Prisma Client queries use transactions under the hood (at least as soon as the queries become a tiny bit complex). Would you want those to be not working, or are you ok skipping the transaction guarantees for your users?

We are working to offer full postgres compatibility, including transactions in the near future. We will even expose a TCP wire protocol connection string and likely have two adapters HTTP and wire protocol. But all of that is blocked until we ship the full thing.


query failed when some fields not exist against DOE

Describe the bug
query failed when i run sql against doris on es. The prompt error is "inner hits node is not an array"
e.g.
CREATE EXTERNAL TABLE `case_replay_for_milimin` (
  `k1` varchar(20) NULL COMMENT "",
  `k2` varchar(20) NULL COMMENT "",
  `k3` varchar(20) NULL COMMENT "",
) ENGINE=ELASTICSEARCH
COMMENT "ELASTICSEARCH"
PROPERTIES (
"hosts" = "...",
...
"enable_docvalue_scan" = "true"
)

This Table has rows like below :
> select  k1 from table;
+------+
| k1   |
+------+
| NULL |
| kkk1 |
| NULL |
+------+

but, actually it returns a error:
> select  k1 from table;
ERROR 1064 (HY000): inner hits node is not an array

To Reproduce
Steps to reproduce the behavior:

put values into index

PUT myindex/_doc/1
{
  "k1": "kkk1"
}
PUT myindex/_doc/2
{
  "k2": "kkk2"
}
PUT myindex/_doc/3
{
  "k3": "kkk3"
}

2、fetch fields contain k1,k2,k3. e.g. select  k1,k2,k3 from table;
Expected behavior
ignore this situation and return ok

maybe you should emphasize  have no value for one batch_size

pr: https://github.com/apache/incubator-doris/pull/3513


Lack of 'Cascade Mask R-CNN' in collections of Metadata for mmdet-2.25.0

Version: openmim-0.1.6, mmcv-full-1.5.3, mmdet-2.25.0
This error popped out when I verified the installation of mmdet. And there is no such error when I downgrade to  mmdet-2.24.0.
$ mim download mmdet --config yolov3_mobilenetv2_320_300e_coco --dest .
Traceback (most recent call last):
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/bin/mim", line 8, in
sys.exit(cli())
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/click/core.py", line 829, in call
return self.main(*args, **kwargs)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/click/core.py", line 782, in main
rv = self.invoke(ctx)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/click/core.py", line 610, in invoke
return callback(*args, **kwargs)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/mim/commands/download.py", line 44, in cli
download(package, configs, dest_root)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/mim/commands/download.py", line 75, in download
model_info = get_model_info(
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/mim/commands/search.py", line 170, in get_model_info
dataframe = convert2df(metadata)
File "/home/users/pli/anaconda3/envs/py38_pt110_mm/lib/python3.8/site-packages/mim/commands/search.py", line 396, in convert2df
for key, value in name2collection[collection_name].items():
KeyError: 'Cascade Mask R-CNN'

Unluckily, i don't have this file in Ubuntu 18.04
I have "locate" and checked .

hard to believe it's a 2 years old bug.

Fixed in mmdet2.25.1


[#33] Cannot register NYM with /ledger/register-nym

Signed-off-by: Thomas Diesler<EMAIL_ADDRESS>

the issue is that the reset value is underscore again so I guess something like https://github.com/hyperledger-labs/acapy-java-client/pull/36 is needed

Fixed with: https://github.com/hyperledger-labs/acapy-java-client/pull/36


Improved naming of variables

Hi @mdekok, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks!


Mobile- Iphone- menus overlap each other- hard to read

Expected behavior
When on mobile- I would expect that when you open an menu, that they should open so that you can see all of the details and they are legible to read
Seems to work on Samsung-

Actual behavior
I tried Iphone 11 max pro and  7 and got same results when you expand menu the contents get merged into whatever is listed below it. See screen shots.
Before you open - menu-

When you open meal prep menu-

When you open products menu-

When you open sales& specials menu-

When you open cooking styles& ingredients

When you open gifts menu-

not attached to party- it does the same thing-

Steps to reproduce behavior
Mobile -  seems to not play nice on iphones I checked 11 pro max  and iphone 7
go to meun
open menu
You will see that when it opens it doesn't display correctly

Got this assigned out to Jen TST-144

Looks good- closing


dynamically upsample data (zoom)

see: be able to dynamically upsample time-series data to look bigger #116

Implemented.


FR: Add Cloudflare Capability

Inspiration: https://github.com/madebymutual/Country-Detect
Docs: https://api.cloudflare.com/#zone-settings-get-ip-geolocation-setting

@mariovass we discussed this on the Craft Slack, but I'll post the answer here for anyone it might help.
Craft 3 allows you to fetch the headers via Twig, so you can achieve this using the following:
{% set country = craft.app.request.headers.get('CF-IPCountry') %}


Rust Abci Execution

Issue being fixed or feature implemented
This PR is for Rust Abci execution. And includes a lot of changes:

Validation:

Validation in RS-Drive-Abci.
Synchronous.
Validation now happens in 3 steps:

Structure : does not look at state (except for getting contract for documents), structure only
Signature : verifies signatures
State : verifies state

Output of each validation is state transition action.

Execution: Groups validation output as State transition Actions.
The advantage is that state transitions are now atomic.
State Storage on Restart: Rewrote this feature in Rust.
Masternode identity port: Rewrote this feature in Rust.
Operator now has a system key for Node id
Signature verification: Commits and withdrawal signatures are verified.
Withdrawals: A lot of fixes
Atomicity of state transitions: State transitions are atomic. Individual state transitions can fail without invalidating the block.
Identity operations are now on Identity V2: Exact Identity calls
Masternode list: Direct new calls to core (more efficient and easier).

What was done?
This is a mega PR that brings with it the main Platform executor written in Rust.
How Has This Been Tested?
A lot of strategy tests.
Breaking Changes
We removed JS-Drive and JS-DPP.
Checklist:

[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

We are going to merge this into to v25 broken. Reviews are going to take place from tomorrow but after merge. v0.25 will only become the main branch after CI is passing.


Possibility to open the file with splits

i'll make this a priority for next release. seems like an important feature.

Added in version 1.0. You will need recompile the latest jfind and use the new lua interface as described in this repo's readme.


keyboard on_key() handler sends incorrect mod mask

Pressing a mod is not immediately reflected in keyboard.get_modifier_masks() which is used to send the modifier state to clients.
To reproduce:
Hold shift and type into a weston-terminal window. The first key will be lowercase and only the subsequent keys after will be capitalized.

Fixed in #544


model.add(Merge) dimension issue.

I have a problem with "merge" function in model.add
========================================================
model_f3 = Sequential()
....bla bla .... long story short.
    model_f3.add(Dense(32*img_p2*img_p2, last_node))
    model_f3.add(Activation('relu'))
    model_f3.add(Dropout(0.5))

    model_f5 = Sequential()

....bla bla .... long story short.
    model_f5.add(Dense(32*img_p2*img_p2, last_node))
    model_f5.add(Activation('relu'))
    model_f5.add(Dropout(0.5))

    model = Sequential()
    model.add(Merge([model_f3, model_f5], mode='concat'))
    model.add(Dense(last_node*2, nb_classes))
    model.add(Activation('softmax'))

=========================================================
Message (when lastnode*2):
ValueError: dimension mismatch in args to gemm (32,1024)x(2048,5)->(32,5)
Apply node that caused the error: GpuDot22(GpuJoin.0, <CudaNdarrayType(float32, matrix)>)
Inputs types: [CudaNdarrayType(float32, matrix), CudaNdarrayType(float32, matrix)]
Inputs shapes: [(32, 1024), (2048, 5)]
Inputs strides: [(1024, 1), (5, 1)]
Inputs values: ['not shown', 'not shown']
Message  (when lastnode):
ValueError: The splits sum to 2048, expected 1024
Apply node that caused the error: GpuSplit{2}(GpuDot22.0, TensorConstant{-1}, MakeVector.0)
Inputs types: [CudaNdarrayType(float32, matrix), TensorType(int8, scalar), TensorType(int64, vector)]
Inputs shapes: [(32, 1024), (), (2,)]
Inputs strides: [(1024, 1), (), (8,)]
Inputs values: ['not shown', array(-1, dtype=int8), array([1024, 1024])]

In the following part of the code,
 model.add(Merge([model_f3, model_f5], mode='concat'))
 model.add(Dense(last_node*2, nb_classes))

when I use [mode='sum'] and change lastnode*2 to lastnode, it works perfectly.
On the other hand, [mode='concat'] and change it to lastnode, it doesn't work.
For paramerter in final "Dense" function,
I've tried lastnode*4, lastnode/2 and whatnots, but none of them has succeeded it.
Any guys have an idea on this ?
I believe [mode = concat] needs doubled number of nodes because it concatenates the nodes. (as the term says, literally),
which is different from [mode = sum]. Because [mode=sum] just sums up the weights into a node.
Am I wrong or is there anything wrong with my code?
Please help me if you have an idea on this thing.
I always appreciate your support and help. thank you.

This issue can be solved by installing brand-new Theano library.
This was an internal bug of the old Theano library.

Yeah even I had the same issue and I got it resolved by installing the latest theano from github (not using pip install)


Correctly construct Region to uphold deallocation safety invariant

Closes #2061
To cite from the comments of the patch:

Shrinking the buffer down to the length is important to uphold a safety invariant by the dealloc method.
Passing in a differing size into the dealloc layout is considered undefined behaviour.
See: https://doc.rust-lang.org/stable/alloc/alloc/trait.GlobalAlloc.html#safety-2

Yeah, I now understand why this fix is correct, thank you! However, this will create a pointless clone of the data in many cases. I think there are two potential other ways to address it:

Don't call build_region(&buffer) but create a region of the entire buffer capacity
We only have this problem when buffers are created by the contract and then destroyed by the VM (see fn call_raw<A, S, Q>). But the whole instance.deallocate(res_region_ptr)?;is pointless because we dropinstance` one line later. This is like cleaning the house before demolishing it.

Don't call build_region(&buffer) but create a region of the entire buffer capacity

Since all of the logic is private, I felt free to update the API of build_region.
Instead of taking a raw byte slice, it now takes something that implements the new unsafe trait RegionSource which is, by default, implemented for &[u8] and Vec<u8>.
The trait exposes the function capacity which has to, as documented in the trait safety section, return the full capacity of the memory region.
I wanna change the trait a bit, but yeah, this is roughly what I envisioned.
And testing against the latest nightly compiler, it fixes it.

The AsRef<[u8]> bound was very Vec<u8> centric since other types which might implement this API only point to a sub-region of the allocated memory with a different starting pointer and so on and so forth..
So this is a better API.

Damn it, I always forget the changelog..

I was just not sure if there is anything practically relevant we need to be careful of

No, not really. It's just something they are experimenting with to give the compiler more information about raw pointers.
Because the more info compilers have about the code, the more assumptions they can make, the more optimization tricks they can do (prime example: Fortran)

@Mergifyio backport release/1.5


[*] editor config / unecessary file / block unused commit file

@robsongajunior thanks a lot. Merged!


Support for Tensorflow 1.14.0

When trying to install I get this error:
Collecting tensorflow==1.7 (from easyfacenet)
  Could not find a version that satisfies the requirement tensorflow==1.7 (from easyfacenet) (from versions: 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.14.0rc0, 1.14.0rc1, 1.14.0, 2.0.0a0, 2.0.0b0, 2.0.0b1)
No matching distribution found for tensorflow==1.7 (from easyfacenet)

pip install tensorflow==1.7.0
Collecting tensorflow==1.7.0
Could not find a version that satisfies the requirement tensorflow==1.7.0 (from versions: 1.13.0rc1, 1.13.0rc2, 1.13.1, 1.13.2, 1.14.0rc0, 1.14.0rc1, 1.14.0, 2.0.0a0, 2.0.0b0, 2.0.0b1)
No matching distribution found for tensorflow==1.7.0

Tensorflow v1.7 is not compatible with the most recent release of Python. It only works up to Python 3.6.x
There is a workaround for this by using a virtual environment:

install python virturalenv with pip pip install virtualenv (optionally for Win: pip install virtualenvwrapper-win

Download an older build of Python (e.g. 3.6.9)

Install it in a separate folder (e.g. C:\Python36\

get a fresh clone of the of the facenet repository

go to your facenet folder

change the requirements.txt to install the correct version of scipy: scipy==1.1.0 (see issue #1060)

create a new virtual environmnet: virtualenv --python=*insert path to older python version* venv

activate it via venv\Scripts\activate

install requirements: pip install -r requirements.txt

hope this helps


os.environ["HOME"] is unsupported on Windows

Encountered a KeyError related to the HOME environment variable while attempting to run FastKafka on Windows platform.
(stats-bot-py3.11) PS C:\Users\nesclass\Workflow\griefyou\stats-bot> fastkafka
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\nesclass\AppData\Local\pypoetry\Cache\virtualenvs\stats-bot-aYPg2ZVp-py3.11\Scripts\fastkafka.exe\__main__.py", line 4, in <module>
...
  File "C:\Users\nesclass\AppData\Local\pypoetry\Cache\virtualenvs\stats-bot-aYPg2ZVp-py3.11\Lib\site-packages\fastkafka\_components\docs_dependencies.py", line 55, in <module>
    local_path = Path(os.environ["HOME"]) / ".local"
                      ~~~~~~~~~~^^^^^^^^
  File "<frozen os>", line 679, in __getitem__
KeyError: 'HOME'

One of the potential solutions for this problem is to use USERPROFILE as a platform-dependent alternative for provided variable.
Also, you can get home folder using os.path as follows:
local_path = Path(os.path.expanduser("~")) / ".local"

Closing it as it's a duplicate, referenced in #130


Middleware is only applied on first request

I have a GraphQL server with some fields you can query if you include auth headers in the request. I'm using angular2-apollo.
// Example query
this.apollo.query({
  query: gql`
    query getUser($guid: String!) {
      user(guid: $guid) {
        name
        email
      }
    }
  `,
  variables: {
    guid: 'guid'
  }
}).then(success => {...}).catch(error => {...});

email here is a restricted field that will be null if there is no auth headers in the request.
I define the middleware like this in my app.module.ts:
const apolloNetworkInterface = createNetworkInterface(url);
apolloNetworkInterface.use([{
  applyMiddleware: (req, next) => {
    if (!req.options.headers) {
      req.options.headers = {};
    }
    if (localStorage.getItem('token')) {
      req.options.headers.token = localStorage.getItem('token');
      req.options.headers.userid = localStorage.getItem('guid');
    }
    next();
  }
}]);
const apolloClient: ApolloClient = new ApolloClient({
  networkInterface: apolloNetworkInterface
});

First time I run the query I'm not logged in, so there is nothing in localStorage and therefore req.options.headers is empty.
The response is as expected:
user: {
  name: "John",
  email: null
}

Now as a user I want to know Johns email, so I decide to login and try to run the same query again. When I login a token and my userid is saved in localStorage.
This time I expect the req.options.headers to have token and userid in it, so that email is not null.
To my surprise req.options.headers is the same as before I logged in, and Johns email is still null.

Can you put a log or debugger to verify when your middleware function is ever called?

And when it is called, what the headers end up being? This is the primary way people use middlewares so I would be surprised if it didn't work.

After some digging, I found out that I needed to add forceFetch: true to my query
// Example query
this.apollo.query({
  query: gql`
    query getUser($guid: String!) {
      user(guid: $guid) {
        name
        email
      }
    }
  `,
  variables: {
    guid: 'guid'
  },
  forceFetch: true
}).then(success => {...}).catch(error => {...});

💣


Fix bugs around week # in nflgame-update-players script

Based on changes suggested in:
https://github.com/derek-adair/nflgame/issues/106
https://github.com/derek-adair/nflgame/issues/105

I'll be evaluating this shortly, I have had some really crazy months.
If anyone is interested in contributing by pushing/merging/maintaining this i'd be greatly appreciative.  Pretty minimal time right now due to personal circumstances.


Site Analytics

Adding site analytics would provide valuable insights into user demographics and platform preferences. This data will directly inform future event planning, helping us tailor events to our audience's interests and ensure optimal outreach strategies.

Like an onboarding process? I feel like easiest way to do that is to have a google form link that users can choose to fill out or skip when onboarding. It's quick to set up and doesn't require much dev time to show a popup modal with the link the first time a user login.


Add the ability for server fns to be submitted via GET requests

Per popular request, I added the ability for server fns to be submitted via GET requests, with their args included in a query string! The encoding "GetJSON" and "GetCBOR" will do that, and return the response either via JSON or CBOR.
Let me know if I broke something, because I probably did. I modified the axum integration to make this work, but not the Actix one yet, and there's plenty of docs that still need to be updated.
Closes issue #788

@Demonthos We're expanding server functions to allow GET requests, with either JSON or CBOR responses. In other words, rather than POSTing a URL-encoded set of arguments, you could GET with them in the URL query string. This is useful because it enables people to set Cache-Control and other headers, and if they're behind a CDN they can cache server function requests across users that way.
Wanted to get your input as this would affect your implementation as well. Basically this just adds a couple new encoding options to be handled.

The new encoding options look great! It might be worth considering changing the get function to return a complete server function object that contains both the encoding and the server function so that we don't need to maintain a separate hashmap for the encoding.

The new encoding options look great! It might be worth considering changing the get function to return a complete server function object that contains both the encoding and the server function so that we don't need to maintain a separate hashmap for the encoding.

I thought about that, the issue being that  REGISTERED_SERVER_FUNCTIONS returns a HashMap of ServerFnTraitObj. It would be fairly trivial to change that to return a struct containing both things though, if you and @gbj  think that's preferable

I was pointing to the wrong line with my comment about GetCBOR.
One of the two commits I just pushed should fix it by encoding GetCBOR requests as GET requests with URL-encoded arguments, but with CBOR responses. Otherwise trying to run a GetCBOR server function just panicked.
The other one tweaks the Actix handler so it can accept any HTTP method (although obviously mostly GET and POST). Note that get() handlers have to added manually in main.rs for Axum/Viz, which is fine.
...The other two commits are cargo fmt and clippy.

Looks good to me, can't believe I missed the panic

All good. Thanks for your great work as always, Ben! Merging now.


fix(radio): update radio group when programmatically uncheck a radio button

added test cases for uncheck
notify all radio buttons after status change and radio group value change to avoid notify radio group twice

For issue  #609

LGTM


Get diet requirement from database

resolve: #17
Short Description
we have a new pages for Volunnteer that they can choose more option about their diet_requirement.So we create a  table to store these data. And we need to get the data about diet_requirement from database.
Implementation

Create a request parser (get_parser) for parsing user_id from the incoming GET request.
Define the diet_requirement_fields dictionary to map the properties of the dietary requirement to appropriate data types.
Create a Flask Blueprint named diet_requirement_retrieval_bp and an Api instance associated with the blueprint.
Add the RetrieveDietaryRequirement resource to the API with the endpoint /getDietaryRequirement.

Testing cases:
write two cases to test whether it get the correct answer from database

[ ] test case 1:
      diet_requirement1 = DietRequirement(
      user_id=2,
      diet_id=2,
      halal=False,
      vegetarian=True,
      vegan=False,
      nut_allergy=False,
      shellfish_allergy=False,
      gluten_intolerance=True,
      kosher=False,
      lactose_intolerance=True,
      diabetic=True,
      egg_allergy=False,
      other="without oil"
  )}

[ ] test 2
  # Test Case 2
  diet_requirement2 = DietRequirement(
      user_id=3,
      diet_id=3,
      halal=False,
      vegetarian=False,
      vegan=True,
      nut_allergy=False,
      shellfish_allergy=True,
      gluten_intolerance=False,
      kosher=False,
      lactose_intolerance=False,
      diabetic=False,
      egg_allergy=False,
      other=""
  )

[ ] test 3
input user_id where is not in database
user_id=1234

Test result
test 1 return
Return correct diet_requirement and return true

test 2 return
Return correct diet_requirement and return true

test 3 return
Return false

The pull request description doesn't need to have implementation details, that's all included in the code.


lwt's cmis magically available through ocamlfind but not its dll

Right now it seems like I can install reason-cli and get ocamlfind. Also it seems like when I run ocamlfind ocamlc -package lwt.unix -package lwt.ppx with that ocamlfind (and no opam installed at all) it will compile.
That's a bit surprising and a bit annoying, because when I try to run my executable I get a runtime dll loading errors:
Fatal error: cannot load shared library dlllwt-unix_stubs
Reason: dlopen(dlllwt-unix_stubs.so, 138): image not found

The reason seems to be that the env variable CAML_LD_LIBRARY_PATH doesn't point to anything so reason-cli's ocamlfind cannot find the lwt dll. I work around this issue by doing:
export CAML_LD_LIBRARY_PATH=/usr/local/lib/node_modules/reason-cli/rel/3.x.x________________________________________/i/opam_alpha__slash__lwt-2.7.0-234bd21d/lib/lwt

I'm thinking there's two ways to go about this:

prevent reason-cli's ocamlfind from compiling with lwt. I'm guessing lwt's just a dependency of something inside reason-cli, and ocamlfind happens to have the right metadata files or env variables to know about lwt's cmi files, which makes it compile. If we make it error that would be great, it'd force people to download opam to get actual lwt and will prevent all sorts of in-between hacks
fill up the CAML_LD_LIBRARY_PATH with lwt paths. I like this solution less because it's a bit magical that lwt happens to work but no other opam deps (well in reality anything that reason-cli depends on would work I'm guessing). But I find this solution better than the current situation.

What do you all think?

The ocamlfind in reason-cli isn't meant to be used for anything except debugging reason-cli at the moment. OCamlfind needs to know where stuff is - and so it needs its env variables / config populated correctly. I think that every project should have its own ocamlfind dependency, and its own config that is populated correctly, for that specific project, which will avoid the problem.
OPAM does this for you at the global level which causes some problems (much like the problem you are seeing here). Esy (which reason-cli is based off of) does it at the project level, for your project's specific set of dependencies.
I think we should stop exposing ocamlfind from reason-cli or at least discourage it for anything other than debugging a reason-cli installation. Thoughts?

@jordwalke Is there any way to call it something other than ocamlfind so that I don't end up accidentally using it? I want to make sure I have a correct opam install so that things don't sometimes work and sometimes not.
If you think we could get away with not shipping it then that solves my problem :)

Yeah, we can realias binaries that are only coincidentally exported to something like reason-cli-ocamlfind. That way they can still be used to debug the installation, but won't conflict.

Hey sorry I didn't comment on this for a while.
It'd be awesome if we could make this happen

Heyyyy this hit us again, where the ocamlfind that comes with Reason-cli shadows opam’s ocamlfind which makes it unusable.
Any chance we can get this fixed quickly?


[Cherry-pick #28625 for 2.51.0] Update Python base container deps

This is a cherrypick of the same commit merged in #28625 to the release-2.51.0 branch.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

Run Python_Coverage PreCommit


Add support for Teradata

Teradata has developer tools for Visual Studio, but they’re showing no love to VS Code.
The ability to connect using LDAP would be amazing!

Hi @C1ARKGABLE , I don't have time to work on this right now.
But if you want to try to develop this, we have a guide for new dialects.
Would you like to try?
Take a look here: #219

Sure @mtxr! I’ll dip my feet in tomorrow. Will you be around if I have questions at this same time?

Can't promise this, I have a lot of work for tomorrow since it's a holiday in US on Thursday. But you can always drop your message and I'll reply as fast as I can.
I really appreciate you working on this since I have no experience with Teradata
Thanks!
On Jul 2 2019, at 7:53 pm, Gable Brown<EMAIL_ADDRESS>wrote:

Sure @mtxr (https://github.com/mtxr)! I’ll dip my feet in tomorrow. Will you be around if I have questions at this same time?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/mtxr/vscode-sqltools/issues/285?email_source=notifications&email_token=AAFMX2N47R6UJLQLH5U3PETP5PL5PA5CNFSM4H4HGMDKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZCYR5A#issuecomment-507873524), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAFMX2ONA2ILFEBHL7EWCDTP5PL5PANCNFSM4H4HGMDA).

@C1ARKGABLE any updates on this?

Haven’t had much luck. I’d have to make calls to Python or Java because there aren’t any JS drivers. I don’t want to jank it together. @mtxr

Got it.
That's sad. It will be a bit hard to do this kind of integration for now.
I'll close this ticket since we don't have real plans for this soon, ok?
I'll tag it as a third-party-blocking to revisit it in the future.
Thanks for your effort @C1ARKGABLE

Hey there, is there still an issue with teradata not having a JS driver? I found this https://github.com/Teradata/nodejs-driver, but wasn't sure if that was something you'd already looked into. Just tossing it out there since I stumbled on this while looking for a decent non-1900's themed teradata IDE.


Pass down KnownDeployProperties to the GenerateCreateScript functionality

This PR allows setting all the possible properties in the GenerateCreateScript functionality. The properties defined in the csproj (the KnownDeployProperties ones) are now automatically set when generating the create script
Solves issue #230

@jmezach I've resolved all the comments. Let me know if the PR is OK for you

@dovic95 Looks good to me.
@ErikEJ @jeffrosenberg Could either of you have a look at this as well, thanks!

LGTM

Hi, are we waiting for another approval? Would be great if we can complete this PR and release the version :)


Support for service/node "meta" parameters

Docs:
https://www.consul.io/docs/agent/services.html
https://www.consul.io/docs/agent/options.html#_node_meta
Error message:
Evaluation Error: Error while evaluating a Resource Statement, Consul::Service[foo]: has no parameter named 'meta'

Hi @sc0rp10, this is currently not implemented. Are you interested in providing a patch for this?

hi, @bastelfreak
Yes, I can try to PR this, but i'm not familiar with puppet tests and can't spent to much time to it.
May you approve PR without tests?

We can help out with writing tests. Just create the PR and we will have a look at it.


Fix/remove Docker tests

The Docker tests currently require docker hub credentials and any contribution made from a fork fails the test making the CI not very reliable unless you look at the logs. Maybe we should push to docker hub only when a request is merged to master.

Yes the Docker push should only be executed if the branch is NOT a pull request coming from external. I think Travis injects an environment variable to determine whether it's an internal branch or an external PR.
We cannot push only on master as we make use of the tags from branches for test deployments as well.
Would you like to make that change, @sohaibiftikhar?


Updated PR for readme, roadmap and merchandise

See #29

@revett will this do?

@jhepkema Will look at this soon - sorry about the wait.

@revett thats allright, Im on a holiday in Asia so no worries :)


[Issue]: api_buffered_tracing throws exceptions

Problem Description
Trying to build and run the api_buffered_tracing example, I get the following logs:
E20240725 12:34:03.379593<PHONE_NUMBER>31616 buffer.cpp:204] buffer callback threw an exception: vector::_M_range_check: __n (which is 1) >= this->size() (which is 0)
Removing the references to client_name_info in tool_tracing_callback fixes this.
Operating System
sles15sp5
CPU
AMD EPYC
GPU
AMD Instinct MI250X
ROCm Version
ROCm 6.1.0
ROCm Component
No response
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

rocprofiler-sdk is not fully compatible with ROCm 6.1 so I am going to close this. Please report back if this issue still exists once ROCm 6.2 is available.


Flutter crash report.

command
flutter run
exception
RPCError: ext.flutter.debugDumpRenderTree: (-32603) ext.flutter.debugDumpRenderTree: Unexpected error from chrome devtools:
text: Uncaught (in promise)
exception:
  description: null
  type: string
  value: Unexpected null value.
attempted JS eval: `require("dart_sdk").developer.invokeExtension(
    "ext.flutter.debugDumpRenderTree", JSON.stringify({}));
`

#0      handleErrorIfPresent (package:dwds/src/utilities/shared.dart:56:5)
#1      ChromeProxyService.callServiceExtension (package:dwds/src/services/chrome_proxy_service.dart:353:5)
<asynchronous suspension>
#2      VmServerConnection._delegateRequest (package:vm_service/src/vm_service.dart:1592:24)
<asynchronous suspension>

#0      new _OutstandingRequest (package:vm_service/src/vm_service.dart:1626:45)
#1      VmService._call (package:vm_service/src/vm_service.dart:2109:21)
#2      VmService.callServiceExtension (package:vm_service/src/vm_service.dart:2080:14)
#3      FlutterVmService._checkedCallServiceExtension (package:flutter_tools/src/vmservice.dart:773:28)
#4      FlutterVmService.invokeFlutterExtensionRpcRaw (package:flutter_tools/src/vmservice.dart:792:48)
#5      FlutterVmService.flutterDebugDumpRenderTree (package:flutter_tools/src/vmservice.dart:554:48)
#6      ResidentHandlers.debugDumpRenderTree (package:flutter_tools/src/resident_runner.dart:661:52)
<asynchronous suspension>
#7      TerminalHandler.processTerminalInput (package:flutter_tools/src/resident_runner.dart:1659:7)
<asynchronous suspension>

flutter doctor
[✓] Flutter (Channel stable, 2.5.2, on Manjaro Linux 5.14.7-2-MANJARO, locale pt_BR.UTF-8)
    • Flutter version 2.5.2 at /home/ajbg/.programas/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 3595343e20 (7 days ago), 2021-09-30 12:58:18 -0700
    • Engine revision 6ac856380f
    • Dart version 2.14.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /home/ajbg/.programas/android-sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /home/ajbg/.programas/android-sdk
    • Java binary at: /home/ajbg/.programas/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /home/ajbg/.programas/android-studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 94.0.4606.81

• No issues found!

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Show task run test output status on task run nodes

Fixes
Fixes HAC-3200
Description
Determines the test status for pipeline run tasks and if the task run is successful, show any test status to indicate the test status issues.
Type of change

[x] Feature

/cc @christianvogt @karthikjeeyar

Just came from a design share and wanted to share some possible variations on the warning but without adding a new status. Let me know what you think.

Red warning is failure and stops progress
Warning with label is test status and;
Warning on a task but doesn't stop the pipeline

@andrew-ronaldson So an test warning just shows the warning status on the node. A test failure shows the warning status along with a Test failed badge?

We can refine this more but this should work as a starting point. Use the warning state with counter that shows a count issues.

Updated to show badges with counts. Updated the description with the screenshot.

/hold cancel

/lgtm
/approve


Update Jackson to <IP_ADDRESS>

Jackson 2.9.9 contains a few vulnerabilities reported by Snyk:
✗ High severity vulnerability found in com.fasterxml.jackson.core:jackson-databind
  Description: Deserialization of Untrusted Data
  Info: https://snyk.io/vuln/SNYK-JAVA-COMFASTERXMLJACKSONCORE-450207
  Introduced through<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>
✗ High severity vulnerability found in com.fasterxml.jackson.core:jackson-databind
  Description: Deserialization of Untrusted Data
  Info: https://snyk.io/vuln/SNYK-JAVA-COMFASTERXMLJACKSONCORE-450917
  Introduced through<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>
✗ High severity vulnerability found in com.fasterxml.jackson.core:jackson-databind
  Description: Deserialization of Untrusted Data
  Info: https://snyk.io/vuln/SNYK-JAVA-COMFASTERXMLJACKSONCORE-455617
  Introduced through<EMAIL_ADDRESS>  From<EMAIL_ADDRESS>><EMAIL_ADDRESS>

Build failed.  The jackson version specified in the com.fasterxml.jackson.version property is for the jackson-bom.  I believe jackson-bom version <IP_ADDRESS>90807 will use jackson-databind <IP_ADDRESS>, so the com.fasterxml.jackson.version property should be set to <IP_ADDRESS>90807

@philsttr is correct and posted about 2 seconds before I was about to submit the same!
Use <IP_ADDRESS>90807  for jackson-bom.  The only thing that differs between this and the BOM for 2.9.9 is the version of jackson-databind.

Thanks guys!

Is there an estimated release date? @philsttr

6.2 released.


MNT: Re-rendered with conda-smithy 2.4.2

Re-rendered with conda-smithy 2.4.2

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks. Looks good.
Somehow a .DS_Store file crept in. May want to go back and remove that.


Cant Read Module

It keeps saying "Failed to open C:\Program Files\WindowsApps\Microsoft.Halo5Forge_1.157.64802.2_x64__8wekyb3d8bbwe\deploy\pc\levels\globals-rtx-1.module for reading" and I do not understand why

Hello? Does anybody know what my problem is?... So I downloaded fivetool, visual studio community, windows sdk 8.1, the artifacts... Fivetool-master and the ModuleExtractor.exe are both on my desktop... I managed to put the globals-rtx.module onto my desktop as well... However I'm still getting the "Extraction failed: Failed to open C:\Users\Zelor\Desktop\globals-rtx-1.module for reading
Press Enter to quit..." issue and I honestly am stumped at this point

This tool isn't being maintained right now and I can't guarantee that it will work with the latest version of the game.
Also, did you dump the Halo 5 files first using a UWP dumper?

Oh sorry, I talked it some people and we got it fixed... I didn't dump the files yet and that was the issue


Build using Jina : Movie recommender search engine !

At Jina, we build crazy fun things using Jina and one such example is the meme search.
Meme search allows you to upload a meme of your choice and renders many similar memes to your meme !.
If you are interested in getting your hands on coding with Jina and creating something, this is the issue for you.
Idea:
Create a movie recommender search engine using Jina !.
What It does?
As you know, Jina is famous for creating out-of-the-box search engines and we would like you to put your creative hats on an create a search engine using Jina which takes the name of the movie as input and renders similar movies to it.
What is the tech stack?

Jina for backend
Streamlit for frontend (or any other front end framework you prefer)
Dataset ? - feel free to use any !

To help you get started , we have a list of simple examples created using Jina here - https://github.com/alexcg1/simple-jina-examples
How do I get started if I am a beginner?

Read Jina's GitHub repo
Run our examples to get a better understanding - https://github.com/jina-ai/jina#run-quick-demo
Go through the docs, they have everything you need!

I am stuck, I need help !!
We hear you, drop into our discord channel and post your queries there , introduce yourself and get to meet some amazing people building Jina and contributing to Jina !
What if you have no idea what Jina is or neural search is?
Try - https://www.youtube.com/watch?v=Ex6vYQoTbdE , we created a video explaining everything about what is neural search and Jina !
Do I get Swags?
We love appreciating good work in open source and every project you create/every contribution you make will be rewarded with some amazing swags. Find out more at - https://jina.ai/blog/swag/

That's an amazing idea. I also think we all should work together. You can see all my contact info on my Github profile if you want to reach me. You can also reach me if you want are facing any issues as I am done with backend and API stuff.

@soumyac19 @achintya-7 @DPR-4 Anyone who wants to work on this go ahead, and post your repo in this thread. If you want to work together, great - otherwise do your own things. Totally up to you!

That's an amazing idea. I also think we all should work together. You can see all my contact info on my Github profile if you want to reach me. You can also reach me if you want are facing any issues as I am done with backend and API stuff.

Can you share your repo here @achintya-7 ?
For frontend someone could easily adapt my fashion search frontend (be sure to use migrate-3 branch until I merge to main)

@alexcg1 I can but I want to use flutter for frontend. Everyone is using streamlit, so I wanted to try something else. I can share the work in progress repo if you want.

Working Repo : https://github.com/achintya-7/Movie_Recommender_Flutter

@soumyac19 @achintya-7 @DPR-4 Anyone who wants to work on this go ahead, and post your repo in this thread. If you want to work together, great - otherwise do your own things. Totally up to you!

so i can work on it ?

Working Repo : https://github.com/achintya-7/Movie_Recommender_Flutter

@alexcg1 This project is complete. Any new input or suggestion is welcome,

@joeyouss can you try running the code on your machine?

@DPR-4 since this is already done, maybe choose a different example? There are so many different things you can search :)

Hi @achintya-7 , thanks for the efforts. We would love to read that blog. Can you share a draft before publishing for us to review? Once written, please share a link here in the comments and we will get back in no time.

Sure. I hope making it on medium will be fine by y'all.

Sure thing @achintya-7

Sure thing @achintya-7

Hey, sorry I took some time but here is the first draft :
https://medium.com/@achintya22052000/movie-search-with-jina-ai-933ebe118c30

Hi, I have completed the project and also have published an article about it on medium. You all can check it out here
https://medium.com/@achintya22052000/movie-recommender-powered-by-jina-ai-933ebe118c30


ERC20 transfers list error on mainnet

ERC20 transfers for mainnet address: 0x4838B106FCe9647Bdf1E7877BF73cE8B0BAD5f97 is erroring on node side in our test server.
Assigning myself to investigate.

This is still an issue. Page 12 of 15 is returning
{
	"error": {
		"code": -32000,
		"message": "method handler crashed"
	}
}

I've seen the same thing happen on other addresses, too.


Move to Glide

Resolves #20
Due to some packages having folders with an underscore, gvt will not vendor the folders.
Moved vendoring to glide.

not needed gvt has -a flag to not ignore folders with an underscore.


Fix room mention

I'm submitting a...
[x] Bug Fix
[ ] Feature
[ ] Other (Refactoring, Added tests, Documentation, ...)

Trying to resolve #1665
Checklist

[ ] Commit Messages follow the Conventional Commits pattern

A feature commit message is prefixed "feat:"
A bugfix commit message is prefixed "fix:"

[ ] Tests for the changes have been added

Description
The mention list variables are completely misused, fixing them so we can actually use the feature properly.
Tested locally with BROLOG_LEVEL=silly with extra console logs to verify that the variables are properly passed through.
Does this PR introduce a breaking change?
[ ] Yes
[x] No

I really don't feel like writing a test for it, testing alone takes out most of my energy. Please advice.

@lhr0909 Good catch! Thanks for fixing it.
I'll merge this PR after @windmemory approved it.
The unit test is better to have, but it's ok to leave it as is, hope we will have enough resources to cover them in the future.

@lhr0909

You've invited Simon Liang to Chatie! They'll be receiving an email shortly. They can also visit https://github.com/Chatie to accept the invitation.


能否添加 websocket 类？

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Same as  #779?


github/workflows :  fail test_suite_pr if PR images are not pushed yet

While investigating the problem with #3557, it appears that Eden requires the "Publish PR build" step (danger.yml) to complete before it can fetch the PR image for testing. However, "Publish PR build" depends on "PR build" (build.yml).
Sometimes, GitHub runners get stuck during certain builds (arm64, I'm looking at you) and we might trigger the Eden run by approving the PR before "PR build" is finished, and subsequently, "Publish PR build" is completed. This can result in Eden failing to pull the PR image because it hasn't been pushed yet.
I'm not a github expert so this will be a bit of trial and error, help is greatly welcomed.

It's tricky. Now, the workflow starts after Publish is done, along with the previous triggers. So the trigger looks like:
if (buld_published OR pushed OR approved)
But it should be
if (buld_published AND(pushed OR approved))

It's tricky. Now, the workflow starts after Publish is done, along with the previous triggers. So the trigger looks like: if (buld_published OR pushed OR approved) But it should be if (buld_published AND(pushed OR approved))

yes, I try another way :(

The desired logic is not supported from scratch by GitHub actions and requires some extra magic...
Maybe @uncleDecart or @yash-zededa could help here.

@shjala, @OhmSpectator is right, behavior is tricky and it won't work as expected. IMO we need to compose build test publish under one workflow because that way we can manage this behavior better. I think it makes sense to have a call for everyone interested tomorrow.

Am I right that it will still fail the Eden tests, but this time, explicitly?

yes, that is the idea

I tested it locally with act, here is how.

created a event config (event.json) with the following content:

{
  "pull_request": {
    "number": 1337
  }
}

I tried to create a pull_request_review event with action and state but I don't think it currently supported, anyways this works for now, btw this values are supposed to be based on github events.

change the "number" to a PR that it's images are already published, like 3568 and run act, telling it to simulate a pull_request_review event with the config file event.json, act looks into all workflows and as expected only runs eden.yml and succeeds to run check_pr_publish as expected :

❯ act pull_request_review -e event.json
WARN  ⚠ You are using Apple M-series chip and you have not specified container architecture, you might encounter issues while running act. If so, try running it with '--container-architecture linux/amd64'. ⚠
[lf-edge/eden Test suite/check_pr_publish ] 🚀  Start image=node:16-buster-slim
[test_suite_master/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
INFO[0000] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker pull image=node:16-buster-slim platform= username= forcePull=true
INFO[0001] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker create image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker run image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/check_pr_publish ] ⭐ Run Main Install curl and jq
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /var/run/act/workflow/0] user= workdir=
Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
Get:2 http://deb.debian.org/debian-security buster/updates InRelease [34.8 kB]
[...]
| 0 added, 0 removed; done.
| Running hooks in /etc/ca-certificates/update.d...
| done.
[lf-edge/eden Test suite/check_pr_publish ]   ✅  Success - Main Install curl and jq
[lf-edge/eden Test suite/check_pr_publish ] ⭐ Run Main Check Image Availability
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /var/run/act/workflow/1] user= workdir=
| Docker image evebuild/danger:pr3568-xen and evebuild/danger:pr3568-kvm are available on Docker Hub.
[lf-edge/eden Test suite/check_pr_publish ]   ✅  Success - Main Check Image Availability
[lf-edge/eden Test suite/check_pr_publish ] Cleaning up container for job check_pr_publish
[lf-edge/eden Test suite/check_pr_publish ] 🏁  Job succeeded
[test_suite_pr/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
❯

to simulate a failure, we need a GITHUB_TOKEN for the comment API, and it seems values like context.repo.owner in the createComment are extracted from the token (or git configurations?), so when I provide my own token it tries to comment a repo I own, so to simulate this I created a PR in my own repo https://github.com/shjala/eve/pull/2 , changed the "number" in the config to 2 and ran it again, here is the result :

❯ act -s GITHUB_TOKEN="$(gh auth token)" pull_request_review -e event.json
WARN  ⚠ You are using Apple M-series chip and you have not specified container architecture, you might encounter issues while running act. If so, try running it with '--container-architecture linux/amd64'. ⚠
[lf-edge/eden Test suite/check_pr_publish ] 🚀  Start image=node:16-buster-slim
[test_suite_tag/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_master/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_tag/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
INFO[0000] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker pull image=node:16-buster-slim platform= username= forcePull=true
INFO[0001] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker create image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker run image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/check_pr_publish ] ⭐ Run Main Install curl and jq
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /var/run/act/workflow/0] user= workdir=
Get:1 http://deb.debian.org/debian buster InRelease [122 kB]
Get:2 http://deb.debian.org/debian-security buster/updates InRelease [34.8 kB]
[...]
| done.
[lf-edge/eden Test suite/check_pr_publish ]   ✅  Success - Main Install curl and jq
[lf-edge/eden Test suite/check_pr_publish ] ⭐ Run Main Check Image Availability
[lf-edge/eden Test suite/check_pr_publish ]   🐳  docker exec cmd=[bash --noprofile --norc -e -o pipefail /var/run/act/workflow/1] user= workdir=
| Docker image evebuild/danger:pr2-xen and evebuild/danger:pr2-kvm are not available on Docker Hub.
[lf-edge/eden Test suite/check_pr_publish ]   ❌  Failure - Main Check Image Availability
[lf-edge/eden Test suite/check_pr_publish ] exitcode '1': failure
[lf-edge/eden Test suite/check_pr_publish ] 🏁  Job failed
[lf-edge/eden Test suite/report_pr_publish] 🚀  Start image=node:16-buster-slim
[test_suite_pr/Test/Smoke tests-1] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-3] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-4] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
[test_suite_pr/Test/Smoke tests-2] 🚧  Skipping unsupported platform -- Try running with `-P buildjet-4vcpu-ubuntu-2204=...`
INFO[0013] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/report_pr_publish]   🐳  docker pull image=node:16-buster-slim platform= username= forcePull=true
INFO[0015] Parallel tasks (0) below minimum, setting to 1
[lf-edge/eden Test suite/report_pr_publish]   🐳  docker create image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/report_pr_publish]   🐳  docker run image=node:16-buster-slim platform= entrypoint=["tail" "-f" "/dev/null"] cmd=[] network="host"
[lf-edge/eden Test suite/report_pr_publish]   ☁  git clone 'https://github.com/actions/github-script' # ref=v6
[lf-edge/eden Test suite/report_pr_publish] ⭐ Run Main actions/github-script@v6
[lf-edge/eden Test suite/report_pr_publish]   🐳  docker cp src=/Users/administrator/.cache/act/actions-github-script@v6/ dst=/var/run/act/actions/actions-github-script@v6/
[lf-edge/eden Test suite/report_pr_publish]   🐳  docker exec cmd=[node<EMAIL_ADDRESS>user= workdir=
[lf-edge/eden Test suite/report_pr_publish]   ✅  Success - Main actions/github-script@v6
[lf-edge/eden Test suite/report_pr_publish]   ⚙  ::set-output:: result=
[lf-edge/eden Test suite/report_pr_publish] Cleaning up container for job report_pr_publish
[lf-edge/eden Test suite/report_pr_publish] 🏁  Job succeeded
Error: Job 'check_pr_publish' failed

and you can see the comment on my PR : https://github.com/shjala/eve/pull/2

Yep, it works because you created a PR from the same repo... We can try to redo the test with a PR from a forked repo... Or just merge and see how it works in our master =D

Yep, it works because you created a PR from the same repo... We can try to redo the test with a PR from a forked repo... Or just merge and see how it works in our master =D

I think it works because the GITHUB_TOKEN="$(gh auth token)" I used has write perms.

oops, sorry, it fails here with the permissions error, yeah...
https://github.com/lf-edge/eve/actions/runs/6825241108/job/18562698195?pr=3572

but it also failed on apt-get, that is odd!

cannot we just make the eden job only run after the build job is completed?
on:
  workflow_run:
    workflows: ["PR build"]
    types:
      - completed

or maybe we should at least add it to the other triggers, so that it starts automatically after the build is finally done.

@europaul it was the very first version of the WF change in this PR =)
See comment here: https://github.com/lf-edge/eve/pull/3572#issuecomment-1804046439

@europaul it was the very first version of the WF change in this PR =) See comment here: #3572 (comment)

@OhmSpectator I thought it might have been :) but there is no easy way to check with all the force pushes that we do :cry:
so my reasoning was as follows:
we replace the other triggers with the one that I described, but we still check whether the PR was approved in the if-block. This way the build job gets triggered on every push (like now) and (I assume) every time it finishes it will also trigger the eden tests.

@europaul I'm not following your comment, is it a solution to the problem? if so, would please elaborate on it?

@shjala yes, I think that your initial implementation was correct, but instead of adding this trigger to the others, we should make it the only trigger. Because push trigger is already included in the build workflow and pull_request_review is not strictly necessary since we already check if the PR is approved in
if: github.event.review.state == 'approved'

@europaul that way it'll be skipped if not approved, and if we push new changes it won't be triggered if we push new changes.Build will be triggered, but no tests. To my best knowledge GHA does not support transitive property :)

@uncleDecart but every time the build workflow completes, the tests will be triggered, right?

@uncleDecart, why did the tag "stable" appeare? We have not yet found the solution for the permissions error )

@OhmSpectator when we will find a fix, we will need to merge it to stable branches

I recall seeing some workflow where we build it from source if the image can not be pulled from docker hub. It would make sense to remove that code, but I don't see any code being deleted in this PR.

That is in eden_old which is disabled.

@europaul, missed your message, sorry. Yes, from what I see that's how danger.yml is dependent on build.yml

@europaul, missed your message, sorry. Yes, from what I see that's how danger.yml is dependent on build.yml

@uncleDecart then: push will trigger the build -> build completion will trigger the tests

@europaul true, but what if arm build will fail? We don't use Arm images in Eden tests, that means if Arm build is failing we are not running tests. Or if it's completed, but not successful we will waste eden running time. And as we can see now, sometimes arm fails because of the runners. We do need to introduce changes to separate build & publish for different architectures so that if one build fails we can continue with other builds and eden tests, or make one big workflow file with high-level dependencies (my favorite option, TBH). But this solution will stop the current bleeding


Unknown flag error for k3d cluster create

Bug Report
When I run:
k3d cluster create --config k3d.yaml --k3s-server-arg "--no-deploy=traefik" --k3s-server-arg "--no-deploy=servicelb"
I get the following error:
FATA[0000] unknown flag: --k3s-server-arg
Description
# k3d version
k3d version v5.1.0
k3s version v1.21.5-k3s2 (default)

Expected Behavior
No error when running the cluster create command.

NOTE: This is running the non-Codespaces version.

I tried replacing with --k3s-arg but got the following error.  I'd recommend adding a requirements section and listing the expected k3s/k3d versions
# k3d cluster create --config k3d.yaml --k3s-arg "--no-deploy=traefik" --k3s-arg "--no-deploy=servicelb"

INFO[0000] Using config file k3d.yaml (k3d.io/v1alpha2#simple)
WARN[0000] Default config apiVersion is 'k3d.io/v1alpha3', but you're using 'k3d.io/v1alpha2': consider migrating.
FATA[0000] K3sExtraArg '--no-deploy=traefik' lacks a node filter, but there's more than one node

This is a breaking change between k3d 4.x and 5.x. The Codespace still uses 4.8.x because of some upstream dependencies. I'll get a fix merged.

updated the Codespace to use k3d 5.x
updated k3d.yaml
updated readme


Document shell completion

Add in a stanza to the Usage section about tab completion

Thanks for the help!

No problem. So glad to have a usable tool that keeps me away from the browser. Thanks for making this tool!


indentation not working

{% extends "base.html" %} {% load static %} {% load cache %} {% block content %} {% cache 500 product_list %}

    {% for product in object_list %}

{{ product.title }}

    {% endfor %}

{% endcache %} {% endblock %}

all the blocks are lined up in one line, and also it is necessary that the li tag has an indent

Not experiencing this.  But in my case, the HTML indentation is handled by this plugin: https://github.com/othree/html5.vim
I tried it with the default $VIMRUNTIME/indent/html.vim and it still worked fine.  You might have another plugin that took over indentation.


BUG: Incorrect logging when default port is already in use

Description:
When the default port (7379) is already in use and the DiceDB server is started, a success message is incorrectly logged showing DiceDB 0.0.3 running on port 7379 before displaying the error: Error finding and binding port: address already in use
Steps to reproduce

Start a service or process that uses port 7379
Start DiceDB server with the default port 7379
Observe the logs.

Expected output
The success message indicating that DiceDB is running should only be logged after the server has successfully bound to the port and started. If the port is already in use, no success message should be logged.
Observed output
The success message DiceDB 0.0.3 running on port 7379 is logged before attempting to bind the port. This causes confusion as the server immediately follows with an error message:
Error finding and binding port: address already in use

Expectations for resolution
The issue seems to stem from the success message being logged before the server attempts to bind the port and start. This message should be moved to a point after the successful binding and startup of the server.

@shardul08 assigned, thanks for contributing.


Create Area of Shapes calculator

Description of Change

Checklist

[ ] Added description of change
[ ] Added file name matches File name guidelines
[ ] Added tests and example, test must pass
[ ] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[x] Relevant documentation/comments is changed or added
[ ] PR title follows semantic commit guidelines
[ ] Search previous suggestions before making a new one, as yours may be a duplicate.
[ ] I acknowledge that all my contributions will be made under the project's license.

Notes:

Read the contributions guidelines for the repo.


Constructing a wall on a pile of rubble returns "A(n) west wall already exists here"

The solution to this depends on the results of destroying a ceiling (see #29). Currently a surface is deemed "rubble" when its health reaches 0.
Response for walls and floor should be "Rubble prevents construction of a new wall/floor."

Fixed. Changes will be pushed to binaries when milestone is complete.


[USM] enable_http_incomplete_buffer config

What does this PR do?
As discussed offline, incomplete buffer as implemented today doesn't associate the corresponding requests and responses so create request/response with high latency
This will reduced the accuracy as there will be less requests/responses hit but reduce the max latency of some endpoints
But the current implementation could wrongly associate response to other endpoint/resource.
This PR add a configuration flag enable_http_incomplete_buffer disabled by default
service_monitoring_config.enable_http_incomplete_buffer = false
EnableHTTPIncompleteBuffer specifies if the HTTP incomplete buffer need to be enable to associate orphan requests and responses due to NAT

Without post /api/v2/series endpoint

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Let's wait for the RFC to have a complete solution for that area

We can enable by default if you prefer (as it's running today)
Today we don't have a way to disable it without drawback (telemetry.dropped++)
The RFC will point the issues and possible solutions, but as discuss offline it would not be an easy/quick fix

As the impact of disabling the incomplete buffer is harming the accuracy, i don't want to have the option to disable it, until we figure out the correct solution for the actual issue


lame typo pull request

Excellent effort, I was looking at Mike Amundsen templates and wondering where I should start with my own media types...

Ahh, I suck, thank you! This is very much a work in progress, so give it a shot and feel free to make more modifications!


Adding a few more requirements

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Additions

[x] added python<38 requirements
[x] added win23 requirements
[x] added build requirements

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [25, 26, 30]
noarch packages can't have selectors. If the selectors are necessary, please remove noarch: python.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Non noarch packages should have python requirement without any version constraints.
Non noarch packages should have python requirement without any version constraints.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender


Add note/reminder about the node URL

:+1:

Current coverage is 100% (diff: 100%)

Merging #127 into master will not change coverage

@@           master   #127   diff @@
====================================
  Files           7      7
  Lines         139    139
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          139    139
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 88b9e3a...1afd324


https://github.com/jslint-org/jslint/issues/new?assignees=&labels=enhancement&template=feature_request.md&title=Feature+request%3A+Your+title+here

Is your feature request related to a problem? Please describe.

Describe the solution you'd like

Describe alternatives you've considered

closing as invalid issue


Unable to click "+ new chat"

! [image] (https://github.com/casibase/casibase/assets/162850856/dbd20ae1-9acd-416d-8e2b-93e0e1a1b7e8)
The Add New Dialogue button has been unable to be clicked, and the configured large model cannot be used at all

The new button will be disabled if you have too many empty chats (chats with no messages in it)


Go to Definition for Magic Methods

I know that this may be very difficult to do, so I guess that this is more of a feedback post.
In laravel and other frameworks (and non-frameworks), magic methods are used for certain features, so I would suggest that it could be possible to define a method/property using PHPDoc.
I don't believe that there is a doc comment for this, so I will make one up called: @called in this example.
It would look something like this:
class A extends ClassWithMagic {
    /**
     * @called MyMethod
     */
    public function scopeMyMethod() {}
}

Example of what the extended class might look like:
class ClassWithMagic {
    public function __call($name, $args){
        $func = 'scope' . $name;
        $this->$func($args);
    }

    public function __get($name) {
        return $this->$name;
    }
}

Then it could be used something like this:
A::MyMethod();
// or
$a = new A;
$a->MyMethod();
// or
(new $a)->MyMethod();

It would know about the method due to the doc comment.

There is support for this in phpdoc. It's called @method. Documentation: https://github.com/phpDocumentor/fig-standards/blob/master/proposed/phpdoc.md#711-method
Looks like intelephense already supports it and works fine.
Would look like this:
/**
 * @method string getString()
 * @method void setInteger(int $integer)
 * @method setString(int $integer)
 */
class Child
{
    <...>
}

That somewhat works, maybe I am doing it wrong, but it doesn't go to the function, just where the comment is. Is there a way to use it so it goes to the method?
/**
 * @method void DefaultSelect()
 */
class GameCache extends BaseGameModel{
    public function scopeDefaultSelect($query, $filters){
		// Do stuff
	}
}

Oh, I see. So in the __call you end up funneling it to the scopeDefaultSelect (as an example)? Yeah, I don't think that phpdoc allows you to mark that at least...

Yeah, that is a Laravel feature for doing database queries:
https://laravel.com/docs/5.5/eloquent#local-scopes
Other frameworks may do something differently

I don't think this could be really helpful as code should not depend on an editor. Adding random (and non-sense for others) phpdoc tags will 'couple' the code to one editor plugin, not even an editor, but a plugin feature, ending on dirty code. I'm in a project where some people use Atom, others Sublime or even PhpStorm. I see no reason to have code with custom stuff so each ones editor works, and I cannot imagine my self tagging a scope with @called for me, tagging whatever method with @forwarded so it works on some random Atom plugin and @scopedMethod for my mate using a Sublime plugin. However, suggesting this tag to the phpDoc team may be helpful, some kind of mix between @method and @see

I did submit something after. No reply yet though

I did submit something after.

Cheers mate!
I'd love to have this feature supported on phpDoc blocks

Good discussion, I don't think introducing a new phpdoc tag is appropriate, I'd rather stick with current standards. @arturock mentions @see. I wonder if this would be suitable in itself. Document _call with @see tags and then go to definition can provide multiple locations. It wouldn't distinguish between scopeMethodA and scopeMethodB but at least provides useful information rather than just going to the @method in the class docblock.

I think the way to go here would be to use inline @see in the @method description.
/**
 * @method void DefaultSelect() {@see GameCache::scopeDefaultSelect}
 */
class GameCache extends BaseGameModel{
    public function scopeDefaultSelect($query, $filters){
		// Do stuff
	}
}

would this also work with, as an example, laravel facades:
https://github.com/laravel/framework/blob/5.5/src/Illuminate/Support/Facades/DB.php
the @see directive is class level and not method level. Currently when using a facade no intellisense is provided.


[source] Missing event because of missing retry

Describe the bug
Missing events because retry is missing
Expected behavior
We expect when a sink post fails, we should retry so we don't have missing events
To Reproduce
Produce half million messages to a topic of more than 100 partitions and have all messages forwarded to a sink with no delay.
Knative release version
Additional context
Add any other context about the problem here such as proposed priority

Duplicate of https://github.com/knative-sandbox/eventing-kafka/issues/639

@lionelvillard Like @slinkydeveloper said, I think supporting delivery spec could be a good option. What do you think?

Do not close on this issue yet. Whether the missing events due to retry works or not depends on our tests results. We need multiple tests to confirm whether the fix works or not.


Amplify pull - The specified key does not exist.

Describe the bug
I want to pull a currently deployed Amplify App into an empty folder with the command provided by the Amplify console amplify pull --appId myAppId --envName dev. Previously I was having credential problems in my .aws/credentials & config files so I cleaned it completely and did aws configure with the access & secret key that has access to the Amplify App as the deafult aws cli profile. This is the error I was getting before cleaning out the credentials:
Amplify AppID: myAppId not found. Please ensure your local profile matches the AWS account or region in which the Amplify app exists.
Failed to pull the backend.
Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1

After cleaning out (currently):
? Do you want to use an AWS profile? Yes
? Please choose the profile you want to use default
? Which app are you working on? <myAppId>
Backend environment 'dev' found. Initializing...
Failed to pull the backend.
The specified key does not exist.
NoSuchKey: The specified key does not exist.
    at Request.extractError (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/services/s3.js:831:35)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/usr/local/lib/node_modules/@aws-amplify/cli/node_modules/aws-sdk/lib/sequential_executor.js:116:18)

I have tried reinstalling the amplify cli with npm uninstall -g @aws-amplify/cli & npm install -g @aws-amplify/cli. The reinstall was successful. Still getting the error. I have also tried running the amplify pull --appId myAppId --envName dev command from the app's locally cloned repository with the same results. The only difference there is it prompts me to choose which app I want to pull (as seen above).
To Reproduce
Steps to reproduce the behavior:

Deploy an Amplify App through the console
Go to "Backend Environments" -> edit -> copy the amplify pull... command
Execute the amplify pull... command in an empty directory
Exdecute the amplify pull... command in the app's git cloned directory

Expected behavior
When I execute the pull command, it pulls the complete deployed app to local and I can continue working on it.
Code Snippet
mkdir myAmplifyApp
cd myAmplifyApp
amplify pull --appId myAppId --envName dev

Screenshots

What is Configured?
aws sts get-caller-identity returns the correct user that has full access to the Amplify app.
Environment
System:
    OS: macOS 10.15.5
    CPU: (4) x64 Intel(R) Core(TM) i5-4258U CPU @ 2.40GHz
    Memory: 86.59 MB / 8.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 14.4.0 - /usr/local/bin/node
    npm: 6.14.4 - /usr/local/bin/npm
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 83.0.4103.116
    Firefox: 78.0.2
    Safari: 13.1.1
  npmGlobalPackages:
    @aws-amplify/cli: 4.24.2
    favicon-emoji: 2.3.0
    npm: 6.14.4
    twilio-cli: 1.12.0

Amplify CLI maintains a S3 deployment bucket for each backend env, there is a file #current-cloud-backend.zip in the deployment bucket, its content is the zipped version of the amplify/#current-cloud-backend folder. That's where the Amplify CLI stores the current status of an backend env, and how the CLI can pull the latest into the local development.
For your issue, the CLI is complaining it can not find that particular file to execute the pull command.
Did you allow the Amplify Console to complete its setup of the backend env? or have you manually modified the content of the S3 deployment bucket?

I checked and there are two buckets in the account for Amplify-deployment. One is indeed missing the #current-cloud-backend.zip file. I have decided to create a new application from scratch and switch the domain over. The new one works without any problems and the original one was a first-iteration anyways. I am not sure what happened to the app's bucket, likely an accidental deletion. Thank you for the help.


Closes: #1

updated validations.py python script
fixed the behaviour of validate_user function in validations.py
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot i signed it!


[NO MERGE] Add Super Economic to Server

The fee estimate endpoint needs to be updated as a predecessor to this PR.
Do not merge until the endpoint has been updated.

Coverage decreased (-0.01%) to 41.245% when pulling 5c0e3af88305e3c8008d2b3030a282bafa9f7151 on brian.supereconomy into b01f6bc8db9d40c7183a2fd81544020ba8fdb1bc on master.