how to create two way binding?

Hi guys,
I am new to BackboneJS, I have created bootstrap modal pop up view as a child view, which is calling from parent view.
In modal pop up, i want to add or hide elements based on dropdown box. but currently modal pop up disappears after rendering..
Below is  code
 var RepeatModalView = Backbone.View.extend({
el: '.repeatmodal_tem',
elements: {
    "uRepeateverybyweek": true,
    "uRepeateverybyday": false,
    "uRepeaton": true,
    "uRepeatby": false,
},
events: {
    'change .select_rctype': 'select_rctype',
    'hidden': 'close'
},
initialize: function() {
  //_.bindAll(this, 'show', 'teardown', 'render', 'select_rctype');
  this.template = _.template($('.repeatmodal_template').html());
  this.render();
},
show: function() {
  this.$el.find('#RepeatModal').modal('show');
},
close: function() {
  this.$el.data('modal', null);
  this.remove();
},
render: function() {
  this.$el.html(this.template(this));
  //this.$el.modal({show:true}); // dont show modal on instantiation
},
select_rctype:function(event) {
    this.rctype = $("#uRepeats option:selected").val();
    console.log('rctype', this)
    if (this.rctype == "d"){
        this.elements.uRepeateverybyweek = false;
        this.elements.uRepeateverybyday = true;
        this.elements.uRepeaton = false;
        this.elements.uRepeatby = false;
    }
    else if (this.rctype == "m"){

    }
    else if (this.rctype == "y"){

    }
    else{

    }
    return this;
}
});

In your view use a backbone model and backbone.stickit (http://nytimes.github.io/backbone.stickit/) to realize two way databinding.

Also you can use https://github.com/gmac/backbone.epoxy, but its an issues and not QA please use https://stackoverflow.com/ or https://gitter.im/jashkenas/backbone
Thanks.

Hi Nicola,
I have checked backbone.stickit but i am not getting how to bind elements with backbone.stickit.
elements: {
    "uRepeateverybyweek": true,
    "uRepeateverybyday": false,
    "uRepeaton": true,
   "uRepeatby": false,
},

Can you provide short example  that will great help?

Hi @kdbusiness90  elements must be a backbone model so below write little example for you:
var RepeatModalView = Backbone.View.extend({
el: '.repeatmodal_tem',
elementsBinding: {
   //here set your  property bindings as specified in styckit documentation:   http://nytimes.github.io/backbone.stickit/#bindings
},
events: {
    'change .select_rctype': 'select_rctype',
    'hidden': 'close'
},
initialize: function() {
  this.elements = new Backbone.Model({
    "uRepeateverybyweek": true,
    "uRepeateverybyday": false,
    "uRepeaton": true,
    "uRepeatby": false
  });
  //_.bindAll(this, 'show', 'teardown', 'render', 'select_rctype');
  this.template = _.template($('.repeatmodal_template').html());
  this.render();
},
show: function() {
  this.$el.find('#RepeatModal').modal('show');
},
close: function() {
  this.unstickit(this.elements);
  this.$el.data('modal', null);
  this.remove();
},
render: function() {
  this.$el.html(this.template(this));
  //this.$el.modal({show:true}); // dont show modal on instantiation
  this.stickit(this.elements, this.elementsBinding);
}
});

This is only the base structure that you should follow. Please refer to styckit documentation for the rest. It's really exaustive.

These kinds of discussions are best had on one of the other channels:

For questions on working with Backbone or general discussions: https://groups.google.com/forum/#!forum/backbonejs, http://stackoverflow.com/questions/tagged/backbone.js, or https://gitter.im/jashkenas/backbone


Patch OSS vulnerabilities

Apply patches for:

CVE-2024-5535 in openssl
CVE-2024-7006 in libtiff

Update following packages to include fixes for reported vulnerabilites:

FFmpeg 7.0.1 -> 7.0.2
curl 8.8.0 -> 8.9.1

17890009, 17890048


Add links to researchers and responsible disclosure programs

Added references to the work of Ken Munro and Pen Test Partners (the team behind the Mitsubishi Outlander vulnerabilities).

Also added links to three major vehicle companies' active bug bounty programs (in a new sub-section).

Thanks so much!

No problem @jaredmichaelsmith! Working on a few more updates; will send a pull request when finished.


Apply API review fixes for MTRPersistentStorageDelegate.h

Rename the protocol (and header).
Fix some comments.

Addresses part of https://github.com/project-chip/connectedhomeip/issues/22420
Issue Being Resolved

Fixes #22542

Change overview
See above.

Fast-tracking platform-specific fix with platform owner review.


bypass403: init at 1.4.0

Motivation for this change
Tool to bypass 403 Forbidden responses
https://github.com/drsigned/bypass403
Related to #81418
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 109820 run on x86_64-linux 1

1 package built:

bypass403

Result of nixpkgs-review pr 109820 run on x86_64-linux 1

1 package built:

bypass403

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109820 run on x86_64-linux 1

1 package built:

bypass403

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109820 run on x86_64-linux 1

1 package built:

bypass403


More snippets!

Great start, would love to see this project expand to contain the full typescript-ified version of the reactjs snippets here: https://github.com/xabikos/vscode-react/blob/master/snippets/snippets.json

I've made a start, but I'll be looking into it this weekend.

And why double quotes?

@Meshock Personal preference, sorry ;).


fix(ActionControllerMixin): Visibility of controller

ActionControllerMixin has a private controller, inaccessible from the class that uses the mixin. This bug makes mixin useless in terms of reducing boilerplate code.

@adar2378 can you please comment?  (and don't merge)

@IlyaZadyabin Nice catch! Yes, I agree that the private controller is inaccessible from the mixin.

Why would we need to access controller outside of the mixin?

Honestly I think the mixin makes zero sense, we should move all functionality to DefaultActionController?


Add tests for the SAML flows

The SAML WS flows do not have tests.

There are tests for the SAML password client flows, but not for the endpoints.


HSEARCH-2679 Improve error handling/reporting around Elasticsearch request sending

https://hibernate.atlassian.net/browse/HSEARCH-2679

merged!


fix the bug causing unnecessary gas fee increases

Issue: When mining FT and NFT, the setting of the satsbyte parameter is ineffective, and the actual gas fee paid is always higher than the set value, sometimes even up to 10 times higher, for example, when mining electron, the gas fee is 22.67U: https://www.blockchain.com/explorer/transactions/btc/1840000a132746b15ff545c075fe12e55ba69f1ffa57856a3bddf21f684a2c38).
Cause: When sending the commit transaction, unnecessary additional fees (this.getTotalAdditionalOutputValues()) are added in the AtomicalOperationBuilder::addCommitChangeOutputIfRequired method while obtaining the fee for the reveal transaction. In fact, this portion of the fee is already included in revealFeePlusOutputs and should not be added again.

Solution: Modify "const totalOutputsValue = this.getTotalAdditionalOutputValues() + fee.revealFeePlusOutputs;" to "const totalOutputsValue = fee.revealFeePlusOutputs;"
Has it been tested: Yes, transaction for mining electron has been tested below.
Gas fee becomes 1.49U when mint electron now: https://www.blockchain.com/explorer/transactions/btc/1840000b56f0a8a9fed31753dfa29fad00048d9b0905dd770f7929e888e3419b

非常感谢，原本都不敢用程序mint了，太贵了~

Great job.

Using current fix.
I had sent a tx with satsbyte=10.
Commit tx 12 sat/vB.
Reveal tx 37 sat/vB.
I have no problem with the 12 sat/vB caus I saw there is an extra_buffer the fee calculations are adding.
But the 37 sat/vB - the issue doesn't seem to be fixed.
Maybe I was in the right direction after all in issue #77.
Now that I look deeper at the original tx published above by @syslink.
It ain't look right to me.
https://bitcoinexplorer.org/address/bc1pjp3rwu4xp97gup8223h4ll9f23f7e6h3jdz2hgupt307cpsdtu6qdwkvn9
commit tx 24 sat/vB
reveal tx 52 sat/vB

A fix was made to --satsbyte, please verify


FO: add error message modal display

I've made some modifications to handle cart adding errors.

Good idea ! thanks

Good idea, thanks !

@reho
I just tested your PR
The modal isn't displayed anymore if the product is out of stock and the product page is not refreshed.
So, it's like add to cart button didn't do anything, but the product is anyway added to the cart and the product page is like blocked
Video: https://drive.google.com/file/d/1F67XapHrknmIH4oAqC1Vk1ljQsQEi5SJ/view?usp=sharing

Interesting idea 🤔
This PR might need some updating as it was done a long time ago, but the idea has potential 👍

Issue was fixed by PR PrestaShop/ps_shoppingcart#41 so this PR is not relevant anymore
Thank your for the contribution anyway, sorry for the late answer


A lot of Pending Requests

Hi. I am using Angular 2 and Webpack for developing an Atom with atom-typescript. On every save I have increasing of pending requests. I tried to find solution, however they are old...

Thanks!

I have a similar issue. Please help us on this one, because working in Atom right now is a bit frustrating :)

Do you mind posting your tsconfig file?

I have same problem. My project is build on: https://github.com/preboot/angular2-webpack with almost no changes... So my tsconfig is:
{
  "compilerOptions": {
    "target": "ES5",
    "module": "commonjs",
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "sourceMap": true,
    "noEmitHelpers": true
  },
  "compileOnSave": false,
  "buildOnSave": false
}

I had the same issue. Basically, the plugin wasn't able to find my ts files and thats why the pending requests were getting queued up. I got around it by including
"filesGlob": [
      "**/*.ts"
    ],

and
"atom": {
    "rewriteTsconfig": true
  }

In my tsconfig. This will rewrite the tsconfig file by scanning through your application for .ts files. Also, make sure that the tsconfig is the root folder of your app src and does not have node_modules as any child.

@DNRN @nikhildev @pstawinski the solusion is very easy:
tsconfig.json
{
  "compilerOptions": {
    "target": "es5",
    "module": "commonjs",
    "moduleResolution": "node",
    "sourceMap": true,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "removeComments": false,
    "noImplicitAny": false
  },
  "exclude": [
    "node_modules"
  ],
  "compileOnSave": false,
  "buildOnSave": false
}

We add:
"exclude": [
    "node_modules"
  ]

and compiler ignore the tone of aditional files.

Thank you!

Excluding node_modules seemed to help briefly, but after a few seconds, I'm back with the same kind of errors:

Related issues: #968, #1037. There are several old related issues as well (#728, #760, #556, #349), but they seem to have been resolved at one point and now the problem is back. @basarat, any ideas?
Here are my latest errors:

So, another data point. I originally had tsconfig referencing files like this:
	"files": [
		"src/ts/main.ts",
		"typings/index.d.ts"
	]

My main.ts file is the entry point and all the other files are referenced via imports. I thought that would be the best way, and I would avoid spending time compiling files that I wasn't currently referencing anywhere. If I change that to this, though, it seems to work fine:
	"filesGlob": [
		"src/ts/**/*.ts",
		"typings/index.d.ts"
	]

This apparently lets TS build its index much faster and seems to be the recommended way. My question is, though, is my first way supposed to be supported?

I'm also quite confused about how different the performance is between atom-typescript and VSCode with an angular-cli generated project (I haven't tried any other project though tbh).
I have a component like this:

Any solution for this? I am having the same problem in a Anguar 2 project, for me Infinity Pending Requests and the number endless increases.
tsconfig.json
{
  "compilerOptions": {
    "declaration": false,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "lib": ["es6", "dom"],
    "mapRoot": "./",
    "module": "es6",
    "moduleResolution": "node",
    "outDir": "../dist/out-tsc",
    "sourceMap": true,
    "target": "es5",
    "typeRoots": [
      "../node_modules/@types"
    ]
  },
  "exclude": [
    "node_modules"
  ]
}


test: fix matching tls certificate error for Go1.20

Go1.20 adds more info to the tls error. Now it reports:
tls: failed to verify certificate: x509: certificate signed by unknown authority
Close #16

Thank you, @zhsj. Fixed.


Result of TakePhotoAsync is a solid grey colored rectangle

Please fill out either the bug or feature request section and remove whatever section you are not using.
Bug
Version Number of Plugin:  2.6.2
Device Tested On: Lenovo Tab3 (TB3_X70F)
Expected Behavior
Image is not stored in gallery but stored correctly in the pictures folder
Actual Behavior
Image is stored in gallery but the image in the pictures folder only contains a solid shade of grey
Steps to reproduce the Behavior
Just use TakePhotoAsync and check file in folder and gallery
Interesting side information... this happens on the default camera app that comes with the installation and only if the picture is taken right away, if I wait for ~10 seconds before taking the picture the image in the pictures folder is correct.
Also, when I install an alternative camera app, the behaviour is correct.

same issue on Sony Xperia Z2.
If you do not wait ~7 seconds before tapping button in camera app, we get a bitmap of the size 1x1

This also happens on LG-G3s, identical behaviour to what @HopScotch47 described.
The image saved is a 1x1 grey square, approximately 630 bytes in size.

I see that the bug is fixed in 2.7+ versions.
but...
version <IP_ADDRESS> is the latest version with normal behavior setting orientation.
in 2.7+ if i take a photo in portrait rotation - returned photo always in landscape.
this is cricical for me. So i'm staying on <IP_ADDRESS>

Seems to be fixed already. Can't reproduce locally at all.


Properly render images linked in support documentation

Updated occurrences of invalid image urls which were forgotten when changing support documentation.

:1st_place_medal:
So nice! Thank you!


[Bug]: Calculate the number of invalid PIN attempts during the Send flow

Describe the bug
It is currently possible to have unlimited attempts to enter the PIN during the Send flow.
This should be limited to 10 attempts, as it is in the Start flow.
Reproduce

Setup PIN
Enable PIN for Send
Try to enter wrong PIN 10 times

Screenshots / Recording
No response
Operating system
iOS, Android
Bitkit version
No response
Log output
No response

fixed in #2311

Verified in v1.0.8
Wipes data after 8 unsuccessful pin attempts on send


Fixed a bug with remote rules.

Take a look at 8ad45e95b3524877ceec7b02fc41e581eae75a83

Needs to be merged into the master.


Added Flux Pavilion and other KubeCon EU info

Signed-off-by: mewzherder<EMAIL_ADDRESS>

fyi @staceypotter @dholbach @scottrigby


fix compile error

fix https://github.com/dmlc/mxnet/issues/4584

Do you know why that bug occurs?
MXNetError: Cannot find argument '', Possible Arguments:

@sethbrin May be we cannot use {{}}?

I know, I just test the unordered_map<string, string> table = {{}}, the size of table is 1
the constructor just insert the empty ""=>"" to the table.


For BE: Service discovery service

Services are on different ports normally automatically done by gateway.
But in the microservices cluster we need to override the IP address and port.
So user service with version X should connect to ip-address XYZ on port X
Add an option to overwrite service endpoints
Add an option to overwrite authentication

import { createClient } from "@extrahorizon/javascript-sdk";

const sdk = createClient({
  microservices: true,
  serviceUrls: {
    users: 'http://<IP_ADDRESS>:3000',
    data: 'http://<IP_ADDRESS>:3000'
  }
});

await sdk.auth.authenticate({
  secret: 'very-secret-string'
});

We could also have a different name like createInternalClient ?
The secret can be used to retrieve tokens or is it just added to the headers?

import { createClient } from "@extrahorizon/javascript-sdk";
export interface ServiceLocatorInfo {
  service: string;
  version?: string;
}
export interface ServiceLocation {
  host: string;
  port: number;
}
const sdk = createClient({
  microservices: true,
  serviceUrlFn: (service: ServiceLocatorInfo):ServiceLocation  => {
    return ServiceUrls[service]
  }
});

await sdk.auth.authenticate({
  secret: 'very-secret-string'
});

import { createClient } from "@extrahorizon/javascript-sdk";

const sdk = createClient({
  microservices: true,
  serviceUrls: {
    users: 'http://<IP_ADDRESS>:3000',
    data: 'http://<IP_ADDRESS>:3000'
  }
});

await sdk.auth.authenticate({
  secret: 'very-secret-string'
});

We could also have a different name like createInternalClient ?
The secret can be used to retrieve tokens or is it just added to the headers?

@jensdev I understand the proposal. But the issue is that services can change from IP and Port very frequently. When you would do it this way the IP and port number are provided to the SDK on initialization and startup of the microservice.

import { createClient } from "@extrahorizon/javascript-sdk";
export interface ServiceLocatorInfo {
  service: string;
  version?: string;
}
export interface ServiceLocation {
  host: string;
  port: number;
}
const sdk = createClient({
  microservices: true,
  serviceUrlFn: (service: ServiceLocatorInfo):ServiceLocation  => {
    return ServiceUrls[service]
  }
});

await sdk.auth.authenticate({
  secret: 'very-secret-string'
});

@jensdev this would indeed be the way to go


Add config files to deploy to IBM Cloud

Detailed steps are at https://github.com/csci-ga-2820-fa22-003/wishlists/issues/58#issuecomment-1316265290
Issue#58

Codecov Report

Merging #69 (94a1c39) into main (34616fe) will decrease coverage by 0.23%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##             main      #69      +/-   ##
==========================================
- Coverage   97.42%   97.19%   -0.24%
==========================================
  Files           7        7
  Lines         389      392       +3
==========================================
+ Hits          379      381       +2
- Misses         10       11       +1

Impacted Files
Coverage Δ

service/routes.py
99.43% <66.66%> (-0.57%)
:arrow_down:

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


Bug: When adding web parts to modern pages the order option is ignored

The order option in the spo page clientsidewebpart add command is not used for anything, while it should make it possible to specify the placement of the web part in the specified column. Currently, web part is always added to the end of the column.

@ypcode if you haven't started working on this, I'd be glad to pick it up. I've include the needed logic (based on your original code) in the page text add command so could easily reuse it.

Hi Waldek, I was planning to do it this week-end, but if you are in the middle of this, it is probably more efficient :) the text command is working well then ? have you had to change anything ?


 I didn't encounter this issue in previous patch

Edit: I've  just encountered this issue for lagoon dragon kaisa too

Currently, Kaisa's skin model will disappear after rebirth. It is estimated that Kaisa is rebirth from the void. I have no idea about that. So I modified the code to require the user to switch Kaisa skins manually. It's better to have a model than nothing, right?
you can download zip from v13.5.2

目前，凯撒的皮肤模型将在重生后消失。据估计，凯撒是从虚空中重生的。我对此一无所知。所以我修改了代码，要求用户手动切换凯撒皮肤。有模型总比没有好吧？
您可以从以下网址下载zipv13.5.2

Can you fix this Kha'Zix intensify R the body will be disordered?

Currently, Kaisa's skin model will disappear after rebirth. It is estimated that Kaisa is rebirth from the void. I have no idea about that. So I modified the code to require the user to switch Kaisa skins manually. It's better to have a model than nothing, right?
you can download zip from v13.5.2

Thanks

This is due to one of the side effects of repeated reads and writes in stack->push, See #337

目前，佳兆业的皮肤模型会在重生后消失。估计佳兆业是虚空重生。我对此一无所知。所以我修改了代码，要求用户手动切换佳兆业皮肤。有模型总比没有好吧？
你可以从v13.5.2下载 zip

卡萨是在虚空中重生，有1.5秒左右的半隐身状态，可以在卡莎复活后的1.5秒后调用换肤函数


compatibility with Ngrok LAN

So, this mod reaaaaally helps out when used with ESSENTIAL, and simple voice chat, because everytime you open LAN, the port always changes, but with custom LAN, voicechat keeps the same port so there is no need for tweaking the config files everytime
but I prefer using Ngrok LAN because for some reason is less laggy and buggy than essential, the only downside being the forever changing port, but Custom LAN is not compatible with Ngrok LAN (crashes when opening the OPEN TO LAN menu), and is the most convinient way to use simple voice chat, so finding a way to pair Ngrok with custom lan would be really cool
maybe it doesnt work because of how ports work n stuff but idunno about that stuff, wouldnt hurt to try tho, if it worked with essential, why wouldnt it work with Ngrok

ngrok support has been added in Custom LAN 2.3.0, using the new Tunnel button on the Open to LAN screen!


⚠ Processors has degraded performance

In c9f1ef8, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 45 ms

Resolved: Processors performance has improved in c102e35.


Make code PEP8 compliant and add automated PEP8 test

Fixes #160 and fixes #24
You can check PEP8 compliance locally by installing flake8 and running it in the top level of the repo.

It would be nice to merge this ASAP as it touches almost every file and will therefore probably cause many merge conflicts.

Seems good to me!


torchrec inference build error

Hi
I am trying to build torchrec/torchrec/inference , I get this error of "/usr/bin/ld: final link failed: Bad value" with libboost_filesystem.a(operations.o)
/usr/bin/ld: /usr/lib/x86_64-linux-gnu/libboost_filesystem.a(operations.o): relocation R_X86_64_PC32 against symbol '_ZN5boost10filesystem16filesystem_errorC1ERKNSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEERKNS0_4pathESC_NS_6system10error_codeE' can not be used when making a shared object; recompile with -fPIC
/usr/bin/ld: final link failed: Bad value
collect2: error: ld returned 1 exit status

I got similar error with folly library too , which after building folly(folly built with gcc 9+, g++ 9+)  with -fPIC got fixed. I install libboost using this command line
./build/fbcode_builder/getdeps.py install-system-deps --recursive
docker image : pytorch/pytorch:1.11.0-cuda11.3-cudnn8-devel
machine : gpu : nvidia tesla v100 ,
Could you kindly help me with debugging this error ?
Thank you
Bhavya

Hi @bmedishe, we've updated our docs recently with new installation instructions adn an updated CMakeLists.txt: https://github.com/pytorch/torchrec/tree/main/torchrec/inference
There is also a server and client example for you to try out.
Please let me know if you have any questions or need any help with the setup.


Return all errors in data graph - SHACL

The ShaclEx library should return all the errors in the data graph.
Currently the SHACL validator of the ShaclEx library  returns only one constraint error in the data graph at random even though there are more errors in the data graph.
To understand this better:
Example data graph:

@prefix ex:    <http://example.org/> .
@prefix oslc_data: <http://open-services.net/ns/servicemanagement/1.0/> .
@prefix sh:    <http://www.w3.org/ns/shacl#> .
@prefix rdf:   <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix xsd:   <http://www.w3.org/2001/XMLSchema#> .
@prefix dcterms: <http://purl.org/dc/terms/> .
@prefix rdfs:  <http://www.w3.org/2000/01/rdf-schema#> .
@prefix sam:   <http://www.sampledomain.org/sam#> .
@prefix oslc:  <http://open-services.net/ns/core#> .
<http://localhost:8080/SampleAdaptor/services/serviceProviders/1/aResources/1>
        a                       sam:AResource ;
        sam:aReferenceProperty  sam:AnotherResource ;
        sam:aSetOfDates         "2017-02-22T13:33:57.639Z"^^xsd:dateTime ;
        sam:aStringProperty     "Test" ;
        sam:anIntegerProperty   "1234"^^xsd:integer .
[ a              rdf:Statement ;
  rdf:object     sam:AnotherResource ;
  rdf:predicate  sam:aReferenceProperty ;
  rdf:subject    <http://localhost:8080/SampleAdaptor/services/serviceProviders/1/aResources/1>
] .

corresponding shacl shape graph:
@prefix ex:    <http://example.org/> .
@prefix oslc_data: <http://open-services.net/ns/servicemanagement/1.0/> .
@prefix sh:    <http://www.w3.org/ns/shacl#> .
@prefix rdf:   <http://www.w3.org/1999/02/22-rdf-syntax-ns#> .
@prefix xsd:   <http://www.w3.org/2001/XMLSchema#> .
@prefix dcterms: <http://purl.org/dc/terms/> .
@prefix rdfs:  <http://www.w3.org/2000/01/rdf-schema#> .
@prefix sam:   <http://www.sampledomain.org/sam#> .
@prefix oslc:  <http://open-services.net/ns/core#> .

ex:S    sh:property     [ sh:datatype  xsd:dateTime ;
                          sh:maxCount  0 ;
                          sh:minCount  0 ;
                          sh:predicate      sam:aSetOfDates
                        ] ;
        sh:property     [ sh:datatype  xsd:integer ;
                          sh:maxCount  1 ;
                          sh:minCount  1 ;
                          sh:predicate      sam:anIntegerProperty
                        ] ;
        sh:property     [ sh:datatype  xsd:string ;
                          sh:maxCount  0 ;
                          sh:minCount  0 ;
                          sh:predicate      sam:aStringProperty
                        ] ;
        sh:targetClass  sam:AResource .

In the above data graph there exists anIntegerProperty and aSetOfDates. But, according to shapes graph, the cardinality of these two properties should be 0.
The ShaclEx validation library SHOULD return maxCountError for both these properties. BUT, it returns only one error at random. Sometimes maxCountError for anIntegerProperty sometimes for aSetOfDates.

@labra @darkLabra

@berezovskyi  @jadelkhoury

Yes, that's something I'm planning to repair in the near future.
I was very busy in the last months but as soon as I find some time, I will adapt the computation model to collect all errors.
The reason for returning only the first error was to increase performance in case of very big RDF data to avoid generating all errors.
Currently SHACL specification says that implementations must potentially be able to generate all errors, but that sometimes they don't need to generate all errors. My first attempt will be to add a flag so users can choose if they want all-errors behavious or first-error-stop behaviour.

I started to work on this in branch shacl-allResults

Hi!
The link you gave is broken. When are you planning to release this bug-fix. Thank you for investing time in it.

Yes, perhaps you forgot to push the branch?

Ops, I had done the job in that specific branch but once I finished it, I just merged to master and deleted the branch. You can see the PR here.
The example from @yashkhatri works now (after updating the name of the properties from sh:predicate to sh:path and adding the type declaration sh:NodeShape).
You can see it working here.
I didn't want to close the issue yet until I checked the feature using the SHACL test suite. To that end, we are working on generating the ValidationReport following the SHACL recommendation.
One problem that needs to be solved is converting the result structure (called result shape map) to validation report results. I hope I can work on this feature soon.

Hi!
"One problem that needs to be solved is converting the result structure (called result shape map) to validation report results. I hope I can work on this feature soon."
Would be nice to know that status of this. Is this implemented now?

Not yet...I was involved in other tasks and couldn't tackle it. It is really one of the first topics in my TODO list but the next two weeks I will have a lot of teaching duties...
I will try to devote time to it as soon as possible. One preliminary step I did was to be able to read SHACL's manifest files which were slightly different from my original proposal. I needed this previous step in order to be able to pass the SHACL test-suite...which is my final goal.

Hi @labra I was using 0.0.63 version of shaclex library (https://dl.bintray.com/labra/maven/es/weso/shaclex_2.12/) in our java project.
This issue#38 is still open. However, I would like to use the feature of getting all the errors in data graph if it is implemented and released. So my question is; does any of the version (0.0.65, 0.0.66, 0,0,67) contains this feature?

The latest release 0.074 already has implemented that feature.
I prefer to keep the issue open because I didn't have time to check all the tests in the SHACL testsuite.

As all the tests in the SHACL test-suite have passed, I think we can close this issue.


Transparent background for VisitableView

Today the VisitableView is opaque with a white background. It would be great if the view background was instead transparent unless explicitly set eg. via styles={{ backgroundColor: .. }}.
Having a transparent background would allow a VisitableView be shown as a screen with presentation: transparentModal, covering the screen, but allowing the content behind to be shown and the  of the web page to control the transparency, eg:

Full page with semi-transparent background

The functionality would allow showing webpages that blend in with the content behind it - for instance dialogs.

You can take a different approach to handle dialogs, don't see any benefits of doing that for now. For example you can use FullWindowOverlay from react-native-screens.


Is that possible to publish a release?

We want to add this tool into hd-home which is a GitHub release asset tool. But there's no release found.

Great! Now people can install grmon with the following command:
gitpod /workspace/http-downloader $ hd install grmon
start to fetch the config
Enumerating objects: 5, done.
Counting objects: 100% (5/5), done.
Compressing objects: 100% (5/5), done.
Total 5 (delta 0), reused 0 (delta 0), pack-reused 0
target package is bcicen/grmon
start to download from https://github.com/bcicen/grmon/releases/download/v0.1/grmon
start to download with 4 threads, size: 6120200, unit: 1530050
Downloading part 3:  [====================================================================] 100%
Downloading part 2:  [====================================================================] 100%
Downloading part 0:  [====================================================================] 100%
Downloading part 1:  [====================================================================] 100%
install grmon to /usr/local/bin/grmon


[Cloud] Remove cloud module

Issue #, if available:
Description of changes:

Remove cloud related stuff. Expect cloud CI to fail. It will be gone once merged in

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Cloud CI fail as expected. Will merge once the regular CI passes. The only change regarding the regular CI that will not be reflected in the PR is the path filter removal, which should be correct


IMPROV: get number of columns in xterm.js based console

We need to detect or retrieve the number of colums so that line wrapping will work in browser based console too.

Fixed in https://github.com/refaktor/rye/pull/385 ... a little confusing, should improve


net/http/httputil: ReverseProxy doesn't handle connection/stream reset properly

What version of Go are you using (go version)?

$ go version
net/http/httputil: ReverseProxy

Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/jzhan/Library/Caches/go-build"
GOENV="/Users/jzhan/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/jzhan/gocode"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.13/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/4p/ch6dsl490v9227wwhd9rnb7w0000gn/T/go-build248622450=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

I  am trying to test the behavior of HTTP forwarding using httputil.Reverseproxy. Especially the behavior when the backend server closes the connection (for HTTP/1) or resets the stream (for HTTP/2). To test the stream reset behavior, the server uses panic(http.ErrAbortHandler) in the handler.
Wrote some tests for these.
For HTTP1: https://github.com/jaricftw/go-http-reverseproxy/blob/master/http1_test.go#L20, which tests:
1). http1 client -> http1 server
2). http1 client -> httputil.Reverseproxy -> http1 server
For HTTP2: https://github.com/jaricftw/go-http-reverseproxy/blob/master/http2_test.go#L25, similarly:
1). h2c client -> h2c server
2). h2c client -> httputil.Reverseproxy (with h2c setup) -> h2c server
What did you expect to see?
I would expect all tests to pass. i.e. the response for W/ and W/O reverseproxy should be the same in the stream reset scenario
What did you see instead?
In the happy case, w/ and w/o reverseproxy tests all passed, for both HTTP/1 and HTTP/2.
However, I see different responses W/ and W/O reverseproxy for the connection/stream reset test cases:
For HTTP/1:
W/O reverseproxy:
PASS: TestHTTP1/direct,_server_panic (0.00s)
        http1_test.go:62: got err: Get http://<IP_ADDRESS>:6666: EOF
        http1_test.go:63: got resp: <nil>

W/ reverseproxy:
FAIL: TestHTTP1/via_proxy,_server_panic (0.00s)
        http1_test.go:62: got err: <nil>
        http1_test.go:63: got resp: &{502 Bad Gateway 502 HTTP/1.1 1 1 map[Content-Length:[0] Date:[Thu, 19 Sep 2019 20:52:56 GMT]] {} 0 [] false false map[] 0xc0001da300 <nil>}
        require.go:248:
                Error Trace:    http1_test.go:65
                Error:          An error is expected but got nil.
                Test:           TestHTTP1/via_proxy,_server_panic

For HTTP/2:
W/O reverseproxy:
PASS: TestHTTP2/direct,_server_panic (0.00s)
        http2_test.go:78: got err: Get http://<IP_ADDRESS>:7777: stream error: stream ID 3; INTERNAL_ERROR
        http2_test.go:79: got resp: <nil>

W/ reverseproxy:
FAIL: TestHTTP2/via_proxy,_server_panic (0.00s)
        http2_test.go:78: got err: <nil>
        http2_test.go:79: got resp: &{502 Bad Gateway 502 HTTP/2.0 2 0 map[Content-Length:[0] Date:[Thu, 19 Sep 2019 19:01:15 GMT]] 0xc00022e700 0 [] false false map[] 0xc000250800 <nil>}
        require.go:248:
                Error Trace:    http2_test.go:81
                Error:          An error is expected but got nil.
                Test:           TestHTTP2/via_proxy,_server_panic
FAIL

Hi @jaricftw,
A 502 from a reverse proxy seems reasonable to me in this scenario, but I am not as familiar with http/2.0 as the package owner @bradfitz.

@toothrot I've update the issue to include HTTP/1 as well, and it seems neither HTTP/1 connection close nor HTTP/2 stream reset is handled by the reverse proxy --- a 502 will be returned to the client
The risk would be the proxy doesn't close the connection to the client in HTTP/1, or doesn't send a RST_STREAM to the client in HTTP/2?

You can change the ReverseProxy's default 502 behavior by setting your own ErrorHandler with whatever policy you'd like: https://golang.org/pkg/net/http/httputil/#ReverseProxy.ErrorHandler

The risk would be the proxy doesn't close the connection to the client in HTTP/1, or doesn't send a RST_STREAM to the client in HTTP/2?

Sorry, I don't follow. Could you reword? Or back up? I read your test code above (thanks for the nice bug report!) but I'm confused by this latest statement/question.

The risk would be the proxy doesn't close the connection to the client in HTTP/1, or doesn't send a RST_STREAM to the client in HTTP/2?

Sorry, I don't follow. Could you reword? Or back up? I read your test code above (thanks for the nice bug report!) but I'm confused by this latest statement/question.

Sorry for the confusion. I don't have enough tests to back it up yet. I meant that the connection error seen by the proxy is not properly returned to the client, which may confuse the client upon receiving a regular status code 502. E.g. client may choose its retry mechanism based on the response code/error. And I am assuming the client <-> proxy connection/stream will still be treated "connected" from the client perspective, although the proxy->server one is closed?


Feature/custom chunk

I needed this feature to add <hr/> nodes. All your other repositories have the functionality to add custom components, except this one. The solution works for atomic blocks, so I guess it does not support all use cases it its present form.
Thanks for the open source editor.

ran into the same requirements would like to have support for custom tags outside of known html tags.

@jpuri any update on this?

Feature released in<EMAIL_ADDRESS>Thanks a lot @nickyout


Get version with importlib.metadata

pkg_resources that was used to get package version is a deprecated API: https://setuptools.pypa.io/en/latest/pkg_resources.html

Thanks!
If I understand correctly, we dopn't need the backport on Python 3.8, when importlib.metadata is in Python? In that case let's hold off to 3.8, otherwise we'll just have another bit of old code to remove later, which means we didn't really gain anything.

OK, implemented, thanks!


BUG: bst_* datasets broken (and thus CircleCI)

CircleCI is failing because the copy.com downloads are failing. We need to:

Update our URLs to use https://mne-tools.s3.amazonaws.com (you can see the paths in there) instead of copy.com.
I think I also changed the archives to be gz instead of bz2 files (much faster decompression with only a little bit bigger files), but just changing the filename should make this work as type is autodetected.
For at least bst_auditory I made it use CTF data instead of the FIFF-converted versions, but hopefully this will only require minimal changes in the example scripts to get working.

@jaeilepp do you have time to try it? If not, I'll have time later in the week.

Yeah I can try.

By the way in circle I think the only bst dataset you need to run examples
might be raw, not auditory or the other one. Worth checking while you're in
there.


Empty nostr.json

Have setup a domain and a user, but have an empty nostr.json.
It was also unclear what the relay field did. Shouldn't NIP-05 be totally independent of relay?

Follow-up ...
I'm seeing a 500 - No API Key found for LN Address error pop-up.
I've re-installed the extension and nuked the DB, but no change.

Have setup a domain and a user, but have an empty nostr.json.

It's kind of counterintuitive, but nostr.json is not a static file, it's generated dynamically every time it's opened. Please, try it with a valid name parameter, like this:
https://core.btcmap.org/nostrnip5/api/v1/domain/QhuWY6kMqu5qVPGhmPSPAN/nostr.json?name={username}
Also, a more informative error message would be nice if the parameter is missing.

It was also unclear what the relay field did. Shouldn't NIP-05 be totally independent of relay?

The relay field is 'recommended', see the specs: https://github.com/nostr-protocol/nips/blob/master/05.md

I'm seeing a 500 - No API Key found for LN Address error pop-up.

Yeah, that was a stumper for me too. The lnurlp (Pay Links) extension needs to be installed (it wasn't installed by default for me). Then the nostrnip5 extension has a settings page that can be accessed as an admin. See the screenshot below:
You can find your Lightning Address Admin Key under your wallet's Node URL, API keys and API docs section.
Don't give up, give it another try; it took me some time, but now it works for me!

Ah, I didn't know it was dynamically generated. However, it should return a non-empty file when is is generated, right? Clients don't provide query parameters afiak.
Even using the filter doesn't work for me.

Ah, I didn't know it was dynamically generated. However, it should return a non-empty file when is is generated, right? Clients don't provide query parameters afiak.
Even using the filter doesn't work for me.

Clients should provide query parameters, as per the specs: https://github.com/nostr-protocol/nips/blob/master/05.md#reasoning-for-the-well-knownnostrjsonnamelocal-part-format
It took a while, but now it works for me: https://lnbits.noszter.hu/nostrnip5/api/v1/domain/JTai6rMtFMhjjSfYt7duaC/nostr.json?name=zoltan
Can you check your nostrnip5 plugin? Is your address present and active?


Cannot download Linux 64 bit 2.1.1

Which version of PhantomJS are you using? Tip: run phantomjs --version.
N/A

What steps will reproduce the problem?

Go to the homepage http://phantomjs.org/download.html

Download Linux 64 bit. Sometimes the link hangs, sometimes it shows XML, but it does not download. I went to the bitbucket to verify. It doesn't appear to be a spelling error. https://bitbucket.org/ariya/phantomjs/downloads/phantomjs-2.1.1-linux-x86_64.tar.bz2

Which operating system are you using?
Chromium Version 50.0.2661.102 Ubuntu 16.04 (64-bit)

Did you use binary PhantomJS or did you compile it from source?
No

Please provide any additional information below.

And now it works.... I don't know. Probably have to check your webserver logs to understand what was wrong.

I looked through my history and found this if it helps:
https://bbuseruploads.s3.amazonaws.com/fd96ed93-2b32-46a7-9d2b-ecbc0988516a/downloads/396e7977-71fd-4592-8723-495ca4cfa7cc/phantomjs-2.1.1-linux-x86_64.tar.bz2?Signature=i6DgQIQiZMg571hwsEf6VoGBMEs%3D&Expires=1465372335&AWSAccessKeyId=AKIAIWY5XSVPZPDQYRQQ&versionId=null&response-content-disposition=attachment%3B filename%3D"phantomjs-2.1.1-linux-x86_64.tar.bz2"
<?xml version="1.0" encoding="UTF-8"?>
<Error>AccessDenied<Message>Request has expired</Message><Expires>2016-06-08T07:52:15Z</Expires><ServerTime>2016-06-08T08:32:43Z</ServerTime><RequestId>37B8399844E7E19B</RequestId><HostId>HBqPzAjZoWvKF7xYFBDyplQOfHMklKuIOfHM6c96MiyrhGXfAWkkr/B3qSIJyhX1Vt2Lm5PpOtk=</HostId></Error>```

Bitbucket prevents you from taking that link and reusing it, using tokens. That means if you want an automated system to download Phantom, host it yourself in Amazon s3 or something similar to that.

Hi! This is a known problem with Bitbucket. We are working on the solution.

Why close the ticket if the solution is not implemented? It is still happening 6 months later


[rt.openssl.org #3913] Add a way to application to know a minimum DH size allowed by the client

Migrated from rt.openssl.org#3913 (status was 'new')
Requestors:

<EMAIL_ADDRESS>
From<EMAIL_ADDRESS>on 2015-06-17 10:38:45:
The current minimum DH size allowed by the client is 768 bits which is a
hardcoded constant. It would be nice if the constant was at least
#define in public headers or even better if there was an API to query
various minimum and maximum bit sizes that are checked in the library
such as the maximum supported key lengths, etc.

--
Tomas Mraz
No matter how far down the wrong road you've gone, turn back.
                                              Turkish proverb
(You'll never know whether the road is wrong though.)

_______________________________________________
openssl-bugs-mod mailing list
<EMAIL_ADDRESS>https://mta.openssl.org/mailman/listinfo/openssl-bugs-mod

Marking as inactive to be closed at the end of 3.4 dev barring further input

@t8m, you're not gonna add the public define? :)


Small cleanups

This branch name has become a bit of a misnomer... I have lumped together a number of fixes and enhancements... in general this is probably a bad practice, but I vote we let it slide for now...
Add badges etc to readme, adjust regression tests links to reflect closure of issues, and add co_reduce regression test, even though it appears it was never a regression to begin with.

Fixes #79
Fixes  #297
Add regression test for #243
Add regression for #172 (currently set to pass when test fails)
Migrates to use of a .VERSION file to make parsing easier for scripts and allow extra comments
Adds auto-upload of release assets upon tagging with git, but requires that the tag is PGP signed (git tag -s <tag> [tree-ish])
This should compute the SHA 256 checksum and create a detached signature of the cryptographic SHA 256 checksum with an encrypted GPG subway I uploaded to the repo/travis. 🔮 🎩 🐇

Current coverage is 45.73% (diff: 100%)

Merging #310 into master will decrease coverage by 4.76%

@@             master       #310   diff @@
==========================================
  Files             2          3     +1
  Lines           903       1019   +116
  Methods          43         64    +21
  Messages          0          0
  Branches        191        193     +2
==========================================
+ Hits            456        466    +10
- Misses          367        476   +109
+ Partials         80         77     -3

Powered by Codecov. Last update 882c371...79d3f51

These commits need to be rebased and cleaned up before merging... Also I set CMake/CTest to indicate pass status even when regression 172 fails... I did this so that I could examine and compare codecov.io output between the passing and failing tests in an attempt to localize the bug if it's in the library.

With value instead of intent(in) here are the coverage changes: https://codecov.io/gh/sourceryinstitute/opencoarrays/compare/882c371d4d7e84364eb7adfba4b4f8d840e3f398...cb9f040bfc7543424a637a43fb447fbbe605a0f8/changes
vs
https://codecov.io/gh/sourceryinstitute/opencoarrays/compare/882c371d4d7e84364eb7adfba4b4f8d840e3f398...1a0e3b7edb6867d9e9371cbd9ed1d8f5b3dd2010/changes with intent(in)

lgtm

Need to account for older CMake where CMAKE_Fortran_COMPILER_VERSION is missing....

LGTM

@rouson: I've added a regression test for #172, but it's still failing. To cope with this, I've set it so that failing indicates success (so that users won't see test failures because it's a known issue...) I'd like to merge this PR, but I could split that test out into a separate one. What do you think? Would you be OK with merging this PR with the failing regression for #172 but have it report a "PASS" status when it fails?

I'm going to go for it... we can always factor out the #172 regression test later.

Is there no way to set a test to expect to fail?

That's what it is now. But then when it's fixed it will report a failure.
It's a logical not/inverse operation.
On Sun, Jan 22, 2017 at 12:34 PM Dr. Andre Vehreschild <
<EMAIL_ADDRESS>wrote:

Is there no way to set a test to expect to fail?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/sourceryinstitute/opencoarrays/pull/310#issuecomment-274345302,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAREPLFu0ebX99t2VRG0NiafrdGZfmeHks5rU5MPgaJpZM4Ljxfg
.

@zbeekman Hopefully you saw my message sent via Hangouts.  If a failing test degrades user confidence, then I believe that a false positive would degrade user confidence even more.  Moreover, I am highly unlikely to remember this exchange 6 months from now when a user reports a related problem and then I spend time trying to track down the cause, only to realize the cause has been under my nose all along, but I forgot about it because I've ben lulled into a false sense of security.   Probably it's best to set up CMake so that there are a group of tests that are cordoned off as known failures and not run unless a flag is passed to run them.  They should probably also be listed somewhere such as in our Known Issues page, which actually might be obsolete at this point because we haven't maintained it.  So probably an even better approach would be to have a Known Issues milestone under which we would the issue numbers related to failing tests and name the tests.  I'd say almost anything is better than having a test falsely report success.

This is clearly the correct way to do it. The only question is: when do we turn on the known failures? During CI tests? For the users by default (with a message telling them it's a known deficiency?) I'm inclined to take the latter approach, in the interest of transparency
As for the known issues page, we should ensure that we have an open issue for each issue and a link to the issues page.

@zbeekman Your suggestion sounds good to me as long as the users see a message indicating that these are known failures.  Ideally the message would even include a URL to the list of known failures.  For Travis-CI, we would build with something like
cmake .. -DBUILD_KNOWN_FAILURES=TRUE
where the above flag would default to FALSE or would be explicitly set to FALSE for CI.

Your suggestion sounds good to me as long as the users see a message indicating that these are known failures.

I 100% agree... My thinking is this: In the known failures test we output clear messages with URLs pointing to the issue tracker pages saying these are known deficiencies that we are working to resolve. Failing tests are turned on by default, and if the user is installing via install.sh we can ensure that we export CTEST_OUTPUT_ON_FAILURE=1 and have any calls to ctest with the --output-on-failure flag. This way they will see the warning message. We can also include a warning in the CMake configure script informing them that we have turned on tests with known failures and giving them instructions on how to turn those off.
As long as the failures are runtime failures we should always build them, since compilation in and of itself is a smoke test.
On Travis-CI, however, I don't think we should run tests known to fail so that we can continue our workflow of finding new bugs and issues, without the noise coming from bugs we already know about. Otherwise the known issues would obfuscate new ones and coding errors/typos etc.


How to use synthesizeAssociation function?

Here is my class;
.h
#import <UIKit/UIKit.h>

@interface UIView (TFUmengAnalytics)
@property (nonatomic, copy) NSDictionary *tf_umengDictionary;
@end

.m
@implementation UIView (TFUmengAnalytics)
synthesizeAssociation(UIView, tf_umengDictionary);
@end

but, I got a error:

Anyone can help me? Please....

You need to prefix synthesizeAssociation with an @ symbol, just like the built-in keywords.

Thank you @jspahrsummers , it works. But why there isn't syntax highlighting for synthesizeAssociation. And also I cannot use command+click to jump to define.


*: update deps

What problem does this PR solve?
This PR updates futures library to latest version to fix security
issues. It also updates several yanked dependencies.
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

No release note.

/bench

/run-test

/run-unit-test

/run-test

/run-integration-ddl-test


Include  for std::unique_ptr

Currently #include <grpcpp/support/interceptor.h> in some compilers (gcc 8, for example) will fail due to missing <memory> include. This PR fixes this issue.
Fixes #23100

@yashykt

Hi @Dlougach , can you create an issue which details the output showing the error on compilation with gcc 8?

Hi @Dlougach , can you create an issue which details the output showing the error on compilation with gcc 8?

Done.


[$250] Distance -Distance rate is not grayed out after editing "Tax rate" & "Tax reclaimable" offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v9.0.19-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp https://expensify.testrail.io/index.php?/tests/view/4849591
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:
Precondition:

Tax is enabled.
Distance rates are enabled.
Track tax in DIstance rates is enabled.

Go to Distance rates.
Go offline.
Tap on any rate.
Edit "Tax rate" and "Tax reclaimable on".
Return to distance rate list.

Expected Result:
The distance rate will be grayed out after editing "Tax rate" or "Tax reclaimable on" offline.
Actual Result:
The distance rate is not grayed out after editing "Tax rate" or "Tax reclaimable on" offline.
Workaround:
Unknown
Platforms:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/71db7762-531e-4430-8249-533e9c29e63f

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01495987645fc6b7df
Upwork Job ID:<PHONE_NUMBER>301545382
Last Price Increase: 2024-08-14
Automatic offers:

ikevin127 | Reviewer | 103534987
FitseTLT | Contributor | 103534988

Issue OwnerCurrent Issue Owner: @ikevin127

Proposal
Please re-state the problem that we are trying to solve in this issue.
Distance rate is not grayed out after editing "Tax rate" & "Tax reclaimable" offline
What is the root cause of that problem?
Here we are missing taxClaimablePercentage and taxRateExternalID
https://github.com/Expensify/App/blob/cc2efc6f7a4f17bc0ee6cdb4b8fb3fcf592c3cbb/src/pages/workspace/distanceRates/PolicyDistanceRatesPage.tsx#L115-L120
What changes do you think we should make in order to solve the problem?

We need to add pendingFields taxClaimablePercentage and taxRateExternalID to the checks here
https://github.com/Expensify/App/blob/cc2efc6f7a4f17bc0ee6cdb4b8fb3fcf592c3cbb/src/pages/workspace/distanceRates/PolicyDistanceRatesPage.tsx#L115-L120
What alternative solutions did you explore? (Optional)

Reproduced. I also verified that we generally do grey out the main option row if there is an offline change to a setting associated with it.

♻ Reviewing proposal today.

@FitseTLT's proposal LGTM. The RCA is correct and the proposed solution fixes the issue according to the Expected result.
🎀👀🎀 C+ reviewed

@FitseTLT is there an ETA for PR here?

PR pending

PR ready

TY! I see it's merged now

⚠ PR was just deployed to production ~10 minutes ago but automation failed to add the hold -> this should be on [HOLD for Payment 2024-08-27] according to today's production deploy from https://github.com/Expensify/App/pull/47591#issuecomment-2299886099.
cc @joekaufmanexpensify

Sounds good!

Payments here are:

$250 to @FitseTLT for PR via upwork.
$250 to @ikevin127 for C+ via upwork.

All set to issue payment!

@FitseTLT $250 sent and contract ended!

@ikevin127 $250 sent and contract ended!

All set, thanks everyone!


Updates Microservice book on Resilient communication features with HttpClientFactory in .NET Core 2.1 with Polly

Summary
Updates Microservice book on Resilient communication features with HttpClientFactory in .NET Core 2.1 with Polly

Updates based on Bill's and Maira's feedback are already pushed into the branch.
Please, merge when possible. 👍

I have pushed additional updates based on Bill's and Maira's feedback. Please, review and merge when possible. Thanks! 👍


Add Competitive Programing Questions from leetcode, geeksforgeeks, codechef etc.

Hey @Mridul-1-Sharma Please assign this task to me.

Hey @Mridul-1-Sharma Please assign this task to me.

Assigned to you now start contributing

Hey can you assign this task me??

Hey can you assign this task me??

OK ASSIGNED

Hi can you assign this task me? I would like to contribute

This repository is excluded from hacktoberfest if want to contribute you can contribute in https://github.com/Mridul-1-Sharma/data_structures_algos


Generator returns the same values

I use this code
$f = Faker::instance()->getGenerator();
$fm->define(User::class)->setDefinitions([
        'first_name'   => $f->firstName,
        'last_name' => $f->lastName,
        'email'  => $f->email,
]);

Each time I create new User there are the same values for names and email.

How about add some @method  annotations to League\FactoryMuffin\Faker\Facade? I can do that.


Update k8s-e2e-external-storage job to use Kubernetes 1.21

Running the job with Kubernetes 1.21 fails (k8s-e2e-external-storage/11):
Summarizing 10 Failures:

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should support multiple inline ephemeral volumes
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with same fsgroup skips ownership changes to the volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should support two pods which share the same volume
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with different fsgroup applied to the volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with different fsgroup applied to the volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, new pod fsgroup applied to volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (Always)[LinuxOnly], pod created with an initial fsgroup, volume contents ownership changed in first pod, new pod with same fsgroup applied to the volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Dynamic PV (default fs)] fsgroupchangepolicy [It] (OnRootMismatch)[LinuxOnly], pod created with an initial fsgroup, new pod fsgroup applied to volume contents
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/utils/utils.go:717

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should create read-only inline ephemeral volume
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

[Fail] External Storage [Driver: cephfs.csi.ceph.com] [Testpattern: Generic Ephemeral-volume (default fs) (late-binding)] ephemeral [It] should create read/write inline ephemeral volume
/workspace/src/k8s.io/kubernetes/_output/dockerized/go/src/k8s.io/kubernetes/test/e2e/storage/testsuites/ephemeral.go:307

Ran 36 of 5978 Specs in 6890.099 seconds
FAIL! -- 26 Passed | 10 Failed | 0 Pending | 5942 Skipped
--- FAIL: TestE2E (6890.14s)
FAIL

Ginkgo ran 1 suite in 1h54m53.465533491s
Test Suite Failed

Running on Kubernetes 1.20 works fine. We'll need to make sure that we are complaint with Kubernetes 1.21 too.

[x] The CI job description k8s-e2e-external-storage.yaml needs to be updated so that it generates a 2nd job for Kubernetes 1.21. This would look similar to the project/template in mini-e2e.yaml
[ ] errors listed above should be addressed in PRs referencing Updates: #2017
[ ] additional errors are expected with the RBD testing, these have not been executed yet

#2532 makes the tests pass. #2015 can be used to enable fsGroupChangePolicy again (once fixed).


Possible bug

Did you intentionally not include the loss associated with 'initial inference' (prior to unrolling for k steps)? Possible that I am misunderstanding paper.

Hi, thank you for reporting this issue. Could you provide more information about "loss associated with initial inference"?
Currently, the implementation follows the paper Mastering Atari, Go, chess and shogi by planning with a learned model. In the loss/default_loss_fn, the loss is calculated as follows.
Here each time the loss function will first encode the raw observation, and iteratively loop through the hidden states and calculate loss with actual observation. The batch.Rn is the bootstrapping value, where the value is calculated by sequence of rewards and value of the root node.
I think the initial inference uses representation to encode the raw observation and prediction to provide prior_logits and value, so I think default_loss_fn includes this part. What do you think? Please share your thoughts.
@partial(jax.jit, static_argnums=(0, ))
def default_loss_fn(muzero_instance, params, batch):

    # Use muzero_instance to access required methods and attributes
    loss = 0
    c = 1e-4
    B, L, _ = batch.a.shape
    batch.r = scalar_to_support(batch.r, muzero_instance._support_size).reshape(B, L, -1)
    batch.Rn = scalar_to_support(batch.Rn, muzero_instance._support_size).reshape(B, L, -1)
    s = muzero_instance._repr_apply(params['representation'], batch.obs[:, 0, :])

    def body_func(i, loss_s):
      loss, s = loss_s
      v, logits = muzero_instance._pred_apply(params['prediction'], s)
      # Appendix G, scale the gradient at the start of the dynamics function by 1/2
      s = scale_gradient(s, 0.5)
      r, s = muzero_instance._dy_apply(params['dynamic'], s, batch.a[:, i, :].flatten())
      # losses: reward
      loss_r = jnp.mean(
        optax.softmax_cross_entropy(r,
        jax.lax.stop_gradient(batch.r[:, i, :])
        ))
      # losses: value
      loss_v = jnp.mean(
        optax.softmax_cross_entropy(v,
        jax.lax.stop_gradient(batch.Rn[:, i, :])
        ))
      # losses: action weights
      loss_pi = jnp.mean(
        optax.softmax_cross_entropy(logits,
        jax.lax.stop_gradient(batch.pi[:, i, :])
        ))

      loss += loss_r + loss_v + loss_pi
      loss_s = (loss, s)
      return loss_s
    loss, _ = jax.lax.fori_loop(1, L, body_func, (loss, s))
    # Appendix G Training: "irrespective of how many steps we unroll for"
    loss /= L

    # L2 regulariser
    l2_regulariser = 0.5 * sum(
      jnp.sum(jnp.square(p)) for p in jax.tree_util.tree_leaves(params))
    loss += c * jnp.sum(l2_regulariser)
    # print(f'loss2: {loss}')
    return loss

Want to connect via email or discord? Either way you can reach me at the email in my bio.

How about email? I have also included my email in bio(just now)


Feature/constants plus source command

Synopsis:
Very basic source command.
Usage examples:

src react, and source tz: Returns the url pointing to the command
src,  src help: Returns the usage of source command.

Constants file:
Idk if anyone likes having a constants file but I added it just in case. It contains constants in case we need to not repeat some stuff in the future.
I also edited the run.sh script.

@tk3369 I have followed the BlueStyle Guide so I resolved the conversation regarding that. However, I need your opinion regarding me moving the constants from main.jl to constants.jl and my edit of src.jl. Thank you :D

@tk3369 Please re-review :smiley:  I have made some changes based on your suggestions. Regarding the log thingy from line 47 in main.jl, I think it would be better for it to be in a separate PR.


12259 concat ws

Summary of the changes / Why this improves CrateDB
Added concat_ws function to the scalar functions library. User is able to concatenate with a delimiter expanding crateDB's features. https://github.com/crate/crate/issues/12259
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Can one of the admins verify this patch?

ok to test

CrateDB team,
I ran the following command locally against this branch and it passes without error.
./gradlew :server:test -Dtests.seed=2DFFC6BE4D571E5 -Dtests.class=io.crate.integrationtests.disruption.replication.logical.SubscriptionDisruptionIT -Dtests.method="test_subscription_metadata_tracker_retries_on_publisher_disconnect" -Dtests.locale=se-SE -Dtests.timezone=Indian/Chagos
Not sure where the differences would be between our environments. Any help would be greatly appreciated. Thanks

I ran the following command locally against this branch and it passes without error.

It seems to be a flaky test, not related to your PR, you can safely ignore it.

All the failed checks seem to have failed via some processes we wouldn't have access too to apply a fix. Please let me know if there is anything I can do to help this pull request along to closure.
Failing Checks:
Test CrateDB SQL on windows-latest:

Error: The operation was canceled

.
Sphinx linkcheck:

Run sphinx-build -n -W --keep-going -q -c docs/ -b linkcheck -E docs/ docs/out/html
/home/runner/work/crate/crate/docs/admin/discovery.rst.rst:128: WARNING: broken link: https://azure.microsoft.com/en-us/documentation/articles/resource-group-authenticate-service-principal-cli/ (404 Client Error: Not Found for url: https://docs.microsoft.com/en-us/api/Redirect/en-us/documentation/articles/resource-group-authenticate-service-principal-cli/)
INFO: Adjusted proxied_api_host to https://crate.io/docs/crate/reference/_
INFO: No adjustments to html_context
sphinx-sitemap warning: No pages generated for site.xml
Error: Process completed with exit code 1.

Vale

Vale: Found 0 suggestion(s), 0 warning(s), and 1 error(s).
Warning: Unable to create annotations; printing Vale alerts instead.
[ { path: 'docs/appendices/release-notes/unreleased.rst',
start_line: 111,
end_line: 111,
start_column: 17,
end_column: 18,
annotation_level: 'failure',
title: '[error] Vale.Spelling',
message: 'Did you really mean 'ws'?' } ]

All the failed checks seem to have failed via some processes we wouldn't have access too to apply a fix. Please let me know if there is anything I can do to help this pull request along to closure.

Hi @rc-enfuse, thanks for your efforts. I left another round of minor suggestions + there is already a comment about first arg  = null -> return null, otherwise looks good.
Sphinx might be failing because of 80 symbols per line, should be fine after applying suggestions. I will take care of Vale failure as a last step before squashing commits, no worries.

@BaurzhanSakhariev Thanks. I'll get right on these changes.

Pending approval I think this pull request is ready to merge. unless someone else has any other comments.

Hi @rc-enfuse, thanks for applying suggestions, looks good. Could you give me permissions to push into your branch? I did some polishing and squashed commits

Hi @rc-enfuse, thanks for applying suggestions, looks good. Could you give me permissions to push into your branch? I did some minor polishing and squashed commits

sure but doesn't github usually give you the ability to squash commits before we merge?

sure but doesn't github usually give you the ability to squash commits before we merge?

I needed permission to do some minor changes as well (didn't want to bother you by new comments as it's really small naming updates/reverting non-related changes)

retest this please

Hi @rc-enfuse, looks good and ready to merge.
Short update: We are about to release version 4.8 and currently have feature freeze on the master. Once we release we merge this PR.
Thanks for your contribution!


Feat: Add features like seeding and fake migrations in the cli

The first case will be to insert values into the database with some useful data after a successful migration.
For reference: https://sequelize.org/master/manual/migrations.html#creating-the-first-seed
Fake migrations will be useful when a database has been created and altered manually and later you want to manage all things with migrations.
Reference: https://docs.djangoproject.com/en/3.1/topics/migrations/#initial-migrations

There is already an issue for seeding: #251
You can also use CLI to import sql dumps. I don't follow what you mean by fake migrations.

@B4nan

You can check djangos reference link

@B4nan i ma leaving here some links for you to follow and get the idea.
https://stackoverflow.com/questions/46772762/django-migrate-fake-and-fake-initial-explained#:~:text=migrations run correctly.-,--fake-initial,preexisted the use of migrations.
https://dev.to/k4ml/django-migrate---fake-initial-gotchas-2fih

I don't think this is possible, migrations in MikroORM are about raw SQL queries, there are no metadata available (e.g. I have no information about what the migration will do, what tables it will create/alter/drop). And I have no plans to change this, I want migrations like this, so users have absolute control and there is no magic. I have no interesting in parsing the queries to get such information (that would be a highway to hell).

Its not that difficult. Suppose you have a database which which wasnt using migration. Now what you do is configure the orm for the database, generate entities using cli  from that database, now generate initial migration. But since the database already exists, now what you have to do is create a migration table and insert the generated migration data only. No any alterations to the entities table needed. I managed to get a workaround for typeorm: https://github.com/typeorm/typeorm/issues/6195  .  Without this, you cannot use the orm migration incases like there is an existing database with some data and you only have to write a webapp

It just inserts the available migration file data to migration table nothing else .

It just inserts the available migration file data to migration table nothing else .

If you just started using the ORM on existing project/db, then there will be no exiting migrations that you would have to insert, that is the bit I don't understand.

so how do we setup initial migrations for an existing db?

You could create the entities from the existing database mikro-orm generate-entities and then generate the migrations based on the current schema diff mikro-orm migration:create.
Even though I have never tried this I think it should work. If it calculates the diff based on the current db and the entities then maybe you need to remove the database before creating the migrations. Maybe @B4nan can confirm this and close the issue or give the correct answer and then close ;)
For more questions I'd recommend using Slack or Stack overflow, Github issues are more for bugs and feature requests.
Slack channel
or Stack overflow.

I'd say the idea of initial migrations is valid, maybe if the opening post was more like "how do I do this", instead of "you need to implement this and that", it would be more understandable (especially when the suggestions come from FW that takes a different approach to migrations)... I really don't see a need for "fake migrations" and this is not about seeding either.
So I am fine with having a command to create the first (initial) migration, not based on diffing the current schema, but basically doing schema:create to get all the create table statements, so other users would just run migrations and end up with a working schema.
Currently with an existing project/db, you would need to distribute this initial schema dump as a separate file and import it (possibly via MikroORM CLI) first, before running the migrations.
As a workaround, one could do npx mikro-orm schema:create -d to get the initial dump and manually store it in the migrations folder, and then manually mark this migration as executed.
So my vision is basically to have npx mikro-orm migration:initial, that will do the schema:create behind the scenes (instead of using schema:update which is the current behaviour). This migration would be already marked as executed for the user who generated it. If there are already some migrations, it would throw - it would be allowed only as the very first migration to be stored.


Store Access Tokens In Cookies

Storing access tokens in local storage exposes the user to XSS attacks, instead we should store tokens in cookies. You can learn more here
AB#6248

We now acquire the token silently each time we intend to show it in the Auth tab. It is not stored anywhere. We are now safe :-)


P1 Decryption key for Luxembourgish Meters

Hi,
where can I enter the P1 decryption key for Luxembourgish Meters?
In WeiGus Code it is writen like so:
//Key for SAG1030700089067
uint8_t KEY_SMARTY[] = {0xAE, 0xBD, 0x21, 0xB7, 0x69, 0xA6, 0xD1, 0x3C, 0x0D, 0xF0, 0x64, 0xE3, 0x83, 0x68, 0x2E, 0xFF};
sorry, I am a beginner in all of this.
Thank you

Hi, this software is designed only for the Swedish, unencrypted version of the P1 protocol, so it doesn't support the encrypted format used in Luxembourg unfortunately. But the built in DSMR module in ESPHome support it out of the box, check here: https://esphome.io/components/sensor/dsmr.html


Does uses stop words embeded?

Or do I have to use spacy or ntlk to remove it?

Hi,
I’m assuming you want to remove stop words from tokenization result? If so, unfortunately currently we don’t have support on that, so try use SpaCy or NLTK for now. But we will support this in future release.
From: Igor Costa<EMAIL_ADDRESS>Sent: Wednesday, April 17, 2019 9:38 PM
To: Microsoft/BlingFire<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [Microsoft/BlingFire] Does uses stop words embeded? (#7)
Or do I have to use spacy or ntlk to remove it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2FBlingFire%2Fissues%2F7&data=02|01|Jiaming.Guo%40microsoft.com|fb92c45e98514503b0c708d6c3b7b040|72f988bf86f141af91ab2d7cd011db47|1|0|636911591036320604&sdata=le2ezV3ooN5aQNMjBNEu0d6XTr6KPMCf2y8ChhaDLfI%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALODWDGM4DI26DFZHXTBAL3PQ73L5ANCNFSM4HGZVK3A&data=02|01|Jiaming.Guo%40microsoft.com|fb92c45e98514503b0c708d6c3b7b040|72f988bf86f141af91ab2d7cd011db47|1|0|636911591036330613&sdata=H8YtjckZNhBAtFd5Rg3Bo1o1zPhKWJiptAgQfkR%2Boe4%3D&reserved=0.


[WIP] Add elish function

Implementation of ELiSH activation function.
Ref. #1448

Thanks for this PR. Although it is still in WIP, the necessity of this activation functions a part of the framework is a bit unclear. In particular, I can't find a lot of references to it and the significance is not obvious. Would you mind giving us a bit of background?

Sorry for not stating the reason for the choice earlier @hvy . I assumed that implementing functions from recent research (though not referenced to a lot) might allow more experimentation to be possible using the framework for research purposes by providing some flexibility on choices. Please correct me if this is not one of the intentions. I wouldn't mind working on another function instead :)

Thanks for the quick response and the clarification. Providing various functions are indeed what we're aiming for but there's for instance a cost in maintaining public APIs. The consensus within the team is that this functions does not seem significant enough. We could be wrong of course, and don't want any discouragements. Maybe it can be reconsidered if it's being developed outside the framework (as a third party repository for instance) and it looks promising. Please let me close this ticket for now but you're very welcome to work on a different function. If unsure, feel free to open an issue first.


Gradle support hung on srclibCollectMetaInformation

Running src config -m program on a Gradle repo, https://github.com/trishagee/cafelito, caused the scan phase in srclib-java to hang. It made no progress in ~30 min.
Output is:
/tmp/sg/github.com/trishagee/cafelito $ src -v config -m program
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-docker/.bin/srclib-docker scan --repo github.com/trishagee/cafelito --subdir .]
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-java/.bin/srclib-java scan --repo github.com/trishagee/cafelito --subdir .]
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-javascript/.bin/srclib-javascript scan-commonjs --repo github.com/trishagee/cafelito --subdir .]
  --> with input {}
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-python/.bin/srclib-python scan --repo github.com/trishagee/cafelito --subdir .]
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-ruby/.bin/srclib-ruby scan --repo github.com/trishagee/cafelito --subdir .]
  --> with input {}
  --> with input {}
Running: [/home/sqs/.srclib/sourcegraph.com/sourcegraph/srclib-go/.bin/srclib-go scan --repo github.com/trishagee/cafelito --subdir .]
  --> with input {}
  --> with input {}
  --> with input {}
SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Using javagraph version '2f6ef8f'
0 POM files found.
1 gradle files found.
Reading ./build.gradle...
Using gradle wrapper script:/tmp/sg/github.com/trishagee/cafelito/./gradlew
warning: "./..." matched no packages

There was no additional output during this time. The running processes (I only looked for gradle, unfortunately, so there may have been others running that I didn't catch):
$ ps aux|grep gradle
sqs       3719  0.5  2.5 9435332 630076 pts/19 Sl+  09:42   0:36 /usr/lib/jvm/java-8-oracle/bin/java -Dorg.gradle.appname=gradlew -classpath /tmp/sg/github.com/trishagee/cafelito/gradle/wrapper/gradle-wrapper.jar org.gradle.wrapper.GradleWrapperMain -I /tmp/srclib-collect-meta7143144763510911167gradle --project-cache-dir /tmp/gradle-cache4508823660131528500 srclibCollectMetaInformation

I then ran rm -rf .srclib-cache and reran it, and it completed successfully :+1:. So I'm just writing this issue in case someone else sees the same issue. Not urgent.

Cannot reproduce with sourcegrapg/srclib-java@b562a32537bac7c2b21e48d90b43f77f0c3a8f23


ValidationError: Resource ContainerInstances does not exist

I'm seeing the following error in /var/log/cfn-hup.log and cfn-wire.log after the container instances have booted:
AwsQueryError: [Errno 400] ValidationError: Resource ContainerInstances does not exist for stack arn:aws:cloudformation:…

This seems to be caused by infrastructure/ecs-cluster.yaml:
"/etc/cfn/hooks.d/cfn-auto-reloader.conf":
    content: !Sub |
        [cfn-auto-reloader-hook]
        triggers=post.update
        path=Resources.ContainerInstances.Metadata.AWS::CloudFormation::Init
        action=/opt/aws/bin/cfn-init -v --region ${AWS::Region} --stack ${AWS::StackName} --resource ECSLaunchConfiguration

Perhaps it should be Resources.ECSLaunchConfiguration.Metadata.… instead of Resources.ContainerInstances.Metadata.…?

I tested this s/ContainerInstances/ECSLaunchConfiguration/ change locally, works for me. Patch attached (promoted issue to PR).

Thanks @pda


Incorrect output from decluster when n=1

Description
The function pysteps.utils.cleansing.decluster returns the first output with an incorrect shape in the specific case of n==1.
How to reproduce
import numpy as np
from pysteps.utils.cleansing import decluster

X = np.array([[0.0, 0.0]]) # ndim==2
V = np.array([[1.0, 1.0]]) # ndim==2
(X_dec, V_dec) = decluster(X, V, 20)
print(X_dec, V_dec)
print(X_dec.ndim)
print(V_dec.ndim)

Prints:
[0. 0.] [[1. 1.]]
1
2

Expected output
According to the documentation of the decluster function:
    Returns
    -------
    out: tuple of ndarrays
        A two-element tuple (``out_coord``, ``output_array``) containing the
        declustered coordinates (l, d) and input array (l, m), where *l* is
        the new number of samples with *l* <= *n*.

That is, both outputs should have ndim==2 even in the edge case of n==1.
Notes
I didn't examine this throughly, but this may mean that pysteps.motion.lukaskanade.dense_lucaskanade does something funny when exactly one motion vector is found by the sparse motion vector search. See call to decluster here: https://github.com/pySTEPS/pysteps/blob/e2c3484ef234e3ef1ac8f1f11b1e64bc6911f35c/pysteps/motion/lucaskanade.py#L273

Hi @jleinonen and thanks for the detailed description - I'll have a look into it!


useUnknownInCatchVariables reset

Please reset useUnknownInCatchVariables to its default value to restore common behavior.
Note that the useUnknownInCatchVariables was disabled in #879
The proposed change:

Remove useUnknownInCatchVariables from tsconfig file

Thanks @gulderov - are you planning to do this in a follow-up PR to #879?

The necessary changes have already been implemented in #879. Once that pull request is merged, this issue will be resolved and can be closed


Average aggregate does not ignore nulls in long column when druid.generic.useDefaultValueForNull=false

Affected Version
Tested in version 0.18.1
Description
While investigating the correctness of druid's null handling by setting druid.generic.useDefaultValueForNull=false I noticed that the avg aggregate is not correctly ignoring nulls in its count. The returned average is the same as if the nulls were filled with 0.
Steps to reproduce

set druid.generic.useDefaultValueForNull=false in common.runtime.properties, i used the the micro quick start to test this (./bin/start-micro-quickstart)

Load the example data from the ingestion spec below, this is an inline datasource. the data looks like this:

{"id":1,"duration":null}
{"id":2,"duration":null}
{"id":3,"duration":98}
{"id":4,"duration":6}
{"id":5,"duration":73}
{"id":6,"duration":1}
{"id":7,"duration":56}
{"id":8,"duration":null}
{"id":9,"duration":null}
{"id":10,"duration":null}

run the following query

select avg(duration) as "average", sum(duration) as "sum", count(duration)  as "count" from just_duration

Actual result :
{"average": 23, "sum": 234, "count": 5}

Expected result: the average should be 234/5 = 46.8
Example ingestion spec
{
  "type": "index_parallel",
  "spec": {
    "ioConfig": {
      "type": "index_parallel",
      "inputSource": {
        "type": "inline",
        "data": "{\"id\":1,\"duration\":null}\n{\"id\":2,\"duration\":null}\n{\"id\":3,\"duration\":98}\n{\"id\":4,\"duration\":6}\n{\"id\":5,\"duration\":73}\n{\"id\":6,\"duration\":1}\n{\"id\":7,\"duration\":56}\n{\"id\":8,\"duration\":null}\n{\"id\":9,\"duration\":null}\n{\"id\":10,\"duration\":null}"
      },
      "inputFormat": {
        "type": "json"
      }
    },
    "tuningConfig": {
      "type": "index_parallel",
      "partitionsSpec": {
        "type": "dynamic"
      }
    },
    "dataSchema": {
      "dataSource": "just_duration",
      "granularitySpec": {
        "type": "uniform",
        "queryGranularity": "NONE",
        "rollup": false,
        "segmentGranularity": "YEAR"
      },
      "timestampSpec": {
        "column": "id",
        "format": "posix"
      },
      "dimensionsSpec": {
        "dimensions": [
          {
            "type": "long",
            "name": "duration"
          }
        ]
      }
    }
  }
}

I've tracked down the issue to here https://github.com/apache/druid/blob/master/sql/src/main/java/org/apache/druid/sql/calcite/aggregation/builtin/AvgSqlAggregator.java#L71
we don't filter this count aggregator factory so it's counting all the rows in the column even if they're null.

took a stab at a fix here https://github.com/damnMeddlingKid/druid/pull/1/files . I check if the column is nullable and then use a filter aggregate factory to perform the count if it is nullable. Not sure if this approach is good, happy to clean up the PR and add tests if this is a good direction.

Hi @damnMeddlingKid,
Looks like this is a bug and you got the right fix. The logic is similar to what's in CountSqlAggregator already, so I'd suggest extracting it to a helper function, for example, CountAggregatorFactory.createFieldCountAggregatorFactory(PlannerContext, RowSignature, VirtualColumnRegistry, RexBuilder, RexNode). Then we could call that helper in AvgSqlAggregator as well. It would be similar to how it already calls out to SumSqlAggregator.createSumAggregatorFactory.
If you raised a PR doing this it would be appreciated! Please feel free to add me as a reviewer on it or at-mention me in a comment.

It's possible both cases (nullable and non-nullable) would be covered by existing tests, but if not, please add enough tests to cover both to CalciteQueryTest.

Thanks Gian, I'll refactor it and put up a PR.

Thanks everyone, i would also like to get opinions on https://github.com/apache/druid/issues/10073
if thats not the expected behaviour i'd like to work on a fix for that too.


fix for issue402: generic from super package resolving

I've not added info to plugin.xml release-notes, in reason to avoid minor file conflicts if possible

I've not added info to plugin.xml release-notes, in reason to avoid minor file conflicts if possible

Just don't add the "release 0.9.9: (community release)" line to multiple change lists.  8)


e2e: rte lane cleanups

improve the e2e rte lane to prepare to for future fixes (see #653 ).

on second thought we probably don't need this split anymore. Focusing on #653


feature request: uninstall

Seems there is no way to undo all the stuff that this tool does to a repository.
Looks like I will have to do it manually. :-(

Specifically, I am getting the below error anytime I commit to a repository, even after I brew uninstall, and delete / re-clone the repository.
[alex.mills@L110615 iris-app 08:29:42]  git commit -a
git: 'secrets' is not a git command. See 'git --help'.

Looks like to resolve this, you can just do rm -rf .git/hooks/*
But this removes ALL githooks. It should only be used by the desperate or reckless, and it does not resolve the core problem this tool has.

I came here looking for the same thing, and you can manually disable this by editing the following three files, or deleting them if there is only the one entry in them:

.git/hooks/commit-msg
.git/hooks/pre-commit
.git/hooks/prepare-commit-msg

Although no git secrets -r command exists, go into your local top level git repo directory and vi ~/.git/config. In config you should see your git secrets near the bottom blocked together under [secrets]. Scroll down to the desired row and type d to delete that secret row, then escape saving your changes.

https://github.com/awslabs/git-secrets/issues/95?email_source=notifications&email_token=ACSMZRNDVDZ3RW3HOL5HKDLRDX4BRA5CNFSM4GGPLGHKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOWYGERUA#event-3054258384 #Closed #95.

+1 to a clean uninstall process.
Current manual process requires you to:

delete secrets from .git/config for all local repositories
delete secrets from .gitconfig
remove commit-msg, pre-commit and prepare-commit-msg hooks from all git repositories
change/remove git template so that it does not configure these hooks for newer repos.
uninstall binary

This is too much manual work.

on top of, what @azazi-sa mentioned
You have to figure out where the binary is actually installed.
Following are the multiple places you can look in the Linux based machine.

By default all the git core command scripts are available in /usr/lib/git-core/ directory.
Apart from that, you can check in /usr/bin/ and /usr/local/bin/ directories as well.
If you are still NOT able to find the binary, then type which git-secrets to identify the place where actually git-secrets is installed.

This works for git hooks. But I cannot find, where a pattern configuration is stored:
git secrets --scan-history cat: 'C:/bin/Git/path/to/secret/file/patterns': No such f


Optimizing the getAttributeNameArray() by overriding the base impleme…

…ntation to remove hotstop on this method
This method from the base implementation is called on every event occurrence where it iterates the attribute list. Optimizing this method to compute the attribute list only once when the stream definition changes and not on every call.
Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues in the following format: Resolves issue1, issue2, etc.

Goals

Describe the solutions that this feature/fix will introduce to resolve the problems described above

Approach

Describe how you are implementing the solutions. Include an animated GIF or screenshot if the change affects the UI (email<EMAIL_ADDRESS>to review all UI text). Include a link to a Markdown file or Google doc if the feature write-up is too long to paste here.

Release note

Brief description of the new feature or bug fix as it will appear in the release notes

Documentation

Link(s) to product documentation that addresses the changes of this PR. If no doc impact, enter “N/A” plus brief explanation of why there’s no doc impact

Automation tests

Unit tests

Code coverage information

Integration tests

Details about the test cases and coverage

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes/no
Ran FindSecurityBugs plugin and verified report? yes/no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes/no

Samples

Provide high-level details about the samples related to this feature

Related PRs

List any other related PRs

Test environment

List all JDK versions, operating systems, databases, and browser/versions on which this feature/fix was tested

There is a check style failure in this PR. I have fixed in [1] with you commit. Please refer that.
[1] https://github.com/siddhi-io/siddhi/pull/1351


IndexError: index 0 is out of bounds for axis 0 with size 0 for plot_topomap based on only one channel

Description of the problem
If the EpochsSpectrum array contains only one channel, we get this error for plot_topomap() method:
IndexError: index 0 is out of bounds for axis 0 with size 0
Steps to reproduce
epochs = epochs.pick_channels( ['Fz'])
  band= {'Delta (0-4 Hz)': (0, 4), 'Theta (4-8 Hz)': (4, 8),
     'Alpha (8-12 Hz)': (8, 12), 'Beta (12-30 Hz)': (12, 30),
     'Gamma (30-45 Hz)': (30, 45),
           'High Gamma (50-80 Hz)': (50, 80)

           }
fig= epochs_selected.compute_psd().plot_topomap(bands={band['Alpha (8-12 Hz)']},show=False)

Link to data
No response
Expected results
..
Actual results
..
Additional information
..

you cannot plot a topomap based on only one channel. By definition a topomap is a spatially interpolated representation of the electromagnetic field on the scalp surface; you cannot interpolate between sensors if you only have one sensor.


scss_files option not working properly?

I've got a Node project with a .scss-lint.yml that looks like this:
scss_files: 'src/app/**/**/*.s+(a|c)ss'
exclude:
    - 'dist/*'
    - 'node_modules/*'
    - 'something else'

I'd expect that only files whose path match the one given in scss_files will be linted. However, I have to have the dist and node_modules entries in the exclude as otherwise scss files in there are also linted, which slows everything down and gives a lot of unwanted output. I don't think this is expected behaviour, or am I doing something wrong?

Hey @sblask,
To properly exclude all items under a directory, you need to use the **/* pattern, e.g.
exclude:
  - 'dist/**/*'
  - 'node_modules/**/*'

Regarding the behavior of scss_files: how are you invoking scss-lint? If you specify a file, then the scss_files option is ignored and that file is used instead. Thus running scss-lint . will ignore the scss_files option, which could explain why your dist and node_modules directories are getting linted.

Thanks for the help! I was running scss-lint . because when I ran scss-lint I got No SCSS files matched by the patterns: (but no pattern was printed to screen, so not a big help figuring out the problem). I found the solution now, this: *.s+(a|c)ss did not work (we are moving from sass-lint to scss-lint and there it did work). I am not even sure it should work, but as all our files are *.scss anyway, I changed scss_files to src/app/**/**/*.scss and now I can run scss-lint and everything gets linted.

Hey @sblask,
The reason the previous pattern didn't work (now that you mention it) is because (a|c) is not the way to do alternation with file glob patterns (though that is the way to do it with regexes). Thus if you had written {a,c} you would have had a working pattern.
Having said that, scss-lint doesn't support Sass anyway, so you wouldn't want to be linting your .sass files.