Multiple AlertConditions proposal

Ref: #218
This proposal introduces the capability to compose AlertConditions in an AlertPolicy using and/or logic. This should allow users to define complex conditions composition, which in turn will enable using better SRE practices like multi-window burn-rate alerting
The change is designed to be backward compatible with the current specification.

Coming back to this I think that at this stage having such logic would be an overkill for the spec implementations and thus spec itself. The AND/OR logic can be still achieved with multiple Alert Policies (OR between them) and multiple conditions for a single policy (AND between them).
Maybe we could just start with removing the limit for 1 condition per policy in v2alpha1?

So taking the initial example in this PR:
expr: (
        job:slo_errors_per_request:ratio_rate1h{job="myjob"} > (14.4*0.001)
      and
        job:slo_errors_per_request:ratio_rate5m{job="myjob"} > (14.4*0.001)
      )
    or
      (
        job:slo_errors_per_request:ratio_rate6h{job="myjob"} > (6*0.001)
      and
        job:slo_errors_per_request:ratio_rate30m{job="myjob"} > (6*0.001)
      )
severity: page

We would translate it like this? This now assumes that multiple conditions are AND between them, which is not the case in the current spec (but is documented to be maximum 1 at the moment, so in theory it should not be conflicting with current tools).
apiVersion: openslo/v1
kind: AlertPolicy
metadata:
  name: High-Burn-Rate-ShortWindow
  displayName: High burn rate
spec:
  conditions:
        - kind: AlertCondition
          spec:
            condition:
              kind: burnrate
              threshold: 14.4
              lookbackWindow: -1h
              alertAfter: 5m
        - kind: AlertCondition
          spec:
            condition:
              kind: burnrate
              threshold: 14.4
              lookbackWindow: -1h
              alertAfter: 1m
---
apiVersion: openslo/v1
kind: AlertPolicy
metadata:
  name: High-Burn-Rate-LongWindow
  displayName: High burn rate
spec:
  conditions:
        - kind: AlertCondition
          spec:
            condition:
              kind: burnrate
              threshold: 6
              lookbackWindow: -6h
              alertAfter: 5m
        - kind: AlertCondition
          spec:
            condition:
              kind: burnrate
              threshold: 6
              lookbackWindow: -30m
              alertAfter: 5m

I do believe this makes sense, and the implementation in go (and most languages) would much simpler than the initial proposal. The only problem I could see is that, if you link both Alert policies to a notification, it would send 2 notifications for the same problem in an OR condition. Bu I think working on that problem will be easier than trying to fit a full and/or logic as before

Do you have some proposal for how we would deal with multiple AlertPolicies bound to a single SLO?
I.e. how would we make that OR happen? 😁

Anyway, that's kind of irrelevant in form of this PR and as @nieomylnieja mentioned, just allowing multiple conditions is already an improvement.
LGTM

One idea that we have as we are discussing this at work with @hy3n4 is that we could make a field in the AlertPolicy that would define the operation we are looking to create from some template in the order of the conditions or conditionRefs as defined in the policy.
Something like:
(0 and 1) or (2 and 3)

... using zero-based indexing (we could do index + 1, idc)
What do you think?

My original idea was to move the alert conditions to a separate files so they could be referenced in the alert policy. Allowing for reuse of the alert conditions when needed. And then in the policy itself support a limited set in a similar manner of operations.
But never though you would want to multiple groups of group of conditions. Interesting.


fix: Incorrect check against Content-Type instead of content-type in httpRequest #3992

Fixes Issue Incorrect check against Content-Type instead of content-type in httpRequest #3992

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 84.543%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
19660

Relevant Lines:
22028

💛  - Coveralls


Docker Container Lost Settings after Restart or container

Describe the bug
When I stop the docker container and start again all settings gone...
To Reproduce
Steps to reproduce the behavior:
Run the docker container, configure the connections, stop and start again the container
Expected behavior
A clear and concise description of what you expected to happen.
Keep the settings
Screenshots
If applicable, add screenshots and gifs to help explain your problem.
Not applicable
System information:
In which OS are you running? GIve all details of where you have deployed the application. Make sure to give detailed version information.
Docker using the image provided in the wiki
Additional context
Add any other context about the problem here.

I have this issue too,
I think we should add disk config to docker container.
But I don't want add whole /app directory to disk

I have fixed adding /app to my local FS (Insede there are config and logs files only)

It's wrong to set whole /app directory sync to your disk in production
I find better solotion.
I read the code and findout if you set MONGO_CON_STR env variable then your data is persistant in Mongo DB.

I guess 'CONFIG_ENV_VAR_NAME' is not respected at all


NativeModules does not resolve - [React Native v.0.5.0]

Receiving Invariant Violation that prevents application from loading:
Unable to resolve module NativeModules from /Users/.../node_modules/react-native-device-uuid/DeviceUUID.ios.js

React Native v. 0.7.1
Edit
Does it have something to do with requiring react-native instead of referencing private modules? Something like this perhaps (adapted from https://github.com/johanneslumpe/react-native-keyboardevents/commit/5e571914903c83c7faf1316c26fa6f94f50d7a19)?:
var {
  invariant,
  NativeModules: {
    DeviceUUID,
  },
} = require('react-native');

Appears like this was unrelated to uuid. Closing for now, but may revisit if it turns out otherwise! Thanks!


Create section for Helm101

I noticed that there are no tutorials on Helm. Has this been worked on before, and is a section on Helm appropriate here? If so, I can contribute.

@Phantom-Intruder You are most welcome to contribute.

#82 PR for this

Thanks. I have merged it. Please add a link at the end of each page so that it becomes easy for readers.

Keep contributing,.

Added links and also added parts related to chart hooks, repositories, and chart testing

Thanks @Phantom-Intruder for all your great work.


Problem with initialDate

Hi man! I try since 2 days set initialDate, but i think is problem with this instead, the current local date is set.
Maybe i make something wrong, but i tried a lot of attempts.

Hi, @ShuffleCodes
Damn! I totaly missed issue. Will check it out.
If you still there can you please confirm issue is that you can't set initialDate?


Use MDoc to automate latest version in readme

Uses MDoc variables to put the latest version in the readme.md. If a new version is released, but the version is still outdated in the readme, a failing build will tell you 😊

@wijtserekker I've also added a GH workflow to update the docs, so you don't have to do it manually any more. It can just be run from the actions (after a new release, for example)


langchain v0.2.17

Checklist

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

Reviewer information
Link to pyproject.toml at v0.2.17: https://github.com/langchain-ai/langchain/blob/langchain%3D%3D0.2.17/libs/langchain/pyproject.toml
Git diff output is included below.
Note that orjson is not required as it is an optional group dependency.
% git diff langchain==0.2.16 langchain==0.2.17 -- libs/langchain/pyproject.toml
diff --git a/libs/langchain/pyproject.toml b/libs/langchain/pyproject.toml
index bcf19da0d..e4d6761b4 100644
--- a/libs/langchain/pyproject.toml
+++ b/libs/langchain/pyproject.toml
@@ -4,7 +4,7 @@ build-backend = "poetry.core.masonry.api"

 [tool.poetry]
 name = "langchain"
-version = "0.2.16"
+version = "0.2.17"
 description = "Building applications with LLMs through composability"
 authors = []
 license = "MIT"
@@ -33,7 +33,7 @@ langchain-server = "langchain.server:main"

 [tool.poetry.dependencies]
 python = ">=3.8.1,<4.0"
-langchain-core = "^0.2.38"
+langchain-core = "^0.2.43"
 langchain-text-splitters = "^0.2.0"
 langsmith = "^0.1.17"
 pydantic = ">=1,<3"
@@ -111,6 +111,7 @@ langchainhub = "^0.1.16"

 [tool.poetry.group.lint.dependencies]
 ruff = "^0.5"
+orjson = '<3.10.0'

 [tool.poetry.group.typing.dependencies]
 mypy = "^1.10"

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

BTW, the langchain==0.2.16 release incorrectly lists the version floor of langchain-core as 0.2.32 instead of 0.2.38.
If this needs to be fixed, a new v0.2.16 branch should be created, and a PR should be opened against that branch. The PR should bump the build number and the version floor.


CephFS: kernel and fuse mount options per cluster

Describe what this PR does
Implemented the capability to include kernel and fuse mount options
for individual clusters within the ceph-csi-config ConfigMap.
This allows users to configure the kernel and fuse mount options
for each cluster separately. The mount options specified in the ConfigMap
will supersede those provided via command line arguments.
Is there anything that requires special attention
Is the change backward compatible?
Yes
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow
guidelines in the developer
guide.
[x] Reviewed the developer guide on Submitting a Pull Request
[x] Pending release notes
updated with breaking and/or notable changes for the next major release.
[x] Documentation has been updated, if necessary.
[x] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

Show available bot commands
These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

/retest ci/centos/<job-name>: retest the <job-name> after unrelated
failure (please report the failure too!)

/test ci/centos/mini-e2e/k8s-1.28

LGTM, can you please check that we don't have any problem for the static PVC where clusterID might not be set but monitor IP's are passed?

Added a check for empty ClusterID.

@mergifyio rebase

@mergifyio queue


Make sure Write doesn't panic.

Fixes #555.

Makes sense how it prevents the panic.
Where did this bug report come from? It's too bad we can't add a failing test -- next time would be good to try and get a reproduction. Something to keep in mind when we see issues come in. But fixing panic === good
LGTM

@sjezewski meme game is on point

I would have really liked to add a failing test too, but have no idea when fuse triggers this bug unfortunately.


Update _sidebar.md

Correcting documentation references.  This is attempt number two, the first time I clicked "commit directly to the main branch."

'Closes #2' and autolink issue #2.


Switch to using dict instead of values (up to 14x speedup, lower model size)

Question | Feature Request
Recently I've been researching how __dict__ and __slots__ works and discovered that if we inherit from class with __slots__ and don't declare __slots__ in child class, __dict__ will be created automatically.
Also I found out that object.__dict__ uses less memory than ordinal dict, as it shares keys between instances (PEP-0412)
Moreover I found an advice to use __dict__ and __slots__ together in cases when fast access to attributes declared in __slots__  needed.
So combining these researches I replaced
__slots__ = ('__values__', '__fields_set__')
to
__slots__ = ('__dict__', '__fields_set__')
and renamed usages of __values__  in BaseModel (yes, that's exactly the only thing I've made) also removed __getattr__, as it's become redundant.
This lowered model size a bit, but what is most interesting, increased attribute access up to 14 times!
All tests are passed without any changes, so as far as I know this won't break any existing code except for one that uses __values__ attribute directly. But I guess that can be solved adding a property:
@property
def __values__(self):
    # Through warning here
    return self.__dict__

Here's a little test showing a difference:
import datetime
import functools
from sys import getsizeof
from timeit import timeit
from types import ModuleType, FunctionType
from typing import Optional, List

from gc import get_referents

from pydantic import BaseModel

class Article(BaseModel):
    id: int
    label: str
    text: Optional[str]

class User(BaseModel):
    id: int
    login: str
    email: str
    password: str
    first_name: str
    last_name: Optional[str] = None
    articles: Optional[List[Article]] = None

    register_date: datetime.datetime
    last_login: Optional[datetime.datetime] = None

user = User(
    id=42,
    login="mylogin",
<EMAIL_ADDRESS>    password="hashed secret password",
    first_name="Some",
    last_name="User",
    articles=[Article(id=1, label="foo"), Article(id=2, label="bar")],
    register_date=datetime.datetime.now(),
    last_login=datetime.datetime.now()
)

globs = {'user': user}
tmt = functools.partial(timeit, globals=globs, number=1000000)

BLACKLIST = type, ModuleType, FunctionType

def get_size(obj):
    """
    sum size of object & members.
    https://stackoverflow.com/a/30316760
    """
    seen_ids = set()
    size = 0
    objects = [obj]
    while objects:
        need_referents = []
        for obj in objects:
            if not isinstance(obj, BLACKLIST) and id(obj) not in seen_ids:
                seen_ids.add(id(obj))
                size += getsizeof(obj)
                need_referents.append(obj)
        objects = get_referents(*need_referents)
    return size

print(f'''
has __dict__: {hasattr(user, '__dict__')}
uses __dict__: {not hasattr(user, '__values__')}
has __values__: {hasattr(user, '__values__')}

getsize(user): {getsizeof(user)} bytes
get_size(user): {get_size(user)} bytes
''')

print(f'''\
getattr:
    user.id: {tmt('user.id')}
    user.articles: {tmt('user.articles')}
    user.articles[0].label: {tmt('user.articles[0].label')}
''')

print('getsizeof:')

try:
    print(f'''\
    user.__dict__: {getsizeof(user.__dict__)} bytes
    user.articles[0].__dict__: {getsizeof(user.articles[0].__dict__)} bytes
    user.articles[1].__dict__: {getsizeof(user.articles[1].__dict__)} bytes
''')
except AttributeError:
    print('   __dict__: skipped')

try:
    print(f'''\
    user.__values__: {getsizeof(user.__values__)} bytes
    user.articles[0].__values__: {getsizeof(user.articles[0].__values__)} bytes
    user.articles[1].__values__: {getsizeof(user.articles[1].__values__)} bytes
''')
except AttributeError:
    print('    __values__: skipped')
...

Results with __values__ used in BaseModel:
has __dict__: True
uses __dict__: False
has __values__: True

getsize(user): 72 bytes
get_size(user): 3620 bytes

getattr:
    user.id: 0.5301102200000969
    user.articles: 0.4485138009995353
    user.articles[0].label: 0.8063585330000933

getsizeof:
    user.__dict__: 112 bytes
    user.articles[0].__dict__: 112 bytes
    user.articles[1].__dict__: 112 bytes

    user.__values__: 368 bytes
    user.articles[0].__values__: 240 bytes
    user.articles[1].__values__: 240 bytes

Result with __dict__ used instead of __values__ (upd. results without getattr):
has __dict__: True
uses __dict__: True
has __values__: False

getsize(user): 64 bytes
get_size(user): 3484 bytes

getattr:
    user.id: 0.03707944100096938
    user.articles: 0.03184107699962624
    user.articles[0].label: 0.06509907500003465

getsizeof:
    user.__dict__: 368 bytes
    user.articles[0].__dict__: 240 bytes
    user.articles[1].__dict__: 240 bytes

    __values__: skipped

This is very interesting! I tried running the benchmark, and it looks like there is no change, but I think the benchmark only relates to parsing speed (basically no Model.__getattr__s involved).
I added the following code to the benchmark to see what it looked like in a slightly more realistic usage scenario:
                    passed, result = test.validate(case)
                    # New here:
                    if passed:
                        for _ in range(200):
                            if result.location is not None:
                                getattr(result.location, "latitude")

If I remove the contribution of the parsing part, I got an approximately 10x speedup on your branch relative to master, whether cythonized or not. As far as I can tell, there was no regression in parsing speed.
However, if I replaced the above getattr with just result.dict() or result.json(), I got no speed up. I'm not familiar enough with the internals to clearly understand why this is the case.
At any rate, given how common an operation attribute access is in practice, this seems like an awesome improvement!
There is another big benefit to this (at least for us PyCharm users 😅): by removing the custom __setattr__, PyCharm now raises warnings when you try to access attributes that don't exist! (This is currently handled properly by mypy, but PyCharm doesn't respect the @no_type_check decorator for this purpose. I raised an issue with PyCharm and got an explicit response that they didn't think that decorator should be used for that purpose, even if mypy does.) This alone would make me excited about this change.
Overall, 👍👍

I noticed that BaseModel.dict speed didn't change too.
I guess that is because we acessing __values__ once or twice per one model and getting it directly with object.__getattribute__, as it are stored in __slots__).

This looks awesome. Thank you very much.
I'll review the code in the PR but reply to the principle here.
I don't think we need to add the __values__ property, that's something internal which I doubt anyone is used, just clear description in HISTORY should be enough.

fixed by #712


Ignore some SslContextBuilderTests on certain JDKs

Motivation:
The tests testUnsupportedPrivateKeyFailsFastForServer and testUnsupportedPrivateKeyFailsFastForClient are failing on RedHat/Fedora distributions of JDK 11.
The test try to assert that a certificate using an exotic curve is rejected by BoringSSL, and throws an SSLException when building the private key.
However, on RedHat JDK 11, this curve - <IP_ADDRESS>.39 - is not in the JDK CurveDB, and so we get an IllegalArgumentException when creating the key manager.
The CurveDB on this distribution has a pretty limited set of curves available, so it might not be possible to find one that is accepted by the key manager, but rejected by the private key build.
Modification:
Make the tests look out for the IllegalArgumentException from the CurveDB, and if observed, abort the test as if by an assumption violation.
Result:
The tests will now be ignored, rather than fail the build, when running on JDKs that do not know about the curve used in the test.

Checkstyle fix…


go/lint: start running govulncheck on moov-io repositories

Docs: https://pkg.go.dev/golang.org/x/vuln/cmd/govulncheck

Codecov Report
Base: 78.37% // Head: 78.37% // No change to project coverage :thumbsup:

Coverage data is based on head (14bfc8e) compared to base (f385fd1).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #262   +/-   ##
=======================================
  Coverage   78.37%   78.37%
=======================================
  Files           1        1
  Lines          37       37
=======================================
  Hits           29       29
  Misses          4        4
  Partials        4        4

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Make it possible to specify None for Where conditions.

This issue will be resolved with #1 Add where conditions other than equal.


No executable found matching command "dotnet-ef"

dotnet ef --version
dotnet : No executable found matching command "dotnet-ef"
At line:1 char:1
+ dotnet ef --version
+ ~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (No executable f...and "dotnet-ef":String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

Steps to reproduce

Create a Web Api core project.
Add some references in csproj file.

<Project Sdk="Microsoft.NET.Sdk.Web">

  <PropertyGroup>
    <TargetFramework>netcoreapp1.1</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Folder Include="wwwroot\" />
  </ItemGroup>
  <ItemGroup>
    <PackageReference Include="Microsoft.ApplicationInsights.AspNetCore" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore" Version="1.1.2" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.3" />
    <PackageReference Include="Microsoft.EntityFrameworkCore" Version="1.1.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Sqlite" Version="1.1.2" />
    <PackageReference Include="Microsoft.Extensions.Logging.Debug" Version="1.1.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Design" Version="1.1.2" />
    <PackageReference Include="Microsoft.EntityFrameworkCore.Tools" Version="1.1.1" />
  </ItemGroup>
  <ItemGroup>
    <DotNetCliToolReference Include="Microsoft.VisualStudio.Web.CodeGeneration.Tools" Version="1.0.0" />
    <DotNetCliToolReference Include="Microsoft.EntityFrameworkCore.Tools.DotNet" Version="1.0.1" />
  </ItemGroup>
</Project>

Run dotnet ef --version

Further technical details
EF Core version: see above
Database Provider: see above
Operating system: win10    Version 10.0.15063 Build 15063
IDE: Visual Studio 2017
Please note, I have run dotnet restore

...and you running dotnet ef in the same directory as the *.csproj file, right?

Hi @bricelam , I run dotnet ef in Package Manager Console.
The insteresting is, dotnet ef can be run in normal .Net Core Console application, but can't be run in .Net Core WebApi Project via Package Manager Console.

PMC defaults to the solution directory. cd to the project directory before running.
Also, we strongly recommend using Add-Migration and friends in PMC since integrate better with Visual Studio to provide a superior experience.

Thanks @bricelam , cd to the project directory works.


Fix Github Actions timeout when downloading dependencies

https://github.community/t/failed-sending-data-to-the-peer-connection-died-tried-5-times-before-giving-up/189130


Enable inline video playback for iOS

Set allowsInlineMediaPlayback = 1 for WKWebview

Thank you for the PR. Could you please address the comment before i can merge this?


Add support for private object/class methods

If the name of a class or object method starts with an underscore, then make it a
private method.  It can be used by class/object methods in the current class
and by classes derived from the current class.

A coding pattern I use more and more frequently  in another language is private constructor and static method that returns an object. I have no idea how useful that would be in vim9script. Of course it's a moot point at this time since
vim9script
class C
    static def Create(): C
    enddef
endclass

gives E1010: Type not recognized: C
And a private constructor could be added later without breaking source compatibility.

Doh! But the following does work
vim9script
class C
    static def Create(): any
        return C.new()
    enddef
endclass

var x: C = C.Create()

To recap, when a class variable is declared "public" it is readable and
writable from outside the class. If "public" is omitted, the variable is
readable but not writable. If the variable name starts with an underscore,
it's private and neither readable nor writable from outside the class.

Yes.  Just to clarify, this PR introduces the support for private class/object
methods and doesn't change the accessibility of the class/object members
described above.

This is already quite a lot to remember, imo.
In my opinion the current syntax for classes is too complex. I would remove
the "public" keyword entirely and just use underscores for both variables and
methods. There's no need for read-only class variables. If someone needs to
access a variable from outside the class just write a setter like in any
other language. Vimscript should stay as simple as possible.
Like others already mentioned on vim_dev, an alternative to underscores is
to use "public" and "private" access modifiers for both variables and
methods.

I do have the changes ready to make the object members "public" by default.
I will send a separate email on this and open a separate PR.

I have created the PR https://github.com/vim/vim/pull/12932 to make the member variable access
public by default.

Like others already mentioned
that maybe was me?
aha, i've pointed out several times, but not sure the final version was going to be now -_-#
anyway, maybe it was designed to be completed and secured, but truly a bit complex,
if you insist leading underscore was better for private var/method, it's ok,
but meanwhile perhaps we may no need public/private keyword anymore,
as for interface/abstract (and 'static'?), vim9 maybe basic class was good enough, specially for scripting.


Add ClamWin 0.99.4

Part of https://github.com/microsoft/winget-pkgs/issues/897
Microsoft Reviewers: Open in CodeFlow

Looks like SourceForge timed out:
-----------------------------------------------------
Installer Availability
-----------------------------------------------------
Status: Started

6/1/2020 8:27:38 AM
Status: Completed
Result: Failed
Error occurred during installer copy from https://sourceforge.net/projects/clamwin/files/clamwin/0.99.4/clamwin-0.99.4-setup.exe/download.
 A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond A connection attempt failed because the connected party did not properly respond after a period of time, or established connection failed because connected host has failed to respond

6/1/2020 8:27:45 AM

****************************************************************************************
Installer Validation
****************************************************************************************
Status: Completed
##[error] Result: Failed

Can someone retrigger the bot?

@MarNicGit Try that: https://github.com/microsoft/winget-pkgs/issues/1588#issuecomment-639263718

@SoftCreatR Seems like that did the trick. Is there a big difference between ammending the commit and force pushing it compared to simply pushing an empty commit? I tend to do this a lot:
git commit --allow-empty -m "Retriggering bot"
git push

@marnicgit Nah, that's fine too.


nn_matching: cannot perform reduction function max on tensor with no elements

Hello @GrumpyZhou ,
Image pair matching sometimes fails with the following error:
File "my_code.py", line 111, in <lambda>
    matcher = lambda im1, im2: model.match_pairs(im1, im2)
File ".../image-matching-toolbox/immatch/modules/caps.py", line 79, in match_pairs
    matches, kpts1, kpts2, scores = self.match_inputs_(im1, gray1, im2, gray2)
File ".../image-matching-toolbox/immatch/modules/caps.py", line 69, in match_inputs_
    match_ids, scores = self.mutual_nn_match(desc1, desc2, threshold=self.match_threshold)
File ".../image-matching-toolbox/immatch/modules/base.py", line 63, in mutual_nn_match
    return mutual_nn_matching(desc1, desc2, threshold)
File ".../image-matching-toolbox/immatch/modules/nn_matching.py", line 28, in mutual_nn_matching
    matches, scores = mutual_nn_matching_torch(desc1, desc2, threshold=threshold)
File .../image-matching-toolbox/immatch/modules/nn_matching.py", line 12, in mutual_nn_matching_torch
    nn12 = similarity.max(dim=1)[1]
RuntimeError: cannot perform reduction function max on tensor with no elements because the operation does not have an identity

My guess is that there is an image, for which no keypoints are detected (yes, there is one such image), desc1 or desc2 array is empty, similarity matrix is empty and therefore the max operation fails. Implementing a test somewhere, checking if desc1 or desc2 are not empty (and dealing with the empty ones) would be great. I've seen the issue only with CAPS, but it could probably appear also for other methods.
Thank you very much!

The same issue appeared with R2D2. Tried to implement quick dirty fix in caps.py and r2d2.py by detecting empty kpts1 or kpts2 and then returning empty matches and scores, which seems to work.

Hi @v-pnk , thanks for reporting the error can you try it again?

Hi @GrumpyZhou ,
It seems to work OK now. Thank you very much for your quick response!


Add Threat Stack to the adopters list

Codecov Report

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

@@           Coverage Diff           @@
##           master     #774   +/-   ##
=======================================
  Coverage   81.84%   81.84%
=======================================
  Files          39       39
  Lines         661      661
  Branches       23       23
=======================================
  Hits          541      541
  Misses        120      120

Continue to review full report at Codecov.

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

Thanks @rpless!


[PUI] Helper to render an instance for an url

This PR adds a helper to render an object from a model type and (API) URL.

@matmair LGTM - please address JS issues and I'll merge ;)

Thanks @matmair


Trace implemented

Added:

Trace implementation
Test cases for trace
Added some changes in Readme for Windows support

Can you squash this into one commit?


data-sort-method='none' still does not work.

data-sort-method='none' still does not work.
The previous topic found a fix and that code added to line 113. By default its there.
https://github.com/tristen/tablesort/blob/gh-pages/src/tablesort.js#L113
But it still does not work. Any ideas?
Thanks

So I was able to fix it this way. Not sure if that the best way to do it.
CSS:
.no-arrow {cursor:default !important; }
.no-arrow:not(.no-sort):hover:after {  visibility: hidden !important;  }
HTML:
Store
Live demo:
https://www.perfumetr.com/perfume/1-million/

Thank for reporting, and for those tests!
Isn't this a duplicate of #137? Please see the comment I just added over there, https://github.com/tristen/tablesort/issues/137#issuecomment-637450512.


apiversion and kind not provide in response

error msg:
expected webhook response of admission.k8s.io/v1, Kind=AdmissionReview, got /, Kind=;
response:
{"response":{"uid":"a7aa4dca-6f37-4e74-9041-c7de0a120c23","allowed":true,"patch":"W3sib3AiOiJhZGQiLCJwYXRoIjoiL21ldGFkYXRhL2Fubm90YXRpb25zL211dGF0ZWQiLCJ2YWx1ZSI6InRydWUifSx7Im9wIjoiYWRkIiwicGF0aCI6Ii9tZXRhZGF0YS9hbm5vdGF0aW9ucy9tdXRhdG9yIiwidmFsdWUiOiJwb2QtYW5ub3RhdGUifV0=","patchType":"JSONPatch"}}
my k8s version:
v1.17.2

Hi @cocotyty !
Could you explain better the problem? e.g

Why and where do you need that?
What is not working for you?
When does this happen?
...

Thanks!

I use this library to modify specific pods. I deployed the webhook program and applied MutatingWebhookConfiguration. When I tried to create a deployment, I found that this error occurred, causing the pod to fail to create.

Hi @cocotyty Sorry for the delay, I missed the issue and forgot to respond earlier.
What is the webhook configuration manifest you are using?
I ask this because Kubewebhook only supports v1beta1 and maybe you are expecting v1...
There is a setting on the manifest to specify:
admissionReviewVersions:
  -v1
  - v1beta1

Check this issue, it happened to others: https://github.com/kubernetes/kubernetes/issues/85681

Hey @cocotyty!
I'm closing this, if you continue with the error after checking the latest comment I let you some days ago, please reopen the issue :)


Questions

I've been poking around, and I have a couple questions/requests.
Can you change the loading screen? The textured background and metal bits are giving me heart burn. The FIRST logo on a plain blue background would be fine.
What am I supposed to log in with? I've tried to login with my TBA account and it doesn't do anything. I've also logged in with like, gibberish and it works. To be totally honest, I don't really get the point of logging in, unless it's linked to TBA.
Not app related, but where did you get the image of the field? I can't find any good ones.
Last thing -  when are the events going to be 2017? The dates are all 2016.
Now, totally unrelated. I'm interested in being the drive coach. When you were driving, what did you want the coach to do? What were you looking for?
Thanks!

Hi!
Thanks for your post in the issues thread!
To answer your questions:

I will not be using the FIRST logo in my app in order to protect myself and future progress of the app. You can read the FIRST policy on usage of Trademarks and Copyrighted Materials [here](http://www.firstinspires.org/sites/default/files/uploads/resource_library/UseofUSFIRSTandLEGOGroupTrade marksandCopyrightedMaterials.pdf) - Maybe you should take some antacid!
There is no necessary login information. During my presentation Mary Jean asked the same thing, and my response was that for future development versions of the app, you will be able to log in to a team account and share data through a cloud service. The login screen will most likely be removed in my first release to the app store/google play.
I (slightly) modified the one that was included in the 'Arena' section of the game manual. The only thing I did was remove where the 'Gate' boxes where (I assume this in compliance within FIRST property usage).
I will send you a private email detailing that.

In other news, I'm working on a way to have you guys be able to download the app without my having to install it for you guys. My current method works, and now I just need to figure it out for iOS devices. Once I have that figured out I will make an official alpha release for my alpha testers. Sorry for the delay! I plan on adding more features/cleanup as well.

Awesome! Thanks!


Soul Cage not responding to redstone signal

1.8.9 v2.2.0-6
Is the soul cage activated/deactivated via redstone signal? If that is the intended behaviour, then it does not work in the above version.
I know you have updated since, so was this fixed if that is the way it's supposed to work?

With the default configs, only a Tier 5 shard responds to redstone.

thanks for the info

There already is. Look in ShardTiers.json.


Week view and month view minuteSlot dimming ?

Is there a way to set the background color of minuteSlot  ?
Days and hours (I have the MinuteSlotSize.minutes60) before "now" are greyed out ?
I didn't see a way to do this ...

Apparently in week view there is no cell Builder ?


Build a sand-alone app: error in pyinstaller

Hello,
I am trying to make my pywebio app a stand-alone. I was able to create a ".spec" file. I have editted the ".spec" file so that it has the import "from pywebio.utils import pyinstaller_datas" , and I chnaged the a=Analysis variable the parameter as suggested  here https://pywebio.readthedocs.io/en/dev/libraries_support.html.
Yet, when I try to run pyinstaller on my spec file I get the following error:
File "main_mushroom_soup.spec", line 2, in
from pywebio.utils import pyinstaller_datas
ModuleNotFoundError: No module named 'pywebio'
What can I do to get an exe file of my app?

It seems that you didn't install the pywebio to the environment which you install pyinstaller to.
Try to start a new python env and install pywebio and pyinstaller to it.


ValidateNewApp limits tables created

Given the line here, https://github.com/securitybunker/databunker/blob/master/src/userapps_api.go#L28 with implementation here: https://github.com/securitybunker/databunker/blob/bd6af1443b35445fc5a7a0c4f4f4c6837c7f020f/src/storage/mysql-storage.go#L871
Does databunker limit the number of tables that can be created for users given the hardcoded 100 for SQL and 10 for SQLite? We seem to be encountering the 405 db limitation error after a while which gets fixed when databunker is re-upped. Can we get clarity on the purpose of the limit? Thank you!

Hi,
There is no real need for this limitation.
I will remove it and publish an updated version later today.

I fixed the code and updated the docker containers.
https://github.com/securitybunker/databunker/commit/bdd7a28b66f1134e53837639b7cd0d36ae2b015f

I created a new patch to support multiple apps in 1 table.
https://github.com/securitybunker/databunker/commit/4876cfe8aa436f356e909318ffe20b87e003cb42
The generic.use_separate_app_tables variable can be configured to use multiple tables (the old way) or use only 1 table for all apps (the default new way).
This patch was done in order to remove the requirement to have higher database permissions.
Those permissions were required to create additional application tables.

Closing this


SPI Definitions for SparkFun boards

Added the #defines for the default SPI pins on SparkFun boards

Oh, but can you rebase against the develop branch please?

You also need to click 'Edit' at the top of this PR and change the base branch :wink:

Thanks @lurch - was trying to figure out the rebase process!

@lurch I don't see a contributing guidelines in this repo. Do you get tired of asking people to rebase on the develop branch? Or are contributions normally limited to a privileged few?

@lurch I don't see a contributing guidelines in this repo. Do you get tired of asking people to rebase on the develop branch? Or are contributions normally limited to a privileged few?

Github has a handy feature where you can set up a template for PRs to a repo. I think this would be useful for this repo, since the author is invariably requested to rebase. While this won't make the PR be against a different branch, it would be an opportunity to automatically request the author to rebase against a specific branch, rather than a human having to spend time requesting this for each PR.

These are questions for @kilograham . I only ask people to rebase to develop, in the hope that they'll be able to do that before Graham then comes online (we're in different timezones) :slightly_smiling_face:

Not a bad idea! Is this issue otherwise OK?

@kilograham I feel this PR mostly satisfies #328 (which is also satisfied by PR #343). I'm ok with closing #328 upon merger of this PR.


Status Code: 404

It's showing me Status 404 in console. whenever I try to play a song. I'm using searchSongs as false in constructor. Butbits not playing song just showing same error. How can I solve that.

ytdl core error

I am having the same error.

I think we are supposed to wait till and update from ytdl

Here's a temporary fix

I found a fix:
Go To : node_modules/ytdl-core/lib/info.js
Replace line 254 with
let jsonStr = body.split("ytInitialPlayerResponse = {")[1].split("}}};")[0] += "}}}";

I found a fix:
Go To : node_modules/ytdl-core/lib/info.js
Replace line 254 with
let jsonStr = body.split("ytInitialPlayerResponse = {")[1].split("}}};")[0] += "}}}";

It's tempory. You can play like 2-3 songs and again it shows 404 error

Should I be concerned?
When I install Distube it tells me this
"<EMAIL_ADDRESS>This package will not be updated anymore."
Anyone could answer?

Should I be concerned?
When I install Distube it tells me this
"<EMAIL_ADDRESS>This package will not be updated anymore."
Anyone could answer?

Because it is deprecated. V3 (alpha/beta) doesn't use it anymore. Anyway, it shouldn't create any bugs at the present.

This is due to YouTube changed their site recently and<EMAIL_ADDRESS>cannot handle that.
This will be fixed after ytdl-core author updates the package
Solution: Use this fork
npm i distubejs/node-ytdl-core

Note: You have to install git before installing a npm package from github (Or basically before installing above package)
And use npm v7 for easier overwritting

ytdl-core has been updated
use
npm i ytdl-core@latest


Go to definition doesn't work with Vue component imported without extension

When importing a single file Vue component without the *.vue extension, then the Go to definition doesn't work.
For example with the code below
<template>
  <img alt="Vue logo" src="./assets/logo.png">
  <HelloWorld msg="Welcome to Your Vue.js App"/>
</template>

<script>
import HelloWorld from './components/HelloWorld'

export default {
  name: 'App',
  components: {
    HelloWorld
  }
}
</script>

<style>
...
</style>

We cannot use Go to definition to go to the definition of HelloWorld Vue component. However if you change the import as follow:
import HelloWorld from './components/HelloWorld.vue'

It works.

This is expected behavior, and vue3 has deprecated the use of no extension.


DDPG Agent batch shape problem

The DDPG Agent wraps the data for the call to process_state_batch with additional square brackets which is unnecessary and causes an error with the batch shape when predict_on_batch is called on the next line. In my case I'd expect the batch shape to be (1, 5, 25), but it is turned into (1,1,5,25).
Seems like that the additional brackets are unnecessary. The same thing is happening also in the DQNAgent. See: #306 and #229

DDPG implementation is not so robust. Agree with ya.
For example: metrics += [mean_q] in compile() function, causes metrics list to explode when we have multiple-agents.


Fix phases

Hi, @thomashoneyman!
After reading about phases in a nixpkgs manual I decided that:

preBuildPhases should be left to downstream users;
the example that shows overriding a phase should have runHook-s so that downstream users are able to run preBuild and postBuild hooks.

Additionally, I read about npm rebuild and package-spec and decided to update package paths used in the override-phase example.

@deemp would you like to be added as a collaborator to this repository so that you can merge PRs like this one at your leisure once approved?

@thomashoneyman, what do you think about transferring this repository to the nix-community organization?
I find slimlock easier to use than dream2nix.

If they would want it! Fine by me, and I support you if you want to explore that.

I have added you as a collaborator; feel free to merge!


[Feature Request] FOSSBilling be conscious of server load when activating a customer order (ie. load balancing)

https://github.com/boxbilling/boxbilling/issues/1284
What is the idea?
Instead of configuring a product to a specific server, you can leave it at random or go to a server with fewer customers
Such a function already exists in other panels
Why would this be beneficial
Avoiding the Creation of Several Equal Products for Different Servers and Avoiding Rewor

Such a function already exists in other panels

I'm not sure which other panels have this currently, if anyone knows I'm curious to check it out.
I'm assuming that whoever wrote the post originally was referring to hosting control panels that support multiple servers, like ISPConfig (though I don't know if that supports this feature), but in that case really it's a task for the control panel rather than the billing solution.

Closing this as nobody has provided a compelling reason why this is something that a billing panel should be aware of rather than being the remit of the hosting control panel.


[backport to v1.8.5] ctmap: GC orphan SNAT entries

Backport of #13912 to v1.8.5

test-backport-1.8

Closing, as the PR was backported.


Need support for PT_4_ControlPointPatchList

Hi,
I have successfully made (WIP) planet rendering in OpenGL/DX but im new to unreal 4 and could find a base to start with, can you kindly add support for PT_4_ControlPointPatchList with tessellation shader support, because a quad is better looking when tessellated than triangles for planet.
Or atleast guide me the places where and what needs to change to support  PT_4_ControlPointPatchList!

Adding support for setting the vbo type is doable, but to my knowledge this wouldn't work with the material system. You'd have to do completely custom shaders. Internally UE4 only really uses one structure of tessellation information since it really just has 2 variants of the same thing.

Haven't heard from you in a while, so I'm closing this issue. Feel free to reopen it if you need further assistance.


Problems with input arrays in form

Hello Jeffrey,
I've run into trouble on a current test, as I'm trying to test a form that has input arrays. The problem is the fact that the input data is formatted differently when submitting the form via the test, than when submitting the form via a browser.
Example:
Output from browser(when i dump the request):
array:3 [
  "facilities" => array:5 [▼
    "FacilityOne" => "1"
    "FacilityTwo" => "2"
  ]
]

Output when submitting the form from the test:
`array:4 [
  "facilities[FacilityOne]" => "1"
  "facilities[FacilityTwo]" => "2"
]

Due to this, I can no longer access the input in my controller by
$request->get('facilities');

Can I see the HTML for this?
We can get it fixed.

Sure, this is an example of it:
<form action="/post-url" method="POST">
   <input type="text" name="facilities[FacilityOne]" />
   <input type="text" name="facilities[FacilityTwo]" />
   <input type="submit" value="Save" />
</form>

Would be great!

same here..would be great to handle sub arrays :+1:

Having the same issue with multiple select-boxes as well.
 <select id="input-tags" name="tags[]" multiple>
   <option value="1">Tag 1</option>
   <option value="2">Tag 2</option>
</select>

@hardevine solutions works though!

I have the same issue when I want to submit a form with multiple checkboxes. Does anyone have a solution for this?

@nivv sir , i am currently facing the same problem. I dont know how to solve it. my form looks like this:
`

Coal
Aluminum
  .
  .

`
How can i select multiple things ?
    $this->select('resource[] , ['Coal' , 'kjkjk'});
is all what i know but i am not able to solve it.


IPv6 static routes

I have an environment with a hand full of VMs with static IPv4 and IPv6 addresses. Until recently this has been a very straight forward setup with just a single gateway, so I just had to configure that one router as the default gateway on the network interfaces.
Now I have enabled IPv6 support on a docker VM, so I want to route a certain IPv6 subnet to that host. Because I use static IP configuration, I can't use radvd to advertise this new route for the other hosts to pick up. Hence I turned to the /etc/conf.d/staticroute file and using the /etc/init.d/staticroute script.
As far as I understand the logic in staticroute, if I populate the staticroute environment variable in /etc/conf.d/staticroute with "net <IP_ADDRESS> netmask <IP_ADDRESS> gw <IP_ADDRESS>" (one of the provided example lines) the init script will actually run this command on start:
route add -net <IP_ADDRESS> netmask <IP_ADDRESS> gw <IP_ADDRESS>

For my IPv6 case I therefore added "2001:ffff:ffff:1:4900::/79 gw 2001:ffff:ffff:1:4800::23" as the staticroute, which gets turned into:
route -A inet6 add -2001:ffff:ffff:1:4900::/79 gw 2001:ffff:ffff:1:4800::23

This fails to be interpreted by route, because of the dash that gets added.
My workaround on my systems is to change the line 91 in /etc/init.d/staticroute to:
				route $family $xtra $2 $args

The implication is that I would now have to change any IPv4 routes to start with that dash that I removed from the script.
Given that this is probably a rare edge case and if changed, would break existing IPv4 routes configuration, I am not sure if this should be changed globally. Please let me know if you would accept a PR to document this in the commented examples in the /etc/conf.d/staticroute configuration file or feel free to close this issue. Alternatively I could provide a patch that prevents the dash to get added only for the IPv6 address family.

Hi,
If I understand right, you shoud use "net" for IPv6 too, so you need for your example:
net 2001:ffff:ffff:1:4900::/79 gw 2001:ffff:ffff:1:4800::23
And your config is missing "net" in the beginning.

OK, I see now the problem. But I am not a maintainer, I was just passing by. I could look how to deal with this issue and make a pull request, for example, to fix it.

Maybe I'll just provide a PR that implements a different logic in /etc/init.d/staticroute when $family is ipv6 and document this case in /etc/conf.d/staticroute. Then the maintainers can decide if it is worth a merge or to marginal of a case.

@elrido I don't think you have an edge case.  I have the very same issue.  But besides changing the logic in /etc/init.d/staticroute, there is in fact a solution. If you look at the code in /etc/init.d/staticroute you see it uses the route command unless you set staticiproute as the key, then it uses the ip route command. When it executes the ip route command, it does not prefix the args with the offending -.
So simply add the following line to /etc/conf.d/staticroute instead:
staticiproute="2001:ffff:ffff:1:4900::/79 gw 2001:ffff:ffff:1:4800::23"
Important is you use the keyword staticiproute, anything other than that will default to using route command.  It will then execute ip route add 2001:ffff:ffff:1:4900::/79 gw 2001:ffff:ffff:1:4800::23 instead of using route add command.
Otherwise I agree, the logic should be changed to not prefix the args with the - when it detected inet6 as the family, otherwise it should do it as it does today,  maybe that is a compromise to keep it backward compatible.

Hello, I've encountered this issue today and I can't get to the workaround discussed in this thread.
I'm trying to add an ipv6 host to reject list in the Ip table with the following command
sudo route add -A inet6 -host fd00:ec2::254 reject

getaddrinfo: #host: -2
#host: Unknown host

Any suggestions for a workaround?

Any suggestions for a workaround would be helpful!

The workaround was to drop -net, you use -host. Given the error message looks similar, you might hit the same incompatibility in net-tools' route regarding ipv6 address parsing? Let us know if it works sans -host parameter.

Yes dropping -host and -net helped work around this issue but there seems to be more ipv6 trouble with route (not related to this issue),
Thanks!.
[ec2-user~]$ sudo route -6 add fd00:ec2::254/128 metric 1 reject
SIOGIFINDEX: No such device

(works with ip route)
[ec2-user~]$ sudo ip -6 route add prohibit fd00:ec2::254 metric 1
[ec2-use ~]$ ip route show table all | grep fd00
prohibit fd00:ec2::254 dev lo metric 1 pref medium


Removing capacity feature from the storageconsumer CR

Duplicate of https://github.com/red-hat-storage/ocs-operator/pull/2083
Created just for test

/test ocs-operator-bundle-e2e-aws


The Problem about QEOM dealing with H2, charge = -1

Environment

Qiskit Nature version:0.72
Python version: 3.9.18
Operating system: MacOS Sonoma

What is happening?
When I use QEOM, I set the driver as
driver = PySCFDriver(
    atom="H 0 0 0; H 0 0 0.7",
    basis="sto3g",
    charge=-1,
    spin=1,
    unit=DistanceUnit.ANGSTROM,
)

It is right in pyscf:

but meets bug in ansatz:

How can we reproduce the issue?
In docs/tutorials/04_excited_states_solvers.ipynb, input the driver as data above.
What should happen?
It throw out an error
ValueError: The number of spatial orbitals 2must be greater than number of particles of any spin kind (2, 1).
Any suggestions?
No response

UCC (and its variants) is currently hard-coding a check to ensure that no spin register is completely occupied: https://github.com/qiskit-community/qiskit-nature/blob/cbba76c516cbf372903a336b17fdeb2c31cd6d57/qiskit_nature/second_q/circuit/library/ansatzes/ucc.py#L394
I think that the code itself should be able to handle the case where the number of particles in a spin exactly fills that spin register but there are no unittests to ensure this. Lifting this limitation will enable this use case to run.

You can already use generalized=True which will ignore the actual occupation that you start from to run the code. But your ansatz will be longer.


vdk-server: optimizations

Increase timeout for deploying jobs
Add override file option to be able to override helm configuration locally when used for testing .

Closing due to inactivity


Fix compilation errors for ros2 client on Humble

Fix applied

response callback signature updated to support galactic and humble. I don't believe this builds on Foxy (haven't tested it).

You would need to sign your commits with GPG before I can merge it, thanks!
Regarding Foxy support, it should not be too big a concern, if anyone requires it, I can create a foxy branch.

I am using Foxy, however i can compile and run, but the goal is being rejected by the server

Could you provide more information about the rejection? Namely

Was it an error or did nothing happen?
How does the server log look like?
How does the client log look like?
What is your setup currently? (OS, version of ROS, simulation or real robot)

at the moment nothing is happening and i am using
Ubuntu 20.04
Ros2 foxy
simulation

Could you provide more information for me to try to re-create the issue? What steps have you run, was the goal through nav2 or a free fleet request?

Thanks for the information @aimethierry.
For long logs, please use https://gist.github.com, or
code blocks

This has grown out of the scope of this PR, please continue the conversation in https://github.com/open-rmf/free_fleet/issues/113, or try out the fixes mentioned in that thread.


StringEncodedValues

minor code cleanup: add generic parameters to Enum references
add StringEncodedValue which encodes Strings based on their index within a predefined list
StringDetails for retrieval via PathDetails

Would it make sense to make this dynamic? I.e. only provide a maximum number of strings at construction time and feed strings as they come while import? This way we could easier introduce new tags. Hmmh, but it would also mean that a smaller area is fine and the larger area requires a higher number and breaks.

Would it make sense to make this dynamic? I.e. only provide a maximum number of strings at construction time and feed strings as they come while import? This way we could easier introduce new tags. Hmmh, but it would also mean that a smaller area is fine and the larger area requires a higher number and breaks.

I though a bit about dynamic approaches but encountered a few problems:

we need the number of bits at construction time so we can't really safe space
if we'd solve this somehow and only encode values we encounter while importing, we would still need to store the encountered values. Otherwise it would be impossible to recreate the StringEncodedValue when loading from an already existing graph.

I dodged the problem in my custom area PR by providing two filter mechanisms(bbox+id) which should allow to conveniently limit the number of areas without having to deal with GeoJSON file itself.

Anything missing?

we need the number of bits at construction time so we can't really safe space

If bits are full we could either throw an exception or everything goes into a special "OTHER" string (?)

we would still need to store the encountered values

Yes, it would be necessary to serialize this into the StorableProperties.
But let us do this later in a separate issue (if required at all).
Can you add an integration test where you collect e.g. "surface" or some other string tag via one of our test OSM PBFs? Because it might be useful to introduce some automatic normalization features like white space removal or lower case conversion etc? Hmmh, maybe we don't add such a test here and have a real integration test using the new Janino scripting (an important optimization there would be to do the comparison based on the index not on the String).

If bits are full we could either throw an exception

From my custom area PR:
   custom_area_files:
     - location: core/files/spatialrules/countries.geojson
       id_field: "ISO3166-1:alpha3"
       encoded_value: country
       filter: deu,aut
       max_bbox: -180,180,-90,90

Would you add an extra option to define the maximum number of encoded entries?

or everything goes into a special "OTHER" string (?)

I fear that this might cause erratic behavior and problems which are hard to reproduce.
e.g if we encode countries and reach our maximum number of entries, the resulting EV would completely depend on the order of the processed ways

Yes, it would be necessary to serialize this into the StorableProperties.

Don't we store the toString representation of the EncodedValues? Maybe we should extend this to a load/store mechanic.

Can you add an integration test where you collect e.g. "surface" or some other string tag via one of our test OSM PBFs? Because it might be useful to introduce some automatic normalization features like white space removal or lower case conversion etc?

The first usecase would be custom areas and i think it's ok to expect the IDs in the GeoJSON to be already normalized.

Hmmh, maybe we don't add such a test here and have a real integration test using the new Janino scripting (an important optimization there would be to do the comparison based on the index not on the String).

Not sure if feasible but you could lookup the String within the EncodedValue array once and rely on reference checks later on if you want to avoid String.equals()

We would also need to switch to a hashmap based approach as the Strings passed in via setString() are unordered, so we can't rely on binary search.

Something like this?
public final class StringEncodedValue extends UnsignedIntEncodedValue {
    private final List<String> values;
    private final ObjectIntMap<String> indexMap;
    private final int maxEntries;

    public StringEncodedValue(String name, int maxEntries) {
        this(name, maxEntries, false);
    }

    public StringEncodedValue(String name, int maxEntries, List<String> values, boolean storeTwoDirections) {
        super(name, 32 - Integer.numberOfLeadingZeros(maxEntries), storeTwoDirections);

        this.values = new ArrayList<>(values);
        this.indexMap = new ObjectIntHashMap<>(values.size());
        int index = 1;
        for (String value : values) {
            indexMap.put(value, index++);
        }
        this.maxEntries = maxEntries;
    }

    public StringEncodedValue(String name, int maxEntries, boolean storeTwoDirections) {
        super(name, 32 - Integer.numberOfLeadingZeros(maxEntries), storeTwoDirections);

        this.values = new ArrayList<>(maxEntries);
        this.indexMap = new ObjectIntHashMap<>(maxEntries);
        this.maxEntries = maxEntries;
    }

    public final void setString(boolean reverse, IntsRef ref, String value) {
        int index = indexMap.get(value);
        if (index == 0) {
            if (values.size() == maxEntries) {
                throw new IllegalStateException("Maximum number of entries reached for " + getName() + ": " + maxEntries);
            }
            values.add(value);
            index = values.size();
            indexMap.put(value, index);
        }
        super.setInt(reverse, ref, index);
    }

    public final String getString(boolean reverse, IntsRef ref) {
        int value = super.getInt(reverse, ref);
        if (value == 0) {
            return null;
        }
        return values.get(value);
    }

    public List<String> getValues() {
        return Collections.unmodifiableList(values);
    }

    @Override
    public boolean equals(Object obj) {
        if (this == obj) {
            return true;
        }
        if (!(obj instanceof StringEncodedValue)) {
            return false;
        }
        StringEncodedValue other = (StringEncodedValue) obj;
        return Objects.equals(values, other.values);
    }

    @Override
    public int getVersion() {
        return 31 * super.getVersion() + staticHashCode(values);
    }
}

Yes, thanks :)
An additional indexOf method would be nice, so we can easier get the int (for fast usage inside Weighting.calc)

Changed my implementation and added a few unittests


create schedule configure tool

https://github.com/looker-open-source/lmanage/blob/8303d07308ea0d1a4333f8b3297dcb7b22ef45ad/lmanage/provision_instance_permission_structure.py#L131

https://github.com/looker-open-source/lmanage/blob/main/lmanage/configurator/content_configuration/create_schedules.py


The only way for DMESG to be a nop operation is setting DEVICE_DMESG_BUFFER_SIZE to 0

Looking at CodalDmesg.h from codal-core: https://github.com/lancaster-university/codal-core/blob/350a794a6f43d8e91b4762f110bad675169dc908/inc/core/CodalDmesg.h#L31
The codal_dmesg functions are only ((void)0) operations when DEVICE_DMESG_BUFFER_SIZE is set to zero (or lower).
Even if we set all the microbit-v2-samples codal.json "debug" flags (DEVICE_DMESG, DMESG_SERIAL_DEBUG, CODAL_DEBUG) to zero, CODAL is still allocating a 1 KB of RAM to codalLogStore, which we can see in the map file from a clean build:
 .bss.codalLogStore
                0x0000000020003a6c      0x404 libcodal-core.a(CodalDmesg.cpp.obj)
                0x0000000020003a6c                codalLogStore

And projects like MicroPython:
 .bss.codalLogStore
                0x0000000020003a74      0x404 /home/micropython-microbit-v2/lib/codal/build/libcodal-core.a(CodalDmesg.cpp.o)
                0x0000000020003a74                codalLogStore

And wouldn't be surprised if we found the same in MakeCode.
So I think we end up with non-debug builds where codalLogStore consumes 1 KB of RAM and the DMESG macros/functions are unnecessarily wasting cycles moving data to that buffer.

Currently DEVICE_DMESG doesn't do anything:

https://github.com/lancaster-university/codal-microbit-v2/issues/296

Right now, the only way not to compile the debug buffer and associated functions, is by setting DEVICE_DMESG_BUFFER_SIZE to 0.
This is non-obvious and not documented anywhere, so my proposal is:

DEVICE_DMESG should enable/disable having the CODAL debug log/buffer and functions.

Enabling this flag on its own keeps the debug log in RAM, which can be accessed via debugger

DMESG_SERIAL_DEBUG enables sending the debug log to serial

This is the current behaviour, no change there

We'll need to raise a couple of PRs to do this, first to change the default values in codal-core config header and then to remove the DEVICE_DMESG/DEVICE_DMESG_BUFFER_SIZE values set in the target.json files in codal-microbit-v2.

https://github.com/lancaster-university/codal-core/pull/160

Let's merge, but use DMESG_ENABLE as the main flag.

Okay, DMESG_ENABLE renamed in the codal-core PR and another PR created in the samples repo for the rename:

https://github.com/lancaster-university/microbit-v2-samples/pull/68

Okay, all PRs have been merged, so this is completed 🎉

https://github.com/lancaster-university/codal-core/pull/160
https://github.com/lancaster-university/codal-microbit-v2/pull/298
https://github.com/lancaster-university/microbit-v2-samples/pull/68


GDScript: Implement soft access restriction for _-prefixed members (as warning by default)

Supersedes: #98136
Closes: https://github.com/godotengine/godot-proposals/issues/641
After the discussion with some devs, with the philosophy of "implementing new features without game crashing". Here is the temporary discussion result in my opinion:
Rather than keywords and annotations, it's better to keep the original form that _ as prefix of private/protected members. There are three reasons for that:
Reasons

Based on @HolonProduction and KaiN's opinions, a virutal method should be private or protected, but I personally prefer it to be private, so a virtual method that begins with _ should be a private method (But in my opinion this is "uncallable except some calls", which would be another proposal and pr...). At first, I thought this would be redundant, until the moment I imagined encapsulation on calling a custom virtual method:

func _custom_virtual():
    <contents>

## Wrapper and encapsulator function to call the custom virtual method:
func call_custom_virtual():
    <some preparations related to the virtual method>
    _custom_virtual()

From the perspective of technical reasons, private and protect need to do checks during the reduction. The original pr did checks by retracing the source GDScriptParser::Node,  and even to iterate each super classes, which is much more performance-costly. However, by operating on p_identifier->name, we do not need to trace back what its GDScriptParser::Node is, which greatly reduces the consumption of CPU performance. Meanwhile, for protected identifiers, we just iterate the super classes of parser->current_class. Compared with the implementation of original pr, this is of balanced cost.
Based on the philosophy of "implementing new features without crashing the games", I turned the error into a warning. Of course you can turn it back to error in GDScript warning settings. However, if your game runs out of your expectation, it's time for you to check your code, especially maybe somewhere you referenced a _-prefixed variable?

Discussions
I know that this would bring to thumbups from those who think this is correct, or thumbdowns from those who think that I'm standing for official devs. No matter what this post will be given with, feel free to share and discuss your ideas here.
Plans

[ ] Introducing __ as the prefix of protected members (or private ones, needs to be discussed)
[ ] Unit tests

Notes
This PR will be converted to a review if there is a determined discussion result.

See also #98606 for another alternative

Personally I prefer this one instead of #98606.
Because, GDScript is simple language. Why you need to write "@protected/private" every time you declare a variable when you can just use a '_' prefix? And what you will do in moments when you forced to violate access limitation? Here, you can just ignore the warning. And in #98606 you can do... nothing. In GDScript there is no need for annotation complexity. Instead, we can just properly document this prefixes.
Also, I think '__' should be private, not protected, because private access is very strict, and I think it will be rarely used - protected is enough in most cases


Didn't parse tomorrow 6pm correctly

Doctor Appointment tomorrow 6pm at Sydney Clinic
Doctor Appointment  6pm
Sydney Clinic
Sunday, May 15, 2016 12:00:00 AM
Thanks for coding this thought, it looks fantastic!

This bug was fixed in 0.0.22, but demo page was not updated.
Sorry.
Shold be fixed now.

version updated


Number of imports

Feature request: Once the mocks have been registered, inform the user of how many.
This will ensure that the user that they have been imported

https://github.com/ng-apimock/core registers how many mocks have been loaded.


SNOW-1269784: Snow Snowpark Build/Deploy Issues (can't manually zip folder)

SnowCLI version
2.1.1
Python version
3.11.7
Platform
macOS-10.16-x86_64-i386-64bit
What happened
This is a bit weird, but I have a need to build my snowpark project, unzip the package, and make a modification to a sub-module. I found a bug in the process, which I have isolated the issue down to simply unzipping and zipping the package, no changes necessary.

if you build the package with snow build, and deploy it with snow deploy, it works fine.
if you build the package with snow build, open the zipped package, re-zip the folder, and then deploy it with snow deploy, it ends up not being able to find modules that are clearly in the package. The deploy does not fail, the procedure being called just fails at runtime.

The screen recording walks through these two paths above, illustrating the breaking point is simply opening and re-zipping the folder generated by snow build.
https://github.com/snowflakedb/snowflake-cli/assets/3344498/da23d298-8936-4cae-9dad-c21613b8210b
Console output
Python Interpreter Error:
ModuleNotFoundError: No module named 'procedures' in function HELLO_PROCEDURE with handler procedures.hello_procedure

How to reproduce

Init a snowpark project
run snow build to build the project zip
open/extract the project zip, delete existing zip file
compress the expanded project folder to create a new zip
run snow deploy for the new zip
call procedure

I did go back and test v2.0.0 and v2.1.0 as well, and they have the same issue.

Hi,
It looks like your manually created zip contains extra app 2/ due to compressing folder and not only its content.
Can you compare content and paths of both archives using unzip -l <name_of_archive>.zip ?

Oh! you're right, compressing the folder does nest everything in the parent folder. Consider this a non-issue, thanks!


fix: correct language in the description of HcCache::drop

Following discussion in team channel, clean up language about the importance of calling drop on an HcCache instance.

Saw you wanted just a review from Andrew, so not going to merge it until he looks at it. :)


Add configs for P45 and simulated AreaDetector

PR based on discussion around https://github.com/DiamondLightSource/dodal/issues/18 and similar issues. It may be a good idea to split it into 2-3 PRs. Mainly interested in opinions for now.
Changes:

Add beamline module for P45
Add beamline module for DLS simulated AreaDetector
Add Ophyd devices for simulated AreaDetector and Aravis GigE camera
Add utils module for instantiating all devices in a beamline module

@DominicOram Have fixed the build issues upstream. Shall we continue the factory format discussion offline? Also shall we merge this in the interim or leave it until decided?

@DominicOram Have fixed the build issues upstream. Shall we continue the factory format discussion offline? Also shall we merge this in the interim or leave it until decided?

Yes, let's get this merged


Use Paymetheus' local AppData directory for wallet.

Add AssemblyResources static class for reading attributes from the
entry assembly.  This is used to access the name of the application
which is used for the local appdata directory.
Fixes #52.

I like the concept.  I do see a potential issue here with ports since they're also named "Paymetheus".  I could see there being a problem where it tries to use the same directory for two different coins based on it.

Code itself looks fine.

Discussed on IRC.  This just means the ports will need to be named something like "Paymetheus-".
OK

Updated again and could use another code review.  It now includes the organization name (so the local appdata dir becomes AppData\Btcsuite\Paymetheus).  This means that forks for other currencies only need to change the organization name, rather than modifying the product name to avoid a conflict.
Also fixed a bug where spaces in the btcwallet datadir path would cause hangs at startup because it would use the wrong directory and the filesystem watcher for the rpc certificate would never complete.

OK


Getting a slew of warnings that a certain form feature is not found in my feature map

Rasa version: 1.10.0
Rasa SDK version (if used & relevant): 1.10.0
Rasa X version (if used & relevant): (not used)
Python version: 3.7.7
Operating system (windows, osx, ...): OS X Catalina 10.15.4
Issue:
Getting a repeated set of warnings when using the rasa interactive command.  There are more than a hundred (probably touching a 1000) identical lines.
This comes up after the lines -
2020-05-18 18:23:17 INFO     rasa.model  - Data (stories) for Core model section changed.
2020-05-18 18:23:17 INFO     rasa.model  - Data (messages) for NLU model section changed.
Training Core model...
Processed Story Blocks: 100%|████████████████████████████| 52/52 [00:00<00:00, 416.37it/s, # trackers=1]
Processed Story Blocks: 100%|████████████████████████████| 52/52 [00:06<00:00,  7.91it/s, # trackers=41]
Processed Story Blocks: 100%|████████████████████████████| 52/52 [00:07<00:00,  6.99it/s, # trackers=50]
Processed Story Blocks: 100%|████████████████████████████| 52/52 [00:07<00:00,  7.36it/s, # trackers=50]
Processed trackers: 100%|████████████████████████████████| 41/41 [00:00<00:00, 96.21it/s, # actions=252]
Processed actions: 252it [00:00, 1140.67it/s, # examples=252]
Processed trackers: 100%|███████████████████████████████████████████| 541/541 [00:00<00:00, 3096.83it/s]

Error (including full traceback):
2020-05-18 18:23:41 WARNING  rasa.core.featurizers  - Feature 'active_form_null' could not be found in feature map.

Command or request that led to error:
rasa interactive

Content of configuration file (config.yml) (if relevant):
# Configuration for Rasa NLU.
# https://rasa.com/docs/rasa/nlu/components/
language: "en"

pipeline:
  - name: ConveRTTokenizer
    intent_tokenization_flag: True
    case_sensitive: False
    intent_split_symbol: "_&_"
  - name: ConveRTFeaturizer
  - name: RegexFeaturizer
  - name: LexicalSyntacticFeaturizer
  - name: CountVectorsFeaturizer
  - name: CountVectorsFeaturizer
    analyzer: "char_wb"
    min_ngram: 1
    max_ngram: 4
  - name: DIETClassifier
    epochs: 100
  - name: EntitySynonymMapper
  - name: ResponseSelector
    epochs: 100

# Configuration for Rasa Core.
# https://rasa.com/docs/rasa/core/policies/
policies:
  - name: MemoizationPolicy
  - name: TEDPolicy
  - name: MappingPolicy
  - name: FormPolicy
  - name: FallbackPolicy
    nlu_threshold: 0.6
    core_threshold: 0.3
    fallback_action_name: "custom_fallback_action"

Content of domain file (domain.yml) (if relevant):
actions:
- custom_confirm_appointment_receipt
- custom_do_nothing
- custom_ask_appointment
- custom_confirm_appointment
- custom_fallback_action
- custom_get_appointment_availability
- utter_ask_prospectName
- utter_ask_prospectPhoneNumber
- utter_cheer_up
- utter_did_that_help
- utter_goodbye
- utter_greet
- utter_happy
- utter_iamabot
entities:
- preferredAppointmentDateTime
- prospectName
- prospectPhoneNumber
forms:
- appointment_form
intents:
- affirm
- deny
- provide_info
- 1.1_inquire_appointment
- goodbye
- out_of_scope
- demo_init
- mood_great
- mood_unhappy
- bot_challenge
responses:
  utter_ask_prospectName:
  - text: Please provide me your full name for the appointment.
  utter_ask_prospectPhoneNumber:
  - text: Please also provide me your phone number where I can text you the appointment
      confirmation.
  utter_cheer_up:
  - image: https://i.imgur.com/nGF1K8f.jpg
    text: 'Here is something to cheer you up:'
  utter_did_that_help:
  - text: Did that help you?
  utter_goodbye:
  - text: Feel free to reach out if you need anything else.  Take care and goodbye!
  utter_greet:
  - text: Hey! How are you?
  utter_happy:
  - text: Great, carry on!
  utter_iamabot:
  - text: I am a bot, powered by Rasa.
session_config:
  carry_over_slots_to_new_session: true
  session_expiration_time: 60
slots:
  preferredAppointmentDateTime:
    type: text
  proposedAppointmentDateTimeHuman:
    type: text
  proposedAppointmentDateTimeStandard:
    type: unfeaturized
  prospectEmailAddress:
    type: text
  prospectName:
    type: unfeaturized
  prospectPhoneNumber:
    type: unfeaturized
  requested_slot:
    type: unfeaturized

Thanks for raising this issue, @dakshvar22 will get back to you about it soon✨
Please also check out the docs and the forum in case your issue was raised there too 🤗

Do you have active_form_null in your stories?

I couldn't find that string in my stories file.  However, if I use rasa interactive to guide the form interaction, then the following two lines get added to that story after all the required slots are filled -
    - form{"name": null}
    - slot{"requested_slot": null}

These are identical with the lines that were added to the previous stories before I upgraded the version to 1.10.0 from 1.7.0.

This is my appointment form in the actions.py file as well -
class AppointmentForm(FormAction):
	def name(self):
		return "appointment_form"
	@staticmethod
	def required_slots(tracker):
		return ["prospectName", "prospectPhoneNumber"]
	def submit(self, dispatcher, tracker, domain):
		return []
	def slot_mappings(self):
		return {"prospectName":self.from_entity(entity="prospectName", not_intent="out_of_scope"), "prospectPhoneNumber":self.from_entity(entity="prospectPhoneNumber", not_intent="out_of_scope")}

these 2 lines are correct

it looks like null is read incorrectly as actual string "null" instead of being converted to None

Ah, is that because of the new format of the training examples?

what do you mean about new format? I'm not sure, it could be smth with the encoding of stories files

The new format of training examples is a little different for synonyms as of 1.10 (I think).  It also allows us to add role and group.
So it went from -
[2-wheeled](wheelCount:2)

to
[2-wheeled]{"entity":"wheelCount", "value":"2", "role":"wheelSpec"}

Would these warnings/encoding of stories files be affecting the training of the bot?

no I don't think, it is related, cause your problem is in reading stories data format

does a formbot example inside rasa open source repo works for you?

I'll try this today and report @Ghostvv.  Thanks!


Enabling vsts properties flow to test context

Description:
VS IDE has support of association in test explorer. Currently this support is disabled for mstest v2.
Support for mstest v2 will be enabled with this PR: https://devdiv.visualstudio.com/DevDiv/Automated Testing/_git/VS/pullrequest/135489?_a=overview
When association is enabled for mstest v2 tests, user can associate mstest v2 tests with a test case and then run it via test plan flow.
In test plan flow, user might need to use tcm properties (like test case id, test run id, test plan id etc) in test method code. This PR contains changes to allow tcm properties to flow to test context.
Validation Scenarios:
Validated that for full framework and net core, tcm properties are properly passed to test context.
Note: In talk with @PBoraMSFT if RFC is required in mstest v2 or not as the flow is controled from IDE. There is a plan to add the documentation on association window in test explorer.

Should we add some logging to show what values got filled in for properties? It will help in debugging issues.

https://github.com/Microsoft/testfx-docs/issues/36

thanks for resolving the issue. I am still not able to get the vsts Properties in testcontext.
Is there any sample project available?
as per the new changes I tried to access the TCM properties using:
IDictionary<TestPlatformObjectModel.TestProperty, object> tcmProperties = TcmTestPropertiesProvider.GetTcmProperties(null);
certainly compile time error will come, cause TcmTestPropertiesProvider is an internal class.
during Testcase execution, it's important to know the testrun id, Please help.

@anshuls01: Request you to not post queries on closed Pull Requests. Please create separate issue for your queries.


Segfault on latest release

What's your CLI version?
1.4.0
Description & steps to reproduce
Our CI jobs started failing with a segfault after Kurtosis got updated to 1.4.0.
Failing run log:
https://github.com/paradigmxyz/reth/actions/runs/11604491194
What is the severity of this bug?
Critical; I am blocked and Kurtosis is unusable for me because of this bug.
What area of the product does this pertain to?
CLI: the Command Line Interface

Same issue here, unable to get it working with 1.4.0

Hey this is a known issue we're on 1.4.0, we're working on releasing a fix ASAP! https://github.com/kurtosis-tech/kurtosis/pull/2579


mgmt: smp: Add function to unregister SMP GATT service

This adds a possibility to unregister SMP service. Using this function,
device can disable Firmware Update functionality, if not needed.
Signed-off-by: Mariusz Skamra<EMAIL_ADDRESS>

Ok, this is confusing. SMP here doesn't mean Security Manager Protocol? Especially when using the other meaning (whatever it is) in combination with Bluetooth, please spell it out instead of using the acronym - otherwise this is just confusing.

OK. This might be confusing. I mean Bluetooth service of Simple Management Protocol ^^ I'll change the commit message to avoid confusion.

yeap, that SMP name is unfortunate.
Lets just name commit "mgmt: Add function to unregister GATT service".
It should be clear enough.

@sjanc @jhedberg done

recheck


one image = one sector

Is it possible to put one image on each sector?

Hi,
Not at this time. It's something that I thought of a while back which would be cool, but no one has asked for it before so its down the bottom of my possible features list.

ok, not at this time )

Another person has asked for this feature so I need to give some priority.

This feature has been implemented in release 2.4


Is it possible to add bookmark URL?

Hi,
the only feature I am missing coming from Zoomify is the bookmark URL: http://www.zoomify.pl/demo/zoomify-bookmarks.htm?x=1040&y=1560&z=31
Very useful in two cases: to link a spacific portion of a large image and to easily switch between two images in a collection, keeping the same view.
Can it be added?

Yes, a plugin is a good option.
Not a developer, just a user, no idea where to start.
If someone is willing to start I can test it and report.

Fair enough... I'm spreading the word:
https://twitter.com/openseadragon/status/960925378154921984
Hopefully someone will be interested.

Great, thanks!

Whoever wants to write this plugin can get some good ideas from the code here:
https://github.com/davidmcclure/osd-dzi-viewer
...or perhaps @davidmcclure would be interested in making such a plugin?

At @jason-green-io's insistence I've gone ahead and made this plugin:
https://github.com/openseadragon/bookmark-url
@tosco please take a look and see if it works for your needs as well!

Great!
Tested and works perfectly: it is exactly what I was looking for.
Thanks

@tosco Excellent... thanks for giving it a try!


Update CMakeLists.txt

make  it same with wechatQrcode module's CMakeLists file 。
opencv/opencv_contrib#2931

Thank you!


Detect ambiguous labels

This feature could be used to throw appropriate Exception when multiple labels with the same name are created (via code generation or etc.).

Good idea! Thanks for the PR, I merged it with c263250.


Null pointer in createOAuth2Session

When calling createOAuth2Session in the Flutter SDK, the method crashes if success or failure is null which is allowed since both params are optional.
Here is the line and a potential fix:

    params.forEach((key, value) {
      if (value is List) {
        for (var item in value) {
          query.add(Uri.encodeComponent(key + '[]') +
              '=' +
              Uri.encodeComponent(item));
        }
      } else if(value != null) { // null check fixes the crash.
        // can not parse null value into a Uri.
        query.add(Uri.encodeComponent(key) + '=' + Uri.encodeComponent(value));
      }
    });

I would love to provide a PR that fixes this issue, but i am not sure if i can just fork the repo. Don't you generate the dart code?
@TorstenDittmann

As an addition I would also propose to add more type information to the method signature:
  Future<void> createOAuth2Session(
      {required String provider,
      String? success,
      String? failure,
      List<String>? scopes,
      })

@JulianBissekkou Thank you for reporting the issue, we have worked the fix and is waiting release
https://github.com/appwrite/sdk-generator/pull/220


Idea about bookmark in VS Code

HI, VS Code.
I'm a lover with VS Code for just one year, just a little suggestion here...
Cause the programmers may review/trace codes, and programmers (at least including me) may forget where they trace/review. A bookmark is a big help for programmers at this time.
Specific advice here:  The programmer can mark the line number in the code after right-clicking on the lino and selecting the "Add bookmark". VS Code saves the bookmarks in the ".vscode" directory before the VS Code is totally closed and left. In the next time when the same directory being opened, the VS Code loads the data from ".vscode" folder and marks where the bookmarks are, and the programmers can start their work from there.
It's seem that the suggestion is over. Hope that the advice can be implemented and help other programmers who have the same problem like me.
Best regards,
TitanEric

@titaneric there is a popular extension available for this and we currently have no plans to implement this in the core.


How to install

Hi, first of all, great script, and is just what I've been looking for to watch anime offline when I'm away.  At the moment I've only been able to use it by chmoding the darwin file and dragging and dropping it in my terminal.
Thanks

Sorry I now found the crunchy-cli in releases when building the project locally on my mac


ModalWidget with Wix-React Native Navigation

I am using Wix-react native navigation,  I am facing the problem of open the modal widget, openModal={(route) => {           navigator.push(route); // The ModalWidget will be opened using this method. Tested with ExNavigator         }}
The react native navigation needs to register the component first and then push/jump to that screen, I don't get how to integrate them together. Another question after this is, how to pop back?
Thank you!!

Have you solved the problem? For help


templates_suffix: "extension prefix" rather than suffix

Changing the file extension has real disadvantages on editors as they are not recognized anymore.
There are extensions taking care of syntax highlighting.
But there are still problems with the editor configuration.
I've an EditorConfig so I could modify that.
But it's annoying and not pretty.
So, wouldn't it be better to use an "extension prefix"? (.tmpl.py instead of .py.tmpl)
Or as I saw in another issue, just processing all files.
Or the more flexible, allow to define patterns to include (e.g. ["*.py", "*.html", ".env"] or regex (^\.env$)|(\.(py|html)$)).

In my experience, using a different default .jinja is the best, because it integrates properly with IDE syntax highlighters, such as https://github.com/samuelcolvin/jinjahtml-vscode/blob/b8fe444cbd838afb4962bb2446e67c85e0443675/package.json#L27-L177.

Or as I saw in another issue, just processing all files.

I think I never tested this, but if you use _templates_suffix: "", does Copier process all files? 🤔

Apparently it doesn't work as expected… #186

In my experience, using a different default .jinja is the best, because it integrates properly with IDE syntax highlighters, […]

As I tried to explain, yes… but no. I'm using .jinja and I've that extension installed but it's not working the same for .py and .py.jinja. The colors are not the same and my .editorconfig is not applied (because I don't want to include all the .jinja variants).
As usual, I'm looking to have things work as transparently as possible.
Being able to configure something like _templates_extension_prefix: ".jinja" would make it absolutely transparent for everything you may have configured for the real extensions.

@ppo wouldn't .jinja.py be recognized as Python only (not Python+Jinja)? And therefore syntax highlighting could break easily?

@pawamoy Not if the editor extensions also support that… I guess I could bother them too with that 😉

I am creating a Django boilerplate project, similar to Cookiecutter Django but with copier. Changing the extension (to .tmpl or anything else) can also break dependabot which looks for specific files in a designated directory. Therefore I cannot keep the dependencies of the required libraries in template up to date as it's possible with cookiecutter. I can imaging any arbitrary tool which works like dependabot might break.

dependabot works even if your requirements file contains jinja markers?

@Yajo Yes, it does.

I am creating a Django boilerplate project, similar to Cookiecutter Django but with copier.

@mazdakb, I'm working on a similar project Django Jazz, if you want to collaborate…
(sorry for the promotion but it's also so that you know why I posted so many issues, or more about their context 😉 )


feat: add ability to assign custom inline styles

Thanks, sorry for the delay.

No problem. Had the PR commit around for so long, I probably should have sent the PR long ago myself 😉


x/net/http2: support graceful close of client connection

I'm implementing my own http2.ConnPool with some health checking done using the new Ping() method. My problem is that my pool doesn't have the capability to dispose client connections because there is no way to cleanly close a client connection other than keeping it's net.Conn on the side and abruptly close it.
What about adding a Close() method http2.ClientConn that would send a GOAWAY frame and wait for the in-flight streams to complete?
Implementation proposal: https://go-review.googlesource.com/#/c/30076/

This PR has been sitting for month now. Is there something to be done to resume the review?

Sorry, that CL (its tests mostly) exhausted me earlier and then I forgot about it during Go 1.8 chaos.
Its tests still need a bit of clarity about what they're trying to do. The test code itself might be fine now, but it's hard to tell without comments saying what the intent is.
It looks like the non-test code is still fine.
/cc @tombergan

I am sorry to hear tests exhausted you :)
I added some comments to clarify the tests.

Thank you @bradfitz

I'm implementing my own http2.ConnPool with some health checking done using the new Ping() method.

@rs sorry to bother you,  I want to find a proper way to use Ping() method, but can hardly find some valuable reference, would you mind sharing your implement of ConnPolol?  thanks

Our final implementation ended up not using ping.


Correct class of other columns in flightsData

flightsData has the following columns:
[1] "AirlineID"          "ScheduleStartDate"
[3] "ScheduleEndDate"    "FlightNumber"
[5] "DepartureAirportID" "DepartureTime"
[7] "CodeShare"          "ArrivalAirportID"
[9] "ArrivalTime"        "Monday"
[11] "Tuesday"            "Wednesday"
[13] "Thursday"           "Friday"
[15] "Saturday"           "Sunday"
[17] "UpdateTime"         "VersionID"
[19] "Terminal"           "num_codeShare"
If you think column "ArrivalAirportID" has wrong class, then
class(flightsData$ArrivalAirportID) # show current wrong class

# put your code line here

class(flightsData$ArrivalAirportID) # show  correct class now

You should replace # put your code line here with your answer code line that can ensure class(flightsData$ArrivalAirportID) has the right class.

fct_DepartureAirportID <- factor(DepartureAirportID)

class(ArrivalAirportID)
fct_$ArrivalAirportID  <- factor(ArrivalAirportID )
class(fct_ArrivalAirportID)

ord_fct_DepartureAirportID <- factor(DepartureAirportID)

correct <-flightsData[,c("DepartureAirportID")]
correct <-factor(correct)
class(correct)

flightsData$ArrivalAirportID <- factor(flightsData$ArrivalAirportID)


[88] fixed multiselect child ticket bug

fixed issue #88

could this potentially resolve this issue as well?
https://github.com/figueiredorui/1-click-child-links/issues/140


Is it possible to set title value of Scene from Actions? Actions.showComponent({ title })?

react-native-router-flux v4.0.0-beta.21
react-native v0.47.0

Yes, you should be able to pass it as a prop when you perform your action. for example, Actions.login({title: 'Sign up!'})


Refactor termstatus platform-specific code

What is the purpose of this change? What does it change?
This refactors the internal/ui/termstatus package to get rid of some higher-order functions and some redundant checks on Windows. Code with higher-order functions is harder to navigate in an IDE and leads to awkward pseudo-method calls, e.g., t.clearCurrentLine(t.wr, t.fd) instead of t.clearCurrentLine().
I've tried to implement this using an interface, but it wasn't better.
Was the change discussed in an issue or in the forum before?
Not that I know.
Checklist

[x] I have read the Contribution Guidelines
[x] I have enabled maintainer edits for this PR
[ ] I have added tests for all changes in this PR
[ ] I have added documentation for the changes (in the manual)
[ ] There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
[x] I have run gofmt on the code in all commits
[x] All commit messages are formatted in the same style as the other commits in the repo
[x] I'm done, this Pull Request is ready for review

I'm not sure why Travis is listed twice, but clicking the "Queued" build's Details points to a successful build.


Replace _version_generated.py for _version.py

Replace the name of the autogenerated file choclo/_version_generated.py for choclo/_version.py. Remove the existing _version.py and make choclo/__init__.py to append the leading v to the version number. Add choclo/_version.py to .gitignore.
Relevant issues/PRs:
Related to #77

Good catch. This was needed before for datasets module to import the version but no longer needed.


error

Traceback (most recent call last):
File "/home/user/mosint/modules/SocialScan.py", line 1, in
from socialscan.util import Platforms, sync_execute_queries
File "/home/user/.local/lib/python3.9/site-packages/socialscan/util.py", line 9, in
import aiohttp
File "/usr/lib/python3/dist-packages/aiohttp/init.py", line 6, in
from .client import (
File "/usr/lib/python3/dist-packages/aiohttp/client.py", line 35, in
from . import hdrs, http, payload
File "/usr/lib/python3/dist-packages/aiohttp/http.py", line 7, in
from .http_parser import (
File "/usr/lib/python3/dist-packages/aiohttp/http_parser.py", line 15, in
from .helpers import NO_EXTENSIONS, BaseTimerContext
File "/usr/lib/python3/dist-packages/aiohttp/helpers.py", line 667, in
class CeilTimeout(async_timeout.timeout):
TypeError: function() argument 'code' must be code, not str

Same here, did u find a solution?

What version of python are you using?

/mosint$ python3 --version
Python 3.7.3
@.***:~/mosint$
On Thu, Feb 3, 2022 at 10:45 AM Alp @.***> wrote:

What version of python are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/alpkeskin/mosint/issues/13#issuecomment-1028794028,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN5WACKML76OUAISQXCWHWTUZJFDJANCNFSM5LLZX4TA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you commented.Message ID:
@.***>

both /mosint$ python3 --version
Python 3.7.3
and Python 2.7 are installed
can I force go to run with Python3?

I think there is a problem with your python version or the libraries you are using. The simple way is you can update the version of python3 and reinstall the aiohttp library.

Hey, @opticoax747 I think this bug is related to socialscan package. Can you share your OS and aiohttp version?
To find the aiohttp version you're using:
python3 -m pip freeze | grep aiohttp


Conflict resolution can be bypassed, resulting in arbitrary version chosen

I've found a situation in which conflict resolution is bypassed, even though there are two versions that should be considered.
Consider a repository with the following dependency structure:
org:one:1.0
\--- org:control:1.0
     \--- org:dep:2.5
org:two:1.0
+--- org:control-1.2-bringer:1.0
|    \--- org:control:1.2
|         \--- org:dep:2.5
\--- org:dep-2.0-bringer:1.0
     \--- org:dep:2.0

Expected Behavior

Depending on org:one:1.0 and org:two:1.0 should resolve to a dependency on org:dep:2.5 via conflict resolution between 2.5 and 2.0.
Current Behavior

If the dependencies are arranged just as described above (the order of the two org:two:1.0 dependencies matter!), then org:dep:2.0 will be selected and no conflict resolution will even take place (as verified with a debugger).
Debugging this, this is what is roughly happening:

DependencyGraphBuilder#traverseGraph is called
Edge from org:control:1.0 -> org:dep:2.5 is visited, selector for org:dep:2.5 created is resolved. ModuleResolveState for org:dep has one selector and that version is selected.
Later, when the org:control-1.2-bringer node is being resolved, the -> org:control:1.2 dependency is visited
The org:control:1.0 selector is release()d because the dependency has been superseded by 1.2. This removes the org:dep:2.5 selector from org:dep's ModuleResolveState.
org:dep:2.0 is visited (from org:dep-2.0-bringer:1.0) and this adds the org:dep:2.0 selector to the org:dep ModuleResolveState. That version is then 'selected' in the ModuleResolveState, as there's only one selector available. So far so good.
The org:control:1.2 -> org:dep:2.5 dependency is finally visited in the end. When creating the EdgeState, the old org:dep:2.5 selector (created in step 2) is retrieved from the ResolveState.
When it gets to performSelectionSerially, the selector for org:control:1.2 -> org:dep:2.5 is never visited because selector.isResolved() (having been resolved earlier in step 2). Thus, 2.0 remains the selected version in org:dep's ModuleResolveState!

Context

We've hit this on a number of projects where we tried to use the latest resolution strategy, as opposed to strict (aka failOnVersionConflict()) + nebula.dependency-recommender + dependencyRecommendations { strategy OverrideTransitives }.
Steps to Reproduce (for bugs)

I'm providing an actual failing test: https://github.com/gradle/gradle/pull/7049
Your Environment

Build scan URL: n/a

Thanks for the report and the reproducible test case. I will be adding a fix to your PR so that it can be merged.