BadHttpRequestException (encoding?)

info: Microsoft.AspNetCore.Server.Kestrel[17]
      Connection id "0HLE7VFNTA097" bad request data: "Invalid request line: 'PUT %2fLM%2fW3SVC%2f2%2fROOT(oXvpnxcuEWqKAgOETha32JavYZ6eCqrpTN5%2fHxWS854%3d)%2fk1im0mymglivv5rttkwn4fle HTTP/1.1\x0D\x0A'"
Microsoft.AspNetCore.Server.Kestrel.Core.BadHttpRequestException: Invalid request line: 'PUT %2fLM%2fW3SVC%2f2%2fROOT(oXvpnxcuEWqKAgOETha32JavYZ6eCqrpTN5%2fHxWS854%3d)%2fk1im0mymglivv5rttkwn4fle HTTP/1.1\x0D\x0A'

Don't even ask me what I'm trying to get connecting. Weird old project that does a lot of stuff by hand.
What's the deal with this though? The best I have is the encoding is odd. Maybe the %2f on the front?

Confirmed. Sticking a / at the beginning of the same PUT works fine.

You're right that this appears to be intentional. See #1409 which was a PR made to close #1393. I can't find any mention in either the issue or PR for why we started rejecting all requests starting with %. @davidfowl comments:

This is the GET % HTTP/1.1 scenario right?

It makes sense that a Request-URI containing just a % should be rejected given it's a partial escape sequence, but I'm not sure why we started rejecting any Request-URI starting with an escape sequence. @Tratcher @davidfowl Any ideas why we might do this?

I don't see any explanation for why a leading % should be treated differently.

Yea I don't know if it was intentional.

After talking to @Tratcher, I think we figured this out. PUT FOO HTTP/1.1 is accepted because it is interpreted as an authority form request and "FOO" contains only valid authority form characters. "F%2f" is also rejected (in a different part of the code) for not containing only valid authority form characters.
It seems that the if (pathStart == -1) check after seeing a % character in the Request-URI is no longer necessary since a request starting with a % would be rejected by Http1Connection.OnAuthorityFormTarget() anyway.

So, okay. But should % actually be rejected?

@wasabii Yes, % should be rejected. From our reading of RFC 7230 Section 5.3, Request-URIs should come in one of 4 forms: origin-form (i.e. absolute-path ["?" query]), absolute-form, authority-form and asterisk-form. Origin-form targets can be percent encoded, but must start with /. Absolute-form targets can have their paths percent encoded but must start with http:// or https:// (given that we're an HTTP server) and must be an absolute-URI as defined in RFC 3986 Section 4.3. Authority-form targets are only valid for CONNECT requests and has a pretty limited allowed charset that doesn't include %. And lastly an asterisk-form target has to be exactly *.

Oh. It did fail. It just made it past the bad request line stuff I was focused on.
No chance of supporting % eh? I got it working by rewriring the method IL inside Kestrel at load. But that is hella bad.

Trying to write an ASP.Net state server in Core, hosted on Service Fabric, using the Actor model. And maintaining compatibility with existing protocol, which is closed source, in original ASP.Net.
Whomever came up with this (MS) didn't use any existing HTTP library, but decided to do it by hand with sockets. The one thing that they got really wrong is the damned slash.

So this is one of those long standing issues facing everybody trying to write awesome software: how close does one stick to technical purity, vs how many weird use cases do you want your software to be used for.
I don't envy the choice!
What I can say though is, given no way to "turn off strict handling" in Kestrel, it's kinda useless for my particular application. I have temporarily solved my issue by using a IL interception library to overwrite the offending check. Which, from my point of view, isn't really maintainable. But it let me proceed with writing the rest of the software.
I think I'd probably end up with a private fork of Kestrel, though, to get this done.
It should be noted that neither HttpSys nor HttpListener work either.
How about this? Kestrel is largely container driven, right?

Note: the request isn't actually missing the preceding slash. It's just all URL encoded. If you decode it BEFORE looking for the /, it's fine. So they're probably actually just encoding the wrong parts. The entire thing instead of the segments.

%2F can't be decoded without changing the semantics of the request path.

We periodically close 'discussion' issues that have not been updated in a long period of time.
We apologize if this causes any inconvenience. We ask that if you are still encountering an issue, please log a new issue with updated information and we will investigate.


Question about node taints with regard to doks-managed 'coredns' deployment

Hello :wave:
I've a bug report/question about the title:
I've recently created a new cluster on DO, with following Terraform configuration
resource "digitalocean_kubernetes_cluster" "main" {
  # ...
  node_pool {
    # ...
    labels     = {}
    tags       = []
    taint {
      key    = "x-resource-kind"
      value  = "apps"
      effect = "NoSchedule"
    }
  }
}

resource "digitalocean_kubernetes_node_pool" "pool-main-storages" {
  # ...
  labels     = {}
  tags       = []
  taint {
    key    = "x-resource-kind"
    value  = "storages"
    effect = "NoSchedule"
  }
}

Basically I want the new nodes spawned to automatically be given a taint, since I want to control my current/future pods resources for internal usages. The clusters & node pools are created fine, and so is the taint
captain@glados:~$ kubectl describe nodes pool-main-fv5zb
# ...
Taints:             x-resource-kind=apps:NoSchedule
# ...

But I noticed that one of the deployments are not running (coredns)
captain@glados:~$ kubectl get deployment -n kube-system
NAME              READY   UP-TO-DATE   AVAILABLE   AGE
cilium-operator   1/1     1            1           10h
coredns           0/2     2            0           10h

captain@glados:~$ kubectl get pods -n kube-system
NAME                              READY   STATUS    RESTARTS   AGE
cilium-operator-98d97cdf6-phw2j   1/1     Running   0          10h
cilium-plbv2                      1/1     Running   0          10h
coredns-575d7877bb-9sxdl          0/1     Pending   0          10h
coredns-575d7877bb-pwjtl          0/1     Pending   0          10h
cpc-bridge-proxy-hl55s            1/1     Running   0          10h
konnectivity-agent-dcgsg          1/1     Running   0          10h
kube-proxy-zfn9p                  1/1     Running   0          10h

captain@glados:~$ kubectl describe pod/coredns-575d7877bb-9sxdl -n kube-system
# ...
Events:
  Type     Reason             Age                      From                Message
  ----     ------             ----                     ----                -------
  Warning  FailedScheduling   31m (x118 over 10h)      default-scheduler   0/1 nodes are available: 1 node(s) had untolerated taint {x-resource-kind: apps}. preemption: 0/1 nodes are available: 1 Preemption is not helpful for scheduling..
  Normal   NotTriggerScaleUp  2m10s (x431 over 7h16m)  cluster-autoscaler  pod didn't trigger scale-up: 1 node(s) had untolerated taint {x-resource-kind: apps}

Is this expected? From the logs I understand why it didn't trigger the scale up, it's just that I don't know whether this is the proper behaviour or not.
It's also that other kube-system pods/deployments are running fine, I think because the tolerations are set up to "always tolerate everything"
captain@glados:~$ kubectl describe pod/cilium-plbv2 -n kube-system
# ...
Tolerations:                 op=Exists
                             node.kubernetes.io/disk-pressure:NoSchedule op=Exists
                             node.kubernetes.io/memory-pressure:NoSchedule op=Exists
                             node.kubernetes.io/network-unavailable:NoSchedule op=Exists
                             node.kubernetes.io/not-ready:NoExecute op=Exists
                             node.kubernetes.io/pid-pressure:NoSchedule op=Exists
                             node.kubernetes.io/unreachable:NoExecute op=Exists
                             node.kubernetes.io/unschedulable:NoSchedule op=Exists
# ...

versus
captain@glados:~$ kubectl describe pod/coredns-575d7877bb-9sxdl -n kube-system
# ...
Tolerations:                 CriticalAddonsOnly op=Exists
                             node.kubernetes.io/not-ready:NoExecute op=Exists for 300s
                             node.kubernetes.io/unreachable:NoExecute op=Exists for 300s
# ...

As per the reference
If this is expected, you can close this issue. If not then the default deployment might need to be adjusted maybe? Though I don't know whether this will affect others

Hey 👋
Most of the managed workloads that are deployed into the data plane are critical Daemonsets that must or should always run to provide core functionality. CoreDNS is in a bit of a mixed state, in the sense that it does provide core functionality but that it must also run on a worker node that is considered healthy (and that it should be moved / evicted to one should its hosting node become unhealthy).
I think we haven't revisited the current tolerations in a while, so there's possibly an opportunity to improve here. That said, I'd be hesitant to give it a blank toleration since, for instance, we wouldn't want CoreDNS to continue to run on a node that's under memory pressure.
There have also been requests to support extending the list of tolerations on CoreDNS by customers to make it better suit custom taints associated to node pools (as you did). This is something we're also considering to do.
With that background shared, I'd be curious to hear what people's preferences are (and why if non-obvious). This is help us plan for the right, next move in this context.


add DS420+ to compatibilty list

Updates the README to include DS420+ on the compatibility list

The failure here is due to the 1.2.1 packages no longer being accessible from pkgs.tailscale.com. I've bumped the version of tailscale used to 1.2.8 in #34 which should help update the libraries as well as fix CI for this PR.

Thank you, @jacobbednarz !


Crude fix to 'first crust_peer not reading user input'

r? @vinipsmaker
(maidsafe_highfive has picked a reviewer for you, use r? to override)

closes #277

closes #272


fix(SidePanel): resolve storybook SlideOver animation flickering in doc page

Closes #6460
What did you change?

Properly set the animationComplete boolean and its validation when opening SidePanel
Addition to that, I just refactored the setTimout in the useEffect used for focus init

How did you test and verify your work?
Storybook

I noticed an issue in production unrelated to this PR while reviewing it. On the docs page, in the Slide Over option for the slide panel component, the page becomes unscrollable when the panel is opened.


fix(Prompts): use location#display and userInterestsConnection for prompt checks

Description
This PR switches some properties we use to check if:

user has a location
user added artists
It looks like when we add an artist to the collection we can't rely on myCollectionInfo.artistsCount since the collection will be empty and location#display is more accurate for users that don't have a city but have a country for example(new signed up users for example)

Other changes:

removed the duplicated helper test under src/app/utils/tests/collectorPromptHelpers.tests.ts

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against abb5055550de72b4c1c0359069db3ea445379325


CFP: Provide support to advertised routes as a summarized address when doing router advertistment

Cilium Feature Proposal
Support advertising services as summarized routes
Is your proposed feature related to a problem?
I was deploying cillium with bgp control plane + bgp service advertisment and was requested by an external network team to advertise a /24 for all services instead of individual /32's (per service)
Describe the feature you'd like
Allow cilium to summarize service advertisements
(Optional) Describe your proposed solution
provide a summarization option under the CiliumLoadBalancerIPPool which summarizes any address to the pool supernet when advertised
apiVersion: "cilium.io/v2alpha1"
kind: CiliumLoadBalancerIPPool
metadata:
  name: "lab-pool"
spec:
  summarize: true <------- new option which will then summarize any service to the block address when advertised
  blocks:
  - cidr: "<IP_ADDRESS>/24"

Can I kindly ask to assigne this issue to me?

Any update?

Any update?

I am working on this issue.

any update?

I still work on this feature. The result will be soon

@aanm Can I kindly ask to reopen this issue?

@aanm Can I kindly ask to reopen this issue?

and @tbotnz  could you pls reopen this?


ADD: Multiapi request support

I few minor fixes and I think we should be good to go!

Looks good to me, going to remove a line of commented code and then get this merged in.


[gopls] No underline for some kinds of diagnostics

Description
The underline for some kinks of diagnostics does not show up. Others such as sign and virtual_text work well.

Neovim version
NVIM v0.9.0-dev-667+gdd200db98
Build type: Release
LuaJIT 2.1.0-beta3
Nvim-lspconfig version
No response
Operating system and version
macOS 13.1
Affected language servers
gopls
Steps to reproduce

Make a directory for testing mkdir test and then cd into it
Run nvim test.go
Type packa abc
Then the sign and virtual text show up saying the error message "syntax: expected 'package', ...", but no underline there.

Actual behavior
No underline for diagnostics for some kinds of errors.
Expected behavior
No response
Minimal config
local on_windows = vim.loop.os_uname().version:match 'Windows'

local function join_paths(...)
  local path_sep = on_windows and '\\' or '/'
  local result = table.concat({ ... }, path_sep)
  return result
end

vim.cmd [[set runtimepath=$VIMRUNTIME]]

local temp_dir = vim.loop.os_getenv 'TEMP' or '/tmp'

vim.cmd('set packpath=' .. join_paths(temp_dir, 'nvim', 'site'))

local package_root = join_paths(temp_dir, 'nvim', 'site', 'pack')
local lspconfig_path = join_paths(package_root, 'test', 'start', 'nvim-lspconfig')

if vim.fn.isdirectory(lspconfig_path) ~= 1 then
  vim.fn.system { 'git', 'clone', 'https://github.com/neovim/nvim-lspconfig', lspconfig_path }
end

vim.lsp.set_log_level 'trace'
require('vim.lsp.log').set_format_func(vim.inspect)
local nvim_lsp = require 'lspconfig'
local on_attach = function(_, bufnr)
  vim.keymap.set('n', '<space>e', vim.diagnostic.open_float, opts)
end

-- Add the server that troubles you here
local name = 'gopls'
if not name then
  print 'You have not defined a server name, please edit minimal_init.lua'
end
if not nvim_lsp[name].document_config.default_config.cmd and not cmd then
  print [[You have not defined a server default cmd for a server
    that requires it please edit minimal_init.lua]]
end

nvim_lsp[name].setup {
  on_attach = on_attach,
}

print [[You can find your log at $HOME/.cache/nvim/lsp.log. Please paste in a github issue under a details tag as described in the issue template.]]

LSP log
https://gist.github.com/rockyzhang24/971fd1635959fa27f66527c4bf968ab9

:h vim.diagnostic.config

@glepnir config.underline is true by default, so the underline should display. This is more likely a bug. Please reopen it. Thanks.

not bug of neovim or this plugin. hi DiagnosticError

I tested using the minimal config and hl-DiagnosticError has been set to red. Could you please try to reproduce it?

hl-DiagnosticUnderlineError and all the related diagnostic hlighgroups are set by using the mininal config. Still no underline. The issue just happens for gopls.

try reproduce it later. thanks for report.

the reason is gopls return 0 for column. so there is no underline
{
  bufnr = 1,
  col = 0,
  end_col = 0,
  end_lnum = 0,
  lnum = 0,
  message = "expected ';', found 'EOF'",
  namespace = 18,
  severity = 1,
  source = "syntax",
  user_data = {
    lsp = {}
  }
}

so there is nothing on neovim . thanks. report it to gopls if you think this is important for you .

Thank you for confirming it. So is it a bug of gopls or a kind of deliberate design? I am not family with the LSP specification. Is it possible for neovim or this plugin to fix it by adjusting col returned by the server?

it should return a range that include lnum and col pos. on plugin level we just get result from server and set highlight through this range.  0 just mean the start col . we don't know the actual wrong pos column.
https://microsoft.github.io/language-server-protocol/specifications/lsp/3.17/specification/#range

I tested on VSCode and it showed the underline normally.

gopls vscode extension write by google . maybe they think 0 is mean whole line ? I don't know why design like this. it's not lsp spec.

Okay I got it. Thank you so much for guiding me. I will close this issue.


JIT class support for object mode on method-by-method basis

I am adding the 'feature_request' label to this one. Also, @seibert - do you think we could remove the 'highpriority' label from this one? Also also, I think I saw something similar to this during the last year, where a feature was requested to execute certain methods, e.g. ones that do I/O, in jitclasses within object mode.

This seems to be related: https://github.com/numba/numba/issues/5924


(chores) experimental ci: fix cache usage and apply minor cleanups

Progressing, but still not functional. Let's merge this for now and I'll take a closer look tomorrow.


i18n

First of all congratulate you for the excellent work you are doing.
I can not find information on how to translate messages into other languages.
What version of React-Table are you using?
6.7.6
What bug are you experiencing, or what feature are you proposing?
All messages are in English. Can I display the labels of the pagination controls in another language?

For others with same question, related props are:
// Text
previousText: 'Previous',
nextText: 'Next',
loadingText: 'Loading...',
noDataText: 'No rows found',
pageText: 'Page',
ofText: 'of',
rowsText: 'rows',


Create 0000-01-02-maryam-z.md

This is a commit!

Pull request successfully merged

Thank you Git Lab!


fig2svg downgrades svg-resolution

For a project I need a lot of images preferably in svg-format.
I have export_fig and fig2svg installed as add-ons.
If I deactivate fig2svg and just run export_fig I get: a 1mb file with good resolution but also a grid of white lines imposed on top of the figure.
If I activate fig2svg and run export_fig with it I get: a 10kb file with poor figure resolution but with properly rendered text (axis etc.) and no white grid.
What is going on here, specifically the resolution issue? And is there a solution?
The code line I run atm is: export_fig vectorfig.svg

This seems to be an issue with fig2svg, but fig2svg is no longer maintained by its author. You could try to contact its owner directly.
Alternatively, try to save to SVG using Matlab's builtin saveas function (this is only supported on recent Matlab releases).


Is there a way to use this script with a DeepL username/password?

Thank you for this fantastic tool!
I would like to know if there is a way to use the DeepL translation not by chunking the file, but with an actual username/password of DeepL subscriber.
Again, thanks a lot for your work and your generosity sharing it.
Angel

Thanks for your reply, and again congratulations for your work!

@angbadillo In the lastest version (0.2.0) available in PyPi, you could use DeeplApi with an API-KEY. Check out the docs and examples. Hope it works.
Feel free to close the issue if it works

Maybe I'm doing something wrong...
main.py: error: unrecognized arguments: --t --auth [and my API key]
I've tried both CLI syntax in the documentation, but both seems to fail.
:(

Did you updated to 0.2.0?
it is -t deep-api --auth apikey

Close due to inactivity. And it should work with an API KEY


Refunds

Brief Description
Started with the refund api, but ran into too many issues with the checkout api to test refunds without some alterations. I fixed the checkout page to include a stripe session redirect. Added a webhook so upon successful payment the server is notified and payment intent is saved.
Issue #77

[ !] Bug fix
[ !] Feature
[ !] Major change (potentially breaking, list anything that is broken below)
[ ] Documentation

Testing done
Stripe CLI webhook testing was done. A lot of this had to be done through the stripe dashboard.

[ !] Docker was used.
[ ] Docker was not used and I have listed my configuration below.

System Specifications:

Operating System: Win 10
Node Version: 16.16.0
NPM Version: 8.18.0

Checklist:

[ !] My code follows the styling guidelines.
[ ] I have added unit tests and verified that they pass.
[ !] I have used the linter and fixed any linting issues.
[ !] I have commented the code.
[ !] I have adjusted the documentation to match the changed code.
[ !] Prior to submitting the PR, I made sure the latest changes were merged with my branch.
[ !] Existing tests pass locally with changes.

Explanation for unchecked
Testing was done through stripe CLI so unit tests weren't implemented yet

Refunds, check out, and webhook added.


AI Team Balancing

Currently, there could be 4 Gold team AI players. link to demo

This is not an issue. Their teams are randomly generated.

This should fall under a game balancing issue, if people are able to look into this, then great, if not, it can be closed by end of semester.

Fixed, teams for AI are now staggered like: Green, Gold, Green, Gold and are in a FIFO queue.  When a AI connects, they grab the first one, and when they leave they push it to the back of the queue.


Inconsistent usage of Enum#name and toString in MapCompositeConverter.

Hi,
I was just working on getting data stored and returned using a Map with an Enum key.  Initially I was looking at it from the point of view of naming of the properties, in that the recommendation on the Neo4j website is:

2.2.1. Naming rules

Must begin with an alphabetic letter.
    This includes "non-English" characters, such as å, ä, ö, ü etc.
    If a leading non-alphabetic character is required, use backticks for escaping; e.g. `^n`.

Can contain numbers, but not as the first character.
    To illustrate, 1first is not allowed, whereas first1 is allowed.
    If a leading numeric character is required, use backticks for escaping; e.g. `1first`.

Cannot contain symbols.
    An exception to this rule is using underscore, as given by my_variable.
    An ostensible exception to this rule is using $ as the first character to denote a parameter, as given by $myParam.
    If a leading symbolic character is required, use backticks for escaping; e.g. `$$n`.

So by default if using a Map it adds a '.' as the separator between the prefix and name.  Whilst the naming rules say property names cannot contain symbols, I guess this means - if you do you will have to put backticks around it?
But the other part is that the Enum is used as the name of the property, which is upper case, with the recommendation in Neo4j being that it should be lower case.  I was wondering about asking/making a change so that a flag could be added to allow these names to be converted to lower case to use as property names?  (I would assume this would need a flag on the annotation, so as not to break any existing saved data?)
However when I was looking at this, I saw that in the addMapToProperties method it uses
graphProperties.put(prefix + entry.getKey(), entryValue);

to put the property into Neo4j.  But when extracting the data it uses:
return keyType.getDeclaredMethod("valueOf", String.class).invoke(keyType, propertyKey);

in keyInstanceFromString.  What I think this means is that it uses the toString() method of the enum to save it, and then the name of the enum to get it back again?  Is that right, in which case it could be a problem.
Thanks,
Matt
Your Environment

OGM Version used:  3.2.0-alpha05
Java Version used: 8
Neo4J Version used: 3.5.2 (Community)
Bolt Driver Version used (if applicable): 3.1.7
Operating System and Version: Windows 10

Regarding naming: Yes, you are correct. Neo4j-OGM escapes those names. When you run PropertiesTest with debug output, you'll see for example:
Request: CREATE (u:User {name:'Frantisek', myProperties.address.city:'London', myProperties.address.zipCode:'SW1A 1AA'}) with params {}
Apart from that, you are correct, overriding toString in an enum breaks our mechanism.
I'm gonna fix this in both 3.2 and 3.1.

You can now achieve this like this

package org.neo4j.ogm.domain.properties;

import static org.neo4j.ogm.annotation.Properties.*;

import java.util.HashMap;
import java.util.HashSet;
import java.util.Locale;
import java.util.Map;
import java.util.Set;
import java.util.function.BiFunction;

import org.neo4j.ogm.annotation.NodeEntity;
import org.neo4j.ogm.annotation.Properties;
import org.neo4j.ogm.annotation.Relationship;

@NodeEntity
public class User {

    public enum EnumA {VALUE_AA}

    @Properties(transformEnumKeysWith = LowerCasePropertiesFilter.class)
    private Map<EnumA, Object> filteredProperties;

    public static class LowerCasePropertiesFilter implements BiFunction<Phase, String, String> {

        @Override
        public String apply(Phase phase, String s) {
            if (s == null) {
                return null;
            }

            switch (phase) {
                case TO_GRAPH:
                    return s.toLowerCase(Locale.ENGLISH);
                case TO_ENTITY:
                    return s.toUpperCase(Locale.ENGLISH);
                default:
                    throw new IllegalArgumentException();
            }
        }
    }
}


Add llama 31 support

What does this PR do?
This PR adds Llama 3.1 8B support. Please refer to the PR  for discussion history.
Fixes

[ ] Bumped TGI version from 2.0.3 to 2.2.0
[ ] Bumped Transformers from 4.41.1 to 4.43.3
[ ] Bumped Rust from '1.77to1.79`
[ ] Added otlp_service_name argument in serve() method with default value text-generation-inference.server
[ ] Added max_input_tokens argument in serve() method with default value None
[ ] Modified modelling_llama.py for fixing rope_scaling issue.
[ ] Added Llama 3.1 8B test in test_decode.py

@tengomucho

Regarding your suggestion REF

you are right, the right parameter to use is "MAX_INPUT_TOKENS", I mixed things up. My point is: what do you want to achieve by adding this to the cli in the text generation server? For now IIRC this value is used in the launcher to define the maximum number of input tokens that can be passed from the router to the server. The server for now does not use that. It is ok to add it to the cli, but to be effective you will also need to add it to the serve function, and do something with it, otherwise it will not have any effect.

We only require MAX_TOTAL_TOKENS and MAX_BATCH_PREFILL_TOKENS, but looking deeper into TGI v2.2.0  launcher's main.rs; we found that launcher always passes --max-input-tokens argument to the serve function. This is why we cli.py serve function is issuing Error: No such option: --max-input-tokens rank=0
So how can we handle this --max-input-tokens parameter inside cli.py?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

This PR is closed as it does not contain recent changes in the main branch. A new PR is created as replacement.

FYI @Bihan, next time you can just rebase to the master branch and force-push:
git checkout main
git pull
git checkout mybranch
git rebase main
# resolve conflicts
git push --force

This way you do not need to open a new PR 🤗


Adding scroll progress bar

@SaranshBangar
I would like to work on this, could you please assign me this under gssoc-ext, hacktoberfest-accepted, level2

@AswaniBolisetti Sure please follow the guidelines from #69


Input Search Problem

On a desktop view whenever we click on the search input it gives weired border which looks very bad to the eyes and whenever we scroll to the middle of the page the search input goes completely unresponsive.

Hey @AsarAman Could you help us with more details? Are you referring to issue with Edge Add-ons site?

I am referring to microsoft browser, in which there is a main search input in the middle of the page i am talking about that one. Whenever we focus on the search input it has some bad looking border and when we scroll down it becomes unresponsive.

Thanks for the details @AsarAman
This community space is to help developers with Edge add-ons development journey and handling issues related to Partner Center and Microsoft Edge Add-ons site.
For browser related issue, please reach out to Microsoft Edge support by following the steps mentioned here.


Write a PHP program to find sum of n natural numbers by recursion

Description
Write a PHP program to find sum of n natural numbers by recursion
Input  : 5
Output : 15

How to contribute

Save the solution in program/find_sum_of_n_natural_numbers_by_recursion.php file
Add find_sum_of_n_natural_numbers_by_recursion.php file in find-sum-of-n-natural-numbers-by-recursion folder

!assign


[WIP] Manual Search mobile fixes

Summary
Currently manual search does not work on mobile because the fuzzy search bar doesn't lose focus properly—this PR (hopefully) resolves that issue.
Test Plan

Test on mobile that manual search works again (requires deploy)

Issues

Looks like it works 🎉


Better ordering of directions?

The way the recipe generating algorithm is currently set up, some steps appear before others in a way that doesn't make logical sense.
Ex: "Transfer to a plate & serve immediately" can randomly appear before "Preheat the oven."
An improvement to this would be to possibly separate directions into groups? Directions that come first, directions that come in the middle, directions that come in the end? Just to make the end recipe more coherent/logically flowing.

Yes I agree, I do want to keep this behaviour, because I find it quite funny sometimes, but this would fit perfectly in the "serious mode" PR: #13 commit: #11

I would like to work on this issue

@gabLaroche Did you notice the PR? #37

@Brainshaker95 Yes I noticed, I'm very sorry, I was swamped at work, I'll take a look at the PR

@gabLaroche I'm kindly asking you to add the hacktoberfest-accepted label to my PR. If this is not something you want to to, please telle me so that I can use the remaining days to hunt for another repo to participate

@Brainshaker95 Yes of course! it's my bad, I thought I had the hacktoberfest label on the project itself, apparently not haha


Issues with website registration on Windows

I've tested this both on my Solo 2a and my 2c
Twitter and github when attempting registration  I get the following screen and nothing happened when I touch the buttons on the key.

Then I get a different issue when I tried to register the key on facebook. It is prompting me to create a pin which leads me to think that it's trying to use PIV auth rather than fido authentication for facebook.

Thanks. It's indeed a very short time out and works if tapped very early on. Not a very convienent solution though.

Same issue here. Both the Pin requirement when using services (like the opensource mailcow project)  an the issue with the timeout. It's very annoying this way. If you're not fast enough you need to take out the key and insert it again to restart the process. Is there a solution for this? Is it an issue with windows or the site, or is it a solokey issue (and can it be fixed with an firmware upgrade)?

Unable to use it at all. By miracle managed to get all 3 keys working with Facebook, but afterwards any try was a fail. Can't get past "Can't read security key" screen and the touch timeout is deffinitely yoo short (not that it matters since keys are unusable at this stage)

The timeout is annoying and it seems to be about 2-3s but otherwise it seems to work pretty reliably. Didn't have this issue with timeout on the Android phone or Mac so it seems to suggest something on the Windows side. The PIN requirement is related to the user verification part of webauthn IIRC and depends on the website if it requires it or not.
@burtek try to play with the key at https://webauthn.me/debugger

@skundrik thanks for that link
Rebooted my PC and got all keys working via USB on most sites apart from OVH, Zoho (both seem to have internal issues tho) and Dropbox (can't get past "can't read security key"/"are you there" messages as described above)
On Android 12 via NFC none of keys seem to work though. When tested on the website linked above, all yield NotReadableError: An unknown error occurred while talking to the credential manager.

Finally making some progress on this. Underlying issue is that the current firmware uses way too much stack memory and hard faults in some situations (namely, USB interrupt during FIDO processing). With current firmware, when this happens, the only approach is replugging the device (so it gets out of hard fault) and try again "fast enough". But the next firmware release should fix this properly.

Finally some good news! Is there any ETA for the next firmware?

Is there a fix for this yet?

Also seeing this issue.

Me too.

Honestly, at this stage I'm starting to think about ditching Solo and getting Yubi. The keys are almost unusable for me...

The currently available firmware (via solo2-cli) is usable and allows U2F/WebAuthn to properly work on Windows (I use it multiple times a week). You may have to be a bit quick to touch your key, but that's about it. Make sure you're running version 20200101.9 and the key should at least ready for everday usage.
An updated firmware is worked on that solves the timeout issue but it's currently not stable enough to publish (breaks NFC). While I don't like the long delays with the firmware updates either, it's not horribly broken at least.

Honestly, at this stage I'm starting to think about ditching Solo and getting Yubi. The keys are almost unusable for me...

New firmware didn't really help. I moved to Yubi at this stage, happy to give solo another shot in future once firmware is fixed

Can i list somehow the key entries of the device? The sites where it is registered are not saved, if i recall correctly, but can i at least get the number of sites entries? That would help to unregister it.

The sites where it is registered are not saved, if i recall correctly

Imo they should be saved, that's how u2f works, isn't it? Unless I got something wrong

i tried
solo2 app oath list
(which is empty)

The new firmware solved this one for me. I had to reset the security key in windows 11 (was asked for a pin, but never had set up one for this key) after the firmware update.


improve german docs

Fixes some typos and grammatical errors in german docs.

Hi @philipsahli .
Just checking if this PR should remain open?


Show a meaningful message when trying to add an external subdomain

Currently we have:
▲ ~ now domains add --external something.google.com
> Error! Not authorized to access domain something.google.com

We should have:
▲ ~ now domains add --external something.google.com
> Some message that tells you to not add the subdomain, but the actual domain instead

I'd be happy to take this on as my first contribution to now :smile:

I think this was resolved in commit https://github.com/zeit/now-cli/commit/2a50c2bd7f6247bddb1df3bee420573d26f05560#diff-d7b946ada4331cbb4103ebc18442f52aR300

Thank you @melvin0008, looks great :)


add functions to play with limits + copy application plan

Meant to answer @lucamaf need to copy an application plan.
This creates:

a command for limits
-> create
-> list
-> delete

It adds

the copy command to app-plan

Example of usage:
3scale-cli app-plan copy -s {SERVICE_ID} -a {APPLICATION_PLAN_ID}

Ping @3scale/support if you have time to test and evaluate this PR :)
@lucamaf feel free to try it out too

Ping @3scale/support to review this PR :)
Thanks


OSError: Could not load shared object file: libllvmlite.so

Previously (a few months ago) I had no problem with "import umap" but now it is generating an error message.  Any ideas what the problem might be here?
Full error message:
---------------------------------------------------------------------------
OSError                                   Traceback (most recent call last)
<ipython-input-1-59579f3be6e0> in <module>
     29 from sklearn.decomposition import PCA as PCA
     30 from sklearn.manifold import TSNE
---> 31 import umap

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/umap/__init__.py in <module>
----> 1 from .umap_ import UMAP
      2
      3 # Workaround: https://github.com/numba/numba/issues/3341
      4 import numba
      5

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/umap/umap_.py in <module>
     22 import scipy.sparse
     23 import scipy.sparse.csgraph
---> 24 import numba
     25
     26 import umap.distances as dist

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/numba/__init__.py in <module>
     13 del get_versions
     14
---> 15 from . import config, errors, _runtests as runtests, types
     16
     17 # Re-export typeof

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/numba/config.py in <module>
     16
     17
---> 18 import llvmlite.binding as ll
     19
     20 IS_WIN32 = sys.platform.startswith('win32')

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/llvmlite/binding/__init__.py in <module>
      4 from __future__ import print_function, absolute_import
      5
----> 6 from .dylib import *
      7 from .executionengine import *
      8 from .initfini import *

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/llvmlite/binding/dylib.py in <module>
      2 from ctypes import c_void_p, c_char_p, c_bool, POINTER
      3
----> 4 from . import ffi
      5 from .common import _encode_string
      6

/share/lanzarolab/opt/conda/vgl/lib/python3.6/site-packages/llvmlite/binding/ffi.py in <module>
    152         break
    153 else:
--> 154     raise OSError("Could not load shared object file: {}".format(_lib_name))
    155
    156

OSError: Could not load shared object file: libllvmlite.so

Well, I don't know what was going on, but I used conda to reinstall numba and that fixed the problem.


Failed to create SampleApplication

Briefly summarize your issue:
Terminal for authorisation, then exits with Failed to create to SampleApplication!
What is the expected behavior?
Terminal should keep checking for authorisation every few seconds, so I have time to go on amazon.com/us/code
What behavior are you observing?
`2018-07-10 11:27:39.251 [  1] I sdkVersion: 1.8.0
configFile /home/pi/sdk-folder/sdk-build/Integration/AlexaClientSDKConfig.json
Running app with log level: DEBUG9
2018-07-10 11:27:39.289 [  1] 0 ConfigurationNode:initializeSuccess
2018-07-10 11:27:39.321 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.617 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.623 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.625 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.630 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.631 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.636 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.638 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.643 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.645 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.650 [  1] 9 MediaPlayer:createCalled
2018-07-10 11:27:39.652 [  1] 9 MediaPlayer:noOutputConversion
2018-07-10 11:27:39.660 [  1] 5 SQLiteBluetoothStorage:create
2018-07-10 11:27:39.664 [  1] 5 CBLAuthDelegate:create
2018-07-10 11:27:39.668 [  1] 5 CBLAuthDelegate:CBLAuthDelegate
2018-07-10 11:27:39.669 [  1] 5 CBLAuthDelegate:init
2018-07-10 11:27:39.669 [  1] 5 CBLAuthDelegateConfiguration:create
2018-07-10 11:27:39.669 [  1] 5 CBLAuthDelegateConfiguration:init
2018-07-10 11:27:39.670 [  1] 5 CBLAuthDelegateConfiguration:initScopeData
2018-07-10 11:27:39.671 [  1] 9 CBLAuthDelegateConfiguration:initScopeDataSucceeded
2018-07-10 11:27:39.672 [  1] 5 SQLiteCBLAuthDelegateStorage:open
2018-07-10 11:27:39.681 [  2] 5 CBLAuthDelegate:handleAuthorizationFlow
2018-07-10 11:27:39.682 [  2] 5 CBLAuthDelegate:handleStarting
2018-07-10 11:27:39.682 [  2] 5 SQLiteCBLAuthDelegateStorage:getRefreshToken
2018-07-10 11:27:39.684 [  2] 0 SQLiteCBLAuthDelegateStorage:getRefreshTokenFailed:reason=stepResultWasNotRow
2018-07-10 11:27:39.685 [  2] 0 CBLAuthDelegate:getRefreshTokenFailed
2018-07-10 11:27:39.685 [  2] 5 CBLAuthDelegate:handleRequestingCodePair
2018-07-10 11:27:39.686 [  2] 5 CBLAuthDelegate:requestCodePair
2018-07-10 11:27:39.688 [  1] 5 CBLAuthDelegate:addAuthObserver:observer=0xfd2f2c
2018-07-10 11:27:39.689 [  1] 5 CBLAuthDelegate:addAuthObserver:observer=0xfbdee0
2018-07-10 11:27:39.691 [  1] 5 CapabilitiesDelegate:addCapabilitiesObserver:observer=0xfbdee4
2018-07-10 11:27:39.698 [  1] 5 InternetConnectionMonitor:startMonitoring
2018-07-10 11:27:39.706 [  3] 5 InternetConnectionMonitor:testConnection
2018-07-10 11:27:39.747 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.749 [  4] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.752 [  1] 9 SpeechSynthesizer:addObserver:observer=0xfd88fc
2018-07-10 11:27:39.756 [  1] 9 PlaybackRouter:create::called
2018-07-10 11:27:39.760 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.761 [  5] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.764 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.765 [  6] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.776 [  1] 9 AlertScheduler:setTimerForNextAlertLocked
2018-07-10 11:27:39.776 [  1] 9 AlertScheduler:executeScheduleNextAlertForRendering::no work to do.
2018-07-10 11:27:39.779 [  7] 1 AlertsCapabilityAgent:executeOnConnectionStatusChanged:status=DISCONNECTED,reason=ACL_CLIENT_REQUEST
2018-07-10 11:27:39.779 [  1] 5 NotificationsRenderer:create
2018-07-10 11:27:39.780 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.781 [  8] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.784 [  1] 5 NotificationsCapabilityAgent:init
2018-07-10 11:27:39.784 [  1] 5 NotificationsRenderer:addObserver
2018-07-10 11:27:39.791 [  9] 5 NotificationsCapabilityAgent:executeInit
2018-07-10 11:27:39.793 [  9] 5 NotificationsCapabilityAgent:notifyObservers:indicatorState=0
2018-07-10 11:27:39.796 [  9] 5 NotificationsCapabilityAgent:executeProvideState:sendToken=false,stateRequestToken=0
2018-07-10 11:27:39.803 [  9] 9 ContextManager:updateStateLocked:action=updatedState,namespace=Notifications,name=IndicatorState
2018-07-10 11:27:39.810 [  1] E Settings:initializeFailed:reason=SettingNotFoundinConfigFile
2018-07-10 11:27:39.811 [  1] E Settings:createFailed:reason=Initialization error.
2018-07-10 11:27:39.815 [  1] E DefaultClient:initializeFailed:reason=unableToCreateSettingsObject
2018-07-10 11:27:39.816 [  1] 5 DefaultClient:DirectiveSequencerShutdown
2018-07-10 11:27:39.817 [  1] 9 DirectiveSequencer:doShutdown
2018-07-10 11:27:39.820 [  1] 5 DefaultClient:AIPShutdown
2018-07-10 11:27:39.823 [  1] 5 DefaultClient:AudioPlayerShutdown
2018-07-10 11:27:39.825 [  1] 1 AudioPlayer:executeStop:playNextItem=false,m_currentActivity=IDLE
2018-07-10 11:27:39.826 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.827 [  5] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.829 [  1] 5 ContextManager:setStateProvider:action=removedStateProvider,namespace=AudioPlayer,name=PlaybackState
2018-07-10 11:27:39.829 [  1] 5 DefaultClient:SpeechSynthesizerShutdown
2018-07-10 11:27:39.829 [  1] 9 SpeechSynthesizer:doShutdown
2018-07-10 11:27:39.830 [  1] 9 MediaPlayer:setObserverCalled
2018-07-10 11:27:39.830 [  4] 0 MediaPlayer:handleSetObserverCalled
2018-07-10 11:27:39.832 [  1] 5 DefaultClient:AlertsShutdown
2018-07-10 11:27:39.833 [  1] 9 AlertsCapabilityAgent:releaseChannel
2018-07-10 11:27:39.834 [  1] 5 DefaultClient:PlaybackControllerShutdown
2018-07-10 11:27:39.835 [  1] 5 DefaultClient:MessageRouterShutdown.
2018-07-10 11:27:39.836 [  1] 5 DefaultClient:ConnectionManagerShutdown.
2018-07-10 11:27:39.837 [  1] 5 DefaultClient:CertifiedSenderShutdown.
2018-07-10 11:27:39.837 [  1] 5 DefaultClient:AudioActivityTrackerShutdown.
2018-07-10 11:27:39.838 [  1] 5 DefaultClient:PlaybackRouterShutdown.
2018-07-10 11:27:39.838 [  1] 5 DefaultClient:NotificationsShutdown.
2018-07-10 11:27:39.839 [  1] 5 NotificationsCapabilityAgent:doShutdown
2018-07-10 11:27:39.839 [  9] 5 NotificationsCapabilityAgent:executeShutdown:currentState=IDLE
2018-07-10 11:27:39.840 [  9] 5 NotificationsCapabilityAgent:executeSetState:previousState=IDLE,newState=SHUTDOWN
2018-07-10 11:27:39.841 [  1] 5 ContextManager:setStateProvider:action=removedStateProvider,namespace=Notifications,name=IndicatorState
2018-07-10 11:27:39.841 [  1] 5 DefaultClient:UserInactivityMonitorShutdown.
2018-07-10 11:27:39.848 [  a] 9 CertifiedSender:CertifiedSender worker thread done.  exiting mainloop.
2018-07-10 11:27:39.858 [  1] 5 InternetConnectionMonitor:stopMonitoring
2018-07-10 11:27:39.909 [  3] 0 InProcessAttachmentReader:readFailed:reason=SDS is closed
2018-07-10 11:27:39.909 [  3] 5 InternetConnectionMonitor:updateConnectionStatus:connected=true
2018-07-10 11:27:39.910 [  3] 5 InternetConnectionMonitor:notifyObserversLocked
2018-07-10 11:27:39.919 [  1] C SampleApplication:Failed to create default SDK client!
2018-07-10 11:27:39.920 [  1] C SampleApplication:Failed to initialize SampleApplication
2018-07-10 11:27:39.921 [  1] 5 CBLAuthDelegate:removeAuthObserver:observer=0xfd2f2c
2018-07-10 11:27:39.933 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:39.934 [  1] 9 MediaPlayer:resetPipeline
2018-07-10 11:27:39.937 [  1] 5 CBLAuthDelegate:stop
2018-07-10 11:27:40.288 [  2] 5 HttpPost:doPostSucceeded:code=200
2018-07-10 11:27:40.288 [  2] 5 CBLAuthDelegate:receiveCodePairResponse:code=200
2018-07-10 11:27:40.289 [  2] 5 CBLAuthDelegate:mapHTTPStatusToError:code=200,error=SUCCESS
2018-07-10 11:27:40.289 [  2] 5 CBLAuthDelegate:setAuthError:authError=SUCCESS
##################################
NOT YET AUTHORIZED
##################################
################################################################################################
To authorize, browse to: 'https://amazon.com/us/code' and enter the code: FMBB8T
################################################################################################
2018-07-10 11:27:40.312 [  1] 5 SQLiteCBLAuthDelegateStorage:~SQLiteCBLAuthDelegateStorage
2018-07-10 11:27:40.313 [  1] 5 SQLiteCBLAuthDelegateStorage:close
2018-07-10 11:27:40.314 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:40.315 [  1] 9 MediaPlayer:resetPipeline
2018-07-10 11:27:40.316 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:40.317 [  1] 9 MediaPlayer:resetPipeline
2018-07-10 11:27:40.317 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:40.318 [  1] 9 MediaPlayer:resetPipeline
2018-07-10 11:27:40.319 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:40.320 [  1] 9 MediaPlayer:resetPipeline
2018-07-10 11:27:40.320 [  1] 9 MediaPlayer:~MediaPlayerCalled
2018-07-10 11:27:40.321 [  1] 9 MediaPlayer:resetPipeline
Failed to create to SampleApplication !
`
Provide the steps to reproduce the issue, if applicable:
Following the step available here :
https://github.com/alexa/avs-device-sdk/wiki/Raspberry-Pi-Quick-Start-Guide
./SampleApp /home/pi/sdk-folder/sdk-build/Integration/AlexaClientSDKConfig.json /home/pi/sdk-folder/third-party/alexa-rpi/models debug9
Tell us about your environment:
What version of the AVS Device SDK are you using?
  1.8.0

Tell us what hardware you're using:

[ ] Desktop / Laptop
[X] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ X] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

HI @jultur,
Could you check if the content inside /home/pi/sdk-folder/sdk-build/Integration/AlexaClientSDKConfig.json?  It seems to be missing the "settings" container.
2018-07-10 11:27:39.810 [ 1] E Settings:initializeFailed:reason=SettingNotFoundinConfigFile
2018-07-10 11:27:39.811 [ 1] E Settings:createFailed:reason=Initialization error.
2018-07-10 11:27:39.815 [ 1] E DefaultClient:initializeFailed:reason=unableToCreateSettingsObject
2018-07-10 11:27:39.816 [ 1] 5 DefaultClient:DirectiveSequencerShutdown
Thanks,
Kenneth

I'm getting the same issue and I checked the AlexaClientSDKConfig.json file and it's taking the values automatically so everything is filled.
Also it's saying
SQLiteCBLAuthDelegateStorage:getRefreshTokenFailed:reason=stepResultWasNotRow
Has anyone gotten a solution for this?

I am closing this issue due to inactivity. Please feel free to re-open it if it has been closed in error.


Using library with MLX90393 product from Adafruit/ Burst Mode

Hello,
I'll preface this by saying I'm very new to Arduino and I2C communication, so I apologize if I'm missing something obvious.
I have a MLX90393 sensor bought from Adafruit Industries last spring. I was able to get measurements using their example code and library (which only supports Single Measurement Mode). Using Single Measurement Mode in a loop with no delay only gives about a 21 Hz sampling rate, but I see in the data sheet that over 400 Hz is possible when the sensor is set to Burst Mode. I tried to modify their library to set the sensor to Burst Mode, but this caused the initialization of the sensor through their begin() function to fail, and after not receiving answers on their forums I looked for other libraries with burst mode support and found this.
My problem right now is that I can't even get Single Measurement mode to work with this library. I've been trying to test/debug using the following code:
#include <Wire.h>
#include <Arduino.h>
#include <MLX90393.h>

MLX90393 sensor = MLX90393();

void setup() {
  Serial.begin(9600);

  while (!Serial) {
    delay(10);
  }

  Wire.begin();

  if (sensor.begin() == 0)
  {
    Serial.println("Sensor connected successfully.");
  }
  else
  {
    Serial.println("Error occured.");
  }
}

void loop() {
  MLX90393::txyz data;

  uint8_t status1 = sensor.readData(data);
  Serial.println(status1, 4);
  Serial.println(data.t, 4);
  Serial.println(data.x, 4);
  Serial.println(data.y, 4);
  Serial.println(data.z, 4);
  delay(1000);
}

I have no problems getting begin() to be successful (it always returns 0), but the displayed data is very obviously incorrect. For example, the temperature value is 459.792, and the magnetic field values are -0.15 or -0.24, and they don't change when I put a magnet near the sensor. I checked to see where there may be an error occuring in readData, and found that checkStatus(status1) (status for startMeasurement() returns 0 in the serial monitor, but checkStatus(status2) returns 3333 (status2 itself is 302). checkStatus(status1) | checkStatus(status2) also returns 3333.
I think the strangest thing is that there was one time where I actually got believable measurements (that responded to a magnet placed near sensor), but when I reuploaded the sketch it reverted back to the previous issue.
My questions would be, do my issues stem from using the Adafruit product, and if so what might I need to modify in the library to get it to work with this product? Also, is it even worth trying to get Burst Mode working with this sensor (I saw elsewhere that readings in Burst Mode often fail)?
Thank you for any help with this.

A couple of questions:

Can you provide a link to the Adafruit hardware you are using?

What Arduino hardware are you using?

Can you explain how the hardware is connected together?

An intermittent condition like you think you saw could be improper I2C pullup resistors. Just a wild guess.
I can't address the specific burst mode questions; never used it.

Thanks for getting back to me. To answer your questions:

Here's a link to the Adafruit MLX90393 product: https://learn.adafruit.com/mlx90393-wide-range-3-axis-magnetometer?view=all.

I've tried this with both an Arduino Nano and an Elegoo Uno R3.

In both cases, I'm using the A4/A5 pins and 5V pin on the board to connect to the SDA/SCL pins and Vin respectively on the sensor (along with connecting ground pins). I haven't been using the interrupt pin.

I also haven't been using my own pull-up resistors since the guide for the product says you should be able to directly connect the 5V pin of 5V microcontrollers to the sensor, and the SDA/SCL pins have their own internal pull-up resistors. The interrupt pin does need to be level shifted to 3.3V, but as I said I haven't used it yet. I haven't really had any problems with the readings with either of the microcontroller boards when using Adafruit's library and their example sketch, although it does fail if I try to upload the sketch after trying my above sketch for this library without unplugging/plugging the USB first.
I'm still trying to figure out why checkStatus2 (related to readMeasurement()) is returning the error I mentioned. Unless I'm misunderstanding something it's not outputting STATUS_ERROR. The time I did get the successful measurements checkStatus2 was 0 if I remember correctly.

Hey Ted,
I have had similar issues with the library but I have maybe isolated the issues a bit more. I have no trouble reading from the mlx 90393 with my code, but if I add a serial.print to the void loop I get the same symptoms that jeg100 is mentioning. Comment out the single line of code and reupload and it goes right back to correctly reading the data from the magnetometer. Does this help clear up what could be going on?

i ordered one of the adafruit boards to see what is going on. it may take a while to get here, but i can finally get to the bottom of the problems people have reported to me about this board.
also, can you provide a full example of a program that shows the erroneous behavior, like in a gist?
and what arduino board are you using?

Hey Ted,
Firstly, I really appreciate you getting back to me and wanting to help out!
I am using Adafruit’s breakout board and I have the same issues no matter what Arduino board I am running, I have tested it with at least two different brands of Arduino UNO and with an M5Stick-C and I have the same issues. I’ve attached the simple code I’m running and you will see in the loop I have a serial print “channel 1: “ commented out. If I run the script with that commented out it works fine, if that serial print is uncommented it won’t work. In my testing so far it looks like it doesn’t matter where that serial print is, even if it is behind an if statement condition that is never reached. Also, this is just the easiest example of what trips it up. For my project I am trying to run 6 MLXs through a multiplexor and switching the channel on the multiplexor causes the same issue. I know it isn’t the multiplexor because I have run it successfully with two other types of magnetometers. If you want code for that too I could run to my lab because I don’t have it one with that issue handy.
I’m not sure if I mentioned it previously but I have also tried using Adafruit’s library for the MLX and it works perfectly, I don’t have any issues like that. However, they don’t have functions for setting OSR, digital filter, or resolution which makes the magnetometer almost useless… I just say that in case it might help you troubleshoot if you can look at the documentation for their libraries knowing it works.
Did it say how long it may take your Adafruit MLX breakout board to come in? I could ship you one if you are willing to share your address since I have several of them. Would this be helpful? Please let me know if there is anything else you need or any other information that would be helpful.
I’m a physiologist with relatively little programming experience so It is difficult for me to fix issues like this if libraries aren’t working properly and it can keep my projects from moving forward, so again I really appreciate your generosity in helping out with this!
Thank you,
Jonathan D. Miller
From: Ted Yapo<EMAIL_ADDRESS>Sent: Friday, August 7, 2020 10:56 PM
To: tedyapo/arduino-MLX90393<EMAIL_ADDRESS>Cc: Miller, Jonathan Daniel<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [tedyapo/arduino-MLX90393] Using library with MLX90393 product from Adafruit/ Burst Mode (#45)
i ordered one of the adafruit boards to see what is going on. it may take a while to get here, but i can finally get to the bottom of the problems people have reported to me about this board.
also, can you provide a full example of a program that shows the erroneous behavior, like in a gist?
and what arduino board are you using?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftedyapo%2Farduino-MLX90393%2Fissues%2F45%23issuecomment-670819210&data=02|01|j980m842%40ku.edu|0c581739129c460e2d0708d83b4eeb8e|3c176536afe643f5b96636feabbe3c1a|0|0|637324557429765643&sdata=CEZyCKkiJvhpERfpmb2pNuIhK3AjOFY8csSCh%2Fyr7wU%3D&reserved=0, or unsubscribehttps://nam10.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKDDZ7XBUYIMB2BQSGULUK3R7TED3ANCNFSM4JEDOOGQ&data=02|01|j980m842%40ku.edu|0c581739129c460e2d0708d83b4eeb8e|3c176536afe643f5b96636feabbe3c1a|0|0|637324557429765643&sdata=q6%2BcwAWy76Ahr1frH1QP5GonnA3ebWH6Rtck1kwrTnQ%3D&reserved=0.
#include <Wire.h>
#include <MLX90393.h>
MLX90393 mlx;
MLX90393::txyz data; //Create a structure, called data, of four floats (t, x, y, and z)
int GAIN = 7;
int RES_X = 0;
int RES_Y = 0;
int RES_Z = 0;
int OSR = 2;
int DIG_FILT = 7;
void setup()
{
Serial.begin(9600);
Wire.begin();
byte status = mlx.begin(); //iic jumpers set
Serial.println(status);
Serial.print("Start status: 0x");
if(status < 0x10) Serial.print("0");
Serial.println(status, BIN);
mlx.setGainSel(GAIN);
mlx.setResolution(RES_X, RES_Y, RES_Z); //x, y, z
mlx.setOverSampling(OSR);
mlx.setDigitalFiltering(DIG_FILT);
}
void loop()
{
mlx.readData(data);
//Serial.println("channel1:  ");
Serial.println(String(millis())+ ","+ data.x+","+data.y+","+data.z);
delay(2);
}

I believe the original issue was fixed by 4b044b532be969a4464e58fa488459d97248596d. Please pull latest and verify.
I tried @jeg100 's code from https://github.com/tedyapo/arduino-MLX90393/issues/45#issue-511402221, and it fails before that commit, but works afterwards
it remains to be seen if @j980m842 's issue is fixed by the same commit. I'm closing this issue since @j980m842 has opened another one.

Hi, I have an application similar to the one of @j980m842 and therefore I need to read data in bust mode. From your previous comment, I did not understand perfectly the steps needed to get reliable data since there were multiple issues opened on the topic. Can someone resume it? Thank you


Docs 799 batch api errors limits

Fixes Docs issue#799
@dgaubert , can you review? I probably need to edit the limit error message as we recently discussed?  Let make sure we covered all the doc enhancements from the Platform channel discussion.
You can view the changes in the staging envn here:  http://799-batch-api-limits.docs.cartodb-staging.com/cartodb-platform/sql-api/sql-batch-api/

Sounds good, updated 'query` description in the job schema table, and added a new section about Retrieving Job Results with those tip details.  Let me know if the errors in the Usage Notes section are correct?

In order to provide a more realistic examples we should replace all SELECT * FROM user_dataset with UPDATE airports SET type = 'international'

Ok, I changed almost all the examples but got stuck at the multiquery section. You can edit this file directly for the example code.  (Also, we'll still need to show how to wrap the query in the retrieving job results section).

Looks good @dgaubert!  All I did was rename Usage Notes to Best Practices, and changed the heading level for the operations (create, read, list, etc.) from H2 to H3. So the H2 titles now appear was job schema, retrieving job results, and best practices. The operations will expand when you click on the job schema section.
Otherwise, docs look good to me if you approve?

👍 :shipit:


docremove useless doc description

Proposed changes
remove useless doc description
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Kitura Extra argument 'query' in call

Context and Description
I'm a student developping an iOS application with Kitura https://github.com/charelF/Radar
I am getting a strange error which seems to be a bug from Kitura, at least I can not possibly understand what I am doing wrong. I implemented the client.get method and it throws an error when I include the query parameter.
Screenshot

Code: https://github.com/charelF/Radar/blob/be0f0d3f517338370559ba15e466ff7b3bfe85af/Radar/Model/DataBase.swift#L77
Environment Details
I am using Xcode 11.1, macOS catalina, swift 5.1
Steps to Reproduce
Nothing particular, as you can see Xcode detects the line as not being correct even though it should be correct.
Lastly, I am sorry to bother you with this probably trivial issue but I searched a lot and could not find a solution as noone else seems to have this problem...

Closed and posted to KituraKit Repo https://github.com/IBM-Swift/KituraKit/issues/57


Docs for visible repos in menu possibly misleading

In the configuration file example available in the README, there is a comment above the repo list item that states:
# Which repos should appear in the menu?

However, I have noticed that if I push a new repo to soft without an entry in the repo list configuration it still shows up in the menu, even for non admin users.
In fact, the only way I have found to be possible hide a repo from the menu is with the following steps:

Create an entry for a new repo in the config
Mark the repo's private setting as true
Create a non-admin user which does not have the new repo in its collab-repos
ssh into soft with said user

Therefore, the documentation stating Which repos should appear in the menu? seems misleading, unless I am missing anything. A suggestion for an alternative doc (taking into account I have very little experience with the tool) could be something like:
# Customization of avaialble repos
# If a pushed repo is not included here, then it will be public and visible by default

Some extra details:

The version I'm working with is Soft Serve v0.1.3 (3aa9bf3)
When reproducing the issue my config for anon-access was no-access, and for allow-keyless was false

Finally, my sincere thanks to the maintainers for this awesome project :smile:

Good catch! The comment we have isn't accurate and it works as you describe. We should make it more clear.

This is resolved in v0.3.3. Soft Serve now doesn't show private repositories for unauthorized users.


Adding Getting Started Learning Resources

Few Learning Resources which I found handy.

I think it's better to move this content to wiki section.

@iamchathu But I usually find people not much going through the wiki section.
I feel community and contribution guideline should be added there.

Then we can link wiki rather than putting it here. Since this project needs to be continue even after SoC


Add XYPoint Field Mapper

Signed-off-by: Naveen Tatikonda<EMAIL_ADDRESS>Description
Field Mapper logic with new content type "xy_point" to index documents with field of type "xy_point" which is similar to geo_point. It indexes documents which contains value as either WKT, Object, String, Array format. It also indexes list of points in all these formats. It supports same parameters as geo_point like ignore_malformed, ignore_z_value, null_value.
Issues Resolved
https://github.com/opensearch-project/geospatial/issues/95
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

It is not triggering the CI check even though I tried to force push the branch.
Link to CI check that succeeded on my fork - https://github.com/naveentatikonda/geospatial/runs/7569583525?check_suite_focus=true

It is not triggering the CI check even though I tried to force push the branch. Link to CI check that succeeded on my fork - https://github.com/naveentatikonda/geospatial/runs/7569583525?check_suite_focus=true
But, still I'm not able to check the code coverage

Can you update PR pattern to feature/** in workflow

It is not triggering the CI check even though I tried to force push the branch. Link to CI check that succeeded on my fork - https://github.com/naveentatikonda/geospatial/runs/7569583525?check_suite_focus=true
But, still I'm not able to check the code coverage

Can you include pattern feature/** for pull_request/ push branches in workflow

Sure, I will give a try. But, when I worked on logstash-input plugin it actually worked by using '*'

It is not triggering the CI check even though I tried to force push the branch. Link to CI check that succeeded on my fork - https://github.com/naveentatikonda/geospatial/runs/7569583525?check_suite_focus=true
But, still I'm not able to check the code coverage

Can you include pattern feature/** for pull_request/ push branches in workflow

Thanks, it worked. I will raise a separate PR for that

Codecov Report

:exclamation: No coverage uploaded for pull request base (feature/xypoint@1b149d5). Click here to learn what that means.
The diff coverage is n/a.

@@                Coverage Diff                 @@
##             feature/xypoint     #100   +/-   ##
==================================================
  Coverage                   ?   84.82%
  Complexity                 ?      296
==================================================
  Files                      ?       42
  Lines                      ?      995
  Branches                   ?       79
==================================================
  Hits                       ?      844
  Misses                     ?      118
  Partials                   ?       33

Help us with your feedback. Take ten seconds to tell us how you rate us.

It is not triggering the CI check even though I tried to force push the branch. Link to CI check that succeeded on my fork - https://github.com/naveentatikonda/geospatial/runs/7569583525?check_suite_focus=true
But, still I'm not able to check the code coverage

Can you include pattern feature/** for pull_request/ push branches in workflow

Sure, I will give a try. But, when I worked on logstash-input plugin it actually worked by using *

I doubt that. We had same problem in k-nn. We fixed it by /* .

I think we need to merge this PR as it is still not triggering CI and other checks

Dod

I think we need to merge this PR as it is still not triggering CI and other checks

Did you do pushed the updated CI triggering code to this branch before raising the PR?

Dod

I think we need to merge this PR as it is still not triggering CI and other checks

Did you do pushed the updated CI triggering code to this branch before raising the PR?

No. I can't because I created this branch from feature/xypoint. But, we merged that code to main branch. So, I think we need to merge that PR to feature/xypoint and then rebase this branch with feature/xypoint

Dod

I think we need to merge this PR as it is still not triggering CI and other checks

Did you do pushed the updated CI triggering code to this branch before raising the PR?

No. I can't because I created this branch from feature/xypoint. But, we merged that code to main branch. So, I think we need to merge that PR to feature/xypoint and then rebase this branch with feature/xypoint

Yes this is what is required to be done so that the checks can run.


Build extended icons and material3 for all targets

Fixes https://github.com/JetBrains/compose-jb/issues/2106
androidx PR: https://github.com/JetBrains/androidx/pull/244
Also added a task on CI. Successful build: https://teamcity.jetbrains.com/buildConfiguration/JetBrainsPublicProjects_Compose_Publish_2_All_2/3875732?hideProblemsFromDependencies=false&hideTestsFromDependencies=false&expandBuildChangesSection=true

I tried out build 0.0.0-igordmn-patch-3-dev722 Android is working fine. But desktop is crashing when I try to display one of the extended icons. For example:
Exception in thread "AWT-EventQueue-0" java.lang.NoSuchMethodError: 'void androidx.compose.ui.graphics.vector.ImageVector$Builder.<init>(java.lang.String, float, float, float, float, long, int, boolean, int, kotlin.jvm.internal.DefaultConstructorMarker)'
	at androidx.compose.material.icons.filled.PauseKt.getPause(Pause.kt:51)

I tried out build 0.0.0-igordmn-patch-3-dev722 Android is working fine. But desktop is crashing when I try to display one of the extended icons. For example:

I just checked, and it works fine if we use Icons.Filled.Pause in commonMain, and run desktop target.
Could you provide a reproducer?

material-icons-extended-js is still missing from https://maven.pkg.jetbrains.space/public/p/compose/dev/org/jetbrains/compose/material/


Added compatibility with symfony 5

Thanks, I need compatibility with symfony 5, other changes is not necessary.


Mccalluc/no more submodule

Redo #136 against devel.

@shirey please let me know once you approve and merge this PR into devel branch. Then I can redeploy on DEV. And if you think it can be part of the release, I'll redeploy on STAGE as well.


decimal part when numbers are aggregated

When we map numerical dimensions to labels, we could have numbers with a very long decimal part due to the chosen aggregations (see image below). We could hardcode a rounding function, but sometimes users would like to keep the decimal part.
I have no idea on how to solve the issue

similar issue in sankey and alluvial diagrams (see #134 for details)


ngFor grid-layout adds new array items to the bottom, even if unshift is used

If a new array item is added to the array using "unshift", I'd expect the item to appear at the top of an ngFor grid-layout list. However, the item is added to the bottom. As a workaround currently, I duplicate each entry in the list and add it to a new array. This handling of course has some negative performance impact.
Is there another way to get new items appear on top? Or is it a bug?

@moritzvieli not exactly workaround but I think a better approach would be to use ListView or RadListView to visualize an array of item templates. This way you will not only have better control on how to add/remove source items but you will also have the goodies of those controls such as virtualization and recycling which is very important if your templates are heavy. The performance optimization would be great compared to using the Angular structural directive. Great blog post on that matter can be found here

@NickIliev Thanks for your fast reply! I switched from ListView to this approach in NS 2.5 because it didn't work with my layout (pull-refresh-combination). But I'll check if I can reproduce this problem in NS 3.


[Some] HTML entity in template gets re-escaped [incorrectly] by vue.js

Version
2.5.16
Reproduction link
https://jsfiddle.net/50wL7mdz/270981/
Steps to reproduce
In the html template, provide an html "title" attribute that contains an HTML entity using ascii codes.
What is expected?
Vue should not mess with or re-escape/entify the literal ampersand character in the HTML template.
What is actually happening?
Vue sees the ampersand character in the html attribute and changes it to the HTML amp entity - resulting in a rendered ampersand character.

Trying to put a newline character into a title attribute within my template file.
Seems like Vue has some recognized entities (the named ones perhaps) which it leaves alone, but entities using ASCII codes get corrupted.
Entities using ASCII codes seem to behave fine when placed as content inside a tag - but not when placed into an HTML attribute.

I guess its more accurate to say Unicode vs ascii codes above.

HTML escapes are different from js escapes. Using that escape in js just doesn't work: https://jsfiddle.net/w85Lvwa6/

Check out this example - the template is not a declared string in JS.  It is just raw chars pulled from a script tag template.
https://jsfiddle.net/50wL7mdz/271032/
Note the inconsistency by Vue in handling the good vs bad span title.

@posva Are you seeing these comments?


Error encoutered when a custom array value is registered in $_ENV superglobal

Q
A

Bug?
yes

New Feature?
no

Version Used
v1.0.1

Actual Behavior
PHP-FFMpeg encounters an unexpected error when a custom array value is registered in super globals $_ENV. This custom array value is parsed and generates an error.
Expected Behavior
It should not produce an error.
Steps to Reproduce
Register an array in $_ENV , for example
 $_ENV['my_custom_value'] = array('foo');
Then when the function start() in Process.php (ligne 293) iterates over the local variable $env in the for loop an error occurs.

In fact the bug lies within symfony/process package, my bad


Control what gets the state value

I would like to get a bit more control over which elements in the markup recieve the state values. This would help when building up elements of more than just a single element. and ensure the state value is applied exactly where I want. It would also help that each item of the states could have unique ID's in the page.
eg could be, not sure how the parsing would work though.
/**
  * @name Input Number (Keyboard)
  *
  * @description An input that shows a pure numeric keyboard on mobile devices.
  *
  * @state :hover - Hover
  * @state :focus - Focus
  * @state :disabled - Disabled
  *
  * @markup
  * <label for="input-password-@state" class="label">Input Number</label>
  * <input id="input-password-@state" class="input input--number @state" type="text" pattern="\d*" />
  */

+1


Add tryParse to reuse existing optional init

In cases like RawRepresentable, the logic for parsing is already present in the optional init function. I am using following tryParse helper (copied over from map). Would this be considered for inclusion in the library?
import Parsing

extension Parser {
  /// Returns a parser that uses the given closure to perform final parsing of the output of this parser.
  ///
  /// - Parameter transform: A closure that transforms values of this parser's output. Returning nil if transform fails.
  /// - Returns: A parser of transformed outputs.
  @inlinable
  public func tryParse<NewOutput>(
    _ parseFn: @escaping (Output) -> NewOutput?
  ) -> Parsers.TryParse<Self, NewOutput> {
    .init(upstream: self, parseFn: parseFn)
  }
}

extension Parsers {
  /// A parser that parses the output of another parser with a given closure.
  public struct TryParse<Upstream, Output>: Parser where Upstream: Parser {
    /// The parser from which this parser receives output.
    public let upstream: Upstream

    /// The closure that parses output from the upstream parser.
    public let parseFn: (Upstream.Output) -> Output?

    @inlinable
    public init(upstream: Upstream, parseFn: @escaping (Upstream.Output) -> Output?) {
      self.upstream = upstream
      self.parseFn = parseFn
    }

    @inlinable
    @inline(__always)
    public func parse(_ input: inout Upstream.Input) -> Output? {
        self.upstream.parse(&input).flatMap(self.parseFn)
    }
  }
}

Hi @surajbarkale! Thanks for starting a discussion. This parser looks a lot like Parser.compactMap:
https://github.com/pointfreeco/swift-parsing/blob/625c1f23eb4902b5742a94e7d7a47d068d2bf68d/Sources/Parsing/Parsers/CompactMap.swift
I think them main difference is that compactMap will backtrack if parsing fails. Does compactMap work for you in your use case?

I explicitly do not want backtracking behaviour. Consider a use case where one of the columns in CSV file is mapped to an enum. When parsing a line, I want the whole line to fail if the enum column contains invalid value. This can be achieved by using .orElse chaining in parser. However, then I have to remember to change the parser when enum changes.
The tryParse allows me to say extract this string and parse using this external function to get a real data type. Another use case is parsing delimited UUID's using UUID.init?(uuidString:) instead of writing explicit UUID parser.

@surajbarkale I'm not sure I understand. Can you provide a working code sample that shows the problem? A test case that shows tryParse succeeding where compactMap fails would help me better understand.

After running some examples I realized that the end result of compactMap is the same. So this extension is not needed.


Status updates occurring close together result in an exeception

We have a custom resource with several operands.  The current code will return UpdateControl.updateStatusSubResource on every call to ResourceController.createOrUpdateResource.  When several operand updates are processed close together we sometimes see:
ERROR [io.jav.ope.pro.EventDispatcher] Error during event processing ExecutionScope
...
io.fabric8.kubernetes.client.KubernetesClientException: Failure executing: PUT
...
the object has been modified; please apply your changes to the latest version and try again

I need to run in debug more to check if the operator sdk is retrying this - if it is retrying, then can this be logged at debug.  If it's not, then is this something that the operator or fabric8 should be handling?

It does appear in debug that the retry handling is kicking in - the logic will look for any new events, and run the update again.  So this does appear to be benign.  @metacosm are you open to a change that would lower the amount of logging related to this exception?

Yep.

I've submitted a pr to change this to an info log without a stacktrace.

Can we close this issue then? Or are there still things that need to be investigated?

No, it seems that this is how it's expected to work.  The only alternative/addition would be to add locking per resource being passed to createOrUpate such that you could serialize it's processing - at least for the single operator case that would typically eliminate these conflicts.

What do you think, @csviri?

Hi @metacosm and @shawkins
Not sure if I understand the situation correctly: We have already serialized processing in terms of operator for a custom resource. If I understand correctly what happens is that an other process(es) changes the CR frequently, that lead to an optimistick locking error.
By locking I guess you mean pessimistic locking. I'm not aware that kubernetes supports such thing. I think what can be done however is not do updates on status subresource using optimistic locking, since the controller is anyways the only process that should change status.
See:
https://github.com/java-operator-sdk/java-operator-sdk/blob/b4c6483ab3c4ddffcdf030681d03c4f3258556f8/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/EventDispatcher.java#L203-L207
I think this needs to be changed in fabric8 client. @metacosm

Hi @metacosm and @shawkins
Not sure if I understand the situation correctly: We have already serialized processing in terms of operator for a custom resource. If I understand correctly what happens is that an other process(es) changes the CR frequently, that lead to an optimistick locking error.
By locking I guess you mean pessimistic locking. I'm not aware that kubernetes supports such thing. I think what can be done however is not do updates on status subresource using optimistic locking, since the controller is anyways the only process that should change status.
See:
https://github.com/java-operator-sdk/java-operator-sdk/blob/b4c6483ab3c4ddffcdf030681d03c4f3258556f8/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/EventDispatcher.java#L203-L207
I think this needs to be changed in fabric8 client. @metacosm

@shawkins just clarify 1 thing what do you mean by operand. Its a resource that is created by the controller and watched by an event source ?
Regarding to this comment:
https://github.com/java-operator-sdk/java-operator-sdk/blob/b4c6483ab3c4ddffcdf030681d03c4f3258556f8/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/DefaultEventHandler.java#L211
This is an edge case, with event sources where as described if we do an update and execute the controller  again because there were already events from different event source than custom resource event source, the cached custom resource could be out of the date (since we cache when it arrvies on a change from a watch). But this code should solved this problem in all cases. So if by actor you mean an event source, then yes this situation only apply when there are additional event sources.

just clarify 1 thing what do you mean by operand. Its a resource that is created by the controller and watched by an event source ?

That is correct.  Our cr that the operator is managing creates another custom resource, configmaps, secrets, deployments, etc.  We watch those resources via an informer and register them as event sources.
For a given resource R we see logs similar to:
2021-04-24 21:43:12,309 DEBUG [org.bf2.ope.eve.ResourceEventSource] Update event received for io.fabric8.kubernetes.api.model.apps.Deployment x for resource R
...
2021-04-24 21:43:12,433 ERROR [io.jav.ope.pro.EventDispatcher] Error during event processing ExecutionScope "with different operand events for R that the previous"

I checked in on this, and I agree with the opinion that you can't make the expectation that other processes won't be changing status. It's just how Kubernetes works. So optimistic locking with just an info log is the right solution imho.

It's not restricted in any ways, it's just how it ment that only the controller updates the status.  But did not see this anywhere expicitly, (maybe will do a round around this on K8S Slack just out of curiosity). But if you have some nice reference about this pls share @adam-sandor.
The thing is that optimistic locking it won't help basically in any scenarios regaring to this, since it is anyways overriden by the controller on the next run, and its always a result of a run. So while it makes sense to to optimictick locking on specs it does not make to status. (Note that status is a subresource, overrinding it does not affect other parts of resource)
But I think this is a very minor detail, it's more interested from the pholosphical aspect of k8s.

As best as I can tell we are observing this with

Custom Resource CR Revision 1 is in cache
ExecutionConsumer to update Custom Resource CR Revision 1 is queued
CR is updated to Revision 2
ExecutionConsumer runs and fails due to the conflict

This typically doesn't happen because there is little time between 2 and 4, but in some scenarios, such as rolling upgrades or mass deletions we see a flood of events such that some of the ExecutionConsumers have been queued with the stale cr.
It seems like a way to narrow this case even further would be to a cache get to refresh, or immediately reject, the ExecutionScope at the start of the run https://github.com/java-operator-sdk/java-operator-sdk/blob/112a8268155a9f6262e1238b592e9813b09e1557/operator-framework-core/src/main/java/io/javaoperatorsdk/operator/processing/ExecutionConsumer.java#L25

Do you think we should re-open the fabric8 issue to support PATCH?

@metacosm that issue was because of the finalizers mainly, since adding finalizer is always an "additive" operation we could do it by patching ("just add this finalizer to the finalizer list") request. But to be honest, I'm also quite happy with the current solution, maybe that would be a little bit nicer and straighforward.
The patch also increased the resource version, so we have to think about that also after the request.
I guess we can create an issue on this if you think it's worth to spend time on this part, but I'm fine with the current approach also. @metacosm

I will create a issue about the sub-resource update, I guess the k8s api does not require optimisitc locking there, (will verify). So we can disuss the pros and const regarding to that under the issue.

I guess the k8s api does not require optimisitc locking there, (will verify)

This seems to require a resource version per the api server.  I've captured https://github.com/fabric8io/kubernetes-client/issues/3066 for fabric8 to consider this.


AdaptedAttention doesn't work with model loaded in 8bit or 4bit

System Info
Python 3.8.16
peft 0.4.0.dev0
transformers 4.29.2
accelerate 0.20.3
Who can help?
No response
Information

[ ] The official example scripts
[X] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder
[X] My own task or dataset (give details below)

Reproduction
import transformers
from peft import AdaptionPromptConfig, get_peft_model, prepare_model_for_kbit_training

model = transformers.LlamaForCausalLM.from_pretrained(
        "decapoda-research/llama-7b-hf",
        load_in_8bit=True,
        torch_dtype=torch.float16,
        device_map="auto",
)

config = AdaptionPromptConfig(
    adapter_len=10,
    adapter_layers=30,
    task_type="CAUSAL_LM",
)
model = get_peft_model(model, config)

Expected behavior
The above code will throw at the this line, complaining that Char type tensor can't do normal_():
https://github.com/huggingface/peft/blob/08cb3dde577747f6ca6638c884fd66fd16cf2e9d/src/peft/tuners/adaption_prompt.py#L297-L299
This is because, unlike LoraModel, AdaptationPromptModel is not aware of the possibility that the base model can be load in 8bit so that model.q_proj.weight.dtype is now a torch.int8. I manually change it to torch.float32 and it works fine.
In summary, AdaptationPromptModel should check loaded_in_8bit = getattr(self.model, "is_loaded_in_8bit", False) then set the correct float type. Somewhere a compute_dtype or similar should be passed in.

Hi @nuance1979
Thanks for the PR and the clean reproducer. https://github.com/huggingface/peft/pull/604 should add the bitsandbytes (4bit & 8bit) support of AdaptionPrompt


How to release stuck lock

I use redis.Lock()  like:
l = settings.REDIS.lock(name, blocking_timeout=expires)

if not l.acquire():
      # Expired
      return

try:
    return self.task_body(*args, **kwargs)
finally:
    if l:
        l.release()

Once (I have't found the reason), the thread escaped without releasing it. And now I have a stuck lock. If I create lock object like above and call l.acquire(False) it writes False (lock is acquired). If I call l.release() it raises not acquired exception.
Questions:

Is the any way to limit lock acquire time? And release it if time has passed?
How to release stuck lock? I can change the name, but it's strange solution to my mind...

I was just reading the source code to understand how the lock mechanism works and I saw this issue.

If you want a lock to be automatically released after some time, you can use the timeout argument (when calling lock());
From the source code there's an underlying validation for an internal token. So if you can get into Redis, you should be able to clear the lock key manually.

@JokerQyou thanks, I've already found it out from source too =) Forgot about this issue opened, closing.


Add weak reference boxing helpers to C++/WinRT

Here's an example:
https://gist.github.com/kennykerr/9f7ad000c8021cbe2d9f32727b9de901
The trouble with this example is that implementing IWeakReference as a classic COM interface requires COM interop support, which isn't always enabled. Fixing that may require some work in implements to make it a bit more flexible in this regard and is thus substantially more work. I would also prefer a more general solution that let developers "box" non-COM/WinRT types for transport through a WinRT API, but I haven't yet found a way to do that in a generic (and type safe) manner.

I think the sample above will suffice. It's not clear that this is a generally useful scenario.


Param scope and body scope as sibling scopes

This change set implements param scope and the body scope as parallel
scopes. The previous logic of having param scope as an inner scope to the
body scope does not work well in eval cases. This changelist only
addresses function definitions in the param scope. I will be working on
the eval cases next. This works in interpreter mode right now. Backend
changes are not done yet.
Two new bytecodes were added. BeginBodyScope marks the beginning of the
body scope where a new closure object is created for the body scope. Until
then closure created for the param scope is used. After BeginBodyScope the
param scope one is not used other than copying the initial value for the
body scope symbols from the param scope symbols. This required the
creation of another new bytecode LdParamSlot. This also include some
refactoring required for enabling these.

@pleath @akroshg @tcare @abchatra Can you please take a look?

@dotnet-bot test this please

Can you show me where the interpreter verifies the index when it executes [Profiled]LdParamSlot?

The LdParamSlot uses the same method used by LdLocalSlot. OP_LdSlot() has the index check.

Okay. Thanks.
--Paul
From: aneeshdk<EMAIL_ADDRESS>Sent: Tuesday, March 1, 2016 2:38 PM
To: Microsoft/ChakraCore<EMAIL_ADDRESS>Cc: Paul Leathers<EMAIL_ADDRESS>Subject: Re: [ChakraCore] Param scope and body scope as sibling scopes (#384)
The LdParamSlot uses the same method used by LdLocalSlot. OP_LdSlot() has the index check.
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/ChakraCore/pull/384#issuecomment-190940268.


chore: update staging folder links

resolve https://github.com/devstream-io/devstream/issues/1228

@Arhell
Hi, thanks for contributing to the community.
You may want to note that as an open source project , the commit information needs to follow https://www.conventionalcommits.org/en/v1.0.0/.
We also mention it in our contributor development documentation: https://github.com/devstream-io/devstream/blob/main/docs/development/git-workflow/commit-messages.md
Please follow this to modify your commit information. For example chore: update staging folder links
Thanks.

@Arhell
Hi, thanks for contributing to the community.
You may want to note that as an open source project , the commit information needs to follow https://www.conventionalcommits.org/en/v1.0.0/.
We also mention it in our contributor development documentation: https://github.com/devstream-io/devstream/blob/main/docs/development/git-workflow/commit-messages.md
Please follow this to modify your commit information. For example "chore: update staging folder links" Thanks.

Done, thanks


[oadp-1.1] upgrade x/net to 0.17.0

Note CI config to enable unit-test: https://github.com/openshift/release/pull/44529


Can't make content bold/italic/underlined while creating article

User should be able to select the text and make it bold/italic/underlined in article view.

It will require a re-work on the editor.
Currently, there is a limited feature in the editor.
A user still can write markdown content(As far as the feature is not included).
These content will be rendered as markdown.


[question] why having a copy of ~/.vim/autoload/plug.vim ?

Is there a reason for adding a copy of the ~/.vim/autoload/plug.vim, instead of using the command in the file :
curl -fLo ~/.vim/autoload/plug.vim --create-dirs \
     https://raw.githubusercontent.com/junegunn/vim-plug/master/plug.vim

no particular reason, just to save a copy at local and do not depend on curl which is not installed by default on ubuntu


Clean up and simplify CMakeLists.txt

I think it is done, but I will verify

Improved even more, now it's done


Plot and automate performance tests

[x] Create Python plotter performance tests
[x] Allow performance tests to be called from command line
[x] Invoke performance tests from Python plotter script

To run it, you must be in the project root directory. Run:
python3 test/plot/plot.py

The plots should appear in the root directory


Check for 404 before parsing coveralls response

For cases like invalid module name coveralls returns a static html and a 404 status code. Add test and implementation for it

PR for it: https://github.com/nodezoo/nodezoo-coveralls/pull/8


Baseten instructions

Hey, sorry but I'm failing to understand how to setup baseten for use with riffusion. Is it possible to add some steps?

this worked for me
create .env.local file in the inference folder
add RIFFUSION_FLASK_URL=http://<IP_ADDRESS>:3013/run_inference/ to the file.

Issue was definitely on the wrong repo. The point here is to use baseten and not this repo :D


Fix grammatical errors in Functions.md

Fixed various grammatical errors to improve readability of Functions.md documentation.

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

Thank you for contributing to the TypeScript Handbook!
We are currently working through a large backlog of open PRs. As part of this, we're asking authors of PRs with active merge conflicts to assess if their changes are still needed, and if so, open a new PR.
For transparency, we are also working on a full-scale reimagining of the Handbook (see the TypeScript Roadmap), meaning we are largely closing down work on the current version of the Handbook. Typo fixes are always appreciated, but any new content should likely wait until the new Handbook is available so that it can be more naturally adapted to that format.


test: deflaking settings test more

Signed-off-by: Alyssa Wilk<EMAIL_ADDRESS>
Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/wait on CI

PTAL!


Add validation function to declaration options

Fixes #1316
Overview:

New validate property for declaration options
New tests for the new properties

Please pay attention to line 279 in src/lib/utils/options/declaration.ts. I removed one of the signatures. This enables TS's type refinement and thence no more type assertion is required when accessing the option. I don't know if this breaks anything else.

Travis is entirely too slow... will merge as soon as it passes, want to pull it into 0.20.


It is failure to find back password. [REPLACEMENT ISSUE]

The original issue
Id: 1574
Title: It is failure to find back password.

could not be created.
This is a dummy issue, replacing the original one. It contains everything but the original issue description. In case the gitlab repository is still existing, visit the following link to show the original issue:
TODO

In GitLab by @wknet123 on Nov 8, 2019, 07:56
Misconfigured in Email settings in board.cfg.
NOTE: Avoid setting as Inspur mail service, because it needs certificate provided.

In GitLab by @luotian-github on Nov 8, 2019, 07:56
I don't know how to configure email identity. Please check the configuration and give me some feedback.

In GitLab by @wknet123 on Nov 8, 2019, 07:56
The log shows "x509: failed to load system roots and no roots provided"
Maybe cause by incorrect environment setup.
please refer this post: https://github.com/golang/go/issues/6391.

In GitLab by @wknet123 on Nov 8, 2019, 07:56
SOLUTION:

Because alpine Docker image is missing 'ca-certificates.crt' file under /etc/ssl/certs/, just add it to there when building image.
Please make sure you have opened the CUSTOMER AUTHORITY in your email service.

In GitLab by @wknet123 on Nov 8, 2019, 07:56
Fixed by PR #1596

In GitLab by @cuiaq123 on Nov 8, 2019, 07:56
failed in
1.0-767-g9cca511

In GitLab by @wknet123 on Nov 8, 2019, 07:56
Continuous update with PR #1606

In GitLab by @wknet123 on Nov 8, 2019, 07:56
Please configure "email from" at board.cfg as
email_from =<EMAIL_ADDRESS>Seems not support pattern with<EMAIL_ADDRESS>

In GitLab by @wknet123 on Nov 8, 2019, 07:56
Please confirm it with next update package.

In GitLab by @cuiaq123 on Nov 8, 2019, 07:56
still failed in
1.0-780-gb2f8e94

In GitLab by @wknet123 on Nov 8, 2019, 07:56
This caused by the server was blocked by email service provider. Please take a proper server environment to support sending email service.

In GitLab by @guyingyan on Nov 8, 2019, 07:56
qq email can be used
163 email, register before 2012 is ok

In GitLab by @guyingyan on Nov 8, 2019, 07:56
sina is ok

In GitLab by @cuiaq123 on Nov 8, 2019, 07:56
resolved in
1.0-784-g00515bf.board.cfg use sina email

In GitLab by @sokril on Nov 8, 2019, 08:25
closed

In GitLab by @sokril on Nov 8, 2019, 08:25
assigned to @cuiaq123


Fully document installation options

https://charm.readthedocs.io/en/latest/quickstart.html#installing-charm says, at the end of section 1.1

For advanced compilation options, please see Section 2.6.1 of the manual.

That section, in turn, says

Details on all the available alternatives for each of the above mentioned parameters can be found by invoking ./build --help.

And, when I try that, after installing CMake after getting the latest from github ( e4a6e7b3f1ae0b):
$ ./build --help
Usage: ./buildcmake <target> <triplet> [other options]
  See the Charm++ manual at
  https://charm.rtfd.io/en/latest/charm++/manual.html#installing-charm
  for full instructions.

At which point we have a circular dependency!
And I am none the wiser about either what the complete set of options are, or of what they mean.  By "options" I include the actual meaning of the triplets.  I don't even know if I have to choose between shared memory and networked communication, or if charm can use both, as appropriate.

Thanks for the report @RossBoylan! I've prepared a patch that restores --help in the cmake build system: #2927. Regarding the meaning of the triplets, I think the most comprehensive documentation is in our README file: https://github.com/UIUC-PPL/charm/blob/master/README.md . In particular, you can combine shared memory and networked communication with the smp option.