Allow Document Upload at the Plan Level

We had a request to upload documents which pertain to the entire DMP, in-order to keep all DMP content in the one place.  (This would be separate to per-question uploads)
This seems feasible to handle through Dragonfly for now, and to refactor over to ActionStorage when we upgraid to Rails 5.
We will still need to think about:
Where/how to display/preview uploaded documents.
How to handle the export of DMPs with attached documents across our various formats.

Added to #2233 - duplicate issue


Add manifold-sql reference

Add manifold-sql reference in 'SQL Clients and IDE that Support DuckDB'

Done :)


macOS runners are now failing with "Unity Hub: Bad CPU type in executable"

After changes in #72 some macOS related jobs are now failing with this:
font-mansalva                            vcam
font-manuale                             warcraft-logs-uploader
font-maple                               wins

You have 4 outdated formulae installed.

==> Downloading https://public-cdn.cloud.unity3d.com/hub/prod/UnityHubSetup.dmg
######################################################################### 100.0%
Warning: No checksum defined for cask 'unity-hub', skipping verification.
==> Installing Cask unity-hub
==> Moving App 'Unity Hub.app' to '/Applications/Unity Hub.app'
🍺  unity-hub was successfully installed!
Unity Hub installed successfully.
npm WARN exec The following package was not found and will be installed<EMAIL_ADDRESS>++ '/Applications/Unity Hub.app/Contents/MacOS/Unity Hub' -- --headless install --version 2022.3.33f1 --changeset b2c853adf198 --module mac-il2cpp --childModules
/bin/bash: line 65: /Applications/Unity Hub.app/Contents/MacOS/Unity Hub: Bad CPU type in executable
++ set +x
Could not install the Unity Editor.
Please try again or open an issue.
Something went wrong.
Please try again or open an issue.

Exited with code exit status 1

Any help would be appreciated :)

i have fixed this!  will submit a PR.


docker.local name cannot be changed

Expected behavior
The app should either detect which hostname to use (docker.local vs. localhost), or allow the user to change it in preferences.
Actual behavior
docker.local appears to be hard-coded, or its setting is non-obvious.
Information about the Issue
This issue is minor and may be a non-issue after integration into Docker for Mac.
Steps to reproduce the behavior

Install Docker for Mac Beta
Start a container that has a open port such as nginx
The web preview does not display properly

This is a duplicate of 1661.  Closing.


Cannot load module

Describe the bug
In the latest version, when I invoke Lspsaga peek_definition I get this error. Similar errors occur for other Lspsaga commands.
Error executing Lua callback: ...e/pack/packer/start/lspsaga.nvim/lua/lspsaga/command.lua:8: loop or previo us error loading module 'lspsaga.definition' stack traceback:         [C]: in function 'require'         ...e/pack/packer/start/lspsaga.nvim/lua/lspsaga/command.lua:8: in function <...e/pack/packer/start/ lspsaga.nvim/lua/lspsaga/command.lua:7>         ...e/pack/packer/start/lspsaga.nvim/lua/lspsaga/command.lua:60: in function 'load_command'         ...m/site/pack/packer/start/lspsaga.nvim/plugin/lspsaga.lua:8: in function <...m/site/pack/packer/s tart/lspsaga.nvim/plugin/lspsaga.lua:7>
Steps to reproduce
open code file. Invoke :Lspsaga peek_definition
Expected behavior
Show definition
Neovim version (nvim -v)
0.8.2
lspsaga commit
0fa7160
Terminal name/version
iTerm2

can't reproduce .

getting the same error

I am not  god . post your lspsaga config.

I don't have any specific config for lspsaga, just default setup like below to call code actions from whichkey. Maybe some other plugin brroke :/
in plugins
  use({ "glepnir/lspsaga.nvim", branch = "main" }) -- LSP UIs

from whichkey.nvim calling
l = {
    name = "LSP",
    a = { "<cmd>Lspsaga code_action<CR>", "Code action" },

you should call lspsaga.setup function . please read doc

thanks, not sure how it was even working without the setup function 😆

I managed to solve my issue. Seem that in addition to calling packer to load and setup lspsaga:
require('packer').use({     "glepnir/lspsaga.nvim",     branch = "main",     config = function()         require('lspsaga').setup({})     end, })
I also had to call the setup function myself:
local saga = require('lspsaga') saga.setup({})

why don't read doc the install section  ?

I did read it. The example configuration there did not work for me.

package mangement should call the config function after plugin load. problem not here.

lazy better than packer.


TraversalModels register the features that they update

In order to more clearly delineate between TraversalModels in a hierarchical arrangement, we can enforce that any TraversalModel "registers" the feature it updates.
This would assert the following conventions + code changes:

state features should not be defined at the root of the TOML in a [state] section, but instead, always come from traversal model configuration
each traversal model implements TraversalModel::register_features() (currently state_features()) and returns the state features that it updates

if a traversal model depends on a feature from an inner model (hierarchical traversal models), it does not register the feature, but instead, assumes that the features has been registered by the inner model
bonus points: the feature names are parameterized in the config JSON used to build a model, allowing us to define the (somewhat arbitrary) feature names at config time

StateModel collects these and builds a state representation as currently implemented
Remove StateModelService

we no longer persist StateModel feature configurations
we no longer define a top-level [state] in the TOML with globally-set state features

Yeah I like this idea, having the default state features come from the traversal model. Then, at query time, the user could propose a new initial condition for any state feature. If it exists in the state vector that was built by the traversal model it will update it, or, if it doesn't exist, either ignore that parameter or fail.

I think this happens for the most part, but, I don't think user queries can update the state model yet, so I'm keeping it open until we resolve that capability.


Could not find version: latest of iTMSTransporter to download

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Question Subject
Summary:

Currently using Fastlane 2.115.0
I'm running the lane upload_to_testflight
I get a successful response: "Successfully uploaded the new binary to App Store Connect"
The App never made it to Testflight
The console shows an error: Could not find version: latest of iTMSTransporter to download.

I'm not submitting this issue as a bug, because we are behind a proxy.
We had issues before with Fastlane trying to reach the Itunes Connect APIs, and I got help from the IT team to configure some proxies, create a whitelist of APIs to ues,  and everything started to work.
We stop using this environment for a couple of months, and now that I'm back I see this new issue.
Question: Is there an API that fastlane is trying to reach to download the latest iTMSTransporter? If that's the case we could include it in the whitelist for the current proxies.
I see in the logs the iTMSTransporter creates this directory: ~/.itmstransporter
Is that where the latest iTMSTransporter is stored?

Question Description
This is the lane we use:
upload_to_testflight
testflight_log.txt
(
username: "myusername",
ipa: ipa_path,
team_name: group_name,
skip_waiting_for_build_processing: true
)
Log attached.

That is probably output from iTMSTransporter, that is trying to update itself. iTMSTransporter is part of Xcode, but seems it decided it has to update itself. Because of your restrictive network configuration it can't do that and failing.
It also seems as if the code doesn't handle that case very well, giving you the wrong success message :/

@joquerod the list of hosts The Transporter connects to can be found here: https://help.apple.com/itc/transporteruserguide/#/apdATD1E112-D1E1A1303-D1E112A1126

(We could probably add a documentation page "fastlane in enterprise environments" or similar, that collects information like that about IPs to unblock etc.)

Yes! Fastlane for enterprises needs its own section.
@sshaw That's what I needed.
Thank you guys.

Could you create a new issue for that @joquerod and already include the information you already have and gave to your IT team? That would be a nice start.

This issue has been resolved removing the ~/.itmstransporter folder from the machine.
We didn't have to add all the new APIs to the whitelist. We didn't try it because it will take more time working it out with the IT team.
We still get an error and I think is related to our proxy "Could not find version: latest of iTMSTransporter to download"
If we ignore the error everything seems to work fine.
Every time we ran this Jenkins job we got a successful response: "Successfully uploaded the new binary to App Store Connect" but we had to check on Itunes Connect to be sure the App was there.
The only difference after removing the cash folder is that the App is actually there!
@janpio do you still want me to create another issue? as a bug? For sure the expectation is to avoid false positives, and receive the right message even if you are behind a proxy.

Yes, feature or bugs report - it will be more a placeholder for "add some enterprise documentation and common gotchas" ;)


Wiki Jakarta Annotations API

So I just found out that v2 supports Nullable/Nonnull Although I don't think you should mention every annotation library ever... I think mentioning this is worth it, since it's the spiritual successor to findbugs jsr305 (I'd argue) and maybe someone else would learn about it this way.

Where do you think this should be better documented?  On the Supported Annotations page we have this, indicating that any @Nullable annotation is supported:
https://github.com/uber/NullAway/wiki/Supported-Annotations#nullability
Is that what you're looking for?

yeah, I was just thinking it would be worth specifically mentioning jakarta, since it's basically a light copy over from javax.annotation. Although it seems everyone? is going to go to jspecify... or something, it's not clear to me what that's actually going to mean.


Problem with single file components

TL;DR: see last sentence.
I started using vue back when it was at version 0.12 or so and built a test application. Back then I didn't use single file components. I had script sections in the html files that had logic in them. It was.. ok-ish to build an app like that, but the problem was, when it got more complex, I couldn't always remember where the code I wanted to change was, and frankly I shouldn't need to remember if you separate concerns correctly.
What I then did was separating the vue (viewmodel) code into their own files and putting them all in a file. Then I had an app.js and imported all these single files using gulp. It worked great, BUT the view (html and css) was still in the according html file. So I then knew where to find the code I wanted or needed to change, but then I had to look for the html file where I used the viewmodel.
I now wanted to try a very simple test application using the guide on http://vuejs.org/guide/application.html to build single file components... but using webpack + vue-loader seemed very impractical to use and I still don't fully know what's going on there (might be me, but the documentation describes this process rather vaguely).
So I basically am looking for an easy, fast, efficient and nice way of separating concerns whilst being able to locate viewmodels and views fastly inside my project.

Use forum for questions next time.
Where is an example with vue-loader to learn how to do it.

Woops, sorry, must have confused tabs.
Anyway, I've seen this example, but it's not very intuitive, is it. And it looks like you have a lot of overhead using this method when creating a new component (importing and registering the component, css).


Add batched inference

[x] Use llama_decode instead of deprecated llama_eval in Llama class
[ ] Implement batched inference support for generate and create_completion methods in Llama class
[ ] Add support for streaming / infinite completion

Silly question, does that also support for parallel decoding in llama.cpp?

Does the newest version support "batched decoding" of llama.cpp?
https://github.com/ggerganov/llama.cpp/pull/3228

This would be a huge improvement for production use.
I tested locally with 4 parralel requests to the built-in ./server binary in llama.cpp and am able to hit some insanely good tokens/sec -- multiple times faster than what we get with a single request via the non-batched inference.

This would be a huge improvement for production use.
I tested locally with 4 parallel requests to the built-in ./server binary in llama.cpp and am able to hit some insanely good tokens/sec -- multiple times faster than what we get with a single request via the non-batched inference.

@LoopControl  How did you do 4 parallel requests to the ./server binary? Can you please provide an example, I'm trying to do the same. Thanks!

@LoopControl How did you do 4 parallel requests to the ./server binary? Can you please provide an example, I'm trying to do the same. Thanks!

There's 2 new flags in llama.cpp to add to your normal command -cb -np 4  (cb = continuous batching, np = parallel request count).

@

@LoopControl How did you do 4 parallel requests to the ./server binary? Can you please provide an example, I'm trying to do the same. Thanks!

There's 2 new flags in llama.cpp to add to your normal command -cb -np 4 (cb = continuous batching, np = parallel request count).

Thanks, that works for me with llama.cpp, but not llama-cpp-python, which I think is expected. Unfortunately, the server API in llama.cpp here  doesn't seem to be as good as the server in llama-cpp-python, at least for my task. Using the same llama model, I get better results with llama-cpp-python. So, I hope this can be added soon!

When will this feature be available? I hope anyone can help solve this problem please.

Let me know if there are any roadblocks - I might be able to provide some insight

Hey @ggerganov I missed this earlier.
Thank you, yeah I just need some quick clarifications around the kv cache behaviour.
The following is my understanding of the kv_cache implementation

The kv cache starts with a number of free cells initially equal to n_ctx
If the number of free cells gets down to 0 the kv cache / available context is full and some cells must be cleared to process any more tokens
When calling llama_decode, batch.n_tokens can only be as large as the largest free slot, if n_tokens is too large (llama_decode returns >1) you reduce the batch size it and retry
The number of free cells decreases by batch.n_tokens on every call to llama_decode
The number of free cells increases when a kv cell in the cache is cleared (.pos = -1 and seq_id.clear())
Calling llama_kv_cache_seq_cp does not use up any new cells, the copy is "shallow" and only adds the new sequence id to the set
Calling llama_kv_cache_shift works by modifying the kv cells that belong to a given sequence however this also shifts this cell in all of the other sequences it belongs to

Is this correct?

Yes, all of this is correct.

Calling llama_kv_cache_shift works by modifying the kv cells that belong to a given sequence however this also shifts this cell in all of the other sequences it belongs to

This call also sets a flag that upon the next llama_decode, the computation will first shift the KV cache data before proceeding as usual.
Will soon add a couple of functions to the API that can be useful for monitoring the KV cache state:
https://github.com/ggerganov/llama.cpp/pull/4170

Has the problem been solved? I pulled the latest service can support concurrency?

@abetlen Is there any progress on this?

+1, would be really great to have this

+1, would be so great to have this!

+1

+1

+1

+1

Guys, any other solution in this??

+1

+1

+1
I use llama-cpp-python as (presently) the sole first-class-support backend in an inference framework I've been developing in private.  I'm not sure how much it would benefit from batching, as I've yet to do performance testing against other backends, but I feel like it could be a significant boon.
What's the current status of this and #951?  I might be interested in taking a look at this, but I'm not certain I'd bring much to the table, I'll have to review the related code more.

I use llama-cpp-python as (presently) the sole first-class-support backend in a project I've been developing.

I would not do this. batching is super important and I had to move to llama.cpp's server (easy to deploy w/ docker or python, or even just the exe) because of lack of features on llama-cpp-python. If you're doing CPU inference, llama.cpp is a great option, otherwise I would use something like vLLM, BentoML's OpenLLM, or Predibase's LoRAx

I would not do this. batching is super important and I had to move to llama.cpp's server

This is something I was considering, appreciate the advice.  I'll likely end up doing that.  I had to do the same with Ollama, but I wasn't on Ollama long and by no means felt it was the right fit for the job, support for it merely started from a peer showing interest and my compulsion to explore all viable options where possible.
I'm doing GPU inference and sadly that means Nvidia's antics have hindered me from getting things running in a container just the way I'd like them to up until now... but that's another story.  I haven't tried vLLM, OpenLLM or LoRAx, llama.cpp and llama-cpp-python have generally been all I've needed up till now (and for longer to some degree, I home-- I really appreciate the work done by all contributors to both projects, exciting that we're at least where we are today).  Are those libraries any good if you're looking to do something with the perplexity of say q6_k on a (VRAM) budget?  I'd prefer to be able to run it on my 1080Ti, even when I have access to more VRAM in another environment.

I am dealing with this right now -- and unfortunately llama-cpp-server has the only chat-completions endpoint that I can find that supports logprobs properly (random hard requirement I have). llama.cpp doesn't natively support it in its API, which is why I use llama-cpp-python.
vllm is great for big GPUs, however, it doesn't support gguf-quantized models or running at full-precision for smaller GPUs (I use T4s).
Whereas ideally I could run llama-cpp-python on an instance with 4 T4s and batch across them (like vllm can do), I am now creating instances with 1 gpu and scaling horizontally.
If anyone knows of a better openai-compatible API endpoint that wraps llama.cpp, I am listening, but I haven't found one.

I am dealing with this right now -- and unfortunately llama-cpp-server has the only completions endpoint that I can find that supports logprobs properly (random hard requirement I have). llama.cpp doesn't natively support it in its API, which is why I use llama-cpp-python.
vllm is great for big GPUs, however, it doesn't support gguf-quantized models or running at full-precision for smaller GPUs (I use T4s).
Whereas ideally I could run llama-cpp-python on an instance with 4 T4s and batch across them (like vllm can do), I am now creating instances with 1 gpu and scaling horizontally.
If anyone knows of a better openai-compatible API endpoint that wraps llama.cpp, I am listening, but I haven't found one.

Have you tried https://github.com/ollama/ollama?

ollama doesn't support batched inference what a silly suggestion.
https://github.com/ollama/ollama/issues/1396

I case this is useful to others, as a workaround until this is implemented, I wrote a tiny python library that

downloads and installs the raw llama.cpp server binary
downloads some model weights from huggingface hub
provides a simpleServer class to control starting/stopping the binary

This was needed because the raw server binary supports batched inference (and supports structured output, another requirement I have). All the heavy logic is already in the upstream C server, so all I needed to do was do the CLI and subprocess logic.

Does this mean that continuous batching is not supported in llama-cpp-python? I assume this is the type of batching under consideration in this issue.

Does this mean that continuous batching is not supported in llama-cpp-python? I assume this is the type of batching under consideration in this issue.

Yes, but, no
Yes: continuous batching is not "utilized" in llama-cpp-python.
No: you can't even just do the simplest batching which encode multiple prompt at the same time, decode multiple sequence at the same time.

Does this mean that continuous batching is not supported in llama-cpp-python? I assume this is the type of batching under consideration in this issue.

Yes, but, no Yes: continuous batching is not "utilized" in llama-cpp-python. No: you can't even just do the simplest batching which encode multiple prompt at the same time, decode multiple sequence at the same time. Continuous batching is somehow "beyond" this

So, Is n_batch parameter is currently useless ?? I am wondering what the function of 'n_batch' is

So, Is n_batch parameter is currently useless ?? I am wondering what the function of 'n_batch' is

Yes it is.

So, Is n_batch parameter is currently useless ?? I am wondering what the function of 'n_batch' is

Yes it is. I played around alot with n_batch, n_threads etc. but it's all useless..
Further, I tried using the futures threadpool as well as the threading module to simulate parallelism. All failed with a crashing kernel. As long as a single thread is running everything is fine, proving that it's not the thread process itself. However, once a second thread is started and tries to propagate it crashes. I assume the weights are locked and cannot be accesses by two separate asynchronous processes.

Any update on this ?

Any updates? @abetlen I Think this is highly anticipated by many...

Any updates? This would be such an awesome feature to have. Looking forward to it

您好：    很高兴收到您的来信，我会尽快阅读并回复您！祝您工作顺利、生活愉快！


[Optimization] Implicit gemm rewrite

Pull Request Template
Checklist

[x] Confirmed that run-checks all script has been executed.
[x] Made sure the book is up to date with changes in this PR.

Related Issues/PRs
Requires https://github.com/tracel-ai/cubecl/pull/309 to land first
Changes
Adds a brand new implicit GEMM implementation that uses the matmul primitives in cubecl. This is slower for small k sizes, but much faster for large ones, and more flexible. I'm keeping the current implementation because it's significantly faster for certain sizes, and uses a significantly different loader strategy (loading only within each warp, which skips cross warp syncs).
Adds a number of new convolution benchmarks to test performance with different sizes and characteristics.
Testing
All non-group tests pass, and CRAFT has the expected output with all layers using the new implicit GEMM. This tests many different and relatively large layers. Adds two new regression tests for bugs discovered during implementation.

Done 👍


Mangakalot Source

Source name
Mangakalot
Source link
https://mangakakalot.com/?u=zido.gq
Source language
English
Other details
Large repo
Acknowledgements

[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

It changed to mangakakalot

This extension already exists under the name Mangakakalot under the English language: https://keiyoushi.github.io/extensions/extensions/#en.mangakakalot


Extract System.Collections

Description

HashTable, Queue and Stack.
Add new assembly System.Collection to friends list.
Add new repo to dependents to update collection.
Update AssemblyNativeVersion to <IP_ADDRESS>.
Bump version to 1.5.0-preview.
API moved to new repo nanoFramework.System.Collections.

Motivation and Context

Further work on squeezing mscorlib to allow running on very constrained devices. Removing these APIs reduces mscorlib PE by ~3kB and native image size by ~9kB.

How Has This Been Tested?

Screenshots
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Signed-off-by: José Simões<EMAIL_ADDRESS>

PUBLISH_RELEASE
UPDATE_DEPENDENTS


Copyright Section Not in Middle on Home page

Currently, the copyright section is not in the middle of the web page as expected. This misalignment affects the overall visual appeal and user experience of the website.
If we fix it and put the copyright section in the middle, the website will follow the right design rules and provide a better user experience.
Before

After

@akshathar-navada , @Divesh2003  please assign this issue to me. I would like to work on it.

Hey @BikramNarayan , we prefer the exisiting one. If there is anything u would like work on, pls go ahead!


[Feature] Add more flexibility

Hi,
it would be great to control the behavior for specific images using the attribute-list syntax in combination with some global options, or even by page using the metadata option.
For example, having the ability to:

enable the light box globally or by page and disable for specific images
disable globally or by page and enable for specific images

I had thought about this feature before, but I didn't implement this to keep this plugin simple at first. Glad to hear someone need this too.
I will add this feature later. Thanks for the advice and great ideas 😄.

Hi @BoxedBrain,
we are support disabling lightbox by a specific image or a specific page since v0.2.0. Thanks for the ideas, they save me a lot of time, but I only implemented disabling logic to keep this plugin setting as simple as possible.
If there is anything advice for this new feature just let me know 😊.


data duplication in mergeTree table

this is a sample table:

CREATE TABLE default.test ( DATE_COLUMN Date,  id Int64,  name String,  numVal Int64) ENGINE = MergeTree(DATE_COLUMN, (DATE_COLUMN, id), 8192); insert into test (DATE_COLUMN,id,name,numVal) values ('2016-12-01',11,'lee',77); insert into test (DATE_COLUMN,id,name,numVal) values ('2016-12-01',11,'shailee',999)
the 2nd row is inserted successfully and now I have duplicates in the table. Am I doing something wrong?

MergeTree doesn't deduplicate data, it just sorts the rows by the primary key expression and compacts the data parts in the background. Perhaps you might want to take look at the ReplacingMergeTree or AggregatingMergeTree ?

But why should it duplicate in the first place? Does primary key mean something else in this context?
Here, the primary key is date,id which is the same for both the entries. That shouldn't happen.

It's not a primary key like you expect in OLTP systems, it's more like a sorting key as I said. You can read about how it works here https://medium.com/@f1yegor/clickhouse-primary-keys-2cf2a45d7324

Alright! ReplacingMergeTree works. but I still have to optimise it for de-duplication.


fix(type=number): Don't parseFloat empty value strings, they NaN

#1100 added automatic number parsing of <input type="number"> fields but if the field is empty it currently applies parseFloat to an empty string, which produces NaN which could be argued either way as right or wrong, but this has the consequence of making the <input> warn in Google Chrome with:

The specified value "NaN" is not a valid number. The value must match to the following regular expression: -?(\d+|\d+.\d+|.\d+)([eE][-+]?\d+)?

This happens on every re-render, once that field has been touched. Using undefined also invokes the warning, the only thing that doesn't is the default empty string or a string that is a valid number.
Because of #1089 I can't create a JSBin to reproduce, but here's some code:
console.clear();

const { createStore, combineReducers } = Redux;
const { Provider } = ReactRedux;
const { reduxForm, Field, reducer: form } = ReduxForm;

/////////////////////////////////////////////////

const Form = reduxForm({
  form: 'example-form'
})(
  () =>
    <div>
      <Field name="count" type="number" component="input" />
    </div>
);

/////////////////////////////////////////////////

const store = createStore(
  combineReducers({ form })
);

ReactDOM.render(
  <Provider store={store}>
    <Form />
  </Provider>,
  document.getElementById('app')
);

If it's too awkward now the value might be an empty string or a parsed number, I think removing the parsing number behavior altogether is probably the right call. Then the behavior is the same as the browser. Especially because parseFloat has known quirks that may causes issues in browsers that don't actually restrict you from typing non-number characters in the input (like Chrome does). e.g. they type '1-dal@41' and parseFloat('1-dal@41') === 1

Published in v6.0.0-rc.1.

It also happens to me when I use a decimal separator that is accepted by Chrome but is not valid JS, e.g. if I try to type 12.3 with a French locale, since the separator should be ., the fields empties when I type . and I get the same alert than you.


incompatibility with vagrant 2.1.3

vagrant version
$ vagrant --version
Vagrant 2.1.3

plugin version
$ vagrant plugin list
vagrant-disksize (0.1.2, global)

Vagrantfile
$ cat Vagrantfile
Vagrant.configure(2) do |config|
  config.vm.box = 'ubuntu/xenial64'
  config.vagrant.plugins = ['vagrant-disksize']
  config.disksize.size = '50GB'
end

issue
$ vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
There are errors in the configuration of this machine. Please fix
the following errors and try again:

Vagrant:
* Unknown configuration section 'disksize'.

i used to work with this amazing plugin for a many years!
right now it does not work and there is no vagrant alternative for this...
can anyone shed the light?

I confirm the issue

Same here on Vagrant 2.1.5

Works for me on 2.1.5.
Sorry, I don't remember the steps to make it working, try to re-install it or something.

Yes, I only see this when installing a new version of Vagrant and the plugins aren't migrated with it. Re-installing the plugin should work (and did for me, on Vagrant 2.2.0) but please let me know if that doesn't work for you and I'll investigate further.

I'm trying the plugin for the first time today (in Vagrant 2.1.2). I seem to have needed to install it twice to get it working. :-/
However, once I got it working, it easily made a tiny Ubuntu box usable! It resized the disk and the partition with a single LoC.

Same issue here


High memory usage with CRS

First, just want to say that I'm really excited about this project - it's great to have an alternative to Apache or nginx with ModSecurity!
I've built a distroless image that uses Caddy, Coraza and the Core Rule Set, based on the directions from your various repos. It took a little while to get things working correctly, but it is now - you can see it here: https://github.com/abstractvector/c3-waf
However, one thing I've noticed is that the memory consumption of the container is incredibly high when the CRS rules are loaded - jumping from ~10MB without the rules to ~200MB with the rules. The addition of these two lines is all it takes:
include /coraza/coreruleset/crs-setup.conf
include /coraza/coreruleset/rules/*.conf

Once the container is loaded (e.g. using ab), the memory usage climbs significantly again. Stress testing the image without the CRS rules loaded will cause the memory usage to rise to ~20MB, but if I add the CRS rules in then memory consumption climbs to over 900MB!
This is easily reproduced using your Dockerfile (https://github.com/jptosso/coraza-waf-docker/blob/master/Dockerfile) and just adding in the relevant crs-setup.conf and rules/*.conf files as per the directions in your README.
Any idea what can be done to reduce the memory consumption? It doesn't appear to drop once the load has been removed.

Hey @abstractvector thank you very much for the information and the test implementation, the memory footprint of the rules cannot be decreased a lot, but I will run some tests.
Regarding the transactions, maybe the GC is not fast enough to clear old transactions, I will run some experiments with a transaction.Close function to flush resources.
Can you provide me your hardware specs and the parameters used for AB? I will try to replicate the issue and create a detailed profiling.

My testing was crude and not particularly extensive, but the following command demonstrates the issue quite well:
$ ab -c 100 -n 10000 http://localhost:8000

Running this against either your Docker image with CRS compiled in or my Docker image linked above shows the situation quite clearly in contrast to the same test with the two CRS config file lines commented out. I'm just running this on a MacBook Pro - Docker has access to 4 CPUs and 2GB RAM.
Having now left it a while, it seems like the memory consumption does eventually drop - it looks like a minute or two after the ab run, the CPU usage for the process jumps to around 11-12% and the memory consumption begins dropping. I'm guessing that's the GC.
My hope is to turn this into a cross-platform build so it could run on a Raspberry Pi or similar, but that memory consumption is too high unfortunately.

You are right about the memory spike but the GC turns down after a few seconds. It should be a priority to reduce the effort for the GC and proposals are more than welcome. I think I should create a function after tx.ProcessLogging(), something like tx.Close()
func (tx *Transaction) Close() error {
  if tx.collections == nil {
    return fmt.Errorf("transaction already closed")
  tx.collections = nil
}

or manually clearing it:
for k := range tx.collections {
    delete(tx.collections, k)
}

tx.collections is the heavier pointer for a transaction, forcing it's deletion should be enough.

I would love to help out, but I know very little about Go so I'm unlikely to be much assistance I'm afraid.

It's cool, thank you for your discovery, I will close this issue once a solution is found, so you are kept posted.

Hey @abstractvector, I have added a new cache feature for transactions so the memory is reused instead of freed and reallocated, your feedback on the branch would be really appreciated: https://github.com/jptosso/coraza-caddy/tree/tx_clean
Thank you


 When your docs you spent so much time putting together get out of sync with your source of truth. Again.
Wouldn't it be amazing if we could just use the same data and the same templates being put INTO the system to DOCUMENT the system itself? What's that? I already put a PR in for that on the original Pattern Lab PHP Core repo... 10 months ago? Oh snap!
In all seriousness, this is a major feature we've been seriously lacking in Pattern Lab's markdown docs approach for forever. This PR should fix that by letting you use the same templates and same data as you would with any other pattern in Pattern Lab, but in your markdown files.
Testing this out locally seems to be working as expected (global data trickling through, local data overriding global data if it exists, and pattern includes w/ namespaces) -- all seem to be working perfectly at first glance!
Before:

After:

Practical Usage Example:

CC @EvanLovely @evanmwillhite @jesconstantine @aleksip @bradfrost

Tested this out and it works for me too. Nice work! Yeah, there's a lot of cool ways this could get used....
@sghoweri Go ahead and merge and release a new version as is if you'd like! Not sure if you want to refine further or not.
This also led me to think about how we document what our forks can do that the original Pattern Lab cannot so people know about these cool features to use. I'm not sure what's the best place. I initially think about the wiki attached to each repo, but that leads to having multiple ones spread about, but it'd work cleanly. Perhaps it'd just be a single wiki page with all the fixes and improvements we have.

@sghoweri Go ahead and merge and release a new version as is if you'd like! Not sure if you want to refine further or not.

@EvanLovely I might sit on this for a day or two but definitely (or not... lemme think about it). Just want to make sure we have a little PR and docs written up before cutting a new release for this.

This also led me to think about how we document what our forks can do that the original Pattern Lab cannot so people know about these cool features to use. I'm not sure what's the best place. I initially think about the wiki attached to each repo, but that leads to having multiple ones spread about, but it'd work cleanly. Perhaps it'd just be a single wiki page with all the fixes and improvements we have

+1 for having a single wiki (or single wiki with different sections) highlighting the key differences we've put in, configuration, differences between this and PL Node project, etc.
The Pattern Lab Node project does have quite a well put together wiki I must say:

https://github.com/pattern-lab/patternlab-node/wiki/Known-Issues-&-Deviations-from-Pattern-Lab-PHP
https://github.com/pattern-lab/patternlab-node/wiki/Configuration
https://github.com/pattern-lab/patternlab-node/wiki/ChangeLog

Hey @sghoweri and @EvanLovely, just letting you know this all looks awesome. I'm ducking in here really just to say that these features are great and we're looking at how to best wrangle documentation and feature requests. As for the lack of attention on the PRs, we're starting to try to address that as well.
Keep all this great thinking up and I'm hoping we can get this great stuff closer to core soon.

Moving this PR over to the main PR repo! https://github.com/pattern-lab/patternlab-php-core/pull/126


Who uses KafkaCenter?

If you find it convenient to use KafkaCenter, you can provide the name of the company.

it's me!


Support returning structs from functions

Version 0.4.22 still gets issues with that;
https://github.com/ethereum/solidity/issues/40
FYI:
https://ethereum.stackexchange.com/questions/7317/how-can-i-return-struct-when-function-is-called
https://github.com/ethereum/solidity/issues/1601
https://github.com/ethereum/solidity/issues/2948
https://github.com/ethereum/solidity/pull/2863
Depends on #32

Is there an ETA for this , next protocol release / Solidity 0.5.0?

EncoderV2 is in 0.4.0 already, you can specify to use it with the following code
let contract = web3.contract(jsonString, at: nil, abiVersion: 2)

Although I didn't have a chance to test it well in read world yet, requires some beta Solidity compiler voodoo. So far it's backward compatible with V1 anyway

I've tried it with the 0.4.22 compiler.
It seems to not play nice with remix and web3 still? Last thing I read is that it only worked with Ether.js so far. Not sure if there have been any updates on this since 29th of jan.

Hello @kyriediculous
I've made the following contract
pragma solidity ^0.4.21;

pragma experimental ABIEncoderV2;

contract Tester {
    struct TestStruct {
        uint256 number;
        string someText;
        uint256[2] staticArray;
        uint256[] dynamicArray;
        string[2] anotherDynamicArray;
    }

    constructor() {

    }

    function testSingle() public pure returns (TestStruct t) {
        uint256[] memory dynArray = new uint256[](3);
        dynArray[0] = 0;
        dynArray[1] = 1;
        dynArray[2] = 2;
        uint256[2] memory stArray = [uint256(0), uint256(1)];
        string[2] memory anotherDynArray = ["Hello", "World"];
        t = TestStruct({
            number: 1,
            someText: "Hello world",
            staticArray: stArray,
            dynamicArray: dynArray,
            anotherDynamicArray: anotherDynArray
        });
        return t;
    }

    function testDynArray() public pure returns (TestStruct[] ts) {
        ts = new TestStruct[](2);
        ts[0] = testSingle();
        ts[1] = testSingle();
    }

    function testStaticArray() public pure returns (TestStruct[2] ts) {
        ts[0] = testSingle();
        ts[1] = testSingle();
    }
}

So far the latest version from @develop branch successfully works with "testSingle" method.
Sincerely, Alex

Full support should be in master now

Full encoding support is also in master now

It's likely that this is compatible with web3 1.0.0 right now but not with anything else; Not with remix or web3 implementations that are compatible with Metamask (since they are web3 1.0.0 incompatible) as of writing.
Of course this isn't at your end.
Remix:
[call] from:0xca35b7d915458ef540ade6068dfe2f44e8fa733c, to:Tester.testSingle(), data:f376e...6e013, return:
{
	"error": "Failed to decode output: Error: Unsupported or invalid type: tuple"
}

Yes, as far as I know remix and web3 are based on ethereumjs-abi JavaScript package and do not support decoding yet (etherjs does support V2 encoding). Nevertheless you can follow an ABI spec from solidity documents directly and implement it. I’ve published this contract and was calling a function as a part of the test, Ethereum network itself works with contracts with experimental ABI, and 0.4.22 looks quite stable with it (they had to implement usage of “returndatasize” assembler instructions to make it possible to return a structure, that was merged as a part of 0.4.22), so I expect ABIv2 to become stable and non-experimental somewhere in 0.4.23 or 0.5 that should come quite soon.
You can look at the tests for web3swift and find a full ABI and contract address on Rinkeby. Any pull requests to add more exhaustive tests and edge cases are welcome.
Sincerely, Alex

From: kyriediculous<EMAIL_ADDRESS>Sent: Sunday, April 8, 2018 9:08:45 PM
To: BANKEX/web3swift
Cc: Alexander; State change
Subject: Re: [BANKEX/web3swift] Support returning structs from functions (#33)
It's likely that this is compatible with web3 1.0.0 right now but not with anything else; Not with remix or web3 implementations that are compatible with Metamask (since they are web3 1.0.0 incompatible) as of writing.
Of course this isn't at your end.
Remix:
[call] from:0xca35b7d915458ef540ade6068dfe2f44e8fa733c, to:Tester.testSingle(), data:f376e...6e013, return:
{
"error": "Failed to decode output: Error: Unsupported or invalid type: tuple"
}
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/BANKEX/web3swift/issues/33#issuecomment-379569862, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AGKv92JmPNUkhlMPo4VtU73ehMoNl3dHks5tmlItgaJpZM4TFEyN.

If I deploy a contract using the ABI experimental function(s) described here for returning a struct from a Soldity function, will it work with Web3.js on Rinkeby and the main net?

Hello @roschler
First of all, ABIv2 encoder/decoder in web3swift is backward compatible with a normal one. It will understand all current encoding types, but will also decode experimental features, such as returning structs. You can already use these features if you want by putting this #pragma is your code, and it will be included in 0.5 version of solidity. Interestingly, solidity docs describing ABI encoding imply features of ABIv2, looks like those features just were not implemented yet in main solidity branch.
So, if you do not need this features - just remove a pragma and everything will work. If you do need them, you will have to supply a pragma and use V2 decoder from web3swift.
Sincerely, Alex


Multiplatform support(iOS + Android) for push notification

Can we use FCM for push notification in VoIP voice call? if I use apn credential its working fine but if I try to use FCM for iOS I am not getting push notification while making call.
If I have to use apn only(as mention in step https://github.com/twilio/voice-quickstart-swift#8-create-a-push-credential-with-your-voip-service-certificate) what should I do if I want to work on both iOS and android.
which value should we use as PUSH_CREDENTIAL_SID on server?

Hi @SunilSpaceo
The Programmable Voice iOS SDK requires a VoIP Service certificate in order to enable the incoming call notifications. It is different from the regular APNS certificate so you will need to follow the quickstart steps to generate one and create Push Credential with the credentials.
Currently Voice iOS SDK does not support notifications using FCM.
thanks
-bobie

@SunilSpaceo
To be more accurate, FCM does support APN service, but not for Apple VoIP Service at the moment.

Thanks for your reply. I understand, we have to use VoIP Service certificate for iOS but which value should we use as PUSH_CREDENTIAL_SID on server?
if I set that value as FCM push credential, it will work for Android but not for iOS
if I set that value as APN push credential, it will work for iOS but not for Android
so which value to set if I want to support for both iOS and Android.
Is there a way we can set PUSH_CREDENTIAL_SID value dynamically so it works for both platform?

Hi @SunilSpaceo
One way is to embed additional platform info in the http request URL parameter list. For example
https://my.server.ai/accessToken?platform=ios_sandbox&duration=86400

and check the value of the platform parameter on your access token server endpoint, then determine which push credential sid to put in the access token.


Minor error message fix

Codecov Report

Merging #580 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master     #580   +/-   ##
=======================================
  Coverage   59.66%   59.66%
=======================================
  Files          32       32
  Lines        2209     2209
=======================================
  Hits         1318     1318
  Misses        616      616
  Partials      275      275

Impacted Files
Coverage Δ

core/keyserver/keyserver.go
44.17% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Mediapipe Iris without facemesh?

I want to use Mediapipe Iris tracking, but I have a different method to crop out the area around the eye. In the documentation of Mediapipe Iris, it says that the position around the eye is cropped out using facemesh.
Since my use case does not involve a full picture of the face, I can't run the face mesh model first.
is there any way to run Mediapipe Iris without facemesh?

Hi @balavenkataraman123,
Unfortunately, we have completely ended support for the legacy solutions, including iris. You can find this information in our documentation. We are sorry, but you will need to find your own solution for your use case. Our primary focus is on improving our Task APIs, and we won't be able to assist further with this issue.
Thank you!!


println! with iex -S mix produces poorly formatted output

It looks like this basically.

Not much we can do about this I believe.
An alternative would be to introduce a new printing function in rustler that uses the BEAM provided printing functions.
Relevant: https://github.com/rusterlium/rustler/issues/72

There is this, which could enable us to change where the default rust print functions send their outputs. This seems to be what the rust test framework uses.
The issue is I don't think there is any way for us to statically execute code on every scheduler thread, so I don't think we can actually use this.
https://github.com/rust-lang/rust/blob/db15596c5747eed0344f2622a9d85ad9534b23f9/library/std/src/io/stdio.rs#L934-L963

FWIW, I've had pretty good luck using env_logger: https://github.com/scrogson/franz/blob/master/native/franz/src/lib.rs#L12

The set_print is also unstable. Maybe we should just document that using print! etc. is a bad idea and people should use the log crate instead.

I am using the env_log crate and am still having this behavior. Is this expected? (ie does this issue go deeper into the rust/erlang logging interaction). If its not expected, is there a code example that might be able to get logging output without the extra spaces?


Add Docs for Union Types via the [@unboxed] feature

Create a standalone page "Union Types", which basically translates the previous BS blog posts into a more formal documentation page:

Union Types in BS
BS 7.2 Holiday release

This includes adding the [@unboxed] decorator to the decorators overview with a link to the actual doc (an example would be too big to just inline it).
\cc @fhammerschmidt

Since I have been investing quite some time in reiterating Tagged Unions w/ Variants, and since there was a lot of  feedback coming from Cristiano's side regarding the unboxed feature (and its blog post), we probably want to split that topic up into more practical examples, independent from the "interop" sidebar category and really think about useful use-cases


Update gen-crd-api-reference-docs dependency

Once https://github.com/ahmetb/gen-crd-api-reference-docs/pull/29 merges, we need to switch back to using that version instead of @jpeach's fork. We originally diverged due to https://github.com/kubernetes-sigs/service-apis/pull/503.

This was fixed in https://github.com/kubernetes-sigs/service-apis/pull/513.
/close


Add Libcloud topic

Thank you for contributing! Please confirm this pull request meets the following requirements:

[x] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[x] Curating a new topic or collection

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[x] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

A bunch of other Apache Software Foundation projects (Cassandra, etc.) already have a topic page, so it would also be good for Apache Libcloud to have one.

Hi,
Thanks for your contribution. However, curated topics are meant to be used for heavily used topics and this topic is only used on 14 public repositories.

I re-read our contribution guidelines and think this falls into the second criteria from this sentence, so I am reopening this.

Valuable topics, for example, include those that are already widely used by repositories, or a topic that currently lacks important information.

@kenyonj Thanks - I removed unused alias and addressed the build failures.


add notification menu to header

*Issue:*https://github.com/bcgov/entity/issues/24636
Description of changes:

Added Notification bar to header, on xl viewpoint and above it shows the word notifications, smaller and it shows just the bell (this matches the existing header)
Added api call to get the number of notifications
Used translations to change what the text is in the menu based on the number of notifications (again matching current functionality)
If a notification exists it links you to the team page as it does currently
Otherwise it says no notifications
Added ability for menu to have a sublabel to mimic the existing menu

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-111-htmvkuct.web.app

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-111-htmvkuct.web.app

Broken unit test?


Wildcard certificate requirement

You should consider using a wildcard certificate to match the application you plan to create.

This statement needs to be reviewed. When creating an external wildcard URL, a wildcard certificate must be created that matches the URL. If you do not use a wildcard certificate, it fails to apply the certificate with an error of "the host names on the certificate do not match any verified domains."

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

ID: d0e1d914-1430-b322-131d-8fabb9177dbd
Version Independent ID: 581aad50-f3c2-35a1-0142-45eea0d10c05
Content: Wildcard applications in Azure Active Directory Application Proxy
Content Source: articles/active-directory/app-proxy/application-proxy-wildcard.md
Service: active-directory
Sub-service: app-proxy
GitHub Login: @kenwith
Microsoft Alias: kenwith

@jezza280 Thank you for your feedback . We will investigate and update the thread.


macOS version on MacOS 10.12.6 is non-functional

downloaded the most recent app for macOS.
dropped it in the Applications directory.
double clicked on the app.
App opens to a empty box with no tabs or distinguishable items.
File -> Open selects a file but nothing appears.
File -> open project folder shows the path in the top of the window above the menubar, but does not display anything (like the navigation tree normally on the left).
version 1.2.4 was operational.
update...after several attempts, the app does open and i can add the project folder tree.
that said, i cannot open a text file.
only after exiting and opennig the app a couple of times will it allow me to view text files.

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently - at the moment we don't have enough information to further triage this issue.
Thanks for understanding and meeting us half way :grinning:


Server waits forever on receiving next(c.p)

https://github.com/mattn/go-xmpp/blob/8a80c8abe319ccc0e61b63684158bffd1a61ca5a/xmpp.go#L315
I placed a few printfs and the server seems to wait on this line forever. No errors thrown either. When I turn debug ON, I was able to see the  tag.
boopathi example(master)> go run example.go -server localhost:5222 -username test@boopathi -password testtest -notls -debug
<?xml version='1.0' encoding='UTF-8'?><stream:stream xmlns:stream="http://etherx.jabber.org/streams" xmlns="jabber:client" from="boopathi" id="fc88ef3d" xml:lang="en" version="1.0">
SE =  {{http://etherx.jabber.org/streams stream} [{{xmlns stream} http://etherx.jabber.org/streams} {{ xmlns} jabber:client} {{ from} boopathi} {{ id} fc88ef3d} {{http://www.w3.org/XML/1998/namespace lang} en} {{ version} 1.0}]}
<stream:features><starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls"></starttls><mechanisms xmlns="urn:ietf:params:xml:ns:xmpp-sasl"><mechanism>DIGEST-MD5</mechanism><mechanism>PLAIN</mechanism><mechanism>ANONYMOUS</mechanism><mechanism>CRAM-MD5</mechanism></mechanisms><compression xmlns="http://jabber.org/features/compress"><method>zlib</method></compression><auth xmlns="http://jabber.org/features/iq-auth"/><register xmlns="http://jabber.org/features/iq-register"/></stream:features>
Mechanism =  DIGEST-MD5
<challenge xmlns="urn:ietf:params:xml:ns:xmpp-sasl">cmVhbG09ImJvb3BhdGhpIixub25jZT0iOXM3dTVMYjByd240VFB6TzBJbUJOMjhXN1dyaHAwT1NzamVtbUFhMyIscW9wPSJhdXRoIixjaGFyc2V0PXV0Zi04LGFsZ29yaXRobT1tZDUtc2Vzcw==</challenge>
<success xmlns="urn:ietf:params:xml:ns:xmpp-sasl">cnNwYXV0aD01MTI1YmIwY2JiNzMxMWMyODlkZmIwMTY3Yzc4Nzc2Yg==</success>
Waiting for next message
</stream:stream>

I'm having the same issue, did you ever figure it out @boopathi ?

cc @mattn

What the server do you use?

I'm trying to use it with OpenFire

This is my output
<?xml version='1.0' encoding='UTF-8'?><stream:stream xmlns:stream="http://etherx.jabber.org/streams" xmlns="jabber:client" from="<IP_ADDRESS>" id="REDACTED" xml:lang="en" version="1.0">
<stream:features><starttls xmlns="urn:ietf:params:xml:ns:xmpp-tls"></starttls><mechanisms xmlns="urn:ietf:params:xml:ns:xmpp-sasl"><mechanism>DIGEST-MD5</mechanism><mechanism>PLAIN</mechanism><mechanism>CRAM-MD5</mechanism></mechanisms><compression xmlns="http://jabber.org/features/compress"><method>zlib</method></compression><auth xmlns="http://jabber.org/features/iq-auth"/></stream:features>
<proceed xmlns="urn:ietf:params:xml:ns:xmpp-tls"/>
<?xml version='1.0' encoding='UTF-8'?><stream:stream xmlns:stream="http://etherx.jabber.org/streams" xmlns="jabber:client" from="<IP_ADDRESS>" id="REDACTED" xml:lang="en" version="1.0"><stream:features><mechanisms xmlns="urn:ietf:params:xml:ns:xmpp-sasl"><mechanism>DIGEST-MD5</mechanism><mechanism>PLAIN</mechanism><mechanism>CRAM-MD5</mechanism></mechanisms><compression xmlns="http://jabber.org/features/compress"><method>zlib</method></compression><auth xmlns="http://jabber.org/features/iq-auth"/></stream:features>
<challenge xmlns="urn:ietf:params:xml:ns:xmpp-sasl">REDACTED</challenge>
<success xmlns="urn:ietf:params:xml:ns:xmpp-sasl">REDACTED</success>
</stream:stream>

Then everything stops and it never gets to the go function to read the talk.Recv

Can confirm this, I made some changes in my fork[1], but I am not ready to PR yet.
[1] https://github.com/dullgiulio/go-xmpp/commit/07352ce6dc735e200c220a747ab5af746aebc0ce

Is anyone working on this bug?
It looks to me like the io.EOF error from func (*xml.Decoder) Token (indicating end of XML stream) is ignored. It should probably trigger connection re-initiation, or at least cause c.Recv to return io.EOF so it can be handled outside the package.
https://github.com/mattn/go-xmpp/blob/8a80c8abe319ccc0e61b63684158bffd1a61ca5a/xmpp.go#L540

Sorry, we stopped using OpenFire and I can't reproduce this any longer.
But wouldn't that line end up in panic(unreacheable) is an EOF is received there?

https://play.golang.org/p/9FiXqI8F0b
I'm afraid not. Since the type switch below don't catch it (the token t is nil), it will simply continue looping forever.
@mattn - Any interest in accepting a pull request that would expose this error to the c.Recv() caller? I can't understand how ignoring that io.EOF could lead to anything but an unrecoverable failure, but I may of course be missing something here.
@dullgiulio - Out of curiosity, do you know if OpenFire implements XMPP Ping? After enabling this on our ejabberd instance, it looks like we're getting io.UnexpectedEOF instead of a non-responsive Recv() call when the connection is broken.

Your patch seems good for me.

I'm sorry. I was looking at (and linking to) an old blob that.
Commit 9dd92e124738bea45d60514e3aa597d7f999d907 changed the behaviour of this function, so my playground example is no longer valid.
Although the patched version behaves as I suggested (https://play.golang.org/p/FAP7Wa8hll), I still suggest that we remove the err != io.EOF statement. It appears misleading, and may still cause issues (depending on the xml package's behaviour).
I'm sending you a PR in a couple of minutes.


SITL, Model: plane, [Err] [gazebo_mavlink_interface.cpp:1166] Error opening serial device: open: No such file or directory

Describe the bug
Sitl simulation of the plane model doesn't work properly, since the gazebo_mavlink_interface doesn't seem to be able to open the serial device.
http://discuss.px4.io/t/sitl-gazebo-plane-error-with-gazebo-mavlink-interface/9852
To Reproduce
Steps to reproduce the behavior:
execute:
~/src/Firmware$ make -j4 px4_sitl_default gazebo_plane
on command line.
Expected behavior
usual Sitl simulation: Opening of Gazebo with world/model + conection to PX4 over QGC
Drone (please complete the following information):

(Not relevant for issue but need SITL to alter model for) Project Dipper (focus project at ETH)
https://dipper.ethz.ch/

Screenshots

I'm not sure if this is your problem, but that error is usually caused when the PATHs aren't setup correctly. I'd need to test the make targets.

After installing the latest version of Ubuntu (18.04 LTS) the Simulation worked properly. It then was an issue related to my system, so the original code must be working properly. Sry for that and thanks for your reply!!


TEST-#6009: use tmp_path fixture instead of ensure_clean_dir as pandas 2.0.0 does

What do these changes do?

This is necessary step to migrate to pandas 2.0.0.

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #6009
[x] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

I wonder whether Pytest is cleaning up its tmp_path after run... I can find nothing about that in the Pytest docs... 😞

By default, after every 3rd time: https://docs.pytest.org/en/6.2.x/reference.html?highlight=tmp_path#std-fixture-tmp_path

@vnlitvinov answered all your comments

LGTM
@vnlitvinov any more comments?

@dchigarev let's merge it then


Add minWidth and minHeight constraints to the window

The problem
As shown above, the UI can get "clamped together" when the window is resized to a small size. The sidebar buttons "Settings" and "Help", which is to be at the bottom of the bar, clamp with the other buttons, making them unusable. The text "The list is empty..." is cut off the screen too.
The solution
Use the "minWidth" and "minHeight" options to make sure the window is large enough. More information can be found at https://github.com/electron/electron/blob/master/docs/api/browser-window.md.
Alternatives
For the sidebar, a responsive layout can be used to tackle this problem, but it would be much more work.
The text, however, can simply trimmed, with "..." at the end, with CSS.
Notes
The five buttons at the bottom-left of the screenshot is not distributed: it was coincidental.

This issue seems to be created because of network problems. Please delete this issue. Sorry to bother. @ransome1


Yi Dome Guard Bootloop after SD card change

So guys I need your help,
my sdcard in the cam was broken and I thought that I could just simply download the new version here, but it on a new SD Card and thats it.
But my camera keeps restarting the whole time. No wifi-connection is created on the router.
What can I do?
The camera works normally, if I insert a card without any hack via the app.
Thanks for your answers and time :)

It can happens when the card is slow.
There is a race condition in the boot script but I was not able to find it.

It can happens when the card is slow. There is a race condition in the boot script but I was not able to find it.

interesting... The card is a 16GB Class10 A1 HC1 is that too slow?
What is recommended?

Ah I seemed I found the issue.
My download was not 100% complete and therefore the issue with the continuus reboot

interesting... The card is a 16GB Class10 A1 HC1 is that too slow?

Maybe too fast... :)

My download was not 100% complete and therefore the issue with the continuus reboot

Good news.


Building issues

Hi, guys! I was not able to build container as README mentiones. Got the following error:
root@selfportal:/home/ubuntu/kube-monkey# make container
Running docker with ''
Sending build context to Docker daemon    112MB
Step 1/3 : FROM ubuntu
 ---><PHONE_NUMBER>37
Step 2/3 : RUN if (dpkg -l | grep -cq tzdata); then         echo "tzdata package already installed! Skipping tzdata installation";     else         echo "Installing tzdata to avoid go panic caused by missing timezone data";         apt-get update && apt-get install -y --no-install-recommends 		apt-utils 		tzdata 	&& rm -rf /var/lib/apt/lists/*;     fi
 ---> Using cache
 ---> a7a57f2b0a76
Step 3/3 : COPY kube-monkey /kube-monkey
COPY failed: stat /var/snap/docker/common/var-lib-docker/tmp/docker-builder735501569/kube-monkey: no such file or directory
Makefile:33: recipe for target 'container' failed
make: *** [container] Error 1

The side question: can I rebuild kube-monkey somehow with a more modern version of client-go, so it would be compatible with Kubernetes 1.15?

https://github.com/asobti/kube-monkey/issues/150
Duplicate of https://github.com/asobti/kube-monkey/issues/150.
Team, please, change the README so it would be more clear about installation procedure. Thanks!

Can't actually run make. Error log:
=== RUN   TestCreateClient
E1118 21:09:59.896858     341 kubernetes.go:38] failed to obtain config from InClusterConfig: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined
--- FAIL: TestCreateClient (0.12s)
    kubernetes_test.go:35:
        	Error Trace:	kubernetes_test.go:35
        	Error:      	Expected nil, but got: &errors.errorString{s:"Unable to verify client connectivity to Kubernetes apiserver"}
        	Test:       	TestCreateClient
=== RUN   TestNewInClusterClient
E1118 21:10:00.021478     341 kubernetes.go:38] failed to obtain config from InClusterConfig: unable to load in-cluster configuration, KUBERNETES_SERVICE_HOST and KUBERNETES_SERVICE_PORT must be defined
--- PASS: TestNewInClusterClient (0.00s)
=== RUN   TestVerifyClient
--- PASS: TestVerifyClient (0.00s)
FAIL
coverage: 75.0% of statements
FAIL	github.com/asobti/kube-monkey/kubernetes	0.145s

Hi @ArseniiPetrovich. Apologies for the delay.
I updated the makefile and the dockerfile a few commits ago. I'm going to close this issue. Feel free to reopen if you still have issues building.


Add support of Kanji Mode

Main change

use zero, unit instead of Poly([0]), Poly([1])
move original tests of QRCode.jl into test/tst_overall.jl

For a highly-optimized algorithm where bandwidth is the bottleneck, UInt8 vs Int can make a lot of difference in performance, e.g., https://github.com/JuliaImages/ImageMorphology.jl/pull/102#issuecomment-1165297247. But for QRCoders, it might be too early for this to make a difference.

I did some tests about type allocations:

It has slight effect when changing func(::Int) with func(::Integer)
However, the efficiency decreases observably when type conversion is involvedstruct foo
    data::Vector{UInt8}
end
data = rand(0:255, 10^4)
uint8data = UInt8.(data)
@btime sum(foo(data).data); # 4.105 μs
@btime sum(foo(uint8data).data); # 836.509 ns

In order to increase the code efficiency, we need to avoid type conversion as possible. (we might consider that in the later stage)

Looks like there's a misunderstanding here about the type annotation on performance. I'll write up a few things and share it with you in private later this week.

I'm a little confused about the purpose of key word and predefined values.
function qrcode( message::AbstractString
       , eclevel::ErrCorrLevel = Medium()
       ; version::Int = 0
       , compact::Bool = false )

Would it be better to write
function qrcode( message::AbstractString
       ; eclevel::ErrCorrLevel = Medium()
       , version::Int = 0
       , compact::Bool = false)

keyword parameter doesn't participate in multiple dispatch
in old Julia (<v1.0 I think), keyword parameter has large overhead. In Julia 1.8, it seems that the overhead of constant keyword parameter is optimized to "zero".

Choosing between keyword parameter and positional argument is a tradeoff between convenience and explicitly -- this is more of a design taste.
Because keyword parameter doesn't participate in multiple dispatch, it has to be type stable regardless of different value choices. For instance,
julia> struct M1 end

julia> struct M2 end

julia> f(; m) = m()
f (generic function with 1 method)

julia> g(m) = m()
g (generic function with 1 method)

julia> @code_warntype f(m=M1) # unstable
MethodInstance for (::var"#f##kw")(::NamedTuple{(:m,), Tuple{DataType}}, ::typeof(f))
  from (::var"#f##kw")(::Any, ::typeof(f)) in Main at REPL[3]:1
Arguments
  _::Core.Const(var"#f##kw"())
  @_2::NamedTuple{(:m,), Tuple{DataType}}
  @_3::Core.Const(f)
Locals
  m::DataType
  @_5::DataType
Body::Any
1 ─ %1  = Base.haskey(@_2, :m)::Core.Const(true)
│         Core.typeassert(%1, Core.Bool)
│         (@_5 = Base.getindex(@_2, :m))
└──       goto #3
2 ─       Core.Const(:(Core.UndefKeywordError(:m)))
└──       Core.Const(:(@_5 = Core.throw(%5)))
3 ┄ %7  = @_5::DataType
│         (m = %7)
│   %9  = (:m,)::Core.Const((:m,))
│   %10 = Core.apply_type(Core.NamedTuple, %9)::Core.Const(NamedTuple{(:m,)})
│   %11 = Base.structdiff(@_2, %10)::Core.Const(NamedTuple())
│   %12 = Base.pairs(%11)::Core.Const(Base.Pairs{Symbol, Union{}, Tuple{}, NamedTuple{(), Tuple{}}}())
│   %13 = Base.isempty(%12)::Core.Const(true)
│         Core.typeassert(%13, Core.Bool)
└──       goto #5
4 ─       Core.Const(:(Base.kwerr(@_2, @_3)))
5 ┄ %17 = Main.:(var"#f#3")(m, @_3)::Any
└──       return %17

julia> @code_warntype g(M1) # stable
MethodInstance for g(::Type{M1})
  from g(m) in Main at REPL[4]:1
Arguments
  #self#::Core.Const(g)
  m::Core.Const(M1)
Body::M1
1 ─ %1 = (m)()::Core.Const(M1())
└──      return %1

For our case, if we can ensure that output is always Matrix{Bool}, we're good to use keyword parameter.

Abstract

Add support of Kanji mode
Separate UTF8 mode from Byte mode (the length method for the two are different)
Minor changes on some of the functions

I summarize the followings that might need to be reviewed before merging this PR. Looking forward to your advice if convenient.

matrix.jl -- line149
Here I have some questions about the efficiency of makemask.
Origin implement of makemasks.
"""
    makemasks(matrix::Array{Union{Bool,Nothing},2})

Create 8 bitmasks for a given matrix.
"""
function makemasks(matrix::Array{Union{Bool,Nothing},2})::Array{BitArray{2},1}
    n = size(matrix, 1)
    masks = [falses(size(matrix)) for _ in 1:8]

    # Weird indexing due to 0-based indexing in documentation
    for row in 0:n-1, col in 0:n-1
        isnothing(matrix[row+1, col+1]) || continue
        if (row ⊻ col) & 1 == 0
            masks[1][row+1, col+1] = true
        end
        if row & 1 == 0
            masks[2][row+1, col+1] = true
        end
        if col % 3 == 0
            masks[3][row+1, col+1] = true
        end
        if (row + col) % 3 == 0
            masks[4][row+1, col+1] = true
        end
        if (row >> 1 + col ÷ 3) & 1 == 0
            masks[5][row+1, col+1] = true
        end
        if (row & col & 1) + ((row * col) % 3) == 0
            masks[6][row+1, col+1] = true
        end
        if ((row & col & 1) + ((row * col) % 3)) & 1 == 0
            masks[7][row+1, col+1] = true
        end
        if (((row ⊻ col) & 1) + ((row * col) % 3)) & 1 == 0
            masks[8][row+1, col+1] = true
        end
    end
    return masks
end

I add a function makemask since only one mask is needed in QRDecoders.jl for each decoding.
_maskrules = Function[
    (x, y) -> (x ⊻ y) & 1,
    (x, _) -> x & 1,
    (_, y) -> y % 3,
    (x, y) -> (x + y) % 3,
    (x, y) -> (x >> 1 + y ÷ 3) & 1,
    (x, y) -> (x & y & 1) + (x * y % 3),
    (x, y) -> (x & y & 1 + x * y % 3) & 1,
    (x, y) -> ((x ⊻ y & 1) + (x * y % 3)) & 1
]
makemask(matrix::AbstractArray, k::Int)::BitArray{2} = makemask(matrix, _maskrules[k])
function makemask(matrix::AbstractArray, rule::Function)::BitArray{2}
    n = size(matrix, 1)
    mask = falses(size(matrix))
    for row in 1:n, col in 1:n
        mask[row, col] = isnothing(matrix[row, col]) && iszero(rule(row - 1, col - 1))
    end
    return mask
end

I would rewrite the makemasks as following.
makemasks(matrix::AbstractMatrix) = makemask.(Ref(matrix), 1:8)

But it would be much slower.
using BenchmarkTools
using QRCoders: empytmatrix
matrix = empytmatrix(20)

@btime makemasks(matrix); # 85.790 μs (17 allocations: 10.23 KiB)
@btime makemask.(Ref(matrix), 1:8); # 215.263 μs (20 allocations: 10.31 KiB)

The decrease in efficiency is partly due to the assignment operation. It would be better if writing
function makemask(matrix::AbstractArray, rule::Function)::BitArray{2}
    n = size(matrix, 1)
    mask = falses(size(matrix))
    @inbounds for row in 1:n, col in 1:n
        if isnothing(matrix[row, col]) && iszero(rule(row - 1, col - 1))
            mask[row, col] = true
        end
    end
    return mask
end

@btime makemask.(Ref(matrix), 1:8); # 131.314 μs (20 allocations: 10.31 KiB)

But still, it's a bit slower than the direct method makemasks.
Here is my question:

I try to avoid if method to trigger higher performance of the for-loop but failed(and cost more)
The new method is still a bit slower makemasks, is that caused by the function call.

table.jl -- line 11
I use the read function to initialize kanji.
"""
Shift-JIS characters download from: https://www.romhacking.net/documents/179/.
"""
_kanji = split.(split(strip(read("src/Kanji.tbl", String)), '\n'), '=')

"""
Allowed characters for `Kanji()` mode.
"""
const kanji = Dict{AbstractChar, Int}(
  first(j) => parse(Int, i; base=16) for (i, j) in _kanji)

This only works in QRCoders.jl. When I try using QRCoders in QRDecoders.jl, it would throw file path error. So I use include method instead.
My questions are, is it reasonable to initialize variables with text file especially when the file is large, and how to avoid path error when dealing with files.


Migrate from nu7hatch/gouuid to satori/go.uuid

nu7hatch/gouuid does not complain the RFC4122 spec (see issue]
Also this fork suggest that implementation could create duplicated keys.
I guess changing how UUIDs are generated could break current installs (if UUID is used to decide if the node exists or not)

Addressed by https://github.com/skydive-project/skydive/pull/2446


fix: change scope name

Problem description

Current scope name does not follow API Design Guidelines
Expected behavior

Change scope name to api-name:action syntax
Additional context

Per comment in review of  #67

Fix provided in fedf4965cafd13500e611c187b6048161548a582


Can't use mouse (via terminal) to highlight and copy text from the notification popup

Sometimes I want to copy the text that is in the popup.
I usually use the mouse for this, and will try to highlight it and then yank and/or copy.
I've tried this using a regular mouse select in Vim, select text, then y to yank. Seems to work (though is a bit awkward as the content in the popup moves around a bit).
But my preferred method when using the mouse, is to hold down shift whilst selecting text. And this seems to use kitty (my terminal) to directly select the text (not nvim), and when I let go of the mouse button, it's usually already copied.  (I just find it faster that way, and works outside of Vim too).
But for some reason, this 2nd method does not work with nvim-notify?
The thing is, I can use shift + mouse select to copy any other text in the Neovim UI without issue (including popups like Telescope produces).  It's just the text in the nvim-notify popups that can't be selected & copied using this method, and I don't get why.
Is there some setting that prevents text in the nvim-notify popups from being selectable by kitty perhaps?

I'm not sure why kitty can't select it, it might be something to do with the window being rerendered but I'm not at all confident on that. You can use the kitty show_scrollback mapping to open the pager (which I have set to nvim but I believe defaults to less) and then mouse select text with that


Add integer check to Question._should_be_shown()

Up till now, we've exclusively used the depends property for filtering by lot, so we're always checking an array of strings for one that matches the filter.
However, some questions are dependent on others having specific properties. Currently, this only applies to socialValueCriteriaOutcomes, which requires that socialWeighting be zero, or between 10 and 20.
eg:
depends:
  - "on": "lot"
    being:
      - user-research-participants
  - "on": "socialWeighting"
    being:
      - min_value: 0
        max_value: 0
        type: integer
      - min_value: 10
        max_value: 20
        type: integer

This adds the ability to check one or several number ranges. It could also be rejigged in the future to allow for checking multiple properties on any question type, but for now only applies to numbers.
socialWeighting and socialValueCriteriaOutcomes only apply to DOS6, but this change doesn't impact existing implementation beyond a couple more checks.

Hmm, this clearly needs more work. Rushed it. Going to convert to draft.


Open file - Read Only

Can I open the file in read only mode?
The file is opened by another process(Office), but I only need to read some data from it using program.
open C:\Users\andrew\OneDrive\REPORT.xlsx: The process cannot access the file because it is being used by another process.

Thanks.

Hi @AngangGuo, please follow by issue template provide code and steps to reproduce the issue.

the minimum program is as the following:
func main() {
	f, err := excelize.OpenFile(`C:\Users\andrew\OneDrive\Work\Label.xlsx`)
	if err != nil {
		fmt.Println(err)
		return
	}
	s:=f.GetSheetMap()
	fmt.Println(s)
}

The Excel file is on my company OneDrive folder. It can be opened and processed using this program without any problem.
But If I open the file using Microsoft Excel first. Then this program will fail and the message is:
open C:\Users\andrew\OneDrive\Work\Label.xlsx: The process cannot access the file because it is being used by another process.

I have OneDrive installed on my own computer. I can open the file (C:\Users\Andrew\OneDrive\Private\register.xlsx)using the above program without any problem.
But if I open that file first from Excel for Office 365 and then run the program, the following error message shows out:
open C:\Users\Andrew\OneDrive\Private\register.xlsx: The process cannot access the file because it is being used by another process.

Is it possible to open the file as readonly or get a snap shot of the file?

My solution:  in the end, I made a copy of the file and read the content from the copy.

For now I found a better solution to work with the file:

Let Excelize program open my local OneDrive file
If I need to work on the file, I'll open the network version from Browser.


Add Sticker From url

is there’s any way to add sticker from url?

For now.  Only local images (png, jpeg, jpg) can be loaded.

2020年2月6日 下午11:00，Ali Al-jiachi<EMAIL_ADDRESS>写道：
is there’s any way to add sticker from url?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/lincf0912/LFMediaEditingController/issues/42?email_source=notifications&email_token=ACNOL4VRXWDIHUBUF35YME3RBQQ2HA5CNFSM4KQ6MULKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ILRHPRQ, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACNOL4SWS6PC7WMDNLRLZSLRBQQ2HANCNFSM4KQ6MULA.

thanx man for this amazing library hope to add this feature soon.

Yes. It will. In fact, it's starting.

Now you can upgrade it. It can already support sticker from URLs.
version 1.4.2


[Bug] The ps-cs service log of linkis reports this error

Search before asking

[X] I searched the issues and found no similar issues.

Linkis Component
linkis-commons
What happened + What you expected to happen
The ps-cs service log of linkis reports this error:

figServletWebServerApplicationContext (559) [refresh] - Exception encountered during context initialization - cancelling refresh attempt: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'eurekaInstanceLabelClient': Invocation of initkaba method failed; nested exception is java.lang.RuntimeException: com.netflix.client.ClientException: Load balancer does not have available server for client: linkis-ps-publicservice

Relevent platform

Reproduction script

Anything else
这个问题来自Linkis社区的QA文档
This question comes from the QA documentation of the Linkis community
QA Doc Link：https://docs.qq.com/doc/DSGZhdnpMV3lTUUxq
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

这个是因为publicservice服务没有启动成功导致，建议手动重启下publicservice sh/sbin/linkis-dameo.sh restart ps-publicservice
The publicservice service did not start successfully. It is recommended to manually restart the publicservice sh/sbin/linkis-dameo.sh restart ps-publicservice


Progress Bar is not showing while checking updates

Describe the bug
When you launch Legacy Update while during checking updates, the progress bar doesn't show.

Steps to reproduce the behavior:

Click Start on the taskbar.
Point to All Programs, and Click Legacy Update at the top. (If you are using classic start menu, just click it on the top one).
While during check updates, See the Progress Bar that isn't shown.

Expected behavior
The Progress bar should appear while during checking update.
Environment and others (please complete the following information):

Legacy Update version: 1.6.2
Host OS: Windows 7 Ultimate SP1 32-bit
Guest OS (in Virtual PC 2007): Windows Fundamentals for Legacy PCs SP2.

Additional context
The Guest OS runs on Windows XP, although it is Embedded version under name Windows FLP. I haven't tested for latest version for some reason. And ofc, It is Internet Explorer 6, I didn't even update IE7/IE8 yet.
Have you checked that no similar bug report(s) exist?

[x] I have searched and didn't find any similar bug report.

This was somewhat intentional, I added an ActiveX progress bar in 1.8, and I switched the site over to it a few days ago because I figured enough time has passed that most users should have updated. That said, I probably can make it fall back to the progress bar gif if the control fails to load.


cilium statedb dump command & bugtool

As requested by :manager::
Adds the API handler /statedb/dump and implements "cilium statedb dump"
on top to dump the contents of StateDB as JSON.
Extends sysdump to call "cilium statedb dump" to include the output in sysdump.

/test

@aanm is on PTO for a bit, api hence covered mostly by Nate - changing the review request to  him.

intermixing writeByte, writeString. If memory serves the later is a bit slower and given we're looping, figured I'll flag.

I'm not sure I follow your comment with writeByte vs writeString - writing the runes of the string individually seems slower, and casting to []byte{} would potentially break runes apart? What's your suggestion instead?
In any case, we're writing to a buffered writer, afaik the writes should amortize to be comparably fast to a raw memcpy; the volume of data is not expected to be huge at this time, since there's just one statedb table.

Given our sysdump perf issues on some clusters might be worth seeing if there is a diet plan and put this on the bench?

I agree that performance hasn't been the foremost concern of sysdumps. I'm reading your comment as a suggestion to put the collection of the statedb information behind a flag? I'm not opposed in general, but so far we've just slurped as much information as possible into a sysdump -- changing that will imo confuse debugging efforts of people unaware, due to missing information. If we want to do this, I'd advocate for doing it in some coordinated fashion over all of the sysdump information. I'd rather not have to go back and forth with requesting sysdumps with different configurations to get to the information needed to debug an issue, and usually I don't know which component is to blame before looking at a sysdump...

@bimmlerd Thank you! I was not advocating introducing a flag, just a general comment on sysdump efficiency and benchmarking to optimize outputs.


Fix small typo in rosdep key and update include header

Hello 👋
Thank you for your repo, very useful!
While building the package on Jazzy, I noticed 2 issues:

Couldn't find the header cv_bridge/cv_bridge.h but the build worked with cv_bridge/cv_bridge.hpp.
Typo in the rosdep key tf2_geometry_msgs in package.xml.

Feel free to merge this PR at your convenience!
Best regards.

Currently, I don't plan to switch to Jazzy since there are still users for Foxy and Humble. Instead, you can just create a symbolic link named cv_bridge.h pointing to cv_bridge.hpp in the /opt/ros/jazzy/include/cv_bridge/cv_bridge/ folder, which will resolve this issue.


Smarter default queries?

Would be nice if the context menu would also have some smarter query generation.
Right now, the generated query is always only: SELECT * FROM "table" WHERE time > now() - 5m .
It would be very useful if there were the possibility to generate not just for the last 5 mintues, but e.g.:

dependent on the available timestamps data, i.e. "last_available_date" - 5m instead of just now()-5m
dependent on the columns, i.e. select col1, col2, col3 from instead of just select * from

Thank you.

It would be nice to be able to configure the default query. t least to define a different value for 'now() - 5m', like 'now() - 60m', ...

I vote for a change of the default query as well.
I changed my local copy to the following queries (which I prefer more - but that's just a personal opinion)
SELECT * FROM "dbname" ORDER BY time DESC LIMIT 1000
AppForm.cs 1487:
queryControl.EditorText = string.Format("SELECT * FROM \"{0}\" ORDER BY time DESC LIMIT 1000", measurement);

Can I download your copy somewhere or the relevant *.dll for this change?


New prettier comment form

Right now when the user clicks on "Questions/Feedback" they are taken to the main PDS comment form, which is horribly ugly. We need a pretty one that looks like OPUS.

The PDS Engineering Node is planning to come up with a consistent way for people to give feedback. We will defer working on this until that is fleshed out so we don't duplicate effort.


tag lts releases

would be cool is allTargets had an .lts === true flag if a release is currently lts

Good idea!


Added new sample MyEmails

Q
A

New sample?
yes

What's in this Pull Request?
Added new sample ACE for showing current user's emails in ACEs

@siddharth-vaghasia thank you for your sample. I am in the process of reviewing it and will get it approved as soon as I can. Question, the ReadMe indicates that this is targeted to 1.13 but the package.json uses 1.14 beta. Do you want to pick one and update? Thanks again!

@dcashpeterson  - sure, will update.

if you can please advise on what the should be a badge for the 1.14 beta? Tried some combinations but it not worked...
or it would be https://img.shields.io/badge/version-1.14.0-green.svg ?

@siddharth-vaghasia go ahead and use the 1.14.0-green. I can check with folks and make sure it's correct and if it's the only change I can update it later if it's incorrect.

@dcashpeterson  - I have updated to spfx 1.14....

@siddharth-vaghasia Thank you for the sample. I apologize it took me so long. @PopWarner will reach out to set up a demo if he hasn't already.


Migrate apache dashboard and update with blog style

What does this PR do?
This PR updates the apache ootb dashboard with blog style and moves the dashboard to integrations-core.

### Motivation

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

LGTM, the only suggestion I have is to add a description under the logo with links to the blog post and integration doc:
https://www.datadoghq.com/blog/monitor-apache-web-server-datadog/
https://docs.datadoghq.com/integrations/apache/?tab=host


Ruby: Remove app_name and app_version

They have been deprecated for over 6 months and should be removed.

Codecov Report

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

@@            Coverage Diff            @@
##             master    #1525   +/-   ##
=========================================
  Coverage     87.05%   87.05%
  Complexity     4126     4126
=========================================
  Files           404      404
  Lines         16618    16618
  Branches       1764     1764
=========================================
  Hits          14466    14466
  Misses         1666     1666
  Partials        486      486

Continue to review full report at Codecov.

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


React Forms not working after installing appcues SDK

Hi! I recently installed the AppCues SDK on a React Native app and noticed that after installing it some things related to react-hook-form have stopped working making the app unusable.
The problem happens on iOS both real devices and simulators.
After the app launch, everything seems to be working fine and the AppCues flow starts correctly and show as configured on the panel except for the Inputs where we use react-hook-form and some functions seems to not work anymore, ex: watch().
The relevant part of our versions are the following:
"@appcues/react-native": "3.1.9",
"react": "18.2.0",
"react-hook-form": "7.49.3",
"react-native": "0.72.6",

Since there's no errors, warnings or traces I can't even start to work on the issue. Do you have any suggestions? Is there any more detail I can provide to help track this down? Thank you!

Hi Jose, thanks for reaching out.
This is quite a strange issue and not one we've seen before. There shouldn't be any reason an Appcues flow would interfere with the behaviour of your form inputs.
A couple questions to clarify:

Your text inputs only stop working after an Appcues flow is shown? ie I want to confirm that just having the Appcues SDK running isn't causing any issues
Does it make a difference if your inputs are already rendered when the Appcues flow is shown? i.e. is It broken in both of these cases:

Form is rendered (and working), Appcues flow shows, Form stops working
Appcues flow shows, navigate to Form, Form isn't working

Does the flow that's displayed include any survey blocks (ie text inputs or other form fields)? I'm guessing it's the "Test Mobile 2" flow you've been working on, so "no"?

I've created a barebones test app to try and reproduce the issue: https://github.com/mmaatttt/appcues-react-native-module-issue-136. It uses the versions you've listed and then has a couple uses of watch() on one TextInput. Then I setup Appcues to be able to launch a flow on demand. After all that, the text input still works fine with watch().

Could you take a look at the test app and let me know if it looks like it should be an adequate reproduction of your use case?
And I know this might be a big ask, but the best thing to help us debug the issue would be if you could provide minimal sample code (like my app above) that reproduces the issue.
Thanks,
Matt

Hi Jose, thanks for replying, I really appreciate it.
Answering your questions:

Yes it stop working as soon as the Appcues flow shown, before that everything works okey, initializing the session, identifying the user and even tracking the screen but as soon as I reach a screen where the flow is triggered it stop working.

The inputs are rendered after the flow is shown, we navigate to a modal which is basically a fullscreen multine input. So is Appcues flow shows, navigate to Form, Form isn't working.

I does not includes any survey block that I know of. Is kind of frustrating since we want to implement this for production, we also tried to update to expo 50 an react-native 0.73.2 but nothing seems to be working and it is breaking a main feature of the app.

About providing a sample code I can't assure anything since I'm just a developer so I should ask.
Thanks,
Jose

@mmaatttt @josarcsal, we've been seeing similar issues on our project. Did we resolve this?
Can provide more info if needed.

@mjd-myob we spent quite a bit of time debugging this, but were not able to come to a satisfying conclusion.
A minimal reproducible example would be most helpful since the issue never surfaced in my test app.
And can you confirm if you're having the same specific problem described in this issue, or it is different in some way?


Bug: Profile header is squashed due to new margins

Describe the bug

Steps to reproduce
View a profile full screen on the laptop.
Browsers
Chrome
Additional context (Is this in dev or production?)
This fix needs to not break any other page.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

The pr https://github.com/open-sauced/insights/pull/1490 broke the profile page because the feed page is using the layout wrapper of the profile page. So when Rita fixed the feed page responsiveness, it actually broke the profile page.
We should probably have layout wrapper for the feed.
Will try to find out more about the card page

Just a note here to prevent such behaviour in the future and be sure that we will not have any issues we might need to have snapshot tests. I opened an issue for that.
Thanks takanome for your comment, it's really useful.

The PR fix: enhance feed layout responsiveness #1538 will fix the issue.


3:2 Pull Down MFXVideoDECODE

Hi
Got an interlaced clip that midway does a pull down, i.e., tb,tb,tb, ... tbt,btb,...,tb, tb ....
I am wondering if MFXVideoDECODE is able to deal with this, and if so, how would I be able to detect such transitions.
Cheers,

Hi @Nastooh-Inca ,
Are you talking about Table D-1? If so, you can get this info from https://github.com/Intel-Media-SDK/MediaSDK/blob/master/doc/mediasdk-man.md#picstruct
MFX_PICSTRUCT_FIELD_REPEATED is dedicated to tbt, btb cases. Please note that in order to get MFX_PICSTRUCT_FIELD_REPEATED you need to initialize decoder with ExtendedPicStruct.
Please also see decoder implementation, just for your reference :
https://github.com/Intel-Media-SDK/MediaSDK/blob/master/_studio/mfx_lib/decode/h264/src/mfx_h264_dec_decode.cpp#L1535
So, in order to detect the transition you need to check presence/absence of MFX_PICSTRUCT_FIELD_REPEATED in output surface PicStruct field.

Thank you for the feedback. Do theses enums hold for all formats, i.e. MPEG2, AVC and HEVC, or as noted in the table, just for AVC?

Hi @Nastooh-Inca ,
Do you have other questions/concerns ?


Extract govuk::user into its own module

In a bid to break up the monolithic govuk module.
I tested this in Vagrant.

Rebased against recent changes in master.

merge when ready


TypeError: undefined is not an object (evaluating 'warnings.length') - Version 0.58.4

Have tried Stackoverflow as well, No solution.
This used to work, I have many other Apps of the same Version and they all are working fine. A couple of months back created a new app with the same react-native init command and its working fine. Now don't know what changed.
Description
TypeError · TypeError: undefined is not an object (evaluating 'warnings.length') This error is located at: in YellowBoxListRow (at YellowBoxList.js:91) in RCTView (at View.js:45) in View (at VirtualizedList.js:1734) in CellRenderer (at VirtualizedList.js:687) in RCTView (at View.js:45) in View (at ScrollView.js:873) in RCTScrollView (at ScrollView.js:1002) in ScrollView (at VirtualizedList.js:1061) in VirtualizedList (at FlatList.js:625) in FlatList (at YellowBoxList.js:87) in RCTView (at View.js:45) in View (at YellowBoxList.js:79) in YellowBoxList (at YellowBox.js:104) in YellowBox (at AppContainer.js:93) in RCTView (at View.js:45) in View (at AppContainer.js:115) in AppContainer (at renderApplication.js:33)
React Native version:
React Native 0.58.4
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

react-native init ProjectName --version=0.58.4
cd ProjectName
react-native run-android

Expected Results
App should have executed and shown the screen
Snack, code example, screenshot, or link to a repository:
Mac OS Catalina
Latest Android Studio Version
Latest Android OS on OnePlus6T

hellow,how did you resolve it?

I didn't, still waiting for some insightful comments from the maintainers. Would be helpful to understand the cause. Would be willing to help raise a PR if I understand the challenge.

I encountered the same problem，😫

Hey guys, same problem here.
In the meanwhile, you can at least run and test the app by using the Debug JS Remotely.

But it has edge cases when you make the Release build. And you can't find the problem because it will not be the same case when Debug mode is on.

Sorry, but the whole objective is to work on the desired version, upgrading will be later part of development this year. Hence the issue is raised. This wasn't the case a month back it was fine then.

And Yes it works on the higher version, for now, I have used 0.59
Removed the 0.58.4 installation so the environment info cant is shared, but it's straight forward to recreate the challenge I have mentioned in the description.

Same here :(

Met the same problem

Hi Team,
More cases are going to be reported on the same, there has to be some solution to this. Not all will be able to upgrade their react-native versions.
Hi Whoever is facing this challenge,
The closest version that works in 0.59 and changing to 0.59 has helped me resolve it without breaking changes. I hope it works for you too.

thanks! I have resolved the problem from upgrading my RN version
At 2020-04-16 12:57:58, "Kai"<EMAIL_ADDRESS>wrote:
Sorry, but the whole objective is to work on the desired version, upgrading will be later part of development this year. Hence the issue is raised. This wasn't the case a month back it was fine then.
And Yes it works on the higher version, for now, I have used 0.59
Removed the 0.58.4 installation so the environment info cant is shared, but it's straight forward to recreate the challenge I have mentioned in the description.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Sounds like original issue filer is unblocked by moving to a near version (and didn't have to go to latest). So yay? Closing out issue.

The same issue. Any ideas?