Add support for preprocessing dash manifests

Have you read the FAQ and checked for duplicate open issues?
Yes
Is your feature request related to a problem? Please describe.
On several occasions we have had to perform pre-processing on DASH manifest, to correct anomalies in the manifest.
Remove empty audio representations
Remove TTML adaptation sets that were causing problems
Change maxSegmentDuration
Currently we do this in a network response filter. This is very inefficient; requiring the following transformation ArrayBuffer -> String -> XmlDocument -> manipulate -> String -> ArrayBuffer, which consumes a lot of CPU and memory resources.
Describe the solution you'd like
It would be preferable to intercept the manifest immediately after it has been converted into an XmlDocument usign a new configuration parameter to provide the preprocessing function.
manifest.dash.preprocess(manifestXml: XmlDocument)
Describe alternatives you've considered
This can be done at the application level by monkey patching Shaka like this
// Monkey patch DashParser to allow manipulation of the manifest as an XMLDocument before it is
// processed by shaka
const originalProcessManifest = getShaka().dash.DashParser.prototype.processManifest_;
const spectrumProcessManifest = async function (this: any, mpd: any, finalManifestUri: any) {
  logger.error('intercepted parser');
  return originalProcessManifest.bind(this)(mpd, finalManifestUri);
};
getShaka().dash.DashParser.prototype.processManifest_ = spectrumProcessManifest;

Additional context
We think it would be better to have a generic manifest preprocessing capability instead of an ever increasing number of flags to address specific situations (e.g. ignoreEmptyAdaptationSet, ignoreSuggestedPresentationDelay etc..)

The application level monkey patching approach does not work with the non-debug build of shaka player.

@ismena, what do you think of a configurable callback to let apps manipulate the DASH XML as a tree of elements, rather than needing to manipulate it as text in a network response filter?

@joeyparrish we will submit a PR for this enhancement if you think it will be accepted

Sounds good to me.  Please add a callback to the config under manifest.dash., and we'll take a look.  The default (in lib/util/player_configuration.js) can be a no-op, but please take a look at the comments around progressCallback in that file, as there is a subtle issue to work around:
      // Need some operation in the callback or else closure may remove calls
      // to the function as it would be a no-op.  The operation can't just be a
      // log message, because those are stripped in the compiled build.
      progressCallback: (content, progress) => {
        return [content, progress];
      },

We use the default values in the config to do type-checking of incoming configs at runtime.  So we have to keep the compiler from changing the number of arguments to the function.  To keep it from stripping unused args, return them to that they are "used".


The plugin is not being installed properly

Hi there! I installed the plugin and it looks like is ok, but I'm not seeing it in the UI, I mean, I do not have any option for S3
bash-5.0$ grafana-cli plugins ls
installed plugins:
tvarit-s3-datasource @ 0.0.1

bash-5.0$ grafana-cli -v
Grafana CLI version 7.1.0

The plugin is not signed. All backend plugins developed by anyone other than Grafana aren't.
To enable this plugin, change the following
[plugins]
...
# Enter a comma-separated list of plugin identifiers to identify plugins that are allowed to be loaded even if they lack a valid signature.
allow_loading_unsigned_plugins =

to
[plugins]
...
# Enter a comma-separated list of plugin identifiers to identify plugins that are allowed to be loaded even if they lack a valid signature.
allow_loading_unsigned_plugins = tvarit-s3-datasource


[Service Bus] Throw error when Entity Path in connection string does not match what is passed to methods that create senders & receivers

In v7 of the Service Bus package, the ServiceBusClient constructor takes connection string which can have the EntityPath specified in it if the user copied it from the policy created for a queue or topic.
The methods to create senders and receivers expect the queue/topic/subscription name to be passed to them.
This issue is to ensure that if there is a mismatch between the two, we should throw an error.
The reason is that a connection string from a policy from one queue/topic when used with a different entity gives a not so helpful error today. We can pre-empt this and throw a meaningful error ourselves instead.
If we are already doing this, please add a comment here sharing the error message and close this issue

We should be good to go.
E           azure.servicebus.exceptions.ServiceBusAuthenticationError: Entity names do not match, the entity name in connection string is servicebustest_qtwodq3ct; the entity name in parameter is servicebustest_qonevxudr.


Update ni-sidebandaudio-ioctl_sbaud_get_volumepropertyvalues.md

@tedhudek - This PR is ready to go live when you have a moment.


Update slf4j-api to 2.0.7

About this PR
📦 Updates org.slf4j:slf4j-api from 2.0.6 to 2.0.7
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.slf4j", artifactId = "slf4j-api" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.slf4j", artifactId = "slf4j-api" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #2486 (7124031) into main (1c144c2) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2486      +/-   ##
==========================================
- Coverage   78.36%   78.35%   -0.02%
==========================================
  Files         441      441
  Lines       15438    15438
  Branches     2498     2498
==========================================
- Hits        12098    12096       -2
- Misses       3340     3342       +2

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


Ticket/1.6.x/11661 ec2 rspec raises not throws

This patch corrects that behaviour. Also, 'throws' wasn't working on our CI
machines since we were using mocha 0.9.x.

Merged in 531f4d8c54a2c0196d25cae09771c17688006345

Merged in 531f4d8c54a2c0196d25cae09771c17688006345


Add basic policy evaluation to crates download

I tested pass/fail policy cases with crates, but not more nuanced sigstore-based policies yet.
Related to #9

@jcrossley3 I reworked this a bit to use ? instead of an unwieldy amount of match statements.

@jcrossley3 WDYT, is this one good to merge?


schedy

Problem/Motivation

seem to be missing module hass_apps_loader at least that's what I seen in appsdaemon4 log

Expected behavior

(What you expected to happen)

Actual behavior

(What actually happened)

Steps to reproduce

(How can someone else make/see it happen)

Proposed changes

(If you have a proposed change, workaround or fix,
describe the rationale behind it)

This add-on does not ship with Shedy. Please report this with Shedy itself.


Fixing application crash issue for plugin

Fixing application crash issue
Updating plugin version

@gss-pranav is that the most recent version of glassfish?

@gss-pranav is that the most recent version of glassfish?

Yes.
This contains implementation for our Jakarta dependency. this is compatible version with out Jakarta dependency.


[MS Language Server] Go to definition fails to find definition

Environment data

VS Code version: 1.35.0
Extension version (available under the Extensions sidebar): 2019.5.18875
OS and version: Ubuntu 18.04
Python version (& distribution if applicable, e.g. Anaconda): 3.6.7 64 bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Language Server (more info #3977): Jed/Language Server

Expected behaviour
When I go to a definition of render_template I get jumped to the definition of render_template
Actual behaviour
Error message No definition found for render_template.
Steps to reproduce:

Use flask repo
Go to blog.py
Line 25, go to definition of render_template

Issue moved to microsoft/python-language-server #1189 via ZenHub


Toilets preset: missing female=yes + male=yes option

The Gender field in the Toilets preset is missing a Female/Male (female=yes + male=yes) option.
I think that gender-segregated toilets are usually mapped with just one node instead of two (i.e. one with female=yes and one with man=yes). Because there's no female=yes + male=yes option, this could lead that people wrongly tag these toilets as unisex.

Unfortunately iDs radio button field can't set multiple values with one radio button.  If the tag keys were namespaced, we could support it with a multiselect.
A lot of people don't really like the existing gender tagging anyway, and there was a lengthy discussion of these issues on #3954, but I don't think it ever achieved enough consensus in the community to push for all the tags to be changed to something better.

Unfortunately iDs radio button field can't set multiple values with one radio button.

But adding tree separate checkboxes – one for female, one for male and one for unisex – like micahcochran proposed and like it's implemented in JOSM shouldn't be a problem, should it?

But adding tree separate checkboxes – one for female, one for male and one for unisex

That's confusing because some people check male=yes and female=yes, and other people check unisex=yes.
I'd rather have people drop a point node for each restroom, and pick a single option for what it is.  That's the least confusing way to tag them currently.
If instead gender field were namespaced and people could pick from many different things in a multiselect, that would be even better, but it's probably not going to happen because nobody can agree well enough on what the existing tags mean in order to do a meaningful tag upgrade.


[Bug] Nested validation using dataclasses

Describe the bug
When using nested dataclasses, validation does not work properly.
curl -X POST localhost:8000/test
⚠ 400 — Bad Request
====================
Invalid request body: Nested. Error: Value '<factory>' is not a dict

To Reproduce
from sanic import Sanic
from sanic.response import text
from sanic_ext import openapi, validate
from dataclasses import dataclass, field
from enum import Enum
from typing import Dict, Optional

app = Sanic("test")

@dataclass
class Choices(Enum):
    foo = "foo"
    bar = "bar"

@dataclass
class Child:
    foobar: Optional[Choices] = field(default_factory=Choices)

@dataclass
class Nested:
    child: Optional[Child] = field(default_factory=dict)

@app.middleware("request")
async def dummy_body_if_empty(request):
    if not request.body:
        request.body = b'{}'

@app.post('/test')
@openapi.body({"application/json": Nested}, required=True)
@validate(Nested)
async def handler(request, body: Nested):
    return text('OK')

Expected behavior
curl -X POST localhost:8000/test works.
Environment (please complete the following information):

OS: Debian/Ubuntu
Version 22.9.0

Hello,
I've been looking into this and it seems that the underlying issue does not have to do with dataclasses or the fact that they're nested with each other.
The culprit for the issue you're encountering seems to be that Enums classes are not a valid type to take into consideration for validation for the moment, as a dict type seems to be expected.
I believe that this is expected behavior having in mind how checks are made (against types, not against specific values) - but I may be wrong though.
Nonetheless, for the time being, you can use custom validation by specifying a post_init function within the class in which you expect an enum response.
Please find a working example in which it can be seen that nested dataclasses are not an issue here:
The code:
from sanic import Sanic
from sanic.response import text
from sanic_ext import openapi, validate
from sanic.exceptions import BadRequest
from dataclasses import dataclass, field
from enum import Enum
from typing import Dict, Optional

app = Sanic("test")

@dataclass
class AnimalTypes(Enum):
    mammal = "mammal"
    bird = "bird"

@dataclass
class Family:
    animal_group: str

    def __post_init__(self):
        self.validate_animal_group()

    def validate_animal_group(self):
        vals = [animal_type.value for animal_type in AnimalTypes]
        if self.animal_group not in vals:
            raise BadRequest("Animal must be of a valid type")

@dataclass
class Animal:
    name: str
    family: Family

@app.post('/test')
@openapi.body({"application/json": Animal}, required=True)
@validate(json=Animal)
async def nested_w_enum(request, body: Animal):
    return text(f'The animal is called: {body.name}, and it is of type: {body.family.animal_group}')

if __name__ == "__main__":
    app.run(host="<IP_ADDRESS>", port=1234, debug=True, auto_reload=True)

Valid request:
curl -X POST -d '{"name":"Lion","family":{"animal_group":"mammal"}}' localhost:1234/test
response:
The animal is called: Lion, and it is of type: mammal
Invalid Request:
curl -X POST -d '{"name":"Lion","family":{"animal_group":"dragon"}}' localhost:1234/test
response:
⚠ 400 — Bad Request
====================
Animal must be of a valid type

PD: You can specify animal_group: Union[AnimalTypes, str] adding Union to the typing imports so that it's visible from the OpenAPI spec.

Thanks for having a look. I'm using the __post_ini__ hack at a different level too, just did not come to my mind for this ;)
https://gitlab.com/morph027/python-signal-cli-rest-api/-/blob/main/python_signal_cli_rest_api/api/groups.py#L168-L181 (Here i also provide different views for openapi/validator as at the time writing this i could not find out how to trick it).

Works so far. I'm still splitting out the classes for openapi docs and validation ((example)[https://gitlab.com/morph027/python-signal-cli-rest-api/-/blob/main/python_signal_cli_rest_api/api/configuration/init.py#L34-L81]) but that's fine.
I'm wondering if using pydantic models will be less hassel...

I'm wondering if using pydantic models will be less hassel...

Almost without a doubt 😆

I'm wondering if using pydantic models will be less hassel...

Almost without a doubt laughing

Good to know, will have a look...some late night after work work :)

Just for the record, pydantic models are doing everything as expected :heart_eyes:


Cannot import habitat_sim anymore

https://github.com/facebookresearch/habitat-sim/blob/8ee3eb45d44cbcd51850d977136fbbe5f9dedaa7/habitat_sim/agent/agent.py#L7
>>> import habitat_sim
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/__init__.py", line 12, in <module>
    from .agent import *
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/agent/__init__.py", line 7, in <module>
    from .agent import *
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/agent/agent.py", line 7, in <module>
    import attr
ModuleNotFoundError: No module named 'attr'

I tried pip install attr but then I get
Type "help", "copyright", "credits" or "license" for more information.
>>> import habitat_sim
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/__init__.py", line 12, in <module>
    from .agent import *
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/agent/__init__.py", line 7, in <module>
    from .agent import *
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/agent/agent.py", line 9, in <module>
    from .controls import ObjectControls, ActuationSpec
  File "/raid/xkcd/external_code/habitat-sim/habitat_sim/agent/controls.py", line 17, in <module>
    @attr.s(auto_attribs=True, slots=True)
AttributeError: module 'attr' has no attribute 's'

So I don't know what attr package is.

It’s in requirements.txt. It’s called attrs on pip, so pip install attrs will do it.

Whoops, that one's on me.


MGMT-9825: create preflight package, responsible for preflight check …

…for single CR
This PR includes the following:

preflight package that executes preflight check for single PR
a) compile helm chart
b) get list of all yamls from the compiled chart
c) go over the list of yamls and validate daemonset image, in case buildconfig is not present
add Init API to runtime, used by preflight
add mock file for helmer package

/assign @qbarrand
/assign @pacevedom
/assign @pmtk
/assign @ybettan

/test lint

/test images

/test e2e-aws-ocp

/test e2e-aws-ocp

/test e2e-aws-ocp

Typo in the commit message:
preflight package that executes preflight check for single PR --> preflight package that executes preflight check for single CR

/lgtm


Don't repeat field name in errors

Fixes #10452 short-term issue.

@quinton-hoole @lavalamp

LGTM!

LGTM

This broke tests, and I didn't notice the big X because Shippable was being flaky. @cjcullen is taking #10588


Cannot play shortly after player is loaded (Android)

@triniwiz/nativescript-youtubeplayer 4.1.4
NS 8.1.4
Android 11
Some time after my page containing a YoutubePlayer is loaded, I get the prints
chromium: [ERROR:web_contents_delegate.cc(228)] WebContentsDelegate::CheckMediaAccessPermission: Not supported.
chromium: [ERROR:web_contents_delegate.cc(228)] WebContentsDelegate::CheckMediaAccessPermission: Not supported.

If I call 'play' prior to these permission prints, it does not play and I get
chromium: [INFO:CONSOLE(1)] "Uncaught TypeError: player.playVideo is not a function", source: https://www.youtube.com/ (1)

If I call 'play' after these permission prints, it works.  The only way I know to tell programmatically whether 'play' was successful is by checking for a state change, so I came up with the following workaround...
            onPlayVideoTap()
            {
                let player = this.$refs.videoPlayer.nativeView;
                let playCount = 0;
                let _this = this;

                Timer.clearTimeout(this.playStartTimerID);
                this.playStartTimerID = Timer.setInterval(function(){
                    if(_this.videoState == 'unstarted'){
                        console.log("play video["+(playCount++)+"]: "+_this.videoSource);
                        player.play();
                    }else{
                        Timer.clearInterval(_this.playStartTimerID);
                        _this.playStartTimerID = -1;
                    }
                }, 250);
            },

...where 'videoState' has been set in response to an event...
            onStateChange(args)
            {
                const STATES = ['unstarted', 'ended', 'playing', 'paused', 'buffering', 'bad', 'cued'];
                let player = this.$refs.videoPlayer.nativeView;
                let stateNum = args.state;

                this.videoState = STATES[stateNum + 1];
            },

Sample output...
chromium: [INFO:CONSOLE(865)] "Unrecognized feature: 'picture-in-picture'.", source: https://www.youtube.com/s/player/9a0939d3/www-widgetapi.vflset/www-widgetapi.js (865)
JS: play video[0]: aZKLf6gV04U
chromium: [INFO:CONSOLE(1)] "Uncaught TypeError: player.playVideo is not a function", source: https://www.youtube.com/ (1)
JS: play video[1]: aZKLf6gV04U
chromium: [INFO:CONSOLE(1)] "Uncaught TypeError: player.playVideo is not a function", source: https://www.youtube.com/ (1)
JS: network connection: WiFi
JS: play video[2]: aZKLf6gV04U
chromium: [INFO:CONSOLE(1)] "Uncaught TypeError: player.playVideo is not a function", source: https://www.youtube.com/ (1)
JS: play video[3]: aZKLf6gV04U
chromium: [INFO:CONSOLE(1)] "Uncaught TypeError: player.playVideo is not a function", source: https://www.youtube.com/ (1)
chromium: [ERROR:web_contents_delegate.cc(228)] WebContentsDelegate::CheckMediaAccessPermission: Not supported.
chromium: [ERROR:web_contents_delegate.cc(228)] WebContentsDelegate::CheckMediaAccessPermission: Not supported.
JS: play video[4]: aZKLf6gV04U
JS: video state: unstarted (-1)
JS: video state: buffering (3)
JS: video state: playing (1)

Rather ugly but it works.

No, I wasn't.  It does work if I only call play after ready event fires.  Thanks!

Issue is resolved by waiting for the "ready" state.


Scan GA testnet address prefilled with amount using QR

scan this via main QR button
Wrong amount populated. Also if should open the test account send and not the checking account send - this is important as we have a test account for learning

Also when tried of HRF donation QR, it gets multiple inputs.

Fixed send routing part(based on testnet/mainnet address)

This is related to iOS device and I don't have that with me.

This seems to be fixed now.


fail to build gitlab algorithm

HKube micro-service
api  server
Describe the bug
when tring to build algorithm from gitlab you are getting
Error response, status=500, message=TypeError: Cannot read property '0' of undefined
build the following algorithm (with or without commit/id)
{
    "name": "gitlabalg",
    "cpu": 1,
    "gpu": 0,
    "mem": "256Mi",
    "minHotWorkers": 0,
    "baseImage": "",
    "options": {
        "debug": false
    },
    "gitRepository": {
        "entryPoint": "master",
        "url": "https://gitlab.com/tamir321/hkube.git",
        "commit": {
            "id": "507aa9b1db90ccda19aef145849fb18362ab1bb7"
        },
        "branchName": "master",

        "gitKind": "gitlab"
    }
}

@tamir321 @yehiyam
fixed, waiting for PR

tested on
"systemVersion": "v1.3.36",
"fullSystemVersion": "v1.3.36-1592826366249",


Change the Placeholder M2M relationship from Page to Title

Move the placeholders many to many field from the Page model to the Title model
Adjust any failing tests
Do not introduce new tests
Create a data migration to:

Create a placeholder for each language the page is tied to (title_set)
Move all plugins for each language into their respective placeholder

Coverage remained the same at 78.189% when pulling 8c4d84a0d691bc9ec0bb66cbd7c1576df5b25884 on Aiky30:placeholder-to-title into<PHONE_NUMBER>0ee9f8c5c078bda8e5cfdd9a639f53 on divio:release/4.0.x.


bosh release is not self contained. requires internet access

trying xip release, looks very promising.
However i can not use it behind our corporate proxies. Here is the log.
Started compiling packages > pdns-3.4.6/daa1d3f85eca4b9333885512e68339c32e61232e. Failed: Action Failed get_task: Task a75249fe-c45f-44b3-4129-1e0fb54a21e3 result: Compiling package pdns-3.4.6: Running packaging script: Command exited with 100; Stdout: Err http://ppa.launchpad.net trusty Release.gpg
Unable to connect to ppa.launchpad.net:http:

Hi @poblin-orange ,
Next time I rebuild this release I'll try to incorporate the necessary pre-requisites as blobs, but it may be a couple of months before I get around to it.
—Brian

Ok. I have tried a quickfix with bosh-gen to reuse the powerdns package from the bosh boshrelease.
I submit it as a PR
Pierre


workshop: add a 100k rows of dataset for importing to TiDB Cloud Dev Tier

As 1 million is still big enough for Dev Tier, so we have to provide a much smaller dataset.
not urgent.

Closed, 100k is too large for Dev Tier, we provide 24k instead.


Page "Items by Location"  procedure SetColumns  [Scope('OnPrem')]

could you delete in page "Items by Location" the Scope onPrem? We can not use this page in our cloud solution or have to complete copy it.
Old Code:
[Scope('OnPrem')]
procedure SetColumns(SetWanted: Option Initial,Previous,Same,Next)
var
    MatrixMgt: Codeunit "Matrix Management";
    CaptionFieldNo: Integer;
    CurrentMatrixRecordOrdinal: Integer;
begin

NewCode:
procedure SetColumns(SetWanted: Option Initial,Previous,Same,Next)
var
    MatrixMgt: Codeunit "Matrix Management";
    CaptionFieldNo: Integer;
    CurrentMatrixRecordOrdinal: Integer;
begin

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[SPARK-45922][CONNECT][CLIENT] Minor retries refactoring (follow-up to multiple policies)

What changes were proposed in this pull request?
Follow up to https://github.com/apache/spark/pull/43591.
Refactor default policy arguments into being an arguments on the class, not within core.py
Why are the changes needed?
General refactoring, also makes it easier for other policies to derive.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Existing coverage
Was this patch authored or co-authored using generative AI tooling?
No

cc @juliuszsompolski

Added RetryException handling here, and fixed comment

Mind retriggering https://github.com/cdkrot/apache_spark/actions/runs/6877183050/job/18704368968? I think it might be related.

Needs conflict resolution after https://github.com/apache/spark/pull/43800 merged.

Done :). Thanks

cc @HyukjinKwon
Fixed some test fails, hope it works now

Merged to master.


feat: DecimalNumber() - added spacing between values and unit

Overview: What does this pull request change?

added the parameter unit_buff_per_font_unit to add additional space between the numeric values and the unit
default value unit_buff_per_font_unit=0 ensures backward compatibility
also added parameter documentation in docstring

Motivation and Explanation: Why and how do your changes improve the library?
According to general rules of typesetting a spacing between a numeric value and a unit is required. In Manim this is currently not possible, since leading non-printing characters in the unit would be stripped automatically.

class decTest(Scene):
    def construct(self):
        num1 = DecimalNumber(34.67, unit=r"\text{km}")
        self.add(num1)
        num2 = DecimalNumber(34.67, unit=r"\text{km}", unit_buff_per_font_unit=0.003)
        self.add(num2.next_to(num1,DOWN))

Further Information and Comments

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

Good idea and makes sense! I worry about the alignment if a unit has a character with a low tail in it like mg, won't it be too high? I don't know of a good general solution to this problem.

Since I only added horizontal space to the existing object, the vertical alignment should not be affected from how it already is today.
But yes, you might be right - already today the vertical alignment will probably be off, because the numbers and units are not aligned to their baseline, since Manim's Tex-rendering currently does not recognize the baseline...


 Error in `zsh': munmap_chunk(): invalid pointer: 0x096daee8 *** expanded

Expected result:
Doesn't crash.
$ git rev-parse HEAD
0b4bba4ca2d6115dabfdb0852ce516e30765d2e6

Note:
This should actually be filled at the zsh issue tracker, but I couldn't find it.
.zshrc
# If you come from bash you might have to change your $PATH.
# export PATH=$HOME/bin:/usr/local/bin:$PATH

# Path to your oh-my-zsh installation.
  export ZSH=/home/bjorn/.oh-my-zsh

# Set name of the theme to load. Optionally, if you set this to "random"
# it'll load a random theme each time that oh-my-zsh is loaded.
# See https://github.com/robbyrussell/oh-my-zsh/wiki/Themes
ZSH_THEME="agnoster"

# Uncomment the following line to use case-sensitive completion.
# CASE_SENSITIVE="true"

# Uncomment the following line to use hyphen-insensitive completion. Case
# sensitive completion must be off. _ and - will be interchangeable.
# HYPHEN_INSENSITIVE="true"

# Uncomment the following line to disable bi-weekly auto-update checks.
# DISABLE_AUTO_UPDATE="true"

# Uncomment the following line to change how often to auto-update (in days).
# export UPDATE_ZSH_DAYS=13

# Uncomment the following line to disable colors in ls.
# DISABLE_LS_COLORS="true"

# Uncomment the following line to disable auto-setting terminal title.
# DISABLE_AUTO_TITLE="true"

# Uncomment the following line to enable command auto-correction.
# ENABLE_CORRECTION="true"

# Uncomment the following line to display red dots whilst waiting for completion.
COMPLETION_WAITING_DOTS="true"

# Uncomment the following line if you want to disable marking untracked files
# under VCS as dirty. This makes repository status check for large repositories
# much, much faster.
# DISABLE_UNTRACKED_FILES_DIRTY="true"

# Uncomment the following line if you want to change the command execution time
# stamp shown in the history command output.
# The optional three formats: "mm/dd/yyyy"|"dd.mm.yyyy"|"yyyy-mm-dd"
# HIST_STAMPS="mm/dd/yyyy"

# Would you like to use another custom folder than $ZSH/custom?
# ZSH_CUSTOM=/path/to/new-custom-folder

# Which plugins would you like to load? (plugins can be found in ~/.oh-my-zsh/plugins/*)
# Custom plugins may be added to ~/.oh-my-zsh/custom/plugins/
# Example format: plugins=(rails git textmate ruby lighthouse)
# Add wisely, as too many plugins slow down shell startup.
plugins=(git rust)

source $ZSH/oh-my-zsh.sh

# User configuration

# export MANPATH="/usr/local/man:$MANPATH"

# You may need to manually set your language environment
# export LANG=en_US.UTF-8

# Preferred editor for local and remote sessions
# if [[ -n $SSH_CONNECTION ]]; then
#   export EDITOR='vim'
# else
#   export EDITOR='mvim'
# fi

# Compilation flags
# export ARCHFLAGS="-arch x86_64"

# ssh
# export SSH_KEY_PATH="~/.ssh/dsa_id"

# Set personal aliases, overriding those provided by oh-my-zsh libs,
# plugins, and themes. Aliases can be placed here, though oh-my-zsh
# users are encouraged to define aliases within the ZSH_CUSTOM folder.
# For a full list of active aliases, run `alias`.
#
# Example aliases
# alias zshconfig="mate ~/.zshrc"
# alias ohmyzsh="mate ~/.oh-my-zsh"

Smaller .zshrc:
export ZSH=/home/bjorn/.oh-my-zsh
ZSH_THEME="agnoster"
plugins=(git rust)
source $ZSH/oh-my-zsh.sh

(Ending with newline)
Results in crash with
$ zsh
 bjorn@pc-bjorn  ~  rustc --pretty flowgraph==
(eval):1: parse error near `(w~^H^Q )'
_values:147: command not found: (
*** Error in `zsh': double free or corruption (out): 0x088dc850 ***
Afgebroken

$ zsh --version
zsh 5.0.2 (i686-pc-linux-gnu)

Way of crashing seems to depend on the exact content of the config file, the history and with what text in the prompt when pressing tab.

Send it to<EMAIL_ADDRESS>


[SIP] Your Title Here (do not add SIP number)

Please make sure you are familiar with the SIP process documented
here. The SIP will be numbered by a committer upon acceptance.
[SIP] Proposal for ...
Motivation
Description of the problem to be solved.
Proposed Change
Describe how the feature will be implemented, or the problem will be solved. If possible, include mocks, screenshots, or screencasts (even if from different tools).
New or Changed Public Interfaces
Describe any new additions to the model, views or REST endpoints. Describe any changes to existing visualizations, dashboards and React components. Describe changes that affect the Superset CLI and how Superset is deployed.
New dependencies
Describe any npm/PyPI packages that are required. Are they actively maintained? What are their licenses?
Migration Plan and Compatibility
Describe any database migrations that are necessary, or updates to stored URLs.
Rejected Alternatives
Describe alternative approaches that were considered and rejected.

Looks like this was opened accidentally... twice. Were you planning on filling out this template? Closing for now, but happy to reopen.


Allow exporting mod pack as JSON string for easier sharing

It would be nice to have an option to export the selected mod pack as a JSON string/file or something that can be copy-pasted/uploaded to share mod packs across the internet.
The string/file will then be imported in the manager and any missing mods will be automatically downloaded using the Factorio Mods API.

Sorry, I misclicked the close issue button.

Modpacks are saved as Json objects, as a matter of fact. You can find them in %AppData%\ModMyFactory\modpacks.json.
However, while I can add an export feature for modpacks, auto-downloading the missing mods is currently not possible. I will come back to this once mod downloading is implemented (which is on my todo list).

Exporting and importing of modpacks is supported as of version 1.2.0.


Updated pod labels

Updated labels that get assigned to pod to match what is assigned on the stateful set to improve traceability throughout the deployment.

@fbordallo Please let me know if there is anything else I need to do to get this approved.  I have already filled out and sent in the document required for making contributions.  Thanks!!!

Hi, thanks for raising this issue, and sorry for the delay in getting back to you. We’ve been transitioning the Fortify Helm charts to Docker Hub, and the older Helm charts that used to be hosted in this repository have now been deprecated. If you think this issue is still relevant for the new Helm charts hosted on Docker Hub, please raise a support ticket through the regular OpenText support channels. We will no longer be monitoring issues raised on this repository. For more information on this change, please see the repository README file.


Add support for 1-dimensional RangeInclusive

I'm interested in using this for 1D and 2D overlapping ranges but with RangeInclusive (specifically, for overlapping text ranges + overlapping 2D grid ranges). I managed to basically brute-force replace every Range with RangeInclusive in the library and got a local version of this working, except it doesn't appear to work right for 2D ranges (for this, instead I decided to use quadtree_rs) -- probably because you cannot implement Ord for 2D ranges reliably (instead, it forms a lattice-like ordering for disjoint ranges).
Anyways, I think it works for 1D RangeInclusive so I thought I'd raise up the issue. I'll try to see if I can get the code uploaded.

see: https://github.com/tprodanov/iset/pull/4


Occur error while deploy example02 chaincode with security + 4 peers + BFTP

Description

I started 4 peers(test_vp0,test_vp1,test_vp2,test_vp3) and one CA. While I deployed example02 chaincode, it will occur the following warn message in the peer log, and occur error while execute query.

Peer Log: 09:37:56.790 [consensus/pbft] recvViewChange -> WARN 1689 Replica 0 found incorrect signature in view-change message: Could not verify message from vp0 (unknown peer)

Query Error: Error: Error querying chaincode: rpc error: code = 2 desc = "Error:Failed to launch chaincode spec(Could not get deployment transaction for a5389f7dfb9efae379900a41db1503fea2199fe400272b61ac5fe7bd0c6b97cf10ce3aa8dd00cd7626ce02f18accc7e5f2059dae6eb0786838042958352b89fb - LedgerError - ResourceNotFound: ledger: resource not found)"

This error does not happen if I disable security.
How to resolve it?
Describe How to Reproduce

membersrvc
docker run --rm -it -p 5000:5000 -p 30303:30303 -e CORE_VM_ENDPOINT=http://<IP_ADDRESS>:2375 -e CORE_PEER_ID=test_vp0 -e CORE_PEER_ADDRESSAUTODETECT=true -e CORE_SECURITY_ENABLED=true -e CORE_SECURITY_PRIVACY=true -e CORE_PEER_PKI_ECA_PADDR=<IP_ADDRESS>:50051 -e CORE_PEER_PKI_TCA_PADDR=<IP_ADDRESS>:50051 -e CORE_PEER_PKI_TLSCA_PADDR=<IP_ADDRESS>:50051 -e CORE_SECURITY_ENROLLID=test_vp0 -e CORE_SECURITY_ENROLLSECRET=MwYpmSRjupbT hyperledger/fabric-peer peer node start
Start other 3 peers one by one
Login, CORE_PEER_ADDRESS=<IP_ADDRESS>:30303 peer network login test_user0
Deploy CORE_PEER_ADDRESS=<IP_ADDRESS>:30303 CORE_SECURITY_ENABLED=true CORE_SECURITY_PRIVACY=true peer chaincode deploy -u test_user0 -p github.com/hyperledger/fabric/examples/chaincode/go/chaincode_example02 -c '{"Function":"init", "Args": ["a","100", "b", "200"]}'
Query, then will happen the following error in console
Error: Error querying chaincode: rpc error: code = 2 desc = "Error:Failed to launch chaincode spec(Could not get deployment transaction for a5389f7dfb9efae379900a41db1503fea2199fe400272b61ac5fe7bd0c6b97cf10ce3aa8dd00cd7626ce02f18accc7e5f2059dae6eb0786838042958352b89fb - LedgerError - ResourceNotFound: ledger: resource not found)"

CORE_PEER_ID must be vpN where N starts from 0
https://github.com/hyperledger/fabric/blob/master/docs/Setup/Network-setup.md#using-a-consensus-plugin

@iiiLiQuiDiii , already fixed using vp0 -> vp4. Thanks.


Why does modified-time access condition derive from etag access condition?

Hi Rick, I'm looking at your PR this https://github.com/Azure/azure-sdk-for-cpp/pull/1592 and working on using this access condition in storage sdk.
I cannot understand why inheritance should be used here. I was under the impression that, inheritance is always something like dog is an animal so animal is base class and dog is derived class. How is request access condition like that?
This inheritance relationship doesn't work for some storage API. Search Set Container Metadata in this page
https://docs.microsoft.com/en-us/rest/api/storageservices/specifying-conditional-headers-for-blob-service-operations, this API only supports If-Modified-Since access condition. Also Delete Container doesn't support etag access condition

@Jinming-Hu MatchConditions and RequestConditions are general purpose conditional request modifiers for all services.  It unfortunate that storage api's are inconsistent and all do not accept the all conditional request modifiers.
I would follow what the other 4 languages do.  I notice that the .NET SDK will accept all 4 even though as you pointed out all 4 are not accepted by the service.  They then throw if the IfMatch or IfNoneMatch are specified.
if (conditions?.IfMatch != default ||
                        conditions?.IfNoneMatch != default)
                    {
                        throw BlobErrors.BlobConditionsMustBeDefault(nameof(RequestConditions.IfMatch), nameof(RequestConditions.IfNoneMatch));
                    }

I don't know what I'm supposed to do here. This looks like pre-condition and according to C++ guidelines, we should crash rather than throw exceptions. Still I do not understand why inheritance has to be used.
@JeffreyRichter @tg-msft Can you please help out here?

@Jinming-Hu is really correct. For C++, let's keep our MatchConditions structure as-is and NOT derive our RequestConditions from MatchConditions - derive it from nothing. Also, let's rename RequestConditions to ModifiedConditions.

@RickWinter I assigned this issue back to you.

Ignoring GoLang (since it doesn't have inheritance), this change means we'll be different from all the other "OOP" friendly languages who decided to keep the inheritance structure. What's the drawback of their approach and is it worth it to be different here?
https://github.com/Azure/azure-sdk-for-java/blob/385a9f53e542689571cb407696b6f458d957196f/sdk/core/azure-core/src/main/java/com/azure/core/http/RequestConditions.java#L14-L16
https://github.com/Azure/azure-sdk-for-net/blob/a318e4154d56728fc6a22f8b4ce1c1d0c36a7d7c/sdk/core/Azure.Core/src/RequestConditions.cs#L11-L12

Other language sdk doing so doesn't justify the correctness. They sometimes make mistakes. Most of the times it's good to take reference from them, but not always.

What's the drawback of their approach and is it worth it to be different here?

If RequestConditions inherits from MatchConditions, it implies that any REST API supports if-modified-since/if-unmodified-since has to also support etag match/non-match . This is simply not true. It's not stated anywhere in any documentation. We do have API that only supports if-modified-since/if-unmodified-since while does not support etag.

Technically, Jinming is correct. From a raw HTTP perspective, MatchConditions and ModifiedConditions are different and many services support MatchConditions while few services support ModifiedConditions. So, from an HTTP perspective, there is no relationship between match & modified conditions and therefore these 2 types should not be related to each other. One could even go so far as to say that many services support If-Math and not If-None-Match so even the match conditions structure could not exist and just the one or two fields could be explicitly specified for the operations that support one or both fields. However, it adds context to customers to wrap these two fields in a thing called MatchConditions and a thing called ModifiedConditions.
The other languages are really mistaken to draw the relationship that Modified (or Request) Conditions is derived from MatchConditions and I think I'm responsible for some of this. When I was on the Storage team, all I cared about was Storage and I think all (or almost all) operations take Match AND Modified conditions, so I merged them together into 1 structure: https://github.com/Azure/azure-storage-blob-go/blob/master/azblob/access_conditions.go#L8
Then, as other languages and services got on board, the separation occurred because many services don't support Modified conditions but the separation was incomplete and the derivation remained. So, we have a chance to do the better thing in C++ and I think we should take advantage of that.


Revised Nest function and type names

I've renamed some of the Nest function and type names as they have confused some people.
I've also changed how the types are consumed rather than using the {} as T hack. In my opinion, this makes it clearer that we are providing types rather than running any significant code at runtime.
Also updated the docs to better explain the role every function and decorator, and how the type safety is done.
Also fixes #122

Codecov Report

Merging #140 (ecc9dfc) into main (8675078) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #140   +/-   ##
=======================================
  Coverage   65.47%   65.47%
=======================================
  Files           9        9
  Lines         252      252
  Branches       60       60
=======================================
  Hits          165      165
  Misses         77       77
  Partials       10       10

Impacted Files
Coverage Δ

libs/ts-rest/core/src/lib/zod-utils.ts
100.00% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

I want to pull this and play with it locally, but this looks great!
I wonder if this is an opportunity to discuss adding codemods for migrations in ts-rest? I've never done them myself so not sure on the effort required on our side to make them.

I want to pull this and play with it locally, but this looks great!
I wonder if this is an opportunity to discuss adding codemods for migrations in ts-rest? I've never done them myself so not sure on the effort required on our side to make them.

I've thought about it too. I think it could be a good idea to do during the next major release if we decide to drop the deprecated versions.

This looks great! LGTM :)

I way prefer the naming, and awesome that you polled the discord 🔥


Suggestion: Add a right-click menu when right-clicking an image inside Fly with options like "Open in explorer", "Copy File Name" and "Copy Path"

Would it be possible to implement a right-click menu in Fly similar to Picasa where, if you right-click the actual image in Fly you get a context menu with functions such as "Open in explorer", "Copy File Name" and "Copy Path"?

Closing as requested feature is already available.

Hmm then I might have a bug, I run 2.2.0 and right-clicking next to a picture makes Fly go windowed but right-clicking on the picture does nothing for me.

Right clicking outside the picture will make Fly go windowed. And right clicking on the picture should yield the windows explorer right click menu itself. I have tested it only on Windows 11.
I use Fly personally also and had noticed once it was not giving the right click menu. Could not reproduce the issue again to debug. I shall add some more logs for right click in next version and we can have a look into your issue.

Right clicking outside the picture will make Fly go windowed. And right clicking on the picture should yield the explorer right click menu itself. I have tested it only on Windows 11.
I use Fly personally also and had noticed once it was not giving the right click menu. Could not reproduce the issue again to debug. I shall add some more logis on that side in the next version and we can have a look into your issue.
Please raise as a bug so that I can keep track.

Thanks for the insight, alright will do!


fix sync crashing with constraint fails, refactor db queries, improve performance

This PR does a few things:

Solves #22
Refactor database schema to use (transaction_hash, log_index) as primary key for chain_events and all related tables
Refactors database queries to avoid running into conflicts, delete unused queries and code
Fixes a bug with ethers-rs that was leading to KeyRegistry events being unable to be parsed properly
Improves performance a bit by fetching logs for each contract in a single call for all events we care about instead of one-by-one per event
Adds a nice little updating progressbar for sync progress instead of basic println! statements

One small thing: could we make the progress bar pick up from where it left off when restarting? Besides that LGTM

yeah i went back and forth on making it start from FARCASTER_START_BLOCK vs the current block it's at
i'll update to make it start from FARCASTER_START_BLOCK to make it look nicer and the % mean something tangible


Drag uploading error gives an unclickable error dialogue

Trying to upload a complex .obj that is having problems and getting this unclickable error dialogue with an overlay on top of it.

Same problem here i guess. We need a cancel button here.

Something like this:

@mattatgit do you have the complex OBJ somewhere for reference? probably something errors out and doesn't come back

@kschzt No, sorry, not sure which one that was. I've tried a fairly wide range of objects, but don't have any really complex ones to hand right now. But pretty much anything i upload usually has problems.


[google_maps_flutter] plugin 0.5.21 regression, polylines updates are not displayed.

Polylines are not updated because the new added hasChanged() method in _PolylineUpdates.from(), always found the previous and current polyline being the same.
As a test, returnig true from hasChanged() restore the correct behavior.
The polyline was used to track the user route on the map, the only thing changing being the number of points.
Now, to restore the behavior, the PolylineId() must be unique, causing the old one to be removed.

This is really severe

@ppicas I agree with you. When we store the state of Polygons, Polylines etc, can we store a clone of the object? Maybe adding clone a method to each of these would make it consistent. Let me know if you can get to this in the next couple of days, if not I can do it.

Of  course @iskakaushik, I'll prepare a pull request :slightly_smiling_face:

this can be closed now


Zod: unstable/breaking variable names for validation lengths

What are the steps to reproduce this issue?
Have a schema which validation for mins and maxes, see that integers are added at the end of names

What happens?
The values are already unique enough I would have thought we don't need these numbers appended to the end of the variables, it means it is quite annoying to consume these variables in the code because every time a new min/max is added to an api the variables change out from under you.
…
What were you expecting to happen?
It is useful to use these variables for example a character counter on a field, it's nice if the backend defines these.
Stable variable names like the rest of the library
If they can't be stable for the whole file, at least stable for a given schema would be nice so if you add a new endpoint it doesn't change every other variable
…
Any logs, error output, etc?
…
Any other comments?
…
What versions are you using?
Operating System: …
Package Version: …
Browser Version: …

@Georgegriff that was introduced because of this issue: https://github.com/anymaniax/orval/issues/1027
A PR to fix both your current scenario but not break #1027 would be great!

I'll see what I can do, it might be tricky to fix without a breaking change if it results it people's Zod variables being renamed 🤔 an option wouldn't feel right. I'll explore it anyway and see what's possible

Maybe somehow you can detect whether its a matrix of values and add the numbers if not don't add the number?  I think I was just lazy and started at +1 etc to fix that issue.

So i just did some digging, cloned the repo and was about to start then realised @anymaniax fixed my exact issue in https://github.com/anymaniax/orval/pull/1330 🥳
I confirmed it by running the yarn run generate:zod and comparing the version i'm using:
6.27.1

vs master

When do you expect this might be released?
I note that @anymaniax mentioned this is a breaking change, i do tend to agree... however the original fix that introduced the unstable variable names was introduced in a 6.x minor version too and that too technical broke variable names, so maybe it doesn't matter.

Soon.. I will close this for now I marked it for 6.28.0

Turns out i can't read git history.. it's already out in 6.28.2.
Just wanted to say thanks for this project, its great!


Update post-lecture-quiz.md

👋  @redhand980 thank you for your contribution! Unfortunately this cannot be merged as this file is a quiz template. However, we do need help with ways to grade quizzes, so check out that issue if you'd like to help with that!


[TF-TRT] Allow bool dtype as IO of TRTEngineOps

This PR allows bool dtype as input or output dtype of TRTEngineOp.

@bixia1 for review
CC: @meena-at-work @drivanov


Add default constructor for HazelcastClient

Added the default constructor for HazelcastClient which simplified client usages.

Linux test FAILed.

Windows test FAILed.

Linux test FAILed.

Windows test PASSed.

Windows test PASSed.

Linux test PASSed.


Fix unchecked errors in clusterloader2 code.

Those were caught by errcheck.
/assign @krzysied

LGTM once compile errors are fixed

PTAL - tests are passing now.

/lgtm
/approve


When run linera-indexer, I get error 'invalid type: null, expected struct Event'

Description
When run linera-indexer, I get error 'invalid type: null, expected struct Event'
Steps
1  Compile linera-indexer in linera-indexer/example
2  Run local test net with linera net up
3  Run node service with linera service
4  Run linera-indexer with RUST_LOG=trace ./target/release/linera-indexer --storage /tmp/linera-indexer/indexer.db run --port 8088 --service-port 8081
Result
We get error Error: ReqwestError(reqwest::Error { kind: Decode, source: Error("invalid type: null, expected struct Event", line: 1, column: 529) })
Commit
f2310b4ab7fccc11108971b53368be09996e24ab

Hi @kikakkz, I'm working on some changes to the Linera GraphQL API at the moment so it's possible that the API has changed in ways that we need to match in the linera-indexer.
However, I'm having some trouble reproducing the error you describe.  If I do:
$ eval "$(linera net helper)"
$ linera_spawn_and_read_wallet_variables linera net up
$ linera service &
$ linera-indexer --storage /tmp/linera-indexer/indexer.db run --port 8088 --service-port 8081

I get only successful output, and indeed I can connect to the GraphiQL instance exposed at localhost:8088 and run queries like
query {
  plugins
  state { chain, block, height }
}

which execute successfully.
Is there an additional step I need to take to see the error?
Alternatively, is it possible that you're mixing an older version of linera (e.g. in your ~/.cargo/bin, from a previous cargo install command) with a newer version of linera-indexer?  Since the GraphQL API changed a bit in version 0.6, linera from before then won't be compatible with linera-indexer built from a recent commit.

Hi @kikakkz, I'm working on some changes to the Linera GraphQL API at the moment so it's possible that the API has changed in ways that we need to match in the linera-indexer.
However, I'm having some trouble reproducing the error you describe. If I do:
$ eval "$(linera net helper)"
$ linera_spawn_and_read_wallet_variables linera net up
$ linera service &
$ linera-indexer --storage /tmp/linera-indexer/indexer.db run --port 8088 --service-port 8081

I get only successful output, and indeed I can connect to the GraphiQL instance exposed at localhost:8088 and run queries like
query {
  plugins
  state { chain, block, height }
}

which execute successfully.
Is there an additional step I need to take to see the error?
Alternatively, is it possible that you're mixing an older version of linera (e.g. in your ~/.cargo/bin, from a previous cargo install command) with a newer version of linera-indexer? Since the GraphQL API changed a bit in version 0.6, linera from before then won't be compatible with linera-indexer built from a recent commit.

hey, @Twey , i didn't have additional step. i'll double test it today and update. and i think i run cargo install --path linera-service after checkout to v0.6.0. but please also let me double confirm again today, 😄

Hey @Twey , I test again

Hey @Twey , I test again
1  I run cargo install --path linera-service on tag v0.6.0
2  I check linera version with linera --version, and I get

3  I run linera net up in terminal 1
4  I export environment with the output of step 3 in terminal 2
5  Then I run linera service in terminal 2
6  I check indexer version with ./target/release/linera-index --version and i got

7  I run ./target/release/linera-indexer --storage /tmp/linera-indexer/indexer.db run --service-port 8081 --port 8088
8  I get error 2023-12-01T01:16:22.353573Z  INFO linera_indexer::runner: config: IndexerConfig { client: RocksDbConfig { storage: "/tmp/linera-indexer/indexer.db", max_concurrent_queries: None, max_stream_queries: 10, cache_size: 1000 }, command: Run { listener: Listener { service: Service { service_port: 8081, service_address: "localhost", tls: false }, start: BlockHeight(0) }, port: 8088, chains: [] } } Error: ReqwestError(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(8081), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) })

Could you please confirm my steps, too ?

@Twey oh, sorry. I think I use wrong port of service node.

No worries, happens to us all! :grin:


graph_optimization: update before validate

this behavior was not tested;
for some reason, when running Rich's unit tests, the backward graphs are already present -- most likely because we run forward a second time?

I'll investigate this issue. Accept to unblock.


UX suggestion

Easy one. Some users have mentioned they don't know what to do from the home page. I might suggest we somehow guide the user to click one of the 4 boxed genomes. e.g. color them, or put a fat arrow to the left of the tree...

in hipchat discussion, @djow2019 suggested using intro.js for this, which I think sounds great.  I think we'd just need 4 steps to the intro: 1) the four strains, 2) the genome search, 3) the keyword search, and 4) the "what's happening" carousel.

Great idea! But will that script come up each time someone accesses the site, or is it smart enough to know I am a return visitor and keep quiet?

It should only run for first time visitors, so it will only display once.

Here is what the tutorial looks like right now:

This looks great! What do people think about having this intro be triggered
after clicking a "get started" button somewhere near the top of the page,
instead of starting by default for first time visitors?
On Sep 30, 2017 4:08 PM, "Derek"<EMAIL_ADDRESS>wrote:

Here is what the tutorial looks like right now:
[image: screenshot from 2017-09-30 16-01-51]
https://user-images.githubusercontent.com/3160053/31050221-4e6734c4-a5f9-11e7-8f83-901b6286798e.png
[image: screenshot from 2017-09-30 16-02-03]
https://user-images.githubusercontent.com/3160053/31050223-510679b0-a5f9-11e7-9189-9689c9843375.png
[image: screenshot from 2017-09-30 16-02-10]
https://user-images.githubusercontent.com/3160053/31050224-53435a7c-a5f9-11e7-821c-751a77726be9.png
[image: screenshot from 2017-09-30 16-02-20]
https://user-images.githubusercontent.com/3160053/31050225-55092b52-a5f9-11e7-8c58-32c1a19fbece.png
[image: screenshot from 2017-09-30 16-02-33]
https://user-images.githubusercontent.com/3160053/31050226-5624c924-a5f9-11e7-9dbf-d6f8e65591b3.png
[image: screenshot from 2017-09-30 16-02-40]
https://user-images.githubusercontent.com/3160053/31050227-57276872-a5f9-11e7-9dae-21a1b3714260.png
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/SuLab/ChlamBase.org/issues/27#issuecomment-333341782,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACg2kTeAtxtIsi9mr4mTeGUnwf44pWAsks5snsnWgaJpZM4PmFEL
.

Personally, I would prefer without the button because it saves me a click. But a button at the top might be nice in case someone wants to watch the tutorial again without clearing the cache.

My vote would probably be for a 'get started' button. I like the tutorial feature.


[C#] Update Grpc.Core target from 4.5 to 4.6.2

Grpc.Core targets .NET Framework 4.5.
Visual Studio 2022 no longer supports developing net45 targets - https://docs.microsoft.com/en-us/dotnet/framework/install/guide-for-developers
net462 is the oldest .NET Framework target that is still supported - https://docs.microsoft.com/en-us/lifecycle/products/microsoft-net-framework
Consider updating Grpc.Core (and all other packages and projects) to net462.
@jtattermusch

Also here:
https://devblogs.microsoft.com/dotnet/net-framework-4-5-2-4-6-4-6-1-will-reach-end-of-support-on-april-26-2022/

@jskeet  will the .NET client libraries drop support for all the targets prior net462 with the upcoming major version bump?

@jtattermusch: That hasn't been decided for sure, but it seems reasonable to me. We'll need to check with some other internal users though.

This has been done as part of https://github.com/grpc/grpc-dotnet/pull/1651.
Ad client libraries: future versions for client libraries (4.0+) will also target net462, so this shouldn't be a problem.


x/vuln: reduce noise in example traces

Reduce the number of example traces that is output by govulncheck
Output of running on x/vulndb:
govulncheck is an experimental tool. Share feedback at https://go.dev/s/govulncheck-feedback.

Using go1.20.3
 and<EMAIL_ADDRESS>with vulnerability data from https://vuln.go.dev (last modified 2023-07-06 20:13:13 +0000 UTC).

Scanning your code and 581 packages across 60 dependent modules for known vulnerabilities...

Vulnerability #1: GO-2023-1840
    Unsafe behavior in setuid/setgid binaries in runtime
  More info: https://pkg.go.dev/vuln/GO-2023-1840
  Standard library
    Found in<EMAIL_ADDRESS>    Fixed in<EMAIL_ADDRESS>    Example traces found:
      #1: cmd/vulnreport/main.go:196:13: vulnreport.main calls log.Fatalf, which eventually calls runtime.Caller
      #2: internal/issues/githubtest/setup.go:38:11: githubtest.Setup calls testing.common.Cleanup, which calls runtime.Callers
      #3: internal/worker/server.go:168:31: worker.responseWriter.WriteHeader calls http.response.WriteHeader, which eventually calls runtime.CallersFrames
      #4: internal/worker/server.go:168:31: worker.responseWriter.WriteHeader calls http.response.WriteHeader, which eventually calls runtime.Frames.Next
      #5: internal/worker/log/gcpjson.go:14:2: log.init calls event.init, which eventually calls runtime.Func.Entry
      #6: internal/worker/worker.go:178:2: worker.CreateIssues calls event.End, which eventually calls runtime.Func.Name
      #7: internal/worker/worker.go:178:2: worker.CreateIssues calls event.End, which eventually calls runtime.FuncForPC
      #8: internal/database/legacydb/diff.go:24:18: legacydb.Diff calls cmp.Diff, which eventually calls runtime.GC
      #9: internal/database/database.go:123:21: database.VulnsIndex.MarshalJSON calls json.Marshal, which eventually calls runtime.GOMAXPROCS
      #10: cmd/vulnreport/main.go:15:2: vulnreport.init calls build.init, which eventually calls runtime.GOROOT
      #11: internal/test/packages.go:19:10: test.VerifyImports calls testing.common.Fatal, which eventually calls runtime.Goexit
      #12: internal/observe/observe.go:103:32: observe.eventHandler.Event calls otel.MetricHandler.Event, which eventually calls runtime.Gosched
      #13: internal/gitrepo/gitrepo.go:94:26: gitrepo.ReadTxtarRepo calls os.File.Write, which eventually calls runtime.KeepAlive
      #14: internal/worker/store/mem_store.go:193:12: store.memTransaction.GetCVERecords calls sort.Slice, which eventually calls runtime.MemProfileRecord.InUseBytes
      #15: cmd/vulnreport/main.go:125:3: vulnreport.main calls pprof.StopCPUProfile, which calls runtime.SetCPUProfileRate
      #16: internal/report/report.go:267:17: report.Report.Write calls os.File.Close, which eventually calls runtime.SetFinalizer
      #17: internal/derrors/derrors.go:45:19: derrors.Report calls errorreporting.Client.Report, which eventually calls runtime.Stack
      #18: internal/worker/update.go:122:24: worker.cveUpdater.update calls runtime.TypeAssertionError.Error
      #19: internal/worker/store/fire_store.go:14:2: store.init calls firestore.init, which eventually calls runtime.Version
      #20: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which calls runtime.efaceOf
      #21: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which eventually calls runtime.findfunc
      #22: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which calls runtime.float64frombits
      #23: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which eventually calls runtime.forcegchelper
      #24: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which eventually calls runtime.funcMaxSPDelta
      #25: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which eventually calls runtime.lockInit
      #26: internal/worker/update.go:122:24: worker.cveUpdater.update calls runtime.plainError.Error
      #27: cmd/vulnreport/main.go:23:2: vulnreport.init calls runtime.init, which eventually calls runtime.throw

Vulnerability #2: GO-2023-1753
    Improper handling of empty HTML attributes in html/template
  More info: https://pkg.go.dev/vuln/GO-2023-1753
  Standard library
    Found in<EMAIL_ADDRESS>    Fixed in<EMAIL_ADDRESS>    Example traces found:
      #1: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.Execute
      #2: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.ExecuteTemplate

Vulnerability #3: GO-2023-1752
    Improper handling of JavaScript whitespace in html/template
  More info: https://pkg.go.dev/vuln/GO-2023-1752
  Standard library
    Found in<EMAIL_ADDRESS>    Fixed in<EMAIL_ADDRESS>    Example traces found:
      #1: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.Execute
      #2: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.ExecuteTemplate

Vulnerability #4: GO-2023-1751
    Improper sanitization of CSS values in html/template
  More info: https://pkg.go.dev/vuln/GO-2023-1751
  Standard library
    Found in<EMAIL_ADDRESS>    Fixed in<EMAIL_ADDRESS>    Example traces found:
      #1: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.Execute
      #2: cmd/worker/main.go:121:56: worker.runServer calls http.ListenAndServe, which eventually calls template.Template.ExecuteTemplate

Your code is affected by 4 vulnerabilities from the Go standard library.

@ianthehat @zpavlinovic Would adding a flag that would just skip the .traces call do the job? https://github.com/golang/vuln/blob/master/internal/scan/text.go#L271
I could make that contribution.
As far as what the flag exactly is I'm not sure. I guess have a new value for show would be the best. Maybe -show=notraces?
https://github.com/golang/vuln/blob/master/internal/scan/flags.go#L222-L229

I think the issue is about not showing all the example traces and we can do something for that, e.g., show 10 traces and then mention there are more that can be seeing with -verbose show.
But not showing traces at all should not be provided as an option here, I think. What is the point of call analysis then? If you want to run package and module level analysis, then you can use -scan package and -scan module.


clashing HKT tag inferences glossed over

Minimum repro based on https://github.com/gcanti/fp-ts/issues/204#issuecomment-343066911, also featuring @gcanti's HKT hack from https://github.com/Microsoft/TypeScript/issues/1213#issuecomment-338379577.
TypeScript Version: 2.6.1-insiders.20171019 (Playground)
Code
interface Applicative<F> {}

interface HKT<URI, A> {
  _URI: URI
}

interface Either<L, A> {
  _URI: 'Either'
}
let right: <L, A>(a: A) => Either<L, A>

interface Option<A> {
  _URI: 'Option'
}
let some: <A>(a: A) => Option<A>
let option: Applicative<'Option'>

// 'curried' version
declare function traverse1<F>(F: Applicative<F>): <A, B>(f: (a: A) => HKT<F, B>) => any
let x1 = traverse1(option)(a => some(a)) // HKT<"Option", number[]>
let x2 = traverse1(option)(a => right(a)) // error :)

// uncurried version
declare function traverse2<F, A, B>(F: Applicative<F>, f: (a: A) => HKT<F, B>): any
let x3 = traverse2(option, a => right(a)) // no error :(, HKT<"Option" | "Either">

Expected behavior: error, probably by intersecting the forward/backward inferences into "Option" & "Either" then realizing there is no valid intersection.
Actual behavior: forward and backward tag inferences are reconciliated as "Option" | "Either", causing the x3 to (erroneously) pass.

I wonder if this has been looked at yet

Picking the largest type among the inference candidates is foundational to the inference algorithm. You'd want #14829 to make the second reference to F be excluded from the candidate set.


telepot.exception.BadHTTPResponse when sending a Photo when using Ansyc

I keep getting an error with my bot of a BadHTTPResponse when sending a photo with the Async library aio.  I get a 504 Gateway time out.  I can not figure out why there is an error.  Has anyone experienced this problem?

504 Gateway timeout likely means that Telegram servers are down. Try again later.

Do you know of a good way to handle that error?

Sorry, there is really no way to handle that error specifically. It's caught inside telepot.loop.GetUpdatesLoop (which is used by MessageLoop). But don't worry. The message loop will keep running despite the error.


Error sending email

Hello, here is a mistake in the terminal, when sAINT tries to send data to the email

why the protocolConnetct returning false? When installing saint on terminal  I checked mail password and login. Its right.

Hello,
Check if you have enabled access to less secure apps on your account. It is required.
https://myaccount.google.com/lesssecureapps?pli=1
And see if you have the lib32ncurses6 package on your OS. I'll fix now the package name, old lib32ncurses5.
Run: apt install zlib1g-dev libncurses5-dev lib32z1 lib32ncurses6 -y

Hello.
I'm having a problem with sending email as well.
I do not know why but when I rode the Windows terminal I noticed the following:
DEBUG SMTP: need username and password for authentication
DEBUG SMTP: protocolConnect returning false, host=smtp.gmail.com, user=Lucas Fraga, password=
DEBUG SMTP: useEhlo true, useAuth true
DEBUG SMTP: trying to connect to host "smtp.gmail.com", port 465, isSSL false
Sending the email to the following server failed : smtp.gmail.com:465
Sending the email to the following server failed : smtp.gmail.com:465
I entered the password correctly. What is wrong?

hey bro I don't receive any emails while the saint-1.0-jar-with-dependencies.jar is runing successfully on target host and the https://myaccount.google.com/lesssecureapps is turned on.
please help me !


Error handling

Coverage decreased (-1.0%) to 95.652% when pulling f3c46d3bf88ed397f3d1f5a8b812725569382137 on error-handling into 50a7740a271b6e47704cfad067eff7db155ea437 on master.


BZ-1274685 - New Repository dialog validation broken when Managed Repository is checked

Simple modification to allow dependency setting to tests stuff.

The following PR is related to this PR and should be previously merged

https://github.com/uberfire/uberfire-extensions/pull/103

@manstis
Hello Mike, would you mind to review this PR?

+1 to merge.

Thanks.
6.3.x: ec17dd2bca1a82ee18e7156fc27c85eb6871d341


Add yarn lockfile

I suggest to add a yarn lockfile to the project. It's going to be ignored when redux-form would be installed as a dependency, but it will make contributing experience much better. For details and reference please see: https://yarnpkg.com/blog/2016/11/24/lockfiles-for-all

Current coverage is 13.78% (diff: 100%)

Merging #2360 into master will decrease coverage by 86.21%

@@           master      #2360   diff @@
========================================
  Files          61         44     -17
  Lines        1269       1139    -130
  Methods         0          0
  Messages        0          0
  Branches        0          0
========================================
- Hits         1269        157   -1112
- Misses          0        982    +982
  Partials        0          0

Powered by Codecov. Last update 070204d...adf200f

Something went wrong with the coverage here 😕

Weird, basically tests fail when NODE_ENV set to test (which is happening in test:cov script). Investigating

Hm, the same issue with coverage was in this commit: https://travis-ci.org/erikras/redux-form/jobs/187009142 You can notice it's failing if uncover npm run test:cov

I wonder if it's one of the updated dependencies messing everything up...

I had a yarn lock file for a while, but yarn kept acting up and breaking the build in strange ways, either with linting or test coverage, so I removed it. I love the idea of yarn, but it doesn't seem quite mature enough yet.

Hm, you're right — if I use npm install instead of yarn — npm run test:cov does not throw anything. That's really odd. Do you mind if I investigate it a bit? If we'll be certain that nothing can be done here, this PR can be closed without merging. 😕

Yes, by all means, please investigate, and if you solve it, re-submit. 👍


Bug in example code

When I run python examples.py I get a syntax error in coinapi_v1:
Traceback (most recent call last):
  File "examples.py", line 1, in <module>
    from coinapi_v1 import CoinAPIv1
  File "/coinapi-sdk/python-rest/coinapi_v1.py", line 196
    self.headers = {**self.DEFAULT_HEADERS, **headers, **header_apikey}

Really is a shame they won't fix this.  My trading company revolves around python for development and data analysis with commodities and is looking for more crypto data. You would think they'd put some effort into their product.

Anyone have a fix for this?

I believe that it's not a bug, but just incompatibility with previous versions of Python. The example code was written for the 3.X version of python interpreter.

Good call. I had been trying it with both 2 and 3 but I think that there was a issue with my installation of 3. Reinstalled 3 in a different main directory from 2 and it works now. Thanks for letting me know it was written for 3.


Invalid version numbers in nuspec trigger an unhandled exception (WSOD) on upload.

Create a .nupkg file by hand whose version number in the nuspec is invalid (for the purposes of this example, let's assume it's <IP_ADDRESS>.0).
Attempt to upload it using the gallery UI.

RESULT:  You'll get an unhandled exception on upload.  Investigating error logs reveals the underlying error is an ArgumentExeption with the error message:
System.ArgumentException: '<IP_ADDRESS>.0' is not a valid version string. Parameter name: version
The full stack trace is available at https://qa.nugettest.org/Admin/Errors.axd/detail?id=2213.
EXPECTED:  We should never WSOD on customers, even in fringe scenarios like this. If we can treat invalid package uploads in the Gallery the same way we do for validation errors, that would be ideal.

We need much better validation.

This no longer happens.
Web UI:

Push from command line:
Pushing e2e.semver1stable.170503.225836.7811<IP_ADDRESS>.nupkg to 'https://dev.nugettest.org/api/v2/package'...
  PUT https://dev.nugettest.org/api/v2/package/
  BadRequest https://dev.nugettest.org/api/v2/package/ 623ms
Response status code does not indicate success: 400 (Failed to read the package file. Ensure it is a valid NuGet package with a valid manifest.).


feat: add option to run in debug mode

Execute function become too big, I propose to put Rc.read and folder creation in an init function.

Doesn't bother me that much (that's quite common in CLI). Spliting with no improvement on how we abstract the implementation is quite useless and just make things harder to maintain later.


[Toggl Button] Rework tag selection

🐛 Bug Report
When I type a tag I have to click on it in the chrome extension menu which breaks flow and is clunky. If I push enter, the menu goes away but none of my tags are added if I had not clicked on them. It's also not clear when a Tag is assigned to a Time Entry.
Expected behavior
Toggle Button extension's tagging interaction should be similar to the desktop app's (macOS 7.4.231 beta)
To reproduce
Attempt to assign a Tag to a Time Entry via Toggl Button, and do the same with the desktop app
Suggestions
Toggle Button extension's tagging interaction should be similar to the desktop app's (macOS 7.4.231 beta)
Context
From this conversation. @dooart requested the creation of this issue here.

Let me work on this :)


Add ability to set Assignee and Epic on Jira sink

Added the ability to set the assignee and Epic (link) in the JIra sink config

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


Use file_cache_path path to store chef-client

Use file_cache_path path to store chef-client instead of /opt

:+1:

The downside here is that the omnibus updater will delete cached copies of Chef Server that have been downloaded into the file cache dir on servers.  This results in the rather large chef server file getting downloaded on every chef run.  This change should happen, but the deletion logic in the omnibus updater needs to be tightened up to not delete server files.

Path updated!


feat: dialog content enhancement

I found that the current implementation of the  CommonDialog cannot display multi-line strings well (cannot parse '\n')

In fact I want to show a list of resources to be deleted when I click the delete unused resources button (in my later commit), so I found this issue and I want to make sure that such a modification is ok.


Add parser for MSVC /sourceDependencies output

This adds depfile support for the new /sourceDependencies format coming in MSVC in Visual Studio 16.7 and currently available in public preview. #1806 has more discussion about this flag.
/sourceDependencies outputs a UTF-8 encoded JSON file with information about the headers used during compilation. With the changes in this PR, Ninja rules that specify deps = msvc_source_dependencies will consume a JSON file specified by depfile to compute dependency information.
Within Ninja, RapidJSON is used to parse the JSON file and extract paths. Because Ninja on Windows uses the ANSI APIs instead of the Unicode APIs, we also have to convert the paths from UTF-8 to the current code page using a combination of MultiByteToWideChar and WideCharToMultiByte. This conversion will only happen on Windows and will partially address #1766.

What's the minimum language standard that Ninja supports? I didn't see anything documented in the coding style, but the Linux and macOS builds are failing because simdjson requires C++11 or greater.

See here for the C++ minimum language standard: https://github.com/ninja-build/ninja/issues/1403

I switched from simdjson to RapidJSON to be C++98 compatible.

@benmcmorran
Honestly I wish you wouldn't. Accommodating the Ninja project's decision to use an ancient language variant is counterproductive and only makes the problem worse.

Thanks for the PR!
Should this be Windows-only? This way we wouldn't include the big JSON dependency on the other platforms (or is there a use-case I'm missing?).

Yes, I can make this Windows-only. The new format is only supported by MSVC (as far as I know).

@jhasse a few questions about including RapidJSON:

Is configure.py deprecated?
If no, are you okay with directly including the RapidJSON source code in the repo like this PR is doing, or should I modify configure.py to have some crude dependency management system?
If yes, can I remove configure.py and just use CMake to manage the RapidJSON dependency? This avoids having RapidJSON checked in.

@jhasse Do you have any other concerns or is this good to merge? I can squash commits to keep history clean.
I removed the UTF-8 to ANSI conversion function, so this PR no longer has a dependency on #1671. It won't correctly handle paths with non-ASCII characters, but that's a pre-existing issue with Ninja on Windows if your console codepage doesn't match the encoding of the build.ninja file.

Yes, please squash the commits.
Before merging this needs further testing and Ninja 1.10.1 needs to be released. Also I won't merge the Microsoft copyright header, someone else needs to do that.

Thanks @jhasse, I've squashed the commits. By "further testing" do you mean adding additional automated tests in this PR, or manually testing on a variety real-world systems? Given that you are unable to merge the Microsoft copyright header, who else is able to merge PRs to the Ninja project?

@benmcmorran nico and evmar

By "further testing" do you mean adding additional automated tests in this PR, or manually testing on a variety real-world systems?

The latter ;) The unit tests are great, very good work.

closing in favor of #2200


Outdated documentation

Hi guys! I think this: https://github.com/airbnb/MvRx/wiki/Advanced-Concepts#dependent-requests is kind of outdated. It makes reference to onSubscribe, which I'm not entirely sure if its an internal Airbnb method or an old method which got replaced by subscribe and asyncSubscribe.
Let me know, I can PR changes if needed

Thank you, I have updated it to:
init {
    asyncSubscribe(MyState::request1), onSuccess = {
        fetchRequest2()
    })
}


Remove DTLS 1.3 specific code

As discussed internally, this code is no longer maintained or tested, and moreover the plan is to implement DTLS 1.3 on top of MPS once TLS 1.3 + MPS are done.

Version of the prototype prior to this PR: https://github.com/hanno-arm/mbedtls/tree/tls13-prototype-dtls13


Fix map controls crashing when invalid key is provided

I'm having the same issue reported on #276.
I tested it and adding optional chaining before calling indexOf fixes it, but let me know if you prefer a more verbose approach. 👍
closes #276

Simple and works, gotta love it :)
Thanks a lot for the fix <3


uninstaller: fixed bug for flooded error

Also, need a detail commit message:

Explain how to reproduce the issue and how fix works
if someone has a similar issue, they should able to google/search the log and found something here.


Menus

Vertical and Horizontal Menus + sub-menus.
Classes for dropdown/-up, positioning and sets.

Closed since Goals and internal marks are not getting tracked on Issues.

Why that in the first place? >_>


Installing from source with go doesn't work

Hey!
I tried installing from source using go. I have set GOBIN in my PATH.
This is the output I get when running the command specified in the README:
> go get -u github.com/gitguardian/src-fingerprint/cmd/src-fingerprint
go: downloading github.com/gitguardian/src-fingerprint v0.14.0
go get<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: srcfingerprint
	        but was required as: github.com/gitguardian/src-fingerprint

Hello @ShaigroRB,
I think the instructions are not exactly correct in this section. Can you try the following :
git clone github.com/gitguardian/src-fingerprint && cd src-fingerprint
make build
./src-fingerprint --version

I got it working by cloning via ssh and building with make.
git clone git@github.com:GitGuardian/src-fingerprint.git
cd src-fingerprint && make build

Thanks for your help!


Indian Railways is misformatted

It's missing the "auth" section, so "https" and "link" are located in the wrong columns.

Fixed in commit db03add118d8a2205b2de63d400865fe4c169f62. Thanks for catching that @JoshCheek !


Features formats can be determined by the YML file for develop

The YAML config file is now capable to set the format of annotated features.
The example chloroplast data set YAML now contains default configuration for

inverted repeates
ndh genes
ycf genes

fixes #89

Approved

Coverage decreased (-0.2%) to 99.128% when pulling 8b6179b76013fa442bcd11798f6402fc6df4820e on hotfix/feature-format-from-yml into d830bece9bb550ba871d4f10b23bcdeffa95b408 on develop.


Create key if does not already exist

Fixes #3

Verify provided key vault exists, if not, error
Get key version of the provided key, if key does not exist, create it
Verify key version if provided, if key version does not exist, error. If key version is not provided, get key version with the provided key name

/lgtm


Add message logging to streaming RPCs.

This PR adds support for logging messages to streaming RPCs where possible. The one case that isn't logged are calls that stream requests, but take a unary response; in this case, if the RPC is invoked asynchronously then the span will be finished before the requests are iterated over, so they can't be logged to the span.

Ok, so I think it's fine to leave logging off the streaming case for now. We can circle back around to it later if someone cares.


feat: added minor perf optimizations for charts

Description, Motivation and Context:
Aims to publish a new release package with Chart Animations and slightly improved performance

Please also update the README.md


Start search when "key enter" is pressed

Hi,
with this PR you can start search when "key enter" is pressed.
I tested this change in local.
If you think it is useful merge it.

Thank you!! This is very useful.


feat: add address.road and address.full methods

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.70%. Comparing base (ac302ac) to head (e5067c8).
Report is 2 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #3   +/-   ##
=======================================
  Coverage   99.69%   99.70%
=======================================
  Files          21       21
  Lines         655      673   +18
  Branches      132      135    +3
=======================================
+ Hits          653      671   +18
  Misses          1        1
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


WIP: #241 shouldi got Cargo-audit for RUST code

added cargo_audit.py and test_cargo_audit.py in examples/shouldi/shouldi and examples/shouldi/tests respectively.

@pdxjohnny  pls review !!!

Closed because of https://github.com/intel/dffml/pull/493


Storport: the hyperlink of HW_INITIALIZATION_DATA should be link to storport not strmini

[Enter feedback here]
the hyperlink of HW_INITIALIZATION_DATA (Remarks part) should be link to storport not strmini...
https://docs.microsoft.com/en-us/windows-hardware/drivers/ddi/storport/ns-storport-_hw_initialization_data~r1

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

ID: f1e290fc-6386-21b8-0af3-242acb582a8e
Version Independent ID: 249f0dd4-053f-e9e0-e0a7-ba8a2308a4af
Content: StorPortInitialize function (storport.h) - Windows drivers
Content Source: wdk-ddi-src/content/storport/nf-storport-storportinitialize.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @lorihollasch
Microsoft Alias: loriwhip

Thanks for catching that, @BlackINT3!
Just checked in the fix. Should be live shortly.
Eliot


[Dubbo-4218] Fix NPE when the TagRouterRule addresses config is null (#4218)

What is the purpose of the change
Fix NPE when the TagRouterRule addresses config is null #4218
Brief changelog
Fix NPE when the TagRouterRule addresses config is null #4218
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4236 into master will decrease coverage by 0.09%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master    #4236     +/-   ##
===========================================
- Coverage     63.01%   62.91%   -0.1%
  Complexity      505      505
===========================================
  Files           769      769
  Lines         33058    33061      +3
  Branches       5217     5217
===========================================
- Hits          20830    20801     -29
- Misses         9823     9853     +30
- Partials       2405     2407      +2

Impacted Files
Coverage Δ
Complexity Δ

...bo/rpc/cluster/router/tag/model/TagRouterRule.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...ache/dubbo/remoting/p2p/support/AbstractGroup.java
45.45% <0%> (-11.37%)
0% <0%> (ø)

.../remoting/transport/netty4/NettyServerHandler.java
61.9% <0%> (-9.53%)
0% <0%> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
51.42% <0%> (-5.72%)
0% <0%> (ø)

...ng/exchange/support/header/HeartbeatTimerTask.java
73.68% <0%> (-5.27%)
0% <0%> (ø)

.../org/apache/dubbo/remoting/ExecutionException.java
15.78% <0%> (-5.27%)
0% <0%> (ø)

...bbo/registry/support/ProviderConsumerRegTable.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

...exchange/support/header/HeaderExchangeHandler.java
64.75% <0%> (-2.46%)
0% <0%> (ø)

.../exchange/support/header/HeaderExchangeServer.java
66.98% <0%> (-1.89%)
0% <0%> (ø)

...he/dubbo/registry/multicast/MulticastRegistry.java
67.87% <0%> (-1.81%)
0% <0%> (ø)

... and 3 more

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 346630e...e038606. Read the comment docs.


needs_external_needs: Support local needs.json

There is a (well-hidden) option in intersphinx to load the objects.in file from a local directory or one or more mirrors.
I'm looking for a way to load an external needs.json file from a local directory.
This could be solved by adding an optional setting like "json_path", which just loads a file instead of attempting HTTP download.
Use Case:
Given two projects, A and B, which are built locally, the following can be used to generate an external-needs mapping:
|- project_A/
|  |- build
|  |  |- needs.json
|  |  |- html/
|  |- src/
|  |  |- conf.py
|- project_B/
|  |- build/
|  |  |- needs.json
|  |  |- html/
|  |- src/
|  |  |- conf.py

project_B/src/conf.py:
needs_external_needs = [{json_path = "../../project_A/build/needs.json" ...}]

I think there is also a hidden feature for needs_external_needs to use a local needs.json.
The json_url can have the format file://my/local/path/needs.json.
So instead of http:// you can use file://.
But this is totally missing in the docs.
So I keep this ticket open, until I have fixed the docs.

well hidden indeed. :-) I ran a local python http server to circumvent the limitation until now.

Please document that the URL has to be file:///home/local/path/needs.json, with three slashes after file:
Or else you get this error:
Traceback (most recent call last):
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinxcontrib/needs/external_needs.py", line 24, in load_external_needs
    response = s.get(source["json_url"])
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/requests/sessions.py", line 555, in get
    return self.request('GET', url, **kwargs)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/requests/sessions.py", line 542, in request
    resp = self.send(prep, **send_kwargs)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/requests/sessions.py", line 655, in send
    r = adapter.send(request, **kwargs)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/requests_file.py", line 34, in send
    raise ValueError("file: URLs with hostname components are not permitted")
ValueError: file: URLs with hostname components are not permitted

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/str2lr/venvs/sphinx-doc/bin/sphinx-build", line 8, in <module>
    sys.exit(main())
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/cmd/build.py", line 294, in main
    return build_main(argv)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/cmd/build.py", line 280, in build_main
    app.build(args.force_all, filenames)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/application.py", line 350, in build
    self.builder.build_update()
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/builders/__init__.py", line 294, in build_update
    len(to_build))
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/builders/__init__.py", line 306, in build
    updated_docnames = set(self.read())
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/builders/__init__.py", line 402, in read
    self.events.emit('env-before-read-docs', self.env, docnames)
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinx/events.py", line 101, in emit
    results.append(listener.handler(self.app, *args))
  File "/home/str2lr/venvs/sphinx-doc/lib/python3.7/site-packages/sphinxcontrib/needs/external_needs.py", line 27, in load_external_needs
    raise NeedsExternalException(f'Getting {source["json_url"]} didn\'t work. Reason: {e}')
sphinxcontrib.needs.external_needs.NeedsExternalException: Getting file://home/.../workspace/build/documentation/needs.json didn't work. Reason: file: URLs with hostname components are not permitted

Ohh thanks for the hint and for testing it.
Then the relative path support is not working.
Will fix this too.

Again: It works for me. IF you spell it file:///...

Sphinx-needs doesn't support relative paths to an "external needs" project, though.
This would come in handy for relocatable multi-project build documentation, i.e. if you put the documentation in such a web folder structure:
<webroot>
|- master
|  |- project_a
|  |- project_b  # reference project A
|  `  ...
|- release
|  |- 1.0
|  |  |- project_a
|  |  |- project_b  # reference project A
|  |  `  ...
|  |- 1.1
|  ` ...

Intersphinx supports this:
intersphinx_mapping = {'project_a': ('../../project_a/html',
                                  (os.path.join(project_root, 'build', 'project_a', 'html', 'objects.inv'), None))}

Result:
:doc: links will be converted to links in the following pattern:
<relative path to my project root>/<relative path to other project root>/<relative document path>.html
e.g. if you give have a role :doc:project_a:tools/index in a file project_b/html/doc/design.rst, the relative link will be:
../../../project_a/html/tools/index.hml
It now does not matter whether the documentation is in a relative path master/ or release/1.0/ or release/1.1/

What would probably be nice from a usability perspective is to check whether the json_url has a scheme component (contains ://), if it doesn't then assume it is a relative path and construct a url in the form file:///<project_root>/<json_url>

I agree.
It was a fast solution to ask only for an url and then let the used request library handle the download.
But now we should do the url vs. path handling by our own, so that it can be more stable.
I like the idea of having another option called json_path.
So the user can decide if he/she uses json_url or json_path.
And then for json_path absolute or relative path handling becomes quite easy,

@twodrops  @danwos, sorry, I was out of office in October!
The json_path option works fine but the relative base path feature I requested in this comment is not yet there. I will create a new issue for it and I hope you can somehow add it to 0.7.3 🙂


trainable attribute seems to be broken in keras 1

The trainable attribute seems to be broken in Keras 1.
From the Layer docstring:
trainable: boolean, whether the layer weights will be updated during training.

Sadly the trainable attribute has no effect. For example both Convolution2D and Dense layers
set  always trainable_weights ignoring the trainable attribute.
My solution would be to rename the trainable_weights  member to _trainable_weights
and then add this method:
@property
def trainable_weights(self):
    if self.trainable:
        return self._trainable_weights
    else:
        return []

If you are fine with my solution, I can get  a pull request ready.

Seems to be fixed now