next function causing issue some times

in my express.js app have an POST route (given below). when user performs an action the POST route gets called and the logic under post /response gets executed and then the user gets redirected to another page. most of the time this flow worked without issues, but one of the time i see POST being invoked and then immediately there is an GET call for the post URL which resulted in 404 and none of the code under post /response block got executed
i am wondering if the next function is causing any issue by converting the POST to GET and not letting the code under POST /respone to execute?
app.use(function (req, res, next) {
  res.locals.user = req.session.user;
  logger.log("PAGE", `Page: ${req.originalUrl}`);
  next();
});

app.post("/response",
 [check('id').exists(),
 check('check').exists()],
 (req, res) => {
//logic here
return res.redirect("page2");
}

Apache log:
xx.xx.xx.xx - - [30/Mar/2020:21:30:00 -0400] "POST /response?c=val1&id=val2 HTTP/1.1" 302 263
xx.xx.xx.xx - - [30/Mar/2020:21:30:00 -0400] "GET /response?c=val1&id=val2 HTTP/1.1" **404** 15974

@veeRN135 closing this issue as cannot confirm issue is express. Happy to re-open if you can post a test case w/o Apache that illustrates the issue.


Remove double quote - return NULL when entityMapper.getTableMetaSpace()

MappingManager manager = new MappingManager(session);
Mapper<Post> m = manager.mapper(Post.class);

m.getTableMetadata() // is returning NULL

We are double quoting the name of the table, and we are not able to find in the table map of the Keyspace, which is a bug

Good catch, your fix looks good to me. Some minor remarks:

should_return_table_metadata would be a better name for the test
the commit message could be improved, mentions "getMetaSpace" and it's not clear whether "throw NPE" is the fix or the initial problem. Suggestion: "Fix quoting issue that caused Mapper.getTablemetadata() to return null".

Could you do these changes and rebase on the current 2.1? We're planning to release 2.1.9 soon and I'd like to include this, so if you don't have time today I'll do the changes myself (keeping proper attribution of the commit).

Cherry-picked as 97363ce, thanks.

Thanks. i was planning to do it.
I would like to know if this fix will be included in the branch 2.2 ?

Yes, the fix will be merged to more recent branches.
However, we plan to retire the 2.2 branch in favor of 3.0, I'll send an announcement about that on the mailing list next week.

Created JAVA-978 for this for tracking.


Why is conforming to ListBindable required?

New issue checklist

[x] I have reviewed the README and documentation
[x] I have searched existing issues and this is not a duplicate

General information

IGListKit version: 3.0
iOS version(s): Any
CocoaPods/Carthage version: 1.2.1
Xcode version: 8.3.3
Devices/Simulators affected: Any
Reproducible in the demo project? (Yes/No): Yes
Related issues: #449

Debug information
Not attaching debug information since this is just a question.
Question
First of all, thank you so much for this binding section controller - I was frustrated about how cumbersome it would be to dynamically update certain parts in our app, and suddenly in a few hours that was gracefully handled by another IGListKit component :)
The only part I didn't like (yet!) though was that I was required to implement the ListBindable protocol. I know: I'm probably doing it wrong, but because of the nature of my project I don't see any issues from what I'm doing (I don't ever update existent cells, I do only append more).
When using the regular ListSectionController class, I was creating and setting up my cells from the cellForItem: method. Now we have cellForViewModel:atIndex: - ok, I've just moved the code over to the new method basically. We're not even using the view model parameter, to be honest, given that the section controller owns an array of objects and thus it knows exactly which object to take from the array (by its index). I can still see my viewModelsFor: being useful - internally - since they're input to IGListDiff.
All good, let's run the project - crash because our cells do not implement ListBindable. So checking your LabelCell example, this method's responsibility is basically hooking a model to the view (couldn't have guessed!). So let's say my cells are also LabelCells: I'm already doing label.text = objects[index] as? String in cellForViewModel:atIndex:, and I really don't see (yet!) a reason to review my architecture in tens of cells just to separate this line into a new method. I mean, I can see why it looks good to have this separated, such as being able to keep things private in the cell, but it has never been necessary on iOS nor on IGListKit, and trust me: we've really done a big architecture with Swift protocols that allows us to export this cellForItem: logic to outer classes, it looks and works good and the question is exactly: why would we need to change?
Given the limitations of adding default implementations to @objc protocols, we've added the boilerplate ListBindable conformance with empty bindViewModel: implementation to every cell. It just works. Even being empty. In IGListBindingSectionController.m, I see a cell being built at an oldUpdatedIndex and then having a model from an indexAfterUpdate hook up to it, so that's kind of a big hint already, except that my view models indices do never update. I'd be convinced that my scenario is just too specific, yet apparently we're not the only ones doing it and after applying the same reasoning to your MonthSectionController example (moved ListBindable implementation from LabelCell into cellForItem: in the section controller), it still works exactly as before.
TL;DR

Is it possible to make ListBindable conformance optional?
If not, what are exactly all the functional issues when not separating code out from cellForItem: into bindViewModel:?

Current code:
UICollectionViewCell<IGListBindable> *cell = [collectionContext cellForItemAtIndex:oldUpdatedIndex
                                                                 sectionController:self];
[cell bindViewModel:self.viewModels[indexAfterUpdate]];

What I propose:
UICollectionViewCell<IGListBindable> *cell = [collectionContext cellForItemAtIndex:oldUpdatedIndex
                                                                 sectionController:self];
if ([cell conformsToProtocol:@protocol(IGListBindable)]) {
    [cell bindViewModel:self.viewModels[indexAfterUpdate]];
}

Thank you so much!

Great questions! And you did quite a lot of investigation too.
You can absolutely still do your cell config in cellForItemAtIndex:. It's exactly the same as doing it in the bind method.
The binding exists because of how common practice it is to configure a cell given a view model when the model drives how many and what type of cells exist. And you're right, it helps separate code.
But if you have your own architecture, that's good too!
Also, sometimes you will still have to do extra work in cellForItemAtIndex:, e.g. If your cell has a delegate.
Why is it required? For perf reasons. Checking protocol conformance is immensely more expensive than checking for class type due too ObjCs dynamic nature with protocols and optionality. This code executes in Instagram, so we want it lightning fast.
Empty bind methods are totally fine (and MUCH faster).

Great, I confess that's news to me :)
I'm usually all for avoiding boilerplate - in which case I'd probably rather not having ListBindable at all - but as I said, I know it is an architecture improvement, so it's fine. Thanks a lot!


Bad minimal NIFTI size check

For INCF/BIDS-examples#48, I've created some minimal NIFTI files, which are just the header (+ 4 pad bytes), and gzipped. I'm getting the following error when validating: This file is too small to contain the minimal NIfTI header.
I suspect it may be checking for a 352 byte minimum for the file itself, not the unzipped contents.

It's hard to figure out what should be the size of a compressed header
(since different compression levels could be used). This seems like an edge
case - would you mind using uncompressed files in your example?
Best,
Chris
On Mon, Feb 5, 2018 at 7:29 AM, Chris Markiewicz<EMAIL_ADDRESS>wrote:

For INCF/BIDS-examples#48 https://github.com/INCF/BIDS-examples/pull/48,
I've created some minimal NIFTI files, which are just the header (+ 4 pad
bytes), and gzipped. I'm getting the following error when validating: This
file is too small to contain the minimal NIfTI header.
I suspect it may be checking for a 352 byte minimum for the file itself,
not the unzipped contents.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/INCF/bids-validator/issues/394, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOkp3NP-3XZipRMZn-M4iUB9wMCcHWvks5tRx5KgaJpZM4R5nkH
.

Sure.


useEventListener - unexpected token

I had this on the latest version:
./node_modules/usehooks-ts/dist/esm/useEventListener/useEventListener.js 10:34
[1] Module parse failed: Unexpected token (10:34)
[1] File was processed with these loaders:
[1]  * ./node_modules/react-scripts/node_modules/babel-loader/lib/index.js
[1] You may need an additional loader to handle the result of these loaders.
[1] |   }, [handler]);
[1] |   useEffect(() => {
[1] >     const targetElement = element?.current || window;
[1] |
[1] |     if (!(targetElement && targetElement.addEventListener)) {

Hi @Johannes5, thanks for reporting the bug! I think it is related to #97, so you can use<EMAIL_ADDRESS>or you also can try our test v3 test-usehooks-ts where we have fixed this problem.


cvat_server container does not start successfully

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

Expected Behaviour

cvat_server container should be start successfully
Current Behaviour

please see logs
Possible Solution

https://github.com/opencv/cvat/blob/5e2eda76014c325c69dc297749d1d123ed315626/Dockerfile#L147
may be this should be added before.
RUN mkdir ${HOME}/supervisord

Steps to Reproduce (for bugs)

git clone https://github.com/opencv/cvat
cd cvat
docker-compose up -d

Context

Your Environment

Git hash commit (git log -1): 916b5dcfd840d066ca4d4673a65fa4226150eb15
Docker version docker version (e.g. Docker 17.0.05): 20.10.6
Are you using Docker Swarm or Kubernetes? no
Operating System and version (e.g. Linux, Windows, MacOS): linux
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Logs
docker logs -f --tail=100 cvat_server

output:
Error: could not find config file supervisord/server.conf
For help, use /usr/bin/supervisord -h
Error: could not find config file supervisord/server.conf
For help, use /usr/bin/supervisord -h
Error: could not find config file supervisord/server.conf
For help, use /usr/bin/supervisord -h
Error: could not find config file supervisord/server.conf
For help, use /usr/bin/supervisord -h
Error: could not find config file supervisord/server.conf
For help, use /usr/bin/supervisord -h

@azhavoro , could you please look?

your docker version ?
Maybe you can try     COPY --chown=“django” supervisord/ ${HOME}/supervisord

@maltoze Hi,
The reason is that the build config doesn't really point to development images, so as a workaround I would suggest 2 options:
specify version CVAT_VERSION=dev docker-compose up -d
or build actual images yourself with
docker-compose -f docker-compose.yml -f docker-compose.dev.yml up -d --build
As generall solution I think we can set dev instead of latest for develop and pin actual version for each CVAT release. Now downloading the compose config from the develop branch, we do not run a really developed version, actually we run the latest release. @nmanovic @klakhov  What do you think?

@azhavoro  I'm encountering the same issue. Could you specify how I should get the latest release of CVAT up and running using docker-compose?

@azhavoro I'm encountering the same issue. Could you specify how I should get the latest release of CVAT up and running using docker-compose?
Try this

CVAT_VERSION=dev docker-compose pull
CVAT_VERSION=dev docker-compose up -d

@azhavoro that works indeed, but it seems strange that we should deploy with the 'dev' tag? Will these always be stable?

@azhavoro that works indeed, but it seems strange that we should deploy with the 'dev' tag? Will these always be stable?

The problem is only relevant for the develop branch, so I don't think it's strange to use dev images with the development branch. If you want to deploy the latest release: https://github.com/opencv/cvat/releases/tag/v2.1.0.

The problem is only relevant for the develop branch, so I don't think it's strange to use dev images with the development branch. If you want to deploy the latest release: https://github.com/opencv/cvat/releases/tag/v2.1.0.

My bad @azhavoro , I didn't notice the default branch on github was the development branch! Thanks for your reply.


Sign out dialog: pressing enter again should actually sign out

Issue: Y2K-785

CI fail is unrelated and is fixed in master.


fix(color-string): bump version to 1.5.5

Codecov Report

Merging #292 (36fd4f1) into master (a5140ea) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #292   +/-   ##
=======================================
  Coverage   44.90%   44.90%
=======================================
  Files          50       50
  Lines        3042     3042
  Branches      533      533
=======================================
  Hits         1366     1366
  Misses       1499     1499
  Partials      177      177

Continue to review full report at Codecov.

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


[wip] Investigate dev publish gate

https://github.com/DataDog/datadog-ci-azure-devops/pull/22/files


Add home link to navindu

You can see here that you have changed 6 files. So that means that all the commits you made for other Pull requests are on this branch too. So all the previous pull requests are redundant. I think you haven't understood the github workflow properly. Go to youtube and understand the basics of branches and commits and PRs, it will be helpful in the future. Happy Hacking. I am merging this because I have nothing to lose., but most repos won't allow such PRs

I was opening pull request for every commit I was making. Am I supposed to wait till PR to get merged before opening another PR?


LSP: Go to type definition support

Problem

Given a variable, foo, it can be difficult to jump to the location that defines the type of foo.
Proposed solution

Implement "Go to type definition...". The behavior in TypeScript is that it jumps to the location that defines the type of the thing you are pointing at. We should implement the same behavior for Sorbet.
Open questions

How should this feature handle complex types? (Union/nilable/etc)

It could return the locations for each component of the type...? Or return nothing.
TypeScript returns a location for each type in a union, but nothing for intersection types.

Differences from "go to definition"
Given a variable foo of type Bar, "Go to definition" goes to the definition of foo. "Go to type definition" goes to the definition of Bar, which is the type of foo.

Note that this feature is different from "go to definition"

(Updated description to clarify)

I don't see anything about IDE support in the docs. If I'd like to learn more about IDEs that support sorbet, where could I read more? Thanks.

@jaredbeck Thanks for your interest! We're still working on making IDE support decent, so we have not documented it yet or made an announcement. When it is released, we will likely make a public announcement. It will work with any editor that supports the language server protocol (either natively or, more typically, via a plugin/extension). Internally at Stripe, we're actively working on VS Code as a first class citizen, but we know that it works with LSP plugins for emacs and vim, too.

Closed in #1704


[Feature] Adding Search Function into each Components

Description
The search function is only available on the home page to search components. I want to add a search function to every component page also, to specifically search any element into that component.
Assign me under JWOC 2024, if approved.
Screenshots
Ex : I will add a search function here to search backgrounds...

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[X] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

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

Proceed @Surajit0573

Status of your PR @Surajit0573

working on it...will PR before end of the day.


gmsv_workshop_linux.dll - attempt to index global 'steamworks' (a nil value)

I am running a x32 server and am attempting to use steamworks but am having difficulties with it. I have added it to the correct path and I also went through the troubleshooting steps but have had no luck as of yet.
I was wondering if I was able to get some help with this. I have also sent you an email.
The error.

Did you put require("workshop") in a serverside file?
Is it being called BEFORE line 144 in sv_workshop.lua?
Did you install the DLL to garrysmod/lua/bin/gmsv_workshop_linux.dll ?
Are there any errors when the DLL is required?

Yes
Yes
Yesssir
No errors appear when workshop is being required

I've just tested this on my side just in case function signatures or anything like that have changed with the latest Garry's Mod updates, but everything works fine.

Are you using a provider for your server? (If yes, which one? If not, what's your operating system?)
Could you show the content of your file to check how the module is loaded?
Could you write status in the server console and show what it contains?

Yeah its weird how its not working for me not sure why, it worked on another dedi we had.
We are  just running the server through pterodactyl (Ubuntu 22.04).
.dll location
Our server is on version : 2023.06.28/24 9040 secure (I have tried using older versions of the module)

yer mate

Yeah, I still get an error when I use lua_run, says the same thing. Hopefully we are able to get it working.


Cipher Algorithms Added

What does this PR do?
Added Cipher Algorithms
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)
Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
Have you read the Contributing Guidelines?
Yes

@Add it in Algorithms/Ciphers

@DeepanshuGOYAL05 add it in Algorithms/Ciphers

@DeepanshuGOYAL05 Add it in Algorithms/Ciphers

Yaa Sure
On Tue, Oct 6, 2020, 5:27 PM Kunal Mali<EMAIL_ADDRESS>wrote:

@DeepanshuGOYAL05 https://github.com/DeepanshuGOYAL05 Add it in
Algorithms/Ciphers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dubesar/Java-A-Z/pull/421#issuecomment-704219107, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6TG73Q3FQVT3ZZ3P6JPTTSJMA3LANCNFSM4SF4XMHA
.

Hey,
Check it now!!
On Tue, Oct 6, 2020, 5:27 PM Kunal Mali<EMAIL_ADDRESS>wrote:

@DeepanshuGOYAL05 https://github.com/DeepanshuGOYAL05 Add it in
Algorithms/Ciphers
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dubesar/Java-A-Z/pull/421#issuecomment-704219107, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6TG73Q3FQVT3ZZ3P6JPTTSJMA3LANCNFSM4SF4XMHA
.

Please review thePR

@DeepanshuGOYAL05 We have made a folder named "Cypher" already in the master branch but as you have forked earlier you are not able to see the folder. Check it once in the main repo of java-a-z

Ohh Okay
On Tue, Oct 6, 2020, 7:18 PM Kunal Mali<EMAIL_ADDRESS>wrote:

@DeepanshuGOYAL05 https://github.com/DeepanshuGOYAL05 We have made a
folder named "Cypher" already in the master branch but as you have forked
earlier you are not able to see the folder. Check it once in the main repo
of java-a-z
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dubesar/Java-A-Z/pull/421#issuecomment-704281371, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6TG753O7EGABUTHYM2T3DSJMN3PANCNFSM4SF4XMHA
.

@DeepanshuGOYAL05 You have git pull command to update your repo with the latest.

@DeepanshuGOYAL05 You have to only add Diffie_Hellman.java in the Cyphers folder as the rsa.java is already there.

Hey,
I have added it. Please check
On Tue, Oct 6, 2020, 7:57 PM Kunal Mali<EMAIL_ADDRESS>wrote:

@DeepanshuGOYAL05 https://github.com/DeepanshuGOYAL05 You have to only
add Diffie_Hellman.java in the Cyphers folder as the rsa.java is already
there.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dubesar/Java-A-Z/pull/421#issuecomment-704306858, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6TG7ZWGHGXZM6ZKYJGDTDSJMSNFANCNFSM4SF4XMHA
.


chore: remove unused fp-ts utils

This is part of the Gatsby fp-ts refactor.

Current dependencies on/for this PR:

main

PR #198

PR #192

PR #193

PR #194

PR #195

PR #196

PR #197

PR #199


HTML template for notifications with FlowDock

I created an HTML template, similar to the existing email body template. I use it for content in the Flowdock message. It's rather generic, and I think it can be used also in other HTML-capable notification receivers like Hipchat.
In Flowdock it looks like:

:+1: otherwise. We will want to migrate this to a generalized and user-configurable templating system soon.

@t0mk Sorry, this one slipped through the cracks due to the lack of another ping after the last update. Could you squash the commits (and rebase on current master if necessary).

Yeah, I'll do it on Monday.
On Sat, Jul 11, 2015 at 7:16 AM, Julius Volz<EMAIL_ADDRESS>wrote:

@t0mk https://github.com/t0mk Sorry, this one slipped through the
cracks due to the lack of another ping after the last update. Could you
squash the commits (and rebase on current master if necessary).
—
Reply to this email directly or view it on GitHub
https://github.com/prometheus/alertmanager/pull/80#issuecomment-120571307
.

uh, let me fix the test

I squashed the commits, fixed the test, and wrote a test for flowdock notification.

@t0mk Thanks! Sorry again - was on vacation :) Now finally merging.


Adds the ability to save/load SF Programs to/from Blackbird files

Adds the module io.py, containing the functions:

to_blackbird(): converts a SF Program object to a Blackbird Program object (for internal use only)

to_program(): converts a BB Program object to a SF Program object (for internal use only)

save(): saves a quantum program to a Blackbird .xbb file.

load(): loads a quantum program from a Blackbird .xbb file.

Import save and load into __init__ to make them top level functions

Adds the blackbird library as a dependency of SF.

Looks good. A couple minor questions, but the main thing that caught my attention is that programs are automatically and implicitly compiled when a bb file is loaded and it has a target. Do we want to do this?

@smite and I had a big discussion about this as I coded it, and went through all possible user expectations, and this one made kind of made the most sense.
One alternative would be to 'trick' the program into thinking it was already compiled, i.e.,
prog.backend = bb.target['name']

but we thought this would lead to too much unexpected behaviour.
The final alternative would be to just ignore the target keyword, but this also seems unexpected from a user point of view.

Autocompilation has been removed. Now, if the target is provided, the attribute program.backend is set.
Todo: add ability to Program.compile() to use existing prog.backend attribute if no argument is provided


Error if CAA 'iodef' record defined

Hi,
It seems if a zone in Namecheap contains a CAA 'iodef' record, the Namecheap Terraform provider fails to create DNS records. If the CAA record is removed, the creation succeeds. I've tested with A and CNAME records, using Terraform 1.4.0 and 1.4.6, with Namecheap provider 2.1.0.
Terraform Version
Terraform v1.4.6

provider registry.terraform.io/namecheap/namecheap v2.1.0

Namecheap provider version
What version of terraform-provider-namecheap are you using? 2.1.0
Steps to Reproduce

Create a CAA record such as below in the zone file on Namecheap

CAA Record
@
iodef
<EMAIL_ADDRESS>

Create Namecheap resource block

resource "namecheap_domain_records" "create-record" {

    domain = "domain.com"
    mode = "MERGE"

    # ACM Certificate Validation Record
    record {
        hostname = "test"
        type = "A"
        address = "<IP_ADDRESS>"
        ttl = 1800
    }
}

Run 'terraform apply'
Output is an error such as below.

Error: Records[10].Address "0 iodef<EMAIL_ADDRESS>must contain a protocol prefix for CAA iodef record

I'm not sure why they're not merging it, but https://github.com/namecheap/terraform-provider-namecheap/pull/66 has the fix you're looking for.

Thanks, and good work! I have a case open with Namecheap and have passed this on. Fingers crossed they actually do something about it.

I just tried to add the following records and failed with the error message below.
If I add it in the Web GUI it works without any complaints.
I'm sure somehow the validation logic does not recognise mailto: correctly.
Code
 record {
    address  = "0 issue \"pki.goog\""
    hostname = "@"
    mx_pref  = 10
    ttl      = 1799
    type     = "CAA"
  }
  record {
    address  = "0 iodef<EMAIL_ADDRESS>    hostname = "@"
    mx_pref  = 10
    ttl      = 1799
    type     = "CAA"
  }

Error
│ Error: Records[18].Address "0 iodef<EMAIL_ADDRESS>must contain a protocol prefix for CAA iodef record
│
│   with namecheap_domain_records.larsartmann_com,
│   on larsartmann.com.tf line 5, in resource "namecheap_domain_records" "larsartmann_com":
│    5: resource "namecheap_domain_records" "larsartmann_com" {
│

Update: if it's fully synced but you want to change any other part of the domain e.g. TXT _dmarc it wouldn't let you/me.

The problem
https://github.com/namecheap/terraform-provider-namecheap/blob/5e44263d262d771fbc46c6cda8b2bf5bcd98810b/vendor/github.com/namecheap/go-namecheap-sdk/v2/namecheap/domains_dns_set_hosts.go#L40
validURLProtocolPrefix checks for :// but mailto: doesn't contain //
See: https://www.rfc-editor.org/rfc/rfc6844#:~:text=5.4.  CAA iodef Property
https://github.com/namecheap/terraform-provider-namecheap/blob/5e44263d262d771fbc46c6cda8b2bf5bcd98810b/vendor/github.com/namecheap/go-namecheap-sdk/v2/namecheap/domains_dns_set_hosts.go#L193
Suggested fix
 if strings.Contains(*record.Address, "iodef") && !(validURLProtocolPrefix.MatchString(*record.Address) && strings.HasPrefix(*record.Address, "mailto:")) {

Please give a try to the latest version of this provider released yesterday.


Remove Numbered Bullets in Object Assignability BBE

Purpose
This to fix the formatting error in displaying bullet points in the "Object Assignability" BBE description.
Related to https://github.com/ballerina-platform/ballerina-lang/pull/21838.
Fixes #21785
Approach

Describe how you are implementing the solutions along with the design details.

Samples

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

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #21997 into stage will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            stage   #21997   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

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


Support & test import() for non-bundled code

Well... this is weird. "Test basic import syntax" passed ... it should've failed. 🤔
Does Jest support import() or is something else in our tool chain handle this?

Now I'm just really confused .. fails locally but not on CI?

Maybe we could try running Jest with --no-cache on CI just in case.

I believe we already do: https://github.com/facebookincubator/create-react-app/blob/d822d8d479ef7986b394f39775aa1dc46540e7b2/tasks/e2e-kitchensink.sh#L138

I want to see a failing CI test 😢

I wonder if our CI tests even work ...

/cc @EnoahNetzach

Back when I was working on CI tests, I tried to make it fail (e.g. with #1160), and my tests always failed correctly..
Also, running this branch in a docker container EnoahNetzach:create-react-app/local-testing won't fail.
Did you link  babel-preset-react-app? I just tested it and without it it was failing.

@gaearon I think the tests are just flaky (for an unknown reason).
Notice the fail and pass when I revert and then revert the revert.
Should I create a new PR?

Flaky as in "sometimes fails when it shouldn't" is understandable and fine.
Flaky as in "sometimes passes when it shouldn't" is weird and we need to figure out why.

Okay, so it passes when the other commit it present. a373aec34a0e8a541bc40a7dff836d29f199ddfb fails when c2ab8fbf375b75332b7f7cb1560de8b318bc5624 is present, so I'm not sure if this is a travis bug or a e2e bug (or just how travis works).

Looks like travis does git fetch origin +refs/pull/1615/merge: and git checkout -qf FETCH_HEAD which means Travis is only capable of testing the latest given commit in a PR, not older commits.

Do you mean that it might display a previous commit has passed while in reality it has grabbed the most recent pushed commit (that hasn't reached the queue yet)?

That's exactly what it does. But only for PRs.

OK, that makes sense.
So what happened is:

You push commit A
Travis enqueues a job
You push commit B
Travis enqueues another job
Travis executes the first job, effectively fetching commit B, but GH integration shows the checkmark for A
Travis executes the second job

That would be it. 😄

OK today I learned. 😅

I noticed this when I pushed the second commit shortly after the first, because Node 4 kitchensink had started (and failed) and then the Node 6 kitchensink (queued not executing when I pushed second commit) succeeded. I then re-validated my theory by only running the first commit alone then pushing the second commit and re-running the first job, seeing it magically switch from failing to passing.
Think this is worthwhile enough to report to travis?

Meh, I guess this is as designed.

Good job figuring it out tho

Thanks! Good to land?

Yep

Just to clarify, because the wording of "effectively fetching commit B" confused me a bit:
This is because Travis wants to use the post-merge commit – ie, when a push on a PR happens and GitHub checks if it is "ok to merge", it actually performs the merge itself between the current state of the PR branch and the branch it's been opened on (eg, master).
The resulting commit (which is only available remotely, as something like +refs/pull/1615/merge – a moving target) is the one that Travis is trying to use because it's a more comprehensive check of how things will probably look if the PR is merged. It could of course just use the commit that was pushed, which wouldn't have race conditions, but that commit could be out of date w/ respect to the branch the PR has been opened on.
If GitHub offered a reference like refs/pull/1615/merge-<commit-sha> then this wouldn't be an issue (but it would also require an order of magnitude more references on GH's end).


[WIP] changes to compile on post-M4 nightlies

not mergeable until M5 is released, because of the msg.formatDefSiteMessage change

Happy to merge this if it's ready.

Superceded by https://github.com/milessabin/shapeless/commit/cb973aa10f066782e8543fdce8d44b2d83996801.


Getting KVStore error for 'image-classification' examples.

I just pulled down 0.5.0 of mxnet and installed it on my Amazon EC2 instance. However, when I try to run train_mnist.py and train_cifar10.py I get the following output:
$ python train_mnist.py
2015-12-15 23:39:23,949 Node[0] start with arguments Namespace(batch_size=128, data_dir='mnist/', gpus=None, kv_store='local', load_epoch=None, lr=0.1, lr_factor=1, lr_factor_epoch=1, model_prefix=None, network='mlp', num_epochs=10, num_examples=60000)
[23:39:25] src/io/iter_mnist.cc:91: MNISTIter: load 60000 images, shuffle=1, shape=(128,784)
[23:39:25] src/io/iter_mnist.cc:91: MNISTIter: load 10000 images, shuffle=1, shape=(128,784)
Traceback (most recent call last):
  File "train_mnist.py", line 122, in
    train_model.fit(args, net, get_iterator)
  File "/home/ubuntu/mxnet/example/image-classification/train_model.py", line 69, in fit
    epoch_end_callback = checkpoint)
  File "/home/ubuntu/.virtualenvs/mxnet/local/lib/python2.7/site-packages/mxnet-0.5.0-py2.7.egg/mxnet/model.py", line 724, in fit
    kvstore, len(self.ctx), self.arg_params)
  File "/home/ubuntu/.virtualenvs/mxnet/local/lib/python2.7/site-packages/mxnet-0.5.0-py2.7.egg/mxnet/model.py", line 145, in _create_kvstore
    raise TypeError('kvstore must be either KVStore or str')
TypeError: kvstore must be either KVStore or str
I've tried manually setting --kv-store "local" in the command line arguments (even though "local" is what the value defaults to). I'm sure there is something simple I'm missing, but I'm not seeing it.

I met the same problem, even when I set --kv-store "local", the problem still exists, do you know other solutions? Thanks

please reinstall the mxnet python package, since we slightly changed model.py

@mli Thanks, it is ok now

@mli: Confirmed, re-installing the mxnet Python package resolved the issue. Thanks!


feat: compat with python3.7

Hello,
I'm the author of django-admin-tailwind (a django admin theme with TailwindCSS and AlpineJS too).
Congratulations for this great work (much better than my theme).
I'd like to add compatibility with python3.7 and I'm ready to make the effort to make it happen if it's okay with you.

Hey Aleksi, thanks for kind words, always nice to meet fellow Django developer :-P I left few comments in your PR but then I checked for how long 3.7 is going to be officially supported and after quick look I noticed that support is planned for another 6 days :-D I'm not sure if it makes sense to put an energy into this topic. What do you think?
https://endoflife.date/python

Hey @lukasvinclav, thanks for your feedback. I've just fixed your comments.
I work for a company that can't upgrade to python3.8 right now. They want an advanced Django Admin (django-unfold is perfect for what they want). I'd like to contribute directly to this repository rather than maintain a fork if you agree.
I could also bring in some new features later :)

Just tested python 3.7 support and it seems that Poetry requires at least 3.8.

@Aleksi44 Regarding the lru_cache, I did some investigation and maybe I would prefer to use it with explicitly setting default values. You can read more here:
https://stackoverflow.com/questions/47218313/use-functools-lru-cache-without-specifying-maxsize-parameter
I think this is the last thing which we are missing and then we are good to go.

Hey @lukasvinclav
I see that the commit you merged is no longer on main. Did you have any problems?

@Aleksi44 I sent you an email last Friday. I used address from your profile.


[macOS, .NET] Fix dotnet binary detection.

macOS app bundles have extremely limited PATH, and will never have dotnet in it, so we need to look for it manually.

Shouldn't we try the default installation path after OS.PathWhich("dotnet") fails to find dotnet in the PATH?
Also, should this method mirror what we do in hostfxr_resolver.cpp to get the default installation path for all OS?
https://github.com/godotengine/godot/blob/bf271ea6af5e4e1dbf12a5428445f0efa54f8651/modules/mono/editor/hostfxr_resolver.cpp#L173-L217

Shouldn't we try the default installation path after OS.PathWhich("dotnet") fails to find dotnet in the PATH?

It's pointeless to look in PATH, it will never have dotnet, for bundles it's locked to /bin and /usr/bin and both are read only (a separate and digitally signed disk volume).

Also, should this method mirror what we do in hostfxr_resolver.cpp to get the default installation path for all OS?

I guess it might be useful to add it for other platforms (or directly call this method), but not as important, most will have it in the path.

Thanks!

Also, should this method mirror what we do in hostfxr_resolver.cpp to get the default installation path for all OS?
https://github.com/godotengine/godot/blob/bf271ea6af5e4e1dbf12a5428445f0efa54f8651/modules/mono/editor/hostfxr_resolver.cpp#L173-L217

The C# implementation should probably use hostfxr_resolver through an internal call.


[Bug] Lightspeed.vim jumps incorrectly on wrapped lines

I've noticed that Lightspeed sometimes would make weird jumps and not land on target. It seems to happen on long, wrapped lines.
Here are two paragraphs with the first being broken into five separate lines and the second one is a single wrapped line. I'm trying to jump to the word "refined" on the third line.
It works correctly on the split-line paragraph, but lands on "work-in-progress" on the second line in the wrapped-line paragraph:

I tried the same in neovim itself and it works correctly in both cases (although I noticed it labels targets a bit differently, but I'm not sure if it matters):

I tired disabling all other VSCode extension, but that didn't change anything.
VSCode version: 1.69.0
neovim version: v0.7.0
VSCode Neovim version: v0.0.87

Yesterday's update:

this issue also affects Leap and Sneak
the incorrect jump described above only happens if the cursor is located on the line immediately above the line you are targeting

Today's update:

I can't reproduce it anymore 🎉🎉🎉
I see that both VSCode and VSCode Neovim versions got bumped up

Hurray!


chore(main): release 0.12.0

:robot: I have created a release beep boop
0.12.0 (2023-08-03)
Features

s E opens search editor to the side (3a00c72)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/MarcoIeni/glimpse/releases/tag/v0.12.0 :sunflower:


Missing GitHub Link

Type: Missing
Summary:
Missing GitHub Link
Details:
No GitHub Link provided
Resolution:
Using GitHub
Affected definitions:

clap 3.0.0-beta.5

Thank you for your contribution! 😄


Framework fails to download with v1.0.134

With v1.0.134, framework fails to download:
$> kubescape download framework nsa --output nsa.json
Error: control 'nsa' not found
Usage:
  kubescape download framework/control <framework-name>/<control-name> [flags]
Supported frameworks: [nsa mitre armobest]

@uncycler Thanks. I'm on it!

Fixed and published. Thank you

it's all good, thx!


X Error of failed request when running roslaunch kinect2_bridge kinect2_bridege

Hello,
I was trying to connect Kinect v2 to the robot and make it work, I am using ssh -X to access the robot's graphics, but every time I try to run roslaunch or rosrun, it shows X Error of failed request.
The result is as follows:
administrator@jackal:~$ roslaunch kinect2_bridge kinect2_bridge.launch
... logging to /home/administrator/.ros/log/3343e64e-7c67-11e7-ad07-bf271f6e5f05/roslaunch-jackal-8959.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.
started roslaunch server http://jackal:51062/
SUMMARY
PARAMETERS

/kinect2_bridge/base_name: kinect2
/kinect2_bridge/base_name_tf: kinect2
/kinect2_bridge/bilateral_filter: True
/kinect2_bridge/calib_path: /home/administrat...
/kinect2_bridge/depth_device: -1
/kinect2_bridge/depth_method: default
/kinect2_bridge/edge_aware_filter: True
/kinect2_bridge/fps_limit: -1.0
/kinect2_bridge/jpeg_quality: 90
/kinect2_bridge/max_depth: 12.0
/kinect2_bridge/min_depth: 0.1
/kinect2_bridge/png_level: 1
/kinect2_bridge/publish_tf: False
/kinect2_bridge/queue_size: 5
/kinect2_bridge/reg_device: -1
/kinect2_bridge/reg_method: default
/kinect2_bridge/sensor:
/kinect2_bridge/use_png: False
/kinect2_bridge/worker_threads: 4
/kinect2_points_xyzrgb_hd/queue_size: 5
/kinect2_points_xyzrgb_qhd/queue_size: 5
/kinect2_points_xyzrgb_sd/queue_size: 5
/rosdistro: indigo
/rosversion: 1.11.21

NODES
/
kinect2 (nodelet/nodelet)
kinect2_bridge (nodelet/nodelet)
kinect2_points_xyzrgb_hd (nodelet/nodelet)
kinect2_points_xyzrgb_qhd (nodelet/nodelet)
kinect2_points_xyzrgb_sd (nodelet/nodelet)
ROS_MASTER_URI=http://localhost:11311
core service [/rosout] found
process[kinect2-1]: started with pid [8977]
process[kinect2_bridge-2]: started with pid [8978]
process[kinect2_points_xyzrgb_sd-3]: started with pid [8979]
process[kinect2_points_xyzrgb_qhd-4]: started with pid [8980]
[ INFO] [1502219220.913502562]: Loading nodelet /kinect2_bridge of type kinect2_bridge/kinect2_bridge_nodelet to manager kinect2 with the following remappings:
process[kinect2_points_xyzrgb_hd-5]: started with pid [9008]
[ INFO] [1502219220.917350282]: waitForService: Service [/kinect2/load_nodelet] has not been advertised, waiting...
[ INFO] [1502219220.922039061]: Initializing nodelet with 4 worker threads.
[ INFO] [1502219220.938193022]: waitForService: Service [/kinect2/load_nodelet] is now available.
[ INFO] [1502219221.047534577]: [Kinect2Bridge::initialize] parameter:
base_name: kinect2
sensor: default
fps_limit: -1
calib_path: /home/administrator/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
use_png: false
jpeg_quality: 90
png_level: 1
depth_method: default
depth_device: -1
reg_method: default
reg_device: -1
max_depth: 12
min_depth: 0.1
queue_size: 5
bilateral_filter: true
edge_aware_filter: true
publish_tf: false
base_name_tf: kinect2
worker_threads: 4
X Error of failed request:  BadRequest (invalid request code or no such operation)
Major opcode of failed request:  153 (DRI2)
Minor opcode of failed request:  1 (DRI2Connect)
Serial number of failed request:  11
Current serial number in output stream:  11
[FATAL] [1502219222.756759642]: Failed to load nodelet '/kinect2_bridgeof typekinect2_bridge/kinect2_bridge_nodeletto managerkinect2'
[kinect2-1] process has died [pid 8977, exit code 1, cmd /opt/ros/indigo/lib/nodelet/nodelet manager __name:=kinect2 __log:=/home/administrator/.ros/log/3343e64e-7c67-11e7-ad07-bf271f6e5f05/kinect2-1.log].
log file: /home/administrator/.ros/log/3343e64e-7c67-11e7-ad07-bf271f6e5f05/kinect2-1*.log
[kinect2_bridge-2] process has died [pid 8978, exit code 255, cmd /opt/ros/indigo/lib/nodelet/nodelet load kinect2_bridge/kinect2_bridge_nodelet kinect2 __name:=kinect2_bridge __log:=/home/administrator/.ros/log/3343e64e-7c67-11e7-ad07-bf271f6e5f05/kinect2_bridge-2.log].
log file: /home/administrator/.ros/log/3343e64e-7c67-11e7-ad07-bf271f6e5f05/kinect2_bridge-2*.log
[kinect2_bridge-2] restarting process
process[kinect2_bridge-2]: started with pid [9180]
[ INFO] [1502219223.060315120]: Loading nodelet /kinect2_bridge of type kinect2_bridge/kinect2_bridge_nodelet to manager kinect2 with the following remappings:
[ INFO] [1502219223.062550693]: waitForService: Service [/kinect2/load_nodelet] has not been advertised, waiting...
^C[kinect2_points_xyzrgb_hd-5] killing on exit
[kinect2_points_xyzrgb_qhd-4] killing on exit
[kinect2_points_xyzrgb_sd-3] killing on exit
[kinect2_bridge-2] killing on exit
The result of rosrun kinect2_bridge kinect2_bridge:
administrator@jackal:~$ rosrun kinect2_bridge kinect2_bridge
[ INFO] [Kinect2Bridge::initialize] parameter:
base_name: kinect2
sensor: default
fps_limit: -1
calib_path: /home/administrator/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
use_png: false
jpeg_quality: 90
png_level: 1
depth_method: opencl
depth_device: -1
reg_method: opencl
reg_device: -1
max_depth: 12
min_depth: 0.1
queue_size: 2
bilateral_filter: true
edge_aware_filter: true
publish_tf: false
base_name_tf: kinect2
worker_threads: 4
X Error of failed request:  BadRequest (invalid request code or no such operation)
Major opcode of failed request:  153 (DRI2)
Minor opcode of failed request:  1 (DRI2Connect)
Serial number of failed request:  11
Current serial number in output stream:  11
I am using Intel graphics chipsets. Both PC and robot have Ubuntu 14.04, ROS indigo.
./bin/Protonect works but has error: [Debug] [DepthPacketStreamParser] skipping depth packet
I have been stuck here for weeks. Please help. Thank you!!!

Protonect does not run correctly (even with the viewer disabled) with X-forwarding enabled. I don't know why this is, but after disabling X-forwarding, I was able to solve a similar issue (possibly the exact same issue) on Ubuntu 16.04 and ROS Kinetic.


Regularization for Linear Regression.

ML - Algorithm

[x] I have not used any Machine Learning Libraries in my code
[x] This ML Algorithm is not available in the repository currently.
[x] If it is, My ML Algorithm is implemented in a different method.
[x] I have starred this repository

Information

ML Algorithm Name:
Linear Regression with Regularization
Added extra prompts too, for better outputs while program execution.

Language:
python.

Go ahead and create a PR @Varun221

Implemented in #68


fix(manifest): yaml unmarshalling

The manifest package operates using the assumption that only the JSON
types are present in the map[string]interface{}:

float64 for numbers
string for texts
bool for booleans
nil for null values

The gopkg.in/yaml package however also uses int, etc. This breaks
our package.
To work around this, the custom yaml.Marshaller has been modified to
not actually unmarshal, but call json.Unmarshal instead, making both
formats behave exactly the same.
Extensive tests have been added to ensure this.

The encoding/json package Number type has a hierarchy for number representations https://golang.org/pkg/encoding/json/#Number, is the premise in the assumption that all numbers are float64 valid?

The encoding/json package Number type has a hierarchy for number representations https://golang.org/pkg/encoding/json/#Number, is the premise in the assumption that all numbers are float64 valid?

Same document sais at https://golang.org/pkg/encoding/json/#Unmarshal:

To unmarshal JSON into an interface value, Unmarshal stores one of these in the interface value:
bool, for JSON booleans
float64, for JSON numbers
string, for JSON strings
[]interface{}, for JSON arrays
map[string]interface{}, for JSON objects
nil for JSON null

These are the assumptions the code in manifest uses. Does that answer your question?


Add more security to MXPR

it would be nice to add more security tomxpr, for example, ability to specify user name & password for remote connection.
It would be also useful to add SSL, but it could be too much work.

mxpr is reusing JMX dacility from JDK so add all the stuff vanila JMX have is not a big deal.
Though, my intent for mxpr was quick and dirty solution. If you want it right it would be probably better to setup JMX property on target JVM.
Am I missing some other useful cases for mxpr

I understand, I'm planning to use it as quick and dirty for solution when customer doesn't setup stuff in advance, and restarting/reconfiguring is not an option...
I'll look, if it's possible to do at all, and come with PR if I can :-)

Oh, one more think. You can do mxpr work  with jcmd for JDK (no shady jars from internet required).
Here is my cheetsheet for JDK tools it has an example https://training.ragozin.info/collateral/JDK_tools_cheatsheet.pdf
jcmd have all security options available.
The only issue with jcmd - in some rare situations JVM RMI port on some their adapter making forwaring of JMX through SSH impossible. So I keep mxpr for such cases.

Thank you very much for cheatsheet!

wont fix


Package export does not process book info

All current package TSV export functions in PackageController & WorkflowController do not output any Monograph fields.
This should probably be fixed in concordance with #8.

To have the reference, I shall reference here own test results:
Tested on dev & QA
With reference to LAS:eR-tickets ERMS-1203 and ERMS-1150
When exporting EBook-packages (KBart and GOKb), no identifiers are being exported.
Tested with:
dev: https://phaeton-dev.hbz-nrw.de/gokb/public/packageContent/org.gokb.cred.Package%3A812354
qa: https://phaeton-qa.hbz-nrw.de/gokb/public/packageContent/org.gokb.cred.Package%3A812408

Fixed in #49


Add integration test for APIRule reconciliation to Error status

Description
Changes proposed in this pull request:

Add test

Related issue(s)

eventually add when you delete VS that status is back to OK? or is it too much?

eventually add when you delete VS that status is back to OK? or is it too much?

@videlov
That is already verified in this test.


Youtube HTTP Error 404: Not Found

Seems there is a bug, can you please fix this issue?
$ youtube-dl.exe --verbose http://www.youtube.com/watch?v=Llu4kNm9-hA
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'http://www.youtube.com/watch?v=Llu4kNm9-hA']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2015.05.15
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] Llu4kNm9-hA: Downloading webpage
ERROR: Unable to download webpage: HTTP Error 404: Not Found (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
File "youtube_dl\extractor\common.pyo", line 312, in _request_webpage
File "youtube_dl\YoutubeDL.pyo", line 1710, in urlopen
File "urllib2.pyo", line 410, in open
File "urllib2.pyo", line 523, in http_response
File "urllib2.pyo", line 448, in error
File "urllib2.pyo", line 382, in _call_chain
File "urllib2.pyo", line 531, in http_error_default

It works for me, can you watch the video in a browser?

Yes, in all browsers it works.

Ok, I think it's network problem. This video was downloaded successfully on machine from another network.
But what may be a cause of the problem?

If on the same computer you can watch it on the browser but not download with youtube-dl, that's definetely strange (specially because it's not that the webpage is missing something, but that the webpage can't even be downloaded). If you are using some proxy/vpn in your broswer (and it's required for watching the video) you also need to configure youtube-dl to use it. Are you in your home network, or in some school/company network?

jaimeMF, thanks for responses very much.
Today it works fine, so I can't reproduce the error.


bugfix/correct-version-in-doc-builds

Link to Relevant Issue
This pull request resolves #58
Description of Changes

I think this is the fix for #58.
My thinking here is that, because we use setuptools-scm to allow the package to be versioned by git tags, we need to give the tag information to the doc CI build prior to package install as well.
The reason that the publish job doesn't need this info is that it only runs on tag releases already.
If my assumptions are correct, the doc versions should now follow the package version BUT will likely still have the dev* bit at the end. That is, when this is merged it should be something like 1.0.4.dev1+g06fa7f2 which is the version I get when I install locally on my #55 branch. If we don't want that dev bit, we will likely need to only run this doc build job on tag releases.

I think we should be clear as to what version of the docs we want published at bioio-devs.github.io.  IMO it should be the latest pypi released version and not anything newer, since the gh-pages site is the only live site we have for the docs, afaik.


feat: Option to create parser-specific Emoji object

Description
Adding option to create parser-specific Emoji object in each project
I've managed to test this with mavenLocal and it does work. If you have any question or see an improvement can be made do not hesitate :)
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Enhancement (Improves existing functionality)

IMPORTANT: By submitting a patch, you agree to allow the project
owners to license your work under the terms of
the Apache License 2.0.

Hello, so here is how I suggest we create EmojiManager now using EmojiManager.create(context: Context, serializer: IDeserializer). AbstractEmojiInitializer still needs to be put out of the main module but this is how it would work.
Tell me what you think about it.

I'll have a look at this PR hopefully in the morning with a fresh mind, I do like the idea of having an extension function for creation

Could you kindly rename the main source set to src/main/kotlin/* for all the new library modules you've created 😃

In my last commit I've added gson and moshi serializer, but I'm a bit clueless on how to make them artifact.
If you need me to modify the PR do tell me :)

Do you need me to do something else in order to merge this ?

Do you need me to do something else in order to merge this ?

Thought you were still busy with it 😄 You have a few failing steps, spotless and unit tests

Oh! haha I was waiting for you as well. Anyway I've updated, I hope the tests passes

For some reason EmojiLoader::class.java.getResourceAsStream("emoji.json") returns null

I'm not sure what's missing for the last checks

I'm not sure what's missing for the last checks

Looks like you're missing AndroidManifest.xml from the contract module

There is an issue with the release-drafter

fatal: unable to access 'https://github.com/AniTrend/android-emojify/': The requested URL returned error: 403

There is an issue with the release-drafter

fatal: unable to access 'https://github.com/AniTrend/android-emojify/': The requested URL returned error: 403

It's alright it's not required, it's probably because your PR is running off your fork so any actions that may change files aren't required. It'll run when we merge ✨

Ok cool, same for instrumented test ?

Ok cool, same for instrumented test ?

I'll look into the instrumented test and resolve it on develop should be something small. This was a lengthy PR process, sorry for taking a while to replying. It's been hectic lately

No worries! There was a lot of changes and it's better to take our time in order to get something clear

True, regardless. Thank you for being patient and taking the time to consider all the options and resolving all the comments, massively appreciated 😃


Apple TV disconnects from Home Assistant when watching media

Home Assistant release (hass --version): 0.51.2
Python release (python3 --version): 3.4.2
Component/platform: apple_tv
Description of problem: Apple TV is connected to Home Assistant. Apple TV disconnects from Home Assistant, producing the error message “Establishing a connection to the AppleTV” when watching media on the Apple TV, namely Netflix. Rebooting Home Assistant fixes the problem temporarily. Problem began appearing around the 0.48 release.
Expected: Playing media/Netflix on the Apple TV should not cause the Apple TV to disconnect from Home Assistant.
Problem-relevant configuration.yaml entries and steps to reproduce:
apple_tv:
  - host: <IP_ADDRESS>
    login_id: 00000001-f255-xxxx-xxxx-d57da19c43b9

Confirm Apple TV is connected to HA.
Play multiple shows on Apple TV using Netflix App
Notice that the Apple TV eventually disconnects from Home Assistant while watching.

Traceback (if applicable):

Additional info:
See this thread for more information: https://community.home-assistant.io/t/appletv-no-longer-reachable-after-watching/25383/2

I get this as well.  I have two Apple TV 4's.  Lots of this in my error log:
WARNING (MainThread) [homeassistant.components.media_player.apple_tv] A <class 'pyatv.exceptions.AuthenticationError'> error occurred: failed to login: 400

I've been thinking a bit about this and don't understand why it would happen. Especially since I haven't made any changes to the protocol logic lately. But it would be super helpful if someone with these issues could enable additional log points in pyatv, preferrably the ones below, and run again.
logger:
  default: warning
  logs:
    homeassistant.components.media_player.apple_tv.pyatv.net: debug
    homeassistant.components.media_player.apple_tv.pyatv.daap: debug
    homeassistant.components.media_player.apple_tv.pyatv.internal.apple_tv: debug

I hope that should work. It's not strictly necessary to change default, but it easier to single out the specified log points.
Follow up questions:

Anyone have problems with ATV3? I don't see this on my device.
Has there been any software updates to tvOS lately that might have changed something? Quite unlikely though.

@postlund I have enabled the verbose logging for apple_tv as requested.
To answer your questions:

I only have a single ATV4 that is experiencing the problem.
No software updates to my knowledge

Added your logging but only get this every 10 seconds in the log file:
WARNING (MainThread) [homeassistant.components.media_player.apple_tv] A <class 'pyatv.exceptions.AuthenticationError'> error occurred: failed to login: 400

I've got some more info on this bug. For me, it ONLY happens when watching the Netflix app. I can watch as much media through the Plex app and the connection never drops. Watch a show or two on Netflix, though, and it drops every time.

Sorry for not replying earlier, been busy with stuff. I know that a lot of people are having various issues with the current implementation. This seems however to only be the case with ATV4 (please correct me if I'm wrong). My current take is to not spend any additional time than necessary on this as I want to put my focus on the MediaRemoteTV-protocol instead. The protocol that is actually made for and used by the ATV4.
I've put a tremendous amount of work into this the last week or so, but I'm no where near a complete implementation. The hard parts, pairing and encryption, is however implemented and I've made the necessary changes to incorporate support for an additional protocol into pyatv (it was harder than one might think). So I hope that we sometime soon can move over to the new protocol (maybe with some additional features, who knows) and solve all of these issues. I also hope that you can understand my decision on this and bear with me until it's done. No one has implemented this before (openly at least), so it will take the time it takes.

Since the last tvOS update (tvOS 11, I think?), I'm no longer experiencing this issue. Will do some more testing tomorrow.


How do you check if an item is currently being dragged?

I want to like show some sort of view in the item itself while it is being dragged. Is there any way to know it is currently being dragged inside the model?

@dev-JE02, you have OnItemMoveListener extends OnActionStateListener with the method onActionStateChanged() that you can implement in your activity OR if you don't need to implement move listener you can override onActionStateChanged() in FlexibleViewHolder, just call super() at first line.


Margin Mode always in isolated mode in gateio

Describe the bug
Operating system: Ubuntu
Environment: Docker
HB Release: v1.26.0
When I'm placing DCA trade using hummingbot it always places trades in isolated mode instead of cross mode, I tried to modify margin from isolated to cross manually, but when I run the hummingbot it returns back to isolated again.
anyone has any idea what's going on?
Steps to reproduce

Trades false back to isolated mode
tried to switch between different operating systems and installed previous versions but the issue still the same

Release version
v1.26.0
Type of installation
Source
Attach required files
No response

Any Help??????????


Utils events tests

Is your feature request related to a problem? Please describe.
Write tests for ./src/utils/events/index.ts
Describe the solution you'd like
n.a
Describe alternatives you've considered
n.a
Additional context
n.a

@lukasaric can I work on this issue


Switch readthedocs to main

Currently the readthedocs set up by @noahsprent in #78 is pointing to the develop branch. I believe that we actually want it pointing to the releases on the main branch instead. We can't do that effectively, however, until we make a release that includes the readthedocs materials added in #78, howver.

I think actually a good solution to this might be to have the 'latest' docs on rtd point to the develop branch, but have a 'stable' docs that points to the most recent release on main. We can set the default docs to be the 'stable' docs, but if people want to see the latest docs for the develop branch (and if we want to see them before we merge to main) then they can select that in the bottom left.
See instructions for doing that here
For now, we can point the default to 'latest/develop' until we have a release that incorporates the docs into main (which I suggest we don't do until we have some docs for people to see!).

I agree with Noah. The best is to have a stable version pointing to master and a latest version pointing to dev. I can set up that but first I would like to have some help writing the docs, any volunteer?

Thanks @Gonza10V! If you know about read-the-docs could I add you as a reviewer of #84? I've just added the bits to create the autoapi reference but there are a few warnings. I think we probably only need that, along with some text on the homepage and a link to the docs in the repository here before that makes it enough to push to main for the time being?  @jakebeal Do they sound like acceptable criteria for a main release?
I think it's probably the priority to focus on the docs for the distro repository right now, which I'm going to work on, but it would be a useful exercise for me to help write more pages for these docs after we've got that going so happy to help then.

@noahsprent We can make a release whenever we have a significant improvement ready to push out --- I'm fine with minor versions and patch versions. If you're OK with #84, go ahead and merge it.

@noahsprent Looks like you managed this to complete the merge of #84. Count with me to help you setting rtd documentations, it is a skill that I also need to improve.

Looks like sticking with "latest" is fine for now, and once it's declared out of pre-release, we'll get a "stable" branch and tagged branches automatically.


config: update operator payloads for v0.8.0 release

This covers step 13 from the release checklist here

@portersrc, please, just take a look at the comments related to enclave-cc.

I bumped it; it looks like that enclave-cc bundle is still building as of this writing though

/test

/test-s390x

/test

/test

/test

@portersrc any special reason to not pin the s390x payload? -> https://github.com/confidential-containers/operator/blob/main/config/samples/ccruntime/s390x/kustomization.yaml

Oh, that's something that was overlook, I guess, nice catch!

@portersrc any special reason to not pin the s390x payload? -> https://github.com/confidential-containers/operator/blob/main/config/samples/ccruntime/s390x/kustomization.yaml

Oh, that's something that was overlook, I guess, nice catch!

Also we are using the CI payload instead of quay.io/confidential-containers/runtime-payload

@portersrc you also need to bump the hash of pre-reqs for peer-pods too
Good opportunity to advertise this: https://github.com/confidential-containers/operator/pull/205 (perhaps someone "cof-cof Chris cof-cof" would like to take over that script).

Also we are using the CI payload instead of quay.io/confidential-containers/runtime-payload

I think the idea is to switch to the runtime-payload bundle in step 19 after we add the release tags for kata and enclave-cc. In the past we have sometimes combined this step with that one, though, and we could do that again if people think it would save time.

Also we are using the CI payload instead of quay.io/confidential-containers/runtime-payload

I think the idea is to switch to the runtime-payload bundle in step 19 after we add the release tags for kata and enclave-cc. In the past we have sometimes combined this step with that one, though, and we could do that again if people think it would save time.

Good point @fitzthum , I forgot kata isn't tagged yet. IMHO, anything to speed up this release is welcomed, so I am okay to switch the payload on this PR to run the CI jobs one more time

Thanks to @wainersm for catching these two things: pinning s390x runtime-payload-ci and also pinning peer-pods' reqs-payload.
These are incorporated into the latest push.

/test

@wainersm my hunch for now is to do another PR for step 19 just to make sure we don't confuse ourselves. I don't think that will take too much longer, but I leave it to @portersrc to decide.

/test

Let's give these test one more chance to behave


Add missing if_data and if_msghdr structs

Looks like ci is failing?

Don't worry, I'm updating the failing platforms. I'll ping you once working. :)

@alexcrichton: Fixed.

@bors: r+

:pushpin: Commit 396bad8 has been approved by alexcrichton

:hourglass: Testing commit 396bad810bf50a3f914f5977e2be105367bbc590 with merge 8e5152a4eee7acebd2317171998357b76aa1a509...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing 8e5152a4eee7acebd2317171998357b76aa1a509 to master...


don't use trim on progress streams in AnsiProgressHandler

Trim deletes ANSI escape codes in the stream messages, so you end up
with everything after the escape code and then the message (sans color
or whatever else the escape code was trying to do).
Now the code explicitly just deletes any '\n' chars at the end of the
string as the comment claims it should be doing.

Those build failures seem transient (too many requests), but it wasn't clear how to re-run it.

Are you able to login to CircleCI to rebuild it?
On Fri, Nov 6, 2015 at 3:36 PM, Wes Morgan<EMAIL_ADDRESS>wrote:

Those build failures seem transient (too many requests), but it wasn't
clear how to re-run it.
—
Reply to this email directly or view it on GitHub
https://github.com/spotify/docker-client/pull/267#issuecomment-154525174
.

--
David Xia
Software Engineer
<EMAIL_ADDRESS>

@cap10morgan agreed those failures are transient, and this change wouldn't cause any problems with that functionality anyway.
Thanks for the contribution!


cannot find name space webdriver when installing with yarn

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see use [gitter](https://gitter.im/mgechev/angular2-seed) or [stackoverflow](https://stackoverflow.com/questions/tagged/angular2)

Current behavior
If you install the dependencies with the yarn command, it picks up the lockfile and install the correct dependencies. However, when running the project, you will receive the cannot find name space webdriver error. If you install the dependencies with npm install, this error does not occur.
Expected behavior
To work out of the box
Minimal reproduction of the problem with instructions

git clone https://github.com/mgechev/angular-seed
cd angular-seed
yarn
npm run build.prod.exp

Please tell us about your environment:
macOS Sierra, VS Code, yarn

Angular Seed Version: https://github.com/mgechev/angular-seed/commit/ff742ea55141cdd4dad4f0dbc03866799fd3e7f6

Node: node --version = v6.9.1

After regenerating the yarn.lock file, everything works as expected. I will file a PR for that.

It seems that "webdriver" makes troubles  lol


custom interpolators

Hi,
It is possible to use custom interpolators like in angular translate?
https://angular-translate.github.io/docs/#/guide/15_custom-interpolators
Thanks,
David

Hello, it's not possible yet, but it shouldn't be too difficult, we just need to add a config param to the parser.

what about making parser injectable?

Hello, the parser is now injectable and you can create your own custom parser if you need, closing this :)


Detect Mouse Buttons?

My mouse has 2 buttons on the side. Would love to be able to rebind the hotkeys to one of them.

If you have a Logitech mouse you can use the remap programm. (Other mouses might have this software as well)


failed to replicate the result of top1, top5 acc for CVAE model

Hi,
I am trying to replicate your chatbot model. I did exactly the same thing as you mentioned in the README. I got more or less similar kl/reconstruction loss for training CVAE. But for emoji classification, top1 and top5 acc for CVAE model is only 30.4% and 54.3%, which is much worse than the results you reported in the paper. Can you give some suggestions about this?
Thanks
Peng

what is the performance of your Base model?
I suggest choosing the breakpoint at 75% of the converged Base model. And train your CVAE model starting from there.

what is the performance of your Base model?
I suggest choosing the breakpoint of a not yet converged Base model as the pretrained model. And train your CVAE model starting from there.
Also, you may need to stop the training of the emoji classifier before it overfits.

Thanks for your reply.
The base model achieves perplexity of 134.244/132.922 at step 18000/27500 on test set. And I choose the breakpoint at step 18000 for the starting point for CVAE training. After CVAE model training, I got recon/kl loss as 42.426/26.412. For the emoji classifier, the best model is 'step': 9000, 'epoch': 2, 'accuracy': 0.3211703300476074, 'loss': 2.8405072689056396, 'top_5_accuracy': 0.5782781839370728 on the test set, which is same as what you reported.
I got the acc result on top1/top5 as 0.304/0.543 for CVAE model from the rl_run.py as you will get the test set performance before training. Any ideas about the problem?
Best
Peng

This looks like the result of the Base model. Have you tried printing the acc results of your Base model?

Thanks for your comment. I will try the base model and let you know the result.

Sorry for the late reply.
I ran the base model and the top1/top5 accuracy is 0.349/0.575, which is similar to what you reported in the paper. However, the CVAE model is even worse than the baseline seq2seq.
Please let me know your opinion.
Thanks!
Peng

Hi! I got the same accuracy problem as you discussed here when replicating the model and i am still confused why.If you guys could give me some advice i will be very appreciated.

@claude-zhou
Dear zhou
I attempt to implements your paper with pytorch
when I read your code,I found that the output you used in calculating the test loss(perplexity) was the output of training decoder but not inference decoder.
As you mentioned in the code "use inference decoder's logits to compute recon_loss"
but the logits was the output of training decoder not inference decoders'
shouldn't we use the output of inference decoder to calculate the test loss(perplexity)?
        with tf.variable_scope("loss"):             max_time = tf.shape(self.rep_output)[0]             with tf.variable_scope("reconstruction"):                 # TODO: use inference decoder's logits to compute recon_loss                 cross_entropy = tf.nn.sparse_softmax_cross_entropy_with_logits(  # ce = [len, batch_size]                     labels=self.rep_output, logits=self.logits)                 # rep: [len, batch_size]; logits: [len, batch_size, vocab_size]                 target_mask = tf.sequence_mask(                     self.rep_len + 1, max_time, dtype=self.logits.dtype)                 # time_major                 target_mask_t = tf.transpose(target_mask)  # max_len batch_size                 self.recon_losses = tf.reduce_sum(cross_entropy * target_mask_t, axis=0)                 self.recon_loss = tf.reduce_sum(cross_entropy * target_mask_t) / batch_size
            # Dynamic decoding             infer_outputs, _, infer_lengths = seq2seq.dynamic_decode(                 decoder,                 maximum_iterations=maximum_iterations,                 output_time_major=True,                 swap_memory=True,                 scope=decoder_scope             )             if beam_width > 0:                 self.result = infer_outputs.predicted_ids             else:                 self.result = infer_outputs.sample_id                 self.result_lengths = infer_lengths

            # Dynamic decoding             infer_outputs, _, infer_lengths = seq2seq.dynamic_decode(                 decoder,                 maximum_iterations=maximum_iterations,                 output_time_major=True,                 swap_memory=True,                 scope=decoder_scope             )             if beam_width > 0:                 self.result = infer_outputs.predicted_ids             else:                 self.result = infer_outputs.sample_id                 self.result_lengths = infer_lengths

I am thinking that we should use "infer_outputs" to  calculate the CrossEntropy and calculate the perplexity  further

I will be appreciate it very much if you could give some advice @claude-zhou


Front end profiles

[x] Create public/private profile view on FrontEnd

[x] Update Ember user model for profile fields

[x] Update Server user model

[x] Allow profile fields to be updated

[ ] Server authorization for updating user model

[ ] Write tests

Sweet, need a link to the profile somewhere though?

Theres one included in Kiki's navigation bar, should there be another one
somewhere?
On Tue, Nov 26, 2019, 13:42 Jake Lee Kennedy<EMAIL_ADDRESS>wrote:

Sweet, need a link to the profile somewhere though?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tunapanda/swag/pull/53?email_source=notifications&email_token=ABJEA4FF5SAMXR3DPL3KTJTQVT4Q5A5CNFSM4JRON7X2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFFR4RI#issuecomment-558571077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJEA4FXFSAQYNOC5XRRZ4DQVT4Q5ANCNFSM4JRON7XQ
.

Is that in a different branch?

Yeah, in master branch now after the Frontend-MVP pull request


Debundle pycodestyle

We have pep8 (renamed pycodestyle upstream) bundled and out code interwoven. This makes it hard to update pycodestyle. This PR removes bundled pep8 (2.2.1a0) and instead install upstream pycodestyle (2.3.1) as a dependency.
Additionally a bit of clean up was done that brings image size down to 59.2MB from the latest being 681MB (11.5x smaller).

@pointlessone nice work here! are there any breaking changes between the version of pep8 we were using and latest pycodestyle that would impact end users (e.g. with respect to configuration options)?
If so, maybe we will want to consider that in how we roll out update

@ABaldwinHunter I don't think so but let me check and get back to you (tomorrow).

@ABaldwinHunter There seem to be no breaking changes between these releases in the changelog.


Setup not working on EKS due to Security Groups issues

What's the issue?
When running the default Load Test scenario provided (i.e. my-scenario.jmx), there is some missing data in the Apache JMeter Dashboard (see screenshot next).

This does not happen when allowing All Traffic in the Security Group(s) associated with my EKS nodes. If, however, I only allow traffic for the components included in the setup (JMeter, Jolokia, InfluxDB, Grafana), I get the issue shown in the picture (see above). The ports to which I've allowed node-to-node Inbound Traffic are:

8086: for InfluxDB
8778: for Jolokia
50000: for JMeter
3000: for Grafana

Is there anything else I'm missing here?  Another port being used that I'm completely ignoring?
How to reproduce the issue?
Follow the steps provided in the repository in a EKS cluster with strict Security Groups rules for the cluster nodes.
Any additional details?

I've also tried to update the grafana version used from grafana/grafana:7.3.3 to grafana/grafana:8.2.6 and started having connection issues with the following browser console error: can’t establish a connection to the server at ws://localhost:3000/api/live/ws. Perhaps something to do with Grafana Live. This was a little weird, since I have other Grafana setup in the same EKS cluster (traffic allowed on port 3000) and hadn't experienced that issue before.

PS: Thanks for this setup. It's really cool.

Hey,
Sorry for the delay,
The port 1099 must be allowed as well.
JMeter master and slaves are communicating through RMI and samples results are send through this protocol.
The communication is bi directional (master to slave, slave to master)
Glad this tool is useful!

Hello @vmendiolalau is your issue fixed ?


[Feature Request] Option to launch PS1 games with retroarch.

Closing issue prior to archiving

Complete, added to v4.02


propose to PHP.MigrationSet to update array/list syntax to short one

I understand "Migration" as a replacement of a deprecated concept by a new one.
Short array syntax is just an alternative way to express your arrays. array() is not deprecated and short array syntax is not meant to replace it.
Just because someone prefers array() syntax doesn't mean they haven't migrated to 5.4
I feel it the rule should stay out of a migration set

I understand "Migration" as a replacement of a deprecated concept by a new one.
Short array syntax is just an alternative way to express your arrays. array() is not deprecated and short array syntax is not meant to replace it.
Just because someone prefers array() syntax doesn't mean they haven't migrated to 5.4
I feel it the rule should stay out of a migration set

When I created Migration rulesets, they were for purpose of "let use the new language/syntax/feature of higher version of PHP/PHPUnit". I agree one can say it's a matter of style. But one can also say that not using ** operator is matter of style, and we aim to use it, if possible. But no worry, one can always disable the rules s/he would not like to follow

When I created Migration rulesets, they were for purpose of "let use the new language/syntax/feature of higher version of PHP/PHPUnit". I agree one can say it's a matter of style. But one can also say that not using ** operator is matter of style, and we aim to use it, if possible. But no worry, one can always disable the rules s/he would not like to follow

Closes #5389

Closes #5389


prometheus-tor-exporter: init at 0.3

Motivation for this change
I want some monitoring for my Tor relay :) This is not a particularly widely used piece of software, but it seems to be still maintained upstream.
First time contributing and packaging things for Nix, please let me know if anything can be improved!
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
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)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-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)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build prometheus-tor-exporter

Success on aarch64-linux (full log)
Attempted: prometheus-tor-exporter
Partial log (click to expand)
setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/prometheus-tor-exporter.service
patching sources
updateAutotoolsGnuConfigScriptsPhase
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
/nix/store/rzk0cmak8fvfrn6g9yc4mfizn5b76l9x-tor-exporter-0.3

Success on x86_64-darwin (full log)
Attempted: prometheus-tor-exporter
Partial log (click to expand)
source root is source
setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/prometheus-tor-exporter.service
patching sources
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
/nix/store/igfgjnmki84dpvq0vnl7k8p04gcmlgm5-tor-exporter-0.3

Success on x86_64-linux (full log)
Attempted: prometheus-tor-exporter
Partial log (click to expand)
source root is source
setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/prometheus-tor-exporter.service
patching sources
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
/nix/store/bpaywx2dp6zxawcw2k5ng7pv8xwl519r-tor-exporter-0.3

Success on x86_64-darwin (full log)
Attempted: prometheus-tor-exporter
Partial log (click to expand)
source root is source
setting SOURCE_DATE_EPOCH to timestamp 315619200 of file source/prometheus-tor-exporter.service
patching sources
configuring
no configure script, doing nothing
building
no Makefile, doing nothing
installing
post-installation fixup
/nix/store/igfgjnmki84dpvq0vnl7k8p04gcmlgm5-tor-exporter-0.3

(Misclick, sorry.)

Thanks and welcome! ;)


[iOS] Notification dots inconsistently used

Summary
The Android and iOS app notification dots behave differently on general notifications (not mentions).
Environment Information

Device Name:
OS Version: iOS 13.3
Mattermost App Version: 1.32.0
Mattermost Server Version: 5.19.0

Steps to reproduce
Setup:

User A: Browser
User B: Android app
User C: iOS app
All users in same private channel

App config for both users B and C:

Mobile push
** Notifications for all activities
** Notifications on all states (online, away, offline)
Mentions
** Username enabled
** Channel wide notifications enabled
Private channel not muted

A writes to channel "Hey all"

B gets a push notification and notification dot
C gets a push notification but no notification dot

A writes to channel "Hey @here"

B gets a push notification and notification dot
C gets a push notification but no notification dot

A writes to channel "Hey @B"

B gets a push notification and notification dot
C gets a push notification but no notification dot

A writes to channel "Hey @C"

B gets a push notification and notification dot
C gets a push notification and notification dot

Expected behavior
With all notifications enabled I would expect both the Android and iOS app to show the notification bubble with every triggered notification, as long as the bubble is generally enabled.
Observed behavior
Apparently the iOS only shows the bubbles on direct mentions. Neither @here nor regular messages are trigger a notification bubble. On Android every push notification also triggers to notification bubble.
Possible fixes
Don't know how to fix this.

Manually created mentions seem to work as expected.

@olivier-magloire @amyblais I can confirm the issue on iOS. On Android, the dot badge always appear. However for iOS, the badge appears intermittently for @all / @here / @channel
Single @here does not trigger badge. @here --> @here also does not trigger badge.
However, when user receives @here --> @all --> @here, then the badge appears for @here. The same goes for @all --> @channel --> @all, only then the badge for @all appears.
DMs and at-mentions seem to trigger the badge properly.
This issue is not a recent regression. Issue only occurs on iOS

Thanks, opened a ticket here: https://mattermost.atlassian.net/browse/MM-27141.

Hi @amyblais, are there plans to implement this anytime soon? We have people waiting to get this resolved. This is quite crucial for working together.

I don't have an ETA on this, the latest comment on the ticket is that one of the dev teams will address the ticket alongside work related to overhauling how unread counts work.

Hey all
i just wanted to ask if this issue is still on the board to fix or if this is closed.
We still have the same issue, that the badges on IOS don't show up if a user receives an message which isn't directly addressed to him.
IOS: 14.4
MM: 1.40.0

We are closing some older tickets that have been inactive for a while. If the issue persists on the latest mobile app version, feel free to re-open this issue.


change all texts to English

change all texts to English

@tangyang9464 @imp2002 please review

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


CLTextTool landscape keyboard

Using the 2017-10-18 build, the CLTextTool does not properly handle the keyboard in landscape orientations (the portrait keyboard always appears instead of the landscape keyboard).
To reproduce the issue:  build the CLImageEditorDemo with Xcode 9 and launch the iPad Simulator in landscape orientation;  tap the Edit button;  tap the Text tool;  tap the New button.
I think the issue is line 618 of _CLImageEditorViewController, because if I change line 618 to unconditionally return UIInterfaceOrientationMaskAll, then the landscape keyboard appears as expected.
A related issue: there's a UITextField that appears on screen while text is being typed.  In the attached screenshot, you can see its blue insertion point just above the New button.

Thanks for reporting!


Implement host-function boilerplate through x-macros.

This moves the declaration of host function information (names, type signatures, mutability-of-self, enclosing module, wasm import names, etc.) into a single x-macro (higher-order macro) that then calls back several different site-specific macros where the information is used:

The site of trait Env being declared
The site of all guest extern "C" native module decls.
The site of the impl Env for Guest that forwards all methods through to the extern "C" modules.
The site of impl Env for WeakHost that forwards all methods through to Host
A new site of resolving imports during WASM module instantiation in the VM.
A new site of dispatch functions that unpack args and forward the call to Host when the VM calls a resolved import.

It does not change implementations of host functions. They are still to be written manually, though the Env trait that declares them all (which Host has to implement) is now generated from the macro, so you are always required to implement all the functions everything else in the system knows about. This accounts for a lot of the "new code" (todo!()s) in the PR.
I know this is a macro-based solution. I know macros detract from comprehensibility, and x-macros doubly so. I tried to make it a fairly clear set implementation, that does not lose or obscure important surface information, but there's no escaping the reality of what's being done. I ask that those negatives be balanced against the following negatives if we follow the non-macro status quo:

The labor of adding, removing and keeping-in-sync edits to 6 different sites as we add/remove/adjust host functions.
The odds of introducing subtle errors due to mismatches in them.
The fact that we are identifying imports by otherwise-unchecked string names in 2 places that must be kept in sync.
The fact that we are identifying imports by otherwise-unchecked integer indexes in 2 places that must be kept in sync.
The possibility there will be more such sites and must-be-kept-in-sync places in the future.

Given these concerns I am advocating this solution. This is exactly the sort of thing macros are in Rust to support.

The one benefit this gives us that I think is meaningful is reduced risk of using the wrong symbol on the guest and host sides. Is there some other way we could specify that, such as having them all defined as constants?

In addition to constants we could probably write a test to check that each constant is associated with functions of the same name, arity, and types. This should address the risk you mentioned without the library code needing to be macro generated.

Thanks for adding the detailed comments for the macro, they help significantly with understanding what's going on. I was able to use rust-analyzer to expand the Env trait which helps too. (Unfortunately the other macro use sites don't expand.)
I still think we're optimizing heavily for a short term initial gain at a long term cost to every future reader, but I guess we'll be iterating on the host fns a lot and this is will speed up (maybe?) that process. I defer to you if this tradeoff is the most appropriate tradeoff to make. I defer to you if you think this is the best tradeoff to make.
I don't love that Rust is forcing us to put vm logic in the host crate but I understand why now 👍.

There's one build failing, the one that has the vm feature turned on for the host crate.
Could you also add an entry to build-common and test-common for the vm feature?


Add project to Heroku

As we now have a backend we need to host this project on Heroku. This is important for user testing at the end of sprint one.

Having difficulties getting Heroku to compile the sass. I tried this approach:

This is depreciated. https://github.com/ddollar/heroku-buildpack-multi


Plan 2021 Roadmap

Let's pick the top priorities for new feature work on async/await this year.
Use this issue for submitting ideas that have their own tracking issues/RFCs. (If one doesn't exist, create one!)
However, I'd like to avoid using this issue for discussion of individual items. Let's use the Zulip topic for that instead:
#wg-async-foundations > 2021 roadmap

I'd like us to consider #23: AsyncRead, AsyncWrite traits

I'd like us to consider #23: AsyncRead, AsyncWrite traits

I'd like us to consider #11: thread::block_on. And see  #12 Stream through to stabilization.

I'd like us to consider #11: thread::block_on. And see  #12 Stream through to stabilization.


[Bugfix] Fix marlin kernel crash on H100

This PR addresses the Marlin kernel H100 crash that was reported here: https://github.com/neuralmagic/nm-vllm/issues/187.
The reason for the crash was the inline PTX assembly that introduced the async_copy with streaming behavior. The solution is to use the more standard PTX for async_copy (without the fractional L2 policy for "evict_first"). There is no performance difference between standard async_copy PTX and the previous one.

I tried various modifications to the PTX to keep the cache-hint, but it did not work.

Is there any data to prove that there is no difference for l2 cache hint method in performance? But why is this method introduced in the marlin original paper?

Is there any way to keep the cache hint? Seems pretty useful but if you measured no difference then it might be alright

Hello, Is there any data to prove that there is no difference for l2 cache hint method in performance? I use a100 to run bench.py on marlin repo and it showes that on models other than llama 7B, l2 cache hint has performance improvement. Although bench.py actually only does matmul operation for simulation but does not have a full inference procedure.


Correct Shutdown Procedure

I'm shutting down individual voters like this:

Voter sends an app-layer Logout request to the leader
Leader removes the voter id: raft.change_membership(new_voter_ids, true, true) by demoting it to learner
Voter waits for confirmation: raft.wait(None).metrics(|x| !x.membership_config.voter_ids().any(|id| id == node_id))
Newly demoted learner shuts down its raft: raft.shutdown().await

If the voter is the leader of a single-node cluster, skip the change_membership entirely.
The only problem with this is I get error messages in the leader's log when shutting down the newly demoted learners, which tells me something is wrong.
{"timestamp":"<PHONE_NUMBER>47s","level":"WARN","message":"error sending AppendEntries RPC to target","error":"NetworkError: HTTP status client error (412 Precondition Failed) for url (http://localhost:11002/v2/append)","target":"openraft::replication","threadId":"ThreadId(3)"}
{"timestamp":"<PHONE_NUMBER>97s","level":"ERROR","message":"error replication to target=2","error":"NetworkError: HTTP status client error (412 Precondition Failed) for url (http://localhost:11002/v2/append)","target":"openraft::replication","threadId":"ThreadId(3)"}

I could fix this by changing to raft.change_membership(new_voter_ids, true, false), but the voter wouldn't get the acknowledgement at step 3, except by way of the leader, which would work but take away from the controlled procedure a bit. Adding a follow-up request would introduce lines of code, and the same acknowledgement dilemma.
This feels like the right procedure except for toning down the error logging when learners don't respond. Any recommendations?

Hmm... it's a typical dilemma with distributed consensus protocol: either to tolerate that a receiver may not receive a message or to tolerate that a sender may block forever.
Thus the leader may give up replicating the last log to a removed voter, and an app should not rely on the removed voter receiving the last log.
To complete the first policy:

Convert it to a learner, then expect the last log on the learner. Another step that makes this policy complete is to send another change-membership request to remove this learner(to get rid of the error in the log). Then shut down the learner.

Or use the second policy:

Or remove the voter at the first step with change_membership(new_voters, true, false). And wait for the second membership log to commit by watching the leader's metrics.

BTW, why is it necessary to wait for the replication of the second membership log to the node being removed?
As soon as the leader commits this log, that voter node can be shut down without trouble.

BTW, why is it necessary to wait for the replication of the second membership log to the node being removed?
As soon as the leader commits this log, that voter node can be shut down without trouble.

Thanks! That's where I ended up, but it's good to have your input. The leader doesn't respond until change_membership returns (which I believe means the Membership entry was committed to the raft), so it's good enough.


[ASV] performance regression in append operation

Pandas_on_ray engine - 44 cpus

<0.8.0>
<0.9.1>
ratio
benchmark

265±8ms
338±9ms
1.28
benchmarks.TimeAppend.time_append(((500000, 20), (1000000, 10)), True)

899±20ms
1.01±0.02s
1.12
benchmarks.TimeAppend.time_append(((5000, 5000), (5000, 5000)), True)

Found regression commits:

7cfc85c311fbe91c653285c46639352dd9ae4321
6f1e0c04b22a82f7a24271a5e9d4139cc5ee5537

Used command:
asv continuous -f 1.05 5b6f73a98cc0ee5a78503704b98b7268709dc797 21707d2dd8c51404db692e27fb1a69203440d2eb  --launch-method=spawn -b time_append  -a repeat=8 -a number=2

Feel free to reassign.

@anmyachev is this just flaky/sporadic or is this consistently slower?

@anmyachev is this just flaky/sporadic or is this consistently slower?

@pyrito consistently slower (but the reason for the degradation of performance may already be different)

Not relevant since append has been removed in pandas 2.X and modin 0.23.0.


Temporary file cleanup can remove unintended files

It looks like in version 0.4.0, the behavior of NativeLoader.cleanup() method changed considerably: it used to delete files matching "webp-imageio-0.5.0*" in java.io.tmpdir, but now it recursively deletes everything matching that pattern.
This has unintended consequences... If you happen to set java.io.tmpdir to $HOME, all copies of webp-imageio-0.5.0.jar and webp-imageio-0.5.0.tar.gz also disappear.  Or if you had set java.io.tmpdir to $XDG_RUNTIME_DIR (/run/user/:uid/), the cleanup might take forever, because slow/huge network file systems might be mounted under /run/user/:uid/gvfs/...
Maybe just restore the previous, non-recursive behavior?

Thanks for the report and sharing context 👍 Looking at the code I think I misunderstood where the tmp files are extracted to. Previous behavior should be now restored in 0.5.1 ✅

Thanks for a super-quick fix!